[Full images datamanager] fix image indexing with cv.undistort and implement eval indices logic

Two things:
1) fix new undistorted image shape  not being the same as input
from my understanding in newK, roi = cv2.getOptimalNewCameraMatrix(K, distortion_params, (image.shape[1], image.shape[0]), 0) the roi that is returned is in pixel notations (px_start, py_start, px_end, py_end)
When slicing an image we should instead do image[y : y + h + 1, x : x + w + 1] to preserve the same shape as the input image. Otherwise we crop the last pixel and output_image.shape != input_image.shape :D
2) add eval indices logic from config
There are a few nice eval options in the config that haven't been implemented. I implemented an easy version for eval_num_images_to_sample_from and eval_image_indices which I think are the most important ones for evaluating.

also added option for mipnerf360 style evaluation in colmap dataparser to hold out every n'th image in train set for eval dataset. Useful for comparing with inria eval scripts

Reverted sampling strategies in colmap dataparser in favour of the new PR #2709 which also implements them. This PR fixes 1) the output image size bug after cv.undistort and 2) implements the logic for "eval_image_indices" and "eval_num_images_to_sample_from" in the datamanager config.

How does this PR interact with #2726 ?

isn't it OK that after the undistortion the shape of the image changes?
Maybe the fact that in Mip-NeRF 360 the images are already undistorted, and the second undistortion caused a loss of pixel (prior to PR #2726), lead to this change?

@maturk For the first issue: do you have sources indicating the roi is in (left, upper, right, lower) convention? I think it is in x, y, w, h convention.

The fix should be done upstream: w and h returned by the latest version of icvGetRectangles are off by 1 pixel, see https://github.com/opencv/opencv/issues/24831

@f-dy okay wow this is actually pretty crazy XD Never would I have thought that I would find a bug in OpenCV! Thank you for making the issue and even tracking down the problem in the source code.

I would recommend adding a unit test to test_datamanager.py, so that we're sure to get the thing right in the end.

And btw we should probably not use cv.undistort, which uses bilinear interpolation, and write our own function that calls cv2.remap with INTER_CUBIC instead. bilinear interpolation creates uneven blur in the image, especially for small distortion amounts, i.e with a small k1 you would see rings of blur in the undistorted images. I'll add some code that demonstrates that

So with opencv 4.8+ this is no longer an issue?

Still an issue, even with 4.9.
Hopefully will be patched in the future, following the issue that @f-dy opened.

Any suggestions what we should do with this issue with the cropped pixel?

@maturk Let's wait opencv fix this bug, I believe they will fix it very soon. Once they fix the issue, we can bump up opencv version dependency again.

Any suggestions what we should do with this issue with the cropped pixel?

gate the fix you did with if 4.8<=opencv_version<4.10, since the issue is marked for OpenCV 4.10, and add a unit test, so that we can adjust the gate if they don't fix it in 4.10

OpenCV 4.10 is out now, and with that I'd like to add a small request: Can we loosen up the OpenCV dependency ever so slightly?
Specifically, can we loosen up at least the 4th digit of the version requirement?
The reason being, Conda cannot select package versions down to the 4th digit and if you install COLMAP via Conda/-forge, it will pull in opencv-python automatically.
You can restrict the version via Conda, but not down to the 4th digit.
Pip, when installing Nerfstudio, cannot modify the opencv-python version installed by Conda, and because of how tight the version check is, it will fail to install.
The only way around this right now is to manually modify the pyproject.toml file to loosen the version check, but this of course only works for installations from git, not from pypi.
So basically: If we could loosen up the version constraints for opencv-python at least to the point where it doesn't care for the 4th version digit (though ideally even looser), this would reduce conda woes quite a bit and simplify installation requirements.

OpenCV 4.10 is out now, and with that I'd like to add a small request: Can we loosen up the OpenCV dependency ever so slightly? Specifically, can we loosen up at least the 4th digit of the version requirement? The reason being, Conda cannot select package versions down to the 4th digit and if you install COLMAP via Conda/-forge, it will pull in opencv-python automatically. You can restrict the version via Conda, but not down to the 4th digit. Pip, when installing Nerfstudio, cannot modify the opencv-python version installed by Conda, and because of how tight the version check is, it will fail to install. The only way around this right now is to manually modify the pyproject.toml file to loosen the version check, but this of course only works for installations from git, not from pypi. So basically: If we could loosen up the version constraints for opencv-python at least to the point where it doesn't care for the 4th version digit (though ideally even looser), this would reduce conda woes quite a bit and simplify installation requirements.

Yeah sounds good to me @SharkWipf , can you do a PR?


GH-517 Run orphan fix-up during Orchestration post-initialize

Fixes #517

Applied to main (=2.1) and branch 2.0.


WhateverGreen

You should use Intelfb patcher to fix framebuffer
Serán my config.plist/Devices/Properties
https://github.com/johnnync13/XiaoMi-Pro
https://www.tonymacx86.com/threads/guide-intel-framebuffer-patching-for-mojave.256490/

Thanks a lot. The instruction is very useful.
Well, since current Clover patch method is available on both High Sierra and Mojave, and I suffered from VRAM 7mb after using the parameters in your config.plist/Devices/Properties(10.13.6), I may keep using KextsToPatch for a moment.
I see a lot potential in WhateverGreen.kext. Because it's still in beta version, I may wait until this kext is stable enough.

Yes, of course. This patch send me another member, and It is not correct but It is an example. In my laptop MSI, I get 2048 MB on VRAM.

I think the actual VRAM is still 1536MB. After increased from 1536MB to 2048MB, the OpenGL score in CineBench R15 doesn't change.
By the way, I patched config.plist in the latest two commits, thank you for instruction.


Check AsyncOperation derives and uses for async stuff

So far SceneManager, AssetBundle, Resources uses it but there could be others, check for it

covered pretty much everything, only thing needs to be done is UnityEngine.Networking.UnityWebRequestAsyncOperation
need to disable network stuff or do something about it


medipens don't show correct volume values before injection.

Description

Screenshots

#12875 :trollface:

Seems to be fixed now. Although the margins of the text are still a little off.

all my love went into hypos and injectors


Replace Cassandra with Dynamodb.

Hi, this is awesome project!. So I've got assignment to change Cassandra to AWS DynamoDB. Do you have guideline or any suggestions?

@nut-abctech did you get anywhere with this?

@utsav-upendra not yet. seem like this project discontinue

Amazon now supports Cassandra in their AWS Managed Cassandra Service  - https://aws.amazon.com/ru/mcs/

this project stop maintain. I close this issue


x/tools/internal/lsp/cache: improve method of getting end positions for errors

When we calculate end positions for errors that originate from go/packages. We do some kind of syntax analysis, but some issues arise when adding new characters to the list in internal/lsp/cache/errors.go#L208
if width := bytes.IndexAny(data[offset:], " \n,():;[]"); width > 0 {
	return ...
}

Take the sample program below:
__ main.go __
package main

import "golang.org/whatever"

func main() {
    if y/2 == 10 { return }
}

We will have two errors: (1) "undeclared name: y" and (2) "could not import golang.org/whatever (no package for import golang.org/whatever)".
In the current state, the diagnostic for error 1 will be on "y+2" and the diagnostic for error 2 will be on "golang.org/whatever".
Now, the diagnostic for error 1 should only underline "y" since that is the undeclared name, to fix this I could add "/" to the character list in internal/lsp/cache/errors.go#L208 but then the diagnostic on error 2 would now underline "golang.org" rather than "golang.org/whatever".
I think gopls needs a better way to compute the end positions for type errors.

Superseded by https://github.com/golang/go/issues/42290.


Hass.io tab empty in 0.75

Home Assistant release 0.75 with the issue:
Hass.io tab is empty
Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):
Hass,io
Component/platform:
Hass.io installed on Raspberry Pi 3B+
Description of problem:
After clicking Hass.io in the menu a blank frame shows. Refreshing doesn't resolve it. Checked in IE, Opera and Chrome on WIN10
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

Traceback (if applicable):

Additional information:

This is my first installation, and I have exactly this issue. I tested 32 and 64 bits image on RPi 3b+

I think it is broken, I tried to upgrade and I am getting nothing. Checked config ok, I am trying to reinstall from scratch and all I see are errors.
2018-08-03 15:15:17 ERROR (MainThread) [aiohttp.server] Error handling request
Traceback (most recent call last):
File "/srv/homeassistant/lib/python3.5/site-packages/aiohttp/web_protocol.py", line 230, in data_received
messages, upgraded, tail = self._request_parser.feed_data(data)
File "aiohttp/_http_parser.pyx", line 297, in aiohttp._http_parser.HttpParser.feed_data
aiohttp.http_exceptions.BadStatusLine: invalid HTTP method

Having the same issue on a fresh install, console shows a 404 error for   /frontend_latest/custom-panel.js

Having the same issue on a fresh install, Able to logon to raspberry pi and get Hassio GUI but the Hassio tab produces a blank screen

Same here.
Request URL: http://hassio:8123/frontend_latest/custom-panel.js
Request Method: GET
Status Code: 404 Not Found
Remote Address: 192.168.x.x:8123
Referrer Policy: same-origin

Called by ha-panel-custom.js:
iframeDoc.write();

I'm having the same problem.  New install 32 bit hassio image on pi 3 b+.  All other tabs / pages seem to work but clicking Hass.io on left side shows just blank page.  Have tried Chrome, Safari, IE all not working.
Thanks for the help!!!

Also, trying to run any hassio command from the console results in:
The HTTP request failed with the error: Get http://hassio/<request>: dial tcp <ipaddr>:00: getsockopt: connection refused

I tested Chrome, Edge and Internet Explorer on Windows / Chrome and Opera in Android. Same problem.

Same problem here running a fresh install of Hass.io 0.75 on Raspberry Pi 3B+.
Tried Safari, Chrome, and Firefox on MacOS 10.13.6 and Safari on iOS (iPhone and iPad), all with the same problem.

If you can get to the hassio cli, hassio ha update -o version=0.74.2 would downgrade to a working version.

If only SSH was a default add-on. It seems that everyone that did a fresh install will have to reflash once a new version comes out.

My Hassio tab is working fine on 0.75.0.... I did upgrade from the beta b0... but it's working fine for me and a few minutes ago I just got a new-frontend notification...

I just installed fresh on a pi3B+ and have encountered the same issue.  I checked with chrome and edge, but can not load the hass.io tab.

I was running just fine until the upgrade now the front end no longer
works. Trying a fresh install and will slowly add my configuration until it
breaks again.
On Fri, Aug 3, 2018 at 10:22 PM, Geert de Graaf<EMAIL_ADDRESS>wrote:

Same issue over here.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/issues/15816#issuecomment-410424871,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB4QpjKVJFVo0Giys9-yf4VtIwwU8faFks5uNS-tgaJpZM4Vuqh3
.

