Update pillar dependency in edgeview and newlog

The main purpose of this PR is to update the indirect dependency on Focinfi/go-dns-resolver package. The previous version of this package was missing license. Also, github.com/rackn/gohai disappeared from go.sum as expected.
Also run go mod tidy for both packages and go mod vendor additionally for newlog
edgeview does not use vendoring - @naiming-zededa do we want to keep it like that or unify it with pillar and newlog? I believe vendoring is used only to shorten the container build time.
Signed-off-by: Milan Lenco<EMAIL_ADDRESS>

@milan-zededa I think the current way is fine. when update for edgeview is needed from pillar, it can update the go.mod instead of copying all the vendor files. thanks.

@milan-zededa I think the current way is fine. when update for edgeview is needed from pillar, it can update the go.mod instead of copying all the vendor files. thanks.
@naiming-zededa @milan-zededa
FYI in #2861 we are trying to run the build without network access as a way of being able to better track which packages (golang, alpine, etc) so that it is easier to track the Software Bill of Materials for a build/release.
That implies that each pkg/foo needs to be more self contained. Hence having a vendor directory with the needed source (which go mod vendor will create). So I think we need such a PR for edgeview.

ok.


build_request: Set 'release' label when provided

[Looking for early feedback]
Currently if the spec provides a 'release' param, it is not passed along
to the orchestrator as a label.  Detect that and provide it.  Also remove
the bump_release plugin if the 'release' param is passed in.
Updated tests to verify change.
Signed-off-by: Don Zickus<EMAIL_ADDRESS>

@lcarva
"I think we want to only set this key if self.spec.release is truthy."
What key?  :-(  The 'release' key for labels?

@lcarva I had a concern about this snippet in my test case update

   if disabled:

       with pytest.raises(NoSuchPluginException):

           get_plugin(plugins, "prebuild_plugins", "bump_release")

   labels = plugin_value_get(plugins, "prebuild_plugins", "add_labels_in_dockerfile",

                             "args", "labels")

   else:

   if not disabled and not release:
       assert plugin_value_get(plugins, "prebuild_plugins", "bump_release",
                               "args", "hub") == hub

       assert 'release' not in labels

   elif not disabled and release:

       with pytest.raises(NoSuchPluginException):

           get_plugin(plugins, "prebuild_plugins", "bump_release")

       assert 'release' in labels

   elif disabled and not release:

       with pytest.raises(NoSuchPluginException):

           get_plugin(plugins, "prebuild_plugins", "bump_release")

       assert 'release' not in labels

   elif disabled and release:

       with pytest.raises(NoSuchPluginException):

           get_plugin(plugins, "prebuild_plugins", "bump_release")

       assert 'release' in labels

The last 'elif' concerns me.  I read it as 'if the hub is disabled but the release is there. then expect the release'.  I thought if the 'hub/secrets/whatever' is disabled then the 'release' code would not be traversed because the bump_version is not added.  Otherwise if you can add a release in the spec and have it auto-added, then my asserts in the early part of my patch should have failed.

What is the expectation of 'release'.  If it is provided always use it? Or only on production builds?

What is the expectation of 'release'. If it is provided always use it? Or only on production builds?

For an orchestrator build there are two places the release value can come from. 1) user has specified it in the Dockerfile via the 'release' label; or 2) user has omitted 'release' label in Dockerfile causing the bump_release plugin to run and determine what this value should be.
By the time a worker build is to be created by the orchestrator build, the release value has already been determined. Whether the value came from Dockerfile label or from bump_release plugin is irrelevant.
For a worker build, the expected behavior with this changes is that the release value is always given when build is created.
But I think your question is regarding what happens if someone creates an orchestrator build with a release value. I think it makes sense to use the value and propagate it down to the worker builds. We can probably achieve that by simply enabling add_labels_in_dockerfile in orchestrator_inner:1.json. On the other hand, we can also restrict this behavior.
@twaugh, what do you think?

Yes, maybe running add_labels_in_dockerfile is the answer to that.


Update Build Documentation for IntelliJ 2023.3.0

I trying to do a release for #91, I checked out the repo and install IntelliJ 2023.3.0. I followed the steps at https://jetbrains.org/intellij/sdk/docs/basics/getting_started/setting_up_environment.html and

Installed the Plugin Devit plugin
Downloaded a JDK through IntelliJ and added it to the project: openjdk-21
Added IntelliJ Platform Plugin SDK: Pointed at install directory for IntelliJ 2023.3.0
Checked out IntelliJ Community source code and added it to the SDK Sourcepath in Platform Settings | SDK

When I try to build the plugin, I get build errors:
clion-cppcheck\src\com\github\johnthagen\cppcheck\Configuration.java:3:36
java: package com.intellij.openapi.options does not exist

@firewave Do the build instructions need updated? I know that the guts of this plugin haven't been updated in a long time (e.g., #69 would be ideal). Are you still able to build it on a clean install of IntelliJ 2023.3.0?

@firewave Thanks for the tip. I was able to build the plugin using 2023.1.5.
1.6.3 has been uploaded and is awaiting moderation from the JetBrains teams. The notice says this may take up to two business days.

I found the issue. You simply need to add a new Plugin SDK again as they split off some code into separate JAR files and they are missing if you have an existing SDK.

If you are able to update the README here: https://github.com/johnthagen/clion-cppcheck?tab=readme-ov-file#development
that would be awesome.


Add thunder changes event

Added an event for when the thunder changes in a world, weirdly a separate event from WeatherChangeEvent

Attempted the above corrections, please let me know if I did anything in a weird way lmao


OpenAPI deepObject serialization for query parameters

First check

[x] I added a very descriptive title to this issue.
[x] I used the GitHub search to find a similar issue and didn't find it.
[x] I searched the FastAPI documentation, with the integrated search.
[x] I already searched in Google "How to X in FastAPI" and didn't find any information.
[x] I already read and followed all the tutorial in the docs and didn't find an answer.
[x] I already checked if it is not related to FastAPI but to Pydantic.
[x] I already checked if it is not related to FastAPI but to Swagger UI.
[x] I already checked if it is not related to FastAPI but to ReDoc.
[x] After submitting this, I commit to:

Implement a Pull Request for a confirmed bug - or implement the feature described in this issue :)

Description
Allow usage of deepObject serialization from OpenAPI 3.
What it could look like
from fastapi import FastAPI

app = FastAPI()

class UserQuery(BaseModel):
    role: str
    firstName: str

@app.get("/")
def read_root(id: Optional[UserQuery]):
    return {"Hello": "World"}

Query:
/users?id[role]=admin&id[firstName]=Alex

Related

https://github.com/tiangolo/fastapi/issues/245#issuecomment-762729917

Links to a good example: https://stackoverflow.com/questions/48491688/how-to-define-parameters-with-square-brackets-in-openapi-swagger

https://github.com/tiangolo/fastapi/issues/203

See also #283 for broader support of encoding styles.
This is the sort of issue where it's not entirely clear if the responsibility of supporting this should be on FastAPI or Pydantic. On one hand, it has to do with value parsing and encoding, which is something FastAPI tends to fully delegate to Pydantic, but on the other hand it's not part of JSON Schema, it's an OpenAPI extension that only applies to route parameter objects, and most of these encoding styles don't make sense in a JSON context (which is what pydantic is generally meant for).
I would like to see these supported as well.

I'm keen to pursue this issue as I think it is a fairly common use-case (for example what to sort by or filter by)
See links on the different supported options below:

OpenAPI
Swagger

The current solution is to create your own parser if you want to support it. However, the only way I have found is to use a regex pattern to display on the OpenAPI spec. See below link for an example of a rough implementation of custom parsing for sortby and filterby
https://gist.github.com/ghandic/21c27470f6797dd856208a2c68f3e43a
Ideally this could look more as follows
from fastapi import QueryMixIn
from pydantic import BaseModel

...

class FilterBy(BaseModel, QueryMixIn):
    name: str
    q: str

    class Config:
        query_style = "deepObject"

class SortBy(BaseModel, QueryMixIn):
    name: str
    by: str

    class Config:
        query_style = "deepObject"

@router.get("/search")
async def search(
    sort: Optional[SortBy] = Query(None, ),
    filter: List[FilterBy] = = Query(None, ),
    limit: int = Query(10, ge=1, le=50),
    page: int = Query(1, ge=1),
) -> List[Record]:
    ...

Then the user could standardize their Query style across their API using inheritance
from fastapi import QueryMixIn
from pydantic import BaseModel

class QueryModel(BaseModel, QueryMixIn):
    class Config:
        query_style = "deepObject"

from custom import QueryModel

...

class FilterBy(QueryModel):
    name: str
    q: str

class SortBy(QueryModel):
    name: str
    by: str

@router.get("/search")
async def search(
    sort: Optional[SortBy] = Query(None, ),
    filter: List[FilterBy] = = Query(None, ),
    limit: int = Query(10, ge=1, le=50),
    page: int = Query(1, ge=1),
) -> List[Record]:
    ...

The tough problem I see would be validation, but since we're using pydantic could the user not use their validators on their custom QueryModel inherited class
Thoughts?

Please see example implementation here: https://github.com/ghandic/FastAPI-deepObject

Ideally, I'd like to not define the types twice, would need to understand more of how the routes get registered for that implementation.

Here is a example of what the OpenAPI spec should look like:
openapi: 3.0.2
info:
  title: FastAPI
  version: 0.1.0
paths:
  /:
    post:
      summary: Demo
      operationId: demo__post
      requestBody:
        content:
          application/json:
            schema:
              $ref: '#/components/schemas/Sort'
        required: true
      responses:
        '200':
          description: Successful Response
          content:
            application/json:
              schema: {}
        '422':
          description: Validation Error
          content:
            application/json:
              schema:
                $ref: '#/components/schemas/HTTPValidationError'
    get:
      summary: Demo
      operationId: demo__get
      parameters:
      - in: query
        name: sort
        schema:
          $ref: '#/components/schemas/Sort'
        style: deepObject.   # ENABLE THIS
        explode: true   # ENABLE THIS
      responses:
        '200':
          description: Successful Response
          content:
            application/json:
              schema: {}
        '422':
          description: Validation Error
          content:
            application/json:
              schema:
                $ref: '#/components/schemas/HTTPValidationError'
