Enable Password Hiding modifier with USER_INFO Credentials

There seems to be some facility by which the Schema Registry client's logger knows how to mask out the value assigned to basic.auth.user.info when logging the connector's configuration at bootstrap time.  Currently, the registry helper's src.basic.auth.user.info and dest.basic.auth.user.info properties both appear in the log entry created by Kafka Connect when first loading the connector.
It is not yet clear whether the client makes any attempt to mask credentials in the URL when the basic.auth.credentials.source property is set to URL instead of USER_INFO.  Recommend observing the existing behavior and mimicking it accordingly for src.basic.auth.user.info and dest.basic.auth.user.info.
I suspect this behavior is driven by field modifiers set through Connect's ConfigDef structure, but that is just a guess for the moment.

Fixed by Pull Request #22


chore: update dependencies

Fixed: https://github.com/puppeteer/puppeteer/issues/9372

Note that the doc check is failing


Communication

Communication

[ ] Must have a meeting at least 4 days a week at 6:00 pm
[ ] Must have either Discord, Zoom or Google meet

Finish


Increase ping pong timeout delay

As spring luamenu is unresponsive while pathing, users drop from the lobby when joining games often

Heartbeat requirement is 45 seconds with a check interval of 30 seconds. How long does it need to be?

Maybe double that, can there be negative side effects of doing so?

Nothing we'd notice at this stage. I'll increase the change in the code tonight and release tomorrow with the username fix.


CLOUD-000 fix broken annotations

@schfeca75

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1457/


Keepalived based HA master

Support automatic keepalived installation/configuration

Extremely useful feature and great idea to support it out of the box, huge +1 :)
May I ask why did you choose Cisco's VRRP over the open-source CARP protocol? Do you just want keepalived and that's it? Or is there a specific use case why one is better than another in light of PKE?
For example, when I was looking for a solution of API server HA, I've found UCarp much easier to install and configure than keepalived (example repo: https://github.com/immanuelfodor/kube-karp). Since then we are using it within our in-house clusters without a problem. I'm honestly interested in why keepalived, maybe we need to switch in the future and good to think ahead.

Extremely useful feature and great idea to support it out of the box, huge +1 :)
May I ask why did you choose Cisco's VRRP over the open-source CARP protocol? Do you just want keepalived and that's it? Or is there a specific use case why one is better than another in light of PKE?
For example, when I was looking for a solution of API server HA, I've found UCarp much easier to install and configure than keepalived (example repo: https://github.com/immanuelfodor/kube-karp). Since then we are using it within our in-house clusters without a problem. I'm honestly interested in why keepalived, maybe we need to switch in the future and good to think ahead.

@immanuelfodor Hi! Thanks for your comment! We are in the same issue. I am building the kubernetes-based PaaS platform with my colleagues and we checked a lot of approached for building HA clusters. Unfortunately, ANY of the possible approaches has its own disadvantages and nuances. For example, keepalived or CARP are working only in L2 environments, AFAIK. So it is not the option for cloud providers and deploying the kubernetes on top of VMs. But on the other hand it's well known and working solution for bare metal and vmware environments.
The approach used by kubespray - deploying of haproxy with the list of all master nodes on any of worker nodes - has another issues. Etc. etc.

@immanuelfodor Hi! Thanks for your comment! We are in the same issue. I am building the kubernetes-based PaaS platform with my colleagues and we checked a lot of approached for building HA clusters. Unfortunately, ANY of the possible approaches has its own disadvantages and nuances. For example, keepalived or CARP are working only in L2 environments, AFAIK. So it is not the option for cloud providers and deploying the kubernetes on top of VMs. But on the other hand it's well known and working solution for bare metal and vmware environments.
The approach used by kubespray - deploying of haproxy with the list of all master nodes on any of worker nodes - has another issues. Etc. etc.


[Bug]: Issue Templates have multi select for contributions

Contact Details
No response
What happened?
For feature and documentation requests, there should not be multiple choice for "Are you able/willing to make the change? (It's ok if you're not!)".
Set multiple to false for both.
Version
v1.0.4
What operating system are you seeing the problem on?
Windows
What version of Node are you seeing the problem on?
16
Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

:tada: This issue has been resolved in version 1.0.5 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Create GPU enabled images for Demo 2

Demo 2 will require GPU enabled model training. For this to work, we need to ensure that the images that are built and used for the training notebook as part of our Elyra Pipeline correctly use GPU  nodes.
Acceptance Criteria:

[x]  Build images with the correct set of requirements to use GPU during training.
[x]  Ensure that GPU's are being used while running the notebooks as a KFpipeline.

We can confirm that GPUs are available in inference pipeline runs as well.


All scripts are running as tool

This is more of gdextension problem, but having wall everything in "!Engine.editorHint" is quite daunting

GDExtension has concept of initialization levels, IIRC this can be done by moving class registration into specific init level, need further investigation.


Is it chargeable ?

I have done exact steps and get it done to run but, just want to know that Will google charge me if I use it ?

This is not an official Google service but an unofficial project for developers. For more information, please refer to Google's service policies. This project is provided as-is with the code.


Improve BCP47 / language code validation

Is your feature request related to a problem? Please describe.
While checking #693 I noticed that our syntax-based validation considers almost any ascii string to be valid.
This is not what I had in mind when I opened this issue and I think it would be much more useful if we could use https://www.iana.org/assignments/language-subtag-registry/language-subtag-registry to verify that the language tags are truly valid and not just syntactically well-formed.
Describe the solution you'd like

Write a utility which converts https://www.iana.org/assignments/language-subtag-registry/language-subtag-registry into an extensional ValueSet and contribute this to our fhir-registry.
Verify that this ValueSet gets picked up by our Validator and use this in place of the syntax-based validation added as part of #693
Establish some plan for keeping this ValueSet up-to-date.

Describe alternatives you've considered
N/A
Additional context
Technically, the valueset cannot be "enumerated" because new codes can be registerred any time.  However, practically speaking, I think its sufficient just validate the codes against the values on the list.  This might take some testing and discussion to prove out.

One concern about defining a ValueSet and enumerating codes is the number of combinations.
Lang: 8178
ExtLang: 239
Script: 202
Region: 304
Grandfathered: 26
For the "extLang" subtag, each is tied 1-to-1 to a "lang" subtag.  However, the script and region subtags are independent, so they can be used in combination with any lang+extlang, as there's no indication of which "typically" are used together.
That gets us to a value set in this approximate size:  521,138,581 = ((8178 + 239) * 203 * 305) + 26

@lmsurpre Can we discuss how these findings affect the requirements/assumptions of this issue?

Sure, happy to discuss.  I did a quick zulip search and, at least in one point, the "reference" validator was validating against the IANA registry as well:
https://chat.fhir.org/#narrow/stream/179167-hapi/topic/HAPI.20Warning.20for.20unresolvable.20URIs.20and.20identifier.20type.20codes/near/173318343
Maybe just focus on the IANA language codes, validating everything up to the first (-) and ignoring the rest?

This looks like a good resource too:  https://www.w3.org/International/articles/language-tags/#registry

Yeah, I can do it that way by adding code that reads in a language tags file and keeps track of the separate pieces, doing a lookup against that. Doing so, doesn't fit the strategy of defining a extensional ValueSet, so as long as that's ok, I think that makes sense.

I ended up going with strategy that validates language subtags registered in the IANA registry, which includes language-extlang-script-region-variant. Extensions and private-use subtags are excluded, since those are not registered in the IANA registry, and therefore do not satisfy interoperability.

I deployed this and confirmed both:

when fhirServer/core/extendedCodeableConceptValidation is set to false, it allows values like

{
  "resourceType": "Patient",
  "id": "{{patientid}}",
  "language": "xxxxxxxx",
  "active": true
}

when fhirServer/core/extendedCodeableConceptValidation is set to false, it rejects resources with invalid language values like:

{
  "resourceType": "Patient",
  "id": "{{patientid}}",
  "language": "xxxxxxxx",
  "active": true
}

The error looks good too.
{
    "resourceType": "OperationOutcome",
    "issue": [
        {
            "severity": "fatal",
            "code": "invalid",
            "details": {
                "text": "FHIRProvider: Element 'language': 'xxxxxxxx-yyy' is not a valid code for value set 'http://hl7.org/fhir/ValueSet/all-languages'"
            },
            "expression": [
                "<no expression>"
            ]
        }
    ]
}

Looks good!


Update winmltools doc for 1.3

This change is to update conversion document to reflect changes in winmltools 1.3

Add tensorflow and quantization code snippet
Add relevant updates for dependent packages

OPS Build status updates of commit 41d7056:
:clock10: Preparing: average preparing time is 45 sec(s)

OPS Build status updates of commit 41d7056:
:clock10: Full build: average full build time is not available, there is no data for this repository.

Note: A full build takes more time than an incremental build.  This is the expected behavior of the build system.

OPS Build status updates of commit 41d7056:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/convert-model-winmltools.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

OPS Build status updates of commit 5039d20:
:clock10: Preparing: average preparing time is 45 sec(s)

OPS Build status updates of commit 5039d20:
:clock10: Incremental building: average incremental building time is 22 sec(s)

OPS Build status updates of commit 5039d20:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/convert-model-winmltools.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

  This is the expected behavior of the build system.

OPS Build status updates of commit 3ee1759:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/convert-model-winmltools.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

OPS Build status updates of commit 986c143:
:clock10: Pending: waiting for processors (13 builds ahead of you)
:warning: OPS Build is busy, click here to report any potential performance downgrade issues, or you could view OPS Portal for trend of build queue length.

  This is the expected behavior of the build system.

OPS Build status updates of commit 986c143:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/convert-model-winmltools.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


OBV indicator throws exception no shape attribute in outputs.py

OBV indicator throws an exception in outputs.py

Hey,..
Be sure you are working with develop branch - https://github.com/CryptoSignal/Crypto-Signal/blob/develop/README.md
If u want share me your config file, then I can check it.
Best

Closed due to no answer.


Set up automated builds on Docker Hub

Docker Hub can use automated builds which are based on the latest Dockerfile commit.
http://docs.docker.com/docker-hub/builds/

That's cool!

It's a neat feature but I don't think we'll be using it anytime since Hulk/Skynet (the builders that currently handle publishing) self bootstrap to build our debs when they build the image.  That's not possible with Docker Hub.