Still have no access via a browser, but was just now able to get into the hass.io page on my iphone ha app.

RPi 2, upgraded from working 0.74.2 to 0.75.  Same result, blank Has.io tab.

RPi 2, HassOS 1.7, Supervisor 122.
Went from 0.75.0b0 to b1, lost the panel on my Windows 10 pc, not on my phone (both Chrome).
Upgraded to stable 0.75.0, still the same situation.
Downgraded back to 0.74.2, all was working again.
Upgraded directly to 0.75.0 stable, seemed to be working on my pc, but not on my phone anymore.

Same problem, not on a fresh install. Getting same error as @jvilalta

cogneato or anyone else - would love to downgrade, have SSH installed, but stopped by default.  Anyone know the format to start it from the services dev tool using hassio.addon_start please?

@darylsargent try {"addon":"ssh"}

Anyone know which release here https://github.com/home-assistant/hassos/releases/ works?

@darylsargent try {"addon":"ssh"}

Thanks @JungDev - you pointed me in the right direction - it was {"addon":"core_ssh"}

Same issue and bad luck; my previous version and setup stopped working so I did a fresh install on a new SD card and came up with this issue so no way to restore. Hoping for a fix soon.

At the moment it should be fixed and currently building. Will be released as 0.75.1, you can check the available version here.

How long does it typically take to build?

For those of us who fresh installed, do we need to reflash, or will there be a way to update without being able to click Hass.io menu on the GUI?

I can see a new image on the web, I will test now.

If you check tjorims post above right now 0.75.0 is still avaliable stable.

Hi, Im new to hass.io. I have just bought a Raspberry Pi 3 B+ and flashed the SD-card and put it in my Raspberry. I´m experience the same issue as many others, that the hass.io tab is empty, which hindering me to proceed with the configuration.
What is the procedure? Wait until there is a new version 0.75.1 and redownload a new image and flash it to the SD-card and try again in my Raspberry?
Thanks for support,
Alex

@alela101 That's 1 option and possibly the easiest if you've nothing set-up yet. I think you can also connect a keyboard and screen to the raspi and manually update via the terminal, but I haven't done this myself yet.

@tjorim Thanks for the swift reply. I have a keyboard connected to my rapi but how exactlly do I udpate it vai the terminal?

@alela101 I believe you can do something like this
hassio ha update -o version=0.74.2
I haven't tried it, i'm on the same boat as everyone else on here. Fresh install and waiting for the new version to be available.

0.75.1 is available for raspi's as well now. If you're having troubles updating, join the HA Discord server.

Hassio on HassOS, Raspberry Pi 2, can confirm the issue has not been fixed.

edit, fixed after a reboot!!

Did a fresh install of hassos_rpi2-1.9.img.gz on RPI2B. The issue is still there.

0.75.1 worked for me eventually.  I did have to do a restart as well though before the hass.io tab would work.

0.75.1 Worked for me as well. make sure to reboot HA.

Looks like 0.75.1 is stable release now!  I'm going to try and re-flash to see if it updates to 75.1 now...

Can anyone share how they successfully updated hass.io from 0.75.0 on RPi 3? I don't have SSH enabled, but can attach a screen if that helps.

I can confirm that you have to, even on a complete clean install of 1.9 (0.75.1), shut down and power up the Pi for the tab to work.

Running hassio homeassistant update through ssh CLI worked to update the broken home-assistant 0.75.0 to 0.75.1. Hassio-panel came visible again.
After that also noticed that my old hassio host was broken somehow and couldn't update it. (maybe it was and old resin os? ...)
Anyhow tried a clean install of hassio through 32-bit image from wiki https://github.com/home-assistant/hassos/releases/download/1.9/hassos_rpi3-1.9.img.gz to my RPI3B (32GB card) and got the "20 minute wait" page but then nothing. No activity in disk, no response from the IP. Tried booting eventually, and nothing after that either (except the disk light seems stuck on)

Same state as bobotus, my second flash. 20 minute wait page for over an hour each time.

Changing the SD card from Trancend 32GB class 10 to Samsung 64GB gave a bit better result, I got the home-assistant running at around 14 minutes after clean hassio install.
But then:
Tried to restart the home-assistant through the UI (to get the non-working hassio-panel up), but the home-assistant died. After power-cycling the RPI it doesn't work anymore. hdmi-console output:
EXT4-fs error (device mmcblk0p8): ext4_lookup:1585: inode #528715: comm python3: deleted inode referenced: 528723
Seems like the home-assistant docker images does a lot of operations after install, so the card can easily be corrupted.

SOLVED.
I installed the new image hassos_rpi3-64-1.9.img.gz, put the sd in Rpi 3b+ and nothing happend, after unplug and plug power the installation finish after 20min, but the hass.io still not visible. After Restart ha the hass.io is visible. yeah!

Confirming here, had same issue.  Updated to .0.75.1 via command line but issue remained.
Had to do full power down and reboot of Pi to get the Hass.io panel back.  Restarting host wasn't enough.

@frenck Stop releasing under / un tested code into production and i wouldnt have to comment on the pull request that introduced new bugs and wasted a lot of peoples time.

@thecodingcheese I'm sorry?  What do I have to do with this?
Please get your facts straight before starting pointing fingers. Thank you.


Including files is not possible with WebCompiler

You can't use one of
      {include file="header.html.mjml"}
      <mj-include path="./header.html.mjml" />

Because the template is just loaded via file_get_contents and send to the API.

MJML seems to dont support mj-include-path on api. See https://mjml.io/api/documentation/

Yes, but an idea would e.g. be to write a mj-include-path resolver which does just string-replace with file_get_contents on those files?

Does the mj-include-path just appending the content? 🤔

Looks like it:
https://github.com/mjmlio/mjml/blob/50b08513b7a651c234829abfde254f106a62c859/packages/mjml-cli/src/helpers/fileContext.js
https://github.com/mjmlio/mjml/blob/c7195156e0cb0db5b429af6044df7dc7f1aa847e/packages/mjml-parser-xml/src/index.js


Early un-escape of \

https://github.com/Ferruslogic/PhotonJSON/blob/6563d62c3c7c0ee4e6264d3638d3e5fe037891ed/PhotonJSON.livecodescript#L233
If un-escaped here, then there is a possibility of converting \\n or \\t improperly for example since the \\ will be changed to \.  Suggest placing it before or inside the _unicodeEscapes function but the existence of \\u will still need to be considered.

Thanks a lot. This problem is already solved in Bob Hall's FastJSON library. So I brought that part of the code over here and adapted it. Using the ECMA-404 standard, we have that \\ n is the string \ n.

Glad to help out.


Refactoring unit testing

Currently, the matrix comparisons in unit test are done with a specific function. This process does not allow to precisely locate where the unit test fail.
We need to refactor all the unit test similarly to 8deb921d9

Closing with #91


Create objects using go-git

Instead of calling out to the git command twice to hash the missing files and then check them out, do the same thing natively using go-git.
I am not sure if this is possible, but it probably should be.

Objects are now created fully in go and file metadata is updated from the index directly. This is an immense speed up to the object creation compared to doing it via the git commandline.


Resque fails unless the before_child_exit hook is defined