components:
  schemas:
    Direction:
      title: Direction
      enum:
        - asc
        - desc
      type: string
      description: An enumeration.
    HTTPValidationError:
      title: HTTPValidationError
      type: object
      properties:
        detail:
          title: Detail
          type: array
          items:
            $ref: '#/components/schemas/ValidationError'
    Sort:
      title: Sort
      required:
        - by
        - direction
      type: object
      properties:
        by:
          title: By
          pattern: ^[a-z]+$
          type: string
          example: name
        direction:
          $ref: '#/components/schemas/Direction'
    ValidationError:
      title: ValidationError
      required:
        - loc
        - msg
        - type
      type: object
      properties:
        loc:
          title: Location
          type: array
          items:
            type: string
        msg:
          title: Message
          type: string
        type:
          title: Error Type
          type: string

For some demo app (proposed solution to be worked on)
from enum import auto

from fastapi import FastAPI, Query
from pydantic import BaseModel, Field
from fastapi_utils.enums import StrEnum

app = FastAPI()

class Direction(StrEnum):
    asc = auto()
    desc = auto()

class Sort(BaseModel):
    by: str = Field(..., example="name", regex="^[a-z]+$")
    direction: Direction

@app.post("/")
def demo(sort: Sort):
    return {}

# NOTE: Currently says:
# AssertionError: Param: sort can only be a request body, using Body(...)
# .... /lib/python3.8/site-packages/fastapi/dependencies/utils.py", line 331, in get_dependant
# This is where we should allow using deepObject
@app.get("/")
def demo(sort: Sort = Query(...)):
    return {}

@tiangolo would you be supportive of this if the effort was put in to build the PR? It would take a good amount of effort and even more testing.

For what it's worth I was able to support the full range of options in Xpresso, maybe that can serve as inspiration for FastAPI: https://xpresso-api.dev/0.42.3/tutorial/query_params/#customizing-deserialization

How about just Query(style="deepObject")?

Rejected in #9768

How about just Query(style="deepObject")?

Rejected in #9768

I don't think Sebastián closed the issue actually, I'll reply there


Benchmark all examples with codspeed

This PR adds benchmarking to CI by running all of our example files with codspeed.
Hopefully, it should not only help us see how PRs affect performance but also give us some profiling through codspeed to diagnose where things speed up or slow down.

I think this needs an admin on the egraphs-good org to approve the app in codspeed, I had to "request" adding it to this org.
I also would need admin permissions on this repo to add the CODSPEED_TOKEN secret, or someone else who already has them can create one and add them here!

In response to some questions offline on any downsides to adding codspeed:
It introduces some more complexity overall in terms of depending on external services, and will add some more noise to PRs about their performance impacts.
Performance is also not an exact science, so it has the possibility of introducing some false assurances, when in fact the impact might be different when running in a different environment. Also, it means that we are judging performance based on the example files which might not representative of performance sensitive workloads.
Overall though I would judge the additional signal to be worth those, in helping us judge the impact of PRs on performance, especially those meant to make things faster. I think it will also encourage us to have representative example files.
I have found codspeed to be invaluable in the Python package to help diagnose the impact or large changes on end to end performance and it has helped me refine those changes so that the impacts are reasonable. It's built in flamegraphs have also helped me narrow down where the slowdowns occur and also see upstream slowdowns from egglog changes. So adding this instrumentation closer to the source would be helpful in getting that information sooner.

This is ready for a review now. There are lots of tradeoffs with how accurate we want benchmarks to be vs their overall time, but I tried to find a good balance for now.
Currently, the benchmarks CI task takes ~ 9 minutes to run. Why does it take so long? Codspeed instruments the code to measure CPU cycles to get more reliable benchmarks on flaky hardware like that in CI:

CodSpeed instruments your benchmarks to measure the performance of your code. A benchmark will be run only once and the CPU behavior will be simulated. This ensures that the measurement is as accurate as possible, taking into account not only the instructions executed but also the cache and memory access patterns. The simulation gives us an equivalent of the CPU cycles that includes cache and memory access.

In order to keep the timings reasonable, I reduce the number of times the eggcc-extraction schedules run from 5 to 2. Currently, on 2 it takes ~4 minutes for that benchmark alone to run. If I switch it even to 3 it takes 46 minutes, which seems too long. One question might be if the eggcc-extraction is doing similar enough work at say 2 runs opposed to 3 runs. Here is a screenshot of different profiles, the 3 on the left (taking around 10x as long) and the 2 on the right:

They look similar enough to probably have reasonable differences in performances based on the shorter one.
Here is a rough outline of where time is spent in the benchmarks currently:

I hope that even if this isn't a perfect representation of production tasks, it can still be helpful as we iterate over PRs. If your PR is minor, you can always merge it in without waiting for the benchmarks to finish if you don't want to wait.

I had increased the "threshold" for codspeed to say whether the performance change was meaningful to 10%. I brought it back down to 5% so it can alert us to more changes:

I think the main thing it affects is the comments on PRs. If you click on the report, you can see a granular breakdown of all changes.

@saulshanabrook Are you sure about this last change? The website says that the default is 10%, and 10% seems more in line with the variance that we're seeing.

Yeah I'm open to that! It just seems like a 5% speedup on the long running. Benchmarks is actually rather significant... And if we are trying to improve them then we might need many smaller performance improvements to get there. I don't see much variability at all on the larger benchmarks.


Can $LATEST be the default?

It would be nice if $LATEST were the default for version and therefore optional.

Thanks for the PR


bower tag

Hi,
seems that you haven't tagged the lasted commit, from bower i can access only to the 0.0.1

Whoopsie :) I'll get to fixing it today.

Thank you and congratulations for this really great widget

I'm still getting 0.0.1 from bower install ion-google-place.
And thanks for the great plugin :)

+1


Feature/toast placement

Description
Fixes DG-370

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Phi-Thien Hoang seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #174 into develop will increase coverage by 0.02%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #174      +/-   ##
===========================================
+ Coverage    91.13%   91.15%   +0.02%
===========================================
  Files          126      126
  Lines         2098     2103       +5
===========================================
+ Hits          1912     1917       +5
  Misses         186      186

Impacted Files
Coverage Δ

src/scripts/main/toast/index.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

I removed the dynamic sticky functionality for now, as it introduced trouble with toast on mobile. The issues with toast on IE is still present


Automated cherry pick of #5529: fix: 避免重装系统失败后imageId依然发生改变

Cherry pick of #5529 on release/2.12.
#5529: fix: 避免重装系统失败后imageId依然发生改变

Codecov Report

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

@@              Coverage Diff               @@
##             release/2.12   #5533   +/-   ##
==============================================
  Coverage                ?   6.82%
==============================================
  Files                   ?     506
  Lines                   ?   86352
  Branches                ?       0
==============================================
  Hits                    ?    5896
  Misses                  ?   79960
  Partials                ?     496

Flag
Coverage Δ

#aFlag
6.82% <ø> (?)

Continue to review full report at Codecov.

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

/lgtm

/approve


read the header of the file when then are not ending in a know suffix

Hello,
i have a case in our project Argonaut https://github.com/fusiondirectory/argonaut that the main programs are not ending in .pm or .pl etc.. so perlcritic find the issues but the sonar-perl scanner doesnt report them to sonar.
it would be nice that if the perl scanner encounter a file without an extension it read the header of the file and try to fin if there is a #!/usr/bin/perl at the beginning of the file so the result of this file could be send to sonar
i attach the perlcritic file and the output of the jenkins running the sonnar scanner
Thanks and happy new year !
perlcritic.txt
sonar-perl-jenkins.txt

Hi,
I don't think it is a good idea, performance-wise. It also does not follow common SonarQube practice. Do your files have no filename extension at all? Or is it just a different one than the usual? In the latter case, just add those extensions to com.github.otrosien.sonar.perl.suffixes config property.

According to http://docs.sonarqube.org/display/SONAR/Analysis+Parameters there's also sonar.import_unknown_files - might help when attaching perlcritic issues to those files, but I haven't tested that.

hello,
no they don't have any extension, but i will test with this sonar.import_unknown_files it seems to be easier don't now if we can do that by project
Cheers

hello,
i have tested and that configuration work perfectly on my sonar-scanner inside jenkins
sonar.import_unknown_files=true
sonar.exclusions=READ*,Change*,COPY*,AUTH*,perlcritic_report.txt
maybe we could a a paragraph about this in the documentation ?
Cheers

Sounds good. Mind to give it a try?


States for stage and pipeline - unscheduled, scheduled, executing, done/failed

unscheduled -> scheduling -> scheduled?

"described" as opposed to "unscheduled"

"queueing",
"dequeueing"="synchronizing"


Bubblechat use new chat events

Bubblechat switch to new chat events

Thanks for your pull request. We are already working on this internally and will be moving the bubble chat script out of the corescripts and into the new chat in january.


Enable integrations for CodeCov and ReadTheDocs

@jrbourbeau, I have tried to set up integrations for CodeCov and ReadTheDocs, I believe that they are waiting on your approval at the organisation level. Can you please review and approve these as appropriate, possibly from https://github.com/organizations/pycondor/settings/installations?

Sorry for the delay @duncanmmacleod -- I'm just getting back from a couple weeks of PTO. I've approved the codecov and readthedocs integration requests. Please let me know if there's anything else needed for this at the moment


fix start date on history

ticker.basic_info["last_price"]

sometimes raises:
GDEVW: 1d data not available for startTime=-2208994789 and endTime=1687780922. Only 100 years worth of day granularity data are allowed to be fetched per request.

Run a floor or ceil on max start, it will help with cache reuse.

I neglected argument - floor('D') or ceil('D'), hopefully now obvious why. If stuck look for other examples in yfinance

thanks, updated

I just noticed the docstrings still say default is 1900. Can you update?

new pull request I guess, right?

Yup, can't modify a merged PR.

where is wiki?
https://github.com/ranaroussi/yfinance/pull/1584/files

https://github.com/ranaroussi/yfinance/wiki


Object Highlighter

Enables players to highlight any in-game object with customizable colors and styles through a simple shift + right-click interface.
Includes individual highlighting or "All"

Why are you trying to remake object indicators? What does this plugin do significantly different that the core plugin doesn't already do (better)?

Why are you trying to remake object indicators? What does this plugin do significantly different that the core plugin doesn't already do (better)?

Mine allows to tag "All" of that object ID as well as individually, which can be useful for certain cases

We no longer allow highlight by id plugins

We no longer allow highlight by id plugins

Ah okay then, sorry for asking but I'm curious as to why

We no longer allow highlight by id plugins

Ah okay then, sorry for asking but I'm curious as to why

There's too much potential for abuse, and it puts a big burden on reviewers.


Update CODEOWNERS for master

@swift-ci smoke test

@swift-ci please smoke test

@swift-ci please smoke test linux

@swift-ci please test

@swift-ci please smoke test

