Improve handling of sensors when remote sensors are in use

Corrects previous behavior that unconditionally treated sensor 0 as indoor and sensor 1 as outdoor. When using any sensor other than the internal temperature sensor as the control source, sensor 1 becomes the control temperature, and sensor 2 the outdoor. Sensors are now matched by name with "Space Temp" used as the indoor temperature when present, and "Thermostat" when it is not. API documentation gives no indication that these names differ based on region or language, so there is no obvious reason to believe they shouldn't remain consistent.
Also appears to resolve #21.
Testing with IP: <IP_ADDRESS>
logging in...
DEBUG:urllib3.connectionpool:Starting new HTTP connection (1): <IP_ADDRESS>:80
DEBUG:urllib3.connectionpool:http://<IP_ADDRESS>:80 "GET / HTTP/1.1" 200 None
Login successful.   I think there is still no support for reading my second sensor, because it's not marked outdoor:
{"sensors":[{"name":"Thermostat","temp":77.0,"hum":20},{"name":"Space Temp","temp":77.0},{"name":"Remote","temp":76.0}]}

You can call get_sensors("Remote", "temp") to get that value. I just added get_sensor_list to enumerate sensor names. Is there something else you're specifically looking for?
Testing with IP: <IP_ADDRESS>
logging in...
DEBUG:urllib3.connectionpool:Starting new HTTP connection (1): <IP_ADDRESS>:80
DEBUG:urllib3.connectionpool:http://<IP_ADDRESS>:80 "GET / HTTP/1.1" 200 None
Login successful. API: 9 Type: residential
DEBUG:urllib3.connectionpool:Starting new HTTP connection (1): <IP_ADDRESS>:80
DEBUG:urllib3.connectionpool:http://<IP_ADDRESS>:80 "GET /query/info HTTP/1.1" 200 None
Was able to get info:{'name': 'Living Room', 'mode': 1, 'state': 1, 'fan': 0, 'fanstate': 1, 'tempunits': 0, 'schedule': 0, 'schedulepart': 255, 'away': 0, 'spacetemp': 73.0, 'heattemp': 74.0, 'cooltemp': 82.0, 'cooltempmin': 35.0, 'cooltempmax': 99.0, 'heattempmin': 35.0, 'heattempmax': 99.0, 'activestage': 1, 'hum_active': 0, 'hum': 41, 'hum_setpoint': 45, 'dehum_setpoint': 55, 'setpointdelta': 2.0, 'availablemodes': 0}
Name is Living Room
Fan is 0
Heat setpoint is 74.0
Cool setpoint is 82.0

DEBUG:urllib3.connectionpool:Starting new HTTP connection (1): <IP_ADDRESS>:80
DEBUG:urllib3.connectionpool:http://<IP_ADDRESS>:80 "GET /query/sensors HTTP/1.1" 200 None
Was able to get sensors:{'sensors': [{'name': 'Thermostat', 'temp': 74.0, 'hum': 41}, {'name': 'Space Temp', 'temp': 73.0}, {'name': 'Outdoor', 'temp': 48.0}, {'id': 18, 'name': 'HVAC Supply', 'temp': 109.0, 'battery': 100, 'type': 'Supply'}, {'id': 19, 'name': 'HVAC Return', 'temp': 72.0, 'battery': 100, 'type': 'Return'}, {'id': 0, 'name': 'Dining Room', 'temp': 73.0, 'battery': 100, 'type': 'Remote'}, {'id': 1, 'name': 'Laundry Bath', 'temp': 73.0, 'battery': 100, 'type': 'Remote'}, {'id': 2, 'name': 'Master Bedroom', 'temp': 73.0, 'battery': 100, 'type': 'Remote'}, {'id': 3, 'name': 'Master Bath', 'temp': 73.0, 'battery': 100, 'type': 'Remote'}, {'id': 4, 'name': 'Kitchen', 'temp': 73.0, 'battery': 100, 'type': 'Remote'}, {'id': 5, 'name': 'Office', 'temp': 73.0, 'battery': 100, 'type': 'Remote'}, {'id': 6, 'name': 'Guest Bath', 'temp': 73.0, 'battery': 100, 'type': 'Remote'}, {'id': 7, 'name': 'Guest Bedroom', 'temp': 73.0, 'battery': 100, 'type': 'Remote'}]}
Indoor temp is 73.0 and humidity is 41
Outdoor temp is 48.0

Thermostat temp is 74.0, humidity is 41, battery is None, and type is None
Space Temp temp is 73.0, humidity is None, battery is None, and type is None
Outdoor temp is 48.0, humidity is None, battery is None, and type is None
HVAC Supply temp is 109.0, humidity is None, battery is 100, and type is Supply
HVAC Return temp is 72.0, humidity is None, battery is 100, and type is Return
Dining Room temp is 73.0, humidity is None, battery is 100, and type is Remote
Laundry Bath temp is 73.0, humidity is None, battery is 100, and type is Remote
Master Bedroom temp is 73.0, humidity is None, battery is 100, and type is Remote
Master Bath temp is 73.0, humidity is None, battery is 100, and type is Remote
Kitchen temp is 73.0, humidity is None, battery is 100, and type is Remote
Office temp is 73.0, humidity is None, battery is 100, and type is Remote
Guest Bath temp is 73.0, humidity is None, battery is 100, and type is Remote
Guest Bedroom temp is 73.0, humidity is None, battery is 100, and type is Remote

DEBUG:urllib3.connectionpool:Starting new HTTP connection (1): <IP_ADDRESS>:80
DEBUG:urllib3.connectionpool:http://<IP_ADDRESS>:80 "GET /query/runtimes HTTP/1.1" 200 None
Runtimes: [{'ts':<PHONE_NUMBER>, 'heat1': 455, 'heat2': 0, 'cool1': 0, 'cool2': 0, 'aux1': 0, 'aux2': 0, 'fc': 0}, {'ts':<PHONE_NUMBER>, 'heat1': 369, 'heat2': 0, 'cool1': 0, 'cool2': 0, 'aux1': 20, 'aux2': 0, 'fc': 0}, {'ts':<PHONE_NUMBER>, 'heat1': 492, 'heat2': 0, 'cool1': 0, 'cool2': 0, 'aux1': 0, 'aux2': 0, 'fc': 0}, {'ts':<PHONE_NUMBER>, 'heat1': 352, 'heat2': 0, 'cool1': 0, 'cool2': 0, 'aux1': 19, 'aux2': 0, 'fc': 0}, {'ts':<PHONE_NUMBER>, 'heat1': 318, 'heat2': 0, 'cool1': 0, 'cool2': 0, 'aux1': 36, 'aux2': 0, 'fc': 0}, {'ts':<PHONE_NUMBER>, 'heat1': 353, 'heat2': 0, 'cool1': 0, 'cool2': 0, 'aux1': 24, 'aux2': 0, 'fc': 0}, {'ts':<PHONE_NUMBER>, 'heat1': 89, 'heat2': 0, 'cool1': 0, 'cool2': 0, 'aux1': 19, 'aux2': 0, 'fc': 0}]
DEBUG:urllib3.connectionpool:Starting new HTTP connection (1): <IP_ADDRESS>:80
DEBUG:urllib3.connectionpool:http://<IP_ADDRESS>:80 "POST /control HTTP/1.1" 200 None
DEBUG:urllib3.connectionpool:Starting new HTTP connection (1): <IP_ADDRESS>:80
DEBUG:urllib3.connectionpool:http://<IP_ADDRESS>:80 "GET /query/info HTTP/1.1" 200 None
Heat setpoint is 60.0
Cool setpoint is 90.0

DEBUG:urllib3.connectionpool:Starting new HTTP connection (1): <IP_ADDRESS>:80
DEBUG:urllib3.connectionpool:http://<IP_ADDRESS>:80 "POST /control HTTP/1.1" 200 None
DEBUG:urllib3.connectionpool:Starting new HTTP connection (1): <IP_ADDRESS>:80
DEBUG:urllib3.connectionpool:http://<IP_ADDRESS>:80 "GET /query/info HTTP/1.1" 200 None
Heat setpoint is 74.0
Cool setpoint is 82.0

Aha, I understand, ok, thanks!

Ok, good to hear that'll work for you.
After some more thought, I added an optional filter for type to get_sensor_list and some logic to infer the sensor type for non-wireless sensors.
Testing with IP: <IP_ADDRESS>
logging in...
DEBUG:urllib3.connectionpool:Starting new HTTP connection (1): <IP_ADDRESS>:80
DEBUG:urllib3.connectionpool:http://<IP_ADDRESS>:80 "GET / HTTP/1.1" 200 None
Login successful. API: 9 Type: residential
DEBUG:urllib3.connectionpool:Starting new HTTP connection (1): <IP_ADDRESS>:80
DEBUG:urllib3.connectionpool:http://<IP_ADDRESS>:80 "GET /query/info HTTP/1.1" 200 None
Was able to get info:{'name': 'Living Room', 'mode': 1, 'state': 3, 'fan': 0, 'fanstate': 0, 'tempunits': 0, 'schedule': 0, 'schedulepart': 255, 'away': 0, 'spacetemp': 73.0, 'heattemp': 74.0, 'cooltemp': 82.0, 'cooltempmin': 35.0, 'cooltempmax': 99.0, 'heattempmin': 35.0, 'heattempmax': 99.0, 'activestage': 1, 'hum_active': 0, 'hum': 40, 'hum_setpoint': 45, 'dehum_setpoint': 55, 'setpointdelta': 2.0, 'availablemodes': 0}
Name is Living Room
Fan is 0
Heat setpoint is 74.0
Cool setpoint is 82.0

DEBUG:urllib3.connectionpool:Starting new HTTP connection (1): <IP_ADDRESS>:80
DEBUG:urllib3.connectionpool:http://<IP_ADDRESS>:80 "GET /query/sensors HTTP/1.1" 200 None
Was able to get sensors:{'sensors': [{'name': 'Thermostat', 'temp': 75.0, 'hum': 40}, {'name': 'Space Temp', 'temp': 73.0}, {'name': 'Outdoor', 'temp': 48.0}, {'id': 18, 'name': 'HVAC Supply', 'temp': 78.0, 'battery': 100, 'type': 'Supply'}, {'id': 19, 'name': 'HVAC Return', 'temp': 73.0, 'battery': 100, 'type': 'Return'}, {'id': 0, 'name': 'Dining Room', 'temp': 74.0, 'battery': 100, 'type': 'Remote'}, {'id': 1, 'name': 'Laundry Bath', 'temp': 74.0, 'battery': 100, 'type': 'Remote'}, {'id': 2, 'name': 'Master Bedroom', 'temp': 74.0, 'battery': 100, 'type': 'Remote'}, {'id': 3, 'name': 'Master Bath', 'temp': 74.0, 'battery': 100, 'type': 'Remote'}, {'id': 4, 'name': 'Kitchen', 'temp': 73.0, 'battery': 100, 'type': 'Remote'}, {'id': 5, 'name': 'Office', 'temp': 74.0, 'battery': 100, 'type': 'Remote'}, {'id': 6, 'name': 'Guest Bath', 'temp': 74.0, 'battery': 100, 'type': 'Remote'}, {'id': 7, 'name': 'Guest Bedroom', 'temp': 74.0, 'battery': 100, 'type': 'Remote'}]}
Indoor temp is 73.0 and humidity is 40
Outdoor temp is 48.0

