[Bug]: o keyword in Alfred not giving option to search Obsidian

Bug Description
When I trigger Alfred and type o, followed by a space and my search term, only the default search options are available.
I do see the option to search Obsidian flash by, but it is replaced by all the default search options. Using a different keyword, like oos doesn't change what happens.
I have run the setup, and the od command works to open my daily note. So I think the workflow is installed and mostly configured.
Is this a configuration issue with Alfred? Any advice is appreciated.
Relevant Screenshot

To Reproduce

Trigger Alfred
Type o followed by a space then a search term

Debugging Log
[12:00:53.791] Logging Started...
[12:01:00.416] Shimmering Obsidian[Script Filter] Queuing argument '(null)'
[12:01:00.727] Shimmering Obsidian[Script Filter] Script with argv '(null)' finished
[12:01:00.730] ERROR: Shimmering Obsidian[Script Filter] Code 1: /Users/shawn/Library/Application Support/Alfred/Alfred.alfredpreferences/workflows/user.workflow.AAABE307-D7C7-4EEA-92C5-7E71386A21EB/scripts/o_search.js: execution error: Error: SyntaxError: JSON Parse error: Unexpected EOF (-2700)
Version Information
v2.18
Make sure you have done the following

[X] I have updated to the latest version of this workflow.
[X] I have read the Troubleshooting Documentation.
[X] If the o ... main search does not work, I have tried rerunning osetup and it still does not work.

please provide the information you have skipped (version information, debugging log, screenshot, etc.) – they are there for a reason. I cannot help you without them. ("only default search options" could mean anything, depending on how you set Alfred up)

I have added the requested information. Thanks for getting back to me.

Could you post a screenshot of your Alfred workflow environment variables?

run oversions and paste the result here?

Metadata JSONs: 1/3
Metadata not found. Please run osetup and retry.
INSTALLED VERSION
macOS: 12.1
Obsidian: 0.13.21
Alfred: 4.6.1
Workflow: 2.18.0
Advanced URI Plugin: 1.17.0
Metadata Extractor: 0.2.4
LATEST VERSION
Obsidian: 0.13.19
Obsidian (Insider): 0.13.21
Workflow: 2.18.1
Advanced URI Plugin: 1.17.0
Metadata Extractor: 0.2.4

Metadata JSONs: 1/3
Metadata not found.

Yeah, there seems to be some issue with the metadata writing, for whatever reason. Could you run oupdate ➞ Manually Refresh Metadata and try again?

That fixed it! Thank you. Great workflow - thank you for sharing it.

Thanks 😊


$formdate->year['selected'] not working

$formdate->year['selected'] in FormDate library of codeigniter not working
https://github.com/bcit-ci/CodeIgniter/wiki/FormDate

FormDate is/was a third party tool, developed for a very early version of CodeIgniter.
Any issues with it should be taken up with the author ... this has nothing to do with CodeIgniter itself, thanks.


Custom installation location - non-admin privileges

Hello,
For non admin users with limited privileges on our computers please have a switch that overrides the location of terraform and terragrunt.
https://github.com/warrensbox/terraform-switcher and https://github.com/warrensbox/tgswitch allows this.
And this single tool terraenv would be much better as not all of us have admin privileges

Thanks for the feedback @jimsmith . I will take care of this in next release.

@aaratn will there be a release soon for this feature ?

Hello @aaratn I'd like to work on this if you'll take a pull request for it. I was thinking a config file in ~/.terraenv with something like PATH=~/.terraenv/bin and then pass the PATH in to install, uninstall, use as part of args. Continue to default to /usr/local/bin. Then update the user docs and output messages to use the new feature.


Unable to control old Leviton switches (VRS05/VRS15)

I have several old Leviton switches (and dimmers) around the house - those are of the first generation of Z-Wave from about 10-15 years ago and I've been using them w/o problems with several different controllers and hubs throughout the years.
I'm setting up everything now in Z-Wave JS UI and noticed that I cannot control them either directly from the UI or from HomeAssistant via WebSocket. Moreover, in the UI under "Binary Switch v0" section there's "Current value" field, but there's NO "Target value" at all, which I normally get for other devices with e.g. "Binary Switch v1":

I'm using Zooz 800-series ZST39 LR stick. The problematic devices are Leviton VRS05 and VRS15 switches, but I haven't tried dimmers yet.
They report Z-Wave protocol version 2.9 and Z-Wave chip firmware version "0.4" or "0.5"
I wonder if this is due to "v0" in the "Binary Switch" class:

Or if it's a driver issue...
I would appreciate any help and gladly provide logs or try different options. Thanks in advance!

This feels like an incomplete interview. Please re-interview one of the nodes and make a driver log (loglevel debug) of that.
Ideally clear the logfile before doing that (open it in the store tab, remove everything, save). I just need the zwavejs_... .log file.
Also let me know the # of the node you re-interviewed

@AlCalzone sure, can do! Please give me a bit more time - trying so shuffle bunch of instances around in between day job...
FYI, just an upfront clarification... This is not a one-off user error - I've already mentioned that I have several of such similar devices (VRS05/VRS15) and all of them show this issue. I also have at least 2 separate instances where I've tried it - HomeAssistant running Z-Wave JS + UI addons and a standalone Docker container with the same. I've tried including and excluding these devices multiple of times, as well as re-interviewing them after the fact. Seems like a systematic issue - all attempts lead to the same outcome.
Regardless, I'll get back to you with the logs you've requested.

Ok, I've been able to confirm that it still fails exactly the same with Home Assistant default Z-Wave JS support (so it's not UI specific) and also with a different controller stick (QuickStick Combo HUSBZB-1 = Nortek Z-Wave+Zigbee)
Here are the logs of including/interviewing the device attached. Please let me know if you need anything else. Thanks!
zwave-js-ui-store2.zip

The interview appears to be successful, except that the CC versions cannot be queried, which might be the reason you're not seeing the values to control the device. I'll look into that.

Ok I can reproduce. The timed out version query was the issue.

@AlCalzone I finally got a chance to update and test the fix - everything now works. Thanks a lot!


Provide or link to efficient ways to read the dataset

Example for (distributed) training

Hugging face dataset
tf dataset
kaggle
webdataset / pytorch
Jax example
keras example

Example for (distributed) inference:

Clip batch
efficient b0 inference

Example for statistics computation:

pyspark to compute stats
pyspark to compute incremental input
dask / pandas

Most of those should be very short .py that do not add any dependency to the repo in an examples folder. Some of them will be links to to other places.

https://webdataset.github.io/webdataset/writing/#writing-filters-and-offline-augmentation
https://github.com/webdataset/webdataset#webdataset

pip install webdataset pyyaml

from tqdm import tqdm
import torch
import webdataset as wds
dataset = wds.WebDataset([f"http://the-eye.eu/eleuther_staging/cah/releases/laion400m/{i:05d}.tar" for i in range(16)]).to_tuple("jpg;txt", missing_is_error=False)
dataloader = torch.utils.data.DataLoader(dataset, num_workers=16, batch_size=16)
for _ in tqdm(iter(dataloader)):
    pass

1500 iter/s

https://github.com/rom1504/clip-retrieval/blob/main/clip_retrieval/clip_inference.py#L105 is a good way to read the dataset, add it here as an example

having a package dedicated to reading datasets in various format could be an idea


Button to mark a road segment as blocked

Useful for disaster planning. Ability to mark segments (and nodes?) as blocked to see what effect it has on traffic so I can adapt my road network accordingly.
The button would also allow me to unblock roads that the game created (eg. due to fire or #132).
https://github.com/VictorPhilipp/Cities-Skylines-Traffic-Manager-President-Edition/issues/133

Win extra internets if it adds traffic barrier or some other prop(s) —eg. traffic cones or something— to the segment whilst blocked.

+1, surprised this isn't a vanilla feature. It can be "faked" by banning all vehicles (but I fear that will break bus routes) or turning speed limit to minimum (but it doesn't actually close the road), but those require remembering to set it back when done.
Seems like not too difficult since you're already altering the pathfinding penalty for things like speed limit. Maybe closing just overrides the speed limit to 1km/h?


[Refactor] refactor materialized_views view

Fixes #issue
Use a simple build_row() function row grab a row instead of build the chunk vertically.
What type of PR is this:

[ ] BugFix
[ ] Feature
[ ] Enhancement
[x] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

@Mergifyio backport branch-3.2


[Bug report]

Bug report
medium-zoom's problem
Description

When I click on the mask outside the picture, the mask disappears but the enlarged picture doesn't disappear with it.
Steps to reproduce

Reproduction link / repo: https://docs.leedom.me/handbook/git/sshkey.html#添加公钥

Expected behavior

I hope when I click on the mask outside the picture, the mask and the picture disappear together
Screenshots

第一步

第二步

第三步

Environment info

Browser: Chrome: 99.0.4844.74
Output of vuepress info:

# Paste output of `vuepress info` here
System:
    OS: macOS 11.6
    CPU: (8) x64 Intel(R) Core(TM) i7-7820HQ CPU @ 2.90GHz
  Binaries:
    Node: 16.13.2 - /usr/local/bin/node
    Yarn: 1.22.17 - /usr/local/bin/yarn
    npm: 8.1.2 - /usr/local/bin/npm
  Browsers:
    Chrome: 99.0.4844.74
    Firefox: 89.0.2
    Safari: 14.1.2
  npmPackages:
    @vuepress/core:  2.0.0-beta.36 (2.0.0-beta.33, 2.0.0-beta.4)
    @vuepress/theme-default:  2.0.0-beta.36
    vuepress: ^2.0.0-beta.36 => 2.0.0-beta.36
  npmGlobalPackages:
    vuepress: 0.8.4

https://v2.vuepress.vuejs.org/advanced/architecture.html#overview
The official website is not having any issues.

https://v2.vuepress.vuejs.org/advanced/architecture.html#overview
The official website is not having any issues.

I remove some unnecessary dependencies, then it works well!
thx 4 ur reply!


Recently opened list not showing in taskbar

VSCode Version: 1.48.2
OS Version: Windows 10 1803

Steps to Reproduce:
N/A

Does this issue occur when all extensions are disabled?: N/A
The "recently opened files and folders" list has disappeared from the VS Code taskbar pin. I only noticed this issue recently and have tried uninstalling and reinstalling vs code but to no avail.
Pinned folders were still showing in that list but I decided to remove them to try and fix this issue but it didn't work and now I'm stuck like this.