If we find ourselves updating this more frequently or having to cherry-pick it across branches, GitHub also allows for groups to be specified as code-owners. That way, something like @apple/swiftpm-team could be used to manage membership.

Agreed.  In this particular case, Rick is also added to the owners for the 5.3 branch (since he is the release manager) but is not on added for master.  So in this case it wasn't a cherry-pick, but maybe we could make this list more versatile using groups.


Improve code coverage on FHIR Core

IMPORTANT: Where possible all PRs must be linked to a Github issue
Fixes #326
Type
Choose one: (Bug fix | Feature | Documentation | Testing | Code health | Release | Other)
Checklist

[ ] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[ ] I have added any strings visible on UI components to the strings.xml file
[ ] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[ ] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[ ] I have built and run the fhircore app to verify my change fixes the issue and/or does not break the app

Pull Request Test Coverage Report for Build #151

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.2%) to 75.844%

Totals

Change from base Build #150:
1.2%

Covered Lines:
876

Relevant Lines:
1155

💛  - Coveralls


Fixes Dijksta's shortest path algorithm

Description 📜
Discussed in detail about Dijkstra’s Shortest Path Algorithm.
Fixes #5920

Type of change 📝

[x] Documentation (Content Creation in the form of codes or tutorials)

Domain of Contribution 📊

[x] DSA

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@sneha-nitdgp  Thanks for suggestions.  Updated recommended changes.


[Question]: I have a integer array. I want to specify maximum and minimum for integer stored in the array.

What are you wanting to achieve?
I have an integer public int[] array {get; set;}.
I have used Maximum and Minimum length to specify a range for the array. I want to specify the same for the integer stored in the array
What code or approach do you have so far?
[Minimum(0), Maximum(100)]
public int[] array {get; set;}
Additional context
No response

You could use the RangeAttribute to do this. But the problem is that right now is not valid the Swagger generated, I will do a PR to fix this

You can use this library if you use FluentValidation in your project. https://github.com/micro-elements/MicroElements.Swashbuckle.FluentValidation. RuleForEach works as expected

I am using Swashbuckle.AspNetCore.Annotations.


Added premixing template; bugfix for wmagent deployment script

For the record, in order to make deterministic pileup really work, we need at least:

set 'DataPileup' instead of 'MCPileup'
set 'DeterministicPileup' to True
have the correct DataMixingModules in the job configuration (proper ConfigCacheID)

Eric will talk to Liz just to make sure we properly understood it and to have everyone on the same page.
BTW, all my jobs failed with "RootInputFileSequence::readOneSequential(): no input files specified for secondary input source.", that's another thing to investigate now.
Anyways, @ticoann feel free to merge it.

Updated the PR with LumiList for the DQMHarvest template

Ignore this for now. Beginning pylint and unit tests.


Create temporary queues on-demand as needed

Currently, the bus queue is always created. Consider creating this on-demand when needed by the request client, and make it an option to specify that they aren't created.
I'm not sure what side effects this will cause, but enough people ask about it so consider it an opportunity to reduce the queue count on the broker.

[ ] RabbitMQ
[ ] Azure Service Bus
[ ] ActiveMQ
[ ] Amazon SQS

Hello, this feature would be very useful to me, is there an update on this effort?
Thank you

Just read on aws whats-new blog there is first class support for temporary queues, https://aws.amazon.com/about-aws/whats-new/2019/07/temporary-queue-client-now-available-for-amazon-sqs/ will have a look if this might be something we could use in the amazon sqs transport related to #1389

Add a receive endpoint configuration flag so that a receive endpoint can opt-out of being started automatically. When a consumer connects to the receive endpoint, it would then start - which may include asking for the receive endpoint's address in the case of it being generated as a temporary queue name.
This would allow the bus endpoint to remain stopped until a request client or something connected a consumer to it. Or until a send/publish using the IBus's IPublishEndpoint was initiated, since it would ask for the input address as the SourceAddress.
(these notes are for me, so that I can actually do it on the v6 branch later)

Hello! i created  temporary queue:
"ConnectHandle handle = busControl.ConnectConsumer();"
then i publish "MyMessage"
why consumer not getting message?

Because the bus endpoint doesn’t bind or subscribe consumers. It’s meant to be used for responses and other messages sent directly to the queue.

Thank! Understood!
On Thu, Jun 4, 2020 at 2:56 PM Chris Patterson<EMAIL_ADDRESS>wrote:

Because the bus endpoint doesn’t bind or subscribe consumers. It’s meant
to be used for responses and other messages sent directly to the queue.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/MassTransit/MassTransit/issues/1390#issuecomment-638801785,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN3TYXS5GSL22FOTJGYMHJDRU6DV3ANCNFSM4HARZT3Q
.


Add validate the input mac address on docker run command

Signed-off-by: Lei Jitang<EMAIL_ADDRESS>when docker run with a invalid mac address, the container will still start and  assigned a
random mac address.
Is this a desired behavior?
I think it should show a message to info the user that the input mac address is invalid.
Fixes #11027

This makes sense--can you add a test for the new validation?

add a test @estesp

ping @jfrazelle

Thanks for the tests! I made a few minor comments; please fix and then I think this is ready to go

@estesp thanks for you suggestion, update

thanks, LGTM

LGTM


Ad blocks WebView

Describe the bug
The ad blocks the WebView, hiding part of the user interface of my app. There does not seem to be any option in initialize() or showBanner() for the WebView to be resized when the ad is loaded and displayed.
To Reproduce
await AdMob.initialize();
let [consentInfo,trackingInfo]=await Promise.all([AdMob.requestConsentInfo(),AdMob.trackingAuthorizationStatus()]);
if(consentInfo.status===AdmobConsentStatus.REQUIRED&&trackingInfo.status==='notDetermined'){
	if(consentInfo.isConsentFormAvailable){
		consentInfo=await AdMob.showConsentForm();
	}else{
		await AdMob.requestTrackingAuthorization();
		trackingInfo=await AdMob.trackingAuthorizationStatus();
	}
}
AdMob.showBanner({
	adId:'ca-app-pub-3940256099942544/9214589741',
	adSize:BannerAdSize.ADAPTIVE_BANNER,
	position:BannerAdPosition.TOP_CENTER,
	isTesting:!!process.env.DEBUGGING,
	npa:consentInfo.status===AdmobConsentStatus.REQUIRED||trackingInfo.status!=='authorized'
});

Expected behavior
The banner should not occlude my app's UI in the WebView. If I use the cordova version of https://github.com/admob-plus/admob-plus the banner does not block my UI (but I am unfortunately unable to get that working on capacitor).
Screenshots
@capacitor-community/admob (blocked UI)

https://github.com/admob-plus/admob-plus (UI not blocked)

Smartphone (please complete the following information):

Device: Samsung Flip 4
OS: Android
Browser: Capacitor

You have to listen to the ads events and resize your app hmtl accordingly.


authors in Zenodo

Hi,
We need clarification about where authors are derived from in Zenodo when importing a GitHub repo. Is it from the GitHub user account, or is it from the CONTRIBUTOR file.
Also note in Zenodo you can add authors.
I could test out where the authors are derived from but if someone has the answer that will be easier.
When we have clarification I'll edit para.
Simon

Hi,
I raised a ticket on this with Zenodo/OpenAIRE as it can become a little nuanced (what a surprise).
QUOTE:
Reference | 01251-2018-07-21
Hi,
I'm writing documentation on submitting GitHub repositories for researchers to Zenodo and I need help on how Zenodo gets author information from GitHub.

Is it a. from the GitHub user names, or b. from a CONTRIBUTOR or other metadata file in the repo.

Also

Do the release versions inherit the author names from the 'concept submission' or from the names in the GitHub repository at the time of release submission.

Many thanks
Simon
END QUOTE
Zenodo have DOI model of (with each DOI having its own editable metadata) - eek, problemo?
v1.0 (specific version): 10.5281/zenodo.60943
v1.1 (specific version): 10.5281/zenodo.800648
Concept (all versions): 10.5281/zenodo.705645
http://help.zenodo.org/
S

Is it from the GitHub user account, or is it from the CONTRIBUTOR file.

Definitely (also) from the git history. I am "co-author" of several Zenodo records, to whose source repo on GitHub I contributed.

Hi,
Ah great thank you. Interesting how the attribution is getting assigned, defo appears that it needs some further systemization. One typo update and I'm senior author ;-).

Only to those who enjoy navel-gazing and tea-leaf-reading more than actual data & its processing ;-D


How to have browser debugger with app on device

This is a question, not a bug report.
Is it possible to run the app through the Expo client app on device, but to have debugging output through the browser on my dev machine?
By default my terminal shows console output but the formatting is all screwed up for debug modules that write with colors/tables etc

Hi @richkeenan! It is possible, on iOS for example you can shake the device to show the dev menu (when your app is open) and then choose "Debug Remote JS". React Native Debugger also works quite well. Check out the usage section to see how to integrate with Expo.

Thanks @brentvatne that's cool. Given that this solution isn't mentioned in the CRNA docs I think I'll make a PR to include this - I'm sure it's useful for people like me, who only have experience of CRNA and not react-native-debugger

@brentvatne does this also work for Android? Your answer only mentions iOS and I don't have an Androis device to test on

Yup. Nice! A PR would be very welcome for that. Sorry I just assumed iOS for some reason, you can do it on Android with the same gesture -- shake the device and choose 'Debug JS Remotely'

@brentvatne Current Android version (Nexus 5) does not invoke any Developer Menu when shaking

Step 1: Ensure device and server on the same network.
Step 2: Deploy the application to the device with debug mode.
Step 3: Shake device -> Select Remote JS Debugging.

I have the same question as @dmongit. we are having considerably different performance on a handset vs a simulator and need to debug the handset but the shake does not bring up a debug menu.

Hi @brentvatne ,
Please give me full step by step suggestion for how to debug application on chrome.


how to automate the test case for upload document(PDF/JPG/ZIP) file using Android Espresso

how to automate the test case for upload document(PDF/JPG/ZIP) file using Android Espresso

how to automate the test case for upload document(PDF/JPG/ZIP) file using Android Espresso


Resized images are actually 9x the specified resolution

Small is actually 1536x1152, medium is 2304x1728, and large is 3072x2304.
The file sizes are also wrong. For the image I tested, large was 2MB, which seems close enough to the estimate given the incorrect resolution, but small was 641KB, which is 16x.
I tested with an iPhone 8 Plus, and the images were originally HEIF before Riot converted them to JPEG. I'm currently using version 0.6.15 on iOS 11.3, but the issue was present in earlier versions on iOS 11.1.

Hmm I can't reproduce this. The sizes are exactly the ones specified. Can you test this again on the latest version?