All sensors:
Thermostat temp is 75.0, humidity is 40, battery is None, and type is Local
Space Temp temp is 73.0, humidity is None, battery is None, and type is Control
Outdoor temp is 48.0, humidity is None, battery is None, and type is Outdoor
HVAC Supply temp is 78.0, humidity is None, battery is 100, and type is Supply
HVAC Return temp is 73.0, humidity is None, battery is 100, and type is Return
Dining Room temp is 74.0, humidity is None, battery is 100, and type is Remote
Laundry Bath temp is 74.0, humidity is None, battery is 100, and type is Remote
Master Bedroom temp is 74.0, humidity is None, battery is 100, and type is Remote
Master Bath temp is 74.0, humidity is None, battery is 100, and type is Remote
Kitchen temp is 73.0, humidity is None, battery is 100, and type is Remote
Office temp is 74.0, humidity is None, battery is 100, and type is Remote
Guest Bath temp is 74.0, humidity is None, battery is 100, and type is Remote
Guest Bedroom temp is 74.0, humidity is None, battery is 100, and type is Remote

Control sensors:
Space Temp temp is 73.0, humidity is None, battery is None, and type is Control

Local sensors:
Thermostat temp is 75.0, humidity is 40, battery is None, and type is Local

Outdoor sensors:
Outdoor temp is 48.0, humidity is None, battery is None, and type is Outdoor

Remote sensors:
Dining Room temp is 74.0, humidity is None, battery is 100, and type is Remote
Laundry Bath temp is 74.0, humidity is None, battery is 100, and type is Remote
Master Bedroom temp is 74.0, humidity is None, battery is 100, and type is Remote
Master Bath temp is 74.0, humidity is None, battery is 100, and type is Remote
Kitchen temp is 73.0, humidity is None, battery is 100, and type is Remote
Office temp is 74.0, humidity is None, battery is 100, and type is Remote
Guest Bath temp is 74.0, humidity is None, battery is 100, and type is Remote
Guest Bedroom temp is 74.0, humidity is None, battery is 100, and type is Remote

Return sensors:
HVAC Return temp is 73.0, humidity is None, battery is 100, and type is Return

Supply sensors:
HVAC Supply temp is 78.0, humidity is None, battery is 100, and type is Supply

DEBUG:urllib3.connectionpool:Starting new HTTP connection (1): <IP_ADDRESS>:80
DEBUG:urllib3.connectionpool:http://<IP_ADDRESS>:80 "GET /query/runtimes HTTP/1.1" 200 None
Runtimes: [{'ts':<PHONE_NUMBER>, 'heat1': 455, 'heat2': 0, 'cool1': 0, 'cool2': 0, 'aux1': 0, 'aux2': 0, 'fc': 0}, {'ts':<PHONE_NUMBER>, 'heat1': 369, 'heat2': 0, 'cool1': 0, 'cool2': 0, 'aux1': 20, 'aux2': 0, 'fc': 0}, {'ts':<PHONE_NUMBER>, 'heat1': 492, 'heat2': 0, 'cool1': 0, 'cool2': 0, 'aux1': 0, 'aux2': 0, 'fc': 0}, {'ts':<PHONE_NUMBER>, 'heat1': 352, 'heat2': 0, 'cool1': 0, 'cool2': 0, 'aux1': 19, 'aux2': 0, 'fc': 0}, {'ts':<PHONE_NUMBER>, 'heat1': 318, 'heat2': 0, 'cool1': 0, 'cool2': 0, 'aux1': 36, 'aux2': 0, 'fc': 0}, {'ts':<PHONE_NUMBER>, 'heat1': 353, 'heat2': 0, 'cool1': 0, 'cool2': 0, 'aux1': 24, 'aux2': 0, 'fc': 0}, {'ts':<PHONE_NUMBER>, 'heat1': 119, 'heat2': 0, 'cool1': 0, 'cool2': 0, 'aux1': 19, 'aux2': 0, 'fc': 0}]
DEBUG:urllib3.connectionpool:Starting new HTTP connection (1): <IP_ADDRESS>:80
DEBUG:urllib3.connectionpool:http://<IP_ADDRESS>:80 "POST /control HTTP/1.1" 200 None
DEBUG:urllib3.connectionpool:Starting new HTTP connection (1): <IP_ADDRESS>:80
DEBUG:urllib3.connectionpool:http://<IP_ADDRESS>:80 "GET /query/info HTTP/1.1" 200 None
Heat setpoint is 60.0
Cool setpoint is 90.0

DEBUG:urllib3.connectionpool:Starting new HTTP connection (1): <IP_ADDRESS>:80
DEBUG:urllib3.connectionpool:http://<IP_ADDRESS>:80 "POST /control HTTP/1.1" 200 None
DEBUG:urllib3.connectionpool:Starting new HTTP connection (1): <IP_ADDRESS>:80
DEBUG:urllib3.connectionpool:http://<IP_ADDRESS>:80 "GET /query/info HTTP/1.1" 200 None
Heat setpoint is 74.0
Cool setpoint is 82.0

Once you two have settled on something that makes both of you happy, let me know and I'll merge it. :)

I'm happy now. Last improvement, I promise.

I like the new additions, I'm happy now too.

I'll try to make some time to crank out a new release on Pypi in the next day or two.  I think I have to resurrect a container to do so.

Perfect! I plan to submit a PR to bump home assistant once that's done.

If you do, can you add support for the additional sensors?

That's the plan, but separately, so the fix for the current temperature gets through quicker. I also plan to modernize the integration to support config flow for flexibility with sensor entity names. It's unlikely that I'll get to it in the next few days, as I'll be dividing my time between shoveling quite a bit of snow and troubleshooting the denonavr integration.
While you wait, enjoy some REST sensor magic:
sensor:
  - platform: rest
    name: Thermostat Sensors
    json_attributes:
      - sensors
    resource: http://<IP_ADDRESS>/query/sensors
    value_template: "sensoring"
    scan_interval: 60
  - platform: template
    sensors:
      average_temperature:
        friendly_name: "Average Temperature"
        device_class: temperature
        unit_of_measurement: "°F"
        value_template: "{{ state_attr('sensor.thermostat_sensors', 'sensors')|selectattr('name','eq','Space Temp')|map(attribute='temp')|list|last|default('None') }}"
      thermostat_room_humidity:
        friendly_name: "Thermostat Room Humidity"
        device_class: humidity
        unit_of_measurement: "%"
        value_template: "{{ state_attr('sensor.thermostat_sensors', 'sensors')|selectattr('name','eq','Thermostat')|map(attribute='hum')|list|last|default('None') }}"
      wireless_temperature_battery_level:
        friendly_name: "Wireless Temperature Sensor Battery Level"
        device_class: battery
        unit_of_measurement: "%"
        value_template: "{{ state_attr('sensor.thermostat_sensors', 'sensors')|selectattr('name','eq','Sensor Name')|map(attribute='battery')|list|last|default('None') }}"

Version bumped to 0.13 and uploaded to pypi.


Changelog: request params changed

request param changed: pattern
request param changed: min request param value increased/decreased
request param changed: min items param value increased/decreased
request param changed: max request param value increased/decreased
request param changed: max items request param value increased/decreased
request param changed: max length request param value increased/decreased
request param changed: min length increased/decreased
request param changed: enum value added/removed

Codecov Report

Merging #343 (2ea5340) into main (dc39db7) will increase coverage by 0.17%.
The diff coverage is 79.86%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #343      +/-   ##
==========================================
+ Coverage   78.30%   78.47%   +0.17%
==========================================
  Files         182      184       +2
  Lines        9254     9372     +118
==========================================
+ Hits         7246     7355     +109
+ Misses       1722     1718       -4
- Partials      286      299      +13

Flag
Coverage Δ

unittests
78.47% <79.86%> (+0.17%)
:arrow_up:

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

Files Changed
Coverage Δ

...cker/check-request-parameters-max-items-updated.go
66.66% <66.66%> (ø)

...ker/check-request-parameters-min-length-updated.go
66.66% <66.66%> (ø)

...heck-request-parameter-pattern-added-or-changed.go
90.16% <77.77%> (-5.76%)
:arrow_down:

...cker/check-request-parameter-enum-value-updated.go
95.45% <100.00%> (ø)

...ker/check-request-parameters-max-length-updated.go
100.00% <100.00%> (ø)

checker/check-request-parameters-max-updated.go
95.00% <100.00%> (ø)

...cker/check-request-parameters-min-items-updated.go
94.87% <100.00%> (ø)

checker/check-request-parameters-min-updated.go
94.87% <100.00%> (ø)

checker/default_checks.go
90.90% <100.00%> (ø)

... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Fail to use clojurewerkz.cassaforte.query

I'm following the getting started guide
clojurewerkz.cassaforte.client/connect works fine, but when I try to use
(ns test-cassansra.core
  (:require [clojurewerkz.cassaforte.client :as cc]
            [clojurewerkz.cassaforte.cql    :as cql]
            [clojurewerkz.cassaforte.query  :refer :all]))

I get
clojure.lang.Compiler$CompilerException: java.lang.NoClassDefFoundError: Could not initialize class clojurewerkz.cassaforte.query__init, compiling:(clojurewerkz/cassaforte/cql.clj:1:1)

Looks like query lib is missing.
My project file:
(defproject test-cassansra "0.1.0-SNAPSHOT"
  :description "FIXME: write description"
  :url "http://example.com/FIXME"
  :license {:name "Eclipse Public License"
            :url "http://www.eclipse.org/legal/epl-v10.html"}
  :dependencies [[org.clojure/clojure "1.6.0"]
                 [clojurewerkz/cassaforte "3.0.0-alpha1"]
                 ])

@tzach please use mailing list for questions. You are trying to use an alpha version from master. The docs cover 2.x and pretty explicitly say so.

@ifesdjeen is this unfinished or an intentional API change?

@michaelklishin it is finished, and I was pretty sure query.clj is still there :/

might be an artifact upload problem, i'm investigating the problem right now

I'm currently affected by this issue. My project.clj:
(defproject apollo "0.1.0-SNAPSHOT"
  :description "FIXME: write description"
  :url "http://example.com/FIXME"
  :license {:name "Eclipse Public License"
            :url "http://www.eclipse.org/legal/epl-v10.html"}
  :dependencies [
                 [org.clojure/clojure "1.8.0"]
                 [seesaw "1.4.2" :exclusions [org.clojure/clojure]]
                 [clojurewerkz/cassaforte "3.0.0-alpha1"]
                 ]
  :main ^:skip-aot apollo.core
  :target-path "target/%s"
  :profiles {:uberjar {:aot :all}})

@codygman i've just tried that once again, it somehow works for me.. could you please try running
rm -fr ~/.m2/repository/clojurewerkz/cassaforte/3.0.0-alpha1

it might be that the artefact got cached or something similar happened...
I've tried with / without AOT, with 1.8 and 1.6 Clojure, works in all cases.


