WMS ID - 9381

Add here your WMS ID
General requirements

Information in the workshop is adequate and updated
Code is correct and working
Links are correct
Make sure you enter the help email link in your manifest.json
Please make sure WMS URLs are updated as needed after your PR is approved

Checklist - Refer to the QA document for the complete list
Please confirm that the following is completed before submitting your PR

[ ] All filenames are lower case (including folders, images, files, etc.)
[ ] Filenames are descriptive
[ ] Your workshop folder structure should be similar to the one used in the sample workshop (https://github.com/oracle-livelabs/common/tree/main/sample-livelabs-templates/sample-workshop)
[ ] Are you using multiple versions (desktop/, sandbox/, tenancy/)? Make sure that each of them contains a manifest.json and an index.html
[ ] Image references in markdown contain an alternate text

Since the lab is to be consumed as part of CLoudWorld hands-on sessions and hence Updated the 'CloudWorld 2022! GetHelp' page

Done the required changes as mentioned.

@anuj-git1412 You need to create separate folders for the CloudWorld version and not update the existing manifest.json files. Please create a separate ocw-desktop, ocw-freetier and ocw-livelabs folders under workshops folder. Copy the index.html and manifest.json files with Get help changes into each folder and revert the changes to existing ones.

@anooshapilli : Done the required changes as mentioned.

The Need Help section should stay as is in the manifest files in livelabs, freetier and desktop folders. There needs to be new folders and files created (you can just copy them and rename the folders)

The Need Help section should stay as is in the manifest files in livelabs, freetier and desktop folders. There needs to be new folders and files created (you can just copy them and rename the folders)

The request to update the 'Need Help' section for the CloudWorld specific folders came from the senior management who are going to present this lab during CloudWorld.

Correct you need to update the manifest.json in the ocw-freetier, ocw-livelabs, ocw-desktop - these are the manifest files the workshop will point to at Oracle Cloud World. The manifest.json in freetier, livelabs and desktop will stay the same for the workshop after cloud world.


Add compress option for ExportToTiff

This suggested change adds the rasterio 'compress' option to the ExportToTiff task.
Apologies for the slightly dirty set of commits - forgot to branch first.

Hi @atedstone,
Thank you for you contribution!
Could you please change target branch to develop and resolve possible conflicts? I will then review and merge the PR.

Hi @atedstone,
I wanted to bump up this PR and what @devisperessutti said about changing the base of your branch and the target branch to develop.
Will wait a bit for feedback otherwise we will probably close this PR and create a new one ourselves.

This PR has been merged in https://github.com/sentinel-hub/eo-learn/pull/219
Thank you for your contribution. :+1:


UserWarning: You have unused kwarg parameters in Row, please remove

Run fast_stable_diffusion_AUTOMATIC1111.ipynb
Start stable-diffusion
3 Got error - /usr/local/lib/python3.8/dist-packages/gradio/deprecation.py:43: UserWarning: You have unused kwarg parameters in Row, please remove them: {'equal_height': True}
warnings.warn(
/usr/local/lib/python3.8/dist-packages/gradio/deprecation.py:43: UserWarning: You have unused kwarg parameters in Button, please remove them: {'full_width': True}
warnings.warn(

check with A1111

check with A1111 - That is, what should I do? I do not get you. This is in Google Colab.

{'equal_height': True} is used by A1111, not me, I'm not responsible for the A1111 repo

Is it possible to do something so that everything works?

it's just a warning that doesn't break the ui

Well, it doesn't start for me and that’s it. I do not know why.

run the colab without the first cell and see if it works

 {'equal_height': True} is used by A1111, not me, I'm not responsible for the A1111 repo
it should be a extensions \npw.py
line 20

with gr.Row(equal_height=True):
I try change to
with gr.Row():
then no this error,
this should be a layout, may be still working, after change!


Validation without using tag in struct definition

Package version eg. v9, v10:
v10
Issue, Question or Enhancement:
Hi everyone,
I'm totally new on go-playground/validator.
Is it possible to validate a struct not using tag on each field ?
For instance:
type User struct {
	Email string
}

func (i *User) Validate() error {
	// HERE Use of validate.Struct( ....) or something else
}

Thanks in advance :+1:

Closing this, seems that it has been duplicated :( here https://github.com/go-playground/validator/issues/722

Closing this, seems that it has been duplicated :( here https://github.com/go-playground/validator/issues/722


Unable to install

When running:
pnpm add -g @archetype-org/create-bos-app

I get
ERR_PNPM_FETCH_404  GET https://registry.npmjs.org/@archetype-org%2Fcreate-bos-app: Not Found - 404

@archetype-org/create-bos-app is not in the npm registry, or you have no permission to fetch it.

@elliotBraem Thanks for reporting. It should be fixed now - was private for some reason. Let me know if you can install it now


Add basic composer.json file

Adds ability to install via composer.

Sorry for the slow response, didn't think to check this dusty old thing for updates :)
Can merge, thanks!


Upgrade to Node 20 on CI, enable actions concurrency

The snap is excluded from this because there is no Node 20 snap yet.

LGTM, thanks :)
If you want to get the concurrency changes in here too, otherwise I can get another PR for them.

Added the concurrency group to all pull-* pipelines. See here for the description of github.ref. So it should now cancel previous runs in the same concurrency group. I will test with a push now.

Cancelling seems to work fine:

Maybe the icon would go to a grey stop sign if all steps in the build are cancelled, but I don't know how to cancel giteabot 😆.

Also I wonder why giteabot is still pending on the cancelled build.

Giteabot won't go grey, it'll stay "pending" until the PR receives all approvals.

I'm not sure if you are seeing this too, but in the required checks area below there are duplicated checks. Perhaps the concurrency stuff is indeed best saved for another PR so as to not hold this one up. Sorry for the extra work you have done.

Hmm I don't see duplicates but there are "6 successful, 8 expected" and the expected ones do not run, strange.

Trying this now: https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#example-only-cancel-in-progress-jobs-or-runs-for-the-current-workflow

Ah yup, I'm not seeing any dupes anymore. What you have done has worked :)

There is one more variation of this cancelling which can be configured so that only pull requests cancel, release builds and main branch builds won't cancel. I'll copy the syntax that the node repo uses.

Done, this new cancellation will only cancel on pull requests. I see the workflows currently only run on: pull_request, but should we ever decide to run them on main branch, this syntax will be good to have.


Carousel with two images slide transitions the wrong way at edge.

components: Carousel
reactstrap version #6.5.0
import method csj
react version #16.6.3
bootstrap version #4.1.3

What is happening?
An image carousel with only two images will not slide transition from slide at index 1 to slide at index 0 in the correct direction. Instead, it slides back (instead of back around, if that makes sense?). Clicking the carets is still semi-user friendly, but if the carousel is swipeable, then the user interaction makes no sense.
What should be happening?
Move from index 0 to index 1, with slide transition right to left. Then again in the same direction.
Steps to reproduce issue

Setup simple carousel as seen here: https://reactstrap.github.io/components/carousel/
Remove one of the items so there are only two.

Error message in console
No error is seen, the logic is carried out, but it may be incorrect.

Code
Second block in if statement, if activeIndex is 1 and nextProps.activeIndex is 0, it will swipe left.
componentWillReceiveProps(nextProps) {
    this.setInterval(nextProps);
    // Calculate the direction to turn
    if (this.props.activeIndex + 1 === nextProps.activeIndex) {
      this.setState({ direction: 'right' });
    } else if (this.props.activeIndex - 1 === nextProps.activeIndex) {
      this.setState({ direction: 'left' });
    } else if (this.props.activeIndex > nextProps.activeIndex) {
      this.setState({ direction: this.state.indicatorClicked ? 'left' : 'right' });
    } else if (this.props.activeIndex !== nextProps.activeIndex) {
      this.setState({ direction: this.state.indicatorClicked ? 'right' : 'left' });
    }
    this.setState({ indicatorClicked: false });
  }

@klapeyre did you ever find a solution to this issue?  I am up against the same problem

I think I ended up patching it client side, guarding for the case where there are two images in the carousel.

@klapeyre yeah I am trying to do the same thing, but haven't come up with a successful patch yet.  Mind sharing any tips on what you did?

With the example they provided in the docs here (https://reactstrap.github.io/components/carousel/) - I added a check in the next() and previous() function implementations. If your activeIndex is 1 and array length is 2 (this confirms your carousel has two images and you are at the end of it), I set the index to 2 and then immediately after that set the index to 0. This gives the visual impression that it is still swiping right. Do the same for the implementation of previous().


Remove suite body tracking from SemanticModel

Summary
The SemanticModel currently stores the "body" of a given Suite, along with the current statement index. This is used to support "next sibling" queries, but we only use this in exactly one place -- the rule that simplifies constructs like this to any or all:
for x in y:
    if x == 0:
        return True
return False

Instead of tracking the state, we can just do a (slightly more expensive) traversal, by finding the node within its parent and returning the next node in the body.
Note that we'll only have to do this extremely rarely -- namely, for functions that contain something like:
for x in y:
    if x == 0:
        return True

Can you add your benchmark numbers from running only the affected rule in the CPython repository before and after? To get an idea of how this affects performance.

I spent some time trying to benchmark this. cargo bench shows no significant difference, but I don't think any of those files actually trigger this rule. So I then added a dedicated file from CPython that does trigger it, and again, no noticeable difference.

I have a lot of trouble running the CPython benchmark for specific rules because the timing is completely dominated by everything else.


How to configure the proxy for the service account on prem?

In the documentation there is a segment Use an existing service account, that says "If you [..] use a proxy that requires authentication, you need to use a managed service account".
We have a situation that requires a proxy to talk to the internet (and Azure). But it is unauthenticated. How do you create and/or (re-)configure the managed service account for Azure AD Connect?
It would be nice to have some guidance for that. Maybe the hints elsewhere about configuring a few XML files need some subtle changes for un-authenticated proxys? Or maybe there is an easy way to add an unauthenticated proxy to a managed service account, or maybe the whole AD server that will run Azure AD Connect (if that is recommended, I suppose not).
The default created service account by the wizard does not work for us.
We tried several approaches, but gave up after a few hours.
It would be an improvement if proxy config for Azure AD Connect was better documented.

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

ID: 21fa537d-81fc-ccc9-f7f8-1486e71dea62
Version Independent ID: 8e578d88-8bcb-a229-95f6-10b760d6f563
Content: Azure AD Connect: Custom installation
Content Source: articles/active-directory/hybrid/how-to-connect-install-custom.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@HenkPoley
Thanks for your feedback! We will investigate and update as appropriate.

For Azure AD Connect there already is some documentation here: https://docs.microsoft.com/en-us/azure/active-directory/hybrid/tshoot-connect-connectivity
Maybe I am mistaken, but C:\Windows\Microsoft.NET\Framework64\v4.0.30319\Config\machine.config seems to be a global configuration. Meaning, any .NET application that goes through "Framework64 v4.0.30319" will get access to the proxy.
Which does not quite seem security-in-depth.
I would expect there to be some kind of proxy config for the managed service account, or maybe a config file for the Azure AD Connect service executable.

@HenkPoley Thank you for the issue.  I have created a work item for this and will be addressing it.  Unfortunately I don't have an ETA for when this change will go in because things move around alot.  I will post an update to this thread once the changes have been staged and are going live.
Thanks!
Bill
#please-close


Question about BN in nas_fpn

Hi, thanks for your code.
I try to implement NAS-FPN in pytorch, but there seems to be some errors with my BN layer (Both 1-stage(RetinaNeet) / 2-stage). I freeze the BN layer in r50 backbone, fine-tune the BN in NAS-FPN (initialized with weight=1 bias=0), am I right?
The log is like this, training seems OK, but eval is always 0 (0.001).
Average Precision  (AP) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.000
Average Precision  (AP) @[ IoU=0.50      | area=   all | maxDets=100 ] = 0.000
Average Precision  (AP) @[ IoU=0.75      | area=   all | maxDets=100 ] = 0.000
Average Precision  (AP) @[ IoU=0.50:0.95 | area= small | maxDets=100 ] = 0.000
Average Precision  (AP) @[ IoU=0.50:0.95 | area=medium | maxDets=100 ] = 0.000
Average Precision  (AP) @[ IoU=0.50:0.95 | area= large | maxDets=100 ] = 0.000
2019-08-08 01:31:15,314 - INFO - Epoch [1][7330/7330]	lr: 0.02000, bbox_mAP: 0.0000, bbox_mAP_50: 0.0000, bbox_mAP_75: 0.0000, bbox_mAP_s: 0.0000, bbox_mAP_m: 0.0000, bbox_mAP_l: 0.0000, bbox_mAP_copypaste: 0.000 0.000 0.000 0.000 0.000 0.000
2019-08-08 01:31:39,196 - INFO - Epoch [2][50/7330]	lr: 0.02000, eta: 8:01:08, time: 0.477, data_time: 0.070, memory: 4057, loss_rpn_cls: 0.0640, loss_rpn_bbox: 0.0223, loss_cls: 0.1917, acc: 96.7520, loss_bbox: 0.0561, loss: 0.3341
2019-08-08 01:31:56,286 - INFO - Epoch [2][100/7330]	lr: 0.02000, eta: 8:00:42, time: 0.342, data_time: 0.007, memory: 4057, loss_rpn_cls: 0.0718, loss_rpn_bbox: 0.0185, loss_cls: 0.2137, acc: 96.5977, loss_bbox: 0.0619, loss: 0.3659

There is another little thing. The RetinaNet (P2 to P6 in your code) is different with the paper (P3 to P7), will (P2 to P6) better than (P3 to P7)?
Thanks again~


Fix Python3 compatibility

This PR fixes some Python 3 compatibility issues in moe code.

:white_check_mark: Build SMT 0.1.354 completed (commit https://github.com/SMTorg/SMT/commit/35242e81a2 by @relf)


FreeCodeCamp Derventa, Bosnia and Herzegovina request

Please provide the following information to aid in adding your group to the directory:
Esssential

[https://www.facebook.com/groups/freecodecamp.derventa/ ] Group URL (Web address)
[ Derventa ] Town or City group is located in
[ Republika Srpska ] State / County / Region
[ Bosnia and Herzegovina ] Country

Nice to have

[ 44.97839 17.90779 ] Co-ordinates

If you have one

[ ] URL of Group Photo

PR submitted


demo new service

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
If updating an existing version, please select the specific language SDKs and CLIs that must be refreshed after the swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No refresh required for updates in this PR

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following appy to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you are using OpenAPIHub to initialize the PR for adding a new version. More details, refer to the wiki.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[ ] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

azure-sdk-for-js
output-folder: sdk/demoservice/newservice-demo-rest
require:
 - specification/demoservice/data-plane/readme.md

azure-sdk-for-python
output-folder: sdk/demoservice/azure-newservice-demo
require:
 - specification/demoservice/data-plane/readme.md

azure-sdk-for-java
output-folder: sdk/demoservice/azure-newservices-demo
namespace: com.azure.iot.newservice.demo
data-plane: true
require:
 - specification/demoservice/data-plane/readme.md

azure-sdk-for-net-track2
output-folder: sdk/demoservice/Azure.IoT.NewServiceDemo
require:
 - specification/demoservice/data-plane/readme.md

/azp run


THRIFT-5456: Disable SIGPIPE on TServerSocket

TServerSocket sets up many socket options, but unlike TSocket, it doesn't disable SIGPIPE.
One example for how this can be triggered is the server receives a request, but takes too long to process it. The sender then times out, and potentially closes its sockets. The server then completes the request, and then attempts to write back the result. Since the socket is now closed, a SIGPIPE signal is sent to the process, and if that signal isn't ignored, the process would crash, with a potential -13 exit code.

Could you trigger the build again, e.g. by force-pushing the branch?

Is this also missing in TNonblockingServerSocket?

@kainjow any news here?

@kainjow any news here?

Will take a look soon.

TNonblockingServerSocket

Done!


maven: upgrade to latest spring-framework release

Upgrades spring framework and reload4j dependencies
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

@blueorangutan package

@rohityadavcloud a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 3144

@blueorangutan test matrix

@nvazquez a Trillian-Jenkins matrix job (centos7 mgmt + xs71, centos7 mgmt + vmware65, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests

Trillian test result (tid-3875)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 38276 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6250-t3875-kvm-centos7.zip
Smoke tests completed. 78 look OK, 3 have errors
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_01_add_primary_storage_disabled_host
Error
0.62
test_primary_storage.py

test_01_primary_storage_nfs
Error
0.13
test_primary_storage.py

ContextSuite context=TestStorageTags>:setup
Error
0.23
test_primary_storage.py

test_03_deploy_and_scale_kubernetes_cluster
Failure
33.09
test_kubernetes_clusters.py

test_07_deploy_kubernetes_ha_cluster
Failure
75.03
test_kubernetes_clusters.py

test_08_upgrade_kubernetes_ha_cluster
Failure
63.96
test_kubernetes_clusters.py

test_09_delete_kubernetes_ha_cluster
Failure
0.04
test_kubernetes_clusters.py

ContextSuite context=TestKubernetesCluster>:teardown
Error
77.10
test_kubernetes_clusters.py

test_01_secure_vm_migration
Error
147.82
test_vm_life_cycle.py

test_02_unsecure_vm_migration
Error
272.06
test_vm_life_cycle.py

test_03_secured_to_nonsecured_vm_migration
Error
142.19
test_vm_life_cycle.py

test_08_migrate_vm
Error
43.83
test_vm_life_cycle.py

test_09_expunge_vm
Error
144.39
test_vm_life_cycle.py

test_10_attachAndDetach_iso
Error
3.01
test_vm_life_cycle.py

test_11_destroy_vm_and_volumes
Error
3.00
test_vm_life_cycle.py

ContextSuite context=TestVMLifeCycle>:teardown
Error
6.01
test_vm_life_cycle.py

Trillian test result (tid-3874)
Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7
Total time taken: 38978 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6250-t3874-xenserver-71.zip
Smoke tests completed. 78 look OK, 1 have errors
Only failed tests results shown below:

Test
Result
Time (s)
Test File

ContextSuite context=TestListIdsParams>:teardown
Error
1.18
test_list_ids_parameter.py

Trillian test result (tid-3876)
Environment: vmware-65u2 (x2), Advanced Networking with Mgmt server 7
Total time taken: 39103 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6250-t3876-vmware-65u2.zip
Smoke tests completed. 79 look OK, 0 have errors
Only failed tests results shown below:

Test
Result
Time (s)
Test File

@blueorangutan test

@Pearl1594 a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@blueorangutan test

@rohityadavcloud a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-3885)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 32756 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6250-t3885-kvm-centos7.zip
Smoke tests completed. 93 look OK, 0 have errors
Only failed tests results shown below:

Test
Result
Time (s)
Test File


Initial version

This is the initial version of access token validation for Identity Server. I written any tests on it as I am not entirely sure if I implemented it correctly and in an expected way. This pull request is mainly to get more feedback. There are a few obvious missing parts:

[ ] Complete InMemoryValidationResultCache.GetAsync implementation (slipped :disappointed:)
[ ] Preserve access token implementation.
[ ] Working sample.
[ ] Extensive logging.
[ ] Tests.

Grammar nazi: introspection is sometimes called introception (in the sample)
Q: So is this supposed to be the dnx version of the middleware?

yes - at some point this will host the asp5/coreClr version. Right now it is more an experiment ;)

sorry, haven't been able to update this. I will get onto it this week.

no probs - we are all busy


Implement mechanism for deactivating users

Description
This PR modifies the logic for refreshing the access token so that the username in its payload is checked to see if it is part of the active usernames list before it refreshes it.
Testing instructions
Add a set of instructions describing how the reviewer should test the code

[ ] Review code
[ ] Check Actions build

Agile board tracking
closes #12

I made some changes so that the env var values used during testing are overridden using pytest.ini instead of using the pytest_configure hook. As such, the tests should now be run with the following command: pytest -c test/pytest.ini test. README was updated accordingly. Please also run pip install .[dev] as pyproject.toml has been updated.


User data usage in examples/compositor/wl_compositor.c

wl_compositor.c stores data in the wlr_surface struct. It should not do this as it will soon be merged into wlroots, and the data field in wlr_surface is to be changed by the user only.

you're right. It's only there as long as we use wl_compositor to iterate over the created surfaces since it needs this data to keep track of surface destruction.

I'll sort this tonight or this weekend.

I did a review and I guess we fixed it.


Spell check support in skylark files.

Skylark Docstrings (skydoc) in particular could do with this.

Hi there! We're doing a clean up of old issues and will be closing this one. Please reopen if you’d like to discuss anything further. We’ll respond as soon as we have the bandwidth/resources to do so.


For HTTP recordings at least, consider support Servirtium's markdown format (optionally)

Ref: https://github.com/servirtium/README
Not to take away from Mountebank's excellent APIs, but support of Servirtium's markdown format could be good too :)

I totally agree.
For now, the best option is a CLI converter than can convert Servirtium's markdown format to something that can be passed to the --configfile CLI option (see http://www.mbtest.org/docs/commandLine#config-file). Right now, that means an EJS template, or simply inlining everything directly into JSON. Optionally, the CLI converter could take the reverse transformation too -- converting a JSON output from mountebank and saving it in Servirtium markdown.

@paul-hammant Good news. The 2.4.0 release supports custom formatters, which allow anyone to create a formatter that can convert to and from Servirtium format. Docs available at http://www.mbtest.org/docs/commandLine#custom-formatters


spring boot build info resource detector

Depends on https://github.com/open-telemetry/opentelemetry-java-instrumentation/pull/10457

@zeitlinger now that the upstream PR is in, do you want to take this out of draft?

@zeitlinger now that the upstream PR is in, do you want to take this out of draft?

It will be deployed in 2.2.0 - which is noted in https://github.com/open-telemetry/opentelemetry.io/issues/4027


Unclear copyright situation

I'm trying to package this crate for inclusion in debian, but the copyright information is a bit unclear.
The cargo.toml file state that Pascal Hertleif is the author, but LICENSE-MIT state that 'The assert_cli Developers' have copyright.
My guess is that the copyright claim in LICENSE-MIT is incorrect, but it would be great to have that verified.

Hah, I guess I forgot to change this when I copied over the license file. Legally, this is fine -- 'the assert_cli developers' at time of copying the file include me and people I know work on Rust CLI projects. I've updated the license to now say 'roff-rs developers' and that is probably also the best thing to use as author. The metadata in the Cargo.toml doesn't include every contributor. I hope that works for you.


Check platform before querying readableVersion

For web platforms, querying the readable version resulted in a crash. The extra platform check makes sure no undefined methods are called.
Description
Fixed issue #796
Added platform check in getReadableVersion and getReadableVersionSync methods.
Compatibility

OS
Implemented

iOS
✅

Android
✅

Windows
✅

Web
✅

Checklist

[X] I have tested this on a device/simulator for each compatible OS
[ ] I added the documentation in README.md
[ ] I mentioned this change in CHANGELOG.md

It is slightly better and I really appreciate the willingness to PR but as I mentioned on #796 because react-native-web is more of a polyfill than anything (which in my mind means the methods themselves shouldn't really be called on the platform, but including the module shouldn't crash) I'm not that interested in return correctness and we already platform-check the underlying methods, so I just chained through to them

If you are willing to actually flesh out web implementations though, that's a much different story and I'd happily merge real web implementations of things


App redirect issue when LinkedIn App is install.

Hi
Demo is running fine in simulator when device is not installed.
But when I tired in real device then its not redirect back to my app when LinkedIn app is install in real device.
Please help me.

Same issue here

Error
li4127213://error?state=GET-ACCESS-TOKEN&info=INVALID_APP_ID&domain=LISDKInvalidRequestError&description=Invalid%20Application%20ID

did you guys do this: Associate your iOS app with your LinkedIn app ?


1Password-gui: 8.0.33-53.BETA -> 8.0.34-stable

This includes an overhaul of the install script, using patchelf to set
the interpreter and the libraries. This reflects a lot of what I've
learned about electron + nixos over the last few months.
This change is motivated by the fact that 1Password's version of
electron changed, probably leading to crashes that I've been seeing when
I try to update the Nix derivation to the latest versions.

Motivation for this change
Things done

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

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

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

Result of nixpkgs-review pr 123515 at 15394ebb run on x86_64-linux 1

2 packages built successfully:

_1password-gui
tests.trivial

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/applications/misc/1password-gui/default.nix:77:3:
   |
77 |   installPhase = ''
   |   ^

Typically the commit messages for updates like this follow the pattern of:
"attribute_name: oldversion -> newversion"

Fixed!

Sorry to be a broken record, but: looks like the commit title didn't change :grimacing:

Sorry to be a broken record, but: looks like the commit title didn't change grimacing

Maybe this time I actually fixed it. :|

I'm hoping for a review from @danieldk or @timstott . And anyone else who has bundled closed source electron apps. I used the Slack app as a bit of a model in my overhaul here, and if this code is good I'm going to apply the same thing to another electron app.

The changes look like a large improvement to me. I made some small changes, the first two are minor nitpicks:
1. Changed the title of the commit message, the first item should ideally be the exact attribute name.

2. Changed the argument order. The canonical order in nixpkgs is: `lib`/`stdenv`, functions used in the derivation, native build inputs, build inputs, check inputs. Within each group, they are typically alphabetically sorted.

3. I moved `rpath` into the `installPhase` scope with a `let`-binding. Having `rpath` as an argument to `stdenv` adds it as an environment variable in the builder. Scoping it keeps the environment clean :).

Though, it seems that the icons and XDG desktop entries are not installed anymore (see the comment). Is there are particular reason for removing them?

Damn, I just stomped some of your changes. :( Let me see if I can rebuild them again.

Damn, I just stomped some of your changes. :( Let me see if I can rebuild them again.

Okay, I think I restored your work now. Please give it a quick check to make sure it's right.


Interpreted vs compiled

Yo! :)
I have read the section "Interpreted vs. Compiled Code" in the README. I think that is not exactly the way it works.
Elixir always compiles your code. Also, always runs your code. The only difference between elixir and elixirc is that the compilation of modules generates a .beam file as a side-effect (I am sure you know this, only building my argument).
The top-level code is no different than regular modules. To execute that code, Elixir wraps the Erlang AST it generates from the Elixir AST into a function called __FILE__/1, and injects that dynamically into a module called elixir_compiler_0. All building Erlang AST by hand. Then compiles that module, and executes the script by calling said dummy function. That is done here.
You can inspect that with this script:
{_, mfa} = :erlang.process_info(
  self(),
  :current_function
)
IO.inspect(mfa)

which outputs
{:elixir_compiler_0, :__FILE__, 1}

So, in the end, you are running compiled code anyway. The only penalty scripts have is that every time you run them, they have to be first compiled. But the code itself should run at the same speed.

Hey @fxn  :wave:
Thanks for coming by here and taking the time to open such a well formed issue :tada: :dancer:
I'm... not quite certain as I don't know this intrinsics of elixir well enough - I'd be so inclined to try and ping @michalmuskala as he brought it to my attention to see what light he can shed. Otherwise I might have to spend some time on the weekend digging into that code :)
Thanks again! :rabbit:
Nala and Al think you're pretty chill!

The key point that may be disconcerting here is that the Erlang compiler is able to compile Erlang Abstract Code on the fly, and that the BEAM is able to load binary code dynamically.
That is what Elixir exploits, these are the fundamental Erlang tools it leverages.
So when you define a module in Elixir source code, both elixir and elixirc define the module in memory, it gets loaded into the VM. Only elixirc generates an artifact for the module. Indeed, you can have three modules in one file. They will be all defined and loaded into the BEAM, and if under elixirc three files generated called after the module names.

I'm not worried about the capability to define multiple modules/exs vs ex files, that's still something I need to fix there (as was pointed out to me before just didn't get tot it yet... bad tobi).
More that part that if I have top level code in a script like the benchmarking code, that that gets automatically wrapped into a module and called. If it is like that, that's amazing. I actually wondered why it wouldn't be like that or that that'd be a nice optimization. :)
I'll try and take a deeper look this evening, thanks again!

@PragTob I am about to publish a post with a summary of how it works :).

There you go! https://medium.com/@fxn/how-does-elixir-compile-execute-code-c1b36c9ec8cf

Yes, after looking more closely at the compiler, and after the great talk @fxn gave at ElixirConf, I think I was confused (I wrote this even before the article, that cleared things up even more).
The Elixir will wrap code outside of modules in a "fake" module, compile that and execute. With defmodule inside, this execution would lead to defining a real module. The case where code is interpreted and not compiled is IEx. One could argue, though that this is just an implementation detail and not a guarantee - it could be changed to work using the interpreter instead of compiling the "wrapper" module first.
So it seems I only introduced noise and occupied the time of everybody. I'm really sorry about that.

@michalmuskala no problem at all man! It took me some weeks to order the ideas and see how was all happening. Once you see the internals, the observation that the speed is the same follows, but before that you could totally believe there was some sort of slow "interpretation" of code.
Also, you prevented me from deleting code from Kernel no less! 😄  ❤

Helloooo!
Cool to hear.
@michalmuskala no worries at all, in fact still lots of props and thanks for bringing it up. I always worried about it a bit and wanted to read up on it and now we finally got to it. We can all err, especially with complex stuff like this. Please do continue to bring up things like this - it's a great help! :rocket:
I need to get going to work now but will adjust the README time. Thanks to you both, and I just asked them and Nala and Al agree that you are both great @michalmuskala @fxn ! :heart: :rabbit: :tada:


Link parsing of README is inconsistent with Github Markdown

Issue
You can see this issue in the README.md file in the status module (GitHub vs Deno). The square brackets in the headings are not followed by a () but they are still parsed as links in the Deno markdown rendering. This is not the case in the Github markdown rendering of the same README.
Possible solution
// components/Markdown.tsx
const renderers = (canonicalURL: string) => ({
  // ...
  // adding linkReference in the renderers
  linkReference: LinkRenderer,
  // ...
});

// components/Markdown.tsx
function LinkRenderer(props: LinkRendererProps) {
  // ...
  // adding a check if the href is actually present
  if (href) {
    return (
      <a href={href} className="link">
        {props.children}
      </a>
    );
  }
  return (<>[{props.children}]</>):
}

Should I open a pull request to implement these changes?

Issue is in react-markdown. But it's not in LinkRenderer issue. Issue is in HeadingRenderer

But it's not in LinkRenderer issue. Issue is in HeadingRenderer

My possible solution was a quick fix on the deno website side. I too saw that the problem resides in fact on the react-markdown but I figured a maybe it was meant to be a feature. Thanks on opening the issue though :)


Better leverage of stream-json to speed things up a little.

This PR is meant to reduce the code in CompressedJSON.ts and speed things up a little bit by leveraging existing stream-json features:

No logic update was attempted. Whatever the code was doing before, it is doing now, right or wrong.
stream-json can pack values (keys, strings, numbers) itself, which is more efficient than doing it in the client code.
The processing pipeline length was reduced — each stream introduces an additional latency.

One algorithm improvement can be to collect only first 64 characters of a string and ignore the rest — it looks like longer values are not used anyway, but it can be a problem if user's data contains huge strings. If it is a worthwhile improvement, I can whip up another PR.
PS: stream-json was not updated. The same version is used.
PPS: I am an author of stream-json. So if you have some questions, requests, suggestions, you know who to ping. :-)

@uhop Thank you so much for this!
Right now it's failing with this:
ERROR: /app/src/quicktype-core/input/CompressedJSON.ts[249, 14]: This type is not allowed in the operand for the '!' operator because it could be undefined. Only booleans are allowed.
  | ERROR: /app/src/quicktype-core/input/CompressedJSON.ts[256, 27]: This type is not allowed in the condition because it could be undefined. Only booleans are allowed.

Sorry we don't have public access to our build logs yet :-(
Did you do any benchmarks by any chance?

Thanks for the log — will fix.
I did some rough benching, and the gain was small — around 5%. I guess the original code was already good (kudos!), and most of the time is spent somewhere else. I decided to go ahead and offer a PR mostly to reduce and streamline the code. OTOH, 5% for free is not that bad. ;-)

Done: types are updated in 208d40eb564a7f7ee6d71fc0295cef4124901995.

Thank you so much!


Finetune: AWAC

Wandb report

Seems like we need to fix the report for our scores:

https://wandb.ai/tlab/CORL/reports/-Offline-to-Online-AWAC--VmlldzozODAyNzQz#door
https://wandb.ai/tlab/CORL/reports/-Offline-to-Online-AWAC--VmlldzozODAyNzQz#relocate

@vkurenkov @Howuhh @Scitator


AddMultiplicationEquality and negative integers ( range [-1, 0, 1] )

What version of OR-tools and what language are you using?
Version: 7.7.7810
Language: Python
Which solver are you using (e.g. CP-SAT, Routing Solver, GLOP, BOP, Gurobi)
Solver: CP-SAT
What operating system (Linux, Windows, ...) and version?
Operating System: Windows
What did you do?
Steps to reproduce the behavior:

Run example program in Attachment
topology.zip

What did you expect to see
Solutions for the constraint Z = X*Y with domains [-1,0,1] of these variables
What did you see instead?
An error-message:
"Not supported
*** Check failure stack trace: ***
...
"
Make sure you include information that can help us debug (full error message, model Proto).
Anything else we should know about your project / environment
This is just an example, many constraints of type Z=X*Y will be automatically generated in the real program, where many variables with the range [-1, 0,  1] are kept in matrices. Multiplication of these matrices leads to such constraints.
Maybe I'm expecting too much using such constraints with a domain including negative, zero and positive values. Probably I can get around it by generating constraints as follows (with quite some extra variables and constraints as penalty ;-) . :
AbsX = abs(X)
AbsY = abs(Y)
AbsZ = abs(Z)
AbsZ = AbsX* AbsY
(X >= 0).OnlyEnforceIf(PositiveX)
(X < 0) . OnlyEnforceIf(PositiveX.not())
(Y >= 0).OnlyEnforceIf(PositiveY)
(Y < 0) . OnlyEnforceIf(PositiveY.not())
(Z >= 0).OnlyEnforceIf(PositiveZ)
(Z < 0) . OnlyEnforceIf(PositiveZ.not()))
(PositiveX == PositiveY).OnlyEnforceIf(PositiveZ)
(PositiveX != PositiveY).OnlyEnforceIf(PositiveZ.not())
Is this the way to go? Or was the original constraint  Z=X*Y  expected to work?

So I tried:
``
Z = X *Y, assuming range of all variables [-1,1]
def multiply(model, Z, X, Y):
AbsX = model.NewIntVar(-1, 1, "")
AbsY = model.NewIntVar(-1, 1, "")
AbsZ = model.NewIntVar(-1, 1, "")
PositiveX = model.NewBoolVar("")
PositiveY = model.NewBoolVar("")
PositiveZ = model.NewBoolVar("")
model.AddAbsEquality(AbsX, X)
model.AddAbsEquality(AbsY, Y)
model.AddAbsEquality(AbsZ, Z)
model.AddMultiplicationEquality(AbsZ, [AbsX, AbsY])
model.Add(X >= 0).OnlyEnforceIf(PositiveX)
model.Add(X < 0). OnlyEnforceIf(PositiveX.Not())
model.Add(Y >= 0).OnlyEnforceIf(PositiveY)
model.Add(Y < 0). OnlyEnforceIf(PositiveY.Not())
model.Add(Z >= 0).OnlyEnforceIf(PositiveZ)
model.Add(Z < 0).OnlyEnforceIf(PositiveZ.Not())
model.Add(PositiveX == PositiveY).OnlyEnforceIf(PositiveZ)
model.Add(PositiveX != PositiveY).OnlyEnforceIf(PositiveZ.Not())
Gives me:
solution 1 X=0 Y=0 Z=0
solution 2 X=0 Y=1 Z=0
solution 3 X=1 Y=0 Z=0
solution 4 X=-1 Y=1 Z=-1
solution 5 X=-1 Y=-1 Z=1
solution 6 X=1 Y=-1 Z=-1
solution 7 X=1 Y=1 Z=1
``
So that is doing the trick. Still the question if  this is the expected way of encoding such constraint, ... .  Thanks in advance!

First time in 20 years that I have seen a real need to multiplication
between 2 general integer variables.
Your encoding is very slow, you can use an allowed assignments constraint
with the 9 possible tuples.
Le lun. 10 août 2020 à 14:29, HenkVandecasteele<EMAIL_ADDRESS>a écrit :

So I tried:
``
Z = X *Y, assuming range of all variables [-1,1]
def multiply(model, Z, X, Y):
AbsX = model.NewIntVar(-1, 1, "")
AbsY = model.NewIntVar(-1, 1, "")
AbsZ = model.NewIntVar(-1, 1, "")
PositiveX = model.NewBoolVar("")
PositiveY = model.NewBoolVar("")
PositiveZ = model.NewBoolVar("")
model.AddAbsEquality(AbsX, X)
model.AddAbsEquality(AbsY, Y)
model.AddAbsEquality(AbsZ, Z)
model.AddMultiplicationEquality(AbsZ, [AbsX, AbsY])
model.Add(X >= 0).OnlyEnforceIf(PositiveX)
model.Add(X < 0). OnlyEnforceIf(PositiveX.Not())
model.Add(Y >= 0).OnlyEnforceIf(PositiveY)
model.Add(Y < 0). OnlyEnforceIf(PositiveY.Not())
model.Add(Z >= 0).OnlyEnforceIf(PositiveZ)
model.Add(Z < 0).OnlyEnforceIf(PositiveZ.Not())
model.Add(PositiveX == PositiveY).OnlyEnforceIf(PositiveZ)
model.Add(PositiveX != PositiveY).OnlyEnforceIf(PositiveZ.Not())
Gives me:
solution 1 X=0 Y=0 Z=0
solution 2 X=0 Y=1 Z=0
solution 3 X=1 Y=0 Z=0
solution 4 X=-1 Y=1 Z=-1
solution 5 X=-1 Y=-1 Z=1
solution 6 X=1 Y=-1 Z=-1
solution 7 X=1 Y=1 Z=1
``
So that is doing the trick. Still the question if this is the expected way
of encoding such constraint, ... . Thanks in advance!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/2133#issuecomment-671325476,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3M7WF4S2R5XJHX2CYDR77R4JANCNFSM4PZ4D7BA
.

@lperron Thanks for the advice!
Actually my coding above is wrong in the case of 1 negative value and the other value 0  :-( .
The assignment-based solution will be easier to verify!
Regards,
Henk

I have pushed support for product with variables spanning across 0.


[Feature request] Add 'Show Change Indicators' in Global settings

Change Indicators is slowing down the browsing. Please add Change Indicators toggle in Settings so it's automatically on/off.

Already added.


HDDS-10796. Avoid dummy KeyOutputStream in OzoneOutputStreamStub

What changes were proposed in this pull request?
OzoneOutputStreamStub.getKeyOutputStream() returns a dummy KeyOutputStream, which has no relation to the underlying stream (the one passed to parent OzoneOutputStream class).
This PR eliminates the need for the dummy KeyOutputStream.  The only caller (ObjectEndpoint) can access it via OzoneOutputStream, so the stub's custom logic is not bypassed.
https://issues.apache.org/jira/browse/HDDS-10796
How was this patch tested?
CI:
https://github.com/adoroszlai/ozone/actions/runs/8928970232

Thanks @ivandika3 for the review.


Facing types issue

please check screenshot

Hi, would you be able to provide some more detail about the issue that you're encountering? Specifically it would be helpful to know what TypeScript version and what Node version you are using. Thanks!

I am facing same issue in my angular project,

details of project are:
Angular CLI: 11.0.7
Node: 14.20.0
OS: win32 x64
Angular: 11.0.9
... animations, common, compiler, compiler-cli, core, forms
... localize, platform-browser, platform-browser-dynamic, router
Ivy Workspace: Yes
Package                         Version
@angular-devkit/architect       0.1100.7
@angular-devkit/build-angular   0.1100.7
@angular-devkit/core            11.0.7
@angular-devkit/schematics      11.0.7
@angular/cdk                    11.2.13
@angular/cli                    11.0.7
@angular/material               11.2.13
@schematics/angular             11.0.7
@schematics/update              0.1100.7
rxjs                            6.6.7
typescript                      4.0.8

Hi, would you be able to provide some more detail about the issue that you're encountering? Specifically it would be helpful to know what TypeScript version and what Node version you are using. Thanks!

Angular CLI: 11.0.7
Node: 14.20.0
OS: win32 x64
Angular: 11.0.9
... animations, common, compiler, compiler-cli, core, forms
... localize, platform-browser, platform-browser-dynamic, router
Ivy Workspace: Yes
Package                         Version
@angular-devkit/architect       0.1100.7
@angular-devkit/build-angular   0.1100.7
@angular-devkit/core            11.0.7
@angular-devkit/schematics      11.0.7
@angular/cdk                    11.2.13
@angular/cli                    11.0.7
@angular/material               11.2.13
@schematics/angular             11.0.7
@schematics/update              0.1100.7
rxjs                            6.6.7
typescript                      4.0.8

Thanks for the info, would it be possible for you to upgrade TypeScript to 4.5 or above to support the new syntax? That would mitigate the issue for you.

We do not support Node 14 either as it is long out of support since April 30th 2023.  We only support Node 18+ as per the current Node lifecycle.

Hi, would you be able to provide some more detail about the issue that you're encountering? Specifically it would be helpful to know what TypeScript version and what Node version you are using. Thanks!

using typescript version : 4.3.5
please check below ss

I think it should be the latest core ? the build has something like dist/commonjs, it comes from the recent change?

but i am using static version number not latest

Hi Team,
Issue fixed.
only import format use like es5


datepicker: selected date circle is out of center in mobile

Bug, feature request, or proposal:
bug
What is the expected behavior?
The selected day and today circle to be vertically centralized around the date number in mobile devices
What is the current behavior?
Both are not vertically centralized in mobile devices (at least in my Android).

What are the steps to reproduce?
Open the material.angular.io site in an Android device.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Android device
Is there anything else we should know?

Seems ok in IOS in both Chrome and Safari app


EF 6.2 Linq Entity Framework Ignoring IS NULL / IS NOT NULL on self referenced foreign key

The following code replicates the issue...
    class Program
    {
        static void Main(string[] args)
        {
            MyContext context =
                new MyContext("data source=(local);initial catalog=TestQuestion;integrated security=True;MultipleActiveResultSets=True;App=EntityFramework");

            IList<ChecklistQuestionEntity> questions =
                context
                    .Set<ChecklistQuestionEntity>()
                    .Where(q => q.OriginalQuestionID.HasValue)
                    .ToList();

            IList<ChecklistQuestionEntity> originalQuestions =
                context
                    .Set<ChecklistQuestionEntity>()
                    .Where(q => !q.OriginalQuestionID.HasValue)
                    .ToList();
        }
    }

    public class MyContext : DbContext
    {
        public MyContext(string nameOrConnectionString) : base(nameOrConnectionString)
        {
        }

        protected MyContext()
        {
        }

        protected override void OnModelCreating(DbModelBuilder modelBuilder)
        {
            // Commenting out the following resolves the issue
            modelBuilder.Entity<ChecklistQuestionEntity>()
                .HasRequired(cq => cq.OriginalQuestion)
                .WithMany(cq => cq.Revisions)
                .HasForeignKey(cq => cq.OriginalQuestionID)
                .WillCascadeOnDelete(false);
        }
    }

    [Table("cklChecklistQuestion")]
    public class ChecklistQuestionEntity
    {
        /// <summary>
        /// Gets or sets the ID
        /// </summary>
        [Key]
        [DatabaseGenerated(DatabaseGeneratedOption.Identity)]
        public int ID
        {
            get;
            set;
        }

        /// <summary>
        /// Gets or sets the OriginalQuestionID
        /// </summary>
        public int? OriginalQuestionID
        {
            get;
            set;
        }

        /// <summary>
        /// Gets or sets the <see cref="ChecklistQuestionEntity"/>
        /// </summary>
        public virtual ChecklistQuestionEntity OriginalQuestion
        {
            get;
            set;
        }

        [Required]
        public string Question
        {
            get;
            set;
        }

        public ICollection<ChecklistQuestionEntity> Revisions { get; set; }
    }

The SQL script to create the table is ....

SELECT
    [Extent1].[ID] AS [ID],
    [Extent1].[OriginalQuestionID] AS [OriginalQuestionID],
    [Extent1].[Question] AS [Question]
    FROM [dbo].[cklChecklistQuestion] AS [Extent1]

when the following code is executed...
    IList<ChecklistQuestionEntity> questions =
        context
            .Set<ChecklistQuestionEntity>()
            .Where(q => q.OriginalQuestionID.HasValue)
            .ToList();

...and then you'll see...
SELECT
    CAST(NULL AS int) AS [C1],
    CAST(NULL AS int) AS [C2],
    CAST(NULL AS varchar(1)) AS [C3]
    FROM  ( SELECT 1 AS X ) AS [SingleRowTable1]
    WHERE 1 = 0

...when the following statement is executed...
            IList<ChecklistQuestionEntity> originalQuestions =
                 context
                     .Set<ChecklistQuestionEntity>()
                     .Where(q => !q.OriginalQuestionID.HasValue)
                     .ToList();

When you exclude the line of code that reads...
            modelBuilder.Entity<ChecklistQuestionEntity>()
                .HasRequired(cq => cq.OriginalQuestion)
                .WithMany(cq => cq.Revisions)
                .HasForeignKey(cq => cq.OriginalQuestionID)
                .WillCascadeOnDelete(false);

and replace it with...
           modelBuilder.Entity<ChecklistQuestionEntity>();
You see the expected SQL appear in SQL Profiler which is...

SELECT
    [Extent1].[ID] AS [ID],
    [Extent1].[OriginalQuestionID] AS [OriginalQuestionID],
    [Extent1].[Question] AS [Question]
    FROM [dbo].[cklChecklistQuestion] AS [Extent1]
    WHERE [Extent1].[OriginalQuestionID] IS NOT NULL

SELECT
    [Extent1].[ID] AS [ID],
    [Extent1].[OriginalQuestionID] AS [OriginalQuestionID],
    [Extent1].[Question] AS [Question]
    FROM [dbo].[cklChecklistQuestion] AS [Extent1]
    WHERE [Extent1].[OriginalQuestionID] IS NULL

I've already discussed this issue on stack overflow here...
https://stackoverflow.com/questions/49295278/linq-entity-framework-ignoring-is-not-null
Whilst the question refers to the issue being in 6.1.3.  I've since tested 6.2 and have replicated the same behaviour.  It seems when you relate a nullable field in a table to the primary key of the table EF stops believing the field can be null and evaluates field.HasValue to true and !field.HasValue to false in the where clause.

Seem as mapping error.
.HasRequired(cq => cq.OriginalQuestion)
must be a
.HasOptional(cq => cq.OriginalQuestion)


EFRS-794: FaceClassifierPredictor verify test added

https://jira.exadel.com/browse/EFRS-794

I have read the CLA Document and I hereby sign the CLA


Exchange node upgrade process

Need to update the node install and upgrade process for wallets and exchanges. This should include either replacing or updating the code at fio.start.

Todd G | FIO, [26.04.21 08:00]
Running a v1 history node with >150,000 keys on a t3.medium works okay.
So a m5.2xlarge will be okay too. I also did some testing on faster hardware, 5800x and nvme ... it was not much faster.
The wallet is 21mb, the blocks + history indices are 15gb, (state is 897mb)

Updated devhub with node upgrade info: https://developers.fioprotocol.io/docs/chain/node-build


Transformer models in Sockeye

This PR adds support for transformer encoders & decoders in Sockeye.
The transformer architecture was proposed in Vaswani et al 2017, Attention is all you need.
From a high-level, users control the choice of encoder and decoder architectures through the --{encoder,decoder} flags.
Integration tests are added for new architecture types.

Looking at the Travis log it looks like py3.4 does not support expressions like [softmax, attention_probs, *states], but rather needs something like [softmax, attention_probs] + states.

This has become a gigantic feature branch :) How are we going to bring this into master? Is this still squashable, given that the branch contains regular merges from master?

We can either merge without squashing, or squash into a single commit (to avoid redoing the merges I had to do before). Lets try to get this in sooner than later to avoid growing this branch even more.
We can still make incremental updates to master after merging this.

yeah, sure. I'm all for merging this asap :)

Have we run end-to-end systems with RNN and transformer for the current state of the branch?  I started a RNN run today but someone else may have already finished one.  Felix can probably comment best on the transformer results.
If we get the expected BLEU scores for RNN and something in the ballpark for transformer, I think we're ready to merge everything into master.  We can continue to iterate on hyper-parameter settings for transformer and training.

I have one finished for RNNs with this branch, produces the same result as master, down to the last digit in ppl.
Still waiting on transformer results. If you are fine with the current state, approve and I will merge once Tobi had a final look and I have transformer results.


Mock api calls

Fixes https://github.com/vnegi10/CryptoDashApp.jl/issues/29

Codecov Report
Patch coverage has no change and project coverage change: -67.75% :warning:

Comparison is base (fd6064e) 67.74% compared to head (eea3713) 0.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #31       +/-   ##
==========================================
- Coverage   67.74%   0.00%   -67.75%
==========================================
  Files           7       7
  Lines         496     496
==========================================
- Hits          336       0      -336
- Misses        160     496      +336

Files Changed
Coverage Δ

src/plotsAV.jl
0.00% <0.00%> (-93.46%)
:arrow_down:

src/plotsCG.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 4 files with indirect coverage changes

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


Fix invisible thrusters on load

Loading into a hovering-state save, the thrusters on your current vehicle (and then all other vehicles) will be invisible. Moving the detection/activation block to OnVehicleFinishedMountingEvent seems to resolve the issue.

Thanks! This definitely helped in my testing, but it's still an issue when replacing let_there_be_flight.archive (something rare for people not developing) - there seems to be some caching issue that I need to reverse in regards to VisualControllerComponent, which is suppose to handle this sort of thing (I think).


3.Optimize pulse collection scanning

3rd optimization
Changes:

(Around 30% faster for 2 pulses) Scan all pulses at the same time when using a pulse collection where each pulse have the same wavelength distribution.

Tests mostly fail with a sign error, but they work well in master.


Database tables don't get cleaned up

Rancher versions:
rancher/server: v1.6.12
rancher/agent: v1.2.7
Infrastructure Stack versions:
healthcheck: v0.3.3
ipsec: v0.2.1
network-services: v0.2.7
scheduler: v0.6.3
Docker version: (docker version,docker info preferred)
v1.12.6
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
Ubuntu 16.04
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
OpenStack
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
single node rancher with external DB
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:
Results:
Our MySQL database is currently taking 23GB and it keeps growing.
It looks like the biggest table is instance which takes over 20GB for 1.6M records. Second most big table is image with 1GB for 1.5M records.
I already launched the rancher/cleanup-1-1:v0.1.2 image.
Is it normal?

Hi,
I think i have the same issue 1.4 Gb cattle.instance table
Most is due to "purged" instance
 select distinct(state), count(*) from instance group by state;
+----------+----------+
| state    | count(*) |
+----------+----------+
| purged   |    35715 |
| removed  |        1 |
| running  |      738 |
| stopped  |     1174 |
| stopping |        1 |
+----------+----------+
other tables looks ok.
SELECT table_name AS "Tables", round(((data_length + index_length) / 1024 / 1024), 2) "Size in MB" FROM information_schema.TABLES WHERE table_schema = "cattle" ORDER BY (data_length + index_length) DESC;
+-----------------------------------------------+------------+
| Tables                                        | Size in MB |
+-----------------------------------------------+------------+
| instance                                      |    1432.23 |
| audit_log                                     |     180.30 |
| image                                         |      87.48 |
| container_event                               |      45.91 |
| instance_label_map                            |      45.20 |

Rancher | v1.6.11
Cattle | v0.183.30
User Interface | v1.6.30
Rancher CLI | v0.6.5
Rancher Compose | v0.12.5

@mcanevet @junkiebev @ValentinOdier
In 1.6.12, we added cleanup logic for databases, but there as still a known issue where we don't clean up purged instances: https://github.com/rancher/rancher/issues/7050#issuecomment-336292654
Can you check to see if most of your purged instances fall in this state?
select id, state, path, instance_id from mount where volume_id in (select volume_id from mount where instance_id in (select id from instance where state="purged") and state="inactive") and state="inactive";```

Hi,
your query now returns 73K items with currently 85K "purged".
Is there something i can do ?

@deniseschannon ,
Similar issue with only setup difference Rancher Server 1.16.10.
90K Instances filling up ~5GB DB out of which 25K are purge records with mount reference (following the query you shared}
Also, there are 102K records @ image as well {though taking just ~80Mb}
Upgraded that to 1.16.14 (stable) as well today but DB cleanup didn't trigger - do I need any special action/configuration for that? Not in favor of trying to do purge cleanup manually.
Thanks in advance!

We're seeing the same... ```
| 13084 | inactive | /var/lib/rancher/etc |       13059 |
+-------+----------+----------------------+-------------+
6376 rows in set (0.04 sec)
Looks like about 50% of those entries are not getting cleaned up.

select count(*) from mount where volume_id in (select volume_id from mount where instance_id in (select id from instance where state="purged") and state="inactive") and state="inactive";
returns 57,020 rows
I see 62,148 agents
the instance table is 1.8GB

There were a few issues causing this:

we were reataining stale data in the mount table, which held a reference to the instance table (this was a intended behavior, but had the unforseen side effect of blocking row deletes)
instances that had a reference held in the mount table would get skipped in our cleanup (this was intentional)
we cleanup rows from the instance table in 1000 row batches. If any batch was comprised entirely of instances that were referenced in another table, the entire cleanup would exit (this was a bug)

To fix, we added special cleanup logic to remove unnecessary stale data in the mount table and fixed the bug that would cause cleanup to prematurely exit

Duplicate of https://github.com/rancher/rancher/issues/7050


Unauthorized Access on fastlane init

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I have read the Contribution Guidelines

Issue Description
I have reviewed the existing github issues relating to this error, and none of them seem to show that this issue has been recently noted, or solved. Others address incorrect retry handling, or lack of two-factor authentication support, or other unrelated issues.
I am initializing  a new project and I am getting an "Unauthorized Access" error. I have verified I am using the latest versions of fastlane and all dependencies, as well as the Xcode command-line tools. fastlane does not crash and I am able to continue with the manual steps once access fails.
I am able to log in to the Developer portal using the same credentials I enter via fastlane. I have two-factor authentication enabled, and I do receive the 6 digit verification code and enter that correctly when prompted.
I am not using an Enterprise account, and the Apple ID I am using is only tied to one developer team. Also, I have the Admin role in my team, and the team_id in the generated Appfile is correct.
I made sure to go back and clear all of the following information before capturing the output below.

delete the deliver entry from the keychain
delete the spaceship* files from /tmp
delete ~/.spaceship
delete the fastlane directory from the project directory

Complete output when running fastlane, including the stack trace and command used
$ fastlane init --verbose
DEBUG [2016-09-12 15:50:06.42]: Checking if there are any plugins that should be loaded...
INFO [2016-09-12 15:50:06.43]: Detected iOS/Mac project in current directory...
INFO [2016-09-12 15:50:06.43]: This setup will help you get up and running in no time.
INFO [2016-09-12 15:50:06.43]: fastlane will check what tools you're already using and set up
INFO [2016-09-12 15:50:06.43]: the tool automatically for you. Have fun!
INFO [2016-09-12 15:50:06.43]: Created new folder './fastlane'.
INFO [2016-09-12 15:50:06.43]: $ xcodebuild clean -showBuildSettings -project ./FastlaneTest.xcodeproj
Your Apple ID (e.g. fastlane@krausefx.com<EMAIL_ADDRESS>WARN [2016-09-12 15:50:14.64]: Verifying if app is available on the Apple Developer Portal and iTunes Connect...
INFO [2016-09-12 15:50:14.64]: Starting login with user<EMAIL_ADDRESS>-------------------------------------------------------------------------------------
The login information you enter will be stored in your Mac OS Keychain
You can also pass the password using the `FASTLANE_PASSWORD` env variable
More information about it on GitHub: https://github.com/fastlane/fastlane/tree/maxxxr/credentials_manager
-------------------------------------------------------------------------------------
Password (for xxx@xxx.com): ********
Two Factor Authentication for account<EMAIL_ADDRESS>is enabled
Please enter the 6 digit code: xxxxxx
Requesting session...
Loading session from<EMAIL_ADDRESS>Loading session from<EMAIL_ADDRESS>Auth error received: 'Unauthorized Access'. Login in again then retrying after 3 seconds (remaining: 4)...
Loading session from<EMAIL_ADDRESS>Auth error received: 'Unauthorized Access'. Login in again then retrying after 3 seconds (remaining: 3)...
Loading session from<EMAIL_ADDRESS>Auth error received: 'Unauthorized Access'. Login in again then retrying after 3 seconds (remaining: 2)...
Loading session from<EMAIL_ADDRESS>Auth error received: 'Unauthorized Access'. Login in again then retrying after 3 seconds (remaining: 1)...
Loading session from<EMAIL_ADDRESS>ERROR [2016-09-12 15:50:57.78]: Unauthorized Access
ERROR [2016-09-12 15:50:57.78]: An error occured during the setup process. Falling back to manual setup!
App Identifier (com.krausefx.app): com.xxx.FastlaneTest
INFO [2016-09-12 15:51:09.77]: Created new file './fastlane/Appfile'. Edit it to manage your preferred app metadata information.
Would you like to create your app on iTunes Connect and the Developer Portal? (y/n)
n
Optional: The scheme name of your app (If you don't need one, just hit Enter):
WARN [2016-09-12 15:51:14.35]: 'snapshot' not enabled.
WARN [2016-09-12 15:51:14.35]: 'cocoapods' not enabled.
WARN [2016-09-12 15:51:14.35]: 'carthage' not enabled.
INFO [2016-09-12 15:51:14.35]: Created new file './fastlane/Fastfile'. Edit it to manage your own deployment lanes.
INFO [2016-09-12 15:51:14.35]: fastlane will send the number of errors for each action to
INFO [2016-09-12 15:51:14.35]: https://github.com/fastlane/enhancer to detect integration issues
INFO [2016-09-12 15:51:14.35]: No sensitive/private information will be uploaded

If I delete the fastlane directory in my project and run fastlane init --verbose again, I get the same results, I'm just not asked for my two-factor authentication info.
Configuration Files
Please copy the complete content of your Fastfile and any other configuration files you use below:
The Fastfile is the default generated by fastlane init
Fastfile:
# Customise this file, documentation can be found here:
# https://github.com/fastlane/fastlane/tree/master/fastlane/docs
# All available actions: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Actions.md
# can also be listed using the `fastlane actions` command

# Change the syntax highlighting to Ruby
# All lines starting with a # are ignored when running `fastlane`

# If you want to automatically update fastlane if a new version is available:
# update_fastlane

# This is the minimum version number required.
# Update this, if you use features of a newer version
fastlane_version "1.102.0"

default_platform :ios

platform :ios do
  before_all do
    # ENV["SLACK_URL"] = "https://hooks.slack.com/services/..."

  end

  desc "Runs all the tests"
  lane :test do
    scan
  end

  desc "Submit a new Beta Build to Apple TestFlight"
  desc "This will also make sure the profile is up to date"
  lane :beta do
    # match(type: "appstore") # more information: https://codesigning.guide
    gym # Build your app - more options available
    pilot

    # sh "your_script.sh"
    # You can also use other beta testing services here (run `fastlane actions`)
  end

  desc "Deploy a new version to the App Store"
  lane :release do
    # match(type: "appstore")
    # snapshot
    gym # Build your app - more options available
    deliver(force: true)
    # frameit
  end

  # You can define as many lanes as you want

  after_all do |lane|
    # This block is called, only if the executed lane was successful

    # slack(
    #   message: "Successfully deployed new App Update."
    # )
  end

  error do |lane, exception|
    # slack(
    #   message: exception.message,
    #   success: false
    # )
  end
end

# More information about multiple platforms in fastlane: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Platforms.md
# All available actions: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Actions.md

# fastlane reports which actions are used
# No personal data is recorded. Learn more at https://github.com/fastlane/enhancer

Environment
fastlane version (run fastlane -v): 1.102.0
Do you use bundler to execute fastlane (i.e. bundle exec fastlane)? NO
Do you use a Ruby environment manager (e.g. chruby, rbenv, rvm)? NO

Regarding the comment above by fastlane-bot, this is not related to code signing.

That is weird, you can run spaceship -u [your email] to quickly try the login without triggering all of fastlane.
Can you confirm that you can login on the following services:

Apple Developer Portal, and list all certificates
iTunes Connect
Login in Xcode in the settings

Hi @KrauseFx
When I execute spaceship -u [my email], it does successfully log me in to iTunes Connect and the Developer Portal.
I can log in to the Developer Portal, and login via Xcode works as well.
When I go to log in to iTunes Connect, it shows me a "Getting Started" page, instead of the one I'm used to seeing, when used with other accounts and teams. I believe the problem may be that I am not the Team Agent, and I may not have been granted any Roles yet.
I'm following up on the Roles issue now.

I was able to verify with one of my co-workers, and he can see me on iTunes Connect, and I am listed as an Admin. Digging further now....

I contacted Apple Developer Support to see if they can fix things. I've seen some mentions on Stack Overflow and elsewhere that other people had the same problems. Contacting DTS fixed the problem.
I don't know if this is a red herring or not. We'll see after I hear back from Apple.

@KrauseFx here's an update:
I had my co-worker double-check, and I am NOT a user on iTunes Connect. Even so, if I execute spaceship -u [my email] I see the following output:
Logging into to iTunes Connect (xxx@xxx.com)...
Successfully logged in to iTunes Connect

Logging into the Developer Portal (xxx@xxx.com)...
Successfully logged in to the Developer Portal

---------------------------------------
| Welcome to the spaceship playground |
---------------------------------------

Enter docs to open up the documentation
Enter exit to exit the spaceship playground
Enter _ to access the return value of the last executed command

Just enter the commands and confirm with Enter
[1] pry(main)>

Within spaceship if I execute Spaceship::Tunes::Application.all I see an Unauthorized Access error, as I would expect.
[6] pry(main)> Spaceship::Tunes::Application.all
Spaceship::Client::UnauthorizedAccessError: Unauthorized Access
from /usr/local/lib/ruby/gems/2.3.0/gems/spaceship-0.32.1/lib/spaceship/client.rb:428:in `block in send_request'

It seems that there are two problems here:

spaceship -u [my email] incorrectly reports Successfully logged in to the Developer Portal for a user who is not in iTunes Connect.
fastlane init --verbose correctly displays an Unauthorized Access error, however for better debugging, it should report which service is reporting the access error.

I have requested to my Team Agent to be added to iTunes Connect. Once that is done, I'll test this all again.

So I'm in iTunes Connect now, and everything is working great. I'm closing this issue and will be opening two more for the items I mentioned above.

@stevekohls Thanks Steve! Really appreciate the update. You are required to be a iTC admin for spaceship to work as expected. Thanks!


Spelling mistake in cart

There in a spelling mistake in Add to cart in Place of cart there is "card"
I want to work on this issue please assign it to me.

please assign this issue to me

Please Review my code.

Thank you @codervivek5 for accepting my first PR <3 .


Adds verify_accounts_hash_and_lamports_for_tests()

Problem
We need to pass in the snapshot storages as a function parameter to verify_accounts_hash_and_lamports() as the fix for startup verification accounts hash mismatch issues due to storages getting cleaned away prematurely.
When we change the function signature by adding a new parameter, it'll require changing a bunch of tests as well. To minimize that noise, and reduce accidentally breaking tests, we could create a test wrapper. Then we'd only need to update a single location for all the tests.
Summary of Changes
Add verify_accounts_hash_and_lamports_for_tests(), which is a wrapper to call verify_accounts_hash_and_lamports_for_tests(). This will be updated by future PRs.

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

Project coverage is 82.1%. Comparing base (a645d07) to head (2176f56).
Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##           master    #1201     +/-   ##
=========================================
- Coverage    82.1%    82.1%   -0.1%
=========================================
  Files         886      886
  Lines      236417   236419      +2
=========================================
- Hits       194267   194247     -20
- Misses      42150    42172     +22


!important scattered throughout Bootstrap CSS files

Bootstrap includes many !important keywords throughout the stylesheets, which makes customization of Bootstrap themes unnecessarily difficult. This happens, for example with colors, such as in the main Bootstrap dark theme.

This is by design so that the utilities have the desired effect.
If one wants to customize things further they have to do it via the Sass files.

Yes, this is intentional.


Key mapping possible bug

Microsoft PowerToys version
0.69.1
Installation method
Scoop
Running as admin
Yes
Area(s) with issue?
Keyboard Manager
Steps to reproduce
Reassign a shortcut: ALT(Left)+F4 to ALT(Left)+Z.
I had this shortcut mapped for a long time and, a couple weeks back (I don't know the exact version), it stopped working. I don't see this shortcut being added as reserved for the operating system in the documentation, so I assume it must be a bug.

✔ Expected Behavior
Be able to close apps with ALT(Left)+Z, just like you would with ALT(Left)+F4.
❌ Actual Behavior
ALT(Left)+Z does nothing, and ALT(Left)+F4 stops working too.
Other Software
No response

it's alt-related, and I believe it's similar to what I encountered - /issues/24902

It seems you're mapping it the other way around.
You're mapping Alt+F4 to Alt+Z instead of mapping Alt+Z to Alt+F4.
Does this help? /needinfo

Oh, you are right :smile:. I changed computers and must have switched the order in my newly installed PowerToys, thus I thought it was an update that broke it. Thank you so much.


Add a Makefile for streamlined development and maintenance operations in OpenTofu Registry

Issue Description:
I'd like to propose adding a Makefile to the OpenTofu Registry project to improve development workflow, enhance build consistency, and streamline common operations. The Makefile will consolidate frequently used commands, making it easier for both new and experienced contributors to interact with the project by standardizing operations.
Motivation:
Including a Makefile can significantly improve the development experience by providing an accessible set of commands for common tasks, reducing the need to recall specific go commands or manually install dependencies. It will also help maintain consistency across contributors by promoting a standardized workflow, ensuring code quality and consistency.
Proposed Targets:
Below are the initial Makefile targets proposed for the registry project:

check-mkcert: Verify if mkcert is installed locally, a prerequisite for running a secure local environment.
create-certificate: Create a new local certificate if one does not already exist.
mod-init: Initialize go mod to handle dependencies.
populate-generated-folder: Populate the generated folder with required files for the registry.
run-server: Start the server, facilitating quick local development and testing.
build-all: Build all the binaries required for the registry.
test: Run tests for all packages to ensure code integrity and functionality.
clean: Remove generated files and binaries, maintaining a clean workspace.
fmt: Format Go code using Go’s formatting standards.
lint: Run static code analysis to maintain code quality.
vet: Analyze code to detect potential issues.
install-tools: Install required development tools to set up the environment.
help: Display available commands and their descriptions.

Expected Benefits:

Improved Developer Experience: A single command interface for common tasks simplifies the onboarding process for new contributors and minimizes setup and maintenance time.
Code Quality Assurance: Linting, vetting, and testing targets encourage a consistent code quality.
Enhanced Maintenance: Clear, standardized commands reduce the chance of errors and streamline project management.

Hi @abstractionfactory
Thank you for the feedback!
I understand the concerns about maintaining shell scripts, and I’m more than happy to help in any way that aligns with the team’s goals.
I would be happy to work on the suggested documentation for using the registry and setting up a fork.
Please let me know if there are specific areas or details you’d like covered, and I’ll make sure to incorporate those.
Looking forward to collaborating further!


Event name emitted should be kebab case

https://vuejs.org/v2/guide/components-custom-events.html#Event-Names
Rename the emitted event name using the recommended kebab case so the events would work on a single html application.
Right now, the following example doesn't work.
<vuejs-datepicker
  v-on:changedMonth="onMonthChange"
></vuejs-datepicker>

it should not be kabab cased if you read the manual.

Where in the manual? In the link I sent you, it's written "For these reasons, we recommend you always use kebab-case for event names."
I'm suggesting this fix because the events were not working when the component is included in a html file.

@charliekassel if you read the docs for vuejs, they indeed should be kebab-case.
Pasted from VueJS docs:

Unlike components and props, event names will never be used as variable or property names in JavaScript, so there’s no reason to use camelCase or PascalCase. Additionally, v-on event listeners inside DOM templates will be automatically transformed to lowercase (due to HTML’s case-insensitivity), so v-on:myEvent would become v-on:myevent – making myEvent impossible to listen to.
For these reasons, we recommend you always use kebab-case for event names.

I've added an example for changedMonth in the codesandbox demo at the bottom -  it seems to be working ok for me on Chrome, Firefox and Safari browsers (Mac).
https://codesandbox.io/s/mpklq49wp

The issue is not when you use a .vue file with Webpack.
Look at the console here and change month: https://codepen.io/anon/pen/LgGaWP

Also, I noticed that haugen86 made a Pull request with the fix.


Expose nib property to Nibable protocol

This is useful eg. when working in a separate framework and there's a need to specify custom bundle for the UINib.

Merge is my second name.


When using something like resession to set the cwd workspace is display incorrectly

Here I launched NeoVim in my home directory (/home/simon) but switched to my dotfiles (~/dotfiles) using resession but cord.nvim still shows simon as the workspace

Closing as this is caused by https://github.com/stevearc/resession.nvim/issues/64


Remove dead link

It loads fine for me.

@sindresorhus sorry, on Firefox it's a white screen, haven't checked it on other browsers

Then I suggest you report the problem to the website owners.


Failed to run test suite by using jest

Test suite failed to run when using Github Actions, getting following error
TypeError: Cannot redefine property: configDescriptor
        at Function.defineProperty (<anonymous>)
    > 1 | import { BncClient, crypto } from "@binance-chain/javascript-sdk";
        | ^
      2 |
      3 | const network = "mainnet";
      at Object.<anonymous> (node_modules/usb/usb.js:54:8)
      at Object.<anonymous> (node_modules/@ledgerhq/hw-transport-node-hid/src/listenDevices.js:6:1)
      at Object.<anonymous> (node_modules/@ledgerhq/hw-transport-node-hid/src/TransportNodeHid.js:14:1)
      at Object.<anonymous> (node_modules/@binance-chain/javascript-sdk/lib/ledger/index.js:41:74)
      at Object.<anonymous> (node_modules/@binance-chain/javascript-sdk/lib/index.js:57:38)

Is there any solution for this.
Here is my workflow file
...
jobs:
  Build:
    name: Build Docker
    runs-on: ubuntu-latest
    steps:
      - name: Install Ubuntu dependencies
        run: sudo apt-get install -y libudev-dev libusb-dev usbutils
      ....
      - name: Install dependencies
        run: yarn install

      - name: Lint code
        run: yarn lint

      - name: Test code
        run: yarn test

Found sollution here https://github.com/node-usb/node-usb/issues/412


Frontend_with_Framework: React 6.4 -> Render Props in React

Description
Contributors are required to create Documentation and explain about the Rendering Props which should cover the questions like:

What is Rendering Props in React ?
Where we use Render Props in React ?
Benefits of Rendering Props in React ?
also give some theory explanation with code
any addons from your side (in-depth of the topic) .

Note :

Plagiarism is strictly prohibited.
Changes should be made inside the Frontend_Web_Development_React_Angular_Vue/ dir inside Frontend_Web_Development_React_Angular_Vue branch.
Files need to be added into the Week 6 folder only.

Domain
Frontend Dev React/Angular/Vue
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign


Error when checking for Solr cores.

Issue Type

Bug Report / Support Request

Your Environment
Vagrant 2.2.5
VirtualBox 5.2.18r124319
ansible 2.8.4
  config file = None
  configured module search path = ['/Users/jnettik/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/local/Cellar/ansible/2.8.4/libexec/lib/python3.7/site-packages/ansible
  executable location = /usr/local/bin/ansible
  python version = 3.7.4 (default, Jul  9 2019, 18:13:23) [Clang 10.0.1 (clang-1<IP_ADDRESS>)]

Your OS

macOS (version)

Full console output
Some of the output was cut off, but here's the part that relates to Solr: https://gist.github.com/jnettik/2e86c24fde6ee530b4724356351112e6
Summary
After updating to the latest version of the DrupalVM box (2.0.3), any new provision errors out at the same step:
TASK [geerlingguy.solr : Check current list of Solr cores.] ********************
fatal: [dvm-50]: FAILED! => {"changed": false, "content": "", "elapsed": 0, "msg": "Status code was -1 and not [200]: Request failed: <urlopen error [Errno 111] Connection refused>", "redirected": false, "status": -1, "url": "http://localhost:8983/solr/admin/cores"}

I'm running into the same issue today when trying to update.

I was able to get around this by switching it to install:
solr_version: "4.10.4"
Maybe it has to do with the Java 8+ note in the documentation here?:
http://docs.drupalvm.com/en/latest/extras/solr/

having very similar problem out of the box, get the
FAILED! => {"changed": false, "content": "", "elapsed": 0, "msg": "Status code was -1 and not [200]: Request failed: <urlopen error [Errno 111] Connection refused>", "redirected": false, "status": -1, "url": "http://localhost:8983/solr/admin/cores"}
using
solr_version: "5.5.5"

Same with me. Anybody has any idea how to fix this?

Does it help to destroy and add this java version to config.yml?
  - openjdk-8-jdk```

@joestewart - It spent 2 days trying different solutions to fix this issue. Adding java_packages worked for me. Thank you so much.

@joestewart - Adding java_packages worked for me too!

This issue is not stale. Adding java_packages to the default config, along with updating the Solr version variable, should be done to make sure Solr is installable and usable out of the box with Drupal VM.

Waiting on the above build to complete; it specifies a Java 8 version on both Debian/Ubuntu and Red Hat/CentOS. Note that for Debian 10, I haven't tested it (it's hard to get Java 8 on Debian 10) but the number of users of Drupal VM on Debian 10 is small enough that I'll worry about that at some other time. Hopefully when Solr 10 or 11 is out and finally supports a newer Java version :D


I am considering to contribute, but need Chines pages in English

[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Ⅰ. Issue Description
I have significant background in transaction processing and transaction models, but my Chinese does not exist.  Do you consider to translate all information into English at some point in time?
Ⅲ. Describe what you expected to happen
Translate Chinese text in the Wiki into English.

@FrankLeymann ，we will soon arrange community members to do this work, and all Chinese documents will have corresponding English documents. Welcome to the community of fescar.

@FrankLeymann ，we will soon arrange community members to do this work, and all Chinese documents will have corresponding English documents. Welcome to the community of fescar.

@slievrly contributors are also willing to offer help if there is a clear translation plan and process to see which part is being translated, I suggest opening issues for each module that can be taken by contributors/members, @FrankLeymann before it's done, you can use something like Google translate plugin to translate, which is usually enough to understand the meaning.

@FrankLeymann ，we will soon arrange community members to do this work, and all Chinese documents will have corresponding English documents. Welcome to the community of fescar.

@slievrly contributors are also willing to offer help if there is a clear translation plan and process to see which part is being translated, I suggest opening issues for each module that can be taken by contributors/members, @FrankLeymann before it's done, you can use something like Google translate plugin to translate, which is usually enough to understand the meaning.

@kezhenxu94  I will , thank u.


Update backup-upgrade-to-vm-backup-stack-v2.md

Changed curly quotation marks to regular.

@trinadhkotturu - Would you review the proposed changes? Thanks

#sign-off


Allow to limit the number of files created by AdvancedFileOutput

Log rotation is not only about splitting logs between multiple files but also about limiting the number of those files.
What do you think about a setting in the AdvancedFileOutput class to limit the count of archived files?
Suggested logic:

the active file _file should not be considered for the count
at every rotation:

count files different to _file in the destination folder
sort files by creation date asc
remove the first file in the list until the list length is not equal to maximum number of files accepted

the default value for maximumArchivedLogFiles should be null, so that it keeps the current behaviour without cancellations

I will be happy to provide a PR, let me know if it could be reasonable

Hi!
This sounds like a good idea, but I am not sure if we can really rely on the creation date of the files.
AFAIK, some file systems and programs tend to be funny with it and either set it to 0 (1970-01-01) or update it along with the modification date.
Additionally, it could be re-set to the current date/time when moved between disks (old file deleted -> new file created).
I would like to have these things checked first.
However, as this would be opt-in anyway, I don't see a reason against providing this option. :+1:

Doing a fast search, also the Dart documentation doesn't recommend to rely on changed property of FileStat: https://api.dart.dev/stable/3.3.3/dart-io/FileStat/changed.html
What do you think about rely on filenames sorted by natural order? The main problem could be the fileNameFormatter property but this function takes a DateTime as input so it's implicit that every filename will have different names.
If this could be acceptable, the documentation must specify this logic to prevent wrong usages

TBH, not really a fan of it :/

this function takes a DateTime as input so it's implicit that every filename will have different names

I don't think so, the time is basically optional and nothing guarantees that it is the right order (e.g. could be random UUIDs).
Leaning on e.g. Log4j, the default PathSorter (which is invoked prior to deletion) sorts after "most recently modified".
Maybe even make it customizable for the user, this could range from a simple boolean flag (use FileStat.modified vs. changed), over a metadata callback for a single file to a whole PathSorter-like class.
As it's the default in a fairly popular logger library, I guess it's reasonable to go this or a similar route (it's opt-in after all).
What do you think?

Yeah, sure, go ahead!


Error compilling console app with System.Data.SqlClient on Linux

I'm getting the following error when I'm trying to compile a simple console app which connects to a Sql Server database in Ubuntu 19.10.
 /usr/bin/ld: obj/Debug/netcoreapp3.0/linux-x64/native/Proj.o: in functionSystem_Data_SqlClient_Interop_NetSecurityNative__ImportPrincipalName':
/__w/1/s/src/System.Data.Common/src/System/Data/Common/BooleanStorage.cs:180: undefined reference to NetSecurityNative_ImportPrincipalName'   /usr/bin/ld: obj/Debug/netcoreapp3.0/linux-x64/native/Proj.o: in function System_Data_SqlClient_Interop_NetSecurityNative__InitSecContext_0':
/__w/1/s/src/System.Data.Common/src/System/Data/Common/ByteStorage.cs:162: undefined reference to NetSecurityNative_InitSecContextEx'

This looks related to: https://github.com/dotnet/corefx/pull/38377 that has been fixed a while ago.
What is the ILC package version that you are using? Make sure that you are restoring the latest one from https://github.com/dotnet/corert/blob/master/samples/HelloWorld/nuget.config#L6

This looks related to: dotnet/corefx#38377 that has been fixed a while ago.
What is the ILC package version that you are using? Make sure that you are restoring the latest one from https://github.com/dotnet/corert/blob/master/samples/HelloWorld/nuget.config#L6

Thanks Jan, I fogret change it, I still have "https://dotnet.myget.org/F/dotnet-core/api/v3/index.json". Now it's working.

Thanks for confirmation. I will try to get the stale myget packages deleted. They are causing a lot of confusion.


security vulnerabilities in localserver-cli-command dependency

The localserver-cli-command package depends on a specific and old version of the package 'handlebars'.
That old version has a dependency on the discontinued optimist which uses a package called minimist that has a known security vulnerability
It would be nice if the dependencies could be updated to newer versions (or, generally, allow newer versions instead of locking everything to a specific version)

Thanks @ori-moisis for bringing this up.
Surely we will look in to this Asap.

Fixed in latest version.


RC Test: Gravity and Orbits 1.4.0-rc.2 PhET-iO and PhET Brand

RC1 testing completed in https://github.com/phetsims/QA/issues/595
This is a placeholder issue, it will be filled out once the next RC is ready.

RC.2 was published to:
https://phet-dev.colorado.edu/html/gravity-and-orbits/1.4.0-rc.2/phet-io/
https://phet-dev.colorado.edu/html/gravity-and-orbits/1.4.0-rc.2/phet/gravity-and-orbits_en_phet.html
There are 5 issues that were corrected for RC.2.  They are all marked as "ready-for-qa" and I will link them below this comment.  When one of those issues has been verified, it may be closed.  When all 5 issues are closed, this sim is ready for production deployment, unless other problems are identified.

This can be a spot-check focused on the above issues.
Beyond this, for general sim testing, let's limit that to ~10 minutes of sanity testing on each of 2 popular platforms.

Issues to verify:

[ ] https://github.com/phetsims/gravity-and-orbits/issues/379
[ ] https://github.com/phetsims/gravity-and-orbits/issues/380
[ ] https://github.com/phetsims/gravity-and-orbits/issues/382
[ ] https://github.com/phetsims/gravity-and-orbits/issues/383
[ ] https://github.com/phetsims/gravity-and-orbits/issues/384

Devices (Not all need to be done, but record which were. Time should be light):

[ ] Win 10 Chrome (Time = )
[ ] Win 10 Firefox (Time = )
[ ] MacOS 11 Safari (Time = )
[ ] MacOS 11 Chrome (Time = )
[ ] ChromeOS (Time = )
[ ] iPadOS (Time = )

For https://github.com/phetsims/gravity-and-orbits/issues/382 I deployed another RC that just changes the time readout font on Android app to match the other platforms:
Deployed: https://phet-dev.colorado.edu/html/gravity-and-orbits/1.4.0-rc.3/phet/gravity-and-orbits_en_phet.html
Deployed: https://phet-dev.colorado.edu/html/gravity-and-orbits/1.4.0-rc.3/phet-io/

QA is done

Thanks, I'll close and open a placeholder for RC.4


Invalid Storage Proof given out

Used the get_storage_value_proof method to get proof for custom pallet OCEX with key as IngressMessages of polkadex chain. It gives out an invalid hash. If the same thing is done using jsonrpsee it provides the correct hash. The problem is only faced in custom  build pallets
Code used to get hash using substrate API client
        let result= self.api
            .get_storage_value_proof("OCEX", "IngressMessages", Some(at))??;// api is of type Api<sr25519::Pair, WsRpcClient, PlainTipExtrinsicParams>

Code used to get hash using jsonrpsee API client
        let ingress_messages: Value = client
            .request(
                "state_getReadProof",
                rpc_params![vec![to_value(storage_key.clone()).unwrap()], block_hash.clone()],
            )
            .await
            .unwrap();
            let read_proof: ReadProof<Hash> = serde_json::from_str(&ingress_messages.to_string()).unwrap();

Interesting. There should be no difference between the pallets - be it custom or from parity.

Which api client version are you using?
Do the storage keys, one generated by the api-client (should be printed as Info log), and the self generated one match?

One possible difference I'm seeing is that the api-client deserializes to an Result<Option<ReadProof<Hash>>>, while for code you posted with the jsonrpsee, I don't see any option involved.
But if that would be the problem, I don't see it working for one pallet but not for the other.

We are using this branch polkadot-v0.9.27 and both the storage keys are matching.

THB I'm a little stuck. I could test the scenario with common pallets, but if it's working with them, that wouldn't help.
Can you provide me with links / example code on how I can reproduce your issue?

@nakul1010 can you provide the full example? I don't see how you build the storage_key for the pure jsonrpsee usage, but I think it is important.
... I'm trying to reproduce the problem with the polkadex/mainnet node from dockerhub (which should be correct (?) ).

I gave it a try to reproduce it on the branch polkadot-v0.9.27 together with the node polkadex/mainnet:v4.0.0 (from docker hub). But for me the storage proof does not differ.
Do you still have this issue? If so, can you provide the full code, that you're running?

Hey @haerdib and @masapr,
I'm sorry for not getting back to you sooner. After some investigation, we discovered that the issue was not with the substrate API client method as we initially suspected. Instead, the culprit was the use of the to_string method, which introduced extra bytes and caused a cascade of problems throughout our codebase.
Thank you both for your help in troubleshooting this issue. With your assistance, we were able to identify and resolve the problem. Therefore, I am closing the issue.