ci: add Windows x64 wheels

Also aim to fix #24.
Thanks @plusiv for https://github.com/ada-url/ada-python/pull/46#issuecomment-1776388272, where he came up with the solution.
It's a little bit tricky: maybe make cannot work with vcvars64.bat to modify the env, so I use a long before-build.
And I don't know how to choose from vcvars64.bat and vcvars32.bat by the OS arch. So only x64 wheels are available.

Some questions:

Why do we include .o, .cpp, .c, .h files in the wheels? Afaik, ada works well without them.
pip install ada_url --no-binary ada_url will fail (on both Linux and Windows) because of the absence of ada.o, same as https://github.com/ada-url/ada-python/pull/46#issuecomment-1776388272. Maybe we can open a new issue for this?

Microsoft Visual C++ 2015-2022 Redistributable is needed at runtime.

Excellent! I'll get a Windows machine to test this on.


fix(legacy): fixes issue in legacy browsers

This ensures default is quoted in Objects.

Coverage remained the same at 89.0% when pulling b62f01705fabf491ed9824ce4f978f4ee0231231 on jnewman-fix-old-ie into cdb2336cc892f2b6292312cf53e4cdc7ba710907 on master.


Document InputMessage component

We currently have an InputMessage component that is used under the hood in a lot of our components. We should expose it and properly document it so it can be used on its own.
In addition, we may want to consider refactoring its current maxCount prop. Since it will now be exposed, we should maybe use a prop more specific to the component to determine whether it should wrap the children with Announce.
AC:

Add to storybook
Add docs site dev page
Add proper usage in design docs

Will do this after doc site has component groups.


No-Bake Cookies

#1

Perfect! Just FYI -- when you're making a pull request between forks, no additional permissions are necessary (i.e. i don't need to be added as a collaborator to your fork). This is how github facilitates public contributions to open source projects. Anyone can fork a public repository and submit a pull request to the original project.

Perfect! Just FYI -- when you're making a pull request between forks, no additional permissions are necessary (i.e. i don't need to be added as a collaborator to your fork). This is how github facilitates public contributions to open source projects. Anyone can fork a public repository and submit a pull request to the original project.


EDU-812 Expand timeline component for color support

Closes https://educandu.atlassian.net/browse/EDU-812
Version 1

Version 2

Looks nice, even more in version 2 :-)


acid_table_compaction_reqs.sql generates invalid commands

The script generates commands like:
ALTER TABLE db.table PARTITION (name=val) COMPACT 'MAJOR'
but when trying to run, they fail with:
Error: Error while compiling statement: FAILED: ParseException line 1:79 cannot recognize input near 'val' ')' 'COMPACT' in constant (state=42000,code=40000)

It looks like we need to surround the val with quotes.  Something like the following seems to work:
ALTER TABLE db.table PARTITION (name='val') COMPACT 'MAJOR'

Somewhat related:
We have some tables partitioned by a timestamp.  They look like:
ALTER TABLE db.table PARTITION (name=2017-12-21 02%3A07%3A18) COMPACT 'MAJOR';
What's the correct way to do this?  I've tried with quotes and replacing %3A with :


[v4] replyTo is not functioning when using batch

replyTo no longer functions after upgrading to v4 and renaming from reply_to. Issue not reproducible on v3.5, when reply_to is still being used.
My usage (simplified):
resend.batch.send([
  {
    from,
    to: email,
    replyTo: data.email,
    subject: data.subject
      ? `[${siteDisplayName}] ${data.subject}`
      : `You got a message from ${siteDisplayName}`,
    react: <ContactFormNotification {...data} />,
  },
  {
    from,
    to: data.email,
    subject: `Got Your Message From ${siteDisplayName}!`,
    react: <ContactFormAcknowledgement {...data} />,
  },
]);

https://github.com/mwskwong/mwskwong.com/blob/20469fb8c20266821fdf5d61db2e2994e3e04bac/src/lib/actions.tsx#L25

I suspect it is due to:
https://github.com/resend/resend-node/blob/0d9568b5f014b50153e30d59b00f3d14f828106f/src/batch/batch.ts#L35
Comparing it to:
https://github.com/resend/resend-node/blob/0d9568b5f014b50153e30d59b00f3d14f828106f/src/emails/emails.ts#L53
The create() function in Batch isn't renaming replyTo back to reply_to when calling post().

Using reply_to is working but it gives a type error.

in my case, after upgrading to v4 of resend npm package, reply_to is always set to the same email as to
I had to downgrade back to v3.2.0 to make it work again

I'm using this solution in the meantime
c.forEach((c) => c.replyTo && ((c as any).reply_to = c.replyTo));
await resend.batch.send(c);


Added exercise kindergarten-garden based on Java track

Another exercise missing in the Kotlin track.

Hell0 there @lathspell.
Thanks for taking the time out to add this exercise.
I'm wondering if you have any time to address the above comments. I'm trying to clean up the PR queue. If you don't have time to address them, then do you mind if I close this?


Assembly performance tests

Added a performance test for the assembly API and in the process added some caching for the contigs. This will actually need to be turned back off for the new objects, once the code is modified to access the pre-computed contigs in the object itself. But for the old objects, it is nice still. Although one could worry about blowing out memory.. depends on how many contigs are being accessed.

By the way, because I was working in master for this and the other PR that tries to run in TravisCI in parallel, this branch has both sets of commits. I'll comment out of this one whatever doesn't work, but actually this might make the merge easier rather than harder later.
Either way I think this needs to get merged pretty soon.

Current coverage is 62.93%

Merging #88 into develop will increase coverage by +0.15% as of 3ac0c59

@@            develop     #88   diff @@
=======================================
  Files            14      14
  Stmts          2112    2142    +30
  Branches          0       0
  Methods           0       0
=======================================
+ Hit            1326    1348    +22
  Partial           0       0
- Missed          786     794     +8

Review entire Coverage Diff as of 3ac0c59

Powered by Codecov. Updated on successful CI builds.

Thanks Dan, merging.


Fix L2CAP API to support OSX 10.14

Fix for issue #344

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Sylvain Rebaud seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Circle errors

Looks like the current circle error is due to to the fact that littler loads so few packages.  You could add methods, stats, graphics, and grDevices to the library calls in the niche-modelling, but I find it easier to omit these and run with:
R -e 'rmarkdown::render("manuscript.Rmd")'

That way you get all the packages a usual R session has (unlike r or RScript).
It also looks like you will then need to install the most recent wicket version from GitHub to resolve another error.
That should get you up to the issue at #8.
HTH

p.s.  since rocker/ropensci is such a big image, I'd consider switching to rocker/geospatial:3.4.1 (using the vesion tag for reproducibility) , and add a DESCRIPTION file to this repo like so:
Package: compendium
Version: 0.1.0
Depends: rmarkdown, tidyverse, rgbif
Remotes: ropensci/wicket@99d3b14

Then I'd have circle-ci run:
run -ti -v $(pwd):/data -w /data rocker/geospatial:3.4.1 R -e 'devtools::install(); rmarkdown::render("manuscript.Rmd")'

made the changes, still getting a wicket error though, not sure why

got a successful run https://circleci.com/gh/sckott/gbifms/58
now trying to get deploy working

working now except deploy is not working, meh, im over it, can't get it to work, can just manually update pdf in repo i guess

yeah, I usually do that anyway so I don't like having to copy my credentials around. (I do wish GitHub would issue repo-specific PATs instead of global PATs!)  Good that it builds on circle, that's always nice for reproducibility!
Anyway, I think this is good to go as a preprint now, yes?

