Tingle.Extensions.Caching.MongoDB throws sporadically "Collection already exists" when using "CreateIfNotExists = true"

I tested Tingle.Extensions.Caching.MongoDB and when scaling up alot of services instances i get sporadically the error below:
MongoDB.Driver.MongoCommandException: Command create failed: Collection already exists. NS: TestCacheDatabase.TestCacheCollection.
   at MongoDB.Driver.Core.WireProtocol.CommandUsingCommandMessageWireProtocol`1.ProcessResponse(ConnectionId connectionId, CommandMessage responseMessage)
   at MongoDB.Driver.Core.WireProtocol.CommandUsingCommandMessageWireProtocol`1.ExecuteAsync(IConnection connection, CancellationToken cancellationToken)
   at MongoDB.Driver.Core.Servers.Server.ServerChannel.ExecuteProtocolAsync[TResult](IWireProtocol`1 protocol, ICoreSession session, CancellationToken cancellationToken)
   at MongoDB.Driver.Core.Operations.CommandOperationBase`1.ExecuteProtocolAsync(IChannelSource channelSource, ICoreSessionHandle session, ReadPreference readPreference, CancellationToken cancellationToken)
   at MongoDB.Driver.Core.Operations.WriteCommandOperation`1.ExecuteAsync(IWriteBinding binding, CancellationToken cancellationToken)
   at MongoDB.Driver.Core.Operations.CreateCollectionOperation.ExecuteAsync(IWriteBinding binding, CancellationToken cancellationToken)
   at MongoDB.Driver.OperationExecutor.ExecuteWriteOperationAsync[TResult](IWriteBinding binding, IWriteOperation`1 operation, CancellationToken cancellationToken)
   at MongoDB.Driver.MongoDatabaseImpl.ExecuteWriteOperationAsync[T](IClientSessionHandle session, IWriteOperation`1 operation, CancellationToken cancellationToken)
   at MongoDB.Driver.MongoDatabaseImpl.CreateCollectionAsync(IClientSessionHandle session, String name, CreateCollectionOptions options, CancellationToken cancellationToken)
   at MongoDB.Driver.MongoDatabaseImpl.UsingImplicitSessionAsync(Func`2 funcAsync, CancellationToken cancellationToken)
   at Tingle.Extensions.Caching.MongoDB.MongoCache.MongoCollectionInitializeAsync()
   at Tingle.Extensions.Caching.MongoDB.MongoCache.ConnectAsync(CancellationToken token)
   at Tingle.Extensions.Caching.MongoDB.MongoCache.GetAsync(String key, CancellationToken token)
   at Microsoft.Extensions.Caching.Distributed.DistributedCacheExtensions.GetStringAsync(IDistributedCache cache, String key, CancellationToken token)

When using "CreateIfNotExists = true" i would assume that such erros would not be thrown. It complains that the collection already exists. Which should be still valid (no error) if "CreateIfNotExists = true" is set.
Using:
MongoDB.Driver: 2.19.1
Tingle.Extensions.Caching.MongoDB: 4.1
Net 7
Linux, Docker and Kubernetes

Is this something that started happening in version 4.1.0 (released on 22 May, 2023)?

I just started with 4.1.0 and have no experience with prior releases.

Could you possibly be configuring the cache twice in your setup resulting the options being overridden?

When i restart my service it works, but i am confused how this should have an effect.

Could you try and reproduce with minimal code, maybe offer a public repro of the same?

Thats a little bit hard. You need a cold start with alot of services to achieve this. We have several test clusters where we start cold from scratch with several dozen of containers in parallel. Thats the scenario where its happening.
What i propose is catching this explicit exception and skipping it when "CreateIfNotExists = true" is on.

If I get you right, you are starting lots of replicas at once targeting the same database and collection but the collection is not created prior?

@ThomasTosik this should be fixed in 4.1.1. Could you give it a try?
NuGet: https://www.nuget.org/packages/Tingle.Extensions.Caching.MongoDB/4.1.1


Update App Developer flow

Discussion around removing the difficulty for an App Developer to follow the docs flows for various App Developer tasks.

One quicker (agile) fix is to add a new role-based page for each fabric role, starting with app dev. role. ... This new page would define the experience and prereqs etc. of an app dev. - and then link to the existing docs for each "step" in their workflows. Defining each app dev task or workflow that we document would be a first step.