Can you try to reproduce with our nightly insider builds? You can give our preview releases a try from: https://code.visualstudio.com/insiders/

The insider build seems to be working fine.

I feel like this might be an issue with windows explorer but I'm not sure.

Yeah I don't think we can fix something here. I suggest you file a bug against Windows.

I was able to fix the issue by moving the old config folder (%APPDATA%\Code) and then reinstalling vscode. I'm using a mostly fresh config now, was able to get the important stuff synced back. Still not sure what caused this issue but let me know if you want to look at something from my old config.


remove throw in fetch onError handler

Currently we are throwing an exception in the error handler of the request. This exception cannot be caught in a try-catch block which results in node crashing when an invalid fetch request is made. This PR fixes that issues by removing the error handler which was only being used to throw the error.

Thanks @Karan-Taneja!

This fix has been released in:
https://www.npmjs.com/package/lightstep-tracer/v/0.30.2
https://www.npmjs.com/package/lightstep-tracer/v/0.30.2-no-protobuf


MessageFilter?

In can_api.cpp:395 I see the witty comment
    // TODO: Hmm, what's about SetProperty? (Lucky me, there's nothing to be set)

Does that mean that the RusokuCAN adapter does not support message filter (masks) then?

Hi,
TouCAN converter supports message filter.
Take a look at:  https://www.rusoku.com/storage/app/media/CANAL Library User Guide.pdf
CanalSetFilter11bit
CanalSetFilter29bit

Thanks. Good to know that the hardware can do it. As far as I understand it though, it doesn't seem to be implemented in the USB driver https://github.com/mac-can/RusokuCAN/blob/52f784a4566fb9e4d4bfd2fa856cd29fa11fa2ce/Sources/TouCAN_Driver/TouCAN_USB.c:
int TouCAN_set_filter_std_list_mask(CANUSB_Handle_t hDevice, Filter_Type_TypeDef type, UInt32 list, UInt32 mask) {
    return (int)TOUCAN_ERROR_OFFSET;
}

int TouCAN_set_filter_ext_list_mask(CANUSB_Handle_t hDevice, Filter_Type_TypeDef type, UInt32 list, UInt32 mask) {
    return (int)TOUCAN_ERROR_OFFSET;
}

Do you have a timeline for implementing this?

Message filtering is currently not realized (in Software) for the TouCAN interface, neither for 11-bit identifiers nor for 29-bit identifiers.
It is correctly observed that is expected to be realized with setting filter code and filter mask by properties. But due to a not understood issue with a CAN interface based on an SJA1000 chip, the realization is postponed for now. I don´t want to run into later incompatibilities among the different CAN interfaces (e.g. PEAK-System or Kvaser).

Since over two years have passed without any message filters… did you abandon work on that?

It is postponed since over ten years (I hate this messing around with binary masks for identifier filtering). But is is still on my agenda (maybe with the next major release).

Ok, let's cross fingers. I can't quite understand how you could get along without those filters for so long. since application-level filtering wastes so much power.

🎁


Install unit files in /etc/systemd/system

Description
Install unit files in /etc/systemd/system. systemd gives this location priority.
Issues Resolved
chef-server #572
Check List

[x] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[x] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[x] The CLA has been signed. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

cc @tas50

I'm 👍 on this overall.  One question I have is whether it would be useful for us to remove the old version of the file in /usr/lib if it exists so that for upgrades we don't have a confusing case where we have two on disk.
While some may complain that packages shouldn't install files to /etc/systemd since that is for local configuration, I think the situation is unclear for non-distribution packages that the administrator installs by hand, which is the case for our server-side packages.  And this is cleaner than trying to deal with the /usr/lib vs /lib split.

@stevendanna I'm happy to defer to the general consensus on removing the unused unit file in /usr/lib.  I went back and forth and then decided against it because I thought it would be a piece of code that lingers and never gets removed even when it is likely no longer being used.

:+1: on the change, but I would like to see the old one removed.  Given the long tail on upgrades, it will be years before it's non-functional.

@marcparadise @stevendanna I initially was thinking like y'all - we should remove the old one, but how likely are we going to be on a system that we're installing to the new location with systemd and the old location exists? Wouldn't this just be failed installs?
Now that I think about it, this would be RHEL 7, given it doesn't have the compilation option that necessitates this fix for Ubuntu. So given that, it would be nice to clean up the old files in a situation like that, @yzl. It'll make for easier debugging if we don't have misleading files left on the system, should we ever be in the situation we're debugging this. I didn't follow your logic for why you left the files in place though @yzl, so maybe I'm missing understanding.
Overall, +1 and think this is fine to go down and the clean up can always be a separate PR.

Thank you @stevendanna @marcparadise @mmzyk -- I will merge this and submit a new PR shortly for the cleanup.


Only Tag providers at the end of the workflow

Without doing this, there is a chance that a user can download the Go SDK when it's not actually ready to be used
//cc @jkodroff @guineveresaenger

@stack72 Is this part of what releasebot does, or part of one of the GH workflows?
If it's the former, this issue should go in pulumi/release-bot. If it's the latter, can we assume we mean release.yml?

That sounds exactly the ticket @guineveresaenger

This issue is now fixed for Terraform providers. Native providers will have to add an SDK push tag step to their release workflow Action.


Update api.rxt in startup with labels

Description:
API RXT should be updated at runtime (for labels)

Fix : https://github.com/wso2/carbon-apimgt/pull/5518


macOS

is it possible to get this to work on macOS?

most likely not, because Google issue different keys with Linux and it didn't work on Linux

most likely not, because Google issue different keys with Linux and it didn't work on Linux

Indeed, same as Win32. Different platform, different. I think in the previous issue there was mention of python script that fakes requests, or works offline? That will still work on Mac, presumably (by signing request with Win64 key). May need recompilation, though. Wasm is platform-independent, but python extensions are not, IIRC.

is it possible to get this to work on macOS?

It probably is possible to make this work on macOS
for macOS
the public-key =
-----BEGIN RSA PUBLIC KEY-----
MIIBCgKCAQEArjGlEeMziUYWyJVB+VVjB21YrtygdUXzZ11R8FUNKbOZE7cVc8NSQLQlg8US5akq22U5j45SvMAAsbfcEmJ1i7KWvmsvAsBfgqfE4NUJE7AFLM/p4zTZVx6+KHaHspa3HuTiZYZN0hxvJwG6gAyxuDyQznU9UqaS+Jq+vBIkL1+ZhQnFHv+vomZC/QJMbiH9iVDOKFwFRTz3XSCiygmcfOpCltJgwKdl3wf9Hu0Ddh0vZYvbL4d70pGiSF4AnaaExzytxNZ9zHuVkWZ4ea2wM91kiKNolu4r79rfcq9Wm6Ou5ZA1lUwDEKx575Hf3LrkUlUZM+hK+81QnLu2MoBM0wIDAQAB
-----END RSA PUBLIC KEY-----

and the modulus =
ae31a511e333894616c89541f95563076d58aedca07545f3675d51f0550d29b39913b71573c35240b42583c512e5a92adb65398f8e52bcc000b1b7dc1262758bb296be6b2f02c05f82a7c4e0d50913b0052ccfe9e334d9571ebe287687b296b71ee4e265864dd21c6f2701ba800cb1b83c90ce753d52a692f89abebc12242f5f998509c51effafa26642fd024c6e21fd8950ce285c05453cf75d20a2ca099c7cea4296d260c0a765df07fd1eed03761d2f658bdb2f877bd291a2485e009da684c73cadc4d67dcc7b9591667879adb033dd6488a36896ee2befdadf72af569ba3aee59035954c0310ac79ef91dfdcbae452551933e84afbcd509cbbb632804cd3

but you are still missing the exponent & other values from wasm_src/codelift.cpp + offsets that will need to match / be updated for macOS. but maybe @Satsuoni wants to look into this as its not yet clear to me myself how to get those values.

is it possible to get this to work on macOS?

It probably is possible to make this work on macOS
for macOS v4.10.2198
the public-key =
-----BEGIN RSA PUBLIC KEY-----
MIIBCgKCAQEArYPFwDFydUNnctoCnrh+Zqg8y/6crO1ASrBKHEan6nK8AyXjwNq5kBf5kesODnvvZQvBPZjX0TKe8RbECulhvbVhupfVavOM5o0b9tL1Tz9UhitLvdFckSivix+Mj2tUjWPanqNC3ppAnPhPZLDN8X3AqmwMXekZUJShvZFmypSrH0Qkb6dRTSc0O2guRNuIP4fMwQ8KvgA2NnOf/waR8Nxwwr1wWPjYXC+B22kF/5zmzuc7zzXeunHUYTAZqhv7OUs/poHIXDJ0cZUryJ30w8Q5Ab1gCABgm2k4dEtXQc2BvnCIwTb/wJvVHfUtE0NMa8cMYisTGNW0umvmcvE5yQIDAQAB
-----END RSA PUBLIC KEY-----

and the modulus =
ad83c5c0317275436772da029eb87e66a83ccbfe9caced404ab04a1c46a7ea72bc0325e3c0dab99017f991eb0e0e7bef650bc13d98d7d1329ef116c40ae961bdb561ba97d56af38ce68d1bf6d2f54f3f54862b4bbdd15c9128af8b1f8c8f6b548d63da9ea342de9a409cf84f64b0cdf17dc0aa6c0c5de9195094a1bd9166ca94ab1f44246fa7514d27343b682e44db883f87ccc10f0abe003636739fff0691f0dc70c2bd7058f8d85c2f81db6905ff9ce6cee73bcf35deba71d4613019aa1bfb394b3fa681c85c327471952bc89df4c3c43901bd600800609b6938744b5741cd81be7088c136ffc09bd51df52d13434c6bc70c622b1318d5b4ba6be672f139c9

but you are still missing the exponent & other values from wasm_src/codelift.cpp + offsets that will need to match / be updated for macOS. but maybe @Satsuoni wants to look into this as its not yet clear to me myself how to get those values.

@knobse  thank you for the above, very much appreciated. Hopefully @Satsuoni will take a look and can help further.

I wrote how I got the constants, etc in the Readme. Ghidra works on dylibs, as far as I can tell, so the same procedure should work (system calls will need to be patched separately). All one needs is basic googling skills, and (a lot of ) time.
And since I don't have convenient access to MacOS, I could not work on it even if I wanted to.  And I don't :P Sorry. Once was quite enough.