Site Request: ITProTV

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2016.11.22. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2016.11.22

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[x ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
youtube-dl -v https://itpro.tv/course-library/vmware-certified-professional-6-vcp6dcv/upgrading-vsphere-6-2/
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['-v', 'https://itpro.tv/course-library/vmware-certified-professional-6-vcp6dcv/upgrading-vsphere-6-2/']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2016.11.22
[debug] Python version 3.4.4 - Windows-10-10.0.14393
[debug] exe versions: none
[debug] Proxy map: {}
[generic] upgrading-vsphere-6-2: Requesting header
WARNING: Falling back on generic information extractor.
[generic] upgrading-vsphere-6-2: Downloading webpage
[generic] upgrading-vsphere-6-2: Extracting information
ERROR: Unsupported URL: https://itpro.tv/course-library/vmware-certified-professional-6-vcp6dcv/upgrading-vsphere-6-2/
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp0yt5fyka\build\youtube_dl\YoutubeDL.py", line 694, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp0yt5fyka\build\youtube_dl\extractor\common.py", line 357, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp0yt5fyka\build\youtube_dl\extractor\generic.py", line 2443, in _real_extract
youtube_dl.utils.UnsupportedError: Unsupported URL: https://itpro.tv/course-library/vmware-certified-professional-6-vcp6dcv/upgrading-vsphere-6-2/
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video:  https://player.vimeo.com/video/100816669?api=1&player_id=vimeoplayer
Single video: https://player.vimeo.com/video/100816670?api=1&player_id=vimeoplayer

Description of your issue, suggested solution and other information
site request for itpro.tv,
i would like to request support for itpro.tv since i would like to have an cli based support for downloading videos from the site when i dont have good internet speeds or i would like to archive the videos to an remote server.  currently itpro.tv needs an paid account to download videos, you can signup for an trial account using the code TWIT30 when signing up.  they currently use two sources for video urls jw player and vimeo. the jw player links expire after 24 hours while the Vimeo links do not.
you can download an Vimeo video using the following youtube-dl command,  youtube-dl -v "https://player.vimeo.com/video/170683353?api=1&player_id=vimeoplayer" --referer "https://itpro.tv/"
or  youtube-dl -v "https://player.vimeo.com/video/170683353?api=1&player_id=vimeoplayer" --referer "https://itpro.tv/" --external-downloader aria2c --external-downloader-args='--max-connection-per-server=16 --split=20'
ITProTV provides the connection for self-study users and the real world experience that career IT people need to be successful. Because our shows are available Live everyday our content portfolio stays fresh and current. We are always covering the latest information of each topic geared toward I.T. candidates who desire to validate their skills/knowledge through vendor certification exams.
Thanks
Dedsec1

You may also try this method http://www.allavsoft.com/how-to/download-itprotv-courses.html

You can use Allavsoft to download all online courses from the entire ITProTV course library into MP4, AVI, MOV, WMV, MP3, WAV, AAC, etc  and find it at
http://www.allavsoft.com/how-to/download-itprotv-courses.html


feat: expose context to handlers

Motivation
Exposing the context could come in handy to allow users to have a bit more control that they would if they were using msw directly and it will expose some helper functions like delay

Requested review from a random @lifeomic/security team member, @bishopb, due to term: expose.
This review is not blocking and is for broader awareness.  Consider if this change requires deeper security review and ask when it is necessary.

Automation | #pr-flagged-sec-review


new address to block

needs blocking = om45odi5th.7cnli.adobestats.io
/jim

ALSO this:  om45odi5th6.7cnli.adobestats.io
Note this second entry has one additional digit in the subdomain name.  Unusual for a server change to happen only hours from the previous one..
Thanks!

ALSO THIS ONE:  om45odi5th6.7cnli.adobestats.io
Note there is one additional digit in the subdomain.  Unusual for them to update the server after only 13 hours...

Added both, thank you for the report!


Add the function bicoloring

Checklist

[x] Appropriate tests were added
[x] Any code changes were done in a way that does not break public API
[x] All documentation related to code changes were updated
[x] The new code follows the
contributor guidelines, in particular the SciML Style Guide and
COLPRAC.
[x] Any new documentation only uses public API

Additional context
#69

In principle this would be good but I suggest we put it on hold until we fully figure out bicoloring and neutral colors in SparseMatrixColorings.jl. Converting this PR to draft in the meantime so that it doesn't get merged accidentally.


One .xib for all screen sizes

Handle autolayout better and use one .xib for all screen sizes for better initialization handling.

Hi @psy2k , I just found an alternative for autolayout.
I just add one line:
        self.popViewController.title = "This is a popup view"
        self.popViewController.showInView(self.view, withImage: UIImage(named: "typpzDemo"), withMessage: "You just triggered a great popup window", animated: true)

        // Autolayout
        self.popViewController.view.frame = self.view.frame

Thanks

@JulienDrecq This does not have always the desired results, especially if it is called before the viewDidLayoutSubviews method of the parent controller.

would love to see this because you are missing a xib file that works for ipad pro


Fix CMVN variance normalization

Variance normalization should be performed dividing by the square root of the variance, not the variance

CLA Signed!

DId you train models with this change? Does it help?

Hi Lukasz
I have tried with the fisher_callhome_spanish task. The epsilon part was necessary to avoid NaNs. An the normalization part also helped the convergence on TPUs. We are still having slow convergence problems on GPUs, even with many GPUs, and I ran out of TPU credit.
I would be pleased to evaluate the impact on librispeech both on TPUs and GPUs, and to provide new models. But I would need first some Google Cloud credit for the TPU part. I have received no answer yet from the TensorFlow Research Cloud program.
  De: Lukasz Kaiser<EMAIL_ADDRESS>
Para: tensorflow/tensor2tensor<EMAIL_ADDRESS>CC: Jose Fonollosa<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Enviado: Viernes 22 de junio de 2018 0:38
Asunto: Re: [tensorflow/tensor2tensor] Fix CMVN variance normalization (#877)
DId you train models with this change? Does it help?—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Move to filters for high-level intent

Coding in Jinja/YAML is error-prone and difficult to troubleshoot. Proposal to move to custom filters (l2vpn, l3vpn) to generate low-level device configuration from high-level intent.

May go for dedicated modules inside roles to 'pythonize' data manipulation


How to disable automatic collection of IDFA in Firebase Analytics SDK

I'm using Firebase Analytics SDK (Xamarin.Firebase.iOS.Analytics 6.9.0) in my Xamarin.Forms App.
We want to disable IDFA collection due to recent iOS 14 user privacy requirements. Can you kindly let me know how this can be achieved ?
I could see the AdSupport class in the Xamarin.iOS.dll, but I couldn't find any relevant information nor have a clue as to how I can ensure that this AdSupport class can be excluded from the app so that IDFA won't be utilized by the Firebase Analytics SDK.
I found that the Google Analytics SDK (Xamarin.Google.iOS.Analytics <IP_ADDRESS>) has a property to enable IDFA collection and as per my understanding the IDFA collection is disabled by default in this SDK.
Please let me know how I can disable IDFA in the Firebase Analytics SDK? Thanks

The Age, Gender, and Interests properties are automatically collected only if your app links to the Ad Support framework. Linking to this framework also automatically collects the Advertising Identifier (IDFA).

Quoting the docs, so does the AdSupport framework get linked automatically to the app from the Xamarin.iOS.dll?
If so, to disable the IDFA, is there any other way other than to remove the SDK from the app?

The Age, Gender, and Interests properties are automatically collected only if your app links to the Ad Support framework. Linking to this framework also automatically collects the Advertising Identifier (IDFA).

Quoting the docs, so does the AdSupport framework get linked automatically to the app from the Xamarin.iOS.dll?
If so, to disable the IDFA, is there any other way other than to remove the SDK from the app?

Hello @PaulsonM ,
I'm facing the same issue. Did you find a way to solve the issue?

I'd like to know this. It's so sensitive that we should have some standard solution.

Hi did you get any answer to this?


Doc search is not working

The search in the documentation does not find anything:

I tried different search terms in different browsers.

Same for me.
Algolia seems to be setup on the docusaurus config 🤔 May the application was deleted on Algodia ?

I'm investigating the issue, in the dashboard I'm able to run the queries successfully 🤔

@wcandillon the search still has some issues
e.g. searching for "fitbox" does not find anything:

And the "fullscreen" search at https://shopify.github.io/react-native-skia/search does not find anything:


Place event first which is directly related to selected event

Currently we are highlighting the source and placing the event to the first for selected root event. There's subsequent requirement that placing the event to the first for selected root event other than source.
To add my idea, we can place events to the first on components which is directly related to select event, so that it can reflect complex topology like below:
S1 -> Projection -> Aggregation -> Join -> Sink
S                               ->

If we select the event which is emitted from S1, events in S1, Projection, Aggregation, Join will be reordered. Sink is not reordered given that the event is co-related (both from S1 and S2).

#1018


Non-Interactive Installation for zsh

zsh compinit: insecure directories, run compaudit for list.

Ignore insecure directories and continue [y] or abort compinit [n]?

And this only happens on travis.


Research hosting options for backend

AWS?

Gunna try to get GCP code runner working if they ever give me the credits for it. Gunna need to dockerize the application though which I will do shortly

Gunna close this and open new issues for the dockerization/GCP hosting on cloud runner see:
https://github.com/sekR4/FastAPI-on-Google-Cloud-Run


Improve design of custom pointers

In alignment with Special pointers, we suggest to align values on the right.
On the picture we see, there may be variable length keys.
Mobile version is also missing.

Fixed in #682


Reset formats to only to ttbin

Hey Ryan
First of all, let me thank you for this tremendously useful piece of software allowing me to throw Windows of my hard disk for good. During my use of the tool I encountered a minor issue which might be due to a bug, might have been just overlooked, or might even be due to a shortcoming from my side by not handling it correctly.
Using ttwatch --set-formats=LIST, one can set additional formats in which the activity files will be stored upon downloading from the watch. However, once specified, I haven't found a way of getting rid of these additional formats such that only the binary (ttbin) will be saved, as was the case initially.
Passing an empty list, that is ttwatch --set-formats=, yields No valid file formats found and thus doesn't work.

I have removed the check to see that at least one format was specified, so ttwatch --set-formats= will now just clear the list.


dev_mode: Add debug trace facility controlled through "dev_trace" setting

As a use case, add tracing to the reload module to get the output advertised in #408.

This is really interesting.  Thanks for your work on the debug mode stuff - it's not typically where we get contributions!

 I started looking at this but I don't understand what is happening in 3bb59cb33da4f6e10fd4b0eb3df4c6361e936ff3. Can one of you explain it a bit and how to use it?

@stoivo, trace is essentially a wrapper around print, mostly adding formatted meta-data and consistent output.  In the context of this larger, it allows for pretty debug output when reloading like you see in https://github.com/divmain/GitSavvy/pull/408.

How would you use it?
When to use dump_var vs trace? That is the difference I don't see.

@divmain you're welcome! I was experimenting on inline diff view -> switched to refactoring ui.py -> encountered reloading issues -> switched to reload -> and here we are, debugging it :wink:
@stoivo trace is essentially an ad-hoc replacement for the standard logging:
log = trace.for_tag("reload")  # read as: logger = logging.getLogger(__name__)

And then scatter around log() calls. As like using the standard logging, one can filter the output using trace tags.

On the other hand, dump and dump_var are more like handy helpers during hacking, useful for printf-style debugging. Usage of these is not supposed to be checked into the repository.
def some_nasty_stuff(foo):
    ... # the real logic

    # XXX remove!
    # and if something performs wrong:
    dump(foo)         # arg0 = ...
    dump(foo=foo)     # foo = ...

    # or, to show all arguments and local variables:
    dump(**locals())  # foo = ...
                      # bar = ...

dump_var is essentially an internal function outputting a single value, a call to dump(foo=foo) is then handled as dump_var("foo", foo). The former is just a bit less to type. :)

Thanks @abusalimov


[Feature Request] AutoComplete no results default.

Existing Component
Yes
Component Name
<el-autocomplete></el-autocomplete>
Description
It would be useful if we could add a default option to the autocomplete for when no results are matched.
For example, if not found I'd like to add a button which says "Create new".

Sorry, we have no plan to support this.


Removed GroupM as alias of PubMatic

Removed GroupM as alias of PubMatic
🏷 Type of documentation

[x] update bid adapter

@pm-azhar-mulla - are you sure you want to delete this doc? This alias is still available before 6.24.
Instead of deleting, we recommend adding a note: "This bidder is not available in PBJS 6.24 and later."

@bretg  Updated the file as per recommendation.


Restore mix.lock erroneously downgraded in 6e4ade6

Pull Request Test Coverage Report for Build b317f1eeaecd4ece22a9ddc4f94bf2c33545344e-PR-759

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 97.001%

Totals

Change from base Build 06cf8215284abb93730c983f02f14dd331d03bbc:
0.0%

Covered Lines:
1520

Relevant Lines:
1567

💛  - Coveralls


Tests are broken

Our test framework seems to be broken for several cases.

Semi-fixed by #647, but this fix removed some visualization tests on some platforms.


Update channel.go

remove Google+ check since it is being dismissed by google

Hi, @ilmanzo 👋 Thanks for contributing! LGTM and ready to merge! 😄


Clean up internal/context

internal/context.go has some tech debt that should be cleaned up including

Commented out code that should either be deleted or enabled https://github.com/temporalio/sdk-go/blob/269b55b8d184849209c6973fe913a427fafc07e3/internal/context.go#L348
Incorrect use of go in a workflow context (this code appears to be dead, but should be removed regardless) https://github.com/temporalio/sdk-go/blob/269b55b8d184849209c6973fe913a427fafc07e3/internal/context.go#L239

@Quinn-With-Two-Ns @Quinn-With-Two-Ns I like to work on this. Before, making a PR I have a question regarding the cleanup of incorrect use of the go routine, what should we do there after the cleanup? I am thinking of adding a return and end the function.

@tharun208 that be great! When I last reviewed the code I conclude that the else statement was never satisfied since parentCancelCtx should always return true. If we can verify that then we can probably just add a panic because that is an undefined state.


How to create annotations for vqa2/vizwiz dataset for grid features?

I need to create annotations file of vqa2/viwiz dataset for VQA task that you've mentioned in the paper. Please let me know how do we create the complete annotations for grid features.

Sir,Do you know how to create the complete annotations for grid features.If I have a customized dataset(just have images and one image to 5 captions)


[wip] bump localization

Include Spanish updates

didn't see this earlier! I'll close as duplicate


feat(#290): Add Java 8 Support

Add Java 8 support.
Closes: #290.
History:

feat(#290): set source and target java versions
feat(#290): make codebase java-8 compatible
feat(#290): fix all qulice suggestions

PR-Codex overview
The PR adds Java 8 support, updates dependencies, and improves exception handling in various classes.
Detailed summary

Updated Java versions to 8, 11, and 20 in multiple files
Changed List.of to Arrays.asList for compatibility
Improved exception handling in various classes
Updated dependencies and versions for consistency

The following files were skipped due to too many changes: src/main/java/org/eolang/opeo/compilation/XmirParser.java, pom.xml

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 5min)


reaction questions

what are the conditions
do we want to extract the order of tags in a separate field?

conditions are HTML


Support for SpringBoot 3.0.X

Hi,
Is there a plan to make a new release of this library, which supports integration with Spring Boot 3.0.X?

There is a PR for this https://github.com/ulisesbocchio/jasypt-spring-boot/pull/341. Just need to wait until it is merged and released.
And as I did not want to wait I imported the AutoConfiguration file manually instead:
`
import com.ulisesbocchio.jasyptspringbootstarter.JasyptSpringBootAutoConfiguration;
@SpringBootApplication
@ImportAutoConfiguration(classes = JasyptSpringBootAutoConfiguration.class) // Remove when Jasypt is released for spring boot 3 (or at least 2.7)
public class Application {
public static void main(String[] args) {
    SpringApplication.run(Application.class, args);
}

}
`

@sickan90 : Thank you.

thank you guys! all merged and released on 3.0.5


Bug fix missing result when parallel outfile enabled

What problem does this PR solve?
fix missing result when parallel outfile enabled
Problem Summary:
avoid lack of logic to use dep to control eos sending
Check List (For Committer)

Test

[X] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No colde files have been changed.
[ ] Other reason

Behavior changed:

[X] No.
[ ] Yes.

Does this need documentation?

[X] No.
[ ] Yes.

Release note

None

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall

run buildall


feat: support @nuxt/components auto-import

Awaits https://github.com/nuxt/components/pull/234

Any progress? I wanted to use unplugin icons in my project (nuxt 2 bridge), but after errors in resolving icon components, I realized that it was not worth it. The documentation doesn't say anything about the lack of icon component auto-import in nuxt, which is confusing.

Hey @antfu, just out of curiosity, why was this PR closed? Is there any other fix or workaround available by now? :bulb:


pkg/report: Do not report a bunch of "fatal errors"

These are internal to syzkaller and confusing to the reader.

These are all coming from Go runtime and with 99% probability they are just different manifestations of kernel-caused memory corruptions. What about bucketing them all to some go runtime error-titled report?
OOM messages, as it's already the case, should cause Suppressed=true. But do we really care about the specific titles of all others?
Cc @dvyukov

I agree "suppressed reports" is a big hummer.  For normal kernels these usually mean kernel corrupting user-space data and with a reproducer can be well actionable. IIRC we had such examples with mouting of some filesystems.
Also consider what if all testing of your kernel of interest just reports gazillion of these and no real kernel testing happens anymore, do you still want to continue ignoring these? ;)
Also note that for gVisor these mean "normal kernel bugs" since it's written in Go and that's what we are testing. See:
https://syzkaller.appspot.com/gvisor/fixed
https://syzkaller.appspot.com/gvisor/invalid
So for gVisor these need to be somehow special cased.
But grouping these for normal kernels looks reasonable since they either indicate the same bug (memory corruption), or just usually less interesting (practically, at least if no happen too often).

I agree "suppressed reports" is a big hummer. For normal kernels these usually mean kernel corrupting user-space data and with a reproducer can be well actionable. IIRC we had such examples with mouting of some filesystems. Also consider what if all testing of your kernel of interest just reports gazillion of these and no real kernel testing happens anymore, do you still want to continue ignoring these? ;)

Fair enough :)

Also note that for gVisor these mean "normal kernel bugs" since it's written in Go and that's what we are testing. See: https://syzkaller.appspot.com/gvisor/fixed https://syzkaller.appspot.com/gvisor/invalid So for gVisor these need to be somehow special cased.

I see.

But grouping these for normal kernels looks reasonable since they either indicate the same bug (memory corruption), or just usually less interesting (practically, at least if no happen too often).

I'm not quite sure what the best mechanism for grouping is but I tried to update the PR to something that seems to be in the direction of what you'd like. :) I hope it's the right stab in the dark!