After thinking about for a long time I think I will set it up so we can have automatic builds but that won't come until I finish working out some of the basics of how we want the images to behave on a final level as even though they are released (and I do use them in production) they are still beta IMO.

We are trying our first automated build right now.

Automated builds are enabled: https://registry.hub.docker.com/u/jekyll/jekyll/build_id/55906/code/bifbugh45hbtmau2qmbgbs4/


If i connect a ps4 controller the addon thinks its a xbox controller

If i connect a ps4 controller the addon thinks its a xbox controller oh and for some odd reason the add-on doesn't allow me to run my game within the editor using split screen mode

If i connect a ps4 controller the addon thinks its a xbox controller

Can you add some additional info from your current setup? It seems you're working from an Android tablet, is that correct? Is the controller connected through bluetooth, or with a cable?

for some odd reason the add-on doesn't allow me to run my game within the editor using split screen mode

Is this the error the screenshot refers to? If yes, it seems you're using a ControllerIconTexture within your code; can you show that part of the code to better understand your current context?
It's also possible that your code is running in/from an autoload, which is registered before the ControllerIcons autoload. If you're using this feature, try to move ControllerIcons to the top, before any of your project-specific code runs.

Can you add some additional info from your current setup? It seems you're working from an Android tablet, is that correct? Is the controller connected through bluetooth, or with a cable?
Yes that is correct and the controller is connected via Bluetooth

Is this the error the screenshot refers to? If yes, it seems you're using a ControllerIconTexture within your code; can you show that part of the code to better understand your current context?
yes
It's also possible that your code is running in/from an autoload, which is registered before the ControllerIcons autoload. If you're using this feature, try to move ControllerIcons to the top, before any of your project-specific code runs.
Okay but I can't do that because I'm using a sonic engine name sonic Worlds next and it need the engine files to be on top for the engine to work