Hey @Satsuoni maybe i understood it wrong but you kind of emulated the logic in c++. Why isn't it possible todo the same for macOS? you don't need access to a macOS device for that right? or do i not understand it correctly? Also on the HOW there is a lot unclear like how PFUN_180119595 was found + how you got rt value in prfnd.py + what to give it as input + how do see whats the right output & what isn't + same with kn in prfold.py ...etc... if i understood what you did i might be able todo it too but the Readme isn't complete.

Hey @Satsuoni maybe i understood it wrong but you kind of emulated the logic in c++. Why isn't it possible todo the same for macOS? you don't need access to a macOS device for that right? or do i not understand it correctly? Also on the HOW there is a lot unclear like how PFUN_180119595 was found + how you got rt value in prfnd.py + what to give it as input + how do see whats the right output & what isn't + same with kn in prfold.py ...etc... if i understood what you did i might be able todo it too but the Readme isn't complete.

It is possible, it is just tiresome, so I don't want to do that. Please refer to Readme, section "Lazy git" for reason as to why.
Now, as to what is necessary to do that:

One needs to get a CDM dylib from MacOS, preferably x64 one and not ARM one.
Load it into Ghidra, and adjust emulator script to run with Mac executable. The entry points should be similar to Windows, since they are in EME standard. The syscalls are all different though.
Run it in emulator to produce signature, or dig through obfuscated code manually.
Extract the part that generates signature. It is easier to deal with, since output is sent through browser, and so has to be in cleartext. It helps to be able to generate that output, though, so it is easier to work in native OS (MacOs) and have a small program for that.
Trace data back through code to the obfuscated input, then to actual protobuf messsage
Just copy out all the code in between into c++
Spend several weeks (or less, if you have any skills) fixing decompiler output and resolving buffer overlaps and filling in runtime-generated constants from emulator memory dumps.
Figure out ways to manipulate input (see cpp code for details)

While extracting second half of key is nice (makes c++ code much smaller) , it is not strictly necessary.
I admit, I was in a hurry to finish readme... PFUN_180119595 was found by Ghidra, I just added "P" in front of the name. It was called pretty much every second line in the exponentiation function, so it is really hard to miss.
"rt" value in prfind is not used. I forgot to delete it. It was/is the first input. You can easily get it by calculating (output^65537)%n, where output is the last big number in the log file that is located in the same folder. You can also see that it is properly padded for PSS signature.
Prfnd consumes log file, example given in the folder, exponentiation.log  Log file is produced by appropriately adjusted emulator running the exponentiation function.
Prfold  is used on the file produced by Prfnd. kn there is not used... since it is after sys.exit() line. I experimented a lot...
If you find exponentiation function using buffers, (intermediate) input is the very first number.
In general, all described can be done with basic googling skills and reading the code (also done by googling) and Tomer's and mine guides. Heck, I was not even aware of Ghidra's existence three months ago >_> So I googled pretty much every line of python scripts. :)

Very cool. Thanks for the extra info! If i have some time i will see how far i can get with this i think i will have some issues but with this i should be able to get started.

@Satsuoni thanks for the extra info, I will have another read through your Readme and Tomer8007's writeup and see how I get on.

@Satsuoni thanks for the extra info, I will have another read through your Readme and Tomer8007's writeup and see how I get on.


WIP: Implementing the primary DGEList class

EDIT: Do not review / merge.

I just set up the Ubuntu Subsystem on Windows today, and I was grabbing the files from the Windows side but manipulating (and committing) them in the subsystem. Maybe that had something to do with it?
I'll get more traditional dual-booting going in the near future and see if that helps. Actual modification happened in _DGEList.py but nowhere else that I'm aware of.

Ah the Windows platform makes sense.
You ended up changing all lines to CRLF line endings.
I'll definitely read through that, although I don't plan on pushing anything from Windows anymore. I've got a Mac at work and I can dual boot at home. Will get that changed and re-push as soon as I've got a minute.


softmax activation in the last layer

In my custom network builder (tried layered network builder as well) I have this as a last layer
x = dense(x, size=len(env.actions), activation='softmax', bias=True, l2_regularization=regularization, scope='dense0')
But when I print action inside the execute() method of the environment I see that the action parts do not sum to 1. Why could this be? I also tried stacking linear layer and then softmax nonlinearity. I also tried
x = tf.nn.softmax(x) after a linear layer

Are you trying to use the softmax layer to output actions? The observations and actions layers (input and output layers of a typical DNN) are handled automatically by TensorForce depending on the action_space and state_space definition of the environment.
For example, if you have an environment which expects 4 inputs for each observation, and outputs one of 3 discrete actions, then you don't actually put 4 or 3 anywhere in your network builder - you build those into your environment definition. In this case, if we wanted a single hidden layer in our DNN, we'd only use network_builder([dict(type='dense', size=32)]) (1 layer).  That's to say, I don't think you want that final softmax, if I'm understanding correctly. Is that maybe the issue?

@lefnire I need to output a single action that is a vector of certain length where each entry represents the percentage of the resources that are allocated to an asset. So I do not need ti output the discrete action, but rather a vector that sums up to 1. I hope I am explaining it clear enough if not please ask.