It looks like the tests fail because I now rename fatal error: out of memory to go runtime bug and these no longer match the Suppress rules.
I will only investigate further if the current change is what you had in mind anyway... ;)

Uh? i'm not sure what happened with the "FlorentRevest requested review from a bazillion persons".

These people were designated as code owners, so they get auto-added each time you modify files with the matching paths. I'm not sure how useful it all in practice though..

It looks like the tests fail because I now rename fatal error: out of memory to go runtime bug and these no longer match the Suppress rules.

If the reports are still to be suppressed, that's fine, just update the tests (run go test ./pkg/report -update). If they are no longer suppressed, then that's indeed a problem.


Add support to DASH

Thanks for this very useful project! Is possible to add DASH support?
Also, can you provide an wallet for me to pay you a beer? 😄

@uliporeggia, DASH support has been added, please update your client with the new revision.
If you wish to buy me a beer you can make a small donation here 15c91dGPndFPburMdtQZygQSoWxx1yUqyv.
Thanks

Thanks!
https://blockchain.info/tx/55cc241d034961cdbc7e090a616482625ebd49b1ca561b38961f450e668939b7

Thanks @uliporeggia, appreciated


Pods should opt-in to webhook mutation directly

Is your feature request related to a problem? Please describe.
Today, the default configuration of the webhook is to intercept all pod mutations:
https://github.com/Azure/azure-workload-identity/blob/6c77e4071a7c6cbd08d16e0c128a25972a7b3ce8/charts/workload-identity-webhook/templates/azure-wi-webhook-mutating-webhook-configuration-mutatingwebhookconfiguration.yaml#L21-L34
and the default mutatingWebhookObjectSelector is empty:
https://github.com/Azure/azure-workload-identity/blob/6c77e4071a7c6cbd08d16e0c128a25972a7b3ce8/charts/workload-identity-webhook/README.md?plain=1#L55
The webhook uses the service account associated with the pod to filter the mutation logic:
https://github.com/Azure/azure-workload-identity/blob/6c77e4071a7c6cbd08d16e0c128a25972a7b3ce8/pkg/webhook/webhook.go#L120-L124
While a user could specify a custom mutatingWebhookObjectSelector and mutatingWebhookFailurePolicy, I believe we can have better defaults around this flow.  The webhook should only be invoked if we actually expect to mutate the pod.  This will reduce the latency impact of the webhook and allow us to fail closed (not as a security boundary, but instead to prevent pods from starting in an unexpected state).
Describe the solution you'd like
The defaults should be changed so that:
objectSelector -> azure.workload.identity/use
https://github.com/Azure/azure-workload-identity/blob/6c77e4071a7c6cbd08d16e0c128a25972a7b3ce8/pkg/webhook/consts.go#L6
failurePolicy -> Fail
The webhook would be updated to mutate the pod if either pod or the service account has the label (eventually these checks will be removed altogether and the webhook would always mutate the pod).
To limit the impact of this breaking change, we could do a few things:

