Fixed: Permission check when creating a Product Facility (OFBIZ-12739)

Typo in the action being checked when creating a product facility prevented users with the FACILITY_CREATE permission, but not the CATALOG_CREATE permission, from creating Product Facilities.

Tested by creating a new user in an environment with demo data loaded. User has following permissions via a Security Group:

OFBTOOLS_VIEW
FACILITY_VIEW
FACILITY_UPDATE
FACILITY_CREATE
CATALOG_VIEW

WG-9943
Successfully created a Product Facility for product WF-9943, meaning the FACILITY_CREATE permission was successfully used as a fallback for the absence of the CATALOG_CREATE permission.


[Extension Request] Page 6550 "Document Line Tracking"

Describe the request
Page 6550 "Document Line Tracking" (formerly NAVDACH page ID 5005399 Document Lines Tracking) needs several event publishers in order to enable either standard business cases not covered yet (e.g. correction credit memo for sales sales invoice without return order) or customer specific documents.
Therefore we suggest at least to changes or add:

make variable SourceDocType an Enum to extend it with more document types

functions FindRecords, AssignLineFieldFromDocument() und ShowRecords() should each provide an event at the end of its CASE block (to handle some more document types):
OnFindRecordsAfterCase(SourceDocNo: Code[20], SourceDocLineNo: Integer, SourceDocBlanketOrderNo: Code[20], SourceDocBlanketOrderLineNo: Integer, var TempDocumentEntry: Record "Document Entry" temporary)
OnAssignLineFieldFromDocumentAfterCase(var DocType: Text[30]; var DocNo: Code[20]; var DocArchive: Text[30]; var DocLineType: Text[30]; var DocLineItemNo: Code[20]; var DocLineDescription: Text[100]; var DocLineQuantity: Decimal; var DocLineUnit: Code[10])
OnShowRecordsAfterCase(TempDocumentEntry: Record "Document Entry" temporary)

procedures FindRecordsRelatedToSalesOrder, FindRecordsRelatedToPurchaseOrder, FindRecordsRelatedToSalesReturnOrder should have an event at the end to show correction credit memos and prepayment docs:
OnFindRecordsRelatedToSalesOrderFindAdditionalDocuments(DocNo: Code[20]; DocLineNo: Integer, var TempDocumentEntry: Record "Document Entry" temporary)
OnFindRecordsRelatedToPurchaseOrderFindAdditionalDocuments(DocNo: Code[20]; DocLineNo: Integer, var TempDocumentEntry: Record "Document Entry" temporary)
OnFindRecordsRelatedToSalesReturnOrderFindAdditionalDocuments(DocNo: Code[20]; DocLineNo: Integer, var TempDocumentEntry: Record "Document Entry" temporary)

Additional context
[everything described above]
Internal work item: AB#496877

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 15565.


[zh-cn] Resync kubelet-in-userns.md

Resync this page by following the en revision of https://github.com/kubernetes/website/pull/30410/files

I'm kicking this in, in hope another PR would adjust the way paragraphs are grouped.
/lgtm
/approve


Update cross compile info

When cross compiling OpenBLAS using make (with TARGET, HOSTCC, CC, and FC specified), I found that if I don't specify CROSS=1 flag, the build process attempts to run the tests which may crash with SIGILL: Illegal instruction if the build machine does not have all the instructions of the target machine. The README doesn't state this clearly enough.

I wonder what kind of machine you were cross-compiling on ? Normally/ideally the build system should detect and set CROSS=1 automatically.

I wonder what kind of machine you were cross-compiling on ? Normally/ideally the build system should detect and set CROSS=1 automatically.

I was compiling on a Zen 3 machine for a Sapphire Rapids machine. Regardless, though, I don't see any mechanism in the Makefile (https://github.com/OpenMathLib/OpenBLAS/blob/develop/Makefile) that would set the CROSS flag automatically. There is only one line (401) where it's being used.

The problem is specific to "cross-compiling" for a more powerful cpu (instruction set) of the same architecture, which arguably is not cross-compiling in the usual sense (where the architectures are completely different so there is not even a common subset of instructions between the build host and target system). In a "regular" cross-(architecture) compilation, the CROSS option is added automatically by the c_check script, so I do not think all of the proposed changes to the documentation are necessary.
That said, I have not come up with a convincing solution for your case (except perhaps forcing a second run of getarch to detect the host cpu when the given TARGET happens to match the host architecture and then comparing capabilities via a fixed list).
So perhaps a more appropriate addition to the documentation would be something like When compiling for a more modern cpu TARGET of the same architecture, e.g. TARGET=SKYLAKEX on a HASWELL host, option "CROSS=1" can be used to suppress the automatic invocation of the tests at the end of the build ?

The problem is specific to "cross-compiling" for a more powerful cpu (instruction set) of the same architecture, which arguably is not cross-compiling in the usual sense (where the architectures are completely different so there is not even a common subset of instructions between the build host and target system). In a "regular" cross-(architecture) compilation, the CROSS option is added automatically by the c_check script, so I do not think all of the proposed changes to the documentation are necessary. That said, I have not come up with a convincing solution for your case (except perhaps forcing a second run of getarch to detect the host cpu when the given TARGET happens to match the host architecture and then comparing capabilities via a fixed list). So perhaps a more appropriate addition to the documentation would be something like When compiling for a more modern cpu TARGET of the same architecture, e.g. TARGET=SKYLAKEX on a HASWELL host, option "CROSS=1" can be used to suppress the automatic invocation of the tests at the end of the build ?

Ah, interesting, I didn't know that's not considered cross compiling. I can modify the PR to reflect what you said

Updated the PR.

Thank you


Store .mov part of livephoto in subdir

Is it possible to store the .mov part of a livephoto in a subdir e.g. livephotos?

It is not currently possible to do this.  The .mov part of the live photo is always given the same path and name as the image part.  You can use --skip-live option to skip exporting the .mov part.

@wormiedk It occurred to me that the --post-function option could be used to approximate this behavior. --post-function allows you to execute a python function immediately after each file is exported.  The function gets passed information about the exported file including the file paths.  Using this information, you can detect if a photo is a live photo, and if so, move the file to a sub-directory.
If you save the following in a file named process_livephotos.py you can run it as follows:
osxphotos export /path/to/export  --post-function process_livephotos.py::process_livephotos
One caveat: this won't work well with --update as the update code will detect the .mov files as missing and re-export them.
""" Move the .mov component of a live photo to livephotos subdirectory in the same folder as the photo"""

import pathlib
from typing import Callable

from osxphotos import ExportResults, PhotoInfo

def process_livephotos(
    photo: PhotoInfo, results: ExportResults, verbose: Callable, **kwargs
):
    """Call this with osxphotos export /path/to/export --post-function process_livephotos.py::process_livephotos
        This will get called immediately after the photo has been exported

    Args:
        photo: PhotoInfo instance for the photo that's just been exported
        results: ExportResults instance with information about the files associated with the exported photo
        verbose: A function to print verbose output if --verbose is set; if --verbose is not set, acts as a no-op (nothing gets printed)
        **kwargs: reserved for future use; recommend you include **kwargs so your function still works if additional arguments are added in future versions

    Notes:
        Use verbose(str) instead of print if you want your function to conditionally output text depending on --verbose flag
        Any string printed with verbose that contains "warning" or "error" (case-insensitive) will be printed with the appropriate warning or error color
        Will not be called if --dry-run flag is enabled
        Will be called immediately after export and before any --post-command commands are executed
    """

    # only process live photos
    if not photo.live_photo:
        return

    for filename in results.exported:
        filename = pathlib.Path(filename)
        if filename.suffix != ".mov":
            continue
        # move the .mov file to the livephotos subdirectory
        livephotos_dir = filename.parent / "livephotos"
        if not livephotos_dir.is_dir():
            livephotos_dir.mkdir()
        new_filename = livephotos_dir / filename.name
        if new_filename.exists():
            # delete any existing file with the same name
            new_filename.unlink()
        filename.rename(new_filename)
        verbose(f"Moved {filename} to {livephotos_dir}")

Great! Thanks a lot.
Is there a way to give live photos a different name e.g
Photo 1.jpg
Photo 2_live.jpg
Photo 2_live.mov
The objective is to be able to exclude Live Photos in Plex to avoid a lot of duplicate photos. With the above I could add a .plexignore with *live.mov to exclude the video part

Is there a way to give live photos a different name e.g