One sort of quick improvement could be to separate the 1) chaincode application dev from the 2) app dev that calls the chaincode. ... e.g. The long tutorial for c.p. below could assume the chaincode already exists, and we separate the chaincode pars out accordingly, with cross links. https://hyperledger-fabric.readthedocs.io/en/release-2.4/tutorial/commercial_paper.html
i.e. These could become 2 separate roles - chaincode dev. and app dev. ... (Whether or not these are often the same person or not we don't need to get into.)

https://hyperledger-fabric.readthedocs.io/en/release-2.4/developapps/scenario.html - fix "corporating" typo - corporation.
In What is c.p.? section we could say in plain English that commercial paper is just an IOU right. I'll (corp.) pay you $100 on maturity date of IOU, but you (lender) only has to pay me $90 for the IOU today.

https://hyperledger-fabric.readthedocs.io/en/release-2.4/developapps/scenario.html - fix "corporating" typo - corporation.

In What is c.p.? section we could say in plain English that commercial paper is just an IOU right. I'll (corp.) pay you $100 on maturity date of IOU, but you (lender) only has to pay me $90 for the IOU today.

Analysis page
Seems odd that paper 00001 is not described as an "asset" on blockchain. This is the definition of a network asset right.

didn't intend to close, should remain open.

Process and Data Design page

Next page. Smart Contract Processing. https://hyperledger-fabric.readthedocs.io/en/release-2.4/developapps/smartcontract.html
We'll see if the title matches what is described. ...

Next page Application -- https://hyperledger-fabric.readthedocs.io/en/release-2.4/developapps/application.html

Next page Application design elements -- https://hyperledger-fabric.readthedocs.io/en/release-2.4/developapps/designelements.html

Plan is to update this section for gateway service.
To keep c.p. example or use asset transfer example instead is tba.

Developing Applications topic has been deleted from the latest doc, closing this issue.


[UNDERTOW-1885] remove escape char from delimitered strings

Issue: https://issues.redhat.com/browse/UNDERTOW-1885

Which one?

I'm marking this one as under verification for now. We discussed internally the impossibility of adding a test, but I want to revisit this issue carefully before merging it.

Thanks @baranowb !

After discussion with @baranowb and @rmartinc I reverted fix commit from this PR and left test commit merged in as https://issues.redhat.com/browse/UNDERTOW-2163 fixed this issue already.


Fix Werewolf agility XP given in skill calculator

Looks like the XP for the Penguin course was copied to the Werewolf course.

I just made an account to fix this myself. Jagex actually doubled the exp for handing in the stick at the end of the course back on July 12th. Prior to that, the course was worth 540 total. Now it's 730, as the stick doubled from 190exp > 380exp.


[Doc] RLMObject.init(forPrimaryKey:String) is not in the documentation

The init method on RLMObject that takes the primary key is not documented anywhere except in some release notes.
http://realm.io/docs/cocoa/0.81.0/api/Classes/RLMObject.html  <-- offending page
This wouldn't be so bad, except you can't browse the class definitions in Swift. It just comes up as a blank page (Xcode bug?)

Hi @voidref, you can't find +[RLMObject objectForPrimaryKey:] because you're looking at outdated documentation from ~8 months ago. Here are the objectForPrimaryKey: and objectInRealm:forPrimaryKey: docs from the latest release (0.91.0).
We'll soon be removing old docs from search engine indexing and displaying a banner on older documentation so this should be less confusing in the near future.

Thanks, I could have sworn I had gotten to that doc page via realm.io, but I was clearly mistaken. Sorry about that!


Repo is empty.

The Repo is empty, like maybe you forgot to commit? or?... unless??

Repo is not empty the .gitignore and LICENCE.md clearly exists at the root.
Closed due to invalid issue.


Dataset

This data set link download is invalid.
Could you please give me a new download link

train_aug set can be downloaded at here

Thank you very much


GitHub repo url could have a trailing slash

A GitHub repository URL is assumed to have

either no trailing slash
or .git at the end

GitHub also allows repo URL to contain a trailing slash (example: https://github.com/owners/repo/).

Double slash in the repo URL causes HTTP request to fail with 404 error.

Potential fix for this issue would be to ensure repo URL doesn't end with a trailing slash when UrlConverter is executed, similar to .git removal logic.

I can send a PR, but not sure why the tests here are disabled.


Fix PipelineConnector to duplicate the events

Description
This fix resolves the issue reported in the bug https://github.com/opensearch-project/data-prepper/issues/1886. This bug is about some pipeline processors not duplicating the events before modifying them. And that was causing wrong behavior when a pipeline is split into two parallel pipelines.
When pipeline1 has two sinks pipeline2 and pipeline3 and pipeline2 is just doing grok match and pipeline3 is doing string substitution, both the pipelines will see the string substitution because same event is used in both cases.
Fix is to make a copy of the event whenever an event is modified.
Added a new end-to-end test case that was failing without the fix and verified that the test case is passing with the fix.
Issues Resolved
Resolves #1886
Check List

[ ] New functionality includes testing.
[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@kkondaka , I believe this is almost ready.
You do have some unused imports, which our checkstyle rules prohibit. See the logs here for guidance: https://github.com/opensearch-project/data-prepper/actions/runs/3230696086/jobs/5300610638
Also, we should run that new end-to-end test. I commented above about how to make it run in GitHub. With that change, we should be able to verify it before merging.
Thanks!

Codecov Report

Merging #1897 (9de5ebc) into main (9f54929) will decrease coverage by 0.27%.
The diff coverage is 82.60%.

:exclamation: Current head 9de5ebc differs from pull request most recent head a406ad2. Consider uploading reports for the commit a406ad2 to get more accurate results

@@             Coverage Diff              @@
##               main    #1897      +/-   ##
============================================
- Coverage     93.89%   93.62%   -0.28%
+ Complexity     1551     1549       -2
============================================
  Files           198      198
  Lines          4589     4611      +22
  Branches        372      374       +2
============================================
+ Hits           4309     4317       +8
- Misses          197      212      +15
+ Partials         83       82       -1

Impacted Files
Coverage Δ

...search/dataprepper/pipeline/PipelineConnector.java
89.36% <80.00%> (-6.94%)
:arrow_down:

...pensearch/dataprepper/model/trace/JacksonSpan.java
98.29% <100.00%> (+0.02%)
:arrow_up:

...opensearch/dataprepper/pipeline/ProcessWorker.java
71.15% <0.00%> (-19.24%)
:arrow_down:

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


Create a Makefile

Create Makefile in the root directory

https://github.com/koki-algebra/image-super-resolution-batch/pull/11


Log status when we fail to init leveldb

Mirrors are crashing at this point, collecting more info.

Obsoleted by #1308.


Improve "mainnet-fixture"'s set of finality signatures in testing

Our current "mainnet data testnet" does not have a realistic set of finality signatures across its data. Tooling needs to be employed to seed it with more realistic conditions.

SME @Fraser999


[SSE Sidecar] Create switch block filtering

Step events indicate a switch block has been processed, which in turn, indicates the end of an era.   Allow the application of a filter to pass through or exclude step events from the sidecar endpoint.
@devendran-m commented on Tue Aug 16 2022
Create switch block filtering

@caspersteve commented on Sun Oct 30 2022
Related to Checkpointing v1.0 and Explorer v1.1

Hi @sacherjj , do you know if this repo still exists please?

https://github.com/CasperLabs/product/

Jakub's recommendation is to close this as the details on this ticket are insufficient. Should this filter is required, then a new request with detailed specification is necessary.

Closing based on @zajko 's recommendations.


[UPSTREAM] Fetch secret generator controller

Description
Fetch secret generator controller from upstream.
How Has This Been Tested?
Install RHODS using the following image:
./setup.sh quay.io/modh/rhods-operator-live-catalog:1.19.0-secretgen-controller

Create the following secret:
cat <<EOF | oc apply -n redhat-ods-applications -f -
apiVersion: v1
kind: Secret
metadata:
  name: dashboard-oauth-client
  annotations:
    secret-generator.opendatahub.io/name: secret
    secret-generator.opendatahub.io/type: random
    secret-generator.opendatahub.io/complexity: "32"
    secret-generator.opendatahub.io/oauth-client-route: rhods-dashboard
type: Opaque
EOF

Verify the oauthclient object is successfully created:
$ oc get oauthclient dashboard-oauth-client -o yaml
apiVersion: oauth.openshift.io/v1
grantMethod: auto
kind: OAuthClient
metadata:
  creationTimestamp: "2022-10-25T11:16:07Z"
  name: dashboard-oauth-client
  resourceVersion: "72656465"
  uid: 75200104-72e7-4cf7-9e5d-56579eeb14ef
redirectURIs:
- https://rhods-dashboard-redhat-ods-applications.apps.samuvl.dev.datahub.redhat.com
secret: Mx2jeVoOShtuuhUFqyoWTz6iVyrlBttg

@cfchase Could you approve and merge this PR?


Python formatters are not respecting selected environment

Type: Bug

Start a new project by creating some folders. At least two level of depth are required e.g. my-project/python/src .
Create a new python project in a deep directory (see 1.).
Enable various tools in the extension settings such as mypy, autopep8, isort, etc.
Set up a virtual environment (virtualenv preferred for reproduction) and enable it in the IDE context via ">Python: Select Interpreter" if not automatically recognized.
Install the tools enabled in the extension settings via terminal (be sure to source the virtual environment there!) via pip. E.g. python -m pip install mypy autopep8 isort.
Create a python file (test.py) and write some code there.
Use one of the tools, namely autopep8 or isort.
VS Code should create a popup which asks to install the tool globally.

Expected behavior: No popup and formatter does its thing.
Additional information:
In my observation i could confirm this behavior for autopep8 and isort. mypy seems to work after reloading the project. autopep8 might work, however, it could also be that VS Code uses a fallback formatter. Isort will never work through the IDE except when using the terminal to trigger it.
There are way more tools such as bandit but i have not tried those not mentioned in the report.
Also i believe that the behavior in the past was that all tools would be working when choosing an interpreter from a virtual environment.
My current assumption is that this might happen because my project is a multi programming language project and the python source is not in the project root.
Extension version: 2023.10.1
VS Code version: Code 1.79.1 (4cb974a7aed77a74c7813bdccd99ee0d04901215, 2023-06-12T16:14:05.102Z)
OS version: Windows_NT x64 10.0.19045
Modes:
Remote OS version: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

System Info

Item
Value

CPUs
Intel(R) Xeon(R) CPU E3-1240 v3 @ 3.40GHz (8 x 3392)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: unavailable_offvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
23.94GB (11.61GB free)

Process Argv
--crash-reporter-id cee6d7dc-4f4d-45ae-bf8a-a639212f4740

Screen Reader
no

VM
0%

Item
Value

Remote
WSL: Ubuntu

OS
Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

CPUs
Intel(R) Xeon(R) CPU E3-1240 v3 @ 3.40GHz (8 x 3392)

Memory (System)
11.65GB (9.26GB free)

VM
0%

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30410667
vscaac:30438847
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
3biah626:30602489
pyind779:30671433
89544117:30613380
pythonsymbol12:30671437
2i9eh265:30646982
showlangstatbar:30737416
vsctsb:30748421
pythonms35:30701012
03d35959:30757346
pythonfmttext:30731395
pythoncmvfstrcf:30756944
fixshowwlkth:30730052
pythongtdpath:30739704
dh2dc718:30763024
pythonnosm12tcf:30765602

@maunzCache For several of these tools we have their own extensions:
https://marketplace.visualstudio.com/items?itemName=ms-python.autopep8
https://marketplace.visualstudio.com/items?itemName=ms-python.black-formatter
https://marketplace.visualstudio.com/items?itemName=ms-python.isort
https://marketplace.visualstudio.com/items?itemName=ms-python.pylint
https://marketplace.visualstudio.com/items?itemName=ms-python.flake8
https://marketplace.visualstudio.com/items?itemName=ms-python.mypy-type-checker
We don't show the install prompt for several of these tools anymore, we actually show a prompt to install the above extensions. If you had previously dismissed the prompt you won't see the extension recommendation prompt. But there should be some info in the logs indicating this. Please check the logs.

Ok that is good to know. It seems that i did not properly read the new message because the popup just looked like the previous warning. I'll try those extensions then.
And to be honest i am not sure which logs to crawl through. I tried the output for the Python extension but there was no hint for the isort extension. But i did notice that my formatting is provided by pycodestyle instead of autopep8 - which is fine. Just too many formatters installed here.


[ansible/artifactory] nginx installation options

Is this a request for help?:

Is this a BUG REPORT or FEATURE REQUEST? (choose one):
FEATURE_REQUEST

Currently both roles artifactory_nginx and artifactory_nginx_ssl are installing and configuring nginx.
The current installation has some downsides:

it's installed with "state: present" and will end up in any old version of the package (e.g. "1.18.0" on Ubuntu 20.04), even that there are newer versions available.
there is no possibility to define a version of the package (or a different package source)
it's not possible to skip the nginx installation completely, in case that you did use a different role to install a custom/newer nginx but want to use artifactory_nginx_ssl to configure it

if you would split up installation and configuration, or at least make them both switchable via boolean variable, that would be great.
Which installer:
ansible
Which product and version (eg: ansible & collection version - 7.24.2):
collection latest version / master branch
Which operating system and version(eg: ubuntu & version - 20.4):
Ubuntu 20.04, but all are affected
Which product license (Enterprise/Pro):
Enterprise

I guess you can roll your own Nginx when you set these, but I didn't test it.
artifactory_nginx_enabled: false
artifactory_nginx_ssl_enabled: false

In production environments, you should set state to present and specify a target version to ensure that packages are installed to a planned and tested version.
Setting state to latest not only installs software, it performs an update and installs additional packages. This can result in performance degradation or loss of service. If you do want to update packages to the latest version, you should also set the update_only or only_upgrade parameter to true based on package manager to avoid installing additional packages.
Installing Artifactory on-premises is a bit hard given the requirements of the postgres and nginx roles in the JFrog platform collection. These rely on postgresql.org and nginx.org, which in our case would need to be allowed in our proxy. Recent versions of RHEL/Rocky/AlmaLinux have NGinx and Postgresql in their appstream, you only need to enable them.

Proxy allow list

galaxy.ansible.com
ansible-galaxy-ng.s3.dualstack.us-east-1.amazonaws.com
nginx.org
download.postgresql.org
releases.jfrog.io
releases-cdn.jfrog.io


TMT integration tests for polarion support

From the failing tests it seems the plans/create_pylero_cfg.sh file has not been included in the commit.

From the failing tests it seems the plans/create_pylero_cfg.sh file has not been included in the commit.

Ah thanks, copied locally, so that I've verifoied that tests works well, but forgot to git add :-)
But currently I'm testing rpm installed TMT version, not sure how it works, if we add remote plan into tmt if there will stay rpm version or locally installed tmt will win @psss ?

But currently I'm testing rpm installed TMT version, not sure how it works, if we add remote plan into tmt if there will stay rpm version or locally installed tmt will win @psss ?

When running here the latest released tmt packages will be tested, when running the remote plan from the tmt repo, the fresh stuff from the git repo will be verified. I think this should be working relatively fine if there not too many pull requests open in the tests repo.

Hm, tests passed but the test output seems to be empty. Any idea what might be wrong?

Hm, tests passed but the test output seems to be empty. Any idea what might be wrong?

Forgot to add .fmf/version file into test data


Feature Request: 2D to 3D Conversion Matrices

Background And Motivation
Our team is currently facing an issue where we are unable to match object mask data to object metadata.
More robustly, at the Oracle level, the step metadata object contains a ground truth object mask image, as well as lists of structural and non-structural objects, with each object's metadata in 3d world coordinates. Currently there are no built in means to relate an object appearing in the mask image to it's corresponding metadata in the object lists. Teams can attempt to compute projection matrices that translate objects in the masks to 3d world coordinates and perform matching operations on these translated representations, but this injects one more point of failure into perception systems.
Solutions
Issue #240 presents one solution to this problem, a color map in StepMetadata that contains object ID string keys and color pixel array values, matching objects to their 2d and 3d representations. This is the most straightforward solution, and is preferred for our teams training needs. We suspect other teams would benefit from this information as well.
Another solution would involve giving teams direct access to the Unity computed translation matrices. Teams could transform 3d object representations into 2d image coordinates with the Camera.worldToCameraMatrix object, or transform 2d map image coordinates into 3d object representations with Camera.cameraToWorldMatrix.
With the latter solution, StepMetadata.worldToCameraMatrix and StepMetadata.cameraToWorldMatrix would contain 4x4  matrices in the OpenGL convention. The OpenGL and Unity conventions have one simple distinction:

Note that camera space matches OpenGL convention: camera's forward is the negative Z axis. This is different from Unity's convention, where forward is the positive Z axis.

This would either need to me made apparent to teams, or the TA2 team would have to convert the matrices into the Unity convention.

Thanks for the post @zshureih . Currently, while running with oracle metadata, our API returns a color property in each object's metadata containing the RGB values used for that object in the object masks:
https://github.com/NextCenturyCorporation/MCS/blob/master/machine_common_sense/API.md#objectmetadata
Issue https://github.com/NextCenturyCorporation/MCS/issues/240 doesn't add any new information, it just moves the existing information from the object's metadata up to the step's metadata (as requested by the CORA team).
Would you be able to use the color property for the moment? Or am I misunderstanding your request?

We would be able to use color for the moment, yes.
Thank you Thomas! I guess I made a false assumption about the color property here, relating it to the rgb image rather than the object masks.


feat: Allow custom client location

This breaks the type & runtime dependency on a hardcoded
@prisma/client client output location, allowing
custom location (eg when having multiple clients).
This adds a layer of runtime safety to validate the DMMF,
which can now be passed in the configuration,
allowing all sorts of runtime hacks.
See #18 & #19.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

20 of 23 (86.96%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 80.634%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/encryption.ts
9
12
75.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
140

Relevant Lines:
169

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
16 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.2%) to 80.634%

Files with Coverage Reduction
New Missed Lines
%

src/index.ts
1
78.95%

src/dmmf.ts
5
86.87%

src/encryption.ts
10
67.5%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
140

Relevant Lines:
169

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
16 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.2%) to 80.634%

Files with Coverage Reduction
New Missed Lines
%

src/index.ts
1
78.95%

src/dmmf.ts
5
86.87%

src/encryption.ts
10
67.5%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
140

Relevant Lines:
169

💛  - Coveralls


cannot upload and unzip large zip

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az webapp deployment source config-zip
Errors:
Traceback (most recent call last):
Local\Temp\pip-unpacked-wheel-taryuzhh\knack\cli.py, ln 233, in invoke
azure\cli\core\commands\__init__.py, ln 660, in execute
azure\cli\core\commands\__init__.py, ln 723, in _run_jobs_serially
azure\cli\core\commands\__init__.py, ln 716, in _run_job
AppData\Local\Temp\pip-unpacked-wheel-l7ikczg1\six.py, ln 703, in reraise
azure\cli\core\commands\__init__.py, ln 694, in _run_job
azure\cli\core\commands\__init__.py, ln 331, in __call__
pip-unpacked-wheel-sc_lkrar\azure\cli\core\__init__.py, ln 808, in default_command_handler
azure\cli\command_modules\appservice\custom.py, ln 395, in enable_zip_deploy_webapp
azure\cli\command_modules\appservice\custom.py, ln 422, in enable_zip_deploy
MemoryError

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az webapp deployment source config-zip -g {} -n {} --src {}

Expected Behavior
Environment Summary
Windows-10-10.0.19041-SP0
Python 3.6.8
Installer: MSI

azure-cli 2.20.0

Additional Context

route to appropriate team

@NON2057 We sincerely apologies for the delay. Please upgrade to latest version of az cli 2.39.0 and retry.


Usernames can be reserved URLs

From the Update my portfolio page, users can request certain usernames that create a conflicting path to public profile.
For example, I was able to request the username account, even though the resulting path to public profile conflicts with http://www.freecodecamp.com/account (the page used to update or manage an account).
Similarly, I was able to request the username challenges, which conflicts with http://www.freecodecamp.com/challenges, and breaks the user's avatar link.
Known usernames I've found that conflict with certain URLs:

Username account conflicts with http://www.freecodecamp.com/account
Username challenges conflicts with http://www.freecodecamp.com/challenges
Username signout conflicts with http://www.freecodecamp.com/signout

This could be fixed by reserving usernames that match directory names.

Hi @strman this issue has gotten fixed in staging so hopefully no more funny accounts!


The test:all script runs on 0.13, 0.14 and 15

LGTM, thanks @bettiolo

Awesome :-) it'd be nice to rebase but otherwise LGTM

@ljharb do I need to do anything?

@bettiolo if you don't mind using a local git client and rebasing these down to 1 commit, that'd be great - if it's a problem, nbd tho

Done ;)