Attempt to issue warnings and audit annotations to help users know that this change is coming
Template the mutatingWebhookObjectSelector YAML so that it can be set to the empty string as a way to restore the old behavior for a period of time
Have the change behind a feature gate for some period of time (and the gate's default state would change from off to on over time)

Describe alternatives you've considered
Not sure what other options there are.
Additional context
xref #540

Closing this issue with https://github.com/Azure/azure-workload-identity/releases/tag/v1.0.0-alpha.0 release.


HDDS-2160. Add acceptance test for ozonesecure-mr compose. Contribute…

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
153
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

0
shelldocs
0
Shelldocs was not available.

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

-1
mvninstall
36
hadoop-ozone in trunk failed.

-1
compile
25
hadoop-ozone in trunk failed.

+1
mvnsite
0
trunk passed

-1
shadedclient
38
branch has errors when building and testing our client artifacts.

-1
javadoc
59
hadoop-ozone in trunk failed.

_ Patch Compile Tests _

-1
mvninstall
35
hadoop-ozone in the patch failed.

-1
compile
27
hadoop-ozone in the patch failed.

-1
javac
27
hadoop-ozone in the patch failed.

+1
mvnsite
0
the patch passed

+1
shellcheck
1
There were no new shellcheck issues.

+1
whitespace
0
The patch has no whitespace issues.

-1
shadedclient
33
patch has errors when building and testing our client artifacts.

-1
javadoc
60
hadoop-ozone in the patch failed.

_ Other Tests _

+1
unit
309
hadoop-hdds in the patch passed.

-1
unit
31
hadoop-ozone in the patch failed.

+1
asflicense
37
The patch does not generate ASF License warnings.

1531

Subsystem
Report/Notes

Docker
Client=18.09.7 Server=18.09.7 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1490/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1490

Optional Tests
dupname asflicense mvnsite unit shellcheck shelldocs compile javac javadoc mvninstall shadedclient

uname
Linux 3b6e16cad131 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / d7d6ec8

Default Java
1.8.0_222

mvninstall
https://builds.apache.org/job/hadoop-multibranch/job/PR-1490/1/artifact/out/branch-mvninstall-hadoop-ozone.txt

compile
https://builds.apache.org/job/hadoop-multibranch/job/PR-1490/1/artifact/out/branch-compile-hadoop-ozone.txt

javadoc
https://builds.apache.org/job/hadoop-multibranch/job/PR-1490/1/artifact/out/branch-javadoc-hadoop-ozone.txt

mvninstall
https://builds.apache.org/job/hadoop-multibranch/job/PR-1490/1/artifact/out/patch-mvninstall-hadoop-ozone.txt

compile
https://builds.apache.org/job/hadoop-multibranch/job/PR-1490/1/artifact/out/patch-compile-hadoop-ozone.txt

javac
https://builds.apache.org/job/hadoop-multibranch/job/PR-1490/1/artifact/out/patch-compile-hadoop-ozone.txt

javadoc
https://builds.apache.org/job/hadoop-multibranch/job/PR-1490/1/artifact/out/patch-javadoc-hadoop-ozone.txt

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-1490/1/artifact/out/patch-unit-hadoop-ozone.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1490/1/testReport/

Max. process+thread count
423 (vs. ulimit of 5500)

modules
C: hadoop-ozone/dist U: hadoop-ozone/dist

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1490/1/console

versions
git=2.7.4 maven=3.3.9 shellcheck=0.4.6

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
104
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

0
shelldocs
0
Shelldocs was not available.

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

-1
mvninstall
35
hadoop-ozone in trunk failed.

-1
compile
25
hadoop-ozone in trunk failed.

+1
mvnsite
0
trunk passed

-1
shadedclient
34
branch has errors when building and testing our client artifacts.

-1
javadoc
56
hadoop-ozone in trunk failed.

_ Patch Compile Tests _

-1
mvninstall
33
hadoop-ozone in the patch failed.

-1
compile
23
hadoop-ozone in the patch failed.

-1
javac
23
hadoop-ozone in the patch failed.

+1
mvnsite
0
the patch passed

+1
shellcheck
0
There were no new shellcheck issues.

+1
whitespace
0
The patch has no whitespace issues.

-1
shadedclient
30
patch has errors when building and testing our client artifacts.

-1
javadoc
48
hadoop-ozone in the patch failed.

_ Other Tests _

-1
unit
150
hadoop-hdds in the patch failed.

-1
unit
25
hadoop-ozone in the patch failed.

+1
asflicense
26
The patch does not generate ASF License warnings.

1238

Reason
Tests

Failed junit tests
hadoop.ozone.container.ozoneimpl.TestOzoneContainer

Subsystem
Report/Notes

Docker
Client=19.03.0 Server=19.03.0 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1490/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1490

Optional Tests
dupname asflicense mvnsite unit shellcheck shelldocs compile javac javadoc mvninstall shadedclient

uname
Linux a01eca9bda51 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / d7d6ec8

Default Java
1.8.0_222

mvninstall
https://builds.apache.org/job/hadoop-multibranch/job/PR-1490/1/artifact/out/branch-mvninstall-hadoop-ozone.txt

compile
https://builds.apache.org/job/hadoop-multibranch/job/PR-1490/1/artifact/out/branch-compile-hadoop-ozone.txt

javadoc
https://builds.apache.org/job/hadoop-multibranch/job/PR-1490/1/artifact/out/branch-javadoc-hadoop-ozone.txt

mvninstall
https://builds.apache.org/job/hadoop-multibranch/job/PR-1490/1/artifact/out/patch-mvninstall-hadoop-ozone.txt

compile
https://builds.apache.org/job/hadoop-multibranch/job/PR-1490/1/artifact/out/patch-compile-hadoop-ozone.txt

javac
https://builds.apache.org/job/hadoop-multibranch/job/PR-1490/1/artifact/out/patch-compile-hadoop-ozone.txt

javadoc
https://builds.apache.org/job/hadoop-multibranch/job/PR-1490/1/artifact/out/patch-javadoc-hadoop-ozone.txt

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-1490/1/artifact/out/patch-unit-hadoop-hdds.txt

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-1490/1/artifact/out/patch-unit-hadoop-ozone.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1490/1/testReport/

Max. process+thread count
292 (vs. ulimit of 5500)

modules
C: hadoop-ozone/dist U: hadoop-ozone/dist

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1490/1/console

versions
git=2.7.4 maven=3.3.9 shellcheck=0.4.6

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.


Fixes Missing/invalid DB name error

This PR fixes #38 in which the error is being thrown: Missing/invalid DB name
Because of where this error is thrown, I don't think there is a way to catch this in the tests, so I did not modify the tests.

merged, see #48


Replace the system keyboard outside your own app

Is it possible to replace the system keyboard outside your own app?  For use in whats app or other messaging apps?

I also want to know wether this function has or not.

Yeah, you can do this:
https://customkeyboarddetails.blogspot.com/2019/02/using-react-native-in-android-custom.html


Field place holders in Expression builder

The values for User, Global, Fields are displaying in the expression builder (report | Value | Edit Expression) as place holders {!UserID}, {@PageNumber}, {numofdays} etc. they need to be changed so they insert the correct value such as =Fields!accepteduntil.Value.

I have just committed a fix for this.  Try it out and let me know if it fixes the problem for you.


Binary name inconsistency

In chapter 3 we defined the app name as app

However, the binary in chapter 5 is referred to as zero2prod. The guide itself therefore does not build.

Keep up the great work!

Fixed and released, thank you!


add search - [closed]

In GitLab by @luotian-github on Sep 1, 2017, 06:16
Merges search -> dev
add search

In GitLab by @wknet123 on Nov 13, 2019, 09:55
closed


filter out all items in the step "Filter down to catholic and public school students and create a  dummy for catholic"

Hi!
thanks for your share for the data and the propensity score tutorial.
when i run the R scrip, after the step "Filter down to catholic and public school students and create a  dummy for catholic", no items exist at all in "dta", i am not familar to the data stucture and do not know what happened, would you help?
thanks!

finally i solved it,  change the  code "filter(l5cathol == 'YES' | l5public == 'YES')"  to "filter(l5cathol == '1' | l5public == '1')"

another quetion: the "r5race" is " "1"  "3"  "8"  "7"  "2"  "5"  "6"  "4"  "-9"",not "white" ,"black"....
so where can i find what the number of "r5race" represent?

Thanks for reporting this issue! The best fix here would be to use something other than the SPSS-compatible data from ECLS (I'm not a huge fan of spss.get(), which very well might be the cause of this issue). For example, there's a .txt file in the Stata-compatible download. Feel free to try that .txt file instead, and if you can't get that to work I'll try to do it myself in the next couple of days :)

I tried the stata data, but did not find a proper way to imported in R. Then I tried the sas data, using the function "foreign::read.xport"  to import data. However, the issue I described above still exist.

@WhitePredator - the issue has been resolved by using read_por() from haven to read the data.


Fix macOS compiling problems

conda-build v20.1+ doesn't have any default value defined for CONDA_BUILD_SYSROOT anymore and the conda_build_config.yaml from .travis is ignored by conda-build.

If @PiotrZierhoffer is happy and CI is happy, then go ahead and merge.

This makes things a bit Travis-specific, but I want to have things stable before we clean it up


Make task action default to the value of the effect

Actions until now would default to throwing a NotImplemented exception, however it may be generally useful to have tasks without side effects combined with tasks that perform side effects for more clean code. For this reason, if undefined, the action property will take the value of the effect property from now on.
Change-type: minor

I self-certify!


Significant delay in latest transfer card showing up

Describe the bug
The latest transfer card with the current processing one doesn't show up for a long time (around 1-2 min depending on the tx confirmation time). I suppose this is the time it takes for the transaction to get confirmed + the subgraph seeing the event.
To Reproduce
Steps to reproduce the behavior:

Go to https://amarok-testnet.coinhippo.io/
Swap
See error

Expected behavior
The card with the current pending transaction should show up immediately upon clicking confirm on metamask.

Also possibly remove the "transferring TEST..." entirely and just leave the card with the processing transaction on the latest transfers list

Latency is same delay from polling the backend and dupe of this shttps://github.com/connext/nxtp/issues/2010
Good callout. Need to update this interaction


Redesign the reader submodule

Codecov Report

Merging #18 (2d223b6) into master (41bbf4b) will decrease coverage by 0.50%.
The diff coverage is 96.77%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master      #18      +/-   ##
==========================================
- Coverage   85.16%   84.66%   -0.50%
==========================================
  Files           6        6
  Lines         445      424      -21
==========================================
- Hits          379      359      -20
+ Misses         66       65       -1

Flag
Coverage Δ

unittests
84.66% <96.77%> (-0.50%)
:arrow_down:

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

Impacted Files
Coverage Δ

lib/connection.lua
73.04% <ø> (ø)

postgres.lua
100.00% <ø> (ø)

lib/reader/single.lua
95.65% <91.66%> (+2.96%)
:arrow_up:

lib/reader.lua
100.00% <100.00%> (ø)

lib/result.lua
96.15% <100.00%> (-1.85%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Queue mode sender concurrency fix

This PR is based on the source of #158 (contains the first three commits) and aims to fix the concurrency problem in
(https://github.com/eclipse/leshan/blob/master/leshan-server-core/src/main/java/org/eclipse/leshan/server/queue/impl/QueuedRequestSender.java#L124
if we assume two threads t1 (user thread that calls LwM2mRequestSender.send()) and t2 (another user thread that calls cancelPendingMessage()):

t1 passes the check mentioned above, but request was not added to messageStore yet
t2 removes all request from queue (could be because client was unregistered)
t1 adds the request to message store (we have a request in the store without a corresponding registered client, i.e. this request may stay there forever)

PS: I will update this PR if #158 is accepted.
Signed-off-by: Bala Azhagappan<EMAIL_ADDRESS>

Also updated this PR after #158 was closed. Rebase on to master did not work due to conflicts between #158 and #157. I had to merge it with the changes in this PR.

Sry we try to keep a clean history, so no merge allowed ;)
Maybe you could try to restart from master and cherry pick the last commit. (This should limits conflict)

Hi Simon,
It worked. Thanks for the tip. I still have some conflicts with master which I think is due to #167 which got merged today. I will rebase on to master again tomorrow and probably have to adapt the implementations too as per new API.

Integrated in master (commit  	a8df823)


feat: add Tabnine for LazyVim

What
Add TabNine plugin for hrsh7th/nvim-cmp
Why
LazyVim has Codeium by this PR https://github.com/LazyVim/LazyVim/pull/1619 so it would be good if we have Tabnine :)

I believe it would be good to leave the options in the opts instead of config to give the user the opportunity to change them, instead of changing the config function.

Refactored a bit and merged. Thanks!


fix: 書之 prints "true", "false" instead of "陽", "陰"

Refer to the third and the fourth result in 歐幾里得法.
I'm not sure whether my changes work, as I haven't got npm installed on my computer. Sorry for the inconvenience.

Nice, thanks!


Control Icons

I'm sure most people use keyboard shortcuts for such things, but I tend to use the mouse to control menus. Would it make sense to add / is there a target group for symbols like in other IDEs (for example Visual Studio or CLion with the Arrow for run, the Square for stop, etc.)
I created a text-only version here, but the text renderer doesn't really like consistency.
~~https://github.com/d3rrial/jucipp/commit/03f04bbdb595f09822f8ea4d3bdd7d823fccdf05~~

Edit: I deleted my fork, here's a patch file that does the same thing:
edit2: nvm github doesn't like me doing patch files. I'll upload it properly in a second.
Here is the patchfile that does what I wanted in a Gist: https://gist.github.com/d3rrial/a3f14a6b806519f546196bbfff9a0707

At least in my opinion, one should use the mouse as little as possible, and it might be kinder to users to actually make them use shortcuts instead of clicking on such symbols. Also, when one has learned and is using the shortcuts, such symbols do nothing but take up screen space. Finally, one can actually compile and run more than one program from juCi++, so which one should the symbols represent? Still, I like the minimal look on the symbols, but not sure how we could place them there, especially when some systems have detached menus.

The reason why I don't like keyboard shortcuts is because I have a terrible memory. Granted, I'm the exception here and at the very least I'll keep those symbols in my own build. I more or less created the issue to see what your stance is on this and if there is interest in a proper implementation.
As for what the symbols do, I haven't put a hover-text or anything on them to clearly mark what they do, but they do exactly what their corresponding menu item would do. The commit is a mere proof of concept

I admit it looks cool:) The main issue I guess, given it looks approximately the same on all platforms (need testing on especially Ubuntu and OS X), is that we can run multiple programs at the same time in juCi++, and if one adds colors to indicate that an application is stoppable or already running, it would be hard to do so. It might be that there are a good solution for this, but my initial thought is that there might not be one. I'll give it some more thought!

Don't worry about it too much. It's just a suggestion and juCi++ is also very solid without it. If it's only beneficial for edge cases like me, then there's no point in actually implementing it after all :)
Unfortunately I don't really know enough about the menus you used, gtk I assume?, to make something better. I'll probably read myself into this topic, but I don't expect to find a properly working solution for a few weeks if not months (unfortunately busy doing other things).

