Add mongoose

Description

Adds mongoose from nest example
Untested, undocumented
Has utility docker compose for just bringing up db, this will probably be moved/changed by whoever is doing docker things

Fixes/resolves #10
Type of change

[ ] New feature (non-breaking change which adds functionality)
[ ] Improvement (non-breaking change which improves existing functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

please don't add PRs to the project :)

I know this is a draft PR, but assuming its close-ish to done:
The database stuff shouldn't be a controller/resource, just a service (nest g service ). That service is then used by other controllers when they need database access.
The design would then have every schema have its own service for access. I.e. if we have users and flats, then there would be a schema and service for users, a schema and service for flats.
Then a controller that wants access to the user store would import the user db service, and a controller that wants access to the flat store would import the flat db service.

Thanks, didn't know how it should have been done because haven't setup a nest project before, only worked on some basic things

Thanks, didn't know how it should have been done because haven't setup a nest project before, only worked on some basic things

no worries, its always tough on the first person because they actually have to do the work 😄. Once the first one is done though it becomes easier for everyone else since theres an example already directly in the code.

That's a good thing! It means that we're achieving our value of learning new things

I've changed dto convention to model, it conflicts slightly with nest models but i feel like it's more accurate and it's just a small class

Just adding update to the db module, hold tight.

I think this is at a good point now. Thanks for the help everyone.


Zigbee2mqtt won't start after updating to 1.17.1-1

After updating to 1.17.1-1 my add-on won't start, printing this error message:
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] socat.sh: executing...
[11:45:15] INFO: Socat not enabled, marking service as down
[cont-init.d] socat.sh: exited 0.
[cont-init.d] zigbee2mqtt.sh: executing...
[11:45:16] INFO: MQTT available, fetching server detail ...
[11:45:16] INFO: Previous config file found, checking backup
[11:45:16] INFO: Creating backup config in '/config/zigbee2mqtt/.configuration.yaml.bk'
[11:45:16] INFO: Adjusting Zigbee2mqtt core yaml config with add-on quirks ...
[cont-init.d] zigbee2mqtt.sh: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.
[11:45:17] INFO: Handing over control to Zigbee2mqtt Core ...
><EMAIL_ADDRESS>start /app
> node index.js
Zigbee2MQTT:error 2021-02-12 11:45:21: Error while starting zigbee-herdsman
Zigbee2MQTT:error 2021-02-12 11:45:21: Failed to start zigbee
Zigbee2MQTT:error 2021-02-12 11:45:21: Exiting...
Zigbee2MQTT:error 2021-02-12 11:45:21: Error: Error while opening serialport 'Error: Error: No such file or directory, cannot open /dev/ttyACM0'
    at Znp.<anonymous> (/app/node_modules/zigbee-herdsman/dist/adapter/z-stack/znp/znp.js:136:32)
    at Generator.next (<anonymous>)
    at /app/node_modules/zigbee-herdsman/dist/adapter/z-stack/znp/znp.js:27:71
    at new Promise (<anonymous>)
    at __awaiter (/app/node_modules/zigbee-herdsman/dist/adapter/z-stack/znp/znp.js:23:12)
    at SerialPort.<anonymous> (/app/node_modules/zigbee-herdsman/dist/adapter/z-stack/znp/znp.js:134:49)
    at SerialPort._error (/app/node_modules/zigbee-herdsman/node_modules/@serialport/stream/lib/index.js:198:14)
    at /app/node_modules/zigbee-herdsman/node_modules/@serialport/stream/lib/index.js:242:12
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: `node index.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2021-02-12T10_45_21_998Z-debug.log

Core: core-2021.2.3
Home Assistant OS 5.11

After this update, zigbee2mqtt don't start.
Log:
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] socat.sh: executing...
[11:53:26] INFO: Socat not enabled, marking service as down
[cont-init.d] socat.sh: exited 0.
[cont-init.d] zigbee2mqtt.sh: executing...
[11:53:29] INFO: MQTT available, fetching server detail ...
[11:53:30] INFO: Previous config file found, checking backup
[11:53:30] INFO: Creating backup config in '/share/zigbee2mqtt/.configuration.yaml.bk'
[11:53:31] INFO: File devices.js found, copying to ./node_modules/zigbee-herdsman-converters/
cp: can't create './node_modules/zigbee-herdsman-converters/devices.js': No such file or directory
[cont-init.d] zigbee2mqtt.sh: exited 1.
[cont-finish.d] executing container finish scripts...
[cont-finish.d] done.
[s6-finish] waiting for services.
[s6-finish] sending all processes the TERM signal.

@fss your error is 'Error: Error: No such file or directory, cannot open /dev/ttyACM0' which means your serial port is unavailable. Are you running in a VM or native HassOS setup?
@Murada99  your error is related to devices.js cutomisation which is not really the recommended (and supported) method for adding custom devices. See the docs here for the new method: https://github.com/zigbee2mqtt/hassio-zigbee2mqtt/blob/master/zigbee2mqtt/DOCS.md#adding-support-for-new-devices

@ciotlosm I'm running HassOS, but it might be my Supervisor version being super-old (and it refuses to update)

@fss you can go to the Supervisor / Hardware as stated in the Notes of docs and find out which named serial port to use:
To find out which serial ports you have exposed go to Supervisor → System → Host system → ⋮ → Hardware

@ciotlosm yeah I know ;) the device is copied from there and it all worked perfectly until today :)

@ciotlosm I managed to update Supervisor to latest version, now the error message is slightly changed:
Zigbee2MQTT:error 2021-02-12 12:17:39: Error while starting zigbee-herdsman
Zigbee2MQTT:error 2021-02-12 12:17:39: Failed to start zigbee
Zigbee2MQTT:error 2021-02-12 12:17:39: Exiting...
Zigbee2MQTT:error 2021-02-12 12:17:39: Error: Error while opening serialport 'Error: Error: Operation not permitted, cannot open /dev/serial/by-id/usb-Texas_Instruments_TI_CC2531_USB_CDC___0X00124B001949CABE-if00'
    at Znp.<anonymous> (/app/node_modules/zigbee-herdsman/dist/adapter/z-stack/znp/znp.js:136:32)
    at Generator.next (<anonymous>)
    at /app/node_modules/zigbee-herdsman/dist/adapter/z-stack/znp/znp.js:27:71
    at new Promise (<anonymous>)
    at __awaiter (/app/node_modules/zigbee-herdsman/dist/adapter/z-stack/znp/znp.js:23:12)
    at SerialPort.<anonymous> (/app/node_modules/zigbee-herdsman/dist/adapter/z-stack/znp/znp.js:134:49)
    at SerialPort._error (/app/node_modules/zigbee-herdsman/node_modules/@serialport/stream/lib/index.js:198:14)
    at /app/node_modules/zigbee-herdsman/node_modules/@serialport/stream/lib/index.js:242:12

@ciotlosm Thanks, I removed the file and now it starts. It was working until this update but now I need to change the way to add new devices.

@ciotlosm is this supposed to work?
chmod 777 /dev/ttyACM0
chmod: /dev/ttyACM0: Read-only file system

@fss I don't think that's supposed to work. Mine works without any mods, so might be worth trying to fix your old supervisor.
@Murada99 I'm sure someone might know how to make it work with the custom devices in the new version, but it's worth learning the new way as it's much more stable between version upgrades.

as said, I was able to update Supervisor to the latest version but file system permissions seem completely borked now. on a friend's setup listing /dev/tty* shows chmod of 777 meaning the port can be written to, whereas on mine it's in some weird audio group and writing is not possible.

@fss where are you running the commands? In the terminal addon or on the host?

@ciotlosm in the terminal addon

@fss i have also the audio group, but I have w enabled, but I'm running HassOS 5.4. I'll upgrade to see if it reproduces. I posted in #27 my details as it seems multiple people have problems with /dev and readonly.

@ciotlosm FYI, I'm running core-2021.2.3, supervisor-2021.02.6, Home Assistant OS 5.11
thanks!

Hi, I am having an issue too. MQTT won't start. I think there is something wrong with one of the zigbee devices from the old setup. I will have a check tonight.
`[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] socat.sh: executing...
[14:07:40] INFO: Socat not enabled, marking service as down
[cont-init.d] socat.sh: exited 0.
[cont-init.d] zigbee2mqtt.sh: executing...
[14:07:49] INFO: MQTT available, fetching server detail ...
[14:07:50] INFO: MQTT server settings not configured, trying to auto-discovering ...
[14:07:52] INFO: Configuring 'mqtt://core-mosquitto:1883' mqtt server
[14:07:52] INFO: MQTT credentials not configured, trying to auto-discovering ...
[14:07:53] INFO: Configuring'addons' mqtt user
[14:07:53] INFO: Previous config file found, checking backup
[14:07:53] INFO: Creating backup config in '/config/zigbee2mqtt/.configuration.yaml.bk'
[14:07:54] INFO: Adjusting Zigbee2mqtt core yaml config with add-on quirks ...
[cont-init.d] zigbee2mqtt.sh: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.
[14:07:57] INFO: Handing over control to Zigbee2mqtt Core ...

