Added CLAHE adjustment the all the anatomical canvases.

Fixes #1660

bug:
~/sct_example_data/mt $ sct_label_utils -i mt0.nii.gz -create-viewer 2,4

--
Spinal Cord Toolbox (image-adjustment/8f288717a815db14d6d2c39a3f31e0a96bc5e262)
Running /Users/julien/code/sct/scripts/sct_label_utils.py -i mt0.nii.gz -create-viewer 2,4
/Users/julien/code/sct/python/lib/python2.7/site-packages/matplotlib/font_manager.py:273: UserWarning: Matplotlib is building the font cache using fc-list. This may take a moment.
  warnings.warn('Matplotlib is building the font cache using fc-list. This may take a moment.')
Traceback (most recent call last):
  File "/Users/julien/code/sct/scripts/sct_label_utils.py", line 824, in <module>
    main()
  File "/Users/julien/code/sct/scripts/sct_label_utils.py", line 814, in main
    processor.process(process_type)
  File "/Users/julien/code/sct/scripts/sct_label_utils.py", line 112, in process
    self.output_image = self.launch_sagittal_viewer(self.value)
  File "/Users/julien/code/sct/scripts/sct_label_utils.py", line 653, in launch_sagittal_viewer
    launch_sagittal_dialog(self.image_input, output, params)
  File "/Users/julien/code/sct/spinalcordtoolbox/gui/sagittal.py", line 100, in launch_sagittal_dialog
    dialog = SagittalDialog(controller)
  File "/Users/julien/code/sct/spinalcordtoolbox/gui/base.py", line 110, in __init__
    self._init_ui()
  File "/Users/julien/code/sct/spinalcordtoolbox/gui/base.py", line 118, in _init_ui
    self._init_canvas(layout)
  File "/Users/julien/code/sct/spinalcordtoolbox/gui/sagittal.py", line 52, in _init_canvas
    self.sagittal = widgets.SagittalCanvas(self, plot_points=True, annotate=True)
  File "/Users/julien/code/sct/spinalcordtoolbox/gui/widgets.py", line 243, in __init__
    self._init_ui(self._image.data[:, :, self._z], dx / dy)
  File "/Users/julien/code/sct/spinalcordtoolbox/gui/widgets.py", line 143, in _init_ui
    self.adjust_image(data),
  File "/Users/julien/code/sct/spinalcordtoolbox/gui/widgets.py", line 177, in adjust_image
    return exposure.equalize_adapthist(data)
  File "/Users/julien/code/sct/python/lib/python2.7/site-packages/skimage/color/adapt_rgb.py", line 40, in image_filter_adapted
    return image_filter(image, *args, **kwargs)
  File "/Users/julien/code/sct/python/lib/python2.7/site-packages/skimage/exposure/_adapthist.py", line 76, in equalize_adapthist
    image = img_as_uint(image)
  File "/Users/julien/code/sct/python/lib/python2.7/site-packages/skimage/util/dtype.py", line 325, in img_as_uint
    return convert(image, np.uint16, force_copy)
  File "/Users/julien/code/sct/python/lib/python2.7/site-packages/skimage/util/dtype.py", line 205, in convert
    raise ValueError("Images of type float must be between -1 and 1.")
ValueError: Images of type float must be between -1 and 1.

Another issue, already reported here, is that CLAHE introduces an artifact when the kernel size is not a multiple of the image dimensions. See below:
WITHOUT CLAHE:

WITH CLAHE:

important bug, please do NOT merge before fixing it.
data: sct_example_data/t2
syntax:
sct_label_vertebrae -i t2.nii.gz -s t2_seg.nii.gz -c t2 -initc2

master/caf11f46d92fe8765490a53325acbf0e2b36b8f1

image-adjustment/6fb676cbde7bf70ee7d3a5490cbc6e705774665c:

closing this because:

no activity for a month
manual adjustment of contrast is now possible, which makes this PR less required


Usage of measureBeforeMount makes all cards be aligned to the left

Describe the bug
Whenever I try to use measureBeforeMount to disable animations, the layout makes all items be aligned to the left.
I'm using width 100% for the parent of the RGL.
Your Example Website or App
https://codesandbox.io/p/sandbox/mj7kcq
Steps to Reproduce the Bug or Issue

Create a layout that needs items side by side.
Use the props measureBeforeMount as true.

Expected behavior
I expected to see my layout being followed and not ignored.
react-grid-layout library version
1.4.4
Operating System Version
Windows
Browser
Chrome
Additional context
No response
Screenshots or Videos
No response

same issue

Same to me, downgrading to 1.3.4 fix the issue


Add remaining teams

There are still a number of teams not configured here. Also gives the chance to rethink the old teams, like FixOurRails.

All teams are configured now.
I archived the FixOurRails repos, deleted the teams, and removed the people from the org that have just been member because of that.


add JSON processing capabilities to sensor_serial

Description:
Reading from the serial line strings/numbers is OK, but most likely you will be reading something more complex. This pull request address devices that publish in JSON format through the serial interface adding the value_template optional configuration (which exists in other sensors) and copying into attributes the JSON values.
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Checklist:
If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable ([example][ex-requir]).
[x] New dependencies are only imported inside functions that use them ([example][ex-import]).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

This also requires an update of the existing documentation.

Well, someone updated the documentation already :)
or am I missing something ?

./homeassistant/components/sensor/serial.py:117:1: D202 No blank lines allowed after function docstring
./homeassistant/components/sensor/serial.py:117:1: D208 Docstring is over-indented


A fatal error occurs when TEE is started.

Block height: 1,217,665
SGX authentication mode: IAS
Ubuntu kernel version: 5.15

TEE will report an error when running on a brand new machine. The error content is as follows:

problem seem to related here:

2024-11-28 latest log: max 15MB
2024-11-28T05:52:48.066277Z  INFO cifrost: bridge() exited with error: Rpc error: RPC error: ErrorObject { code: ServerError(-32008), message: "Response is too big", data: Some(RawValue("Exceeded max limit of 15728640")) }

**Caused by:
    RPC error: ErrorObject { code: ServerError(-32008), message: "Response is too big", data: Some(RawValue("Exceeded max limit of 15728640")) }**

Stack backtrace:
   0: anyhow::error::<impl core::convert::From<E> for anyhow::Error>::from
   1: cifrost::batch_sync_storage_changes::{{closure}}
   2: cifrost::bridge::{{closure}}
   3: <tokio::future::poll_fn::PollFn<F> as core::future::future::Future>::poll
   4: tokio::runtime::park::CachedParkThread::block_on
   5: tokio::runtime::context::runtime::enter_runtime
   6: tokio::runtime::runtime::Runtime::block_on
   7: cifrost::main
   8: std::sys_common::backtrace::__rust_begin_short_backtrace
   9: std::rt::lang_start::{{closure}}
  10: std::rt::lang_start_internal
  11: main
  12: __libc_start_main
  13: _start


Metadata updates for release 8.7.1

Metadata changes:

Updated phone metadata for region code(s): AG, AM, BJ, BO, CL, EE, EG, GD, HK, IN, KE, KW, LK, US
Updated short number metadata for region code(s): BJ, EE, EG, IN, KW
New geocoding data for country calling code(s): 1564 (en)
New carrier data for country calling code(s): 1268 (en)
Updated carrier data for country calling code(s): 56 (en), 254 (en), 374 (en), 852 (en, zh, zh_Hant), 1473 (en)
Updated / refreshed time zone meta data.

@SumathiSundara Release 8.7.1 jar file does not contain either the time zone map data (file map_data) or the corresponding time zone utility class (file PhoneNumberToTimeZonesMapper.class). Have they been renamed or refactored into some other location? Please advise. We need time zone mapping feature missing in 8.7.1, but somehow alluded to in the release notes ("Updated / refreshed time zone meta data.").

@mike-chen-tff I just verified that
(a) this release included a carrier mapping update (this github PR includes a map_data update)
(b) this release included (as expected) a new carrier mapper release on Maven Central https://mvnrepository.com/artifact/com.googlecode.libphonenumber/carrier/1.66
(c) that carrier mapper jar includes the .class file and data.
All working as intended. What am I missing?

@andyst you are right. I was looking in the wrong place (libphonenumber-x.x.x.jar instead of geocoder-x.xx.jar), my bad. Question is retracted.


still can't make playlists work (autoplay policy ?)

