Add file logging, more control over what is logged

Fixes #14.
I took some liberties here, let me know if you want to back any of them out:

-l and --level are gone, and have been replaced with the more specific --log-level-file and --log-level-stderr. Those default to DEBUG and INFO, respectively.
I changed shortname to name, to make it easier to figure out exactly which logger is spamming you, so it can be silenced with LOGLEVEL.
I added a LOGLEVEL variable, which is inspired by PYTHONWARNINGS, and gives precise control over which loggers are used. I begun by trying to implement it with --level and argparse, as Trinity does, but realized that the env variable would require a lot less code, and makes it easier to manage, using either an .env file or an export.

Here's a command which gets rid of all the upnp spam:

LOGLEVEL=INFO:ddht.upnp,INFO:urllib3.connectionpool,INFO:Service,INFO:Action,INFO:ssdp,INFO:Device ddht --log-level-stderr 10

It's unwieldy to write this out every time. But after an export we can get all the benefit of not being spammed by upnp, while still running just

ddht --log-level-stderr 10

In the future it might make sense to read configuration from pyproject.toml, but for now this seems nice and convenient.
Cute Animal Picture

btw, make lint-roll is your friend.

Added a note to the README and appeased lint.

Just realized this is dumping logs in the current working directory.
I think we need these to be in $XDG_DATA_HOME/ddht/logs/

Also, logs should rotate which is pretty easy to setup.

Adding this in a sec!


Results from Tasks query rendering center justified

Describe the bug
I've embedded a tasks plugin query in a cell - which is incredibly useful! For narrow width cells results display left-aligned, as they should, but the results are rendering center-justified when cell width is set wide. I'm using the Minimal theme and am on a Mac.

Steps to reproduce
Embed a Tasks plugin query in a text cell. Use a fairly large amount of text for the task description. Resize width of cell to see alignment of task items move between left and center alignment.
Expected behavior
Items should remain left justified, regardless of cell width
Are you using the mobile app?
Yes
Obsidian debug info
SYSTEM INFO:
Obsidian version: v1.3.5
Installer version: v1.1.16
Operating system: Darwin Kernel Version 22.5.0: Mon Apr 24 20:51:50 PDT 2023; root:xnu-8796.121.2~5/RELEASE_X86_64 22.5.0
Login status: logged in
Catalyst license: vip
Insider build toggle: on
Live preview: on
Legacy editor: off
Base theme: light
Community theme: Minimal v6.3.3
Snippets enabled: 3
Restricted mode: off
Plugins installed: 27
Plugins enabled: 25
1: Excalidraw v1.9.3
2: Kanban v1.5.3
3: Advanced Tables v0.18.1
4: Show Current File Path v0.5.2
5: Calendar v1.5.10
6: Dataview v0.5.56
7: Periodic Notes v0.0.17
8: Enhance Copy Note v0.2.1
9: Style Settings v1.0.6
10: Minimal Theme Settings v6.3.1
11: Find orphaned files and broken links v1.9.0
12: Customizable Page Header and Title Bar v4.7.2
13: Contextual Typography v2.2.5
14: QuickAdd v1.0.2
15: Templater v1.16.0
16: Natural Language Dates v0.6.1
17: Buttons v0.4.19
18: Creases v0.6.6
19: Tasks v3.10.0
20: ReadItLater v0.3.0
21: Tag Wrangler v0.5.11
22: Pandoc Plugin v0.4.1
23: Obsidian Enhancing Export v1.6.1
24: Notion-Like Tables v6.18.6
25: File Order v0.0.6
RECOMMENDATIONS:
Custom theme and snippets: for cosmetic issues, please first try updating your theme and disabling your snippets. If still not fixed, please try to make the issue happen in the Sandbox Vault or disable community theme and snippets.
Community plugins: for bugs, please first try updating all your plugins to latest. If still not fixed, please try to make the issue happen in the Sandbox Vault or disable community plugins.
Relevant log output
No response

@clshafer Thanks for the bug report. I will fix this today.


2D LSTM attention

Can we use the same code for 2D LSTM attention ?

@basma-b  did you solve this ?


Add array support for q and Q typecodes, update renamed methods

Resolves #433.
There are quite a few test failures still occurring in test_array.py, but it doesn't look like any of them are a direct result of this. test_tofromstring() fails with an incorrect number of warnings thrown for calling the newly deprecated methods, but this is an implementation detail and has changed within CPython following 3.4 to match

Why close this? I think there were just a couple of recommendations from @slozier. It looked good otherwise!

I ran into some implementation difficulties/broken functionality when trying to clean it up, and wanted to make sure the pieces were all there prior to submitting any updates. I don't think I'll have time to do that in the next few days, so I closed it for now. I'll reopen it when I have @slozier's recommendations (which I agree with) in and the module works again.


allow for more then one virtual environment

Be sure to check the existing issues (both open and closed!), and make sure you are running the latest version of Pipenv.
Check the diagnose documentation for common issues and the PEEP list before posting! We may close your issue if it is very similar to one of them. Please be considerate and follow the PEEP process, or be on your way.
Make sure to mention your debugging experience if the documented solution failed.
Is your feature request related to a problem? Please describe.
yes but not to anything you have posted.
I'm trying to switching between pypy and cpython virtual environment  to do testing on speed issue and laging in the python programs I am using
A good example of the problem I having is install piipenv ( follow the contributor guide) and running

pipenv install --dev

work ok except uses wrong python as already mention in issue  #4339 (pipenv does not use default python version under pyenv)
but if i want to also install a virtual environment for python 3.6 and run

pipenv install --dev --python /usr/local/pyenv/versions/pypy3.6-7.3.1-src/bin/python

it says

Virtualenv already exists!
Removing existing virtualenv…

not what I want. I want to have both.
Right now i don't think there is a way to do this.

A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

I'm always frustrated when

It is creating a pipfile when I don't want one (it demands a specific python, would not be a problem if was not for this)
I can't run pipenv shell and drop into the virtual environment shell I want to run
I keep running into more problems
the  --python option does not work well

Describe the solution you'd like
I would like to run

pipenv --python /usr/bin/pypy

have it create a environment for pypy
then run

pipenv --python /usr/bin/python

have it create an environment for python
then run

pipenv shell

have it print something to the likes of

you have more then one python enviorment installed they are
1 /usr/bin/python
2 /usr/bin/pypy
please use the --python flag

A clear and concise description of what you want to happen.
done above
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context

Like all issues i make I want to look into implement this but not sure if i can( I am not that good of programmer) and want to see if this type of thing is welcomed.
Please run $ pipenv --support, and paste the results here. Don't put backticks (`) around it! The output already contains Markdown formatting.
If you're on Linux, run the following:
$ pipenv --support | xclip

return nothing
but pipenv --support returns
$ pipenv --support
Pipenv version: '2020.6.2'
Pipenv location: '/usr/local/pyenv/versions/3.8.2/lib/python3.8/site-packages/pipenv'
Python location: '/usr/local/pyenv/versions/3.8.2/bin/python3.8'
Python installations found:

3.8.2: /usr/local/pyenv/versions/3.8.2/bin/python3
3.8.2: /usr/local/pyenv/versions/3.8.2/bin/python3.8
3.8.2: /usr/local/pyenv/versions/3.8.2/bin/python3
3.7.3: /usr/bin/python3
3.7.3: /usr/bin/python3.7m
3.7.3: /usr/bin/python3.7
2.7.17: /usr/local/pyenv/versions/2.7.17/bin/python2
2.7.16: /usr/bin/python2
2.7.16: /usr/bin/python2.7

PEP 508 Information:
{'implementation_name': 'cpython',
 'implementation_version': '3.8.2',
 'os_name': 'posix',
 'platform_machine': 'armv7l',
 'platform_python_implementation': 'CPython',
 'platform_release': '5.4.47-v7l+',
 'platform_system': 'Linux',
 'platform_version': '#1322 SMP Wed Jun 17 17:58:52 BST 2020',
 'python_full_version': '3.8.2',
 'python_version': '3.8',
 'sys_platform': 'linux'}

System environment variables:

SHELL
SESSION_MANAGER
WINDOWID
PYENV_HOOK_PATH
QT_ACCESSIBILITY
COLORTERM
PYENV_SHELL
XDG_CONFIG_DIRS
XDG_MENU_PREFIX
SSH_AUTH_SOCK
DEBFULLNAME
DESKTOP_SESSION
SSH_AGENT_PID
PYENV_VERSION
GTK_MODULES
XDG_SEAT
PWD
PIPX_BIN_DIR
LOGNAME
XDG_SESSION_TYPE
CXXFLAGS
XAUTHORITY
HOME
LANG
LS_COLORS
XDG_CURRENT_DESKTOP
VTE_VERSION
DEBEMAIL
INVOCATION_ID
PYENV_DIR
GLADE_CATALOG_PATH
XDG_SESSION_CLASS
TERM
USER
PIPX_HOME
DISPLAY
SHLVL
XDG_VTNR
UBUNTU_MENUPROXY
XDG_SESSION_ID
GLADE_MODULE_PATH
XDG_RUNTIME_DIR
GLADE_PIXMAP_PATH
PYENV_ROOT
JOURNAL_STREAM
XDG_DATA_DIRS
PATH
CFLAGS
DBUS_SESSION_BUS_ADDRESS
TEXTDOMAIN
PIP_DISABLE_PIP_VERSION_CHECK
PYTHONDONTWRITEBYTECODE
PIP_SHIMS_BASE_MODULE
PIP_PYTHON_PATH
PYTHONFINDER_IGNORE_UNSUPPORTED

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

PATH: /usr/local/pyenv/versions/3.8.2/bin:/usr/local/pyenv/libexec:/usr/local/pyenv/plugins/xxenv-latest/bin:/usr/local/pyenv/plugins/python-build/bin:/usr/local/pyenv/plugins/pyenv-virtualenv/bin:/usr/local/pyenv/plugins/pyenv-update/bin:/usr/local/pyenv/plugins/pyenv-installer/bin:/usr/local/pyenv/plugins/pyenv-doctor/bin:/usr/local/poetry/bin:/home/pi/.local/bin:/home/pi/bin:/usr/local/pyenv/shims:/usr/local/pipx/bin:/usr/local/pyenv/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/local/games:/usr/games
SHELL: /bin/bash
LANG: en_US.UTF-8
PWD: /home/pi/pip-sources/pyvim-env/src/pyvim

Contents of Pipfile ('/home/pi/pip-sources/pyvim-env/src/pyvim/Pipfile'):
[[source]]
name = "pypi"
url = "https://pypi.org/simple"
verify_ssl = true

[dev-packages]

[packages]
prompt-toolkit = "*"
six = "*"
pyflakes = "*"
pygments = "*"
docopt = "*"
pyvim = {editable = true, path = "."}

[requires]
python_version = "3.6"