form.getFields() should handle widget field without an appearance.

What were you trying to do?
I have a pdf with a widget without an appearance set. When calling form.getFields() which gets the following exception:
Error: Expected instance of PDFName, but got instance of undefined.
I'm attaching the sample file I'm using.
f1040 filled (4).pdf
Why were you trying to do this?
I need to enumerate all fields.
How did you attempt to do it?
Just called the api getfields();
What actually happened?
I get an exception: Error: Expected instance of PDFName, but got instance of undefined
File: PDFContext..js Function PDFContext.prototype.lookup throws the exception
throw new UnexpectedObjectTypeError(types, result);
What did you expect to happen?
Get the list of fields including the widget without the appearance.
How can we reproduce the issue?
Load the attached file into the library
Get form and call form.getFields();
f1040 filled (4).pdf
Version
https://cdn.jsdelivr.net/npm/pdf-lib/dist/pdf-lib.js
What environment are you running pdf-lib in?
Browser
Required Reading

[X] I have read www.sscce.org.
[X] My report includes a Short, Self Contained, Correct (Compilable) Example.
[X] I have read Smart Questions.
[X] I have read 45 GitHub Issues Dos and Don'ts.

Additional Notes
No response

This report does not include complete, executable code.
Despite this, I tried to reproduce it anyways and was unable to.
Here is an example of a proper, reproducible bug report: https://github.com/Hopding/pdf-lib/issues/1001

hello
I have exactly the same problem
I made a small zip with enough to reproduce the problem
pdf-bug.zip


Add Projects

Description
Adds Projects (groupings of ESC environments) to the ESC CLI
Testing
setup: personal/default org = syeh2. other orgs = ambiguous
Init
❯ go run ./cmd/esc env init env1
Environment created: syeh2/default/env1

❯ go run ./cmd/esc env init new-project/env1
Environment created: syeh2/new-project/env1

# Legacy pattern (org/env) because I have an org called `ambiguous`
❯ go run ./cmd/esc env init ambiguous/myenv
Environment created: ambiguous/default/myenv

# Test creating a project with the same name as an org
❯ go run ./cmd/esc env init syeh2/ambiguous/env1
Environment created: syeh2/ambiguous/env1

# Because now I have an existing env under the `ambiguous` project, I can create a new env under the same project with just `ambiguous/env2`
❯ go run ./cmd/esc env init ambiguous/env2
Environment created: syeh2/ambiguous/env2

Get
Setup: Created 'syeh2/ambiguous/myenv' and 'ambiguous/default/myenv` envs beforehand
❯ go run ./cmd/esc env get ambiguous/myenv --value json
Error: ambiguous path provided

Environments found at both 'syeh2/ambiguous/myenv' and 'ambiguous/default/myenv'.
Please specify the full path as <org-name>/<project-name>/<env-name>
exit status 1

❯ go run ./cmd/esc env get syeh2/ambiguous/myenv --value json
{
...
}