Related to https://github.com/cookpete/react-player/issues/1072, I still can't make playlists work; player is "blocked" by the browser when the tab is inactive.
I suspect that this might be related to your code, since it even does not work when the website is whitelisted for autoplay (for chrome, at chrome://settings/content/sound).
This example on codepen has an audio playlist which works when tab is in background (even if not whitelisted) https://codepen.io/danielhoegel/pen/OZNqNo

After a media has ended and that the next one is requested to play (custom code), react-player fires onReady but does not play the media.
Other Information
Demo here : http://v2.spiff-radio.org/
I also filled a question on stackoverflow here.
Thanks a lot !

With your demo it doesn't work here either on either Firefox or Chrome.
I use the library on a Nextjs project and there it autoplays the next video.
I use onEnded={..} which calls a function which updates the current url={...}.

Hi @TheManFromEarth1,  it helped !
As I knew it was possible because of your feedback, I kept trying to find out what was the bug.
Well, at the beginning of my function resolveTracks(), I had a line
setSourceUrl('');

Which was "resetting" the stream before loading the next URL.
That was the cause of my bug.  Without that line, going to the next track works !
Thanks a lot for your help.
(no need for stopOnUnmount here)


[Bug] Open lapce here does not work on Windows

And modals don't exit when Esc is pressed

Thanks for opening this issue!
I believe modals should close now when Esc is pressed. Could you please test?
I'm not sure about "Open Lapce here", but I don't think explorer integration is implemented yet.

Fixed in #1158


[Bug] Info and resolution for white screen

🐛 Bug
Here are the details I've found and what I've done to fix the white screen issue locally when the language is not set to English.
To reproduce
Steps to reproduce
Install Carpe on non english system/brower.
Print screen or Debug/Console error message
[HMR][Svelte]"Unrecoverable HMR error in <AccountSwitcher>: next update will trigger a full reload"
Error: [svelte-i18n] Cannot format a message without first setting the initial locale.

System information
Please complete the following information:

Carpe 1.0.0
MacOS 14 Sonoma

Change
In src/lang/i18n.ts
Add is_loading in import and export
import { locale, init, _, getLocaleFromNavigator, register, isLoading } from 'svelte-i18n'
...
export { _, setupI18n, getLocaleFromNavigator, locale, isLoading }

In AccountSwitcher.svelte
Wrap the dropdown with {#if !isLoading} after import it.

@BBK912

Thank You for debugging this @Ziltosh

fixado


post-mvp: deduplicate content

when porting v1 content over to this repo, some content shared between tracks has been duplicated to ship the mvp. lets dedupe before fixes start to be applied in one place and not another. e.g., the solidity intro lesson is shared by the erc721 and erc20 tracks.

I was hoping to only have one copy of that lesson, but used in two places. Just didn't figure out how to do that (was a late night, I believe). I should have created an Issue for it.

What about if having the fundamentals page we can just link to that lesson path URL from each Lesson Card on the other Tracks needed ? Will that work and solve the lesson duplication?

What about if having the fundamentals page we can just link to that lesson path URL from each Lesson Card on the other Tracks needed ? Will that work and solve the lesson duplication?

For my understanding, that sounds like a perfect fix! ✊🏼 😁

What about if having the fundamentals page we can just link to that lesson path URL from each Lesson Card on the other Tracks needed ? Will that work and solve the lesson duplication?

For my understanding, that sounds like a perfect fix! ✊🏼 😁
apologies for the misunderstanding this quote created @Markkos89


What's the VNC password?

We're using the docker flow, and the VNC client is asking for a password. What is it?

secret


Not spawning robot in docker compose

Hello sir,
When bringing up a robot along with a simulator container using docker compose, the robot is not getting spawned into the environment. However the ros2 topic for the robot is active in the robot container. However, on manually launching the robot from another terminal, the robot becomes visible in the simulator container.
How to solve this?

I'll need a minimal example to reproduce the issue.
Side note, we're creating a single launch file to launch simulation and spawn robots in https://github.com/osrf/mbzirc/pull/126. Maybe that helps. We'll have more instructions once the PR is merged.


[Authoring] We cannot add any style to a button and the align-center property doesn't work

Target Application
Microsoft Teams
Application Operating System
iOS
Schema Version
1.5
Problem Description

We cannot add any style to adaptive card. The adaptive card has a property called style. It has 2 optional values. One is positive, the other is destructive. If I change the style and render it by Bot Framework Web Chat, it will be like:

But if I change the render engine to Teams, it will be like:

If we set horizontalAlignment of ActionSet to true, the button inside it will be like:

But it doesn't work in Teams app. The button will be align-right automatically:

Expected Outcome

We can change style of buttons.
We can align buttons to left, right or center.

Actual Outcome

The style of buttons don't change.
The button alignment cannot be controlled by us. It will be align-right automatically.

Card JSON
{
    "type": "AdaptiveCard",
    "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
    "version": "1.5",
    "body": [
        {
            "type": "ActionSet",
            "actions": [
                {
                    "type": "Action.Submit",
                    "title": "positive",
                    "style": "positive"
                },
                {
                    "type": "Action.Submit",
                    "title": "destructive",
                    "style": "destructive"
                }
            ],
            "spacing": "None",
            "horizontalAlignment": "Center"
        }
    ],
    "actions": [
        {
            "type": "Action.Submit",
            "title": "default"
        }
    ]
}

Repro Steps
No response

This issue has been copied over to Microsoft Teams repo here: https://github.com/MicrosoftDocs/msteams-docs/issues/9299

Hi @qjkobe,
Could you please raise this issue directly with the Teams mobile team? The host app needs to support the style for it to function properly.


Add href attribute to pf-sort links

Fixes #213
/cc @jwforres @rhamilto

You can try using href="javascript:void(0);" rather than href="" to get the tests to pass.

@jeff-phillips-18 I can make the change, but I'm unable to run the tests because of this error:
PhantomJS 1.9.8 (Mac OS X) pf-select Page with pf-select directive should respond to changes in ng-options FAILED
	Expected 4 to be 3.
	Error: Expected 4 to be 3.
	    at /Users/sam/git/angular-patternfly/test/select/select.spec.js:54
PhantomJS 1.9.8 (Mac OS X) pf-select Page with pf-select directive should respond to ng-model changes FAILED
	Expected 4 to be 3.
	Error: Expected 4 to be 3.
	    at /Users/sam/git/angular-patternfly/test/select/select.spec.js:83
	Expected '' to be 'a'.
	Error: Expected '' to be 'a'.
	    at /Users/sam/git/angular-patternfly/test/select/select.spec.js:88

Updated.

I don't see any issues with the pf-select tests.

I don't see any issues with the pf-select tests.

It could be environmental, but they consistently fail for me.

You might try cleaning out the node_modules and lib directories and redoing npm install and bower install

You might try cleaning out the node_modules and lib directories and redoing npm install and bower install

Thanks, that fixed the test error. Still seeing the diff after grunt build... Any ideas?

I see... it is a problem with the previous commit not rebuilding :(
Next commit will fix it
LGTM

Rebased.


Write object level properties of mesh objects in 3mf model file.

Additional fix for #132
Default mesh object property was not written in 3mf xml file even after exposing the APIs for the same. This commit fixes the same.

Thanks, @shahchintan14 !
I included your suggestion in the PR that implements the API for object-level properties https://github.com/3MFConsortium/lib3mf/pull/147


Climate history not showing state after 0.96

Home Assistant version
0.96.5
SmartIR version
1.7.0
SmartIR configuration

Add configuration here
climate:
  - platform: smartir
    name: Room AC
    unique_id: room_ac
    device_code: 1382
    controller_data: <IP_ADDRESS>
    temperature_sensor: sensor.room_temperature
    humidity_sensor: sensor.room_humidity
    # power_sensor: binary_sensor.ac_power

Describe the bug
A clear and concise description of what the bug is.
After 0.96 the climate history graph is not showing the status shaded part (if it's cooling or not).
here is my entity status:

and the history graph:

Used to be something like this:

Debug log
Add your logs here.

2019-07-28 00:09:46 WARNING (MainThread) [homeassistant.loader] You are using a custom integration for hacs which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you do experience issues with Home Assistant.
2019-07-28 00:09:46 WARNING (MainThread) [homeassistant.loader] You are using a custom integration for smartir which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you do experience issues with Home Assistant.
2019-07-28 00:10:05 WARNING (MainThread) [homeassistant.setup] Setup of influxdb is taking over 10 seconds.
2019-07-28 00:10:05 WARNING (MainThread) [homeassistant.setup] Setup of input_boolean is taking over 10 seconds.
2019-07-28 00:10:07 WARNING (MainThread) [homeassistant.setup] Setup of hacs is taking over 10 seconds.
2019-07-28 00:10:09 WARNING (SyncWorker_19) [homeassistant.components.influxdb] Database host is not accessible due to 'HTTPConnectionPool(host='<IP_ADDRESS>', port=8086): Max retries exceeded with url: /write?db=home_assistant (Caused by ConnectTimeoutError(<urllib3.connection.HTTPConnection object at 0x71d23450>, 'Connection to <IP_ADDRESS> timed out. (connect timeout=5)'))', please check your entries in the configuration file (host, port, etc.) and verify that the database exists and is READ/WRITE. Retrying again in 60 seconds.
2019-07-28 00:10:11 WARNING (MainThread) [homeassistant.loader] You are using a custom integration for rpi_power which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you do experience issues with Home Assistant.
2019-07-28 00:10:22 WARNING (MainThread) [homeassistant.components.sensor] Setup of platform dht is taking over 10 seconds.
2019-07-28 00:10:28 WARNING (MainThread) [homeassistant.components.template.sensor] Could not render template door_sensor_battery, the state is unknown.
2019-07-28 00:10:28 WARNING (ThreadPoolExecutor-1_0) [pycec] Not initialized. Waiting for init.
2019-07-28 00:10:29 WARNING (ThreadPoolExecutor-1_0) [pycec] Not initialized. Waiting for init.
2019-07-28 00:11:26 WARNING (SyncWorker_19) [homeassistant.components.influxdb] Database host is not accessible due to 'HTTPConnectionPool(host='<IP_ADDRESS>', port=8086): Max retries exceeded with url: /write?db=home_assistant (Caused by ConnectTimeoutError(<urllib3.connection.HTTPConnection object at 0x6ea27fd0>, 'Connection to <IP_ADDRESS> timed out. (connect timeout=5)'))', please check your entries in the configuration file (host, port, etc.) and verify that the database exists and is READ/WRITE. Retrying again in 60 seconds.
2019-07-28 00:11:44 WARNING (SyncWorker_1) [pychromecast.controllers] PLAY command requested but no session is active.
2019-07-28 00:13:28 ERROR (InfluxDB) [homeassistant.components.influxdb] Write error: HTTPConnectionPool(host='<IP_ADDRESS>', port=8086): Max retries exceeded with url: /write?db=home_assistant (Caused by ConnectTimeoutError(<urllib3.connection.HTTPConnection object at 0x6dba0cb0>, 'Connection to <IP_ADDRESS> timed out. (connect timeout=5)'))
2019-07-28 00:14:22 ERROR (InfluxDB) [homeassistant.components.influxdb] Resumed, lost 26 events
2019-07-28 00:14:28 WARNING (SyncWorker_11) [pychromecast.controllers] PLAY command requested but no session is active.
2019-07-28 00:15:28 ERROR (InfluxDB) [homeassistant.components.influxdb] Write error: HTTPConnectionPool(host='<IP_ADDRESS>', port=8086): Max retries exceeded with url: /write?db=home_assistant (Caused by ConnectTimeoutError(<urllib3.connection.HTTPConnection object at 0x6eabf1f0>, 'Connection to <IP_ADDRESS> timed out. (connect timeout=5)'))
2019-07-28 00:15:46 WARNING (MainThread) [homeassistant.helpers.entity] Update of sensor.raw_hall_temperature is taking over 10 seconds
2019-07-28 00:16:04 WARNING (MainThread) [homeassistant.helpers.entity] Update of sensor.raw_room_humidity is taking over 10 seconds
2019-07-28 00:16:07 WARNING (MainThread) [homeassistant.components.sensor] Updating dht sensor took longer than the scheduled update interval 0:00:30
2019-07-28 00:16:26 ERROR (InfluxDB) [homeassistant.components.influxdb] Resumed, lost 23 events

Additional context
Add any other context about the problem here.

Seems that there needs to be a new attribute called hvac_action which needs to be set to "cooling" or "heating" etc.

I'll close this until things get clear https://github.com/smartHomeHub/SmartIR/pull/210


Bosh Templating Language RFC

Proposal for adding a new templating language for bosh releases
Preview

ARP releases  do a lot of "check to see if this property is provided, if not then use the property from a bosh link". Is this type of logic someone that would be provided?
But really, I second what @rkoster said above asking for examples of what you want to include in your templating language, that way I can be more specific in my worries 😅

I come bearing examples of template logic!

Most of our jobs use a function like this one to do the "check to see if this property is provided, if not then use the property from a bosh link" logic. link
IP validation. We do this in many jobs.
basic int logic greater than/less than. link
string must be one of a set. link
type checking, it must be an array. link
if the property is provided, and once we strip whitespace, then it must be longer than 50 chars. link
requiring certain properties, but only when other properties are set. link
turn bosh machine info into json. ink
if a certain thing is a certain number, then do all these other things. link
cert validation link
validating cert SANs link
at least one CA has a certain SAN link
and those examples are all from gorouter!
here we use a property as a deprecation warning to alert users that nats (non-tls) is going away soon. link
pulling out db params from a string link
turn array into a comma separated list link

After going through all these files I see that nearly every one of ours has this helper method defined. It would be great if you owned these types of helper functions :)
  def parse_ip (ip, var_name)
    unless ip.empty?
      ip = ip.split(":")[0]
    begin
      parsed = IPAddr.new ip
    rescue  => e
      raise "Invalid #{var_name} '#{ip}': #{e}"
    end
    end
  end

If we want to push logic out of the templates why not drop the central template rendering altogether and allow release others to consume the YAML directly? The provided YAML would be placed in the config directory of the job and the release decides how to proceed with it.
This would also work around the core issue here which I think is dependency management: the release authors are implementing against a templating system which is controlled by the user of the release (e.g. the operator) and not the release author. By explicitly making the template rendering a part of the release itself we solve that issue. Bumping the template renderer becomes the responsibility of the release author which can properly test and resolve any issues with version bumps. And the operator no longer risks breaking a bunch of releases just because the ruby version or some required dependency is bumped in their environment.
If we still want BOSH to be able to distinguish between the templating and pre-start phase we could add an additional lifecycle step. If the release author provides a script for that step the ERB rendering is automatically disabled. And for release which don't want to switch a Ruby script which mimics the behaviour of the old template rendering could be provided / injected to speed up the transition.


Fix release date for 2.0.73

Docs Build status updates of commit afe3a3c:
:clock10: Full build: the average full build time is 6 min(s) 46 sec(s), this is based on the last 197 build(s) for this repository.

Docs Build status updates of commit afe3a3c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/release-notes-azure-cli.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

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


feat(cli): Add missing upstream warning during 'keptn upgrade'

Signed-off-by: odubajDT<EMAIL_ADDRESS>

This PR

adds missing upstream warning during keptn upgrade

Related Issues

Fixes #6308

Issue about refactoring the upgrade.go code: https://github.com/keptn/keptn/issues/6439


niche-Vlaanderen: Invalid use of the Relevé extension

Invalid use of the Relevé extension: there are many Relevés : single event. In theory, each Relevé should represent a single sampling event. Plus there is very little variation between all these Relevés. Presumably they were carried out for different vegetation layers, but this isn’t indicated?

@kbraak  you mean that, for example, these 3 records, pointing at the same relevée should actually be merged to one record?:

 This demo dataset shows the proper usage of one Relevé per sampling event.

@kbraak I ve updated the DwC-A, you can find it here.
dwca-inboveg_niche_vlaanderen-v1.4.zip
Can you check the mapping?


fix: TestCafe grabAttributeFrom method not works

Motivation/Description of the PR
Applicable helpers:

[ ] Webdriver

[ ] Puppeteer

[ ] Nightmare

[ ] REST

[ ] Appium

[ ] Protractor

[x] TestCafe

Description of this PR, which problem it solves
grabAttributeFrom() method not returning the attribute value

A link to the corresponding issue (if applicable).
https://stackoverflow.com/questions/59133688/grabattributefrom-method-not-returning-the-attribute-value

Type of change

[ ] Breaking changes
[ ] New functionality
[x] Bug fix
[ ] Documentation changes/updates
[ ] Hot fix
[ ] Markdown files fix - not related to source code

Checklist:

[x] Tests have been added
[ ] Documentation has been added (Run npm run docs)
[x] Lint checking (Run npm run lint)
[x] Local tests are passed (Run npm test)

Seems fine, but test failed

 Here is an overview of what got changed by this pull request:

Clones added
============
- test/helper/webapi.js  6

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Clones added
============
- test/helper/webapi.js  2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Clones added
============
- test/helper/webapi.js  2

See the complete overview on Codacy

Don't understand, why travis build for TestCafe fails every time with catched error from previous test
And why circle-ci with webdriver fails

 Here is an overview of what got changed by this pull request:

Clones added
============
- test/helper/webapi.js  2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Clones added
============
- test/helper/webapi.js  2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Clones added
============
- test/helper/webapi.js  2

See the complete overview on Codacy

Skipped some failing tests for TestCafe.
But somewhy Webdriverio tests fails in Circle CI, does not know why.

Looks like we have a problem with test. Cause if some test in test suite thows error (and catches it), then all next tests fails

webdriverio v5.16.12 was released between first (passed) and second (failed) circle CI builds.
https://github.com/webdriverio/webdriverio/releases

 Here is an overview of what got changed by this pull request:

Clones added
============
- test/helper/webapi.js  2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Clones added
============
- test/helper/webapi.js  4

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Clones added
============
- test/helper/webapi.js  4

See the complete overview on Codacy


Fix pixelate not working well with alpha channel

closes  #1046
Before

After

:rocket: PR was released in v1.1.2 :rocket:


Modified xmsh package to track the master branch

This allows user to get the latest version automatically.

Your desire is already satisfied by the --head install parameter. VCPKG aims to fix versions when installing packages, so your PR might not be accepted...

Thanks for the heads up.


Integrate Kevin's Advantage Tracker

Basic functionality has been coded. Includes:

Ability to set rolltype via !shaped-at command
accepts either --advantage --disadvantage or --normal
Updated the associated token with the green status marker for advantage, or red for disadvantage

Still to do:

Update a newly placed token with the appropriate status marker if rolltype is not normal

Want to do:

Create a new config option that would allow for adv/dis to be set with a 'revert' flag that would revert the rolltype back to normal (or whatever it was before) after a roll is made.


InvalidArgumentError: Only ranks up to 5 supported

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): CentOS inux release 7.6.1810
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: No
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 1.14.0
Python version: 3.7.6
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: V9.1.85
GPU model and memory: Tesla K80 24C
I am working with Keras to build the following (simplified) model:
class Model():
   def __init__(self):
       pass

   def build_model(self):
       inp = Input(shape = self.outputs.shape)

       x = Dense(128, activation = 'elu')(inp) # We'll start exploring with one hidden layer
       outp = Dense(1, activation = 'elu')(x)

       self.model = Model(inputs = inp, outputs = outp)

       self.model.compile(loss = dice_loss,
                            optimizer = Adam(),
                            metrics = [dice_coef])

   def run_model(self):
      # For the sake of simplicity
      self.outputs = np.random.rand(8,64,12,86,98,1)

      # labels shape = (64, 12, 86, 98) to get (1,64,12,86,98)
      self.labels = np.expand_dims(self.labels, axis = (0,-1))
      # outputs shape (8,64,12,86,98,1) to -> (1,64,12,86,98,8)
      self.outputs = np.swapaxes(self.outputs, 0,-1)

      self.build_model()

      self.model.fit([self.outputs, self.labels],
                                batch_size = 1,
                                epochs = 200)

However, upon trying to fit the model, as:
model = Model()
model.run_model()

The following error appears:
  File "/home/kdqm927/miniconda3/envs/segment/lib/python3.7/site-packages/tensorflow/python/keras/engine/training_v1.py", line 785, in fit
    use_multiprocessing=use_multiprocessing)
  File "/home/kdqm927/miniconda3/envs/segment/lib/python3.7/site-packages/tensorflow/python/keras/engine/training_arrays.py", line 634, in fit
    shuffle=shuffle)
  File "/home/kdqm927/miniconda3/envs/segment/lib/python3.7/site-packages/tensorflow/python/keras/engine/training_v1.py", line 2308, in _standardize_user_data
    batch_size=batch_size)
  File "/home/kdqm927/miniconda3/envs/segment/lib/python3.7/site-packages/tensorflow/python/keras/engine/training_v1.py", line 2335, in _standardize_tensors
    exception_prefix='input')
  File "/home/kdqm927/miniconda3/envs/segment/lib/python3.7/site-packages/tensorflow/python/keras/engine/training_utils.py", line 573, in standardize_input_data
    'with shape ' + str(data_shape))