i think so, okay if i submit?
(for deploy, i'll see if can get a deploy to S3 at least, so we have a stable URL to link to)

yup, submit away! 🚀
For stable URL, let's just activate the Zenodo toggle and tag a preprint release.  Of course that gives us only a DOI for the repo as a whole, but anyone just wanting the pdf file should probably use the DOI that goes to the preprint anyway.

Right, makes sense  on Zenodo front, will do

thoughts @cboettig ?

computer sci i guess?

I would go with Life Sci, since I think that's more the 'audience' of the piece.


DROP DATABASE can fail even after last connection is closed

https://github.com/edgedb/edgedb/pull/4565 hacks around this to prevent some test flakes, but it is worth fixing the bug.
We should revert https://github.com/edgedb/edgedb/pull/4565 when we do so.

_ensure_database_not_connected actually does have a retry loop that waits for all the connections to be dropped. I strongly suspect that the issue is that the connection pool is reestablishing connections


[HIGH-HIGH] Change delivery Email Address of Contact Form submissions with subject: "Trouble with Website"

hey @JasonTheAdams! When Users submit contact form with the "nature of your inquiry" selected as "Trouble with Website", it is not sent to info@ email address. We need to change this. Right now these emails are probably being sent to an account that is not checked.
Pretty sure that the 3 other Contact Inquiry subjects are being delivered to info@, but please double check that, too.

@dreazee I just switched the contact form over to Gravity Forms. It now sends all information to info@ no matter what the nature of the inquiry is. It also sends out a notification to the person sending the email as before.
You should have received 4 emails from me, one per inquiry type. Please confirm that you've received these. I can confirm that I received a notification to my email for each submission.


Konflux update test-component-pac-llhnae

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-llhnae-on-pull-request has failed.
Namespace: build-e2e-hzzb-tenant
PipelineRun: test-component-pac-llhnae-on-pull-request-2v7dr

Task Statuses:

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
21 seconds
clone-repository

❌ Failed
1 minute
build-container

✅ Succeeded
35 seconds
show-summary

Failure snippet:
task build-container has the status "TaskRunImagePullFailed":
the step "upload-sbom" in TaskRun "test-component-pac-llhnae-on-pull-request-2v7dr-build-container" failed to pull the image "". The pod errored with the message: "Back-off pulling image "quay.io/redhat-appstudio/cosign@sha256:c883d6f8d39148f2cea71bff4622d196d89df3e510f36c140c097b932f0dd5d5"."

Pipelines as Code CI/test-component-pac-llhnae-on-pull-request has successfully validated your commit.
Namespace: build-e2e-hzzb-tenant
PipelineRun: test-component-pac-llhnae-on-pull-request-z6hfb

Task Statuses:

StatusDurationName

✅ Succeeded
10 seconds
init

✅ Succeeded
19 seconds
clone-repository

✅ Succeeded
41 seconds
build-container

✅ Succeeded
8 seconds
sast-snyk-check

✅ Succeeded
16 seconds
apply-tags

✅ Succeeded
9 seconds
push-dockerfile

✅ Succeeded
26 seconds
ecosystem-cert-preflight-checks

✅ Succeeded
15 seconds
deprecated-base-image-check

✅ Succeeded
1 minute
clair-scan

✅ Succeeded
49 seconds
clamav-scan

✅ Succeeded
17 seconds
sbom-json-check

✅ Succeeded
6 seconds
show-sbom

✅ Succeeded
9 seconds
show-summary

Pipelines as Code CI/test-component-pac-llhnae-on-pull-request has successfully validated your commit.
Namespace: build-e2e-hzzb-tenant
PipelineRun: test-component-pac-llhnae-on-pull-request-49prs

Task Statuses:

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
19 seconds
clone-repository

✅ Succeeded
59 seconds
build-container

✅ Succeeded
13 seconds
ecosystem-cert-preflight-checks

✅ Succeeded
44 seconds
clamav-scan

✅ Succeeded
9 seconds
apply-tags

✅ Succeeded
10 seconds
clair-scan

✅ Succeeded
9 seconds
deprecated-base-image-check

✅ Succeeded
9 seconds
sbom-json-check

✅ Succeeded
8 seconds
push-dockerfile

✅ Succeeded
7 seconds
sast-snyk-check

✅ Succeeded
7 seconds
show-sbom

✅ Succeeded
6 seconds
show-summary

Pipelines as Code CI/test-component-pac-llhnae-on-pull-request has successfully validated your commit.
Namespace: build-e2e-hzzb-tenant
PipelineRun: test-component-pac-llhnae-on-pull-request-fsl5h

Task Statuses:

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
35 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
22 seconds
ecosystem-cert-preflight-checks

✅ Succeeded
51 seconds
clamav-scan

✅ Succeeded
21 seconds
clair-scan

✅ Succeeded
18 seconds
deprecated-base-image-check

✅ Succeeded
23 seconds
sast-snyk-check

✅ Succeeded
19 seconds
sbom-json-check

✅ Succeeded
21 seconds
push-dockerfile

✅ Succeeded
17 seconds
apply-tags

✅ Succeeded
14 seconds
show-summary

✅ Succeeded
7 seconds
show-sbom


re #32 revise readme for first release

resolves #32

Codecov Report

Merging #33 (362cf9b) into main (215e803) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #33   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines           73        73
  Branches        27        27
=========================================
  Hits            73        73

Flag
Coverage Δ

unittests
100.00% <ø> (ø)

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

Continue to review full report at Codecov.

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


Can't edit Clojure files without getting spammed by Linter messages

Error: Got invalid response from Linter, Type: undefined
    at Object.module.exports.validateResults (<anonymous>:6:17)
    at EditorLinter.setMessages (<anonymous>:31:36)
    at <anonymous>:69:12

I had linter-clojure installed, but once I uninstalled it, I still get these messages, which doesn't make any sense to me

This should be filed over on linter-clojure, although it sounds like you might have an issue with your Atom compile-cache hanging onto things it shouldn't. Can you close Atom, delete ~/.atom/compile-cache, start Atom and see if you still see this?

No dice - here's my list of packages:
/Users/paul/.atom/packages (33)
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>

Aha, this is a linter-plus bug that I one-off installed via apm link. Apologies for the noise! I'll keep that compile-cache trick in mind.

Ah, yea 1.0.0 is just a tad old :stuck_out_tongue:.


SIDM-6606 - Added ability to get a assignable roles of user

JIRA link
https://tools.hmcts.net/jira/browse/SIDM-6606
Change description

Added new functions idem-api endpoints

getAllRoles - Get's all roles in system
getRoleByName - Get a role by its name
getAssignableRoles - Get all assignable roles for given array of roles

Added array of assignable role for logged in user into req.session.user.assignableRoles
Updated User and AuthedRequest types

Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

Closed due to being superseded by https://github.com/hmcts/idam-user-dashboard/pull/235


Any way to filter?

Is there any way to filter certain action so they don't appear?  Some of my actions occur several times per second - and I'm not really interested in those ones.

Filtering has been added.


test(HttpResponseMessage) add testing for HttpResponseMessage

Also seperated out header parsing to a static method of headers.
This makes testing slightly easier and it also feels like it belongs to headers.

This library is going to take a while to add all of it's testing so I thought I would make smaller more frequent pull requests.

What you are doing is a huge help. Small or large is welcome :)

Ah shucks, I'm happy to help.


change default pyrsia node port to 7888 #259

change default pyrsia node port to 7888

removing the closing of ticket in the first comment as we do not want to close the issue until we have this port registered in IANA. @betarelease will be doing that work so will assign that ticket to him after this merged


Link to next page is broken

There's a markup syntax error that causes the "next page" button to not be rendered.

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

ID: 0434976b-9b0b-1680-293a-a9f990c828c5
Version Independent ID: 0434976b-9b0b-1680-293a-a9f990c828c5
Content: Select a template to configure forecasting
Content Source: ce/sales/select-template-forecast.md
Service: dynamics-365-sales
GitHub Login: @udaykirang
Microsoft Alias: udag

@gregowens, thanks for your inputs. I've fixed the bug and will publish the doc.
Thanks,
Uday


Can't run bot

So i have done all the steps before and i have changed the directory to the new file created, no problems there, but when i want to run the python program this error pops up and i don't know what to do. So if anyone does know what to do then please tell me, or if there is already a thread addressing this problem then please link the thread.

Same problem bro

@tanujyadav97 used it even but didn't worked ?? How did get that working ?

whats the error now?

@tanujyadav97 Ok i used the command you suggested and it didnt give me the same error but gave me a new one

Any clue whats to do with this one?

replace line 64 with "sample_questions = json.loads(open("Data/questions.json",encoding="utf8"))

@tanujyadav97
It shows an error regarding
(Traceback ):
File "answer_bot.py" , line17, in
Import urllib.request as urllib2
Importerror : No module named request
This is the error hope you can figure it out !!

replace "import urllib.request as urllib2" with "import urllib2"

I ve replaced urllib.request as urllib2.request but it doesn't work and even in the error it shows the same but with urllib2 now
Error :
File "answer_bot.py" , line17, in
Import urllib2.request as urllib2
Importerror : No module named request

@nanip9997 please read my comment properly

Ok thanks @tanujyadav97 it worked

hey i ve got  a new error please help with this

I changed line 64 with the one you said, but i got a new error message

@nanip9997 use raw_input() instead of input() in python 2.7

@tanujyadav97 should I change it the answer_bot.py code ??

@nanip9997 yes

@tanujyadav97  sorry for asking many questions I APPRECIATE YOUR PATIENCE.
I've been getting 2 more errors one while running sample questions and the other is while i pressed 's'

this contains the error of sampq

this contains the second error please help for one lasttime

@nanip9997 the error is self explanatory

@tanujyadav97 I've installed tesseract-ocr and also added the tessdata to environment variables but it shows the error ..
Should we get going the live game so that the tesseract recognises or it is not installed correctly

@tanujyadav97  dude can you help with my issue, i know you are dealing with two issues but i think its only appropriate to help the person who opened this thread. Thanks for the help anyway.

I also have the same problem as @Kablamoo123 . Please help :)

@Kablamoo123 @tkirky either use "json.load()" with the same arguments or use "json.loads(open(same arguments).read())"

@tanujyadav97 I used json.load() and that worked and i don't get that error anymore, but i now have this when i type in either "s" or "sampq":

I'm getting these errors when run with 's'. How do I resolve this?

Hey, this has been really helpful so far, but I'm getting the same error as above. Any idea how to fix this? Thanks a lot!

Either it is the problem with the installation of your tesseract or the
screen grab position you ve placed the mirrored screen
On Sat, 12 May 2018, 7:28 pm rsewell97<EMAIL_ADDRESS>wrote:

Hey, this has been really helpful so far, but I'm getting the same error
as above. Any idea how to fix this? Thanks a lot!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sushant10/HQ_Bot/issues/62#issuecomment-388557127,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ab-xlXZVk4yTqMyAeOqMJJdg-wkAenRhks5txup-gaJpZM4TC1EH
.

Hey, so I've uninstalled and reinstalled both tesseract and opencv and I've checked the screenshot  coordinates using an online application. I'm still getting this error, any other ideas??
Thanks a lot!


OpticalSystem arbitrary plane field

It would be nice to query an OpticalSystem at an arbitrary plane and get the field there.

This is technically already possible, but we should probably make some examples and filters to make this easier: https://flax.readthedocs.io/en/latest/guides/extracting_intermediates.html#use-capture-intermediates

I think this is a really cool idea, and I don’t think it’s solved by simply capturing intermediates. Currently, we do propagation with special propagators for 2f lenses, etc. Such propagators only let us capture the field at those particular special planes a focal distance away from the lens.
But what if you wanted to capture the field at a plane that’s not at one of those locations? Right now, that requires that we construct a new optical system which potentially included a Fresnel propagation to the desired plane.
Can we separate the definition of an optical system, which should really just be a description of the hardware components of the system, from queries about the light field at any arbitrary plane along the propagation axis?
Message ID: @.***>


chore(primitives): revise signature interface and transaction encode

Closes: #922
Motivation
revise signature interface
After working on #914 , I'd like to revise the interface of Signature because:

encode, decode are not symmetric. Even though encoding with encode_inner_legacy() or encode_eip155_inner(), we'll decode w/ only decode_eip155_inner(). The transaction w/ eip155 is still a legacy transaction. So I'd like to unify the interface as encode_legacy() and decode_legacy().
NIT but the interface name are not consistent. i.e, encode_inner_legacy() and encode_eip155_inner(). encode_eip155_inner() should be encode_inner_eip155() for consistency even though I'd like to remove it.
For non legacy transaction, the signature encoding is processed in Transaction.  For consistency, Signature should also have encode() and decode() for non legacy transaction.

revise transaction encode
During revising signature interface, I've found that transaction encode has also consistency and symmetric issue in terms of len() and encode(). I'd like to clean it up.

Codecov Report

Merging #928 (e405556) into main (82b10fa) will decrease coverage by 0.12%.
The diff coverage is 94.66%.

@@            Coverage Diff             @@
##             main     #928      +/-   ##
==========================================
- Coverage   74.05%   73.94%   -0.12%
==========================================
  Files         292      292
  Lines       31722    31728       +6
==========================================
- Hits        23493    23460      -33
- Misses       8229     8268      +39

Flag
Coverage Δ

unit-tests
73.94% <94.66%> (-0.12%)
:arrow_down:

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

Impacted Files
Coverage Δ

crates/primitives/src/transaction/mod.rs
89.74% <92.00%> (+1.11%)
:arrow_up:

