Error in RHEV - Build a VM * catalog items

When requesting a VM from oVirt I get an error instead of list of available vm templates.  evm.log has:
[----] E, [2016-04-22T09:00:45.990981 #3202:fdf034] ERROR -- : <AutomationEngine> Method STDERR: (druby://<IP_ADDRESS>:32809) /var/www/miq/vmdb/lib/miq_automation_engine/engine/miq_ae_service_model_base.rb:179:in method_missing: undefined method miq_templates for #<MiqAeMethodService::MiqAeServiceManageIQ_Providers_Redhat_InfraManager:0x0000000357b608> (NoMethodError) [----] E, [2016-04-22T09:00:45.991696 #3202:fdf034] ERROR -- : <AutomationEngine> Method STDERR: 	from <code: provider.miq_templates.each do |template|>:115:in <main> provider.miq_templates.each do |template|
If I go to  Integration/RedHat/RHEV/DynamicDialogs/list_rhev_template_guids,  and comment out  searching for templates on a provider (lines 114-120), I can get a list of all oVirt templates.
So, to me it looks like I do get a provider object, but it is missing miq_templates method. Should this work with  ManageIQ/oVirt, or does it require Cloudforms/RHEV?
I have ManageIQ Capablanca and oVirt 3.6

Honestly, we did not test this with the upstream versions of either product. I have no reason to believe they shouldn't work, but we just haven't had time to look at it yet.
If you go into a rails console and run something like the following...what do you get back?
Enter rails console and enter MIQ workspace (make sure you have sourced cloudforms_essentialsrc for alias). After entering the console, past the $evm=xxx line that is echo'd:
railsc
Set your provider and see what methods are available:
provider = $evm.vmdb(:ManageIQ_Providers_InfraManager).all.detect { |x| x.name == "<your-provider-name>" }
provider.methods
I've tested this against both RHEV 3.5 & 3.6 and I am able to pull back the templates with each.
You can also add a line of code just after 112 to log in automation.log what the provider object is so you know what it is picking up before it fails and exits.
log(:info, "Provider found: #{provider.inspect}")

@sgenchev - Were you able to get past this issue?

I am sorry, was out of the whole computer business.
Log line for provider below:
There is no mid_templates method. Closest one is total_miq_templates. Where does this method come from? oVirt SDK, Miq?
Thanks!
Provider found: #<MiqAeServiceManageIQ_Providers_Redhat_InfraManager:0x897f678 @object=#<ManageIQ::Providers::Redhat::InfraManager id: 2, name: "oVirt-prod", port: "443", hostname: "olaxpa-ovirtm01.garmin.com", ipaddress: nil, created_on: "2016-02-02 19:39:01", updated_on: "2016-05-05 13:13:57", guid: "9bf6565e-c9e4-11e5-8ac2-001a4a160158", zone_id: 1, type: "ManageIQ::Providers::Redhat::InfraManager", api_version: "<IP_ADDRESS>", uid_ems: nil, host_default_vnc_port_start: nil, host_default_vnc_port_end: nil, provider_region: nil, last_refresh_error: nil, last_refresh_date: "2016-05-05 13:13:57", provider_id: nil, security_protocol: nil, realm: nil, tenant_id: 1>, @virtual_columns=["aggregate_cpu_speed", "aggregate_cpu_total_cores", "aggregate_disk_capacity", "aggregate_logical_cpus", "aggregate_memory", "aggregate_physical_cpus", "aggregate_vm_cpus", "aggregate_vm_memory", "authentication_status", "cpu_usagemhz_rate_average_avg_over_time_period", "cpu_usagemhz_rate_average_high_over_time_period", "cpu_usagemhz_rate_average_low_over_time_period", "derived_memory_used_avg_over_time_period", "derived_memory_used_high_over_time_period", "derived_memory_used_low_over_time_period", "emstype", "emstype_description", "last_refresh_status", "max_cpu_usage_rate_average_avg_over_time_period", "max_cpu_usage_rate_average_avg_over_time_period_without_overhead", "max_cpu_usage_rate_average_high_over_time_period", "max_cpu_usage_rate_average_high_over_time_period_without_overhead", "max_cpu_usage_rate_average_low_over_time_period", "max_cpu_usage_rate_average_low_over_time_period_without_overhead", "max_mem_usage_absolute_average_avg_over_time_period", "max_mem_usage_absolute_average_avg_over_time_period_without_overhead", "max_mem_usage_absolute_average_high_over_time_period", "max_mem_usage_absolute_average_high_over_time_period_without_overhead", "max_mem_usage_absolute_average_low_over_time_period", "max_mem_usage_absolute_average_low_over_time_period_without_overhead", "region_description", "region_number", "total_clusters", "total_hosts", "total_miq_templates", "total_storages", "total_vms", "total_vms_and_templates", "total_vms_never", "total_vms_off", "total_vms_on", "total_vms_suspended", "total_vms_unknown", "zone_name"], @associations=["ems_clusters", "ems_events", "ems_folders", "hosts", "provider", "resource_pools", "storages", "tenant", "vms"]>

So here are my associations for my provider:
@associations=["customization_specs", "ems_clusters", "ems_events", "ems_folders", "hosts", "miq_templates", "provider", "resource_pools", "storages", "tenant", "vms"]
I am wondering if something got missed in the upstream ManageIQ project?  Again, CloudForms_Essentials was built for the downstream stable version.
I would highly recommend that you submit this question to talk.manageiq.org because this method should be there. Closing this issue.


Is this plugin must be registered globally?

i read the docs for vue2, and can't find the locally regisered example, only the globally registered example. so i try it out, and failed. So is this plugin can only be registered globablly, or am i miss something?
i read the issue #1 , it seems related, but i'm using one single version of highlightjs 10.7.2, so i'm not quite clear whether or not this plugin supports locally register.
thanks.
NODE_MODULES

+---@highlightjs
|   \---vue-plugin
|       +---dist
|       \---src
|           \---lib
+---highlight.js
|   +---lib
|   |   \---languages
|   +---scss
|   +---styles
|   \---types

GOOD version (Globally registered）:

<template>

    <highlightjs language="javascript" code="var x = 5;" />

</template>

BAD version (Registered in single file component）:

<template>

    <highlightjs language="javascript" code="var x = 5;" />

</template>

errors in chrome console:
vue.runtime.esm.js?2b0e:619 [Vue warn]: Unknown custom element: <highlightjs> - did you register the component correctly? For recursive components, make sure to provide the "name" option.

found in

---> <TestTarget> at src/views/WebView/SourceCodeViewer.vue
       <TestPage> at src/pages/TestPage.vue
         <App> at src/App.vue
           <Root>

if i give a name，the error message change, but still cannot render the 'var x = 5'
export default {
  components: { 'highlightjs': hljsVuePlugin }
}

vue.runtime.esm.js?2b0e:619 [Vue warn]: Failed to mount component: template or render function not defined.

found in

---> <Highlightjs>
       <TestTarget> at src/views/WebView/SourceCodeViewer.vue
         <TestPage> at src/pages/TestPage.vue
           <App> at src/App.vue
             <Root>

it seems that the project name 'vue-plugin' already indicate that this is a vue plugin not component, which can only be installed using Vue.use and use global registration, according to the vue.js docs here。

The vue2 version exports component also, though Vue 3 version doe now.  Perhaps that would work?
export default {
  components: { 'highlightjs': hljsVuePlugin.component }
}

@joshgoebel yes it works! thanks for the help~~

@Trinovantes Is there a reason this export was removed from the Vue 3 version?

Just an oversight on my part. It should be fine to export for a local component

Would mind resolving maybe? 🙃


checkhealth not working for me

After execute :checkhealth which_key, NeoVim does no pass the following output:
which_key: health#which_key#check
========================================================================
## WhichKey: checking conflicting keymaps

Any idea what could it be? I'm using which-key default plugin configuration.
Using Neovim v0.8.0

Same.

I'm having the same issue here.

Running checkhealth which_key produced an error for me:
Vim(lua):E5108: Error executing lua .../pack/packer/start/which-key.nvim/lua/which-key/keys.lua:372: attempt to index field 'health' (a nil value)

I patched it locally in ~/.local/share/nvim/site/pack/packer/start/which-key.nvim/lua/which-key/keys.lua with
local health = vim.health or require'health'

and replaced vim.health with health. Probably someone can prepare a PR for this?


NavigationLink binding lead to view onAppear

I have a strange problem, I need use NavigationLink push other view. If I use this method when pushed can lead to current view onAppear.
NavigationLink(
isActive: viewStore.binding(get: .isNavigationActive, send: JenkinsJobDetailAction.setNavigation(isActive:)),
destination: {
Text("...")
},
label: {
Image(systemName: "hammer")
.font(Font.system(size: 15))
.foregroundColor(.primary)
}
)
If I use another method, not lead to current view trigger onAppear.
NavigationLink {
JenkinsJobBuildView(store: store.scope(
state: .buildJob,
action: JenkinsJobDetailAction.buildJob
))
} label: {
Image(systemName: "hammer")
.font(Font.system(size: 15))
.foregroundColor(.primary)
}
Please help me analyze it. Thanks!

Hi @15620946376, I'm going to convert this to a discussion since it's not an issue with the library.


Add new on-campus landing page

⚠ WIP; needs copy and images updating

Trello card
Trello-3538
Context
The Direct Marketing team want to try a new on-campus landing page; Havas have came up with a design which we have implemented with a view to reusing the new components.
Changes proposed in this pull request

Add PhotoQuoteListComponent for on-campus landing page

The new on-campus landing page designed by Havas has a list of quotes and associated photographs.
Add component to render a list of quotes with photos; the layout is fairly complex so I've split the grid layout in the CSS to make it easier to read.

Add food for thought page

This is the new on-campus landing page. It uses the new PhotoQuoteListcomponent to display a number of quotes. The markdown scss had to be updated to allow for a yellow box-style heading on this page.

Make subject specific header more generic

We want to use the subject specific header/hero in other areas of the website.
Refactor it to be a generic campaign header that can be used on other pages.

Update on-campus hero section

Use the generic campaign header with an updated cake image.
Guidance to review
The page can be previewed here.
Checklist of things still to do

[ ] Noindex the page
[ ] Consider adding a folder structure eg /landing
[x] Add a promo module
[ ] Add intro paragraph and heading from Havas
[ ] Add case study copy and images
[ ] Update promo module link (and photo) prior to merging

Paging @EllieNodder @GDW7 to ask if we should have a (light-touch) CTA at the end of this landing page, possibly to the ETA page?  Promo module? Something else? Or nothing?

Paging @EllieNodder @GDW7 to ask if we should have a (light-touch) CTA at the end of this landing page, possibly to the ETA page? Promo module? Something else? Or nothing?

Seems like the perfect page to me for adding an ETA promo - exactly the right audience

Paging @EllieNodder @GDW7 to ask if we should have a (light-touch) CTA at the end of this landing page, possibly to the ETA page? Promo module? Something else? Or nothing?

Seems like the perfect page to me for adding an ETA promo - exactly the right audience

Would you be able to add this (next week) @EllieNodder ?

Thanks for adding the promo @EllieNodder! That looks fine - if we get a proper ETA image, it would probably be worth updating it here (or otherwise shortening the 'Explore teaching advisers' title so the person's eyes aren't covered on mobile?). Page is not due to go into production until end of next week though so let's leave for now

@ethax-ross have added an ETA promo module - it seems to be coming through in the review app, just not sure I've done it in the right way. The backend of the page looks diff to elsewhere where we have the promo modules so have literally just copied the whole promo rather than the usual 'promo_content:' thing in the frontmatter. Does it look okay to you?

Thanks for adding the promo @EllieNodder! That looks fine - if we get a proper ETA image, it would probably be worth updating it here (or otherwise shortening the 'Explore teaching advisers' title so the person's eyes aren't covered on mobile?). Page is not due to go into production until end of next week though so let's leave for now

Yep, am chasing Steph/Andy about the image

@gemmadallmandfe @EllieNodder @ethax-ross I spotted and corrected a small typo in quote 4. Otherwise it looked good to me.


Crash on any nuke useage.

Any time I try to use/launch a nuke, I recieve a crash with this message:
The mod True Nukes (0.3.1) caused a non-recoverable error.
Please report this error to the mod author.
Error while running event True-Nukes::on_tick (ID 0)
Entity creation of entity-ghost failed
stack traceback:
[C]: in function 'create_entity'
True-Nukes/scripts/blast-system.lua:111: in function 'damage_entity'
True-Nukes/scripts/blast-system.lua:252: in function 'move_blast'
True-Nukes/scripts/blast-system.lua:294: in function 'move_blast'
True-Nukes/scripts/blast-system.lua:294: in function 'move_blast'
True-Nukes/scripts/blast-system.lua:294: in function 'move_blast'
True-Nukes/control.lua:78: in function <True-Nukes/control.lua:65>
I am using K2+SE, although neither seem to be the issue? I'm not sure, though. Sorry for the vague details.

OK, I think my last release should have fixed this, although its not a very elegant fix.


Clicking 'Run' on a suite or single test does nothing

Describe the bug

[ ] test load missing suites/tests
[ ] test load errors
[x] test running errors

Test Explorer lists all my tests correctly.  I can debug any test normally.  When I click the run button (in the explorer or in my code), nothing happens at all.  I enabled the Logpanel setting, and I get this output:
[2020-10-03 16:33:51.905] [INFO] Running Jest tests... ["undefined▪a:\\source\\project\\test.spec.ts♦name"]
[2020-10-03 16:33:51.905] [INFO] Finished loading Jest tests.

When I click 'Run All' for the entire project, it works fine (a Blue 'play' icon appears, then a while later it changes to a green check, although I don't see any log output during this process?).
I assume that 'undefined' shouldn't be there in the path?  I tried setting the 'Path to Jest' setting (although it's unclear exactly what it should be set to, or how to tell when the auto detection has failed) to "${rootPath}\node_modules.bin\jest", but that had no effect on the 'undefined' in the logs.
My project is typescript based, using jest + ts-jest.  jest.config.js is at the root of the workspace, along with the node_modules folder where jest is installed.
Desktop (please complete the following information):

OS: Windows 10 x64

Can you have a look at v0.7.5, just released to marketplace?  @TheSench has contributed which may fix this issue.

Can you have a look at v0.7.5, just released to marketplace?  @TheSench has contributed which may fix this issue.


SQL MI Manual and Scheduled Start/Stop APIs

ARM API Information (Control Plane)
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

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

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
November 2022.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
November 2022.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

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

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

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

Change to data plane APIs
Adding new properties
All removals

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

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

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

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

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

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

      "200": {

indicate the final response of the long running operation with the 200 status code

Refers to: specification/sql/resource-manager/Microsoft.Sql/preview/2022-08-01-preview/ManagedInstances.json:498 in 0c3919c. [](commit_id = 0c3919c165fdc77f3903a35b6b68d328c710a38b, deletion_comment = False)

      "200": {

same here, the final response of the long running operation needs to be indicated along with the 200 status in the swagger

Refers to: specification/sql/resource-manager/Microsoft.Sql/preview/2022-08-01-preview/ManagedInstances.json:541 in 0c3919c. [](commit_id = 0c3919c165fdc77f3903a35b6b68d328c710a38b, deletion_comment = False)

      "readOnly": true

why is x-ms-identifier being dropped?

Refers to: specification/sql/resource-manager/Microsoft.Sql/preview/2022-08-01-preview/ManagedInstances.json:707 in 0c3919c. [](commit_id = 0c3919c165fdc77f3903a35b6b68d328c710a38b, deletion_comment = False)

      "200": {

indicate the final response of the long running operation with the 200 status code
Refers to: specification/sql/resource-manager/Microsoft.Sql/preview/2022-08-01-preview/ManagedInstances.json:498 in 0c3919c. [](commit_id = 0c3919c, deletion_comment = False)

@rkmanda Not sure what you mean by this, can you elaborate, or give me some example?

❌ ResourceNameRestriction
❌ CreateOperationAsyncResponseValidation
❌ LroLocationHeader
These failed pipelines were introduced in Oct 2022 (less than two months ago). Can we ask for the exception for this API version (2022-08-01-preview) ? Because 2022-08-01-preview was planned and developed before Oct 2022.
We will try to address these newly introduced policies in the next API version release. Thanks ~
same as https://github.com/Azure/azure-rest-api-specs/pull/21733#issuecomment-1345847508

      "200": {

Not fixed

In reply to:<PHONE_NUMBER>

Refers to: specification/sql/resource-manager/Microsoft.Sql/preview/2022-08-01-preview/ManagedInstances.json:498 in 0c3919c. [](commit_id = 0c3919c165fdc77f3903a35b6b68d328c710a38b, deletion_comment = False)

      "200": {

Please fix these comments

In reply to:<PHONE_NUMBER>

Refers to: specification/sql/resource-manager/Microsoft.Sql/preview/2022-08-01-preview/ManagedInstances.json:541 in 0c3919c. [](commit_id = 0c3919c165fdc77f3903a35b6b68d328c710a38b, deletion_comment = False)

      "200": {

Not fixed
In reply to:<PHONE_NUMBER>
Refers to: specification/sql/resource-manager/Microsoft.Sql/preview/2022-08-01-preview/ManagedInstances.json:498 in 0c3919c. [](commit_id = 0c3919c, deletion_comment = False)

Both this and the other one are fixed, 200 is completely removed. Line referenced here is in an older commit, this has been addressed last week.

@mmaricic-microsoft , let me know if you would like to fix modelValidation errors in this PR.

@mmaricic-microsoft , let me know if you would like to fix modelValidation errors in this PR.

@raych1 we discussed this a few weeks ago (the whole giant chat regarding swagger errors was actually started by me). In there we agreed that, since I do not own any of the APIs that have failing validation, and the files in which it is failing are not changed by me, I should just reach out to owning teams in order to create repair items for this.
The email thread is "Fixing Swagger examples for owning operations".

As discussed offline, we agree to fix the model validation errors in this workitem: https://msdata.visualstudio.com/Database Systems/_workitems/edit/2097649


Iss32 - merges custom css files into style.css

Fix for issue #32
Removed in style.css:
.row {
  padding: 0 0 2em;
}
footer {
  padding: 20px 0;
}
#maincontent {
  padding-top: 50px;

.row is a bootstrap class. If rows need to be modified, use custom class
redundant footer, using instead: footer {
padding: 1em 0 4em;
overflow: hidden;
}
use body body {
font-family: 'Open Sans', sans-serif;
margin-top: 50px;
}
also removed index_style.css and modern-business.css with changes above

I just looked at the homepage from the master branch and this branch and the carousel looks broken to me on this branch.  I haven't checked for other differences between master and this branch.
To confirm the purpose of this branch:
This branch should result in the website looking identical to master, right?  I take it this is a entirely code style clean up and not a functional change.

Some of the css paths needed to be changed to css/style.css. Everything should look the same now.


Caching hardware capabilities

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


(PUP-10316) Update "http" report processor to use the http client

Blocked on https://github.com/puppetlabs/puppet/pull/8051
Update the "http" report processor to send reports using the http client. The normal client is used when running puppet apply. When running in puppetserver, the external http client is used, which delegates to puppetserver.
Adds an integration test to verify puppet apply submits a report to an HTTPS URL whose server cert is issued by the puppet CA.
Adds an "https client" shared context to simplify test setup.

CLA signed by all contributors.


fix: collections not being initialized without User-Agent header

ip and global collections are not initialized for requests without a User-Agent header. This may allow malicious users to bypass rules using these collections.
Fixes #3642.

@fzipi Sure but is there a way how can i know if collections were or were not initialized? How to write a test for it?

Don't know what to say here. @dune73 do you have a clue on how to test this?

Testing this with ftw is complicated.
If you want to test it, you would need an additional rule that checks for a variable in the collection. If you do not want to do that, then you add a plugin with an effect based on the collection.
Honestly, I think we can add a remark here and forget about the testing for the time being, it's not worth the hassle.

Proposal:
The creation of the IP and the GLOBAL collection is not being tested as
of this writing due to limits in ftw and our testing setup.
Proper testing would involve the checking of a variable in the said collections.

Done!


[seatunnel] move dijiekstra from committer to ppmc

move dijiekstra from committer to ppmc

You only update the zh-CN and en need be update too.


Create uri-from-top-55-most-popular-apps.txt

Removed trailing slashes

How was this list made?
Whats the source?

https://github.com/danielmiessler/SecLists/pull/781#issuecomment-1168353194
https://twitter.com/nullenc0de/status/1425973675715612672
https://gist.github.com/nullenc0de/e9d1f2a8a0a38c9bfcb5bdb9fc7191ea

Thank you!


BatchNorm documentation

There are a lot of things I don't like about this BatchNorm op. Now there are more.

This is an example of what we're currently doing in the bridge to work around this: https://github.com/NervanaSystems/ngraph-mxnet/pull/207

TESTNOW

I've changed this to what I think we want BatchNorm to be.


How to use this with a Camera3D  ?

I attempted to use this plugin with a Camera3D . There is no documentation so i gave it a whirl and it seems like the UI elements all appear in the top left corner all jumbled together without proper vertical spacing at all... why ?


chore: extends dropzone and upload field

Description
Tweaks to Upload and Dropzone components, making them more extendable.

Dropzone adds prop to allow multiple files

Upload correctly sets url if state is initialized with a File

[ ] I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

[x] Chore (non-breaking change which does not add functionality)

Checklist:

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Existing test suite passes locally with my changes
[ ] I have made corresponding changes to the documentation

@JarrodMFlesch nice changes! Could you please also look into exposing the component? https://github.com/payloadcms/payload/pull/7581

@carlcs absolutely, I'll make that happen in this PR

That’s great, looking forward to this getting merged! And no worries, I’ll close my PR.


Fix issues with missing jar files

Without building the root project (as suggested in the proposed edit), required dependencies cannot be found e.g. dice-provider-emulator-1.0.0.jar, thus the project fails to build.

@kirpasingh : Thanks for your contribution to the Azure documentation! The author, @msebolt, has been notified to review your proposed change.

Changes look good to me.
Adding @dominicbetts for sign off.

#sign-off

@dominicbetts: I'm sorry - only the author of this article, @msebolt, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@msebolt - Hi Matt, looks like you need to do the sign off - go ahead please.

Sure, no problem.
#sign-off


Improved Label widget

Hi there,
I would like to have a label widget with some extra features:

Highlight single characters (typically underline) to mark mnemonics
Control overflow behaviour - do not wrap, but render an ellipsis
Maybe have some extra state for mouse hover to allow showing a tooltip

Context: I plan to create an application that has to be fully controllable using the keyboard. I will start to create that widget myself.

(1) and (2) are blocked by #33, as most text-related issues.
(3) can probably be achieved with the Tooltip widget.

about (1) and (2) - I will work around that - get the positions based on grapheme clusters (using TextInput as inspiration), and then just draw a line so it approximates an underline for a single character. It may not look perfect and break apart with with some fonts or non-latin scripts, but it will be "good enough" for now.
about (3) - as far as I understood, the info if a widget is hovered is needed to display tooltips correctly - otherwise all tooltips would follow the mouse constantly. It may turn out that no such feature is needed, instead box the label (and related widget) into another container that has a "hovered" state

I think not only for labels, but text overflow as a whole should be configurable. Currently, text seems to only wrap on white-spaces, so an overflow configuration could have 5 options:

TextOverflow::WrapWords (default, wrap at whitespace)
TextOverflow::WrapChars (wrap at char that would overflow, may break a word)
TextOverflow::Clip (Clip text)
TextOverflow::Ellipsis (Clip text and add ellipsis to it "...")
TextOverflow::Allow (Allow for text to overflow, maintaining the text as it is)


MB-14972 Implement BuildReService

Jira ticket for this change
Summary
This PR replaces calls to testdatagen.MakeReService with the new factory.BuildReService
Setup to Run Your Code
Run make server_test and make sure tests pass
Verification Steps for Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

Frontend

[ ] User facing changes have been reviewed by design.
[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] There are no new console errors in the browser devtools
[ ] There are no new console errors in the test output
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.

Backend

[ ] Code follows the guidelines for Logging
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Database
Any new migrations/schema changes:

[ ] Follows our guidelines for Zero-Downtime Deploys
[ ] Have been communicated to #g-database
[ ] Secure migrations have been tested following the instructions in our docs

Screenshots

Messages

:book:
:link: MB-14972

  Generated by :no_entry_sign: dangerJS against 1542082f79ff1d098c627929186bf4c589ecc762


create unit tests

I'm happy to do this, but if you could give some pointers that would be cool

Its basically just testing each method in our program. You can hand write the pseudo code which shows how you will test to see if each method works. He actually gave examples of how to do it in the slides he gave today

just got home for the day.. sorry I've been lagging behind you on this assignment. I'm currently breaking up the dependency.py file into more than just "getDependenciesAndScan" for easier unit testing


4 |  MedianOfTwoSortedArrays | C++

Title -
What will change -
Type of Issue -
Please add/delete options that are not relevant.

[x] Adding New Code
[ ] Improving Code
[ ] Improving Documentation
[ ] Bug Fix

Programming Language
Please add/delete options that are not relevant.

[ ] Python
[x] C++
[ ] Java

Self Check

Ask for issue assignment before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/Ftbj4a
Happy Coding,

I want to work on this problem.

I want to work on this problem, can you assign me?

[x]  Assigned to @gadm21

Can you assign me this issue, I have already created a pull request, please check that, my code takes case of all corner cases

@Rohit-varma Hi since this Issue has already been assigned to @gadm21 ,  you can create another Issue and wait for it to be assigned to you and only after that you can make a Pull Request


CSI external-resizer project updates 1-23 to 1-29

Description of changes:
CSI external-resizer project updates 1-23 to 1-29
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm


Skip rest segment button does not clear the next toast notification

Repro:

Rest segment starts
Click skip rest segment
Observe the "time to focus" notification still appears even though we're already focusing.

It would be a bit hacky, but wouldn't it be possible to pause the toast service for a couple seconds so that the notification doesn't send?

Fixed in v4.3


FIX: Ensure permission denied in geolocation.getCurrentPosition rejects the Promise

fixes https://github.com/facebook/react-native/issues/22535
Moves the permission request logic on Android M and above to Native side to mimic IOS behavior.
Changelog:
[Android] [Fixed] - Ensure permission denied in geolocation.getCurrentPosition rejects the promise
Test Plan:
Was originally reviewed and accepted as https://github.com/facebook/react-native/pull/22843 as part
of core react-native. That PR was verified via my test project which requested location via
Geolocation service. If the permission request was denied by the user the promise was rejected as well.
https://github.com/Jyrno42/rn-geoloctest

Hello @Jyrno42 @janicduplessis !
I'm facing the same issue with android geo_error callback. How can I access the newer version with this fix?


samplegen: fixes to and for check-calling-form-in-baseline: yaml file updates, tag string format

Some additional changes are needed on top of #2012

The library_gapic.yaml should be migrated to proper callingFormCheck strings. Sorry I overlooked this in #2012
We should add callingFormCheck lines for the additional languages
The multi-line yaml literal suggested for the multi-language callingFormCheck lines is not working for me. Let's either update the doc or revisit the format
As a result of the above, we should deal with any resulting error messages from the check
The callingFormCheck string should not require specifying the value_id, since it is already embedded in a value set. Let's avoid the repetition
If it's easy, error messages should ideally also output the method name where the expected value set/calling form combination was not found. This would make it easier to fix errors, but is not a strict requirement because we're only using this for testing the generator itself, so we can name value sets distinctly to make finding errors easy.

See a PR in progress in my fork that begins to address items 1 and 2. I am blocked on 3, so will let @pongad handled that, as well as doing 5.

Resolved by #2049


Create TOS Content

The app needs to display the Terms of Service, which closely mimic the TOS from Stimulus.com since the content isn't to be duplicated or reused without permission and credit.
Definition of Done
The following tasks need to be completed.

[x] displays TOS
[x] provides link to Stimulus' TOS

Tech Notes
Implement the AboutView and TermsOfServiceView as wrappers to display markdown content in a popup dialog.

Done. Pasted markdown into code. Needs to be integrated into the app.

Done. Pasted markdown into code. Needs to be integrated into the app.


refactor(CLIMATE): create separate directive

Experimenting with separating each tile's code into its own directive.
To summarize, the specific tile's code is separated into its own directive and HTML template.
The directive's controller contains tile-specific code and is exposed as ctrl in the template.
Since item and entity are bound to the isolated scope of the directive, there is no need to pass those from the tile's template anymore since the controller has access to them already. Only in case, we want to call the APIs from the main controller (exposed as ctrl.main in the tile's template) we need to pass them. That part is a bit dirty right now but could be made better eventually.
Please don't merge yet. Still figuring out if this is the best way to do it.
Please have a look and provide any feedback @alphasixtyfive @akloeckner

Please don't merge yet. Still figuring out if this is the best way to do it.

Isn't there a "draft PR" feature for this situation?

Any more movement in this direction? I've been wanting to add some new tiles but the current structure makes that a bit difficult, so having them split out to separate files would be awesome.

Sadly, I don't think I can currently devote that much time to get all of those refactors done.
Though I guess this PR might be "mergable" already, if we ignore the raised improvement suggestions. Maybe that would kick-start more changes from other people.

I'd certainly be up for helping split some more tiles out to separate files if there was a framework in place for it 🙂

My 50ct: Yes, please ignore my suggestions, if they are blocking improvement! If there is interest in advancing this, we should go with the usual spirit... Improve step by step and handle problems if and when they arise.
So, Charles, if you are planning on adding some tiles, you will certainly stumble accross some of these issues. Meaning, they'd even be adressed in a timely manner. And by the time you're done with the new tiles, the framework will have settled to something consistent. I'd be happy to provide some more 50ct from my non-ng-expert perspective any time, if you like.
If we don't want this convergence to happen on the main branch, we could set up a feature branch and merge when appropriate. However, since the main branch receives updates only once per month by now, I would just wait for this convergence to settle before creating a new release...


Small fixes to HLL Tests.

Tweak if-else conditional formatting to follow PSR2. Fix BucketOperationsTest check for HLL bucket type. Fix HLL bucket props test. Fix HllOperationsTest name.

:shipit:


[SPARK-29938][SQL][FOLLOW-UP] Improve AlterTableAddPartitionCommand

All credit to @Ngone51, Closes #27293.
What changes were proposed in this pull request?
This PR improves AlterTableAddPartitionCommand by:

adds an internal config for partitions batch size to avoid hard code
reuse InMemoryFileIndex.bulkListLeafFiles to perform parallel file listing to improve code reuse

Why are the changes needed?
Improve code quality.
Does this PR introduce any user-facing change?
Yes. We renamed spark.sql.statistics.parallelFileListingInStatsComputation.enabled to spark.sql.parallelFileListingInCommands.enabled as a side effect of this change.
How was this patch tested?
Pass Jenkins.

The PR description is not right anymore.

The PR description is not right anymore.

Do you mean the config name is different?


Create library for third party consumption of OONI data

This is about exposing some functionality of the faspath as a python module/CLI that third party developers can pip install and use for consuming OONI data.

For the record, this is my solution to this problem:
https://github.com/Jigsaw-Code/net-analysis/tree/master/netanalysis/ooni
It does everything I need as someone that analyzes OONI data. It can be used as a cli tool, or as a library in a Jupyter notebook.
You can download data for a specific test type, country and date range. It doesn't download extra data, which is especially critical in the old S3 bucket. Instead it reads the index files to determine the byte ranges to fetch from the report archives.

@fortuna thanks for sharing this!
We have discussed this internally and have concluded that we would like to integrate your tool for downloading OONI measurements inside of the ooni/pipeline codebase.
The plan for how we are going to go about doing this is as follows:

[ ] We create a new directory structure in ooni/pipeline that will host all data analysis code that is to be used by third parties
[ ] We take the codebase from https://github.com/Jigsaw-Code/net-analysis/tree/master/netanalysis/ooni and integrate it into ooni/pipeline
[ ] Refactoring is done to make this code be used by the fastpath
[ ] Publish this as a package on pypi that people can easily pip install it
[ ] Update mining OONI data post to reference this new tool (https://ooni.org/post/mining-ooni-data/)
[ ] Add to the CI support for pip installing the tool and running it on windows, macOS and linux

Great, I'm glad to hear!
I believe it can be beneficial to have the analysis code separate from the pipeline.
Why would someone depend on the entire pipeline if they don't want to run the pipeline?
On the other hand, I can see the analysis code being used in many different tools and contexts.
I'd argue that the dependency should go the other way around: the pipeline should depend on a data analysis library.
Making the analysis library separate from the pipeline also makes sure that it doesn't depend on any part of the pipeline, thus more portable.
@FedericoCeratto @hellais Perhaps create a ooni/analysis?

@bassosimone explained to me that they have different packages in the same repo. That should do it. Though it would be helpful to update the root README to not distract people that get there for data analysis with all the pipeline stuff.

I think what I can do is create a setup.py for the existing sync_measurements. Then we can just copy the folder into the right place.

So I did a bit more research into this problem and I will share some notes about it here.
What I would like is that end users can do pip install $pkgand they will get some tools useful for this purpose. Ideally the code of $pkgwould be the same that is used by fastpath so that we aren’t duplicating codebases and we ensure that it’s nicely tested.
This package should be then used by end-users both as a CLI tool and as a set of functions they can call programmatically from python code.
I started looking at how we would integrate netanalysis/ooni so that we would be using that code in the fastpath, but then realised that maybe it's easier to adapt the fastpath code to have the same functionality as netanalysis/ooni and where relevant integrate missing bits of code from netanalysis/ooni into fastpath directly.
That way we would have from day 1, something that is already used in production and we wouldn't be risking too much by integrating a new codebase.
I would therefore like to do a bit of refactoring of fastpath (https://github.com/ooni/pipeline/tree/master/af/fastpath/fastpath) so that code that is specific to our infrastructure is separated from code that would be useful to third parties.
The main functions that I would imagine an analyst being interested in are:
OONI data sync
Functions to fetch OONI data from s3 based on some search parameters abstracting away the complications related to the data being split across different buckets in different data formats and offering them on disk in a consistent and uniform data format.
I think the API that @fortuna came up with for this is pretty nice and is basically what most users care about for this use-case, see: https://github.com/Jigsaw-Code/net-analysis/tree/master/netanalysis/ooni.
He’s not the first person to write such a tool, as David Fifield wrote the same tool in go a while back: https://github.com/ooni/ooni-sync.
Having an official way of doing this I think should be considered P0.
I am looking at the codebase for pipeline and the function that seems most relevant to this is stream_cans inside of s3feeder (https://github.com/ooni/pipeline/blob/master/af/fastpath/fastpath/s3feeder.py#L303).
The problems of exposing this as-is are a few (a lot of these apply to the other use-cases too):

They all depend on statsd, which I don’t think we should have a requirement for third party data analysts. My plan for solving this is to support passing to all functions that require emitting metrics a custom metrics object. In the case of third party data users, this metric object would support optionally printing to stdout the value of the emitted metrics and would otherwise just be a noop. An alternative approach could be that of automatically using the mocked metrics object in setup_metrics if the import fails, but I think this could be potentially a bit risky as it could lead to a deployment not logging metrics and us not noticing.
The usage of the opaque confobject is a bit awkward. I think it would significantly improve the API if we were to just expand the values of the conf object that each function call needs. They aren’t actually many, so this doesn’t seem much of a challenge.
My understanding of how the s3feeder works, is that it stores on disk the cans in the original format. I think that to improve usability we should instead rewrite all of them in a consistent data format to ease the consumption by third parties.
Compared to the library of @fortuna, we are also missing one feature, which is that of trimming strings that are longer than a predefined length.
I don’t think it’s too much effort to add support for this and we should probably add it as well. The code he wrote for this looks neat and I think we can re-use it as-is.
The other feature we are missing in multi-threading support for the fetching of cans. This also isn’t too much effort, but it might break how it interacts with the pipeline, so maybe we can put this on hold and do it as future work.

Running OONI measurements through analysis
I think it would also be nice if we supported the ability to apply the scoring logic to a set of measurements the user has downloaded to their disk.
The reason for this is a few:

Currently there is no way for a user to fetch a full database dump or to map the results coming from the API (that includes the scoring) to the raw measurements they downloaded.
It would also be helpful for us to be able to more quickly iterate and improve our scoring logic on a sample of data inside of something like a jupyter notebook

The task of splitting this out seems a little bit more challenging, though, because at a cursory glance it looks a bit more tightly coupled to the database as well as things that are very specific to our deployment (ex. the creation of directories that assume you are running on a linux system: https://github.com/ooni/pipeline/blob/master/af/fastpath/fastpath/core.py#L66).


fix: give update-semver additional permissions

This should update the existing v1, v1.9 tags to point to the latest released tag.

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


Add mne.count_annotations()

Similar to mne.count_events(), I think a function to count unique annotations would be quite useful. This PR adds a function mne.count_annotations() as well as a method mne.Annotations.count().

Thanks @cbrnr !


debug env message should mask key values by default, only exclude the unmasked_keys

see this line: https://github.com/teracyhq-incubator/secret-manager-action/blob/develop/src/config.ts#L165
You must create a secret named ACTIONS_STEP_DEBUG with the value true to see the debug messages.
expected: values should be masked, only exclude the values from unmasked_keys
actual: all the values were displayed without any mask

actually, the key values were all in clear text when ACTIONS_STEP_DEBUG=true, so we need to check and make sure to add mask before output the debugging message.


can't use mysql DATE_FORMAT function

Make sure these boxes are checked before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if any
[ ] I have reproduced the issue with at least the latest released version of superset
[ ] I have checked the issue tracker for the same issue and I haven't found one similar

Superset version
master
Expected results

Actual results

Steps to reproduce
select DATE_FORMAT(now(), '%h%i%s');

other
It looks like confusion with python syntax

double %, i usually do this

@tanjxyz thanks,when i use double % , it's work.

how to fix
if i query like is
select indb_time, TotalFiles, BlocksTotal
from hdfs_info_logs
where statis_date>=20171101
and DATE_FORMAT(indb_time, '%%H%%i%%S') = '000000';

and use Visualize , it will auto add double % in my query

I tried to fix this in the past and it was more tricky than expected. It has to do with the fact that different drivers behave differently in this case, and even the same driver can behave differently based on how it's called. We should iron this out in the future, in the meantime double your %%...


Avoiding special run to initialize on-disk store?

If I understand correctly, right now this is needed for incremental parsing:
goaccess -f first.log --keep-db-files

Followed by:
goaccess -f next.log --load-from-disk --keep-db-files
goaccess -f another.log --load-from-disk --keep-db-files

Is there a way to avoid the first invocation, and just have the store initialized if it doesn't exist already?
If not, is this a reasonable way of initializing the store?
goaccess -f /dev/null --db-path=/path/to/db  --keep-db-files

I'm not sure I follow the use case, however, you should be able to do the following against the first.log
goaccess -f first.log --load-from-disk --keep-db-files

If you just want to initialize the store, you could run goaccess as you posted above.
goaccess -f /dev/null --keep-db-files

Just make sure you have a valid date/time/format in your config file so it won't complain about it.

I'm not sure I follow the use case, however, you should be able to do the following against the first.log

goaccess -f first.log --load-from-disk --keep-db-files

With first.log I meant to say that this would be the first time that goaccess is run, so db files wouldn't exist, and if I understand the man page correctly, they need to exist before goaccess can be invoked with --load-from-disk, no?
       --load-from-disk
              Load previously stored data from disk. Database files need to exist. See keep-db-files.

You can run goaccess for the first with --load-from-disk. e.g.,
goaccess -f first.log --load-from-disk --keep-db-files

In fact, you can run goaccess as:
goaccess -f first.log

It will create the database files even if they don't exist. Passing --load-from-disk for the first time, it's like a no-op.

Closing this. Feel free to reopen it if needed.

Thank you! It works as you described. Perhaps it's worth checking that man page entry for ambiguity. Then again, if I'm the only one who's misunderstood, PEBKAC. :)

Thanks. I have updated the man page and the site as well.


v4.2.8

replace html-minifier with html-minifier-terser (#157)
add rule to people.com
update dependencies

Coverage increased (+99.6%) to 99.648% when pulling 237350ad28ef2600d886abd0896aeabb3d8282af on v4.2.8 into f1e18b005a96390391d348c1c212ea6a66144df9 on main.

Coverage increased (+99.6%) to 99.648% when pulling 237350ad28ef2600d886abd0896aeabb3d8282af on v4.2.8 into f1e18b005a96390391d348c1c212ea6a66144df9 on main.


Plugin version does not match release version

https://github.com/Tagirijus/AddShortcuts/blob/8baa59233405266acbc395acbc91194e31143527/Plugin.php#L45

Yep, went wrong during my plugin rename session. Thanks for the hint, I just fixed it.


How can i get the virtual machine power on status?

As the title, how can i get the virtual machine status, such as power off, deallocated from the sdk?

I believe you want InstanceView() but not totally sure.  Can you try it?  If it doesn't work we can engage somebody from the compute team to answer.
https://github.com/Azure/azure-sdk-for-go/blob/master/services/compute/mgmt/2018-10-01/compute/virtualmachines.go#L574

yes, i have seen that the InstanceView returns statues[1].code with the value PowerState/running, but i'm not sure it is the right answer because it is un-documented.

@hyonholee can you help answer?

I'd like this as well. Please let it not be added to the list of issues that will be ignored.

Statuses[1].DisplayStatus of InstanceViewStatus will show power state.

Are there links to documentation which will display all the valid values of InstanceViewStatus/DisplayStatus?

Are there links to documentation which will display all the valid values of InstanceViewStatus/DisplayStatus?

@tariq1890 have you found a list of possible values?

@tariq1890 @anton-yurchenko
I found this:
https://docs.microsoft.com/en-us/azure/virtual-machines/states-lifecycle


[Internal] Diagnostics: Fixes performance of Region Contacted Information

Description
Getting region Contacted information from the diagnostics is very inefficient today. It is affecting Client Telemetry Performance by 2% to 3% .
Right now, Overall impact is 5% (Comparing run with Telemetry and Without Telemetry)
Performance Test:
Without Change:

After this change:

Reduction in memory allocation and increase RPS is noticed for this particular function.
Client Telemetry Stats (Calculating Overall Impact after the Change):

Current Impact is:
Maxima Difference = 11446 (w/o Telemetry) - 10985 (with Telemetry) = 461 = 4%
Minima  Difference = 10573 (w/o Telemetry) - 10229 (with Telemetry) = 344 = 3%
Overall Improvement in Telemetry is 1% - 2%.
Type of change

[] Bug fix (non-breaking change which fixes an issue)

Closing issues
To automatically close an issue: closes  #2846 and closes #2952

What's the impact with-out regions-contracted collection in telemetry?

What's the impact with-out regions-contracted collection in telemetry? This data point will help us understand how much is it contributing even with optimization.

https://github.com/Azure/azure-cosmos-dotnet-v3/issues/2952 This issue has all my RCA documented.


What is the initial password and default IP in Hotspot mode?

Hello and thanks for this cool project!
I'm trying to connect to a Wattpilot, that has been factory resetted with the RFID card.
What is the default ip? My client gets an IP address in <IP_ADDRESS>/24. But the Wattpilot doesn't seem to respond to pings, so it's not trivial to find the host. And what is the initial password? Is it the “hotspot key“ or the ”V.code“?
Best,
Matthias

Hi, the initial IP is <IP_ADDRESS> and the wattpilot should also answer pings.
The initial password is empty, and can only be set with the Solar.Wattpilot app, but on hotspot you never need a password.

Hey, thanks for the quick answer!
And how do I set an empty password? wattpilotshell terminates itself:
$ wattpilotshell
Traceback (most recent call last):
  File "/usr/local/bin/wattpilotshell", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.10/dist-packages/wattpilot/wattpilotshell.py", line 1185, in main
    Cfg.set(os.environ)
  File "/usr/local/lib/python3.10/dist-packages/wattpilot/wattpilotshell.py", line 1152, in set
    assert not d.required or d.val, f"{var.name} is not set!"
AssertionError: WATTPILOT_PASSWORD is not set!

I am not familiar with wattpilotshell, but I guess maybe setting any password will just work. You can also connect with the Fronius app first, and actually set a password which will the also allow you to connect over local network.

The Fronius app for Android is completly broken. I tried to just set the wifi on three different days now and it just hangs. I need to make it work with this project or by an iphone, where the app shall work.


Status report: allow exports

Exports were deactivated from the status report, as the current report export functionality is setup around longitudinal data.
The status report export should only include latest records, it should be similar to the questionnaire export functionality.
This will likely need backend changes.

 Status report


bug: compose/enviroment.sh does not work for podman

That is because it sets an alias docker=podman and then tries to use it.
Fix:

still set the alias
but also set an env variable docker=docker or docker=podman
change command line to $docker from docker

ALSO
alias ec='docker compose'
should be
alias dc='docker compose'

Propogate this change to ioc-adsimdetector and release as 4.0.1

I believe works on our diamond workstations because by default we have alias expansion enabled:
[esq51579@pc0146 ~]$ [ "$(shopt -p expand_aliases)" == "shopt -s expand_aliases" ] && echo "Enabled" || echo "Disabled"
Enabled

However it failed on a training laptop, we should probably not rely on alias expansion in a script


Concurrent update with upsert=true results in E11000

Hi,
I am using the latest version of ReactiveMongo 0.11.9 and Mongo 3.2. I have three update operations:
val query = BSONDocument("_id" -> e.id.value)
val update = BSONDocument(
      "authorId" -> e.authorId.value,
      "fileName" -> e.fileName,
      "start" -> e.start,
      "stop" -> e.stop,
      "duration" -> e.duration
)
    questionCollection.update(query, BSONDocument("$set" -> update), upsert = true)

questionCollection.update(BSONDocument("_id" -> e.id.value), BSONDocument("$inc" -> BSONDocument("friendsLikes" -> 1)), upsert = true)

questionCollection.update(BSONDocument("_id" -> e.id.value), BSONDocument("$inc" -> BSONDocument("friendsLikes" -> -1)), upsert = true)

All of above operations are executed concurrently and sometimes they fails with such response:
reactivemongo.api.commands.UpdateWriteResult: DatabaseException['E11000 duplicate key error collection: maindb.questions index: _id_ dup key: { : "28e6f90f0e8a04a7f3800b0354eebf362b3c40366080aa4e2dde74e98c53b655" }' (code = 11000)]

As far as I understand update with upsert=true should create document only if it does not exist. So why I am receiving the error with duplicated _id while I am updating the document?

Hi, please use the MailingList to ask question. Best regards.


Azure Embedding Quota Limit

Describe the bug
I am running a detailed report with Azure Openai, and am hitting quota limits. While I have a rate limit activated of 500k tokens per minute, it seems to still throw an error and not handle the throttling request well.
openai.RateLimitError: Error code: 429 - {'error': {'code': '429', 'message': 'Requests to the Embeddings_Create Operation under Azure OpenAI API version 2024-02-15-preview have exceeded call rate limit of your current OpenAI S0 pricing tier. Please retry after 86400 seconds. Please go here: https://aka.ms/oai/quotaincrease if you would like to further increase the default rate limit.'}}
Of course the error message should be false, as my rate limits are not per day, and waiting 24h is not an option.
Expected behavior

The log to the user (e.g. websocket) should indicate that there is throttling in place.
The embedding should resume after a while, or indicate what the user can do instead.

i have the same issue.

Fair point, we'll have to think how to make this smoother.
A) @danieldekay, is it the same docs that you're running reports on.
Have a look at this PR: "Documents, crawled urls, and website will be chunked and loaded to the inputted vector store if vector_store is not None."
https://github.com/assafelovic/gpt-researcher/pull/838
Meaning, if you run GPTR with the same Langchain vectorstore, perhaps it will cut down the embeddings processes.
B) the "cooling off" feature is also a good idea. Did you mention somewhere that there's a Langchain method we can leverage to get the required "cool off" period?
Once we have that, we can go about adding the websocket message. Adding an exception handle would also be a good first step which publishes a websocket message to the frontend

Awesome.
Adding to the resilience channel on Discord.
For anyone reading who hasn't joined the Discord, Join here to access the above link

I solved the issue by
_embeddings = AzureOpenAIEmbeddings(
model=model,
timeout=60,
chunk_size=1000,
azure_endpoint=os.environ["AZURE_OPENAI_ENDPOINT"],
openai_api_key=os.environ["AZURE_OPENAI_API_KEY"],
openai_api_version=os.environ["AZURE_OPENAI_API_VERSION"],
**embdding_kwargs,
and changing to azure_openai:text-embedding-3-large
If you think this is correct solution i can add a pull request
The suggested solution (chat_model_rate_limiting) is not working for AzureOpenAIEmbeddings in the current version of Langhian. https://python.langchain.com/docs/how_to/chat_model_rate_limiting/

Ah - just realized 86400 seconds is equal to 1 day :) - that's a good amount of time.
I haven't played much with the Azure config, so I prefer to delegate to @danieldekay :)
Whatever you think will make the flow smoother Dan - maybe we should set a default embedding model in the config?

Has someone solved this?

Sure @roninio,
Green light for the PR - maybe we should also set a default azure embedding model in the config?

Hi create a pull request https://github.com/assafelovic/gpt-researcher/pull/979
also updated documentation regarding Azure


pdf24

Add pdf24 to tools:  https://www.pdf24.org/en/

I can work on this, can you assign this issue to me?

And could you add gssoc 2k23 label

Made the pr @JasonDsouza212


Consider using personal-files interface in Snap

Snapcraft added a new interface that allows snaps to access files under $HOME: https://forum.snapcraft.io/t/the-personal-files-interface/9357 This means our Newsboat snap can now store its config and data in ~/.newsboat or XDG equivalents just like non-snap version.
TODO:

[ ] find out what the upgrade path will be. Does personal-files interface prevent users from using Snap's versioned paths?
[ ] add it to Snapcraft.yaml

Actually, using ~/.newsboat from Snap doesn't seem like a good idea anymore. Snap is all about isolation, and going against that doesn't seem necessary. Also, no-one requested this, so I'm closing this issue without any guilt.


Unhandled Error in output-app_hooks

Hi everone,
I want to call a python-script after a successful login. The configuration is below:
#-----------------------#
#   APPLICATION HOOKS   #
#-----------------------#
[output-app_hooks]

# Enable app_hooks output plugin
#
# input: true/false
# required: YES
# default: false
enabled = true

[...]

# Calls the script when a login is successful with the following arguments
# ./script LOGIN_SUCCESSFUL DATETIME IP USERNAME PASSWORD
#
# input: path of script to run
# required: NO
login_successful = /home/honssh/test.py

After a successful login i got this error message:
2018-06-03 00:44:25+0200 [-] Unhandled error in Deferred:
2018-06-03 00:44:25+0200 [-] Unhandled Error
	Traceback (most recent call last):
	  File "/usr/lib/python2.7/threading.py", line 801, in __bootstrap_inner
	    self.run()
	  File "/usr/lib/python2.7/threading.py", line 754, in run
	    self.__target(*self.__args, **self.__kwargs)
	  File "/usr/lib/python2.7/dist-packages/twisted/_threads/_threadworker.py", line 46, in work
	    task()
	  File "/usr/lib/python2.7/dist-packages/twisted/_threads/_team.py", line 190, in doWork
	    task()
	--- <exception caught here> ---
	  File "/usr/lib/python2.7/dist-packages/twisted/python/threadpool.py", line 246, in inContext
	    result = inContext.theWork()
	  File "/usr/lib/python2.7/dist-packages/twisted/python/threadpool.py", line 262, in <lambda>
	    inContext.theWork = lambda: context.call(ctx, func, *args, **kw)
	  File "/usr/lib/python2.7/dist-packages/twisted/python/context.py", line 118, in callWithContext
	    return self.currentContext().callWithContext(ctx, func, *args, **kw)
	  File "/usr/lib/python2.7/dist-packages/twisted/python/context.py", line 81, in callWithContext
	    return func(*args,**kw)
	  File "/opt/honssh/honssh/output/output-app_hooks.py", line 82, in login_successful
	    self.runCommand(command)
	  File "/opt/honssh/honssh/output/output-app_hooks.py", line 163, in runCommand
	    sp = subprocess.Popen(args, shell=False, stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
	  File "/usr/lib/python2.7/subprocess.py", line 711, in __init__
	    errread, errwrite)
	  File "/usr/lib/python2.7/subprocess.py", line 1343, in _execute_child
	    raise child_exception
	exceptions.OSError: [Errno 13] Permission denied

Is this a bug in honssh or is it my mistake? Adding the code in the output-app_hooks.py works, but is not suitable.

For anyone else reading this, please ensure the script is executable and have a hash bang at the top, e.g.
#!/usr/bin/env python


Clean up DisplayServerAndroid::window_get_native_handle() with the GLES3 renderer

This is a small cleanup for DisplayServer::window_get_native_handle() on Android.
This is a bug, because without these changes, calling window_get_native_handle(DisplayServer::OPENGL_CONTEXT) will do something unpredictable if we're using Vulkan, and window_get_native_handle(DisplayServer::DISPLAY_HANDLE) will always return 0.
This also updates openxr_opengl_extension.cpp to use this to get the EGLDisplay, rather than interacting with EGL directly.

Thanks!


Share button is broken

Previously it opened a dialog that allowed to copy a share URL,
now it navigates to https://github.com/dart-lang/dart-pad/wiki/Sharing-Guide

Thanks for reporting! Yes, we got impacted by GitHub deprecating anonymous gists. #805 tracks bringing reenabling a sharing flow via an auth flow. Closing this bug as for now the button works as intended.


Pin twine to v1.15.0 to support Python 3.5 for longer

Codecov Report

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

@@          Coverage Diff          @@
##           master    #16   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           7      7
  Lines         128    128
  Branches       16     16
=====================================
  Hits          128    128

Continue to review full report at Codecov.

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


Update setup to reflect README.rst

Massive fix. Update the setup.py profile to the proper README extension. Currently pip install doesn't work because of this bug.

Thanks! Btw it's an RST now because sphinx does everything with RST, not markdown.

Yep, I've been wranging RST/sphinx for climpred. Although it's super low priority since we haven't got a really functional version going yet. Working on transitioning to object-oriented currently.


Update the contact section

Enhance the UI of the contact section

Hello, I want to work on this issue please assign me this work under label hacktoberfest.
Thank You!

Hey , I want to work on this issue. I'm sure that I'll help in this . Assign me please


Corrected Naming of the stack_template file

should I create it as an issue rather than PR?
because this is a very minute change

no need


"Stepper evaluation errored" on call to token.approve(spender, value)

How to reproduce: KEVM symb-testing, modified to run this solidity code:
    function test_approve() external {
        ERC20 token     = SymbolicTestBuiltins(this).new_ERC20_with_arbitrary_storage();
       address spender = SymbolicTestBuiltins(this).create_symbolic_address();
        uint256 value   = SymbolicTestBuiltins(this).create_symbolic_uint256();
        bool success = token.approve(spender, value);
    }

Execution time: >6 hours, left overnight.

step2440-and-before.txt

Issue fixed at KEVM end so probably no longer relevant.


Maximum index

Using 4x4 matrix: How do you ensure that two rows does not have their maximum values at the same column indexes?

How do you implements 3x3 jigsaw puzzle?


Add Rust impl of wasmtime_ssp_fd_prestat_get

This PR addresses #120. It ports wasmtime_ssp_fd_prestat_get from C to Rust.
I'd like to use this PR to discuss a few things regarding all manner of prestat_ related things in wasmtime. Firstly, please note that I've decided to try and go the least intrusive way possible when porting fd_prestate_get functionality; that is, my goal was to change as few things at Rust's side of wasmtime-wasi as possible.
Summary of changes:

I've made all rwlock_ functions in locking.h non-static and non-inlined to be able to link to them via bindgen. The main reason for this was to avoid having to mess around with re-implementing in Rust fd_prestats struct which requires read-write locking. Having said that, however, IMHO it should ultimately be rewritten in safe Rust.
I've made fd_prestats_get_entry non-static and bindable by bindgen. However, since the functionality of this function is relatively simple, it shouldn't prove hard for it to be rewritten in Rust, and hence, should be done ASAP.
I've moved the definition of fd_prestat from posix.c into posix.h in order for bindgen to be able to properly bind it. Here, I was mainly after getting access to the only field that the struct features, namely const char* dir.
wasmtime_ssp_fd_prestat_get is currently a direct translation of C version into unsafe Rust, so any suggestions in improving its safety are more than welcome.

Please leave as many suggestions and comments as possible!

I think your approach of making things non-static makes sense. Once everything is in Rust, these will go away anyway :-).

Exactly my thinking as well! :-)

I reckon that looks good!


LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20240204164051081 to main

LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20240204164051081 to main with localized lcls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.002%) to 41.759%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/dialog/configurePython/configurePathPage.ts
1
57.02%

Totals

Change from base Build<PHONE_NUMBER>:
-0.002%

Covered Lines:
30828

Relevant Lines:
69082

💛  - Coveralls


#113 [Bug] [Tech] Locale loading fails if browser language is set to us_US

Closes #113 . Hopefully.


Create a docker image with postgres

Hey,
When you want to use this awesome project with k8s and need PostgreSQL, it's extremely hard to achieve without a prebuilt docker image that already contains the PostgreSQL driver preconfigured.
Can you please add to your release?
Thanks,
Dan

Yes! The easiest option would be to include the Postgres driver directly. For example, like it was done recently for ZeeQS: camunda-community-hub/zeeqs#163

Done by #307 :tada:


Parse to tibetan other wylie dictionaries.

[ ] convert {wylie} to tibetan script

Hard to realize it to work fast and correct.


C# Port

Извиняюсь, если я должен был задать этот вопрос в другом месте.
Я очень люблю эту библиотеку, но мне также очень нравится многофункциональная среда, такая как общий языковой среды выполнения (CLR) и C#.
Меня интересует, возможно ли мне портировать следующие библиотеки на C#:
browser-with-fingerprints
playwright-with-fingerprints
Или это невозможно из-за необходимости портировать дополнительный код?
Буду признателен за вашу помощь и ответ на этот вопрос.
English
I apologize if this should be posted somewhere else.
I really love this library but I also really love a proper multi threading environment such as common language runtime and C#
I wish to know is it possible for me to port these libraries
browser-with-fingerprints
playwright-with-fingerprints
to C#?
or is this not possible because I need to port additional code
Thank you

Причина заключается в том, что я хочу написать код, который будет запускать один новый скрейпер-браузер каждые 10 секунд, и управление таким потоком кажется нелепым в Node.js из-за его однопоточности. Я предпочитаю использовать C# вместо рабочих процессов и пула потоков Node.
English
The reason is because I wish to write code that will run one new scraper browser every 10 seconds and managing that through put is just silly with node.js because of single threaded and I prefer c# over node workers / threadpool

C# is not supported.

The reason is because I wish to write code that will run one new scraper browser every 10 seconds and managing that through put is just silly with node.js because of single threaded and I prefer c# over node workers / threadpool

You don't need to create threads to start browsers or interact with them in parallel. All methods are async.

Thank you I really appreciate you writing back.

You don't need to create threads to start browsers or interact with them in parallel. All methods are async.

Async != Multithread - could you please comment on how you run a multi-threaded app using your code? Is it just node workers? It would really help if you could show a few lines of code to run multithreaded
Also if I ported the plugin code to c# do you feel it would work? or is there other reasons it can not work due to the overall system design.
Thank you

Ok, here is example
https://github.com/CheshireCaat/puppeteer-with-fingerprints/blob/master/examples/parallel.js

Also if I ported the plugin code to c# do you feel it would work? or is there other reasons it can not work due to the overall system design.

Library can be ported to C#


Added CI workflow to build example app for iOS and Android

This closes https://github.com/ably/ably-flutter/issues/419. I've implemented a new GitHub action to build example app for iOS and Android that runs along integration tests for merge requests.

Leaving this as a draft, because new steps will fail untill https://github.com/ably/ably-flutter/pull/421 is merged


Dropdown menu for Data source list

User Story:
The browser presents a list of graph genomes that can be browsed.  The user selects on from a list.  Program automatically switches to displaying the new graph genome from the beginning.
Currently, the user needs to know the exact name of the folder they'd like to open in Pantograph.  tryJSONpath checks to see if that folder exists and contains a bin2file.json.  Ideally, we'd just check /public/test_data and auto-populate a dropdown with valid folder names.
Related: Basically the same feature for a different input #41

I suspect that it is impossible to fetch all folder located under /public/dataset from JavaScript. That's because front-end JavaScript does not allow to access folders directly for security reasons. Instead, I assume we need to put folderlist.json or something to let Schematize know the entire list of valid folder names.

Or should we create a server which provides dataset (which is not good for JBrowse2 integration, though) ?
If we have a SPARQL endpoint which provides the entire dataset, then this problem is solved because we can ask the entire list of dataset.

Oh ouch, even if the place it's looking is in public/test_data/?  That doesn't sound correct.  It's already loaded in the npm server, and the Javascript can do HEAD checks to see if a file is there.

Sorry, what means the npm server? If we have known the file name, then it is possible.
Could you tell me the way to fetch the folder name list under public/test_data?
For example, every case seen in https://stackoverflow.com/questions/20822273/best-way-to-get-folder-and-file-list-in-javascript or https://www.npmjs.com/package/list-react-files, it requires fs to access the file system. However, fs does not work in client-side JavaScript.
One possible idea is to run a simple Http server on /public folder which lists all folders. But I am not sure that works both of the production build and development server.


yield local dataframe

Codecov Report

Merging #298 (9cffe63) into master (d1584c2) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #298   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           98       100    +2
  Lines         9339      9402   +63
=========================================
+ Hits          9339      9402   +63

Impacted Files
Coverage Δ

fugue/__init__.py
100.00% <100.00%> (ø)

fugue/execution/execution_engine.py
100.00% <100.00%> (ø)

fugue/registry.py
100.00% <100.00%> (ø)

fugue/workflow/_tasks.py
100.00% <100.00%> (ø)

fugue/workflow/workflow.py
100.00% <100.00%> (ø)

fugue_duckdb/__init__.py
100.00% <100.00%> (ø)

fugue_duckdb/execution_engine.py
100.00% <100.00%> (ø)

fugue_duckdb/registry.py
100.00% <100.00%> (ø)

fugue_sql/_visitors.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Support NVIDIA T4 and assign T4 pods to EKS cluster

Closes #635.
Main changes:

Tell Vivaria that the main cluster supports GPUs (by setting config.gpuMode and K8sHost#hasGPUs correctly)
Tell Vivaria to put H100 runs on the GPU cluster and other runs on the EKS cluster
Add GPU-related nodeSelectors to pods (not strictly necessary because the GPU cluster only has H100s and the EKS cluster only has T4s, but that could change in the future)

Covered by automated tests. Manual testing: I ran this code in staging and was able to use it to run a task that used T4 GPUs. I was also able to run a task that didn't have a GPU assigned. I haven't tested a task that uses H100s because there are many active pods on our GPU cluster.

Where would the explicit flag live? In manifest.yaml? Or maybe Vivaria configuration, like in server/.env?
It did cross my mind that this isn't particularly configurable and we'll need to make code changes when we want to add new GPU or other resource types, or move existing resource types between clusters.
I don't think manifest.yaml is the right place. I don't think users should get to decide that e.g. their general/count-odds runs get to run on the GPU cluster.
It doesn't seem worth making this configurable yet. That's more of a vibe thing -- the code to read the config seems complicated.

Gotcha. OK yeah maybe it isn't so bad. We could have per-cluster GPU_MODELS environment variables that are comma-separated lists of GPU models.
OK maybe in a follow-up PR!

I pushed a commit that... doesn't actually fix a bug. Because VIVARIA_API_IP_FOR_K8S_GPU_CLUSTER happens to be a Tailscale IP, which agents in the EKS cluster can contact just as well as they can contact API_IP. But it still seems good to have the logic here be correct.


Add Markov Chain or Markov Process algorithm

Wikipedia: https://en.wikipedia.org/wiki/Markov_chain
Implementations:

Random pronounceable words generator
Example in coffee-script (8 years old) here: https://github.com/SyntaxColoring/Markov-Word-Generator/tree/Markov.coffee

Random sentence generator
Example in ruby/python (7 years old) here: https://github.com/hrs/markov-sentence-generator

This sounds interesting, can I be assigned this?


[Enhancement] Integration with arrow2 and polars

Hi,
I want to make some contributions by integrating arrow2 and polars into this crate.
Like:

query -> arrow2 array -> polars DataFrame
polars DataFrame / arrow2 array -> query ->  polars DataFrame

The idea looks like this: https://github.com/therealhieu/rust-duckdb-workaround
If this makes sense, I will contribute a PR

Great idea. I had wanted to do this long time ago but haven't got the chance, good you can help!
Let's try split this into multiple PR, I think the first step is to replace all arrow-rs dependency to arrow2
Thanks!

I will keep the arrow-rs as current, becaucse I think it's doing well and essential.
Instead,  I will add some feature flags for this integration/conversion between arrow-rs and arrow.
Do you agree?

Also fine, but I may wondering how you gonna design the API.
From my side, if we still want to keep the arrow-rs, we may have an API like query_arrow2() so that we can avoid to convert from arrow to arrow2, it might be more efficient and more maintainable

Anyway, you can try first. Happy to review the code


Fix docker-hub uploader tagging

What
Describe what this PR is doing.
Why ?
Justification for the PR. If there is existing issue/bug, please reference. For
Bug fixes why and what can be merged in a single item.
How ?
It is optional, but for complex PRs please provide information about the design,
architecture, approach, etc.
Testing ?
How was this feature tested? Did you add unit tests? Did you add E2E tests?
Special triggers
Use the following phrases as comments to trigger different runs

bot:retest rerun Jenkins CI (to rerun GitHub CI, use "Checks" tab on PR page and rerun all jobs)
bot:upgrade run additional update tests

how did you test it ?

how did you test it ?

I uploaded a dummy container to dockerhub and deleted it after that


Add report actions when the card is deactivated, limit type or amount is changed

Problem
When admin takes an action regarding user's expensify card - deactivating, replacing a card, changing limit amount or limit type, the effect of this change can influence the users ability to use the card, for example they might not even be aware that the card was deactivated when this happens and left confused, maybe asking admin for clarification.
Solution
Adding a report action to the workspace chat communicates this well and leaves an audit trail as well for actions taken regarding the cards.
We would like to introduce 4 new report actions:

CARDDEACTICATED
CARDREPLACED
CARDLIMITAMOUNTCHANGED
CARDLIMITTYPECHANGED

The copy of the messages would be:

deactivated card ending in XXXX
replaced card ending in XXXX
changed limit to $X for card ending in XXXX (previously $Y)
changed limit type to X for card ending in XXXX (previously Y)

Issue OwnerCurrent Issue Owner: @mountiny

This is internal I will tackle this one

Reassigning as I'm going OOO for the next two weeks.

How are we doing with this @mountiny ?

Its in my backlog, this is polish and we are prioritizing the #quality bugs with workspace feeds

Making this monthly as its nice to have

Monthly

Monthly


feat(util): add (git) blame-nvim

adding https://github.com/FabijanZulj/blame.nvim with default settings and overriding <leader>gb keymap

Also have you tried f-person/git-blame.nvim? That's what I'm using, but I'm interested if you've tried both and can do a comparison between them. This is not a suggestion to change the plugin, just a question from a user perspective if possible.

<leader>gb is a default keymap that shows a git history for the cursor location.

I also don't really see the point of this extra, since it's simply adding the plugin.
Users can easily do that themselves

Also have you tried f-person/git-blame.nvim? That's what I'm using, but I'm interested if you've tried both and can do a comparison between them. This is not a suggestion to change the plugin, just a question from a user perspective if possible.

I didn't know about this. I'll give it a try :)

<leader>gb is a default keymap that shows a git history for the cursor location.
it is labeled "Git Blame Line" so I thought it would make sense to "Git Blame File" instead :)

I also don't really see the point of this extra, since it's simply adding the plugin. Users can easily do that themselves

oops! I thought it would make a nice addition. closing :)


Ensure the build check fails if there are new untracked files

One gap in the current check run is that if a PR generates new files during a build which aren't checked as part of the PR, we ignore this.
This at least creates confusion as to why there is are untracked files locally when building on main, but in the worst case could cause us to fail to check in something that might be required in certain runtime environments.

My expectation is that check dist/ verify build will fail as we currently have some untracked files on main's build. I'll add these in a follow-up commit so we have a clear red > green signal this is resolving an issue.


_get_workflow_data in Finder is broken in 11.3

Trying to run finder.model_id(model_id) on the metlife-dev.indico.domains and the call to self.get_workflow_data just hangs. Is it possible the query backing it is not in IPA v4.11.3?
Also begs the question on how to handle versioning. It's likely customers will not be on the most recent version of IPA and we need a way to control for older versions. Decorators on functionality that check if users IPA version is > what's needed for the function?
Also, is the IPA version exposed in the python client?

Should we allow users to filter the kind of results that get returned?

I wonder if above should be filed as its own IPA bug?


Refactor sagas.js to sagas/index.js

This refactoring puts all of the core sagas together in src/sagas and will allow future refactoring to further break down the new src/sagas/index.js to be a much easier and encapsulated task.
NOTE: With the file renames detected by git, it should not matter if this PR gets merged before other PRs with saga changes.

[ wait until end of next week to merge, since 4.3.3 is building mid week ]


Class small-centered not working v6.4.1

this code below used to work when i was using foundation-sites 6.3.1. the column is well centered with the class small-centered.

Test

But since i update to version 6.4.1, the column is no longer centered. the class small-centered has no effect.
Am i missing something?

@AleeBadara Seems to be working OK for me with a blank CodePen: https://codepen.io/brettsmason/pen/Ogvwwq
The grid was changed to use XY grid by default from 6.4 so the float and old flex grid are not the defaults. How are you using Foundation? Is using Sass you need to make sure you set the correct grid includes.

Thank you @brettsmason. I use npm to install foundation-sites and webpack to bundle all my files. I think it's me. I'm missing something. I'll have a look.

@AleeBadara  I'm facing the same issue, have you fix it?

@AleeBadara @BondoS The small-centered class is part of the float grid which is no longer the default grid.
By default with 6.4 the variable $global-flexbox is set to true, which will set it to use the xy grid (or legacy flex grid). In order to use the float grid set this to false (assuming you are using @include foundation-everything).  See here for more details on enabling/disabling certain components too. Alternatively if you are using precompiled CSS then make sure you use foundation-float.css as this has the class you want in it.
Does that help?

@BondoS if you are using the foundation.css file, you also have to import the foundation-float.css as @brettsmason said.
here is my code. A React-Webpack project:
require('style-loader!css-loader!foundation-sites/dist/css/foundation.min.css');
require('style-loader!css-loader!foundation-sites/dist/css/foundation-float.min.css');
Thanks to you 2.

Going to close this as this should be solved, but please reopen if you are still having issues.


Visualization for Uint8Arrays

Node buffers are of type Uint8Array. I would be nice to have the same visualization features of buffers from base64 strings.

@pepkin88 Fixed. Thanks :smile:


Improved joinURLs so that two qualified URLs can be joined.

Previously, joining http://www.domain.com/some/path with http://www.domain.com/another/path would result in http://www.domain.com/another/path. Now it results in http://www.domain.com/some/path/another/path which is more intuitive and useful.
Also tweaked CesiumTerrainProvider so that the tile URL templates can be overridden from the options.

Thanks @keattang!  This change seems fine to me, does anyone else have an opinion?

Done :)

@keattang and @hpinkos I'm not really sure the new logic makes sense to me.  In my mind, joinUrls is similar to .NET's Path.Combine or Apache Commons' [FilenameUtils.concat](https://commons.apache.org/proper/commons-io/javadocs/api-1.4/org/apache/commons/io/FilenameUtils.html#concat(java.lang.String, java.lang.String)), both of which will return the second path and ignore the first if the second is absolute.  I'm not really sure why you would want joinUrls to work in the way you've described.  If http://www.domain.com/another/path is a valid URL, there's no reason to expect that http://www.domain.com/some/path/another/path is a URL that makes any sense at all.

Hey @kring I made the changes so that I could feed CesiumTerrainProvider a url that looks like https://cloudfront.com/some/folder/ and a list of URL templates that look like https://cloudfront1.com/{z}/{x}/{y}.terrain, https://cloudfront2.com/{z}/{x}/{y}.terrain, etc. and get URLs out that look like https://cloudfront1.com/some/folder/{z}/{x}/{y}.terrain.
It's a bit hacky but the changes to joinUrls to support it seemed to make sense. However if you think they're not appropriate I'm happy figure out another way to do it. Either way though I'll probably need the ability to override the tiles option so I might make that into a separate PR.

Sorry, I still don't understand where such URLs would come from.  If you'll be overriding the tiles option anyway, why not just provide the correct absolute path in the first place?

@kring What's the status here, are we just closing this?

Yeah, thinking about this more, I don't think it makes sense for how we intended the joinUrls function to work.  I'm going to close this PR.


fix(cordova/apple/macos): fix popover click and location issues

I think this fixes the following popover issues on macos:

Clicking events are incorrectly caught and close the popover #1727
The popover is opened in the bottom right corner on initial load #1653

This should allow us to release a fixed old macos client while we try to resolve the VpnExtension's gomobile issues for Catalyst on arm64.

Any idea why the iOS build is failing?


Auto select profile

Help request
Your Environment

Netflix add-on version:
Operating system version/name:
Device model: Nvidia shield pro

Used Operating system:

[x] Android
[ ] iOS
[ ] Linux
[ ] OSX
[ ] Raspberry-Pi
[ ] Windows

Describe your help request
Hello, is it possible somehow to auto select the profile instead of staying and select the one i want to use every time i start a movie ?  I have 2 profiles on my netflix account and every time i start a movie i have to select one of the profile.
Thank you
Screenshots

https://github.com/CastagnaIT/plugin.video.netflix/wiki/FAQ-(Audio%2C-Video%2C-Subtitle%2C-Other)#how-to-set-the-automatic-selection-of-a-profile


Always ouput Resolving dependencies... to user

$ bundle exec pod install --ansi --verbose
  [DEPRECATED] Your Gemfile contains multiple primary sources. Using `source` more than once without a block is a security risk, and may result in installing unexpected gems. To resolve this warning, use a block to indicate which gems should come from the secondary source. To upgrade this warning to an error, run `bundle config set disable_multisource true`.
  Resolving dependencies...

The Resolving dependencies... is output by bundler, Is it a bug?
The bundler is 2.1.0.pre.2.

If you have a Gemfile.lock file, it shouldn't print that because it doesn't need to resolve dependencies. Can you rerun with the --verbose flag to see why it's re-resolving?

I guess you found out about the culprit here? Could you give me, and anybody reading this with the same problem, a bit more info about the resolution? Thank you!


Medical roles should be able to change the holocard of a marine in the health analyzer UI

Does a holocard system currently work or even exist?

Does a holocard system currently work or even exist?

Not yet

Is it okay to allow anyone using a medical scanner or medHUD to change holocards or should there be a skill check?

Is it okay to allow anyone using a medical scanner or medHUD to change holocards or should there be a skill check?
There should be a skill check, in cm13 it requires level 2 in medical skill.

When should the hud show a red skull? I've seen it a few times and I think it's related to holocards, but never understood the specifics.


Need some help for experiments on other dataset.

I change some configs and use other dataset. The size of my dataset is relatively small. So I reduce the total iterations to 10k.
For test results, the mAP of denseteacher is much lower than corresponding supervised model(relative to the coco dataset). There may be several factors:

The total iterations are too small for model convergence.
The size of dataset is too small.
The strategy of unlabel image sampling affects a lot.

I do not kown which is the main factor. Can you give me some advice? Here are some logs.

2022-10-07 21:39:25.569 | INFO     | cvpods.utils.dump.events:write:253 - eta: 0:18:44  iter: 38780/40000  total_loss: 1.656  loss_cls: 0.143  loss_box_reg: 0.181  loss_quality: 0.426  distill_loss_logits: 0.421  distill_loss_quality: 0.412  distill_loss_deltas: 0.141  fore_ground_sum: 171.450  time: 0.8847  data_time: 0.0075  lr: 0.002500  max_mem: 5055M
2022-10-07 21:39:34.673 | INFO     | cvpods.utils.dump.events:write:253 - eta: 0:18:35  iter: 38790/40000  total_loss: 1.792  loss_cls: 0.115  loss_box_reg: 0.158  loss_quality: 0.406  distill_loss_logits: 0.600  distill_loss_quality: 0.416  distill_loss_deltas: 0.174  fore_ground_sum: 175.650  time: 0.8847  data_time: 0.0084  lr: 0.002500  max_mem: 5055M
2022-10-07 21:39:44.712 | INFO     | cvpods.utils.dump.events:write:253 - eta: 0:18:26  iter: 38800/40000  total_loss: 1.694  loss_cls: 0.121  loss_box_reg: 0.164  loss_quality: 0.413  distill_loss_logits: 0.508  distill_loss_quality: 0.416  distill_loss_deltas: 0.156  fore_ground_sum: 163.533  time: 0.8847  data_time: 0.0119  lr: 0.002500  max_mem: 5055M

thanks, we do have some experience on data scaling.
Firstly, if your sup dataset is small and training time is short, consider using smaller burn_in_step so the teacher shouldn't overfit when SSL begin. If you want to check whether training has converged, check if the mAP is stable, a converged mAP curve is like:

Secondly, the TRAINER.DISTILL.RATIO should be changed for different dataset, this hyper-param is sensitive to dataset as introduced in our paper, and its optimal value can be estimated using dataset.
Besides, we don't see the effect of different unsup data sampling on coco, it won't matter if you have enough unsup data.

Thanks for your sincere advice~
I tried several seeds of sampling sets and got some normal results(similar with experiments on coco).
It seems my small dataset and the imbalance category distribution lead extreme sampling sets.  For example, results appear normal with seed5 and appear strange with seed1.


feat: compute content source for helix 4 config

Please ensure your pull request adheres to the following guidelines:

[ ] make sure to link the related issues in this description. Or if there's no issue created, make sure you
describe here the problem you're solving.
[ ] when merging / squashing, make sure the fixed issue references are visible in the commits, for easy compilation of release notes

If the PR is changing the API specification:

[ ] make sure you add a "Not implemented yet" note the endpoint description, if the implementation is not ready
yet. Ideally, return a 501 status code with a message explaining the feature is not implemented yet.
[ ] make sure you add at least one example of the request and response.

If the PR is changing the API implementation or an entity exposed through the API:

[ ] make sure you update the API specification and the examples to reflect the changes.

If the PR is introducing a new audit type:

[ ] make sure you update the API specification with the type, schema of the audit result and an example

Related Issues
Thanks for contributing!

:tada: This PR is included in version 1.68.0 :tada:
The release is available on:

v1.68.0
GitHub release

Your semantic-release bot :package::rocket:


Measuring Uncertainty - epistemic & aleatoric

Firstly, thanks for the amazing repo. I'm a complete beginner and this repo really broke BNNs down simply in a very hands-on manner.
Could you guide me on how to calculate the uncertainty and break it into its epistemic & aleatoric components using TyXe?
Thanks!

I believe that this library may only handle epistemic uncertainty currently. From section 2.1.2 of the TyXe paper:

At this time, we restrict the probabilistic model definition to weight space priors.

Weight uncertainty only accounts for the epistemic uncertainty of the model. To model the aleatoric uncertainty, one would need to incorporate a model for noise on either the inputs/outputs (or both!) a la Decomposition of Uncertainty in Bayesian Deep Learning for Efficient and Risk-sensitive Learning.


Auto-upgrade fails when using fetchTarball with a .tar.gz

Issue description
I'm using the following...
{
    # ...
    unstable = (import (fetchTarball https://github.com/NixOS/nixpkgs-channels/archive/nixos-unstable.tar.gz) combinedOptions);
    # ...
}

... to install a number of packages from the unstable channel. This works fine when running a nixos-rebuild manually, but it fails in the auto-upgrade service, with the following error:
[...]
Aug 24 04:40:46 desktop-home nixos-upgrade-start[22952]: unpacking ‘https://github.com/NixOS/nixpkgs-channels/archive/nixos-unstable.tar.gz’...
Aug 24 04:40:46 desktop-home nixos-upgrade-start[22952]: tar (child): gzip: Cannot exec: No such file or directory
Aug 24 04:40:46 desktop-home nixos-upgrade-start[22952]: tar (child): Error is not recoverable: exiting now
Aug 24 04:40:46 desktop-home nixos-upgrade-start[22952]: tar: Child returned status 2
Aug 24 04:40:46 desktop-home nixos-upgrade-start[22952]: tar: Error is not recoverable: exiting now
Aug 24 04:40:46 desktop-home nixos-upgrade-start[22952]: error: program ‘tar’ failed with exit code 2
Aug 24 04:40:46 desktop-home nixos-upgrade-start[22952]: (use ‘--show-trace’ to show detailed location information)
[...]

Presumably, this is caused by this line in the service, which appears to only specify a limited set of packages that doesn't include gzip. It seems the systemPackages are not included in the PATH at all.
I'm not sure how this could be solved correctly; the quick fix would be to add gzip to the path attribute, but this doesn't really scale to other custom logic that people might have in their configuration.nix. Perhaps all systemPackages should be explicitly added to the path as well, so that the behaviour is the same as when running nixos-rebuild from a shell?
Steps to reproduce

Use fetchTarball in configuration.nix, pointing at a .tar.gz file.
Set up the autoUpgrade service: system.autoUpgrade.enable = true;
Wait for the timer to roll around, or trigger the unit manually :)

Technical details

System: 17.03.1700.51a83266d1 (Gorilla)
Nix version: nix-env (Nix) 1.11.13
Nixpkgs version: "17.03.1700.51a83266d1"
Sandboxing enabled: false

You should be able to override in your own configuration.nix:
{
  systemd.services.nixos-upgrade.path = [];
}

Hmm, that's not really a generalizable solution though - if I use something new, this will break again, and it wouldn't be obvious unless I diligently checked my nixos-upgrade unit logs.
It'd probably be more practical to have something as a default that just uses the current system environment, since that way it'd match the command-line behaviour and therefore provide the least astonishment.