Yes. You can use the --filename option with a template to do this. For example:
osxphotos export /path/to/export --filename "{photo.live_photo?{original_name}_live,{original_name}}"
This checks to see if the photo is a live photo:
{photo.live_photo?
and if it is, appends _live to the original filename:
{original_name}_live
otherwise uses the original filename:
,{original_name}

Can you see why this does not append _live to Live Photos?
osxphotos export -V --folder "- nyere -" --update --cleanup --convert-to-jpeg --original-suffix "{edited?_original,}" --edited-suffix "" --exiftool --directory '{folder_album,Fotos ikke i album}' --album-keyword --filename '{created.date} {created.hour}.{created.min}.{created.sec}{photo.live_photo?_live,}' /Volumes/Photos/Export

Can you see why this does not append _live to Live Photos?

The --filename template you're using should append _live to live photos.  I've tested it and indeed it does on my system:
Exporting IMG_0728.JPG (51F2BEF7-431A-4D31-8AC1-3284A57826AE.jpeg) as 2019-12-29 19.01.15_live.JPG (23/29)
Exported /Users/rhet/Desktop/export/2019-12-29 19.01.15_live.JPG
Exported /Users/rhet/Desktop/export/2019-12-29 19.01.15_live.mov

Can you paste output from -V showing results for a live photo?  What does the photo actually get named?


How to add custom scripts to plugin folder

Hi
I need to run custom bash script during one of checks.
There is no obvious way to do it.
I can add to to custom_monitor_definitions in helm chart but i cannot set permission for it, so actual check will fail because of wrong permissions.
The only way i see is to inherit docker image and add my scripts to it, but its too much problem for something so simple.
How its suppose to work?

There are a few examples under node-problem-detector/config/plugin/, e.g., network_problem.sh. And node-problem-detector/config/network-problem-monitor.json is the configuration file that uses it.
You can also refer to the original custom plugin doc on the interface between the plugin and NPD: https://docs.google.com/document/d/1jK_5YloSYtboj-DtfjmYKxfNnUxCAvohLnsH5aGCAYQ/


DRILL-5832: Migrate OperatorFixture to use SystemOptionManager rather than mock

The OperatorFixture provides structure for testing individual operators and other "sub-operator" bits of code. To do that, the framework provides mock network-free and server-free versions of the fragment context and operator context. As part of the mock, the OperatorFixture provides a mock version of the system option manager that provides a simple test-only implementation of an option set.
With the recent major changes to the system option manager, this mock implementation has drifted out of sync with the system option manager. Rather than upgrading the mock implementation, this change uses the system option manager directly – but configured for no ZK or file persistence of options.
The key reason for this change is that the system option manager has implemented a sophisticated way to handle option defaults; it is better to leverage that than to provide a mock implementation.
Work is divided into a series of commits. The first is just routine code cleanup.
Options Manager Updates
The second commit introduces three changes to the options manager:

Provide a test-only constructor for the system option manager that “persists” options to memory only.
Provide classic getType(String name) methods for the option manager with types Long, Double, Boolean and String. This allows clients to provide only a name, not an option validator, to get an option value.
In the setLocalValue(String name, Object value) method, allow values that are ints (mapped to longs) or floats (mapped to doubles). This saves having to remember to write things like 10L or 1.0D in test code.
Since we are now using the system option manager for the test framework, the BaseOptionSet class is no longer needed and was removed.

Use System Option Manager for Tests
The next commit deals with replacing the ad-hoc TestOptionSet with the SystemOptionManager class.

The SystemOptionManager is extended with a test-only constructor that uses an in-memory “persistent” store to avoid polluting the disk or ZK with test-only settings.
The OperatorFixtureBuilder class is extended to gather a set of name/value pairs for options to set. These are set in the OperatorFixture once the system option manager is created. This mechanism uses the generic type mechanism to allow the values to be any valid Java type, rather than just strings. That is:

OperatorFixtureBuilder builder = …
builder.systemOption("myOption", 10);
builder.systemOption("yourOption", "yourValue");
…

Rename FixtureBuilder
Drill now has two “fixtures”: ClusterFixture and OperatorFixture. The class FixtureBuilder is confusing: which fixture does it build? To resolve this, the class was renamed to ClusterFixtureBuilder to be consistent with other fixture builders.
Use ALTER SESSION RESET
Many tests set a session option, then reset the option back to the defaults. Much code did this the hard way:

Get the default value of the option from the code
Set the session option using this default.

In removing the TestOptionSet class, we removed a method used to get the default. We need an alternative.
As it turns out, there is a cleaner way to accomplish the goal. Rather than forcing the option to the default value explicitly, we can just use the ALTER SESSION RESET command.
A new method, resetSystemOption, was added to BaseTestQuery and to ClientFixture. For tests that don’t use these classes, special one-off solutions were added.
This change has the added benefit of being more accurate. In the options system, an option set to the (presumed) default value is not the same as an unset option that automatically reverts to the default. Using ALTER SESSION RESET therefore puts the system back into the true "default" state where as explicitly setting the default does not. (And, trying to explicitly set the default can mask bugs, as described next.)
TestParquetWriter and DRILL-5833
When testing the fixes, testing revealed a random failure in TestParquetWriter due to inconsistent resetting of session options. As it turns out, failure to reset the option to select the “new” Parquet reader caused the testDecimal() test sometimes use the “old” version. When the “old” version was used, it hit a long-standing bug in the way the code reads Decimal9 and Decimal18 types.
This bug was fixed, code visited during the investigation was cleaned up, the tests modified to always reset system options to the default value, and the failing decimal test setup to run both the “new” and “old” readers.
This particular change is somewhat complex, please see the writeup in DRILL-5833 for details.

@ilooner, please review the option-related changes. @sachouche, can you review the Parquet parts?

@paul-rogers Some tests are failing on my machine
Failed tests:
  TestMetadataProvider.tables:154 expected:<OK> but was:<FAILED>
  TestMetadataProvider.tablesWithTableNameFilter:219 expected:<OK> but was:<FAILED>
  TestMetadataProvider.tablesWithSystemTableFilter:191 expected:<OK> but was:<FAILED>
  TestMetadataProvider.tablesWithTableFilter:180 expected:<OK> but was:<FAILED>

Tests in error:
  TestCustomUserAuthenticator.positiveUserAuth » UserRemote SYSTEM ERROR: URISyn...
  TestCustomUserAuthenticator.positiveUserAuthAfterNegativeUserAuth » UserRemote
  TestInfoSchema.selectFromAllTables » UserRemote SYSTEM ERROR: URISyntaxExcepti...
  TestViewSupport.infoSchemaWithView:355->BaseTestQuery.testRunAndReturn:356 » Rpc
  TestInfoSchemaFilterPushDown.testFilterPushdown_NonEqual » UserRemote SYSTEM E...
  TestParquetScan.testSuccessFile:60->BaseTestQuery.testRunAndReturn:356 » Rpc o...
  TestDynamicUDFSupport.setup:91->BaseTestQuery.updateTestCluster:148->BaseTestQuery.updateTestCluster:166 » Runtime
  TestSchemaChange.org.apache.drill.TestSchemaChange » Rpc CONNECTION : java.net...
  TestSchemaChange>DrillTest.finiDrillTest:106 » Interrupted sleep interrupted
  TestDynamicUDFSupport.setup:91->BaseTestQuery.updateTestCluster:148->BaseTestQuery.updateTestCluster:166 » Runtime
  TestSelectivity.org.apache.drill.TestSelectivity » Rpc CONNECTION : java.net.N...
  TestSelectivity>DrillTest.finiDrillTest:106 » Interrupted sleep interrupted
  TestDynamicUDFSupport.setup:91->BaseTestQuery.updateTestCluster:148->BaseTestQuery.updateTestCluster:166 » Runtime
  TestDynamicUDFSupport.setup:91->BaseTestQuery.updateTestCluster:148->BaseTestQuery.updateTestCluster:166 » Runtime
  TestSelectWithOption.org.apache.drill.TestSelectWithOption » Rpc CONNECTION : ...
  TestSelectWithOption>DrillTest.finiDrillTest:106 » Interrupted sleep interrupt...
  TestDynamicUDFSupport.setup:91->BaseTestQuery.updateTestCluster:148->BaseTestQuery.updateTestCluster:166 » Runtime
  TestDynamicUDFSupport.setup:91->BaseTestQuery.updateTestCluster:148->BaseTestQuery.updateTestCluster:166 » Runtime
  TestDynamicUDFSupport.setup:91->BaseTestQuery.updateTestCluster:148->BaseTestQuery.updateTestCluster:166 » Runtime
  TestDynamicUDFSupport.setup:91->BaseTestQuery.updateTestCluster:148->BaseTestQuery.updateTestCluster:166 » Runtime
  TestDynamicUDFSupport>DrillTest.finiDrillTest:106 » Interrupted sleep interrup...
  TestTpchDistributed.tpch14 » User CONNECTION ERROR: Connection /<IP_ADDRESS>:4...
  TestTpchDistributed.tpch15:104->testDistributed:28->BaseTestQuery.test:402 » User
  TestTpchDistributed.tpch16:109->testDistributed:28->BaseTestQuery.test:402 » User
  TestTpchDistributed.tpch17:114->testDistributed:28->BaseTestQuery.test:402 » User
  TestTpchDistributed.tpch18:119->testDistributed:28->BaseTestQuery.test:402 » User
  TestTpchDistributed.tpch20:135->testDistributed:28->BaseTestQuery.test:402 » User
  TestTpchDistributed.tpch19_1:130->testDistributed:28->BaseTestQuery.test:402 » User
  TestTpchDistributedConcurrent.testConcurrentQueries:195 » ConcurrentModification

Tests run: 2335, Failures: 4, Errors: 29, Skipped: 138

Otherwise option related changes LGTM

@paul-rogers It looks like the unit test failures were caused by an unrelated issue. My Mac sometimes create a wifi*.log file in /tmp . And the presence of the file in /tmp causes some unit tests to fail. Everything LGTM.

@paul-rogers Some of the changes I am making on top of https://github.com/apache/drill/pull/978/ as part of DRILL-5730 will likely conflict with this change. When do you think this could make it in? It would be helpful to have it merged sooner to avoid more conflicts down the line :) .

There is a funny thing about the way Drill works. I can review your changes and commit them as soon as I provide a +1. My changes must wait until another committer to find time in their very busy schedules to consider this work. So, we'll likely commit yours first, I'll rebase mine on top of it, then wait for another committer to find time to consider it. The one exception would be if a non-committer can give this PR a +1 and a committer agrees to do a bulk commit this week.

@paul-rogers Will this be going in anytime soon? I'd like to fix the annoying
test(String.format("alter session..."))

statements in the tests after this goes in.

Addressed comments. Rebased on master. Resolved merge conflicts. Squashed commits.
@ilooner, @sachouche please do a quick final review to check for loose ends.

+1, LGTM.


[shardformer]enable padding vocabulary size.

📌 Checklist before creating the PR

[x] I have created an issue for this PR for traceability
[x] The title follows the standard format: [doc/gemini/tensor/...]: A concise description
[x] I have added relevant tags if possible for us to better distinguish different PRs

🚨 Issue number

Link this PR to your issue with words like fixed to automatically close the linked issue upon merge
e.g. fixed #1234, closed #1234, resolved #1234

#5488
📝 What does this PR do?

Summarize your work here.
if you have any plots/diagrams/screenshots/tables, please attach them here.

enable padding vocabulary size to make it choose a faster kernel.
💥 Checklist before requesting a review

[x] I have linked my PR to an issue (instruction)
[x] My issue clearly describes the problem/feature/proposal, with diagrams/charts/table/code if possible
[x] I have performed a self-review of my code
[x] I have added thorough tests.
[x] I have added docstrings for all the functions/methods I implemented

⭐ Do you enjoy contributing to Colossal-AI?

[ ] 🌝 Yes, I do.
[ ] 🌚 No, I don't.

Tell us more if you don't enjoy contributing to Colossal-AI.

All tests have passed.

All tests have passed.

Hi @ver217 @flybird11111 , thank you for the great work. I recent found a bug in hybrid parallel execution (#5539) that HybridParallelOptimizer holds unsharded parameters, and it was because of vocab embedidng padding (in each model policy's preprocess, calling self.model.resize_token_embeddings() creates a new nn.Embedding leaving Optimizer to look the old embedding parameter. Since this function call has been removed in this PR, I wonder if it solves my problem.

Yes, this pull request can resolve your problem.


fix: ensure Fastly-Key is stripped from the dump

To protect customers who use the FASTLY_DEBUG_MODE we should redact the Fastly-Key value.

An alternative approach here might have been to clone the request with req.Clone() and then explicitly unsetting (or resetting to eg. XXXXXX) the Fastly-Key header.


Webhook subscriptions.messaging.knative.dev unable to downgrade

0.15.x to 0.16.0 goes fine, but 0.16.0 to 0.15.x fails on eventing.
Error message when downgrading from 0.16.0 to 0.15.x for eventing:
{"level":"info","ts":"2020-07-13T16:36:36.614Z","logger":"knative-operator.manifestival","caller":"zapr@v0.1.1/zapr.go:69","msg":"Merging","diff":"{\"metadata\":{\"labels\":{\"eventing.knative.dev/release\":\"v0.15.2\"}},\"spec\":{\"conversion\":{\"strategy\":\"None\",\"webhook\":null},\"validation\":{\"openAPIV3Schema\":{\"properties\":{\"spec\":{\"properties\":{\"channel\":{\"description\":\"Channel that forwards incoming events to the subscription.\",\"properties\":{\"apiVersion\":{\"minLength\":1,\"type\":\"string\"},\"kind\":{\"type\":\"string\"},\"name\":{\"minLength\":1,\"type\":\"string\"}},\"required\":[\"apiVersion\",\"kind\",\"name\"],\"type\":\"object\"},\"delivery\":{\"description\":\"Subscription delivery options. More information: https://knative.dev/docs/eventing/event-delivery.\",\"type\":\"object\",\"x-kubernetes-preserve-unknown-fields\":true},\"reply\":{\"description\":\"the destination that (optionally) receive events.\",\"properties\":{\"ref\":{\"description\":\"a reference to a Kubernetes object from which to retrieve the target URI.\",\"properties\":{\"apiVersion\":{\"minLength\":1,\"type\":\"string\"},\"kind\":{\"minLength\":1,\"type\":\"string\"},\"name\":{\"minLength\":1,\"type\":\"string\"},\"namespace\":{\"minLength\":1,\"type\":\"string\"}},\"required\":[\"apiVersion\",\"kind\",\"name\"],\"type\":\"object\"},\"uri\":{\"description\":\"the target URI or, if ref is provided, a relative URI reference that will be combined with ref to produce a target URI.\",\"minLength\":1,\"type\":\"string\"}},\"type\":\"object\"},\"subscriber\":{\"description\":\"the subscriber that (optionally) processes events.\",\"properties\":{\"ref\":{\"description\":\"a reference to a Kubernetes object from which to retrieve the target URI.\",\"properties\":{\"apiVersion\":{\"minLength\":1,\"type\":\"string\"},\"kind\":{\"minLength\":1,\"type\":\"string\"},\"name\":{\"minLength\":1,\"type\":\"string\"},\"namespace\":{\"minLength\":1,\"type\":\"string\"}},\"required\":[\"apiVersion\",\"kind\",\"name\"],\"type\":\"object\"},\"uri\":{\"description\":\"the target URI or, if ref is provided, a relative URI reference that will be combined with ref to produce a target URI.\",\"minLength\":1,\"type\":\"string\"}},\"type\":\"object\"}},\"required\":[\"channel\"],\"type\":\"object\"},\"status\":{\"type\":\"object\",\"x-kubernetes-preserve-unknown-fields\":true}}}},\"versions\":[{\"name\":\"v1alpha1\",\"served\":true,\"storage\":false},{\"name\":\"v1beta1\",\"served\":true,\"storage\":true}]}}"}
{"level":"info","ts":"2020-07-13T16:36:36.617Z","logger":"knative-operator.manifestival","caller":"zapr@v0.1.1/zapr.go:69","msg":"Updating","name":"/subscriptions.messaging.knative.dev","type":"apiextensions.k8s.io/v1beta1, Kind=CustomResourceDefinition"}
{"level":"error","ts":"2020-07-13T16:36:36.715Z","logger":"knative-operator.manifestival","caller":"zapr@v0.1.1/zapr.go:128","msg":"Failed to update merged resource, trying overwrite","error":"CustomResourceDefinition.apiextensions.k8s.io \"subscriptions.messaging.knative.dev\" is invalid: [spec.conversion.webhookClientConfig: Forbidden: should not be set when strategy is not set to Webhook, spec.conversion.conversionReviewVersions: Forbidden: should not be set when strategy is not set to Webhook]","stacktrace":"github.com/go-logr/zapr.(*zapLogger).Error\n\tgithub.com/go-logr/zapr@v0.1.1/zapr.go:128\ngithub.com/manifestival/manifestival.Manifest.update\n\tgithub.com/manifestival/manifestival@v0.5.1-0.20200702141132-93669fa9179b/manifestival.go:160\ngithub.com/manifestival/manifestival.Manifest.apply\n\tgithub.com/manifestival/manifestival@v0.5.1-0.20200702141132-93669fa9179b/manifestival.go:150\ngithub.com/manifestival/manifestival.Manifest.Apply\n\tgithub.com/manifestival/manifestival@v0.5.1-0.20200702141132-93669fa9179b/manifestival.go:101\nknative.dev/operator/pkg/reconciler/common.Install\n\tknative.dev/operator/pkg/reconciler/common/install.go:50\nknative.dev/operator/pkg/reconciler/common.Stages.Execute\n\tknative.dev/operator/pkg/reconciler/common/stages.go:36\nknative.dev/operator/pkg/reconciler/knativeeventing.(*Reconciler).ReconcileKind\n\tknative.dev/operator/pkg/reconciler/knativeeventing/knativeeventing.go:103\nknative.dev/operator/pkg/client/injection/reconciler/operator/v1alpha1/knativeeventing.(*reconcilerImpl).Reconcile\n\tknative.dev/operator/pkg/client/injection/reconciler/operator/v1alpha1/knativeeventing/reconciler.go:169\nknative.dev/pkg/controller.(*Impl).processNextWorkItem\n\tknative.dev/pkg@v0.0.0-20200625173728-dfb81cf04a7c/controller/controller.go:434\nknative.dev/pkg/controller.(*Impl).RunContext.func2\n\tknative.dev/pkg@v0.0.0-20200625173728-dfb81cf04a7c/controller/controller.go:372"}
{"level":"error","ts":"2020-07-13T16:36:36.816Z","logger":"knative-operator","caller":"knativeeventing/reconciler.go:210","msg":"Returned an error","knative.dev/traceid":"f00dd239-bd3f-4245-9390-9035ac711c82","knative.dev/key":"knative-eventing/knative-eventing","targetMethod":"ReconcileKind","error":"failed to apply non rbac manifest: CustomResourceDefinition.apiextensions.k8s.io \"subscriptions.messaging.knative.dev\" is invalid: [spec.conversion.webhookClientConfig: Forbidden: should not be set when strategy is not set to Webhook, spec.conversion.conversionReviewVersions: Forbidden: should not be set when strategy is not set to Webhook]","stacktrace":"knative.dev/operator/pkg/client/injection/reconciler/operator/v1alpha1/knativeeventing.(*reconcilerImpl).Reconcile\n\tknative.dev/operator/pkg/client/injection/reconciler/operator/v1alpha1/knativeeventing/reconciler.go:210\nknative.dev/pkg/controller.(*Impl).processNextWorkItem\n\tknative.dev/pkg@v0.0.0-20200625173728-dfb81cf04a7c/controller/controller.go:434\nknative.dev/pkg/controller.(*Impl).RunContext.func2\n\tknative.dev/pkg@v0.0.0-20200625173728-dfb81cf04a7c/controller/controller.go:372"}
{"level":"error","ts":"2020-07-13T16:36:36.817Z","logger":"knative-operator","caller":"controller/controller.go:449","msg":"Reconcile error","error":"failed to apply non rbac manifest: CustomResourceDefinition.apiextensions.k8s.io \"subscriptions.messaging.knative.dev\" is invalid: [spec.conversion.webhookClientConfig: Forbidden: should not be set when strategy is not set to Webhook, spec.conversion.conversionReviewVersions: Forbidden: should not be set when strategy is not set to Webhook]","stacktrace":"knative.dev/pkg/controller.(*Impl).handleErr\n\tknative.dev/pkg@v0.0.0-20200625173728-dfb81cf04a7c/controller/controller.go:449\nknative.dev/pkg/controller.(*Impl).processNextWorkItem\n\tknative.dev/pkg@v0.0.0-20200625173728-dfb81cf04a7c/controller/controller.go:435\nknative.dev/pkg/controller.(*Impl).RunContext.func2\n\tknative.dev/pkg@v0.0.0-20200625173728-dfb81cf04a7c/controller/controller.go:372"}
{"level":"info","ts":"2020-07-13T16:36:36.817Z","logger":"knative-operator","caller":"controller/controller.go:436","msg":"Reconcile failed. Time taken: 6.139282429s","knative.dev/traceid":"f00dd239-bd3f-4245-9390-9035ac711c82","knative.dev/key":"knative-eventing/knative-eventing"}
{"level":"info","ts":"2020-07-13T16:36:36.817Z","logger":"knative-operator.event-broadcaster","caller":"record/event.go:281","msg":"Event(v1.ObjectReference{Kind:\"KnativeEventing\", Namespace:\"knative-eventing\", Name:\"knative-eventing\", UID:\"44949a2b-b3f8-44c9-895a-574a703733e1\", APIVersion:\"operator.knative.dev/v1alpha1\", ResourceVersion:\"1741\", FieldPath:\"\"}): type: 'Warning' reason: 'InternalError' failed to apply non rbac manifest: CustomResourceDefinition.apiextensions.k8s.io \"subscriptions.messaging.knative.dev\" is invalid: [spec.conversion.webhookClientConfig: Forbidden: should not be set when strategy is not set to Webhook, spec.conversion.conversionReviewVersions: Forbidden: should not be set when strategy is not set to Webhook]"}

0.15.3 CRD:
apiVersion: apiextensions.k8s.io/v1beta1
kind: CustomResourceDefinition
metadata:
  name: subscriptions.messaging.knative.dev
  labels:
    eventing.knative.dev/release: "v0.15.3"
    knative.dev/crd-install: "true"
spec:
  group: messaging.knative.dev
  preserveUnknownFields: false
  names:
    kind: Subscription
    plural: subscriptions
    singular: subscription
    categories:
    - all
    - knative
    - eventing
    shortNames:
    - sub
  scope: Namespaced
  subresources:
    status: {}
  conversion:
    strategy: None
    #    strategy: Webhook
    #    webhookClientConfig:
    #      service:
    #        name: eventing-webhook
    #        namespace: knative-eventing
  additionalPrinterColumns:
  - name: Ready
    type: string
    JSONPath: ".status.conditions[?(@.type==\"Ready\")].status"
  - name: Reason
    type: string
    JSONPath: ".status.conditions[?(@.type==\"Ready\")].reason"
  - name: Age
    type: date
    JSONPath: .metadata.creationTimestamp
  validation:
    openAPIV3Schema:
      type: object
      properties:
        spec:
          required:
          - channel
          type: object
          properties:
            channel:
              type: object
              description: "Channel that forwards incoming events to the subscription."
              required:
              - apiVersion
              - kind
              - name
              properties:
                apiVersion:
                  type: string
                  minLength: 1
                kind:
                  type: string
                name:
                  type: string
                  minLength: 1
            subscriber:
              type: object
              description: "the subscriber that (optionally) processes events."
              properties:
                uri:
                  type: string
                  description: "the target URI or, if ref is provided, a relative
                    URI reference that will be combined with ref to produce a target
                    URI."
                  minLength: 1
                ref:
                  type: object
                  description: "a reference to a Kubernetes object from which to retrieve
                    the target URI."
                  required:
                  - apiVersion
                  - kind
                  - name
                  properties:
                    apiVersion:
                      type: string
                      minLength: 1
                    kind:
                      type: string
                      minLength: 1
                    name:
                      type: string
                      minLength: 1
                    namespace:
                      type: string
                      minLength: 1
            reply:
              type: object
              description: "the destination that (optionally) receive events."
              properties:
                uri:
                  type: string
                  description: "the target URI or, if ref is provided, a relative
                    URI reference that will be combined with ref to produce a target
                    URI."
                  minLength: 1
                ref:
                  type: object
                  description: "a reference to a Kubernetes object from which to retrieve
                    the target URI."
                  required:
                  - apiVersion
                  - kind
                  - name
                  properties:
                    apiVersion:
                      type: string
                      minLength: 1
                    kind:
                      type: string
                      minLength: 1
                    name:
                      type: string
                      minLength: 1
                    namespace:
                      type: string
                      minLength: 1
            delivery:
              description: "Subscription delivery options. More information: https://knative.dev/docs/eventing/event-delivery."
              type: object
              x-kubernetes-preserve-unknown-fields: true
        status:
          type: object
          x-kubernetes-preserve-unknown-fields: true
  versions:
  - name: v1alpha1
    served: true
    storage: false
  - name: v1beta1
    served: true
    storage: true

0.16.0 CRD:
apiVersion: apiextensions.k8s.io/v1
kind: CustomResourceDefinition
metadata:
  name: subscriptions.messaging.knative.dev
  labels:
    eventing.knative.dev/release: "v0.16.0"
    knative.dev/crd-install: "true"
spec:
  group: messaging.knative.dev
  versions:
  - &version
    name: v1alpha1
    served: false
    storage: false
    subresources:
      status: {}
    schema:
      openAPIV3Schema:
        type: object
        # this is a work around so we don't need to flush out the
        # schema for each version at this time
        #
        # see issue: https://github.com/knative/serving/issues/912
        x-kubernetes-preserve-unknown-fields: true
    additionalPrinterColumns:
    - name: Ready
      type: string
      jsonPath: ".status.conditions[?(@.type==\"Ready\")].status"
    - name: Reason
      type: string
      jsonPath: ".status.conditions[?(@.type==\"Ready\")].reason"
    - name: Age
      type: date
      jsonPath: .metadata.creationTimestamp
  - !!merge <<: *version
    name: v1beta1
    served: true
    storage: true
  - !!merge <<: *version
    name: v1
    served: true
    storage: false
  names:
    kind: Subscription
    plural: subscriptions
    singular: subscription
    categories:
    - all
    - knative
    - eventing
    shortNames:
    - sub
  scope: Namespaced
  conversion:
    strategy: Webhook
    webhook:
      conversionReviewVersions: ["v1", "v1beta1"]
      clientConfig:
        service:
          name: eventing-webhook
          namespace: knative-eventing

@jcrossley3 Have you seen this error?
FYI @vaikas

@jcrossley3 Have you seen this error?

No. And I don't fully understand how the v1 and v1beta1 CRD schemas are supposed to reconcile.

kubectl patch crd subscriptions.messaging.knative.dev --type='json' -p '[{"op" : "remove", "path" : "/spec/conversion/webhook"},{"op":"replace", "path":"/spec/conversion/strategy","value" : "None"}]

As @vaikas said, we need to run the above command.
How can we add it into knative operator 0.15 branch?

As we do three-way merging with manifestival, v1beta1 -> v1 is fine. But v1->v1beta1, crd is still v1.

Perhaps we could release a .15.4 that has the subscription CRD definition changed like this:
Current:
    strategy: None
    #    strategy: Webhook
    #    webhookClientConfig:
    #      service:
    #        name: eventing-webhook
    #        namespace: knative-eventing

Change it to this:
    strategy: Webhook
    webhookClientConfig:
      service:
       name: eventing-webhook
        namespace: knative-eventing

Do you have a way to try out with that change to see if the downgrade goes clean? If it does, we could cut a release with that.


feat(obstacle_cruise_planner): support pointcloud-based obstacles

Description
Make obstacle_cruise_planner compatible with sensor_msgs::msg::PointCloud2-based obstacles.
The node subscribes to a sensor_msgs::msg::PointCloud2 topic and converts the point cloud into obstacles by the following steps:

transform the cloud into the trajectory frame
downsample the cloud by voxel grid and apply Euclidean clustering to get candidates of obstacles
check if each point in a cluster is within the stop / slow down margin
when any points are inside the stop / slow down area, create a new obstacle object containing:

the nearest point from the ego for stop obstacles
the nearest and the farthest from the ego for slow down obstacles

Stop and slow down planning are performed with the same trajectory generation algorithm as for ML-based (autoware_perception_msgs::msg::PredictedObject) obstacles.
Cruising and yielding are not supported since point cloud-based obstacles have no velocity information.
Margin adjustment on curves (https://github.com/autowarefoundation/autoware.universe/pull/4952) is also unsupported due to the following reason:

Storing whole point cloud is required to implement the function.
Unlike PredictedObject polygons, point clouds do not necessarily capture the entire obstacle. This could lead the margin to be smaller than in the ML-based case.

Related links

Tests performed
Psim
ros2 launch autoware_launch planning_simulator.launch.xml map_path:=$HOME/autoware_map/sample-map-planning vehicle_model:=sample_vehicle sensor_model:=sample_sensor_kit

with the following changes to obstacle_cruise_planner.param.yaml:
/**:
  ros__parameters:
    common:
+    use_pointcloud: true

     stop_obstacle_type:
+      pointcloud: true

     slow_down_obstacle_type:
+      pointcloud: true

    behavior_determination:
+    pointcloud_search_radius: 5.0
+    pointcloud_voxel_grid_x: 0.05
+    pointcloud_voxel_grid_y: 0.05
+    pointcloud_voxel_grid_z: 100000.0
+    pointcloud_cluster_tolerance: 1.0
+    pointcloud_min_cluster_size: 1
+    pointcloud_max_cluster_size: 100000

Notes for reviewers

Interface changes

Effects on system behavior
This PR does not affect the behavior when using PredictedObject.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[ ] I've confirmed the contribution guidelines.
[ ] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

Removed changes of dummy_perception_publisher: https://github.com/autowarefoundation/autoware.universe/pull/6907/commits/64a3191c82fe3cee6b167b0001f5b892c6a68110, https://github.com/autowarefoundation/autoware.universe/pull/6907/commits/d5260f751db42978a6eb193a066371857c59c27d, https://github.com/autowarefoundation/autoware.universe/pull/6907/commits/d8c8dbb9d4e905c45ed2577f8e422a3bdd72b7e3


Replace scikit-build-core / CMake with setuptools

Using scikit-build-core and CMake was overkill for copying two files into site-packages. This change replaces them with setuptools, and also modifies some of the package names to use a dash (i.e. numba-cuda) which I think is more "standard" in the places in which I've done it here.
Manual testing suggests that the packages built before and after this change are identical, with a couple of small exceptions:

The LICENSE file is placed in the root by setuptools instead of a folder called licenses.
Metadata differs slightly (WHEEL, RECORD, METADATA) due to them being generated slightly differently between the build tools.
setuptools also includes a top_level.txt file.

Many thanks to @vyasr for pointing me to the solution to copy files to site-packages with setuptools, as used in
rapids-dask-dependency.

My review has no power here, but this LGTM.

On the contrary, I would not merge it without your approval :slightly_smiling_face:


How to debug KSCrash in XCode?

I want to know how KSCrash works when a mach exception happened, especially how to get call stack, so I need  to set breakpoints in KSCrash.

@yuzhiyunAtTencent You can debuge KSCrash through KSLogger.Have a look at KSLogger.h.

Have you solved your problem? It seems that none of the demos he provided have the function of capturing logs

@yuzhiyunAtTencent Have you solved your problem? It seems that none of the demos he provided have the function of capturing logs

You need to modify both lldb and KSCrash.
As lldb handles mach exceptions without any options provided to users and kscrash forbiddens almost all crash log when traced (by lldb when you debug).

@Folobe26 Thank you, because of your tip, my problem is solved.

@Folobe26 Thank you, because of your tip, my problem is solved.

how to do that, with Xcode I can not make breakpoint in
static void handleException(NSException* exception, BOOL currentSnapshotUserReported)

I want to know how KSCrash works when a mach exception happened, especially how to get call stack, so I need to set breakpoints in KSCrash.

have you resolved the problem? , please help me，I don't know to make breakpoint in KSCrash

I want to know how KSCrash works when a mach exception happened, especially how to get call stack, so I need to set breakpoints in KSCrash.

have you resolved the problem? , please help me，I don't know to make breakpoint in KSCrash

+1，how to debug KSCrash by breakpoint


Messages

K guys I need some help here This is what I get:

Will you just please review the code and comment on anything that looks questionable and Ill go back to the drawing board with it.

K guys I need some help here This is what I get:
Will you just please review the code and comment on anything that looks questionable and Ill go back to the drawing board with it.

i just woke up, i'll take a look

If you want you can start working right from this repository and get rid of your fork.

https://www.google.com/imgres?imgurl=https%3A%2F%2Fwww.tiktok.com%2Fapi%2Fimg%2F%3FitemId%3D6871276244734954754%26location%3D0%26aid%3D1988&imgrefurl=https%3A%2F%2Fwww.tiktok.com%2Fdiscover%2Fcool-wolf-wallpaper&tbnid=vUN3Lyw_1u0xvM&vet=12ahUKEwjk0IeC5ML8AhXBvHIEHcWMB_4QMygcegUIARCdAg..i&docid=UjLQSNfN_WCp1M&w=720&h=1280&q=cool wolf&hl=en&ved=2ahUKEwjk0IeC5ML8AhXBvHIEHcWMB_4QMygcegUIARCdAg
Will one of you guys get me this video pretty please?

Will one of you guys get me this video pretty please?

It is redirecting to a tiktoks search

Ohhh :disappointed: it supposed to be a 12 sec video gif thing

Ohhh 😞 it supposed to be a 12 sec video gif thing on tictoc and its blocked from me

is that it? https://v16-webapp.tiktok.com/4dd8855bd23d5e2091daf3c4acb19de4/63c0c731/video/tos/maliva/tos-maliva-ve-0068c800-us/94fe2add86254e0a8b584ab54712d0c9/?a=1988&ch=0&cr=0&dr=0&lr=tiktok_m&cd=0|0|0|0&br=1576&bt=788&cs=0&ds=3&ft=4b~OyMXh8Zmo01dJy64jVMQxJpWrKsdm&mime_type=video_mp4&qs=0&rc=ZGRpN2Y5OjlnPDc6aGY5OUBpamV4PGg6ZndnOTMzNzczM0AwMWIwL2M0NWAxMDUxMTM0YSNvLWAwcjRnZi5gLS1kMTZzcw%3D%3D&l=202301122051043FA188787F3EA357FE66&btag=80000

 I dont know :face_with_head_bandage: I hit the firewall
It soposed to be the same wolf image I used in the banner but it does something

Oh my goodness @Ronnie018  I found Build a REALTIME CHAT APP in 5 MIN with React and Firebase and its will v8.2 Lets try this out

Oh my goodness @Ronnie018 I found Build a REALTIME CHAT APP in 5 MIN with React and Firebase and its will v8.2 Lets try this out

can you access mega?

What is mega?

What is mega?

its similar to google drive

https://user-images.githubusercontent.com/86536923/212185601-30f9b2bf-eaf4-44cc-a5ad-70de3bc5db2c.mp4

No I don't have Mega. I'd share a drive with you but someone just lost their laptop yesterday for sharing a google folder with someone we could make a private repository on our team
I love that video so hard but would you want to use the whole video in the banner. Also we can share on Canva. The lady at the end looks a bit like Angelina Jolie

@Ronnie018 Do you know the song

Marilyn Manson - Sweet Dreams ( Are Made Of This ) - Official Audio HD i was trying to find a closed captions youtube video for you but I can't It's a very intense and real song

Marilyn Manson - Sweet Dreams ( Are Made Of This ) - Official Audio HD i was trying to find a closed captions youtube video for you but I can't It's a very intense and real song

I thought that was the video you sent me

No the video I sent you is the same as the picture in the banner but I love this video and I love the song I listen to rap more then anything but I love all music

Sweet dreams are made of this
Who am I to disagree?
Travel the world and the seven seas
Everybody's looking for something
Some of them want to use you
Some of them want to get used by you
Some of them want to abuse you
Some of them want to be abused
Have you ever heard it before?

Have you ever heard it before?

Yes a lot

That was the only video i found with this image

Hey what do you think of the new network page?

I don't know  I found it from a google  image search and that was the link to it. It's fine. I'll find something better eventualy I just hated the face of our organization being plain

I love the networking page I wrote to you about it in the discussion page You should swap it out with the current networking page
I'm having a rock concert in my room and the women are not used to me listening to rock so I've had 4 stop by and ask what's going on since i found your video

Oh my goodness @Ronnie018 I found Build a REALTIME CHAT APP in 5 MIN with React and Firebase and its will v8.2 Lets try this out

is worked out?

I love the networking page I wrote to you about it in the discussion page You should swap it out with the current networking page

Thanks, i already did this

Three Days Grace are my favorite rock band but Animal I Have Become is my favorite song from them

I consider myself a rock guy, but i never heard of this band, i'll listen to it now

What is your favorite band?

About the messages app I have not been able to finish it because I was running technical assistance with Narcotics Anonymous then some yahoo deleted the wifi password from the great room tv and they wanted me to fix it and I cant without entering in the wifi password and if I entered it I'd get into trouble because I'm not supposed to know what it is. But even the Officer was like come on you must know what the wifi password is. That was a whole puddle of hot water I was not going to step in. I've been working on some styling here and there while I did all that. Plus, my son just opened up to me about feeling like a weirdo at school. It's hard being a parent from here but I think I gave him good advice.

So far so good I have 5 minutes left to the tutorial and it's working with our versions then I just have to tie it into out site.

Ha Ha Ha, it's fracking working with this version of firebase!!!! Ha Ha Ha. Whoo, Winning so F ing HARD!!! I'm seriously going to tear my hair out if this will not incorporate in to this site.
The guy that's doing the tutorial is a douche and will past stuff in and switch screens or he imports stuff without showing you. I've got half a mind to go leave a comment in his discussions.


powershell: 7.1.4 -> 7.2.0

Motivation for this change
This release adds support for .NET Core 6 and Apple Silicon (aarch64-darwin)
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Seems like linux has a new runtime dependency on systemtap now:
Build fails with:
autoPatchelfHook could not satisfy dependency liblttng-ust.so.0 wanted by /nix/store/x5znpnviwinda1sai7jp4c9x0a7srfax-powershell-7.2.0/share/powershell/libcoreclrtraceptprovider.so
Add the missing dependencies to the build inputs or set autoPatchelfIgnoreMissingDeps=true

Okay, I will further test this on my Linux box at home, I just quickly worked on this on my macbook at work 😅

It's actually lttng-ust that needs to go into buildInputs

Okay, everything should be fine now, I tested compilation on aarch64-darwin and x86_64-linux.


[Component] Popover

### Tasks
- [x] Use the RichTooltipBox as base
- [x] Update the previews to represent the states
- [x] Address the deprecated API in TooltipBox
- [x] Add slots for component placements
- [x] Update the Popover documentation from the api changes
- [x] Add paparazzi test for every Popover state possible
- [ ] Design review for Popover
- [ ] #1585 Readjusting the popover

alright, FYI
https://github.com/adevinta/spark/issues/1360

CC
https://github.com/adevinta/spark/issues/1585

ℹ tests will be done by @kazaky.
We agreed that importants things to test that are not visible on demo app are :

with or without tip
with short / long text in the container (to test scrolling)
alignment left/right/start/end


Teddy/add polygon mvi eth2x fli p

Pull Request Template
Type of Change
Select a category that relates to the content of your pull request (choose all that apply).

[ ] Bug Fix
[x] Content
[x] New Feature
[ ] Documentation
[ ] Code Refactoring

Summary of Changes

[x] Additional tokens to list (Polygon: MVI, ETH2x-FLI-P)

Add to MaticTokens

[x] Create new export for just Index Coop products IndexCoopMaticTokens. The use case for this would be consumers that are also fetching their a different list from another source and we only want to add IC tokens (ex. coop.tax)
[ ] Any changes to get this published as a package?

Issue https://www.notion.so/index-coop/Coop-tax-Use-Indexcoop-TokenLists-and-include-MVI-and-ETH2x-FLI-P-2690c2bd681b4820bdcc50ccb6847dd1

Test Data or Screenshots
Tested local via npm link

Submission Reminders
By submitting this pull request, you are confirming the following to be true:

I have reviewed the Contribution Guidelines.
I have performed a self-review of my own code.
I have updated my repository to match the master at SetProtocol/index-ui.
I have included test data or screenshots that prove my fix is effective or that my feature works.

@0xModene Won't have permissions to actually publish the package. Happy to help however I can; otherwise, I'll leave this last bit of you 😸


Upgrade to Gradle 6.7.1

Hello,
I've been working on refactoring the android-gpgs implementation because it uses deprecated Google APIs. However, when building the project I noticed that the project is based on an older Gradle version. I figured I'd migrate the project to Gradle 7.0.3 before proceeding with the actual refactoring.
Here is a PR that contains the needed changes to build / deploy the project with Gradle 7. Basically all of the gradle scripts needed some tuning to get this to work.
Building with gradlew clean publishToMavenLocal on Windows or gradle clean publishToMavenLocal on Linux works great and the local repository populates with libraries I can use in my project.
Obviously I haven't tested deploying the package to the external repository but thats something maybe you, the developer, can test.
If you need me to do any additional fixes or other changes please let me know.

Hi,
please repair the GitHub actoins build.
There are some changes that seem unnecessary (renaming minSdk, targetSdk etc. to *Ver) and cause changes that could be avoided.
Setting ios and core modules to Java version 8 is dangerous until all platforms supported support Java 8. While RoboVM can handle some Java 8 language features, it is still better to keep the source level on 7 so problems get caught on compile time, not on runtime.
Are you sure Gradle 7 is needed? For most things, Gradle 6.7 is still a viable alternative which causes much less problems in terms of backwards compatibility.

Hi,
Thanks for the comments. I will take a look at the issues you raised and will update the PR accordingly.
Br.
Kari

Hello,
I looked into the issues you mentioned and here are some thoughts:
There are some changes that seem unnecessary (renaming minSdk, targetSdk etc. to *Ver) and cause changes that could be avoided.
I will go through the changes and remove any unnecessary changes. However, with the new Gradle 7 the compileSdk, targetSdk and minSdk are reserved keywords and thus cannot be used. That is why we need to use a different name for the variables.
Setting ios and core modules to Java version 8 is dangerous until all platforms supported support Java 8. While RoboVM can handle some Java 8 language features, it is still better to keep the source level on 7 so problems get caught on compile time, not on runtime.
Fair enough I will revert back to Java 7. I was refactoring the gpgs-client and decided to use lambdas which are supported in Java 8. I will revert the changes.
Are you sure Gradle 7 is needed? For most things, Gradle 6.7 is still a viable alternative which causes much less problems in terms of backwards compatibility.
I made some investigation and couldn't find any source that declares one shouldn't use Gradle 7. As far as I know Gradle 7 is the latest stable version and incorporates many improvements and bugfixes over Gradle 6. What is your preference? Should we stick to Gradle 7 or should I downgrade to Gradle 6?

Reverted to Java 7 and made a slight change to compile.yml
Had to change:
run: ./gradlew clean uploadArchives -PSNAPSHOT=true -PLOCAL=true
to
run: ./gradlew clean publishToMavenLocal -PSNAPSHOT=true -PLOCAL=true
publishToMavenLocal  copies all defined publications to the local Maven cache, including their metadata (POM files, etc.).
When publishing a release version one should use
`run: ./gradlew clean publish -PRELEASE=true

Gradle 7 needs Java 11, so not useable on environments still running jdk 8. That's why 6.7 is widely used at the moment. I would prefer to use it if it works with the latest dependencies.

Hi,
Understood. I downgraded Gradle to 6.7.1 and com.android.tools.build:gradle module to version 4.2.2 which supports Gradle 6.7.1+.
Builds fine with ./gradlew clean publishToMavenLocal -PSNAPSHOT=true -PLOCAL=true

Updated the PR.

Libgdx back to 1.9.8.
Moved dependencies to submodules where they are used.
Added some comments to the question about publishing the artifacts.


Image does not show when selected from SDcard

Expected Behavior
I want to show the stream image in an Image tag
Actual Behavior
The image does not show when pick from Sdcard.
Steps to Reproduce the Problem

FileData fileData = await CrossFilePicker.Current.PickFile(new string[] { "image/png", "image/jpeg" });
img.Source = ImageSource.FromStream(() =>
{
return imageProduct.FileData.GetStream();
});

Specifications
I copy the same image into the sdcard and into the phone. I select the image from the internal storage, Image tag shows the image. But, when I select the image from the sdcard, the image does not show up.

NuGet package version: 2.1.34
Platform(s): Android

@Guervyl what was the solution in the end? Might want to share?


[Backend] Uncomment Compiler and CompilerBase

Let's uncomment Compiler and CompilerBase classes with a test.
ONE-vscode-DCO-1.0-Signed-off-by: Yongseop Kim<EMAIL_ADDRESS>

PTAL @jyoungyun


User action doesn't allow you to pass on GUI

User action on cmd_line says space for pass and escape for fail.
GUI needs a pass fail button for the user_action

With #108, the cmd_line actually changed, so space no longer forces a pass. (and the prompt was change accordingly).
I don't think this is a problem anywhere it has been used to date, and seemed to be a more consistent with the concept of the user_action call. So, closing this now essentially as a won't fix. Happy to re-introduce that capability in the future if a need arises.


jsLint complaint about 'for' statement

Hi, first at all, thanks for this great tool called 'jslint'
I have this "problem" or issues when I parse my code with jslint.
/*jslint multivar:true */
function looping2(n) {
"use strict";
var a = 0, b = 1, f = 1;
for(var i = 2; i <= n;  i = i + 1) {
f = a + b;
a = b;
b = f;
}
return f;
}
JSLint gave this warning:
Unexpected 'for'.
Unexpected 'var'.
Then I changed my code to this:
function looping(n) {
"use strict";
var a = 0, b = 1, f = 1, i;
for (i = 2; i <= n; i = i + 1) {
f = a + b;
a = b;
b = f;
}
return f;
}
...and still complaint about the 'for' statement.
What's your thoughts about for statement?
I think you saw 'for' only to be used with arrays and not for simple looping.
Thank you, and please don't complaint abot my english grammar, I'm foreing.

Read the instructions or use the 'for' option.

Instructions:

JSLint does not recommend use of the for statement. Use array methods like forEach instead.

How can I use forEach  in this code? When is clearly a simple "loop"  that said: "go from zero to a max number(N).
Please I don't ge it.

...or use the 'for' option.

I ended up using a while loop statement.
Anyway, thanks about the tip using the 'for' option to avoid the warning.
I'd prefer having an explanation about the JSLint rule with 'for' statements and how to solve rather than using the silent option 'for;
Best regards


Use empty title bar on macOS

Wondering if it is possible for the IDE to use empty space in the title bar on macOS? The resize icons seem to not be used very often (?) but the empty title bar is taking up valuable screen real estate.

https://marketplace.visualstudio.com/items?itemName=iocave.customize-ui makes this possible.
Would this make a good feature to support by default vs. overwriting code by an add-in?

Duplicate of #12377
TL;DR, Code team has not decided what's the best way to do it.
Code team also not recommends modifying UI with extensions.


The Simpsons testing

Great job!

This's a great idea! But there is a minor issue in animation timing. But I'll try to fix this.


borgbackup: cleanup inputs

This fixes https://github.com/NixOS/nixpkgs/issues/16171. See commit message for details. Tested basic functionality.

My backup still works.

My backup still works.


Handle parameter without setting explicit set to true

I've encountered an bug on the handling of the parameters, which I didn't noticed on the testing.
GET {{baseurl}}?select=NextMajor&accept_insiderEula=true&DoNotRedirect=true
GET {{baseurl}}?select=NextMajor&accept_insiderEula&DoNotRedirect=true

When a =true is included it works, but without this explicit, is doesn't add the param to the BCCH command.

@Arthurvdv thanks again. Merged, built and published as v1.4.1. Lesson learned: Really click through all tests...

one more comment: I forgot to bump the version in the code :( Therefore, I now built 1.4.2 which only has the new version so that the header in the response shows the correct information, but no other code has changed

@tfenster, thank you for resolving this so quickly!

Lesson learned: Really click through all tests...

Yeah, I also cut some corners here I believe and I'll also will help to execute all the tests (and review the content of the response).


hoverEdge function not working

the hoverEdge function doesn't work on the canvas. I am not getting any console output when I tried to log the event. Looks like the function doesn't get triggered at all. Can we fix this?

Still not working for me


Change Project settings for Project onboarding when talking about "legacy system"

At the end of the document, in the following section:
https://docs.microsoft.com/en-us/dynamics365/fin-ops-core/dev-itpro/migration-upgrade/upgrade-data-sandbox#run-the-data-upgrade-deployable-package
The documentation says: "If no data upgrade packages appear in the list, go to your project settings in LCS, and make sure that your legacy system is set to AX2012 Upgrade."
But, this options is not on the project settings in LCS, but in the Project Onboarding in LCS, in the Project Overview tab, so, it must be:
"If no data upgrade packages appear in the list, go to your project onboarding in LCS, to the project overview tab, and make sure that your legacy system is set to AX2012 Upgrade."
Thanks

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

ID: 6717d4f3-0b93-713a-6a61-405cce70dfe0
Version Independent ID: 050db2bb-9dd4-82ff-3f27-ec7a8d0ea64c
Content: Upgrade from AX 2012 - Data upgrade in sandbox environments - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/migration-upgrade/upgrade-data-sandbox.md
Service: dynamics-ax-platform
Product: ****
Technology: ****
GitHub Login: @laneswenka
Microsoft Alias: laswenka

@jatomas, thanks for the feedback.
@laneswenka, please review.

Bug 532040 created to track this issue.


initial stab at a send-only OSC client

this allows data to be sent to, say, Max MSP or Ableton Live etc.

ping @nikhan you like?


Add auto project detection.

Fixes #629
Changes proposed:

[x] Add
[ ] Prepare
[ ] Delete
[ ] Fix

Things I've done:

[x] My pull request fixes an issue, I referenced the issue.

Unfortunately, the code functionality isn't covered via tests. This is something that should be done regardlessly.

OK. Because I'm making a lot of changes to the code to support testing, let's keep this PR as a WIP.

I agree.

If it's ok for you I'll publish what we have right now.

I'll probably have it ready by tomorrow but you can never be sure. So the decision is yours.

I think we can publish what we have right now and develop and test your great new addition without pressure. If it's ready during the week it will be published next Sunday. I think this is more than acceptable. Besides that, it doesn't introduce breaking changes to the extension so it won't pose a problem to the users.


devel/git: add cross-compilation support

.FIXES #142

updated.
i did the variables like buildroot:
https://github.com/buildroot/buildroot/blob/master/package/git/git.mk

Thanks!


Origin Chain Fee displayed is delayed

Description the bug
Origin Chain Fee displayed is delayed
To reproduce

Open the extension
Go to XCM Transfer screen
Select Origin Chain = Kintsugi & Input valid data for the all fields
Change Origin Chain to Karura then click on Transfer button
On the Authorize Transaction screen, observe the Origin Chain Fee displaying

Actual: Origin Chain Fee = x KINT
Notes: it takes a while to display properly
(See file attachment below)

Version test:

https://file.cdmteck.com/s/4oimk9zDJ8R5Aai/download?path=%2F&files=./348-merge-build-0.4.5-1655446043317.zip or
https://file.cdmteck.com/s/4oimk9zDJ8R5Aai/download?path=%2F&files=./346-merge-build-0.4.5-1655381117968.zip

Related: https://github.com/Koniverse/SubWallet-Extension/issues/411

Do not reproduce this behavior in the new UI version


CUPS driver for Zebra LP 2824 Plus on Mac OS X 10.11 doesn't print as previewed

Version: -feature
CUPS.org User: blasto333
CUPS driver for Zebra LP 2824 Plus on Mac OS X 10.11 doesn't print as previewed. In Chrome the label shows up correctly in print preview as taking up one label. When printed it prints the wrong direction (rotated 90 degrees) and over 2 labels.

hello blasto333,
maybe you have problems with autorotation of the label.
try to disable it with
lpoptions -p   -o pdfAutoRotate=false
@michaelrsweet is it possible to show this propertie at the webinterface?
it is not easy to check this settings after month :-(

Hi
I Have the same problem with my Mac OS X 10.11 and my SATO CG412TT label printer.
I tried "lpoptions -p -o pdfAutoRotate=false", but it doens't work.  What can I do? Thanks

pdfAutoRotate is specific to Linux, and is not a standard IPP attribute (which is what the web interface shows).
This sounds like it is a Chrome printing bug and not something we can fix in the driver.


Extends 'spo site apppermission get'. Closes #2409

Extends 'spo site apppermission get'. Closes #2409
Extends the spo site apppermission get command to show the roles.

Thank you for such a quick turnaround. We'll review it asap.

Hey @nanddeepn, it seems like some extra commits got pulled in in the PR. Could you please try to clean it up? If that will turn impossible, please close this PR and open a new one. Thank you!

Hi @waldekmastykarz
I will close this PR and will raise a new one soon. Thank you.

Awesome. Appreciate your help


Details: System.InvalidOperationException: Token StartAttribute in state Element Content would result in an invalid XML document.

Hi. I don't know is it a bug or not, but just decided to post. If it is not a bug, please, close it.
This "bug" comes from setup:
PageType = ListPart;
Caption = 'Bank Account Ledger Entries';
Editable = false;
SourceTable = "Bank Account Ledger Entry";
//    SourceTableView = SORTING("Bank Account No.","Posting Date") ORDER(Descending);
SourceTableTemporary = true;
InsertAllowed = false;
ModifyAllowed = false;
DeleteAllowed = false;
ShowFilter = false;
LinksAllowed = false;
If I uncomment the line "SourceTableView = SORTING("Bank Account No.","Posting Date") ORDER(Descending)" I receive:

Microsoft (R) AL Compiler version 0.12.28462
Copyright (C) Microsoft Corporation. All rights reserved
[Error - 11:41:13 PM] Please report this issue to https://github.com/microsoft/al/issues including information on how to reproduce it, if possible.Details:
System.InvalidOperationException: Token StartAttribute in state Element Content would result in an invalid XML document.
at System.Xml.XmlWellFormedWriter.ThrowInvalidStateTransition(Token token, State currentState)
at System.Xml.XmlWellFormedWriter.AdvanceState(Token token)
at System.Xml.XmlWellFormedWriter.WriteStartAttribute(String prefix, String localName, String namespaceName)
at System.Xml.XmlWriter.WriteAttributeString(String localName, String value)
at Microsoft.Dynamics.Nav.CodeAnalysis.Emit.ObjectMetadataEmitHelper.WritePropertyValueMetadata(XmlWriter xmlWriter, PropertySymbol property) in C:\a\316\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Emitter\ObjectMetadataEmitHelper.cs:line 80
at Microsoft.Dynamics.Nav.CodeAnalysis.Emit.ObjectMetadataEmitter.WritePropertyValueMetadata(XmlWriter xmlWriter, PropertySymbol property) in C:\a\316\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Emitter\ObjectMetadataEmitter.cs:line 54
at Microsoft.Dynamics.Nav.CodeAnalysis.Emit.ObjectMetadataEmitHelper.WriteAttributeProperties(XmlWriter xmlWriter, IEnumerable1 properties, Action2 writePropertyMetadata) in C:\a\316\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Emitter\ObjectMetadataEmitHelper.cs:line 69
at Microsoft.Dynamics.Nav.CodeAnalysis.Emit.ObjectMetadataEmitHelper.WriteProperties(XmlWriter xmlWriter, Symbol declaringSymbol, IEnumerable1 properties, Action2 writePropertyMetadata, Boolean shouldOutputDefaultProperties) in C:\a\316\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Emitter\ObjectMetadataEmitHelper.cs:line 38
at Microsoft.Dynamics.Nav.CodeAnalysis.Emit.PageMetadataEmitter.<>c__DisplayClass3_0.b__1() in C:\a\316\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Emitter\PageMetadataEmitter.cs:line 90
at Microsoft.Dynamics.Nav.CodeAnalysis.Emit.ObjectMetadataEmitHelper.WriteStartEndElement(XmlWriter xmlWriter, String elementName, Action action) in C:\a\316\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Emitter\ObjectMetadataEmitHelper.cs:line 25
at Microsoft.Dynamics.Nav.CodeAnalysis.Emit.ObjectMetadataEmitHelper.WriteStartEndElement(XmlWriter xmlWriter, String elementName, Action action) in C:\a\316\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Emitter\ObjectMetadataEmitHelper.cs:line 25
at Microsoft.Dynamics.Nav.CodeAnalysis.Emit.PageMetadataEmitter.b__1_0(XmlWriter xmlWriter) in C:\a\316\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Emitter\PageMetadataEmitter.cs:line 33
at Microsoft.Dynamics.Nav.CodeAnalysis.Emit.ObjectMetadataEmitter1.EmitMetadata(Action1 content) in C:\a\316\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Emitter\GenericObjectMetadataEmitter.cs:line 37
at Microsoft.Dynamics.Nav.CodeAnalysis.MethodCompiler.CompileObject(ObjectTypeSymbol containingType) in C:\a\316\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Compiler\MethodCompiler.cs:line 248
at Microsoft.Dynamics.Nav.CodeAnalysis.MethodCompiler.<>c__DisplayClass17_0.b__0() in C:\a\316\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Compiler\MethodCompiler.cs:line 216
at Microsoft.Dynamics.Nav.CodeAnalysis.Utilities.UICultureUtilities.<>c__DisplayClass5_0.b__0() in C:\a\316\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Utilities\UICultureUtilities.cs:line 152
at System.Threading.Tasks.Task.Execute()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Dynamics.Nav.CodeAnalysis.MethodCompiler.WaitForWorkers() in C:\a\316\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Compiler\MethodCompiler.cs:line 161
at Microsoft.Dynamics.Nav.CodeAnalysis.MethodCompiler.CompileMethodBodies(Compilation compilation, ModuleBuilder moduleBeingBuiltOpt, Boolean hasDeclarationErrors, DiagnosticBag diagnostics, Predicate1 filterOpt, CancellationToken cancellationToken) in C:\a\316\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Compiler\MethodCompiler.cs:line 145    at Microsoft.Dynamics.Nav.CodeAnalysis.Compilation.CompileImpl(ModuleBuilder moduleBuilder, Boolean emittingPdb, DiagnosticBag diagnostics, Predicate1 filterOpt, CancellationToken cancellationToken) in C:\a\316\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Compilation\Compilation.cs:line 806
at Microsoft.Dynamics.Nav.CodeAnalysis.Compilation.Compile(ModuleBuilder moduleBuilder, Boolean emittingPdb, DiagnosticBag diagnostics, Predicate`1 filterOpt, CancellationToken cancellationToken) in C:\a\316\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Compilation\Compilation.cs:line 783
at Microsoft.Dynamics.Nav.CodeAnalysis.CommandLine.CommonCompiler.PackageTheModule(Compilation compilation, CompilerTextWriter consoleOutput, CancellationToken cancellationToken, ErrorLogger errorLogger, AnalyzerDriver analyzerDriver) in C:\a\316\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\CommandLine\CommonCompiler.cs:line 531
at Microsoft.Dynamics.Nav.CodeAnalysis.CommandLine.CommonCompiler.RunCore(CompilerTextWriter consoleOutput, ErrorLogger errorLogger, CancellationToken cancellationToken) in C:\a\316\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\CommandLine\CommonCompiler.cs:line 413
at Microsoft.Dynamics.Nav.CodeAnalysis.CommandLine.CommonCompiler.Run(CompilerTextWriter consoleOutput, CancellationToken cancellationToken) in C:\a\316\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\CommandLine\CommonCompiler.cs:line 312
at Microsoft.Dynamics.Nav.EditorServices.Protocol.LanguageServer.Extensions.CreatePackageRequestHandler.d__3.MoveNext() in C:\a\316\s\source\Prod\Microsoft.Dynamics.Nav.EditorServices.Protocol\LanguageServer\Extensions\CreatePackageRequestHandler.cs:line 32
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Dynamics.Nav.EditorServices.Protocol.RequestRegistry.d__6.MoveNext() in C:\a\316\s\source\Prod\Microsoft.Dynamics.Nav.EditorServices.Protocol\Endpoints\RequestRegistry.cs:line 59

HI @RedFoxUA,
This GitHub repository is dedicated to handling issues and feedback for the latest Developer Preview releases of the AL development tools for Dynamics 365 Business Central.
If you can reproduce the problem on the latest Business Central Developer preview version we will be happy to look further into it. Otherwise please contact the regular support channel.
You can open Support Request to CSS through PartnerSource portal or contact your Service Account Manager (SAM) in the local subsidiary to understand what is included in your contract as of support incident and PAH (Partner Advisory Hours). Your SAM might also step by step direct you how to open a support request or how to get credentials if this is the first time for you / your company.
For more details please read:
https://blogs.msdn.microsoft.com/nav/2018/05/03/al-developer-previews-multiple-releases-and-github/
https://github.com/Microsoft/AL/wiki/Frequently-Asked-Questions#i-found-an-issue-on-nav-2018-what-do-i-do

Hi,
We do highly value your feedback and want to fix the bugs that you run into.
Our team is only so big and we don't want to investigate issue that has already been fixed, so that is why we urge you to test on the latest release.
That issue you are facing seems to fixed in a later release and is a duplicate of #781

Hi Thomas Pedersen.
Thank you very much for your answer and clarification.
I'm using the NAV 2018 ('US Dynamics NAV 11.0' cumulative Update 10 for Microsoft Dynamics NAV 2018 Build 24742) and I don't have a possibility now to check it on the latest version, sorry.
But again, thank you for your answer.


edf reader fails

I have some iEEG data in .edf format. mne_edf2fiff works just fine. read_edf_file() is able to read the metadata, but when it tries to read the iEEG data it crashes.
In [3]: ra = mne.io.edf.read_raw_edf('raw.edf', preload=True)
Extracting edf Parameters from raw.edf...
Setting channel info structure...
Creating Raw.info structure...
Reading raw data from /local_mount/space/truffles/2/users/rlaplant/ecog/raw.edf...
Reading 0 ... 3781782  =      0.000 ...  7567.997 secs...
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)

/local_mount/space/truffles/1/users/rlaplant/.canopyrc/User/lib/python2.7/site-packages/mne-0.9.dev-py2.7.egg/mne/io/edf/edf.py in _read_segment(self, start, stop, sel, verbose, projector)
    276                     print int(sfreq), n_chan, blocks
    277                     data = data.reshape((int(sfreq), n_chan, blocks),
--> 278                                         order='F')
    279                     for i in range(blocks):
    280                         start_pt = int(sfreq * i)

It is trying to reshape a 232352736x1 matrix into a 499x129x7568 matrix.
It don't even know where to begin debugging this -- it seems like it is not figuring out correct information in the header? Maybe there is some encoding issue?
@dgwakeman suggested to tag @teonlamont
I don't know if I can provide the raw data as it is clinical data that may not be deanonymized, but does anyone have any idea what might be going on?

First, just to check, have you updated to the most recent master? There were some bugs that were resolved over the past couple of days.
Do you have a file you could share to help troubleshoot?
The reshaping is done because edf data are written in segments in time (records or blocks) and it will ultimately need to be in a channel by time.
Envoyé de mon iPhone

Le 28 janv. 2015 à 19:06, Roan LaPlante<EMAIL_ADDRESS>a écrit :
I have some iEEG data in .edf format. mne_edf2fiff works just fine. read_edf_file() is able to read the metadata, but when it tries to read the iEEG data it crashes.
In [3]: ra = mne.io.edf.read_raw_edf('raw.edf', preload=True)
Extracting edf Parameters from raw.edf...
Setting channel info structure...
Creating Raw.info structure...
Reading raw data from /local_mount/space/truffles/2/users/rlaplant/ecog/raw.edf...
Reading 0 ... 3781782  =      0.000 ...  7567.997 secs...
ValueError                                Traceback (most recent call last)
/local_mount/space/truffles/1/users/rlaplant/.canopyrc/User/lib/python2.7/site-packages/mne-0.9.dev-py2.7.egg/mne/io/edf/edf.py in _read_segment(self, start, stop, sel, verbose, projector)
276                     print int(sfreq), n_chan, blocks
277                     data = data.reshape((int(sfreq), n_chan, blocks),
--> 278                                         order='F')
279                     for i in range(blocks):
280                         start_pt = int(sfreq * i)
It is trying to reshape a 232352736x1 matrix into a 499x129x7568 matrix.
It don't even know where to begin debugging this -- it seems like it is not figuring out correct information in the header? Maybe there is some encoding issue?
@dgwakeman suggested to tag @teonlamont
—
Reply to this email directly or view it on GitHub.

Yes this is on a semi-recent dev version.
I will try it again soon and update.

@aestrivex I found a solution to the problem with pr #1771.


Documentation missing important details

Describe the bug
A clear and concise description of what the bug is.
Documentation of management_zone_v2 does not say what the possible values look like for the rule types for management zones. I can see it in the UI, but there's no information on how to properly add that to terraform. Very, very, very frustrating.
To Reproduce
N/A
Expected behavior
Options in documentation provided
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
NA
Smartphone (please complete the following information):
NA
Additional context
Add any other context about the problem here.

Hi @narora138,
We apologize for the inconvenience, the schema documentation in general was being created with what was returned by the Dynatrace API. There were recent enhancements that were made on our end to include additional pieces of information including the possible values based off of the defined enums.
Unfortunately, management_zone_v2 was one of the older resources that were created without this improvement. I'll update the documentation for management_zone_v2 to be included in our next release. We'll be reviewing all of the other older resources to improve documentation in future releases but it may take some time as it will be a manual effort, we appreciate your patience.


Never start a goroutine without knowing how it will stop

Source: https://dave.cheney.net/2016/12/22/never-start-a-goroutine-without-knowing-how-it-will-stop

Or perhaps the related, from Bryan C. Mills:

Start goroutines when you have concurrent work to do now.


Can't find hand.dae .

I notice that in inference.ipynb, you have a file named "hand.dae"
gripper_trimesh = trimesh.load('/mnt/huangjiangyong/VRKitchen/hand.dae', force='mesh')
Could you tell me how to get this file?

This file is borrowed from PerAct. I forget where I got this file. Send me an email<EMAIL_ADDRESS>and I will reply with this file.


How to properly configure return path to receive bounces in the correct mailbox

Hi there, I've been working with Postal since years ago, but now I'm awaring of a thing that I didn't realize before, and that is that we are not receiving the bounces emails to our mailboxes (for example autoresponders). So what is the correct way to configure the postal server so that the next example works fine (asuming domain.com is a domain configured in Postal that is installed in mailserver.com acting as outgoing/smtp server) ?

<EMAIL_ADDRESS>send mail to<EMAIL_ADDRESS>that has an autoresponder, so<EMAIL_ADDRESS>receive a message with the auto-response.
<EMAIL_ADDRESS>send mail to<EMAIL_ADDRESS>that has an autoresponder, so<EMAIL_ADDRESS>receive a message with the auto-response.

I use a two users/address examples to remark the need of each one receive his own "bounced" autoresponder, and not to have a "catch-all bounces" address or something like that.

In our case, based on our tests, the auto-responses are been sending to<EMAIL_ADDRESS>which could be good but are getting trapped in Postal (and never reach the original sender mailbox) as you can see in the screenshot. Moreover it cannot link it to the original mail sent  :

My postal.yml:
mx_records:
 -mx.mail.mailserver.com
smtp_server_hostname: mail.mailserver.com
spf_include: spf.mail.mailserver.com
return_path: rp.mail.mailserver.com
route_domain: routes.mail.mailserver.com
track_domain: track.mail.mailserver.com

My DNS zone for the mailserver:
;; A Records
mail.mailserver.			1	IN	A	xxx.xxx.203.204
mailserver.				1	IN	A	xxx.xxx.203.204
track.mail.mailserver.		1	IN	A	<IP_ADDRESS>

;; AAAA Records
mail.mailserver.			1	IN	AAAA	fe80::250:56ff:fe01:28dc
track.mail.mailserver.		1	IN	AAAA	fe80::250:56ff:fe01:28dc

;; CNAME Records
mx.mail.mailserver.			1	IN	CNAME	mail.mailserver.
rp.mail.mailserver.			1	IN	CNAME	mail.mailserver.

;; MX Records
routes.mail.mailserver.		1	IN	MX	10 mx.mail.mailserver.
rp.mail.mailserver.			1	IN	MX	10 mx.mail.mailserver.

;; TXT Records
mailserver.				1	IN	TXT	"v=spf1 include:mx.ovh.com ~all"
rp.mail.mailserver.			1	IN	TXT	"v=spf1 a mx include:spf.mail.mailserver ~all"
spf.mail.mailserver.			1	IN	TXT	"v=spf1 ip4:xxx.xxx.203.204 ip4:xxx.xxx.147.102 ip4:xxx.xxx.14.126 ip4:xxx.xxx.124.163 ~all"
postal._domainkey.rp.mail.mailserver.	1	IN	TXT	"v=DKIM1;......."

My DNS zone for the domain configured in Postal:

All that been said, we are using our own separated Incoming Mail server for the "domain.com".
Thanks
PS: All info and data has been "changed/obfuscated" even in screenshots for security.

Postal is designed to handle transactional email from software and as such you currently need to configure a return path route so that Postal knows what to do with the returned emails, i.e. direct them back to the application that sent them in a way it can understand.
There is more detail in the wiki https://github.com/postalhq/postal/wiki/Auto-Responders-&-Bounces
It sounds like you're using this for direct person-to-person email so I'm not sure this will serve you that well.


vcpkg install blaze:x64-windows --head broken

>vcpkg.exe install blaze:x64-windows --head
The following packages will be built and installed:
    blaze[core]:x64-windows (from HEAD)
Starting package 1/1: blaze:x64-windows
Building package blaze[core]:x64-windows from HEAD...
-- CURRENT_INSTALLED_DIR=D:/Devel/vcpkg/installed/x64-windows
-- DOWNLOADS=D:/Devel/vcpkg/downloads
-- CURRENT_PACKAGES_DIR=D:/Devel/vcpkg/packages/blaze_x64-windows
-- CURRENT_BUILDTREES_DIR=D:/Devel/vcpkg/buildtrees/blaze
-- CURRENT_PORT_DIR=D:/Devel/vcpkg/ports/blaze/.
-- Downloading https://api.bitbucket.com/2.0/repositories/blaze-lib/blaze/refs/branches/master...
-- Downloading https://api.bitbucket.com/2.0/repositories/blaze-lib/blaze/refs/branches/master... OK
CMake Error at D:/Devel/vcpkg/scripts/cmake/vcpkg_download_distfile.cmake:54 (message):
  vcpkg_download_distfile requires a FILENAME argument.
Call Stack (most recent call first):
  D:/Devel/vcpkg/scripts/cmake/vcpkg_from_bitbucket.cmake:159 (vcpkg_download_distfile)
  D:/Devel/vcpkg/ports/blaze/portfile.cmake:3 (vcpkg_from_bitbucket)
  D:/Devel/vcpkg/scripts/ports.cmake:72 (include)

Error: Building package blaze:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: blaze:x64-windows
  Vcpkg version: 0.0.103-2018-01-30-5700754bd11fe9b22728bd8682c9cfe67a69a462

Additionally, attach any relevant sections from the log files above.

@ras0219-msft sorry, but It still doesn't work. I have tried to add my own library from BitBucket and look to Blaze as an example. It seems that Blaze is only one library from BitBucket which added to vcpkg.


Right to Left support

you just need to set gravity for views , or you could add a separate layouts

I have push the update to support RTL,please pull from the update (not release a new version)

version 0.1.19 support RTL


Siamese network dropout

How does Keras handle when we decide to use dropout with in branches of Siamese layer?, Is it possible to sync dropped weights?
@farizrahman4u

I'd be interested in that as well! Is dropout synchronized anyway because one reuses the same model in Keras to build the two branches of the siamese network? Or are different neurons dropped in the two branches when passing a sample pair?


The filter for the monitor works poorly and there is no documentation for cmake. (IDFGH-1654)

idf.py monitor --print-filter="*:E GATTS_TABLE_DEMO:I"
Displays a message ( Error: No such command "GATTS_TABLE_DEMO:I". )

idf.py monitor --print-filter="GATTS_TABLE_DEMO:I"
this works good, but I need the rest of the tags

Which os and shell?  Did you try escaping the space?

Hi @ArTyGill. Thanks for reporting the issue but unfortunately I cannot reproduce it based on the information you've provided. The print-filter used by you is working for me.
So please provide also the full log message and IDF version beside the previously mentioned OS and shell. Thanks!

@ArTyGill Do you have any update regarding your issue?

@ArTyGill Thanks for reporting the issue, will close this due to short of feedback. Feel free to reopen if you still have this issue, thanks.


don't run cargo fmt --all in pre-commit hook

This action started failing because cargo fmt was added to the pre-commit hook and the tool is not available without running Rust setup. Instead, we should just skip applying hooks when creating commits from this workflow.
~Since the create-pull-request doesn't have a way of disabling hooks, simply remove the file before husky installs it into place.~ Let cargo fmt run on individual files through lint-staged, but don't run it on everything on every precommit.
Test Plan: Manual workflow run on this branch which opened #3401

is it possible to only run cargo fmt when relevant rust files change? this looks like it's not going to touch any of those files, so the pre-commit hook shouldn't apply. (I'm not sure if lint-staged is already supposed to be doing that?)

it actually does already: https://github.com/vercel/turbo/blob/a3bfa40ccd67862b3640731d06a2529a455fd2e5/package.json#L51
...but cargo fmt --all was added to the hook in #3179: https://github.com/vercel/turbo/commit/daaeabfb502d3b1f6c1acfae31d0d2cf8a740463#diff-d2bc4bbf14eadc292d84e0ef5f7a93115c23557f533809fc80b896934291529dR5
@ForsakenHarmony was there a reason to format everything on every change? cargo fmt is probably fast enough to do so, but it requires the rust toolchain in situations like these.

@wbinnssmith we can remove it

Why do we have the google font data in our repo though, I thought we wanted to use it from the next package directly

Why do we have the google font data in our repo though, I thought we wanted to use it from the next package directly (@next/font/dist/google/font-data.json)

Yeah, we do. We just haven't gotten to it yet, and this was breaking.


Create "Dolphin" sample

@Foxar please describe the plan for "Dolphin" sample if you could.

Dolphin sample:
Simple mini-game about swimming around as a dolphin, collecting collectibles by jumping and exploring, and avoiding threats. Possibly having multiple levels that switch upon enough collectibles being found.

Dolphin sample TO-DO

[ ] Animated water(mesh,code)
[ ] Collectible (mesh,code)
[ ] Seabed (mesh,code)

Please dont forget about readme.md also :)


SAM crashes when provided a kWh/kW rate in only some months

Describe the bug
A bug report from @drevastation came in regarding TOU rates

Still not fixed, in 2020.11.29r2 and 2021.12.2. Using a Time-of-Day rate from Public Service company of Oklahoma GUID: 5dae1b2f5457a3cc62d2ae4f This crashes both versions.

This rate has kWh rates in June through October, but kWh/kW rates the rest of the year. SAM crashes when attempting to run this rate.
To Reproduce
Steps to reproduce the behavior:

Open attached SAM file:
psco_tiered_rates.zip
Click simulate
See crash

Expected behavior
SAM should support the Oklahoma rate without crashing
Operating System and Version

OS: Windows
Version 2020.11.29r2, 2020.12.02r0

Closed in #889


[VirtualMachineClient] ListComplete response (iter.Value()) is not consistent with Get API and Azure CLI

Bug Report

github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2021-12-01/compute
SDK version -v61.4.0, v63.0.0
Go verion - go version go1.17.8 linux/amd64

What happened?
Resource field (struct) has properties, location and tags fields set to nil for VirtualMachine response received from ListComplete call.

However, Azure CLI command (az vm list --resource-group {group_name}) and VirtualMachineClient's Get call return values for those fields.

What did you expect or want to happen?
Get consistent VirtualMachine data when using ListComplete, Get, and CLI commands
How can we reproduce it?
Compare outputs from ListComplete, Get and CLI command (az vm list)

Thanks!

@tadelesh, could you help to take a look?

@diyor-uptycs I've tried the API and it works fine. Could you please paste your code here?

@tadelesh Can you expand Resources field? Can you see properties and location fields being not nil?
Try to compare those two fields to the result (the same fields) from the CLI command (az vm list --resource-group {group_name}).
Result of the CLI command provides value to those fields.

@diyor-uptycs Sorry for misunderstanding. You are right. The HTTP response for list API seems lose some param. CLI might use some special logic to put back such param. I'll involve the service team to have a look. Thanks for the feedback.

@Drewm3 Can you please take a look at this VM issue?

Moving Jayla's issues over to me.

@diyor-uptycs or @navba-MSFT, could you clarify the bug that you believe is occurring in the the REST API along with the specific API that you feel has the bug? It is unclear from the issue above.

Resolving this due to no update


Update backend services to be compatible with PDB2PQR v3.1.0

Resolves #98
NOTE: merging these changes will make the backend incompatible with the frontend until PR Electrostatics/apbs-web#38 is also merged

@intendo @mmacduff
Can we hold on merging this to main until we can alert site users via banner/alert (see Electrostatics/apbs-web#39)


ENH: Improve itk::ResampleInPlaceImageFilter coverage

Improve itk::ResampleInPlaceImageFilter class coverage: test the
exceptions.
PR Checklist

[X] No API changes were made (or the changes have been approved)
[X] No major design changes were made (or the changes have been approved)
[X] Added test (or behavior not changed)

The macOS builds are failing without testing the patch set; failures are being reported since a few days in the master.


Added support for HmIP-DRSI1

Please point your pull request at the devel branch. Also provide some information about the changes.
This pull request:

fixes issue: -
adds support for HomeMatic device: HmIP-DRSI1

New class: -
Home Assistant platform: -

does the following: enable switch

nevermind i didn' t see that the pull request for HmIP-DRSI1 was already there.


[fuchsia] use the binary tools in the fuchsia SDK

This commit moves from using the bash scripts to using the binary tools.

Review is https://dart-review.googlesource.com/c/sdk/+/221024


fix(PathSelector): problem about PathSelector which gets the  wrong ArrayDepth

Before submitting a pull request, please make sure the following is done...

[ ] Ensure the pull request title and commit message follow the Commit Specific in English.
[ ] Fork the repo and create your branch from main.
[ ] If you've added code that should be tested, add tests!
[ ] If you've changed APIs, update the documentation.
[ ] Ensure the test suite passes (npm test).
[ ] Make sure your code lints (npm run lint) - we've done our best to make sure these rules match our internal linting guidelines.

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

when repeat dragging arrayTable into arrayTable, and the name of ArrayColumn is the same as the component inside the ArrayColumn, the PathSelect will render these repeatedly named components multiple times，as the value of the node is composed of array depth, but the depth calculation is not right


Added catppuccin-mocha theme

Added a theme for catppuccin mocha.
Used Colors
bgcol = Base
fgcol = Text
hical = Green
hical2 = Rosewater
hical3 = Peach
errorcol = Red
Link to the theme: https://github.com/catppuccin/catppuccin

Preview of the theme:


cachepath without subdirectories

Setting cache path to j:\tmp\test.json, BestSource creates files like: J:\tmp\test.json\TestClips&Co\files\test.avi.0.bsindex, would be nice if instead of the whole path simply a hash of the path could be used?
So instead of J:\tmp\test.json\TestClips&Co\files\test.avi.0.bsindex` use:

J:\tmp\test.json\<crc32 Hash of Path to input file>.0.bsindex or
J:\tmp\test.json\<crc32 Hash of Path to input file><input file name>.0.bsindex or
J:\tmp\test.json\<crc32 Hash of Path to input file>\input file name>.0.bsindex.

Thanks for considering.

Done


Importing this library from a plain javascript project depends on tslib

@accordeiro created a new javascript project from scratch. He installed soroban-client, and did npm install.
His index.js was:
var SorobanClient = require('soroban-client');
var server = new SorobanClient.Server('http://localhost:8000/soroban/rpc');

// get the sequence number for an account
server.getAccount(
  'GASOCNHNNLYFNMDJYQ3XFMI7BYHIOCFW3GJEOWRPEGK2TDPGTG2E5EDW'
).then(function(r){ console.log(r); });

But when trying to run it, the error was:
><EMAIL_ADDRESS>start
> node index.js

node:internal/modules/cjs/loader:936
  throw err;
  ^

Error: Cannot find module 'tslib'
...

npm install tslib resolved the issue, but this shouldn't be a requirement for users.
From a quick google it looks like we should set importHelpers: false in this library's tsconfig.
Note: This might also affect js-stellar-sdk?

Also should probably change line 2 in the example to:
var server = new SorobanClient.Server('http://localhost:8000/soroban/rpc', {allowHttp:true});

Since it rejects non-https RPC servers by default. Happy to also file a separate issue for this one, let me know.

This is no longer true.