crates/primitives/src/transaction/signature.rs
100.00% <100.00%> (ø)

crates/net/eth-wire/src/types/status.rs
63.21% <0.00%> (-11.18%)
:arrow_down:

crates/primitives/src/header.rs
88.31% <0.00%> (-6.17%)
:arrow_down:

crates/stages/src/stages/bodies.rs
91.36% <0.00%> (-1.73%)
:arrow_down:

crates/net/network/src/manager.rs
49.79% <0.00%> (-0.21%)
:arrow_down:

crates/executor/src/config.rs
98.59% <0.00%> (-0.02%)
:arrow_down:

crates/primitives/src/chain_spec.rs
73.79% <0.00%> (-0.02%)
:arrow_down:

crates/net/network/src/peers/manager.rs
83.42% <0.00%> (+0.18%)
:arrow_up:

... and 4 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

I thought about this a bit, and legacy is actually useful qualified name for the functions since they're only used for legacy transactions.
So +1 on the renames, but I'd love to see a brief not in the functions docs that states this.

one naming change. let's avoid legacy as much as we can.

IMHO, generally Legacy term is too broad so I definitely agree on removing as much as we can if Legacy is used as general meaning. Otherwise, to my understanding, Legacy transaction could be specific term because many ethereum implementation and library use Legacy transaction term to indicate before EIP2930.

Legacy transaction might be a specific term because many ethereum implementation and library use Legacy transaction term to indicate before EIP2930.

this is the same for reth transaction types.
I guess the confusion here is, that a legacy tx needs to be handled differently if it supports EIP155 or not, which should be included in the naming.


error parseWorksheet

I'm getting this error all the time
typeMismatch(Swift.String, Swift.DecodingError.Context(codingPath: [CodingKeys(stringValue: "sheetData", intValue: nil), CodingKeys(stringValue: "row", intValue: nil), XMLKey(stringValue: "365", intValue: 365), CodingKeys(stringValue: "c", intValue: nil), XMLKey(stringValue: "2", intValue: 2), CodingKeys(stringValue: "f", intValue: nil)], debugDescription: "Expected to decode String but found SharedBox instead.", underlyingError: nil))

Hi @stonkeep, could you please attach the file for reproduction or try the master branch for this file? My understanding is that this issue was fixed in master, but I hope we could double check with your file. Thanks!

This looks very similar to #59 so unless more data is provided, I'd like to assume this is the same issue, which was fixed in #61. The fix is now available in master and will be included in the next 0.6 release.


[REG2.064a] Issue 11223 - inline ice with tuple assignment and if/else

http://d.puremagic.com/issues/show_bug.cgi?id=11223
Superseded fix of 36a44cbcea9fbb18b221f77e00f2dca77f21bc88

This pull request introduced a regression:
https://issues.dlang.org/show_bug.cgi?id=15210


skrooge: 2.12.0 -> 2.13.0

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/skrooge/versions.
These checks were done:

built on NixOS
ran ‘/nix/store/l83qij463yvk1nqxahdcm9amfnp7zdm8-skrooge-2.13.0/bin/skroogeconvert -h’ got 0 exit code
ran ‘/nix/store/l83qij463yvk1nqxahdcm9amfnp7zdm8-skrooge-2.13.0/bin/skroogeconvert --help’ got 0 exit code
found 2.13.0 with grep in /nix/store/l83qij463yvk1nqxahdcm9amfnp7zdm8-skrooge-2.13.0
directory tree listing: https://gist.github.com/699ec87fffb73399c86ad11632ed9b17

cc @jokogr for testing

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: skrooge
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: skrooge
Partial log (click to expand)
shrinking /nix/store/l83qij463yvk1nqxahdcm9amfnp7zdm8-skrooge-2.13.0/lib/qt-5.10/plugins/skg_file.so
shrinking /nix/store/l83qij463yvk1nqxahdcm9amfnp7zdm8-skrooge-2.13.0/lib/qt-5.10/plugins/skrooge_tracker.so
shrinking /nix/store/l83qij463yvk1nqxahdcm9amfnp7zdm8-skrooge-2.13.0/lib/qt-5.10/plugins/skrooge_import_gnc.so
shrinking /nix/store/l83qij463yvk1nqxahdcm9amfnp7zdm8-skrooge-2.13.0/lib/qt-5.10/plugins/skrooge_operation.so
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/l83qij463yvk1nqxahdcm9amfnp7zdm8-skrooge-2.13.0/lib  /nix/store/l83qij463yvk1nqxahdcm9amfnp7zdm8-skrooge-2.13.0/bin
patching script interpreter paths in /nix/store/l83qij463yvk1nqxahdcm9amfnp7zdm8-skrooge-2.13.0
checking for references to /build in /nix/store/l83qij463yvk1nqxahdcm9amfnp7zdm8-skrooge-2.13.0...
postPatchMkspecs
postPatchMkspecs

Failure on aarch64-linux (full log)
Attempted: skrooge
Partial log (click to expand)
[4738/17529] CXX obj/third_party/icu/bundled_icuuc/uhash_us.o
[4739/17529] CXX obj/third_party/icu/bundled_icuuc/ucurr.o
[4740/17529] CXX obj/third_party/icu/bundled_icuuc/uiter.o
[4741/17529] CXX obj/third_party/icu/bundled_icuuc/udata.o
[4742/17529] CXX obj/third_party/icu/bundled_icuuc/ustr_cnv.o
[4743/17529] AR obj/third_party/angle/libtranslator.a
[4744/17529] CXX obj/third_party/icu/bundled_icuuc/ulistformatter.o
building of '/nix/store/j8zfhsjxn7vkm8lzps9202wa0skjnygx-qtwebengine-5.10.1.drv' timed out after 3600 seconds
cannot build derivation '/nix/store/47213805xyssc1l94mfrpbjiqn2d3z1i-skrooge-2.13.0.drv': 1 dependencies couldn't be built
error: build of '/nix/store/47213805xyssc1l94mfrpbjiqn2d3z1i-skrooge-2.13.0.drv' failed


Loading modules with css files fails when using together with Awesomium.Net

Using ocLazyLoad together with Awesomium (<IP_ADDRESS>) (http://www.awesomium.com) does not work if there is a css file in the files array. The promise gets resolved but the ng module does not work. I have posted a minmal example on stackoverflow: http://stackoverflow.com/questions/28541731/lazyloading-angularjs-module-with-css-file-fails-with-awesomium-web-view
To reproduce you need to install the Awesomium sample that are shipped with the SDK (for Windows), see: http://www.awesomium.com/download.
As it does work on all browser expect Awesomium I suppose it's a bug on their side, but I am not sure of course. OcLazyLoad does also not throw any error. Is there anything interesting that can be tested to narrow down the cause of this issue?

Hmm interesting, I'll check it out when I can.
Do you know if the script files are inserted into the DOM ? (they should be if the promise is resolved).
If you only try to lazyload the css file then what happens ?

All files (script and css) are inserted into the DOM.
If I only lazyload the css file then the styles get applied.

Interesting, then I need to investigate because I have no idea what the problem is right now :)

Btw. apparently Awesomium build on top of the same core as Chrome 18. I determined the version using the method described here: http://stackoverflow.com/questions/5916900/how-can-you-detect-the-version-of-a-browser
I have tried to find this version but it seems very hard to find downloads for old versions. Unfortunately I did not succeed. As I use Awesomium actively I would be happy if this could be fixed. On the other hand the error could be due to a bug in this chromium version.
Do you already test the lib with browserstack?

I will try using another async loader.


Optimize UI responsiveness in scanning state

The UI is very laggy while the scanning is in progress, and it makes the program almost unusable during the first few seconds of startup. The bottleneck is jwalk iterator, it may block the UI thread sometimes. In this PR, I alleviated the situation by some optimizations, and the UI is now super responsive at startup.
Here is what it looks like:
https://github.com/Byron/dua-cli/assets/3109144/3b06b888-c4ec-4c45-9468-36b0059cbe6c
Some implementation notes:

The traversal operation is moved to another thread to prevent the UI thread from being blocked, and its results are sent to the UI thread via a channel.
Use crossbeam::channel to receive UI events and jwalk entries. In contract to std::sync::mpsc, we can select from a set of channels without busy polling using try_recv.
Created our own version of input_channel. The function provided by crosstermion returns a std::sync::mpsc::Receiver value, which is not compatible with crossbeam::select!.

I have tested this change and indeed it is now super smooth 🎉  Thanks!

Thanks a million, it works just as advertised!
It's so good to have this issue finally fixed!
And because we had to wait so long for this to be improved, I didn't want anybody to wait even longer: here is the new release


Added ability to configure upload and download executables from the command line

Closes #45
@aleksakrstic

Hi @bryanOrtiz,
Thanks for the contribution!
These changes will be available in the next release.


Bug: Character Descriptions cut off weirdly

Version
5..3
Issue Checklist

[X] Have you checked the Issues/Discussions to see if this hasn't been reported yet?
[X] Have you tried reinstalling the mod?
[X] Have you tried disabling every mod except Funker Selector?
[X] Have you tried reproducing this issue on a fresh install of FNF?

Context (Provide images, videos, etc.)
got this weird issue wjere the description doesn't load in completely.

Hanging w my family todaY probably wont be at the computer I'm having the issue with for a lil bit. My bad lol

I've encountered this issue before releasing 2.0.0, but haven't found a proper fix for it yet. I'll look into it when the Song Pack Pitstop is out.


CSS support

Hi!
Greet work!
How about CSS?

Hi @michail-vestnik , thanks!!
GCloc already supports CSS, you can install the latest version of it by clicking here or you can update your docker image to use the latest version: docker pull joaodanielrufino/gcloc:latest

Thanks!!!


MINOR: Add version check on enable-systest-events flag

Recent patch adding enable-systest-events flag without any version check breaks all uses of versioned console consumer. E.g. upgrade tests, compatibility tests etc.
Added a check to only apply the flag if running 0.10.0 or greater.

@apovzner @ewencp
The issue was introduced in https://github.com/apache/kafka/pull/1278 (which I also reviewed...)
Unfortunately I didn't think of this in the recent code review, but I noticed this in a subsequent branch build. We'll see a large chunk of test failures without this present patch.

@granders Good catch. Question about when exactly we do version bumping (and maybe about how exactly the comparison operator works?).

I assume this patch needs double commit as it is relevant to the 0.10 branch, but I want to have people explicitly request double commits. This is the case, correct?
Beyond the impact of this specific patch, what would have helped catch this? PR/branch builder? or does this require running against multiple branches to protect against? I want to figure out what the right process is so we catch these issues earlier.