Gotcha, I'll let someone else chime in then. Re input/output, the way these RL setups work is on an environment that takes actions based on an observation. Different lingo than supervised/unsupervised ML: an observation is your input/features (it's a vector of numbers), and an action is your output. Because observations/actions are defined in the environment (whether it's a video game or a car or trading bot), TF defers to the environment to provide those bits. That's to say, the input/output layers are handled behind the scenes.
See CartPole for example. When you get past quickstart.py in TF, you're going to be building your own custom OpenAI Gym environment which takes actions and provides observations.
The other bit is that each RL model handles the final layer in the network differently. Some are discrete, some are continuous, some are multimodal distributions, some are softmax, etc etc. Those final layers are in the definition of said model, so TF removes that step from you since it's part of the deal.

Don't have to add anything to @lefnire's explanation -- as explained, network outputs can be considered 'internal embeddings', which are used to parameterize the actual stochastic action policy (which, in turn, is specified by the environment's action requirements). Hence it's not really possible to directly influence the action calculation, but the same model configuration applies to different environments without changes.
You could achieve the type of action you want by doing the softmax 'externally', essentially as part of the environment, while the actions then are the 'logits' of the distribution, arbitrary continuous values. However, it might be equally reasonable to consider this a proper action type, "distribution" or so, which could be integrated into TensorForce.
If this answers the question, feel free to close this issue. :-)


Refactor policy tracing tab and banner

Fix #500
This fixes the issue with the tracing banner not displaying correctly and converts it into more of a "checklist" for the user to get relevant information on what is necessary to achieve tracing.
It will update as services are found and gives a link to update the kubewarden-controller chart
Also the Tracing data needed to be refactored to properly display the rows.
https://github.com/rancher/kubewarden-ui/assets/40806497/7715f709-50fc-4fc4-b884-67b2afa66dc1

@viccuad Nice idea! I added a property to disable the button when either of the two prerequisites haven't been made.
Disabled

Enabled

Also added the cluster id to the trace object to ensure traces with the same name don't appear for multiple clusters.


Uninformative error message for a write without full permission

The following Pulse program fails to verify, which is expected since we only have p0 permission to r0, but need 1 to be able to write:
module PM = Pulse.Main
open Steel.ST.Reference
open Steel.FractionalPermission
open Pulse.Steel.Wrapper

```pulse
fn simple_write (r0: ref nat)
               (#p0:perm)
    requires pts_to r0 p0 0
    ensures pts_to r0 p0 1
{
    r0 := 1;
}

However, the error message we get is
"(Error 228) user tactic failed: Pulse checker failed (see also FStar.Tactics.V2.Derived.fst(447,4-449,16))
(Error) inference failed in building head, no solution for ?n_0"
which is not really helpful

I have a fix for this in my branch, which I'll soon push to main. The error message now looks like this:
ExistsWitness.fst(12,8-12,8): (Error 228) user tactic failed: `Pulse checker failed` (see also FStar.Tactics.V2.Derived.fst(447,4-449,16))
ExistsWitness.fst(18,4-18,11): (Error) Could not infer some implicit arguments: ?.n;
Failed to prove the following goals:
  1. Steel.ST.Reference.pts_to r0 Steel.FractionalPermission.full_perm (FStar.Ghost.reveal ?.n)
The remaining conjuncts in the separation logic context available for use are:
  1. Steel.ST.Reference.pts_to r0 p0 0
The typing context is:
  p0 : Steel.FractionalPermission.perm
  r0 : Steel.ST.Reference.ref Prims.nat

2 errors were reported (see above)

The first error is always reported when a Pulse term fails to check: It's the top-level failure raised by the Pulse tactic.
The second error is the relevant one here:

It points to the source location of the r0 := 1
It says that it could not infer some of the implict arguments, in this case ?.n. That n refers to the name of the bound variable in the primitive Pulse.Steel.Wrapper.write ... so, it's not very meaningful in this case, but for a user-defined Pulse function, the name of the uninferred implicit should help.
It prints the part of the precondition that couldn't be proven, showing the unresolved occurrence of the ?.n
It shows what was available in the context for matching and all the other typing hypotheses in the environment

I guess it could be nicer if we could somehow highlight the difference why the goal couldn't be matched in the context, e.g., highlighting the full_perm vs p0. But that's for another day ...
Thanks for the report!


FR: Swift Package Manager support for MLKit by Firebase team consultants

Feature proposal
I request you that the Firebase team consultants to support MLKit for Swift PM.
MLKit was originally supported as a Firebase product, which is why MLModelDownloader is still in this repo.
As of March 2020, MLKit began to be offered differently from Firebase as a new standalone SDK.
This decision in itself makes sense.
However, MLKit's lack of open source code and support for CocoaPods only has hindered its migration to Swift PM.
We have also observed that the MLKit team is also struggling to support it. https://github.com/googlesamples/mlkit/issues/180
It would have been nice if it were completely independent of Firebase, but like Firebase, it relies on common libraries, including GoogleUtilities, etc., so trying to migrate only Firebase to Swift PM
This would have resulted in duplicate symbols.
I decided to support Swift PM on my own and converted the MLKit pre-built binaries to XCFramework.
However, I am encountering some problems due to the lack of serious support for Swift PM.

Using google-toolbox-for-mac, which is not supported for Swift PM (already removed in Firebase)
Classes derived from Old Protobuf are being used. This is already supported in Firebase to use apple/SwiftProtobuf.
It is difficult to convert from pre-built binaries to the architecture for the Arm64 simulator, and simulator builds cannot be performed on Apple Silicon Macs. This is a known problem with installations with CocoaPods.

As you know, the first and second problems have already been resolved in Firebase due to the efforts of the Firebase team to support SwiftPM.
This issue is a significant degradation of the development experience not only for MLKit users, but also for developers who use Firebase in conjunction with MLKit.
We appreciate Firebase's diligent efforts to support developers on a daily basis and ask that you consider taking action to address this issue.

@d-date Sorry about the trouble and thanks for the request.  I'm going to close as out of scope. While we can help and have helped other Google teams add Swift PM support, the requirement needs to be driven through those teams.
cc: @bcdj


Fix to send parser to prettier API properly

What does this implement/fix? Explain your changes.
With the implementation of prettier, I didn't pass the parser into the options properly so it can't beautify anything other than javascript.  This fixes so it works with other supported languages.
...
Does this close any currently open issues?
Closes #2070
...
Any other comments?
...
Checklist
Check all those that are applicable and complete.

[X] Merged with latest master branch
[X] Regenerate documentation with npm run docs
[ ] Add change details to CHANGELOG.md under "Next" section
[X] Added examples for testing to examples/ directory
[X] Travis CI passes (Mac support)
[X] AppVeyor passes (Windows support)

Awesome, thanks @szeck87! @Glavin001 when about are you planning to ship this in a new release? I can't wait 😄

Sometime in the next couple days. I usually like to wait and try to review and/or merge some more PRs into the next release.

Published to v0.32.2


Remove meta viewport for now.

Temporary fix for #4.
Right now, the website is not mobile-friendly, so
meta viewport makes things worse. At least it will be usable on
mobiles, albeit a bit clumsy.
@EIREXE could you please add this?

Hey man, I appreciate your contribution, however this is the wrong repository, I forgot to publish it under the ModulousSmash group, but the current code now resides here: https://github.com/ModulousSmash/ModulousNext
No need to make a pull request there, I will put this in the next update along with tags.

You should put that in the README and the subtitle.


git clone https:xxx error

remote: Counting objects: 3964, done.
remote: Compressing objects: 100% (3138/3138), done.
error: RPC failed; result=18, HTTP code = 200
fatal: The remote end hung up unexpectedly
fatal: early EOFs:   2% (80/3964)
fatal: index-pack failed

Thanks your feedback!

See http://stackoverflow.com/questions/17683295/git-bash-error-rpc-failed-result-18-htp-code-200b-1kib-s for solutions.

Close for now but open for continuing discussion.


Add game mode info to CommanderEntry

And also parse the groups' name as part of the GameMode enum. This merge request depends on #96

After thinking about it a bit more, I think it makes more sense to move the game_mode field to the LogState struct.. Because the next step would be to also include the current ship info which can change during a session.


For a multi-branch Pipeline, the tag cannot be triggered automatically

For some use cases, we build the container image once a new git tag is created. According to the feature of Jenkins, you will see the log output like this:
No automatic build triggered for v0.0.2
See the details from the changelog of github-branch-source-plugin
Open question

How to support triggering a new tag automatically?

See https://github.com/kubesphere/ks-devops/issues/598


Stringify indentifier for elementClick event

Hey @topicus,
Apologies for my previous poor commit. My version pushed seems to be missing a set of quotes around one of the identifiers. I have amended the elementClick code to be what it was originally intended to be.

No worries. Thanks again.


WFLY-17308 - Update resteasy-spring to 3.0.0.Final

The resteasy-spring is moved from preview-ee back to standard-ee.

I've placed a hold on this as a build of this server has errors when testing with https://github.com/resteasy/resteasy-spring/tree/main/testsuite/integration-tests-spring-web/deployment

@jamezp I'll check the problem.

After several days working with James, here is a possible solution: https://github.com/resteasy/resteasy-spring/pull/67/files

@jamezp Thanks for checking!

Thanks @liweinan

relative PR for reference: https://github.com/resteasy/resteasy-spring/pull/71


Keep a (short, n=5) list of previous replies.

Fixes (as an enhancement) #747
There is now a List<FtpReply> LastReplies with max 5 elements.
The previous LastReply still exists - it is fed from LastReplies[0]. If you want the next reply previous to that, you need LastReplies[1] and so on.
Especially useful to access intermediate messages on download (RETR) and upload (STOR) commands

My actual intent was: Do we do this like this, and keep LastReply as well, so as not to break code for existing users?

Closed in order to re-open a cleaner version.


Adding bilibili into the companies list of superset users in README

bilibili is the most popular video sharing website in China(NASDAQ: BILI), we are using superset in our search system and recommend system. Superset is a powerful analytical tool, thanks for making the great project open-sourced!

Codecov Report

Merging #8591 into master will increase coverage by 5.2%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #8591      +/-   ##
=========================================
+ Coverage    66.5%   71.7%    +5.2%
=========================================
  Files         450     118     -332
  Lines       23120   12587   -10533
  Branches     2453       0    -2453
=========================================
- Hits        15376    9026    -6350
+ Misses       7595    3561    -4034
+ Partials      149       0     -149

Impacted Files
Coverage Δ

...sets/src/explore/components/ExploreChartHeader.jsx

superset/assets/src/components/Checkbox.jsx

...perset/assets/src/dashboard/util/getEmptyLayout.js

...ts/src/dashboard/util/getComponentWidthFromDrop.js

.../assets/src/dashboard/reducers/dashboardFilters.js

...rc/explore/components/controls/AnnotationLayer.jsx

superset/assets/src/welcome/DashboardTable.jsx

...erset/assets/src/dashboard/util/injectCustomCss.js

...explore/components/controls/ColorPickerControl.jsx

...ets/src/dashboard/components/dnd/DragDroppable.jsx

... and 322 more

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 3d0c0f5...a60e10b. Read the comment docs.


support wasm32-unknown-emscripten

Related to #189
To support wasm32-unknown-emscripten no need to do anything special like c api. It just works as is, because emscripten emulates OpenGL ES 3.0, so just disabling web path for it works fine (tested on vange-rs project)

Tested also in hello-triangle example of wgpu (https://github.com/gfx-rs/wgpu/pull/2233)


Fix/graphviz

Fixes a bug in graphviz whereby actors were not quoted

Coverage increased (+7.3%) to 76.667% when pulling 5d1c5258d6dcba25ea163612fdc6d9341ad42235 on fix/graphviz into 8821d89be9e3d5443ac4c369768554a74a1fcc22 on master.


Compatibility with client-side JS

I've been using phx-update="ignore" in a LiveComponent so I can use an element that should be entirely handled by client-side JS. The script just replaces the textarea with a WYSIWYG editor. It's something like this:
<div phx-update="ignore">
    <link rel="stylesheet" href="editor.css">
    <textarea id="editor"></textarea>
    <script src="editor.js"></script>
</div>

When I try the same thing with Surface, the element just goes blank.
Is there any way to get this working?
(To be honest I'm not sure I should be putting the script and link tags inside the ignored node... Any suggestions are welcome!)
Thanks in advance!

Hey, @izzues!
It's hard to tell what's going on by just seeing the snippet above. Can you provide a sample project reproducing the issue? If there's no syntax error, it should work the same way as in LiveView. Also, you may need to add an id to the div with the phx-update. I believe LV requires it.

To be honest I'm not sure I should be putting the script and link tags inside the ignored node...

I believe they should probably go in the <head> section.

Sorry for taking your time, you're right, all I had to do was add an ID to the element. Thank you so much!


Tasks build up

Instead of building Tasks in one shot, like,
cdp.Tasks{
		cdp.Navigate(`https://www.google.com`),
		cdp.Sleep(2 * time.Second),
		cdp.WaitVisible(`#hplogo`, cdp.ByID),
		cdp.SendKeys(`#lst-ib`, q+"\n", cdp.ByID),
		cdp.WaitVisible(`#res`, cdp.ByID),
		cdp.Text(sel, res),
		cdp.Click(sel),
		cdp.Sleep(2 * time.Second),
		cdp.WaitVisible(`#footer`, cdp.ByQuery),
		cdp.WaitNotVisible(`div.v-middle > div.la-ball-clip-rotate`, cdp.ByQuery),
		cdp.Location(site),
		cdp.Screenshot(`#testimonials`, &buf, cdp.ByID),
		cdp.ActionFunc(func(context.Context, cdptypes.Handler) error {
			return ioutil.WriteFile("testimonials.png", buf, 0644)
		}),

how to build Tasks gradually, like
cdp.Tasks{
		cdp.Navigate(`https://www.google.com`),
}
if NeedExtraWait {
cdp.Tasks{
		cdp.Sleep(2 * time.Second),
}
}
if WaitForMore {
cdp.Tasks{
		cdp.WaitVisible(`#hplogo`, cdp.ByID),
}
}

how to put such code together?
Sorry I did read around https://godoc.org/github.com/knq/chromedp#Tasks several times, but still unable to figure that myself.
Please bear with me and help. Thx.

Tasks is defined as the following:
// Tasks is a sequential list of Actions that can be used as a single Action.
type Tasks []Action

As such, you can simply compose your Tasks by using the standard slice operations:
t := Tasks{}
if cond {
    t = append(t, cdp.Navigate(`http://example.com`))
}
err := c.Run(context, t)


Big search query flows under the search and filter icons

Describe the bug
A big search query in the input box shows under the icons.
Please refer to the screenshots below.
Screenshots

Desktop (please complete the following information):

OS: Linux
Browser: Chrome

Proposal
The font size of the query box is 130%. We can dynamically increase/decrease the font size based on the length of the query user types.
Sample
let fontSize = Math.floor(130/fontSizeFactor);

May i try to solve this issue ??

I don't think that this is a good alternative as it seems hacky and will not be asthetically pleasing.

How about a ui change .. can we make the text field a little smaller than those buttons ?

The icons inside the input field look good and minimal.
I was thinking about something like in this codepen

The icons inside the input field look good and minimal.
I was thinking about something like in this codepen

Yeah this will work perfectly i guess .. may i attempt to go for this one ??

@makkoncept  I was thinking the same :)

@thegitone23 Go ahead!

@thegitone23 Nice work man!
I would still prefer that you remove the border around the text box and try to go for the horizontal scroll on text-overflow. The reason being that it is not aesthetically pleasing and kind breaks the flow. You are almost there. All you have to do is that break the search bar into text box and icons and then blend them together so that they look like one element but in reality its textbox + search area at exact same location without any internal borders separating them.
Hope it helps

