ENH: ImageJ ImgLib2 example

Example from:
https://github.com/CellProfiler/notebooks/blob/95a2558060075b0c8c9ad254413fac8dacfc2d12/imagej_pyjnius_demo.ipynb
Co-authored-by: Curtis Rueden<EMAIL_ADDRESS>

CC: @ctrueden


NIFI-10965 PutGoogleDrive

Summary
NIFI-10965
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[x] Apache NiFi Jira issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-10965
[x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-10965

Pull Request Formatting

[x] Pull Request based on current revision of the main branch
[x] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[ ] Build completed using mvn clean install -P contrib-check

[x] JDK 8
[x] JDK 11
[x] JDK 17

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[x] Documentation formatting appears as expected in rendered files

Will review


Use -ListAvailable for Get-Module PSReadline bug template

My VM is a bit weird where some modules do not auto-load in some PowerShell instances (at least the first time before they are being used by a different host such as e.g. the ISE). Therefore the issue template did not work for me.

This tells me every version of PSReadLine installed on the machine, but not the one currently being used. I suppose I could assume you're always using the latest, but I shouldn't assume such things.

Ok, I see, I'll close then


Redirect of USB device failed

I have installed USBDK, but  I excute the command“UsbDkController.exe -r ID SN”in cmd on windows7(32 bit), the error "Driver file operation error. DeviceIoControl failed (The system cannot find thefile specified.  Error code = 2)"  will appear. How to solve this problem?
thank you！
    The cmd screenshot is following：

C:\Program Files\UsbDk Runtime Library>UsbDkController.exe -i
Installing UsbDk driver
UsbDk driver installation succeeded
C:\Program Files\UsbDk Runtime Library>UsbDkController.exe -n
Enumerate USB devices
Found 5 USB devices:
0. FilterID: 1, Port: 1, ID: 8087:0024, Configs: 1, Speed: 3
USB\VID_8087&PID_0024 1
Descriptor for configuration #0: size 25

FilterID: 3, Port: 1, ID: 8087:0024, Configs: 1, Speed: 3
USB\VID_8087&PID_0024 1
Descriptor for configuration #0: size 25
FilterID: 4, Port: 2, ID: 413c:2107, Configs: 1, Speed: 2
USB\VID_413C&PID_2107 2
Descriptor for configuration #0: size 34
FilterID: 4, Port: 4, ID: 17ef:6019, Configs: 1, Speed: 2
USB\VID_17EF&PID_6019 4
Descriptor for configuration #0: size 34
FilterID: 5, Port: 1, ID: 0951:1665, Configs: 1, Speed: 3
USB\VID_0951&PID_1665 60A44C413C4EFEC1798423E8
Descriptor for configuration #0: size 32

C:\Program Files\UsbDk Runtime Library>UsbDkController.exe -r 0951:1665 60A44C413C4EFEC1798423E8
Redirect USB device 0951:1665, 60A44C413C4EFEC1798423E8
Driver file operation error. DeviceIoControl failed (The system cannot find thefile specified.  Error code = 2)
Redirect of USB device failed
C:\Program Files\UsbDk Runtime Library>UsbDkController.exe -r 0951:1665 60A44C413C4EFEC1798423E8
Redirect USB device 0951:1665, 60A44C413C4EFEC1798423E8
Driver file operation error. DeviceIoControl failed (The system cannot find thefile specified.  Error code = 2)
Redirect of USB device failed

Hello and thank you for your report.
The correct syntax for what you’re doing is
UsbDkController.exe -r "USB\VID_0951&PID_1665" 60A44C413C4EFEC1798423E8
Hellp screen of UsbDkController.exe is indeed misleading, I will fix it in the next UsbDk version.
Best Regards,
Dmitry

Help screen clarifications committed at 51ca12740b03237715bffb576c1d78635590b72e


make sure example #17 works after first time, and when that image is …

example 17  would work only the first time, and only if that image was the first one to be clicked
not sure if this is the best way to do it, but it seems to be working fine

Thanks, I also added it to the gh-pages branch


Task: Improve performance of Accessibility Statement page

Describe the task
Clean and Green Philly's Accessibility Statement page has a Google Lighthouse performance score of 74/100 on mobile devices, compared to 97/100 on desktop devices. Reduce this page's image loading time and Improve the score to high 80s or low-to-mid 90s.
Acceptance Criteria

[ ] Reduce image loading time for Accessibility Statement page
[ ] Generate Lighthouse report for mobile devices, showing improvement in performance score
[ ] Post screenshots of results in this issue's comments section

Additional context

See #740 for details
For instructions on using Lighthouse, see https://developer.chrome.com/docs/lighthouse/overview

Will revisit performance later in 2025 after we've upgraded UI


Remove the syntex dependency

It's deprecated. It has a long build time. We should get off of it.
I think the new hotness is syn (or maybe synom?) and quote?

I'm working on this.

Unfortunately, it looks like using the quote crate doesn't result in pretty printed Rust code.

You can pass the output through rustfmt::format_input to make it pretty.

Unfortunately, rustfmt is nightly only now, and bindgen can't be nightly only.
I don't think we want to stick to an old rustfmt (one that still worked on stable) either. That's no better than sticking with syntex.
We recently gained the ability to run the $PATH's rustfmt on the generated bindings, and I think we can rely on that functionality here. It's "just" a matter of CI and testing integration.

This is hItting a bug in rustfmt that is making the formatted rust input invalid: https://github.com/rust-lang-nursery/rustfmt/issues/1914

Can work around it by using a custom rustfmt.toml and passing it to the rustfmt invocation we use.

Update: I have a branch without the syntex dependency that can generate bindings for all but one of our test headers (it just hangs in that one test header, and I haven't investigated it yet) and the emitted bindings have changed a little in inconsequential ways but are all passing their layout tests. There is also still some clean up to do and some constify module enums things.
Getting pretty close though.

I have it passing all tests \o/
There is still some code clean up and minor rebasing to be done before this gets a PR.


Task - Filter collection object, more facets

with/out the 3 buffered fields (param name is the field name)
preparation type preparation_type_id[]= or preparation_type_id (the former for this facet, picklist style)

Needs model from @mjy.


Add nodes/metrics to allowable resources

Pulled in the RBAC yaml for the prometheus-operator and saw that it couldn't target our kube state / node exporter metrics, complaining about a 403.

Looks like this failed CI because it needs an update to the generated docs.
@jpreese can run make generate and commit the generated doc updates?

@pgier pushed!


[Pulsar Admin] Add support for creating replicated subscriptions

Is your feature request related to a problem? Please describe.
pulsar-admin topics create-subscription doesn't currently support creating replicated subscriptions.
Describe the solution you'd like
Add option for specifying the replicated parameter for subscription creation.

I will try to make it.

@Technoboy- One of the challenges in making the changes is keeping the interfaces in Pulsar Admin Java API backwards compatible. Please consider that when doing the changes.

The issue had no activity for 30 days, mark with Stale label.


Controlling the interval for streaming data

Hi!
While reading data from a CSV file and streaming it, I would like to set the time interval (say, 1ms or 10s) for reading each line of data and streaming it. This would allow me to create a truly real time streaming effect for IoT data. I did explore tweaking the read.max.wait.ms parameter, but apparently that parameter has a different purpose.
Is there an easy way to accomplish what I'm trying to do?
Thanks!

Hi @sbanga, the read.max.wait.ms config property is used to wait fore more data when the task has reached the end of file. Currently, there is no mechanism to wait before sending next records.
If this is only for testing purpose you can write a simple bash script that sleep for 1 sec before writing a new record to a CSV file.

Hi @sbanga, the read.max.wait.ms config property is used to wait fore more data when the task has reached the end of file. Currently, there is no mechanism to wait before sending next records.
If this is only for testing purpose you can write a simple bash script that sleep for 1 sec before writing a new record to a CSV file.

Thanks @fhussonnois. The approach you outlined is what I have currently implemented.

Thanks @fhussonnois. The approach you outlined is what I have currently implemented.


Revert ROUNDTRIP Rules

Revert https://github.com/Azure/azure-rest-api-specs/pull/25609 for the following reasons:

Azure Terraform requires that onboarded APIs must comply with the roundtrip rules.
These rules are not described in any other alternative places.
These rules are still existing in oav

/pr RequestMerge


Latest BP Goal is not always respected

In the sandbox, I updated the BP goal of a patient from 135/85 to 140/85. While the graph indicated that the patient had achieved their goal, the recommendation did not. Both the old and new goal are passed in the prefetch. If I remove the old goal from the prefetch, the recommendation is correct. So something in the CQL is not respecting the most recent Blood Pressure goal. Here is the hook request that should result in "Goal Achieved" to the monitoring plan definition:
goalbug.txt

There are a few problems with the current definition of the BP Goal and the logic to determine the most recent goal. This is how it's described in the Workflow Logic document:

Definition
Meaning

Qualifying Blood Pressure Goals
A Goal with a lifecycleStatus in ('proposed', 'planned', 'accepted', 'active', 'completed'). Goal must have a single target, with one measure in the ValueSet for Systolic Blood Pressure (http://cts.nlm.nih.gov/fhir/ValueSet/2.16.840.1.1137<IP_ADDRESS>4.2) and one measure in the Value Set for Diastolic Blood Pressure (http://cts.nlm.nih.gov/fhir/ValueSet/2.16.840.1.113883.3.526.2.1045). Both measures must have detail that is a quantity with unit mm[Hg].

Most Recently Established Blood Pressure Goal
Most recent by start of Qualifying Blood Pressure Goals. "start" can be a date or a codeable concept. We are relying on this being present and being a date for the logic to work.

The lifecycleStatus filter is including goals that aren't actually active. It's questionable whether 'proposed' or 'planned' should be included, but 'completed' definitely shouldn't be.
The CQL is only looking at the startDate of the Goal. COACH is not sending a start date. It is probably more correctly sending a statusDate which is the date that the current lifecycle status took effect.
It is possible for neither start date nor status date to be present, and it is possible for "start" to be a CodeableConcept rather than a date. So we should make allowances for this.

Since "start" may not be a date at all, I decided not to try and fall back on it. The assumption is that there will be a statusDate and that if we have more than one "active" or "accepted" BP Goal, the one with the latest status date is most relevant. Other lifecycle statuses are no longer considered.

I tested this once deployed. I had to update the patient's goal to 140/86 for the goal reports to align. There may be a difference between COACH and the recommendations in terms of whether the BP average must be strictly less than or less than or equal to the goal.

This turns out to be a rounding difference. The actual average of the last BP set is:
TEST Get Average Most Recent BP Set=Tuple {
"systolic":<PHONE_NUMBER>4
"diastolic": 85.36603529
}
So technically, they are still above goal. But the graph rounds the average to 137/85 first and uses these numbers to compare to goal. This is an edge case. We can consider addressing in the future.

We got an error report today and after digging in, it seems the COACH is throwing an NPE because goals coming from the EHR do not contain a start date. COACH is still sending a status date and not a start date when passing the Goal to CQL. On reflection, it seems like the Goal start date is more important if known. The start date should denote when the patient first started working on the goal, while the status could be updated periodically throughout that time. We have decision logic that computes how long someone has worked toward the goal in order to determine whether to send them to Pharma recommendations, so 'start' is more appropriate.

The COACH tasks should still be implemented, but CQL will now use the goal start is present and a date and goal status if not.


[Documentation] the alt and id attributes on some examples are not up to date

On some examples on scenegraph,  the alt and id attributes are not up to date with the corresponding models.
https://modelviewer.dev/examples/scenegraph/

PRs welcome 😁

Done. my first PR here :  3040


Missing win-arm64 msi build for 7.0

The earlier builds have a win-arm64 build but I do not see one from 7.0 (even the RCs have them). Would it be possible to get one so we can use on Microsoft's own Surface Pro X device without x86 emulation. Thanks.

/cc @heaths for information.

@iSazonov I don't own the installers. I'm not sure who does. I'm only fixing the many upgrade issues.

@heaths I ping you only for information so that this request is not a surprise for you.

Thanks. There's no platform specific to ARM/64 for Windows Installer. x86 and x64 packages work.

I see. That will have to be a separate fix, I believe. The WiX authoring needs to be updated to support it, but there's various conditions that assume BUILD64 and whatever else are only two options, unless ARM64 is part of BUILD64. I haven't looked at that specifically. If it were, that's fine - you only need a minimum of 2 UpgradeCodes since x64 and x86 can be installed SxS. Not sure if that's true of ARM/ARM64 but you can't mix the ARM and x86-based platforms together for MSI, I don't believe.
There is an easier, more permissive way to write all this but it would be a huge undertaking. I'm trying to make significant but less invasive changes to keep the churn down and avoid unforeseen problems. I'm also trying to consolidate the authoring and conditions into the .wxs files themselves, rather than splattering various conditions in the calling scripts as well (wasn't necessary, and there are better ways than setting environment variables in the calling process, which might affect some other scripts later).

WiX added ARM64 support recently: https://github.com/wixtoolset/issues/issues/5558

I see. That will have to be a separate fix, I believe. The WiX authoring needs to be updated to support it, but there's various conditions that assume BUILD64 and whatever else are only two options, unless ARM64 is part of BUILD64. I haven't looked at that specifically. If it were, that's fine - you only need a minimum of 2 UpgradeCodes since x64 and x86 can be installed SxS. Not sure if that's true of ARM/ARM64 but you can't mix the ARM and x86-based platforms together for MSI, I don't believe.
There is an easier, more permissive way to write all this but it would be a huge undertaking. I'm trying to make significant but less invasive changes to keep the churn down and avoid unforeseen problems. I'm also trying to consolidate the authoring and conditions into the .wxs files themselves, rather than splattering various conditions in the calling scripts as well (wasn't necessary, and there are better ways than setting environment variables in the calling process, which might affect some other scripts later).

Yes, on ARM64 x86 and ARM64 can be installed SxS. MiX support for ARM64 has been upstream quite some time ago now. ARM64 should not require any special handling.
Do you think we can get this added?
Thanks.

+1 for this, it's quite annoying that there are .msi installers for the x86 and x64 builds, but none for ARM64.

@TravisEz13 Is this important for 7.1 release?

I think we'll see about doing this for 7.2.

Even with the 7.1 release being on dotnet 5.0 and the big thing there is that it works on windows arm64, there still isn't an arm64 windows build?

There is a build:  https://github.com/PowerShell/PowerShell/releases/download/v7.1.0/PowerShell-7.1.0-win-arm64.zip

But it's not an msix
On Thu, Nov 12, 2020, 19:47 Travis Plunk<EMAIL_ADDRESS>wrote:

There is a build:
https://github.com/PowerShell/PowerShell/releases/download/v7.1.0/PowerShell-7.1.0-win-arm64.zip
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PowerShell/PowerShell/issues/12116#issuecomment-726268047,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB4EVEAGR4UDKAJPL3ZB6V3SPQUUJANCNFSM4LGP4JZA
.

There is a build: https://github.com/PowerShell/PowerShell/releases/download/v7.1.0/PowerShell-7.1.0-win-arm64.zip

Yeah, no setup of any kind tho, right? No MSI or MSIX.

@harrisonmetz @pmsjt Welcome to contribute if you have an interest! PowerShell team is very small to address all requests quickly.

While I still think we need an .msi installer, I did manage to "install" it for my needs from the zip file and configure a profile in Windows Terminal. I first installed PowerShell 7.1 x86 to get a profile added in Windows Terminal, which I could copy and edit to point to PowerShell 7.1 ARM64.
I downloaded PowerShell-7.1.0-win-arm64.zip and extracted its contents to "C:\Program Files (Arm)\PowerShell\7.1.0"
In Windows Terminal, I clicked the downward arrow and then clicked "settings" to edit settings.json. I copy/pasted the PowerShell 7.1 x86 entry to end up with the following:
    {
        "guid": "{574e775e-4f2a-5b96-ac1e-a2962a402336}",
        "hidden": false,
        "name": "PowerShell 7 (ARM64)",
        "commandline": "C:\\Program Files (Arm)\\PowerShell\\7.1.0\\pwsh.exe"
    },
    {
        "guid": "{34a13805-9b56-5e89-9234-f02de44bd971}",
        "hidden": false,
        "name": "PowerShell 7 (x86)",
        "source": "Windows.Terminal.PowershellCore"
    },`

`
I now have "PowerShell 7 (ARM64)" and "PowerShell 7 (x86)" in Windows Terminal, alongside other entries like Windows PowerShell and the old Command Prompt. It is good enough for me now, but I'd rather have an .msi installer.

Even though PowerShell MSIs or MSIXs are not being made available here, they are somehow managing to make their way into the Store.
https://www.microsoft.com/store/productId/9MZ1SNWT0N5D
Installing from the store is actually more convenient - at least for me - because it takes care of the updating automatically for me.

Can confirm the version from the Store is native ARM64 🚀 Thanks for sharing @pmsjt!


[ktx] update to 4.0.0-beta6

Updates version to 4.0.0-beta6
It is impossible to use previous (beta5) version
CMake complains about dll and then INTERFACE_INCLUDE_DIRECTORIES

Hi @skaarj1989
Thanks for your PR.
Could you please merge master and execute the command vcpkg.exe x-add-version ktx?

Hi @skaarj1989
Thanks for your PR.
Could you please merge master and execute the command vcpkg.exe x-add-version ktx?

Done

Done

The failures on x64-windows and x64-windows-static like this:
-- Configuring done
CMake Error at CMakeLists.txt:167 (add_dependencies):
  The dependency target "mkvk" of target "ktx" does not exist.
Call Stack (most recent call first):
  CMakeLists.txt:387 (commom_lib_settings)

CMake Error at CMakeLists.txt:167 (add_dependencies):
  The dependency target "mkvk" of target "ktx_read" does not exist.
Call Stack (most recent call first):
  CMakeLists.txt:388 (commom_lib_settings)

Could you please look into this?

The failures on x64-windows and x64-windows-static like this:
-- Configuring done
CMake Error at CMakeLists.txt:167 (add_dependencies):
  The dependency target "mkvk" of target "ktx" does not exist.
Call Stack (most recent call first):
  CMakeLists.txt:387 (commom_lib_settings)

CMake Error at CMakeLists.txt:167 (add_dependencies):
  The dependency target "mkvk" of target "ktx_read" does not exist.
Call Stack (most recent call first):
  CMakeLists.txt:388 (commom_lib_settings)

Could you please look into this?

I can't reproduce that failure.

I can't reproduce that failure.

/azp run

/azp run

https://github.com/KhronosGroup/KTX-Software/releases
KTX 4.0.0 is released


chore: fix vulnerabilities

Description
Fixes a vulnerability in the webpack-dev-server and storybook packages.
Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own changes
[x] I have run yarn lint to make sure my changes pass all tests
[x] I have run yarn test to make sure my changes pass all linters
[x] I have pulled the latest changes from the upstream main branch
[ ] ~I have tested both the react and the CDN versions on local and integration environments~
[ ] ~I have added the necessary labels to this PR in case a new release needs to be published after merging into main (e.g. release and patch)~

Contribution guidelines
For contribution guidelines, styleguide, and other helpful information please
see the CONTRIBUTING.md file in the root of this project.

:rocket: PR was released in v2.29.0 :rocket:


Build breaks  when updating  mongoose

It seems that the latest update of mongoose has added native typescript definitions, which are more accurate than the @types/mongoose definitions used previously. This causes some of the definition to break, mostly around some of the null validation methods added in #1550 .
There is also a mismatch between the API input and the Database input for the simulators.
I will be working to fix this in #1618

Lets set aside the updating of mongoose for later. We need to focus on getting minimal functionality working before 12/20.
I've been trying to minimize the mismatch between the API and database. We really need to implement a test that checks this. What is the mismatch between the API and database?

Agreed. I skipped the update, but did fix some of the typing issues in #1628

The update revealed some type issues that were not being checked in the previous version

The number of new type issues isn't that big. I don't seen anything that appears to be related to the changes that I made with the handling of null. The issues are related to type casting and only having some attributes defined in some places (e.g., meta of BiosimulatorsMeta).
A good test of the APIs would be to upload a document to the API, have the database save it, retrieve the document from the API and check that the documents are the same (up to ApiResponseProperty properties). This would help identify any unintended type casting that Mongoose is doing.

Yup, I was able to fix the changes with the validators. There was a field that was spelled incorrectly, and a method that explicitly needed undefined passed to it to satisfy the compiler. The other type issues seems to be a case of the compiler being less sure of the type than  we are. That will probably just need to be cast.


Behaviour after selecting multiple emails

In an inbox (including unified) if you select multiple emails and Mark as read our delete them, the inbox remains in select multiple mode, so when you tap another email to open, it's added to the non existent selection rather than behaving as expected.
V5.733

For "mark as read" this is intentional. Maybe the user wants to move the messages to another folder afterwards. It'd be annoying having to select all previously selected messages again. Whereas it's just one tap to exit selection mode if marking those messages as read is all the user wanted to do.
After using the "delete" action selection mode should be exited. This is working as expected on all of my test devices. Can you provide more information on how to reproduce the issue? And/or record a video of the behavior you're seeing?


missing references

hi, you seem to be missing some crucial references to mr LeCam! Thanks for fixing

added! so good you caught that pesky missing reference


result_inline.html character encoding issue

Copied from wq/wq.db#76:

When creating a campaign with a field name containing a single quote(ex: Child's Name), the rendered label in result_inline.html partial doesn't encode the quote properly(ex: Child's Name). However the description containing the same text renders correctly(ex: Child's Name).

It might be possible to fix this by updating the template to use triple Mustache bracket which will trigger verbatim rendering ({{{type_label}}}).
Note that I am in the process of updating several dependencies and I will revisit this app once that is done. (See wq/vera#5)

The triple mustache bracket does indeed fix the HTML rendering.  In the future, I feel like we might want to address the issue server-side.  /reports/new.json returns the html-escaped label format(Child's Name) and probably shouldn't.

Ok, I think once wq/vera#6 is merged, this can be closed.


correct URL opens in new tab

Can you move the code to separate file facebook.js which exports a function and takes the browser as argument ? As tests increase this file will start getting bigger.


feat: implement Validate for Cow

Implements the Validate trait for Cow when the type it's wrapping implements Validate already, similar to all the other smart pointer types.

The nightly test task seems to be failing due to a mismatch in the output of the ui/compile-fail/pattern_mismatched_types.rs test.
This fails the same way when my change is not present when I run it locally, so I'm guessing it's a matter of rustc's version.
The output is the same, but there are two lines that switched places here:
Expected
   = help: the following other types implement trait `Matcher`:
            once_cell::sync::Lazy<T>
            Regex

Actual
   = help: the following other types implement trait `Matcher`:
            Regex
            once_cell::sync::Lazy<T>

I'm not sure how to proceed.


Refactor native contract unit tests to use test runner

Summary or solution description
Part of an overall refactoring.
Changed the execution tests of the compiled smart contracts related to Neo Native Contracts to use TestRunner instead of TestEngine
Tests
Rerun the modified unit tests

Task linked: CU-864drmw8u Substituir a TestEngine em testes unitários dos smart contracts nativos do Neo

Coverage: 91.157% (+0.07%) from 91.083% when pulling def45fcacd5290130a4fc1f4a100ed79f8fab864 on CU-864drmw8u into 3250ba0c1274e6a53ecfd0856afd4b526417e1cc on development.


Ability to organize shortcuts to apps, docs and excels in a dashboard by dragging and dropping

Suggested new feature or improvement
Dashboard should have the capability to add short cuts to various applications and/or docs or excel sheets for developer to access easily.
Developer  should be able to drag and drop any application from the computer to the dashboard.
Scenario
Developer wants to organize all the apps documents that he uses into the dashboard so everything is easily accessible.
Developer should be able to drag and drop these into the dashboard.
Additional details
No response

I'm curious why a user would prefer putting these on the dashboard over using Start Menu, Taskbar, or the Desktop for launching.


Set default sylius payment selector to avoid failure during form type…

This is second issue I had while creating functional order test with mollie installed. Happens on PUT /api/v1/checkouts/select-payment/ request. When using cod (Cash on delivery) payment method.
ArgumentCountError: Too few arguments to function Sylius\Bundle\ResourceBundle\Form\Type\AbstractResourceType::__construct(), 0 passed in /var/www/app/vendor/symfony/form/FormRegistry.php on line 91 and at least 1 expected

/var/www/app/vendor/sylius/resource-bundle/src/Bundle/Form/Type/AbstractResourceType.php:32
/var/www/app/vendor/symfony/form/FormRegistry.php:91
/var/www/app/vendor/symfony/form/FormFactory.php:74
/var/www/app/vendor/symfony/form/FormFactory.php:38
/var/www/app/vendor/sylius/resource-bundle/src/Bundle/Controller/ResourceFormFactory.php:42

I figured that others also are struggling with this issue because there is post that is proposing a fix for it here: https://gitmemory.com/issue/Sylius/PayPalPlugin/154/724763754
I think it is related to Sylius\PayPalPlugin\SyliusPayPalPlugin being installed next to Mollie (but I can not confirm or deny it). It is overriding Sylius\Bundle\CoreBundle\Form\Type\Checkout\SelectPaymentType and then this default value is lost for Mollie. And so sylius is not able to find correct payment selector.
Let me know if this is not related to Mollie bundle. In the end can be fixed with config directly in src/Resources/config/services.xml

Hi, do you still have this issue? If so could you provide me more details?

We're closing this PR due to inactivity.


Add function: *-ServiceDeskProblem

New-ServiceDeskProblem
Remove-ServiceDeskProblem

# Unsure of correct verb to attach request(s) to a problem
Add-ServiceDeskRequest
Add-ServiceDeskProblem
Connect-ServiceDeskProblem
Join-ServiceDeskProblem
Push-ServiceDeskRequest

Add is looking more like the verb to use for attaching requests to a problem:

Adds a resource to a container, or attaches an item to another item. For example, the Add-Content cmdlet adds content to a file. This verb is paired with Remove.

Get-ServiceDeskProblem -Id 12345 | Add-ServiceDeskRequest -Id 67890, 12345


fix(core): Access configmap from another namespace for KEP-1755

Change kubernetes CLI usage to actually get the configmap local-registry-hosting in namespace kube-public for kind action registry configuration from KEP-1755.
Release Note
fix(core): Access configmap from another namespace for KEP-1755

OLM checks should be fixed now. You can rebase please.

OLM checks should be fixed now. You can rebase please.

Done, but I still need to fix some configuration to really test KEP-1755.

@squakez @oscerd could one of you please run again the failing tests ? :pray:

All green. Is it ready to merge?

Yes please.


package manager: the install buttons wrap with long version numbers

Describe the bug
with our long 1.0.0-preview.GUID versions, we get a ugly text wrapping in the button.

I forgot, did we already try truncating like this?


Plugin does not appear to work in Sketch v40.3 (33839)

Sketch shared text styles plugin does everything except actually export the text-styles.json file in Sketch 40.3.

my sketch v40.3 is crashing on importing via shared text styles plugin

Doesn't import anymore: I'm on Version 41.2 (35397)


Pipeline Structure Cache fills up immediately, never drops

Looks like there is no equals() method on PipelineData so you will always get a cache miss.


[Vulkan][Unittests] Add parametrization to vulkan unit tests.

This also switches to using vulkan -from_device=0 by default, and marks tests as pytest.xfail if the device does not support the
functionality being tested.

Potential reviewer: @masahi


Browserify support

First of all, I want to thank you for creating such a great tool, and sorry for my bad English, hope you will be able to understand what I am trying to ask.
I am new to things like RequireJS and CommonJS, so for learning purposes I am now working on a small jQuery-based chat application, based on CommonJS approach for including dependencies and Gulp with Browserify for browser support. I also want your plugin for scrollbars. I've installed npm-package, and now, when I am trying to include it in my "chat.js" file with "require(...), it behaves just like if I include it using "script" tags, assigning itself to a global jQuery object. One of my aims was to create a chat module, which I can just take, and place into another project, without worrying about dependencies. So the chat will have it's own local npm-required jQuery object, and all other necessary libraries included locally. My question is: how can I assign this plugin to some local jQuery object in my module, to use it with browserify? Or is this a wrong approach of using CommonJS? I've seen some other tools like browserify-shim, but I am not sure is this what I really need. I understand that maybe this question is a little offtopic, but anyway hope you can give me some ideas.

Hi, you're right, really we need the same because to pollute the global namespace is a bad practice. Moreover is a must in situations like described here: https://github.com/twbs/bootstrap/pull/15640#issuecomment-71096192
I'm trying to make this library compatible with browserify (in this way) but I thin I need help from developers of the project. :(

:+1: bad globals

@jesusprubio browserify-shim could help, but it looks like dirty solution for me, leading to some another inconveniences. We need real browserify support, but unfortunately it looks like the author is not going to implement it. Are there any alternatives to this plugin, compatible with browserify?

Yep I think it's dirty, we don't like it. As a last alternative we're going to fork the repo and make it compatible.

I currently don't have time to deal with this. I'm hoping to do it the next update.

@malihu waiting patiently.

Please @malihu don't close the issue, let it open a reminder (unless the author prefers it to be closed) :)

@ibc I've closed it accidentally :)

Updated plugin (version 3.0.8) and npm. Thanks to @xqwzts

Awesome, thanks to all.


SCP files and Autocreate Mechfiles

Added support for scping files and the ability to create mechfiles when there is not one detected

I like the idea but I have some qualms about the implementation.
I think it would probably be better to use the prompts that are already in clint instead of using the Python 2 specific raw_input. Especially the y/n one.
I also think the implementation should more closely mimic Vagrant SCP or docker cp

I agree with the prompts, however my gripes with Vagrant SCP is the necessity of knowing the vm name to be able to pull files. To me, it is more clear to state that you either want to "push" or "pull" from the target system. Since Mech's only way of uniquely identifying a given machine is either through the path to the vmx or IP, I do not believe these are not intuitive enough to remember. That being said, a feature that might be worthwhile is allowing the user to specify a friendly name for a given box.

https://github.com/invernizzi/vagrant-scp#examples
You don't need to know the VM name to do scp with Vagrant SCP. The colon specifies the direction that you're copying from so the VM name isn't really necessary. Also mech commands with the exception of list and status are always directed at the mech in the current folder or a mechfile that's higher up in the folder tree.

Updated to reflect SCP syntax update and removed the terrible raw_inputs

I went through the code and commented where I thought necessary. Once those are resolved I'll accept.
Thanks for the PR, it helps a bunch.

Commits squashed and everything you requested is resolved. The mechfile automatically creates a user field (with mech as default) so a user may go in and change it.


unmap pos_inds and neg_inds in anchor_target_single

In anchor_target_single https://github.com/open-mmlab/mmdetection/blob/master/mmdet/core/anchor/anchor_target.py#L151 , the labels, label_weights, bbox_targets and bbox_weights are unmapped to original anchors but pos_inds and neg_inds are not. So the returned inds does not match the returned targets. Would it be better to also unmap the inds. For example,

pos_inds = unmap(unmap(torch.ones_like(pos_inds), anchors.shape[0], pos_inds), num_total_anchors, inside_flags).nonzero()
neg_inds = unmap(unmap(torch.ones_like(neg_inds), anchors.shape[0], neg_inds), num_total_anchors, inside_flags).nonzero()

Thanks! We will check it.


lvgl input driver returning key-off immediately after key-on

Describe the bug
The new input driver code has a non blocking call to get data off the queue (https://github.com/zephyrproject-rtos/zephyr/blob/main/modules/lvgl/input/lvgl_common_input.c#L19), but does not check if the queue was empty. The LVGL callback has no way of detecting that the new data is invalid either, so in practice for touch events this result in the touch getting detected and immediately released in the subsequent call. This also means that dragging across the screen does not work.
The kscan lvgl driver was handling this correctly, by keeping a local copy of the last event and giving it back to the callback if no new data was available.
lvgl_input_read_cb should be modified to behave in a similar way, saving a copy of the last valid message in the common structure and reusing it if ``k_msgq_get returns with no messages available.
@faxe1008 do you have time to look into this? Sent you some more details on discord too.
Thanks!!
To Reproduce
Build the lvgl sample, touch and hold the button, the button press animation is suppressed after one frame, the button should stay highlighted until release.

Will look into this, will be back next Monday. Thanks for raising this 👍, fix should be pretty much what you wrote.


Update order payment method

This PR:

adds the paymentMethodReferenceInput type to GraphQL v2
creates the updateOrder v2 mutation, which for now only updates the payment method, but in later PRs will be expanded to also update the frequency, tier, or amount of the order

Question: with type: new GraphQLNonNull(PaymentMethodReferenceInput) I don't want the paymentMethod to be non-null, because we might be updating the amount, which would not need the payment method id, for example. However I'm not sure how to initialise the new PaymentMethodReferenceInput otherwise? I tried type: new GraphQLInputObjectType(PaymentMethodReferenceInput) but that did not work.

@Betree Pushed up the lastest changes with our attempt at useEffect.
Basically it doesn't seem to wait for the data the first time so our defaultPaymentMethod ends up being undefined, but if we open another card's payment method menu, then it does set the defaultPaymentMethod option.

Today was really tough trying to find the right context and methods for creating a new payment method..... I was using the wrong one for a few hours 😓  I know there's definitely work to be done before this can be merged but extra eyes on it and feedback can't hurt.


Moving Intellisense files to single location

Moving Intellisense to single location in Foundation repo and updated XMLs with latest information.

I think we should reconsider this.  It breaks encapsulation of transport packages.  And it introduces a risk of APIs and corresponding Intellisense being out of sync.

/azp run


Use tracking code to tag season content

Seasons consist of stories, exhibitions, events and catch up videos. Reporting is hampered by the fact that there is no way to compile data except by creating large content groupings or Data Studio filters based on individual urls.
Previously, series had been tagged and used in Data Studio as a custom dimension. As a first idea, this could look like: pageview: { groupType: season, id; seasonId }
This spreadsheet has a tab, "asset list" which lists all pages within the seasons so far:
https://docs.google.com/spreadsheets/d/1yIaLeCk9wgjSlOR0nSLQuuFowPD2gqUSsCksl9cFeqQ/edit#gid=0

This is being covered by https://github.com/wellcomecollection/wellcomecollection.org/pull/6912#issuecomment-900213899


Refs #30231 -- Fixed selenium tests with headless mode

Admin inlines are not visible on screen sizes just larger than the breakpoint when the sidebar is open.
This fixes the new selenium tests which were introduced in #13436 but fail in headless mode.
@felixxm -- While this fixes the test (I've run locally in headless mode anyway)  I'm not sure if it's a suitable patch?

buildbot, test on selenium.


Support multiple --input files with split-video command

As per PySceneDetect v0.5, if multiple -i / --input videos are given, the files will be appended together.  This works for everything except splitting the video with ffmpeg currently (see the TODO in the scenedetect.video_splitter module as per v0.5-beta-1).
Currently, only support for multiple input/appended videos is provided for mkvmerge (i.e. using the split-video command with the -c / --copy option).  This issue is to track the progress of implementing this for the regular split-video command using ffmpeg, which may require writing a temporary file list to disk and deleting it after in order to solve this in a cross-platform way.

Related to #77

Closing as for v1.0 concatenation of input videos will no longer be a supported feature of PySceneDetect.  Videos should be stitched together before running through PySceneDetect.


Bug 2213757: external: separate rgw endpoint validation from cephfs and rbd storage…

…class
currently if the rgw endpoint is not reachable
all the storageclass creation fails
Fix will make the rgw endpoint validation separate and will create rbd and cephfs storageclasses if rgw endpoints validation fails

/auto-cc


FIX: Button going outside the Card

The UI is broken and the button is going outside the Card

I would like to work on this issue, Please assign it under gssoc

I would like to work on it , Please assign this issue to me  and please add level labels

I would like to work on this. Please assign me this issue.

Hey @aakshitaa can you please assign this issue to me under gssoc 24


Word VBA Selection.GoTo: Added links; grouped examples together

Added links to the referenced enumerations and objects
Moved the "Examples" header above the examples
Removed extra blank lines and trailing whitespace

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

VBA/Word-VBA/articles/selection-goto-method-word.md
:white_check_mark:Succeeded

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

VBA/Word-VBA/articles/selection-goto-method-word.md
:white_check_mark:Succeeded

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


add lstm to r1.13

Closing this PR as this not against master, please open a new PR against master
CC @mihaimaruseac


[Azure Devops] Auto complete support

:sparkles: What kind of change does this PR introduce? (Bug fix, feature, docs update...)
Feature.
:arrow_heading_down: What is the current behavior?
Cannot mark PR as autocompleted.
:new: What is the new behavior (if this is a feature change)?
Can mark PR as autocompleted by using new configuration parameter (setautocomplete) in repo command.
:boom: Does this PR introduce a breaking change?
No, parameter is optional and set to false on default.
:bug: Recommendations for testing
N/A
:memo: Links to relevant issues/docs
https://github.com/NuKeeperDotNet/NuKeeper/issues/938
:thinking: Checklist before submitting

[x] All projects build
[x] Relevant documentation was updated

Thank you for your code @Berreek ! I used it in my fork and it works perfectly.

@skolima Any update for it?

Could you update from master so that this branch builds, please? I'm also not sold on the naming - it's specific to Azure, while adding a general option.  Azure calls this "set auto-complete" flag https://docs.microsoft.com/en-us/azure/devops/repos/git/pull-requests?view=azure-devops while GitHub (and Dependabot) call this auto-merging ( https://github.com/dependabot/feedback/issues/954 ) and that's naming I'm more familiar with.
What would you say about naming the option setautomergeflag? (docs updated accordingly once we settle on a name).
@AnthonySteele could you perhaps comment?

I updated the branch.
Regarding the name - I called it auto-complete beacuse I looked only on Azure to be honest. I agree with you that auto-merge will get better understanding since complete is Azure specific as you said.
Let me know if we stick with it.

@skolima I renamed the option from setautocomplete to setautomerge - is there something else that need to be done in order to merge that PR?

@Berreek @skolima do you have any ETA for this? It will be very helpful

Truth be told, I don't think any of the original developers uses NuKeeper any more. But I'll try to find time to release this.


Modal with passed trigger element, buttons can't close modal- usage question/feature request

Steps
Pass the Modal component a trigger element instead of using it as a controlled component.
Expected/Desired Result
Clicking the trigger element will open the Modal, and you have the option of passing a button in the <Modal.Action /> section a prop that can trigger closing the Modal. Maybe have a <Modal.Button /> or another prop that can trigger a close?
Actual Result
You can no longer use buttons in <Modal.Action /> or any other child elements to close the modal. You can only click outside the Modal to close it and/or use a closeIcon.
Version
latest
Testcase
Basic Modal example in the docs (the second example)

This is the intended usage.  In React, there are no implicitly parent-child relationships.  So, the buttons cannot automatically close the Modal unless they are explicitly wired to do so (e.g. onClick -> set state).
The key is here:

instead of using it as a controlled component

In order to do this, the Modal must be controlled.
In the future, we plan to support defining Modals with shorthand props.  This would allow you do accomplish what you are asking for, but, you wouldn't do it with children, you would do it with props that we provide.  This is possible because we can take the prop values and internally instantiate all the children and, while doing so, add the appropriate event listeners and state mutations.
It might look something like this (note, this is not a planned API, just an example):
<Modal
  trigger={<Button>Basic Modal</Button>}
  basic
  size='small'
  header={{ icon: 'archive', content: 'Archive Old Messages' }}
  content='Your inbox is getting full, would you like us to enable automatic archiving of old messages?'
  actions={[
    { basic: true, color: 'red', inverted: true, icon: 'remove', content: 'No', },
    { color: 'green', inverted: true, icon: 'checkmark', content: 'Yes' },
  ]}
/>

As opposed to the current:
<Modal trigger={<Button>Basic Modal</Button>} basic size='small'>

  <Modal.Content>

Your inbox is getting full, would you like us to enable automatic archiving of old messages?

  </Modal.Content>
  <Modal.Actions>
    <Button basic color='red' inverted>
      <Icon name='remove' /> No
    </Button>
    <Button color='green' inverted>
      <Icon name='checkmark' /> Yes
    </Button>
  </Modal.Actions>
</Modal>

PRs welcome!

Thank you for the quick response and explanation.

I've been playing around and trying to implement something like what you described above. I don't think the way i've written this is ideal, but I think its a start? I would love to work on implementing this feature. Here are the changes I made.
const modalJSX = (
      <ElementType {...rest} className={classes} style={{ marginTop }} ref={this.handleRef}>
        {Icon.create(closeIconName, { onClick: this.handleClose })}
        {children}
        {_.isNil(children) && header && ModalHeader.create(header)}
        {_.isNil(children) && content && ModalContent.create(content)}
        {_.isNil(children) && actions && ModalActions.create(actions.map((action, i) => {
          if (action.triggerClose) {
            const onClick = (callback) => (e) => {
              if (callback) callback(e, this.props)
              this.handleClose()
            }
            action.onClick = onClick(action.onClick)
          }
          return Button.create(action, { key: i })
        }))}
      </ElementType>
    )

The above would implement the below modal with shorthand props. A triggerClose can be passed to the shorthand action items that would allow clicking the buttons to close the modal (I would love to not have to use a open boolean in state all the time).
<Modal
  trigger={<Button>Basic Modal</Button>}
  basic
  size='small'
  header={{ icon: 'archive', content: 'Archive Old Messages' }}
  content='Your inbox is getting full, would you like us to enable automatic archiving of old messages?'
  actions={[
    { basic: true, color: 'red', inverted: true, icon: 'remove', content: 'No', triggerClose: true },
    { color: 'green', inverted: true, icon: 'checkmark', content: 'Yes', triggerClose: true },
  ]}
/>

If you'd like to open a PR for this it'd be great.  We can help out there with more visibility and comments.


Error when trying CME2.py

Hello,
Love your work - trying to use it to learn from!
When I try to run CME2.py, I get the error: "string indices must be integers" & it points to the line output['prior settle']=[i['priorSettle'] for i in df['quotes']]
I am a novice when it comes to webscraping, but hoping you may be able to shed some light on the issue. Thanks so much!

I'm sorry but I didn't get the error when I ran it just now.

 Make sure you are using the latest version.

Have you modified the script? In that case, you have to print out variable i,df,output to figure out where the erreur comes from. Can't help you from my end.


Mannheim

When I test this all parking lots are shown with 0 free lots and nodata although there is a free datum on the website.

my fault.
fixed.

Fixed by https://github.com/offenesdresden/ParkAPI/commit/01439a11a901f50aefe412f9f04b2e2cbab143a1.


Initial pass at webhook-subscription-2022

Here's the initial pass for webhook-subscription-2022. Primarily, this updates webhook-auth to correspond to the Solid-OIDC spec, instead of its own spec.
Note: I noticed that Solid-OIDC is very different than it once was. I'll need to ask about that next meeting.

Should this be merged in?


Where i can found C# code examples?

Hi All,
Where i can found C# code examples?

There aren't any at present, but you can easily use an online converter to create C# from the VB.


Update make.go - Changing the standards version

Updating the standards version to follow the latest as per the latest standards version of Debian policy.
4.1.3

the PR fixes it.


Batch integration

This PR adds the batch_integration_graph task including datasets, methods and metrics as well as a docker container (openproblems-python-batch-integration).
Submission type

[ ] This submission adds a new dataset
[x] This submission adds a new method
[x] This submission adds a new metric
[x] This submission adds a new task
[x] This submission adds a new Docker image
[ ] This submission fixes a bug (link to related issue: )
[ ] This submission adds a new feature not listed above

Testing

[x] This submission was written on a forked copy of SingleCellOpenProblems
[x] GitHub Actions "Run Benchmark" tests are passing on this base branch of this pull request (include link to passed test: )
[ ] If this pull request is not ready for review (including passing the "Run Benchmark" tests), I will open this PR as a draft (click on the down arrow next to the "Create Pull Request" button)

Submission guidelines

[x] This submission follows the guidelines in our Contributing document
[x] I have checked to ensure there aren't other open Pull Requests for the same update/change

Codecov Report

Merging #241 (ed94d7c) into main (a20bbc6) will decrease coverage by 13.04%.
The diff coverage is 50.00%.

@@             Coverage Diff             @@
##             main     #241       +/-   ##
===========================================
- Coverage   89.30%   76.25%   -13.05%
===========================================
  Files          73      100       +27
  Lines        1533     2232      +699
  Branches       90       94        +4
===========================================
+ Hits         1369     1702      +333
- Misses        135      498      +363
- Partials       29       32        +3

Flag
Coverage Δ

unittests
76.25% <50.00%> (-13.05%)
:arrow_down:

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

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

@LuckyMD do you think we could already merge that? Doesn't contain all methods but this would make further testing easier

I think it's fine to have a subset of the task here (especially to make GA tests faster). Minimum requirements are 1 dataset, 3 methods, 1 metric, and all tests passing (and tests actually being run ^^). I guess we have to wait for the CI to finish to assess whether this is the case.

Just talked to @scottgigante about some design decisions that will affect the Batch integration PR specifically. We'll need to adapt the following:

[ ] add layer log_scran to dataset loader for immune and pancreas data (instead of overwriting and re-normalizing)
[ ] Re-add valenzi batch
[ ] Remove log_scran() in task dataset function

https://github.com/danielStrobl/SingleCellOpenProblems/actions/runs/2338972919

Looks great! If tests pass here, then this is mergable :)

I'm not seeing the codecov report... Just want to confirm all methods are imported before we merge.

Thanks for your comments! I think those should be addressed by my last commit


Unable to upload symbols

Describe the bug
I'm not able to upload symbol file using nuget
C:\Program Files (x86)\Jenkins\workspace\libLogger>c:\nuget\nuget.exe push "C:\Program Files (x86)\Jenkins\workspace\libLogger\bin\Debug\Logger.*.snupkg"<PHONE_NUMBER> -Source http://<IP_ADDRESS>:5000/v3/index.json
Pushing Logger.2.0.0.snupkg to 'http://<IP_ADDRESS>:5000/api/v2/symbol'...
PUT http://<IP_ADDRESS>:5000/api/v2/symbol/
BadRequest http://<IP_ADDRESS>:5000/api/v2/symbol/ 140ms
Response status code does not indicate success: 400 (Bad Request)

Thanks for filing this bug, I’ll try to reproduce this! What version of NuGet are you using? Is there anything special about your project’s snupkg? Does BaGet’s logs indicate anything out of the ordinary, like an exception?

I am seeing the same error with symbol uploads using version 0.1.70 and 0.1.71 however when running BaGet built from master within VS Code to debug I do not get an error. git diff tells me there is no difference between the tag for 0.1.71 and the current master. I am not sure where to go with that.
.NET Core SDK (reflecting any global.json):
 Version:   2.2.105
 Commit:    7cecb35b92

Runtime Environment:
 OS Name:     centos
 OS Version:  7
 OS Platform: Linux
 RID:         centos.7-x64
 Base Path:   /usr/share/dotnet/sdk/2.2.105/

Host (useful for support):
  Version: 2.2.3
  Commit:  6b8ad509b6

.NET Core SDKs installed:
  2.2.105 [/usr/share/dotnet/sdk]

.NET Core runtimes installed:
  Microsoft.AspNetCore.All 2.2.3 [/usr/share/dotnet/shared/Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.2.3 [/usr/share/dotnet/shared/Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 2.2.3 [/usr/share/dotnet/shared/Microsoft.NETCore.App]

v0.1.71
 ~/NuGetPackages/Deploy $ dotnet nuget push -s http://localhost:5000/v3/index.json MTK-Logger-devel.<IP_ADDRESS>16.nupkg
warn : No API Key was provided and no API Key could be found for 'http://localhost:5000/api/v2/package'. To save an API Key for a source use the 'setApiKey' command.
info : Pushing MTK-Logger-devel.<IP_ADDRESS>16.nupkg to 'http://localhost:5000/api/v2/package'...
info :   PUT http://localhost:5000/api/v2/package/
info :   Created http://localhost:5000/api/v2/package/ 1008ms
info : Your package was pushed.
warn : Found symbols package 'MTK-Logger-devel.<IP_ADDRESS>16.snupkg', but no API key was specified for the symbol server. To save an API Key, run 'NuGet.exe setApiKey [your API key from http://www.NuGet.org]'.
warn : No API Key was provided and no API Key could be found for 'http://localhost:5000/api/v2/symbol'. To save an API Key for a source use the 'setApiKey' command.
info : Pushing MTK-Logger-devel.<IP_ADDRESS>16.snupkg to 'http://localhost:5000/api/v2/symbol'...
info :   PUT http://localhost:5000/api/v2/symbol/
info :   BadRequest http://localhost:5000/api/v2/symbol/ 57ms
error: Response status code does not indicate success: 400 (Bad Request).

Debug build
 ~/NuGetPackages/Deploy $ dotnet nuget push -s http://localhost:50561/v3/index.json MTK-Logger-devel.<IP_ADDRESS>16.nupkg
warn : No API Key was provided and no API Key could be found for 'http://localhost:50561/api/v2/package'. To save an API Key for a source use the 'setApiKey' command.
info : Pushing MTK-Logger-devel.<IP_ADDRESS>16.nupkg to 'http://localhost:50561/api/v2/package'...
info :   PUT http://localhost:50561/api/v2/package/
info :   Created http://localhost:50561/api/v2/package/ 1164ms
info : Your package was pushed.
warn : Found symbols package 'MTK-Logger-devel.<IP_ADDRESS>16.snupkg', but no API key was specified for the symbol server. To save an API Key, run 'NuGet.exe setApiKey [your API key from http://www.NuGet.org]'.
warn : No API Key was provided and no API Key could be found for 'http://localhost:50561/api/v2/symbol'. To save an API Key for a source use the 'setApiKey' command.
info : Pushing MTK-Logger-devel.<IP_ADDRESS>16.snupkg to 'http://localhost:50561/api/v2/symbol'...
info :   PUT http://localhost:50561/api/v2/symbol/
info :   Created http://localhost:50561/api/v2/symbol/ 319ms
info : Your package was pushed.

Thanks for the report @fuzzybair. I installed BaGet v0.1.71 and .NET Core 2.2.105 and I can't reproduce this issue. Some more questions:

What storage type are you using?
What database type are you using?
Is there anything special about your snupkg? Would you mind attaching a nuget package and its snupkg here that reproduces this issue?
How are you running BaGet? Is it dotnet BaGet.dll or through the docker image?
If you run BaGet on another operating system, does this issue still reproduce?

FileSystem
Sqlite
NuGet and symbol package built on a Jenkins build server(windows). using "nuget.exe pack Logger.csproj -Properties "Platform=x64;tag=-devel" -Symbols -SymbolPackageFormat snupkg" contents of the snupkg is just a pdb file set to Portable

I can not release our code but I will work on getting a test package that reproduces the error.

I am running "dotnet BaGet.dll" via systemd.
BaGet.service

[Unit]
Description=A lightweight NuGet and Symbol server.

[Service]
ExecStart=/usr/bin/dotnet BaGet.dll
WorkingDirectory=/opt/BaGet/server
EnvironmentFile=/opt/BaGet/conf/BaGet.env
PassEnvironment=ASPNETCORE_URLS
Restart=always
RestartSec=120
SyslogIdentifier=BaGet

[Install]
WantedBy=multi-user.target

BaGet.env
ASPNETCORE_URLS=http://*:5000

appsettings.json
{
  "ApiKey": "",
  "PackageDeletionBehavior": "Unlist",
  "AllowPackageOverwrites": false,

  "Database": {
    "Type": "Sqlite",
    "ConnectionString": "Data Source=/opt/BaGet/data/baget.db"
  },

  "Storage": {
    "Type": "FileSystem",
    "Path": "/opt/BaGet/data"
  },

  "Search": {
    "Type": "Database"
  },

  "Mirror": {
    "Enabled": false,
    "PackageSource": "https://api.nuget.org/v3/index.json"
  },

  "Logging": {
    "IncludeScopes": false,
    "Debug": {
      "LogLevel": {
        "Default": "Warning"
      }
    },
    "Console": {
      "LogLevel": {
        "Default": "Warning"
      }
    }
  }
}

tessting on windows 10 with dontnet 2.2.103 and version 0.1.71(default configuration) using the same nupkg/snupkg does not seen to have the same issue.

I am not opposed to switching to the Docker based approach. However I have not had much luck in the past getting docker containers running as Linux that will come up after system reboot. If you could provide a link or some documentation on how to get BaGet running under systemd with a docker, I am willing to try it.

@fuzzybair https://docs.docker.com/engine/reference/run/#restart-policies---restart

@7thcubic Thanks for the link, with it I was able to get the docker instance working and restarting after reboot.  As a note after setting the docker services to restart on boot, two changes to the docker run command were needed for it to work

Replace --rm with  --restart=unless-stopped
Add a :Z to the end of the volume mount (for SeLinux context)

docker run --restart=unless-stopped --name nuget-server -p 5555:80 --env-file /opt/BaGet/baget.env -v "/opt/BaGet:/var/baget:Z" loicsharma/baget:latest

with the docker version I am able to upload symbols. I am not sure why running under systemd directly did not work.

I was able to upload but Visual Studio did not find the symbols I believe the cause is Issue #224

Hey @fuzzybair, I wasn't able to reproduce the issue in https://github.com/loic-sharma/BaGet/issues/224. I created a project, targeted the x64 platform, packed it using nuget.exe, and loaded the symbols using Visual Studio 2017. This did not reproduce the issue. Could you provide a sample project that when packed reproduces this issue?

@MatijaSestak Could you provide a sample project that when packed reproduces this issue?

Sure, https://github.com/MatijaSestak/AppLogger

I am having the same issue as mentioned above. Here are my development environment

Visual Studio 2017

.Net Framework Project (4.6.2)

NuGet.exe (v5.2.0)

I am not sure, but this issue could be because BaGet uses Nuget v2 api for symbols for snupkg packages. But, as stated here it is only supported in NuGet api v3 for Symbols

If the symbol package isn't published, check that you've configured the NuGet.org source as https://api.nuget.org/v3/index.json. Symbol package publishing is only supported by the NuGet V3 API.

Sure, https://github.com/MatijaSestak/AppLogger

@MatijaSestak This line  looks suspicious, you should use <DebugType>portable</DebugType> in both debug and release builds (it looks like your release builds should work though, see this line)
@bhattvishal @SiOuz Please make sure you don't have <DebugType>full</DebugType> or <DebugType>pdbonly</DebugType> anywhere in your .csproj files. You should use <DebugType>portable</DebugType> instead as BaGet does not support Windows PDBs yet. See: https://github.com/loic-sharma/BaGet/issues/153

I am not sure, but this issue could be because BaGet uses Nuget v2 api for symbols for snupkg packages. But, as stated here it is only supported in NuGet api v3 for Symbols

TLDR: BaGet is doing the right thing here. More detailed explanation: the V3 API uses the V2 API for package uploads. If you go to nuget.org's V3 package source, you'll find the following JSON:
{
  "version": "3.0.0",
  "resources": [
    ...

    {
      "@id": "https://www.nuget.org/api/v2/package",
      "@type": "PackagePublish/2.0.0"
    },

    ...
  ]
}

That being said, you MUST use a V3 package source URL (like https://api.nuget.org/v3/index.json) instead of a V2 package source URL (like https://nuget.org/) to publish symbol packages. The NuGet protocol can be awfully confusing 😅

That being said, you MUST use a V3 package source URL (like https://api.nuget.org/v3/index.json) instead of a V2 package source URL (like https://nuget.org/) to publish symbol packages. The NuGet protocol can be awfully confusing 😅

What should I put into -Source parameter if I want to push a Symbol Package?
I build BaGet from the source code and deploy to IIS (which is running on my company server).
The BaGet site is now serving on a sub-domain: nuget.mycompany.com.
I have been pushing the .nupkg file using the following command: nuget push -Source http://nuget.mycompany.com/v3/index.json MyPackage.1.0.0.nupkg -ApiKey api-key-set-in-appsettings.json-file and it worked without problems.

Folks,
reading this thread, it seems I have a similar issue;
I push several package (with .snuget) to my BaGet server without problem. Only one of them fails with:
PUT http://localhost:5000/api/v2/symbol/
  BadRequest http://localhost:5000/api/v2/symbol/ 2038 ms
error: Response status code does not indicate success: 400 (Bad Request).

It turns out that this one package contains a C++/CLI project and the pdb from it. I can fix it easily by removing the offending .pdb from the .snuget before uploading.
So the problem must be that the one .pdb is not in portable format!
The server outputs an exception which also points to this:
Unable to index symbol package due to exception
System.BadImageFormatException: Invalid COR20 header signature.

Is it actually the same what the issue starter encountered? Then my information can help.
And, of course, I would be really glad about a suggestion on how to fix this. (Skip the non-portable .pdb or make it acceptable)


Custom search param for active groups

IMPORTANT: Where possible all PRs must be linked to a Github issue
Closes #2233
Engineer Checklist

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

Code Reviewer Checklist

[ ] I have verified Unit tests have been written for any new feature(s) and edge cases
[ ] I have verified any strings visible on UI components are in the strings.xml file
[ ] I have verifed the CHANGELOG.md file has any notable changes to the codebase
[ ] I have verified the solution has been implemented in a configurable and generic way for reuseable components
[ ] I have built and run the FHIRCore app to verify the change fixes the issue and/or does not break the app

this also looks like it needs explicit testing for the resource type group being filtered and not, interesting that the new lines are showing that they're covered, what's missing is at least some test data with active and inactive groups and asserts verify only the active groups remain after filtering

Yea I think we can open an issue for that, because I feel like to make that worthwhile we’d need to load some functionality from the URL in addition to letting a client configure it.

On Apr 12, 2023, at 3:37 AM, Benjamin Mwalimu @.***> wrote:
@dubdabasoduba commented on this pull request.
In android/engine/src/main/java/org/smartregister/fhircore/engine/configuration/app/ConfigService.kt https://github.com/opensrp/fhircore/pull/2235#discussion_r1163742256:

@@ -66,4 +68,24 @@ interface ConfigService {
}

fun provideConfigurationSyncPageSize(): String
+

/** Provide a list of custom search parameters */
fun provideCustomSearchParameters(): List {
val activeGroupSearchParameter =

 SearchParameter().apply {

   url = "http://smartregister.org/SearchParameter/group-active"

@pld https://github.com/pld Yes, that's true. I guess my concern is making it hard-coded. It means a client can't change it easily if they have their own spec definitions.
We can clean it up later though.
—
Reply to this email directly, view it on GitHub https://github.com/opensrp/fhircore/pull/2235#discussion_r1163742256, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAMMEVYGSSRDU4S7LOJNE3XAZLSDANCNFSM6AAAAAAWU6STLE.
You are receiving this because you were mentioned.


Double-backticks can crash the editor

Describe the bug
This is a duplicate report of https://github.com/microsoft/vscode/issues/90369, where a user added a new identifier with double-backticks and the editor crashed.  I have seen it as well on personal projects intermittently, as has Mathias Brandewinder.

Duplicate of https://github.com/ionide/ionide-vscode-fsharp/issues/1073


⚠ Bofrost.lu has degraded performance

In d7e6da3, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1314 ms

Resolved: Bofrost.lu performance has improved in 5b2757d after 51 minutes.


MapBox Studio Classic 0.3.1 - kartotherian/osm-bright.tm2

When i try to open the style kartotherian/osm-bright.tm2 the error message appears: "Failed to load fonts from: node_modules/osm-bright-fonts/fonts/ in Map"

Have you followed the instructions for installing the fonts, like outlined in the ReadME?
https://github.com/kartotherian/osm-bright.tm2#viewingediting-this-style
Since you are using the @kartotherian version of osm-bright, it might be a good idea to post your question over there.


Notes from discussion from real-time synchronization of widgets

This is from a meeting between @rgbkrk, @ivanov, @jasongrout, @SylvainCorlay, and Ian Rose on Friday, Jan 6, discussing widgets and real-time collaboration.

Widgets shouldn't directly talk with the kernel. Widgets should interact with a model store, and the model store should carry out the syncing. Right now, the base widget model manages the state quite closely - instead this should be abstracted out to a state store. The state store can then implement syncing to talk to a kernel directly, or to sync with a server state store, or propagate changes over some real-time sync framework like Google's.

(We also opened #1032 and #1033 from related discussions.)

Nice!

Closing since the discussion is now over :).

Thanks for typing up the notes!

Re-opening, since obviously this isn't implemented. Since it's only one item, I'm just changing the title.


[CMAKE] Add TritonIR dependency on TritonGPU TypeInterfaces Tablegen

https://github.com/triton-lang/triton/commit/133109de1248ddf934fd46409dece99d78ef59d5 added a TypeInterfaces tablegen include (triton/Dialect/TritonGPU/IR/TypeInterfaces.h.inc) to include/triton/Dialect/TritonGPU/IR/Types.h. This file is included by https://github.com/triton-lang/triton/blob/main/lib/Dialect/Triton/IR/Traits.cpp#L6 via https://github.com/triton-lang/triton/blob/main/include/triton/Dialect/TritonGPU/IR/Dialect.h which only explicitly depends on the Triton Dialect Tablegen. This results in the occasional build failure:
2024-12-02T13:21:12.3924074Z   FAILED: lib/Dialect/Triton/IR/CMakeFiles/TritonIR.dir/Traits.cpp.o
2024-12-02T13:21:12.4016048Z   /usr/bin/c++ -DGTEST_HAS_RTTI=0 -I/runner/_work/intel-xpu-backend-for-triton/intel-xpu-backend-for-triton/python/build/cmake.linux-x86_64-cpython-3.9/lib/Dialect/Triton/IR -I/runner/_work/intel-xpu-backend-for-triton/intel-xpu-backend-for-triton/lib/Dialect/Triton/IR -I/runner/_work/intel-xpu-backend-for-triton/intel-xpu-backend-for-triton/include -I/runner/_work/intel-xpu-backend-for-triton/intel-xpu-backend-for-triton/. -I/home/runner/.triton/llvm/llvm-86b69c31-ubuntu-x64/include -I/runner/_work/intel-xpu-backend-for-triton/intel-xpu-backend-for-triton/python/build/cmake.linux-x86_64-cpython-3.9/include -I/runner/_work/intel-xpu-backend-for-triton/intel-xpu-backend-for-triton/third_party -I/runner/_work/intel-xpu-backend-for-triton/intel-xpu-backend-for-triton/python/build/cmake.linux-x86_64-cpython-3.9/third_party -D__STDC_FORMAT_MACROS  -fPIC -std=gnu++17 -Werror -Wno-covered-switch-default -fvisibility=hidden -g  -fno-exceptions -funwind-tables -fno-rtti -MD -MT lib/Dialect/Triton/IR/CMakeFiles/TritonIR.dir/Traits.cpp.o -MF lib/Dialect/Triton/IR/CMakeFiles/TritonIR.dir/Traits.cpp.o.d -o lib/Dialect/Triton/IR/CMakeFiles/TritonIR.dir/Traits.cpp.o -c /runner/_work/intel-xpu-backend-for-triton/intel-xpu-backend-for-triton/lib/Dialect/Triton/IR/Traits.cpp
2024-12-02T13:21:12.4044994Z   In file included from /runner/_work/intel-xpu-backend-for-triton/intel-xpu-backend-for-triton/include/triton/Dialect/TritonGPU/IR/Dialect.h:12,
2024-12-02T13:21:12.4047664Z                    from /runner/_work/intel-xpu-backend-for-triton/intel-xpu-backend-for-triton/lib/Dialect/Triton/IR/Traits.cpp:8:
2024-12-02T13:21:12.4050715Z   /runner/_work/intel-xpu-backend-for-triton/intel-xpu-backend-for-triton/include/triton/Dialect/TritonGPU/IR/Types.h:11:10: fatal error: triton/Dialect/TritonGPU/IR/TypeInterfaces.h.inc: No such file or directory
2024-12-02T13:21:12.4054755Z      11 | #include "triton/Dialect/TritonGPU/IR/TypeInterfaces.h.inc"
2024-12-02T13:21:12.4055787Z         |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
2024-12-02T13:21:12.4056738Z   compilation terminated.

The cmake file you referred to is the Triton cmake file but not the TritonGPU cmake file.
It seems to be that Traits.cpp shouldn't include any TritonGPU related files. Maybe we should clean up verifyTensorLayouts

Yes I agree, the root cause is a dependency on the Triton file on Triton GPU code. I'm happy to take a look at that, too - my first thought would be to split the explicit ttg stuff to a free function and then handle that dependency during linking, but I am not sure what the best place for the ttg free function would be - one of the ttg utility files? Do you have another suggestion?

Maybe we can implement LogicalResult MyDialect::verifyOperationAttribute(Operation *op, NamedAttribute attribute);?
cc @Mogball and @ThomasRaoux , any suggestions?

After converting to TritonGPU IR, the Triton IR ops are still considered part of the Triton dialect, right? If so, would we have the same problem with the verifyOperationAttribute that we do now since we would want to verify the layout attribute on the Triton ops and would need to reference the appropriate ttg objects?
Assuming the analysis above is correct, I think creating a free function that is essentially the body of checkLayout and placing that in something like lib/Dialect/TritonGPU/IR/LayoutVerifier.cpp would resolve the problem, b/c there should be no problem finding the definition at link time and no ttg attributes would leak through the function signature, just the mlir::Value and probably the ModuleOp.

Your suggestion would fix the build issue, but it wouldn't fix the layering violation. Triton should be logically independent of TritonGPU, otherwise is might lead to problems with non-GPU backends.
IMO we should have an interface on the encoding to verify it's valid. The verification code is already specializing on specific encoding types and this would allow third party backends to add verification for their layout types.

Maybe we can implement LogicalResult MyDialect::verifyOperationAttribute(Operation *op, NamedAttribute attribute);?
cc @Mogball and @ThomasRaoux , any suggestions?

Yes I think we could make an interface like we do for DialectInferLayoutInterface. That should allow us to decouple the layout verification from TritonIR which is a clear layering violation

Closing in favor of #5312


add Big heading landing in home page

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the main branch of freeCodeCamp.
[ ] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #XXXXX

Hi, this is my first time contributing to open source if I have missed a step or I'm doing something wrong. please let me know, I want to learn.

I'll leave some more specific comments directly on the files to help clarify my concerns. 🙂

so I should remove the package-lock.json?
On Wed, Feb 24, 2021 at 3:38 PM nhcarrigan<EMAIL_ADDRESS>wrote:

@nhcarrigan requested changes on this pull request.
In client/i18n/locales/english/translations.json
https://github.com/freeCodeCamp/freeCodeCamp/pull/41251#discussion_r582383086
:

"big-heading-2": "Build projects.",

"big-heading-3": "Earn certifications.",

"big-heading-2": "Learn to solve problems.",
"big-heading-3": "Build projects.",
"big-heading-4": "Earn certifications.",

This change here is adding a new key to be translated through the
reacti18next functions...
In client/src/components/landing/components/LandingTop.js
https://github.com/freeCodeCamp/freeCodeCamp/pull/41251#discussion_r582383467
:

@@ -30,6 +30,7 @@ function LandingTop({ page }) {

{t('landing.big-heading-2')}
{t('landing.big-heading-3')}

{t('landing.big-heading-4')}

And the key is used here, but is not replacing any existing un-translated
text. So I am confused about the addition of text to the client. Why is
this being added?
In package-lock.json
https://github.com/freeCodeCamp/freeCodeCamp/pull/41251#discussion_r582383918
:

@@ -1,8 +1,19712 @@
{
"name": "@freecodecamp/freecodecamp",
"version": "0.0.1",

"lockfileVersion": 1,

"lockfileVersion": 2,

lockfile Version 2 is used in npm versions 7 and higher - our codebase
officially supports npm major version 6. The changes to this file should
be removed from your commit.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/pull/41251#pullrequestreview-598032058,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMKTZINYCCTF6W5UH4GZFNTTAWEXZANCNFSM4YFN6PAQ
.

You'll want to revert the changes made to the package-lock.json, yes. Note that you don't want to delete the file entirely. git offers a few ways to do this - personally, I'd reset against our main branch and make the other two changes again.
If needed, I can revert it for you.

yes. go ahead and do it for me.
I'm sorry
On Wed, Feb 24, 2021 at 3:52 PM nhcarrigan<EMAIL_ADDRESS>wrote:

You'll want to revert the changes made to the package-lock.json, yes.
Note that you don't want to delete the file entirely. git offers a few
ways to do this - personally, I'd reset against our main branch and make
the other two changes again.
If needed, I can revert it for you.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/pull/41251#issuecomment-785470554,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMKTZIKIDUGEYJT26V4ZOATTAWGKRANCNFSM4YFN6PAQ
.

it would be appreciated if you could send me some resources contributing to
open source.
On Wed, Feb 24, 2021 at 3:55 PM Manuel Velasco<EMAIL_ADDRESS>wrote:

yes. go ahead and do it for me.
I'm sorry
On Wed, Feb 24, 2021 at 3:52 PM nhcarrigan<EMAIL_ADDRESS>wrote:

You'll want to revert the changes made to the package-lock.json, yes.
Note that you don't want to delete the file entirely. git offers a few
ways to do this - personally, I'd reset against our main branch and make
the other two changes again.
If needed, I can revert it for you.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/pull/41251#issuecomment-785470554,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMKTZIKIDUGEYJT26V4ZOATTAWGKRANCNFSM4YFN6PAQ
.

A great place to start is our contributing documentation.

Thank you for opening this pull request.
This is a standard message notifying you that we've reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Reason: The content of our landing page directly impacts our SEO - as such, we generally do not make changes without input from Quincy.
Thank you and happy coding.

you want me to close it?
delete the branch?
On Wed, Feb 24, 2021 at 4:54 PM nhcarrigan<EMAIL_ADDRESS>wrote:

Closed #41251 https://github.com/freeCodeCamp/freeCodeCamp/pull/41251.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/pull/41251#event-4373939449,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMKTZIPQU2UU2VFCB65OP7LTAWNTHANCNFSM4YFN6PAQ
.


Develop renko chart

Add renko chart feature,

This PR did not include any commits not present in the develop branch.


Bump fast-json-stringify in "unescaped" branch.

Per https://github.com/davidmarkclements/quick-format/pull/16#issuecomment-386836403

thanks for the PR's @wtgtybhertgeghgtwtg - we just need to hold off here,
we need to check that this doesn't break inter-relationships between dependents (in particular, pino and pino-noir)

The idea was to bump to the version the RC version of pino was using.  I'm not sure how it'll work with the current version, though.


need to connect event listeners

At the moment you can't use event listeners and values. It would be nice to have one of the following:
1.) every check/uncheck changes the original SELECT-element value and triggers 'change' and 'input' events at this one
OR
2.) provide your own event functions, e. g. 'onafterwritinglive' or similar, with checked values as arguments
it's a nice work, thanks a lot

Using testing branch, you can try to initialize multiselect in this way:
    $("#menu").zmultiselect({

        filter: true,
        filterResult: true,
        selectAll: true,
        get: "zmultiselect",
        placeholder: "I'm a MultiSelect",
        live: '#menu_live',

        onLoad: function(id){
            console.log("id " + id);
        },

        onChange: function(id, live){
            console.log('change trigger', id, live);
        },

    });

I need to test it out a little more.

@rteinze Between version 5 and 6, I found I needed to use the updateNativeSelect: true option.  Documentation doesn't make reference to it, so I did some source diving to find it.


feat: add mjs support

Allow users to use these in mjs
import { safeLoad } from 'js-yaml';

Fixed by https://github.com/nodeca/js-yaml/commit/1b5fea8526fea734041334db3687ae5a919a0dfc


Removed legacy network annotations

This cleans up our pods a little bit, removing "legacy" old style
annotations like EniId in favor of the new style that comes from
kube-common.
This can be merged and shipped after
https://github.com/Netflix/titus-control-plane/pull/1257
is fully deployed.

I think we've made it, and I don't believe these legacy annotations are in use by the CP anymore.


jQuery 3 and Semantic UI Rating

jQuery 3 seems to break Semantic UI rating
https://github.com/Semantic-Org/Semantic-UI/issues/4561
We are seeing this on CI builds under the ember-release scenario because Ember 2.8 allows jQuery <4. It currently pulls in 3.1.1.
In Ember 2.7 and below it is <3.

I'm not sure if this is the best way to go about fixing this issue, but specifying jQuery <3 in the ember try scenario makes the build pass.
Perhaps a note saying that rating only works with jQuery <3 for the time being will allow things to progress.

Yeah that works, I'm fine specifying jquery < 3 for older ember builds. If you want to put a pull request out we can get it merged in. Thanks!


Add travis CI

Can you also change the badge in the readme (and make sure it reflects the build status)?


Inventory UI

Spells appear in an inventory
They can sometimes swap
They sometimes show
🤷

sometimes???? 😭

The UI is more than a bit dodgy. I'll put a full to-do list that I've noticed on trello


Introduce Columns Amount and Duplicates to RuleTable

Columns Amount and Duplicates were added to RuleTable
Added functions to calculate values for those columns
Set cursor style to default in Filter column to emphasize that this input  is not editable

shows - if no data is given

Are duplicates correctly calculated? Looks a bit odd... but I might be wrong. The column titles could be enhanced e.g. Sigma for sum and some other sign for dupes? with tooltips giving the full explanation?

@marcjansen

Are duplicates correctly calculated? Looks a bit odd... but I might be wrong.

Yes, they are. The example in the screenshot might confuse a little.

The column titles could be enhanced e.g. Sigma for sum and some other sign for dupes? with tooltips giving the full explanation?

Done in https://github.com/terrestris/geostyler/pull/639/commits/112934b78e89d7a11b2dbbab61bd40799feb8bbe

Please don't use "copy" as it suggests that you can copy something here.

Done in https://github.com/terrestris/geostyler/pull/639/commits/e467152f125709cfb52e7e9688bd9b8941f6af50


Where is the option for multiple selection of Image?

Hello, this app is too good, but can i select multiple image? need help regarding this matter.

https://github.com/Yummypets/YPImagePicker/issues/54


CallData: Do not deserialize Result too early

Currently, CallData.Result is of type object? and is thus passed through PerperBinarySerializer.Deserialize immediately on reading the call data in Agent.CallAsync. However, this is wasteful and likely incorrect; consider making a type of objects which should be deserialized later and using that instead of object? in CallData.
This might actually also apply to *Data.Parameters.

Note: This actually causes an issue with StreamData.Parameters since 0360412#Stream.cs:181, as now subscribing to a stream is only possible if the subscriber is able to deserialize the parameters of the stream. (Workaroundable with blank streams)

The CallData.Result issue was fixed in 0.7, but reintroduced with 98017049e8a2e32a5a5c666b42372c1a42c0cdf3.
StreamData.Parameters has been fixed in a92d71fc7c0cec42c49a9f85959a20f7edba7260.
However, as we no longer have PerperBinarySerializer, completely resolving this issue would require using IBinaryObject for CallData, which has the potential to cause more problems down the road due to incomplete deserialization of nested object or object[] fields, in addition to decreasing code legibility.
Hence, closing as a wontfix at the current stage--the impact of the spare CallData.Result deserialization is likely less than the impact of fixing it.


Tiles don't always fade-in with the new tile loading logic

After merging the new tile loading logic #2382, background panes that become the main panes don't fade in, so there's an impression of tiles not always fading in.

The issue is that with throttled pruning and deferred removal, tiles can remain in the DOM and get reused during quick zoom-in-zoom-out operations. When that happens, the CSS transition for loaded tiles is not trigged.
Even if pruning wasn't throttled, we don't want to remove the leaflet-tile-loaded class from background tiles immediately, because they need to be cross-faded with loading tiles from the current layer. The other thing that I tried was to remove and re-add the leaflet-tile-loaded class from existing tiles in the current layer at the end of _updateLevels, but it doesn't look like this triggers CSS transitions. You probably have to remove the class in one frame and then re-add it in another for the transition to animate, and that would add even more asynchronous complexity to tile loading, which I'd like to avoid.
My preference would be to switch to JS-based transitions, which would also allow us to fix the _deferRemove hack. But I know that you are resistant to that. So what do you suggest?

Several years ago, I always assumed native transitions would be faster than animating manually frame by frame. For some cases, that is a valid assumption, but for others, it has proved me wrong. For example, I recently ditched transition-based panning animation.
So if we try it out and measure, it may turn out that it works pretty good, especially considering that we can calculate each tile in the view in one running context, without having to keep lots of independent timers for each tile and try to handle hard to track race conditions. Lets make a simple naive implementation, just to evaluate performance, and if it works, implement it properly.

The PR above fixed a lot of issues with tile loading behavior and flickering and made it awesome, but now that I clearly see a big difference between transitions & reqAnimFrame performance (on a full screen map), we should see if it's possible to bring transitions back while keeping the current behavior:

You probably have to remove the class in one frame and then re-add it in another for the transition to animate, and that would add even more asynchronous complexity to tile loading, which I'd like to avoid.

We can't afford degrading performance significantly with a 1.0 release, so if there's no way to get around this without adding complexity, we'll have to add it.

Done here after will-change changes


Running OmniDB via docker behind jwilder/nginx-proxy results in websocket errors

I try to run OmniDB 2.14.0 via docker behind a nginx reverse proxy (jwilder/nginx-proxy). The application starts, but I always get the "cannot connect to websocket server with ports..." error. The jwilder reverse proxy generates the nginx configs automatically. Here is the output configuration.
I guess the problem is that I only get one "location" but I would need antoher one to redirect the wss requests like at the bottom of the official docs where there is a "/wss" location and a "/" location.
Any idea how I could manage to get OmniDB work with the jwilder/nginx-proxy and it's ssl companion container? How can I correctly forward the requests to the websocket server in this setup?
Any help is very appreciated!

Ok, I got it working now! Here is how I did it, in case some one else is also struggling with it:
My solution was to add an additional nginx proxy inbetween jwilder's nginx-proxy and the omnidb container. In the additional nginx proxy I mounted a default.conf into /etc/nginx/conf.d/default.conf with basically this configuration.
This is my docker-compose.yml. It uses a custom docker image I adjusted myself to easily be able to set the "external websocket port" via an environment variable.
And here is the .env file.
And of course you need an jwilder/nginx-proxy with ssl companion container already running.


Train-Validation-Test dataset

Good Morning
Is there any option to do the classic Train-Validation-Test split? Or at least does labelstudio support a validation dataset to be paired with training dataset? (as well as test dataset)
Thanks in advance

Hi @RiccardoMaistri, thanks for your question. Label Studio does not handle pre-processing of the datasets.


Use Save button as a .submit example

Description
Change the Save button in the example to use the .submit class and remove the type="button" that break the JQuery submit logic.
I think this documents the issue better, but we could call it out a bit more. I originally added a separate submit button, but really that is what Save should do.  Maybe rename Save to Submit would be better?

Closes #11994 but already closed

Types of changes

[X] Documentation
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (anything that would change an existing functionality)
[ ] Maintenance (refactor, code cleaning, development tools...)

Checklist

[X] I have read and follow the CONTRIBUTING.md document.
[X] The pull request title and template are correctly filled.
[X] The pull request targets the right branch (develop or develop-v...).
[X] My commits are correctly titled and contain all relevant information.
[X] I have updated the documentation accordingly to my changes (if relevant).
[X] I have added tests to cover my changes (if relevant).

Maybe rename Save to Submit would be better?

This could lead to unwanted side-effects and break things. I think the proposed changes should be fine.


REFACTOR: removed all OrderedDict

LGTM, quick comment:
In the pyproject.toml in the exclude section we still have this:
'HistoryProps.',  # bad RT05 because of the base class named OrderedDict.
@maxcapodi78


Update MetalLB to latest autodiscovery patch

Right now we use a patched version of MetalLB which was crafted from an early version of https://github.com/metallb/metallb/pull/593. The current version contains a bug which causes BGP sessions to flap in some cases.
Since it could take a while until https://github.com/metallb/metallb/pull/593 gets merged and released, we should update our patched version of MetalLB to the latest revision of the code in this PR.
Steps to perform:

Create new images for the speaker and controller from https://github.com/metallb/metallb/pull/593.
Push the images to quay.io/kinvolk/metallb-speaker and quay.io/kinvolk/metallb-speaker.
Update the MetalLB manifests to the manifests from the latest revision (the upstream MetalLB manifests are changed in https://github.com/metallb/metallb/pull/593).

@iaguis please keep in mind I am currently refactoring https://github.com/metallb/metallb/pull/593 following https://github.com/metallb/metallb/pull/593#discussion_r463856330 and https://github.com/metallb/metallb/issues/682.

Do you think we should wait until that work is finished or should we take the current state of the PR for now? I understand taking the current state means we'll need to do some more work in Lokomotive when the final feature is merged but I'd like to solve the flapping issue ASAP.

I think my latest refactoring as a better chance to solve the flapping sessions bug as it converges node peers with regular peers. I can likely finish the refactoring by EOD tomorrow.

PR #885 is up to fix this.


Update compile-cpp.yml to use macos-12

The runner name macos-latest has been migrated from macos-12 to macos-14, so we need to specify macos-12 specifically.
https://github.blog/changelog/2024-04-01-macos-14-sonoma-is-generally-available-and-the-latest-macos-runner-image/

Ok sounds good. Do we not want to replace 14 with latest so that it will update in due course?

I'll create an issue for changing macos-14 to macos-latest. I would argue that there are pros and cons to it. Keeping it pinned to 14 means that we know it should build, and if it doesn't then we know that something we've done has probably cause the break. If we go with latest then we reduce some of the future burden of adding more mac versions to build on, but we also run into this problem again rather unexpectedly.
I'm happy to do either and I see the merit of both, but it's a choice that the long term maintainers should make.


How to change the blue background color?

Problem
I am trying to change the blue background color. So I cloned the project and started to replace the #2980B9 with something else, but it doesn't work because the color is generated from the sass theme.
So I tried to replace all the $blue with $red into _theme_variables.sass. That background color is still coming from somewhere.
Reproducible Project
$ git clone https://github.com/rtfd/sphinx_rtd_theme.git
$ cd sphinx_rtd_theme
$ sed -i s/blue/red sass/*
$ grunt
$ grep -hor 2980B9 **/*.css | wc -w
38 # This 2980B9 is coming from somewhere... Where...

Additional details
It seems nothing has changed on the theme when I modify the sass. I have tried to change the icon on the admonitions:
diff --git a/sass/_theme_rst.sass b/sass/_theme_rst.sass
index 40cb58a..5252029 100644
--- a/sass/_theme_rst.sass
+++ b/sass/_theme_rst.sass
@@ -93,7 +93,7 @@
   .admonition-title
     @extend .wy-alert-title
     @extend .fa
-    @extend .fa-exclamation-circle
+    @extend .fa-book

When I save the file, grunt automatically refresh the theme, but I cannot see any change on http://localhost:1919/demo/demo.html?highlight=admonition#admonitions
Why?

Solution in #567