$ref: Type is not defined

I'm struggling to understand what a proper path should be for the following File type:
Project structure

routes
###
#  summary: get files by FC hash
#  responses:
#    200:
#      description: success
#      schema:
#        $ref: '#/definitions/models.File'
###
GET /get/:hash                controllers.HashStore.getFiles(hash: Long)

Error

I am wondering if you included the models package as a domain package name when generating the swagger spec as in https://github.com/iheartradio/play-swagger#step-2?

I didn't. Thanks a lot!
Maybe it's a good idea to add a hint like "are you sure you added 'xxx' package as a domain one?"

I agree with Vasily. I spent two days trying to see the model in Swagger UI because I didn't add the domain package that i should. Some people have restricted knowledge of the terms (in my case i didn't understand the term domain package). Therefore in step1 of the instructions where we are instructed to put swaggerDomainNameSpaces := Seq("models") in the sbt file I I put it exactly identical to the instruction. My defined model case class was in the domain settings.models in my project.
It would be nice to add the following in this instruction:
If you have a case class defined under the package "something.models" and you would want to swagger to generate the definition for it (to access with #ref:definitions/), then make sure that the following is added in sbt: swaggerDomainNameSpaces := Seq("something.models")
If you have other case classes in other packages (e.g. other.models) the add them also in the above sequence like: swaggerDomainNameSpaces := Seq("something.models","other.models")

@AngelosPrastitis thanks for your feedback. Would you help by submitting a PR to the readme file?


Added More Functionality

Exceptions can be captured with extra data in the LogEvent properties
Levels are now translated from NLog to Sentry
Internal logger used when failure
Immediate error on bad DSN
Added UnitTests

BREAKING:

Message capture will only occur if CaptureMessages = true (Message
capture sucks anyway)

@bjarneriis, your initial implementation was a great start. I filled in some missing functionality and threw together some unit tests. If you don't mind, I'd like to grab an AppVeyor and setup continuous integration - with your permission of course.
I keep seeing bits and pieces of a NLog Sentry target around the web. It's about time somebody created a GitHub repo and brought it all together. Great Job!

Hi Dillon.
Thank you for your contribution !!
You have changed this project from a small company in-house code snippet to a general purpose project. Thank you !!
An AppVeyor would be a very nice addition to the project :-)
I have made a few comments on your code. Please let me hear what you think.

 Good feedback on the pull request. I made some updates and had a few questions. Let me know what you think. I'd be honored to be on the authors list! Thanks!
As far as the AppVeyor goes, if you're interested of course and since you're owner, you'll have to sign-up (it's free) and create a job for this repository. That way it'll link the two together. I don't know if you've used AppVeyor before but it's a great (free) tool to build the project, make sure tests are passing before getting merged, and it'll generate the NuGet package when you want to release. Pretty cool, but of course, up to you if you want to add it.

Hi @thedillonb. I think that we are close to a merge now :-) Thank you again for your work. I definitely  would like to add AppVeyor to the project. I will look into it some time next week.

@bjarneriis I updated the pull request with the previous feedback. Let me know what you think.

@thedillonb: I have merged your changes, bumped to version 1.2 and release to nuget. Once again thank you for your contribution :-)


Update TG Pro.app to v2.8

Thank you for the contribution. It was merged directly as ce05e50d7f60526b87cdc341c64236660e359775 to keep commit history cleaner. Your contribution is still credited to you.


Can't get the chat window inside the main body

Hi
I'm trying to add the webchat in the main body but the chat window appears on top of the main body div's containers, overriding everything. The examples show how to use the bot code as a complete website, but I want to use it as a smaller chat popup window.
How should my HTML look like in order to fix this?

Please take a look at our samples: 12.customization-minimizable-web-chat


Using a private source repository with login and no verify_ssl the SSL verfication fails

When using a private repository with login and setting verify_ssl = false the resulting pip command is not correct and the installation of a package fails.
$ python -m pipenv.help output
Pipenv version: '2018.6.25'
Pipenv location: '/usr/lib/python3.6/site-packages/pipenv'
Python location: '/usr/bin/python'
Other Python installations in PATH:

3.6: /usr/bin/python3.6m

3.6: /usr/bin/python3.6

3.6: /bin/python3.6

3.6.5: /usr/bin/python

3.6.5: /bin/python

3.6.5: /usr/bin/python3

3.6.5: /bin/python3

PEP 508 Information:
{'implementation_name': 'cpython',
 'implementation_version': '3.6.5',
 'os_name': 'posix',
 'platform_machine': 'x86_64',
 'platform_python_implementation': 'CPython',
 'platform_release': '2.10.0(0.325/5/3)',
 'platform_system': 'MSYS_NT-10.0',
 'platform_version': '2018-02-09 15:25',
 'python_full_version': '3.6.5',
 'python_version': '3.6',
 'sys_platform': 'msys'}

System environment variables:

USERDOMAIN
OS
COMMONPROGRAMFILES
PROCESSOR_LEVEL
PSModulePath
CommonProgramW6432
CommonProgramFiles(x86)
NetHome
MSYSCON
LANG
TZ
MSYSTEM_CARCH
HOSTNAME
PUBLIC
OLDPWD
CONFIG_SITE
WD
CONTITLE
MSYSTEM_CHOST
UATDATA
LOGINSHELL
USERNAME
LOGONSERVER
PROCESSOR_ARCHITECTURE
tmp
LOCALAPPDATA
COMPUTERNAME
USER
!::
DEFLOGDIR
SYSTEMDRIVE
USERPROFILE
PATHEXT
SYSTEMROOT
USERDOMAIN_ROAMINGPROFILE
PROCESSOR_IDENTIFIER
PWD
HOME
TMP
MSYSTEM_PREFIX
https_proxy
http_proxy
LocalShare
!C:
PROCESSOR_REVISION
USERDNSDOMAIN
PROMPT
NUMBER_OF_PROCESSORS
ProgramW6432
COMSPEC
APPDATA
SHELL
TERM
WINDIR
ProgramData
SHLVL
PRINTER
PROGRAMFILES
MANPATH
ORIGINAL_TEMP
ORIGINAL_TMP
ALLUSERSPROFILE
TEMP
temp
MSYSTEM
SESSIONNAME
ProgramFiles(x86)
PATH
PS1
HOMEDRIVE
PKG_CONFIG_PATH
INFOPATH
HOMEPATH
ORIGINAL_PATH
LocalHome
VS140COMNTOOLS
_
PYTHONDONTWRITEBYTECODE
PIP_PYTHON_PATH

Pipenv–specific environment variables:
Debug–specific environment variables:

PATH: /usr/local/bin:/usr/bin:/bin:/opt/bin:/c/Windows/System32:/c/Windows:/c/Windows/System32/Wbem:/c/Windows/System32/WindowsPowerShell/v1.0/
SHELL: /usr/bin/bash
LANG: en_US.UTF-8
PWD: /home/USER

Contents of Pipfile ('/home/USER/Pipfile'):
[[source]]
url =<EMAIL_ADDRESS>verify_ssl = false
name = "artifactory"

[packages]

[dev-packages]

[requires]
python_version = "3.6"

Contents of Pipfile.lock ('/home/USER/Pipfile.lock'):
{
    "_meta": {
        "hash": {
            "sha256": "415dfdcb118dd9bdfef17671cb7dcd78dbd69b6ae7d4f39e8b44e71d60ca72e7"
        },
        "pipfile-spec": 6,
        "requires": {
            "python_version": "3.6"
        },
        "sources": [
            {
                "name": "pypi",
                "url": "https://pypi.org/simple",
                "verify_ssl": true
            }
        ]
    },
    "default": {},
    "develop": {}
}

Expected result
When using USER="name" PASS="test" pipenv install numpy -v I am expecting to have the numpy package installed.
Note that the USER and PASS variables are set in my Pipfile (see details above)
Actual result
When using the command the SSL authentication fails. And this is not surprising given the corresponding generated pip command obtained using the -v option:
/home/USER/.local/share/virtualenvs/USER-_5wChY8u/bin/pip" install   --verbose    "numpy" -i<EMAIL_ADDRESS>--trusted-host name --exists-action w
As can be seen the trusted-host parameter is not the server name but the user name...
Additional comments and questions