ValueError: Error when checking input: expected input_1 to have 7 dimensions, but got array with shape (1, 64, 12, 86, 98, 8)

If I tweak the code by adding another dimension to self.outputs, following the exception from the previous error:
self.model.fit([np.expand_dims(self.outputs, axis = 0), self.labels],
                                batch_size = 1,
                                epochs = 200)

The error I get is:
Traceback (most recent call last):
  File "./file.py", line 257, in <module>
    s.fit_metamodel('outputs_validation_arr')
  File "./file.py", line 157, in fit_metamodel
    callbacks = callbacks)
  File "/home/.../miniconda3/envs/segment/lib/python3.7/site-packages/tensorflow/python/keras/engine/training_v1.py", line 785, in fit
    use_multiprocessing=use_multiprocessing)
  File "/home/.../miniconda3/envs/segment/lib/python3.7/site-packages/tensorflow/python/keras/engine/training_arrays.py", line 666, in fit
    steps_name='steps_per_epoch')
  File "/home/.../miniconda3/envs/segment/lib/python3.7/site-packages/tensorflow/python/keras/engine/training_arrays.py", line 386, in model_iteration
    batch_outs = f(ins_batch)
  File "/home/.../miniconda3/envs/segment/lib/python3.7/site-packages/tensorflow/python/keras/backend.py", line 3632, in __call__
    run_metadata=self.run_metadata)
  File "/home/.../miniconda3/envs/segment/lib/python3.7/site-packages/tensorflow/python/client/session.py", line 1472, in __call__
    run_metadata_ptr)
tensorflow.python.framework.errors_impl.InvalidArgumentError: Only ranks up to 5 supported: [1,1,64,12,86,98,128]
	 [[{{node dense/BiasAdd}}]]

@danielfllaneza
I ran the code shared by you, itseems incomplete, please find the error faced in gist here.
Meanwhile as suggested in the link can you try to reshape
link link1 link2

Oh, I forgot mocking labels, for the sake of simplicity we can create self.labels too, so the code would be:
class Model():
   def __init__(self):
       pass

   def build_model(self):
       inp = Input(shape = self.outputs.shape)

       x = Dense(128, activation = 'elu')(inp) # We'll start exploring with one hidden layer
       outp = Dense(1, activation = 'elu')(x)

       self.model = Model(inputs = inp, outputs = outp)

       self.model.compile(loss = dice_loss,
                            optimizer = Adam(),
                            metrics = [dice_coef])

   def run_model(self):
      # For the sake of simplicity
      self.outputs = np.random.rand(8,64,12,86,98,1)
      self.labels = np.random.rand(64, 12, 86, 98)

      # labels shape = (64, 12, 86, 98) to get (1,64,12,86,98)
      self.labels = np.expand_dims(self.labels, axis = (0,-1))
      # outputs shape (8,64,12,86,98,1) to -> (1,64,12,86,98,8)
      self.outputs = np.swapaxes(self.outputs, 0,-1)

      self.build_model()

      self.model.fit([self.outputs, self.labels],
                                batch_size = 1,
                                epochs = 200)