Contents of Pipfile.lock ('/home/pi/pip-sources/pyvim-env/src/pyvim/Pipfile.lock'):
{
    "_meta": {
        "hash": {
            "sha256": "4709cd6234a8a4ed09266873409dc3a4e1f4e26f93b006a245598e7acc6dee79"
        },
        "pipfile-spec": 6,
        "requires": {
            "python_version": "3.6"
        },
        "sources": [
            {
                "name": "pypi",
                "url": "https://pypi.org/simple",
                "verify_ssl": true
            }
        ]
    },
    "default": {
        "docopt": {
            "hashes": [
                "sha256:49b3a825280bd66b3aa83585ef59c4a8c82f2c8a522dbe754a8bc8d08c85c491"
            ],
            "index": "pypi",
            "version": "==0.6.2"
        },
        "prompt-toolkit": {
            "hashes": [
                "sha256:563d1a4140b63ff9dd587bda9557cffb2fe73650205ab6f4383092fb882e7dc8",
                "sha256:df7e9e63aea609b1da3a65641ceaf5bc7d05e0a04de5bd45d05dbeffbabf9e04"
            ],
            "index": "pypi",
            "version": "==3.0.5"
        },
        "pyflakes": {
            "hashes": [
                "sha256:0d94e0e05a19e57a99444b6ddcf9a6eb2e5c68d3ca1e98e90707af8152c90a92",
                "sha256:35b2d75ee967ea93b55750aa9edbbf72813e06a66ba54438df2cfac9e3c27fc8"
            ],
            "index": "pypi",
            "version": "==2.2.0"
        },
        "pygments": {
            "hashes": [
                "sha256:647344a061c249a3b74e230c739f434d7ea4d8b1d5f3721bc0f3558049b38f44",
                "sha256:ff7a40b4860b727ab48fad6360eb351cc1b33cbf9b15a0f689ca5353e9463324"
            ],
            "index": "pypi",
            "version": "==2.6.1"
        },
        "pyvim": {
            "editable": true,
            "path": "."
        },
        "six": {
            "hashes": [
                "sha256:30639c035cdb23534cd4aa2dd52c3bf48f06e5f4a941509c8bafd8ce11080259",
                "sha256:8b74bedcbbbaca38ff6d7491d76f2b06b3592611af620f8426e82dddb04a5ced"
            ],
            "index": "pypi",
            "version": "==1.15.0"
        },
        "wcwidth": {
            "hashes": [
                "sha256:beb4802a9cebb9144e99086eff703a642a13d6a0052920003a230f3294bbe784",
                "sha256:c4d647b99872929fdb7bdcaa4fbe7f01413ed3d98077df798530e5b04f116c83"
            ],
            "version": "==0.2.5"
        }
    },
    "develop": {}
}

This can be achieved by:
PIPENV_PYTHON=/path/to/python pipenv install

Thankyou
@frostming
For you time and  the command bellow.

PIPENV_PYTHON=/path/to/python pipenv install

I would like to have a better way to do(And wanted to work on) it, since IMHO it is not very user friendly.
For example
I want to and a config file for common option that pipenv just does not do the right thing on like issue
Have an options in the config some like
# option to control which python pipenv uses
#ask on install virtual environment
VIRENV= ask
#allows for only one environment deletes the old
VIRENV=OnlyOne
#to allow it to  create for more than one..
VIRENV=Multiple

DefaultVirPython= /path/to/python

or just when you specify the correct python with the flag have it not delete it.
If I made the changes in pipenv soucecode that allowed for this would it be acceptable or is there some undocumented way to do this.  Adding something in the env file which is unexplained maybe?
I apologize That this is getting to close to a stackoverful question and I understand if you don't want to talk about it anymore.


double flat

It's about twice the flat. ♭ ♭ is not displayed in verovio. Are there other ways to encode double  ♭ signs?

Should be filed at rism-ch/verovio


Forward amount parse validation errors as they are and introduce AmountExceedsMaximum

sanitize_uint() is used in different validations so make it forward the actual error and handle that error appropriately downstream for each validation.

This test failure is because of the error message change. you'd need to update the test: https://github.com/Chia-Network/chia_rs/actions/runs/3596319974/jobs/6056852275#step:9:112

This test failure is because of the error message change. you'd need to update the test: https://github.com/Chia-Network/chia_rs/actions/runs/3596319974/jobs/6056852275#step:9:112

Thanks to this insight I opted for adapting the code to keep the test's ReserveFeeConditionFailed expectation intact.

One way to bring down the size of this change might be to make sanitize_uint() take another error code indicating what error to return if the amount exceeds maximum.
That way you would not have to add so much code to map specific error messages at call sites.

going back to why we're making this change in the first place, I realized the only case we get the wrong (or an unintuitive) error code is in CREATE_COIN with an amount that exceeds maximum. That's the only thing that needs changing, nothing else.
One other place that I also noticed, though, is when parsing a spend. We identify which coin is being spent by its amount, if that amount is negative, we really shouldn't be getting an AmountNegative error, we should get an InvalidAmount error. The former is used for CREATE_COIN with a negative amount.

I think this is a smaller (and a bit simpler) change: https://github.com/Chia-Network/chia_rs/pull/99

I think this is a smaller (and a bit simpler) change: #99
Agreed, thanks.


Finnish translation

Added Finnish translations and an option to the language select dropdown.

Thank you, @niklas-e! Looks amazing! Welcome to contributors 🎉


2.4.4 Release?

Hi, any plan for release 2.4.4 tarball?
I always having issue with building from repo (autogen.sh stuff), so I always build from tarball to get the latest release. Thanks.

No plans yet... I don't think there is a lot (or anything?) new in 2.4.4 yet. Or am I forgetting something?

It's only because I saw you made an update to v2.4.4 on your latest commit. But, I think I can happily wait for it.
Thanks, Joe.

Go to https://github.com/joewing/jwm, select "Code" and choose "Download ZIP" (you'll receive the jwm-master.zip file). Unzip, cd to jwm-master, run ./autogen.sh, followed by ./configure, then "make". cd to src and run "./jwm -v" -- the version displayed is "JWM v2.4.4 by Joe Wingbermuehle".
Is there anything new? substantive? Ask Joe.


v1.9: docs: Fix cilium-runtime image bump instructions

This is a process that reliably works, unlike the previous one which
relies on Quay.io's ability to pull images from DockerHub. It does
require a core maintainer to prepare the changes, but that's fine for
now since v1.9 should change infrequently now.

cc @tklauser FYI this will only be for the v1.9 branch.

Only doc changes no need to re-run the CI.


[4.1] Fix FetchOperation with custom query

PR related to #2817.
Fixes a pair of errors caused when passing a custom query to the fetch() operation via an array as shown in the documentation.
I think this is a simple fix, but I could be totally missing something here! Happy to help test further or do some more digging if needed.

A new inspection was created.

The inspection completed: No new issues

Hello @dave-mills
Thank you very much for your submition.
Indeed you are right. I just tested your solution and it works as expected.
Going to mark this a ready for merge.
Best to you \o

Thanks a lot for the PR @dave-mills - and for the review @pxpm . Let's do this! Merged into master, it'll be tagged later today, or on Monday the latest.
Cheers!


Request for clipboard monitoring

It would be great if implement clip board monitoring.
Regards

I will add this feature soon.

@xp4xbox thank you so much . I will be waiting for this.


[#596] feat(hive): Hive catalog supports to impersonate users to execute operations in simple mode.

What changes were proposed in this pull request?
Hive catalog supports to impersonate users to execute operations in simple mode.
For Kerberos mode,  I have created an new issue. I will finish it in the later pull request.
We use a Hive client cache pool referring to the Iceberg cache pool.  We use user name as the key of cache pool.
Why are the changes needed?
Fix: #596
Does this PR introduce any user-facing change?
Yes, we will add a new document.
How was this patch tested?
Add a new IT

@mchades @jerryshao @yuqi1129 Could you help me review this pr?

@FANNG1 @jerryshao @yuqi1129 @diqiu50 @Clearvive Could you help me review this pr?

All comments are addressed. @yuqi1129 Could you give me review, again?

@mchades @yuqi1129 @jerryshao Could you help me review this pr?


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 1737 ms

Resolved: Bofrost.de performance has improved in 11a6ee0.


How to use api key with angular2-google-maps.

Thanks for a great repo. Can you say me how to use API key with angular2-google-maps.

Hi @santhoshreddy31. Here's an example:
import {provide} from 'angular2/core';
import {ANGULAR2_GOOGLE_MAPS_PROVIDERS, LazyMapsAPILoaderConfig} from 'angular2-google-maps/core';

bootstrap(AppComponent, [
  ANGULAR2_GOOGLE_MAPS_PROVIDERS,
  provide(LazyMapsAPILoaderConfig, {useFactory: () => {
    let config = new LazyMapsAPILoaderConfig();
    config.apiKey = 'mykey';
    return config;
  }})
])

There are also several more options you can configure: https://angular-maps.com/docs/api/latest/ts/core/LazyMapsAPILoaderConfig-class.html
I close this issue now. Feel free to reopen or comment if you have further questions.

Still getting the error after using the LazyMapsAPILoader and the versions 3.18,3.19,3.23,3.24
Any help?

Hi @SebastianM I entered mykey like how you showed above. I don't have any errors, but map is not loading. Any idea why?

same here @bkshashi30  did you fix it?

how to use API key in angular 2.0?

@paruvellyvishwanath You need to import the module along with the information about your API key using forRoot static method of AgmCoreModule. You can see the example in the docs.


Auth::user() return roles automatically

I am working with Laravel and Entrust and the basics work fine. I do have one issue that doesnt seem to come with the package. What I would like is when I do Auth::user() if they are logged in it returns their Roles as well. Currently I know I can do Auth::user()->roles, but for that to work I need to check if they are a user first. I would image there is an easier way and convenient, for when you return the auth user you get there roles as well. Not sure if this can be done in the package or not, but would be nice.

$roles = (Auth::check()) ? Auth::user()->roles()->get() : false;

I'm not sure exactly what you mean, Auth::user() will return the model or a null value depending on if the user is logged in. To actually check if a user is logged in you use Auth::check().

@MineSQL I ended up using the following code:
'user'=> Auth::user() ? Auth::user()->load('roles') : null,
But basically what I wanted was to get the user object with the roles in one object, not get the roles by themselves for the current user.

With Laravel 5.4 you can edit  your User model by passing a protected $with = ['role']; and setting up a Model and define a relationship with eloquent.
class User extends Authenticatable{
   ...
   protected $with = ['role'];
   ...
   public function role(){
      return $this->belongsTo('App\Models\Role');
   }
   ...
}

and then use it
$userRoleName = User::find(1)->role->name;

note that i always store my Models in App\Models\Role in your app could be different - if you are using a professional IDE it should be suggested the correct path of it.

@PaoloFalomo I get what your are saying here, so basically the same thing I am doing with load but in the user model. When I do Auth::user->role->name I get the error Trying to get property of non-object

@PaoloFalomo  protected $with = ['roles']; in the user function ran as I intended.

@packytagliaferro great

@PaoloFalomo you know if there is a way to limit what Roles returns? Right now it gives me everything (created_at, description, display_name, id, name, pivot, updated_at) when I all really need is the Name. Or does that even affect site speed? I am just trying to become a better dev and not just grab everything.


Add full support for Vertica 11

What does this PR do?

Adds support for Vertica 11 for projection storage and storage containers metrics, which were missing due to changes in the related system tables from previous Vertica versions.
Refactors how aggregates for those two sets of metrics are computed by leveraging SQL queries instead of computing them in Python.

Motivation
AI-2467 (and original GitHub issue)
Follow up to #12394.
Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Hey, I don't know if it's the right place to ask. Please, feel free to forward me. As far as I can see, Vertica 12 is released.
https://www.vertica.com/docs/ReleaseNotes/12.0.x/Vertica_12.0.x_Release_Notes.htm
Is it possible to assure everything will work with the Vertica 12 as well?


refactor: drop pug template and use plane html in .md file

Check List

[x] Others (Update, fix, translation, etc...)

Languages: All

What is for?
IMHO, we no need to use .pug for index page. We should just replace them from .pug to .md (plane html).

Thank you :)