@ewencp Thanks for review! here are a few answers:
version bumping... I don't think it really matters in this case given how comparison works? (see my previous version-related comments)

Yes, we want double commit here
PR/branch builder would've caught this no matter irrespective of the branch. Even a run of just the sanity checks would have caught this particular issue.

Running sanity checks here:
https://jenkins.confluent.io/job/system-test-kafka-branch-builder/425/console
Full build here:
https://jenkins.confluent.io/job/system-test-kafka-branch-builder-2/54/console

@ewencp Sanity checks now pass

@granders thanks for fixing this.

Full build had one failure, but its an existing transient issue. LGTM, thanks @granders!


Add clubstep x angry birds by uwu

Song Name
clubstep x angry birds
Artist Name
uwu
Source
Youtube
Youtube Link
TO5gv5iC-n2SuM_G
Direct File Link
https://youtu.be/a8SyA0fTp-M?si=TO5gv5iC-n2SuM_G
Song ID
13
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 1347537

reject
invalid YouTube link


ctrl-m for newline+indent

TIL that electric-indent-mode is now the emacs default!
So I agree this should be the default behavior, preferably with an option to emulate switching the mode off.


Change to noarch: python package

Fixes #1, #18

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin please update cirlce

@conda-forge-admin please update cirlceci


Bug: Unit names are not random enough

Use UUIDs for unit names so we don't replace live units when spawning new ones.

slightly resolved with #35 , by checking if a name is already in use before spawning a unit.
I Think we would still prefer to switch to uuid's, so we can keep this one open.
I'm going to rename this to be a feature though


typo in CMakeLists.txt LZ4_INCLUDEDIR

hello,
In (master) CMakeLists you use LZ4_INCLUDE_DIR but pkg_config defines LZ4_INCLUDEDIR ...
this breaks the case in which you don't want to use the supplied lz4.
in another note: i'm trying to crosscompile, and pkg_config also gives a library directory which is not being taken into account, but that's not so simple to fix as target_link_directories only exists in very recent versions of CMake. i work around by symlinking liblz4.a.
Frank

Hi Frank,
Thanks a lot for reporting this issue. It's really nice to know that someone is actually using this.
The typo is easy enough to fix, but as you mention it's probably not that easy to fix having the actual lz4 library in a non standard location.
I will try to add another target to the automated builds (eg. travis) that installs lz4 to another location (eg. /usr/local) once I've come up with a solution.
Thanks for the hint on target_link_directories. You are right that it only seems to be supported on very recent versions of CMake and although I haven't really made up my mind on which versions of compilers and tools to support, I would like to avoid such as requirement.
I'll try to find a solution ASAP. Please let me know if you figure something out (or possibly open a PR).
Thanks.
/Kasper

hello,
none of my machines runs a recent enough cmake for target_link_directories. so i guess that rules it out for me now. For the time being we switched back to your old CMakeLists which didn't use pkg_config to find lz4. i have the impression that using pkg_config and crosscompiling now is not really supported well.
i needed the non-bundled approach because i'm crosscompiling, otherwise i would go for the bundled approach too.
Frank

And adding... a solution that would be working for us would be bundling a findLZ4.cmake
Something like this (untested, if it looks workable i could test):
find_library(LZ4_LIBRARY
    PATHS ${LZ4_ROOT}
    NAMES liblz4.a
)

find_path(LZ4_INCLUDE_DIR
  lz4frame.h
  PATHS ${LZ4_ROOT}
  NO_DEFAULT_PATH)
include(FindPackageHandleStandardArgs)
find_package_handle_standard_args(lz4 DEFAULT_MSG
  LZ4_INCLUDE_DIR LZ4_LIBRARY)

if(lz4_FOUND)
    add_library(lz4_stream::lz4_stream INTERFACE IMPORTED)
    set_target_properties(lz4_stream::lz4_stream PROPERTIES
        INTERFACE_INCLUDE_DIRECTORIES "${LZ4_INCLUDE_DIR}"
    )
    set_target_properties(lz4_stream::lz4_stream PROPERTIES
        INTERFACE_LINK_LIBRARIES "${LZ4_LIBRARY}"
    )
endif()

Hi Frank,
I'm not really sure why I chose to use pkg_config in the first place, since I used to bundle a FindLZ4.cmake, so it might be a good idea to reintroduce that.
If you could wrap this into a pull request I will definitely have a look at it, so please consider that.
Thanks and kind regards,
/Kasper


imply not null in if

Description
First attempt, tests haven't been updated

for some reason, the optimiser doesn't work when these extra constraints are added.
this halves the speed of the generator, so is something that should be worked out


How to add multiple styles using the IDictionary<string, object> in Blazor attributes ?

Below is the code blow which throws an exception as this key value already exists.
"
"
@code {
private IDictionary<string, object> _Attributes = new Dictionary<string, object>();
// This adds to the element
_Attributes.Add("style", "width:" + this.width);
// This trows an exception that the key value already exists
_Attributes.Add("style", "height:" + this.height);
}
Can anyone suggest a solution for this  to achieve in any other way.?

Thanks for contacting us, @Indrajith-Sync.
Attribute Splatting (the @attributes directive) is meant to be used for applying different attributes to an  element. In this case you're trying to apply multiple sub-values to the same style attribute.
You have two options here, either concatenate both width and height pieces into a single string when applying to style (using the attribute splatting technique), or directly set the style attribute on the div.

Here is a sample:
@page "/"

Hello, world!

Welcome to your new app.

option 1

option 2

@code{
    public IDictionary<string, object> attributes = new Dictionary<string, object>();

    public string Style { get; set; }

    public int Width { get; set; } = 300;
    public int Height { get; set; } = 15;

    protected override void OnInitialized()
    {
        this.Style = $"width: {Width}px; height: {Height}px; border: 1px solid red;";

        this.attributes.Add("style", this.Style);
    }
}

Hi @mkArtakMSFT
Thanks on your suggestion.
From your code block for example:-
protected override void OnInitialized()
{
this.Style = $"width: {Width}px; height: {Height}px; border: 1px solid red;";
    this.attributes.Add("style", this.Style);
    // using this to add another style based on some condition
    if (Height) {
      this.height= $"height: {Height}px; ";
      this.attibutes.Add("style", this.height);
   }
}

But in my case i am using two style attributes in a condition based, in that case i got an exception as
An item with the same key has already been added. Key: style'
can i check weather the atttibutes is present or not ?

Hi @mkArtakMSFT
Thanks it works fine..!


performance

any performance of this code on SQuAD?

much lower than the paper's. I recommend this implementation:  https://github.com/HKUST-KnowComp/R-Net

Thanks a lot.


Awesome-Git :: features highlight

Git 2.36.1 --> 2.37.0 Highlight

A few new features for my preference

core.sparseCheckout
mergetool.vimdiff
--since-as-filter=
git-archive :: --add-file=

git-config.html
: --show-scope
Similar to --show-origin in that it augments the output of all queried config options
with the scope of that value (worktree, local, global, system, command).
: core.fsyncMethod
A value indicating the strategy
Git will use to harden repository data
using fsync and related primitives.
  fsync uses the \
      fsync() system call or platform equivalents.

  writeout-only issues pagecache writeback requests,
      but depending on the filesystem and storage hardware,
      data added to the repository may not be durable in the event of a system crash.
      This is the default mode on macOS.

  batch enables a mode that uses writeout-only flushes \
      to stage multiple updates in the disk writeback cache and \
      then does a single full fsync of a dummy file to trigger the disk cache flush at the end of the operation.

  Currently batch mode only applies to loose-object files.
  Other repository data is made durable as if fsync was specified.
  This mode is expected to be as safe as fsync on macOS for repos \
      stored on HFS+ or APFS filesystems and on Windows for repos stored on NTFS or ReFS filesystems.

: core.sparseCheckoutCone
Enables the "cone mode" of the sparse checkout feature.
When the sparse-checkout file contains a limited set of patterns,
this mode provides significant performance advantages.
The "non cone mode" can be requested to allow specifying a more flexible patterns by setting this variable to false.
See git-sparse-checkout[1] for more information.
: add.interactive.useBuiltin
Set to false to fall back to the original Perl implementation of the interactive version of git-add[1]
instead of the built-in version. Is true by default.
: branch.autoSetupMerge
Tells git branch, git switch and git checkout to set up new branches so that git-pull[1] will appropriately merge from the starting point branch.
Note that even if this option is not set, this behavior can be chosen per-branch using the --track and --no-track options.
The valid settings are: false — no automatic setup is done; true — automatic setup is done when the starting point is a remote-tracking branch; always — automatic setup is done when the starting point is either a local branch or remote-tracking branch; inherit — if the starting point has a tracking configuration, it is copied to the new branch; simple — automatic setup is done only when the starting point is a remote-tracking branch and the new branch has the same name as the remote branch.
This option defaults to true.
: branch..remote
When on branch , it tells git fetch and git push which remote to fetch from/push to. The remote to push to may be overridden with remote.pushDefault (for all branches). The remote to push to, for the current branch, may be further overridden by branch..pushRemote. If no remote is configured, or if you are not on any branch and there is more than one remote defined in the repository, it defaults to origin for fetching and remote.pushDefault for pushing. Additionally, . (a period) is the current local repository (a dot-repository), see branch..merge's final note below.
: gc.cruftPacks
Store unreachable objects in a cruft pack (see git-repack[1]) instead of as loose objects. The default is false.
将无法到达的对象存储在一个cruft包中（见git-repack[1]），而不是作为松散对象。默认为false。
: gc.pruneExpire
When git gc is run, it will call prune --expire 2.weeks.ago (and repack --cruft --cruft-expiration 2.weeks.ago if using cruft packs via gc.cruftPacks or --cruft). Override the grace period with this config variable. The value "now" may be used to disable this grace period and always prune unreachable objects immediately, or "never" may be used to suppress pruning. This feature helps prevent corruption when git gc runs concurrently with another process writing to the repository; see the "NOTES" section of git-gc[1].
: gpg.ssh.defaultKeyCommand
这允许脚本在静态配置 user.signingKey 不可行的情况下动态查找正确的公钥
This command that will be run when user.signingkey is not set and a ssh signature is requested. On successful exit a valid ssh public key prefixed with key:: is expected in the first line of its output. This allows for a script doing a dynamic lookup of the correct public key when it is impractical to statically configure user.signingKey. For example when keys or SSH Certificates are rotated frequently or selection of the right key depends on external factors unknown to git.
: http.curloptResolve
Hostname resolution information that will be used first by libcurl when sending HTTP requests.
: mergetool.vimdiff.layout
The vimdiff backend uses this variable to control how its split windows look like.
Applies even if you are using Neovim (nvim) or gVim (gvim) as the merge tool.
See BACKEND SPECIFIC HINTS section in git-mergetool[1]. for details.
: push.autoSetupRemote
If set to "true" assume --set-upstream on default push when no upstream tracking exists for the current branch; this option takes effect with push.default options simple, upstream, and current. It is useful if by default you want new branches to be pushed to the default remote (like the behavior of push.default=current) and you also want the upstream tracking to be set.
  Workflows most likely to benefit from this option are simple central workflows \
      where all branches are expected to have the same name on the remote.