That's not a bug, but an error in your code.
The input_shape of your model is (None,1,64,12,86,98,8), but the shape of x data you feed into it is (1,64,12,86,98,8). They don't match.
In Tensorflow2+ or keras, all the model will leave the first dimension of data's shape for batch-size, for example, one single sample of X has a shape (8,8),  corresponding, the input-shape of model will be (None, 8, 8), where "None" stands for "batch-size" .

@danielfllaneza
Can you please follow the links shared, the issue reported is due to error in your code.

Thanks for your replies.
I have tried the following:

#10053

# method def run_model
(...)
self.labels = self.labels.reshape([-1,1,64,12,86,98])
(...)


DR Center Minor Fixes

copy change

when buyer raises dispute, i get this error on developer tools. dispute doesn't go through


Adds Centro Universiátio Mário Cesar Jucá

Formerly known as "Faculdade de Tecnologia de Alagoas". Website is http://fat-al.edu.br/.

Link to IT-related course: http://www.fat-al.edu.br/graduacao/bacharelado/engenharia-da-computacao

Also contains link to the course managers e-mail nathalia.alves at fat-al.edu.br, a proof of domain usage

School addresses: http://www.fat-al.edu.br/institucional/campus (link to Google Maps location)

Link to IT-related course: http://www.fat-al.edu.br/graduacao/bacharelado/engenharia-da-computacao

Also contains link to the course managers e-mail nathalia.alves at fat-al.edu.br, a proof of domain usage

School addresses: http://www.fat-al.edu.br/institucional/campus (link to Google Maps location)

@tarsiswt-fat-al Pull request merged. Thank you!


CRDs for tigera-operator are not updating with helm upgrade, Migration is not possible.

Trying to disable default csi-node-driver daemonset created with calico helm chart installation, By setting kubeletVolumePluginPath=None with helm --set args
Ref:  https://github.com/projectcalico/calico/blob/v3.24.5/charts/tigera-operator/crds/operator.tigera.io_installations_crd.yaml#L4253
Running version v3.24.0 of calico & migrating to v3.24.5
Expected Behavior

When upgrading calico chart from v3.24.0 to v3.24.5, Expecting with helm upgrade all CRD's and CR's should be updated to the newer version also the default Daemonset for csi-node-driver should not be present.
Current Behavior

When upgrading calico chart from v3.24.0 to v3.24.5 default Daemonset for csi-node-driver is still present, After checking a little bit found that CRD's are not updated with field kubeletVolumePluginPath  and hence helm upgrade fails with below Error
Error: UPGRADE FAILED: error validating "": error validating data: ValidationError(Installation.spec): unknown field "kubeletVolumePluginPath" in io.tigera.operator.v1.Installation.spec

Possible Solution

To get CRD's and CR's updated had to remove helm release and delete CRD's and then do fresh install which is not a migration scenario which is more like delete and create a new installtion for calico
Steps to Reproduce (for bugs)

Install calico version v3.24.0 using calico chart

helm repo add projectcalico https://docs.projectcalico.org/charts
helm repo update
helm upgrade --install calico projectcalico/tigera-operator --namespace=tigera-operator --create-namespace --version v3.24.0 --set installation.registry=quay.io --set apiServer.enabled=false

Once calico is installed MIgrate to v3.24.5, with --set installation.kubeletVolumePluginPath=None

(root:~) helm upgrade --install calico projectcalico/tigera-operator --namespace=tigera-operator --create-namespace --version v3.24.5 --set installation.registry=quay.io --set apiServer.enabled=false --set installation.kubeletVolumePluginPath=None
Error: UPGRADE FAILED: error validating "": error validating data: ValidationError(Installation.spec): unknown field "kubeletVolumePluginPath" in io.tigera.operator.v1.Installation.spec

Upgrade fails since CRD's and CR's are not yet updated to support the kubeletVolumePluginPath field

Context

When upgrading calico chart from v3.24.0 to v3.24.5 default Daemonset for csi-node-driver is still present, After checking a little bit found that CRD's are not updated with field kubeletVolumePluginPath  and hence helm upgrade fails with below Error
Error: UPGRADE FAILED: error validating "": error validating data: ValidationError(Installation.spec): unknown field "kubeletVolumePluginPath" in io.tigera.operator.v1.Installation.spec

Your Environment

Calico version : v3.24.0 upgrading to v3.24.5
Orchestrator version (e.g. kubernetes, mesos, rkt): k8s version 1.23
Operating System and version: amazon-linux2 ami
Link to your project (optional):

Related to https://github.com/tigera/operator/issues/2249 I would say, I have the same problem

yeah we 100% need to update our upgrade documentation to handle this ASAP

@Shivam0609 I've been able to successfully do upgrades through helm  by splitting it into three helm charts:

First have a crds only helm chart for calico, calico-crds. We basically copy and paste the existing calico helm chart's crds into the templates directory of calico-crds which allows upgrading of the crds through the helm chart.
Next, we install the provided calico helm chart with the --skip-crds flag.
Finally, we have a third helm chart for deploying and instances of the crds, called calico-policies.

This has allowed us to do smooth upgrades with out issues so far and have it successfully deployed through flux with the helmrelease

@Chili-Man +1 for the dedicated charts (I know istio did that at some point), the other solution would be maintaining a docs saying when there are CRDs upgrade to apply and publish the CRDs ready to be applied on Github. I know [kube-prometheus-stack] does this here
This makes it more easier for people to follow the CRDs release, and also to manage it independently, for example with TF like here

Thank you both @Chili-Man and @ArchiFleKs for your contributions, 2 valid solutions that would definitely help going forward.
Simplest would be the second solution, since people can still continue to use the current helm-chart with CRD.
But users concerned about upgradability would use --skip-crds and load the CRDs by hand or automation (terraform). They just need to be properly versioned so we can reference the YAML of the CRDS by code as suggested.

@aquam8 is there a place where we can find the versionned CRDs right now ? outside of the packaged helm charts

@ArchiFleKs you can find them here for now: https://projectcalico.docs.tigera.io/manifests/operator-crds.yaml
Or https://projectcalico.docs.tigera.io/archive/v3.24/manifests/operator-crds.yaml
Or for recent versions, on GH: https://raw.githubusercontent.com/projectcalico/calico/v3.25.0/manifests/operator-crds.yaml

Latest upgrade instructions properly explain to upgrade the CRDs manually https://docs.tigera.io/calico/latest/operations/upgrading/kubernetes-upgrade#all-other-upgrades


testsuite failure in ccg.doctest (python 2.6.6)

The following example in ccg.doctest:
for parse in parser.nbest_parse("you prefer that cake".split(),1):
chart.printCCGDerivation(parse)
for parse in parser.nbest_parse("that is the cake which you prefer".split(), 1):
chart.printCCGDerivation(parse) # doctest: +NORMALIZE_WHITESPACE
is failing with python 2.6.6, since the expected and produced outputs differ as follows:
--- exp   2012-02-11 17:48:17.000000000 +0100
+++ got   2012-02-11 17:48:12.000000000 +0100
@@ -1,7 +1,7 @@
      you    prefer      that   cake
      NP   ((S\NP)/NP)  (NP/N)   N
-         --------------------->B
-              ((S\NP)/N)
+                      -------------->
+                            NP
          --------------------------->
                    (S\NP)
     --------------------------------<
@@ -23,4 +23,6 @@
           ------------------------------------------------------------->
                                      (S\NP)
     -------------------------------------------------------------------<
-
+                                     S
+
+

Yes, we're aware of this problem, and are continuing to fix the doctests.


Fix broken stacks traces on Go code

In this simple Go example, it's clear that Go's stack switch causes stacktraces to wander off the right hand side of the screen. I think this is easy to fix: when trace_writer.ml, sees the symbol runtime.newstack, it should mark all currently-open stack frames as closed.
I'm not sure if there is any more custom control flow in Go code. e.g. do Go stacks shrink? Please file more bugs if you notice any.

Hi, I just discovered this project and it looks like an amazingly useful tool, and I'd love to get it working better with Go. I haven't tried running myself yet, but some initial notes below:

I think this is easy to fix: when trace_writer.ml, sees the symbol runtime.newstack, it should mark all currently-open stack frames as closed.