❯ go run ./cmd/esc env get ambiguous/default/myenv --value json
{
...

Please confirm that this is what you'd like to do by typing `syeh2/new-project/env1`: syeh2/new-project/env1
Environment "syeh2/new-project/env1" has been removed!


Make game responsive

At the moment the game is not playable on devices without a keyboard and only works on large resolution displays.
Making it responsive and playable on all devices includes:

[x] rescaling on viewport/size change
adding inputs for touch devices:

[x] jumping the cat on swiping up
[ ] adding buttons/areas for loading time-travel batteries and traveling

[ ] scaling down sprites for small resolution displays

Yay. With the latest commits in #27 The full scene is now smoothly resizeable, to whatever resolution or screen size you have 😃.
I took an HD reference resolution (1080x1920) which I used for setting the default proportions and scaled based on the height. Changing the screen width will just infinitely scale the landscape and alter the position of the cat.


New PR to test SignCLA at 2019-07-09 11:49:30.360361759 -0700 PDT m=+3.254931003

Body of PR

indiv-pass-fork-staging: test succeeded


Topic recognition - S4745193 Stable Diffusion on ADNI

Hey Teaching Team,
This is a pull request the ingrates a solution for generating new MRI scans for AD or NC based on the ADNI dataset using stable diffusion. The goal was to create reasonably good looking brain images with style/conditioning where during generation you can enter whether you want to generate AD or NC images. I Believe this goal has been achieved as the model supports the label conditioning and can supply during generation and it generated reasonably good looking brain images for the type you entered.
Sample of generated Images:

Files Added
47451933_Stable_Diffusion
├── Results (train results + generation outputs + diagrams)
│   ├── BrainSeq1.png
│   ├── Brains1.png
│   ├── COMP3710SD_diagram.png
│   ├── COMP3710SD_diagram_infrence.png
│   ├── Diffusion_loss.png
│   ├── Diffusion_loss_2.png
│   ├── Diffusion_loss_21.png
│   ├── Diffusion_loss_22.png
│   ├── Diffusion_loss_23.png
│   ├── Diffusion_loss_24.png
│   ├── UMapPlot.png
│   ├── VAE_loss.png
│   ├── VAE_loss_2.png
│   ├── VAE_loss_21.png
│   └── VAE_loss_22.png
│   └── AD_Genrated_Brains.png
│   └── NC_Genrated_Brains.png
│   └── test_brains.png
├── .gitignore
├── README.md
├── dataset.py (for loading in the ADNI dataset, train,test,validation splits, loading transformations)
├── modules.py (contains all the networks)
├── predict.py (reads the trained models from file and uses them to generate completely new images from noise)
├── project_setup.py (use to setup folder structure and check dataset is available)
├── requirements.txt (what python libraries re required)
├── train.py (trains the model, both the vae and U-net, can change settings to output intermediate results and save the models to file)
└── utils.py (contains extra functions used by most the files)

Documentation
The ReadMe.md contains all the information about the model and the results including example generated outputs, loss curves from training and Umap visualization of the latent space.
Kind Regards,
Jamie Westerhout
s4745193

This is an initial inspection, no action is required at this point
Stable Diffusion ADNI brain data ---- Hard Difficulty

Category

Marks
Comments

Algorithm solves the problem
5
4.5
Conditional. OK images, perhaps more time steps?

Implementation functions as intended
3
2.5

Good design
1
1
Modular, Reusable

Commenting
1
1
Meaningful docstrings and comments

Algorithm above Normal Difficulty
5
5

Algorithm is Hard difficulty
5
5
Hard Difficulty

Section IV : Max mark 20 from 20

19

TSNE / UMAP:
Good, clear separation.
Suggestions:
Might be good to have mote time steps to improve image clarity.
Good Setup/Reproducibility/Requirements would be good to have.

i have updated the "Reproducing Results" section to be better explain the Setup/Reproducibility/Requirements. Tried increasing the number time steps but didn't create a significant improvement in image clarity.

reverted back to version originally submitted based on ed thread #504

Marking

Good/OK/Fair Practice (Design/Commenting, TF/Torch Usage)
Good design and implementation.
Spacing and comments.
Header blocks.
Recognition Problem
OK solution to problem. Images need more clarity.-1
Driver Script present.
File structure present.
Good Usage & Demo & Visualisation & Data usage.
Module present.
Commenting present.
No Data leakage found.
Difficulty : Hard.  Stable Diffusion (Hard Difficulty)
Commit Log
Good Meaningful commit messages.
Good Progressive commits.
Documentation
Readme :Acceptable. Minor typos and issues. Usage needs more content.-1
Model/technical explanation :Good.
Description and Comments :Good.
Markdown used and PDF submitted. PDF Checked.
Pull Request
Successful Pull Request (Working Algorithm Delivered on Time in Correct Branch). On time.
No Feedback required.
Request Description is good.
TOTAL-2

Marked as per the due date and changes after which aren't necessarily allowed to contribute to grade for fairness.
Subject to approval from Shakes

Some additional feedback:

Incorporating metrics like Frechet Inception Distance (FID) or structural similarity index (SSIM) to validate the generated images' quality quantitatively would have been helpful.
It would help to include more information on specific hyperparameters used during training. Provide a rationale for the choice of hyperparameters
The VAE training loss graph shows signs of overfitting. To address this, you might consider adding regularization methods like dropout in the VAE or employing an early stopping criterion.
While the training and validation splits are well-handled, expanding on the testing methodology would add rigor to the evaluation.

Overall, good job!

Marking

Good/OK/Fair Practice (Design/Commenting, TF/Torch Usage)
Good design and implementation.
Spacing and comments.
Header blocks.
Recognition Problem
OK solution to problem. Some overfitting in VAE.-1
Driver Script present.
File structure present.
Good Usage & Demo & Visualisation & Data usage.
Module present.
Commenting present.
No Data leakage found.
Difficulty : Hard. Stable Diffusion (Hard Difficulty)
Commit Log
Good Meaningful commit messages.
Good Progressive commits.
Documentation
Readme :Acceptable. FID/SSIM metrics needed.-1
Model/technical explanation :Good.
Description and Comments :Good.
Markdown used and PDF submitted. PDF submitted.
Pull Request
Successful Pull Request (Working Algorithm Delivered on Time in Correct Branch). On time submission.
No Feedback required.
Request Description is good.
TOTAL-2

Marked as per the due date and changes after which aren't necessarily allowed to contribute to grade for fairness.
Subject to approval from Shakes


Maven repository is missing

I'm using dropwizard-jobs in my maven project. When maven is downloading this library from Sonatype repository it is complaining that the library is missing there.
[ERROR] Failed to execute goal on project procurement-service: Could not resolve dependencies for project edu.sjsu.cmpe:procurement-service:jar:0.0.1-SNAPSHOT: Could not find artifact de.spinscale.dropwizard:dropwizard-jobs:jar:1.0-SNAPSHOT in #sonatype-nexus-snapshots (http://oss.sonatype.org/content/repositories/snapshots) -> [Help 1]
Looks like you need to re-upload your jar.
Temporary solution would be to build this jar file and then install it with the following command:
mvn install:install-file -DgroupId=de.spinscale.dropwizard -DartifactId=dropwizard-jobs -Dversion=1.0-SNAPSHOT -Dpackaging=jar -Dfile=/path/to/file

hi,
is it released to any public repository?
Thanks !

@yunspace wanted to do it but we have'nt heard from him for a long time.
@eyalgo, would you like to take it over?

Hi,
I would like that.
Please note, I am fairly new with DW.
What are the open issues? Or is it just deploying it?
Thanks !
 Please don't print this e-mail unless it's really necessary
On Wed, Feb 25, 2015 at 5:25 PM, Hakan Dilek<EMAIL_ADDRESS>wrote:

@yunspace https://github.com/yunspace wanted to do it but we have'nt
heard from him for a long time.
@eyalgo https://github.com/eyalgo, would you like to take it over?
—
Reply to this email directly or view it on GitHub
https://github.com/spinscale/dropwizard-jobs/issues/10#issuecomment-75979601
.

@eyalgo, we've already agreed to release it with the current state. Just remove the "-SNAPSHOT" suffixes and roll it. :-)

ok.
i'll keep you updated.
should i fork pull request?
 Please don't print this e-mail unless it's really necessary
On Thu, Feb 26, 2015 at 6:22 PM, Hakan Dilek<EMAIL_ADDRESS>wrote:

@eyalgo https://github.com/eyalgo, we've already agreed to release it
with the current state. Just remove the "-SNAPSHOT" suffixes and roll it.
:-)
—
Reply to this email directly or view it on GitHub
https://github.com/spinscale/dropwizard-jobs/issues/10#issuecomment-76208417
.

Would be nice.

hi,
I did a pull request.
Is there a plan for merging?
thanks,
Eyal

merged.

well, it in maven central.
closing the issue.


write stats log from main event loop

@woobianca saw yo might be working on CLI logging, here's an idea to clean up the stats logging during connection errors.

make --stats logging a bit cleaner for the user by logging in the main recv loop instead of in a separate thread.
this also suppresses stats logging during connection errors.
the downside is, no stats will be logged if no telemetry or antenna messages are sent, so even if data transfer is stalled, --stats does not show the updated metrics.

closing i think this has negligible impact


Link to implementation documentation broken