@thegitone23 Nice work man!
I would still prefer that you remove the border around the text box and try to go for the horizontal scroll on text-overflow. The reason being that it is not aesthetically pleasing and kind breaks the flow. You are almost there. All you have to do is that break the search bar into text box and icons and then blend them together so that they look like one element but in reality its textbox + search area at exact same location without any internal borders separating them.
Hope it helps

Thanks for the reply first of all, the text box and search area are i think already of the same width but that is the reason causing the overflow, i think javascript would be required to achieve the required effect, can you please suggest a way to do it without using javascript or otherwise should i add a new js file to do that for me, i am a little confused about where should i put the js to do stuff for me.

@thegitone23
You don't need javascript for this. It can be done solely in CSS. I did quick CSS editing directly in chrome:

This is just to give idea that only CSS is required

Thanks for the reply Sir, going straight to the issue the original effects can't be achieved as the serach area doesn't have the same focus properties as the input box, though I can set tabindex for search area but still the autofocus won't work for the search area and I don't think just using css I would be abel to apply borders and shadows to the search area when text area will have focus set to true. I hope I am not in the wrong direction here but I think the problem can simply be solved if I could set properties of search area when focus property of input box is set to true.

Hi @thegitone23, thanks for working on this. I am sorry, I didn't get the email notification of this and therefore was not able to take part in the discussion.
Just like @codingonHP said, you are very close. I have checked your PR locally and can see that the problem is due to the focus animation present in the input field. I would suggest you remove the focus animation altogether from the input field. Just specify outline: none there so that the default blue highlight did not occur on focus.

As far as giving the feedback to the user of the input field getting in focus is concerned, we can look for other ways to do it. After trying it myself, I even think that it might not even be necessary.

Thanks for the reply , I got your point, actually my initial solution was quite like the one you mentioned but the feedback issue was bothering me, anyways I will now work it out using your approach.

Hi @thegitone23, thanks for working on this. I am sorry, I didn't get the email notification of this and therefore was not able to take part in the discussion.
Just like @codingonHP said, you are very close. I have checked your PR locally and can see that the problem is due to the focus animation present in the input field. I would suggest you remove the focus animation altogether from the input field. Just specify outline: none there so that the default blue highlight did not occur on focus.

As far as giving the feedback to the user of the input field getting in focus is concerned, we can look for other ways to do it. After trying it myself, I even think that it might not even be necessary.

Thanks for the reply , I got your point, actually my initial solution was quite like the one you mentioned but the feedback issue was bothering me, anyways I will now work it out using your approach.

Hi @thegitone23, thanks for working on this. I am sorry, I didn't get the email notification of this and therefore was not able to take part in the discussion.
Just like @codingonHP said, you are very close. I have checked your PR locally and can see that the problem is due to the focus animation present in the input field. I would suggest you remove the focus animation altogether from the input field. Just specify outline: none there so that the default blue highlight did not occur on focus.

As far as giving the feedback to the user of the input field getting in focus is concerned, we can look for other ways to do it. After trying it myself, I even think that it might not even be necessary.

I have submitted the PR, i hope it does the job this time.


WIP: chore: Add test to close channel with dlc

The test is reproducing the bug reported in #543.

closing this PR as it's tests are covered in other PRs.


Check if secret key is less than 256 bits

Addresses #39

Thanks, @allantl for the PR! Looks good to me. It's merged already. Will release this now.

closes #39


Key error in reading holicity dataset - visualize_exported_dataset notebook

Hi,
How are you?
Thanks so much for the help! I appreciate it.
When I play with the "visualize_exported_dataset" notebook and go to the holicity section, it seem to have the data loading issue shown below.
After printing out the keys of loaded "data1", it seems only have the following keys:
dict_keys(['image', 'junctions', 'junction_map', 'line_map', 'heatmap', 'valid_mask', 'file_key'])

KeyError                                  Traceback (most recent call last)
 in
6
7 # Reference data
----> 8 ref_img = data1['ref_image'].numpy().squeeze()
9 ref_junc = data1['ref_junctions'].numpy()
10 ref_line_map = data1['ref_line_map'].numpy()
KeyError: 'ref_image'
Any advise is appreciated.
Thanks so much and have a good weekend!

Hi,
The 'type' parameter of the dataset config file controls what will the dataloader return. When 'type=single', it returns only 'image', when 'type=paired_desc', it returns 'ref_image' and 'target_image'.
So in your case you need to use the 'paired_desc' option.

Thanks for the quick advise!
I was able to fix it by changing "type" to "paired_desc" and enabled "augmentation".
Have a good week!

Just wondering what are the "object" exactly.

ref_img = data1['ref_image'].numpy().squeeze()

ref_junc = data1['ref_junctions'].numpy()

ref_line_map = data1['ref_line_map'].numpy()

ref_line_points = data1['ref_line_points'].numpy()

"ref_img": are these the detected junction points?

"ref_junc": are these the subset of junction points that can be used to form lines ( with lines drawn )

"ref_line_map": not sure what it contain

"ref_line_points": are these the ground-truth junction points and associated lines?

Thanks so much!

Hi,

"ref_img": the original image.
"ref_junc": the detected junction points in this image.
"ref_line_map": a 2D matrix where cell (i, j) contains a 1 if there is a line between junctions i and j.
"ref_line_points": sampling of points along every ground truth line, which is used in the descriptor training.

The corresponding "target_*" objects are the same, but in the warped view, yes.

Thanks, @rpautrat!
Have you thought about point distribution issue in some of the images?

You mean if the points used in the descriptor training are not well spread across the image? This probably happens from time to time, but I don't think that it is a big issue. Or what did you mean exactly?


Fix lock (key - "kmm-asset-lock") not obtained

After failing to create a control plane, the kmm terminates cleanly (with error) but lock is NOT cleaned up OR can't be re-obtained:
Apr 19 16:03:19 ip-172-31-4-32.eu-west-2.compute.internal docker[24422]: time="2017-04-19T16:03:19Z" level=info msg="Saving:"/etc/kubernetes/scheduler.conf""
Apr 19 16:03:19 ip-172-31-4-32.eu-west-2.compute.internal docker[24422]: [apiclient] Created API client, waiting for the control plane to become ready
Job for kmm.service failed because the control process exited with error code.
See "systemctl status kmm.service" and "journalctl -xe" for details.
Apr 19 16:04:37 ip-172-31-4-32.eu-west-2.compute.internal systemd[1]: kmm.service: Start-pre operation timed out. Terminating.
Apr 19 16:04:37 ip-172-31-4-32.eu-west-2.compute.internal systemd[1]: kmm.service: Control process exited, code=exited status=2
Apr 19 16:04:37 ip-172-31-4-32.eu-west-2.compute.internal systemd[1]: Failed to start Kubernetes Multi-master.
Apr 19 16:04:37 ip-172-31-4-32.eu-west-2.compute.internal systemd[1]: kmm.service: Unit entered failed state.
Apr 19 16:04:37 ip-172-31-4-32.eu-west-2.compute.internal systemd[1]: kmm.service: Failed with result 'exit-code'.
Apr 19 16:04:47 ip-172-31-4-32.eu-west-2.compute.internal systemd[1]: kmm.service: Service hold-off time over, scheduling restart.
Apr 19 16:04:47 ip-172-31-4-32.eu-west-2.compute.internal systemd[1]: Stopped Kubernetes Multi-master.
Apr 19 16:04:47 ip-172-31-4-32.eu-west-2.compute.internal systemd[1]: Starting Kubernetes Multi-master...
Apr 19 16:04:47 ip-172-31-4-32.eu-west-2.compute.internal docker[25216]: time="2017-04-19T16:04:47Z" level=info msg="Cloud Provider Initialized ["aws"]"
Apr 19 16:04:47 ip-172-31-4-32.eu-west-2.compute.internal docker[25216]: time="2017-04-19T16:04:47Z" level=info msg="Getting "kmm-asset-key" key value..."
Apr 19 16:04:47 ip-172-31-4-32.eu-west-2.compute.internal docker[25216]: time="2017-04-19T16:04:47Z" level=info msg="Assets not present in etcd...
Apr 19 16:04:47 ip-172-31-4-32.eu-west-2.compute.internal docker[25216]: "
Apr 19 16:04:47 ip-172-31-4-32.eu-west-2.compute.internal docker[25216]: time="2017-04-19T16:04:47Z" level=info msg="Transaction didn't succeed - we didn't create lock!"
Apr 19 16:04:47 ip-172-31-4-32.eu-west-2.compute.internal docker[25216]: time="2017-04-19T16:04:47Z" level=info msg="Lock allready created..."
Apr 19 16:04:47 ip-172-31-4-32.eu-west-2.compute.internal docker[25216]: time="2017-04-19T16:04:47Z" level=info msg="Getting "kmm-asset-lock" key value..."
Apr 19 16:04:47 ip-172-31-4-32.eu-west-2.compute.internal docker[25216]: time="2017-04-19T16:04:47Z" level=info msg="Lock (key - "kmm-asset-lock") not obtained, TTL exists:"2017-04-19T16:05:13Z""
Apr 19 16:05:07 ip-172-31-4-32.eu-west-2.compute.internal docker[25216]: time="2017-04-19T16:05:07Z" level=info msg="Getting "kmm-asset-key" key value..."
Apr 19 16:05:07 ip-172-31-4-32.eu-west-2.compute.internal docker[25216]: time="2017-04-19T16:05:07Z" level=info msg="Assets not present in etcd...
Apr 19 16:05:08 ip-172-31-4-32.eu-west-2.compute.internal docker[25216]: "
Apr 19 16:05:08 ip-172-31-4-32.eu-west-2.compute.internal docker[25216]: time="2017-04-19T16:05:08Z" level=info msg="Transaction didn't succeed - we didn't create lock!"
Apr 19 16:05:08 ip-172-31-4-32.eu-west-2.compute.internal docker[25216]: time="2017-04-19T16:05:08Z" level=info msg="Lock allready created..."
Apr 19 16:05:08 ip-172-31-4-32.eu-west-2.compute.internal docker[25216]: time="2017-04-19T16:05:08Z" level=info msg="Getting "kmm-asset-lock" key value..."
Apr 19 16:05:08 ip-172-31-4-32.eu-west-2.compute.internal docker[25216]: time="2017-04-19T16:05:08Z" level=info msg="Lock (key - "kmm-asset-lock") not obtained, TTL exists:"2017-04-19T16:05:13Z""

The fix for this has to include tests:
Write interfaces (for etcd and kubeadm) to test kmm.GetAssets().
Tests for this use case (failed control plane, re-obtain lock).