runtime.newstack is calling into the scheduler [1] (I think this is the truncated runt... bit under gopreempt_m on https://github.com/janestreet/magic-trace/pull/98), which selects a new goroutine to run. The precise moment we switch to the new goroutine stack and start executing is in gogo, so I think this is the best point to mark stack frames as closed. Note the symbol is gogo, not runtime.gogo (runtime.gogo jumps to gogo).

I'm not sure if there is any more custom control flow in Go code. e.g. do Go stacks shrink?

Many places within the runtime switch from the "goroutine stack" to the "system stack", which is a fixed sized thread stack where do things that aren't safe on the goroutine stack. We switch back to the goroutine stack before returning to user code. morestack actually switches to the system stack before calling newstack, so it seems this is already working fine.
Signal handlers run on yet another "signal stack". This is set up with sigaltstack to apply automatically on signal delivery, so my guess is that it is already handled correctly.
Go stacks can shrink. This can either occur in newstack, or concurrently during the GC. In the GC case, we temporarily freeze the goroutine using our standard preemption mechanisms, copy the stack to a small stack, and then allow it to resume with the new SP. It is unclear to me if moving stacks (grow/shrink) will be problematic in and of themselves. Do you track actual SP values to determine when stack frames return?

[1] A bit more background: the newstack call in your example isn't actually growing the stack. Most functions contain a prologue that checks SP against the stack bounds to see if we need to grow the stack by calling newstack. For cooperative preemption, we poison the stack bounds so the prologue thinks it needs to grow the stack, but newstack does the more complex work do decide that no, we really just need to preempt.

Thanks for taking a look at this. It's nice to have someone who actually knows how Go works (as opposed to us) working on this.
To answer your question: magic-trace doesn't track or know anything about stack pointers. The stack frames are entirely inferred by the history of application's control flow (e.g. call, ret, jmp). From your description, it sounds like stack shrinking is uninteresting to magic-trace.

I've put up a draft of a PR that works on a very simple demo program. Thanks for the gogo hint, it was a huge help.
Could you please take a look at that PR and try that version of magic-trace on some interesting go programs you may have lying around?


How to stream with gstreamer to jsmpeg

Hi As mention in documention we can stream with ffmpeg and gstremer. Can we add similar example of gstremer as well
I already posted a question here https://stackoverflow.com/questions/69794587/jsmpeg-camera-stream-with-gstremer
Inshort need same command with gstreamer. and we can add in documentation also.
ffmpeg \
    -f v4l2 \
        -framerate 25 -video_size 640x480 -i /dev/video0 \
    -f mpegts \
        -codec:v mpeg1video -s 640x480 -b:v 1000k -bf 0 \
    http://localhost:8081/supersecret

Thanks

We can consider adding in documention
GST_DEBUG=3 gst-launch-1.0  v4l2src device=/dev/video0 ! videoconvert ! video/x-raw,framerate=30/1 ! avenc_mpeg1video ! mpegtsmux ! curlhttpsink location=http://<IP_ADDRESS>:8081/supersecret

I think gstreamer in the doc/readme would be a valuable addition because it provides observably lower latency than ffmpeg.
Thoughts @phoboslab ?


Update broken link

Toml now uses toml.io for released specifications and the github repo for development.  Also the old link was for the 0.4 specification, while cargo uses toml_edit, which uses toml 1.0 (reference: https://github.com/toml-rs/toml/blob/main/crates/toml_edit/CHANGELOG.md#030---2021-09-13). Finally, the link was referring to discussion of "Bare keys" vs "Quoted keys".  This discussion was in the #table section in the 0.4 specification, but is under #keys in the 1.0 specification.

If https://github.com/rust-lang/fmt-rfcs/pull/177 is merged, this will be unneeded.

The style guide has indeed now been moved, and your commit is now in the version in rust-lang/rust. Thank you!


using 'audio' generates static video

Hi~ @Hangz-nju-cuhk
When I use 'audio' as the test_type, I got the static video all the time. All the generated images are the same. However when I use 'video' as the test_type, it works fine. Do you have any idea for this? Thank you!

Hi, we recently found out that the audio part is not runnable on Pytorch version >=0.4.0, please check your environment to be the same as ours (Pytorch 0.2.0 with python 2.7).

@sarah-zhu Sorry that there was a mistake in our testing data loading code, we have now fixed it. Thank you for pointing it out.

@Hangz-nju-cuhk Thanks for fixing it! Is the audio part  runnable on  Pytorch version >=0.4.0 now?

@sarah-zhu there could be a bug on version >= 0.4.0 that needs fixing. We recommend using Pytorch version < 0.4.0 for now. I will be working on releasing the new version for Pytorch 0.4.1 in a month.

sorry, when I run the testing script to generate videos from audio use the modified code, I still face the following mistake:
Traceback (most recent call last):
File "test_all.py", line 34, in
test_folder = Test_VideoFolder(root=opt.test_root, A_path=A_path, config=opt)
File "/home/dtt/workspace/Talking-Face-Generation-DAVS/Dataloader/Test_load_audio.py", line 92, in init
self.vid = self.loader(self.root, self.A_path, config=self.config)
File "/home/dtt/workspace/Talking-Face-Generation-DAVS/Dataloader/Test_load_audio.py", line 67, in Test_Outside_Loader
raise ("mfccs = 0")
TypeError: exceptions must be old-style classes or derived from BaseException, not str

when I remove the code
        if require_audio:             # ipdb.set_trace()             k4 = 0             for mfcc_num in pair:                 # for s in range(-1,2):                 mfcc_path = os.path.join(path, str(mfcc_num) + '.bin')                 if os.path.exists(mfcc_path):                     mfcc = np.fromfile(mfcc_path)                     mfcc = mfcc.reshape(20, 12)                     mfcc_block[k4, 0, :, :] = mfcc                     k4 += 1                 # else:                 #     raise ("mfccs = 0")
in “Dataloader/Test_load_audio.py”, the code work!

@duantiantian I am a bit confused for I just checked the code, it runs normally on my computer.
The error means no audio files can be found, and you just removed the part for loading audio files, then how can it work? ><

@Hangz-nju-cuhk Thanks. I tested and it works on 0.4.0. Another problem: does this model also work for sentence level audio inputs?

@sarah-zhu I am not sure what do you mean by "sentence-level audio inputs", maybe our kind of method is segment-level audio inputs? We did crop the mfcc by time shifts to let it align with images, so if this process can be realised online in the dataloader, then it is possible to work with a sentence level mfcc feature.

@duantiantian I am a bit confused for I just checked the code, it runs normally on my computer.
The error means no audio files can be found, and you just removed the part for loading audio files, then how can it work? ><

@duantiantian I am a bit confused for I just checked the code, it runs normally on my computer.
The error means no audio files can be found, and you just removed the part for loading audio files, then how can it work? ><

Sorry, I just Comment out "else: raise ("mfccs = 0")", I thought after loading all the audio files, the code will raise an error rather break the for loop when the next audio file does not exist.
Thank you very much for your reply!!! ^-^

@Hangz-nju-cuhk Sorry, I just comment out "else: raise ("mfccs = 0")", I thought after loading all the audio files, the code will raise an error rather break the for loop when the next audio file does not exist.
Thank you very much for your reply!!! ^-^

@Hangz-nju-cuhk Hi, I tested your model on both LRW and Voxceleb datasets using provided mfcc extraction and face alignment code. However, I found the result is more accurate on Voxceleb dataset than on LRW. Is the provided model trained on Voxceleb dataset?

@sarah-zhu I am glad that you did further experiments!
If I remember correctly, this model is trained on the training set of LRW only, and so were the results provided in our paper and videos. However, we did further experiments to train our model on a small sample of Voxceleb (from the first 100 classes), and we do not find any difference in our testing results on Voxceleb or LRW comparing with before.
I think the main reason is that Voxceleb provides normally cleaner audio clips than LRW, and humans are less sensitive to long-term results in Voxceleb than shorter ones in LRW.

@Hangz-nju-cuhk getting this error using audio in google drive bin files
Traceback (most recent call last):
File "test_all.py", line 34, in
test_folder = Test_VideoFolder(root=opt.test_root, A_path=A_path, config=opt)
File "/content/Talking-Face-Generation-DAVS/Dataloader/Test_load_audio.py", line 92, in init
self.vid = self.loader(self.root, self.A_path, config=self.config)
File "/content/Talking-Face-Generation-DAVS/Dataloader/Test_load_audio.py", line 67, in Test_Outside_Loader
raise ("mfccs = 0")
TypeError: exceptions must be old-style classes or derived from BaseException, not str

@ak9250 @duantiantian modifiy raise ("mfccs = 0") to raise Exception("mfccs = 0") can fix the bug

@duantiantian I am a bit confused for I just checked the code, it runs normally on my computer.
The error means no audio files can be found, and you just removed the part for loading audio files, then how can it work? ><

Hello, the problem is not code, the problem is command line
change from
python test_all.py  --test_root ./0572_0019_0003/audio --test_type audio --test_audio_video_length 99 --test_resume_path ./checkpoints/101_DAVS_checkpoint.pth.tar
to
python test_all.py  --test_root ./0572_0019_0003/audio --test_type audio --test_audio_video_length 98 --test_resume_path ./checkpoints/101_DAVS_checkpoint.pth.tar
Because there are only 98 mfcc bin files


Init fails when database username contains '-'

Describe the bug
The CLI crashes when trying to initialize a new Zitadel instance when the init username contains a -.
To Reproduce
Steps to reproduce the behavior:
Create a new Zitadel instance configured for PostgreSQL with a username containing a -.
e.g. in my case
init-zitadel crashes
init_zitadel works
Expected behavior
Valid PostgreSQL usernames should be escaped thus not resulting in a fatal error.
Additional context
HelmChart version: 3.3.2
Zitadel version: v2.8.2
time="2022-10-18T20:44:38Z" level=info msg="initialization started" caller="/home/runner/work/zitadel/zitadel/cmd/initialise/init.go:72"
time="2022-10-18T20:44:38Z" level=warning msg="postgres is currently in beta" caller="/home/runner/work/zitadel/zitadel/internal/database/postgres/config.go:61"
time="2022-10-18T20:44:39Z" level=info msg="verify user" caller="/home/runner/work/zitadel/zitadel/cmd/initialise/verify_user.go:38" username=init-zitadel
time="2022-10-18T20:44:39Z" level=fatal msg="unable to initialize the database" caller="/home/runner/work/zitadel/zitadel/cmd/initialise/init.go:65" error="ERROR: syntax error at or near \"-\" (SQLSTATE 42601)"

Just got hit with this on v2.34.1
{"caller":"/home/runner/work/zitadel/zitadel/cmd/initialise/init.go:65","error":"ERROR: syntax error at or near \"-\" (SQLSTATE 42601)","level":"fatal","msg":"unable to initialize the database","time":"2023-08-26T07:04:04Z"}

Issue still exists on v2.35.3
level=fatal msg="unable to initialize the database" caller="/home/runner/work/zitadel/zitadel/cmd/initialise/init.go:65" error="ERROR: syntax error at or near \"-\" (SQLSTATE 42601)"

@zookatron and @adlerhurst I assume that code in https://github.com/zitadel/zitadel/pull/6928 could close this @skeletorXVI issue, isn't it?
@hifabienne, I'm reviewing open bugs in search of something quick that I can fix this week :smile:

@doncicuto Yes, that PR should fix this issue for you!

So I will close this issue, feel free to reopen, if there are still some problems.
@doncicuto thanks


RFC 85: Remove unused metadata

@marpaia if you could pull this one in... It's a small fix/change but it's causing us issues. Thx!

Thanks!


fix: Prevent Account section's action bar title from changing

Fixes #1892

@anhanh11001 That's set in AndroidManifest


Cannot harden design using option 2 and no macros

I'm attempting to harden my design using option 2 (https://github.com/efabless/caravel/tree/master/openlane#option-2). When I attempt make user_project_wrapper I get several errors and warnings and it finally errors out with:
[ERROR]: Floorplanning failed
[ERROR]: module decred_top not found in /project/openlane/user_project_wrapper/runs/user_project_wrapper/tmp/merged.lef
[ERROR]: Check whether EXTRA_LEFS is set appropriately

I don't have any macros in my design so all of the code is listed under VERILOG_FILES, I don't have any blackbox files or LEF or GDS imports.
If I create a similar script as a separate design (like option 1), I can get through that type of flow. It seems that the script execution between the two options are different, at least in the beginning.
The script being used is https://github.com/SweeperAA/skywater130_decred_miner/blob/master/openlane/user_project_wrapper/config.tcl
I've also modified the interactive.tcl as the instructions indicated (https://github.com/SweeperAA/skywater130_decred_miner/blob/master/openlane/user_project_wrapper/interactive.tcl).

    (file "/openLANE_flow/flow.tcl" line 168)
Makefile:25: recipe for target 'user_project_wrapper' failed
make: *** [user_project_wrapper] Error 1

@SweeperAA: The instructions were a bit lacking in the past, but they were updated in the current master. Please, try the new instructions out and inform us if you're still having any issues, here or on slack.

@SweeperAA: with the updated documentation and the removal of the interactive script, following option 2 has become much simpler and those who tried it out on mpw-one-b didn't face any issues with it. Thus, could we close this issue?


feat(vue-renderer): Hook VueMeta  SPA / Static to modify VueMeta data while generatating

Types of changes

[ ] Bug fix (a non-breaking change which fixes an issue)
[x] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
For SPA and static site builds, the generated HTML pages all use the meta options from nuxt.config.js. This is a problem for SEO (obviously) and for link sharing where the meta content of the HTML will be parsed to give a preview. While a universal or server-sided rendered app would solve this, I'd rather not have to deal with the complexities involved when there is a fairly simple solution (I only need the meta pre-rendered, nothing else).
What I'm proposing is a hook for vue-meta allowing the app to decide on a more granular level what meta to use for each route. I believe this is in-line with existing core functionality and could open up other use cases.
I would hope one day the vue-meta plugin could just work for this for static / spa built sites but I think this can be a good intermediate solution.
Example: with this code block a user can generate all their static HTML to have individual titles without any effort
hooks: {
    'vue-renderer': {
      spa: {
        meta ({ head, renderContext }) {
          const urlParts = renderContext.url.split('/')
          head.title = startCase(urlParts.pop())
        }
      }
    },
  },

Checklist:

[x] My change requires a change to the documentation. - No documentation page for vue-renderer hooks that I can see?
[ ] I have updated the documentation accordingly. (PR: #)
[ ] I have added tests to cover my changes (if not applicable, please state why) - Wasn't sure exactly where to put these tests, wasn't a SPA generate unit test from what I could see?
[ ] All new and existing tests are passing.

In retrospect this was really obvious. When I was experimenting with universal the first time I somehow ended up with a client / server folder, which I assumed wouldn't work for s3, I guess this was down to me not configuring it correctly.
Have just tried now and it's all good... So will close this PR

@loonpwn @pimlie can you explain to me what was obvious? I'm trying to deploy as a static site to S3, but I need SEO, while wanting the cost savings of serving statically from S3, I've explained the issue I'm having with nuxt generate in another issue thread here.
Is it possible to do nuxt build nuxt start then deploy that on S3?  What configurations were you alluding to that I can tweak to set this up?
Is there a guide I can consult?  The deployment guide for S3 only talks about using nuxt generate.
Thanks!

@Sford4
Is using the universal build mode with the configured routes, not an option? My use case is that using universal deployment (which server-side renders static HTML first) was causing some issues that were too hard to debug and it was increasing build time significantly.
We are now deploying a static s3 build of the site using spa/static mode. The code from this issue was actually deploying in another PR I did and was merged, so you can do the above to hack in head into the static HTML when using that spa mode.
The syntax is the same as the above and you would need to map out each individual page in the function, there is no way for it to read the head content of the component since it's not actually running the pages. To reiterate, this is a hacky solution and should only be used as a last resort, but it will sort out some SEO issues, our main issue was including the social sharing data.

@loonpwn Thanks for getting back!  I'm more than happy to use universal build mode with the configured routes, is there a guide for deploying this to AWS?  Right now deploying a static build to S3 works smoothly, and costs next to nothing to keep live.
Can I deploy the universal build to S3, set up for static web hosting, perhaps with the entry point .nuxt/server/index.ssr.html?  Pardon my ignorance, If there's documentation I can consult to get this up on S3 please let me know how I might find it!

@Sford4 It's a bit off-topic from this PR, but I'll try to cover it.
Firstly, you can't run the SSR scripts on the S3 bucket, the S3 web hosting only has support for serving content, not for running it on a server-side level.
Within your nuxt.config.js you should have your target set to 'static' and your mode set to the default (universal I think).
You should have a deploy script which looks like the following (this is mine with cloudfront support).
# Install deps
- yarn install --frozen-lockfile --non-interactive
# Generate dist folder
- nuxt generate
# Push Non-HTML assets with cache headers
- aws s3 cp dist s3://$S3_BUCKET --recursive --acl=public-read --include="*" --exclude "*.html" --cache-control max-age=31536000,public
# Push HTML pages with no cache headers
- aws s3 cp dist s3://$S3_BUCKET --recursive --acl=public-read --exclude="*" --include "*.html" --cache-control max-age=0,no-cache,no-store,must-revalidate
# Invalidate caches
- aws cloudfront create-invalidation --distribution-id=$CLOUDFRONT_ID --paths '/*'

If you're using cloudfront you set the root element to index.html and handle 404's with the 200.html file. This is probably explained in the nuxt docs somewhere if you dig around enough
Happy to assist you further over a DM via Twitter if you're still stuck.

Do we have any solution currently for the social sharing data not updated for dynamic routes for an AWS S3 deployment approach?


Improve performance

It takes several seconds to fill a 640x480 frame - not good!

Moved debugger connection to its own thread; 4x speedup from ~500k ticks/sec to 2M ticks/sec. Better, but still room for improvement.

Recent improvements and fixes have taken performance from ~250k ticks/sec (debug-unittest) to 27M ticks/sec in release. Good enough for now!


Allow setting local host name by an environment variable.

Output of restic version

restic 0.9.2 compiled with go1.11 on linux/amd64
What should restic do differently? Which functionality do you think we should add?

When the host name of my machine changes, making a new backup takes almost as much time as if it was the very first backup (though deduplication still works).  I think restic should allow setting host name with an environment variable, for example, RESTICLOCALHOSTNAME, by analogy with UNISONLOCALHOSTNAME for Unison.
What are you trying to do?

I would like to be able to make "incremental" backups quickly even when the local host name of my machine changes.
Did restic help you or made you happy in any way?

Yes, I use it for backing up my personal files.

Hey, thanks for the suggestion. Did you know there's a command-line option to do exactly that?
$ restic backup --help
[...]
  --hostname hostname        set the hostname for the snapshot manually. To prevent an expensive rescan use the "parent" flag
[...]

You can pass a host name explicitly, which is then used by restic.

No, i didn't know, sorry. Then it may be enough for me, thanks.

Cool, then I'll close this for now. I'd like to avoid adding more environment variables, at least unless we have a consistent story for configuring restic. Thanks a lot for taking the time to file this issue though! :)

Indeed, command line parameters are fine.
However, I just tried it on a dual-boot system where the same partition is mounted as /home, so i tried to make a backup under the same host name, but with different versions of restic: first 0.9.2, then 0.8.3.  The parent snapshot was correctly chosen as the previous one, but the operation took like 25 minutes, though the were no changes.  Is this because of different restic versions?

You've run into the new behavior of the new archiver code introduced in 0.9.0: https://github.com/restic/restic/blob/master/changelog/0.9.0_2018-05-21/issue-549

Ok, thanks for the explanation.  I suppose then that with the same version there will be no problems.


Where is documentation for Snapshot Policy

I haven't yet been given access to ANF on my subscription so I can't take a look at what the Snapshot Policy blade allows. I'm hoping that it will allow scheduling of snapshots and their deletion, so that it can be used for backup on hours/days/weeks/months/years schedules.

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

ID: 4e33e773-ccdf-47e6-ecdd-cb99ec16d029
Version Independent ID: f5a92bb9-68a0-666c-37cf-20a5ad5ced88
Content: Manage snapshots by using Azure NetApp Files
Content Source: articles/azure-netapp-files/azure-netapp-files-manage-snapshots.md
Service: azure-netapp-files
GitHub Login: @b-juche
Microsoft Alias: b-juche

I'm not sure about the support of snapshot policy for Azure NetApp Files.
cc @lichunli-ms and @ntap512 for them to respond.

@Aitor1972 if you still haven't been whitelisted for ANF, could you please send your subscription ID to<EMAIL_ADDRESS>and we can fasttrack your request?
Snapshot policy is not available at the moment but we're working on adding it.

@Aitor1972 please reach out again if you have further questions.  I'm closing this ticket for now.
#please-close

Thanks - I am whitelisted and I have seen a workaround involving a logic app, but it would be nice to have "in-blade" support, and also Azure Backup support for ANF

Thanks - I am whitelisted and I have seen a workaround involving a logic app, but it would be nice to have "in-blade" support, and also Azure Backup support for ANF

Thanks, @Aitor1972.  Copying @lichunli-ms so that she sees your input.

@Aitor1972 yes, we're working on adding a native support for snapshot policy. Adding native backup support is also on the roadmap.


Parsing issues with 2 name cities, spelled out numbers, and street directionals

First off, thank you for this awesome library!
Here are a few bad parse things I have encountered:
Sample set 1
P O BOX 21 OLD WASHINGTON OH 43768
BOX 67 OLD WASHINGTON OHIO 43768
PO BOX 67 OLD WASHINGTON OH 43768
P O BOX 152 OLD WASHINGTON OH 43758
OLD is being parsed as StreetNamePreModifier and should be PlaceName

Sample set 2
70433 OLD TWENTY ONE CAMBRIDGE OH 43725
66025 OLD TWENTY ONE CAMBRIDGE OH 43725
ONE is being parsed as PlaceName and should be StreetName

this one parses fine if ONE is followed by a StreetNamePostType, or is hyphenated  (TWENTY-ONE), or a numeral is used (21)

Sample set 3 - In all of these, W is parsed as PlaceName and should be StreetNamePostDirectional;
85 IRWIN S W SAFETY HARBOR FLA 34695
431 FRONT AVE S W NEW PHILADELPHIA OHIO 44663
7439 PALNER RD S W PATASKALA OH 43062

this is where the StreetNamePostDirectional was entered with a space between SW

7938 S CO RD 425 W SPICELAND IN 47385
52206 ZEP RD W PLEASANT CITY OH 43772
*W is parsed as PlaceName and should be StreetNamePostDirectional
Cheers,
blt2589

These are awesome, thanks @blt2589! We'll add them to the next round of training data.


plasma5 powerdevil: power management kcm broken

Describe the bug
Cannot adjust brightness in "Energy saving" KCM because it errors saying it could not be loaded because the power management service is not running.
And indeed, it failed to start:
$ journalctl -xb | grep powerdev
Nov 30 14:31:05 fr-laptop systemd[1241]: Not generating service for XDG autostart app-powerdevil-autostart.service, only Type=Application is supported.
Nov 30 14:31:21 fr-laptop systemd[1473]: Not generating service for XDG autostart app-powerdevil-autostart.service, only Type=Application is supported.
Nov 30 14:31:30 fr-laptop xsession[1639]: powerdevil: "The shared library was not found."
Nov 30 14:31:30 fr-laptop xsession[1639]: powerdevil: KDE Power Management System init failed!
Nov 30 14:31:30 fr-laptop dbus-daemon[651]: [system] Activating service name='org.kde.powerdevil.discretegpuhelper' requested by ':1.36' (uid=1000 pid=1639 comm="/nix/store/gy01blsvn2z4cgc7m1450z5xcfkprv5g-powerd" label="kernel") (using servicehelper)
Nov 30 14:31:30 fr-laptop org.kde.powerdevil.discretegpuhelper[1695]: QDBusArgument: read from a write-only object
Nov 30 14:31:30 fr-laptop org.kde.powerdevil.discretegpuhelper[1695]: QDBusArgument: read from a write-only object
Nov 30 14:31:30 fr-laptop org.kde.powerdevil.discretegpuhelper[1695]: QDBusArgument: read from a write-only object
Nov 30 14:31:30 fr-laptop dbus-daemon[651]: [system] Successfully activated service 'org.kde.powerdevil.discretegpuhelper'
Nov 30 14:31:30 fr-laptop xsession[1639]: powerdevil: org.kde.powerdevil.discretegpuhelper.hasdualgpu failed
Nov 30 14:31:30 fr-laptop xsession[1639]: powerdevil: "DBus Backend error: could not contact the helper. Connection error: No such method 'performAction' in interface 'org.kde.kf5auth' at object path '/' (signature 'sayay'). Message error: "
Nov 30 14:31:30 fr-laptop xsession[1546]: org.kde.plasma.session: Starting autostart service  "/run/current-system/sw/etc/xdg/autostart/powerdevil.desktop" ("/nix/store/mpnp0rq6addxjpmx1fvppnrd72zflaim-powerdevil-5.18.5/libexec/org_kde_powerdevil")

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
21.03.20201124.6f0c009
github:nixos/nixpkgs/6f0c00907bbd81b47052eef59f7b284926a77289

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute: powerdevil
# a list of nixos modules affected by the problem
module: plasma5

Not restoring session on startup (https://github.com/NixOS/nixpkgs/issues/24563#issuecomment-315056605) results in a functioning power management kcm again.

Related issue https://github.com/NixOS/nixpkgs/issues/69150.

OK, so disabling restoring session startup fixed the kcm. Then, enabling it again and rebooting allowed me to adjust screen brightness.

related https://github.com/NixOS/nixpkgs/issues/102253


Solution can not be load in VS 2017(V15.7.4)

Solution can not be load in VS 2017(V15.7.4)

Maybe it's a bug inside Global.json
Open file Global.json and remove Sdk property
Old:
{   "projects": [ "src", "test" ],   "sdk": {     "version": "2.1.200"   } }
New:
{   "projects": [ "src", "test" ] }

thanks, fixed!


No docs for config module

The config module that every provider exposes is not currently documented in the Resource docs.  Here is the equivalent in the Node.js specific docs:
https://www.pulumi.com/docs/reference/pkg/nodejs/pulumi/aws/config/

It should be noted that the resource docs do expose a Provider resource which effectively documents the same surface area.

It should be noted that the resource docs do expose a Provider resource which effectively documents the same surface area.


spacing of cdot between numbers

$0 \cdot 5$ is rendered with unexpected spacing.
$0$$\cdot$$5$ works fine.

Here is a comparison of $0\cdot 5$1 in TeX and MathJax:

MathJax seems to be spot on to me.  If this is not what you are seeing, then please provide a screen shot that shows the results you are getting.

If you really want no spacing around the dot, then $0{\cdot}5$ is probably a better choice than $0$$\cdot$$5$.

$0{\cdot}5$
Thanks, this is exactly what I wanted to know.


Update managed-networks.md

Not a python expert but this example seems to deadlock if there's a single hanging TCP connection refusing to close.
Can repro with ...
$ openssl s_client -connect <IP_ADDRESS>:3333

... and a parallel curl.
Basically simulating two WARP clients trying to connect to the server. First client leaves connection hanging, second client can't connect and is unable to detect Managed Network.

@dh-cf


bug: inmemory provider panic

In certain circumstances the inmemory provider panics and crashes the dns operator
@mikenairn Add more details

Fixed in https://github.com/Kuadrant/dns-operator/pull/284


Removing before and after hooks...

@olefredrik @colin-marshall Working through some refactoring of things. What do you think of removing all the hooks for before and after content, comments, etc.? Are people using these at all? Taking them out makes things look a lot cleaner and possibly easier to upkeep.

I personally don't use them but others might. @olefredrik I would like to hear your opinion on this and also could you please add a "feedback wanted" label so we can hear from others on this? Thanks!

I never had to use them... Most of the time I let them here, but...

I've actually used these on my business website - they're really handy. I don't know why you would remove a core feature of Wordpress?

I agree with @FlatspinZA . Don't think we should remove it as it's a core feature of WordPress that some people find handy.

@FlatspinZA @olefredrik we're not talking about removing a core feature of WordPress. We're talking about the FoundationPress specific hooks like do_action( 'foundationpress_post_before_entry_content' ).
It makes more sense to me to just edit the template files with whatever you want to put before or after the content instead of writing functions that use hooks to echo out html elements. You get a clearer picture of what's going on in the template that way. I also see no similar hooks in underscores.

Hooks are a WP core feature, although the FoundationPress spesific hooks is not. I'm sure there are many like us that thinks it makes more sense to just edit the template files instead of using hooks in functions. I feel the same way.
But I'm not 100% sure if it's cool to simply remove the hooks, if people find it useful. The Genesis Framework uses hooks. Underscores doesn't.

I just used this specific hook in a current project today. I guess people can make any hook they like when needed, but I like this one because I can target just before the content.
Maybe it is more about those that like a theme where most of the changes reside in the functions.php file. Others, like Colin, prefer to view each template file with special content inside. I  can see the benefits of both. If people know how to use a hook, they probably know how to add one...

Doesn't Wordpress already have hooks built in where you can put html snippets before and after things like the_title and the_content?
I think Underscores is a pure "starter"  theme. So, it is very barebones. Where something like Genesis is often used with a child theme, so it has a lot of stuff in it.
I guess it just depends the direction you want to take FoundationPress. I come from roots/sage where I think they take glee in eliminating things from the template files.
It's all good.

Good feedback so far guys, keep it coming.
@olefredrik for Genesis they make sense because people build themes on top the of the Genesis Framework. We're not a framework per se. Also, it seems to me that a lot of themes that have actions to hook into, actually have code in there that already uses them, whereas ours are there to use if you want to but they don't do anything without you adding a function that uses them.
@ChrisAynesworth good points. I see where it might make the code more "DRY" to only use functions and hooks. I personally would use template-parts to accomplish the same thing without functions.
Maybe a good compromise here would be to keep the content action hooks and remove the comment ones?

@colin-marshall They are really all over the place. In the sidebar.php, header.php, footer.php . Basically across all template files.

I found this post in a WP core issue discussion very interesting/relevant, specifically:

I was one of the original theme authors to pioneer the use of template hooks. They were meant to solve one problem: Allow theme users to add stuff into a parent theme because WordPress did not allow child themes to overwrite templates. Now, with the ability to overwrite templates and the get_template_part() function, template hooks are becoming rather useless.

So it looks like @JPOak is correct that template hooks were meant for child themes, and 5 years ago they were essentially declared obsolete by the guy who came up with them.
Now if somebody wants to use hooks over get_template_part(), that's their choice, but I'm all for the leaner is better approach to a starter theme. And as @ChrisAynesworth pointed out already,

"If people know how to use a hook, they probably know how to add one..."

Actually I was not aware that hooks were meant for a parent / child theme setup. FoundationPress was never meant to be a parent theme, but a plain starter-theme that people can use directly as a starting point for new projects. I agree with your arguments to remove the hooks. If we do, I suggest we remove both the content and comment hooks. Feel free to create a pull request :)

Well, then, @colin-marshall - you have made your point. Probably me just being lazy-ass. :P

@colin-marshall  @olefredrik Does this mean we can we move them from the entire theme? Including in the header.php, sidebar.php, footer.php and everywhere else? If so, i’ll do the PR . I’ve been wanting to do this since I first used FP.

@JPOak yes I believe it's ok to remove all hooks you mentioned.

I'm late to this, but thank you @JPOak! In my own forks I've removed all these hooks long ago but they still haunt me every time i diff for changes, so I'm very happy they are gone! 👍

@Aetles Yes. It was a good time.


Still have error in reproducing “Tutorial.ipynb”: deconvolution part

Hi,
I observed that you have updated the codes. However, some problems have not been solved yet.
1) In "Codes/Deconvolution/Stereoscope_pipeline.py", Line 51: change "scvi.data.setup_anndata" to "RNAStereoscope.setup_anndata"
2) In "Codes/Deconvolution/Stereoscope_pipeline.py", Line 58: change "scvi.data.setup_anndata" to "SpatialStereoscope.setup_anndata"
3) In "Codes/Deconvolution/Cell2location_pipeline.py", Line 47: change "scvi.data.setup_anndata" to "cell2location.models.RegressionModel.setup_anndata"
4) In "Codes/Deconvolution/Cell2location_pipeline.py", Line 77: change "scvi.data.setup_anndata" to "cell2location.models.Cell2location.setup_anndata"
5) In "Codes/Deconvolution/Cell2location_pipeline.py", Line 78: mute it
6) In "doc/Tutorial.ipynb", change all "my_python_path" to "python_path" to let it be identical to "Benchmarking/DeconvolutionSpot.py"
7) In "Codes/Deconvolution/SpatialDWLS_pipeline.r", let output_path = args[5] and let my_python_path = args[4]. Just letting it be identical to "Benchmarking/DeconvolutionSpot.py"
8) "Codes/Deconvolution/DSTG_pipeline.r" is missing.