we lost ko.
https://hexo.io/ko/

we lost ko.

Sorry, I forgot to transform it 😢
I submitted a new PR https://github.com/hexojs/site/pull/2068


Add installation instructions for Accio & list as supported

This library already supports Accio but doesn't document it in the list of supported dependency managers.

No need as Accio is being deprecated with the SwiftPM resources support in Xcode 12 / Swift 5.3. See https://github.com/JamitLabs/Accio/issues/88.


Storage migration to triple reference counting will always fail for chains that are not already upgraded to dual reference counting.

The current versions of substrate attempt to perform an automatic migration of the storage to triple reference counting, assuming all chains are already upgraded to dual reference counting. It also provides helpers to upgrade from single reference counting and even from u8 reference counting to triple ones, but those are now rendered useless since they are effectively impossible to use in a migration. Here is why:
This code is always called after custom migrations, and will always call the function that upgrade from dual to triple reference counting in case this is not already upgrade. The problem with this is that, in case of custom migrations, one has to either be able to upgrade to dual reference counting before this call for this to work, or - if moved to triple reference counting during the custom migration, - to ensure that UpgradedToTripleRefCount::<T> is set to true.
One issue with this is that there is no way to set UpgradedToTripleRefCount::<T> directly. And there is no way to upgrade to dual reference counting in a custom migration, since all function offered explicitly all directly upgrade to triple reference counting.
An example: this function is intended to upgrade from single u8 reference counting to triple reference counting, but is never going to work, since if called during a custom migration, it will indeed perform the right upgrade, but then, the code above would trigger an upgrade from dual to triple reference counting that would fail.
Another minor issue is this function that does not actually do what the name suggests (the comment is correct though).
What this function does is migrating from single reference counting to triple one, so should better not be called migrate_to_dual_ref_count to avoid confusion.
I propose to rename those functions to make this more consistent, and to trigger an update of UpgradedToTripleRefCount::<T> on each of the helpers that actually upgrade to triple reference counting, regardless of the state they are starting from, ( then it becomes unnecessary on `on_runtime_upgrade').
This should keep the default behaviour but fix other issues.

Just created pull request #10337 that implements the changes proposed, if any useful.


Add support for placeholder text

if you like it...
most of the code was shamelessly copied from http://stackoverflow.com/questions/1328638/placeholder-in-uitextview

Thanks, but if you override drawRect anyway we shouldn't use a UILabel, we should draw a string in the drawRect via CoreGraphics.

yeah, I agree, as noted, I shamelessly copied from that suggested
solution...
perhaps it's beter to add a UILabel in initWithFrame and forget about
drawRect
On Wed, Feb 1, 2012 at 9:48 AM, Hans Pinckaers <
<EMAIL_ADDRESS>
wrote:

Thanks, but if you override drawRect anyway we shouldn't use a UILabel, we
should draw a string in the drawRect via CoreGraphics.

Reply to this email directly or view it on GitHub:
https://github.com/HansPinckaers/GrowingTextView/pull/9#issuecomment-3754971

--
/Ran
http://tavory.com


dbt deps on BigQuery requires either gcloud auth or "project" set

Describe the bug & Steps To Reproduce
Running dbt deps with a profiles.yml which a) doesn't have a project and b) in an environment without a valid gcloud auth (such as a docker build):
nimbus:
  target: user
  outputs:
    user:
      type: bigquery
      method: oauth
      dataset: my-dataset
      timeout_seconds: 3600
      threads: 24
      # project: my-project

Raises an error:
 => ERROR [dev 11/12] RUN dbt deps                                                                                5.2s
------
 > [dev 11/12] RUN dbt deps:
dbt-labs/dbt#36 1.615 Running with dbt=0.19.0
dbt-labs/dbt#36 4.925 Encountered an error:
dbt-labs/dbt#36 4.925 Could not automatically determine credentials. Please set GOOGLE_APPLICATION_CREDENTIALS or explicitly create credentials and re-run the application. For more information, please see https://cloud.google.com/docs/authentication/getting-started
------
executor failed running [/bin/sh -c dbt deps]: exit code: 2

These works:

Running with the project: uncommented
Running with dbt deps --target=null

Expected behavior
Does dbt deps need the database / project name at that stage? If so, this is the correct behavior. But if not, there's no need to break in a docker build.
System information
Which database are you using dbt with?

[ ] postgres
[ ] redshift
[x] bigquery
[ ] snowflake
[ ] other (specify: ____________)

The output of dbt --version:
installed version: 0.19.0
   latest version: 0.19.0

Up to date!

Plugins:
  - bigquery: 0.19.0
  - snowflake: 0.19.0
  - redshift: 0.19.0
  - postgres: 0.19.0

The operating system you're using:
MacOS
The output of python --version:
Python 3.8.8

I get the same issue when I run dbt compile
While I have authenticated my terminal using gcloud auth login, when I run dbt compile, I get
06:25:06  Running with dbt=1.1.1
06:25:06  Found 12 models, 2 tests, 0 snapshots, 0 analyses, 668 macros, 0 operations, 0 seed files, 3 sources, 0 exposures, 0 metrics
06:25:06
06:25:06  Encountered an error:
Database Error
Runtime Error
dbt encountered an error while trying to read your profiles.yml file.

Could not automatically determine credentials. Please set GOOGLE_APPLICATION_CREDENTIALS or explicitly create credentials and re-run the application. For more information, please see https://cloud.google.com/docs/authentication/getting-started


F sharp merge 10 2015 b

Gave this another shot by following @rojepp's suggestion of using Beyond Compare. Even with that I found it difficult to assess which of the three changes to take: HEAD, BASE or FSharp.

Something also went wrong with this - there are too many commits in the history here.  Will take a look at the command you use in the other PR


What version of MAgent is compatible with MARLlib?

According to the doc about Environments, it would be fine to just use pip install pettingzoo[magent]. However, the latest version of pettingzoo has moved MAgent to a dedicated project, so its unusable now.
I've compared the code in envs/base_env/magent.py with pettingzoo's previous version.
https://github.com/Replicable-MARL/MARLlib/blob/e1ddcef519b5d6736f140bcf8ad75689de62bff7/envs/base_env/magent.py#L5-L14
It seems like we have to use version before 1.15.0 (e.g. 1.14.0). Yet, after pip install pettingzoo[magent]==1.14.0 and run a test, it reports problem as: "cannot import name 'aec_to_parallel' from 'pettingzoo.utils.conversions'"
I've tried newer version of MAgent and changed magent.py accordingly, but it will raise other problems.
So which version has been tested exactly?

Okay I've found a solution. The following version works for me pip install supersuit==3.3.2 pettingzoo[magent]==1.14.0. Higher version of pettingzoo doesn't match the supersuit check.
I think the document/readme need an update.

A requirements.txt is here for you to refer. Yet there is no need to strictly follow it.
The document/readme will be updated.


[v2.12.0] Ensure CI/documentation reflect changes to default admin credentials #1310

Background
Previously, when installing the security plugin demo configuration, the cluster was spun up with the default admin credentials, admin:admin. A change was made in main and backported to 2.x for the 2.12.0 release, which now requires an initial admin password to be passed in via the environment variable OPENSEARCH_INITIAL_ADMIN_PASSWORD. This will break some CI/testing that relies on OpenSearch to come up without setting this environment variable. This tracking issue is to ensure compliance with the new changes.
Coming from: https://github.com/opensearch-project/security/issues/3624
Acceptance Criteria

[ ]  All documentation references to the old default credentials admin:admin are removed
[ ]  Ensure that CI/testing is working with main and 2.x branches

Resolved by: https://github.com/opensearch-project/data-prepper/pull/3869

@DarshitChanpura , @derek-ho ,
We have a few CI tests that are working currently because OpenSearch 2.12 is not yet released. These still assume the admin password.

These work with older versions, so we probably need to set the password here to admin:

https://github.com/opensearch-project/data-prepper/blob/df2bde6cc4d752013dc9a6f9266f651b43668b23/.github/workflows/opensearch-sink-opensearch-integration-tests.yml#L23-L38

Some of the Docker examples are using the latest version 2 and do not set or use the password.

https://github.com/search?q=repo%3Aopensearch-project%2Fdata-prepper+"opensearch%3A2"&type=code

Also, see this comment:
https://github.com/opensearch-project/data-prepper/pull/3869#issuecomment-1874618527

@dlvenable These need to be addressed post 2.12 release. OR a PR can be raised now that conditionally uses the correct password based on the version. The new password can be with env variable and it will only be picked up for >= 2.12 and will not be used on installs <2.12


Add input to change gha_creds_*.json file location

TL;DR
An input that overrides the env.GITHUB_WORKSPACE used as the path for storing the resulting credentials. This conflicts with checks that look for changed / uncommitted files and for use-cases where actions/checkout is run an additional time later in the job or in a nested composite action later.
Detailed design
Add an input for credentials_path that overrides the use of env.GITHUB_WORKSPACE to be a configurable base path.
https://github.com/google-github-actions/auth/blob/23620afd0f216f749395e1c4f798d973a0cd329a/src/main.ts#L180-L184
The workspace defaults to /home/runner/work/{{ repo }}/{{ repo }} and that is also where code is checked out. There is some risk that the creds file gets committed to the repo if the user has an action that does git add . or similar.
Example config:
jobs:
  job_id:
    # ...

    # Add "id-token" with the intended permissions.
    permissions:
      contents: 'read'
      id-token: 'write'

    steps:
    # actions/checkout MUST come before auth
    - uses: 'actions/checkout@v3'

    - id: 'auth'
      name: 'Authenticate to Google Cloud'
      uses: 'google-github-actions/auth@v1'
      with:
        credentials_path: '/home/runner/work/.config/gcloud/'
        service_account<EMAIL_ADDRESS>
Additional information
No response

Duplicate of https://github.com/google-github-actions/auth/issues/109
Duplicate of https://github.com/google-github-actions/auth/issues/134
Similar to https://github.com/google-github-actions/auth/issues/123
Similar to https://github.com/google-github-actions/auth/issues/212
This is also documented heavily in the README and TROUBLESHOOTING documentation.

I'm not sure I follow:

This conflicts with checks that look for changed / uncommitted files...

This is the exact use case I described above.

...and for use-cases where actions/checkout is run an additional time later in the job or in a nested composite action later.

You can always run the auth action again, which I would definitely recommend doing in composite actions. It feels weird that your composite action would assume authentication existed (but not assume that a checkout existed). You can also use path on the checkout action to check out to a different path.
You're free to move or rename the creds outside of the default path, but this introduces a security risk for self-hosted runners, so we intentionally don't expose such functionality in the package.


PayEmbed context cleanup, Add PayEmbedUI

PR-Codex overview
The focus of this PR is to enhance the wallet connection and chain switching functionalities in the ConnectWallet screens for buying and swapping.
Detailed summary

Added switchChain function for chain switching in various components.
Updated props and imports related to Account and Chain.
Improved handling of active wallet, account, and chain in UI components.

The following files were skipped due to too many changes: packages/thirdweb/src/react/web/ui/ConnectWallet/screens/Buy/tx-history/FiatDetailsScreen.tsx, packages/thirdweb/src/react/web/ui/ConnectWallet/screens/Buy/fiat/PostOnRampSwap.tsx, packages/thirdweb/src/react/web/ui/ConnectWallet/screens/Buy/tx-history/BuyTxHistory.tsx, packages/thirdweb/src/react/web/ui/PayEmbed.tsx, packages/thirdweb/src/react/web/ui/ConnectWallet/screens/Buy/swap/ConfirmationScreen.tsx, packages/thirdweb/src/react/web/ui/ConnectWallet/Details.tsx, packages/thirdweb/src/react/web/ui/ConnectWallet/screens/Buy/BuyScreen.tsx, packages/thirdweb/src/react/web/ui/PayEmbedUI.tsx

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

question: should we add a new component PayEmbedUI?

TODO: test this


Test build for algebras merge

Do not merge
Just a test PR for travis to make sure it can build the algebras PR after merging master into it.

Current coverage is 81.15%

Merging #1008 into master will decrease coverage by -0.90% as of 034aa73

@@            master   #1008   diff @@
======================================
  Files           88     121    +33
  Stmts         7891    8298   +407
  Branches       256     280    +24
  Methods          0       0
======================================
+ Hit           6475    6734   +259
  Partial          0       0
- Missed        1416    1564   +148

Review entire Coverage Diff as of 034aa73

Powered by Codecov. Updated on successful CI builds.


stackvis share does not work inside Manta jobs

The tool checks too aggressively for Manta environment variables that aren't strictly required.

Before, when run inside a Manta job:
# env | grep MANTA_
MANTA_JOB_ID=f9a2846a-b13e-4b23-cb10-de2072737ed3
MANTA_USER=dap
MANTA_REDUCER=0
MANTA_OUTPUT_BASE=/dap/jobs/f9a2846a-b13e-4b23-cb10-de2072737ed3/stor/reduce.0.
MANTA_NO_AUTH=true
MANTA_URL=http://localhost:80/

dap@manta # json name version < package.json
stackvis
0.4.0
dap@manta #

dap@manta # ./cmd/stackvis share < stacks.htm
MANTA_USER, MANTA_URL, and MANTA_KEY_ID must be set in the environment.
For details, see http://apidocs.joyent.com/manta/#getting-started
dap@manta #

With the change:
dap@manta # json name version < package.json
stackvis
0.5.0

dap@manta # ./cmd/stackvis share < stacks.htm
http://localhost:80//dap/public/stackvis/74a7b645-ff80-4cd0-9d6d-abf1e2bbb264/index.htm
dap@manta #

It's unfortunate but expected that the link refers to "localhost:80".  If you fix that to point at whatever Manta you're in, the link does work as expected.
The program still fails if you haven't set either of the other two variables:
dap@manta # MANTA_USER= ./cmd/stackvis share < stacks.htm
MANTA_USER and MANTA_URL must be set in the environment.
For details, see http://apidocs.joyent.com/manta/#getting-started

dap@manta # MANTA_URL= ./cmd/stackvis share < stacks.htm
MANTA_USER and MANTA_URL must be set in the environment.
For details, see http://apidocs.joyent.com/manta/#getting-started
dap@manta #

Published as v0.5.0.


Reagarding animation

This is issue occurs when we scroll up fast from 3rd or 4th page it touches the top and come back to 2nd or 3rd page and similar occuring in last pages.
Expected: when move fast to top page from where page start that will not come back to 2nd or 3rd page it should be stop where page start and similarly in pdf end when it should be stop at the end of the pdf not on second last or third last.

Possible duplicate of #61?

Can you provide me some solution?

I've been investigating the issue. Currently, no solution :(

If you can add some other animation effect may be this will help


Bump netaddr from 1.5.1 to 2.0.4

Review https://github.com/ministryofjustice/moj-ip-addresses/pull/48
Do we want to update? Is there any impact from update?
If no issues, carry out update.

Done


style: fix formatting after CI commit

Seems like 8e5fa3a24cf217b730d35a7f825ba0509856a1bb broke the formatting.

Thanks for fixing, but we just need to ignore JSON from linting/formatting. Otherwise npm/pnpm and Biome will be fighting over the formatting of these files

Fixed in #1797; just ignored from Biome. Whenever pnpm touches this it’ll reformat it anyway; no sense in linting.


Invisible icon in dark mode

It's not a big problem, but I wanted to report it anyway. The extension icon is not visible if the browser is in dark mode.

Good point. Thanks for reporting.
Since the app icon serves no purpose, you can simply hide it in the plugin menu.
Manifest v3 supports multiple icons for light and dark theme. I should be able to fix this by adding a dark-theme icon or simply add a white outline.


move sensu plugins to own git repo

Aside from Ursula-monitoring not having recent changes to monitoring plugins looks good.
Once this is merged, please backport to 1.1.x and 1.0.x so we don't need to maintain checks in three different places.

updated ursula-monitoring to match exactly what we have in ursula.

Once this merges I'll create the backports and lock them to a specific sha/release of ursula-monitoring.


InvokeAsync causes program to hang in WPF-based use case

I'm using Dispatcher.InvokeAsync to run continuous tasks on the main thread, while giving UI the chance to respond to events. WPF has this as intended behavior: https://docs.microsoft.com/en-us/dotnet/desktop/wpf/advanced/threading-model?view=netframeworkdesktop-4.8
In practice, I'm seeing the opposite of the WPF behavior in Avalonia, and the UI freezes instead.
A simple example, added to the axaml.cs for a single button's Click event:
public void Button_Click(object sender, RoutedEventArgs e)
{
	Dispatcher.UIThread.InvokeAsync(TestMethod, DispatcherPriority.SystemIdle);
}

public void TestMethod()
{
	Console.WriteLine("Tick");
	Dispatcher.UIThread.InvokeAsync(TestMethod, DispatcherPriority.SystemIdle);
}

Full example in my repo: https://github.com/audinowho/WPFAvaloniaTest/tree/master/AvaloniaTest
Scheduled TestMethods appear to be dequeued immediately after, instead of giving control back to UI to update once before rechecking queue.  Not sure if intended.

I'm using Dispatcher.InvokeAsync to run continuous tasks on the main thread, while giving UI the chance to respond to event

I believe await Task.Yield() usually used for that.
Although I think InvokeAsync approach should also work at least for specific priorities like SystemIdle.

Should be fixed by the recent dispatcher rework


[BUG] Microsoft.Azure.NotificationHubs Version=4.2.0

Describe the bug
We are getting errors while trying to update the Microsoft.Azure.NotificationHubs' Version='4.2.0'.
According to NuGet Gallery | Microsoft.Azure.NotificationHubs 4.2.0 the dependency in .NETStandard 2.1 is for package
Microsoft.Extensions.Caching.Memory with version (greater or = 6.0.1) but at runtime version 6.0.0 is expected.
Exception or Stack Trace
Error Message:
System.IO.FileNotFoundException
HResult=0x80070002
Message=Could not load file or assembly 'Microsoft.Extensions.Caching.Memory, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60'. The system cannot find the file specified.
Source=Microsoft.Azure.NotificationHubs
StackTrace:
at Microsoft.Azure.NotificationHubs.Auth.TokenProvider..ctor(Boolean cacheTokens, TimeSpan cacheExpirationTime)
at Microsoft.Azure.NotificationHubs.Auth.TokenProvider..ctor(TimeSpan cacheExpirationTime)
at Microsoft.Azure.NotificationHubs.Auth.SharedAccessSignatureTokenProvider..ctor(String connectionString)
at Microsoft.Azure.NotificationHubs.Auth.SharedAccessSignatureTokenProvider.CreateSharedAccessSignatureTokenProvider(String connectionString)
at Microsoft.Azure.NotificationHubs.NotificationHubClient..ctor(String connectionString, String notificationHubPath, NotificationHubSettings settings)
at Microsoft.Azure.NotificationHubs.NotificationHubClient..ctor(String connectionString, String notificationHubPath)
at Microsoft.Azure.NotificationHubs.NotificationHubClient.CreateClientFromConnectionString(String connectionString, String notificationHubPath)
To Reproduce
Step 1 - Update Microsoft.Azure.NotificationHubs to Version=4.2.0 in Azure function version 3.
Step 2 - Update Microsoft.Extensions.Caching.Memory  to version 6.0.1 in the same Azure function
Step 3 - Call the following method available in Microsoft.Azure.NotificationHubs: NotificationHubClient.CreateClientFromConnectionString(azureNotificationHubConnectionString, azureNotificationHubName).
Code Snippet
notificationHub = NotificationHubClient.CreateClientFromConnectionString(azureNotificationHubConnectionString, azureNotificationHubName);
Expected behavior
Notification hub client instance created; No dependency to Microsoft.Extensions.Caching.Memory  to version 6.0.0
Screenshots

Setup (please complete the following information):
Visual Studio 2019

Version of the Library used - 4.2.0

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[OK ] Bug Description Added
[OK ] Repro Steps Added
[ OK] Setup information Added

Hi @SanandABB,
Thanks for reaching out. I'm working on repro'ing this bug independently. Can you give me more information about the version of .NET that you're using to host the Azure Functions instance where you are seeing this issue?
As an aside, I've confirmed that using ILSpy, only the major version of the dependency is being rendered. So despite having 6.0.1, ILSpy still reports <IP_ADDRESS>.

Ioana

From: Martin Strobel @.>
Sent: 25 April 2024 02:43 AM
To: Azure/azure-notificationhubs-dotnet @.>
Cc: Sanand IV @.>; Mention @.>
Subject: Re: [Azure/azure-notificationhubs-dotnet] [BUG] Microsoft.Azure.NotificationHubs Version=4.2.0 (Issue #316)
BeSecure!
This email comes from outside of ABB. Make sure you verify the sender before clicking any links or downloading/opening attachments.
If this email looks suspicious, report it by clicking 'Report Phishing' button in Outlook or raising a ticket on MyIS.
Hi @SanandABBhttps://urldefense.com/v3/__https:/github.com/SanandABB__;!!NLW3fF9v!MyGj5kKRhy30l3J-GLM63IVO_NxR8le7xJ7ADxn7Bou2-8MLgM4h9ltbwZSRXaH4ryqirp-Ljtk7uq5w5RMahWQw$,
Thanks for reaching out. I'm working on repro'ing this bug independently. Can you give me more information about the version of .NET that you're using to host the Azure Functions instance where you are seeing this issue?
As an aside, I've confirmed that using ILSpy, only the major version of the dependency is being rendered. So despite having 6.0.1, ILSpy still reports <IP_ADDRESS>.

Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/Azure/azure-notificationhubs-dotnet/issues/316*issuecomment-2075853601__;Iw!!NLW3fF9v!MyGj5kKRhy30l3J-GLM63IVO_NxR8le7xJ7ADxn7Bou2-8MLgM4h9ltbwZSRXaH4ryqirp-Ljtk7uq5w5Z2eTnX5$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/BHYXZQEACVUA5E2YNUAKYJDY7AN43AVCNFSM6AAAAABGEAQKNOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZVHA2TGNRQGE__;!!NLW3fF9v!MyGj5kKRhy30l3J-GLM63IVO_NxR8le7xJ7ADxn7Bou2-8MLgM4h9ltbwZSRXaH4ryqirp-Ljtk7uq5w5fAQQWZM$.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

I have the similar issue and find out that it may be related to "assembly binding redirect" in App.config or Web.config to map the version from <IP_ADDRESS> to <IP_ADDRESS>.  If the bindingRedirect is manually removed, the Error "Could not load file or assembly 'Microsoft.Extensions.Caching.Memory, Version=<IP_ADDRESS>" will throw.
Usually the bindingRedirect is generated automatically. But in some rare cases it can't.  For example,
the reference "Microsoft.Extensions.Caching.Memory" is removed under the Project References in Visual Studio.
Not sure if it is the same case of @SanandABB. My fix is to uninstall both "Microsoft.Azure.NotificationHubs" and "Microsoft.Extensions.Caching.Memory" in NuGet and then install back the NotificationHubs. Then confirm the bindingRedirect is exist in Web.config or App.config.
<configuration>
  ...
  <runtime>
    <assemblyBinding xmlns="urn:schemas-microsoft-com:asm.v1">
      <dependentAssembly>
        <assemblyIdentity name="Microsoft.Extensions.Caching.Memory" publicKeyToken="adb9793829ddae60" culture="neutral" />
        <bindingRedirect oldVersion="<IP_ADDRESS>-<IP_ADDRESS>" newVersion="<IP_ADDRESS>" />
      </dependentAssembly>
    </assemblyBinding>
  </runtime>
</configuration>

I'm also getting the same issue and I did tried @austinlau  solution but didnt work.
Below are the framework and package versions
.NET Framework 4.7.1
Microsoft.NET.Sdk.Functions 1.0.31
Microsoft.Azure.WebJobs 2.3.0
Microsoft.Azure.NotificationHubs 4.2.0

I have same issue, I had clean project, reinstall nuget, but not woking.

Howdy folks - I've accepted a new role and am no longer working on the ANH product. I'm going to unassign myself, but I'll send some emails internally to get this issue attention.

Hi @SanandABB !
Hope all is well. It is possible that the issue could be related to your Azure Functions configuration. I have found some other examples of similar dependency resolution errors involving Azure Functions on StackOverflow - [here is an example of such an issue]. (https://stackoverflow.com/questions/64809716/could-not-load-file-or-assembly-microsoft-extensions-configuration-abstractions)
One possible workaround seems to be updating the Azure Function version to v4. Would that be something you could try and report back the results?
Thanks!

You can solve Azure NH 4.2.0 working by having all the Microsoft.Extensions.* packages to 6.0.0. Here is my packages.config (.NET Framework 4.8 project).
<package id="Microsoft.Azure.NotificationHubs" version="4.2.0" targetFramework="net48" />
<package id="Microsoft.Extensions.Caching.Abstractions" version="6.0.0" targetFramework="net48" />
<package id="Microsoft.Extensions.Caching.Memory" version="6.0.0" targetFramework="net48" />
<package id="Microsoft.Extensions.DependencyInjection.Abstractions" version="6.0.0" targetFramework="net48" />
<package id="Microsoft.Extensions.Logging.Abstractions" version="6.0.0" targetFramework="net48" />
<package id="Microsoft.Extensions.Options" version="6.0.0" targetFramework="net48" />
<package id="Microsoft.Extensions.Primitives" version="6.0.0" targetFramework="net48" />

For this, you need to force ot install Microsoft.Extensions.Caching.Memory to 6.0.0.
You can do this by this NuGet Package Manager Console command:
Install-Package Microsoft.Extensions.Caching.Memory -IgnoreDependencies -version 6.0.0

These possible solutions mentioned above haven't worked for me. I'm unable to ignore dependencies in NuGet Package Manager. Planning to update Azure Function to version 4 but any other ways to fix this issue would be of great help.
Thank you

@TAbbott117
The most upvoted suggestions are to downgrade Microsoft.Extensions.* to a 3.1.X version.  When I attempt to do so, It also wants me to downgrade the Microsoft.Azure.NotificationHubs version to 4.1.0 which I do not want to do.
Other suggestions on this thread have not helped me out either. Not sure if anyone else as another work around idea.  I am a bit lost on this one.


Communications Site Template

This bug is about new "Communications Site" template and SEWP react web part.
Communications Site have top navigation menu , which do not load page completely.
So, when serfing between pages , SEWP do not re-execute scripts.

I have already fixed that on my side .

Added hidden label on page , indicates,  page was already loaded or not .
Used separate script loading for both cases .
I only worry , how many surprises Modern Pages have ? My customer ask me , will it be working stable someday :)

@mstivoi this is not really an issue with modern pages, but with the Script Editor Web Part not handling navigate events correctly.

Yes , i understand . I mean some modern pages functionality , i don't know about ...
I know one more modern SEWP issue/bug . I not able to load Angular Formly .And i am not sure , why .

@mstivoi for repro I did this:

Created a comm site
Create a page a.aspx
Added the wp and alert on that page
Added a link in top nav to /teams/CommSite/SitePages/A.aspx
Navigated to the home page
Click the nav entry, and the alert displayed

..so what am I doing wrong for repro?

Does page "a.aspx" Modern page ? Can you confirm, browser window loads partly , when you navigates between pages ?

@mstivoi it's modern indeed and I get a fetch with: https://tenant.sharepoint.com/teams/CommSite/SitePages/A.aspx?as=json

I have deployed the latest code built on 1.4.1, where I add the wp explicitly to the site as an app.

May be that was old code . You can close this issue...

The codelogic itself hasn't changed, and If you can repro with the latest build please re-open another issue so I can possibly fix it :) If you can create a small angular sample I can deploy easily create an issue for that as well and I can test.


help identify open http client sockets

If a socket is an http client, print some extra details to help identify it.
Looks like this:
[WTF Node?] open handles:
- File descriptors: (note: stdio always exists)
  - fd 1 (tty) (stdio)
  - fd 2 (tty) (stdio)
- Sockets:
  - ::1:51817 -> ::1:51816
    - POST http://localhost/the-path <--------- the new stuff
  - ::1:51816 -> ::1:51817
- Servers:
  - :::51816 (HTTP)
    - Listeners:
      - request: (anonymous) @ /Users/matt/code/wtfnode/tests/test-http-client.js:11

Hi there. Thanks for the contribution/suggestion! Just wanted to acknowledge that I've seen it, but I may not be able to check it out in more detail for some days yet.

Merged / published - thanks again :)


Develop microsim

Getting ready to merge in the microsimulation work

Will hold open whilst batch job work is ongoing


dobot is connect with windows laptop

pydobot: COM3 open
pydobot: >> AAAA:18:81:3:00004843000048430000484300004843:128
pydobot: >> AAAA:10:83:3:0000484300004843:148
Traceback (most recent call last):
File "", line 1, in
File "C:\python3\lib\site-packages\pydobot\dobot.py", line 47, in init
self._set_ptp_common_params(velocity=200.0, acceleration=200.0)
File "C:\python3\lib\site-packages\pydobot\dobot.py", line 132, in _set_ptp_common_params
return self._send_command(msg)
File "C:\python3\lib\site-packages\pydobot\dobot.py", line 66, in _send_command
response = self._read_message()
File "C:\python3\lib\site-packages\pydobot\dobot.py", line 80, in _read_message
msg = Message(b)
File "C:\python3\lib\site-packages\pydobot\message.py", line 11, in init
self.len = b[2]
IndexError: index out of range

is this exception happening when using the basic example? can you please update to version 1.3.0 (just released) and confirm if it still happens? thanks

Issue still occures as of now, using the latest release.

Hi, I'm also trying to get this working on windows and have the same error. Just installed pydoboy with pip install, I'm using python 3.7. Dobot connects fine to Dobot Studio but I'd prefer using just python. Any support much appreciated 😊

@jKostet  Please report you version of windows, version of pydobot (latest probably), the program which fails to function (connect to the robot) and the error message you see (the same as saravanakumarBalakrishnan?).

@jonascj Windows 10 Enterprise, latest pydobot latest / 1.3.2. I'm trying to run the example in the README.
In addition I have the Dobot DLLs from the latest official Dobot 64bit python demo copied to where my python.exe is located.
Here is the full error msg:
`& 'C:\Program Files\Python37\python.exe' .\example.py
available ports: ['COM5', 'COM3']
pydobot: COM5 open
pydobot: >> AA AA:2:240:1::15
pydobot: >> AA AA:2:245:1::10
Traceback (most recent call last):
File ".\example.py", line 9, in
device = pydobot.Dobot(port=port, verbose=True)

File "C:\Users\USERNAME\AppData\Roaming\Python\Python37\site-packages\pydobot\dobot.py", line 31, in init
self._set_queued_cmd_clear()

File "C:\Users\USERNAME\AppData\Roaming\Python\Python37\site-packages\pydobot\dobot.py", line 241, in _set_queued_cmd_clear
return self._send_command(msg)

File "C:\Users\USERNAME\AppData\Roaming\Python\Python37\site-packages\pydobot\dobot.py", line 89, in _send_command
response = self._read_message()

File "C:\Users\USERNAME\AppData\Roaming\Python\Python37\site-packages\pydobot\dobot.py", line 80, in _read_message
msg = Message(b)

File "C:\Users\USERNAME\AppData\Roaming\Python\Python37\site-packages\pydobot\message.py", line 11, in init
self.len = b[2]

IndexError: index out of range`

@jKostet  Thank you. I've just tried the example from the readme on Windows 10 Home and it works without installing Dobot Studio and without installing additional drivers[1]. This library does not communicate with the Dobot via the DLL-files as the official Python example, this library communicates with the Dobot directly over the serial-over-usb connection.
I notice that you have two available COM-ports, COM3 and COM5. Could it be that you (the readme example really) are using the wrong port?
You could try this change to the example:
port = available_ports[1].device # Changed from avilable_ports[0].device

Or try this instead, it lets you choose COM-port:
available_ports = list_ports.comports()
print('Available COM-ports:')
for i, port in enumerate(available_ports):
    print(f"  {i}: {port.description}")

choice = int(input('Choose port by typing a number followed by [Enter]: '))

port = available_ports[choice].device

[1] https://www.silabs.com/developers/usb-to-uart-bridge-vcp-drivers#software This driver might be necessary, if Windows does not have a serial-over-usb driver installed already.

@MikeMikeHHN Thank you for reporting the issue as well. Would you provide your Windows version, pydobot version (1.3.2 probably), the python program which fails (the readme example?) and the exact error message you are seeing?

@jonascj COM5 is what I use with Dobot Studio. Also if the Dobot Studio is connected to the robot, pydobot will give the appropriate error, something like "Dobot is reserved". I'll try COM3 as well, sometimes the ports work a bit weird.
I got pydobot working on a raspberry pi, which will be the actual device I'll be running the Dobot from, so that's good. Getting it working on Windows would be great for easier development though. 😃

@jonascj
Issue arrises under Win10 Home 10.0.19042 Build 19042, pydobot 1.3.2 in the readme example aswell as a selfwritten program that uses pydobot. Error message:
 available ports: ['COM4'] Traceback (most recent call last):   File "C:\Users\Hati\Documents\GitHub\pydobot\basic_example.py", line 9, in <module>     device = pydobot.Dobot(port=port, verbose=False)   File "C:\Users\Hati\Documents\GitHub\pydobot\pydobot\dobot.py", line 31, in __init__     self._set_queued_cmd_clear()   File "C:\Users\Hati\Documents\GitHub\pydobot\pydobot\dobot.py", line 244, in _set_queued_cmd_clear     return self._send_command(msg)   File "C:\Users\Hati\Documents\GitHub\pydobot\pydobot\dobot.py", line 92, in _send_command     response = self._read_message()   File "C:\Users\Hati\Documents\GitHub\pydobot\pydobot\dobot.py", line 83, in _read_message     msg = Message(b)   File "C:\Users\Hati\Documents\GitHub\pydobot\pydobot\message.py", line 11, in __init__     self.len = b[2] IndexError: index out of range
Anything else I can do to help?
On the same note, there is a bad workaround I found, usually flashing the firmware again aswell as reseting via reset switch results in the error not occuring for a while. I know, not a good way but its what I got so far ^^


Add "CaaS"

Hi @iamNoah1, can you please say if you are interested to add the definition of CaaS yourself, or is this issue is for asking help from a contributor?

Hi @sandipanpanda, nice to see you here again :) I think @iamNoah1 wanted to work on it himself but I'll leave it up to him to respond.

@sandipanpanda my primary intention was to raise awareness that the term should be added. I do not have to work on it necessarily, so if there is someone from the community who wants to work on it: go for it :) Otherwise I would probably pick it up.

@sandipanpanda it's all your if you want to take a crack at it.

Hi there, apologies for the late response. I was off GitHub for a month, so didn't know that this issue was assigned to me. Sending a PR to this issue ASAP.


index.d.ts typings for TypeScript

typings for TS

👍

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Allow arbitrary input for -e/--command

This would allow one to use functions and similar constructs there.
Currently trying to do for example alacritty -e 'echo test; sleep 3' will error with
[2019-10-14 17:18] [ERROR] Failed to spawn command: No such file or directory (os error 2)

xterm already allows/handles this.

You need to run it in shell. So you should use alacritty -e sh -c 'echo test; sleep 3'. It fails because you're giving it a command echo test; sleep 3(Thats a full command name), e.g. command -e echo test is fine (echo is a command and test is its argument), the ; is a shell thing so without running sh,bash, etc it's not possible.

@Clyybber would it be fair to rename this to "Allow Shell commands for -e/--command"?
While xterm does allow this, other don't. I might be interested to hear how others think this should work. I feel like it might re reasonable to launch a shell here myself.

@nixpulvis What should we do on Windows in this case (like you're running wsl or whatever)? What shell should we run? What if your shell doesn't use ; as a separator for multiple commands. What should we do about && and || stuff? There are more questions then answers in my opinion, so I think I do understand why no one other then xterm support it. We can implicitly use shell's -c, but if it's not presented?

This has been asked before and I don't think we should ever change this.
If we automatically spawn a shell, there's no way not to. If we don't, the user can always spawn a shell themselves. If the user understands what's going on, our solution is just better for picking exactly what you want. The only drawback would be brevity and sh -c is hardly too much to ask for.
Especially since the userbase of Alacritty is a bit different than something like gnome-terminal, which comes preinstalled on many systems, I think we should be confident in the ability of our users in this regard.

We should improve the error message then, so people don't keep opening this kind of issue.
We at least need to show the string we failed to execute ideally.

We should improve the error message then, so people don't keep opening this kind of issue.

There's not necessarily going to be an error. And I think 'no such file or directory' should already be pretty clear for everyone familiar with unix. If we change that to something we might find more user-friendly, we just risk changing it to something users are unfamiliar with.

I think our error message is fine. Every other terminal reports the same thing, some of them less clear(gnome-terminal). We already have clear --help output
 -e, --command <command>...                     Command and args to execute (must be last argument)

We at least need to show the string we failed to execute ideally.

Just to add on my previous message, this would be no problem. Sounds like a good idea to me and would be a trivial change.

@nixpulvis Fixed it here: https://github.com/jwilm/alacritty/pull/2879, let me know if that's what you're talking about.

For context, I'm trying to use alacritty to launch a program with setsid, in this example alacritty.
I want alacritty to echo something, spawn a new alacritty instance, and exit/close without waiting for the spawned alacritty to terminate.
Unfortunately this doesn't work with the pattern alacritty -e sh -c "echo something; setsid alacritty; exit". In xterm I tried it with `xterm -e "echo something; setsid xterm; exit"; this works, leading me to believe it was an issue with the aforementioned pattern, which it is not. I'm sorry for that.
So the actual issue is in my snippet and xterm was just behaving a tad bit different to alacritty with setsid.

Maybe setsid alacritty& works? I guess xterm backgrounds itself when it launches.

@NickeZ Yeah thanks that works, although for some reason it only works for me (having fish as the default shell) using alacritty -e fish -c "echo something; setsid alacritty &; sleep 1; exit" and not using alacritty -e sh -c "echo something; setsid alacritty &; sleep 1; exit"...
Also because it is setsid that is started in the background I have to insert a sleep so that it doesn't get killed too early to actually start the process, though I'm sure theres a better way to accomplish that.


waitForNewPast() is laggy? Infinite loop?

Hey !
I got an issue with this function, but not all time...
Check this code :
`# Infinite loop, wait for new paste
while True:

    print("clip waiting")

    pyperclip.waitForNewPaste()

    try:

        clipboard_collected = winclip32.get_clipboard_data("unicode_std_text")

    except:

        pass`

I use winclip32 get and set clipboard data, and pyperclip to detect a new paste.
But when I run this loop, this happens (check the screenshot) until I clear my clipboard. But when I copy a new thing, the same infinite non stop loop restart...

I think I have an idea: I had an open VM and maybe that was the problem, I'll update the topic if it's right

Ok, I found where the problem comes from : When I launch 2 scripts which use WaitForNewPaste(), I don't know what happens but it goes to hell :')
So I close this issue but if someone find how to solve this issue : "Using 2 (or more) scripts which are using waitForNewPaste() in an infinite loop without having a flood like in my screenshot" I'd like to know ! Thanks ! :)