Tested with Resque 2.0, the workers will not start unless a before_child_exit hook has been defined.
gems/resque-2.0.0/lib/resque/worker.rb:647:in `run_hook': undefined method `empty?' for nil:NilClass (NoMethodError)
    from gems/resque-multi-job-forks-0.5.0/lib/resque-multi-job-forks.rb:124:in `release_fork'
    from gems/resque-multi-job-forks-0.5.0/lib/resque-multi-job-forks.rb:105:in `release_and_exit!'
    from gems/resque-multi-job-forks-0.5.0/lib/resque-multi-job-forks.rb:35:in `work_with_multi_job_forks'
    from gems/resque-2.0.0/lib/resque/tasks.rb:20:in `block (2 levels) in <top (required)>'
    from gems/rake-12.3.2/lib/rake/task.rb:273:in `block in execute'
    from gems/rake-12.3.2/lib/rake/task.rb:273:in `each'
    from gems/rake-12.3.2/lib/rake/task.rb:273:in `execute'

I can reproduce this exception in the test suite by commenting out the hook in the test helper: https://github.com/stulentsev/resque-multi-job-forks/blob/master/test/helper.rb#L67

Fixed in #20


Check for web3 global variable, use it if available

This is going to sound like wallet-ception, but bear with me, it might just be crazy enough to be cool.
I'll illustrate by describing a user experience:
I'm a Mist/MetaMask user, and I hear about some cool new feature that was added to MyEtherWallet.  It's a voting tool for some important upcoming fork or something, I donno.  I click a link to MyEtherWallet.
Normally at this point, MEW would show a "create a new account" screen, but in this alt-world, MEW instead does a little web3 check before assuming no wallet, like this:
if (typeof web3 !=== 'undefined') {
  var userAccount = web3.eth.accounts[0]
  initWith( userAccount );
} else {
  // Start MEW as usual...
}

When it identified a global web3 object as present, rather than suggesting creating a new account, it just proceeds to the normal, full-featured MEW interface, with that account available to it.
The hardest part would probably be swapping out the key-signing you're currently doing, since the web3 api actually makes it a lot easier, any call to sendTransaction prompts a user signature request within Mist or MetaMask (you don't need to touch private keys with this API).
As a MetaMask/Mist user, this is awesome, because I just got to enjoy the MEW UI without having been reliant on it as my previous key manager.
For MEW, this might seem more dubious, since you damn near support any account type already.  Well to this, I have a few things to say:

Your UI is awesome and stands on its own, so rather than worrying about undermining your key signing, maybe this is a cool way to elevate your "ethereum home page" status.
This alleviates load on your backend, since Mist/MM users bring their own blockchains.
Both MetaMask and Mist are planning on adding more and stranger key-signing methods, hardware wallets, weird contract wallets, and if you embrace the web3, you get all that for free in your UI, you leverage the work we do in key signing. #BetterTogether

Anyways, that's my pitch.  I know it's pretty far off your current core feature path, so I'd understand if you shave it off as not a priority, but as a Mist/MM user, I think it would be cool.

@flyswatter HIIIIIIIIIIIIIII!
(not urgent at all)
So we're making lists and stuff and I was wondering if you envisioned the broadcast via Metamask or via MEW? (remember the token sales. remember the blame game. 😂)
We currently broadcast Ledger & Trezors via MEW so it would basically be the same flow, accept that instead of confirming on your HW wallet, you would confirm in Metamask.

We don't really have a way right now for passing the rawTx back to the web UI (although I proposed a way to a new EIP over here).
So the way I had imagined it was definitely that MetaMask would be responsible for broadcasting these txs.  This also aligns with how MetaMask allows users to select their own providers, and uses it, so if MetaMask + MEW were used together, the user could even use their own node at the same time.
I could see this being useful in situations like:

A user likes performance, so they use geth or parity locally
They use MetaMask b/c they use some dapps
They prefer a MEW UI for some service (Maybe ENS, for example).

In that case, it could be a pretty seamless experience if you just defaulted to using the injected web3.eth.sendTransaction method.

Btw always fun to think about!  I know no rush, we both have plenty of more important things as well, I'm sure, but it's cool how interoperable this ecosystem lends itself to being.


Fix spelling mistake

Summary of the Pull Request
Fixes a spelling mistake in the src/settings-web/src/components/GeneralSettings.tsx file

References
It was already noticed, & suggested by another contributor, in https://github.com/microsoft/PowerToys/pull/494, but they forgot to commit that suggestion.

PR Checklist

[X] CLA signed. If not, go over here and sign the CLA
[X] Tests added/passed

Detailed Description of the Pull Request / Additional comments
Changes Chose to Choose

Validation Steps Performed
Manual

Hi @ItzLevvie
thank you very much!


Add -RepeatHeader to Format-Table to enable repeating header for each screen full

PR Summary
When using a screen reader or just getting the output of a table with lots of rows, the header is no longer on the screen and the columns may no longer make sense without the context.  This PR adds a -RepeatHeader switch to Format-Table to enable re-outputting the header after every screen full (minus 1 row).  Expectation is that the user is piping the output to a pager (e.g. less) which uses the bottom row for pager information.  I followed the AutoSize parameter as the way to get the parameter from the cmdlet into the deep formatting object where it's needed.
Fix https://github.com/PowerShell/PowerShell/issues/8455
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Change is not breaking
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

User-facing changes

[ ] Not Applicable
OR
[x] User-facing https://github.com/PowerShell/PowerShell-Docs/pull/3443 (https://github.com/PowerShell/PowerShell/blob/master/.github/CONTRIBUTING.md#pull-request---submission)

[ ] Issue filed - Issue link:

Testing - New and feature

[ ] Not Applicable or can only be tested interactively
OR
[x] Make sure you've added a new test if existing tests do not effectively test the code changed

[ ] Add [feature] if the change is significant or affects feature tests

Many of the codefactor issues are not related to this PR or I'm adopting the current conventions in the source code for consistency but breaks CodeFactor rules.


Official blobs as emoji

Google has released new blob emoji as stickers, could remake them (or somehow obtain) as emoji instead. https://www.androidpolice.com/2021/06/15/gboards-amazing-emoji-kitchen-will-soon-show-the-perfect-emoji-mashup-at-the-perfect-time/

I know, but I don't think these could be used:

They're not emojis, but mashup stickers, so the overlap will be quite small
If there is a way to extract them, it will be most likely not in SVG format
Gboard is proprietary software, so even if they could be extracted in a way that works, it would cause legal issues
It's actually not the Blobs that are that difficult to make, but all the other emojis, especially when it involves skintones and genders (in that case, simply copying the Noto ones doesn't work due to the changed art style)


domains, plural.

's to s.
What is this input supposed to be? Is it automated or manually managed? Is this supposed to be a comma-separated list if there are more than 1 domain?

@walker that input can be whatever the user wants actually.  It is not an automated process by any means, that input just goes to Mailchimp and then I manually update a seeder file on the API side with the data.  This allows me to set the actual host names for the user.  Depending on how the app is going to be used, I very well may just use * despite them entering a more specific domain if it looks like their service might be used by more than one domain.
So ya, certainly this is a case by case thing for now and that's why I did not put heavy restrictions on what was actually being entered in this field.  It's literally just going to my email to review.

@manifestinteractive: Great. I figured that might be the case.


How to use Medoo framework object

Hi @mikecao
As per #348 , I have registered the Medoo framework class with flightphp. But not able to get any data and I'm getting 500 error. Can you please tell me whats wrong with the code?
Flight::route('GET /getclients', function(){
    $conn = Flight::db();
    $data = $conn->select("client", [
        "[>]state" => ["client_state" => "state_id"],
        "[>]country" => ["client_country" => "country_id"],
    ]);
    echo json_encode($data);
});

There's not enough in your code to determine the issue.
I would suggest:

Try routing without database connection
Try reviewing PHP error logs for hints
Use xdebug to debug

If these fail, please share more code and ideally a link to your code online.
My guess is that it is a database init/connection issue.

Full Code here: Before using Medoo I was using PDO and it was working fine. Only with Medoo I'm getting 500 Internal Server Error.
<?php
require 'vendor/autoload.php';
require  'Medoo.php';

$medoo_Config = [
	// required
	'database_type' => 'mysql',
	'database_name' => 'xxxx',
	'server' => 'localhost',
	'username' => 'yyyy',
	'password' => 'zzzz',
	// Enable logging
	"logging" => true,
];

Flight::register( "db", new Medoo( $medoo_Config ), [ $medoo_Config ] );

/* Flight::register('db', 'PDO', array('mysql:host=localhost;port=3306;dbname=xxxxx', 'yyyy', 'zzzz'), function($db) {
        $db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
        $db->setAttribute(PDO::ATTR_DEFAULT_FETCH_MODE, PDO::FETCH_OBJ);
}); */

Flight::route('GET /getclients', function(){
    $conn = Flight::db();
    /*$conn = Flight::db();
    $idata = [];
    $SQL = "SELECT c.*, s.state_name,s.state_id, cn.country_name FROM client c, state s, country cn WHERE c.client_state = s.state_id AND c.client_country = cn.country_id";
	$dbdata = $conn->query($SQL);
	if(!empty($dbdata)) {
	    foreach($dbdata as $row) {
	        array_push($idata, $row);
	    }
	}

    echo json_encode($idata);*/
    $data = $conn->select("client");

    /* , [
        "[>]state" => ["client_state" => "state_id"],
        "[>]country" => ["client_country" => "country_id"],
    ]); */
    echo json_encode($data);
});

Flight::start();
?>

If someone needed. Line 19 has to be like this:
Flight::register("db", 'Medoo\Medoo', [ $medoo_Config ]);

FWIW, I'm putting this in the framework in the next little bit. Just a simplified Pdo_Wrapper https://github.com/flightphp/core/compare/master...database-class


Crashes on launch

Every time I try to launch pump.io, I'm faced with the following error message:
-> % ./bin/pump
{"name":"pump.io","hostname":"pump","pid":2881,"component":"dispatch","level":30,"msg":"Dispatch setup complete.","time":"2013-07-15T21:06:53.025Z","v":0}
{"name":"pump.io","hostname":"pump","pid":2881,"component":"dispatch","level":30,"id":1,"msg":"Setting up worker.","time":"2013-07-15T21:06:53.031Z","v":0}
{"name":"pump.io","hostname":"pump","pid":2883,"level":30,"msg":"Initializing pump.io","time":"2013-07-15T21:06:53.446Z","v":0}
{"name":"pump.io","hostname":"pump","pid":2883,"level":30,"msg":"Connecting to databank with driver 'mongodb'","time":"2013-07-15T21:06:53.515Z","v":0}
{"name":"pump.io","hostname":"pump","pid":2883,"level":30,"msg":"Setting up HTTP server.","time":"2013-07-15T21:06:53.561Z","v":0}
{"name":"pump.io","hostname":"pump","pid":2883,"component":"sockjs","level":30,"msg":"Setting up sockjs server.","time":"2013-07-15T21:06:53.580Z","v":0}
{"name":"pump.io","hostname":"pump","pid":2883,"level":30,"firehose":"ofirehose.example.null","msg":"Setting up firehose","time":"2013-07-15T21:06:53.582Z","v":0}
{"name":"pump.io","hostname":"pump","pid":2883,"component":"firehose","firehose":"ofirehose.example.null","level":30,"msg":"Setting up firehose.","time":"2013-07-15T21:06:53.582Z","v":0}
{"name":"pump.io","hostname":"pump","pid":2883,"level":30,"msg":"Listening on 31337 for host pump.mydomain.net","time":"2013-07-15T21:06:53.583Z","v":0}

net.js:1055
    if (port && handle.getsockname && port != handle.getsockname().port) {
                      ^
TypeError: Cannot read property 'getsockname' of undefined
    at net.js:1055:23
    at Object.1:1 (cluster.js:587:5)
    at handleResponse (cluster.js:171:41)
    at respond (cluster.js:192:5)
    at handleMessage (cluster.js:202:5)
    at process.EventEmitter.emit (events.js:117:20)
    at handleMessage (child_process.js:318:10)
    at Pipe.channel.onread (child_process.js:339:9)

Closing for lack of activity. OP, if you still want help with this, then comment and I'll reopen. Otherwise, file a new issue.


Emailing on heroku

Did anybody had success with emailing through Mailgun on Heroku?

Followed the instruction steps, but no luck so far.

works :)


created bar graph for frequency of each input object

Not sure if I did the descending order correctly. I can change if need be.

sorting is harder than I thought so we can leave that be for now


Adding remoteDownload user plugin

@uriahl , thanks for submitting this PR!
One small comment:
It is better to have the fetchAndDeploy method report an error by throwing an exception, rather then having it return an error code.


Expose methods area_set_area_monitor_callback and area_set_monitorable

Expose methods area_set_area_monitor_callback and area_set_monitorable in PhysicsServer and Physics2DServer

If this is merged, a cherry pick for 3.0 will be appreciated :D

Now that Godot 3.1 has been released, we don't plan to cherry-pick new features and enhancements to the 3.0 branch, unless there is very strong support in favor of it. Removing cherrypick label for 3.0.


[Bug]: changing the width of the child widget is moving the widget beyond the parent widget’s width

https://www.loom.com/share/eda5f95f052d48debf661523c99a1b2f

We are seeing a different issue but related to the same steps showed in the video, so raising a separate issue. Moving this issue to done.


Updated Windows installation instructions

I have updated the Windows installation instructions with some corrected commands.
These have been tested on a second computer.

Thanks!


Updating docker file to point to new repo

Hi, thank you for your contribution to Mantl! Before we can accept any code into
master, we need it to meet the following criteria. If there are any you can't
satisfy yourself, go ahead and open the pull request anyway and we'll help you
test. Feel free to delete this message once you're done. Thanks again!

[ ] Installs cleanly on a fresh build of most recent master branch
[ ] Upgrades cleanly from the most recent release
[ ] Updates documentation relevant to the changes

@thomasvincent @stevendborrelli : How do we get this merged? Looks like a pretty simple change here.

Thanks @thomasvincent !


HBASE-23821 Generate CHANGES.md and RELEASENOTES.md for 2.1.9

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 38s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+0 :ok:
markdownlint
0m  0s
markdownlint was not available.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-2.1 Compile Tests _

_ Patch Compile Tests _

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

_ Other Tests _

+1 :green_heart:
asflicense
0m 18s
The patch does not generate ASF License warnings.

2m 13s

Subsystem
Report/Notes

Docker
Client=19.03.5 Server=19.03.5 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1156/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1156

Optional Tests
dupname asflicense markdownlint

uname
Linux ffa040391262 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1156/out/precommit/personality/provided.sh

git revision
branch-2.1 / aa5bf14475

Max. process+thread count
47 (vs. ulimit of 10000)

modules
C: . U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1156/1/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.


i18n support

Hi,
Does your module support or plan to support i18n?

Have you tried using angular-translate? It should work nicely together with ng-fab-form. To do so, you can edit the template.
While it should be quite easy to implement such functionality within ng-fab-form, I'm not sure if the module should be really responsible for translations as their probably is not only relevant for validations.

Thanks for the suggestion @johannesjo.  I was able to use angular-translate with your module.  I even got it to work with their partial loader.  Nice indeed.


mlton: 20180207 → 20210107

Description of changes
https://github.com/MLton/mlton/releases/tag/on-20210117-release
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@thoughtpolice @sheganinans as maintainers of downstream urweb, can I get a review? :)

@toastal looks good!

Metis builds as well.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/mlton-package-has-wrong-version-number-how-to-fix-this/42479/1


use bundled libsqlite3 on windows

diesel mentions that you can do this and it’s the easiest way to do everything on windows. it might also be easier on macOS but i can’t conveniently check if macOS ships with libsqlite3 out of the box although i suspect it does not

thats fine


Optimize should probably behave as much like spiv-opt as possible.

Most importantly, if optimize doesn't produce a new output file, it should exit with a non-zero result code.  If the output file wouldn't be usable, optimize should probably not produce output and should exit with a non-zero result code.
Here are some specifics.

If the source file wasn't trimmed, just copy it and succeed, or fail.
If the source file didn't have a compression format supported by optimize, report an error and exit with failure code rather than silently succeed and don't produce output.
If the source file couldn't be optimized any further, just copy it and succeed, or fail.

Not sure if I understood the above correctly, but just in case: I think the worst possible outcome here is producing a usable result that isn't the result that was requested (by the cmd line options), because in this case you might mistakenly use it and produce wrong data. This is especially important when optimize is part of an automation setup where you might not be reading warning messages on the output.


fix(zero-react): added zero client overrides for SSR

Fixes Zero throwing an error in SSR where WebSocket is not available.

seems reasonable to me as a stop gap to prevent blowing up in ssr environments. @aboodman

Moved to https://github.com/rocicorp/mono/pull/3445

Great, thanks!


dropdown will appear when application is rejected

When an application is rejected in any round and a mail is sent to applicant, the drop-down menu for other actions should not disappear.

.

@abhishek-pokhriyal is this ready for review? I see myself as reviewer but the "ready for review" is missing.

@rathorevaibhav I think this issue is better handled in this PR 499 https://github.com/ColoredCow/employee-portal/pull/499/


L-850 Fix flaky tests

Cache doesn't seem to have any effect (even the loading from different Java version)

The failures in #15 might have been because there are real requests made in integration tests 🤔
The test flakiness doesn't seem to be an issue now.


reset Vao after draw call

Currently VAO will not be set to null after drawCall.
Array buffer or element buffer inside VAO could be updated unintentionally in operations after drawcall such as:

regl.buffer(info)
regl.elements(info)

This is to set vao to null after every drawcall to defend these situations.

It's a waste to call setVao(null) after every drawCall inside batch.
However I haven't found where to add setVao(null) right after batch body.


Error when trying to download via member ID

Here is the link to what shows up in the pixivutil.log
http://pastebin.com/qxPJQWYC

Looks like the login failed, also can you upload Error page for member 119199.html to pastebin?
Check your username/password in config.ini.

The username/password in config.ini is correct.
Here is the pastebin of the file. http://pastebin.com/4Vsk7HgY
Attached is a zip with the file in it (if it worked)
Error page for member 119199.zip

For some reason, the logon is failed, it should show Logging in, return url: http://www.pixiv.net/ when login is successful. The uploaded page shows it (it is the login page).
I just tried with my client. Double check your username/password. If not successful, login in your browser and copy the cookie value (refer to readme.txt: Q3: Error at process_image(): (<type 'exceptions.AttributeError'>, )

ooooh, i figured out the problem. Because in the readme.txt Q3 it says to login via firefox, i had to log in without remember my password/etc. It turns out i was writing my nickname instead of my email as the username. Thanks for the help!!!


Discord Categories

I went looking around for this and couldn't find any functions for creating categories, which are a type of channel iirc.
I believe you can manage categories by searching for them in the channel list, but perhaps there could be more specific functions for managing them added?

Possible duplicate of #495

I closed this, but I'm reopening it because while you can move channels into categories, I'm not sure that you can create categories

You can create categories like this:
channel, err := session.GuildChannelCreate(guildID, "my new category", "4")
I noticed that GuildChannelCreate does not use the ChannelTypes defined in structs.go. I will create a pull request to change that later if I don't forget it. I don't think sending text or voice as the channel type still works. This will be a breaking change in the API, but since the old way doesn't work anymore I think it should be fine.

#511

There are also constants for the various channel types so you don't need to remember the numbers.
https://godoc.org/github.com/bwmarrin/discordgo#ChannelType


Mutate answers in custom action type

I'm not sure if I'm going about this wrong maybe, but I'm trying to enrich my answers object to make my templates a little bit simpler. What I did is create a custom action type called prepare. It takes a prepare config prop, which mutates the answers object it receives (I Object.assign the result onto the original answers. I know it's not ideal but I'm just trying to make this work first). In the next step however, they're not available. Is this intended behavior?
Here's my custom action type:
exports.prepare = (answers, config) => {
  const { prepare } = config;
  Object.assign(answers, prepare(answers));
  return `Data prepared: ${JSON.stringify(answers, null, 2)}`;
};

I see it log the enriched answers object correctly, but the new props aren't available in the templateFilePath of the next step.

I'm just looking at the merged PR in node-plop, and the easiest solution for my use-case, which I think has general applicability, is to allow data to be either an object, or a function that is called with data, so a thunk sort of. Thoughts?

yeah, I think this is actually a defect that was introduced as called out in #117. I'm going to take a look at this soon, I'm pretty sure it can just be addressed with a bug fix.
I also really like your idea to make the data prop optionally take a function 👍

Any updates on this fix?

this is fixed in v2.1.0, released this morning.


Are there plans to update the polyfill-service package

What

I see that the polyfill-service npm package was last updated 6 months ago but there have been updates to the service since then. Are there plans to publish the updated service to NPM?
Details
I have been using the polyfill-service npm package to self host a copy of this service and noticed today that it had not been updated in a while. Is the recommended way to host this service now just to fork the repo or download a tarball from the releases page instead?

The npm package was meant for people to use the programmatic API for the polyfill-service, which is now in a separate package called polyfill-library.
You are correct, the recommended way to host a copy of the service would be to either download a tarball or to clone the repo and checkout the specific tag of the service you are wanting to use.
I'll be closing this issue as I believe we have answered your question, please do comment on this issue if you have any other questions about this :-)


Potential issue with team table

Teams = teamResults
                .OrderByDescending(t => t.Points)
                .ThenByDescending(t => t.Difference)
                .ThenBy(t => t.Name)
                .ApplyRanks()
                .ToList(),

See #175

Here's an example from the old software:

UAT


Majority test

I think I caused a bug in "sparse_isometry.py" resolving a conflict. To fix it, I removed line 32 "pivot_circuit = qiskit.QuantumCircuit(n_qubits)", as I believe it was included improperly.


RC fail for the bitstream 720p5994_parkrun_ter.yuv

hi,
I met one issue when encoding the stream 720p5994_parkrun_ter.yuv with bitrate 2000kbps, but actually the output is 6161Kbps. I use the framerate 30fps.
I noticed it is related to the parameter "SpatialBitrate". if I set it as 2000, the RC will failed. If it is 600, the output bitrate is still ~2000kbps.
BTW, I test on one 1080P stream and did not see the issue.
is this an issue?
attach the cfg file:
Cisco Scalable H.264/AVC Extension Encoder Configuration File
#============================== GENERAL ==============================
UsageType        0            #0: camera video 1:screen content
SourceWidth      1280          #input video width
SourceHeight     720          #input video height
InputFile       /home/bqu/workspace/stream/YUV_files/720p5994_parkrun_ter.yuv # Input  file
OutputFile              enc.h264               # Bitstream file
MaxFrameRate            30                     # Maximum frame rate [Hz]
FramesToBeEncoded       -1                    # Number of frames (at input frame rate)
TemporalLayerNum       1                     # temporal layer number(1--4)
IntraPeriod            0                    # Intra Period ( multipler of GoP size or -1)
EnableSpsPpsIDAddition  1
EnableFrameCropping 	1 		       # enable frame cropping flag
EntropyCodingModeFlag   0                #0:cavlc 1:cabac
MaxNalSize            0            # Unit:Byte, Maximum Nal size
#============================== LOOP FILTER ==============================
LoopFilterDisableIDC       0                   # Loop filter idc (0: on, 1: off,
# 2: on except for slice boundaries,
# 3: two stage. slice boundries on in second stage
# 4: Luma on but Chroma off (w.r.t. idc=0)
# 5: Luma on except on slice boundaries, but Chroma off in enh. layer (w.r.t. idc=2)
# 6: Luma on in two stage. slice boundries on in second stage, but Chroma off (w.r.t. idc=3)
LoopFilterAlphaC0Offset	0                      # AlphaOffset(-6..+6): valid range
LoopFilterBetaOffset	0                      # BetaOffset (-6..+6): valid range
#============================== SOFTWARE IMPLEMENTATION ==============================
MultipleThreadIdc			    1	# 0: auto(dynamic imp. internal encoder); 1: multiple threads imp. disabled; > 1: count number of threads;
#============================== RATE CONTROL ==============================
RCMode			        1				        # 0: quality mode;  1: bitrate mode;                                             # 2: buffer based mode,can't control bitrate; -1: rc off mode;
TargetBitrate			2000				    # Unit: kbps, controled by EnableRC also
MaxOverallBitrate       0                    # Unit: kbps, max bitrate overall, 0 - unspecified
EnableFrameSkip			0		#Enable Frame Skip
#============================== DENOISE CONTROL ==============================
EnableDenoise                   0              # Enable Denoise (1: enable, 0: disable)
#============================== SCENE CHANGE DETECTION CONTROL =======================
EnableSceneChangeDetection			1			# Enable Scene Change Detection (1: enable, 0: disable)
#============================== BACKGROUND DETECTION CONTROL ==============================
EnableBackgroundDetection		 1     # BGD control(1: enable, 0: disable)
#============================== ADAPTIVE QUANTIZATION CONTROL =======================
EnableAdaptiveQuantization			1			# Enable Adaptive Quantization (1: enable, 0: disable)
#============================== LONG TERM REFERENCE CONTROL ==============================
EnableLongTermReference             1              # Enable Long Term Reference (1: enable, 0: disable)
LtrMarkPeriod                       30             # Long Term Reference Marking Period
#============================== LAYER DEFINITION ==============================
PrefixNALAddingCtrl		0						# Control flag of adding prefix unit (0: off, 1: on)
# It shall always be on in SVC contexts (i.e. when there are CGS/MGS/spatial enhancement layers)
# Can be disabled when no inter spatial layer prediction in case of its value as 0
NumLayers              1                      # Number of layers
#//LayerCfg                layer0.cfg		# Layer 0 configuration file
#//LayerCfg                layer1.cfg		# Layer 1 configuration file
LayerCfg                 enc_layer_default.cfg		# Layer 2 configuration file
enc_layer_default.cfg
Layer Configuration File
#============================== INPUT / OUTPUT ==============================
FrameWidth     1280                     # Input  frame width
FrameHeight    720                    # Input  frame height
FrameRateOut    30                     # Output frame rate [Hz]
ReconFile       rec.yuv          # Reconstructed file
#============================== CODING ==============================
ProfileIdc      66          # value of profile_idc (or 0 for auto detection)
InitialQP       24			# Quantization parameters for base quality layer
#================================ RATE CONTROL ===============================
SpatialBitrate		2000		# Unit: kbps, controled by DisableRC also
MaxSpatialBitrate   0     # Unit: kbps, max bitrate for current layer, 0 - unspecified
#============================== MultiSlice Slice Argument ==============================
for S/M Slice(s) mode settings
SliceMode			0		# 0: sigle slice mode; >0: multiple slices mode, see below;
SliceSize			1500
SliceNum			1		# multiple slices number specified
SlicesAssign0		960		# count number of MBs in slice #0
SlicesAssign1		0		# count number of MBs in slice #1
SlicesAssign2		0		# count number of MBs in slice #2
SlicesAssign3		0		# count number of MBs in slice #3 -- seting here is for better testing
SlicesAssign4		0		# count number of MBs in slice #4
SlicesAssign5		0		# count number of MBs in slice #5
SlicesAssign6		0		# count number of MBs in slice #6
SlicesAssign7		0		# count number of MBs in slice #7
DESIGN OF SLICE MODE
0 SM_SINGLE_SLICE			| SliceNum==1
1 SM_FIXEDSLCNUM_SLICE	| according to SliceNum			| Enabled dynamic slicing for multi-thread
2 SM_RASTER_SLICE			| according to SlicesAssign		| Need input of MB numbers each slice. In addition, if other constraint in slice_argument is presented, need to follow the constraints. Typically if MB num and slice size are both constrained, re-encoding may be involved.
3 SM_ROWMB_SLICE			| according to PictureMBHeight	| Typical of single row of mbs each slice?+ slice size constraint which including re-encoding
4 SM_DYN_SLICE			| according to SliceSize		| Dynamic slicing (have no idea about slice_nums until encoding current frame)
5 SM_AUTO_SLICE			| according to thread number	| the number of slices is decided by the number of threads,SliceNum is ignored

@ruil2

could you attach openh264 output trace?

no more info


🎨  Improve block processing

This change implements a blockQueue that forces the SDK to process blocks sequentially. It replaces the tests in run.live.spec.ts to accommodate this new change.

Found an error, making a quick fix now


Specification for findElement

TODO

Prototype: https://github.com/GoogleChromeLabs/chromium-bidi/issues/67
Specification isn't needed anymore, c.f. https://github.com/w3c/webdriver-bidi/issues/150


Pass context to displaced element

davidtheclark/react-aria-modal#9

Thanks for this! Will be released in 2.1.0 as soon as the tests pass in Travis.


(#171) Added flake8 options and added to CI

Fixes #171

I can't approve this as it's technically my own PR but mostly it's @MattPrit's work. I'm happy with it.


TextField label overlaps when programmatically set and original value is undefined

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

TextField label overlaps with form value when the controlled value starts as undefined and is subsequently set programmatically. Issue does not occur if original value is a string (empty or otherwise).

Expected Behavior 🤔

Label should not overlap with form value.
Steps to Reproduce 🕹

Demo: https://codesandbox.io/s/agitated-dijkstra-ns11w
Steps:

Click "Set values" button
"value1" label will because original value is undefined
Other fields don't overlap because original value is an empty string

Context 🔦

Discovered this after trying to automatically fill form fields based on an external action.
Very easy to workaround.
Your Environment 🌎

Tech
Version

Material-UI
v4.11.0

React
16.13.1

Browser
Chrome 84.0.4147.125 (MacOS)

Please check the console.

Please check the console.

Console says:

value prop on input should not be null.

If it should not be null, why not default it to empty string '' if it is null?


Persian/Farsi language in text-extractor

Description of the new feature / enhancement
please support Persian/Farsi language in text-extractor of PowerToys.
Scenario when this would be used?
It's impossible ocr Persian/Farsi texts from images. there is arabic support but full support for Persian/Farsi language is needed.
Thanks.
Supporting information
I checked these articles:
https://learn.microsoft.com/en-us/windows/powertoys/text-extractor#supported-languages
https://github.com/MicrosoftDocs/windows-dev-docs/issues/4568

/feedbackhub


Problem with getting client IP address in ASP.NET CORE web Api ?

I get server IP Address instead of client IP by using below method in ASP.NET CORE Web API .
Please tell me where I'm going wrong. I have used ServerVariables["HTTP_X_FORWARDED_FOR"] in asp mvc before and that has worked correctly.
private string DetectIPAddress(HttpRequest request)
        {
            var _IP = "RemoteIp:" + request.HttpContext.Connection.RemoteIpAddress.ToString() + " - LocalIpAddress:" +
                          request.HttpContext.Connection.LocalIpAddress;
            try
            {
                _IP += " IP.AddressFamily:" + Dns.GetHostEntry(Dns.GetHostName()).AddressList[1].ToString();
                _IP += " HostName:" + Dns.GetHostEntry(Dns.GetHostName()).HostName;
            }
            catch (Exception e)
            {
            }
            return _IP;
        }

https://stackoverflow.com/questions/56525043/problem-with-getting-client-ip-address-in-asp-net-core-web-api

FYI here's the doc: https://docs.microsoft.com/en-us/aspnet/core/host-and-deploy/proxy-load-balancer?view=aspnetcore-2.2
Also, you don't need ForwardedHeaders.All in that code.


x/vuln/vulncheck: DeleteSyntheticNodes for all callgraphs?

Currently, wrapper for only the final callgraph in vulncheck are inlined via callgraph.DeleteSyntheticNodes. Should the same be done for intermediate ones as well?

The intermediate calls to DeleteSyntheticNodes do not bring any benefit. In fact, they slow things down for initial cha call graph. The last call for deleting synthetic nodes should stay as it is cheap and simplifies resulting vulnerability graphs.


Allow specifying health check on ELB

http://docs.aws.amazon.com/AWSRubySDK/latest/AWS/ELB/Client.html#configure_health_check-instance_method

+1

(thumbsup)

bump, ELBs are tough to use without health checks set right.

If anyone could kindly give me a push start for this, I maybe able to work on it some this week.

@danieljimenez any chance you could make it to office hours this Thursday at 1pm PST?  We could get started on it in there!

Hi Tyler,
When would the next office hours be?

The next office hours are Thursday July 30th at 1pm PST.  They are coordinated via http://www.meetup.com/Chef-Office-Hours/events/223831967/

+1


Add Terms of Use (ToU) page to the website

The Terms of Use (ToU) page is currently not added to this website. I wish to create the whole ToU page to outline the rules and guidelines for using the site.
/Assignee
@jahnvisahni31

go ahead @Zaki2409

on it !!

Hi @jahnvisahni31,
Could you please review my pull request for the TOU PAGE section? Your feedback would be appreciated!
Thanks!


Splitting Ordinals

I accidentally consolidated some of my inscriptions in two transactions when using  a command similar to ord wallet send <ADDRESS> 12345sat
https://mempool.space/tx/fa0dd811e35cf9aa533171c0f13b4a9b186bc5792f980090566989e2d246f59e
https://mempool.space/tx/9da61fb1f6f0a836084da18d39b2887da0665917a0d9f4ad548b3e3320b8f707
At first i thought i had burnt the items, but now i see them on the explorer all in one output
https://ordinals.com/output/fa0dd811e35cf9aa533171c0f13b4a9b186bc5792f980090566989e2d246f59e:0
https://ordinals.com/output/fa0dd811e35cf9aa533171c0f13b4a9b186bc5792f980090566989e2d246f59e:0
My question is how do i split them when i want to send to someone else? Is it just ord wallet send ADDRESS INSCRIPTION_ID?

This is theoretically possible but at the moment we do not support it. We decided that it is more important not to accidentally lose inscription than to allow complex disaggregation of outputs. For now you will have to hold on to the inscriptions in this way but we do plan to allow this fine grained control of outputs in the future.

We might also be able to support this easily simply by making the transaction checks more liberal. The idea would be to move checks that we're sending inscriptions to the end of transaction construction, instead of the beginning. That way, if the transaction builder happens to be able to send the inscription you want and all the other ones wind up going back to change, it'll just work.

any updates on this @casey I have a bundle stuck together

I think you can split them out one-by-one by specifying the satpoint of the last inscriptions in the UTXO. Might also work if you specify the inscription id of the last inscription in the UTXO, but not sure about that.

https://ord-mirror.magiceden.dev/tx/575e5d3cdcb6c6f505013884e4e69e4cd927d7d0de178399a1b659a0516f4e5d:0:662


Feature Request: Include the org or user from the repo slug in the local path

Feature Requested
When you clone a repo using the gh verb in Alfred, the resulting clone gets placed at the root of the Cloning: directory.  This layout leads to potential name collisions and prevents cloning multiple forks from GitHub using this Alfred workflow.
I could see either adding a checkbox to the configuration panel to Include repo owner in path or allowing a {owner} substitution in the Cloning: textbox.  Then the workflow command would just need to ensure that the owner dir existed under the Cloning: dir when performing the clone.
Relevant Screenshot
For example:

then using gh github/gitignore followed by ^+Enter would clone the repo into ~/github.com/github/gitignore.
Checklist

[X] The feature would be useful to more users than just me.

I have solved the problem differently, the workflow now automatically renames repos of the same name to prepend the owner name, so that forks are distinguishable.
Available in the Alfred Gallery in 1-2 days, or directly by downloading the latest release here: https://github.com/chrisgrieser/gitfred/releases/download/1.6.0/gitfred.alfredworkflow


superset can not know the string/varchar columns from impala table

Make sure these boxes are checked before submitting your issue - thank you!

[* ] I have checked the superset logs for python stacktraces and included it here as text if any
[ *] I have reproduced the issue with at least the latest released version of superset
[* ] I have checked the issue tracker for the same issue and I haven't found one similar

Superset version
0.18.4
Expected results
when Add Sqla Table, superset can get all right columns from impala table
Actual results
when Add Sqla Table, superset can not know the string/varchar columns from impala table
Steps to reproduce
1.have impala table:

2.Add Database from impala
3.Add Sqla Table
edit table->list table column, I find that the string/varchar columns can not be known by superset

Sure this is an issue in superset and not in the sqlalchemy dialect / plumbing?

@xrmx I am a superset beginner, no idea about the problem.Do you use superset on impala table that has the string/varchar columns and succeed？

i had the same issue with 0.18.5

Same issue. I'm on 0.18.5 as well. This maybe another issue, but Time Grains don't show for Impala as well.

I wonder if the UNKOWN value comes out of the SQLAlchemy dialect or if it comes from us here:
https://github.com/apache/incubator-superset/blob/a141695b2b73df1d05d74c8ed8e22341e6e25edc/superset/connectors/sqla/models.py#L586
Do you see stack traces in your log when saving a Impala table? If so please paste it here. If not that means it's coming out as UNKNOWN out of the driver/dialect and needs to be fixed there.

@JImBurris about time grains, they haven't been contributed yet for Impala. Someone needs to send a PR with a DBEngineSpec with time grains defined for impala, it's as simple as:
https://github.com/apache/incubator-superset/blob/master/superset/db_engine_specs.py#L193

Here, can someone that has impala test this?
https://github.com/apache/incubator-superset/pull/3225

Notice: this issue has been closed because it has been inactive for 264 days. Feel free to comment and request for this issue to be reopened.


v2 demo: Class 'Interop\Async\Loop' not found

I am interested in amphp project and I try to figure out all things together. Currently looks like dependencies are misconfigured:
$ php vendor/bin/aerys -c vendor/amphp/aerys/demo.php
PHP Fatal error:  Uncaught Error: Class 'Interop\Async\Loop' not found in /Users/maxvoloshin/projects/amp-demo/vendor/amphp/aerys/bin/aerys:92
Stack trace:
#0 {main}
  thrown in /Users/maxvoloshin/projects/amp-demo/vendor/amphp/aerys/bin/aerys on line 92

Fatal error: Uncaught Error: Class 'Interop\Async\Loop' not found in /Users/maxvoloshin/projects/amp-demo/vendor/amphp/aerys/bin/aerys on line 92

Error: Class 'Interop\Async\Loop' not found in /Users/maxvoloshin/projects/amp-demo/vendor/amphp/aerys/bin/aerys on line 92

Call Stack:
    0.0011     368144   1. {main}() /Users/maxvoloshin/projects/amp-demo/vendor/amphp/aerys/bin/aerys:0

FAIL: 255

composer.json:
{
    "name": "maxvoloshin/amp-demo",
    "require": {
        "amphp/amp": "2.0.x-dev",
        "amphp/stream": "dev-master",
        "amphp/parallel": "dev-master",
        "amphp/loop": "dev-master",
        "amphp/cache": "dev-amp_v2",
        "amphp/aerys": "dev-amp_v2",
        "amphp/file": "dev-amp_v2",
        "amphp/socket": "dev-amp_v2",
        "amphp/dns": "dev-amp_v2",
        "amphp/windows-registry": "dev-amp_v2",
        "amphp/artax": "dev-amp_v2"
    },
    "minimum-stability": "dev"
}

Thanks for the help!

Right, we didn't update all libraries to the latest draft yet. I'd use the normal version for now, so ^0.4.4.

Yeah, recently Interop\Async has been renamed to AsyncInterop namespace, which hasn't been updated here yet - v2 is still a bit of a moving target here.

This should have been fixed within the most recent commits to amp_v2 branch; closing.


Postgres SUPERUSER - CREATE EXTENSION hstore and done?

When running schema migrations or on first use, a non-SUPERUSER postgres user cannot create the schema because they cannot CREATE EXTENSION hstore.
This is not well-documented, not beginner-friendly, and I only stumbled about the detail in http://docs.miniflux.net/en/latest/cli.html whereas I think it should be mentioned in http://docs.miniflux.net/en/latest/installation.html
The recommendation to temporarily give superuser privileges for migration might be general, but maybe it would be better to just document the required privileged operations and let the users perform them manually (they will have to create the database in the first place anyways...)

I've also had a bit of a hard time with that, and only managed to find it with some obscure SO answer.
No one wants to give SU privileges to a "normal" user in their DB, but it may be important to document the fact that, when creating the user/db, the HStore extension should also be created in this newly created DB.

All these steps are already documented here: https://docs.miniflux.net/en/latest/cli.html#run-database-migrations

In addition to to that, there are several tutorials: https://docs.miniflux.net/en/2.0.6/tutorials.html

I don't think burying the requirement in the copy-and-paste bash code in some tutorial that will eventually be outdated is particularly useful.


[Feature Request] Display the XCreds version to the new "System Info" pop-up

Low priority, but it would be nice to have the option to show the XCreds version into the new XCreds 5 System Info pop-up (shouldShowSystemInfoButton). One less key  to manage, and keeps the UI a little cleaner for those admins who choose to show the version at login. If I am choosing to enable the new 'show information on login window' option,  it seems reasonable to also add the XCReds version to the list along with OS version, IP, serial etc. Purely cosmetic, but worth mentioning.

added and moved system info to left side.


ENH Website

The website looks great. I just noticed a few things when I was browsing. Feel free to address as you see fit.
In mobile browsing:

[ ] there's an overlap in the widget containing the number of lines of code and the documentation section
[ ] the university logos are off-centered with the package logo
[ ] red bar about dev version is hidden by the toolbar

In general:

[ ] missing sections from reference: read_dig_montage and linear_regression_raw.

I actually don't think they're off-centered, I think it's a visual thing due to the fact that the logos aren't rectangular. At least according to the HTML they should be centered

I don't see the hiding or the overlap, either, this is what mine looks like:

What browser are you using?

@Eric89GXL I meant browsing on iPhone and other mobile devices. I think it's just how bootstrap is set up to the wrap.

Ahh, sorry, missed the mobile bit

reference fixes are more than welcome

Fixed by https://github.com/mne-tools/mne-python/commit/632e49f0470fc9526936dbb474fd6aa46501fe4d. I didn't fix the logo centering, though, because I didn't see an easy way to do it, and it's a minor point. If someone else knows how or wants to do it, feel free.


command-execute default set false behaves as an override would

Doing /cf and clicking on the default setting for command-execute and setting it false in a basic claim, winds up causing the flag to affect even the claim owner, leaving you with no recourse to fix it unless you edit the flag in /luckperms/sponge-data/defaults/default.json and restart the server.
I did not test this in an Admin claim, very short on time at the time, but the behavior may extend to it and possibly the wilderness.

I added a workaround for admins, i'll add a fix for owners soon.


Modify PUT /entities/<id> (update_entity) to default to returning simple message instead of entity dictionary

The new default response should be:
return jsonify({'message': f"{normalized_entity_type} of {id} has been updated"})

[ ] Add support for a query parameter to be passed to return the normal dictionary
[ ] Make sure that ?return_all_properties=true is still supported
[ ] Find the places that we currently call this endpoint and add the query parameter to return the normal dictionary (e.g. bulk registration in the Ingest API, various Portal locations)

https://github.com/orgs/sennetconsortium/projects/9/views/1?pane=issue&itemId=64182378


docs suggestion: add perma-links to headers in API reference

We can link to specific sections of the API. For example, https://gateway-api.sigs.k8s.io/v1alpha2/references/spec/#gateway.networking.k8s.io/v1alpha2.HTTPRequestMirrorFilter.
However, there is no way to click no the header and generate a link. Many docs have a little anchor icon there to get it, or make the text itself a link

It would be great to have similar

I've done this with anchorjs previously. It works pretty well.

/lifecycle frozen

While grooming we saw that this one was open for a long period of time without anyone with a strong use case to champion it. We're going to close this as we don't expect anyone's ready to drive it forward, but if you still want this feature and have a strong use case we will be happy to reconsider this and re-open.

Actually I think we completed this one and just never closed it, either way, thanks for closing this one out.

:+1:

This was completed in https://github.com/kubernetes-sigs/gateway-api/pull/1046


access zf2 container in RestAPI

Hi,
i try to access a value stored in a zf2 session container. In my ressource class (extends AbstractResourceListener) it is unfortunately NULL;
Why does this not work? Why is the container stored value empty?
fetchAll($params = array()):
$container = new Container("store_login");
$container->value = NOT NULL
create($data)
$container = new Container("store_login");
$container->value = NULL
thanks for help
ternes3

This behavior depends on the php version, with 5.5.28 is everything fine, with the newest php (6.6.13) version, i get the described behavior.

This repository has been closed and moved to laminas-api-tools/api-tools; a new issue has been opened at https://github.com/laminas-api-tools/api-tools/issues/45.


[Feature Request] Support mobile attachments

Hello there,
I wonder if it is possible that the plugin supports uploading attachments from mobile.

I've had this request before.  I don't know how to do it properly.  It worked at one time, and then it stopped.  There have been some suggestions on how to debug it, which I tried, but didn't manage to get anything working.

Okay, let me try

Okay, what I have deduced so far:

It is not a mobile related problem. The plugin doesn't handle "Insert attachment" command in general.
This command opens a file picker, saves it in the vault, and creates a Wiki-link for it.

Solutions I tried in so far with no luck:

Intercept the callback function of the file picker and pass the file to your PasteFunction
Override the native command and a custom functionality, which opens the file picker and then pass the file to the PasteFunction

async onload() {
...
...
...
   // Wait for Obsidian to load everything, so we make sure that the "Insert attachment" is created by Obsidian
    this.app.workspace.onLayoutReady(() => {
        // Get the command
        const attachFileCommandDefinition = this.app.commands?.commands?.["editor:attach-file"];
       // Store the original callback
        const attachFileCallback = attachFileCommandDefinition?.editorCallback;
        // Check if the callback is stored as expected
        if (typeof attachFileCallback === "function") {
            // Override the existing command with the new callback
            this.app.commands.removeCommand(attachFileCommandDefinition .id);
            this.addCommand({
                id: "editor:insert-attachment",
                name: "Insert Attachment",
                editorCallback: newCallback,
            });
    }
}

I am sharing the progress, in case you have any hint on how to proceed. Meanwhile, I am looking into it.

maybe some inspiration here? https://github.com/trganda/obsidian-attachment-management/blob/main/src/main.ts

Yes, I installed it, but unfortunately it doesn't support attachments via the command. But I will look into that direction, I might get inspired from other plugins.

It is not a mobile related problem. The plugin doesn't handle "Insert attachment" command in general.
this might be the key insight

So, I have spent some more time investigating this popular request.  I would like to support it, however, unfortunately, my existing code path is frankly a pigs breakfast.  While it would be easy to make "something happen" it would be a lot harder to support all of the configuration options that are present in the existing upload mechanism.  This would require me to refactor it substantially.  Perhaps I am just looking for justifications for my laziness, but another issue came to mind.  In the existing cut and paste insert workflow, the user has implicitly decided exactly where in the note that the image/file to be uploaded should go - that's where the cursor is at in the current note. If I understand the workflow for mobile, its a bit different.  This would enable it to be added from the share menu, or from the camera roll, etc.  In this case, I don't know where in which note to actually put the link.
For those of you following this issue - do you have any desired behavior, or insight into to how it actually should work?  Remember, I don't need or use this feature, so I don't have any clue how you might want it to work.  Please do explain.

Thank you for giving it another shot.
Here is my mobile workflow:

My daily note contains all my life management, so any input I take mostly taken from my mobile.
Sometimes, I need to add a photo taken by mobile camera.
So I open the note, insert the cursor at the location
Then, execute "Insert Attachment" command either from command palette or from context menu.

The problem is that the plugin doesn't intercept this action. Also, the mobile app doesn't support Copy/Paste or Drag and Drop actions.
What I tried so far, knowing that I have no JS experience nor Obsidian API:

Intercept the built-in command function right after the file picker. So that the plugin handles the placement as usual.
Replace the built-in function while maintaining the same command name for convenience. For example, in my personal use case, I use slash commander that already has "Insert Attachment" command, also Commander plugin that could place a button to trigger the command anywhere. Although it is just for convenience but not big deal
The last option, that I couldn't do at all, is to implement the full functionality, which is a file picker and handle the placement of the attachment as usual

I hope you would have a much better luck, and be able to implement this feature.


Translatable mapping drivers

Resolves #77
TODO:

[x] YAML driver.
[x] Reference docs
Invalid mapping tests. [separate PR]

@fsi-open

@fsi-open

@fsi-open

@fsi-open

@fsi-open

@rn0


Update stable version in repo from 0.0.5 to 0.0.6

Think about update stable version to 0.0.6
Version: stable=0.0.5 (545 days old) => latest=0.0.6 (15 days old)
Installs: stable=50 (83.33%), latest=8 (13.33%), total=60
Click to use developer portal
Click to edit
Note: This is an automatically generated message and not personally authored by bluefox!

This issue seems to be outdated.
This issue suggests to update the stable version of this adapter to 0.0.6 but in the meantime an update to version 0.1.0 is suggested.
So this issue will be closed and replaced by an updated one.
@mcm195 for evidence


Speed up node reinstall

Right now the kickstart sets up ansible-pull cronjob that runs every 15 minutes.
Then the ansible-pull itself changes that to every 120 minutes.
This means on a reinstall there's quite a long downtime before the the node is configured.
Possible solution:

make kickstart run the ansible-pull-script.sh in /etc/rc.local
have ansible-pull remove it from there and add it as a cronjob

rc.local now run ansible-pull-script.sh on every reboot (as well as in a cronjob).

Done since in 4bcd3bbaf74959ce56b6144c2475f8b8b7cb76ab


Do not include scapegoat class files in assembled jar

When I add the scapegoat plugin
addSbtPlugin("com.sksamuel.scapegoat" %% "sbt-scapegoat" % "1.0.4")
Its class files are being included in my assembly when I run sbt assembly.

Moved to https://github.com/sksamuel/sbt-scapegoat/issues/47


Set OpenRouter's referer header

Is your feature request related to a problem? Please describe.

Not a problem, but OpenRouter gives apps the ability to specify an optional HTTP-Referer header to identify them and make them discoverable to users: https://openrouter.ai/docs/requests#request-headers
Describe the solution you'd like

Setting the header would allow aichat to appear in the "app showcase" list on OpenRouter's home page, which would be good for visibility if that's something desired.
Describe alternatives you've considered

Additional context

Thank you for your suggestion, but we will not implement this feature.
Reasons:

AIChat only sends necessary data to the server, and these two HTTP headers are not essential.
AIChat provides an optional user_agent configuration for users who need to identify requests sent by AIChat.
AIChat is a non-commercial product, and there is no need to promote it in this way.


Pskt

Description:
This PR introduces the implementation of Kaspa's version of Partially Signed Transaction (PST) handling, inspired by Bitcoin's PSBT (Partially Signed Bitcoin Transaction). The implementation includes various roles and functionalities necessary for creating, updating, signing, combining, finalizing, and extracting PSTs.
Key Features

Format and Constraint Operations: The implementation focuses on the format and constraint operations for PSTs.
Integration: Designed to be integrated into a wallet framework or any other wallet implementation.
API Clarity: The API is clear for integration, exposing the PST and waiting for signatures in the case of the Signer role.
No Private Key Handling: It does not accept private keys, nor does it handle script details like multisig or HTLC directly.
Signature Handling: In the Finalizer role, it accepts a callback that constructs the final signature.
Example Provided: A comprehensive example is included demonstrating how partial signatures, redeem scripts, and mixing operations are handled.

Upkeep stuff:

please update the PR title to be something clearer - to me it's kaspa's version of PSBT but it might not be immediately clear to anyone else
please add a description for what this PR is about and how to test it

please add a description for what this PR is about and how to test it

Description is added.
Regarding to test - there is no clear answer. I guess it should be integrated to wallet framework or any other wallet implemtation.
It mostly just  limited set of operations according to roles and format of the pskt


long ckt address report error: 'parse address: "invalid checksum"

Describe the bug
For long ckt address, capsule returns with error: parse address: "invalid checksum
Screenshots
Here is the command and error return

capsule deploy --address ckt1qzda0cr08m85hc8jlnfp3zer7xulejywt49kt2rr0vthywaa50xwsqtp5rgl5262g5m2w7r9n4wc0wywsgvgk9cwc2mu8
thread 'main' panicked at 'parse address: "invalid checksum"', /root/.cargo/registry/src/mirrors.ustc.edu.cn-61ef6e0cd06fb9b8/ckb-capsule-0.10.0/src/bin/capsule.rs:378:48
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace

If I change the long address to short address ckt1qyqxrgx3lg4553fk5auxt82as7ugaqsc3vtszxgefk. The error disappered.
Additional context
Capsule 0.10.0
ckb-cli    installed v1.4.0 (required v1.2.0)

Proposal:

For 0.10 next release and README / wiki / docs, add a warning that capsule deploy is deprecated in favor of ckb-cli deploy
For 0.11, remove capsule deploy or make it a thin wrapper of ckb-cli deploy

WDYT?

This issue per se shouldn't be too hard to fix, I'll fix it for 0.10.


Update PyPI package documentation for v22.06.00

Update CHANGELOG.md and README.md for PyPI v22.06.00 release

rerun tests

@gigony could you please fix up the conflicts here?

@gigony could you please fix up the conflicts here?

It was only a couple of added lines, so I went ahead and resolved it from the web UI.

@gpucibot merge

Thanks all! 🙏


DOCS-9101 initial content for this issue

@sarjeet2013  @kmendiratta-crm I copied most of this over from the gravitational docs. Can you please review and advise?
It's for [SE-13453]

after discussion with @sarjeet2013 and @gustavo-caracoglia -- closing this PR as we don't want customers to mess with this configuration


Testing: introduce mock objects for calling LLM APIs

We currently rely on calling APIs (mainly OpenAI) inside our tests. To isolate the unit tests from the outside conditions of API connectivity (having a key, reacting to model updates), it would probably be best to rely on mock classes that generate fake API responses for the individual tests, e.g. embeddings or text summaries.
This way, the CI procedure can safely be moved to a GitHub runner without the risk of accumulating high API costs, for instance in the case of a workflow bug that causes many runs.

@fengsh27 as I am currently noticing, it is not even purely about the cost / connectivity issue. PRs do not have access to repository secrets because that may cause security issues (understandably). So we cannot use API keys in CI that runs on PRs.

@slobentanzer , Sorry for the late reply, I just notice this comment and your comments in https://github.com/biocypher/biochatter/pull/24#issuecomment-1792890944. I will look into the PR# 24.
I'm intrigued that PRs can't access repository secrets. Because the secrets are specially designed for actions to access, and their values can only be seen by repository owner. Here is the documentation for secrets https://docs.github.com/en/actions/security-guides/using-secrets-in-github-actions. Additionally, please pay attention to step 3 of the documentation (In the "Security" section of the sidebar, select  Secrets and variables, then click Actions.)

@slobentanzer , BTW we need these secrets:

We do have these secrets in the repo; as long as we're inside the org (ie, a PR from within biocypher), it is fine. CI is passion as long as we're on main or inside the org. The issue is with PRs from foreign repos, because they could inject malicious code and steal the secrets. See e.g. here: https://securitylab.github.com/research/github-actions-preventing-pwn-requests/
Since the PR came from Andi's private GitHub account, secrets were not available.

I got it. That make sense.


Update testcafe-hammerhead to v6.0.0

There is a breaking change in the https://github.com/DevExpress/testcafe-hammerhead/issues/410 PR. We should modify our code according to this.

I'm working on this.


Wiki documentation does not match ER Diagram Viewer behavior for creating new table

Description
The project wiki, at https://github.com/dbeaver/dbeaver/wiki/New-Table-Creation , says "From ER Diagram: In the Database Navigator, double-click on the database name or table name. In the window that opens, navigate to the ER Diagram tab, right-click on the window, and choose Create New Table. Alternatively, in the Database Navigator, right-click on the database/table name and click View Diagram. In the window that opens, you can right-click and choose Create New Table."
However, these options do not appear on the right-click menu (screenshot attached).

The project wiki should be updated appropriately.
DBeaver Version
Community Edition 24.0.3
Operating System
Ubuntu 23.10
Database and driver
MariaDB.
Steps to reproduce
In the database navigator, double-click a database name. Navigate to the ER Diagram tab, and right-click within the diagram.
Additional context
No response

Hello @michael-s-crouch
Thanks for noticing.
This menu action is visible only in the Edit mode in the ERD diagram.
This Edit mode is only available in PRO versions. We need to add this information to our wiki.


Question: How to load JSON as a separate file that will be added to the output directory.

Summary
How to load JSON as a separate file that will be added to the output directory. (Lottie json file animation)
Basic example
This is how I add animation atm:
  const animObj = lottie.loadAnimation({
      wrapper: this.$refs.animationRef,
      animType: 'svg',
      loop: true,
      autoplay: true,
      animationData: require('~/assets/animations/animation-hero-home.json')
    })

It generates the js bundle with that JSON inside.
The problem with that - if I have 20 animations throughout my static site, my bundle file will be around 20mb (each animation is 200-500kb)
What I want to do
Ideally I want to use path prop, that accept relative path to the file.
  const animObj = lottie.loadAnimation({
      wrapper: this.$refs.animationRef,
      animType: 'svg',
      loop: true,
      autoplay: true,
      path: require('~/assets/animations/animation-hero-home.json')
    })

Which will generate .json file inside my output directory, and replace require with a path to the file. So It will load the file depending on the page I'm in. (Plus I can gzip on the server separately)
The only resource I've found that address that is here: https://github.com/webpack/webpack/issues/6586
But not sure how to implement that in Gridsome.
Any help would be greatly appreciated.

If you want to add extra webpack configuration to Gridsome's build process, you can use the configureWebpack and chainWebpack options in gridsome.config.js.
You can also use the server API within gridsome.server.js by adding handlers via api.configureWebpack(fn) and api.chainWebpack(fn).
If configuring webpack doesn't work, let me know!

Thanks @Sergix.
If anyone needs a solution, resolved like this:
  configureWebpack: {
    module: {
      rules: [
        {
          type: 'javascript/auto',
          test: /\.json$/,
          include: [path.resolve(__dirname, 'src/assets/animations')],
          use: [
            {
              loader: 'file-loader'
            }
          ]
        }
      ]
    }
  },


WIP: [V2] CFE-971: create and mirror a graph image for ocp releases

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Expected Outcome
Please describe the outcome expected from the tests

I see the e2e test failing here
reading subuid mappings for user \"1003670000\" and subgid mappings for group \"1003670000\": no subuid ranges found for user \"1003670000\" in /etc/subuid"

I see the e2e test failing here
reading subuid mappings for user \"1003670000\" and subgid mappings for group \"1003670000\": no subuid ranges found for user \"1003670000\" in /etc/subuid"

E2E is failing because these tests would be running in a container on a k8s cluster.
Something like the following would allow it to run, but that would require changes in the ci-configs...
securityContext:
       capabilities:
         add:
           - CAP_SETGID
           - CAP_SETUID

I'll privilege first a way to use unshare only in v2, so that e2e aren't affected for the moment.

/test images

Team decision is to use a different implementation (with go-containerregistry):
This containers/buildah implementation has heavy consequences on debugging and running e2e tests in CI


warden svc up

Version of Warden
0.14.1
Operating System and Installation Method
Mac Os 14 via Brew
Describe the Bug
Unable to run warden svc up and dnsmasq is not starting up
 ⠹ Container dnsmasq          Starting                                                                            0.2s  Error response from daemon: Ports are not available: exposing port UDP <IP_ADDRESS>:53 -> <IP_ADDRESS>:0: command failed
To Reproduce

Go to terminal
Run command warden svc stop
warden svc up
See error

After checking ports it is observed that mDNSResponder is blocking the port
Expected Behavior
dnsmasq should run properly and we should be able to access the website at app.example.test
Additional context
No response

I found a fix at
https://github.com/wardenenv/warden/issues/262


Customise onNodePressed to be able to set it in the nodes you want

Will fix this issue https://github.com/fjmorant/react-native-nested-listview/issues/50

duplicated


Investigate basic auton examples on ChiefDelphi

With https://github.com/Open-RIO/2018-Minimum-Viable-Autonomous I believe we are covered for C++/Java. LabVIEW will likely require a little more care to ensure it works with their periodic routines.


Check for previous/existing GitHub issues/module proposals

[X] I have checked for previous/existing GitHub issues/module proposals

Check this module doesn't already exist in another project that is being evolved into AVM

[X] I have checked for that this module doesn't already exist in CARML or TFVM

Terraform or Bicep?
Terraform
Module Classification?
Resource Module
Module Name
avm-res-network-bastionhost
Github Repo name - terraform-azurerm-avm-res-network-bastionhost
Module Details
a terraform module to create a Azure Bastion resource based of CAF best practices patterned on this deprecated module
https://registry.terraform.io/modules/aztfmod/caf-azure-bastion/azurerm/latest
Do you want to be the owner of this module?
Yes
Module Owner's GitHub Username
herms14

@herms14 Can you provide more details how you plan to use mention deprecated module? AVM resource modules  MAY contain references to other resource modules, however MUST NOT contain references to non-AVM modules nor AVM pattern modules.

@prjelesi I don't plan to reference that deprecated module. What I meant was to revive the module and bring it to avm specifications since it has now been archived.

Cool! @herms14
Let's continue with formal part:
Thanks for requesting/proposing to be an AVM module owner.
We just want to confirm you agree to the below pages that define what module ownership means:
https://azure.github.io/Azure-Verified-Modules/specs/shared/team-definitions
https://azure.github.io/Azure-Verified-Modules/specs/shared
https://azure.github.io/Azure-Verified-Modules/help-support/module-support
Any questions or clarifications needed let us know.
If you agree please just reply to this issue with the exact sentence below (as this helps with our automation 👍):
"I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER"
Thanks
The AVM Core Team

@herms14
I just edit module name  - avm-res-network-bastionhost
Please use Github Repo name - terraform-azurerm-avm-res-network-bastionhost

Thanks @prjelesi
I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER"

@herms14 Thanks! moved to "In development"

Update on this modile - I already have the code available on my own repo. Still waiting for CELA to approve the repo under the Azure org - https://ossmsft.visualstudio.com/Reviews/_workitems/edit/48748

UPDATE - I have submitted a PR for review - https://github.com/Azure/terraform-azurerm-avm-res-network-bastionhost/pull/1

Hi @herms14, thank you for requesting a review of the PR and module, have provided feedback on the PR, appreciate if you could look into it and in case you have any questions or concerns than please feel free to reach out.
CC: @PmeshramPM, @matt-FFFFFF, @segraef

@herms14 recommend looking into the tests failing before merging and publishing version 0.1.0 module to the terraform registry.

module has been published in TF registry
https://registry.terraform.io/modules/Azure/avm-res-network-bastionhost/azurerm/latest

Thank you so much @herms14 for owning and developing this module.
Closing this proposal as module has been published to terraform registry below.
https://registry.terraform.io/modules/Azure/avm-res-network-bastionhost/azurerm/latest
Further discussion on this module including issues & PRs to take please on its repo below.
https://github.com/Azure/terraform-azurerm-avm-res-network-bastionhost


cms files are JSON but his extension is .js

Maybe this is just annoying for IDE people like me, where the IDE try to show the JS syntax but fails because they are json files

What do you think to allow both kind of files there, JSON and JS (even more) where the most important thing is the return a valid JSON object

JSON are allowed write/read (like .settings, .users, etc)
JS are read only

merged #214, thank you @CosmoMyzrailGorynych. Published the changed in 1.4.46
@rkmax let us know if this solves your ide issues.

Great!


introduce a way to ask mint about the state of the handshake

Should be merged after #145.

@bifurcation: I just rebased this PR, should be ready for merging now.

@marten-seemann: OK, so this is kind of unfortunate, I relied on being able to run the handshake and have all its states run to completion. Now, it seems like I have to loop around the handshake and wait for the state to be at a specific point where it's waiting for input. Is that what you do?

Yes, that's correct, you'll need to call State() after every step and check if it is mint.State{Client,Server}Connected. On the other hand, I was able to get rid of using mint.AlertWouldBlock, so that made the code a bit easier.

That doesn't work right for what I need, because I don't want to know just when it's connected but rather when it's done everything that it can do without more data. So now I'm going to need a new way to ask "do you need more data, or do you just need me to call handshake() again". Ugh.

I'm now regretting that I didn't review this myself. Can you explain why you need this semantic? To the best of my knowledge, no other TLS stack does this.

I didn't look at how you're doing things in minq (point me to the file, then I can have a look), but you still can use AlertWouldBlock if you rely on that. I just found it a nice property that I don't need it any more in quic-go.
In quic-go we have the session, which is a pretty complex object, handling s loss recovery, flow control, congestion, various timers and much more. Using this API, I can defer allocating this much state until after I received the ClientHello containing a valid cookie, but before sending the certificates (so certificates will already be covered by the normal QUIC retransmission mechanism).

The code is in https://github.com/ekr/minq/blob/master/tls.go#L97
The problem is that you are returning AlertNoAlert after processing CH but
before sending any data.
It looks like I can loop around if alert is AlertNoAlert and state !=
Connected, except that in the case of StatelessRetry .hs is nil and so
GetHsState() segvs.
On Sun, Dec 17, 2017 at 6:20 PM, Marten Seemann<EMAIL_ADDRESS>wrote:

I didn't look at how you're doing things in minq (point me to the file,
then I can have a look), but you still can use AlertWouldBlock if you
rely on that. I just found it a nice property that I don't need it any more
in quic-go.
In quic-go we have the session
https://github.com/lucas-clemente/quic-go/blob/master/session.go, which
is a pretty complex object, handling s loss recovery, flow control,
congestion, various timers and much more. Using this API, I can defer
allocating this much state until after I received the ClientHello
containing a valid cookie, but before sending the certificates (so
certificates will already be covered by the normal QUIC retransmission
mechanism).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/bifurcation/mint/pull/151#issuecomment-352309394, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABD1oVt2KMNsLtJUg8Ekug-jb1nFu1Kwks5tBcvkgaJpZM4Q46PU
.

You want some special treatment of StatelessRetry, don't you? The whole point of performing a stateless retry is that you can discard all your state.
I don't understand your last sentence. When is GetHsState nil?

You want some special treatment of StatelessRetry, don't you?

No.

The whole point of performing a stateless retry is that you can discard all your state.

You're conflating HRR with StatelessRetry. HRR doesn't need to be used with the server discarding state, but, yes, it allows that.  Previously, Mint allowed you to perform the whole CH-HRR-CH cycle on the same mint.Connection, and that's how Minq uses it. I may make Minq purely stateless in the future, but right now, I don't. Unfortunately, after your patch, if you call GetHsState() on a mint.Connection after it has sent HRR, you get a null pointer deref.

I see. I assumed that if a mint user goes through the trouble of using the non-blocking mode, he would only do so to use the stateless retry.
If you don't want to make minq stateless at this moment, you could just recreate the mint.Conn when an HRR is sent. I can't reproduce the nil pointer dereference (I get StateServerStart when calling State() after mint returned the AlertStatelessRetry), so I'm not sure what to do about that. Maybe there's an easy fix for that?