The link to the documentation for installation and usage instructions is pointed to a page that does not render well (https://federalist-proxy.app.cloud.gov/site/18f/identity-style-guide/).

fixed!


SolidJS basic SSR example is broken

The basic SSR example for SolidJS builds with various warnings then when a request is made to server it throws an exception about React not being found, causing a 500 error response.
Relevant warning (the rest are the same):
[vite] warning: The JSX import source cannot be set without also enabling React's "automatic" JSX transform
1  |  /** @jsxImportSource solid-js */
   |                       ^
2  |  import { Link, Style, useHead } from "@solidjs/meta";
3  |  import { splitProps } from "solid-js";

  Plugin: vite:esbuild
  File: node_modules/.pnpm/@vinxi+solid<EMAIL_ADDRESS>
The 500 error after making a request:
[h3] [unhandled] ReferenceError: React is not defined
    at eval (app/server.tsx:22:83)
    ... 5 lines matching cause stack trace ...
    at async Server.toNodeHandle (file://node_modules/.pnpm/h3@1.9.0/node_modules/h3/dist/index.mjs:1885:7) {
  cause: ReferenceError: React is not defined
      at eval (app/server.tsx:22:83)
      at file://node_modules/.pnpm/solid-js@1.8.7/node_modules/solid-js/web/dist/server.js:262:34
      at createRoot (file://node_modules/.pnpm/solid-js@1.8.7/node_modules/solid-js/dist/server.js:58:14)
      at Proxy.renderToStream (file://node_modules/.pnpm/solid-js@1.8.7/node_modules/solid-js/web/dist/server.js:260:14)
      at eval (app/server.tsx:22:40)
      at async Object.handler (file://node_modules/.pnpm/h3@1.9.0/node_modules/h3/dist/index.mjs:1675:19)
      at async Server.toNodeHandle (file://node_modules/.pnpm/h3@1.9.0/node_modules/h3/dist/index.mjs:1885:7),
  statusCode: 500,
  fatal: false,
  unhandled: true,
  statusMessage: undefined,
  data: undefined
}

Thanks for catching this. Fixed in fcc469358b3063cff8da6221224f9dec9f4f2feb


usernames are lowercase, they should be capitalized

this should be handled when creating a user not anywhere else in the ui.


Add Sample using Rancher RKE2 and OpenEBS

Hi,
Thanks for taking the time to create this pull-request and sorry it has taken me sometime to review.
Firstly can you verify that you agree with the contribution terms available here.
I think this is a great addition to the samples, so in general I am supportive, but have a few questions/updates that I have made within the pull request.

You are welcome ! I will read your comments and adapt the pull request accordingly.
I agree with the contribution terms available here, there is a way to formally accept it ? or my comment is enough ?

Thanks - your comment in this pull request is adequate for accepting the contribution terms.

@callumpjackson I let you review my changes according to your comments. I have added the Native HA with 3 pods but since there is a Pod AntiAffinity in the chart we have to create a 3 nodes RKE2 cluster. To be able to test on a single node RKE2 cluster, I have removed this Pod AntiAnffinity manually.


Update Go version

Scion dependency quic-go requires Go 1.20.
I therefore suggest to update to the latest 1.21.1

Agreed, we did this in 8aac7109c17e8c3a87606e7bb4dffa5f2048ea1b and 11bc7030931241f94064063e560788e7db115767.
Did we miss something there?


[BUG] Equal Focus On All Buttons

Description
All the buttons on the hero section of the website have equal focus of user. As per common practices, one of them should be in higher order of focus in order to attract user to it and act as a CTA. We should try to redesign the buttons in such a way that the active state of one is normal state of other.
Also the Github Icon should be responsiveness as per the state.
Screenshots

Browser 🥦
Brave
Checklist ✅

[X] I checked and didn't find similar issue
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (blank for no)

Hi @Souravvmishra , great idea ⚡
However i want to know, what type of design changes are we talking about, and also how will the structure also change.
Currently it looks like this:

As per common practices, one of them should be in higher order of focus in order to attract user to it and act as a CTA. We should try to redesign the buttons in such a way that the active state of one is normal state of other. Also adjust the logo as per the state of button.

Interesting, you can make a PR and then we can I guess further talk about it.

Sorry for the delay. I was out of station for a while. I'll complete it soon.

Hi there, please make a PR when possible :)

Notice 🚩
Hi there, unfortunately, I have to close this Issue/PR, as there is no work currently going on for the same. There might be more Issues/PR which are waiting for this to be completed - so it is better to pace things up. I generally spend/wait for 3 weeks before closing.
If you think that this is a mistake, please drop a mail at<EMAIL_ADDRESS>and we will get back to you.
Regards,
Team Milan.


broadcast_in_dim specification does not say anything about the order of broadcasted_dimensions

What happened?
The broadcast_in_dim specification does not seem to state anything about the expected order of broadcasted_dimensions, and the given example on the page (reproduced below) seems to allow dimensions to be provided in arbitrary orders.
%result = "stablehlo.broadcast_in_dim"(%operand) {
  broadcast_dimensions = array<i64: 2, 1>
} : (tensor<1x3xi32>) -> tensor<2x3x2xi32>

I think however that XLA expects those dimensions to be sorted in increasing order, and that JAX likewise requires dimensions sorted in increasing order for its broadcast_in_dim_p primitive. Sorting also makes more sense to me, otherwise your broadcast has also become a transpose.
Given that the example is explicitly broadcasting with "reversed" broadcasting dimensions, I'm not sure if this is intentional or an oversight---hence I'm filing this bug :)

The example here is intentional and demonstrates the fact that broadcast_dimensions allows adding dimensions anywhere in the result shape.
Actually, the specification is similar to what we have in XLA https://openxla.org/xla/operation_semantics#broadcastindim where broadcast_dimensions maps the dimensions of operand to the dimensions of the target shape, i.e. the i'th dimension of the operand is mapped to the broadcast_dimension[i]'th dimension of the output shape.
Please let me know if this helps.

Thanks for the explanation! I'm not entirely sure I understand the rationale for supporting also unsorted broadcast dimensions. (Is anyone actually using that? As we both saw above, XLA and JAX definitely don't.)
Regardless, I see this is a point where StableHLO has decided to take a different path and this is thus not an oversight. Thanks again Sandeep, I'll close this now.


Sidebar doesn't show individual entries when it is collapsed

The sidebar doesn't behave properly when it is collapsed. It always shows the home entry, even though you put the mouse over any other entry.

@arbulu89 solved in PR #131 and (embarrassingly) in PR #133


Translated Python bool code samples to spanish

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

LGTM

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 🎉


Add bindQueriesToElement to remove need for duplicated code in libraries built on dom-testing-library

There's a block of code used by vue-testing-library, react-testing-library and in the tests here:
   const containerHelpers = Object.entries(queries).reduce(
    (helpers, [key, fn]) => {
      helpers[key] = fn.bind(null, element)
      return helpers
    },
    {},
  )

I propose adding a bindQueriesToElement utility function to encapsulate this code.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Port lpad/rpad from postgresql-10.9

Summary of the changes / Why this improves CrateDB
Checklist

[ ] User relevant changes are recorded in CHANGES.txt
[ ] Touched code is covered by tests
[ ] Documentation has been updated if necessary
[ ] CLA is signed
[ ] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

Can one of the admins verify this patch?

@ysz Thanks a lot for your contribution.
In general:

Please check test failures
All arguments can have NULL values.
Could we use the same code for rpad and lpad to avoid duplication?

@seut Thank you for prompt review.
Comments have been addressed.

Hi Yakov,
Thank you very much for implementing the [l/r]pad functions, we very much would like to incorporate them onto CrateDB. We will take over from here.
Once again thank you very much, we value your contributions!!
Kind regards


Add devise starter files

For this feature me and my partner did the following

Made sure you can register a new user.
User can login with a combination of email and password.
Made sure hashed password should be stored in the database.
Ask for confirmation of email.
Can reset password.

@bellom I've pushed

@Chimoneg27 - kindly give me a few minutes, I will check and revert shortly.

Hi @Chimoneg27 @erickma1
Approved ✅
Great work addressing all the requested modifications from the prior code reviews!
Pleas go ahead and merge 🚀
Keep rocking 🚀!


Print current stack height after receiving Unreachable trap when test-support is enabled

As debugging Unreachable trap is difficult, we can log current stack height after executing wasm was trapped.

A replacement PR is ready to review.


🔧 disable button on expired offer

PR type

[x] Bugfix

What's new?

[x] PR closes #3790

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Had issue bounty label?

[x] Fill up your KSM address: Payout

Screenshot

Note
Would it be possible for the currentOwner to remove expired offers?

pay 50 usd

😍 Perfect, I’ve sent the payout
💵 $50 @ 43.37 USD/KSM ~ 1.153 $KSM
🧗 DVYy1qnocE8t6ZvUfPx3rEjG829khNRXx3YrCGVHHj19Lcb
🔗 0x827ac0ea8f5e6597493fd423d4cf4af3eff413b1106f62dac5aee49ad9f5f580
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


⚠ Avatar API Endpoint has degraded performance

In 77b4643, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 2674 ms

Resolved: Avatar API Endpoint performance has improved in 9578e23 after 20 minutes.


dashboard: Fix issue causing blank page instead of login view

This fixes cases where a blank page is rendered instead of the login view.
#1696

LGTM


Provide record context to attachment upload

We need to push some extra params to the back-end when uploading attachments.

lgtm.


upcoming outage due to upgrade to Ubuntu 22.04

Google Colab runtime will be upgraded to Ubuntu 22.04 in the near future, see
https://github.com/googlecolab/colabtools/issues/3778
The moment this happens all packages will need to be rebuilt.

Upgrade has been pushed to https://github.com/googlecolab/backend-info/commit/1750ed37cd39fbd2d0f70c571ef571272feef59b, but is not live yet on runtime instances.

Upgrade is now live on runtime instances, see https://medium.com/google-colab/colab-updated-to-ubuntu-22-04-lts-709a91555b3c
Currently rebuilding all packages

All packages have been rebuilt.


installation of virtualenv failed using Catalina: errno 13

Hey
After having had so many difficulties with my old Mac I have decided for a new (brand-new) MacBook Pro 16'' running on Catalina. It was time anyway.
Following the tutorial I have installed
home-brew
python
Cairo
pkg-config
installing of the virtual environment it gives  the following errno 13:
_Could not install packages due to an EnvironmentError: [Errno 13] Permission denied: '/Library/Python/3.7'
Consider using the --user option or check the permissions.
You are using pip version 19.0.3, however version 20.0.2 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
trying to upgrade pip to 20.0.2 gave the same errno 13
found out that it might be an issue with rights but am not sure what exactly to do
any advice?
Would be thankful for help

Use the sudo prefix when running the command to generate the Virtual Environment.
Something like this:
sudo python3 -m venv ManimEnv

many thanks. It worked. Was guessing, but did not want to mewl with sudo without knowing that this is the right thing to do.
Now got another issue. Will open separately


Window is always borderless and hides mouse cursor under Wayland

When running under Wayland the created window is always borderless. Also there no cursor while it is over the window.
Screenshot (Gnome on X11):

Screenshot (Gnome on Wayland, sway looks similar):

Running with WAYLAND_DEBUG=1 gives the following log:
[1604604020.617104] -> wl_display@1.get_registry(2)
[1604604020.617145] -> wl_display@1.sync(3)
[1604604020.617348] <- wl_display@1.delete_id(3)
[1604604020.617373] <- wl_registry@2.global(1, "wl_drm", 2)
[1604604020.617413] <- wl_registry@2.global(2, "wl_compositor", 4)
[1604604020.617451] <- wl_registry@2.global(3, "wl_shm", 1)
[1604604020.617481] <- wl_registry@2.global(4, "wl_output", 2)
[1604604020.617516] <- wl_registry@2.global(5, "wl_output", 2)
[1604604020.617556] <- wl_registry@2.global(6, "zxdg_output_manager_v1", 3)
[1604604020.617607] <- wl_registry@2.global(7, "wl_data_device_manager", 3)
[1604604020.617672] <- wl_registry@2.global(8, "zwp_primary_selection_device_manager_v1", 1)
[1604604020.617733] <- wl_registry@2.global(9, "gtk_primary_selection_device_manager", 1)
[1604604020.617808] <- wl_registry@2.global(10, "wl_subcompositor", 1)
[1604604020.617876] <- wl_registry@2.global(11, "xdg_wm_base", 3)
[1604604020.617912] <- wl_registry@2.global(12, "zxdg_shell_v6", 1)
[1604604020.617943] <- wl_registry@2.global(13, "wl_shell", 1)
[1604604020.617991] <- wl_registry@2.global(14, "gtk_shell1", 3)
[1604604020.618019] <- wl_registry@2.global(15, "wp_viewporter", 1)
[1604604020.618153] <- wl_registry@2.global(16, "zwp_pointer_gestures_v1", 1)
[1604604020.618200] <- wl_registry@2.global(17, "zwp_tablet_manager_v2", 1)
[1604604020.618253] <- wl_registry@2.global(18, "wl_seat", 5)
[1604604020.618287] <- wl_registry@2.global(19, "zwp_relative_pointer_manager_v1", 1)
[1604604020.618396] <- wl_registry@2.global(20, "zwp_pointer_constraints_v1", 1)
[1604604020.618459] <- wl_registry@2.global(21, "zxdg_exporter_v1", 1)
[1604604020.618508] <- wl_registry@2.global(22, "zxdg_importer_v1", 1)
[1604604020.618542] <- wl_registry@2.global(23, "zwp_linux_dmabuf_v1", 3)
[1604604020.618569] <- wl_registry@2.global(24, "zwp_keyboard_shortcuts_inhibit_manager_v1", 1)
[1604604020.618628] <- wl_registry@2.global(25, "zwp_text_input_manager_v3", 1)
[1604604020.618678] <- wl_registry@2.global(26, "gtk_text_input_manager", 1)
[1604604020.618730] <- wl_callback@3.done(1739)
[1604604020.618789] -> wl_registry@2.bind(18, "wl_seat", 5, 3)
[1604604020.618831] -> wl_seat@3.get_keyboard(4)
[1604604020.618853] -> wl_seat@3.get_pointer(5)
[1604604020.619123] -> wl_registry@2.bind(2, "wl_compositor", 4, 6)
[1604604020.619168] -> wl_registry@2.bind(3, "wl_shm", 1, 7)
[1604604020.619203] -> wl_compositor@6.create_surface(8)
[1604604020.619245] -> wl_shm@7.create_pool(9, 7, 1920000)
[1604604020.619278] -> wl_shm_pool@9.create_buffer(10, 0, 800, 600, 3200, 1)
[1604604020.636476] -> wl_registry@2.bind(11, "xdg_wm_base", 1, 11)
[1604604020.636561] -> xdg_wm_base@11.get_xdg_surface(12, 8)
[1604604020.636596] -> xdg_surface@12.get_toplevel(13)
Failed to create server-side surface decoration: Missing
[1604604020.636626] -> wl_surface@8.commit()
[1604604020.636648] -> wl_display@1.sync(14)
[1604604020.638806] <- wl_display@1.delete_id(14)
[1604604020.638853] <- wl_keyboard@4.keymap(1, 8, 52842)
[1604604020.638893] <- wl_keyboard@4.repeat_info(33, 500)
[1604604020.638929] <- xdg_surface@12.configure(66)
[1604604020.638949] -> xdg_surface@12.ack_configure(66)
[1604604020.638970] -> wl_surface@8.commit()
[1604604020.638988] <- wl_callback@14.done(1739)
[1604604020.639027] -> wl_surface@8.attach(10, 0, 0)
[1604604020.639051] -> wl_surface@8.damage(0, 0,<PHONE_NUMBER>,<PHONE_NUMBER>)
[1604604020.639072] -> wl_surface@8.commit()
[1604604020.639136] -> wl_shm@7.create_pool(14, 10, 1024)
[1604604020.639169] -> wl_compositor@6.create_surface(15)
[1604604020.639291] -> xdg_toplevel@13.set_title("Window")
[1604604020.673760] -> wl_shm@7.create_pool(16, 9, 1920000)
[1604604020.673829] -> wl_shm_pool@16.create_buffer(17, 0, 800, 600, 3200, 1)
[1604604020.674683] -> wl_surface@8.attach(17, 0, 0)
[1604604020.674734] -> wl_surface@8.damage(0, 0,<PHONE_NUMBER>,<PHONE_NUMBER>)
[1604604020.674766] -> wl_surface@8.commit()
[1604604020.678512] <- wl_buffer@10.release()
[1604604020.678539] <- wl_buffer@17.release()
[1604604020.681444] -> wl_surface@8.attach(17, 0, 0)
[1604604020.681492] -> wl_surface@8.damage(0, 0,<PHONE_NUMBER>,<PHONE_NUMBER>)
[1604604020.681511] -> wl_surface@8.commit()
[1604604020.683957] <- xdg_toplevel@13.configure(800, 600, [4, 0, 0, 0])
[1604604020.684008] <- xdg_surface@12.configure(67)
[1604604020.684035] <- wl_keyboard@4.modifiers(1741, 0, 0, 16, 0)
[1604604020.684073] <- wl_keyboard@4.enter(1741, 8, [])
[1604604020.697978] -> xdg_surface@12.ack_configure(67)
[1604604020.698046] -> wl_surface@8.attach(17, 0, 0)
[1604604020.698074] -> wl_surface@8.damage(0, 0,<PHONE_NUMBER>,<PHONE_NUMBER>)
[1604604020.698097] -> wl_surface@8.commit()
[1604604020.698209] <- wl_buffer@17.release()
[1604604020.714811] -> wl_surface@8.attach(17, 0, 0)
[1604604020.714883] -> wl_surface@8.damage(0, 0,<PHONE_NUMBER>,<PHONE_NUMBER>)
[1604604020.714912] -> wl_surface@8.commit()
[1604604020.715009] <- wl_buffer@17.release()
[...]

I'm not sure whether this may be an upstream issue of Smithay/wayland-rs; I've found no related issues.

Hey @nifker any idea about this one?

It uses the ZxdgDecorationManagerV1 protocol(which looks like gnome doesnt support) for the decoration but I dont know if others exist.
Seems like the default pointer_visibility is for some reason false. We need to change that value.

It seems GNOME doesn't support Server Side Decorations: https://drewdevault.com/2018/01/27/Sway-and-client-side-decorations.html.

@mvforell You say sway doesnt draw serve-side decorations onto the window?


Error on OSX

Running on OSX, running deletefb with the passed parameters produces the following:
> Traceback (most recent call last):
>   File "./deletefb.py", line 139, in <module>
>     run_delete()
>   File "./deletefb.py", line 58, in run_delete
>     is_headless=args.is_headless
>   File "./deletefb.py", line 81, in delete_posts
>     driver = Chrome(options=chrome_options)
>   File "/Users/joe/Library/Python/2.7/lib/python/site-packages/selenium/webdriver/chrome/webdriver.py", line 73, in __init__
>     self.service.start()
>   File "/Users/joe/Library/Python/2.7/lib/python/site-packages/selenium/webdriver/common/service.py", line 76, in start
>     stdin=PIPE)
>   File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 710, in __init__
>     errread, errwrite)
>   File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 1335, in _execute_child
>     raise child_exception
> OSError: [Errno 20] Not a directory