Add separate bundle with streams, make bundler size printing prettier and more flexible

There's now also a browser-full.js entry point and mithril-full.js/mithril-full.min.js bundle that include mithril/stream inside them.
I also fixed the .gitattributes as a driveby.
@lhorie WDYT?

To clarify, the full bundle includes m.stream = require("mithril/stream") (and its submodules as individual methods).

This stemmed from a Gitter discussion.

I was planning to make stream its own thing, rather than having a core bundle with it in it.

I feel there still should be a bundle with the streams, even if it's
separate, just so you can more easily use them in fiddles, pens, etc. That
was the purpose behind the PR.
On Wed, Nov 23, 2016, 08:21 Leo Horie<EMAIL_ADDRESS>wrote:

Closed #1420 https://github.com/lhorie/mithril.js/pull/1420.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lhorie/mithril.js/pull/1420#event-869678716, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AERrBNBTbXjeKntpIe7stbsscB_MxIxRks5rBD3agaJpZM4K6eps
.

I'll update this PR to have a bundle with just streams instead (instead of
opening an identically named PR).
On Wed, Nov 23, 2016, 17:21 Isiah Meadows<EMAIL_ADDRESS>wrote:
I feel there still should be a bundle with the streams, even if it's
separate, just so you can more easily use them in fiddles, pens, etc. That
was the purpose behind the PR.
On Wed, Nov 23, 2016, 08:21 Leo Horie<EMAIL_ADDRESS>wrote:
Closed #1420 https://github.com/lhorie/mithril.js/pull/1420.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lhorie/mithril.js/pull/1420#event-869678716, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AERrBNBTbXjeKntpIe7stbsscB_MxIxRks5rBD3agaJpZM4K6eps
.