<EMAIL_ADDRESS>start /app
node index.js
assert.js:105
throw new AssertionError(obj);
^
AssertionError [ERR_ASSERTION]: Converter field exposes is undefined
at validateDefinition (/app/node_modules/zigbee-herdsman-converters/index.js:52:16)
at Object.addDefinition [as addDeviceDefinition] (/app/node_modules/zigbee-herdsman-converters/index.js:59:5)
at new ExternalConverters (/app/lib/extension/externalConverters.js:13:38)
at new Controller (/app/lib/controller.js:77:34)
at Object. (/app/index.js:25:20)
at Module._compile (internal/modules/cjs/loader.js:1063:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1092:10)
at Module.load (internal/modules/cjs/loader.js:928:32)
at Function.Module._load (internal/modules/cjs/loader.js:769:14)
at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:72:12) {
generatedMessage: false,
code: 'ERR_ASSERTION',
actual: undefined,
expected: undefined,
operator: 'notStrictEqual'
}
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: node index.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2021-02-12T13_08_23_975Z-debug.log
[14:08:25] INFO: Handing over control to Zigbee2mqtt Core ...
<EMAIL_ADDRESS>start /app
node index.js`

@NL18958 your MQTT is fine, it just says it auto-discovered the connection. The problem is probably with a customization you are running. Are you using any custom "devices.js" in your setup?

I did change devices.js previously for a zigbee device that is not standard available. Never got it to work, when upgrading to 1.17.0. I will remove the entry in devices.js tonight and post back here.

The usb stick is detected but as a GENERIC !  wich i think that there is a driver issue or kernel issue on the HA side .
Also output from my zigbee2mqtt add-on
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] socat.sh: executing...
[14:51:57] INFO: Socat not enabled, marking service as down
[cont-init.d] socat.sh: exited 0.
[cont-init.d] zigbee2mqtt.sh: executing...
[14:51:58] INFO: MQTT available, fetching server detail ...
[14:51:59] INFO: Previous config file found, checking backup
[14:51:59] INFO: Creating backup config in '/share/zigbee2mqtt/.configuration.yaml.bk'
[14:51:59] INFO: Adjusting Zigbee2mqtt core yaml config with add-on quirks ...
[cont-init.d] zigbee2mqtt.sh: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.
[14:52:00] INFO: Zigbee Herdsman debug logging enabled
[14:52:00] INFO: Handing over control to Zigbee2mqtt Core ...

<EMAIL_ADDRESS>start /app
node index.js
Zigbee2MQTT:info  2021-02-12 14:52:07: Logging to console and directory: '/share/zigbee2mqtt/log/2021-02-12.14-52-04' filename: log.txt
Zigbee2MQTT:info  2021-02-12 14:52:07: Starting Zigbee2MQTT version 1.17.1 (commit #1.17.1)
Zigbee2MQTT:info  2021-02-12 14:52:07: Starting zigbee-herdsman (0.13.59)
2021-02-12T14:52:08.004Z zigbee-herdsman:adapter:zStack:znp:error Failed to determine if path is valid: 'Error: ENOTDIR: not a directory, lstat '/dev/tty/ttyS0''
2021-02-12T14:52:08.047Z zigbee-herdsman:adapter Failed to validate path: 'Error: spawn udevadm ENOENT'
2021-02-12T14:52:08.052Z zigbee-herdsman:controller:log Starting with options '{"network":{"networkKeyDistribute":false,"networkKey":[1,3,5,7,9,11,13,15,0,2,4,6,8,10,12,13],"panID":6754,"extendedPanID":[221,221,221,221,221,221,221,221],"channelList":[11]},"serialPort":{"path":"/dev/tty/ttyS0"},"databasePath":"/share/zigbee2mqtt/database.db","databaseBackupPath":"/share/zigbee2mqtt/database.db.backup","backupPath":"/share/zigbee2mqtt/coordinator_backup.json","adapter":{"concurrent":null,"delay":null}}'
2021-02-12T14:52:08.056Z zigbee-herdsman:adapter:zStack:znp:log Opening SerialPort with /dev/tty/ttyS0 and {"baudRate":115200,"rtscts":false,"autoOpen":false}
Zigbee2MQTT:error 2021-02-12 14:52:08: Error while starting zigbee-herdsman
Zigbee2MQTT:error 2021-02-12 14:52:08: Failed to start zigbee
Zigbee2MQTT:error 2021-02-12 14:52:08: Exiting...
Zigbee2MQTT:error 2021-02-12 14:52:08: Error: Error while opening serialport 'Error: Error: Not a directory, cannot open /dev/tty/ttyS0'
at Znp. (/app/node_modules/zigbee-herdsman/dist/adapter/z-stack/znp/znp.js:136:32)
at Generator.next ()
at /app/node_modules/zigbee-herdsman/dist/adapter/z-stack/znp/znp.js:27:71
at new Promise ()
at __awaiter (/app/node_modules/zigbee-herdsman/dist/adapter/z-stack/znp/znp.js:23:12)
at SerialPort. (/app/node_modules/zigbee-herdsman/dist/adapter/z-stack/znp/znp.js:134:49)
at SerialPort._error (/app/node_modules/zigbee-herdsman/node_modules/@serialport/stream/lib/index.js:198:14)
at /app/node_modules/zigbee-herdsman/node_modules/@serialport/stream/lib/index.js:242:12
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: node index.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2021-02-12T14_52_08_173Z-debug.log

I did change devices.js previously for a zigbee device that is not standard available. Never got it to work, when upgrading to 1.17.0. I will remove the entry in devices.js tonight and post back here.

Finally got it to work, I had to change file 'freepad_ext.js'. Once I copied back the example code, the add on works again. I hope this helps someone that has the same issue. The problem was that my code for a device that is not supported wasn't working.

Have exactly the same issue as topic starter.  More info here.
https://github.com/zigbee2mqtt/hassio-zigbee2mqtt/issues/27#issuecomment-778636591

Closing this issue as it's covered under #27 for the readonly problem and under #25 for the devices.js problem.


Configurable timeout for kubernetes driver

This PR adds following ,

vendor update for stork (govendor)
vendor update for sched-ops(govendor)
Update torpedo scheduler driver to use updated interfaces in sched-ops

Does hardcoed timeout in driver/k8s also needs to be configurable , those are used with task.DoRetryWithTimeout ?


Automatic Door is no longer per player

Is there an existing issue for this?

[X] I have searched the existing issues

Mod version
1.4.0
Is this a bug caused by this mod alone?
This mod alone did not reproduce the bug
List of mods to reproduce bug
Lots (24) but it was working fine before updating this mod to 1.4.0 (no other mods were updated). Discovered it earlier today and didn't get a chance to test it without other mods.
What happened?
Expected: I(host) can have Automatic Door enabled while another player has it disabled and doors will only open automatically for me (or that the host's setting will override other players' whenever the host is present).
Actual: Whoever was first to enter an area (or has otherwise been there the longest) will override the other's setting.
Note: I was hosting locally (via the "Start Server" menu option) and have Automatic Door enabled, other player does not.
Side note: I think this might happen to other features (automatic processing) as well but haven't properly tested them.
Steps to reproduce the bug

Have Automatic Door enabled
Enter an area no one else is in (e.g. via long distance portal)
Note functioning auto-doors
Have another player with Automatic Door disabled enter area
Note Automatic Doors apply for all
Have original auto-door player leave area and return
Note doors no longer open automatically.

Logs and screenshots
No response

The door not closing automatically is a bug. Fixed in commit 50313953d00ba1797f0475cf6f820aaa603e125c.
It is not a bug that automatic doors are enabled for players other than the host; it is a specification. Automatics are not designed to work only for the host. (Automatics was never designed to work on the server side in the first place...)


Change reader.Read in readObjectFile to fix #18

See https://github.com/gogits/git/issues/18

As it will return an io.EOF when done the error needs multiple checks right? Something like
if err != nil || err != io.EOF

Yes, I think we should first:

Read any bytes left.
Check err, return if not io.EOF
Break out if it's io.EOF.

Sounds right?...

Sounds good, I'm AFK until tomorrow so will change then 😊

what about using something like https://golang.org/pkg/bufio/#Reader.ReadBytes and delimit on a space. that way a lot of work can be offloaded to the stdlib and it's probably less error prone.

Ok, hang on. I had another look at the function and I'm not sure wether it's leaking the file. the way I see it:

file opened here: https://github.com/gogits/git/blob/master/repo_utils.go#L285
if there's an error, gets closed https://github.com/gogits/git/blob/master/repo_utils.go#L290-L296
gets wrapped into a zlib reader https://github.com/gogits/git/blob/master/repo_utils.go#L298
get reseeked to the beginning https://github.com/gogits/git/blob/master/repo_utils.go#L335
gets wrapped into a zlib reader again https://github.com/gogits/git/blob/master/repo_utils.go#L340

From this point on there is no direct access to f anymore, only behind io.Readers. I already started on fixing this and then saw the same pattern in https://github.com/gogits/git/blob/master/repo_utils.go#L172
So i propose another solution.

I'd like to change readerDecompressed to take an io.ReadCloser and make the Close() method to close both the decompressor and the underlying file.
Rewrite the function and get rid of the seeking and using 2 zlib decompressors.

The changes aren't done yet. I'll make another pull request as soon as I can.

@lemmi good catch!


feat: Determine IT modules to test by files changed

This has been merged in via: https://github.com/googleapis/google-cloud-java/pull/8110


Use selected API server port in manifest templates

Just patching in #555 which is a good change but seems abandoned.

Ah, url.Port requires Go 1.8+
We can hold off on this until Bootkube officially moves to Go 1.8

coreosbot run e2e

coreosbot run e2e

coreosbot run e2e

@aaronlevy don't think that affects bootkube anymore since we pulled the apiserver out, and the fix was cherry-picked all the way back to 1.5.


Command line with Azure CLI to check if Ultradisk is available for a VMsize in Availability can be little improved

For VM in availability zones :
The following command give a result difficult to interpret :
az vm list-skus --resource-type virtualMachines  --location $region --query "[?name=='$vmSize'].locationInfo[0].zoneDetails[0].Name" --subscription $subscription
This one is more human readable and understandable:
az vm list-skus --resource-type virtualMachines  --location $region --query "[?name=='$vmSize'].locationInfo" --subscription $subscription -o jsonc

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

ID: a5164427-03a0-3529-959d-98167155d654
Version Independent ID: 443fda4f-c715-540c-19a9-edb7222ba6b8
Content: Ultra disks for Linux VMs - Azure managed disks - Azure Linux Virtual Machines
Content Source: articles/virtual-machines/linux/disks-enable-ultra-ssd.md
Service: virtual-machines
Sub-service: disks
GitHub Login: @roygara
Microsoft Alias: rogarana

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Thanks for the feedback on this doc.
At this point in time we are opting to leave the document as is.
#please-close


tasks requiring confirmation

When I run a task that requires confirmation from vim-taskwarrior such as deleting an order the command fails.
:!task 107 delete

Delete task 107 'update order'? (yes/no) Task not deleted.
Deleted 0 tasks.
shell returned 1

I'm using neovim 0.2.2. Is there any way to confirm the task deletion from vim-taskwarrior?

Same issue here.

ditto / tritto (three people with the same issue?) lol

This is a really old issue but has this been resolved. Is it possible to simply use something like Dispatch so that it doesn't use the default ?

taskwarrior 2.4.0 starts to allow rc.confirmation=no for a non-interactive shell command.
Since this repo is not maintained any more, I've fixed it in my fork (hopefully will keep adding new things), and available here:
https://github.com/xarthurx/vim-taskwarrior

The hero we need but don't deserve 😭

The undo should also be fixed now.
The main problem is that vim doesn't encourage interactive-type of command.
So what I do in the fork is to fake a confirmation with vimscript, and suppress the shell interaction with rc.confirmation=no.
I'll be first implementing meaningful fixes based on the issues in this repo for the past 2 years, then see if it make sense to add more functionalities from newer TaskWarrior.
There's also an alternative way to interactive with TaskWarrior with tasklib.
(This is my beginning attempt to work with TaskWarrior from WSL by contributing PRs, but doesn't work out well with it + taskwiki -- I end up here with this plugin)
Please feel free to try out the fork and submit issues/PRs.

for the command of deleting, please use either :TWDelete or the shortcut D instead of pure shell command :!task xx delete.
The latter is a pure shell cmd and vim cannot handle the interaction.
Though of course you can do :!task xx delete rc.confirmation=no if you like.

@xarthurx : Tried your fork, but I am getting errors. Where can I submit an issue?

Thank you for the test. Yeah, there're several annoying things I've
noticed, and I'm constantly adding things recently, though during weekends.
Please submit in my fork, so that I'll follow them and look into them.
On Thu, 16 May 2019 at 19:45, Philipp Woelfel<EMAIL_ADDRESS>wrote:

@xarthurx https://github.com/xarthurx : Tried your fork, but I am
getting errors. Where can I submit an issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/blindFS/vim-taskwarrior/issues/142?email_source=notifications&email_token=AAOVGVRL2NYF5TAXZHJII6LPVWMTJA5CNFSM4EKXFFY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVSR2HA#issuecomment-493165852,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOVGVWSO5AALR7SZHO5FB3PVWMTJANCNFSM4EKXFFYQ
.

--
MA Zhao
"SAVE PAPER - PLEASE THINK BEFORE YOU PRINT!"


Where's m_flDetectedByEnemySensorTime offset? QUESTION

This offset just disappears from client.dll in commit of update 14004, what happened?

It was removed by Valve. Please look at the closed issues first before opening a new one next time.

Srr


remove outdated information

https://github.com/MicrosoftDocs/windows-itpro-docs/issues/8472
source: https://techcommunity.microsoft.com/t5/microsoft-defender-for-endpoint/edr-capabilities-for-macos-have-now-arrived/ba-p/1047801

Dear @JohanFreelancer9,
kindly do the copy editor review for this PR.
Dear @e0i, for your information
Thank you

#sign-off

@Dansimp PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Foreign Traveling tools

I believe adding the ability to convert currencies would be a great feature, I don't know how it would be done without internet connection since values constantly change and fluctuate, but it is an incredibly useful tool when traveling to a foreign country. In the same logic, maybe a minimal guide to languages, like a basic dictionary on how to thank, salute, etc... a foreigner in his native language. Thank you! <3

Hi, thank you for the suggestion - given that currency rates fluctuate, I don't think it would be feasible for this to be in Trail Sense without the Internet. I don't really want to venture into areas involving money, so I am not going to add this feature to Trail Sense.
What I can do though, is to find a good open-source currency converter and link it in the recommended apps guide (I'm going to repurpose this issue to reflect that)


Fix automatic labels for components

Fixes #1077
Does this PR introduce a user-facing change?:
Fix automatic labels for components

/cherrypick release/1.0

/cherrypick release/1.0

/cherrypick release/v1.0


[v0.2.2] Release Tracker

ETA: Nov 3rd (Fri) - Nov 6th (Mon).
Major changes
TBD
PRs to be merged before the release

[ ] CUDA 12 #1527
[ ] Support embedding inputs #1265
[x] Yarn #1264, #1161

@zhuohan123 @simon-mo Please feel free to add if you have any PR that should be merged for the next release.

I can release the corresponding docker image as well! Hopefully we can get logits processor in as well?

It seems that there is something wrong with batch processing at the current main branch.
When i open a api server that serving wizardcoder-1B model, and testing humaneval by using batch requests and greedy sampling, it gives me less than 10% on main branch whereas 0.2.1-post1 gives me 23.17%.
related issue: #1570
Hope this will be addressed at the v2.2.0. Thanks!

It seems that there is something wrong with batch processing at the current main branch.
When i open a api server that serving https://huggingface.co/WizardLM/WizardCoder-1B-V1.0 model, and testing humaneval by using batch requests(164 concurrent reqs) and greedy sampling, it gives me less than 10% on main branch whereas 0.2.1-post1 gives me 23.17%.
related issue: #1570
I have no idea about this and hope this will be addressed at the  coming v0.2.2. Thanks!

#1546 Fix this.

When is the v0.2.2 version scheduled to be released?

@incrementallearning It's in the process. We are planning to release it asap. ETA is Nov 17th (Fri) - 19th (Sun).


I am studying currntly for contribution !

Hi @ayansiddiqui007,
Thank you for your contribution! We really value the time you've taken to put this together.
Before we proceed with reviewing this pull request, please sign the Lightbend Contributors License Agreement:
https://www.lightbend.com/contribute/cla

Is this still October? https://blog.domenic.me/hacktoberfest/


Failed to mint

INFO [05-05|17:44:52.572] Calling mint function in contract
INFO [05-05|17:44:52.575] Submitting transaction to token contract tokenaddress=0x395E925834996e558bdeC77CD648435d620AfB5b
ERROR[05-05|17:44:52.578] Error occured while minting: insufficient funds for gas * price + value
INFO [05-05|17:45:01.174] found new head                           head=17,196,001 synced=true
INFO [05-05|17:45:02.579] Minting                                  receiver=ac5beb433bbbc14745e573c0755cd0a62fdd506f txID=90b36861227db15aaec8d98b60575a4babb13c2c4f62b94dc1aea6c10ff96765
ERROR[05-05|17:45:02.681] failed to get signature from             err="failed to connect to host id '12D3KooWLEdJsPFe3utrnc3teHJvS9oN9MfuBWRW9hxmoEGZGVfm': failed to dial 12D3KooWLEdJsPFe3utrnc3teHJvS9oN9MfuBWRW9hxmoEGZGVfm:\n  * [/p2p/12D3KooWGttLPcYHMGLep8F9G2xAxM1wLT3JvraXhV2z9BeYmmok/p2p-circuit] dial backoff"
INFO [05-05|17:45:07.596] got a valid reply from a signer
INFO [05-05|17:45:07.697] got a valid reply from a signer
INFO [05-05|17:45:07.699] Calling mint function in contract
INFO [05-05|17:45:07.702] Submitting transaction to token contract tokenaddress=0x395E925834996e558bdeC77CD648435d620AfB5b
ERROR[05-05|17:45:07.705] Error occured while minting: insufficient funds for gas * price + value
INFO [05-05|17:45:12.530] found new head                           head=17,196,002 synced=true
INFO [05-05|17:45:17.706] Minting                                  receiver=ac5beb433bbbc14745e573c0755cd0a62fdd506f txID=90b36861227db15aaec8d98b60575a4babb13c2c4f62b94dc1aea6c10ff96765
INFO [05-05|17:45:17.808] got a valid reply from a signer

Tx got confirmed after a while:
INFO [05-05|18:29:12.712] Noticed mint event                       receiver=0xAC5bEB433bbBC14745E573c0755cD0a62Fdd506F amount=2,000,000,000,000 "TFT tx id"=c0c33a..9ad632

005c932ef9fa5b9bab1bd45a943bee527f1d83dd


fix should not use selectHint in any place (#41260)

pick from master #41260
because all comment has been redirect to channel 2, we should not use hint in any place. selectHint only use to parse hint.

run buildall


maintainers: add eklairs

Description of changes
Add eklairs as maintainer
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Hi @eklairs! Will you be maintaining any packages? Generally, maintainer-list entries are added as separate commits alongside a package adoption/update/fix.

Hi @SigmaSquadron!
Oh i did not know about that - I will be maintaining a package, I thought I will make two PRs for it (one for maintainership and other for adoption). I will add the package adoption commit with this PR :)


Please add action "remove kernels with non existing vmlinuz"

eclean-kernel had the mentioned mode. It was possible to remove unneeded kernel files manually and use eclean-kernel for cleaning. Possible use cases: LTS kernel and current kernel installed simultaneously; bisecting kernel, which produces different versions. It would be useful in these cases to be able to select which kernels to remove, either by deleting vmlinux files manually, or providing versions to eclean-kernel -l , e.g. eclean-kernel --keep-build-dir 4.6.3-gentoo 4.4.6-gentoo

Indeed; actually, eclean-kernel (1) used to by default remove all kernels not referenced by the bootloader, whereas eclean-kernel2 only allows keeping a set number of newest kernels.

Different removal modes are work-in-progress. Can't promise when I'll have time to finish it though.


spritz_random32_uniform

this random function will work in practice, however depending on the UPPER bound (esp large values) it is not uniform
imagine your random generator generates 0 .. 9
and you set the upper bound to  7
then the numbers 0,1,2 have twice as many chance of being chosen than numbers 3,4,5,6,7,8 and 9.
this is because UPPER and MAX random are same order of magnitude.
When UPPER bound is small  it is still not uniform but the bias will be substantial smaller (less problematic)

This algorithm is used almost in all crypto libs and in operating systems such as FreeBSD OS RNG.
This is made to avoiding modulo bias which you are mentioning are you sure it's wrong/broken algorithm?

I do not see how this algorithm avoids the modulo bias. Do you have a link to a good explanation?

This algorithm is used started in OpenBSD OS, and now it's used in many places such as Linux, OpenSSL, OpenSSH, Android, tc...
the original code is in: https://cvsweb.openbsd.org/cgi-bin/cvsweb/~checkout~/src/lib/libc/crypt/arc4random.c?rev=1.55&content-type=text/plain&only_with_tag=HEAD
Here is an explanation for it in an RNG library: https://www.pcg-random.org/posts/bounded-rands.html#debiased-modulo-twice-openbsds-method
You can contact the dev. from OpenBSD for more documentation about it.

OK thanks for the link (and all patches)
I will close this issue and only reopen it if needed.

The link is a good explanation. Thanks,

worst case happens when upper_bound is 2^31 + 1 ==> min = 2^31 -1   so the range of valid values is minimal.(~50%)
drawback is algorithm needs two % operations which are relative expensive if there is no co processor

point (2) can partially be optimized in case the upper_bound stays constant over multiple subsequent calls.
(speed vs memory exchange). Is that a typical use?
uint32_t
spritz_random32_uniform(spritz_ctx *ctx, uint32_t upper_bound)
{
  static uint32_t min = 0, ub = 0;    // to hold last used values
  uint32_t r;

  if (upper_bound < 2)
  {
    return 0;
  }

  if (ub != upper_bound)      //  only recalculate if needed.
  {
    ub = upper_bound;
    /* 2**32 % x == (2**32 - x) % x */
    min = (uint32_t)(-ub % ub);
  }

  /* This could theoretically loop forever but each retry has
   * p > 0.5 (worst case, usually far better) of selecting a
   * number inside the range we need, so it should rarely need
   * to re-roll.
   */
  for (;;) {
    r = spritz_random32(ctx);
    if (r >= min)
    {
      return (uint32_t)(r % ub);
    }
  }
}

The modulo in line return (uint32_t)(r % ub); is not needed if and only if the range [min, 2^32] === [0, upper_bound]
That is for ~50% of the possible values for upper_bound.
Note: This is independent of the above caching strategy of upper_bound.
uint32_t
spritz_random32_uniform(spritz_ctx *ctx, uint32_t upper_bound)
{
  static uint32_t min = 0, ub = 0;    // to hold last used values
  uint32_t r;

  if (upper_bound < 2)
  {
    return 0;
  }

  if (ub != upper_bound)      //  only recalculate if needed.
  {
    ub = upper_bound;
    /* 2**32 % x == (2**32 - x) % x */
    min = (uint32_t)(-ub % ub);
  }

  /* This could theoretically loop forever but each retry has
   * p > 0.5 (worst case, usually far better) of selecting a
   * number inside the range we need, so it should rarely need
   * to re-roll.
   */
  for (;;) {
    r = spritz_random32(ctx);
    if (r >= min)
    {
      if ( ub <= (1 << 31)) return (uint32_t)(r % ub);
      else return r - min;      // no 'expensive' modulo needed.
    }
  }
}

if upper_bound is always below 2^31 this 'optimization' will never be used and in fact add a 32 bit compare in every call.

If upper_bound is known to be 0..31 there is another fast trick - bit_count
uint8_t_t
spritz_random32_uniform_32(spritz_ctx *ctx, uin8_t upper_bound)
{
  uint8_t count = 0U;
  uint32_t r;

  if (upper_bound < 2) return 0;

  r = spritz_random32(ctx);
  for (uint8_t i = 0; i < upper_bound; i++)
  {
    if (r & 1U ) count ++;
    r >>= 1;
  }
  return count;
}

in fact, one could cache the value of r (static) as it may hold enough bits for next call.
(used this in the past to simulate 5 throws of a dice with one 32 bit call to random()  )

The very similar trick can also be used for larger values too.. One 32 bit random number equals two 16 bit patterns, three 10 bit patterns, four 8 bit patterns or five 6 bit patterns up to  32 one bit patterns.
So if upperbound < 16 bits only one call to random is needed...
uint16_t_t
spritz_random32_uniform_16(spritz_ctx *ctx, uin16_t upper_bound)
{
  static uint32_t bits;
  static uint8_t bitsleft = 0;

  if (upper_bound < 2) return 0;
  uint8_t cnt = bits needed for upperbound (some loop with shifts +1)

  for (;;)
  {
    if (bitsleft < cnt)
    {
      bits = spritz_random32(ctx);
      bitsleft = 32;
    }
    r = bits >> cnt;
    bitsleft -= cnt;
    if (r < upper_bound) return r;
  }
}

just to share some thoughts that popped up

Yes it's possible to do that but spritz_random32_uniform_16() will not be thread-safe anymore for an RTOS, correct me if I'm wrong.

if upper_bound is always below 2^31 this 'optimization' will never be used and in fact add a 32 bit compare in every call.

Usually this function used to make visible passwords/keys, so yes, there is no sweet way to optimize this and stay "a universal" code that can run in any machine and OS.
Any further optimization from the current code point to more specific usage case that will bring a negative side.
The only nice optimization that can not give out much side effects is a parallel u32_uniform() that fill out an array of length N:
void u32_uniform(spritz_ctx *ctx, uin16_t upper_bound, uint32_t *array, size_t array_len)
And about the modulo op, if the machine is very basic like ATmega328P it will not take noticeable time to make something like 16-length password/key which is the usual usage for this:
https://android.googlesource.com/toolchain/gcc/+/3f73d6ef90458b45bbbb33ef4c2b174d4662a22d/gcc-4.6/gcc/config/avr/libgcc.S#472
The time-consuming op is __udivmodsi4 which the function will use two times (min & output).
Your research is interesting, thank you.

Earlier I did give you the wrong link from OpenBSD (new version code moved),
So here is the right link: https://cvsweb.openbsd.org/cgi-bin/cvsweb/src/lib/libc/crypt/arc4random.c?rev=1.40&content-type=text/x-cvsweb-markup
Since you are interested, check this old version of it that has some optimization code:
https://opensource.apple.com/source/OpenSSH/OpenSSH-166/openssh/openbsd-compat/bsd-arc4random.c
uint32_t
arc4random_uniform(uint32_t upper_bound)


Pr ch06 json worth a thousand words

Chapter 06 - "Incorporate the relationships into include"
The rest of the JSON before this is all using user with id of 2. When we get to the final code listing the relationships user id is 1 but the include has remained 2.

There's an end quote missing on the English and Spanish but present in the French. I've corrected the English and Spanish.

Great! Thanks you!

Great! Thanks you!


Update build to use sbt 1.1.0

Also, update dependencies. There are a lot of code format changes due to Scalariform update.

I noticed the following:

The test failure may be caused by https://github.com/travis-ci/travis-ci/issues/8836.

Replacing sudo: false with sudo: required seems to work as a workaround.
Or --no-sandbox flag may work.

Maybe Scala versions of .travis.yml can be updated (along with those of interplay).

Sorry if I bothered you.

@hirofumi, thanks for your suggestions. I will try them now. :-)

Hi @Dasiu,
Thank you for your contribution! We really value the time you've taken to put this together.
Before we proceed with reviewing this pull request, please sign the Lightbend Contributors License Agreement:
http://www.lightbend.com/contribute/cla

Sorry @Dasiu. I messed up rebasing and --force pushing and your GitHub handle appeared on this PR by mistake triggering the CLA Validator. You are not required to sign the CLA for this PR.

@marcospereira, I messed a rebase and ended up squashing the original commits you had pushed into 4e0b312

Redid the rebase.
The upgrade of scalariform and mima surfaced more code changes and a couple of mima issues (introduced in #136)

(I did two mistakes trying to rebase this old PR on top of current master).
The PR (https://github.com/playframework/scalatestplus-play/pull/110) was created from master a long time ago.
Since late Jan 2019, master and 4.0.x diverged with some PRs not back/forward-ported.
Yesterday I rebased the commits from the PR on top of 4.0.x and today I rebased the branch from 4.0.x to master. On my rebase today git not only took the commits from the PR but also the commits in 4.0.x that had not been forward-ported to master (e.g. https://github.com/playframework/scalatestplus-play/pull/110/commits/0022f6b54ea0bf7dee1aabda032a6d73ccff4c19 which enables MiMa).
Summing up:

the PR (as it is now) is not just bumping sbt to 1.0 but also forward porting #131 and #134

TIL: GH lists the commits on the PR ordering by date of commit (not position in the git tree)


Fix _SS_earnings_c to index by wage rate

Sends proper parameter name (with '_' prefix) to
indexing_rates_for_update method to get the proper rates for
propagating value forward in time

resolves #224

@talumbau, Thanks for the quick fix on this!


Shopping

Kindly check and review it.............

Hi @Ellyboi
Your project is complete! There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉


Having trouble with Facebook-like popover, and silent route triggering

Hi there! I am building an application where one route can have two states (hear me out!). This route is defined at /search/:query. There are two ways to access it. The first is by entering the search term into the search bar, at which point all of the current route's content blurs, and the search page appears on top of it. To do this, I need to render the search page on top of the current stuff, without replacing it.
The other way to access the page is to simply refresh /search/:query at which point the search results will appear on a plain white background. I think a good analogy is the facebook image gallery.
I have managed to make this work! My problem is that in the first case, when I am done with the blurred-popover search results page, and I click a <Link/> to a new route, that route's onEnter function is not called, and I don't know a better way to ensure that the blur effect is removed and the popover page is removed.
In short, I am currently looking for a way to navigate to a route without triggering a whole-page replace, but still letting the router know that we have indeed navigated, so that navigating away to another page triggers onEnter functions. I am currently using router.replaceWith() to do the replace, but I'm a little stumped. Thanks in advance!

what your describing sounds two states that share a url path... i would check out the rackt/history sister project to react-router (already a dep). it looks like the pushState api would work for your use case. something like history.pushState({ q: 'test', modal: false }, '/search').

Hi Justin! Thanks for the response. I will definitely take a look at that

Any update on this?

Past expiry date on this ticket. Reopen if still a problem.


Integrate Telescope with other meteor app

I would like to share the users collection between my main meteor app and telescope. By that I mean, whenever a user registers on the main app, he/ she should be able to login with the same username / password.
There is a similar question on http://stackoverflow.com/questions/28135868/can-i-use-the-same-db-for-multiple-meteor-apps and some answers.
I am sure many others would like to have this functionality.
Can you please guide the correct way to do it. Maybe add it to documentation.
Thank you!

There's a related question and solution here: https://github.com/TelescopeJS/Telescope/issues/963
I haven't tried it myself so I haven't had a chance to add a tutorial to the docs yet. But maybe you can give it a try.


AC-3 is not detect

Version
Media3 1.2.1
More version details
This url can't detect ac-3 but MXPlayer can detect
exoplayer only have "mp2"

mxplayer have two audio track, track 1 is mp2, track 2 is ATSC A/52A

Devices that reproduce the issue
Xiaomi 13
Devices that do not reproduce the issue
No response
Reproducible in the demo app?
Yes
Reproduction steps
play url by exoplayer
need to add basic auth header
Expected result
have two audio tracks
Actual result
only one audio track

Media
http://11:1212@www.yuntvh.cn:981/stream/channelid/309326390?ticket=6a254587cf68b2fa4860f500b9d94c636b952f8c&profile=pass
Bug Report

[ ] You will email the zip file produced by adb bugreport to<EMAIL_ADDRESS>after filing this issue.

Here is ijkplayer log

If I forced -1 to ac-3, the ac-3 track is appear

I found the url ac-3 streamType is 0x06
so going to readEsInfo
but data.getPosition() = descriptorsEndPosition
so don't get real streamType

I found the document
https://www.atsc.org/wp-content/uploads/2015/03/A52-201212-17.pdf

@ojw28 Why the 0x06 check is remove
https://github.com/google/ExoPlayer/commit/f91ea9039dae92793fc62caa4080d4a58a947771


Function hashFiles seems to be OS-dependent

I noticed that on Windows the value of ${{ hashFiles('**/pom.xml') }} always seems to be different than on Linux/MacOS, where it is identical. Is this due to the fact that Git checks out files with CRLF line endings in Windows rather than LF on unixoid systems?
The related comment https://github.com/actions/cache/issues/102#issuecomment-555092357 and https://docs.github.com/en/actions/reference/context-and-expression-syntax-for-github-actions#hashfiles did not answer this question, unfortunately.
If I am right, is there a way to do some dynamic dos2unix kind of transformation on the set of input files before calculating the hash?

Very interesting!  The hashFiles function is implemented over in the actions/runner repo - https://github.com/actions/runner/blob/be9632302ceef50bfb36ea998cea9c94c75e5d4d/src/Misc/expressionFunc/hashFiles/src/hashFiles.ts#L38 - so we'll want to move this over there so it's seen by the right people.  I'll transfer this issue shortly after posting this comment :smile:

Or not...I apparently don't have permissions in that repo to transfer this over.  I'll need to find someone else (or close and open a new issue).

I forgot to mention: For backward compatibility, an optional, additional parameter such as conversion with a default of none and other possible values of dos2unix and unix2dos or something similar to the same effect should be added.


Spamming the command bar key and enter can result in the command bar being opened but invisible

Spamming the command bar key and enter over and over again can result in the command bar being opened but invisible.

Pretty sure this was fixed


Implemented DropDown in StudyZone

All Submissions:

[ ] Have you followed the guidelines in our Contributing document?

New Feature Submissions:

[ ] Does your submission pass build tests locally?
[ ] Have you indented your code locally prior to submission?

Changes to Core Features:

[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you successfully ran tests with your changes locally?

Hope that you have followed the contribution guidelines. Please don't make a request anywhere to get a review. One of the maintainers will get back to it within a day or so. 😃
(Powered by GitMate.io)

Perfect. But the cache files are still not removed.


Throw an error if dev scripts contain now dev in package.json

This PR throw an error message if dev scripts contain now dev in user package.json.

Codecov Report

Merging #2507 into canary will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           canary    #2507      +/-   ##
==========================================
- Coverage   11.55%   11.54%   -0.01%
==========================================
  Files         257      257
  Lines        9254     9265      +11
  Branches     1031     1034       +3
==========================================
+ Hits         1069     1070       +1
- Misses       8072     8082      +10
  Partials      113      113

Impacted Files
Coverage Δ

src/commands/dev/index.ts
0% <0%> (ø)
:arrow_up:

src/util/dev/builder-cache.ts
66.37% <0%> (-1.52%)
: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 cfc326b...683a2e1. Read the comment docs.

@sophearak Is this because @now/static-build now respects dev script or now-dev script?

@TooTallNate Yes, exactly.  Since zero-config is expected dev as a command to build static-site or app instead of now dev.

can i ask what the intention is for the rest of who have configured our projects?

Hi guys! How are you? Sorry for writing here but it would be great if you tell us how the script must be in the package-json because this fix is really working 😛 Thanks!

@pushplaybang When now dev was introduced, it needed a separate now-dev script to detect a server such as gatsby https://zeit.co/blog/now-dev
This has since been changed so that the dev script can be used. This PR is necessary to prevent an infinite loop where now dev invokes now dev.


DS-2374 authority indexing matching metadata by value

https://jira.duraspace.org/browse/DS-2374

+1

I tested the PR by deleting the authority index ([dspace]/solr/authority/data), deploying b50efa6a5a047a41f763424a8acb5ce047d94bed and running the index-authority script. Unfortunately, it exibits the same behaviour as before - for the author I tested, the same number of documents were created in the authority index as the number of items shown in the browse index for that author. What could have gone wrong?

@helix84 If you want to test with an existing Database you will also need to clear the authority column in the metadatavalue table. If certain metadata values already have authority keys these keys will be used every time. (We don't want to be throwing away data)

@KevinVdV Thanks, that was indeed the "problem" and it also explains whether and how these new generated authority IDs are persisted.
+1, tested


Failed attempt to run code in command window

I ran through "Build and run your code in Visual Studio" with apparent success. But when I tried the next step, "Run your code in a command window" there was no .exe file to be found. The debug folder contains:
a folder: HelloWorld.tlog
and the files:
HelloWorld.Build.CppClean - a text document
Hello World - a text document
HelloWorld - a 3D Object
HelloWorld.vcxproj.FileListAbnsulute
vc142.idb
vc142.pdb
and nothing else! So - what to do next?

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

ID: d597b582-59ec-2014-699b-6aac8db69cf6
Version Independent ID: 81941537-4446-9c5a-6af3-25318396420d
Content: Build and run a C++ console app project
Content Source: docs/build/vscpp-step-2-build.md
Product: visual-cpp
Technology: cpp-tools
GitHub Login: @corob-msft
Microsoft Alias: corob

Same here, more or less, but I just have:
HelloWorld.tlog (directory)
HelloWorld.log
HelloWorld.obj
vc142.idb
vc142.pdb

no HelloWorld.exe file.

A fix should appear later today that clarifies that it's the Debug directory in the HelloWorld solution folder, not the Debug directory in the HelloWorld project folder, that has the executable. Yes, it's confusing.


tests/formulae: skip attestation verification when building gh

Fixes
Error: A `brew install --formula gh` process has already locked /opt/homebrew/Cellar/gh.
Please wait for it to finish or terminate it to continue.

Homebrew/homebrew-core#192550
https://github.com/Homebrew/homebrew-core/actions/runs/11133201051/job/30938871245?pr=192550#step:3:53

Could/should maybe be better handled in brew, but I couldn't see a quick/nice fix for it.


docs: call setEmitters before load

Suggest to add additional documentation to the guide, to call setEmitters before load or loadAll. Otherwise it will trigger AkairoError [INVALID_TYPE]: Value of 'emitter' was not an EventEmitter. API users should not usually have to concern themselves with what is going on behind the scenes, therefore documentation should state if certain steps have to be carried out in a specific order.

I think "...before loadAll" should be "before loading".


Week 5- Proctored MCQ Week1 : Quiz1, Quiz2 Scores

Score: /5, Grader: Stanley Yan
AP Testers

Review ticket with Crossover Score on Monday 4/25 in class

[ ] Start review ticket with study plan and study reflection for week, due EOD Monday 4/18

Tangible

[ ] GH Indi Page with score and corrections

Tangible

[ ] Finals-Quiz1 score/50 and corrections

Tangible

[ ] Finals-Quiz2 score/50 and corrections

Tangible

[ ] Create Task code/runtime and 1 min video are reviewed by another person, you may request Teacher review as time permit

Code
Video

Score: /5
Score: 4.25/5 (last .5 to be determined by teacher)
Review ticket with Crossover Score on Monday 4/25 in class

[ ] Start review ticket with study plan and study reflection for week, due EOD Monday 4/18
good study plan but I would try to be more specific (-.25)
[ ] GH Indi Page with score and corrections
GH individual page shows show screenshot of quiz score, as well as corrections for missed questions
[ ] Finals-Quiz1 score/50 and corrections
screenshot of full score
[ ] Finals-Quiz2 score/50 and corrections
screenshot of completed score and corrections
[ ] Create Task code/runtime and 1 min video are reviewed by another person, you may request Teacher review as time permit
video of create task shows detailed code and runtime of create task


Support Kafka client's "auto.offset.reset = throw exception if no offset" setting

Feature Request: Support Kafka client's auto.offset.reset configuration:
From: https://kafka.apache.org/08/documentation.html
What to do when there is no initial offset in Zookeeper or if an offset is out of range:
* smallest : automatically reset the offset to the smallest offset
* largest : automatically reset the offset to the largest offset
* anything else: throw exception to the consumer. If this is set to largest, the consumer may lose some messages when the number of partitions, for the topics it subscribes to, changes on the broker. To prevent data loss during partition addition, set auto.offset.reset to smallest

It seems like sarama currently (1.16.0) supports the first two.

As far as I can see, Sarama supports none of these options; as soon as it sees an out of range offset, it closes the message channel of the partition consumer. For comparison, here's what the Java client does:
https://github.com/apache/kafka/blob/837f31dd1850b179918f83338b4b4487486b2c58/clients/src/main/java/org/apache/kafka/clients/consumer/internals/Fetcher.java#L962
It ignores stale offsets sent from server that it knows have been superseded, then falls back to a new offset based on the value of auto.offset.reset consumer config, and if nothing is set then it throws an error back to the client.

Closing the message channel is effectively equivalent to throwing an exception, since Go doesn't have exceptions in a meaningful sense. When this happens, the client is able to call ConsumePartition with OffsetNewest or OffsetOldest in order to achieve the behaviour of the other java config options.

It ignores stale offsets sent from server that it knows have been superseded

This is only valid behaviour when dealing with consumer-groups, which is something that happens in the sarama-cluster library.


Medication Dispense /<EMAIL_ADDRESS>

Document
Section
Comment
Organization Name
Mail

Medication Dispense
test
test
<EMAIL_ADDRESS>HL7.BE.FHIR.MEDICATIONIG\Artifacts Summary - FHIR v4.0.1

Closed due to not having a real task attached to it


Format correction for keyboard shortcut page / PDF.

Issue Type: Bug
When we use a shortcut (CTRL+K + CTRL+R) to open keyboard shortcut help, a PDF gets opened. However, there are more than one shortcut keys displayed on single line. It might correct visually, but for screen reader users bit difficult to read the page since the screen reader is reading the keystrokes without any pause. So the correct format would be tabular or there must be a single keystroke on a line.
 However are you refering to this PDF on this Page?
https://code.visualstudio.com/shortcuts/keyboard-shortcuts-macos.pdf
If yes then unfortunetly that keybinding table is a bit outdated?
Or are you referring to the bulit in VS Code keybinding editor?

@nitinzd thanks for providing feedback. However are you refering to this PDF on this Page?
https://code.visualstudio.com/shortcuts/keyboard-shortcuts-macos.pdf
If yes then unfortunetly that keybinding table is a bit outdated?
Or are you referring to the bulit in VS Code keybinding editor?

This issue seems to be about the Keyboard Shortcuts help PDF file.

This issue seems to be about the Keyboard Shortcuts help PDF file.


Dump flags after references in PO files

Closes #52
PR elixir-gettext/gettext#310 by @wpiekutowski

Pull Request Test Coverage Report for Build 2ce7e96a4f3475792f9c999cc6629d0f7d277710-PR-54

2 of 200 (1.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-34.4%) to 65.625%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/expo_po_parser.erl
0
198
0.0%

Totals

Change from base Build bf0c6131c2a281ae4aea478d687c074578a9cadd:
-34.4%

Covered Lines:
378

Relevant Lines:
576

💛  - Coveralls


Support for Arirflow v2.0.0

Please Support for Arirflow v2.0.0

Is there any work ongoing to make this plugin compatible with Airflow v2?

No ongoing work for now but I will try to find some time to put effort into it.


Documentation examples are unusably slow

I really enjoy using UI Kitten, but since I've only been using it for a few weeks I find that I need to refer to the documentation a lot, and that's a problem: the examples are absolutely unusably slow. The Button examples take 20+ seconds to render. And during the time that it's loading the code tab (<>) isn't accessible, so I have to sit there and wait for it to finish loading before I can see any code samples.
Great library, great documentation, but the use of these React Native Web examples in the documentation make me want to throw the whole thing away and use a different library.

I thought it was just me, I'd call myself somewhat of an "advanced" user. Typescript autocomplete and code exploring helps a lot, so I rarely go for the docs... but yeah,  I've noticed this too since the release of v5 (although in beta). I use Brave Browser and haven't tested others, I assumed Brave was blocking something in the docs that slowed it down.
Sadly you probably can't switch back to v4 docs because there's a few API changes that are completely incompatible with v5+, and since you're a new user you're probably using v5...

@ryanvanderpol thanks for the feedback. Thats the payment for having clickable examples.
Our docs run Angular, and to run clickable examples inside it - in general, it runs a small RNW app inside iframe for each example.
I don't know possible ways to optimize, because I'm not familiar with Angular. However, I can discuss it with our web team.

Gotcha. It's a real annoyance for me, and I assume many others, and to be honest, I don't really see what the value is in clickable examples.  This library is very basic: it's just buttons, lists, icons, menus, etc. The only thing I care about is the syntax and properties. I don't need you to prove to me that I can click a button.

Gotcha. It's a real annoyance for me, and I assume many others, and to be honest, I don't really see what the value is in clickable examples. This library is very basic: it's just buttons, lists, icons, menus, etc. The only thing I care about is the syntax and properties. I don't need you to prove to me that I can click a button.

The problem with that particular approach is that this library is updating pretty frequently, and using images could make updating the docs quite inefficient. I'm not sure if there's a way to automate that...

@lesmo comparing to v4, v5 has much more examples, plus some developer notes for each example. It just became meaningful in this version. But with annoying performance issues.

That's kinda terrible. I think it could make the library be perceived as slow.
I suffered performance issues when using react-navigation v4 with rather complex views (top navigation, list view, buttons, images, menus, etc in around three screens), which is kinda' comparable to the many examples per component page in the docs. The problem with react-navigation is that before v5 all the screens in the stack are always in memory and being executed, so there's a lot of components at the same time - v5 solves that by using react-native-screens... so perhaps a lazy loading of the examples could help? Showing the code first and then showing the example after clicking on the live tab?

@lesmo thanks for the explanation. Do you have an experience running react-native-screens in web?

@lesmo yep, that helps. I also used splitting some examples for a particular component to a separate tabs in the documentation. Thanks for taking a part in the discussion, this will be closed with release of v5 stable.

@lesmo thanks for the explanation. Do you have an experience running react-native-screens in web? Would like to try it if it works, but I'm not really sure this is a core of the problem.

Not directly, though. I just implemented it in a past project through react-navigation, immediately you can "feel" it's faster. I don't think it would help under react-native-web because on Web there's no real concept or feature of "native screen" component, so components on previous screens remain "rendered".

@lesmo yep, that helps. I also used splitting some examples for a particular component to a separate tabs in the documentation. Thanks for taking a part in the discussion, this will be closed with release of v5 stable.

My pleasure!


aphorme: init at 0.1.19

Description of changes
Add the small and fast program launcher called aphorme

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Lgtm (and builds on my machine). One thing though: To speed up future time-to-review, consider adding a testVersion test. aphorme --version appears to contain the version :)

Done :)

Could you explain why you used your fork instead of the original one?

Could you explain why you used your fork instead of the original one?

The original creator changed the app version in the Cargo.toml to 0.1.19 and then pushed that change without rebuilding the corresponding Cargo.lock.
The release I made to my fork only adds one commit, which updates the Cargo.lock.

I have been talking with the author and the next release is planned to be switched to the official repo, but there is no timeframe for when the next release will happen.
I will keep maintaining the Nix package either way, as they are not involved with Nix.


Ghost grave

so the problem is that i tried to pickup my grave in the same exact moment it got destroyed by a creeper and now i'm stuck in a position in which i can't open my inventory and i have infinite slowness

2

You closed the issue so I understand you found a solution?


Add Varsig Multiformat

Hey all 👋 We ended up wanting a LOT more flexibilty in Varsig than a codec tag, and broke it out into its own spec here: https://github.com/ChainAgnostic/varsig
This is essentially multisig, except that sequence of letters has a widely-used namespace conflict in multisig wallets. To avoid confusion, it's "varsig", but almost certainly belongs in the block with the other multiformats.
If useful, we can remove the previously-added draft signature types from this table, but I know  historically there's been resistence to that. I'm happy either way 👍

Awesome thanks @vmx :) @rvagg any objections?

sgtm; although "nonstandard-sig" - is this just a legacy thing now and the rename is a way of deprecating it? If so, should we just add a note in the description column that it's going to be removed, or at least that it's not intended to be used for anything going forward?

should we just add a note in the description column that it's going to be removed

Yeah totally — happy to deprecate those. Will do shortly


Supporting SolrCloud mode with core and shard status

I love this prometheus exporter so much! It works very very great for me.
By the way, I running SolrCloud mode but I cannot find a way to looks for shard healthy status, which node up/down/recovering/etc.
Can you support this metric?
Thank you so much.

Hi, thanks, I will look if i can implement this.


Connection with WTPs and Controller

Is it possible to connect the WTPs with the CPP and the CPP with the legacy switch ,if the CPP has 1 ethernet interface?
we can open some VLANs and therefore have 4 virtual interfaces over one physical on the CPP

In principle yes. I never tried to use it with VLANs.
On 15 May 2017 at 16:17, Georgios Tsiatsios<EMAIL_ADDRESS>wrote:

Is it possible to connect the WTPs with the CPP and the CPP with the
legacy switch ,if the CPP has 1 ethernet interface?
we can open some VLANs and therefore have 4 virtual interfaces over one
physical on the node
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/5g-empower/empower-lvnf-agent/issues/5, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAdzTLujAHiXwnGSr6K4ilb4LqUK1jR_ks5r6F5xgaJpZM4NbMrs
.


drag & drop not working

With macos Sierra , PathFinder (application) version 7.4, macvim 8.0 (110).
drag and drop a file from PathFinder, instead of opening a new tab with the content of the file,
just inserts the filepath (e.g "/tmp/test.java") into the current tab

It was an issue with PathFinder, a new version of PathFinder fixed the problem.


[MetricsAdvisor] Made constructor of DataFeed parameterless

Part of https://github.com/azure/azure-sdk-for-net/issues/16321.
Part of https://github.com/Azure/azure-sdk-for-net/issues/18318.
Fixes https://github.com/Azure/azure-sdk-for-net/issues/17719 by accident.

/azp run net - metricsadvisor - tests


Instant chat message not working properly

Hello, chat message does not work for me.
In fact it did work if i start League Auto Accept while the champ select is already here.
I suppose it is bcause my pc is too slow and it didnt write the message in time.
Is it possible to add for the next version a timer like you already did for the ban et champ selec ?

I assume I could just make it verify that you are connect to chat before sending a chat message, will do it some day

Okay thx !

@hutigh hey, I have made some potential fix, could you compile it and test if it works?

Hello ! I ll test it later today thx !

@hutigh hey, I have made some potential fix, could you compile it and test if it works?

Sorry i'm pretty noob about it, i got an error while trying to compile it i got this error idk how to change the kit sdk : Code NETSDK1045 can you tell me how do i change the .net sdk kit to 7.0 or 6.0 ?

I think you just need to download the SDK which you probably don't have installed? idk but try it
https://dotnet.microsoft.com/en-us/download/visual-studio-sdks
The app is currently using .NET 7 so get both x64 and x86 for .NET 7 and try again

Any update?

Hello sorry I didn't had  time to play recently I ll try this weekend!

Hello ! i finally got some time to try it, i see my chat in the champ select but my 2 friends with me in the champ select didnt saw it

Thanks for testing.
I think they probably joined champ select late? I thought about it before and I'm not sure what to do. I can either make it wait for everyone to join which might cause the message to be delayed too long or I can make it resend it every time someone rejoins but that sounds too spammy. Should I just add 100ms fixed delay and call it a day?

yeah i think adding 100 ms should be just fine tell me when its done and i ll try it

Done. I tested it with a friend and he could see the message. Let me know if you are satisfied with it.

Thx you ! I can't test it until next week end (this weekend I will be at lec final yeah !) I'll let you know if it works and I'll close the thread !

I compiled it myself and in 99% of the cases it works. Just for people with really bad connection not. Thanks for the improvement

Since op has not been responding and I assume the fix is working well enough I'll just close it as is


Intro TH new

@NTaylorMullen @danroth27

Looks like the diff is too large for GitHub to showcase. Could you zip-up the generated bits and put them somewhere for me to review in HTML format?

Seems like there's a ton of code included in the project which isn't referrenced all that much. There a reason for all of it?

:watch:

:watc

Looks good to me. :shipit: once this comment is addressed and @danroth27 signs off :smile:

@danroth27 ready for you to sign off.

Go ahead and push. If it's good enough for @NTaylorMullen it's good enough for me.

Fixes #125


Update readme

Moved Documentation & Examples sub-heading higher up the page
Added Contributors section to README
Renamed Usage sub-heading to Quick Usage Example

Pull Request Test Coverage Report for Build 70

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

Totals

Change from base Build 61:
0.0%

Covered Lines:
528

Relevant Lines:
577

💛  - Coveralls


feature request: add END_LINE magic constant

this is allowed to implement, run spec for any line inside it, very handful for big tests;
  it "bla" do
     ...
     ...
     ...
  end

I've thought about this a bit, and maybe we can introduce a couple of entities inside macros. Right now we have @type, but we can add @def and @block. So for example:
Example with @def:
def foo
  # a.k.a. __method__
<EMAIL_ADDRESS># => "foo"
end

foo

Example with @block:
def foo
<EMAIL_ADDRESS># => 7
<EMAIL_ADDRESS># => 9
  yield
end

foo do
  1
end

The compiler has this information, only that it doesn't exposes it. Of course this info is available at compile-time, not at runtime (info about block contents and locations is lost), so it makes sense to expose this at compile-time too, in macros. With this we can implement __method__, running specs between given line numbers (I thought about implementing this by making it be a macro, and then we have access to the block with &block, but I realized the same should be possible without using a macro definition), and probably some other nice things.

I think the suggested solution would apply to my post Error: "undefined local variable or method '__method__'" at https://groups.google.com/d/msg/crystal-lang/wNcMzNw1bLg/Z27sz4VJAAAJ

Also #1582

After implementing @def and @block I remembered that spec's it captures the block, so there's no @block to access (because it's not a compile-time block that will be inlined) so to correctly implement it with what we have right now we'd have to turn it into a macro, which brings some more complications.
I think __END_LINE__ is a good simple solution. For calls without a block it will be the same as __LINE__, but with a block it will be the last line of the block.

I'll probably add @def too, but I won't add @block because it doesn't seem to be very useful now, and can bring some confusion inside macros compared to &block and yield.

After the next version is released I will add this feature to spec.
I saw how rspec does it and it seems to only do it for it blocks. For example, if I specify a line that is in the middle of a describe block but in no particular it block then I think it runs the closest previous spec. Is that expected behaviour? In any case I think the most useful case is when you are debugging a particular spec but don't want to remember its first line.


[TensorIR] Enhance PointerTypeRewrite

Our current GPU backends sometimes can generate the following code:
void kernel(float* global_data, float* res) {
    float data[4];
    ((float4*)data)[0] = ((float4*)global_data)[0];
     res[0] = data[0] + data[1] + data[2] + data[3];
}

Here we are doing vectorized store into a local memory then do a scalarized access from the local memory. While such pattern can be supported by compilers like cuda, it is not always the most robust path. Additionally, backends like vulkan, webgpu comes with the restriction that we can only have one kind of access to the data, resulting the code to be scalarized.
A better version of the code is as follows, when the following conditions are met:

We detect we have aligned vector store into data
For all read access to the data, they are scalar access, and can be represented by extract_element(Shuffle), this can be done by running modular analysis and check that all access have coeff that is multiple of the vector lane

When such condition are met, we can still rewrite data into the vector type, and rewrite all read access of the data to extract_element(represented via shuffle).
void new_kernel(float4* global_data, float* res) {
    float4 vec_data[1];
    vec_data[0] = global_data[0];
    // access the elements of the vector.
    res[0] = vec_data[0].x + vec_data[0].y + vec_data[0].z + vec_data[0].w;
}

This is a pretty common case in GPU programs so it would be desirable to upgrade  PointerTypeRewrite to support such cases. This would likely help to robustify our overall support as the new_kernel is also what most programmer will write by handle
Action Items

[ ] Enhance the var analysis to decouple buffer read and write/opaque access,

The read part needs to contain a coeff entry via Modular analysis that summarizes whether rewrite to extract_element is possible
For all read access, do Modular analysis to get coeff, so we know that index = x * coeff + offset
If coeff % vector_lane == 0, then it means we can rewrite all scalar access to vector access vec_data[(index - offset % veclane) // vec_lane].extract_element(offset % veclane). We can keep GCD of all coeff access to trigger the condition
To keep backward compatibility, we can put it behind a flag rewrite_scalar_read_to_vector_shuffle

[ ] Rewrite scalar read access into extract_element when we find such cases.
[ ] Support extract_element shuffle pattern (more general shuffle pattern is OK but extract_element is the main one) in codegen , then turn rewrite_scalar_read_to_vector_shuffle on for the cases

cc @Hzfengsy @junrushao @quic-sanirudh @shingjan

I'm interested in working on this

I really like this idea, and it will simplify a lot of the codegen-time analysis that occurs to efficiently handle Ramp nodes.
Benefits:

This mimics what is already done as part of the LLVM codegen (link), in case of a
This would remove the need for the CodeGenC generation of vector loads from Ramp indices(link), since any vectorizable loads would already be expressed with scalar indices.  After the enhanced PointerTypeRewrite, bool can_vector_load would always be false in CodeGenC::VisitExpr_(const BufferLoadNode*).
The special handling that Vulkan has (link could be removed entirely, since the lowered module would already satisfy the constraint buffer_load.dtype == buffer_load.buffer.dtype.

Potential breakages

Vector types that whose struct representation contains padding.  For example, LLVM uses 32-bits to store int8x3, with padding in bits 24-31.  For these types, the address ((int8x3*)ptr)[N] is not the same as ((int8*)ptr)[3*N].
Padded vectorization of AllocateConst.  When a buffer's dtype is vectorized, the underlying data in AllocateConst is kept as-is.
DLTensor arguments.  The size of a DLTensor is computed (link) assuming that elements are padded out to the nearest byte (e.g. padding a 1-bit DataType::Bool() out to 8 bits), but does not introduce any padding between vectorized elements (e.g. padding a 24-bit DataType::Int(8, 3) to 32 bits).

Likely follow-ups:

Any remaining scalar read access would be updated from ((float*)ptr)[index] to extract_element(((float4*)ptr)[index/4], index%4), and the ((float4*)ptr)[index/4] would likely benefit from CSE.  Currrently, the tir.transform.CommonSubexprElimTIR pass forbids removal of BufferLoad statements entirely (link), as their contents may change.  We will probably want to loosen up this restriction, to allow CSE elimination of input buffers.


Display description from GraphQL schema in comments

The contents of "description" fields should be converted into comments. Add wrapping and whitespace to make these easier to read.

Looks done to me.


fix(alerting): add condition results to ntfy

show the condition results in ntfy notifications like below.

Codecov Report
All modified lines are covered by tests :white_check_mark:

Comparison is base (0402bdb) 80.25% compared to head (0195d55) 80.36%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #582      +/-   ##
==========================================
+ Coverage   80.25%   80.36%   +0.10%
==========================================
  Files          56       56
  Lines        4477     4486       +9
==========================================
+ Hits         3593     3605      +12
+ Misses        702      700       -2
+ Partials      182      181       -1

Files
Coverage Δ

alerting/provider/ntfy/ntfy.go
65.07% <100.00%> (+5.82%)
:arrow_up:

... and 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Thank you for the contribution!


Support for Minecraft 1.20.4

Request Description
Hey,
Was just wondering if you could add support for minecraft 1.20.4?
Much appreciated.
Minecraft Version
1.20.4

That's already covered by Mouse Wheelie

Sorry, I forgot to mention if it could be for Neoforge? Mouse wheelie only supports Fabric.

Ah, right. Unfortunately backporting is liable to be a pain with the changes I've made for 1.21. I'll look into it if I get time, but I'm short on that right now so I can't promise anything.


fix(framework): InputLeftElement and padding

Give appropiate padding to the input element
depending on the width of InputLeftElement

Have you tested this in app? So stuff doesnt mess up


accounting: time-based balance snapshots

During the recent call with several lnd users, the notion of being able to obtain a balance "snapshot" at any period of time came up. I think the use case here is going back and looking at the historical flow of funds within a channel.
FWIW, this is already available in lnd as is, since we store the entire "revocation" chain. However this isn't indexed by "time" and instead uses an auto-incrementing sequence number of each new channel state. The only notion of time for the revocation chain is the absolute expiation heights on any HTLCs possibly present on the commitment transaction.
Alternatively, this can be done by polling lnd periodically, or having faraday use the streaming balance RPCs to update its own local state.

+1 on time-based balance snapshots.
Related feature that would be useful is time-based balance snapshots for each Bitcoin address (on-chain wallet and channel funding txn outputs). I'm not sure how this fits into the Harmony Reporting Standard, so might be a different report entirely.


ffs: data size

I'm trying to see how we can report on the size and total size of data stored in ffs. The only size I see for data associated with a cid is CidInfo.Hot.Size. Seems like in the case of cold-only storage, we won't have any size information. Should we move Size up a level to CidInfo and always set it?

Looks like the size calculation actually comes from storing the data in hot storage. Looking to see if there is something similar for cold

Appears that filecoin data size can be queried after a storage deal is made by calling Lotus' ClientGetDealInfo. We're not really exposing this method, but we can potentially, or at least call it internally to get the size information to set on some struct.

Hmm, this is a bit complicated and I don't have enough information to figure out the answer. Size can be reported by Lotus' GetDealInfo, but that might be for a single "piece". If a stored cid is split across multiple pieces, you'd have to sum them to get a total size. Additionally, PG supports storing data in multiple miners, each with a separate deal, thus separate (but hopefully equal) answers for Size of each piece deal. I'm only about 85% confident of that analysis, though ;)

Ok, because of some recent updates to Lotus and Powergate (https://github.com/textileio/powergate/pull/414/files), size is now available on both hot and cold storge info.
Just some context, @jsign, should we expect the size reported in hot and cold info to be the same or different? If they're different, why? (that is fine, just want to be sure we understand why/how to use the data)

Ok, because of some recent updates to Lotus and Powergate (https://github.com/textileio/powergate/pull/414/files), size is now available on both hot and cold storge info.
Just some context, @jsign, should we expect the size reported in hot and cold info to be the same or different? If they're different, why? (that is fine, just want to be sure we understand why/how to use the data)

I think we should expect both to be the same now. It should be the cumulative size of the DAG.

Ok thanks, that's easy.


Fix BOLOS_UX_IO_RESET message when app boots

The BOLOS_UX_IO_RESET allows to avoid unwanted PIN lock when USB layer is reinited
A defect in the OS make the very first os_ux message ignored

Re-send the message allows to ensure the BOLOS_UX_IO_RESET message is received by the UX layer.
Changes include

[x] Bugfix (non-breaking change that solves an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)
[ ] Tests
[ ] Documentation
[ ] Other (for changes that might not fit in any category)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 60.19%. Comparing base (86054b8) to head (462fdb7).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #722   +/-   ##
=======================================
  Coverage   60.19%   60.19%
=======================================
  Files          12       12
  Lines        1726     1726
=======================================
  Hits         1039     1039
  Misses        687      687

Flag
Coverage Δ

unittests
60.19% <ø> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Update user_session xpath

https://github.com/3scale/porta/pull/3295 may be breaking some tests, since the href is being remove from the header icons.
-      a.PopNavigation-trigger.u-toggler class=('is-current' if active_menu?(:topmenu, :help)) href="#documentation-menu" title="Documentation"
+      a.PopNavigation-trigger title="Documentation"

@josemigallas LGTM! we will wait with the merge until it is present in alpha to avoid problems in teststuite

@mganisin we can merge this one


Clicking outside does not emulate native behaviour of dropdowns

Are you reporting a bug?
Yes

Provide the steps to reproduce the issue, e.g.:

Focus on the input field
Type c, and wait for suggestions to appear
While suggestions are shown click (or hover) on some link on the page

Observed behaviour: You are navigated to the respective page link (can hover the link)
Expected behaviour: Suggestions should be closed, navigation should not happen (can't hover the link)

Perhaps, it should be a component's prop to opt-in/opt-out of that, but it feels like the default behaviour should be emulating the native one.

it feels like the default behaviour should be emulating the native one.

What's the reason for that?
Could you provide a pointer to accessibility guidelines that suggest that this is the recommended behavior for autosuggest components?
Out of interest, I checked the autosuggests on google.com, facebook.com, and bing.com, and they all allow you to navigate to other links on the page.
react-autocomplete and react-select components also behave similarly.

Checkout any dropdown for example even on github.

First action clicks it away, and only on the second you can press anything. That's because you want people to easily close dropdown not forcing them to try and find an empty space on the screen (which plus might be visually empty, but there might be some other element there)

I agree that this behavior is the norm for dropdowns, but not convinced this should be the case for autosuggests (where input field exists).
Can you find any autosuggest examples online that implement this behavior?

Closing as agreed with Misha.
For those who wants a workaround here's a rough idea how you can do that

        <ReactAutosuggest/>

and style it as a sibling
.containerOpen + .autosuggestBackdrop


bug: cute types are not resolved

@bot.on.callback_query()
async def buy(cb: CallbackQuery):
    await cb.message.unwrap().only().unwrap() # Message, must be MessageCute

Full example to reproduce: https://gist.github.com/timoniq/832b9ba70d2c13f492cb81220e0f8db6

resolved in https://github.com/timoniq/telegrinder/commit/11922289801b7dcb77435064f20c86bda59e6718


No delete func on the site

i kinda of created a blog post but can't find where to delete it on the site....would love to make a pr to fix tthis

@Lftobs please do!

@Lftobs please do!

i don't have write access ... you will have to invite me to the repo

@Lftobs you did a great job. Thank you for contributing!

@Lftobs, you are now listed as a contributor on this project. However, when I look up the contributors via contrib.rocks, I only see myself. My thinking is that it's due to your name being rm -rf haha


Adjust system index alignment in the debugger

Current behavior
Currently the system index and system name in the debugger are all inline.
Proposed improvement
It would be better if the system index were either center or right aligned and the system name were left aligned, so that all of the system names and system indices are flush to a vertical line (imaginary, not visible) going between them. This will make things easier to visually parse.

Once flex options are introduced we should be able to do this much easier


Draft proposal for a o11y index page

This is a draft proposal for #19 that I'd like to get some opinions on our next meeting.
What I'd like to discuss:

What people think about this table-driven layout?

Are we "collecting" materials about Observability only, or do we want project-specific as well? E.g., my questions at #18 were strongly related to Prometheus, do we want to provide the answer here or to leave it to the Prometheus team?

How do we accept community contributions to this index?

We probably would like to avoid pages that provide the same information
We probably would like to avoid pages with a non-inclusive language
Do we accept a new entry in which the contributor is not the author? E.g., me adding Richard's o11y 101 video.

We need a written guide with all the requirements to accept a new entry.
Signed-off-by: arthursens<EMAIL_ADDRESS>

Added :slightly_smiling_face:

Revisiting this, is this draft still active?  Have discussions stalled?

No response on this PR. Closing for now. Let's reopen if there is renewed interest.


Adding Symfony HttpClient support

Symfony released a new HttpClient component, with support of asynchronous requests and Http2 support. This PR add an adapter for this new HttpClient.
The big change is the  support of HTTP2, the examples/03-http-client.php has been adapted to show the performances gain: up to 4x times faster in Http2 🎉 . Of course, this example is an ideal case, where all requests are made on the same server supporting Http2, but it's promising.
Note: Symfony client does not support PSR-7 interfaces natively, then you need to provide some factories (PSR-17)to instantiate PSR-7 responses, https://github.com/Nyholm/http-factory is suggested

Current status: Travis does not support HTTP/2 example, seems related to curl… I'm trying to see if Sf client is able to fallback to HTTP/1 automatically…

🎉
It took me a while to realize that the bug was in the Guzzle HttpClient!
Symfony performs a smart check to ensure that HTTP/2 is supported (and fallback if necessary), but there is nothing similar in Guzzle, the responsibility seems currently delegated to user according to the HTTP/2 thread.
I copied logic from SF to Guzzle client 😄


fix(table): show toolbar if has custom content

Closes #
Summary

show table toolbar when there is custom content but no other options

Change List (commits, features, bugs, etc)

Table Toolbar

Acceptance Test (how to verify the PR)

table with custom content in toolbar and no other options
toolbar should be rendered

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Submissions: pushes to "submissions/submission" branch trigger builds of "submission" branch

What I've done:

[x] created a private Git repository
[x] copied the deploy key and added it to the repository
[x] created a branch called submission
[x] downloaded a starter kit
[x] committed and pushed the starter kit
[x] waited for more than 10 minutes

Here's the problem:
We noticed that the submissions page now mentions the branch name, and my teammate @pink-snow got curious how the "submissions/submission" branch would be displayed. So here's what I did:

pushed a commit to the "submission" branch. It built fine, and passed the tests;
branched off "submission" into "submissions/submission", and added another commit;
pushed "submissions/submission".

In a few minutes, a new entry appeared on the site, but it mentions the same SHA-1 and commit message as the commit on the "submission" branch. When I pushed more commits to "submissions/submission", it created more builds of the "submission" branch.
Please don't ban us for this ._.

Thanks for reporting this! After a brief investigation we came to conclusion that you've discovered a bug in TeamCity which we use as a CI tool. Kudos for that :)


Compiling extension statically into PHP fails

PHP Driver version or file name 5.3.0
Client operating system macOS 10.14.1
PHP version 7.2.12
Microsoft ODBC Driver version <IP_ADDRESS>
Problem description
Currently it seems not possible to compile the sqlsrv and pdo_sqlsrv extensions statically into PHP. Compilation fails because the extensions main header files php_pdo_sqlsrv.h and php_sqlsrv.h are included from C files, while both files currently extensively include C++ specific code.
The main header files should be splitted to allow static compilation.
Expected behavior and actual behavior
This should work:
phpsrc $ rm configure
phpsrc $ ./buildconf --force
phpsrc $ ./configure --with-pdo_sqlsrv
phpsrc $ make

Actual behaviour:
In file included from main/internal_functions_cli.c:77:
In file included from php-7.2.12/ext/sqlsrv/php_sqlsrv.h:25:
In file included from php-7.2.12/ext/sqlsrv/shared/core_sqlsrv.h:41:
In file included from php-7.2.12/ext/sqlsrv/shared/FormattedPrint.h:24:
In file included from php-7.2.12/ext/sqlsrv/shared/xplat_winnls.h:24:
In file included from php-7.2.12/ext/sqlsrv/shared/typedefs_for_linux.h:23:
php-7.2.12/ext/sqlsrv/shared/xplat.h:27:10: fatal error: 'iostream' file not found
#include <iostream>
         ^~~~~~~~~~
1 error generated.

Please note, that this specific error is only a symptom.
Repro code or steps to reproduce

Download and extract PHP source
Extract Extension source to ext/
Run rm configure && ./buildconf --force && ./configure --with-pdo_sqlsrv && make

As indicated here, shared objects allow us to distribute the extensions as separate packages. Another advantage is one can update the extensions without rebuilding PHP.

I don't argue that separate builds are beneficial in most cases, I just think statical builds should be possible. I assume splitting the header file should not be to complicated or introduce a maintenance burden.

Is there any strong reason you want to compile our drivers statically into PHP?

As far as I know static building is possible for almost all extensions (except Xdebug, for functionality reasons). It simplifies configuration (file pathes, zend_extension, etc) and ensures compatibility when upgrading PHP. These may not be too strong, but if you don't have any objections I may try to implement this. Or do you see further issues with a C only header file?

@jjeising sure you're more than welcome to try this out. When you're ready, please submit your pull request to the dev branch, not the master branch.

Closing this issue as we have already merged your pull request, @jjeising
Please feel free to reopen if you like


CoreLibs does not properly detect third-party cookies

For instance, when I sign into Github, it detects GH cookie as third-party while it's not.
When referrer is empty, cookie should be considered first-party.

I guess it can be related to the https://github.com/AdguardTeam/CoreLibs/issues/586

core/pull-requests/811


Custom propositions in node wallet

https://github.com/HorizenOfficial/Sidechains-SDK/issues/184
https://horizenlabs.atlassian.net/browse/SCHAINS-421


Postgres UUID type in models

Hi! Sorry if I missed this part in the docs. Is it possible to have a field in my models which would map to a UUID column type in the database? ManagedPropertyType does not seem to have it.

Can you consider reopening/adding this? UUIDs are preferable for primary keys in several instances. Alternatively, if there was an option to specify a custom Postgres data type that would work too. But it currently doesn't seem possible at all, as only the types specified in ManagedType can be used...

Any plan to add uuid type?

please implement this feature, 99% of my PK use UUID

Is there a workaround at the moment, except manually storing UUIDs in a String column? I imagine this has serious performance penalties.

Is there a workaround at the moment, except manually storing UUIDs in a String column? I imagine this has serious performance penalties.

To be honest using uuid instead of auto increment has performance impact.
Can you explain your use case @hero101111 ?

A UUID can be a fire and forget regarding database insertions, using an autoincrement integer requires submitting the request and then querying it back again to get the ID.
It's also just in general a common practise to use UUID primary keys in postgres (depending who you ask).
I think @hero101111 's point was that using a UUID in a string data type instead of the natively supported UUID type adds unnecessary performance penalties, despite the database fully supporting the intended data type.

A UUID can be a fire and forget regarding database insertions, using an autoincrement integer requires submitting the request and then querying it back again to get the ID.

Yed that would be a use case but if I have such a problem with slow inserts I would rather write in a different database.

It's also just in general a common practise to use UUID primary keys in postgres (depending who you ask).

I was one of those people. It is really bad as you have a significant slowdown compared to big int. Now if I want to hide the key itself from the end user I will make additional uuid column use it to query a resorse by it but join through the primary key.

I think @hero101111 's point was that using a UUID in a string data type instead of the natively supported UUID type adds unnecessary performance penalties, despite the database fully supporting the intended data type.

I am not sure what his point is as I never hear it so I would not assume. Depending on the use case different approaches could be taken do you could use uuid.

Is it the case that Aqueduct is opinionated against implementing the UUID data type? I'd like to do a PR to implement this..

@rageycomma I think if you implement and write tests for it @joeconwaystk  will be glad to merge it.

Yes, would be happy for someone to implement a UUID data type. The more data types exposed to the ORM the better.