The bug could be fixed by #39 but needs a specific unit or integration test to ensure.


Improve reload experience when using with site generator such as cargo doc

Thanks for making this tool! I was looking into making something similar but it would take me way longer if I were to start from scratch... I found some issues when using this as a live-reload service for cargo doc. Please find the details and fix below
Problem Description

When using a static site generator that nukes the output and regenerates it like cargo doc, the server can send update before the build is done, causing massive flickering and in the worst case, stuck on a 404 page.
(If we pretend the above is fixed), location.reload() can still cause slight flickering because assets are not cached
When server restarts, the client doesn't automatically reconnect

The issue(s) can be observed by running live-server target/doc and cargo watch -x doc side-by-side (requires cargo-watch)

Issue screenshot

Fix Description

Client: Probe the page using a query parameter ?reload and ensure it can load before triggering the reload. The server injects a meta tag if the page is a success response for reload (to distinguish it from 404 page)
Client: Instead of calling location.reload(), the client:

refreshes the cached assets
reload the page inside an iframe
swap the head and body into the main document
Because the assets are cached, the reload experience is smoother

Client: Automatically try to reconnect after disconnecting
CLI/Lib: There are probably some edge cases/special scripts that doesn't work with the soft reload approach, so I add hard_reload()/--hard to turn the optimization off and call location.reload() instead. This still ensures the page can be loaded so (1) is not an issue.

Testing

Updated test.rs
Tested soft and hard reload in Chrome, Edge and Firefox. Didn't test in Safari because I don't have a mac

Screenshots:

Hard Reload (notice the slight flicker)

Hot Reload

Other
Not related to everything above, I also found an issue where links in the output of cargo doc were broken because ../ behaves differently when the path is <origin>/live_server vs. <origin>/live_server/ or <origin>/live_server/index.html.
so I added a check to redirect directories that don't end with slash. This might break scenarios that rely on this bug

Thanks for your contribution. However I'm quite busy lately and can't spare extra time for reviewing currently. I will review it after I finish my work in about one or two week(s) :)

I also just tested it on my laptop with 11th gen i5. It's pretty smooth for lower-end device as well :)

Thanks for the review, I will update it soonish

LGTM. Now just needs some adjustments to pass the checks.

Should be fixed now


Which tables need to be migrated from the old database to the new database

Should we pull the entire database and run the migration?
Autocomplete values for subjects and languages would not be needed, because we are now using Questioning Authority for those.
Try migrating everything and see what breaks?

Mysqldump,
mysqlimport
rake db:migrate
test

Tables in Prod:
+---------------------------------+
| Tables_in_SSprodRails           |
+---------------------------------+
| bookmarks                       |
| checksum_audit_logs             |
| content_blocks                  |
| domain_terms                    |
| domain_terms_local_authorities  |
| featured_works                  |
| file_download_stats             |
| file_view_stats                 |
| follows                         |
| local_authorities               |
| local_authority_entries         |
| mailboxer_conversation_opt_outs |
| mailboxer_conversations         |
| mailboxer_notifications         |
| mailboxer_receipts              |
| migrate_audits                  |
| proxy_deposit_requests          |
| proxy_deposit_rights            |
| schema_migrations               |
| searches                        |
| single_use_links                |
| subject_local_authority_entries |
| sufia_migration_survey_items    |
| superusers                      |
| tinymce_assets                  |
| trophies                        |
| user_stats                      |
| users                           |
| users_10042015                  |
| version_committers              |
+---------------------------------+

SS 2.0 has some tables that I don't think need anymore. They're all authority tables:
> domain_terms
> domain_terms_local_authorities
> local_authorities
> local_authority_entries
> subject_local_authority_entries

These have all been subsumed into two new tables, created via QA, qa_local_authorities and qa_local_authority_entries which handle authority entries for subjects and languages. The entries table is quite large, ~ .5 million rows, and re-importing everything from is rdf sources takes many hours.
Steps should probably be:

export qa tables
drop target database, i.e. stage-new or qa-new
import current production database, minus the above tables
run rake db:migrate
restore qa tables

FYI if we synchronize the version_committer table after we run the import it overrides the generated version information.  That table would be one I suggest we also drop if we are not going to import the entire database.

Actually, there's already a migration in Scholarsphere to drop those old authority tables. I put it in there and completely forgot about it. So, we could import the entire database. In looking at the QA authority tables on staging, they're empty so we'll have to do an authority import anyway.

WE also may not want to import the tiny mce assests as that would override any changes made on the QA or stage system prioir to migration.  Aka we could override any of @olendorf's changes that he has staged on qa or staging.  I may have done that this morning when I ran a full import on staging.

tinymce_assets hasn't been changed since it was first created, so no problem there.

Using production data, without the tinymce_assets table, work file. Running db:migrate seems to work. I've updated the migration instructions. We'll need to run it through a complete process to make sure it all works.


refactor(model,util)!: remove store channel type

On March 10th Discord has removed store channels (reference).
Closes: #1573

Tests are failing. Should the rust version be updated in this PR?

The PR to the Discord docs has been merged now.


mediapipe 0.10.0 version is not supported ubuntu 18.04 OS?

Have I written custom code (as opposed to using a stock example script provided in MediaPipe)
None
OS Platform and Distribution
Linux ubuntu 18.04
Mobile device if the issue happens on mobile device
No response
Browser and version if the issue happens on browser
No response
Programming Language and version
python
MediaPipe version
0.10.0
Bazel version
No response
Solution
Gesture
Android Studio, NDK, SDK versions (if issue is related to building in Android environment)
No response
Xcode & Tulsi version (if issue is related to building for iOS)
No response
Describe the actual behavior
my development env is ubuntu 18.04 with python 3.7 & pip 22 version
when i try install mediapipe by using pip
there was only 0.8.9 version available

i want to install mediapipe 0.10.0 version
is not support on ubuntu 18.0.4?

Describe the expected behaviour
my development env is ubuntu 18.04 with python 3.7 & pip 22 version
when i try install mediapipe by using pip
there was only 0.8.9 version available

i want to install mediapipe 0.10.0 version
is not support on ubuntu 18.0.4?

Standalone code/steps you may have used to try to get what you need
my development env is ubuntu 18.04 with python 3.7 & pip 22 version
when i try install mediapipe by using pip
there was only 0.8.9 version available

i want to install mediapipe 0.10.0 version
is not support on ubuntu 18.0.4?

Other info / Complete Logs
my development env is ubuntu 18.04 with python 3.7 & pip 22 version
when i try install mediapipe by using pip
there was only 0.8.9 version available

i want to install mediapipe 0.10.0 version
is not support on ubuntu 18.0.4?

@psw7420fusion,
We only support the python version 3.8 to 3.11 as described here. Please build using the supported versions and let us know if this works for you. Thank you!!

@psw7420fusion,
Could you please update the status of the above. Thank you


Integrated terminal terminate running tasks on close

VSCode Version: Code - Insiders 1.14.0-insider (fda3ed38449fea27b97698ad0950c2870bf6afaa, 2017-06-14T05:13:12.994Z)
OS Version: Windows_NT ia32 10.0.15063
Extensions:

Extension
Author
Version

vscode-eslint
dbaeumer
1.2.11

prettier-vscode
esbenp
0.19.0

cpptools
ms-vscode
0.11.4

debugger-for-chrome
msjsdiag
3.1.3

java
redhat
0.5.0

If I use the integrated terminal for executing watch scripts, or any background running tasks, shouldn't it terminate automatically upon closing vscode?
Steps to Reproduce:
For simplicity, create a CRA app and open it in vscode.

Open the Integrated Terminal and execute npm start
Close the vscode window and try to access http://localhost:3000.

It doesn't have to be a CRA app. Try running express or a http server instance.

Duplicate https://github.com/Microsoft/vscode/issues/26807


Expire items in bulk to reduce chance of races

Expire items to reduce chances of backup racing with file expiration:
https://github.com/home-assistant/core/issues/67026

Codecov Report

Merging #274 (8dc6316) into main (7ea15e8) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #274      +/-   ##
==========================================
+ Coverage   91.82%   91.84%   +0.02%
==========================================
  Files          18       18
  Lines        2018     2024       +6
==========================================
+ Hits         1853     1859       +6
  Misses        165      165

Impacted Files
Coverage Δ

google_nest_sdm/event_media.py
95.07% <100.00%> (+0.05%)
:arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Rotate a matrix clockwise and anti-clockwise | C++

Given a matrix, I will display it after rotating it clockwise and anticlockwise by 90 degrees.
Input:
1 2 3
4 5 6
7 8 9
Output:
Clockwise:
7 4 1
8 5 2
9 6 3
Anti-clockwise:
3 6 9
2 5 8
1 4 7

I want to solve this problem in C++

should I contribute

\
They are not considering dsa,cp,algorithm repository :(
all the pr will not be considered in hacktoberfest but your contribution will be for lifetime, this will help many students to practice dsa .
Thank you for your contribution :)
you can contribute to other repo


bug: drawer-end makes horizontal scrollbar permanently visible.

What version of daisyUI are you using?
v4.4.19
Which browsers are you seeing the problem on?
Chrome
Reproduction URL
https://daisyui.com/components/drawer/#drawer-that-opens-from-right-side-of-page
Describe your issue
Horizontal scrollbar is permanently visible when using drawer drawer-end.
This issue is also visible in the docs.

I'm observing the same behavior in Edge. Firefox renders correctly.
I'm using Next.js 14, tailwindcss 3.3.6, daisyui 4.4.19
I'm using drawer drawer-end in a root layout and that's when this happens. If I change drawer drawer-end to a template or page, the scroll disappears, but the drawer slides from the left, instead of right.
Also, I'm observing some styles not rendering correctly in Edge (oversized components) but correct in Firefox.
This was working fine last week, so this bug must have been introduced in the most recent versions.
I will test with previous releases.

hi! solved the issue by adding overflow-x-hidden tailwind class to <div class="drawer-side"

hi! solved the issue by adding overflow-x-hidden tailwind class to <div class="drawer-side"

In my project, by adding overflow-x-hidden can be solved on the desktop, but on the mobile phone, the entire screen can be swiped left or right through the touch screen.
After returning to 4.4.14, this problem was solved.

If that's the case, you can do md:overflow-x-hidden or something to do this fix on certain screen sizes.

Using version 4.4.19, this error is still occurring. It's not noticeable on macOS Chrome, but on Windows, an unscrollable scrollbar appears