I tried this on Manjaro and it worked great, but I had to double click on the symbols to make them work. Do you have the same issue? I like that the symbols does not take up extra screen space!
The symbols did not show at all on OS X, which is as expected, and I have not tested it on Ubuntu (sorry Ubuntu fans, but Ubuntu really needs to start fixing their GUI bugs).

I have the same issue. The problem is that I'm ad-hocing this on the gtk menu item without a thorough understanding of it. As I have it right now, the interface thinks of these "icons" as a dropdown menu and so it doesn't immediately activate the function, but just goes into the ready state, so it only works after clicking twice. I'm sure there's an easy fix for this, but I'm not knowledgable enough in gtk to figure out how yet.

Not sure if this ever was intended for gtk menus either. But its a great idea! Someone (that is, someone that has spare time, not me!) could maybe create an issue at the gtk bugzilla and ask them to fix this. Or better yet, send a commit to them, but that requires more work of course.

I'm not even sure yet that it's an issue on the end of GTK, it's most likely just me being incompetent and not using the correct method. I'll be looking into this a bit further and I'll come back to you if I have any results

@zalox are you happy with these symbols by the way? If we can make it work? It'll probably only work on Linux in the near future though.

After trawling through a lot of gtk documentation my result seems to be that it would require a completely different approach. Right now the menu is built by hand (as it seems to me). If the menu were built with glade I could make it work. That'd be a major rewrite tho, so I'll leave it as is right now. I'm going to ask on stackexchange if that's alright with you

There are two ways of building menus in gtk, one is deprecated. We use the new non-deprecated approach. Also, it should not matter if the file is in a string (like in juCi++) or in a file. Although, having it in a string makes it more flexible. We can for instance disable these symbols on OS X with a #ifndef.
Sure, feel free to ask on stackexchange.

Hi, and thanks for your contributions!
First, I don't think the debugging buttons belong there. However, they might become useful near the debugging panel when that feature is complete.
The two remaining buttons aren't behaving as they currently indicate. Stop is entirely different from kill last process and start runs compile and run the project. Therefore, if we are adding buttons, we need to define the stop and start artefacts.  Right now, I can't see any way to determine these features in an adequate matter since we need to write a parser to find executables in the build system used, in any case, it would be best effort, not a feature.

Alright, I'll put my approach on ice then. Thanks for the feedback!
Please close the issue at your discretion :)


aaaaa

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
aaaaa
What part(s) of the article would you like to see updated?
aaaa
Additional information
aaaa

سلام


added region variable and argument to vpc module

If I may offer a suggestion @stfusanguinette , you probably don't/won't need this. The region always comes from the provider blocks. So if you're trying to make two VPCs in the same code repository you can just add a second provider with an alias.
Check out some of the backup repositories how that can be accomplished. You'll see there are two providers in main.tf set and they use aliases.

If I may offer a suggestion @stfusanguinette , you probably don't/won't need this. The region always comes from the provider blocks. So if you're trying to make two VPCs in the same code repository you can just add a second provider with an alias.
Check out some of the backup repositories how that can be accomplished. You'll see there are two providers in main.tf set and they use aliases.

Dude, hey, long time no talk! I was going the 2nd provider block route, but Sean wanted me to do it this way. Jake was out so he told me to just make the code and wait for Jake to get back. When Jake got back, we discussed that, yes, the 2nd provider block was the way to go. I'm pretty happy to know I was thinking correctly about this; thanks for confirmation! Hope you're well!


Notification Emails not being sent

When notification for a new lesson is sent, the SendGrid addon for the rssf-worker heroku app seems to be sending emails instead of the SendGrid addon for the rssf heroku app. As a result no one receives emails about new Rails School lessons.

Somehow the heroku config for rssf-worker changed to incorrect sendgrid params. I have fixed this. However because most of our users have gmail and gmail blocks unauthenticated yahoo addresses like Donald's, I had to try and send the gmail users messages from my own account. But I screwed this up and blew all our sendgrid credits for the day. I will send the messages to the gmail users tomorrow morning.


$apolloHelpers.onLogin(token, '/', { expires: 7 }) creating a cookie with different path.

Hello.
I use $apolloHelpers.onLogin when logging in but also in method mounted in the default layout,  to recover the user record when the browser is refreshed.
If after logging in the user goes to, let's say, to path /wallets and refreshes the page a 2nd cookie with path /wallets is created and user management doesn't work.
How can I force $apolloHelpers.onLogin to create a cookie only in  path '/'?
Thanks.
code to recover user data when refreshing browers:
  async mounted() {
    const token = this.$apolloHelpers.getToken()
    if(!!token) {
      const user = await this.$apollo.query( { // how to call a query when needed.
        query: gql`query user($token: String) {
          user(token: $token) {
            _id
            nick
            defaultAddress
            opts {
              _id
              name
              userId
              value
            }
          }
        }`,
        variables: {
          token: token,
        },
      }).then(({data}) => data && data.user)
      console.log("user:", user)
      if(user) {
        await this.$apolloHelpers.onLogin(token, '/', { expires: 7 })
        this.$store.commit("user/set", user)
      }
    }
  },

Apollo configuration in nuxt.conf.js:
apollo: {
   tokenName: 'apollo-token', // optional, default: apollo-token
   cookieAttributes: {
     /** * Define when the cookie will be removed. Value can be a Number
       * which will be interpreted as days from time of creation or a
       * Date instance. If omitted, the cookie becomes a session cookie.  */
     expires: 7, // optional, default: 7 (days)
     /** * Define the path where the cookie is available. Defaults to '/' */
     path: '/', // optional
     /** * Define the domain where the cookie is available. Defaults to
       * the domain of the page where the cookie was created. */
     /** * A Boolean indicating if the cookie transmission requires a
       * secure protocol (https). Defaults to false. */
     secure: false,
   },
   includeNodeModules: true, // optional, default: false (this includes graphql-tag for node_modules folder)
   authenticationType: '', //'Basic', // optional, default: 'Bearer'
   // (Optional) Default 'apollo' definition
   defaultOptions: {
     // See 'apollo' definition for example: default query options
     $query: {
       loadingKey: 'loading',
       fetchPolicy: 'cache-and-network',
     },
   },
   // optional
   errorHandler: '~/plugins/apollo-error-handler.js',
   // required
   clientConfigs: {
     default: {
       // required
       httpEndpoint: 'http://localhost:4000/graphql',
       // optional. See https://www.apollographql.com/docs/link/links/http.html#options
       httpLinkOptions: {
         credentials: 'same-origin',
         headers: {
           'user-agent': 'Server agent'
         }
       },
       // You can use `wss` for secure connection (recommended in production)
       // Use `null` to disable subscriptions
       wsEndpoint: 'ws://localhost:4000/graphql', // optional
       // LocalStorage token
       tokenName: 'apollo-token', // optional
       // Enable Automatic Query persisting with Apollo Engine
       persisting: false, // Optional
       // Use websockets for everything (no HTTP)
       // You need to pass a `wsEndpoint` for this to work
       websocketsOnly: false // Optional
     },
     test: {
       httpEndpoint: 'http://localhost:4000/graphql',
       wsEndpoint: 'ws://localhost:4000/graphql',
       tokenName: 'apollo-token'
     },
   }
 },