I am not sure this is a very good idea to show the login and password when using a verbose output (as it appears in the resulting pip command after the -i option)
I am using verify_ssl = false because I could not find any certificate option (I tried cert = path/to/cert/pem/format without success - it does not appear as a --cert pip option)

I think I found the culprit:
vendor/requirementslib/utils.py line 162
I think the line:
["--trusted-host", urlparse(sources[0]["url"]).netloc.split(":")[0]]
might/should be:
["--trusted-host", urlparse(sources[0]["url"]).hostname]
As for the other list of sources. Is there a specifc reason why this is different then for other sources ? If yes than an alternative would be to configure a second source as a private repo and keep the first pypi source in the Pipfile.

Might be related to (sorry for not noticing that earlier)
In addition I confirm that the error does not occure

Thanks for tracking this down, I'm not totally sure -- @uranusjr do you remember?

Ah… I think I was trying to be too cute with urlparse’s SplitResult. hostname is not a namedtuple attribute, and I feared (I think) our _replace calls might ruin it. I’ll need to dig into the docs (and maybe urlparse source) to make sure if we can safely use hostname. Or we need to implement some better parsing logic here.
I’ll look into this later tonight.

I do not know about the _replace calls but just in case it was unclear: The few lines below line 162 applies to other hosts than the first. these lines are using hostname. So whatever solution you choose it should probably be applied to all sources in the same way ?

It’s a bit complicated because of how the parser works. But the split is there for ports I assume? You might want netloc for that?

Checked the API, hostname should work here. PR opened at sarugaku/requirementslib#22.

I wonder if this is related to #2389 also


version number mismatch?

Dear nanopolish developers,
I did git clone --recursive https://github.com/jts/nanopolish.git a minute ago. After building the files, I found the version is still v0.6.2, not v0.7.  Did I miss something?
$ ./nanopolish --version
nanopolish version 0.6.2
Written by Jared Simpson.
Thanks!
Sharon

Sorry, forgot to push the commit with the new version number. Please try again!

Thanks Jared! It workded!


SetLocation

Hi!
This may sound stupid, but how does the setLocation function actually work?
I have this code:
Pokeio.SetLocation(location2, function(err, location){
          if (err) throw err;
          console.log('err' + err);
          console.log(location);
          console.log('location2');
          console.log(location2);

          location.latitude = 52.40937;
          location.longitude = 4.895058;
          location.altitude = 0;

          console.log(location);

        });

What does it actually do in the game?
Thanks.

atm this method is used to change the location in the playerInfo json created for the player, is not yet changing the location in game, not yet because heartbeat send also a position update so when we will complete the heartbeat method SetLocation will update in-game location too

allright, thanks!

Do you know what the eta is on finishing the heartbeat ?


[5412] Prevent Excel from using scientific notation when displaying HESA IDs

Context
https://trello.com/c/G1JsHEus/5412-s-handle-17-digit-hesaids-in-the-bulk-recommendation-csvs
Changes proposed in this pull request

Prefix HESA IDs with a single quote character to to prevent Excel from displaying it in scientific notation
Modify CSV importer to ignore single quote character

Guidance to review

Log in as a provider
Go through the bulk recommendation process and open CSV files in Excel to confirm that HESA IDs are no longer show in scientific notation

Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?
[ ] Do we need to send any updates to DQT as part of the work in this PR?
[ ] Does this PR need an ADR?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

Hello! looks great. Some things to look into:
When I download the CSV to 'review errors' now, the HESAID does not have the apostrophe at the front of it. So it would have the rounding issue when it's 17-digit I think.
@edwardhorsford and I were also talking and I think we'd agreed it would be better to have an apostrophe at the end as well - so they can see it regardless of which Excel/Numbers/Google spreadsheet viewer they are using. Is that ok John? Sorry for the slight change!

@claire-hughez Made the change. I can't replicate the issue where downloading the errors CSV doesn't show the single quote (apostrophe).


Add vertical regridding

Blocked by #71
Describe the solution you'd like

Vertical regridder that can remap hybrid vertical coordinates to standard pressure levels
Acceptance criteria:

[ ] Remapping to z-levels
[ ] Add more....

Additional context

cdms has reconstructPressureFromHybrid, logLinearInterpolation, and linearInterpolation

I came across this script that we may be able to refer to..

@tomvothecoder @chengzhuzhang the xgcm code base (also xarray) could be a useful reference, see https://xgcm.readthedocs.io/en/latest/transform.html

@durack1 Thank you for the reference.

Meeting Notes (7/20):

Alternative solution: Ryan Abernathey mentioned vertical regridding is available in xgcm

Maybe we can try to give this a go before implementing it ourselves?

Otherwise, added this item for potential implementation for v1.0.0 depending on the complexity

Add a feature specification doc
Jason and Jill work on this together?

@tomvothecoder Thank you for summarizing and updating this issue. I added CDAT's vertical regridding code as a reference.
Just a note here that the GeoCAT solution actually calls the interpolate from metpy package to actually do the vertical remapping. By a brief examination, I think both solutions should meet our requirements. And perhaps performance and use APIs will be two aspect we should focus on.

@jasonb5 Hey, Jason. I spent some time to experiment with the existing vertical regridding methods. Please see the jupyter notebook here: https://github.com/xCDAT/xcdat_test/blob/main/validation/v1.0.0/vertical_interp/Vertical_interp.ipynb
Notice that only xgcm method (the third one in the notebook) is working. And I have hard time to trouble shooting the first two methods. When you have time, would you please also take a look for me. Let me know if you'd like any clarification on this. Thank you!

@chengzhuzhang Thanks, I'll give it a look over.

Hey @jasonb5 , as discussed I added a 4th method, which is to use the cdms2/cdutil approach. The note book now is updated. https://github.com/xCDAT/xcdat_test/blob/main/validation/v1.0.0/vertical_interp/Vertical_interp.ipynb Thank you for looking over!

Hi @jasonb5 – this looks great!
Adding a few comments based on a minimal atmospheric example (code below for reference):