md:overflow-x-hidden

+1


[Metal] Unable to CreateFramebuffer with more than 3 textures in Metal Backend

Create a frame buffer with 3 or more textures bind at the Metal backend.
var depthStencil = factory.CreateTexture(TextureDescription.Texture2D(w, h, 1, 1, PixelFormat.R16_UNorm, TextureUsage.DepthStencil ));
var colorRT  = factory.CreateTexture(TextureDescription.Texture2D(w, h, 1, 1, PixelFormat.R8_G8_B8_A8_UNorm, TextureUsage.RenderTarget | TextureUsage.Sampled));
var normalRT = factory.CreateTexture(TextureDescription.Texture2D(w, h, 1, 1, PixelFormat.R8_G8_B8_A8_UNorm, TextureUsage.RenderTarget | TextureUsage.Sampled));
var depthRT = factory.CreateTexture(TextureDescription.Texture2D(w, h, 1, 1, PixelFormat.R8_G8_B8_A8_UNorm, TextureUsage.RenderTarget | TextureUsage.Sampled));
var mrtFramebuffer = factory.CreateFramebuffer(new FramebufferDescription(depthStencil, colorRT, normalRT, depthRT));

Creating a PipelineDescription using this framebuffer and calling CreateGraphicsPipeline() will throw System.IndexOutOfRangeException.
var mrtPipeline = factory.CreateGraphicsPipeline(pipelineDesc);

For one or two textures, this exception does not seem to occur.
Also, the OpenGL backend does not seem to raise any exceptions.
I have not tried Vulkan  Backend because I could not figure out how to run it on a Mac.