This question is available on Nuxt community (#c241)

I think it's a bug. I specify "/" as the path but, somehow, it creates the cookies with the page current path .
Thanks.

Sorry for late response,
First of all, currently we use universal-cookie to handle cookie on both server side and client side. They have the following options https://github.com/reactivestack/cookies/tree/master/packages/universal-cookie#setname-value-options
...
path (string): cookie path, use '/' as the path if you want your cookie to be accessible on all pages
...

/ seem to allot all page to access it. I think another package like js-cookie will be the same.
The second thing, you seem to using wrong onLogin function, (maybe my bad in documented, my english is not very well)
onLogin: async (token, apolloClient = apolloProvider.defaultClient, cookieAttributes = COOKIE_ATTRIBUTES, skipResetStore = false)

the second argument is the client you want to set the token, then
await this.$apolloHelpers.onLogin(token, '/', { expires: 7 })

should look like this
await this.$apolloHelpers.onLogin(token, undefined, { expires: 7, path: '/' })

Thank you very much!!!!. Now I can refresh the page anywhere within the web.


Default project

Maybe add a program argument to build a new project.
Maybe something like:
$ cards new "My Game"

The created project would probably contain only the basic things to get you going:
my-game/
  templates/
    card.html
  cards.csv
  definitions.csv

card.html would be extremely basic:
<div style="width: 100%; margin-top: 1cm; text-align: center;">
  {{ text }}
</div>

cards.csv would be setup with 1 card:

@count
@template
text

1
templates/card.html
This is your first card

definitions.csv would have default values:

@definition
@value

_title
My Game

_description
Description of My Game

_copyright
CC-BY 4.0

_version
1

As of 9a1f5ad there's a basic empty example that can be copied and used as a starting point.

Added in eb2c3fdfab0c716b47eb3d6ec1e512a6e7d980ca
$ cards new "My Fantastic Game"


Consolidate kiam configuration

This is a consolidation of kiam parts into the kiamSupport configuration
It adds the listeners for server and agent as configurable options
The previously KIAMImage type is now underneath kiamSupport, so this is a possible breaking change to existing cluster.yaml files
Now looks like this
  kiamSupport:
    enabled: true
    image:
      repo: quay.io/uswitch/kiam
      tag: v2.8
    server: localhost
    agent: kiam-server

The original DNS health check/server settings are preserved as defaults
The kiam image is bumped to v2.8

Fixes #1350

@mumoshu My original commit failed the test build. It was a very cryptic exit 1 after a diff statement. After a bit of thinking I could only put it down to the go fmt which turned out to be a correct assumption. Kubernetes has a verify stage which checks for that sort of stuff which might make it clearer? What are your thoughts?

Codecov Report

Merging #1354 into master will increase coverage by 0.07%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1354      +/-   ##
==========================================
+ Coverage   36.84%   36.91%   +0.07%
==========================================
  Files          70       70
  Lines        4359     4364       +5
==========================================
+ Hits         1606     1611       +5
  Misses       2526     2526
  Partials      227      227

Impacted Files
Coverage Δ

core/controlplane/config/config.go
64.07% <100%> (+0.2%)
: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 3c38f29...d0f7b0d. Read the comment docs.

@kevtaylor Are you talking about https://travis-ci.org/kubernetes-incubator/kube-aws/builds/388804226?utm_source=github_status&utm_medium=notification specifically?
In that case, how about extracting format verification to its own make target like make verify-fmt? Would it be clearer for you?

@mumoshu Yes, that was the error I was referring to. Wasn't immediately obvious what the error condition was that was occurring. A verify-fmt would be a nice option I feel.

We've had problems with additional behaviour in the client in 2.8 that waited for an address (fix is here: https://github.com/uswitch/kiam/commit/87e067ef41f4411a9de29c40881b53bf26e7a1e2#diff-56019486e31148cc73e6431f00cf320f). This caused the health check to immediately exit and fail
which prevented the server from successfully starting.
For now I'd suggest sticking with 2.7 until we do another release. I'm hoping this will be within the next few weeks but will likely move to 3.0 as there'll be a few significant changes.

@pingles Okay - will push a commit downgrading to 2.7

@mumoshu I have applied the serverAddresses: construction as suggested and downsized to 2.7
/cc @pingles
/cc @c-knowles

I squashed the commits

Edited the commit message to include your nice PR description.


Add systemd service file

Cherry Picked Commit, Untested Service File, Will Update When Ready To Merge

Closing due to the commit affecting multiple things.


Compress CFrames using axis-angle representation

Effectively halves the size of CFrames from 12 f32s down to 6 f32s.
Does not deal with non-orthogonal CFrames but this is not really a big issue as discussed in the discord channel.
EDIT: For future readers, I'll explain the reasoning behind this.

Non orthogonal CFrames can only really be made using the specific CFrame.new constructor where you specify every element of the matrix, or with CFrame.fromMatrix. Most users will never use these constructors as they are only used when doing complex math. They can also come about from accumulated error, but correcting that is probably for the best anyway.
Furthermore, the use of non orthogonal CFrames are extremely limited - apparently being used mostly for skewing cameras. These use cases are rarely, if ever, going to require networking of the skewed CFrame.
And finally, as documented here, https://github.com/Data-Oriented-House/Squash/pull/28, for the squash library, Roblox itself seems to orthonormalize CFrames before being passed over the network anyway. So, Zap isn't missing any features from Roblox's implementation anyway.

Please review harshly, I am still not super fluent with Rust so I don't know the common patterns and what good, clean, Rust code looks like.

adding docs don't merge yet


[CICD] CD Test

📝 Jacoco Test Coverage Report

Overall Project
60%
:x:

There is no coverage information present for the Files changed


Errors not displayed with nested data

Describe the bug
<Form.FieldErrors /> seems to not display error with dataType: 'json'. Manually setting the error with {$errors.address?.postcode != undefined ? $errors.address?.postcode : ""}, however, works.
Reproduction
<form method="POST" use:enhance class="flex flex-col gap-2">
	<Form.Field {form} name="address.postcode">
		<Form.Control let:attrs>
			<Form.Label>Postcode</Form.Label>
			<Input {...attrs} bind:value={$formData.address.postcode} />
		</Form.Control>
		// {$errors.address?.postcode != undefined ? $errors.address?.postcode : ""}
		<Form.FieldErrors />
	</Form.Field>

	<Form.Button class="mt-6 ">Submit</Form.Button>
</form>

const form = superForm(data, {
		SPA: true,
		resetForm: false,
		validators: zodClient(formSchema),
		onUpdate: ({ form }) => {
			console.log(form);
			if (form.valid) {
				$createWarehouseMutation.mutate();
			}
		},
		dataType: 'json'
	});
const { form: formData, enhance, errors } = form;

import { z } from "zod";

export const formSchema = z.object({
  name: z.string(),
  address: z.object({
    country: z.string(),
    state: z.string(),
    city: z.string(),
    streetName: z.string(),
    streetNumber: z.string(),
    building: z.string(),
    apartment: z.string(),
    postcode: z.string().min(4).max(10),
  })
});

export type FormSchema = typeof formSchema

Logs
{
    "valid": false,
    "posted": false,
    "errors": {
        "address": {
            "postcode": [
                "String must contain at least 4 character(s)"
            ]
        }
    },
    "data": {
        "name": "",
        "address": {
            "apartment": "",
            "building": "",
            "city": "",
            "country": "Polska",
            "state": "Mazowieckie",
            "postcode": "",
            "streetName": "",
            "streetNumber": ""
        }
    },
    "constraints": {
        "name": {
            "required": true
        },
        "address": {
            "country": {
                "required": true
            },
            "state": {
                "required": true
            },
            "city": {
                "required": true
            },
            "streetName": {
                "required": true
            },
            "streetNumber": {
                "required": true
            },
            "building": {
                "required": true
            },
            "apartment": {
                "required": true
            },
            "postcode": {
                "minlength": 4,
                "maxlength": 10,
                "required": true
            }
        }
    },
    "id": "uy3her",
    "shape": {
        "address": {}
    }
}

System Info
System:
    OS: Windows 10 10.0.19045
    CPU: (12) x64 12th Gen Intel(R) Core(TM) i5-12400F
    Memory: 12.79 GB / 31.86 GB
  Binaries:
    Node: 18.17.1 - H:\Program Files\nodejs\node.EXE
    npm: 10.4.0 - H:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Chromium (122.0.2365.80)
    Internet Explorer: 11.0.19041.3636
  npmPackages:
    @sveltejs/kit: ^2.0.0 => 2.5.2
    formsnap: ^0.5.1 => 0.5.1
    svelte: ^4.2.7 => 4.2.12
    sveltekit-superforms: ^2.6.2 => 2.6.2
    zod: ^3.22.4 => 3.22.4

Severity
annoyance

I'll look into this specific issue with the Field, but you may be able to use FieldElement in the meantime:
https://github.com/svecosystem/formsnap/issues/146


The link "read instruction" doesn't work now

The link "read instruction" doesn't work now
I am about modal window where I can read mnemonic code and some other things

Thanks, will be fixed in next update.

Fixed in recent release.


Latest version of Jupyterhub extension 2023.7.100 cannot be installed

The latest version of the JupyterHub extension, 2023.7.100, cannot be installed due to incompatibility with the most recent version of code-server 4.16.1. As a result, version 2023.3.100 is installed instead, which does not provide seamless connectivity to the remote JupyterHub.
code-server: v4.16.1
Code: 1.80.2

Solved with the previous pre-release version of the Jupyter extension.

closed


rename back to pyDIS?

It has been suggested to rename this package back to the original working title of "pyDIS", since all the effort has gone in to making it work for the APO 3.5-m DIS data.
Leave a comment if you have any thoughts!
cc: @StellarArmy @tdwilkinson @kolbylyn @jlurie @adleorocks

Agreed. Or pySPEC or else...? pyDIS does still sound like something
contagious, but I like it being specific to python/DIS/both
On Apr 25, 2015, at 3:08 PM, James Davenport<EMAIL_ADDRESS>wrote:
It has been suggested to rename this package back to the original working
title of "pyDIS", since all the effort has gone in to making it work for
the APO 3.5-m DIS data.
Leave a comment if you have any thoughts!
cc: @StellarArmy https://github.com/StellarArmy @tdwilkinson
https://github.com/tdwilkinson @kolbylyn https://github.com/kolbylyn
@jlurie https://github.com/jlurie
—
Reply to this email directly or view it on GitHub
https://github.com/jradavenport/spectra/issues/29.

I always liked pydis and you probably aren't going to spend time to generalize it (soon anyway) and pretty sure no one else will have the name pydis!
Sent from my iPhone

On Apr 25, 2015, at 3:14 PM, James Davenport<EMAIL_ADDRESS>wrote:
It has been suggested to rename this package back to the original working title of "pyDIS", since all the effort has gone in to making it work for the APO 3.5-m DIS data.
Leave a comment if you have any thoughts!
cc: @StellarArmy @tdwilkinson @kolbylyn @jlurie @adleorocks
—
Reply to this email directly or view it on GitHub.

OK, I'm sold, will rename to pyDIS! Links to SPECTRA should automatically redirect...

Done!


Add a scrollbar to the Standings page

https://github.com/sopra-fs23-group-39/sopra-client/commit/e131d4f9007aca5334c0d5bc5920d0841efe8662


Phoenix/sampledata view

Description
sample data api view to return all samples from a particular group, returning specificially:
Private Id -> Sample.private_identifier
Public Id -> Sample.public_identifier
Upload Date -> SequencingRead.sequencing_date
Collection Date -> Sample.collection_date
Collection Location -> Sample.location

Issue
ch120101
Test plan
wrote unit test for view

@ttung is it cool if i merge this?


Make a link to https://github.com/robertfeldt/BlackBoxOptim.jl

In response to the kindness at https://github.com/robertfeldt/BlackBoxOptim.jl#blackboxoptimjl, I suggest we do something similar.

This is solved by commit 80ed3d904e7c137f1c4c7adecaaaf78dff06e827 when PR https://github.com/libprima/PRIMA.jl/pull/17/commits is merged.


find a bug in your'e code  extension doesn't work properly

const extension = file.type.split('/')[1]
this line of code causes a problem
in some types like "text/plain" it doesn't work properly its better extract extension from file name

I create a pull request for this bug
https://github.com/rouftom/react-mui-fileuploader/pull/35


[Slack bot] Match mentions with + in addition to ~

Description

Risk

Deploy Plan

LGTM
I presume we could have just one regex? But fine as is

Indeed ! Did it


copr: Fix search_null incorrect result when scanning small rows for column id > 255

What problem does this PR solve?
Issue Number: close #9119
Problem Summary: Coprocessor may return incorrect results due to wrapping a column id without checking.
What is changed and how it works?
What's Changed:
Check whether column id overflows before wrapping it.
Related changes

Need to cherry-pick to the release branch

Check List
Tests

Unit test
Manual test: Tested against test cases in #9119

$ mysql --host <IP_ADDRESS> --port 4000 -u root < test.sql
C260
260

Release note

Fix a bug that Coprocessor may return wrong result when there are > 255 columns.

/run-all-tests

LGTM

/merge


feat: migrate to ApolloServer

Summary
Migrates sx-api to use ApolloServer.
Test plan
Open localhost:3000 and run more complex query, for example:
query ($spaceId: String!) {
  proposals(first: 100) {
    id
    proposal_id
    space {
      id
      name
    }
  },
  space(id: $spaceId) {
    id
  }
}

With variables
{
  "spaceId": "0x04b7cff71219e275676e0ca23579f41b99dd1d1bd01adc7d7f1bc917d448e57d"
}

This will be annoying to work with, because when linking we will now have two different copies of graphql so linking packages becomes a mess, probably could be avoided if we include ApolloServer as integral part of checkpoint.

@Sekhmet Maybe we can have some kind of Checkpoint modules made of NPM packages with ApolloCheckpoint and ExpressGraphQLCheckpoint which would includes their dependencies, and have a structure that give more freedom on the API side, I can imagine someone may want to expose a JSON-RPC or REST API from it.


1.10.0 is breaking in at least one of our environments.

previous version was fine.
here is the error
the ntp cookbook is called from our base role
================================================================================
Recipe Compile Error in /var/chef/cache/cookbooks/role_base/recipes/default.rb
================================================================================

LocalJumpError
--------------
no block given (yield)

Cookbook Trace:
---------------
  /var/chef/cache/cookbooks/ntp/recipes/default.rb:115:in `block in from_file'
  /var/chef/cache/cookbooks/ntp/recipes/default.rb:112:in `from_file'
  /var/chef/cache/cookbooks/sncr_common_base/recipes/_ntp.rb:10:in `from_file'
  /var/chef/cache/cookbooks/sncr_common_base/recipes/default.rb:14:in `block in from_file'
  /var/chef/cache/cookbooks/sncr_common_base/recipes/default.rb:8:in `each'
  /var/chef/cache/cookbooks/sncr_common_base/recipes/default.rb:8:in `from_file'
  /var/chef/cache/cookbooks/role_base/recipes/_common.rb:52:in `from_file'
  /var/chef/cache/cookbooks/role_base/recipes/default.rb:9:in `from_file'

Relevant File Content:
----------------------
/var/chef/cache/cookbooks/ntp/recipes/default.rb:

108:    action :run
109:    only_if { node['ntp']['sync_hw_clock'] && !(platform_family?('windows') || platform_family?('freebsd')) }
110:  end
111:
112:  service node['ntp']['service'] do
113:    supports status: true, restart: true
114:    action [:enable, :start]
115>>   timeout 30
116:    retries 3
117:    retry_delay 5
118:  end
119:

I'm adding a guard to only set that on Windows, but what version of Chef are you running?  I tested this on a suite of Linux variants without issue.

This is breaking in our AWS environments, where we are using chef-client version 11.12.8 hitting an EC11 server from (predominantly) CentOS 6.[67] hosts.  We are stuck at that version of the chef-client because of some architectural and proxy behavior issues with the chef-client and our enterprise chef.

I pushed a new release up. Can you try that out and let me know if that fixed your issue.

I was just able to test it this morning.  The new version (1.10.1) worked in the test environment where 1.10.0 was broken.
Thank you.


Week1_vishnu

Hey, good job. You can add a screenshot of the website to PR description.

Thank you for your suggestion! I have added a screenshot of the work to the pull request for reference. Let me know if you have any further questions or concerns.


Call HandleExitCoder for all members of MultiError.Errors

The document of HandleExitCoder say:

If the given error is a MultiError, then this func is called on all members of the Errors slice.

but HandleExitCoder is called only once for the first element of Errors slice.
Because HandleExitCoder which is called in HandleExitCoder will os.Exit.
    if multiErr, ok := err.(MultiError); ok {
        for _, merr := range multiErr.Errors {
            HandleExitCoder(merr) // will exit
        }
        return
    }

Hi @hirose31,
I believe I see your intention, but this implementation wouldn't handle the case where the MultiError itself consists of ExitCoders with differing status codes besides 1. I can see the case for printing all of the error messages before exiting, but I think we'll want need to make a decision about which status code to exit with (I'm inclined to exit with the code of the first matching ExitCoder and 1 if none in the list are MultiErrors).
What do you think?

I'm inclined to exit with the code of the first matching ExitCoder and 1 if none in the list are MultiErrors

I agree 👍

@jszwedko
I've just added document on exit code and test cases. Could you please review commits?

@hirose31 thank you! Just one more note, but otherwise this is looking good.

Perfect, thank you @hirose31


Auth UI SSO login part 1

require #1331
connect #1320
The pr only covers login with general sso login, tested with google login only.
Question:

Now when there is error in the sso flow, we redirect user back to login page. Maybe we should update the error display to a more generic place? Currently it shows above the login id form

TODO:

Login with apple flow should be different? Will further study.
Link with sso provider, the connect buttons should be appear in the settings page

Per our discussion, we want to put the csrf token into sso state before user redirect to the provider auth endpoint. When the provider callback the auth gear, we can check the csrf token to verify the callback is handled in the same user agent.
But I found that the reason why we cannot get back the csrf cookies from the callback, it is not the problem of gorilla csrf setup, it's because we set response_mode to form_post. In browser, if the form post is triggered by another domain, the cookies will not be included. If the callback type is a GET request, the cookies will be included in the request. I think we will not want to change the response_mode to GET, right? Still thinking any better way to verify the callback is in the same browser... any suggestion?

But I found that the reason why we cannot get back the csrf cookies from the callback, it is not the problem of gorilla csrf setup, it's because we set response_mode to form_post. In browser, if the form post is triggered by another domain, the cookies will not be included.

I think we should set SameSite=None for this to work?

I think we should set SameSite=None for this to work?

That would defeat the purpose of CSRF cookie?

That would defeat the purpose of CSRF cookie?

We're using double-submit cookie pattern for CSRF protection. As long as attacker cannot read/write the cookie, the protection should work. SameSite=None still only allows the cookie domain to read/write it, so there should be no problem.

Thanks @louischan-oursky @kiootic. Will check on this:)
TODO:

preserve whole encoded query in sso state, not only for redirect_uri but also other parameters which will be used in future
Try obtain the csrf  cookies from callback by setting SameSite=None
Update callback endpoint path

Rebased and updated:)