I am still tring to reproduce it and will pose bug to this if I meet.
I wish authors should provide every packages' versions, which will largely facility users to reproduce. For example, it seems that authors used a very old version of SPOTlight (from observing the code. The latest and the old versions of SPOTlight are totally different). However, users cannot repeat or successfully finish the tutorial if they follow the published article to install packages.

Hi~  thank you so much for your suggestion! ! !
We used every packages' versions as: Cell2location(Version 0.6a0), DestVI (scvi-tools Version 0.11.0), Stereoscope (within the scvi-tools Version 0.11.0), Seurat(Version 4.0.5), RCTD(Version 1.2.0), SPOTlight(Version 0.1.7), SpatialDWLS(by Giotto of Version 1.0.4), Tangram(Version 1.0.0), STRIDE(Version 0.0.1b), DSTG(Version 0.0.1)
For DSTG, It is more easier to run by their github tutorial(https://github.com/Su-informatics-lab/DSTG).
We will check and update the codes for new version later.
Thanks again!

May I ask you what Python version are you using? It seems that the scvi-tools in version 0.11.0 is conflict with Python 3.6.


Magic calls cause issues with forward()

Given a full mock of a class with magic methods:
class MagicClass
{
    public function __call($name, $arguments)
    {
        return 'real';
    }
}

$proxy = Phony::fullMock('MagicClass');

Using forwards() does not work:
$proxy->nonexistent->forwards();
var_dump($proxy->mock()->nonexistent()); // outputs NULL

This is because __call is still returning null, and also needs to be
forwarded in order to work as expected:
$proxy->__call->forwards();
$proxy->nonexistent->forwards();
echo $proxy->mock()->nonexistent(); // outputs 'real'

One solution could be to detect magic methods when forwards() is called, and
automatically forward __call() also. Unfortunately this would probably forward
ALL other magic calls, which could be very bad. For example, you would not want
this when mocking an Isolator.
The other solution is to bypass the mocked __call method. I think this might be the best solution, but it would mean that you could no longer stub __call and have it affect all magic methods.

@jmalloc Would value your opinion on this one.

I think the current solution will be fine, so long as it's documented well.


MINOR Fix trunk build scan

Trunk builds are run off of "push" events rather than "pull_request". We were missing some logic in the is-public-fork condition that mistakenly caused some trunk builds to skip the build scan.

I also noticed some builds failed to publish their build scan due to ge.apache.org being offline. We should archive the build scan in all cases so we can have the possibility of manually publishing them later.


newsraft: init at 0.21

Description of changes

https://codeberg.org/grisha/newsraft
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Also something related to the nixpkgs PRs that I've seen is that each should contain a single commit with the good convention name.
I don't think people are squashing the commits before merging.

Make the commits into this history

maintainers: add arthsmn
newsraft: init at 0.21

Looks good. Gave it a try and worked as expected. Nice one!

@Steinhagen would you go to the files section and add a review approval if you can. This will help the pr get merged. Thanks for your help!

Thank you Anderson!

@arthsmn Yo, thanks for adding this! I've had it on my list for weeks to add this one next, but here I see you already did. Cheers.


Argument of type Record<string, string> is not assignable to parameter of type Traits | Callback | null | undefined

Breaking change introduced
I have a breaking change trying to upgrade 1.41.0 -> 1.42.1
Expected
I should be able to pass any Record<string, string | number | JSONValue> as traits into the identify call.
Actual
I can only only pass Record<string, JSONValue>
Culprit
Looks like here was a breaking change in<EMAIL_ADDRESS>I believe https://github.com/segmentio/analytics-next/pull/561 is the culprit, specifically these changes

@zygopleural I wasn't able to quickly reproduce.
This works:
const traits: Record<string, string> =  {name: "hello"}
analyics.identify("foo", traits)

const { id, picture, signInMethod, ...traits } = user
identify(id, traits)

I'm seeing

Even if I do
const t = traits as Traits

@zygopleural  This is an interesting one. Can you create a type and then cast it to that?
type UserTraits = {
  name: string
  age: string
}
const { id, picture, signInMethod, ...traits } = user
identify(id, traits as UserTraits)

@zygopleural  be fixed in 1.42.2

FYI @zygopleural there is a recently active open issue about this scanario: https://github.com/microsoft/TypeScript/issues/48014

FYI @zygopleural there is an open Typescript issue about this scenario / bug: https://github.com/microsoft/TypeScript/issues/48014

Ah nice find!


Add status code to pb to indicate the status actually

closed by https://github.com/dst-project/dst/pull/99


Remove example of reference() using apiversion[0]

This pattern leads to weakly versioned templates.  We shouldn't promote it in the docs.

Hi @rjmax, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (ryjones). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;


Add initial fuzz target: mp4parse.

How do these get run?

cargo rustc -- -C passes='sancov' -C llvm-args='-sanitizer-coverage-level=3' -Z sanitizer=address

CI is coming.

We should probably check in a script that lets you fuzz these (or somehow make it work with cargo-fuzz)
Also, shouldn't we use RUSTFLAGS? otherwise mp4parse won't be instrumented.

Is setting RUSTFLAGS required to instrument?

Yes, without it only your fuzz script will be instrumented, so the fuzzer can't be as smart about it


⚠ Bofrost.lu has degraded performance

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

HTTP code: 200
Response time: 2382 ms

Resolved: Bofrost.lu performance has improved in 54abac8 after 8 minutes.


Chrome OS not responding when back from Ubuntu

Hi,
First I wanna say good job to make this work so smoothly in general. Unfortunately I'm getting some weird behavious right now...
Since this morning when I get back on Chrome OS from crouton (without quitting it) I find that I can't type anymore except to enter my password (as it came back from sleep both times). But the key combination to switch to Ubuntu works fine and I can still type in it (which I'm doing right now, as the bug appeared for the second time -- this morning I just rebooted the laptop without thinking any further).
As I'm writing this message I tried different and I thought was a bug linked to the keyboard seems more like a global Chrome lag. I can open the application drawer and the notifications but that's pretty much all. The tabs in Chrome become unresponsive and I get an alert asking me if I want to close them or wait.
I trie closing all the tabs (leaving the browser totally closed -- even the shell, as Ubuntu is running in the background) and all the Chrome OS apps but it didn't change anything: only Ubuntu works properly.
The only thing that changed recently is that I tried grive yesterday (but removed it just after) and installed Insync yesterday night, which is running right now and syncing some of my Google Drive folders (as that's the only way I found to access these files from Ubuntu, even though Chrome OS allows direct access to them).
Any idea what might be causing this?

Can you post the output of croutonversion from your chroot?

crouton: version 1-20141208100324~master:7b8ffff0
release: trusty
architecture: amd64
targets: audio,chrome,xfce,keyboard,touch
host: version 6680.58.0 (Official Build) stable-channel peppy

You might try removing the touch target. I tried it and found it messed with everything.
I can't tell you how to remove the target. I deleted the chroot and started over.

It looks like the 'touch' target installs the 'touchegg' package and
creates a file named '/etc/touchegg.conf'.
On Thu, Mar 5, 2015 at 4:13 PM, Watney<EMAIL_ADDRESS>wrote:

You might try removing the touch target. I tried it and found it messed
with everything.
I can't tell you how to remove the target. I deleted the chroot and
started over.
—
Reply to this email directly or view it on GitHub
https://github.com/dnschneid/crouton/issues/1473#issuecomment-77452774.

--
DennyL@GMail

I'm having this same issue with no touch installed. I have an Asus C300. croutonversion output is below:
crouton: version 1-20150423153254~master:2f55723c
release: trusty
architecture: amd64
xmethod: xiwi
targets: xiwi,xorg,x11,kde,keyboard
host: version 6812.75.2 (Official Build) stable-channel quawks
kernel: Linux localhost 3.10.18 #1 SMP Tue Apr 14 20:45:15 PDT 2015 x86_64 x86_64 x86_64 GNU/Linux
freon: no

@rmkraus I think you are experiencing #1475 please respond to that issue if you can confirm it is the same so we can reopen it.

I believe I am experiencing that issue. Thanks.


Return empty matrix immediately on request for browsers:

Codecov Report

Merging #227 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #227      +/-   ##
==========================================
+ Coverage   62.81%   62.85%   +0.04%
==========================================
  Files          36       36
  Lines        2291     2294       +3
  Branches      151      152       +1
==========================================
+ Hits         1439     1442       +3
  Misses        852      852

Impacted Files
Coverage Δ

lib/client/api_matrix.es6.js
57.83% <100%> (+0.47%)
:arrow_up:

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 931be45...d10a3f8. Read the comment docs.


Default mode for git reset

The git reset command can be performed with various modes (--soft, --mixed, --hard, --merge, --keep)
Trying to perform a reset through sourcegit opens a pop-up asking to choose a mode among those
The default choice in the pop-up (--soft) does not match the default mode in the git command line (--mixed)
I assume that this is not on purpose (though if there is an actual demand for changing the default to --soft, I suggest adding a field in the Preferences menu instead to let the users tune the behavior)

Now the default reset mode has been changed to --mixed, you can download the latest build from Github Action.


Clearly separate tests that depend on torque

Right now, some tests require torque to have been previously loaded. As suggested here, we can create two bundles of tests (one using just cartodb.js and another using cartodb.js with torque).
cc: @viddo

Liked I mentioned in https://github.com/CartoDB/cartodb.js/pull/799/files#r45091920 there already exist a test bundle for "using cartodb.js with torque" so I think it's just a matter of extracting the tests to be run there instead.


Download Qt sources from archive instead of git repo

At the origin, the idea of using the git repo was to make the download faster by only downloading the modules that will be built. However, the result is not positive. Cloning the git repo takes a lot of time and often fails due to network errors.
The package should be updated to download the sources from the archive instead.

May I suggest downloading only the selected module, for example using github : https://github.com/qt/qt5/archive/v5.8.0.zip https://github.com/qt/qtbase/archive/v5.8.0.zip etc.
Qt itself also proposes split modules src packages ( http://download.qt.io/official_releases/qt/5.8/5.8.0/ ) but I cannot find the root qt5 archive with top level configure

The architecture of Conan implies that the source step should not depend on options:

Note that there is only one source folder for all the binary packages. If some source code is to be generated that will be different for different configurations, it cannot be generated in the source() method, it has to be done in the build() method.

Therefore, I believe it is inappropriate to conditionally sync submodules in the source step based on options. I need to be able to build a new configuration without rerunning the source step.
Downloading the archive would fix this issue, since it has everything. Or we could indiscriminately sync all submodules during source.
If we want to optimize the size of the fetch by only pulling what we need, we'd need unique conan recipes for each module.

Cloning the git submodules is still the longest solution. The fastest is to download all the module archives from qt.io, as I did in the pull request #35 . Unfortunately it seems @osechet does not have much time for maintaining this repository during the last weeks.

Yes, the archive download seems appropriate in this case.
FYI. In recent discussions with the conan maintainers, they favor splitting up monoliths like Qt into individual conan recipes. Although that sounds good in theory, Qt has a single configure/build entry point for the collection of modules. In order to modularize the recipes the build system needs to be modularized as well, much like Boost's more recent cmake modularization.

Thanks for all these ideas. I agree with @kenfred, using separate modules is not possible. I tried it with another previous package (conan-qtbase) and it couldn't work. My idea when I created this issue was to directly download the official source archive from Qt's website (from http://download.qt.io/archive/qt/). The archive contains everything and it would make it possible to rebuild later with others options.


POC: Moving redirectMap + hot reload to custom element

-- DO NOT MERGE --
This is a proof of concept of moving the redirect map and hot reload functionality to a custom element -- then this custom element is used to provide the redirect map to an app.
Option 1
Custom element provides redirect map to the Widget, widget passes it through config to children
https://github.com/NEARBuilders/near-bos-webcomponent/assets/16282460/37681842-6aeb-470b-a4ac-8577556ce603
hot reload still does not work
Option 2
Custom element takes an "index" src, and handles the rendering for <Widget inside with redirect Map
https://github.com/NEARBuilders/near-bos-webcomponent/assets/16282460/27380e3d-edba-452c-b276-e4b681f4baeb
hot reload still does not work

Not practical, closing


Fix keystone services and endpoints

Remove the cinderv2 keystone service and endpoints. Cinder's APIv2 was removed in Xena.
The cinderv3 endpoints are updated to remove the project_id from the API URLs. Cinder no longer requires a project_id in the URL as of the Yoga release.
NOTE: This patch does not remove stale cinderv2 artifacts from an existing deployment. Due to another issue with the finalizers (to be addressed separately), the following commands can be executed to remove the v2 service and endpoints:
$ oc patch keystoneservice/cinderv2 --type json
--patch='[ { "op": "remove", "path": "/metadata/finalizers" } ]'
$ oc delete keystoneservice/cinderv2
$ oc patch keystoneendpoints/cinderv2 --type json
--patch='[ { "op": "remove", "path": "/metadata/finalizers" } ]'
$ oc delete keystoneendpoints/cinderv2

/lgtm