Could you provide a bit more information, especially regarding the other properties of the GraphicsPipelineDescription? I suspect there might be a mismatch in the number of attachments that you have in BlendStateDescription.AttachmentStates as that does not seem to be directly validated anywhere (and would very likely behave differently across backends if the size wasn't correct).


Added deployment architecture note for is km docs.

Purpose
Adding deployment architecture note for iskm.

@Mariangela Added requested changes 003365a7a9118fc557296d64430fef260b05bbe8


 Black would solve these problems, but until we have Black, we can at least lint against them with the flake8-commas extension.
Includes #14532 for Flake8 itself.

Historically, it's only been our style to use them for lists/dicts/sets/etc., not when calling functions.  I personally find it a little distracting to see them today, probably because essentially all the Python code I read doesn't use that convention (so I'm expecting another argument when I see a trailing comma).  I imagine that problem will disappear after some time if we merge this.
It is the PEP8 recommendation (https://www.python.org/dev/peps/pep-0008/#when-to-use-trailing-commas) so maybe we should just switch.

I'm strongly in favor of trailing commas in functions.   They're worth the minor distraction factor to me.

I suppose we can merge the non-lint part of this PR even if the linter change has performance issues (See #14532).

Rebased.

One thing I notice while looking at this is a lot of our Nagios checks duplicate that dictionary mapping numbers to state strings; @mateuszmandera would you be up for doing a quick PR to move that to zulip_tools.py?

Merged everything but the linter (still blocked on performance, sigh) as the series ending with a7d84f6084e536adf2cba4cc4606c32e6cd294aa.

Oops, 69730a78cc3ac655e97761fe1da7348edbc8bfa1; had to rebase one last time.

I suppose since the linter has its own PR, we can close this bulk migration PR (and just move the remaining lint enforcement commit to the end of the switch-to-flake8 PR)?


Adjustments to modularization, additional functionality for .msg

Converted remaining commands into modules
Changed behavior for .msg to display the message in the channel the message was sent in
Added support for user-set time zones (for displaying message sent times in the recipient's local time).

To set a timezone, use .settimezone followed by a numeric offset, e.g. .settimezone -5 or .settimezone -03:30

Also please note that this does require some modifications to the database schema of course, so it should not be pushed live until I get those to you. We should probably include an SQL file for setting up tables at some point, but now didn't seem like a good time as the update would require us to change the 0's from the isPrivate field to the name of a channel anyway.


fix: make the plugin works again

This PR restore the backward compatibility with Clappr 0.2.x and make it work again with latest.
Major changes :

Playback rates are internally handled as Number
Added player external interface getPlaybackRate and setPlaybackRate methods
Removed MEDIACONTROL_PLAYBACKRATE from #3 (supersede by external interface)
upgraded all dependencies to latests (Webpack to v4)
Added code lint npm tasks (npm run lint)
Added local dev. server npm task (npm start)
To compile unminified es5 : npm run build
To compile minified es5 : npm run dist
Added release npm task to run lint and compile all (npm run release)
Updated documentation

@bikegriffith i also added a new rateSuffix option (default to 'x') which is only used when unexpected rate value is set using setPlaybackRate external interface  method.
For example :
var player = new Clappr.Player({/* [...] */});
player.setPlaybackRate(0.456) // will display "0.456x"

var player = new Clappr.Player({playbackRateConfig: {rateSuffix: ' times'}});
player.setPlaybackRate(0.456) // will display "0.456 times"


added description for 'blacklist'

Coverage remained the same at 81.37% when pulling a05101154c432232a31945c55b8ea75aecd2659a on iFreezy:description_blacklist into 75fb332e62f849ec89c87aeb8d48284819aca8b5 on alfa-laboratory:master.


fixed image hover in left corner and client x-y

Thanks - will take a look!
On Fri, Jan 23, 2015 at 2:03 PM, Jerel R Hass<EMAIL_ADDRESS>wrote:

You can view, comment on, or merge this pull request online at:
https://github.com/timruffles/ios-html5-drag-drop-shim/pull/31
Commit Summary

fixed image hover in left corner and client x-y

File Changes

M ios-drag-drop.js
https://github.com/timruffles/ios-html5-drag-drop-shim/pull/31/files#diff-0
(8)

Patch Links:

https://github.com/timruffles/ios-html5-drag-drop-shim/pull/31.patch
https://github.com/timruffles/ios-html5-drag-drop-shim/pull/31.diff

—
Reply to this email directly or view it on GitHub
https://github.com/timruffles/ios-html5-drag-drop-shim/pull/31.

The left corner fix here is no longer needed or even valid, as it was fixed with #36. So the following code should be removed from this patch:
this.dragImage.style["left"] = "-999";
this.dragImage.style["top"] = "-999";

Thanks again for the contribution! If you could take a look at the comment we'd be happy to merge.


I'm assigning this to @kristjanmik since he holds the key to the server. (But if you need a hand, just ping me!)

This is happening via CodeShip right now :)


Add Rails 5.2 and 6.0 on Ruby 2.6 to CI

@santry, I will test our this PR against my Rails 5.2 setup, as I've got prety good test coverage on the logging.

@pboling that sounds great, thanks! let me know how it turns out. once we confirm it works I'll put out a new release of the gem.

@santry It works great :). Thanks!


How to increase the font size in the terminal?

I did not manage to increase the font size. My custom CSS is ignored and I could not find any option that configures it. I suppose there is some magic behind this... So asking.

If you don't care about IE and Edge then you can use css variables:
.terminal {
  --size: 2;
}

or if you need to support IE and Edge you need to set the size in pixels (em don't work):
.terminal, .cmd, .terminal .terminal-output div div, .cmd .prompt {
    font-size: 20px;
    line-height: 24px;
}

there is info abot it in style section of the api reference

Tank you!


How do I add events to the calendar?

I read the front page documentation, the source code and tried the example.
I would like to add an array of dates to the calendar, but
I am unclear how to add a singular event to the calendar.
I am unclear how to add an array of events to the calendar.
Is there not a call for "addEvent" or "addDateToCalendar" ?

You need to use func supplementaryView(shouldDisplayOnDayView dayView: DayView) -> Bool
Within that function you either return true or false if you want the supplementary indicator drawn - a blue circle by default, but you can do whatever you want.  You will need to hold your even information somewhere else, and interrogate it based on the dayView value.
It took me a LONG time to find that if you are making these changes dynamically, you will need to call
calendarView.contentController.refreshPresentedMonth()
before you see the changes, as this method appears to assume that it will set up the supplementary view once at initialisation, and you need to force the refresh

@CarterMiller is right, if you need further help please open a new issue.


Text should be  justified to center

I think if the text will be aligned to the center then it will look more good

Thank you for creating this issue! 🎉 We'll look into it as soon as possible. In the meantime, please make sure to provide all the necessary details and context. If you have any questions or additional information, feel free to add them here. Your contributions are highly appreciated! 😊
You can also check our CONTRIBUTING.md for guidelines on contributing to this project.

Can you please assign this to me.

pls assign me the issue,i will fix the task.

Sir I would really like to work on it please assign me this issue.

I would like to work on this


Error when completing a pledge and funding from a collective

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Click on "Complete My Pledge"
Change "Contribute as" to a collective
Click "donate"
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: macOS
Browser: Firefox 63.0
Version: 10.14 (18A391)

Additional context
Using the normal "Contribute" flow from a collective page works as expected. I only see the above error when completing a pledge.

Thanks for creating an issue @sloria. I'll look into this today.

@sloria Can you provide a link to the pledge completion form so I can get the id of order in the database?

@HipsterBrown sure thing. https://opencollective.com/pledges/16923

@sloria I just tried it on our staging area and was able to complete the pledge. Can you share a full screenshot of the pledge form in your view?
I think it may have been a race condition between updating the pre-commit collective with a host and the completing of the pledge.

@HipsterBrown I'm not able to reproduce the issue since I already went through the normal contribution flow on pre-commit's collective.
I didn't change anything else on the form except for the two fields in my screenshots above ("Contribute As" and "Credit Card").

While working with @piamancini, we figured out the issue. I'm working on a patch now.

@sloria The fix has been deployed to production. Thanks again for creating this report. 👍

@HipsterBrown Thanks for the quick resolution!


Fungi frog maze V2

Hi! I'm launching the v2 of my game, with some new assets, new levels and improved mechanics. 🐸

Thanks @LucasHT22 :)


chore: move basic lemmas to basic files

synchronize with https://github.com/leanprover-community/mathlib/pull/17882
Make disjoint_singleton @[simp 1100] because the linter complains it is not in simpNF (because of disjoint_singleton_left and disjoint_singleton_right).

synchronize with leanprover-community/mathlib#17882

Can you please update the commit SHA's recorded at the top of all these files to indicate that you are bumping them to match mathlib3 counterparts.
(That's what you are doing, right?)


fix(Dropdown): fix arrow keys after no results

Fixes #1275
The Dropdown attempts to set the selectedIndex (highlighted item) to undefined after encountering the "No results." message.
This PR resets the selectedIndex to 0 after every search change.  This is inline with SUI core and was our original intention for this component.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@0372af2). Click here to learn what that means.

@@            Coverage Diff            @@
##             master    #1279   +/-   ##
=========================================
  Coverage          ?   95.91%
=========================================
  Files             ?      883
  Lines             ?     4926
  Branches          ?        0
=========================================
  Hits              ?     4725
  Misses            ?      201
  Partials          ?        0

Impacted Files
Coverage Δ

src/modules/Dropdown/Dropdown.js
100% <ø> (ø)

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 0372af2...9ab1199. Read the comment docs.

Released in<EMAIL_ADDRESS>


Merge service worker client into "environment settings object"

I guess we'll need to keep our APIs to it, as they're snapshots.

A service worker client is now an environment settings object. Closing.


Tutorial?

So I tried to do the exact steps but their are times where it won't let me do it.
Example, I cannot do git clone because it said "access denied"

Try running this:
git clone https://github.com/pxlsspace/Pxls

adding on to this perhaps if a dev could find the time to do it, a video tutorial on getting a instance running would be nice. especially for people like me who dont have much experience running things like this and need a little extra help.

As mentioned in #423, this will track a video tutorial while #423 will be for a text-based instructional/tutorial.

@FlyingSixtySix Can you rename this issue to "Video Tutorial Creation" or something similar?


how to use custom results.yaml?

I am trying to use the new "custom results.yaml" feature from PR #1488 for improving rpminspect testing, in https://github.com/fedora-ci/rpminspect-runner/pull/80 and https://github.com/fedora-ci/rpminspect-pipeline/pull/64 . But I am confused how this should be used, as this whole PR talks about test level results.yaml.
As a reference, look at this rpminspect run, especially its artifact dir. You see that it has a plan-level rpminspect/execute/results.yaml:
/license:
    result: pass
    log:
      - data/license/output.txt
    duration: 00:05:30
# [... more tests ...]

But in the test-level artifacts dir there is no results.yaml (not in data/ either, that's empty). This also makes sense as the plan-level results.yaml from above already enumerates the tests in the plan and their results.
The above PRs rework rpminspect to create a custom results.yaml in $TMT_TEST_DATA, see this run. Its test level artifacts in rpminspect/execute/data/rpminspect/data/ contain my custom results.yaml:
- name: /rpminspect
  result: fail
  log:
    - verbose.log
    - results.json

The format of that is of course a little bogus, as this is a single test and thus should not be a list and assign name:. But that's the format that the documentation describes.
However, this isn't reflected at all in the plan-level results in /rpminspect/execute/results.yaml:
/rpminspect:
    result: fail
    log:
      - data/rpminspect/output.txt
    duration: 00:03:49

i.e. the extra log files are not shown at all. Now, the testing farm still runs tmt 1.16 and thus does not contain this PR yet. But I ran it locally with tmt 1.18.0, and the plan-level result in /var/tmp/tmt/run-001/rpminspect/execute/results.yaml looks the same, except for some new (and uninteresting/empty) ids:
/rpminspect:
    result: fail
    log:
      - data/rpminspect/output.txt
    duration: 00:03:01
    ids:
        id:
        extra-nitrate:
        extra-task:

The documented custom results.yaml format would actually make much more sense at the plan level, as it enumerates the tests. So I tried that, i.e. put it into $TMT_PLAN_DATA. Re-running tmt locally, I get
$ find /var/tmp/tmt/run-002 -name results.yaml
/var/tmp/tmt/run-002/rpminspect/data/results.yaml
/var/tmp/tmt/run-002/rpminspect/execute/results.yaml

the first one is my custom file, but it's just "data":
$ cat /var/tmp/tmt/run-002/rpminspect/data/results.yaml
- name: /rpminspect
  result: fail
  log:
    - data/rpminspect/output.txt
    - verbose.log
    - results.json

and the second one is the actual results, but it's just the standard format, without my custom log files:
$ cat /var/tmp/tmt/run-002/rpminspect/execute/results.yaml
/rpminspect:
    result: fail
    log:
      - data/rpminspect/output.txt
    duration: 00:03:22
    ids:
        id:
        extra-nitrate:
        extra-task:

So, I am none the wiser. It seems to me like #1488 did not change anything user-visible at all both with the documented ${TMT_TEST_DATA}/results.yaml approach nor with putting it into $TMT_PLAN_DATA. How can this be used?

@martinpitt  Have you changed your tests' metadata to contain 'result: custom` ?

Ah, I didn't, thanks! https://tmt.readthedocs.io/en/latest/spec/tests.html#result is a little unclear where I'd set that in our .fmf file. I messed around with that a little, and this works:
+++ b/rpminspect.fmf
@@ -6,6 +6,7 @@ discover:
     - name: /rpminspect
       framework: shell
       test: rpminspect_runner.sh $TASK_ID $PREVIOUS_TAG
+      result: custom
       duration: 720m

I'm now getting the extra files:
$ cat /var/tmp/tmt/run-001/rpminspect/execute/results.yaml
/rpminspect/rpminspect:
    result: fail
    log:
      - data/rpminspect/output.txt
      - verbose.log
      - results.json
    ids:
        id:
        extra-nitrate:
        extra-task:

Now I just need to figure out how to turn that /rpminspect/rpminspect: into just /rpminspect:, but that should be easy :grin:
Thanks, and sorry for the noise! :bow:

There's still that structural bit that I don't understand, though: The docs say, that the custom results.yaml should look like this:
    - name: /test/passing
      result: pass
      log: pass_log
      duration: 00:11:22
      note: good result
    - name: /test/failing
      result: fail
      log: fail_log
      duration: 00:22:33
      note: fail result
    - name: /test/no_keys
      result: pass

But how can that be? This is the result of a single test, how/why does it describe the result of multiple tests? It would seem a lot more logical to me if this was:
result: pass
log: pass_log
duration: 00:11:22
note: good result

I just need to figure out how to turn that /rpminspect/rpminspect: into just /rpminspect:, but that should be easy

FTR, I spent half an hour on this and can't figure it out. I'll just accept it as-is, and hope that it doesn't break anything (it will most probably just look weird)

tmt allows multiple results to be reported (which should exactly rpminspect case) so custom results file is list. Having both list or dict accepted would just make it harder to validate.

OK, thanks! Is there some way to report just a single result, without inventing a new name:? I already tried to drop name: (fails with a KeyError) or set it to / (but then the final name also does, which is weird).

We actually don't want to split it up, as that would make it a lot harder to find the extra log files and the viewer.html link. We really want to report these on the plan level, not test level, but I didn't find a way to do that. So minimizing number of tests and plans to 1 seems to be the best that we can do right now?

We actually don't want to split it up, as that would make it a lot harder to find the extra log files and the viewer.html link.

Is there something tmt can do better to use the multiple results from single test? I'm not sure why it is harder to find logs as their paths should be copied to final results.yaml

Is there some way to report just a single result, without inventing a new name:?

I'm afraid we have a bug here. I'd say that name: / should map to the original name, instead of appending the '/'
E.g. this should report pass /foo but it does report pass /foo/
WDYT @ZelenyMartin ?
discover:
  how: shell
  tests:
  - name: foo
    result: custom
    test: |
      cat > ${TMT_TEST_DATA}/results.yaml <<EOF
        - result: pass
          name: /
      EOF

Is there something tmt can do better to use the multiple results from single test?

No, that's fine. The idea is that we have a custom rpminspect viewer which we want to expose in the Testing Farm/tmt runs for rpminspect. I.e. the generic TF viewer doesn't need to do any effort. At some point we should directly open the custom viewer, but that will take some steps. Until then, we just want to expose that viewer.html as a test artifact, and make it as easy to find as possible.


Make it possible to configure an array of scripts per hook

This way, you don't need a wrapper script to perform several checks for one
hook, e.g.:
{
  ...
  "pre-commit": [
    "git-hooks/pre-commit/no_console_log",
    "gulp lint"
  ],
  ...
}

The scripts are executed in the order that they're defined in package.json.
Configuring just one script remains possible:
{
  ...
  "pre-commit": "gulp lint",
  ...
}

@gtramontina what do you think of this functionality and this patch? I'm still seeing that Node 0.10 causes the build to fail, but this seems unrelated to my patch and I'm not sure how to fix it.

Hey there @fibbers, sorry for the late response…
First of all, thank you for your contribution. Supporting an array of scripts is something we've already discussed in the past (https://github.com/gtramontina/ghooks/issues/25) and decided to stick with either a single entry point script or making use of && to achieve what you're proposing.
We're actually considering deprecating ghooks in favor of husky (as discussed here). I'm, in fact, looking for people to support ghooks at this stage, as I've been really busy lately…
As far as your PR, to provide some feedback, I noticed you've implemented it in a really nice way and I appreciate it very much! From looking at the code, it will, potentially, spawn multiple commands at the same time, and it seems that there's no way to guarantee an order… or ensure that the second command would only run if the first exited with 0 (like && would).

Hi @gtramontina, thanks for your response (and mine is also a bit late now:-)).
Good catch about the order and about the exit-codes of the processes.
I've read the discussion in #25 and understand the decisions that were made. I also see the deprecation notifce in favor of husky.
It was a nice practice for me, creating a pull-request for ghooks, but as far as I'm concerned, this pull-request can be closed. Maybe I'll create a PR for husky then :-)


Begun implementation of RawSockets

Does not compile. It's a modified Udp copy-paste and any inputs/pointers on how proceed would be much appreciated.

I have some good news and some bad news.
The bad news: this raw sockets implementation will never work, and - by looking at the raw sockets API in smoltcp - I don't think it is possible to implement a proper raw sockets API on top of it. The thing is, the so called "raw" smoltcp sockets youve wrapped in embassy-net are not exposing the raw Ethernet MAC addresses as packet source or as a destination address for a raw packet. In other words, there is no way - with the smoltcp API you've wrapped - to send a datagram to a specific MAC address that you - as a user - choose.
The good news (for you) is that two weeks ago, when unentangling the edge-* crates from embedded-nal-async, I've accidentally regressed the DHCP server in a way that it might not currently work with all DHCP clients in the field (need to test that), but it now requires only a UDP socket impl and not a true raw socket impl.
So what you can do is try it with the regular Embassy UDP stack and test it with a few DHCP clients.

All right, thank you. I will look into using Udp instead.