openlayers performance

Using openlayers and geojs to display 90,000 features, when zooming or panning, the openlayers will lag for 1~2 seconds, but geojs works smoothly. Is the usage of openlayers not correct?
Here is my reference code:https://github.com/nipeone/geojs_vs_openlayers

Performance might be improved by setting renderBuffer to recommended value(the size of the largest symbol, line width or label; default is 100)

Another option is to use an ImageVector layer instead of a Vector layer.

Another option is to use an ImageVector layer instead of a Vector layer.

Thanks for your reply, do you mean the class ol.layer.VectorImage? I am using this class and it does perform better than ol.layer.Vector. But still not as good as geojs performance.

@nipeone I don't think the renderBuffer advice applies here.

@nipeone If you want to use the WebGL renderer (which might improve performance), you need to write different code. See e.g. https://openlayers.org/en/latest/examples/webgl-vector-layer.html. Looks like what you have now comes from an OpenLayers v3 example.

that's

@nipeone If you want to use the WebGL renderer (which might improve performance), you need to write different code. See e.g. https://openlayers.org/en/latest/examples/webgl-vector-layer.html. Looks like what you have now comes from an OpenLayers v3 example.

@ahocevar Thanks again, performance improves dramatically with the above example, but if increase the number of features to 360000, an error will be reported like this: RangeError: Invalid array length