I notice that the target_lev dataset that I create needs to include the coordinate lev, though standard pressure level data is typically named plev. Would it be hard to give user control over the name of the level name (perhaps later as an improvement)? Or should the user just rename this after regridding?
Note the FutureWarning – should the syntax Grid.transform be implemented now (I didn't actually look at the code to see if this comment makes sense or not – this is based only off the warming)
Is it easy to get rid of the logger warning about missing latitude units (by solving the underlying issue)? Why is this happening for latitude and not longitude?
Can the method options be added to the docstring or would this be problematic (maybe this is actually a xgcm option)? If this is easy to add and isn't bad practice, it would be helpful to add it.
Does the xgcm method have a docstring that you can link to from ds.regridder.vertical? You do this for ds.regridder.horizontal (pointing to xcdat.regridder.xesmf.XESMFRegridder, for example). I'm curious what theta is in the docstring example (maybe this addresses my first comment?).
The output units on the regridded dataset are incorrect. The minimal example below doesn't provide enough information to add the units, so this makes sense, but it seems like the output units / attributes for lev should be pulled from the target dataset rather than copied from the source dataset. It could be good to address this.
Suggestion for an API Change (for consideration / comment):

This comment is based on the behavior from CDAT, which accepts a list of target levels. Could the target_grid (called output_grid in the docstring) be either a List, Dataset, or String? I'm envisioning the following behavior:

If a dataset – keep the core logic the same (except copy the level attributes to the output dataset);
If a List – generate a target dataset, which can then be used for vertical regridding (output level attributes would be largely un-populated since they are not specified);
If a String (either standard_atmosphere or standard_ocean): create a standard target dataset (with appropriate attributes, e.g., axis, units, name, etc.), which is then used for vertical regridding.

Simple example code
import xcdat as xc
import numpy as np

fn = '/p/css03/esgf_publish/CMIP6/CMIP/MPI-M/MPI-ESM1-2-HR/historical/r8i1p1f1/CFmon/ta/gn/v20190710/*.nc'
ds = xc.open_mfdataset(fn)
p = ds.ap + ds.b*ds.ps
target_lev = xc.create_grid(lev=np.linspace(100000, 1, 13))

ds_regrid = ds.regridder.vertical("ta", target_lev, method="log", target_data=p)

/../xgcm/grid.py:987: FutureWarning: From version 0.8.0 the Axis computation methods will be removed, in favour of using the Grid computation methods instead. i.e. use Grid.transform instead of Axis.transform
warnings.warn(
2023-04-18 14:09:52,753 [WARNING]: bounds.py(_create_bounds:398) >> The 'lat' coordinate variable is missing a 'units' attribute. Assuming 'units' is 'degrees_north'.

Example output
ds_regrid.lev

<xarray.DataArray 'lev' (lev: 13)>
array([1.000000e+05, 9.166675e+04, 8.333350e+04, 7.500025e+04, 6.666700e+04,
5.833375e+04, 5.000050e+04, 4.166725e+04, 3.333400e+04, 2.500075e+04,
1.666750e+04, 8.334250e+03, 1.000000e+00])
Coordinates:

lev      (lev) float64 1e+05 9.167e+04 8.333e+04 ... 1.667e+04 8.334e+03 1.0
Attributes:
bounds:         lev_bnds
units:          1
axis:           Z
positive:       down
long_name:      hybrid sigma pressure coordinate
standard_name:  atmosphere_hybrid_sigma_pressure_coordinate
formula:        p = ap + b*ps
formula_terms:  ap: ap b: b ps: ps

Tagging @mzelinka / @chengzhuzhang – to see if they have comments on my suggestions / comments.


pigar unable to recognize html5lib

So this is the code snippet I was working with:
from bs4 import BeautifulSoup
import requests

r = requests.get("some-url")
soup =BeautifulSoup(r.content, "html5lib")
...

When I later generated requriements.txt with pigar, bs4 and requests were there, but not the parser html5lib.

You can import it explicitly to resolve this issue.
I don't see a good and simple solution for this, since analyzing the dependencies of dependencies is complicated and we can not execute users' program safely to do some magical things.

You can import it explicitly to resolve this issue.
I don't see a good and simple solution for this, since analyzing the dependencies of dependencies is complicated and we can not execute users' program safely to do some magical things.

I recommend you try some package management tools, pigar isn't one.

I recommend you try some package management tools, pigar isn't one.


Remove the warning that appears on every page.

I personally don't think this warning is necessary, and it seems a little scary.  I also think there is sufficient notice on the Introduction page that this is incomplete and non-normative.  Additionally, there is now a small link in the upper-right corner of every page to file github issues.
Of course this is up to @Havvy who added this in #122.  Do you think the reference has come along enough in the past 2 years to remove this?
Closes #681

It's probably fine to remove the warning. I don't know if everything has been vetted so far, but most of it has.
r? @Centril


Closing a path does not close all shapes.

When a path has multiple shapes, clicking “closed” only closes the last one. I think a z should be added before any m or M command in addition to adding one at the very end.

You're right. It will be fixed in the next update.


quickemu: fails w/ "Couldn't find current GLX or EGL context"

Describe the bug
quickemu 4.7 fails:
$ quickemu --vm alpine-latest.conf
/nix/store/kvwq3y9xfbiiiymirgm34hr35gl5b8ix-quickemu-4.7/bin/.quickemu-wrapped: line 1746: kill: (23959) - No such process
Quickemu 4.7 using /nix/store/h601yz544qv1vw8r621ixlqrf71bnf78-qemu-8.0.0/bin/qemu-system-x86_64 v8.0.0
 - Host:     "openSUSE Tumbleweed" running Linux 6.3 (pop)
 - CPU:      AMD Ryzen 9 5900X 12-Core Processor
 - CPU VM:   1 Socket(s), 8 Core(s), 2 Thread(s), 8G RAM
 - BOOT:     EFI (Linux), OVMF (/nix/store/lrlk1f5c1vd6b3mi21q42pxj9cgf94w5-OVMF-202211-fd/FV/OVMF_CODE.fd), SecureBoot (off).
 - Disk:     alpine-latest/disk.qcow2 (16G)
             Looks unused, booting from alpine-latest/alpine-virt-3.18.0-x86_64.iso
 - Boot ISO: alpine-latest/alpine-virt-3.18.0-x86_64.iso
 - Display:  SDL, virtio-vga-gl, GL (on), VirGL (on)
 - ssh:      On host:  ssh user@localhost -p 22220
 - WebDAV:   On guest: dav://localhost:9843/
 - 9P:       On guest: sudo mount -t 9p -o trans=virtio,version=9p2000.L,msize=104857600 Public-myuser ~/myuser
 - smbd:     On guest: smb://<IP_ADDRESS>/qemu
 - Network:  User (virtio-net)
 - Monitor:  On host:  nc -U "alpine-latest/alpine-latest-monitor.socket"
             or     :  socat -,echo=0,icanon=0 unix-connect:alpine-latest/alpine-latest-monitor.socket
 - Serial:   On host:  nc -U "alpine-latest/alpine-latest-serial.socket"
             or     :  socat -,echo=0,icanon=0 unix-connect:alpine-latest/alpine-latest-serial.socket
qemu-system-x86_64: ../src/dispatch_common.c:885: epoxy_get_proc_address: Assertion `0 && "Couldn't find current GLX or EGL context.\n"' failed.
 - Process:  Starting alpine-latest.conf as alpine-latest (28946)
~/tmp [1] c quickemu-4.7

Steps To Reproduce
Happens systematically on my system (opensuse tumbleweed, quickemu installed via home-manager)
Notify maintainers
@FedX-sudo
Metadata
$nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.3.1-1-default, openSUSE Tumbleweed, noversion, nobuild`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.3`
 - channels(giorgio): `"home-manager"`
 - nixpkgs: `/nix/store/cyqa3d8y0q5dn6mmpyvpr10cz946scqf-source`

non-NixOS
GL-related error

https://nixos.org/manual/nixpkgs/unstable/#nix-on-gnulinux

If you are using a non-NixOS GNU/Linux/X11 desktop with free software video drivers, consider launching OpenGL-dependent programs from Nixpkgs with Nixpkgs versions of libglvnd and mesa.drivers in LD_LIBRARY_PATH.

Check if https://github.com/guibou/nixGL works around it for you.

nix run --override-input nixpkgs nixpkgs/nixos-21.11 --impure github:guibou/nixGL -- quickemu --vm alpine-latest.conf works.
Thank you very much for the assistance.
I assume this is the expected behaviour and so I'm closing this issue.

It's sadly a known problem, see https://github.com/NixOS/nixpkgs/issues/9415 for the relevant issue.

Adding  --display spice worked for me.
quickemu --vm macos-catalina.conf --display spice


Adds option to create event annotations where actual event(s) binding is configurable

You did it @rakeshnpatel!
Thank you for signing the Anthem Opensource Software CLA.

Overall the code looks beautiful!
Just like the change you have for State Event annotation, could you please extend the same to other Event annotations (Config annotations, Txn annotations, Command Lifecycle annotations, etc). We are yet to implement the event handlers and event management system for the rest.


Unable to build ruby on ARMv7 architecture

Hi rbenv team,
I tried to install ruby on my scaleway C1 instance, ARMv7 architecture but so far I haven't been able to do it. Here's the error log
root@pup:~# rbenv install 2.5.0-dev
Cloning https://github.com/ruby/ruby.git...
Installing ruby-trunk...

BUILD FAILED (Ubuntu 18.04 using ruby-build 20170726)

Inspect or clean up the working tree at /tmp/ruby-build.20181012154306.7388
Results logged to /tmp/ruby-build.20181012154306.7388.log

Last 10 log lines:
making mjit_config.h
generating mjit_compile.inc
compiling node.c
compiling numeric.c
compiling pack.c
generating parse.c
make: bison: Command not found
uncommon.mk:822: recipe for target 'parse.c' failed
make: *** [parse.c] Error 127
make: *** Waiting for unfinished jobs....

Could you please help me out here ?

I'm using this link as my guide
https://qiita.com/takeru56/items/ab09e239dc38e4da5ea1

Hi, the correct project that handles building ruby versions is: https://github.com/rbenv/ruby-build/wiki
Not this one. Your build log suggests that you don't have bison installed. Maybe first check that you have all requisite build tools installed (sudo apt-get install build-essentials) and only then try to compile Ruby versions?
Thank you

Ohh, I see!
Thanks Mislav +1
On Tue, Nov 20, 2018 at 6:44 PM Mislav Marohnić<EMAIL_ADDRESS>wrote:

Hi, the correct project that handles building ruby versions is:
https://github.com/rbenv/ruby-build/wiki
Not this one. Your build log suggests that you don't have bison
installed. Maybe first check that you have all requisite build tools
installed (sudo apt-get install build-essentials) and only then try to
compile Ruby versions?
Thank you
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rbenv/rbenv/issues/1124#issuecomment-440269047, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AMNNXpON9xdByELpZl2m0nk_90BOqbvPks5uxABBgaJpZM4XZvEt
.


Picovoice Documentation Issue (/docs/quick-start/console-signup/)

What is the URL of the doc?
What's the nature of the issue? (e.g. steps do not work, typos/grammar/spelling, etc., out of date)

I don't actually see an issue! It's just the template... closing.


Fix syntax error in example code.

This example code doesn't compile.

Hi @datermine - before we merge this, you need to sign the CLA. Would you please take a minute to do that.  Thanks!

I've already signed it.

Thanks - I've confirmed that @datermine has signed the CLA. The bot is cranky because it sees that the PR was authored by georgemike.com - not by "you". I'll override and merge, thanks.


allow to specify model and audio file for whisper

Adds cli arguments to choose the model and audio file. Multilingual doesn't seem to work though, it'll need a bit of more work.

Closing for #2074


shims: add new to fix xml2-config --libs output

[ ] Have you followed the guidelines in our Contributing document?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

Although this is coming far too late to be of much practical use, it's still useful information.
A few formulae currently set ENV["SDKROOT"] = MacOS.sdk_path to fix building with libxml2 on Sierra. But, the setting of SDKROOT causes the build to fail with "C compiler cannot create executables" / "SDK ... cannot be located" if xcode-select -p indicates the active SDK is Xcode's, resulting in formulae that suddenly won't build if Xcode is installed.
First, why was adding SDKROOT needed in the first place? The most common comment says it fixes "ld: file not found: /usr/lib/system/libsystem_darwin.dylib" for lxml, or some similar message relating to libxml2. That particular error occurs only on 10.12 because a .tbd file in the 10.13 SDK references a library that is not actually included with 10.12.
Ordinarily this wouldn't be a problem, except that the xml2-config that ships with macOS and the libxml2 formula makes the error of using -L instead of -isysroot to refer to the Xcode SDK when it's active. If the CLT is active, the -L reference is not generated.
# on macOS 10.12
$ sudo xcode-select -r
$ xcode-select -p
/Applications/Xcode.app/Contents/Developer
$ /usr/bin/xml2-config --libs
-L/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.13.sdk/usr/lib -lxml2 -lz -lpthread -licucore -lm

$ sudo xcode-select --switch /Library/Developer/CommandLineTools
$ xcode-select -p
/Library/Developer/CommandLineTools
$ /usr/bin/xml2-config --libs
-lxml2 -lz -lpthread -licucore -lm

And since xml2-config is a shell script, we can see that this is occurring only because xcrun -show-sdk-path returns a path only when Xcode's is active, and no value when the CLT is active.
$ xcrun -show-sdk-path

$ sudo xcode-select -r
$ xcrun -show-sdk-path
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.13.sdk

On later macOS versions, xcrun -show-sdk-path prints the path to the CLT's SDK when the CLT is active, causing xml2-config --libs to always include -L<sdk path> in its output, but since /usr/lib/system/libsystem_darwin.dylib is present, it doesn't cause an issue.
For example, libslax builds on Sierra with or without SDKROOT being set as long as the CLT is active. From ~/Library/Logs/Homebrew/libslax/01.configure:
# CLT without SDKROOT fix
  libxml libs:       -lxml2 -lz -lpthread -licucore -lm

# CLT with SDKROOT fix
  libxml libs:       -L/Library/Developer/CommandLineTools/SDKs/MacOSX10.13.sdk/usr/lib -lxml2 -lz -lpthread -licucore -lm

When Xcode is active, it fails with either "libsystem_darwin.dylib" or fails to find clang at all.
# Xcode without SDKROOT fix
  libxml libs:       -L/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.13.sdk/usr/lib -lxml2 -lz -lpthread -licucore -lm

# Xcode with SDKROOT fix
  checking whether the C compiler works... no

It remains unknown why setting SDKROOT causes issues when Xcode's SDK is active:
# CLT active
$ SDKROOT=/Library/Developer/CommandLineTools/SDKs/MacOSX10.13.sdk/ clang --version
Apple LLVM version 9.0.0 (clang-9<IP_ADDRESS>)
Target: x86_64-apple-darwin16.7.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin

# Xcode active
$ SDKROOT=/Library/Developer/CommandLineTools/SDKs/MacOSX10.13.sdk/ clang --version
xcodebuild: error: SDK "/Library/Developer/CommandLineTools/SDKs/MacOSX10.13.sdk/" cannot be located.
clang: error: unable to find utility "clang", not a developer tool or in PATH

What this shim does is take the output of xml2-config --libs from the next xml2-config in the path (whether built-in or from the formula) and change -L to -isysroot. This allows the following formulae to build under macOS 10.12 when Xcode is active and their SDKROOT fix is removed… checks notes… xmlrpc-c, sqliteodbc<EMAIL_ADDRESS>flowgrind, uwsgi. The others have either been fixed by using a newer lxml or aren't fixable by the shim alone.
Like I said, a rather late and probably ignorable fix, but I'll be linking some formula PRs back to here shortly to explain my changes.

Review period will end on 2022-04-08 at 01:39:37 UTC.

And since xml2-config is a shell script, we can see that this is occurring only because xcrun -show-sdk-path returns a path only when Xcode's is active, and no value when the CLT is active.
$ xcrun -show-sdk-path

$ sudo xcode-select -r
$ xcrun -show-sdk-path
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.13.sdk

This is interesting. What happens if you move this out of the if condition:
https://github.com/Homebrew/brew/blob/fa4f239cd6e7b96de54825d779ca2a8ac3ff7ec2/Library/Homebrew/extend/os/mac/extend/ENV/super.rb#L94-L98
and then invoke xcrun -show-sdk-path in brew sh?

Reluctant to introduce a new shim if its just for Sierra. Interested in thoughts on @Bo98's question.

Review period ended.

That block in super.rb isn't being reached in this case, but it did lead me to the shim-free secret sauce solution: in most cases, setting ENV["HOMEBREW_DEVELOPER_DIR"] = MacOS::CLT::PKG_PATH in the formula instead of changing SDKROOT fixes the issue.

Well yes it's not being reached at the moment because it's conditioned to not run prior to Mojave. What I'm proposing to fix that is:
diff --git a/Library/Homebrew/extend/os/mac/extend/ENV/super.rb b/Library/Homebrew/extend/os/mac/extend/ENV/super.rb
index 9e96f2f7e..ec713cca7 100644
--- a/Library/Homebrew/extend/os/mac/extend/ENV/super.rb
+++ b/Library/Homebrew/extend/os/mac/extend/ENV/super.rb
@@ -90,16 +90,16 @@ module Superenv
     if MacOS.sdk_root_needed? || sdk&.source == :xcode
       Homebrew::Diagnostic.checks(:fatal_setup_build_environment_checks)
       self["HOMEBREW_SDKROOT"] = sdk.path
-
-      self["HOMEBREW_DEVELOPER_DIR"] = if sdk.source == :xcode
-        MacOS::Xcode.prefix
-      else
-        MacOS::CLT::PKG_PATH
-      end
     else
       self["HOMEBREW_SDKROOT"] = nil
-      self["HOMEBREW_DEVELOPER_DIR"] = nil
     end
+
+    self["HOMEBREW_DEVELOPER_DIR"] = if sdk&.source == :xcode
+      MacOS::Xcode.prefix
+    else
+      MacOS::CLT::PKG_PATH
+    end
+
     generic_setup_build_environment(
       formula: formula, cc: cc, build_bottle: build_bottle,
       bottle_arch: bottle_arch, testing_formula: testing_formula


package cleanup

The Problem

The Solution

Some of these packages are needed. We need ts-natsutil in non-dev dependencies.


Restic 0.9.6 release

Restic just got a new release: https://github.com/restic/restic/releases/tag/v0.9.6

Thanks. Seems to be a smaller one. Will have a look at it soon.


Modifications necessary to compile against Java8, fixes javadoc errors.

The javadoc compiler in Java 8 is more strict than previous versions.  This change fixes the javadoc errors so that the project can build built with Java 8.

Mahmoud Ben Hassine » jPopulator #115 SUCCESS
This pull request looks good
(what's this?)


Support for grails-schwartz plugin

Grails Schwartz is also very popular for quartz scheduling in Grails3.

I've created a Grails Quartz plugin agnostic version, which supports the Quartz and Schwartz plugins. Interested in it? If yes, I send a PR.

Happy to take a look

See #42 which is open since JUN/01/17.


Reference svg clip path problem when combined with 3d transforms

Bug being tracked here:
https://code.google.com/p/chromium/issues/detail?id=350724

Added as known issue.


[19.03 backport] Cleanup VFP during overlay network removal

clean cherry-pick of #2524

/cc @selansen + @euanh


[GOBBLIN-1774] Util for detecting non optional unions Hive tables

This util will be used across GMIP and compaction to handle non optional unions Non optional unions are compatible with Avro / Orc but not in Iceberg, so special workarounds are necessary to have tables with both types of data

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[ ] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-XXX

Description

[ ] Here are some details about my PR, including screenshots (if applicable):

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Codecov Report

Merging #3632 (bd2f2fc) into master (240aee2) will increase coverage by 2.43%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #3632      +/-   ##
============================================
+ Coverage     46.57%   49.01%   +2.43%
+ Complexity    10667     9200    -1467
============================================
  Files          2133     1751     -382
  Lines         83539    67899   -15640
  Branches       9286     7741    -1545
============================================
- Hits          38910    33282    -5628
+ Misses        41064    31522    -9542
+ Partials       3565     3095     -470

Impacted Files
Coverage Δ

...che/gobblin/hive/metastore/HiveMetaStoreUtils.java
52.32% <100.00%> (+5.96%)
:arrow_up:

...g/apache/gobblin/hive/orc/HiveOrcSerDeManager.java
57.29% <0.00%> (-3.13%)
:arrow_down:

.../modules/scheduler/GobblinServiceJobScheduler.java
63.03% <0.00%> (-1.43%)
:arrow_down:

...main/java/org/apache/gobblin/yarn/YarnService.java
15.03% <0.00%> (-0.66%)
:arrow_down:

...a/org/apache/gobblin/azkaban/AzkabanJobRunner.java

...obblin/metrics/influxdb/InfluxDBEventReporter.java

.../extractor/extract/google/GoogleDriveFsHelper.java

...bblin/compliance/restore/ComplianceRestoreJob.java

...org/apache/gobblin/kafka/tool/KafkaCheckpoint.java

...bblin/metrics/graphite/GraphiteConnectionType.java

... and 382 more

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

Squashed


Comment reference picture assignment

Coverage remained the same at 80.613% when pulling 3dc8dd0a1e08e797def95c9d62e2191b43354baa on tdaede:comment_references into 98f4d66d1fc5456a4b3178c8694fb2d03d44504a on xiph:master.


build(modules): add :modules:reportModuleOrder task

I wanted to know the best order in which to push module updates so that when they build on CI, a module won't try to build before its dependencies have been pushed.

JGraphT looks pretty nice! If I understand correctly this hooks up to gradlew calls. I see that this is the logical place where we have the module dependency graph anyways, but on the other hand I'm not sure whether our central build tooling is the place where this should go. Would this fit into the picoCLI groovyw tooling bucket (once we get there)?
Reasoning for this comment is that I don't think that this info is/should be used by the CI, at least not in the current setup. It is more about developer tooling, isn't it?
Identifying these clusters (or levels) would be really helpful to know when you hit the point where you have to wait before pushing more things. For full automation, we'd need to figure out how to get informed that the build for a commit was finished, and the push-train can move on... not sure whether/how the Github API could help us with that, I might have a look at that 🙃

For full automation, we'd need to figure out how to get informed that the build for a commit was finished, and the push-train can move on... not sure whether/how the Github API could help us with that.

hmm. We know that PRs have checks attached to them, but can be the same be said for other branches? Does GitHub have a way to attach a status report to the current state of develop?
I'd be inclined to look straight to Jenkins without involving GitHub as an intermediary. Or, heck, even the package repository, because we're really waiting until a new SNAPSHOT build has been uploaded, right?
but on the other other hand, I was thinking of this as the quick and dirty kludge that could be done with only local information. If it turns in to a project of integrating with other service APIs and synchronizing to changes in remote server state, I'd be inclined to invest that effort in improving the workflow on the Jenkins side instead.


Blue Feedback icon does not disappear after removing feedback

Describe the bug

The new blue "!" exclamation mark does not disappear despite the feedback field being empty.
To Reproduce

Expected behavior

The icon should be gone if no feedback is given on an element (behaviour before the update).
Environment

 - OS: macOS Mojave 10.14.3
 - Browser: Chrome 75.0.3770.142
 - Version: Artemis 3.4.1

@TobiasPr maybe you can have a quick look, if this problem still occurs. If yes, this might be easy to fix.

Yes, that is easily fixable. One question though: Whats the logic behind marking something with the blue exclamation mark? Is it just that there exists a written feedback?
Curernt logic in Apollon looks like this:
{assessment.score === 0 && <FeedbackIcon />}
{assessment.score > 0 && <CorrectIcon />}
{assessment.score < 0 && <WrongIcon />}

so it does not take the feedback into account at all, its just evaluating the current score. Once set, the current score can never become null again, that's why we always have the exclamation mark

fixed in apollon 2.0.11


Enable toggle behaviour for buttons that trigger popups

So they actually highlight when their popups are open:

While this looks cool to me, this breaks compatibility for ColorPicker, OptionButton and MenuButton IINM, so it can't really be considered for 3.1 at this stage.
We can see if it can be adapted for 3.2 so that it doesn't break compat, otherwise it would have to wait for 4.0.

@akien-mga How so? Unless someone really depends on these buttons to not be in toggle mode, I don't see how this breaks compatibility.

Thanks!

@Calinou Could you mention in the changelog that ColorPicker, OptionButton and MenuButton now have a toggle mode with is enabled by default (thus a behaviour change compared to 3.1).

@akien-mga Done :slightly_smiling_face:


General setup question

AllSky is working fine for me but want to get a better understand of one of the folders.
In the ftp-settigns.sh there is a directory that is called IMGDIR.  That folder is created on the hosting company site.  Been looking in that folder and it appear that is empty.
So is that folder a temp folder for file processing of images? should there be something in that folder? is something missing on the current setup being used?
Thanks
Mike

@maserowik, good questions, and they may help us improve the documentation, so thanks for asking.
The variables in ftp-settings.sh should point to whatever directories you create on your FTP server.  These are totally independent of the folders on your Pi, and you could, if you were crazy, create a "startrails" folder on the FTP server and have Allsky copy the keogram to that folder.
The defaults in ftp-settings.sh when you install Allsky are there simply to give you an idea of what to put in the variables.
On my FTP server, I put everything in the topmost directory, and the FTP server balks if I use "/", so all my variables look like this:     IMGDIR=''"
Allsky doesn't create any folders on your FTP server - you have to do that before you try to upload files, otherwise uploads will fail.
Let me know if you have more questions about this, otherwise, please close the issue.  Thanks.

I have those folder on my web hosting site.  I have data in the startrails and keograms but I have nothing in the images folder.  Just trying to figure out what files or ?? go in to that folder.  The data.json and the image.jpg are in the root of the allsky folder on my hosting site.
here are my current settings I use

@maserowik, do you have UPLOAD_IMG="true" in config.sh?  That will upload the current image (e.g., "image.jpg") to IMGDIR so your web page can display it.  Assuming the name of the uploaded image doesn't change, IMGDIR will only ever hold one file, the most recent image.
The other 3 directories on your web server will have many files - each with the date in its name.
Do you have "allsky-website" installed on your web server (it's the one that allows you to overlay the star field on the image)?  It knows about those directories.
Let me know if you have more questions.

Eric,
I had a typo in the setup on my end.  I did not add the images to the imgdir in the ftp-settings.sh.
Thanks for you help!!


goto definition not working/inconsistent in vim

Steps to reproduce

Install purescript-vim
Configure coc-settings.json like so
npm install -g purescript-language-server
Observed behavior
:CocInfo shows that the Purescript language server has started
Code completion/typeahead works but code navigation fails with the error [coc.nvim] Definition not found

However if you

Quit vim
Start VSCode with the PureScript IDE and PureScript Language Support plugins installed
Start vim (same purescript project)
Observed behaviour
:CocInfo shows vim finds and attaches to existing server (started by vscode)
Code navigation works even though the [coc.nvim] Definition not found error flashes at the bottom of the screen.

That sounds like your VSCode and vim settings are just not the same, code navigation will only work if purs ide is provided the correct source globs, in particular for your dependencies.  e.g. if using Spago, set purescript.addSpagoSources true - you can check your config in VSCode to see what config you want to set, it should show the correct config options.
https://github.com/nwolverson/vscode-ide-purescript/blob/master/package.json

@s-u-m-e Could you double check if this issue is still the same for you after https://github.com/nwolverson/purescript-language-server/pull/90 ? (Once released, or use master) It might have solved your issue.
If it still the case, could you add more details, and an open source repository with instructions on how to reproduce the issue? Thanks.

@andys8 just tried it out and code navigation works. ( The message [coc.nvim] Definition not found still flashes but that's another thing)

Can you post/link to your complete configuration? Maybe there is something duplicated and a second process answers with a negative answer?

@andys8 thanks for the quick response, here's my current setup
https://github.com/s-u-m-e/nvim

I notice you have "--config {}" but also purescript under settings, if settings work then you should not set --config on the command line, it may prevent them from working. (You can also check the command line of the purs ide server process launched by the purescript-language-server process, if the source globs include your packages or not)

So, from my side it's only guessing.
There is this line mentioning vimmerps:
https://github.com/s-u-m-e/nvim/blob/310f55ca0fcd55777235aeb94cbb4de53a066743/init.vim#L167
But it's not installed anymore. Could it be the case that it wasn't removed cleanly? With the described behavior it could be that two plugins or servers are executed. The language server shouldn't work and not work at the same time. Wild guess.

I commented out the --config{} line from the config, here's the output of :CocInfo. @nwolverson how can I check if the source globs include my packages? running purs ide server from the command line seems to work but there is no output
## Output channel: languageserver.purescript
  [Info  - 6:50:51 PM] {"processId":37477,"rootPath":"/Users/sume/tormeda/ps/purescript-book/exercises/chapter6","rootUri":"file:///Users/sume/tormeda/ps/purescript-book/exercise
  Starting with args: ["/usr/local/bin/node","/usr/local/bin/purescript-language-server","--stdio"]
  Starting with cwd: /Users/sume/tormeda/ps/purescript-book/exercises/chapter6 and using root path: /Users/sume/tormeda/ps/purescript-book/exercises/chapter6
  Got new config (client push)
  [Info  - 6:50:51 PM] Starting IDE server
  [Info  - 6:50:51 PM] Using sources from psc-package/spago packages (PURS_IDE_SOURCES not set)
  PureScript Language Server started
  [Info  - 6:50:51 PM] Resolved IDE server paths (npm-bin: false) from PATH of /Users/sume/.yarn/bin:/Users/sume/.config/yarn/global/node_modules/.bin:/Users/sume/google-cloud-sd
  [Info  - 6:50:51 PM] /usr/local/bin/purs: 0.13.8

  [Info  - 6:50:51 PM] Found existing port from file: 15912
  [Info  - 6:50:51 PM] Starting IDE server on port 15834 with cwd /Users/sume/tormeda/ps/purescript-book/exercises/chapter6
  Started IDE server (port 15834)
  [Info  - 6:50:52 PM] [Info] cachedb was changed from: Nothing, to: Just 2020-08-22 03:02:16.87983146 UTC @(main:Command.Ide app/Command/Ide.hs:205:25)

Use ps to inspect the command line of the existing process, eg ps aux|grep purs

Looks like there's only one process running which stars when I open a .purs file, and it stops when I exit vim. Everything seems to work except sometimes goto definition doesn't work the first time and shows the message [coc.nvim] Definition not found but works on the second try.
https://asciinema.org/a/R7AAhspJH15YWACKOM7AGATdi
I can close this issue since the initial problem of not being able to navigate to definition is fixed. I'm not really bothered by the error message

The behavior you describe is happening for me too and I guess expected. It takes some time to initially load stuff (I guess). Afterwards it start's to work. 👍

Initial issue is resolved


Add Lens Flare System

🎉 New feature
Closes #1909
Summary

Added Lens Flare System that adds Lens Flare Render Pass to the camera and allows users to enable lens flare in camera images. The lens flares can be configured using <scale>, <color>, and <occlusion_steps> tags in the SDF.
Added an example camera_lens_flare.sdf file for testing which contains 2 cameras(wide angle and normal) with different configurations for the lens flare (one default and other custom).

Test it

Make sure to use the gz-sim8 and compatible versions of other packages especially gz-rendering8 since it contains the Lens Flare Pass.
Launch the examples/worlds/camera_lens_flare.sdf file from the gz-sim repo using:

gz sim camera_lens_flare.sdf

Checklist

[x] Signed all commits for DCO
[x] Added tests
[x] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[x] codecheck passed (See contributing)
[x] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

🎈 Release
Preparation for <X.Y.Z> release.
Comparison to <x.y.z>: https://github.com/gazebosim//compare/<LATEST_TAG_BRANCH>...<RELEASE_BRANCH>

Needed by <PR(s)>
Checklist

[ ] Asked team if this is a good time for a release
[ ] There are no changes to be ported from the previous major version
[ ] No PRs targeted at this major version are close to getting in
[ ] Bumped minor for new features, patch for bug fixes
[ ] Updated changelog
[ ] Updated migration guide (as needed)
[ ] Link to PR updating dependency versions in appropriate repository in gazebo-release (as needed):

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

can you also sign your commits?

@iche033 I have made all the suggested changes in the latest commit and have signed all previous commits as well ✔

@iche033 Not sure how I missed that one 😅. Anyways, I have made the required changes.


Old react dependency

Old react and reactdom dependencies are interfering with development with newer packages as react is on version 16. Will this package be updated?

Check the next branch. I'll merge and publish when I have the time.

sweet deliverance! Thanks


Emit metrics on deprecated APIs dynamically

Signed-off-by: Daniel Sover<EMAIL_ADDRESS>Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1958296

Description of the change:
Motivation for the change:
Reviewer Checklist

[ ] Implementation matches the proposed design, or proposal is updated to match implementation
[ ] Sufficient unit test coverage
[ ] Sufficient end-to-end test coverage
[ ] Docs updated or added to /doc
[ ] Commit messages sensible and descriptive

Closing in favor of another PR.


Little released version 9.10.0 and 10.1 is not released correctly

The components pom.xml is released incorrectly which makes the download of the component from maven an issue.
For 9.3.3, the pom was released as litle-sdk-for-java-9.3.3.pom (which is correct)
But for release 9.10.0 and 10.1 it was released as pom.xml . This makes the download to maven based projects fail.
Can you please release this properly?

Maven repository has been fixed for 9.10.0, version 10.0 is not release for GA, so I will not be changing the pom file at this time.

Thanks for the quick response.

While updating, I just noticed that the pom is still not correct.
The file name should be litle-sdk-for-java-9.10.0.pom and not litle-sdk-for-java-9.10.0.xml
Can you please correct this?

.pom file should have the correct extension now


Make top & bottom .label padding the same

Use to look like this:

padding: .2em .6em .3em

top
horizontal
bottom

.2em
.6em
.3em

Now looks like this:

padding: .3em .6em;

vertical
horizontal

.3em
.6em

CC: @mdo

Sorry, but we won't be changing v3's CSS for stuff like this at this point in time. Way too far in it's lifecycle to make changes to these kind of values. Critical fixes only at this point. Thanks though!


KAFKA-2402: Create IsrChangeNotificationPath when server statrs.

The change looks good although I wonder if there's anything we can do to make it less likely for similar problems to occur in the future. An automated test would be ideal, but if that is hard then a potential alternative would be to move the Seq of common paths in setupCommonPaths to a val near the other constants along with a comment explaining under what circumstances new paths should be added there.

LGTM! However, an unit test or integration test will be helpful.

@ijuma @samshah , unit test might be a little bit hard to do here simply because the issue will only occur during version upgrade. We need a rolling upgrade test as proposed in KAFKA-1888. This is probably a non-trivial work and maybe out of the scope of this simple fix?

@becketqin, fair enough. Do you think it's worth moving the Seq to be near the constants along with a comment as I suggested? In any case, this is an improvement, so LGTM if you don't think it's worth it.
Note that this branch doesn't merge cleanly anymore so that needs to be fixed before it can merged.

@becketqin, I think this change was included as part of https://github.com/apache/kafka/commit/2c55bd8aa2666ea11ee2814b04f782eef8fa52c6#diff-f4fe18d654e20048317014cfe9d53968R61 right? If so, can you please close this PR?
And thanks for making the change I suggested.

@ijuma Yes, this patch was put in KAFKA-2406 already. I'll close it.


http://json-schema.org/draft-03/schema#/properties/required attribute not supported (fails validation)

The online tool doesn't understand the http://json-schema.org/draft-03/schema#/properties/required boolean attribute.
try:
{
"type": "object",
"properties": {
"id": {
"type": "string",
"required":true,
"ipsum": "id"
}
}
}
returns:
[{"uri":"urn:uuid:da24b2e4-4ab4-4397-bc49-8992177370e4#/properties/id/required","schemaUri":"http://json-schema.org/draft-03/schema#/properties/required","attribute":"type","message":"Instance is not a required type","details":["boolean"]}]
It should accept "required":true or "required":false (default).
Could you please fix?

Looks like this is a duplicated issue . . . https://github.com/jonahkagan/schematic-ipsum/issues/19


Add functionality required by std.log

Describes all the changes to druntime required by std.log:

Add posix module utsname for getting system information.
Modifed the windows module to add support fo getting the computer
name and user name.

Anything else I need to do to get this merged into druntime? Having this there would be nice so we can start reviewing std.log. It would be nice if reviewer are able to run the std.log code while reviewing the module.
Thanks!


fix opacity issue in scene graph layer

Change List

enable opacity in scene graph layer

Coverage increased (+2.5%) to 83.552% when pulling 061a67b0c058551022f11bfea1f8fdc1518e6b02 on fix_opacity into 980435246bb2da323edb4d6f327746c86f3b59dd on master.

There was already a PR fixing this. https://github.com/uber/deck.gl/pull/3488 Please check existing PRs before opening new ones.

Different mesh layers...
@jianh merge the other one as well before publishin?


Console output from bash.exe and lxrun.exe are formatted incorrectly when run from within bash (or from cmd.exe inside bash)

I'm running build 15063.138.
Running bash.exe inside bash fails (see #2023). Its console output is however displayed incorrectly: it looks like it is interpreted as ANSI instead of Unicode ("spaces" between letters) and umlauts are omitted. The issue with the umlauts is probably connected to #2009, but even with the right codepage, the umlauts are not displayed correctly (see below).

I also discovered that trying to run lxrun in the same manner (which should just output its usage help) also results in weird formatting like that, but it also cuts of the start of the help text:

Trying the same from xterm does not show spaces between letters and does not cut off lxrun's help, but has apparently still encoding issues. On top of that, the "Press any key to continue prompt" doesn't react on keys in xterm.

As mentioned above, the umlaut issue is probably from #2009, but even when changing the codepage to 65001 to 850 (as it should be on my computer) or 437, the output is wrong:

This does look like some sort of bug in passing the input through WSL back to cmd.exe. Do you happen to know of any other Windows programs that can repro this output behavior in bash (via interop)?
@adiviness for visibility, he did some buffer work for CU.

It happens also without cmd involved, when just running bash.exe from inside bash.
Until now I saw it only with bash.exe and lxrun.exe. I checked a few other commands and they work OK.

It makes sense, bash.exe writes back to conhost in wide characters, subprocesses of bash write to the emulated tty in utf8. So you're seeing Unicode written back to the utf8 channel between bash and its subprocesses.
This happens with any app that uses WriteConsole instead of writing to the output in a manner that can be properly intercepted by WSL, no?

This issue was moved to Microsoft/console#110


Support for togethercomputer/GPT-JT ?

Any plan on adding support for togethercomputer/GPT-JT (https://huggingface.co/spaces/togethercomputer/GPT-JT).
Seems like the closest alternative to GPT-3. What do you think? I would love to help but I don't know where to start

The above is an app, do you have a link to the model implementation and params from HF? :)

Oh sorry! Yes this one is the app https://huggingface.co/togethercomputer/GPT-JT-6B-v1. Does this have similarities with other models that I could try to implement or even use directly? I would have no idea where to start but willing to help

Ah, nice! We support GPT 2, so maybe that can be used as a building block? Or at least you can compare GPT 2 Python's implementation with our GPT 2 and then to the same to implement your own. :)

Thanks! I will give it a try :) thanks! Any MR I can look at for reference?
BTW this model already knows some Elixir

❤

The reference hf/transformers implementation of GPT-J is here. The implementation should be for the most part similar to any other text model we have, like GPT-2. By a brief look I think we may need to adjust/extend our attention implementation to support the rotary position embedding, but it's fine to modify the current code as necessary and we can find the best way to make it configurable later.

Working on this but it is gonna take a while 'cause I am new to transformers


SOLR-15365: New cluster row in Grafana Dashboard

See https://issues.apache.org/jira/browse/SOLR-15365

@thelabdude Please have a look at this. I tested in grafana and did an export through "share" button -> export -> "export for sharing externally". But the diff is all messed up, so hard to review.
I might attempt to just copy/paste the changed and added sections. Any hints?

Very cool @janhoy . I imported into a live K8s cluster running 8.8.2 with the customized exporter config and see the attached.
Not sure what that extra dimension with =: is on the Replica State History panel? Might have a look at that ... It's up now if you want to try it out: http://<IP_ADDRESS>:3000/d/yt5eXHbMz5/solr-dashboard-main-fixed?orgId=1&refresh=30s&from=now-5m&to=now (I'll slack you the login creds)

Not sure what that extra dimension with =: is on the Replica State History panel?

Since the state label was not there before, you have historic data points without any value. So if you change to last 15 minutes they will disappear...
Thanks for feedback. Will push a commit with some of the things we discussed on slack.

Pushed changes

state history is now 0 for active, and any non-active will stack up in the graph. Much easier to spot failures
hide legends of replicas that have never been shard leader
change title of the dashboard to "Solr Dashboard"


how to prevent XSS?

thanks for your project.
it's pretty good, I have a question: how to prevent XSS?
does this project provide this function?
thanks :-)

Hey Caratpine. If you use my fork (or if I could get softvar to finally merge my PR you get XSS protection included. For that you simply pass encode=True to the convert call, this will escape everything nasty and also guarantee that the html looks good for non-ascii characters. Alternatively you can call .encode('ascii', 'xmlcharrefreplace') on the string returned by convert (and you should do that for any other untrusted data that you return back by http response).

@muellermichel is right. Also, #24 is now merged. So, the latest version 1.2.0 will fix this.

@muellermichel  @softvar thanks for your relies😄
I use this example:
from json2html import json2html
from flask import Flask
import json

app = Flask(__name__)

@app.route('/')
def index():
    s = "<script></script>"
    return json2html.convert(json=json.dumps(s), encode=True)

app.test_client().get('/').data

it returns <script></script> not  '<script></script>'
so i use this method to prevent XSS:
def escape(s):
    return s.replace('&', '&').replace('>', '>').replace('<', '<')

s = '<script></script>
s = json2html.convert(json=escape(s))

Ah I see your usecase - sorry I was wrong, xmlcharrefreplace in that case doesn't do what you want, it only escapes non-html characters to html. I think it would make sense to do your escaping on the text node level by default. For that we could simply use cgi.escape for all text node types. Doing so by default would be analogous to what most templating engines do.
Btw. cgi.escape does exactly what your escape function is doing - more info here: https://wiki.python.org/moin/EscapingHtml
I therefore advocate to leave this issue open. The code to do this with json2html is only 3 lines plus some additional tests, so it wouldn't be a big deal to support.

@muellermichel  thanks for your patience to answer 😄

I've created the PR linked above. See lines 191 and follwing in https://github.com/muellermichel/json2html/blob/master/test/run_tests.py : This should fix XSS for good.
However there is one drawback: There is currently no whitelisting going on. If someone uses json2html together with text nodes containing formatting tags (like  as an example), it would escape it and break formatting, so in that case the user would now have to set escape=False. Since formatting was never a documented feature in json2html I think this should be OK however.

Thanks for pointing this out @Caratpine
It's fixed now in version 1.2.1.
Thanks @muellermichel for the patch.


Bump version to 17.3.1 and cherry-pick loc files

We need to wait as we will need to also cherry-pick #3967

We now have all required translations. We can move forward with this PR and the servicing release processes.


Pulseaudio Passthrough

I've tried many different chrome browser docker implementations and can't seem to get audio working in any of them. I like your implementation the best, I can launch it on Ubuntu 18.04 by removing --device /dev/video0 from your launch.sh file. But no matter how I try to connect pulseaudio to the session, it fails. Would you be open to trying to trying to accomplish sound in Ubuntu 18.04? Cheers.

I'm thinking:
http://wiki.aplinelinux.org/wiki/Sound_Setup might be possible in your dockerfile.

I changed parts of your Dockerfile to install ALSA and Pulseaudio, fix groups, etc. I modified many parts of your launch.sh to attempt to connect Pulseaudio to my host. Never worked, ALSA always failed finding PCM device. I modified the Dockerfile again to run aplay -l and got the "No Soundcards" error.  So I seem to be missing an important step in regards to Alpine Linux and Docker providing a sound device, as I can use the Ubuntu in Docker to play sounds using the answer posted at this site: https://askubuntu.com/questions/972510/how-to-set-alsa-default-device-to-pulseaudio-sound-server-on-docker


[APP] Deprecated crisp-edges settings during yarn build

Summary
[21:01:10] gulp-autoprefixer:
  autoprefixer: /Users/will/git/duelyst/app/ui/styles/application.css:10747:3: There is no browsers with crisp-edges rendering support.Maybe you mean pixelated?
[21:01:10] gulp-notify: [Gulp notification] /Users/will/git/duelyst/app/ui/styles/duelyst.css

Impacted services

[x] The frontend app
[ ] The game servers
[x] Game builds or automation scripts
[ ] Tests or Github Actions
[ ] Infrastructure (Terraform)

Steps to reproduce

yarn build

More info from MDN: https://developer.mozilla.org/en-US/docs/Web/CSS/image-rendering


[SR-15036] JSONEncoder sometimes encodes dictionaries as arrays

Previous ID
SR-15036

Radar
rdar://81606906

Original Reporter
@franklinsch

Type
Bug

Environment
Ubuntu 18.04 using the toolchain swift-5.5-DEVELOPMENT-SNAPSHOT-2021-08-04-a-ubuntu18.04, which has:
Swift version 5.5-dev (LLVM 4afce2110750060, Swift 4ae44a5e0eac8a0)
Target: x86_64-unknown-linux-gnu

Additional Detail from JIRA

Votes
0

Component/s
Foundation

Labels
Bug

Assignee
None

Priority
Medium

md5: 89b9b09cfba7115d37589ea5171b1014

Issue Description:
For the following program:
public struct DataAsset: Codable {
    public var sizes = [ImageSize: String]()
}

public struct ImageSize: Codable, Hashable {}

let data = try! JSONEncoder().encode(DataAsset(sizes: [ImageSize(): ""]))
print(String(data: data, encoding: .utf8)!)

The output I'm getting is `{"sizes":[{},""]}` on Linux (sizes should be a dictionary instead), which is unexpected. On macOS I'm getting `{"sizes": {}}`.

This should be transferred back to the Swift repo, because it appears to be an issue with the compiler-synthesized Codable conformance and not JSONEncoder specifically.

At least I think that's the problem, it's worth verifying that. On macOS I do seem to now be getting {"sizes":[{},""]} with the newer JSONEncoder implementation.


Differentiate between "Time of Day" and "Time Duration"

Currently Slice sees everything as "Time of Day" which can give confusing information and incorrect validations back for Time Durations.
Add a flag for time values to be set as one or the other.

Make sure to also add time duration validator, as done in #212.


Fix BackBufferCopy rect property appearing when not relevant in inspector

master version of https://github.com/godotengine/godot/pull/69775.
The rect property is only effective if copy_mode is Rect.
This closes https://github.com/godotengine/godot/issues/69772.

Thanks!


automatic deploy fails with "Failed to get FD to bundle executable"

Prerequisites

[X] I have written a descriptive issue title
[X] I have searched issues to ensure it has not already been reported

Cake runner
Cake .NET Tool
Cake version
Cake.Xamarin, Version=<IP_ADDRESS>
Operating system
macOS
Operating system architecture
64-Bit
CI Server
No response
What are you seeing?
Hi
I tried to add cake to an existing project. When I run the app from Visual Studio everything works as expected.
The moment I run a cake build and try to deploy the app via Xcrun simctl i get the following error:
An error was encountered processing the command (domain=IXUserPresentableErrorDomain, code=1): Unable To Install “Mvx Scaffolding” Please try again later. Failed to get FD to bundle executable at /Users/HP/Library/Developer/CoreSimulator/Devices/E411AE78-2115-4A84-A334-C4583C7776CD/data/Library/Caches/com.apple.mobile.installd.staging/temp.CIEO1c/extracted/MyXamarinFormsApp.iOS.app/MyXamarinFormsApp.iOS Underlying error (domain=MIInstallerErrorDomain, code=4):         Failed to get FD to bundle executable at /Users/HP/Library/Developer/CoreSimulator/Devices/E411AE78-2115-4A84-A334-C4583C7776CD/data/Library/Caches/com.apple.mobile.installd.staging/temp.CIEO1c/extracted/MyXamarinFormsApp.iOS.app/MyXamarinFormsApp.iOS
I have no issues to deploy any other xamarin project which I build with exact same Cake script.
here my project to reproduce the issue:
https://github.com/hulluP/mvvmCrossCakeBuild
What is expected?
the produced iOS app should deploy in the CI CD
Steps to Reproduce

pull https://github.com/hulluP/mvvmCrossCakeBuild
run the buildForms.sh

Output log
An error was encountered processing the command (domain=IXUserPresentableErrorDomain, code=1):
Unable To Install “Mvx Scaffolding”
Please try again later.
Failed to get FD to bundle executable at /Users/HP/Library/Developer/CoreSimulator/Devices/E411AE78-2115-4A84-A334-C4583C7776CD/data/Library/Caches/com.apple.mobile.installd.staging/temp.CIEO1c/extracted/MyXamarinFormsApp.iOS.app/MyXamarinFormsApp.iOS
Underlying error (domain=MIInstallerErrorDomain, code=4):
Failed to get FD to bundle executable at /Users/HP/Library/Developer/CoreSimulator/Devices/E411AE78-2115-4A84-A334-C4583C7776CD/data/Library/Caches/com.apple.mobile.installd.staging/temp.CIEO1c/extracted/MyXamarinFormsApp.iOS.app/MyXamarinFormsApp.iOS

this seems to have been solved with the latest release