Encountered this before?

Hi, it looks like an issue with the Chrome driver for Selenium. Can you tell me how you installed the Chrome driver, and what version it is?
If you haven't set it up yet, then I think brew cask install chromedriver should get you going. I will add something to the README about MacOS.

Better error messages implemented in https://github.com/weskerfoot/DeleteFB/pull/31/commits/fddd373d142a36f0161406b7975fb920632bf88f


default Image is not visible

I have added the default Image as well the lazyload image path .
Did not get anything visible until the lazyload Image loaded.
Same as on the example

Hi,
The example page was using the following css style:
img {
  opacity: 0;
}
img.ng2-lazyloaded {
  opacity: 1;
}

eg, it was intended to hide the image (and the default image) until the lazy-loaded image was loaded.
However, if you remove opacity: 0; you should be able to see the default image before the lazy-loaded image is loaded.
I have updated the example page with some more examples. Go to http://tjoskar.github.io/ng2-lazyload-image/ and click on "Default image" to see an example of using default images.
If you are using chrome, you can also use the "network throttling" in dev tools to simulate a longer loading time.
Did I answer your question?

You have example code here: https://github.com/tjoskar/ng2-lazyload-image/blob/master/example/default-image.component.ts

@tjoskar I have tried everything, still not working, the default image doesn't show up at all, and the path is correct because if i replaced the lazyloder image with the default image and the default image loads, the issue is when i checked google chrome inspector tool, i see that there is no where the default image was attached to the src attribute of the image, instead the lazyloader image is attached by default. Please is there any other way around this? Waiting for your response.
Thanks

@kelvinrolex, I'm not sure that I follow. If the image that should be lazyloaded is already in the browser cache, the default image will never be visible. The algorithm follows:

Set default image to src
Check if the image is in viewport

2.1 If the image is in viewport start download the lazyloaded image
2.1.1 When lazyloaded image is downloaded, set lazyloaded image to src
2.2 Else go to 2
I recorded a video to show how the default image is loaded: https://www.youtube.com/watch?v=p4hds8Au_qI
Do you have a code snippet that you can share so I can get a better understanding of what you are trying to do.
Thanks

Looks like I have the same problem. If I use [src]="defaultImage" (and, of course have a defaultImage property), the src attribute of the img tag is not written before the image is loaded (comes into view port).
If I use no property binding (e.g.: ), than the default image is loaded and replaced as expected.

@taseroth, Sorry for the late reply. What browser and OS are you using?
Just to be clear; if you visit http://tjoskar.github.io/ng2-lazyload-image/fade-in-image and then pressing the link "default image". Aren't seeing any picture on Nicolas cage then?

I have the same issue. Default image is nod rendered (Only if I put it explicitly with "src" & hardcoded placeholder url).
Also I've noticed that "scrollspy" is not working correctly, some times lasyload loads first 2 or 3  elements &
all of the others will sty unchanged even after "intense" scrolling XD. Offset param seems not affecting this issue. There is one trick: if I re-load the page & scroll down before contaners are loaded, it loads all of the images...
There is no errors in the console or during compilation.

@bsh314, I do not follow.

Default image is nod rendered (Only if I put it explicitly with "src" & hardcoded placeholder url).
You should use src for the default image.
Let's say you are using the following code: . Angular will compile that and give  to the browser (given defaultImage = 'default-image.jpg') and when the images is in the viewport, will the directive (ng2-lazyload) update the image-node to  (given image = 'lazyload-image.jpg').

Also I've noticed that "scrollspy" is not working correctly ...
Are you referencing to bootstraps scrollspy?
Can you reproduce the error without scrollspy?

@tjoskar No, i'm not 8using bootstrap, that was only the way to name that type of functionality.
My default image issue:
 -> this works
 -> this dont work
Thanks for the reply!

I have the same problem. It only works if the img url is hard-coded in the template.
You have it hard-coded in your example:
// https://github.com/tjoskar/ng2-lazyload-image/blob/master/example/default-image.component.ts

@filipsuk: yup, that seems to be the issue. I can't believe I missed that detail.
This issue should be resolved in version 2.3.2. Please give it a try.

@tjoskar Problem seems to be fixed in 2.3.2. Thank you so much for your work! :)

I'm going to close this now. Please let me know of any of you still have the issue.

Hi,
I am still encountering this issue, using this CSS
img {
transition: opacity 1s;
opacity: 0;
}
img.ng-lazyloaded {
    opacity: 1;
}

and also hardcoded the src for defaultImage but the default image is still not visible,
If I comment out the opacity:0, the image is visible but no transition takes place


Support VMs in multiple working directories in the same K8S cluster

Currently we only allow VMs in the same working directory to join the same K8S cluster.
This is not a necessary limitation.
If the localInstanceID is set to Path+VMName at the very beginning, we should be able to support different workingDir VMs easily. Of course multiple functions need to be adjusted but this won't be a significant change.

It's still unclear what the priority of this issue is till we talk to the customer.

This will be address as part of supporting k8s spanning multiple VCs. I'm closing this issue. Feel free to reopen with valid reasons.


Ampersands not shown in code block

If I try to use ampersands in a code block, they are hidden in the Gfm Preview tab:

$ sudo bash -c "cd /usr/local/bin && curl -fsSLo boot https://github.com/boot-clj/boot-bin/releases/download/latest/boot.sh && chmod 755 boot"`
$ export BOOT_CLOJURE_VERSION=1.8.0

appears as

$ sudo bash -c "cd /usr/local/bin    curl -fsSLo boot https://github.com/boot-clj/boot-bin/releases/download/latest/boot.sh    chmod 755 boot"`
$ export BOOT_CLOJURE_VERSION=1.8.0

Notice the missing ampersands.

Hm... that's strange. It works perfectly fine on my machine. Can you please provide screenshot and version of IDE(and the operating system) you use?

IntelliJ IDEA 2016.1.1
Build #IU-145.597, built on March 29, 2016
JRE: 1.8.0_40-release-b132 x86_64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Mac OS 10.11.4
On 13 April 2016 at 12:10, shyyko.serhiy<EMAIL_ADDRESS>wrote:

Hm... that's strange. It works perfectly fine on my machine. Can you
please provide screenshot and version of IDE(and the operating system) you
use?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/ShyykoSerhiy/gfm-plugin/issues/109#issuecomment-209599457

--
Chris Devine | Software Engineer
<EMAIL_ADDRESS><EMAIL_ADDRESS>|
415-813-5245
747 Front St, 4th Fl | San Francisco, CA 94111
Our Mission: To build a better financial world
Unless specifically indicated, this e-mail is not an offer to sell or a
solicitation of any investment products or other financial product or
service, an official confirmation of any transaction, or an official
statement of Funding Circle USA.  This e-mail is meant only for the
intended recipient of this transmission, and contains trade secret and
strictly confidential information belonging to the sender. It is unlawful
for unauthorized individuals to review, use, copy, disclose, or disseminate
confidential information. If you have received this e-mail in error, please
notify the sender immediately by telephone at<PHONE_NUMBER> or by return
email and promptly delete this message from your system.