@nipeone If you want to use the WebGL renderer (which might improve performance), you need to write different code. See e.g. https://openlayers.org/en/latest/examples/webgl-vector-layer.html. Looks like what you have now comes from an OpenLayers v3 example.

When using WebGLLayer to render massive vector features, zooming or moving the map causes the memory to increase sharply until the web page crashes. In which version can this problem be solved?

@jia-mx you can try my example code, i did not test max limitation, when len(features)=90000, everything is ok.if len(features)=360000 will report error

@jia-mx you can try my example code, i did not test max limitation, when len(features)=90000, everything is ok.if len(features)=360000 will report error

@nipeone Your example has the same problem.  Continuously zooming or moving the map will cause memory to increase dramatically until the page crashes. @ahocevar In which version can this problem be solved?

This is something for @jahow to answer.

@ahocevar  is there an interface to free memory after panning and  zooming? I'm not sure about the memory management rules of openlayers, from the phenomenon, it should be caused by not releasing the memory in time after each pan and zoom.

When uploading shp of a file of about 12 MB to the webgl layer using the WebGLVectorLayerRenderer (the feature is about 5000 multipolygons), a rangeerror invalid array length occurs.

When uploading shp of a file of about 12 MB to the webgl layer using the WebGLVectorLayerRenderer (the feature is about 5000 multipolygons), a rangeerror invalid array length occurs.

oh my... I solved it simply with VectorImageLayer. thank you

For WebGL, @jahow should be able to help.

老哥不会是长沙腾讯的吧

Unfortunately there are still unsolved issues regarding memory consumption with the WebGL vector renderer, I'm sorry for not having a better answer to that :/
Also 360k features sounds like it might be too much for the renderer at that time to be honest. You might want to look into using vector tiles to achieve btter results.

mabye you can try deck.gl

Hi @nipeone, could you please try out the latest build of OpenLayers and give a feedback on whether performance has improved? thanks a lot!

i have updated openlayers to 10.2.1, Unfortunately the err is stills here.


[FLINK-8286][Security] Fix kerberos security configuration for YarnTaskExecutor

What is the purpose of the change
Fix broken YARN kerberos integration for flip-6.
Brief change log

Fix kerberos onfigurations.
Refactor code.
Add unittest.

Verifying this change
This change added tests and can be verified as follows:

*Added test that validates kerberos credentials are set correctly *

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): ( no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no )
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

@suez1224 I'll now merge the actual fix, but I'm not 100 % sure the refactoring is correct.
After the fix, we have roughly this path through the code:
if (keytabPath != null && remoteKeytabPrincipal != null) {
	configuration.setString(SecurityOptions.KERBEROS_LOGIN_KEYTAB, keytabPath);
	configuration.setString(SecurityOptions.KERBEROS_LOGIN_PRINCIPAL, remoteKeytabPrincipal);
}