stunnel: use a stable source url

Description of changes
The previous url on https://www.stunnel.org/archive/ only works with the latest version of stunnel, making stunnel impossible to build as soon as a new release has been made (which is frequent enough for stable versions of nixpkgs)
This ftp url apparently contains all the archives, and the hash are the same, so the link can just be updated without side effects
Fixes: #200288
Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Please ignore this, the url has already been fixed at some point -- /archive is fine, the one in the ticket was /download.


[bug] System tray icon gets blurry on Windows after changing system UI scaling

Describe the bug
System tray icon gets blurry on Windows after changing system UI scaling
Reproduction
Go to settings and change system UI scaling to 175% and change it back to 100%, you can do it multiple times to make the icon even more blurry
Expected behavior
Not blurry
Full tauri info output
[✔] Environment
    - OS: Windows 10.0.19045 X64
    ✔ WebView2: 123.0.2420.65
    ✔ MSVC: Visual Studio Community 2022
    ✔ rustc: 1.76.0 (07dca489a 2024-02-04)
    ✔ cargo: 1.76.0 (c84b36747 2024-01-18)
    ✔ rustup: 1.27.0 (bbb9276d2 2024-03-08)
    ✔ Rust toolchain: stable-x86_64-pc-windows-msvc (default)
    - node: 20.10.0
    - pnpm: 8.15.4
    - npm: 10.5.0

[-] Packages
    - tauri [RUST]: 2.0.0-beta.13
    - tauri-build [RUST]: 2.0.0-beta.10
    - wry [RUST]: 0.37.0
    - tao [RUST]: 0.26.2
    - @tauri-apps/api [NPM]: 2.0.0-beta.6
    - @tauri-apps/cli [NPM]: 2.0.0-beta.11

[-] App
    - build-type: bundle
    - CSP: connect-src ipc: http://ipc.localhost ws: localhost:* <IP_ADDRESS>:*
    - frontendDist: ../dist
    - devUrl: http://localhost:3000/
    - framework: SolidJS
    - bundler: Vite

Stack trace
No response
Additional context
No response

This doesn't seem like a bug at all but rather the icon size is not optimal, I am using a 32x32 icon and here is how it looks:

100% scaling

175% scaling

I also included 3 icons of electron apps that I use and they are more blurry than the icon I used
Can you provide the icon you used for tray icon?

I mean it's blurry after you switch back from 175% to 100%, here's a screen shots for comparison
Initial state

After going back and forward a few times

I can finally reproduce, seems like it happens when there is at least one Window open, and works fine if no windows are open at all. Seems like changing the display scaling makes the window unresponsive (@Legend-Master can you configrm this too?) and it somehow affects the tray icon.

My window is fine though, I can interact with it just fine after changing the scaling

That's weird, I will investigate later. I should also mention although I get a slightly blurier (or rather more pixelated) icon than the original, it is not the same as what you see.

I did maybe 4-5 times, and the icon used was https://github.com/tauri-apps/tauri/blob/dev/examples/.icons/icon.ico

I was using https://github.com/tauri-apps/tauri/blob/dev/examples/.icons/32x32.png can you see how this one behaves?

I'll give it a try now

https://github.com/tauri-apps/tauri/blob/dev/examples/.icons/32x32.png after 3 times:

I guess the blurriness are not consistent across machines or rather across Windows versions as I am using Windows 11.


use gauge tire

Thresholds are incorrect for psi units

@sdwalker thanks for the notice, it's solved in https://github.com/CarlosCuezva/dashboards-Grafana-Teslamate/releases/tag/v<IP_ADDRESS>


fix(symbolize): cleanup

Codecov Report
Attention: 14 lines in your changes are missing coverage. Please review.

Comparison is base (734ab34) 59.79% compared to head (210682c) 59.84%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1048      +/-   ##
==========================================
+ Coverage   59.79%   59.84%   +0.04%
==========================================
  Files          19       19
  Lines        2572     2570       -2
  Branches      913      913
==========================================
  Hits         1538     1538
+ Misses        692      690       -2
  Partials      342      342

Files
Coverage Δ

src/symbolize.cc
54.62% <26.31%> (+0.33%)
:arrow_up: