disk-smart3: Some issue detecting disk behind a "DELL" raid card

Hi,
The plugin is unable to detect drive behind a DELL Raid BUT the smartctl daemon is running perfectly and able to detect as we are receiving email from the smartd daemon!
For example:
The following warning/error was logged by the smartd daemon:

Device: /dev/bus/2 [megaraid_disk_03], SMART Failure: DATA CHANNEL IMPENDING FAILURE DATA ERROR RATE TOO HIGH

Device info:
[HP       DG146ABAB4       HPD5], lu id: 0x5000c5000517eb53, S/N: 3NM1QWK600009742E1XH, 146 GB

And unable to get data from the boot disk too!
smartctl failed with exit status "Device open failed, device did not return an IDENTIFY DEVICE structure, or device is in a low-power mode."

What does lsblk --nodeps --output name,type --noheadings --exclude 9,11,230,252,253,254 returns? Do you see your MegaRAID devices there? Is your boot device listed?
What does smartctl --xall /path/to/your/boot/disk returns?

The lsblk command return
sda  disk sdb  disk sdc  disk sdd  disk sde  disk sdf  disk sdg  disk sdh  disk sdi  disk sdj  disk sdk  disk sdl  disk sdm  disk sdn  disk sdo  disk sdp  disk sdq  disk sdr  disk sds  disk sdt  disk sdu  disk sdv  disk sdw  disk sdx  disk sdy  disk sdz  disk sdaq disk sdar disk sdas disk sdat disk sdau disk sdav disk sdaw disk sdax disk sday disk sdaz disk sdba disk sdbb disk sdbc disk sdbd disk sdbe disk sdbf disk sdbg disk sdbh disk sdbi disk sdbj disk sdbk disk sdbl disk sdbm disk sdbn disk sdbo disk sdbp disk sdbq disk sdbr disk sdbs disk sdbt disk sdbu disk sdbv disk sdbw disk sdbx disk sdby disk sdbz disk sdca disk sdcb disk sdcc disk sdcd disk
The megaraid disk is sdb (that's the LUN itself)
and the smartctl on bootdisk give us
The output of megaclisas-status will give us:
`-- Controller information --
-- ID | H/W Model           | RAM    | Temp | BBU    | Firmware
c0    | PERC 6/i Integrated | 256MB  | N/A  | Good   | FW: <IP_ADDRESS>2
-- Array information --
-- ID | Type   |    Size |  Strpsz | Flags | DskCache |   Status |  OS Path | CacheCade |InProgress
c0u0  | RAID-5 |    953G |   64 KB | RA,WB |  Default |  Optimal | /dev/sdb | None      |None
-- Disk information --
-- ID   | Type | Drive Model                            | Size     | Status          | Speed    | Temp | Slot ID  | LSI ID
c0u0p0  | HDD  | HP DG0146FAMWL HPDC3SD1SY0W            | 136.1 Gb | Online, Spun Up | Unknown  | 28C  | [32:0]   | 0
c0u0p1  | HDD  | HP DG146BB976 HPDC3NM5GL54             | 136.1 Gb | Online, Spun Up | Unknown  | 30C  | [32:1]   | 1
c0u0p2  | HDD  | HP DG146BB976 HPDF3NM8QF0P             | 136.1 Gb | Online, Spun Up | Unknown  | 32C  | [32:2]   | 2
c0u0p3  | HDD  | IBM MBE2147RC 6010D300ST2P601060106010 | 136.1 Gb | Online, Spun Up | Unknown  | 30C  | [32:3]   | 3
c0u0p4  | HDD  | HP DG146BB976 HPDC3NM7T80Z             | 136.1 Gb | Online, Spun Up | Unknown  | 29C  | [32:4]   | 4
c0u0p5  | HDD  | HP DG146ABAB4 HPD73NM3BNDL             | 136.1 Gb | Online, Spun Up | Unknown  | 30C  | [32:5]   | 5
c0u0p6  | HDD  | HP DG0146FAMWL HPDB3SD0RV8T            | 136.1 Gb | Online, Spun Up | Unknown  | 28C  | [32:6]   | 6
c0u0p7  | HDD  | HP DG146BB976 HPDF3NM5A3PR             | 136.1 Gb | Online, Spun Up | Unknown  | 30C  | [32:7]   | 7
`
`smartctl 7.2 2020-12-30 r5155 [x86_64-linux-5.3.13-1-pve] (local build)
Copyright (C) 2002-20, Bruce Allen, Christian Franke, www.smartmontools.org
=== START OF INFORMATION SECTION ===
Model Family:     Phison Driven SSDs
Device Model:     KINGSTON SA400S37120G
Serial Number:    50026B7682BD52D8
LU WWN Device Id: 5 0026b7 682bd52d8
Firmware Version: S1Z40102
User Capacity:    120,034,123,776 bytes [120 GB]
Sector Size:      512 bytes logical/physical
Rotation Rate:    Solid State Device
TRIM Command:     Available
Device is:        In smartctl database [for details use: -P show]
ATA Version is:   ACS-3 T13/2161-D revision 4
SATA Version is:  SATA 3.2, 6.0 Gb/s (current: 3.0 Gb/s)
Local Time is:    Tue May  3 09:45:51 2022 EDT
SMART support is: Available - device has SMART capability.
SMART support is: Enabled
AAM feature is:   Unavailable
APM feature is:   Disabled
Rd look-ahead is: Enabled
Write cache is:   Enabled
DSN feature is:   Unavailable
ATA Security is:  Disabled, frozen [SEC2]
Wt Cache Reorder: Unavailable
=== START OF READ SMART DATA SECTION ===
SMART overall-health self-assessment test result: PASSED
General SMART Values:
Offline data collection status:  (0x00) Offline data collection activity
was never started.
Auto Offline Data Collection: Disabled.
Self-test execution status:      (   0) The previous self-test routine completed
without error or no self-test has ever
been run.
Total time to complete Offline
data collection:                (  120) seconds.
Offline data collection
capabilities:                    (0x11) SMART execute Offline immediate.
No Auto Offline data collection support.
Suspend Offline collection upon new
command.
No Offline surface scan supported.
Self-test supported.
No Conveyance Self-test supported.
No Selective Self-test supported.
SMART capabilities:            (0x0002) Does not save SMART data before
entering power-saving mode.
Supports SMART auto save timer.
Error logging capability:        (0x01) Error logging supported.
General Purpose Logging supported.
Short self-test routine
recommended polling time:        (   2) minutes.
Extended self-test routine
recommended polling time:        (  10) minutes.
SMART Attributes Data Structure revision number: 1
Vendor Specific SMART Attributes with Thresholds:
ID# ATTRIBUTE_NAME          FLAGS    VALUE WORST THRESH FAIL RAW_VALUE
1 Raw_Read_Error_Rate     -O--CK   100   100   000    -    100
9 Power_On_Hours          -O--CK   100   100   000    -    20971
12 Power_Cycle_Count       -O--CK   100   100   000    -    6
148 Unknown_Attribute       ------   100   100   000    -    0
149 Unknown_Attribute       ------   100   100   000    -    0
167 Write_Protect_Mode      ------   100   100   000    -    0
168 SATA_Phy_Error_Count    -O--C-   100   100   000    -    0
169 Bad_Block_Rate          ------   100   100   000    -    0
170 Bad_Blk_Ct_Erl/Lat      ------   100   100   010    -    0/0
172 Erase_Fail_Count        -O--CK   100   100   000    -    0
173 MaxAvgErase_Ct          ------   100   100   000    -    0
181 Program_Fail_Count      -O--CK   100   100   000    -    0
182 Erase_Fail_Count        ------   100   100   000    -    0
187 Reported_Uncorrect      -O--CK   100   100   000    -    0
192 Unsafe_Shutdown_Count   -O--C-   100   100   000    -    5
194 Temperature_Celsius     -O---K   034   054   000    -    34 (Min/Max 28/54)
196 Reallocated_Event_Count -O--CK   100   100   000    -    0
199 SATA_CRC_Error_Count    -O--CK   100   100   000    -    0
218 CRC_Error_Count         -O--CK   100   100   000    -    0
231 SSD_Life_Left           ------   088   088   000    -    88
233 Flash_Writes_GiB        -O--CK   100   100   000    -    4429
241 Lifetime_Writes_GiB     -O--CK   100   100   000    -    4594
242 Lifetime_Reads_GiB      -O--CK   100   100   000    -    2831
244 Average_Erase_Count     ------   100   100   000    -    129
245 Max_Erase_Count         ------   100   100   000    -    174
246 Total_Erase_Count       ------   100   100   000    -    55399
||||||_ K auto-keep
|||||__ C event count
||||___ R error rate
|||____ S speed/performance
||_____ O updated online
|______ P prefailure warning
General Purpose Log Directory Version 1
SMART           Log Directory Version 1 [multi-sector log support]
Address    Access  R/W   Size  Description
0x00       GPL,SL  R/O      1  Log Directory
0x01           SL  R/O      1  Summary SMART error log
0x02           SL  R/O      1  Comprehensive SMART error log
0x03       GPL     R/O      1  Ext. Comprehensive SMART error log
0x04       GPL,SL  R/O      8  Device Statistics log
0x06           SL  R/O      1  SMART self-test log
0x07       GPL     R/O      1  Extended self-test log
0x10       GPL     R/O      1  NCQ Command Error log
0x11       GPL     R/O      1  SATA Phy Event Counters log
0x30       GPL,SL  R/O      9  IDENTIFY DEVICE data log
0x80-0x9f  GPL,SL  R/W     16  Host vendor specific log
0xde       GPL     VS       8  Device vendor specific log
SMART Extended Comprehensive Error Log Version: 1 (1 sectors)
No Errors Logged
SMART Extended Self-test Log Version: 1 (1 sectors)
No self-tests have been logged.  [To run self-tests, use: smartctl -t]
Selective Self-tests/Logging not supported
SCT Commands not supported
Device Statistics (GP Log 0x04)
Page  Offset Size        Value Flags Description
0x01  =====  =               =  ===  == General Statistics (rev 1) ==
0x01  0x008  4               6  ---  Lifetime Power-On Resets
0x01  0x010  4           20971  ---  Power-on Hours
0x01  0x018  6<PHONE_NUMBER>  ---  Logical Sectors Written
0x01  0x020  6       180365214  ---  Number of Write Commands
0x01  0x028  6<PHONE_NUMBER>  ---  Logical Sectors Read
0x01  0x030  6        28256879  ---  Number of Read Commands
0x07  =====  =               =  ===  == Solid State Device Statistics (rev 1) ==
0x07  0x008  1              14  ---  Percentage Used Endurance Indicator
|||_ C monitored condition met
||__ D supports DSN
|___ N normalized value
Pending Defects log (GP Log 0x0c) not supported
SATA Phy Event Counters (GP Log 0x11)
ID      Size     Value  Description
0x0001  4            0  Command failed due to ICRC error
0x0002  4            0  R_ERR response for data FIS
0x0005  4            2  R_ERR response for non-data FIS
0x000a  4            6  Device-to-host register FISes sent due to a COMRESET
`

Do you run the check plugin with root privileges? Try running it under the root account, or, if configured, try sudo /path/to/disk-smart3.
What does disk-smart3 return in case of the MegaRAID controller?

How does your /etc/smartmontools/smartd.conf look like?

The smartd.conf file is the default one with nothing in it except
DEVICESCAN -d removable -n standby -m root -M exec /usr/share/smartmontools/smartd-runner

If the problem still exists, a remote session might help clarifying some things. Drop us a note at our info mail address on how to get in contact. Closing this for now as we can't help in this special case.


IIS app init, idle timeout, lAPI, and versioning update

Fixes #12232
Internal Review Topic (App init and timeout section)
The most important updates here are for the IIS topic:

Move the app init/preload/idle timeout coverage to the IIS topic from the ANCM topic.
Enhance the language+layout of the content.
Group app init/preload and idle timeout under a 2nd-level heading and provide each with a 3rd-level heading.
Include an additional resources just for this section to make those resources easy to access from here.
While I'm here, I add a remark on why No Managed Code is set.

Other updates include:

Dates
Versioning
API link updates

Note WRT testing the settings: I don't have a lot of 🕐 to test unless you prefer that I do so. My concern from @sebastienricher's remarks is that the app (its background hosted service) wasn't auto-starting under all scenarios with these settings. @sebastienricher, would you like to add a remark here?
If so, list for the engineers:

Scenarios that worked (e.g., restart of server, restart of WAS)
Scenarios that didn't work (e.g., recycle of worker process)

Thanks 🎷 @sebastienricher :guitar: for the issue and convo!
Note WRT the IIS topic: I find the IIS topic confusing—the delta between app init and preload isn't clear to me from that text. However, it hardly matters over here. The topic here can focus on implementation.

@shirhatti Do you want to look? If not, I'll call for the docs review.


Is it possible to use PRelu activation layers and LeakyRelu layers?

From your guides, I can see that we can use activation layers from the functional api as such:
x = Conv1D(params['filters'], 3, padding='same',activation=params['activation'])(x)
Can we also use hyperparameter optimisation with LeakyRelu or PRelu layers like so:
x = Conv1D(params['filters'], 3, padding='same')(x)
x = PReLU()(x)

Can I specify that I want this advanced activation layer to be either PRelu, LeakyRelu, Relu, Elu?
Thanks for any help

I don't see any reason why you can't do this. Talos API is just a workflow layer that can be added on top of Keras API i.e. generally speaking Talos does not support specific features of Keras API, but Keras API.
Closing here. Feel free to open new issue if anything else.

Thanks for your help.
I was able to achieve this using a list of strings like so:
x = Conv1D(filters, 3, padding = 'same', name = 'enc1')(inputTensor1)
if params['activation'] == 'leakyrelu':
    x = keras.layers.LeakyReLU(alpha = 0.1)(x)
elif params['activation'] == 'prelu':
    x = keras.layers.PReLU(alpha_initializer = 'zeros', alpha_regularizer = None, alpha_constraint = None, shared_axes = None)(x)
else:
    x = Activation(params['activation'])(x)

This only works for custom layers already built into keras. If you want to use cuttom layers from e.g. keras_contrib, then you're out of luck as soon as you want to deploy or fetch the best model. Unfortunately, I discovered this issue after running five days worth of hyperparameter optimization.

then you're out of luck as soon as you want to deploy or fetch the best model. Unfortunately, I discovered this issue after running five days worth of hyperparameter optimization.

@bjtho08 Can you explain what do you mean by this? This should be really straightforward to resolve. Better open a new ticket though.

I will do it within the next hour or so. I'm on my phone right now, but as soon as I get to my office, I can sit down and make a ticket.


updating comparison criteria and adding platform-based conditions in requirements file

sorry, maybe next time :D


⚠ ClamAV has degraded performance

In d584adf, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 223 ms

Resolved: ClamAV performance has improved in 2350a7f after 5 minutes.


Use kyverno namespace

Towards https://github.com/giantswarm/giantswarm/issues/22435 and https://github.com/giantswarm/giantswarm/issues/23971
Moves kyverno to its own namespace on Giant Swarm MCs. Does not affect the behavior of the app in workload clusters.
Checklist

[ ] Update changelog in CHANGELOG.md.
[ ] Make sure values.yaml and values.schema.json are valid.

Tested using a branch of the azure app collection on gremlin


fix security issues

fix bug mentioned in https://github.com/oblac/jodd/issues/787
add urlencode in jodd.http.HttpRequest#path()

Need to add tests, will merge it today


Run multiple instances on same machine

ODB can't currently run correctly on
same machine as both instances will
share globalDb. By passing in nodeId
to the DB name we can remove this
contention

Supersedes https://github.com/omnilaboratory/obd/pull/19

This is fixed.

Is this fixed? @obdCoder

OK, after chatting with team, here is how to run multiple instance on one machine:
copy obd working fold to multiple directories, and use different port number:
---
    |------path 1: /home/lightning/node1/obd.exe, /configure, port 1000001
    |------path 2: /home/lightning/node2/obd.exe, /configure, port 1000002
    |------path 3: /home/lightning/node3/obd.exe, /configure, port 1000003


Configure: fix handling of build.info attributes with value

This line wasn't properly handled:
SCRIPTS{misc,linkname=tsget}=tsget.pl

It generated an attribute "linkname=tsget" with the value 1, instead of
what it should have, an attribute "linkname" with the value "tsget".
Fixes #12341

This pull request is ready to merge

Merged
ea4ee152a7aa022dd87f193cc6e16a7ffbfb455a Configure: fix handling of build.info attributes with value


Nav Bar gets stuck midway through transition state when using the interactivePopGestureRecognizer

When swiping to go back to a previous view controller, it's possible to get the NavBar in a bad state where it seems to be stuck midway through a transition. This is reproducible in the demo app. I've attached a video showing it happening (zipped because movie files arent allowed).Depending on what else you have in the Nav Bar, the results can be more problematic. In one app i'm attempting to use this project in, you can get stuck in the current view controller and are unable to go back via either the back button or the swipe back gesture.
bug_repro.zip

I'd be interested in possible workarounds.

me too


ESPN errors out

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2020.05.08
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log
[debug] System config: [u'--prefer-ffmpeg', u'--restrict-filenames']
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'https://www.espn.com/watch/player/_/id/a0514bdc-2c04-423b-b1bf-26220d6e5d5e/bucketId/35312']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2020.05.08
[debug] Python version 2.7.16 (CPython) - Linux-4.19.0-9-686-pae-i686-with-debian-10.4
[debug] exe versions: avconv 4.1.4-1, avprobe 4.1.4-1, ffmpeg 4.1.3-static, ffprobe 4.1.3-static, rtmpdump 2.4
[debug] Proxy map: {}
[ESPNArticle] 35312: Downloading webpage
ERROR: Unable to extract video id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 797, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 530, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/espn.py", line 208, in _real_extract
    webpage, 'video id', group='id')
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 1005, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract video id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description
WRITE DESCRIPTION HERE
Show episode on ESPN site.  Probably does require credentials for just about ANY USA cable provider (ESPN is on probably every provider in the United States) however the error does not indicate a credential failure.  It appears the URL is a new format that doesn't exactly match any of the formats the extractor knows about.  Also tested leaving off the '/bucketId/35312' at the end of the URL but that did not work either.

The current code still expects a numeric ID there... it'd be great to get a quick fix if possible :)
youtube-dl --verbose -F https://www.espn.com/watch/player/_/id/e67e0893-2ba6-4033-be6f-bddfe31bf267
[...]
[debug] youtube-dl version 2020.11.19
[...]
[ESPNArticle] e67e0893-2ba6-4033-be6f-bddfe31bf267: Downloading webpage
ERROR: Unable to extract video id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp1_szunss\build\youtube_dl\YoutubeDL.py", line 797, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp1_szunss\build\youtube_dl\extractor\common.py", line 532, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp1_szunss\build\youtube_dl\extractor\espn.py", line 208, in _real_extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp1_szunss\build\youtube_dl\extractor\common.py", line 1010, in _search_regex
youtube_dl.utils.RegexNotFoundError: Unable to extract video id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.```

OK, after checking the site's code it's evident the new ID signals a slightly different API with a new entrypoint and Adobe Pass integration (which is already available within the current codebase) so if anyone with a little more Python expertise than me could try adapting the code to keep this important site working I'd certainly help with the metadata parsing and such 👍

What show/episode are you trying to watch?


Why the template of HasEdge and HasEdgeWith differs in the sqlgrph.To target table?

In the latest commit and the earliest sqlgraph commit of entc/gen/template/dialect/sql/predicate.tmpl:
https://github.com/ent/ent/blob/2c63d1d70e/entc/gen/template/dialect/sql/predicate.tmpl#L60
https://github.com/ent/ent/blob/f7ce61fc3c/entc/gen/template/dialect/sql/predicate.tmpl#L67
The HasEdge implements with sqlgraph.To TableConstant in the condition !HasCompositeID, but the HasEdgeWith use InverseTableConstant in the same condition.
The implementation keeps a long time (3 years), which confuses me whether I have misunderstood, or it is a bug?

Hey @Epsirom 👋
Just to ensure, do you mean why the dialect/sql/predicate/edge/has template was changed? It was modified a few months ago to add support for Edge Schemas.
Do you mean the difference between these 2:
https://github.com/ent/ent/blob/master/entc/gen/template/dialect/sql/predicate.tmpl#L58-L64
https://github.com/ent/ent/blob/master/entc/gen/template/dialect/sql/predicate.tmpl#L94-L100

Hey @Epsirom 👋 Just to ensure, do you mean why the dialect/sql/predicate/edge/has template was changed? It was modified a few months ago to add support for Edge Schemas.
Do you mean the difference between these 2: https://github.com/ent/ent/blob/master/entc/gen/template/dialect/sql/predicate.tmpl#L58-L64 https://github.com/ent/ent/blob/master/entc/gen/template/dialect/sql/predicate.tmpl#L94-L100

Thanks for your reply.
I figure out that it is my mistake. I wrote an additional has template for soft deletion, which adds a sqlgraph.HasNeighborsWith to the sqlgraph step. The addition generates SELECT SQL in the same table and confused me.
Thank you again for your work.

I submitted https://github.com/ent/ent/pull/3144 to clarify that Has<E> does not need the sqlgraph.To option.


feat(admin-service): admin service authorization (DSP-1772)

resolves DSP-1772

@mdelez Please make a note somewhere, of all the things that are now hardcoded, what configuration needs to be supplied to the app at runtime.

@mdelez Please make a note somewhere, of all the things that are now hardcoded, what configuration needs to be supplied to the app at runtime.

should I do that in the docs? I need to update them anyways

@mdelez Please make a note somewhere, of all the things that are now hardcoded, what configuration needs to be supplied to the app at runtime.

should I do that in the docs? I need to update them anyways

yes, you could maybe add something like a deployment configuration section.


sw-offline-omniture-analytics

Library Affected:
sw-offline-omniture-analytics
Browser & Platform:
all browsers
Issue or Feature Request Description:
There's some interest in replicating the general approach of sw-offline-google-analytics, but making it work with the URLs used by the Omniture/Adobe analytics platform.
Bonus points if this was done in a way that abstracted the idea of queueing/replaying requests in generic core library, and then we can just add in specific logic to match the URL patterns used by each of the platforms we care about, with helper methods to adjust the URL parameters appropriately before the URLs are replayed.

Any thoughts on what to do with the responses? Seems difficult to return them to the caller in a reasonable way. Maybe fetchMaybeLater() could return an integer id (similar to setTimeout) that can be used to cancel requests and retrieve responses. (The caller will probably need to poll.)
Another possibility is to throw the responses away.

@ithinkihaveacat Did you have any interest in working on this one? :)

Why yes, yes I do.
You got any thoughts on what to do about responses? (I haven't got any better ideas in the month since I last thought about this…)

I'm much more familiar with the Google Analytics platform, but at least there, the responses to the hit requests are universally 200 OK, and aren't meaningful. The code that replays failed Google Analytics hits just ignores the responses rather than trying to communicate them to the controlled clients.

Closing this since the next google analytics should be based off of bg-sync which someone could use to write this library if they wanted.


Determine ontology storage/lookup plan for M1/M2

Questions to answer:

[ ] What should be stored in the ledger?
[ ] What should be stored in the artifact?
[ ] What is the responsibility of the consumer ETL jobs?

What should be stored in the ledger?
One table that holds all terms from the relevant databases.
The table uses the term ID as the primary key (e.g. EFO:0001182)
All other fields in the table are filled directly from the ontology database (label, synonyms)
Other tables will link to this by submitting only the term ID
What should be stored in the artifact?
All metadata from the ontology table that is linked to another object.
(The unlinked ontology metadata could be included, it just doesn't serve much purpose)
What is the responsibility of the consumer ETL jobs?
Display the ontology metadata in the artifact as it would any other metadata
So I think an appropriate question to add is: How is the ontology metadata loaded to the ledger?

@parthshahva our discussion is listed above. If this meets your requirements, this issue can be closed.


public class ClipboardModule extends NativeClipboardModuleSpec

After npx react-native run-android , I get :
/Users/andreyradkevich/dev/native_cabinet/node_modules/@react-native-clipboard/clipboard/android/src/main/java/com/reactnativecommunity/clipboard/ClipboardModule.java:39: error: ClipboardModule is not abstract and does not override abstract method removeListeners(double) in NativeClipboardModuleSpec
public class ClipboardModule extends NativeClipboardModuleSpec {
       ^
/Users/andreyradkevich/dev/native_cabinet/node_modules/@react-native-clipboard/clipboard/android/src/main/java/com/reactnativecommunity/clipboard/ClipboardModule.java:240: error: method does not override or implement a method from a supertype
  @Override
  ^
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note<EMAIL_ADDRESS>uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
2 errors

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':react-native-clipboard_clipboard:compileDebugJavaWithJavac'.
> Compilation failed; see the compiler error output for details.

* Try:
> Run with --info option to get more log output.
> Run with --scan to get full insights.

BUILD FAILED in 7s
error Failed to install the app. Command failed with exit code 1: ./gradlew app:installDebug -PreactNativeDevServerPort=8081
/Users/andreyradkevich/dev/native_cabinet/node_modules/@react-native-clipboard/clipboard/android/src/main/java/com/reactnativecommunity/clipboard/ClipboardModule.java:39: error: ClipboardModule is not abstract and does not override abstract method removeListeners(double) in NativeClipboardModuleSpec
public class ClipboardModule extends NativeClipboardModuleSpec { ^
/Users/andreyradkevich/dev/native_cabinet/node_modules/@react-native-clipboard/clipboard/android/src/main/java/com/reactnativecommunity/clipboard/ClipboardModule.java:240: error: method does not override or implement a method from a supertype @Override ^
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note<EMAIL_ADDRESS>uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
2 errors FAILURE: Build failed with an exception. * What went wrong:
Execution failed for task ':react-native-clipboard_clipboard:compileDebugJavaWithJavac'.
> Compilation failed; see the compiler error output for details. * Try:
> Run with --info option to get more log output.
> Run with --scan to get full insights. BUILD FAILED in 7s.

package.json
    "@react-native-clipboard/clipboard": "^1.14.1",
     "react-native": "^0.73.4",

    ext {
        buildToolsVersion = findProperty('android.buildToolsVersion') ?: '34.0.0'
        minSdkVersion = Integer.parseInt(findProperty('android.minSdkVersion') ?: '23')
        compileSdkVersion = Integer.parseInt(findProperty('android.compileSdkVersion') ?: '34')
        targetSdkVersion = Integer.parseInt(findProperty('android.targetSdkVersion') ?: '34')
        kotlinVersion = findProperty('android.kotlinVersion') ?: '1.8.10'

        ndkVersion = "25.1.8937393"
    }
    ```

same error

It seems to me #237 broke build on android?
react-native 0.73.4, new arch enabled
Changing type in ClipboardModule.java fixed it for me
before:
  @Override
  public void removeListeners(int count) {

  }

after:
  @Override
  public void removeListeners(double count) {

  }

为何一直不更新库 ，每次编译都报错，需要手动更改

this issue is come from "@react-native/codegen" version 0.74.85
to fix that, we need to make sure that use "@react-native/codegen" version 0.74.84
"dependencies": {
   "@react-native/metro-config": "0.74.2",
   "react-native": "0.74.2"
}

or force resolutions
"resolutions": {
   "@react-native/codegen": "0.74.84",
}

Facing the same issue with RN 0.74.4 and new architecture enabled. Any non-hacky solutions other than patching?


[manylinux] dlopen: cannot load any more object with static TLS

Hi,
I am afraid the manylinux silx package is not working. Or maybe i have a problem on my env. I am using a Debian 8.
$ pip install silx==0.5 --user
Collecting silx==0.5
  Downloading silx-0.5.0-cp27-cp27mu-manylinux1_x86_64.whl (5.8MB)
    100% |████████████████████████████████| 5.8MB 233kB/s
Successfully installed silx-0.5.0

Then
$ ./bootstrap.py pyFAI-calib2
WARNING:root:Failed to bind rconsole to socket port, port=54321.
WARNING:silx.gui.plot.backends._matplotlib:matplotlib already loaded, setting its backend may not work
Traceback (most recent call last):
  File "./bootstrap.py", line 200, in <module>
    run_entry_point(target, argv)
  File "./bootstrap.py", line 125, in run_entry_point
    func()
  File "/workspace/valls/pyfai.git/build/lib.linux-x86_64-2.7/pyFAI/app/calib2.py", line 385, in main
    widget = CalibrationWindow()
  File "/workspace/valls/pyfai.git/build/lib.linux-x86_64-2.7/pyFAI/gui/calibration/CalibrationWindow.py", line 44, in __init__
    self.__tasks = self.createTasks()
  File "/workspace/valls/pyfai.git/build/lib.linux-x86_64-2.7/pyFAI/gui/calibration/CalibrationWindow.py", line 54, in createTasks
    from pyFAI.gui.calibration.ExperimentTask import ExperimentTask
  File "/workspace/valls/pyfai.git/build/lib.linux-x86_64-2.7/pyFAI/gui/calibration/ExperimentTask.py", line 37, in <module>
    import silx.gui.plot
  File "/home/valls/.local/lib/python2.7/site-packages/silx/gui/plot/__init__.py", line 66, in <module>
    from .PlotWindow import PlotWindow, Plot1D, Plot2D  # noqa
  File "/home/valls/.local/lib/python2.7/site-packages/silx/gui/plot/PlotWindow.py", line 41, in <module>
    from . import PlotActions
  File "/home/valls/.local/lib/python2.7/site-packages/silx/gui/plot/PlotActions.py", line 79, in <module>
    from silx.math.histogram import Histogramnd
  File "/home/valls/.local/lib/python2.7/site-packages/silx/math/__init__.py", line 31, in <module>
    from .medianfilter import medfilt, medfilt1d, medfilt2d
  File "/home/valls/.local/lib/python2.7/site-packages/silx/math/medianfilter/__init__.py", line 30, in <module>
    from .medianfilter import *
ImportError: dlopen: cannot load any more object with static TLS

If i use
pip install silx==0.5 --user --upgrade --no-binary :all:

it is fine.

Sounds old, @vallsv can this be closed?

I guess. I dont really remember the env or what.


Add two step updates for major version updates

RELEASE_NOTES=n/a
Signed-off-by: Dominik Schulz<EMAIL_ADDRESS>

Codecov Report

Merging #1378 into master will decrease coverage by 0.15%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #1378      +/-   ##
==========================================
- Coverage   57.96%   57.81%   -0.15%
==========================================
  Files         193      193
  Lines       10959    10987      +28
==========================================
  Hits         6352     6352
- Misses       3796     3823      +27
- Partials      811      812       +1

Impacted Files
Coverage Δ

internal/action/update.go
36.36% <0.00%> (-8.08%)
:arrow_down:

internal/updater/update.go
3.77% <0.00%> (-0.74%)
:arrow_down:

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 d0be405...4bec9b3. Read the comment docs.


New Feature: A granite UI (touch UI) widget for editing HTML

This is a simple granite UI widget, for Touch UI dialogs, used for editing HTML with code highlighting using CodeFlask.
Also adds two vendor clientlibs: Prism.js (only HTML language) and CodeFlask

Coverage remained the same at 31.042% when pulling ab21330c56304c22250077a8c1728ff5202ac16d on ahmed-musallam:feature/html-editor-widget into ec21baa52d6a63bc1fee39f74b6969f8ecd5d91c on Adobe-Consulting-Services:master.

TBH, this makes me a little nervous. While the widget in and of itself isn't a problem, it seems to encourage a pattern of allowing authors to hand-edit HTML which is known to create security risks, specifically XSS. @ahmed-musallam how do you forsee the XSS risk resulting from the usage of this widget being mitigated?

@justinedelson fair point, let me look into using sling's XSSFilter with an appropriate context to remove potential XSS  risks. Alternatively, I can look into a JS validation solution to warn against XSS risks.

generally, Adobe's best practice (and I think the industry best practice as well) is to filter on output, so it isn't really an issue for this widget per se. But can you verify that https://github.com/Adobe-Consulting-Services/acs-aem-commons/pull/1155/files#diff-f628067e555e0c13693083f40ba13facR52 doesn't require additional XSS filtering?

@justinedelson The JavaDoc for AttrBuilder  does mention that attributes are protected with XSSAPI.

A builder to generate HTML attributes. This builder is designed to be secured using XSSAPI. It will encode the value automatically. If the value is null, it will be ignored.

However, I think it may be using XSSAPI#encodeForHTMLAttr which will only encode the html and protect against script-running attributes like onload or onclick, but not necessarily against an HTML string containing a <script> I have tested it, and it does not filter out <script></script> for example.
I don't think I should filter XSS issues for the widget since authors should see exactly what they authored in the dialog and no XSS will happen on author side since the code protects against it: encodes HTML in the attribute and does not add the HTML to the dialog DOM as-is.
Like you said, XSS prevention should be done on output, i.e. in the component that is rendering the HTML string to the page: with HTL display context or in JSP with sling XSSFilter, which is what is done today for the rich text editor (even though it has frontend XSS protection).
Let me know your thoughts.

@ahmed-musallam I think as long as the widget doesn't display the rendered HTML, this will be OK. The problem isn't the text <script>alert("owed");</script>, it's what happens when the browser interprets as HTML :)
I want to think about it a bit more and make sure I'm not missing anything. Been a busy week but I'll try to get back to this by mid-next week.


install AND update vundles on rake update

Following on from https://github.com/skwp/dotfiles/issues/588#issuecomment-69034629, vundle has :BundelInstall!, that has been around since at least 2011 (the documentation was confusing before that, but I think it has always been there and done the same thing).

so you think this will cause vundle to selfupdate?

yes, because of https://github.com/skwp/dotfiles/blob/master/vim/vundles.vim#L18

But can we combine this with the other changes to Plugin? or people have to run this first, get new vundle, then we can switch it

I'm trying to figure out a way to have a clean upgrade path that doesn't fubar everyone like the reported issue

Sure, so I'm suggesting you leave the other vundle plugin change for now.  This will update vundle but not use the new syntax so yes, people would have to run this before the other vundle plugin change.
In the future when you add the vundle plugin change there could be a fall back of manually pulling the vundle repo if vundle is too old, not sure how to do that cleanly but seems like getting all plugins updated on update is a good thing independently.
Sorry if I'm confusing matters!


Error When running on noPlartform.

It does run on noHardware but I cant seem to access any functionalities.The noAPI cant deliver any data either

The README file precise
By running the following command you can deploy your application absolutely nowhere. You should try to run this command:


Is there a test start event, which our runner expects to show what test(s) are currently running.

Filing on behalf of @citizenmatt.
Is there is no test start event, which our runner expects to show what test(s) are currently running.

There are no such events per test. But user can rely on ActiveTests property in Test Run Statistics response payload to get the currently running/active tests.
Fixed the documentation for Test Run Statistics via https://github.com/Microsoft/vstest-docs/pull/22
Reference : https://github.com/Microsoft/vstest-docs/blob/f606f0c6f4ab05602889f95aebd02579fc972e49/RFCs/0007-Editors-API-Specification.md#test-run-statistics-response

Thanks for the update. That will suffice.


browser.target().page() returns null

Steps to reproduce
Tell us about your environment:

Puppeteer version: (puppeteer-core): 5.4.1
Platform / OS version: Windows 10 64bit / 19041.630 (build)
URLs (if applicable): https://kor.lottedfs.com/kr/search?comSearchWord=2066576109&comCollection=GOODS&comSort=RANK%2FDESC&comListCount=20&txtSearchClickCheck=Y
Node.js version: 14.15.0 (top level async is possible)

What steps will reproduce the problem?
Please include code that reproduces the issue.
I wanted to make a backbone demo, but I think it's easier for you to just run this code.
const sleep = (ms: number) => new Promise(resolve => setTimeout(resolve, ms));

const browser = await puppeteer.launch({
	executablePath: "C:/Program Files (x86)/Google/Chrome/Application/chrome.exe",
	headless: false
});
const page = await browser.newPage();

let pendingTask = false;
page.on("response", async () => {
	pendingTask = false;
});

await page.goto("https://kor.lottedfs.com/kr/search?comSearchWord=2066576109&comCollection=GOODS&comSort=RANK%2FDESC&comListCount=20&txtSearchClickCheck=Y", {
	waitUntil: "networkidle0"
});
const productEl = await waitForSelector(".productMd > a");
await productEl.click({
	button: "right"
});
const newTabButton = await waitForSelector(".custom-contextmenu a");
browser.once("targetchanged", () => {
	pendingTask = false;
});
pendingTask = true;
await newTabButton.click(); // this will create a new tab
while (pendingTask) {
	console.log("Waiting on the product page to be loaded...");
	await sleep(1000);
}

const productPage = await browser.target().page(); // returns null

What is the expected result?
await browser.target().page() expected to return current page
What happens instead?
It returns null

How about use below statement If you want to get new tabs page?
const productPage = await browser.pages().slice(-1)[0];


EnvoyFilter is failed to do http communication with sidecar

istio version 1.5.4
Problem Statement : Trying to develop EnvoyFilter for external authorization. but if http- prefix is not added on ports.name in service.yaml file then http_connection_manger filter is not incorporate in proxyconfig [istioctl proxy-config listeners hello-world-t2-548c8d58cb-hzlxj --port 15006 -o json -n nexus-sitb] .
hence any http communication from EnvoyFilter do not work.
Any workaround to insert http_connection_manger without using http-.
My Envoy Filter
############################################################
# Envoy External Authorization filter that will query OPA.
############################################################
apiVersion: networking.istio.io/v1alpha3
kind: EnvoyFilter
metadata:
  name: ext-authz
  namespace: nexus-sitb
spec:
  filters:
  - insertPosition:
      index: FIRST
    listenerMatch:
      listenerType: SIDECAR_INBOUND
      listenerProtocol: HTTP
    filterType: HTTP
    filterName: "envoy.ext_authz"
    filterConfig:
      with_request_body:
        max_request_bytes: 8192
        allow_partial_message: true
      failure_mode_allow: true
      grpc_service:
        # NOTE(tsandall): when this was tested with the envoy_grpc client the gRPC
        # server was receiving check requests over HTTP 1.1. The gRPC server in
        # OPA-Istio would immediately close the connection and log that a bogus
        # preamble was sent by the client (it expected HTTP 2). Switching to the
        # google_grpc client resolved this issue.
        google_grpc:
          target_uri: <IP_ADDRESS>:9191
          stat_prefix: "ext_authz"

My Service.yaml
---
apiVersion: v1
kind: Service
metadata:
  name: hello-world-t2
  namespace: nexus-sitb
  labels:
    app: hello-world-t2
    environment: dev
spec:
  ports:
  - port: 9898
    name: hello-world-t2
    protocol: TCP
    targetPort: 9898
  selector:
    app: hello-world-t2
    environment: dev
---

I don't understand the use case? If you want http then declare the port as http?

Any more info?


Navigation2D pathfinding issue on large navigation region

Godot version
v4.0.1.stable.mono.official [cacf49999]
System information
Windows 10
Issue description
Carving holes in the navigation polygon by adding outlines creates areas inaccessible to navigation agents in places where they should not be. This is usually one zone for each hole in one direction towards the edge of the navigation region.
https://user-images.githubusercontent.com/73339803/232428050-60fe0e35-4341-45a4-b890-2cbc787101a2.mp4
Steps to reproduce
You just need to add a small outline polygon to the navigation region with points about 50k pixels from the center of the scene (I gave this figure to reproduce the bug for sure, it also works with much smaller values) and then try to lead the agent around this outline, at least from one side he should encounter an inaccessible zone. This works only at a great distance from the edges of the navigation region, creating holes in the region at a relatively short distance from the edges does not give such an effect regardless of the size of the navigation region.
Minimal reproduction project
NavBug.zip
In this MRP, the problem area is approximately in the same place as in the video.

@ShoulinSS Please upload a minimal reproduction project to make this easier to troubleshoot.

I'm sorry I didn't do it earlier, now it's there.

As you may have noticed the path debug and the navigation polygon debug are correct so the pathfinding, NavigationRegion or navigation polygon / mesh can not be the source of the problem which leaves the NavigationAgent and your custom movement script.
That the navigation debug path updates nearly every frame is a sign that you reset the path every frame which can make the agent bounce between two positions and get stuck which is exactly what you are encountering here.
You may trigger this path update unintentional because the agent ends outside the set path_max_distance due to your movement code. That code is also the source of the issue here.

var vel = position.direction_to(agent.get_next_path_position()).normalized() * 100

You are using a relative to character position for calculating the direction to a global path position hence why everything breaks when you shift one of your character parent nodes or the main node at the world origin by some offset. Change position to global_position and it works again.
On another note, if you use a physics CharacterBody2D and velocity based physics movement please use _physics_process() in your movement script and not frame _process().
The NavigationAgent also updates its internals once each physics tick so if you do rapid movement updates every frame all the distance checkse still only happen once between each physics tick, e.g. distance checks to increase the path index, update paths or end the path following.

You are running into float precision issues due to your enormous scale and very long, thin edges.
You either need to build Godot with double precision or cut down your scale and polygon edges to a more reasonable size.
If you look at that red area that you marked that is exactly where a very thin triangle exists when your convex polygon face with 3+ vertices gets split into triangles to find the closest position. Due to the distance checks failing on precision this middle triangle loses the distance checks against the other two triangles that are larger and less affected by precision. With the middle triangle ignored the point is snapped to one of the other two remaining triangles.

Well, I assumed that it was float precision issue, but I thought that it shouldn't have worked like that specifically from a certain side, and not from all at once.
Thanks for the answer.

Actually, if you need a huge single navigation region, then it can be covered with small holes that will play the role of landmarks for the navigation agent.


[MM-44944] Give info to user in regard of what they are allowed to do in town square.

Summary
In this PR we try to improve UX by informing users what they are allowed to do when trying to click manage in Town Square. More info in the ticket but TLDR is: The new banner appears only when the user has user management permission, is NOT a channel admin, and is on Town Square.
The first commit contains the "feature" change, the second commit is a port of the channel role selector from js class to ts functions
Ticket Link
https://mattermost.atlassian.net/browse/MM-44944
Related Pull Requests

Release Note

NONE

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

Nice enhancement @JulienTant. I wonder if @cwarnermm would have an improved message recommendation?

In the default channel, the only action available is removing guests.

Nice enhancement @JulienTant. I wonder if @cwarnermm would have an improved message recommendation?

In the default channel, the only action available is removing guests.

Thanks and agreed for the message!

Thanks, @matthewbirtch!
@JulienTant -- what do you think about this message: "In this channel, you can only remove guests. Only channel admins can manage other members."
Would it be possible to include a link for the text "channel admins" that goes to https://docs.mattermost.com/welcome/about-user-roles.html#channel-admin?
Would this proposed UI text fit based on the smallest form factor you're developing for?

Thanks, @matthewbirtch! @JulienTant -- what do you think about this message: "In this channel, you can only remove guests. Only channel admins can manage other members."
Would it be possible to include a link for the text "channel admins" that goes to https://docs.mattermost.com/welcome/about-user-roles.html#channel-admin?
Would this proposed UI text fit based on the smallest form factor you're developing for?

Yep this is perfect, here's the result:


How to save form data between pages (forms)

Hi, this is not a bug report "per se" but more of a question. I have an order process where I have to save the form data between pages (form steps). What is the best way to do that in grav? Sesssions? js + cookies? hidden fields? thank you!

Frankly, the best way to approach this is to have a single form, and use JS/CSS to switch between sections of the form as if it was multiple steps.  We have done this before using Fieldsets to arrange the form into sections we can then work with. An example:
form:
    name: newbuild
    client_side_validation: true
    keep_alive: true
    clear_redirect_url: '/dashboard'

    fields:
        step_1:
            type: fieldset
            id: build_step_1
            classes: active
            legend: 'What do you drive?'
            fields:
                year:
                    type: selectize
                    label: 'Year'
                    data-options@: 'Grav\Plugin\CustomPlugin::getBuildsYears'
                    validate:
                        required: true
                        message: 'This field cannot be blank'
                make:
                    type: modeltrim_selectize
                    label: 'Make'
                    validate:
                        required: true
                        message: 'This field cannot be blank'
                model:
                    type: modeltrim_selectize
                    label: 'Model'
                    validate:
                        required: true
                        message: 'This field cannot be blank'
                trim:
                    type: modeltrim_selectize
                    label: 'Trim'
                    options:
                        '': Trim (optional)
        step_2:
            type: fieldset
            id: build_step_2
            classes: partial-requirement
            legend: 'What mods have you done?'
            fields:
                engine:
                    type: tags
                    label: Engine
                    placeholder: 'ex: Superchips Programmer'
                    validate:
                        type: commalist
                intake:
                    type: tags
                    label: Intake
                    placeholder: 'ex: Airaid Cold Air intake'
                    validate:
                        type: commalist

There's also the Sequential Form plugin which I have never tried.

thanks guys...i ended up using storejs but would be interested to find out the grav way. each page is refreshed so storing data in session would be awesone..


Firefox doesn't see host pulseaudio, audio doesn't work

Issue description
When running Firefox 58.0.1 (/nix/store/6sblfm52hkvzr2b4fp8y5bn8fvj2j6ag-firefox-58.0.1/bin/firefox) on a non-nixos Linux system (Debian stretch in this case), audio doesn't work.  I can see the following messages on the console after running firefox and trying to play, for example, a youtube video:
[Child 14641, MediaPlayback #2] WARNING: 7fe5a381fd00 OpenCubeb() failed to init cubeb: file /build/firefox-58.0.1/dom/media/AudioStream.cpp, line 387
[Child 14641, MediaPlayback #2] WARNING: Decoder=7fe5aa798c00 [OnMediaSinkAudioError]: file /build/firefox-58.0.1/dom/media/MediaDecoderStateMachine.cpp, line 3642

Doing some searching, it appears that this error appears for people who aren't running pulseaudio, but I am running pulse via start-pulseaudio-x11 in my ~/.xinitrc (this is Debian's pulseaudio, version 10.0).
Steps to reproduce

Start non-nix-installed pulseaudio via start-pulseaudio-x11
Install firefox via nix-env -i firefox and run it
Try to play audio in firefox

Technical details
 - system: `"x86_64-linux"`
 - host os: `Linux 4.9.0-5-amd64, Debian GNU/Linux, 9 (stretch)`
 - multi-user?: `no`
 - sandbox: `no`
 - version: `nix-env (Nix) 1.11.16`
 - channels(jad): `"nixpkgs-18.03pre127228.91eb41b1383"`
 - nixpkgs: `/home/jad/.nix-defexpr/channels/nixpkgs`

Did this previously work for you?
I ask because some googling of my own turns up results like:

https://www.reddit.com/r/archlinux/comments/7tdtis/firefox_flash_no_audio/
https://bbs.archlinux.org/viewtopic.php?id=233904
and particularly:
https://bugzilla.mozilla.org/show_bug.cgi?id=1430274
which suggests this is a recent bug in that alsa no longer works.

(I know it's about pulseaudio but if the fallback stopped working perhaps that's related)
Anyway looking at the store path you posted (YAY nix!), it looks like the LD_LIBRARY_PATH set there (it's a wrapper script) does not contain libpulseaudio, suggesting the wrapper wasn't generated with pulseaudio support.
(which apparently is controlled by config.pulseaudio:)
https://github.com/NixOS/nixpkgs/blob/master/pkgs/applications/networking/browsers/firefox/wrapper.nix#L74
So perhaps this problem can be reduced to "how do I set config options on non-NixOS?".
If so, the answer somewhat depends on how you're doing things but probably setting "pulseaudio = true;" in your ~/.config/nixpkgs/config.nix would do the trick.  If you don't have this file already, try something like:
$ mkdir -p ~/.config/nixpkgs
$ echo "{ pulseaudio = true; }" > ~/.config/nixpkgs/config.nix

and see if that fixes things (after re-installing/updating firefox)?

Adding { pulseaudio = true; } to ~/.config/nixpkgs/config.nix worked!  Thank you!


Wallpaper Image Resolution too low

_wallpaper images are loaded directly related to the display resolution of the device, however the pictures don't look good on some displays with very high pixel density (i.e. new iphone).
Calculation for size should be changed in javascript

fixed somewhere along the way but forgot which commit


Issued certificate has expired

postal auto-upgrade
Upgrading from the stable channel
Backing up previous application files
Downloading latest version of application
--2018-08-13 13:03:08--  https://postal.atech.media/packages/stable/latest.tgz
Resolving postal.atech.media (postal.atech.media)... <IP_ADDRESS>, 2a00:67a0:a:1::1b
Connecting to postal.atech.media (postal.atech.media)|<IP_ADDRESS>|:443... connected.
ERROR: cannot verify postal.atech.media's certificate, issued by ‘CN=Let's Encrypt Authority X3,O=Let's Encrypt,C=US’:
  Issued certificate has expired.
To connect to postal.atech.media insecurely, use `--no-check-certificate'.

gzip: stdin: unexpected end of file
tar: Child returned status 1
tar: Error is not recoverable: exiting now
Failed to run: wget https://postal.atech.media/packages/stable/latest.tgz -O - | tar zxpv -C /opt/postal/app

Looks like this has been fixed now


Failed to load resource: the server responded with a status of 404 (Not Found)

Project no longer works per instructions.


Rule enhancement: Don't do unnecessary destructuring

What the rule should do:
What problems does it solve:
Example of things the rule would report:
view (Model model) =
   div []
    [ viewA (Model model)
    , viewB (Model model)
    ]
-->
view model =
   div []
    [ viewA model
    , viewB model
    ]

Example of things the rule would not report:
view (Model model) =
   div []
    [ viewA (Model model)
    , viewB model -- using the extracted model
    , viewC model.x -- also using the extracted model
    ]

Should this be part of the Simplify rule or should it be a new rule?

I think it could be part of the Simplify rule.
I am looking for:

Someone to work on this :)

I'd use it if it existed, but...
In practice, rules handling patterns in this form like below are pretty useless because

the rule can't provide a good default name for the undestructured

→ doing the refactor is dumb work but still takes effort

revisit in the future when fixes can take user input? 👀

→ deciding on a name isn't always easy and needs context
→ the rule might be seen as annoying

adding the variant name isn't hugely verbose
variant names etc. add some descriptive names
in code that's changing a lot, you might still want to quickly be able to access the value(s)

→ the rule might be seen as annoying

false negatives for records without types: "How many fields does the original record have?"\{ field } -> f { field = field }

type Id = IdUnsafe Int

toString2 combine (IdUnsafe aInt) (IdUnsafe bInt) =
    combine (IdUnsafe aInt |> toString) (IdUnsafe bInt |> toString)

orElse alternative maybe =
    case maybe of
        Nothing ->
            alternative
        Just content ->
            Just content

tupleCompare ( first, second ) =
    compair ( first, second ) Descending

attempts at making names
toString2 combine idHoldingAInt?? idHoldingBInt?? =
    combine (Id aInt |> toString) (Id bInt |> toString)

orElse alternative maybe =
    case maybe of
        Nothing ->
            alternative
        maybeHoldingContent?? ->
            maybeHoldingContent??

\firstSecond?? -> compair firstSecond?? Descending

Where you can already feel that generic names like idHoldingAInt don't cut down on verbosity while being awkward to read.
My subjective conclusion is that the small readability benefits are not worth at the moment the work of creating this rule

I think my proposal wasn't very clear, or not precise enough. I should have added a more explicit description.
The idea of the change is only to remove the destructuring of single-record variants if the unwrapped value gets wrapped again everywhere that it is used. I also only envisioned this only The example I gave is basically the only one I wanted to cover.
Taking back my original example
view (Model model) = -- Model is unwrapped into model
   div []
    [ viewA (Model model)  -- model in wrapped again
    , viewB (Model model) -- model in wrapped again
    ]
--> we can simplify it to this:
view model =
   div []
    [ viewA model
    , viewB model
    ]

If we had an alias to the argument, then we could use that name instead
view ((Model m) as model) =
   div []
    [ viewA (Model m)
    , viewB (Model m)
    ]
-->
view model =
   div []
    [ viewA model
    , viewB model
    ]

I agree that it could be interesting to do this for tuples, but as you say, that requires giving it a name.
That said, maybe the name could end up being bad in the example I gave :thinking:

Tho your example could (?) end up being simplified by NoUnusedPatterns
view ((Model m) as model) =
   div []
    [ viewA (Model m)
    , viewB (Model m)
    ]
-->
view ((Model m) as _) =
   div []
    [ viewA (Model m)
    , viewB (Model m)
    ]
-->
view (Model m) =
   div []
    [ viewA (Model m)
    , viewB (Model m)
    ]

But only suggesting a fix when you have an alias makes sense. Maybe we can extend it to
view model =
  let
    (Model m) = model
  in
  div []
    [ viewA (Model m)
    , viewB (Model m)
    ]
-->
view model =
   div []
    [ viewA model
    , viewB model
    ]

Might be a little harder to implement.
What I don't yet understand: Do we want to always report an error to folks, saying they should add an alias if none is present? That doesn't really seem like a good solution

folks have to explicitly navigate to the reported section, add the alias
when they want the same name, like (Model model) as model, could it trip up some logic? At the very least this won't compile (which would go against your idea of compiler first, then elm-review)

Funny edge case: a type with phantom parameters should not be reported by this check.
E.g. this code I found in elm-review is no a no-op:
|> (\(ModuleRuleSchema moduleVisitorSchema) -> ModuleRuleSchema moduleVisitorSchema)

or
List.foldl
    (\addVisitors (ModuleRuleSchema moduleVisitorSchema) ->
        addVisitors (ModuleRuleSchema moduleVisitorSchema)
    )
    ...

That's right :+1:
This means we have to make sure that either:

The type does not have type variables
The type's type variable in this context is unbound, which I think is do-able in the context of a argument function declaration


Fix spline bodies (please)

The example provided contains basic b-spline routines that should work on a GPU but cause problems when used inside ParametricBodies. Having this functionality would be a major boost as it would allow creating complex shapes and kinematics.

#4 Implements NURBS splines on GPUs!


Show just the NDS for clarity, gradient doesnt show infractions at a glance

This was done manually

GF wants to focus on Aldeia
Would be interesting to spit out a report of all the survey numbers with NDS
Then hyperlinked back to amche to view in context with the layers like RDP, elevation tilt

Show just the NDS for clarity

Technically possible but might need several iterations to get a usable result. The nature of the source data (CartoDEM) is a raster and to convert it into polygons will require us to make some arbitrary decisions on boundary conditions of the slope. So we always need to interpret any final result with a fuzzy margin of error and not an exact slope zone boundary.
In any case it may be worth doing a first pass an collecting feedback. Will also document the process for more transparency as it is important to get multiple eyes to fine tune the slope outputs.


Docs.MS Release Updates for azure-spring-boot-starter

Update docs metadata and targeting for release of azure-spring-boot-starter

Docs Build status updates of commit ce52ac6:
:clock10: Full build: the average full build time is 86 min(s) 18 sec(s), this is based on the last 724 build(s) for this repository.

Close this PR since all the commits have been moved https://github.com/Azure/azure-docs-sdk-java/pull/1225/.


enable grpc health service

22ef6cb99f60918add8b4be3ba6f40f7a54b50f0 enabled compatibility with Docker 1.13 but didn't include the mandatory healthcheck service. Therefore the healthcheck fails. Running dockerd in debug mode leads to repeating messages as follows:
DEBU[2218] libcontainerd: containerd health check returned error: rpc error: code = 12 desc = unknown service grpc.health.v1.Health

This PR enables grpc health service in runv containerd.
Signed-off-by: Peter Morjan<EMAIL_ADDRESS>

LGTM


Shortcut Guide overlay was permanent - had to close down Powertoys entirely to get rid of it

I accidentally opened the overlay for the Shortcut Guide and somehow it stayed open even after I released the windows key and had no way of closing it, nor was there any indication of what it was while it was open.  Please add some sort of name to the top of the overlay, preferably add a taskbar entry that can be right-click closed and ideally have the same key that opens it, close it in cased this happens again.

@raysmith59 trying to figure out what could have caused this.
was PT being run as admin?

what apps were in focus, i know it was a long time ago but lack of ability to repro would make this hard to diagnosis

I'm going to close this off, it happened a few times back then but not lately so one of the recent updates likely fixed the issue.

Hi,
I've got some repro steps, this is happening for me with 0.35:

Press and hold the Windows key to show the Shortcut Guide overlay.
Press the ',' to Peek the Windows Desktop.
Keep the Windows key held-down. The Desktop is shown permanently rather than peeked as long as the key is held down.
Release the Windows key. The overlay is redisplayed permanently. I had to sign out and back in to remove it.

This happened to me right now. I run one AutoHotKey script that moves my windows to primary monitor (because Windows just for some reason puts my windows to secondary monitor for no reason when I lock out my acount and then login again), anyway after pressing the activating shortcut (ALT+CTRL+F) the script moved all my windows to primary monitor, but for some reason The Windows Shortcut Guide appeared on my primary monitor and I can't get rid of it. It is just there, if I press the winwows button I see start men appear below the overlay. If I press the ALT+TAB I see windows changing below the overlay. But I can't get rid of it. Please send help.


Ripple effects of list items are stacking on scrolling in mobile

actually it is duplicate of https://github.com/angular/material2/issues/10082
but I don't know how to "ReOpen" that ticket. So...
Bug, feature request, or proposal:
Bug
What is the expected behavior?
Ripple effects of list items shouldn't stack on scrolling in mobile. Ideally, effect even shouldn't be visible when you scroll list. It should be visible only on "tap" (like menu in Google Chrome).
What is the current behavior?
Please see video: https://youtu.be/O_H2MG8Torw
What are the steps to reproduce?
Is reproducible on https://material.angular.io/
Steps to reproduce:

Go to https://material.angular.io/guides ON MOBILE DEVICE using Chrome
Click on any link (for example "Getting started")
Click Back button on device
Try to scroll list quickly
You'll see "stack" of ripples

What is the use-case or motivation for changing an existing behavior?
It doesn't look good. In Some scenarios it looks horrible :(
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
My site uses<EMAIL_ADDRESS>and<EMAIL_ADDRESS>but behavior same as on https://material.angular.io/

Go to  https://material.angular.io/components/ripple/examples ON MOBILE DEVICE using Chrome
try to scroll you can see this.
can you help fixing this.

+1

Any update on this? I wanted to use Angular Material for my project, created a simple PoC and hit this ugly issue...


⚠ Bofrost.de has degraded performance

In c205182, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1135 ms

Resolved: Bofrost.de performance has improved in 8ad6b07.


Remove PIDSTORE_RECID_FIELD config

When PUT or PATCH records, Invenio uses loaders to validate the request payload. Currently, loaders are inheriting from RecordMetadataSchemaJSONV1. This class, defined in invenio-records-rest, needs some refactoring because it hardcodes a global config PIDSTORE_RECID_FIELD.
What it does is:

remove any extra field that is sent in the payload and is not defined in the schema class
automatically read the pid param from the url, for example when doing PUT /api/documents/1234
inject a new field in the schema, defined by the config PIDSTORE_RECID_FIELD, with value 1234

This means that if PIDSTORE_RECID_FIELD is defined as item_pid, it will add the document schema item_pid and not document_pid!!!
To do:

refactor invenio-records-rest to remove this field or to have as default value to avoid breaking APIs, in case some Invenio solutions call the PID field always with the same field name for all records.
allow this field to be overridable. Instead of reading this PID field name by config, first check if it is configured as class attribute, or read it from a getter/abstract method

Addressed https://github.com/inveniosoftware/invenio-records-rest/pull/252


cni/install: Handle fs permission issues during binary copy

Checking directory permissions by writing a file can have race conditions or side-effects depending on the underlying filesystem. This patch avoids those types of issues entirely by simply handling permissions errors at the time the operations are performed.
Also, the Exists() function did not specifically check for file existence and could give incorrect results if there are permissions issues. We now explicitly check for permissions errors and report the result.

[ ] Ambient
[x] Configuration Infrastructure
[ ] Docs
[ ] Dual Stack
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@tonya11en: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@tonya11en: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test release-notes

unit-tests-arm64_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@tonya11en: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test release-notes

unit-tests-arm64_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test unit-tests-arm64

unit-tests_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@tonya11en: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test release-notes

unit-tests-arm64_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test unit-tests-arm64

unit-tests_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test unit-tests

integ-ambient_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@tonya11en: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test release-notes

unit-tests-arm64_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test unit-tests-arm64

unit-tests_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test unit-tests

integ-ambient_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test integ-ambient

integ-operator-controller_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test integ-operator-controller

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@tonya11en: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test release-notes

unit-tests-arm64_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test unit-tests-arm64

unit-tests_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test unit-tests

integ-ambient_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test integ-ambient

integ-operator-controller_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test integ-operator-controller

integ-cni_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test integ-cni

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@tonya11en: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test unit-tests

integ-ambient_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test integ-ambient

integ-operator-controller_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test integ-operator-controller

integ-cni_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test integ-cni

unit-tests-arm64_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@tonya11en: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ambient_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test integ-ambient

integ-operator-controller_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test integ-operator-controller

integ-cni_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test integ-cni

unit-tests-arm64_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test unit-tests-arm64

unit-tests_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@tonya11en: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-operator-controller_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test integ-operator-controller

integ-cni_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test integ-cni

unit-tests-arm64_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test unit-tests-arm64

unit-tests_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test unit-tests

integ-ambient_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@tonya11en: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-cni_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test integ-cni

unit-tests-arm64_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test unit-tests-arm64

unit-tests_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test unit-tests

integ-ambient_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test integ-ambient

integ-operator-controller_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test integ-operator-controller

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@tonya11en: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test unit-tests-arm64

unit-tests_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test unit-tests

integ-ambient_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test integ-ambient

integ-operator-controller_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test integ-operator-controller

integ-cni_istio
a32ca39df2a245139bd519ab7d6b02af96777175
link
true
/test integ-cni

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@tonya11en: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-cni_istio
4d5536a4ae0a6a113c42d75d0127671095e37504
link
true
/test integ-cni

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Can't copy messages/code blocks on mobile

Decided to hide the button on mobile because we can't run this function for security reasons, that being said I think they are workarounds but we can come back to this issue later:
	function copyContent() {
		navigator.clipboard.writeText(content);
	}

:tada: This issue has been resolved in version 0.3.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add helper to simplify life with LIKE queries

Alternative to #5506

Looks good to me, needs a better name

Looks good to me, needs a better name

I like Liker :wink:
Any better suggestion?

Looks good to me, needs a better name

What about Concrete\Core\Database\Query\Lover ? 🤣

What about LikeQueryBuilder or LikeBuilder?

What about LikeQueryBuilder or LikeBuilder?

Done (LikeBuilder)

Tyvm, nice tests!

One thing that just crossed my mind, should we make use of the laravel container ->when method? I think this would work the same:
$app->when(LikeBuilder::class)
    ->needs('$otherWildcards')
    ->give(function() {
        rerturn [ 'other' => 'wildcards' ];
     });

should we make use of the laravel container ->when method?

I never user it... pull requests are welcome :wink:

I never user it

That's because 8.0 was the first time we could :) But it's meant for exactly what you're trying to do here

And what's the benefit of that syntax?

One thing that just crossed my mind, should we make use of the laravel container ->when method?

@KorvinSzanto I just switched to the syntax you suggested, and it's working great :wink:


Issue: My bow doesn't change it's texture

I have been following the tutorial series for a while now and have made a bow for my mod, but when the bow is pulled, the texture does not change.
Notable things to mention are:

No warnings are present regarding the textures.

The model file and texture file names match.

I have looked back on the code in this repository multiple times and my code is how it should be (In reference to the ModItemProperties, ModClientEvents and model files).

No-one else seems to have experienced this issue.

============================================

Bow not yet drawn.

Bow at full draw.

============================================
I am not entirely sure as to whether here is the correct place to post this issue, but I thought it would be best to try here out of ease.

Although I am not the one and only Kaupenjoe, I am happy to help if you want. If you do want me to help, could you send me a zip file of your src folder? I can't really understand until I can see your code.

Okay, thank you. Here is the src folder:
src.zip


Fixed broken link in README.md

Description
Fixed the broken Zero Trust Imaging link.
The issue was due to a typo: 'Imaging', instead of 'imaging'
Issue reference
The issue this PR will close: N/A
Checklist
Please make sure you've completed the relevant tasks for this PR out of the following list:

[x] All acceptance criteria in the backlog item are met
[x] The documentation is updated to cover any new or changed features
[x] Manual tests have passed
[x] Relevant issues are linked to this PR

Thank you for the fix!


Help command to open service

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem?
Recently, I created a custom command to launch the xhgui service in a browser.
This is currently quite complex;

checks if HTTP / HTTPS
checks for Gitpod or Codespaces
checks if DEBUG is enabled
case for linux
case for darwin
case for window

It would be convient to have a centralized helper function that custom command developers can call.
This would allow all open-in-browser commands to be updated if, say, Codespaces is updated.
Describe your solution
I would like DDEV to provide a helper function to create a new command.
For example:
#!/bin/bash shell

## #ddev-generated
## Description: Launch a browser to XHGui
## Usage: xhgui
## Example: "xhgui"

HTTP_PORT = 8142
HTTPS_PORT = 8143

launch_service(HTTP_PORT, HTTPS_PORT )

Describe alternatives
No response
Additional context
Several modules, including the launch command would benefit from this:
Eg.

ddev-xhgui
ddev-adminer

Maybe launch and phpstorm and adminer commands could be rolled into this...

Is this fixed with

https://github.com/ddev/ddev/pull/6024
?

Yes, that resolves this issue.  Glad we got this feature in.
Test

Install ddev-adminer

ddev get ddev/adminer

Update global adminier command as below.

DDEV_ADMINER_PORT=9100
DDEV_ADMINER_HTTPS_PORT=9101

if [ ${DDEV_PRIMARY_URL%://*} = "https" ]; then
    ddev launch :$DDEV_ADMINER_HTTPS_PORT
else
    ddev launch :$DDEV_ADMINER_PORT
fi

Restart DDEV

ddev restart

Confirm update command generates correct URL

$ DDEV_DEBUG=TRUE ddev adminer
FULLURL https://30days-l11-demo.ddev.site:9101

Confirm it opens browser to Adminer.

ddev adminer

DDEV: v1.23.0-rc1
OS: WSL2 Ubuntu on Win10.
UPDATE:
Look like I just confirmed what @stasadev  said here

Thanks.
I assume you'd like to be able to update existing commands to use this feature though also, right?

Yes, as I come across addons that benefit, I'll make PRs as nessasary.


"New" label is incorrectly invalidated

It seems as if under certain circumstances an infoItem is not invalidated properly and its "New"-label will be displayed in the topic history of the meeting series.
Expected Behavior:
I would expect the "New"-label to show, only if the infoitem was added in the last minute. But that is not the case.
Steps to reproduce:

Add new minute
Add Topic with an infoitem
Finalize minute
Add new minute
Add another infoitem to the same topic
Finalize minute

Now you can add and finalize as many minutes as you like without the "New"-label ever disappearing in the history.
Additional info: I havent found a way for the infoitems to get invalidated correctly after the incident. Moreover every infoItem that is added to the topic will show the same behavior and keep its "New"-label in the history.

I think, Ich found the root cause of this bug: In line https://github.com/4minitz/4minitz/blob/4622a76aa1c7d6229b88db68a3b05293d6edb074/imports/services/finalize-minutes/meetingSeriesTopicsUpdater.js#L12 we should use the Method TopicsFinder.allTopicsOfMeetingSeries instead.


adjust mass bug fix

calling adjust mass desyncs the debug graphics from the real positions of bodies

Oops wrong branch


Branch 1.4

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

Can one of the admins verify this patch?

Close this PR @GuoNing89

@GuoNing89  please close this PR.

@srowen @vanzin - Shouldn't some automated process be picking up your comments ("close this PR") and closing this PR? I thought we had something like that.

@nchammas Yeah the script or process for that disappeared .. but all it would do is push an empty commit with "closes xxx" in its message. I can do that manually without much work. One moment ...


Host agent changes

What this PR does / why we need it:
The host agent changes for executing install and uninstall script when --useinstallercontroller flag is set when running the host-agent binary.
Which issue(s) this PR fixes:
Fixes #563

Codecov Report

Merging #564 (a1ea7a4) into main (b09a31d) will increase coverage by 0.37%.
The diff coverage is 82.05%.

:exclamation: Current head a1ea7a4 differs from pull request most recent head 47bf7e8. Consider uploading reports for the commit 47bf7e8 to get more accurate results

@@            Coverage Diff             @@
##             main     #564      +/-   ##
==========================================
+ Coverage   67.22%   67.59%   +0.37%
==========================================
  Files          27       27
  Lines        2346     2416      +70
==========================================
+ Hits         1577     1633      +56
- Misses        665      675      +10
- Partials      104      108       +4

Impacted Files
Coverage Δ

agent/main.go
19.44% <0.00%> (-0.11%)
:arrow_down:

agent/reconciler/host_reconciler.go
81.41% <83.11%> (-0.08%)
:arrow_down:

@anusha94 Please review and approve the PR


How do we change the styling / color of the seeekbar, etc?

You can not set the styling of the player seek bar.
Some props are provided for implementing a custom controller (disableDefaultControl, onProgress, etc).

I know this is a long shot, but is there any way you could help me style the BrightCove player directly?
That would be the difference between one or two lines of code and essentially recreating a control structure for the player.
I could pay you for your time.

OK. I'll try to find the solution when I have time.

Thank you so much.
I need to be able to color the seekbar and change which buttons are present in the controls (toggle closed captions, RR and FF, and show only live appropriate controls)

@manse is there any way we could work together on this today or tomorrow?

@technoplato
Sorry but I can not. Please make pull request yourself, or wait until releasing fixed version.


Add automatic periodic check of URLs in jobs

@notactuallypagemcconnell yeah, sure, the only problem I don't like in send_after approach is that I have no idea how to test the recurring part.

@brain-geek yes I will push up a commit with an example sometime today

@brain-geek heres an example, I just will paste here since this branch is on your fork and I'm lazy.
# lib/companies/periodic.ex
defmodule Companies.Periodic do
  use GenServer
  require Logger

  def start_link({_mod, _func, _argument_list, _interval} = state) do
    GenServer.start_link(__MODULE__, state)
  end

  @impl true
  def init({_mod, _func, _argument_list, interval} = state) do
    schedule_work(interval)

    {:ok, state}
  end

  @impl true
  def handle_info(:work, {mod, func, argument_list, interval} = state) do
    apply(mod, func, argument_list)
    schedule_work(interval)

    {:noreply, state}
  end

  defp schedule_work(interval) do
    Process.send_after(self(), :work, interval)
  end
end

Code to run off {mod, func, args, interval} where func is applied to mod using args in interval
# test/companies/periodic_test.exs
defmodule Companies.PeriodicTest do
  use ExUnit.Case

  import ExUnit.CaptureIO

  require Logger

  test "it can run a given module's given function with the given arguments in the given interval" do
    assert capture_io(fn ->
             {:ok, _pid} = Companies.Periodic.start_link({IO, :puts, ["hi"], 1})
             :timer.sleep(5)
           end) =~ "hi\nhi"
  end
end

This test proves that it runs the task twice and has minimal overhead (3 milliseconds spent sleeping) but proves the premise is sound and tests it at an OTP level. I think it works well enough. We use some code similar to this (but with some more sophistication added) at work.

@notactuallypagemcconnell Thanks for the snippet, that's a clever way to test it! Applied.

@doomspork maybe, but then we need to mark this automatic user somehow.

Recently I found a blog post that may help with the implementation:

https://blog.smartlogic.io/genserver-nightly-task/
https://github.com/oestrich/grapevine/pull/78/files

@gemantzu @brain-geek

@lccezinha Thanks for the suggestion, reworked it according to your suggestion.

@doomspork Could you check out the new version? It's now creates pending changes request instead of deleting something, so it should be more reliable. 😄


Subparser Demo with common arguments

Hello, i'm trying to do an simple interface with a feature but i don't know if it's possible. I have three actions in the left column with common arguments, how can i design it?
I can write the arguments with every action but the code is redundant. I'd like to have a fixed block with the common args and below, the specific args for every action.
I see the action names are lowercase, is there some way to set it to uppercase? Thank you.

I think you've already figured out the first part. A 'fixed block of code' sounds like a function to me.
Something like this perhaps?
def main():
  parser = GooeyParser(description='example')
  subs = parser.add_subparsers(help='commands', dest='command')

  subparser_1 = subs.add_parser('option1')
  apply_my_common_args(subparser1)

  subparser_2 = subs.add_parser('option2')
  apply_my_common_args(subparser2)

  subparser_3 = subs.add_parser('option3')
  apply_my_common_args(subparser3)

def apply_my_common_args(subparser):
    subparser.add_argument('--foo', help='something1')
    subparser.add_argument('--bar', help='something2')
    subparser.add_argument('--baz', help='something3')

You can control how the fields are displayed by supplying a metavar argument when adding arguments to the parser. However! You'll need to use the GooeyParser in order for it to work.

from gooey import GooeyParser

def main():
    my_cool_parser = GooeyParser(description='Example')
    my_cool_parser.add_argument("FileChooser", help=file_help_msg, metavar="My Custom Display Name")

I'm going to go ahead and close this since it's not really an issue with Gooey itself. But feel free to still comment here if you've got more questions.

It works! Thanks. Is there some way to show an equivalent "metavar" for add_parser?
another_parser = subs.add_parser('My_option', help='My Option')

Maybe showing the attribute help instead the name but i don't know how to do it. I read the documentation of argparse without luck.

@jslirola, good catch! There currently is not a way to do that.
I did some quick poking around and it looks like I can piggy back on argparse's prog option as a means of customizing the display.

However, while displaying it in the GUI is simple, getting it to actually work will require changing some of the Gooey internals so it keeps track of both the subparser and custom display name.
Do you want to raise this as a new issue?
I'll roll the functionality into the next commit (which may take a bit due to work schedules and holidays..).

Done. I don't want bother you. If you think is not very difficult to customize i'll wait your commit.
Happy holidays.


UI randomly goes blank

When looking at machine UIs, they randomly go empty and then you have to escape and right click back into it to see the items again. This happens quite frequently.
Version: Built from latest 1.10 branch as of Jan 29 2017 (7e3c806a8f136526129967ba1b25946d8f05ccd9).

Do you have a screenshot of it happening? That would help a lot with trying to solve it.

Sure. I believe this happens with all the machines, but take the metallurgic infuser for example. Best if you don't add speed upgrades. I put in 4 compressed redstone:

Then I put in 32 iron ingots:

If you just sit there and watch it, the UI will randomly blank the items, the redstone and RF meters remain filled and the arrow does continue the filling animation:

Once that one item finishes processing and the next one begins, the items reappear (until they randomly disappear again) -- I guess because the UI updates the numbers of items in the slots.
And then once all the processing finishes, you can just stare at the completed items and they will eventually disappear (and then you have to escape and re-enter the UI to see them and pull them out). This is a screenshot of after completion, there are 32 enriched alloys in the machine and I just waited (did nothing) for about 10-15 seconds after the crafting completed and they disappeared:

ESC and then right-click the machine again:

Are you using Optifine by chance?

I just disabled optifine and confirmed that the bug still happens without optifine. On average, every two or three ingots, the UI goes blank til the next ingot refreshes it.
However, there is something more to reproducing it and I'm not sure what that is yet... I teleported to the end and set up just a metallurgic infuser and ultimate energy cell, and tried the above and the UI did not go blank at all. I switched to my single player test world and I have tried setting up various machines that are in my base and have not been able to reproduce the UI bug. But every time I try it in my base in the multiplayer server, it happens within just a couple ingots.
I can do some more intensive trial and error later this week. Maybe download the map from the server, delete chunks via MCedit and see if I can narrow down how to reproduce this issue. Any other ideas?
Could this be at all related to the #3977 "Machines rotating by themselves"? I have also been unable to reproduce that in my single player world so far, and it also happens ALL THE TIME in my base on the server. Also happens to the other player on my server, whose base is over 1000 blocks away from mine; and he also sees this UI bug.

Root cause is the same as #3977, as I suspected. It's fixed with my pull request. Closing this issue.


Catch-22 on initial RF generation

Hi TD,
I've noticed there's a catch-22 on getting started with RF generation. Both the Basic Solar Panel and the Uranium RTG require crushed materials for crafting, however the Manufactory needed to crush them needs RF. Unless I'm missing something, it's necessary to have an RF provider from another mod in order to get started with NC.
Suggestion 1: Add a quartz glass recipe oreDict'd with AE2's quartz glass and use three pieces of that as the top layer for the solar panels, instead of crushed quartz and crushed graphite.
Suggestion 2: Let Graphite be created by smelting Coal in a regular furnace (as is done in BigReactors) and smelt the Graphite Bars into Crushed Graphite in a regular furnace.
Cheers!

I see the issue - how about adding an extra inefficient recipe for the solar panel that uses quartz blocks instead of crushed quartz?

I see the issue - how about an extra inefficient Uranium RTG recipe using 6 lead ingots, 2 coal blocks and a block of uranium?

Make it work like a single-use battery (i.e. has a non-refillable RF buffer) and I think that would work.
What about a potato battery? :P I don't think I've seen a mod do potato batteries before, Would be funny, using potatoes to start with nuclear power.

I'd rather not have to add pre-charged battery blocks or implement RF-holding items as they're both a massive pain.

how about an extra inefficient Uranium RTG

Not a fan. The Uranium RTG already produces a mere 4RF/t.  If xenoflot's (my, logged in with wrong acct) suggestions don't grab you, perhaps you could just add a coal-burning generator?

The thing is that most people play this mod in big packs which almost certainly already have other mods which add coal generators - I'd like NC's initial power gen to be a little more unique than that, or there's basically no point spending my time adding it.
An idea I had a while back was a very cheap heat converter that used the heat generated by radioactive decay in adjacent blocks to generate RF - if they generated, say, 4 RF/t per adjacent uranium/thorium block, placing a few of these down could certainly get you started.

My point re. single/temporary use is to discourage people from just spamming the (what is essentially an all-day solar panel) simple RTG block for mass power gen. If you're averse to implementing some form of RF-storage, what about just making the block expire after a certain number of ticks? Kind of like a half-life I guess?
The heat converter is pretty good too, since it makes it no longer be a one-block infinite power provider. Also could make it randomly consume the heat-producing adjacent blocks to further discourage people from staying at that (earliest) tier.

I guess I could have it delpete the uranium/thorium blocks after a certain amount of time...

Ok, I think this is sorted - the generator uses the heat of adjacent thorium and uranium blocks to generate energy - thorium and uranium blocks generate 4 RF/t each, while the depleted thorium and uranium blocks generate 1 RF/t. After an average of ~30 mins, thorium and uranium blocks will decay and deplete.
Hopefully this works as a nice way to start generating power in NC :)

Love it! Thanks!

Cheers :)
The depleted blocks have been slightly nerfed in v2.7 to produce 0.5 RF/t so that Uranium RTGs aren't made redundant.

Bah! /goes back to edit the wiki page he just wrote :D

Uh.. sorry... :P


Fixed a typo in README.md, and it now uses _name to set the mappings name

This allows you to call :official_amis more than once so you can have multiple AMI mappings set-up.

Thanks for catching that typo! I'm going to need to think through the name thing a little bit. On its face, it makes sense, but I want to make sure that it doesn't conflict with any expected behavior...