Why do two invalidate operations cause security hole？

quote：

Until a modified cache block has updated memory, a CBO.INVAL instruction may expose stale data values
in memory if the CSRs are programmed to perform an invalidate operation. This behavior may result in a
security hole if lower privileged level software performs an invalidate operation and accesses sensitive
information in memory.

https://github.com/riscv/riscv-CMOs/blob/ce72670f9843d94952c685346cf52d43b969ccdf/cmobase/csr_state.adoc。
A scene I guess is like this：

In the machine mode, execute the cbo.inval instruction for an address (such as 0x83000000), and some data(0x1234) in the cache corresponding to the address is invalid
Switch to the user mode and execute the cbo.inval instruction for the same address (such as 0x83000000). Similarly, some data in the cache corresponding to this address is invalid. User mode does not get data 0x1234

So why does it lead to security hole when users access sensitive data?

Adjusting your example, the scenario is:

Some sensitive data (0x1234) is written to a physical address (such as 0x83000000) in backing memory.
Some new sanitizing data (0x0000) is written to that same physical address in a (write-back) cache.
Execute the cbo.inval instruction for the same address (such as 0x83000000), so that the sanitizing data (0x0000) is discarded (not written back) from the cache.
Execute a load for the same address (such as 0x83000000), which reads the old sensitive data (0x1234) from backing memory.

thanks 👍


Don't error hard on ignored argument

First off, great project!
Problem
Currently a hard "invalid C format string" error is flagged when format specifiers have a position argument and the positions are not continuous, meaning that a format argument is omitted:

Firstly, the format string is not actually invalid, it just means that an argument will be ignored. Consider this valid source code:
    printf("On %2$s I eat %3$s and walk for one hour.", 1, "Wednesday", "fish");

Secondly, this may very well be intentional when the counter in the singular case has been replaced with a word, as is the case here.
possible solution
If the same argument is being ignored in the source text as well, no error or warning should be flagged.
If the argument is only ignored in the translation then it could still be intentional, but a warning is probably helpful in case it wasn't.

Firstly, the format string is not actually invalid, it just means that an argument will be ignored.

Ignoring information in the translation that the programmer intends to be included is, by definition and self-evidently, invalid. It breaks the software being translated!
This is not Poedit's behavior, but GNU gettext's, so I can't change it in Poedit. Furthermore, the behavior is correct and identifies a serious bug in the translation.

As you see in the screenshot, the argument is being ignored by the programmer as well.
My program works as intended with the MO file that is currently produced by Poedit. All is good, except the warning.

Again:

This is not Poedit's behavior, but GNU gettext's, so I can't change it in Poedit

Again, GNU gettext's msgfmt works excellently with this:
#: source/app.d:19(main.report)
#, c-format
msgid "On %2$s I eat %3$s and walk for one hour."
msgid_plural "On %2$s I eat %3$s and walk for %1$d hours."

This produces a valid C format string, as in: it works for printf.
Everything works as intended, except that Poedit flags an error which should at most be a warning.

This is a validation error from msgfmt -c. If you think msgfmt -c shouldn't report an error in this case, fix it upstream.
Poedit will not ignore msgfmt -c validation errors.


fix: validate configuration inheritance for Endpoints and EndpointGro…

…up APIM-607

📚  View the storybook of this branch here

🚀 CI was able to deploy the build of this PR, so you can now try it directly here
Notes: The deployed app is linked to the management API of APIM master. (Same login and password as APIM master)

I will moved this PR to draft as the inheritance mechanism is challenged by the team.

will be fixed by the split  of the EndpointCoinfiguration


Restore node 0.10/0.12 support

Submitted as per https://twitter.com/sindresorhus/status/786035215919517696

Would be nice to see it landed, but if not there is one with 0.10 support: https://www.npmjs.com/package/stripbom :)

Wait, what? Why not use an older version if you need 0.12 support?

Because npm outdated exists and is used by many users. Fielding those requests is frustrating for maintainers. The arrow function isn't adding anything to this module.

And things like bithound.com would complain and mark your package as "needed attention" which will distract from the real issues.

@mathiasbynens also, the implementation changed when it broke 0.10/0.12 support so what if we want that?

@sindresorhus ping

Sorry, I'm not interested in reversing this. Just use an older version if you need 0.10/0.12 compatibility. Most modules are moving on. This will soon not be the only module that you'll not be able to upgrade.

If your modules are done, why did you make this change? More importantly, why did you make it with other actual changes? You could have easily broken them into 2 separate semver releases (and still can).

I did two breaking changes in the current major release. First, was to break the buffer part into a separate module so there would be one less dependency here. Second, since it was already a breaking change, I took advantage of it and moved to require Node.js 4.
It's clear we see things differently. I'm not really interested in discussing this any further though.


Let's Encrypt Guide

Added a supporting guide that shows how to get SSL certs for Matrix with the Let's Encrypt automated CA.
Signed-off-by: William A Stevens V<EMAIL_ADDRESS>

thanks! @oddvar - can you take a look please? (might want to merge dave's blog post into a guide too)

great stuff!

now live @ https://matrix.org/docs/guides/lets-encrypt.html


hopefully publish a linux gem

Motivation
We haven't had a linux gem publish in the last 3 days, and I think it is because we are getting unlucky with the linux gem being later in the pipeline and the metadata is getting updated between the ruby publish and the linux one? Sounds implausible but honestly, this code is pretty fragile and I think it is better to fail open and continue to publish as much as we can (especially since we have retries) than to silently drop gems we think are already published.
Test plan

See included automated tests.

Fixes https://github.com/sorbet/sorbet/issues/2250

looking at https://rubygems.org/gems/sorbet-static/versions it seems things are in random order too

@ptarjan This makes it so that the publish script never fails, which means when we're looking at Buildkite, we'll never see a failing publish-ruby-gems.sh job.
I'm fine with this change as is to unblock things immediately, but what do you think about a more proper fix in the future?

I don't know what a more proper fix would look like. We can talk about it in the meeting if you have ideas but I sat and though and couldn't come up with something that was better

I think I know a proper fix(have a single if for all of them), I can take over if you want

as-in, move   if ! gem list --remote rubygems.org --exact 'sorbet-static' | grep -q "$release_version"; then out of the loop

I guess I don't understand what the failure mode is. Do we not want a day where we don't make any master commits to have a red failing build for the publish? Or do we sometimes want to re-run the job and want it to not fail but still keep publishing?
In my mind I want this job to publish as much as it can possibly do and never fail, but I rarely look at it so if you want a different shape, I'm happy to do whichever you want.

(feel free to make any changes you want, i'm not in love with the approach or current code)

Do we not want a day where we don't make any master commits to have a red failing build for the publish?

yes, that's that the current code was achieving, but the proposed one also will make it green if rubygems fail to publish(what they did frequently in the past).
I'll may a PR

== dmitry


stronger contract on flat_buffer, flat_static_buffer

Instead of:

Buffer sequences returned by basic_flat_buffer::data and basic_flat_buffer::prepare will always be of length one.

We can make a stronger statement:

The type of buffer sequence returned will be net::const_buffer or net::mutable_buffer

^ This should also take into account the dynamic_buffer developments in boost .asio

We have tried to update Beast's usage of dynamic buffers to match the style in Asio and the result has only been extreme pain and breakage. I think I have just accepted that Beast's dynamic buffers are going to be different than the ones in Asio. If anything, we should make this more clear in the documentation.
In my new HTTP library, I am avoiding the use of dynamic buffers entirely, so this becomes a non-problem.


KDE5: plasma 5.6 responds slowly

Issue description
Yesterday I upgraded to Nix master just after the merge with staging. Plasma seems to respond very slow, buttons typically take half a minute before the response occurs. CPU usage is low. Applications behave good, there is no delay with them.
My NetworkManager passwords appear to be gone as well, since I have to enter them again. Actually, I think it's because kwallet doesn't appear to be prompted.
Technical details

System: NixOS 16.09.git.bb73e25

I will try an upgrade to Plasma 5.6.4. and Applications 16.04.1.
cc @ttuegel

Plasma is pretty well-known for exposing graphics driver bugs. What type of graphics hardware do you have, and what drivers are you using? You may find other Plasma users with similar hardware are having the same experience. It may also help to change the rendering backend under System Settings > Display and Monitor > Compositor.

Changing compositor settings doesn't seem to make a difference. I created a new user, and there the default session works just fine.
It really appears to be as if something is blocking calls. It happens, and it happens fluently, but just half a minute after I pressed a button.
It would at least appear that this is an issue for upstream.

I removed a lot of dot files and now my session works good again. There is just a high CPU usage but that appears to be related to Baloo.


audible-cli integration

The updated README is pretty self explanatory, this is an attempt to make AAXtoMP3 work in synergy with audible-cli, allowing to get info not normally available in the .aax file

Closed, incoming new pull request with same features and aaxc decryption support


Unable to generate Swagger docs due to type definition not found error

Hello,
I'm having trouble generating Swagger documentation for my Go project using Swaggo. I'm encountering an error that seems to be due to Swaggo not being able to find a type definition that is located in a different directory.
Here's the error message I'm getting:
Error parsing type definition 'main.Neighbors': hashEquals: cannot find type definition: model.NeighborsNeighborsHashEqual
Error parsing type definition 'main.JsonResponse': neighborsData: Neighbors: hashEquals: cannot find type definition: model.NeighborsNeighborsHashEqual
ParseComment error in file /Users/nathannaveen/go/src/github.com/nathannaveen/guac/cmd/rest/known.go :JsonResponse: neighborsData: Neighbors: hashEquals: cannot
ParseComment error in file /Users/nathannaveen/go/src/github.com/nathannaveen/guac/cmd/rest/known.go :JsonResponse: neighborsData: Neighbors: hashEquals: cannot find type definition: model.NeighborsNeighborsHashEqual

The type model.NeighborsNeighborsHashEqual is defined in another directory (/Users/nathannaveen/go/src/github.com/nathannaveen/guac/pkg/assembler/clients/generated/operations.go), and I'm running swag init from /Users/nathannaveen/go/src/github.com/nathannaveen/guac/cmd/rest.
I've tried the following steps to resolve this issue:

Checked my import statements to ensure the package containing model.NeighborsNeighborsHashEqual is correctly imported.
Checked my GOPATH and GOROOT environment variables.
Ensured my Go project is correctly organized according to Go's project layout conventions.
Ran swag init from the root directory of my Go module.
Used the --parseDependency flag when running swag init.
Used the -d flag when running swag init to specify the directory where swaggo should search for types.
Ensured the package is included in my go.mod file and ran go mod tidy.

Despite these steps, I'm still encountering the error. Any help would be greatly appreciated.
Thank you!

@nathannaveen could you try runninng this command swag init -g **/**/*.go from project root

@ranefattesingh That works, thank you!


Improve/refactor animation API

The one major improvement I want to apply here is to move away from these two:
struct term_animate * animated(float value);
void animate_release(struct term_animate *);

It needs to be simpler and not require malloc/free. To avoid that, struct term_animate must be public API, and not an opaque type. The reason that I didn't go this way to start with, was that I felt it's a complicated object and the values it holds are meant for consumption by the accompanying API functions; not the caller, i.e. it might be confusing to use. But I think in the end there's more pros than cons to opening it up.
Also consider whether this module even should be part of the core termlike library. It could be an optional addition.

Looking at the animate_to function, usage can be unclear; for example:
animate_to(fade_anim, 1, 2, step);

Which of the values is the target, and which is the duration?
Consider adding a #define like SECONDS(2) or MILLISECONDS(2000).

Consider making values double instead of float. This would reduce potential warnings like:

and

I don't think there would be any noticeable performance difference; it would require more memory, sure, but in the grand scheme of things it would probably be negligible.
Though I kinda do like how currently doubles are used for anything related to time, whereas floats are used for "values".

Consider adding some common way of determining finished animation, without having to resort to establishing the pattern below over and over.
For example, some point during init:
fade = animated(TERM_COLOR_TRANSPARENT);

...later on, during a tick:
float f;

animate_get(fade, &f);

// checking prior to animating so that we always get the final frame
if (f >= TERM_COLOR_OPAQUE) {
    animate_reset(fade, TERM_COLOR_TRANSPARENT);
} else {
    animate_to(fade, TERM_COLOR_OPAQUE, SECONDS(2), step);
}


Dynamic imports in generator don't work

I have this code (to lazy-load a module in my redux-saga):
function myGenerator() {
  const { lookup: lookupMIMEType } = yield call(() => import("mime-types"));
}

Problem
See here: https://flow.org/try/#0PQKgBAAgZgNg9gdzCYAoVUCuA7AxgFwEs5twBzAU2wAoBKMAb1TDFxIGd9GwAHAQwBO7CmAC+YALxgAnoQowAJqz4wY1OpIB8YQgFsecAfmoAiTAJgnatANzoWAivnPZeg4XdFA
The flow try tool seems to crash silently. On my CLI I get the following error:
$ flow --max-warnings 0
Launching Flow server for /Users/leoselig/nexenio/web
Spawned flow server (pid=50359)
Logs will go to /private/tmp/flow/zSUserszSleoseligzSnexeniozSweb.log
Monitor logs will go to /private/tmp/flow/zSUserszSleoseligzSnexeniozSweb.monitor_log
Error ┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈ packages/app-files/src/js/routing/index.js:0:1

Internal error: uncaught exception:
Context.Require_not_found("\"/Users/some/path/to/file.js\":
(90, 63) to (90, 75)")
Raised at file "map.ml", line 130, characters 10-25
Called from file
"hack/utils/collections/myMap.ml", line 15, characters 13-23

Found 1 error

Expected
Get the same result as with the workaround here

It no longer crashes


What is Breaking Changes "Extension Changed" in the 7.0.0 release?

Is there an existing issue for this?

[X] I have searched the existing issues

Your question
I'm upgrading to the most recent major version of Fuse. In the changelog I noticed for the 7.0.0 version under "Breaking Changes" there is a bullet point for "Extension Changed."
I'm not entirely sure what that means, and would love some insight if anyone could help.

Nvm! Figured this out from this issue. Closing this.

Closing.


2.5.0

support macOS from 10.15 (#153)
remove tutorial (#158)
rename ViewLayout.coordinators to ViewLayout.attachments and improve implementation of UIView.sublayout
apply SL suffix to Layoutable (#159)
test/enchance anchors (#161)
add options to SwiftLayoutPrinter (#162)
revert appkit supporting
update github action
test/performance (#165)
add native api view test (#167)
add run-performance.sh
update run-performance for sorting
update run-performance for sorting
using build in run-performance.sh
update run-performance
update run-performance for test result filtering
Feature/layout debug print (#169)
comment snapshot test
update test sample
comment test from run-performance.sh
update run-performance.sh
update performance
run-performance testing only performancetests
update performance - NSLayoutConstraint+Description.shortDescription
update performance - AnchorsBuilder.build[Block|Array]
update performance - typecheck 50ms
update typecheck script
update performance - NSLayoutConstraint+Description type-check
update performance - function body type-check 100ms
update performance - typecheck Anchors 50ms
update performance - some function body type-check 50ms
cores move to Anchors
convenience move to Anchors
relation move to Anchors
models move to Anchors
layoutguide move to Anchors
update performance test
Activator.layoutIfNeeded run time performance improvement
performance test accuracy up
remove Anchors+LayoutGuide.swift
remove constraint sorting from Anchors
update constraint sorting from Anchors
update how to store sublayout and anchors in ViewLayout
update build performance of ViewLayout.sublayout(_:)
remove script
Removed optional in LayoutExplorer.Component.anchors
remove ViewInformationSet
Revert "remove ViewInformationSet"
Remove unnecessary type validation
add swift ui sample
remove swift ui sample, update tests
add archive.sh
add xcframework
support xcframework
update github action command
add SwiftLayoutWithoutAssistantsView
add SwiftLayoutWithoutIdentifyingView
remove ViewInfomationSet
update archive for xcframework
LayoutElements - create viewDic from components
update xcframework
fix archive process (#170)
remove performance test
remove xcframework things
Revert "feature/remove xcframework"
Revert "Revert "feature/remove xcframework"" (#174)
remove performance test resources
add README_ko.md (#175)
Contributing (#177)
Cleanup for 2.1.0 (#178)
update version of the readme to 2.1.0
clean Anchors.size
add test
fix SwiftLayoutPrinter misprinting
reordering size and add default value to size(length:)
improving of README
printing second item if with size attributes and none zero constant
fix test failure
support UIStackView (#182)
improvement Anchor.size
add ModularLayout (#184)
Feature/animatable layout property (#187)
re-architecting Anchors (#186)
Update README.md
Update README_ko.md
remove Equatable from AnchorsAttribute
remove unnecessary tests
subscripts for any object call self infinite in some conditions (#189)
ViewTags.tag to generic for prevent type erasing (#190)
Revert "ViewTags.tag to generic for prevent type erasing (#190)" (#191)
Revert "subscripts for any object call self infinite in some conditions (#189)" (#192)
Split the library into SwiftLayout and SwiftLayoutUtil (#188)
fix typo

Thank you for your effort.
Again, we took one step forward. 🎉

Thank you for your devotion.
I feeling we in very enterprising way


TTA Method in DeeplabV3+ | Evaluation

What is the intension of the TTA method? Does it deliver better results?
I want to use this network as an input to another network and im not sure, whether I should use the model.tta method.
with torch.no_grad():
    for batched in valid_loader:
        images, labels = batched
        images_np = images.numpy().transpose(0, 2, 3, 1)
        labels_np = labels.numpy()

        images, labels = images.to(device), labels.to(device)
        preds = model.tta(images, net_type='deeplab')
        preds = preds.argmax(dim=1)
        preds_np = preds.detach().cpu().numpy()

        images_list.append(images_np)
        labels_list.append(labels_np)
        preds_list.append(preds_np)

        if len(images_list) == 4:
            break

TTA stands for Test Time Augmentation.
If you want to improve accuracy, please use TTA.
If response speed is important, please use pred_resize method.
https://github.com/nyoki-mtl/pytorch-segmentation/blob/7a30cabb1e169f819329a5e3bd9d39f21d41c1c1/src/eval_cityscapes.py#L32


Add Check Service Principal workflow

Context
Add a workflow that runs on a schedule that checks the secret expiry of the SP secret. Alert if it's in the next N days.
Changes proposed in this pull request
Add workflow
Guidance to review
Test: https://github.com/DFE-Digital/apply-for-teacher-training/runs/8089107490?check_suite_focus=true
Link to Trello card
See Conversation
Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] 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
[ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[ ] API release notes have been updated if necessary
[ ] Required environment variables have been updated added to the Azure KeyVault

 Add & Update CheckServicePrincipal workflow for BAT


Assets cleanup

Some cleanup on our assets. See commits for each change
Related to #320

Ignore the codacy check for now, I did the setup of codacy for this repo just to test some stuff.


Unit test support

I would like an easier way to unit test my ApiResource classes, specifically, to verify how they are serializing in regards to related resources.
It would be helpful if ResourceSerializer were made public for this purpose. Then I can write unit tests that are similar to the unit tests found in ResourceSerializerTests.

I want the bulk of the implementation to live in an internal class, because I want to be free to e.g. add dependencies or refactor. However, I do see the value in being able to serialize things outside of web api.
Maybe we can add a new class that calls into the serializer internally, that can be used for this purpose. I guess it would supply things like the PaginationContext somehow, without exposing them to the outside world. Would that serve your needs?

Sure. That would work.
On Tue, Oct 27, 2015 at 10:00 AM, Jouke van der Maas <
<EMAIL_ADDRESS>wrote:

I want the bulk of the implementation to live in an internal class,
because I want to be free to e.g. add dependencies or refactor. However, I
do see the value in being able to serialize things outside of web api.
Maybe we can add a new class that calls into the serializer internally,
that can be used for this purpose. I guess it would supply things like the
PaginationContext somehow, without exposing them to the outside world.
Would that serve your needs?
—
Reply to this email directly or view it on GitHub
https://github.com/joukevandermaas/saule/issues/17#issuecomment-151530796
.

:+1: I'll see if I can get something working for you this week.


[backup] asyncify restore cli

Motivation
in preparation for new Adaptor trait.
Have you read the Contributing Guidelines on pull requests?
yes
Test Plan
unit tests
Related PRs

@bors-libra r=wqfish

:pushpin: Commit 569fb0a has been approved by wqfish

:hourglass: Testing commit 569fb0a2d73777e91c55e2df3b186420d0bd5fdc with merge 1303b95bce2c3c2ecd35e2028af609140cf71af4...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: wqfish
Pushing 1303b95bce2c3c2ecd35e2028af609140cf71af4 to master...