: repack.cruftWindow
: repack.cruftWindowMemory
: repack.cruftDepth
: repack.cruftThreads
Parameters used by git-pack-objects[1] when generating a cruft pack and the respective parameters are not given over the command line. See similarly named pack.* configuration variables for defaults and meaning.
: revert.reference
Setting this variable to true makes git revert behave as if the --reference option is given.
: safe.directory
This config setting is only respected when specified in a system or global config, not when it is specified in a repository config, via the command line option -c safe.directory=, or in environment variables.
  ...

  As explained, Git only allows you to access repositories owned by yourself, i.e. the user who is running Git, by default. When Git is running as root in a non Windows platform that provides sudo, however, git checks the SUDO_UID environment variable that sudo creates and will allow access to the uid recorded as its value in addition to the id from root. This is to make it easy to perform a common sequence during installation "make && sudo make install". A git process running under sudo runs as root but the sudo command exports the environment variable to record which id the original user has.

  If that is not what you would prefer and want git to only trust repositories that are owned by root instead, then you can remove the SUDO_UID variable from root’s environment before invoking git.

: transfer.credentialsInUrl
  A configured URL can contain plaintext credentials in the form <protocol>://<user>:<password>@<domain>/<path>. You may want to warn or forbid the use of such configuration (in favor of using git-credential[1]). This will be used on git-clone[1], git-fetch[1], git-push[1], and any other direct use of the configured URL.

  Note that this is currently limited to detecting credentials in remote.<name>.url configuration, it won’t detect credentials in remote.<name>.pushurl configuration.

  You might want to enable this to prevent inadvertent credentials exposure, e.g. because:

      The OS or system where you’re running git may not provide way way or otherwise allow you to configure the permissions of the configuration file where the username and/or password are stored.

      Even if it does, having such data stored "at rest" might expose you in other ways, e.g. a backup process might copy the data to another system.

      The git programs will pass the full URL to one another as arguments on the command-line, meaning the credentials will be exposed to other users on OS’s or systems that allow other users to see the full process list of other users. On linux the "hidepid" setting documented in procfs(5) allows for configuring this behavior.

      If such concerns don’t apply to you then you probably don’t need to be concerned about credentials exposure due to storing that sensitive data in git’s configuration files. If you do want to use this, set transfer.credentialsInUrl to one of these values:

  allow (default): Git will proceed with its activity without warning.
  warn: Git will write a warning message to stderr when parsing a URL with a plaintext credential.
  die: Git will write a failure message to stderr when parsing a URL with a plaintext credential.

git-remote.html
: --verbose
Be a little more verbose and show remote url after name.
For promisor remotes, also show which filter (blob:none etc.) are configured.
NOTE: This must be placed between remote and subcommand

git-rev-list.html
: --since-as-filter=
Show all commits more recent than a specific date.
This visits all commits in the range, rather than stopping at the first commit which is older than a specific date.
这将访问该范围内的所有提交，而不是停在第一个比特定日期更早的提交。

git-revert.html
: --reference
Instead of starting the body of the log message with
"This reverts .",
refer to the commit using "--pretty=reference" format (cf. git-log[1]).
The revert.reference configuration variable can be used to enable this option by default.

git-shortlog.html
: --since-as-filter=
Show all commits more recent than a specific date.
This visits all commits in the range, rather than stopping at the first commit which is older than a specific date.

git-archive.html
: --prefix=/
Prepend / to paths in the archive.
Can be repeated; its rightmost value is used for all tracked files.
See below which value gets used by --add-file and --add-virtual-file.
: --add-file=
Add a non-tracked file to the archive.
Can be repeated to add multiple files.
The path of the file in the archive is built by
concatenating the value of the last --prefix option (if any)
before this --add-file and the basename of .
: --add-virtual-file=:
Add the specified contents to the archive.
Can be repeated to add multiple files.
The path of the file in the archive is built by
concatenating the value of the last --prefix option
(if any) before this --add-virtual-file and .
  The <path> argument can start and end with a literal double-quote character;
      the contained file name is interpreted as a C-style string, i.e. the backslash is interpreted as escape character.
  The path must be quoted if it contains a colon,
      to avoid the colon from being misinterpreted as the separator between the path and the contents,
      or if the path begins or ends with a double-quote character.

  The file mode is limited to a regular file, and the option may be subject to platform-dependent command-line limits.
  For non-trivial cases, write an untracked file and use --add-file instead.

: git archive -o latest.tar --prefix=build/ --add-file=configure --prefix= HEAD
  Creates a tar archive that contains the contents of the latest commit on the current branch
  with no prefix and the untracked file configure with the prefix build/.

git-branch.html
: --track[=(direct|inherit)]
The branch.autoSetupMerge configuration variable
specifies how git switch, git checkout and git branch
should behave when neither --track nor --no-track are specified:
The default option, true, behaves as though
--track=direct were given whenever the start-point is a remote-tracking branch.
false behaves as if --no-track were given. always behaves as though --track=direct were given.
inherit behaves as though --track=inherit were given. simple behaves as though --track=direct were given only
when the start-point is a remote-tracking branch and the new branch has the same name as the remote branch.

git-gc.html
: --cruft
When expiring unreachable objects, pack them separately into a cruft pack instead of storing them as loose objects.
: gc.cruftPacks
Store unreachable objects in a cruft pack (see git-repack[1]) instead of as loose objects. The default is false.
: gc.pruneExpire
When git gc is run, it will call prune --expire 2.weeks.ago (and repack --cruft --cruft-expiration 2.weeks.ago if using cruft packs via gc.cruftPacks or --cruft). Override the grace period with this config variable. The value "now" may be used to disable this grace period and always prune unreachable objects immediately, or "never" may be used to suppress pruning. This feature helps prevent corruption when git gc runs concurrently with another process writing to the repository; see the "NOTES" section of git-gc[1].

git-log.html
: --since-as-filter=
Show all commits more recent than a specific date. This visits all commits in the range, rather than stopping at the first commit which is older than a specific date

git-mergetool.html
: mergetool.vimdiff.layout
The vimdiff backend uses this variable to control how its split windows look like. Applies even if you are using Neovim (nvim) or gVim (gvim) as the merge tool. See BACKEND SPECIFIC HINTS section for details.
: vimdiff
- https://git-scm.com/docs/git-mergetool

git-read-tree.html
: SPARSE CHECKOUT
Note: The skip-worktree capabilities in git-update-index[1] and read-tree predated the introduction of git-sparse-checkout[1].
Users are encouraged to use the sparse-checkout command in preference to these plumbing commands for sparse-checkout/skip-worktree related needs.
However, the information below might be useful to users trying to understand the pattern style used in non-cone mode of the sparse-checkout command.
注意: git-update-index[1] 和 read-tree 中的 skip-worktree 功能早于 git-sparse-checkout[1] 的引入。我们鼓励用户使用 sparse-checkout 命令，而不是这些与 sparse-checkout/skip-worktree 相关的管道命令。然而，下面的信息可能对试图理解 sparse-checkout 命令的非核心模式下使用的模式风格的用户有用。

Git 2.37.1 --> 2.37.2

git.html
_git_commands.html
_guides.html
    GIT_ALLOW_PROTOCOL
        If set to a colon-separated list of protocols, behave as if protocol.allow is set to never,
        and each of the listed protocols has protocol.<name>.allow set to always
        (overriding any existing configuration).
        See the description of protocol.allow in git-config[1] for more details.

git-config.html
    core.sparseCheckoutCone
        Enables the "cone mode" of the sparse checkout feature.
        When the sparse-checkout file contains a limited set of patterns,
        this mode provides significant performance advantages.
        The "non-cone mode" can be requested to allow specifying more \
        flexible patterns by setting this variable to false.
        See git-sparse-checkout[1] for more information.

git-daemon.html
    It verifies that the directory has the magic file "git-daemon-export-ok",
    and it will refuse to export any Git directory that hasn’t explicitly been
    marked for export this way (unless the --export-all parameter is specified).
    If you pass some directory paths as git daemon arguments,
    the offers are limited to repositories within those directories.

    <directory>
        The remaining arguments provide a list of directories.
        If any directories are specified, then the git-daemon process
        will serve a requested directory only if it is contained in one
        of these directories. If --strict-paths is specified, then the
        requested directory must match one of these directories exactly.

git-diff-index.html
git-diff.html
    git-diff-files [<pattern>…]
        - sha1 for "dst"; 0{40} if deletion, unmerged or "work tree out of sync with the index".

git-rebase.html
git-rev-parse.html
     -->
gitrevisions.html

    [<branchname>]@{upstream}, e.g. master@{upstream}, @{u}
    A branch B may be set up to build on top of a branch X
    (configured with branch.<name>.merge) at a remote
    R (configured with branch.<name>.remote).
    B@{u} refers to the remote-tracking branch for the branch X
    taken from remote R, typically found at refs/remotes/R/X.

    [<branchname>]@{push}, e.g. master@{push}, @{push}
    The suffix @{push} reports the branch "where we would push to"
    if git push were run while branchname was checked out
    (or the current HEAD if no branchname is specified).
    Like for @{upstream}, we report the remote-tracking branch
    that corresponds to that branch at the remote.


Move code from .razor to .razor.cs

Pull Request
We're moving all code from the .razor filesto .razor.cs files to make it easier to add comments etc.
As part of the move we are also removing unused usings and going to file scoped namespaces
📖 Description

🎫 Issues

#134
👩💻 Reviewer Notes

📑 Test Plan

✅ Checklist
General

[ ] I have added tests for my changes.
[ ] I have tested my changes.
[ ] I have updated the project documentation to reflect my changes.
[ ] I have read the CONTRIBUTING documentation and followed the standards for this project.

Component-specific

[ ] I have added a new component
[ ] I have modified an existing component

⏭ Next Steps

@vnbaaij can you please add editor.config as well to the repository, so that tabs and space and other things will be consistent across the code.
Some more info => https://docs.microsoft.com/en-us/dotnet/fundamentals/code-analysis/code-style-rule-options