Can you try opening this website (https://hardwaretester.com/gamepad), and check the name of the reported controller ?

This is what my ps4 shows up as

and this is what my generic controller shows up as

Could you try modifying the addon code to print the reported controller name? It's possible it is some variation not currently checked, like "PS4 Controller".
Go to res://addons/controller_icons/Mapper.gd script, and print the current controller name under the _get_joypad_type function:
extends Node
class_name ControllerMapper
...

func _get_joypad_type(device, fallback):
    ...
    var controller_name = Input.get_joy_name(device)
    print("Controller name: ", controller_name) # Add this line here
    ...

Could you try modifying the addon code to print the reported controller name? It's possible it is some variation not currently checked, like "PS4 Controller".
Go to res://addons/controller_icons/Mapper.gd script, and print the current controller name under the _get_joypad_type function:
extends Node
class_name ControllerMapper
...

func _get_joypad_type(device, fallback):
    ...
    var controller_name = Input.get_joy_name(device)
    print("Controller name: ", controller_name) # Add this line here
    ...

It says playstation controller

Thank you, so it seems some correct name is being reported, and not a generic name.
Unfortunately it's a vague name, and cannot be used differentiate between PS3/PS4/PS5 variants. The name being this way, it could be from the specific firmware version, from the specific brand if the controller is not an official Sony one, or from a quirk in the Android system, etc... This is simply the reality of relying on controller names for detecting their icons: for these cases, there isn't much that can be done. I'm considering other sources of information to improve this in the future with #66.
For your case specifically, you can implement a custom mapper if you wish to override this behavior for your project. Check out the Changing controller mapper section on the docs (and Settings to apply it) on how to do it.
Since it's not an issue with the addon, I'll close this issue now. If you need more help on this topic though, feel free to continue the discussion.


Always release option?

This might be out of the scope of this action but we are looking for a way to have a scheduled release that runs no matter what.
I've searched for similar release actions and so far have found yours to be one of the best, it just doesn't quite seem to work for our use case.
Use Case
I am building a dev container for Go and I want the release to happen once a week with the patch version to increment each time.  As part of my build process I run dnf upgrade to upgrade any packages with security updates in the main Docker container.  So in this use case there are no commits or dependabot checks that would account for these updates.
Can it do this and I'm missing a config?  Can this option be added?  Or is this out of scope for your project?

I'm using a different release action this is working for me now so this can be closed.


Update spec for adding volunteering roles

Context
When I updated the add another volunteering role link in #3911, copy and pasting a link from another view resulted in me shipping a change that broke this journey (the link pointed to adding another course instead).
Changes proposed in this pull request

Test that clicking ‘No’ shows the review page with a link to the Get school experience service
Test that in this state the links say ‘Add a role’
Test that when there are roles added, the links says ‘Add another role’
Test that when clicking on either of these links, the user is actually taken to the ‘Add role’ form

Guidance to review

I inserted these additional tests in the existing journey; do these revised test journeys work?
I ended up adding the new methods to the bottom of the file; is that correct?

Link to Trello card
https://trello.com/c/pOlhOCkF/
Things to check

[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] API release notes have been updated if necessary
[ ] New environment variables have been added to the Azure config

normally we declare the methods in the same order as they are appear in the scenario block though it's not always possible if methods are reused.

Yeah, I was unsure of this, as they all follow the method that made sense to go after then_i_can_check_my_answers.

normally we declare the methods in the same order as they are appear in the scenario block though it's not always possible if methods are reused.

Yeah, I was unsure of this, as they all follow the method that made sense to go after then_i_can_check_my_answers.


make test broken

Following the instructions in CONTRIBUTING ends up with error:
make test
python generate_brick.py
Checking 3 containers
Traceback (most recent call last):
  File "/big/Brick/venv/lib/python3.9/site-packages/docker/api/client.py", line 268, in _raise_for_status
    response.raise_for_status()
  File "/big/Brick/venv/lib/python3.9/site-packages/requests/models.py", line 943, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 404 Client Error: Not Found for url: http+docker://localhost/v1.40/containers/create?name=agraph

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/big/Brick/venv/lib/python3.9/site-packages/docker/models/containers.py", line 811, in run
    container = self.create(image=image, command=command,
  File "/big/Brick/venv/lib/python3.9/site-packages/docker/models/containers.py", line 870, in create
    resp = self.client.api.create_container(**create_kwargs)
  File "/big/Brick/venv/lib/python3.9/site-packages/docker/api/container.py", line 430, in create_container
    return self.create_container_from_config(config, name)
  File "/big/Brick/venv/lib/python3.9/site-packages/docker/api/container.py", line 441, in create_container_from_config
    return self._result(res, True)
  File "/big/Brick/venv/lib/python3.9/site-packages/docker/api/client.py", line 274, in _result
    self._raise_for_status(response)
  File "/big/Brick/venv/lib/python3.9/site-packages/docker/api/client.py", line 270, in _raise_for_status
    raise create_api_error_from_http_exception(e)
  File "/big/Brick/venv/lib/python3.9/site-packages/docker/errors.py", line 31, in create_api_error_from_http_exception
    raise cls(e, response=response, explanation=explanation)
docker.errors.ImageNotFound: 404 Client Error for http+docker://localhost/v1.40/containers/create?name=agraph: Not Found ("No such image: franzinc/agraph:v7.0.0")

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/big/Brick/venv/lib/python3.9/site-packages/docker/api/client.py", line 268, in _raise_for_status
    response.raise_for_status()
  File "/big/Brick/venv/lib/python3.9/site-packages/requests/models.py", line 943, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 500 Server Error: Internal Server Error for url: http+docker://localhost/v1.40/images/create?tag=v7.0.0&fromImage=franzinc%2Fagraph

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/big/Brick/generate_brick.py", line 29, in <module>
    from bricksrc.quantities import quantity_definitions, get_units
  File "/big/Brick/bricksrc/quantities.py", line 11, in <module>
    g.expand(profile="brick")
  File "/big/Brick/venv/lib/python3.9/site-packages/brickschema/graph.py", line 153, in expand
    return self.expand("owlrl+shacl+owlrl", backend=backend)
  File "/big/Brick/venv/lib/python3.9/site-packages/brickschema/graph.py", line 149, in expand
    self.expand(prf, backend=backend)
  File "/big/Brick/venv/lib/python3.9/site-packages/brickschema/graph.py", line 183, in expand
    self._inferbackend.expand(self)
  File "/big/Brick/venv/lib/python3.9/site-packages/brickschema/inference.py", line 134, in expand
    agraph = self._client.containers.run(
  File "/big/Brick/venv/lib/python3.9/site-packages/docker/models/containers.py", line 814, in run
    self.client.images.pull(image, platform=platform)
  File "/big/Brick/venv/lib/python3.9/site-packages/docker/models/images.py", line 446, in pull
    pull_log = self.client.api.pull(
  File "/big/Brick/venv/lib/python3.9/site-packages/docker/api/image.py", line 430, in pull
    self._raise_for_status(response)
  File "/big/Brick/venv/lib/python3.9/site-packages/docker/api/client.py", line 270, in _raise_for_status
    raise create_api_error_from_http_exception(e)
  File "/big/Brick/venv/lib/python3.9/site-packages/docker/errors.py", line 31, in create_api_error_from_http_exception
    raise cls(e, response=response, explanation=explanation)
docker.errors.APIError: 500 Server Error for http+docker://localhost/v1.40/images/create?tag=v7.0.0&fromImage=franzinc%2Fagraph: Internal Server Error ("Get https://registry-1.docker.io/v2/: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)")
Makefile:4: recipe for target 'Brick.ttl' failed
make: *** [Brick.ttl] Error 1

Missed the dependency on a working docker service to work with the Brick ontology.


TAB77 OnBeforeSetReportLayout include function parameters in event

Could it be possible to add the parameters of PrintDocumentsWithCheckGUIYesNoCommon to the event OnBeforeSetReportLayout ?
LOCAL PrintDocumentsWithCheckGUIYesNoCommon(ReportUsage : Integer;RecordVariant : Variant;IsGUI : Boolean;AccountNoFieldNo : Integer;WithCheck : Boolean;TableNo : Integer)
OnBeforeSetReportLayout(RecordVariant);

using NAV2018CU3.
(If it's already implemented in BC, I will close this issue and use it as a reference for the regular NAV support).

This is already implemented in BC.

Ty! :)


Fix: Consolidation of Copyright notices #91

These changes has been made as required from Consolidation of Copyright notices #91
I have cleaned up by removing the existing copyright headers from all existing code, and ensuring that the single copyright clause has been updated in the LICENCE.txt

Hi @ahmedsaheed, thanks for knocking this ticket out!

Hi @ahmedsaheed, thanks for knocking this ticket out!


Add/Remove a sym-link for docker tray Kitematic menu item

Add a symbolic link to allow the docker tray Kitematic menu option to work after installing Kitematic via the choco package.
Description
The docker tray icon has a menu item 'Kitematic'. Clicking on this when Kitematic is not installed displays a message to install it to C:\Program Files\Docker\Kitematic. This package installs Kitematic into the chocolatey directory under C:\ProgramData...
This change adds a symbolic link which points from C:\Program Files\Docker\Kitematic\kitematic.exe to C:\ProgramData\Chocolatey\lib\docker-kitematic\tools\kitematic.exe
This allows the docker tray icon menu item to work when installing Kitematic from a choco package.
Motivation and Context
After installing Kitematic the user doesn't need to manually move or install into the %ProgramFiles%\Docker\Kitematic directory.
How Has this Been Tested?
create new package

choco pack

basic install. not already installed, no Docker\Kitematic dir or sym-link

choco install docker-kitematic -dv -s .

install again.

choco install docker-kitematic -dv -s . -Force

uninstall

choco uninstall docker-kitematic

No warnings or errors.
Screenshot (if appropriate, usually isn't needed):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Migrated package (a package has been migrated from another repository)

Checklist:

[x] My code follows the code style of this repository.
[ ] My change requires a change to documentation (this usually means the notes in the description of a package).
[ ] I have updated the documentation accordingly (this usually means the notes in the description of a package).
[x] All files are up to date with the latest Contributing Guidelines
[x] The added/modified package passed install/uninstall in the chocolatey test environment.
[x] The changes only affect a single package (not including meta package).

:x: Package verification failed, please review the Appveyor Logs and the provided Artifacts before requesting a human reviewer to take a look.

I'm not sure how the kitematic.zip file is added to the tools directory in this CI process. I was having trouble with this when testing locally and seems to cause the appveyor test to fail in the same way too. I manually downloaded the 17.6 Kitematic zip file and put it into tools, then choco pack-ed it.
Is this expected?

:white_check_mark: Package verification completed without issues. PR is now pending human review

I'm not sure how the kitematic.zip file is added to the tools directory in this CI process. I was having trouble with this when testing locally and seems to cause the appveyor test to fail in the same way too

It's downloaded to the tools directory by running the update.ps1 script: https://github.com/chocolatey/chocolatey-coreteampackages/blob/master/automatic/docker-kitematic/update.ps1
You'll need AU to be able to run that script though.


NotchDrop

Cask name
NotchDrop
GUI or CLI
GUI
Software official website
https://github.com/Lakr233/NotchDrop
Last version
2.2.8
Description
Transform your MacBook's notch into a convenient file drop zone.
Download URL
https://github.com/Lakr233/NotchDrop/releases/download/2.2.8/NotchDrop.zip
Charging model
Free

https://github.com/Lakr233/NotchDrop/issues/18
https://github.com/Brewforge/homebrew-extras/commit/217bbf24561d6ac871db355866c42c3c73f24f73


[WIP] Improve FileTarget performance

Current coverage is 76% (diff: 81%)

Merging #1593 into master will decrease coverage by <1%

@@             master      #1593   diff @@
==========================================
  Files           270        272     +2
  Lines         16410      16613   +203
  Methods        2621       2648    +27
  Messages          0          0
  Branches       1787       1824    +37
==========================================
+ Hits          12519      12656   +137
- Misses         3488       3528    +40
- Partials        403        429    +26

Powered by Codecov. Last update cdb1fcf...a0242b9

this PR: https://github.com/NLog/NLog/pull/1593/commits/9e13a569d3cc3b0f5b2548cd48039a583a7bbdca

master: ~4.3.8 (cdb1fcf14f6e505317ab9cdd00322fe60ba4b4ca)

tag 4.2.3

So still not fully back in performance, but almost there

@bhaeussermann Hey!
This isn't done yet, and maybe I revert removing FileCharacteristics, but if you have time, please let me know your thoughts on these changes :)

Regarding the failing test (with " a/test.log "). I suspect Travis runs on Unix, and on Unix Path.VolumeSeparatorChar is /.
On Unix we should probably ignore the volume separator.

On Unix absolute paths begin with //, not true?

I suspect Travis runs on Unix
Mh... but then "d:\test.log" would have failed.

I don't notice any other issues.
Well done with the performance improvement so far!


Add VA attributes to handle 'extension' content in annotation-source.yaml

Exploring potential to move content from two metakb-cvc extensions into proper VA fields. Lower priority IMO, but worth at least mentioning ahead of the plenary.
- ClinVar collection method data is currently in a clinvar-scv-methods Extension, but could be framed to fit into the VA Statement.has_evidence_of_type attribute.
- ClinVar review status is currently in a clinvar-review-status Extension,  but could be framed to fit into the VA InformationEntity.information_quality attribute.

Locking metakb-cvc for plenary. Created #89 to revisit these points at a later date if desired.


Clustering doesn't work with RabbitMQ 3.6

Once rabbitmq 3.6 was released clustering stopped working properly.

This isn't an issue with rabbitmq 3.6


Updated gon version to v0.2.5

Description
Updated gon version to v0.2.5

TODOs
Read the Gruntwork contribution guidelines.

[x] Update the docs.
[x] Run the relevant tests successfully, including pre-commit checks.
[x] Ensure any 3rd party code adheres with our license policy or delete this line if its not applicable.
[x] Include release notes. If this PR is backward incompatible, include a migration guide.

Release Notes (draft)

Added / Removed / Updated [X].
Migration Guide

LGTM!


<EMAIL_ADDRESS>

Simple code with Constructor and Setter-based injections.
It has some Logback configuration as well, but otherwise, it is pretty clean.

Switching for Annotations


Migrate DHCPv4 tests to new net_pkt API

Codecov Report

Merging #13738 into master will increase coverage by 0.47%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #13738      +/-   ##
==========================================
+ Coverage   52.24%   52.71%   +0.47%
==========================================
  Files         307      306       -1
  Lines       45413    44951     -462
  Branches    10508    10351     -157
==========================================
- Hits        23726    23696      -30
+ Misses      16898    16473     -425
+ Partials     4789     4782       -7

Impacted Files
Coverage Δ

arch/posix/core/posix_core.c
90.9% <0%> (-1.02%)
:arrow_down:

subsys/net/ip/dhcpv4.c

subsys/net/ip/net_core.c
59.11% <0%> (+0.47%)
:arrow_up:

include/net/net_pkt.h
88.62% <0%> (+1.24%)
: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 23ba884...0262ede. Read the comment docs.


[FancyZones Editor]: The name property must not contain any character in the private Unicode range for 'Edit layout' and 'Create custom layout' control.

Microsoft PowerToys version
"0.35.0"
Running as admin

[ ] Yes

Area(s) with issue?
FancyZones Editor
Steps to reproduce
Test Environment:
OS: Windows 10 Build 21354.1
App version: 0.35.0
App name: PowerToys
Screen Reader: Narrator
Tool: Accessibility Insight for Windows
Repro Steps

Launch 'PowerToys' application.
Application will get open with default list item 'General' selected.
Navigate to 'FancyZones' at the left side of the pane and activate it.
Now navigate to 'Launch layout editor' button and activate it.
FancyZones editor window will get open.
Run 'Accessibility Insight' tool on the application.
Observe the issue.

User Impact: Priority-2
If the object role, state(s), properties, boundary, name, and description are not programmatically determinable, it would be difficult for the platforms that provide Accessibility APIs, and would in turn affect the screen reader dependent users to determine the object properties.
MAS Reference:
https://microsoft.sharepoint.com/:w:/r/sites/accessibility/_layouts/15/Doc.aspx?sourcedoc={B661D774-60D0-4277-A121-73F986EBEE76}&file=MAS 4.1.2 – Name%2C Role%2C Value.docx&wdLOR=cE56E09CE-4478-4D20-B785-78D1B90569C1&action=default&mobileredirect=true&cid=577e2918-ba37-4bc7-bdd5-df0122160d1f
Note: Same issue repro for Grid and layout on Choose layout dialog.

✔ Expected Behavior
The name property must not contain any character in the private Unicode range for 'Edit layout' and 'Create custom layout' control.
Modify the Name property, removing all characters in the range U+E000 to U+F8FF and replace them with meaningful, standard text content.
❌ Actual Behavior
The name property contains character in the private Unicode range for 'Edit layout' and 'Create custom layout' control.
Other Software
No response

@dedavis6797 @crutkas Please advise: in WPF we can only use Segoe MDL2 Assets (the icon font) with unicode characters. For example, we cannot use Content="Edit" but Content="". Should we mark this issue as blocked for WinUI 3?

Can't we manually set the name?

@crutkas Yeah we do that using AutomationProperties.Name on the Button. Accesibility Insights is still throwing an error though that the "Text" is a Unicode character.

Closed with the release of v0.45.


False Positive error reported for oneOf schema

What version of Ajv are you using? Does the issue happen if you use the latest version?
Ajv - 6.10.0
Ajv options object

{ allErrors: true }

JSON Schema

{
  "type": "object",
  "properties": {
        "type": {
          "type": "string",
          "enum": ["1", "2"],
          "default": "1"
        }
      },
      "required": ["type"],
      "dependencies": {
        "type": {
          "oneOf": [
            {
              "properties": {
                "type": {
                  "enum": [
                    "1"
                  ]
                },
                "role": {
                  "type": "string"
                }
              },
              "required": [
                "role"
              ]
            },
            {
              "properties": {
                "type": {
                  "enum": [
                    "2"
                  ]
                },
                "ID": {
                  "type": "string"
                }
              },
              "required": [
                "ID"
              ]
            }
          ]
        }
      }
}

Sample data

{
  "type": "1"
}

Validation result, data AFTER validation, error messages
[
  {
    "keyword": "required",
    "dataPath": "",
    "schemaPath": "#/dependencies/type/oneOf/0/required",
    "params": {
      "missingProperty": "role"
    },
    "message": "should have required property 'role'"
  },
  {
    "keyword": "enum",
    "dataPath": ".type",
    "schemaPath": "#/dependencies/type/oneOf/1/properties/type/enum",
    "params": {
      "allowedValues": [
        "2"
      ]
    },
    "message": "should be equal to one of the allowed values"
  },
  {
    "keyword": "required",
    "dataPath": "",
    "schemaPath": "#/dependencies/type/oneOf/1/required",
    "params": {
      "missingProperty": "ID"
    },
    "message": "should have required property 'ID'"
  },
  {
    "keyword": "oneOf",
    "dataPath": "",
    "schemaPath": "#/dependencies/type/oneOf",
    "params": {
      "passingSchemas": null
    },
    "message": "should match exactly one schema in oneOf"
  }
]

What results did you expect?
Sample Data contains type: 1 which is part of the allowed values,  but still getting error for it -
 {
    "keyword": "enum",
    "dataPath": ".type",
    "schemaPath": "#/dependencies/type/oneOf/1/properties/type/enum",
    "params": {
      "allowedValues": [
        "2"
      ]
    },
    "message": "should be equal to one of the allowed values"
  }

Error should not be reported is value matches any of the allowed values. It should contain errors related to other missing fields. Let me know if i have missed something here.
Are you going to resolve the issue?
Don't think so.

the actual validation error is: "should match exactly one schema in oneOf", and neither is matched.
One fails enum: [1], and another fails required: [role]

Yes, other errors make sense. But value of type is part of the allowed values ["1", "2"],  then why should we get that error?


Mark node as NotReady when ovs-ovn failed

Feature request
When ovs-ovn on when node fails, scheduler will still puts pods into this node and leads further problems.
We need to mark the node as NotReady to prevent this, one simple way is that kube-ovn-cni prob check if ovs is not ready and remove the cni conf.

if the node status is notready, kube-controller-manager can't put pods into notready nodes.
Can you draw a picture with descript your means?
maybe, we only use shell scripts in k8s prob check.

If node not ready eviction will be triggered which is not what we want.


Push release "latest" tags for all docker sc4s images

:tada: This PR is included in version 3.5.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fixed bug in model command where loose string comparison would override properties for model relations

This is the same as https://github.com/barryvdh/laravel-ide-helper/pull/804 without the refactoring (and adapting the tests).
I don't thin that refactoring so many lines in contrast for a single line change makes sense and could be the reason this wasn't merged already.
Note: I cherry-picked the original commit, which means @markokeeffe as the author is kept.

Note: the failing test is unrelated to this change, it's because of phpcs complaining about the change from https://github.com/barryvdh/laravel-ide-helper/pull/862


[Merge] MONDO:0011543	BRCA3

Mondo term (ID and Label)
MONDO:0011543	BRCA3
Reason for deprecation
OMIM merged these
Term to be merged with
MONDO:0016419	hereditary breast carcinoma

Don't merge these, they are different concepts
OMIM has a description of the putatitive BRCA3 concept:
https://omim.org/entry/114480
From my reading of this, BRCA3 should indeed by obsoleted. Cite 114480 as rationale. Use a consider link to point to MONDO:0016419


Better test filtering for the 2020 ytp tests

we need a way to filter youtube playback subtests by type some devices can't run specific codecs so we need to filter those out.
For example we would line to run all HFR Tests tests excluding VP9

Thanks for reporting, will address this internally.

Thanks for reporting, will address this internally.


⚠ GitHub has degraded performance

In b94294b, GitHub (https://www.githubstatus.com/api/v2/status.json) experienced degraded performance:

HTTP code: 200
Response time: 157 ms

Resolved: GitHub performance has improved in d7bfe58.


Registry config storage s3 AWS EC2 instance role issue

$ docker images
REPOSITORY              TAG                 IMAGE ID            CREATED             VIRTUAL SIZE
registry                2                   dfb4481bd21b        10 days ago         223.4 MB

Documentation on the s3 storage driver has this explicit Note:

accesskey: Your aws access key.
secretkey: Your aws secret key.
Note You can provide empty strings for your access and secret keys if you plan on running the driver on an ec2 instance and will handle authentication with the instance's credentials.

The note leads you to believe you should include YAML keys named accesskey and secretkey and leave them blank if you intend to use an IAM Role to grant s3 access to your instance. However, it fails every time.
storage:
    cache:
        blobdescriptor: inmemory
    s3:
        accesskey:
        secretkey:
        region: us-east-1
        bucket: foo.docker.registry
        encrypt: false
        secure: false
        v4auth: false

What you have to do is not include the key fields and then things work as expected.
storage:
    cache:
        blobdescriptor: inmemory
    s3:
        region: us-east-1
        bucket: foo.docker.registry
        encrypt: false
        secure: false
        v4auth: false

I suspect the code is using the blank values from YAML instead of ignoring them.

@joecrotty : good observation.  What happens when empty double quotes are used?

@RichardScothern Empty double quotes worked fine.
storage:
    cache:
        blobdescriptor: inmemory
    s3:
        accesskey: ""
        secretkey: ""
        region: us-east-1
        bucket: foo.docker.registry
        encrypt: false
        secure: false
        v4auth: false

In that case,  the documentation is correct.  I suspect you need to provide empty strings so yaml can determine the type of the empty value.

@RichardScothern Can't argue w/ your logic there. My main issue was in not know what a YAML empty string looked like. Would be super helpful if the docs stubbed out an example of an empty accesskey/secretkey as not everyone intuitively knows YAML specs.

We should have an IAM recipe.
Ping @djenriquez who recently did a good job at documenting CF :-)

Blank accesskey/secretkey now supported in the yaml file as of #1414


Make landing section of the site mobile friendly

What was the feature/problem?
fixes #99 - Primary way the kids are going to interact with this site is via a tablet
How does this PR address the above feature/problem?

Additional fixes

Removed the sliding "hello, hola..." animation as it was distracting and helped with the text overlap issue on mobile screen.
Have made the login, registration, and wish list landing pages mobile-friendly.
Fixed few accessibility issues and HTML5 semantic issues (hover pointer on wish type and the section on landing header)
Removed the use of the CSS grid as it is not fully supported in all browsers.

Check lists

[x] Test passed
[x] Coding style (indentation, etc)

Codecov Report

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

@@           Coverage Diff           @@
##           master     #105   +/-   ##
=======================================
  Coverage   95.39%   95.39%
=======================================
  Files          28       28
  Lines         391      391
  Branches       36       36
=======================================
  Hits          373      373
  Misses         15       15
  Partials        3        3

Flag
Coverage Δ

#api
97.12% <ø> (ø)
:arrow_up:

#ui
94% <ø> (ø)
:arrow_up:

Impacted Files
Coverage Δ

ui/src/header/index.js
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 f1e1799...8d39bfc. Read the comment docs.

Wow thanks so much for the contribution! I will sit with our designer on this and walk them through the experience and either accept or leave feedback as soon as I can.

@crstamps2 Do I need to create a new issue to make all the text black, as well as update the logos to the black versions or is that part of #98 ??? Not sure if they are pointing to same gfx. Just LMK.

@hellonoosh I think we are going to keep the design updates in a separate issue/PR, since someone else is working on those already. This PR will just be for making the app mobile responsive.

@chekkan when I boot up the app and use chrome dev tools to force it to be mobile sized, this is what I get.

And this is what I get if it's an iPad Pro

I will look into those issues and get it fixed 😁 any other feedback? Are you happy with me getting rid of the hello, hola animation?

@crstamps2 those issues should be solved now. Please let me know if I missed something.

@chekkan Taking a look here shortly. Concerning the hello animation, while it is a little gimmicky and very 90s-esque animation style, some of the decision makers like it. I think it's ok to not have it in a mobile phone view, but on the standard desktop view, I believe we are going to keep it. Will get back to you on that.

@chekkan can you update with master and resolve the conflicts and then I will review and merge.

@crstamps2 any update on this?


fix(compiler): return handler value for event modifiers instead of un…

fix(compiler): return handler value for event modifiers instead of undefined

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] It's submitted to the dev branch for v2.x (or to a previous version branch), not the master branch
[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] All tests are passing: https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#development-setup
[x] New/updated tests are included

If adding a new feature, the PR's description includes:

[x] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:
Without modifiers compiler return original vm function, so we can expect to handle returned value (as example - handle it in directive)
on: {
    click:  _vm.test($event)
}

With modifier it always return "undefined" because of missing return value
on: {
    click: function($event) {
        $event.preventDefault()
        _vm.test($event)
    }
}

This PR add correct return value if modifier wrapper is used
on: {
    click: function($event) {
        $event.preventDefault()
        return _vm.test($event)
    }
}

How will it behave with the following example? https://codepen.io/anon/pen/ddwWKW
The handler code is alert('x'); alert('y'), so it will be compiled to return alert('x'); alert('y') and the second alert won't be shown, am I right?

Agree with @jacekkarczmarczyk.
@posva , @Kingwl : Now return added only for single handler expressions. And result all the same with and without modifiers
@click="test" =>click:  _vm.test
@click="test($event)" => click: function($event) { return  _vm.test($event) }
@click="test($event)**;**" => click: function($event) { return  _vm.test($event) }
@click="test1($event);test2($event)" =>
    _vm.test1($event)
    _vm.test2($event)
} ```

And the same behavior with modifiers

What about @click="alert(';')"?
split(';') doesnt' seem to be the good solution

@jacekkarczmarczyk what about RegExp like /(?:"|'[^"']*"|')\)(\s+)?;/

Maybe we should return only when the provided expression is a function: <div @click="doSomething"> but not when it is a statement or expression: <div @click="doSomething()"> It always possible to wrap things when you need to pass parameters:
<button @click=" e => doSomething('a parameter', e)">Click me</button>

@posva you're totally right. But there is a problem when we use modifiers. They wraps handler and we can't get result.
For now i'm stuck at parsing handler to check is multiple expressions inside of it
/)(?:\s*);(?:\s*)(?!.*'|")/
http://take.ms/TcERo
If it contains single expression - add return; If not - leave handler as is.
A will be appreciate if someone help to check it correctly.
@jacekkarczmarczyk, @posva, @Kingwl maybe you could help? :)

thanks for your pr
please add more tests for the case

For now i'm stuck at parsing handler to check is multiple expressions inside of it
/)(?:\s*);(?:\s*)(?!.*'|")/

IMO parse expression with regex or other string operation is not a good chooice
we need to reconsider this feature
ping @yyx990803

implement something like handler.rawValue to get access to non-wrapped handler?

we have not instanced the rawValue yet, the modifiers generated code has mixin with handler
instanced the handler.rawValue will lead to runtime code size increases
Maybe we can implement it in a limited way as @posva said

I also think using a regex is a bad idea

@posva, @jacekkarczmarczyk, @Kingwl,  Morning. I've re-implemented exactly as @posva proposed before.
Now we expected that
@click="test" => click: _vm.test
@click.prevent="test" =>
click: function($event) {
    $event.preventDefault()
    return _vm.test($event)
}

@click.prevent="test1();test2()" =>
click: function($event) {
    $event.preventDefault()
    _vm.test1()
    _vm.test2()
}

@posva, @jacekkarczmarczyk, @Kingwl,  Morning. I've re-implemented exactly as @posva proposed before.
Now we expected that
@click="test" => click: _vm.test
@click.prevent="test" =>
click: function($event) {
    $event.preventDefault()
    return _vm.test($event)
}

@click.prevent="test1();test2()" =>
click: function($event) {
    $event.preventDefault()
    _vm.test1()
    _vm.test2()
}

@posva checked this case:
@click.prevent="(e) => test1(e)" compiled to:
click: function($event) {
                $event.preventDefault()
                ;(function(e) {
                  return _vm.test1(e)
                })($event)
              }

So seems that no need to make any other changes

@posva checked this case:
@click.prevent="(e) => test1(e)" compiled to:
click: function($event) {
                $event.preventDefault()
                ;(function(e) {
                  return _vm.test1(e)
                })($event)
              }

So seems that no need to make any other changes

@posva pushed change. Now ^ example returns
click: function($event) {
                $event.preventDefault()
                return (function(e) {
                  return _vm.test1(e)
                })($event)
              }

dd some other test case for method path and function expression please

@Kingwl there are 7 function expression tests was there. I've added more cases with methods and so on.


clang compile targetting wasm failed

I built llvm with setting -DLLVM_TARGETS_TO_BUILD="host;X86;RISCV;WebAssembly" -DLLVM_ENABLE_PROJECTS="llvm;clang;lld;compiler-rt;clang-tools-extra for cmake
chao@DESKTOP-B79IRTG:~$ which clang
/usr/local/llvm/bin/clang

chao@DESKTOP-B79IRTG:~$ llc --version
LLVM (http://llvm.org/):
  LLVM version 11.0.0git
  Optimized build.
  Default target: x86_64-unknown-linux-gnu
  Host CPU: skylake

  Registered Targets:
    riscv32 - 32-bit RISC-V
    riscv64 - 64-bit RISC-V
    wasm32  - WebAssembly 32-bit
    wasm64  - WebAssembly 64-bit
    x86     - 32-bit X86: Pentium-Pro and above
    x86-64  - 64-bit X86: EM64T and AMD64

I also has installed WASI
chao@DESKTOP-B79IRTG:~$ ls /opt/wasi-sdk/share/wasi-sysroot/
include  lib  share

Then I tried to compile a helloworld with clang
chao@DESKTOP-B79IRTG:~$ clang helloworld.c  --sysroot=/opt/wasi-sdk/share/wasi-sysroot/ --target=wasm32-wasi
wasm-ld: error: cannot open /usr/local/llvm/lib/clang/11.0.0/lib/wasi/libclang_rt.builtins-wasm32.a: No such file or directory
clang-11: error: linker command failed with exit code 1 (use -v to see invocation)

Hi!
The builtins need to be installed as well (/usr/local/llvm/lib/clang/11.0.0/lib/wasi/libclang_rt.builtins-wasm32.a).
You can get them by compiling clang_rt from the wasi-sdk package, or directly download them here (libclang_rt.builtins-wasm32-wasi-11.0.tar.gz).

@jedisct1
Thanks !
but after copying the .a I ran into another linker error
chao@DESKTOP-B79IRTG:~/a$ ls /usr/local/llvm/lib/clang/11.0.0/lib/wasi
libclang_rt.builtins-wasm32.a
chao@DESKTOP-B79IRTG:~/a$ clang helloworld.c  --sysroot=/opt/wasi-sdk/share/wasi-sysroot --target=wasm32-wasi
wasm-ld: error: /opt/wasi-sdk/share/wasi-sysroot/lib/wasm32-wasi/libc.a(__original_main.o): undefined symbol: main
clang-11: error: linker command failed with exit code 1 (use -v to see invocation)

Looks like your helloworld.c is missing a main function?

I tried to reproduce this. LLVM HEAD was compiled from source using the configuration above, and the linker found the main function just fine.
clang version 11.0.0 (https://github.com/llvm/llvm-project.git 8799ebbc1f03a348f732fc14242ad4c395076bcc)
Target: aarch64-unknown-linux-gnu

Maybe the version you used needs an update?
If you don’t have a main function, add -mexec-model=reactor to the command line.

I think this is fixed by following @jedisct1's instructions (and this issue is a bit stale); feel free to reopen it if that does not fix the issue.


More methods in Algo

This PR makes several major changes:

The names of functions in BaseAlgo class has been updated to be more standardized and descriptive.
Added get_option_auto() and is_day_trade() to Algo class
Fixed major bugs in PaperBroker
Added BaseLogger class to track transaction history.

Again force merging as it contains important bug fixes. In the future I'll try to isolate bug fixes into bug-fixes branch so I don't have to keep doing force merges.


Test parameter file should fill based on last version

Feature Request
When adding a new version to a tool you have to manually set set the location of the 'Test Parameter Files' each time.
Desired behaviour
If a test parameter file is defined on the last 'default branch' it should be copied to the new version if found to exists in same location of repository.  If the path is no longer valid warn user that it is no longer mapped.
Environment (Browser or OS and Dockstore version)
UI 1.2.2

I've actually started working on this feature already.  On my working build, I can set a default test CWL/WDL test parameter file path (similar to default CWL descriptor path) for each tool.  All new versions/tags upon refresh will automatically add that path to its list of test parameter files.

see also #607
It should be possible to set a default path akin to the default cwl path and default wdl path. After that, any additional test json defined can be merged as in the OP

@garyluu  FYI, update this ticket

Still in the process of testing the updated web service.  For new tools that are added via refresh all tools, currently "/test.json" is added as the defaultTestParameterFile for the tool as well as all its versions/tags.  I'm also currently updating UI2 to allow editing of the default test parameter file paths.  However, editing the default test parameter file path in a manual mode tool will not propagate the change to all its versions.  New tags/versions that are manually added will autofill with the default test parameter file path though.

Related to #745 testing


cant build plugins

Brief description of your issue
all plugins:
error D8016: '/ZI' and '/GL' command-line options are incompatible
Steps to reproduce (optional)
No response
Expected behavior (optional)
No response
Actual behavior (optional)
No response
Environment (optional)
No response

i dont open in VS, only build_release.cmd


PPrint fails to derive a printer for case classes with more than 1 argument list

@ case class A(x: List[String], y: Int)
defined class A
@ A(List.fill(10)("long string"), 0)
res12: A = A(
  List(
    "long string",
    "long string",
    "long string",
    "long string",
    "long string",
    "long string",
    "long string",
    "long string",
    "long string",
    "long string"
  ),
  0
)
@ case class B(x: List[String])(y: Int)
defined class B
@ B(List.fill(10)("long string"))(0)
res14: B = B(List(long string, long string, long string, long string, long string, long string, long string, long string, long string, long string))

This also appears to impact upickle:
scala> case class B(x: List[String])(y: Int)
defined class B

scala> upickle.default.write(B(List.fill(10)("long string"))(0))
<console>:13: error: type mismatch;
 found   : Option[List[String]]
 required: Option[(List[String], Int)]
Error occurred in an application involving default arguments.
       upickle.default.write(B(List.fill(10)("long string"))(0))

From what I can tell, it’s fixed somewhere in 0.5.0. At least ammonite 0.9.0 no longer has this problem.


[Renderer] Poor performance/stutters/frame drops when flashlight is on

Map from video can be found at https://www.patreon.com/posts/32731370
Map name: de_hankagai_esc
Probably because of flickering map lights mostly.
https://user-images.githubusercontent.com/3608544/189210878-92afb614-0f59-46aa-8a2f-07ec0f922ea0.mp4

That is probably because the whole scene has to be rendered twice when flashlight on.

The performance is not too good even when the flashlight is off. Try the map yourself if you want. It happened on other maps too but can't remember which.

In my case (as of this month, which is when I discovered this awesome mod) is the shadows that affect my performance. Not sure why that is exactly but my game starts to freeze randomly, aka stuttering, which strangely  (or maybe not at all) doesn't happen when playing with the default broken shadows.


Binding redirect diagnostics and troubleshooting

In cases where there are assembly binding redirect conflicts, the diagnostic messages you get from msbuild can be pretty bad and sometimes provide inaccurate/incorrect advice. For example, I'm troubleshooting a scenario right now where each binding redirect error gives me advice that just leads to the other error:
MSB3276: Found conflicts between different versions of the same dependent assembly. Please set the "AutoGenerateBindingRedirects" property to true in the project file
Then if as advised, I set the property:
The explicit binding redirect on "Microsoft.Build.Framework, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a" conflicts with an autogenerated binding redirect. Consider removing it from the application configuration file or disabling autogenerated binding redirects.
I don't necessarily view either of these errors as msbuild defects - I think they're probably both correct representations of whatever is going on when I'm trying to compile msbuild locally. I wanted to file this issue though because I think these are bad diagnostic messages and they make it hard to debug the problem.
As far as I can tell there's no guidance given here on where the redirects that conflict are defined, or where the conflicting versions are referenced. When I look around for info on these errors I can't find any information on how you would figure that out, either, just lots of people talking about turning the auto generate mode on or off.
So as a starting point, I think that "please set" message might be inappropriate since that implies it will fix it, when it may just create a different issue - we're encouraging people to just set that property on when that may be masking some other root cause they could address.
More importantly though there needs to be some way to get insight into where the binding redirects are coming from, preferably described in the error message so people can actually find it. The diagnostic info is definitely not something you will find by searching for MSB3276 on google (or in a brief search through the msbuild docs).
Could we expand msbuild to, for example:

For the first one, list out all the conflicting versions and where those references are from
For the second one, list out the source of the explicit binding redirects along with the autogenerated redirect

For the first one, list out all the conflicting versions and where those references are from

Tracked by #608.

For the second one, list out the source of the explicit binding redirects along with the autogenerated redirect

Yes, definitely. Possibly just linking the path of and line number in app.config would communicate this nicely (but I don't know if we currently track that). Let's use this item for that.

Has this (nearly 4-year old) issue been resolved? I am currently running into the same problem and have ping-ponged back and forth as @kg has described. How do I even determine which package is is responsible for the explicit binding redirect? I have looked through the dependencies of all of my NuGet packages, and I don't see a dependency on the later version of System.Net.Http.


Change default value for HiddenFieldStyle.setterCanReturnItsClass

IMO there's no reason to treat differently classic setters and fluent setters

@murdos Fair point 😉  thanks for opening this PR. Merging 👍


style: update troubleshooting.md

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


feat(Block): Multiple titles

To be able to create TUIs such as that of btop, I believe it to be necessary to be able to have multiple titles for a single block. This pull request adds the ability to have multiple titles per block. Each title can have a different alignment thought multiple titles can have the same alignment.
For backwards compatibility sake, I let the old title system and just added a new one that takes precedence over the old one in cases it has been set.
In the new system, you declare your titles in the following ways:
Block::default()
    .title("old system"),
    .titles(vec![
        Title::default()
            .content("Title 1")
            .alignment(Alignment::Right),
        Title::default()
            .content("Title 2")
            .alignment(Alignment::Right)
            .position(Position::Bottom),
        Title::default()
            .content("Title 3")
            .alignment(Alignment::Center)
            .position(Position::Bottom),
        Title::default()
            .content("Cen")
            .alignment(Alignment::Center)
            .position(Position::Bottom),
        Title::default()
            .content("Center 1")
            .alignment(Alignment::Center),
        Title::default().content("Cen").alignment(Alignment::Center),
    ])
    .borders(Borders::ALL);

where the content is just a Line like in the old system.
Note: I haven't added tests, documentation, or bothered with code style.

Can you explain the need for this a bit more. How would you explain the feature to someone that hasn’t seen btop? I took a look at the screenshot and can’t identify how multiple titles is useful.

I believe it would allow for a better user experience. For example, similarly to Btop, the titles can be useful for indicating to users keyboard shortcuts they can make use of in a block; one might also want to add additional information in a block's border such as a status bar with word counts, current location, and such. Here are other uses I found:

Hierarchical information representation. E.g. a main title and a subtitle;
Navigation. E.g. arrows on each side of a block;
Step-by-step instructions or tutorials. I.e. something like a wizard;
Tabs; and,
Filtering and sorting options.

Sorry, I think I wasn't clear. Help me understand how the desired state is desirable. I'm looking for a description of the feature that tells me the effect of this change. How does it render? What are the features that it implements and how do they interact (multiple titles, alignment, on_bottom)? How does it interact with the current features of the block / borders?
For on_bottom: bool, idiomatic rust would suggest making this a position enum (Position::top/bottom) rather than a bool.

Oops, sorry for misunderstanding. I copied what was there before in the Block widget thinking that's how this library wanted it. Let me make the change real quick.

Got it
Allow the block to render multiple titles which each can be positioned either top or bottom,  aligned (Left / Center / Right), and individually styled.
I wonder if we can change title() to accept Into<Title> instead of Into<Line> (and then impl From<Line> for Title). Then  we could allow title() to be called multiple times. e.g.:
Block::default()
    .title(Title::default()
        .content("Left")
        .alignment(Alignment::Left)
        .style(Style::default().fg(Color::Yellow))
    .title(Title::default().content("Center").alignment(Alignment::Center))
    .title(Title::default().content("Right").alignment(Alignment::Right))
    .title(Title::default().content("Left").alignment(Alignment::Left).position(Position::Bottom))
    .title(Title::default().content("Center").alignment(Alignment::Center)).position(Position::Bottom)
    .title(Title::default().content("Right").alignment(Alignment::Right)).position(Position::Bottom)

Thoughts?

Yes, true. I believe that to be a viable option. I'll work on it sometimes soon.

I implemented the suggestion you made but had to remove the old system's title_alignment and title_on_bottom functions. As such, I rewrote some demo examples and some tests.
I'm now uncertain whether or not to bring them back for backward compatibility. As reading them would mean people won't have to make any changes to their current library. We could just mark them deprecated.
Please let me know your opinion on this.

I think leave them in if possible and mark them deprecated.
Perhaps make the field used for the default for the alignment?

I added tests, and brought back title_alignment and title_on_bottom. All that is left is adding documentation and deprecating    title_alignment and title_on_bottom.
Backward compatibility-wise, people shouldn't have to change anything. Thought I haven't tested this.

The examples are generally good for checking the backwards compatibility as they show what a user will see if they are using the deprecated methods. Assume the next major version for a deprecation (v0.22.0).

All done I believe.

A lot of my comments come down to picking an API that:

avoids breaking changes when possible and reasonable
is the simplest least surprising interface
avoids repetition

I think that
Block::new()
  .title("A")
  .title("B")
  .title_style(Style::default()...)
  .title_position(Position::Bottom)

is fine

P.s. Thanks for sticking with it on this review.
I think the next change you make will probably be ready to merge. When you push can you please squash the changes into a single commit and update the commit message to use the conventional commit format with a header and a body that describes the feature enough that it works well for the changelog.
I didn't mention in the review, but would like to see it some tests that show how overlapping titles are handled (e.g. what happens when there is only 10 characters wide and there are 3 titles (left, center, right aligned) and they are each 4 characters wide...?)

I tried using Commitizen for the commit, message but it would seem not to have worked.

Wait, did I mess something up while rebasing? I'm new to this and I'm not sure.

Yes maybe, but it's not the end of the world.
If you want to get back to exactly where you were, take a look at the output of git reflog There will a line that is just before a line that says rebase (start): checkout main. You can write git reset --hard <the SHA1 hash on that line> and you're exactly where you were.
I'd recommend taking the following steps:
git switch main

# assuming the this repo is your upstream remote:
git pull upstream main

git switch multiple-titles

# this is optional - it just creates a branch that helps you easily go back to where you were
git branch mutliple-titles-before-rebase

# assuming that you've already reset the branch back to where you want to merge
git rebase -i <some commit hash that is not your code>
# edit each line that is your code except the first and mark it as s or squash. close the file
# this will result in the code that you've written being a single commit
# now rebase on main

git rebase main
git push origin multiple-titles --force

Thanks a lot, that really helped.


[#291] Add function for CRC32C checksum

You need to check for SSE 4.2 in CMakeLists.txt and define a constant that you can use in an #ifdef

See CI, and remember to squash

@jesperpedersen The implementation is complete. Have verified the hash values with other tools like rhash and gsutil.

I tested the code on a 810MB text file. Using table lookup it took 2.446s and with SSE it took 0.260s.

cat /proc/cpuinfo
...
flags		: fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx mmxext fxsr_opt pdpe1gb rdtscp lm constant_tsc rep_good nopl nonstop_tsc cpuid extd_apicid aperfmperf rapl pni pclmulqdq monitor ssse3 fma cx16 sse4_1 sse4_2 movbe popcnt aes xsave avx f16c rdrand lahf_lm cmp_legacy svm extapic cr8_legacy abm sse4a misalignsse 3dnowprefetch osvw ibs skinit wdt tce topoext perfctr_core perfctr_nb bpext perfctr_llc mwaitx cpb cat_l3 cdp_l3 hw_pstate ssbd mba ibrs ibpb stibp vmmcall fsgsbase bmi1 avx2 smep bmi2 cqm rdt_a rdseed adx smap clflushopt clwb sha_ni xsaveopt xsavec xgetbv1 cqm_llc cqm_occup_llc cqm_mbm_total cqm_mbm_local clzero irperf xsaveerptr rdpru wbnoinvd cppc arat npt lbrv svm_lock nrip_save tsc_scale vmcb_clean flushbyasid decodeassists pausefilter pfthreshold avic v_vmsave_vmload vgif v_spec_ctrl umip rdpid overflow_recov succor smca sev sev_es

but
-- Performing Test HAVE_SSE42
-- Performing Test HAVE_SSE42 - Failed

CI...

@jesperpedersen The test is now only failing for macos. I dont have any experience with macos. Can you please help here?

You can start with #ifdef HAVE_LINUX - but test with your FreeBSD installation as well...

@jesperpedersen The macos Actions uses arm64 which does not has SSE4.2 instructions. So the HAVE_SSE42 should fail. Have updated CMakeLists.txt to clear the residual -msse4.2 flag which was failing the test.

Merged.
Thanks for your contribution !


Fix docstring

We have great tests 😄 . I'm guessing this is due to a change in rasterio, since the tests are pulling in rasterio 1.2.6?


Function arguments should not be allowed to shadow

Summary:
We have intentionally disallowed shadowing (masking) of variable names from an outer scope by ones in an inner scope almost everywhere in the language, so as not to confuse novice programmers. One place where we have not done so is for function arguments. I think it should be enforced there as well.
Description:
Function arguments can currently clash with math library primitive names or with previously defined function names.
Reproducible Steps:
The following example model is currently accepted by the Stan parser:
functions {
  void foo(){}

  real relative_diff(real x, real y, real max, real min, int foo) {
    real abs_diff;
    real avg_scale;
    abs_diff = fabs(x - y);
    avg_scale = (fabs(x) + fabs(y)) / 2;
    if ((abs_diff / avg_scale) > max)
      reject("user-specified rejection, difference above ",max," x:",x," y:",y);
    if ((abs_diff / avg_scale) < min)
      reject("user-specified rejection, difference below ",min," x:",x," y:",y);
    return abs_diff / avg_scale;
  }
}
transformed data {
  real a =  -9.0;
  real b = -1.0;
  real mx = 1.2;
  real mn = 1.1;
}
parameters {
  real y;
}
model {
  real c;
  c = relative_diff(a,b,mx,mn, 1);
  y ~ normal(0,1);
}

Current Output:
The parser does not complain.
Expected Output:
I would expect it to complain for two different reasons: 1. the argument max clashes with a math library function name; 2. the argument foo clashes with the name of a previously defined function.
Current Version:
v2.18.0

We can eliminate the max case (keyword), but not foo (masking).  The problem is that if we have libraries of functions, we don't want their use or even well-formedness to depend on what calls them.

fixed in stanc3


Don't work with 3 values

The control is not displayed correctly
<SegmentedControl           values={['Abiertas', 'Vencidas', 'Por vencer']}           selectedIndex={state.selectedIndex}           onChange={handleIndexChange}         />

Try reloading your app by pressing 'r' in the development server console. It fixed the issue for me.

I am also getting same issue, the exact scenario was

initially it have 2 segment values, after an action done in the screen
values are increased by 1
then it become like above image

Note: it was fine when pop and push the screen once


update initail API calls for all devices

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] The code follows the code style and conventions of this project.
[ ] The change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.

Description of the change

update initail API calls for all devices

Motivation for the change

fix #625

@maple10001,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Feature request: Make exposed port configurable

This works just great. Thanks a lot! I tried it today and even though I had no experience with docker it worked right away. My production setup includes an nginx reverse proxy, which is listening on port 80 on the same rpi. I thought it should be easy to change the ccu2-docker port to something else but this seems to be more complicated. If you could point me to the right direction, I would love to provide a PR.

@rdrongow: have you tried "-p 8080:80" ?  -p host_port:container_port

@holgerimbery: yes. I think the --net=host is the problem as it exposes the whole network, when I understand it correctly from the docker documentation. (https://docs.docker.com/engine/reference/run/#network-host)

A docker run --name $DOCKER_ID --net=bridge -tid -p 8080:80 -p 2001:2001 --device=${SERIAL_DEVICE}:/dev/mmd_bidcos -v /sys/devices:/sys/devices -v /sys/class/gpio:/sys/class/gpio -v ${DOCKER_CCU2_DATA}:/usr/local ccu2 worked for me. I could provide a pull request and make network mode and port forwarding configurable. Let me know, if you're interested.

The settings such as the exposed port are set at build time: at restart ccu2.service restarts the container with the same settings.
I will add two variables to the build.sh to make easier to change these values. Then you can change them and call build.sh again.
Example: CCU2_REGA_PORT=8080 ./buid.sh


Add sqlite connection helper

Uses rusqlite for lightweight sqlite bindings
Implements custom error to deal with unexpected behavior

I've worked with diesel in the past, but I think it is slightly better to just avoid ORMs all-together.
closes #6

I'm curious to know why you chose not to use an ORM? Are there any advantages to writing raw SQL queries?

I'm curious to know why you chose not to use an ORM? Are there any advantages to writing raw SQL queries?

A number of reasons actually:

I like the declarative nature of sql
An ORM is always a subset of actual sql
Learning all the ins and outs of an ORM is just as much work as learning sql
I like the idea of having your queries stored in separate files where their version history is totally unambiguous

You'd still be using an ORM (you still serialize/deserialize your results), only it is an ORM that you have made yourself. Thanks to serde this is minimal overhead and then your ORM defines all the interactions with the database making your intentions very clear.


website: updagre text size

I've got some reports that font size is too small and not accessible enough, actually yes it is a bit small, I have a big screen and looks really small and headers looks a bit bigger in comparison.

@semoal do you mind if I merge this?

Lgtm


How to remove button "Enter manually"

Hi guys,
ask a question ,How to remove button "Enter manually"?
Thanks.

sorry,i got it. disableManualEntryButtons = true


Improve terminology and operations involving "circuit size"

There are at least two related issues here:

We do not make a clear distinction in code between the number of true gates in a circuit and the next-power-of-2-circuit size. Cody suggested something like dyadic_size or dyadic_circuit_size for the latter. Maybe num_gates or number_of_gates is ok for representing the number of gates before padding. Decide on terminology and make this consistent everywhere.
In the Ultra composers (and maybe elsewhere), we compute the "dyadic circuit size" (often called subgroup_size in this context) multiple times using various vaguely named constants. This is brittle and easily leads to bugs. Let's simplify that logic and ensure that dyadic_size is computed once and for all in each composer using clearly named variables.

This was handled during the early stages of goblin dvelopment. dyadic circuit size is now computed once and for all in the ultra circuit builder


wow.js with animate.css load content late

wow.js not working properly when i load the page for the first time and scroll a little fast the content will not show and i have to wait around 2 second to load the content. i dont know whats the issue every thing is working fine for me when i start the project.

i have the same "problem". i "solved" it by setting offset to -200 - but maybe someone has another hint

I am having the same problem but unsure why. It seems to be hit and miss, some of it loads correctly and some of it gets stuck for a few seconds.

I am noticing the same thing on chrome on Mac.


VS Community 2019 version 16.10.1 - Intellisense is not working

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: Community - 16.10.1
OS Version: Windows 10 Enterprise - Version 2004 - Build 19041.1052

Steps to Reproduce:

Install the visual studio community version 16.10.1
This script is created through Unity and hence the unity engine namespace. Enter the below code

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
public class Testing : MonoBehaviour
{
public Light myLight;
void Awake()
{
    int myVar = AddTwo(9, 2);
    Debug.Log(myVar);
}

// Start is called before the first frame update
void Start()
{

}

// Update is called once per frame
void Update()
{

}

}

The MonoBehaviour is also not color coded.
When I type, AddTwo or Debug.Log it does not suggest anything.

Please report this issue against Visual Studio. This repo is for VS Code


t

Problem description

Problem location

I saw a problem at the following URL:

I couldn't find the information I wanted. I expected to find it near the following URL

Other:

Project version(s) affected

Suggestions for a fix

Thanks for reporting the broken link. Please report this issue on the source page where the link is located.


fix wiki links on readme file

The current links do not work and that leads to 404 pages. This fix leads to correct pages and it will be beneficial for a newcomer to save time finding the deploy docs.

@vaibhavmule Welcome to the Hypha team!


Dashboard page: "major" accessibility issues that may not need design

From https://docs.google.com/spreadsheets/d/1Fs5sY1zgpovWVdHMqpJWRu2Zvyr53jw7, here are "major" accessibility issues for the Dashboard page  that we can fix without design changes.   (Note: Mike addressed accessibility level A and AA problems noted by the SiteImprove, WAVE and Axe browser plugins in PR #3159)
Note that solutions may be suggested in https://docs.google.com/spreadsheets/d/1Fs5sY1zgpovWVdHMqpJWRu2Zvyr53jw7.
These can be separate PRs, and/or split into separate tickets.

[x] The "Cite" button is repeated multiple times: form field labels should be unique. See #3188
[x] When "Request access to another collection" is selected and the tooltips are visible, the controls that open the tooltips are not associated with the tooltip content. See #3179
[x] There are link elements that do not have meaningful text.  Within the table, there is link text that is not unique, such as the "Copy persistent link" links. See #3190
[ ] The error messages appear briefly, but they do not remain visible.  Also in #3171
[ ] There is no visual indication of keyboard focus when using TAB or SHIFT+TAB to navigate through controls of the website.  This includes the "Deposits in progress -- please complete" and "Your collections" buttons and the Cancel button in the Help modal dialog.  NOTE: may be fixed by PR #3179
[ ] When the page is resized and hamburger menu is open, focus moves into the links area but focus is not visible.
[x] Some elements are not well formed.  There are multiple <h5>  instances with an id="content-type-prompt" present (id must be unique??) See #3159.
[x] (Minor level) There are list items that are not direct descendants of list containers (<ol> or <ul>). For the list of collections, there is an element between the <ul> and the <li>.
[x] (Minor level) There are headings that have structural markup that does not accurately communicate the heading level.  The modal dialogs each have <h5> headings instead of <h2>. #See 3332

Each of the above points has been resolved, is in progress in a PR, or ticketed separately, closing.


FIX styling of the Addon Info propTables

Issue: #5278
What I did

[x] Made the existing stub table components into their own component files
[x] Added a stylesheet for the table (this may not be something you want me to, please let me know. I'm very happy to change my approach, especially considering that this is my first contribution to Storybook)
[ ] Update existing snapshots (for some reason unrelated tests are failing blocking my ability to update the snapshots for the time being.

Screenshot:

How to test
Just launch storybook and head to the button component which has a story with the Info addon.

Is this testable with Jest or Chromatic screenshots?
Yes
Does this need a new example in the kitchen sink apps?
No
Does this need an update to the documentation?
No

If your answer is yes to any of these, please make sure to include it in your PR.

@ebrearley thanks for this! I'll take a look at it asap.

Should be fixed

Hi Guys, I have been trying to find instructions on how to style the propTable, however I can't find anythings. withInfo only has the option to style propTableHead. Any ideas? It currently looks terrible.


Spelling mistake: Bluetooth "fine"/finde

Blockly --> Bluetooth --> "Fine" iBeacon mit ID

Fixed but new build is needed to apply the changes


Problem-Error downloading ERTflix videos

hello mr,
i'm trying download this link " https://www.ertflix.gr/xenes-seires/to-mikro-spiti-sto-livadi-s04ep10v-o-machitis-meros-v/?series=to-mikro-spiti-sto-livadi " and i receive this error

the same with this link too, similar error " https://www.ertflix.gr/xenes-seires/to-mikro-spiti-sto-livadi-s04ep11-tha-synantithoyme-sto-panigyri/?series=to-mikro-spiti-sto-livadi "
for previous episodes, working fine, but for these last two it has these errors... i'll try tommorow to download the new ones they will upload and i'll write again.
thank you very much for your work! hope fix it soon ;)

The newest update must have solved any problems with ertflix.

Could probably be an error mention on this issue https://github.com/AngelouD/ert-downloader/issues/17#issuecomment-940064843 fixed with a2f02a08bd2b71b80175cd11a2fc72b5146160f3 ?


Swagger-UI 3.x update?

What are the chances of this getting updated to use Swagger UI 3.x? There are some known issues with response type encoding that should be fixed in newer versions.

Dupilcate

If you are in urgent need try my fork:
https://www.nuget.org/packages/Swagger-Net/


Clean context.TODO in pkg and other minor nits

TODO is for migration, we know we want background there
other nits

/assign mattmoor

L33T! :)


Error out immediately on startup if environment incorrectly configured

@francinelucca can you give this a shot on the latest from the main branch to see if it errors out as expected? I think the changes I integrated on Friday might have fixed this.

Okay, thanks for giving it a whirl. In the interest of keeping the production server running as closely to Next.js defaults as possible, let's close this issue as a "wontfix". The risk is pretty minimal, and we'll see it in the logs pretty quickly if/when it happens.


save session file locally

https://github.com/foundation-ui/glass/commit/fe37a454b9a5157341dab885850c63804534fbe9


: Add Initial Support for TiDB Vector Store

This pull request introduces initial support for the TiDB vector store. The current version is basic, laying the foundation for the vector store integration. While this implementation provides the essential features, we plan to expand and improve the TiDB vector store support with additional enhancements in future updates.
Upcoming Enhancements:

Support for Vector Index Creation: To enhance the efficiency and performance of the vector store.
Support for max marginal relevance search.
Customized Table Structure Support: Recognizing the need for flexibility, we plan for more tailored and efficient data store solutions.

Simple use case exmaple
from typing import List, Tuple
from langchain.docstore.document import Document
from langchain_community.vectorstores import TiDBVectorStore
from langchain_openai import OpenAIEmbeddings

db = TiDBVectorStore.from_texts(
    embedding=embeddings,
    texts=['Andrew like eating oranges', 'Alexandra is from England', 'Ketanji Brown Jackson is a judge'],
    table_name="tidb_vector_langchain",
    connection_string=tidb_connection_url,
    distance_strategy="cosine",
)

query = "Can you tell me about Alexandra?"
docs_with_score: List[Tuple[Document, float]] = db.similarity_search_with_score(query)
for doc, score in docs_with_score:
    print("-" * 80)
    print("Score: ", score)
    print(doc.page_content)
    print("-" * 80)

@IANTHEREAL feel free to ping when the PR is ready for review

@baskaryan The TiDB Vector feature is expected to release a beta version around the end of January. I will make the final adjustments then and invite you to review it 😊

@hwchase17 @baskaryan
Quick update on the TiDB Vector Store PR:

Development Complete: The core functionality of TiDB Vector Store is developed and now in internal testing.
Staging in February: A stable staging environment will be ready by February for thorough review.
Launch & Documentation in March: Planning for a March release with comprehensive documentation, including Langchain + TiDB Vector use cases.

Feel free to reach out with any questions or concerns. Excited to see Langchain and TiDB Vector Store in action!

Recently, there have been more and more code conflicts. If possible, please help review this PR.  @baskaryan @hwchase17
And I've DM'd the access method for the tidb vector staging environment to @baskaryan, hope it's useful for this PR reviewing

Thanks for reviewing @baskaryan


Fix/heading visibility

Related Issue
#1572
Fixes:  #1572
Description
This fixed the visibility issue of the headings including Testimonials and destination guide along with this it enhanced the destination guide input text area with visually appealing colors.
#1572
Type of PR

[x] Bug fix
[x] Feature enhancement

Screenshots

Checklist:

[x] I have made this change from my own.
[ ] I have taken help from some online resources.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my code.
[x] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

@swati-204 Your pull request has been successfully merged! Don't stop here—continue contributing by addressing more issues and aiming to become a top contributor. Exclusive swag awaits for those who go the extra mile!
Great contribution ! 🎉


Remove navigate to top in the home component

Is your feature request related to a problem? Please describe.

Remove this in new-board component.

Hey, if the issue's still open can I work on this issue?

hello @sjana0,
yes sure! Please go ahead!

Hello @sjana0,
Are you working on this issue?


Frontend employee management

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.28%. Comparing base (71be5bd) to head (d8dbae1).

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #99      +/-   ##
==========================================
- Coverage   96.29%   96.28%   -0.02%
==========================================
  Files          66       66
  Lines        1700     1695       -5
  Branches      161      160       -1
==========================================
- Hits         1637     1632       -5
  Misses         62       62
  Partials        1        1

Flag
Coverage Δ

frontend-customer
96.56% <ø> (ø)

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

Files with missing lines
Coverage Δ

backend/src/employees/employees.module.ts
100.00% <ø> (ø)

backend/src/employees/employees.service.ts
100.00% <ø> (ø)


Tech Elevator Code School Addition

Hi team -
This pull request is for changes to the config/code_schools.yml. We added two Tech Elevator locations (Cleveland, Ohio and Columbus, Ohio).

Hi @jtucholski - thanks for the PR. You should be live momentarily.


Temporarily Remove validation steps

This is a temporary step to skip validation on-prem while versioning of transformed data is resolved.

It could maybe have used a comment in the source code, unless we are going to return to this soon?


Correction for raw affiliation CNRS, UMR 5239, Lyon, France; Department of Cytology, Hospices Civils de Lyon, Lyon, France

Correction needed for raw affiliation CNRS, UMR 5239, Lyon, France; Department of Cytology, Hospices Civils de Lyon, Lyon, France
raw_affiliation_name: CNRS, UMR 5239, Lyon, France; Department of Cytology, Hospices Civils de Lyon, Lyon, France
new_rors: 029brtt94;02feahw73;01502ca60;01bj4fd12
previous_rors: 029brtt94;02feahw73;01502ca60
works_examples: W2523346005
contact: 91a8e8536e1db2cff6053c5036ef702d:11ca76cf4440a4cd06e11c9ad4c1 @ ens-lyon.fr

This issue was accepted and ingested by the OpenAlex team on 2024-11-01. The new affiliations should be visible within the next 7 days.


What about Lightning network developers?

Can we add ourselves?

Sure, everyone contributing to the ecosystem at large is welcome. In the same way designers, educators, etc. are invited to submit PRs.


Compiler errors when enabling tls features

How to reproduce the problem:
https://github.com/gngpp/reqwest-impersonate/blob/67ce1ec25cdb93b99f1b5c2d8ba4547c4977de67/Cargo.toml#L24-L25
Add default-tls or native-tls or rustls-tls to default:
[features]
default = ["boring-tls", "impersonate", "rustls-tls"]

This library is only suitable for boring tls.

To use other tls, you need to turn off the default features. And the compilation may fail. If you have time to fix it, please pr


Added support of event listeners for blocks

For each block, that converts to HTML, can be added an event listener - callback function with $block array as an argument. The one-time listeners are also supported.
Examples are available in modified README.md.

This is pretty useful, but why just not add node visitor?
class MyTableProcessor extends NodeVisitor
{

    public function leaveTable(array $block) {
        $item = 0;

        // thead and tbody.
        foreach ($block['element']['text'] as &$section) {
             // tr.
             foreach ($section['text'] as &$rows) {
                  $rows['attributes']['class'] = $item++ % 2 ? 'odd' : 'even';
             }
        }
    }
}

or at least add visitTable/leaveTable events.

@BR0kEN-
It'd be helpful if you could briefly explain the problem that this solves, thanks.

@fesor, what do you mean? As I know that library have no objects of such type. Please explain your idea.
@erusev, please read the updated README.md from my fork for understanding what event listeners does.

@BR0kEN- it is true, without objects for nodes or event without intermediate structure node visitor is almost useless. But event listeners also. Anyway sometimes it will be useful to know that parser get some element and leave it. For example when parsing lists, i want to know when parser visit new list level and when it completes one.

@fesor, I've made a thing that's useful for me and, maybe, another developers. And you're talking about another feature, not related to this. So, I'm not against of implementation, but not in this task.

@BR0kEN-
I'm more interested in specific scenarios that would benefit from the feature.

@erusev, for example, you able to wrap any block with additional HTML or change it attributes or add some styles and javascript if some block present or, if necessary, modify content.

@BR0kEN-
Isn't this easy enough to do through extensions?
(see wiki for more info on extensions)

@erusev, of course, but don't you say that creating additional objects for that is a good way?

@BR0kEN- if you need to extend functionality for one single case, extensions is what should be used.
Events system will be very useful for some cases, but without visit/leave or start/complete events, which will allows you to track nested elements, is isn't cover most of possible use cases.

@erusev @aidantwoods Any chance to get this merged?


Update sql-server-linux-configure-msdtc.md

Statement of support for SQL 2017 had SQL 2019 preview also incorrectly mentioned. Removed that reference.

@tejasaks : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@VanMSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks @tejasaks! Sorry for the delay. I made a minor edit.
#sign-off


Error downloading Pyhon

I'm submitting a…
Bug report (I searched for similar issues and did not find one)
The Run command doesn't work , and Save model as well , when trying to download Python in Convert tab  this error occurs
reparing metadata (setup.py): started
  Preparing metadata (setup.py): finished with status 'error'
  ERROR: Command errored out with exit status 1:
    from termcolor import VERSION
  ModuleNotFoundError: No module named 'termcolor'
  ----------------------------------------
WARNING: Discarding https://files.pythonhosted.org/packages/termcolor-1.1.0.tar.gz# (from https://pypi.org/simple/termcolor/). Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.
ERROR: Could not find a version that satisfies the requirement termcolor>=1.1.0 (from tensorflow) (from versions: 0.1, 0.1.1, 0.1.2, 0.2, 1.0.0, 1.0.1, 1.1.0)
ERROR: No matching distribution found for **termcolor**>=1.1.0
INFO: pip is looking at multiple versions of h5py,coremltools,winmltools - to determine which version is compatible with other requirements. This could take a while.
Error: Child process C:\Users\hi\AppData\Local\WinMLDashboard\python\python.exe exited with code 1
    at ChildProcess.<anonymous> (file:///C:/Users/hi/AppData/Local/WinMLDashboard/app-0.7.0/resources/app/build/static/js/main.5085b59a.js:1:264362)
    at ChildProcess.emit (events.js:182:13)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:237:12): Child process C:\Users\hi\AppData\Local\WinMLDashboard\python\python.exe exited with code 1

Minimal reproduction of the problem with instructions
I've Python 3.7 Python 3.9 already installed
when using a model then in convert tap , download a new python binary for WInML Dashboard .
Environment
Windows Build Number: 19042
OS Version : Win10
WinMLTools specific:
- Source training framework: (e.g. CoreML, Scikit-learn, …)(TF converted to k5 and other binary model  using Google teachable machine
- WinMLTools version: 0.7.0
Visual Studio

[x] 2019 LTS
[x] 2021 Preview

Error  when loading Keras model foreach loop isn't a function

Thanks for posting, we're looking into this issue.