SecurityConfiguration sc = new SecurityConfiguration(configuration);

SecurityUtils.install(sc);

SecurityUtils.getInstalledContext().runSecured(new Callable<Void>() {
	@Override
	public Void call() throws Exception {
		TaskManagerRunner.runTaskManager(configuration, new ResourceID(containerId));
		return null;
	}
});

after the fix, that becomes
// in main()
SecurityUtils.getInstalledContext().runSecured(
   YarnTaskExecutorRunnerFactory.create(System.getenv()));

// in YarnTaskExecutorRunnerFactory.create()
if (keytabPath != null && remoteKeytabPrincipal != null) {
	configuration.setString(SecurityOptions.KERBEROS_LOGIN_KEYTAB, keytabPath);
	configuration.setString(SecurityOptions.KERBEROS_LOGIN_PRINCIPAL, remoteKeytabPrincipal);
}

SecurityConfiguration sc = new SecurityConfiguration(configuration);

SecurityUtils.install(sc);

return new Runner()

Meaning, that if someone messes with how things are called it can happen that SecurityUtils.getInstalledContext() is called before SecurityUtils.install(sc) is called in YarnTaskExecutorRunnerFactory.create. I think this can potentially lead to problems and the old path is much clearer. What do you think?

Merged, could you please close the PR after discussion of the refactoring?

@aljoscha that's a very good catch. I added another commit to make sure the call order is always correct (https://github.com/apache/flink/pull/5896/commits/1298b04ea80b9160ac7e4ae8a171f80112a95945). Could you please let me know what you think? I think we should have the unittest in place since this is the second time it breaks. I can create another PR for the refactoring commit.

This PR seems to obfuscate the fix.  The issue was with the interpretation of the keytab path, right?  But the bulk of the change was to clarify the ordering of context installation vs use?

@EronWright the fix is really one this: https://github.com/apache/flink/commit/ba3e2711a2091b7c0907a5caf3ea527f837a442f But I haven't yet managed to reproduce a failure on my system without the fix. Still working on it.

@suez1224 Did you rebase on master? I think that krb5-specific could should not be there anymore.

Closing this PR as it seems to be abandoned.


ponyc not installable on FreeBSD

Trying to install ponyc on FreeBSD 12.1 fails with a error
(20:34:01 <~>) 0 $ ponyup default x86-64-unknown-freebsd-12.1
(20:40:44 <~>) 0 $ ponyup update ponyc release
error: unexpected selection: ponyc-release-x86-64-unknown-freebsd-12.1

I have also tried with other platform strings, though none seems to work.

@Theodus turning on the select test per #159 demonstrates that we indeed have a bug here. I also am working on a PR #161 to add bootstrap testing on FreeBSD 12.1.

This should be resolved now. Sorry for the delay.


impl Command for <impl FnOnce(&mut World)>

This is my first contribution to this exciting project! Thanks so much for your wonderful work. If there is anything that I can improve about this PR, please let me know :)
Objective

Fixes #2899
If a simple one-off command is needed to be added within a System, this simplifies that process so that we can simply do commands.add(|world: &mut World| { /* code here */ }) instead of defining a custom type implementing Command.

Solution

This is achieved by impl Command for F where F: FnOnce(&mut World) + Send + Sync + 'static as just calling the function.

I am not sure if the bounds can be further relaxed but needed the whole Send, Sync, and 'static to get it to compile.

@alice-i-cecile I have now updated the tests with an assertion that only succeeds if the commands have run.

@alice-i-cecile I have now updated the tests with an assertion that only succeeds if the commands have run.

bors r+


serialize.VarInt errors

128     00
255     7f
16512   8000
etc
python 3.7.7, 2.17.18, Fedora 30 x32

You may be looking for python-bitcoinlib.

No, it is not the same.
LevelDB (index/*) stores variants exactly like your's VarInt (WriteVarInt and ReadVarInt in serialize.h of vanilla bitcoin-core).
But without same errors.

Sorry, I responded late last night. Thank you for the big report, that is very helpful.
What I mean is that python-bitcoin is abandon-ware. No, it is worse than that: I need it to support some obscure altcoin tool I’m still using but in the process of winding down, so I’m the mean time I’m only supporting the features that project needs, a small subset having to do with serialization of transaction data, and I’m not even trying to maintain compatibility with bitcoin itself. After that it will truly be abandoned.
So again, thanks for the bug report. I’m leaving this issue open as a reminder to check if this bug affects my use of the library, and if so fix it.
But I wonder, respectfully, why you are using this library, and if you’d be better served by using python-bitcoinlib, an actively developed library maintained by my good friend @kanzure.


Rename feature_map encoding_circuit

Previous PR had an issue due to wrong pycharm refactoring. Easiest solution was to start over again.

Bisher brute force search and replace.

Manche Sachen wie examples fehlen noch

tests laufen durch

Is this ready to be merged or is there something left todo?

sorry, since I had to replace the original PR, this has lost its draft status. I will finish the PR during the next week and let you know, once it's done


Usernames are not displayed for peers

Describe the problem
When Keycloak is used in connection with netbird, no usernames are displayed at the peers.
To Reproduce
Steps to reproduce the behavior:

Go to 'Peers'
Click on one 'Peer'
In the sidebar the username empty and the field is grayed out

Expected behavior
Username of peer should be displayed
Screenshots

Hey @giezi
The names are not displayed because there is no integration with Keycloack user API.
We have only one yet - Auth0. See https://github.com/netbirdio/netbird/tree/main/management/server/idp.
NetBird doesn't store any personal info locally.
But the implementation should be simple. What is your experience with Golang? Maybe you could help us out with it? :)

Hi @braginini -unfortunately, I don't have any experience in Golang. However, we would like to support the project and see if we can find someone on Upwork who will implement this and provide it back into the project.

@giezi
That's awesome! Thank you.
The implementation should implement this interface https://github.com/netbirdio/netbird/blob/ed4f90b6aa7193e9da99f6d05d89da531bb16e76/management/server/idp/idp.go#L12
Here is the example https://github.com/netbirdio/netbird/blob/ed4f90b6aa7193e9da99f6d05d89da531bb16e76/management/server/idp/auth0.go#L23
I belive that this Keycloack API reference can be used
https://www.keycloak.org/docs-api/15.0/rest-api/index.html#_users_resource

@braginini The work is done: https://github.com/netbirdio/netbird/pull/746

@braginini The work is done: #746

Damn awesome! We will have a look

@braginini I saw that the pull request was accepted, and the feature is implemented :) Happy weekend!


Request: Migrate library to Kotlin Multiplatform

I'd like to be able to use json5k in a Kotlin Multiplatform project. Currently json5k only targets JVM, so this is not possible.
There's basically 3 main tasks to do this

update the Gradle config
replace JVM specific code with multiplatform equivalents
update the tests

The docs will also have to be updated.
Update Gradle config
Nothing too exciting here, but this should be the first step.
Initially json5k can be updated to be a multiplatform project that only targets the JVM. Additional targets can be added once the following steps are done.
JVM specific code
json5k has a limited amount of dependencies on JVM classes

java.io.BufferedWriter
java.io.ByteArrayInputStream
java.io.ByteArrayOutputStream
java.io.Flushable
java.io.IOException
java.io.InputStream
java.io.OutputStream
java.io.Writer

I haven't looked in detail, but these can probably be replaced with the multiplatform library Okio. If you don't want to introduce a dependency, then it's possible to create expect/actual definitions per platform (although that could be a lot of work!).
Update the tests
JUnit is not multiplatform, so should be replaced.
Kotest (my recommendation) or kotlin.test are both suitable JUnit replacements for testing multiplatform code.

Thank you for your suggestion!
I am actually quite confident that all java.io dependencies could be completely removed from the core of the library. Therefore, a dependency on Okio (or similar) might not even be necessary.
For the parsing process, an InputReader interface to decouple business logic from the input provider is already in place. I think we can just modify the Json5.decodeFromString function to forward JSON5 input sequences directly to this interface. Then, the Json5.decodeFromStream function would be the only location where expect/actual definitions are necessary. Since it's just one location, the effort should be manageable.
An analogous interface for the generator is not yet available, but it would probably be even easier to define and use.
Please let me know what you think. If you agree, I will try to get it done in the next weeks.

I am actually quite confident that all java.io dependencies could be completely removed from the core of the library. Therefore, a dependency on Okio (or similar) might not even be necessary.

Ah great!
I also had a further look into the testing. I saw that json5k already uses kotlinx.test, and only uses @Nested from JUnit, so that should be easy to migrate.

Please let me know what you think. If you agree, I will try to get it done in the next weeks.

Sounds good 👍
If you like I can get started by making a PR to convert the Gradle config? No code changes, only the build config. I've already done something similar in https://github.com/Kantis/ks3, so I can copy and paste stuff across.

Absolutely, I am happy to accept a PR on this. Thank you for offering your support!

Thank you for your PRs! In the meantime, I have removed all java.io dependencies from the core of the library and added a second target for testing purposes:
kotlin {
    js(IR) {
        nodejs()
    }
}

It is available in the multiplatform branch. With this version, Json5.encodeToString and Json5.decodeFromString should now work on JavaScript platforms as well.
Unfortunately, I had to add expect/actual definitions for writing out floating-point values during the serialization of an object hierarchy. Previously, the library just used Double.toString(), but this call seems to behave inconsistently across different targets. Therefore, a platform-independent version of the library core depends on:

https://discuss.kotlinlang.org/t/decimal-format-in-a-multi-platform-build/5692
https://youtrack.jetbrains.com/issue/KT-21644

Did you have a specific target platform in mind when you opened the issue?

Nice work!
I want to use json5k on Windows, macOS, and Linux.

Just a quick update on this: Serializing floating-point values in a portable manner turned out to be more difficult than expected. In my opinion, the json5k output should be independent of the utilized backend. As described in this issue, however, such an independence is currently not even implemented by the official JSON format.
For the time being, I therefore decided to merge the support for the JVM backend into the main branch. In case you want to give the current snapshot version a try, you can find it here.

Thanks for your effort, everyone! 🙏  Would you mind adding support for iosArm64, iosSimulatorArm64 and iosX64 targets to the snapshot? Or is this out of scope of this issue?

Sure, not a problem at all! :slightly_smiling_face:
I activated a bunch of native targets (including the ones you are asking for) and added a CI job to publish snapshots automatically. Please find the latest artifacts here. I am still a bit hesitant to release a stable multiplatform version, though. Experience reports on how the current snapshot behaves on native target platforms would be highly wecleome!

Awesome, many thanks @xn32 ! 🙏  Will try it out in the next days and let you know how it went!

@xn32 just tested it out, and it's working great! 🤗

Thank you for your feedback. In the next days, I will switch to v1.5.0 of kotlinx.serialization and use this opportunity to publish a stable version of all the currently available targets.

v0.3.0 is now available on Maven Central. It is tested to the best of my ability, but please keep in mind that it is still an experimental library without an entirely stable API. If you run into problems, I would appreciate your feedback. Support for the JS target still depends on the availability of a portable String.format function.
Therefore, I am going to close this issue for now. Thank you for your contributions, everyone!


I don't found the presentation mode

Hello I don't find the button for the presentation mode, I have the frame but not the presentation, is there a solution? Thank

Hello,
Presentation mode is only available in Excalidraw+


Disable SYS_close syscall when Thread Sanitizer is active

Thread Sanitizer doesn't intercept syscall(SYS_close, fd) that's used
instead of close(fd); on Linux.  That leads to false positives as Thread
Sanitizer thinks the descriptor is still being used by the thread.
Disabling the usage of SYS_close won't help in common case where the
libuv is not instrumented, but at least it gives the developers an
option to prepare a Thread Sanitizer compatible build by compiling an
instrumented version.
Some extra notes:

Is there a place where to document this?
Is the performance benefit (for streams) that high that it justifies the use of SYS_close? I understand the need of syscalls() for interfaces that might not be in libc, but is that really necessary for close()?

syscall(SYS_close) is for correctness, not performance. The commit log for 2138dd2f08853b376c4154edd3b1a50262746692 has a bit more detail but, in a nutshell, close() is a cancellation point and therefore a potential resource leak.

@bnoordhuis Thanks for the explanation, somehow I missed the comment at the top of the function.
Meanwhile, after more chat with LLVM/TSAN developer, I changed the commit to use pre- and post- sanitizer wrappers, which has the same effect, but keeps the syscall(SYS_close, ...) instead of calling close(...).

bump

Landed in https://github.com/libuv/libuv/commit/c3fe3cf1bc78cd454cfa114a59f631b9bae5c0d5. Thanks!


Fix jchar segfault

JChar operators did not consider the case in which the order of arguments was reversed.  The add method called the wrong slot when adding to unicode variables.   Both problems are fixed.
Fixes #1053

@hx2A please verify the behaviors make sense.

@Thrameos Looks good to me. Your unit tests are very thorough. Thank you!


Pass serde object as input to revm

The powdr runtime provides get_data to receive data as circuit inputs.  we need to pass the whole transaction/block and new State as public inputs to the revm circuit.

This is actually being developed in powdr_revm


 If you have one thousand instances of the component, the same events will be bind to the document one thousand times.
I'm not sure if it is possible to use event delegation or some other strategy to handle things differently, but I think that performances are acceptable for most use cases. Honestly I don't see the point of supporting up to one thousand instances.
An improvement has been made in version 2, in the sense that events are registered only when an element is active.

Yes, I also tried to change it and add an event to him when it is activated.


The resetToFullScale，How to maintain the rotation angle of the object

resetToFullScale.js
if I want to maintain the rotation angle of the object，What should I do?

Hi jxyun99.
The angle is recorded in Tag.
It would be an easy way to get the value from there and then restore the angle.
For example:
var item = app.activeDocument.selection[0];
var tag = item.tags[0];
var value = tag.value;

// convert radian to degree
var deg = 180 / Math.PI * value;

resetToFullScale(item);

// restore the angle
item.rotate(deg);

https://github.com/jxyun99/test/blob/main/test.gif
I seem to have found a bug. If an object is rotated by an angle, such as -180°, the position after resizing will not be the original position

I have confirmed the bug here as well.
I'll fix it.

Fixed a bug.

Thank you. I just tested it is ok


I18n cn

cn trans underwent multiple proofreaders, i checked structure, layout and links.  looks awesome

I don't know chinese :) has this been reviewed?

I don't know chinese :) has this been reviewed?

yes they did translation, first proofreading, final proofreading.
should be all fine.


No display found on X11 with bspwm

Hey folke,
You've created the project that's been brewing in my head for the past few months (seriously what haven't you written?), and I would love to try it out before taking a swing at it myself. Unfortunately I get a segfault when trying to run polydock. Here is my error:
(gjs:5505): Wnck-WARNING **: 20:38:01.033: libwnck is designed to work in X11 only, no valid display found
^C/sbin/polydock: line 13:  5505 Segmentation fault      (core dumped) gjs "$dist/polydock.js" $@
[restarting]

I did, however, install polydock from the AUR with yay -S polydock instead of building from source. Can you offer any advice on how to get it working? Thanks!

Same issue here on AwesomeWM. Using Arch, and installed from zip in releases

Just to add another test: i3 with a fresh build or with the binary from the releases doesn't work.

Same issue here Arch with i3.

Hello has anyone been able to fix this? I am running into the same issue with i3 and Arch.

Glad to know other people have been running into this issue. I'm having this issue running Arch and gnome


Revert "Fix broken headings in Markdown files"

Reverts blockchain/api-v1-client-python#46

This seems to be the old github markdown? Regardless, the markdown was updated with the last client upgrade. Thanks


Simple bed files

Hi, is there a simple way to visualize basic bed files in higlass? I guess it's possible to go through the whole gene annotation process, but this seems very convoluted for just showing a few lines.

Yes! You just have aggregate them using clodius:
      clodius aggregate bedfile \
          --assembly hg19 \
          somefile.bed

And then ingest them using --filetype beddb and --datatype bed-like:
sudo docker exec higlass-container python higlass-server/manage.py \
  ingest_tileset \
  --filename /tmp/somefile.beddb \
  --datatype bed-like \
  --filetype beddb

There is documentation in the new draft documentation here and here.

Ah, cool, thank you!

OK mmm I am not sure whether I should just continue this here, but somehow after ingesting the file doesn't appear in the list of available datasets... I even restarted the whole docker container. And it is listed in the admin interface.

Sorry, can you try with --datatype bedlike? (no dash in bedlike)

Yep, that's it, thanks!


Hovering over certain buttons, it plays multiple click or hover sounds being really loud.

Added it to my ui, works fine except for some Buttons which for some reason play 9 sounds at a time (I printed _play_stream) not really sure why it makes it really loud or puts that many hover and click sounds over one button.
even changing max_depth to 3, it still plays 9 sounds from the button

I appreciate the feedback! I scanned through the code, but I'm still not sure why that would be happening. Which buttons are causing the issue?
Here are some things I'd try in order to debug this further.

Check your scenes for redundant UISoundController nodes. The option is provided to use either a project wide autoload (ProjectUISoundController), or the nodes in individual scenes, but you shouldn't need to use both.
Disable the "Persistent" option on whichever UISoundController you are using and see if the problem persists (it may introduce new problems, but at least it'll narrow down the issue).
Add a print(node, ":", signal_name) around https://github.com/Maaack/Godot-UI-Sound-Controller/blob/53def554ab4b4497f26d24a73c1249fa512cb84c/addons/maaacks_ui_sound_controller/base/scripts/UISoundController.gd#L129 and see if the same button is getting connected to the same signal multiple times.

Thanks for reporting the issue. Please let me know if you get any results from the above.


Feature/doctests

This PR makes some updates to the documentation, including:

resolving #74
adding doctests to the Travis CI routine
minor refactors

Codecov Report

Merging #75 (b33c98f) into develop (de18385) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop      #75   +/-   ##
========================================
  Coverage    69.35%   69.35%
========================================
  Files           10       10
  Lines         3567     3567
========================================
  Hits          2474     2474
  Misses        1093     1093

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update de18385...b33c98f. Read the comment docs.


DuckDuckGo adds a "." in front of most urls if you dont put "https://" before it

This issue is pretty self-explanatory. This also seems to affect android in a different way, instead of putting a "." it crashes the app.

https://user-images.githubusercontent.com/43621653/185707539-b321daa6-1468-4bc4-ab38-78fb1f6db68b.mp4

To re-produce, go to any url (without adding https://) and it should put a "." in front of it, essentially not allowing the site to be reached.


Give an option to opt out type down-casting

First I need to thank for this great library. It bridges the typing of database and api, which is revolutionary.
Problem
Schema.shape is not accessible when using schema.
I have the following generated code:
import { z } from 'zod';

import type { Prisma } from '@prisma/client';

const Schema: z.ZodType<Prisma.UserCreateInput> l= z
  .object({
    email: z.string(),
    hashedPassword: z.string(),
    lastLogin: z.date().optional(),
    createdAt: z.date().optional(),
    updatedAt: z.date().optional(),
  })
  .strict();

export const UserCreateInputObjectSchema = Schema;

Seems I cannot access Schema.shape because it is not on z.ZodType<Prisma.UserCreateInput>. After removing it, I get
type Schema = z.ZodObject<{
    email: z.ZodString;
    hashedPassword: z.ZodString;
    lastLogin: z.ZodOptional<z.ZodDate>;
    createdAt: z.ZodOptional<z.ZodDate>;
    updatedAt: z.ZodOptional<z.ZodDate>;
}, "strict", z.ZodTypeAny, {
    ...;
}, {
    ...;
}>

and Schema.shape =
(property) ZodObject<{ email: ZodString; hashedPassword: ZodString; lastLogin: ZodOptional<ZodDate>; createdAt: ZodOptional<ZodDate>; updatedAt: ZodOptional<...>; }, "strict", ZodTypeAny, { ...; }, { ...; }>.shape: {
    email: z.ZodString;
    hashedPassword: z.ZodString;
    lastLogin: z.ZodOptional<z.ZodDate>;
    createdAt: z.ZodOptional<z.ZodDate>;
    updatedAt: z.ZodOptional<z.ZodDate>;
}

With this, we can access metadata when defining the database. However, this leads to more complicated type and might not work on bigger projects.
Suggested solution
Provide a way to remove the type annotation on Schema
z.ZodType<Prisma.UserCreateInput>

Alternatives

Additional context

Hey @hanayashiki
Glad that you find this project useful.
As I remember, the type annotation was added to fix a recursive type issue here.
If I understand you correctly, you currently use the schemas to build a database, with some sort of converter, right?
Could you describe or show an example of how you intend to use this project?

Hey @hanayashiki Glad that you find this project useful.
As I remember, the type annotation was added to fix a recursive type issue here.
If I understand you correctly, you currently use the schemas to build a database, with some sort of converter, right? Could you describe or show an example of how you intend to use this project?

I now think you have good reason to do this.
For my project, I have a trpc api layer and a prisma orm, and sometimes I'd like to use the definition directly from the prisma schema to define my apis.
Like you have a CreateUser api input object and a User table, you don't want to redefine some of the fields shared by them.
Currently I found this library is not what I expected it would be, but I'm using zod-prisma which works for me so far.

Hey @hanayashiki
Would just providing an option solve the issue for you?
You can confirm this by running the project locally then copying the generated schemas to your project folder. There you should run your project and maybe even try building it for production. If it builds and runs okay, then we can say that this would work fine for you.
Let me know if you still need the flag

Just revisiting this -- and wondering if there might be a workaround for this use case:
We have a number of auto-populated columns (createdAt, updatedAt, etc...). There's no danger of a user accidentally setting them -- database triggers keep them updated -- but it would be nice for us to be able to hide them in our tRPC input shapes.
For example, given this auto-generated definition from prisma-zod-generator:
const Schema: z.ZodType<Prisma.PracticeContentCreateWithoutSectionsInput> = z
  .object({
    id: z.string().optional(),
    title: z.string(),
    createdForSectionId: z.string(),
    createdAt: z.coerce.date().optional(),
    updatedAt: z.coerce.date().optional()
  })

If we try and restrict that by using zod's omit we get a TS error, even though .omit infact exists on z.object()'s output:

Removing the z.ZodType solves things. (Using ZodObject<>, ZodArray<>, etc... might help, though seems to raise other TS errors in the definitions file when I try.)

my workaround is to cast the schema to z.AnyZodObject