Can't configure content type in generated client

Problem statement
The generated client only allows you to set params and there isn't a way to override the content type used if the endpoint accepts multiple content types. The runtime library itself just uses the first content type in the list of ConsumesMediaTypes https://github.com/go-openapi/runtime/blob/3d6572c5a80826d5e7f6362d4c681875207c3ef6/client/runtime.go#L333-L341.
Suggestion
Instead of hardcoding the ClientOperation https://github.com/go-swagger/go-swagger/blob/312366608bbf17dd219190b66ab63bdc8b4d0db0/examples/todo-list/client/todos/todos_client.go#L36-L48 we could have some kind of function that produces the DefaultAddOneOperation and in addition to AddOne have an AddOneWithOperation that takes in a ClientOperation as an argument.
That way, a user can create a default operation, modify it, and then pass it in to the client. What do you think?
Or, since it's generated code, the existing AddOne can remain unmodified, and we can just generate the additional helper functions.
Environment
swagger version: 0.18.0

but you have a way to control what goes in there, by the order of the spec. That way the server can tell you what they prefer. You can also use ;q= values in the consumes and it will use that to prioritize

You can also use ;q= values in the consumes and it will use that to prioritize
Is this something in the swagger spec or in the runtime lib? This might be what I'm looking for.

I figure my use case might be a little obscure, but I actually want to use the non-preferred content type in the client in order to test that accepting that particular input works on the server side

You can also just call the submit operation directly on the runtime. That way you can present your own client operation.
yeah you'd have to make that change in the swagger spec.

Yeah calling the Submit allows me to do anything, but then I'd have to build the entire ClientOperation by hand which would lose the benefits of swagger generating the request (namely the ID and Method) that the default generated functions have within them. Calling Submit might not be so bad if there were a DefaultAddOneOperation() or something that would create a filled-in operation that I could modify and pass to Submit though

@nlsun here is my 2 cents on how we could enhance the client code to achieve this goal:
type ClientOption func(*runtime.ClientOperation)

func (a *Client) WithConsumes(media []string) ClientOption {
    return func(c *runtime.ClientOperation) {
        c.ConsumesMediaTypes = append(media, c.ConsumesMediaTypes ...)
    }
}

func (a *Client) WithProduces(media []string) ClientOption {
    return func(c *runtime.ClientOperation) {
        c.ProducesMediaTypes= append(media, c.ProducesMediaTypes ...)
    }
}

func (a *Client) WithSchemes(schemes []string) ClientOption {
    return func(c *runtime.ClientOperation) {
        c.Schemes= append(schemes, c.Schemes...)
    }
}

// operation
func (a *Client) AddOne(params *AddOneParams, authInfo runtime.ClientAuthInfoWriter, opts ...ClientOption) (*AddOneCreated, error) {
&runtime.ClientOperation{
		ID:                 "addOne",
		Method:             "POST",
		PathPattern:        "/",
		ProducesMediaTypes: []string{"application/io.swagger.examples.todo-list.v1+json"},
		ConsumesMediaTypes: []string{"application/io.swagger.examples.todo-list.v1+json"},
		Schemes:            []string{"http", "https"},
		Params:             params,
		Reader:             &AddOneReader{formats: a.formats},
		AuthInfo:           authInfo,
		Context:            params.Context,
		Client:             params.HTTPClient,
	}
	for _, apply := range opts {
	    apply(
	}
        result, err := a.transport.Submit(op)
	if err != nil {
		return nil, err
	}
	return result.(*AddOneCreated), nil
}


Strongly tie events to instructions

I'd like to see all events instructions produces. And not by looking at function implementation, but by looking at some trait or some associated constant.
In this ticket I'll to do a research on this topic and imlement it if possible

After some attempts I think it will be an overkill in our current state. We don't need such events analysis for now.
Although it will bring more formal verification to our code, I think, we have some more important tasks for now.


Question: Can you perform an update without loading the entire document first

Does LiteDB support update of a document (including potentially adding fields) to a document without loading the entire document into memory first?
I had a look at the documentation and tried performing two updates on the same document (with different shapes) in the same collection but the behaviour seems to be "last one wins".
Thanks

@isaacabraham  I'm facing the same issue right now, and at the current version can't understand how to update some fields in the document through that expressions. LiteDatabase neither LiteCollection don't have Execute method anymore and docs says nothing about that :C


TypeError: Cannot read property '1' of undefined

while give command ng build --prod getting below error
\node_modules\clean-css\lib\optimizer\restore-from-optimizing.js:54
property.value[property.value.length - 1][1] += IMPORTANT_TOKEN;
^
TypeError: Cannot read property '1' of undefined

I have sloved this error by spotting the bad char in code. By intalling plung-in in Vs called Highlight Bad Chars.


Updated config.ts capabilities

@googlebot I signed it!

THis PR has become stale. And as protractor is no longer maintained closing this PR!


Show QR code for hotspot mode

When in hotspot mode, it would be useful to have the QR code to join the SSID displayed on the OLED display (maybe with a center button click?)
cc: @adriangranados

This already exists (think I had a bad button mapping on my v1.0 unit), closing.


[NOREF] Support for removing multiple solutions

NOREF
Changes and Description

Modify removePlanDocumentSolutionLinks to take the same parameters as createPlanDocumentSolutionLinks (a single solution ID, multiple document IDs)

Follows up on https://github.com/CMSgov/mint-app/pull/296
How to test this change

Ensure go tests pass with scripts/dev up:backend and scripts/dev test:go

Ensure functionality works as expected

Create a model plan
Create a custom operational solution
Link / unlink multiple documents at once

PR Author Review Checklist

[ ] Met the ticket's acceptance criteria, or will meet them in a subsequent PR.
[ ] Added or updated tests for backend resolvers or other functions as needed.
[ ] Added or updated client tests for new components, parent components, functions, or e2e tests as necessary.
[ ] Updated the Postman Collection if necessary.

PR Reviewer Guidelines

It's best to pull the branch locally and test it, rather than just looking at the code online!
Check that all code is adequately covered by tests - if it isn't feel free to suggest the addition of tests.
Always make comments, even if it's minor, or if you don't understand why something was done.

Thanks Clay,
This looks good. I pulled locally and used the endpoint, and it removed the links as anticipated.
One interesting behavior to note with the join condition, it only needs one combination of document_id and solution_id to work. Example, I sent the request with two doc_ids, one was linked to the solution, one was not.
It successfully unlinked the document that was linked to the solution, and it doesn't do anything to the other combination (because there is no record). As long as one combination results in a link being removed, you will get a success message. I think this is likely as it should be expected.

Hmm - I think that makes sense, given that DELETE ... WHERE ... will succeed even if the WHERE returns nothing - though, I wonder if it'll error if one of the deletes actually fails 🤔

Thanks Clay,
This looks good. I pulled locally and used the endpoint, and it removed the links as anticipated.
One interesting behavior to note with the join condition, it only needs one combination of document_id and solution_id to work. Example, I sent the request with two doc_ids, one was linked to the solution, one was not.
It successfully unlinked the document that was linked to the solution, and it doesn't do anything to the other combination (because there is no record). As long as one combination results in a link being removed, you will get a success message. I think this is likely as it should be expected.

Hmm - I think that makes sense, given that DELETE ... WHERE ... will succeed even if the WHERE returns nothing - though, I wonder if it'll error if one of the deletes actually fails 🤔

Good question, I don't know how that would happen 🤔 . If there isn't anything to check, you do get a no row in result set error (which I guess is different). I can't think of a situation where a valid matched entry would be deleted, but another valid match wouldn't be (unless we had some dependent foreign key relationships in the DB, which I'm pretty sure we don't have on this table)


[PROTOCOL] Clarify checkpoint names

Which Delta project/connector is this regarding?

[ ] Spark
[ ] Standalone
[ ] Flink
[ ] Kernel
[x] Other (Protocol)

Description

Resolves #2842
How was this patch tested?

Does this PR introduce any user-facing changes?

No

@dennyglee could this be assigned a reviewer? :pray:

@ryan-johnson-databricks can you confirm the correctness of this?

Yes this is correct. The 20 character zero padding is needed to make sure lexiographical listing puts them in the right place. It is mentioned at couple of places in the spec already.
Thanks for fixing the rest.


M1 Mac woes using libpostal

Hi!
I tried installing libpostal on M1 Mac through these two methods listed below:

https://github.com/openvenues/libpostal#installation-maclinux
https://ports.macports.org/port/libpostal

I cannot use the parser api  using the python bindings when library is installed by (1)  or (2).
For example, after I installed the postal package in my  python 3.8.6 virtual environment, i.e.,pip install postal, I am receiving this consistent error when doing a simple import :
venv) libpostal % python
Python 3.8.6 (v3.8.6:db455296be, Sep 23 2020, 13:31:39)
[Clang 6.0 (clang-600.0.57)] on darwin
Type "help", "copyright", "credits" or "license" for more information.

from postal.parser import parse_address
Traceback (most recent call last):
File "", line 1, in
File "/Users/bdiller/gitlab/project/venv/lib/python3.8/site-packages/postal/parser.py", line 2, in
from postal import _parser
ImportError: dlopen(/Users/bdiller/gitlab/project/venv/lib/python3.8/site-packages/postal/_parser.cpython-38-darwin.so, 0x0002): symbol not found in flat namespace '_libpostal_address_parser_response_destroy'

What is wrong?

What you can do:

start the terminal in Rosetta mode. If you type in arch it should say i386
add the --disable-sse2 flag ( because https://github.com/openvenues/libpostal/issues/551 )

If it still does not work try to use Python 3.9 or 3.10. I was able to get it working by switching from 3.8 to 3.9 or 3.10. Then the nasty error message "symbol not found in flat namespace '_libpostal_address_parser_response_destroy'" went away.

What you can do:

start the terminal in Rosetta mode. If you type in arch it should say i386
add the --disable-sse2 flag ( because make not working on M1 Mac #551 )

If it still does not work try to use Python 3.9 or 3.10. I was able to get it working by switching from 3.8 to 3.9 or 3.10. Then the nasty error message "symbol not found in flat namespace '_libpostal_address_parser_response_destroy'" went away.

I have tried the mentioned steps but it still dint work. My arch is 'AMD64'. I have also tried swithcing to 3.9. It still doesnt work. Any help on this would be appreciated

I am facing the same issue with M2 Mac I tried using all the methods but it does not work out for me if @albarrentine or @JochenFromm or anyone who could help to resolve this issue is really appreciated.

% gcc -v
Apple clang version 14.0.3 (clang-14<IP_ADDRESS>.1)
Target: arm64-apple-darwin22.5.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin
Here is exactly what I did and make check succeeds:
brew install curl autoconf automake libtool pkg-config
mkdir test
cd test
git clone https://github.com/openvenues/libpostal
cd libpostal
./bootstrap.sh
./configure --datadir=/tmp/libpostal --prefix=$PWD
make check

I tried --disable-sse2 like configure suggests but it looks like someone auto-detected the ARM architecture to disable that automatically and if you actually specify --disable-sse2 it breaks things.  I have an M1 but I doubt that matters here.

This works for me. Thanks ❤
Could you also help me in that how I can use it for python.
>>> from postal import _parser
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ImportError: dlopen(/Users/varunpawar/.pyenv/versions/3.8.13/lib/python3.8/site-packages/postal/_parser.cpython-38-darwin.so, 0x0002): tried: '/Users/varunpawar/.pyenv/versions/3.8.13/lib/python3.8/site-packages/postal/_parser.cpython-38-darwin.so' (mach-o file, but is an incompatible architecture (have 'x86_64', need 'arm64')), '/System/Volumes/Preboot/Cryptexes/OS/Users/varunpawar/.pyenv/versions/3.8.13/lib/python3.8/site-packages/postal/_parser.cpython-38-darwin.so' (no such file), '/Users/varunpawar/.pyenv/versions/3.8.13/lib/python3.8/site-packages/postal/_parser.cpython-38-darwin.so' (mach-o file, but is an incompatible architecture (have 'x86_64', need 'arm64'))

Run file /Users/varunpawar/.pyenv/versions/3.8.13/lib/python3.8/site-packages/postal/_parser.cpython-38-darwin.so
Then the same on the python executable that you are running.  It looks like the postal binaries that you installed with pip are Intel x86_64 and your python is arm64.  Make sure your pip is also arm64 as that would be required for you to get the right Python packages... though I really have no experience with the Python bindings, so can't help.  You might want to file an issue on that project (I think it is separate).

I have  the same issue. Will not run in python 3.7 or 3.8, works with 3.9.
On a Mac M2 Max.
 % gcc -v Apple clang version 14.0.3 (clang-14<IP_ADDRESS>.1) Target: x86_64-apple-darwin22.5.0 Thread model: posix InstalledDir: /Library/Developer/CommandLineTools/usr/bin

Main reason is pypostal uses a particular includes and bin directory and for Mac some times that would not be that we have in pypostal.
So to resolve the issue I install python 3.10 and tweaked my path for libpostal includes and bin.

i am using pyspark and getting below error on mac M1. i tired to use python 3.9 version too . It is working with python but not with pyspark :
ImportError: dlopen(/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/postal/_parser.cpython-310-darwin.so, 0x0002): symbol not found in flat namespace '_libpostal_address_parser_response_destroy'
@varunpawar652000 can you share on how you set the path ?

@nikil-katturi can you please share your python version?

python version 3.10

Is pypostal is working as you want?

yes it is working without pyspark

You can avoid Rosetta and build libpostal natively for Apple Silicon macs (M1 etc.), too. Here’s how I did it:
export CC=clang
export CXX=clang++
./configure --datadir=<your_dir_of_choice> --disable-sse2
make
sudo make install
make check

Checking the result:
> file /usr/local/lib/libpostal.dylib
/usr/local/lib/libpostal.dylib: Mach-O 64-bit dynamically linked shared library arm64


本地环境搭建,迁移db报错提示iam migrate fail

case: 本地环境搭建(想做二次开发)
问题: 迁移失败
版本: <IP_ADDRESS>5
环境: centos7 python3.6.12(pyenv)
(bk-itsm-<IP_ADDRESS>5) [root@c7 pycharm_project_173]# python manage.py migrate
You are using Python 3.6. This version does not support timestamps with nanosecond precision and the OpenTelemetry SDK will use millisecond precision instead. Please refer to PEP 564 for more information. Please upgrade to Python 3.7 or newer to use nanosecond precision.
This is the first deployment
发现非2.6.0 之前的升级版本，已跳过修复
init redis settings
autodiscover rpc: rpc.components
autodiscover trigger: action.components
init api public key
init api public success
System check identified some issues:

WARNINGS:
?: (mysql.W002) MySQL Strict Mode is not set for database connection 'default'
	HINT: MySQL's Strict Mode fixes many data integrity problems in MySQL, such as data truncation upon insertion, by escalating warnings into errors. It is strongly recommended you activate it. See: https://docs.djangoproject.com/en/3.2/ref/databases/#mysql-sql-mode
ticket.SignTask.is_passed: (fields.W903) NullBooleanField is deprecated. Support for it (except in historical migrations) will be removed in Django 4.0.
	HINT: Use BooleanField(null=True) instead.
ticket.TicketRemark.ticket_id: (fields.W122) 'max_length' is ignored when used with IntegerField.
	HINT: Remove 'max_length' from field
ticket.TicketRemark.update_log: (fields.E010) JSONField default should be a callable instead of an instance so that it's not shared between all field instances.
	HINT: Use a callable instead, e.g., use `dict` instead of `{}`.
ticket.TicketRemark.users: (fields.E010) JSONField default should be a callable instead of an instance so that it's not shared between all field instances.
	HINT: Use a callable instead, e.g., use `dict` instead of `{}`.
workflow.Workflow.is_task_needed: (fields.W903) NullBooleanField is deprecated. Support for it (except in historical migrations) will be removed in Django 4.0.
	HINT: Use BooleanField(null=True) instead.
workflow.WorkflowVersion.is_task_needed: (fields.W903) NullBooleanField is deprecated. Support for it (except in historical migrations) will be removed in Django 4.0.
	HINT: Use BooleanField(null=True) instead.
Operations to perform:
  Apply all migrations: account, admin, apigw, auth, auth_iam, component_framework, contenttypes, data_migration, django_celery_beat, django_celery_results, django_nyt, django_signal_valve, engine, iadmin, iam_migration, log, pipeline, postman, project, role, service, sessions, sites, sla, sla_engine, task, thumbnail, ticket, ticket_status, trigger, variable_framework, weixin_core, workflow
Running migrations:
  Applying contenttypes.0001_initial... OK
  Applying contenttypes.0002_remove_content_type_name... OK
  Applying auth.0001_initial... OK
  Applying auth.0002_alter_permission_name_max_length... OK
  Applying auth.0003_alter_user_email_max_length... OK
  Applying auth.0004_alter_user_username_opts... OK
  Applying auth.0005_alter_user_last_login_null... OK
  Applying auth.0006_require_contenttypes_0002... OK
  Applying auth.0007_alter_validators_add_error_messages... OK
  Applying auth.0008_alter_user_username_max_length... OK
  Applying account.0001_initial... OK
  Applying account.0002_init_superuser... OK
  Applying account.0003_verifyinfo... OK
  Applying account.0004_create_cache_table... OK
  Applying admin.0001_initial... OK
  Applying admin.0002_logentry_remove_auto_add... OK
  Applying admin.0003_logentry_add_action_flag_choices... OK
  Applying apigw.0001_initial... OK
  Applying auth.0009_alter_user_last_name_max_length... OK
  Applying auth.0010_alter_group_name_max_length... OK
  Applying auth.0011_update_proxy_permissions... OK
  Applying auth.0012_alter_user_first_name_max_length... OK
  Applying iam_migration.0001_initial...do migrate
_call_iam_api fail. method: http_get path: /api/v1/model/systems/itsm/query error: not found:system(itsm) not exists
do api_query fail not found:system(itsm) not exists
_call_iam_api fail. method: http_post path: /api/v1/model/systems error: bad request:Host is required
execute operation [upsert_system] bad request:Host is required fail, error message: id=itsm
Traceback (most recent call last):
  File "manage.py", line 43, in <module>
    execute_from_command_line(sys.argv)
  File "/root/.virtualenvs/bk-itsm-<IP_ADDRESS>5/lib/python3.6/site-packages/django/core/management/__init__.py", line 419, in execute_from_command_line
    utility.execute()
  File "/root/.virtualenvs/bk-itsm-<IP_ADDRESS>5/lib/python3.6/site-packages/django/core/management/__init__.py", line 413, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/root/.virtualenvs/bk-itsm-<IP_ADDRESS>5/lib/python3.6/site-packages/django/core/management/base.py", line 354, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/root/.virtualenvs/bk-itsm-<IP_ADDRESS>5/lib/python3.6/site-packages/django/core/management/base.py", line 398, in execute
    output = self.handle(*args, **options)
  File "/root/.virtualenvs/bk-itsm-<IP_ADDRESS>5/lib/python3.6/site-packages/django/core/management/base.py", line 89, in wrapped
    res = handle_func(*args, **kwargs)
  File "/root/.virtualenvs/bk-itsm-<IP_ADDRESS>5/lib/python3.6/site-packages/django/core/management/commands/migrate.py", line 246, in handle
    fake_initial=fake_initial,
  File "/root/.virtualenvs/bk-itsm-<IP_ADDRESS>5/lib/python3.6/site-packages/django/db/migrations/executor.py", line 117, in migrate
    state = self._migrate_all_forwards(state, plan, full_plan, fake=fake, fake_initial=fake_initial)
  File "/root/.virtualenvs/bk-itsm-<IP_ADDRESS>5/lib/python3.6/site-packages/django/db/migrations/executor.py", line 147, in _migrate_all_forwards
    state = self.apply_migration(state, migration, fake=fake, fake_initial=fake_initial)
  File "/root/.virtualenvs/bk-itsm-<IP_ADDRESS>5/lib/python3.6/site-packages/django/db/migrations/executor.py", line 227, in apply_migration
    state = migration.apply(state, schema_editor)
  File "/root/.virtualenvs/bk-itsm-<IP_ADDRESS>5/lib/python3.6/site-packages/django/db/migrations/migration.py", line 123, in apply
    operation.database_forwards(self.app_label, schema_editor, old_state, project_state)
  File "/root/.virtualenvs/bk-itsm-<IP_ADDRESS>5/lib/python3.6/site-packages/django/db/migrations/operations/special.py", line 190, in database_forwards
    self.code(from_state.apps, schema_editor)
  File "/tmp/pycharm_project_173/iam/contrib/iam_migration/migrations/0001_initial.py", line 16, in forward_func
    migrator.migrate()
  File "/tmp/pycharm_project_173/iam/contrib/iam_migration/migrator.py", line 65, in migrate
    raise exceptions.MigrationFailError("iam migrate fail")
iam.contrib.iam_migration.exceptions.MigrationFailError: iam migrate fail

如果我把 USE_IAM = False 后 ,可以成功迁移. 但是本地访问报错:

好的，感谢


NonJoinedStream[ALL] bypasses max_block_size

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

A test?

The test can be easily implemented with blockSize function.


pfVerticalNavigation: on mobile, move the masthead utility items to the vertical navigation

On mobile (less than 768 px), the masthead utility items (Help, About, Logout, etc.) are hidden but the pfVerticalNavigation component doesn't support dynamically showing them in the vertical navigation. I suggest adding a new property to the items objects to flag what items should be hidden by default and shown on mobiles.
More details in the PatternFly Vertical Navigation design docs.

Hi @alexkieling,
Is this what you had in mind?  I added a mobileOnly flag to the items.
Note that 'Help' and 'User' vert. nav items only appear when in mobile (< 768px) mode, and are hidden when not in mobile mode:

Yes, that's exactly what I had in mind. It seems to match the PatternFly design perfectly.
@tadayosi, what do you think?

@alexkieling Looks good to me. Thanks.

Fixed via #747, not sure why it didn't auto-close


Update decimal places according to ISO4217 standard

A project I am working on uses JS-Money on the front and PHP-Money in the back. An unbelievable number popped up once when we let them work together on the HUF currency.
PHP-Money uses 2 decimal digits for HUF, where JS-Money uses 0.
A Hungarian colleague told me 0 decimal digits would be most logical, due to the smallest unit Fillér not being made anymore.
Although 0 is the a logical choice, following the currency standard is a better choice, which uses 2 decimal digits for HUF currency. An awesome side effect is that it allows PHP-Money and JS-Money to be used in unison!
Changes
This PR updates the decimal digits with the most recent version; ISO 4217:2015. If this PR is accepted, all used currencies are up-to-date ✨.

[X] AFN: 0 -> 2
[X] ALL: 0 -> 2
[X] AMD: 0 -> 2
[X] COP: 0 -> 2
[X] CRC: 0 -> 2
[X] HUF: 0 -> 2
[X] IDR: 0 -> 2
[X] IQD: 0 -> 3
[X] IRR: 0 -> 2
[X] LAK: 0 -> 2
[X] LBP: 0 -> 2
[X] MGA: 0 -> 2
[X] MMK: 0 -> 2
[X] MUR: 0 -> 2
[X] PKR: 0 -> 2
[X] SOS: 0 -> 2
[X] SYP: 0 -> 2
[X] TZS: 0 -> 2
[X] UZS: 0 -> 2
[X] YER: 0 -> 2

Source
ISO 4217 is the International Standard for currency codes.

Coverage remained the same at 95.588% when pulling df569896d58cff249c962909f66a8e98b7f0da37 on PimvanGurp:synchronise-decimal-places-with-ISO4217 into 551199499bdbba3829e91655a673b7c49fddada9 on davidkalosi:master.

Thank you for the contribution. will do an npm release later this day

@davidkalosi did you manage to get the release out the door? We're really waiting for this fix 😄

Hi,  version 0.6.1 is available on npmjs


Change colour scheme of UI

Fix issue #156

Codecov Report

Merging #161 (5864802) into master (492384c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #161   +/-   ##
=========================================
  Coverage     74.28%   74.28%
  Complexity      691      691
=========================================
  Files            95       95
  Lines          2154     2154
  Branches        256      256
=========================================
  Hits           1600     1600
  Misses          425      425
  Partials        129      129

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 492384c...5864802. Read the comment docs.

Hi @EssWhyy, thanks for updating the UI theme! 👍  May I know if there's any update pertaining to the rounded corners for command box & result display that we conversed about in the Telegram group chat earlier on?


Yu-Ris character names showing up in textbox

VN: https://vndb.org/v31434
Using VnTextPatch to extract and insert the scripts, the names of characters show up in the textbox instead of above it. The scripts extracted to the excel file are missing the Japanese 「」quotations which I think are required for the game engine to recognize a line as a spoken dialogue. Also, it seems when inserted into the .ybn files, all 「」quotations are replaced with ‘’.
I've had success with editing the script and having the names show up properly when using OzmafiaScriptThing.7z (source code included).
I've included example scripts inserted by VnTextPatch and OzmafiaScriptThing. Note, the file "yst00071.ybn" contains the name lists of characters.
Yu-Ris.zip

Read this and use version v0.0.15. For now, newer versions than 0.0.15 will result in the names show up in textbox

These ‘...’ quotes were for a translation project I was involved in myself (where we customized the game to use those quotes instead of 「」). Shouldn't have been committed. They're reverted to 「」 now.


Breaking intermittently with VS Code v1.27.2

My settings cycler setup worked for a while, but then fails to work after a while and after lots of tweaking I can't get it to work again this time.
The error displayed in a popup is "command 'settings.cycle.miniView' not found".
The "settings.cycle.logLevel": "log" option doesn't seem to produce any additional output.
keybindings.json:
[
    {
        "key": "ctrl+;",
        "command": "settings.cycle.miniView",
    }
]

settings.json User Settings:
{
  "settings.cycle":
  [
    {
      "id": "miniView",
      "overrideWorkspaceSettings": false,
      "values":
      [
        { // full dev mode
          "editor.minimap.enabled": true,
          "editor.folding": true,
          "editor.glyphMargin": true,
          "editor.lineNumbers": "on",
          "scm.diffDecorations": "all",
          "scm.diffDecorationsGutterWidth": 3,
          "workbench.activityBar.visible": true,
          "window.menuBarVisibility": "default",
          "workbench.statusBar.visible": true,
          "workbench.editor.showTabs": true,
        },
        { // compact mode
          "editor.minimap.enabled": false,
          "editor.folding": false,
          "editor.glyphMargin": false,
          "editor.lineNumbers": "off",
          "scm.diffDecorations": "none",
          "scm.diffDecorationsGutterWidth": 1,
          "workbench.activityBar.visible": false,
          "window.menuBarVisibility": "toggle",
          "workbench.statusBar.visible": false,
          "workbench.editor.showTabs": false,
        }
      ]
    }
  ],
}

Just received a notification about the 1.28.1 update moments after posting this and am pleased to report this issue seems to have disappeared again. If anyone can see an obvious issue with my settings configuration please let me know anyway!


comment from user regarding VDCs

"2 VDCs in Syangja, Birgha Archale and Biruwa Archale have been swiped with each other."
"The names are swiped with each other. As per the map, the geographic area of Birgha Archale has been named Biruwa Archale and vice versa."
Source: https://www.facebook.com/groups/e4nepal/10157956666190430/?notif_t=like&notif_id=1482679103818905

@ravinepal
Thanks! I will take a look.

easiest is just to go to www.openstreetmap.org , log in (you can use github for that), zoom to the right spot and click on "Edit" in the left corner
You can edit the map directly in your browser

@individual-it
Thanks! That's good to know. I can make the change for NepalMap's data when and if @ravinepal or other members of the team feel that Facebook feedback is accurate and should be followed.
However, I don't know that part of Nepal well enough to be qualified to make the change in the underlying OpenStreetMap. I feel that that change should be done by someone with more knowledge of Syangja than I have.
Are you or someone you know familiar enough with Syangja to determine whether or not a change is valid and should be made and make the change?

@prashishh - would you mind posting this qs re Syangja's two VDCs and see if someone can corroborate this claim a user made? pls see this thread for details. cc @studenton


Builder enhanced to support integration with OpenAPI / SmallRye

See the OCI's Example as background.
What is happening here behind the scenes is that there is an OpenAPI spec yaml where GetBucketResponse and GetBucketRequest schemas trigger generation of pojo's in a builder style. And then the generated pojo instances are built (builder().namespaceName("myNamespace").bucketName("myBucket").build()) and then passed to the service interface.
This functionality would be a nice enhancement to integrate into our Helidon Builder, where we would delegate directly to BuilderCreator to generate the backing pojo types for this use case.
Details to be worked out through a POC if we think this is worthwhile to pursue.

The OpenAPITools generators in general--and the ones our team provided to support Helidon in particular--generate POJO classes for each of the schemas declared in the OpenAPI document. These POJOs include getters and setters for the attributes as illustrated in this generated POJO from our example generated project https://github.com/helidon-io/helidon/blob/helidon-3.x/examples/openapi-tools/quickstart-mp/mp-server/src/main/java/org/openapitools/server/model/Message.java.
One possibility could be to enhance the Helidon OpenAPITools generators with an optional flag to detect a new, optional flag (supported only if the Helidon version is 4.x?) and then:

Add the @Builder annotation to the generated POJO classes.
Generate the project pom.xml (if selected) to include the builder annotation processor.

We should consider how valuable we think users would find such a feature vs. simply using the already-generated setters and getters on the POJO classes. We also need to weigh our own priorities for various possible enhancements to our generators.

Several factors:

The generated POJO classes are not part of Helidon but become part of the user's project. With that in mind, the generated code encourages best practices as appropriate but should not necessarily impose any of our own stylistic preferences.
The Helidon generators support both 2.x (requires Java 11+) and 3.x (requires Java 17+). Records do not appear in Java until Java 14. Generating POJO classes when the user targets Helidon 2.x but records when the user targets 3.x or 4.x would add complexity to the generator and templates.
Changing how the generators create POJOs would be backward-incompatible and would need to await a major release of the generator.
Some users might not want to use builders with the generated POJOs (whether classes or records). They might not use Helidon-based builders at all in their apps, particularly for MP apps, so there would be no "standard" experience to preserve for such users.

Any or all of what you describe could be done, technically, but, again, we have to prioritize carefully among the possible enhancements to the generators (and our other work).


Hash whole audio data

Currently the hash used is completely useless - it hashes the entire file minus its first 128 KiB and last 128 bytes. If an ID3v2 tag is changed in the file, the hash changes as well, because of the length change.
We should instead use LibAV to read the raw audio data and hash it all - this will hopefully prevent changes to the ID3 tags from changing the file's hash.

Fixed in #56


Referenceless QE metrics require reference input

🐛 Bug
When using comet from the evaluate package, it seems to require the reference also for QE metrics. There's an easy but dirty hotfix.
To Reproduce
comet_metric = evaluate.load("comet", config_name='wmt21-comet-qe-da')
# erorr!
comet_metric.compute(predictions=["Hallöchen"], sources=["Heyo"])

More info
Passing a reference with an empty string appears to satisfy the library. I checked that the output scores are not dependent on the reference:
# {'mean_score': 0.08140071481466293, 'scores': [0.08140071481466293]}
comet_metric.compute(predictions=["Hallöchen"], sources=["Heyo"]. references=[""])
# {'mean_score': 0.08140071481466293, 'scores': [0.08140071481466293]}
comet_metric.compute(predictions=["Hallöchen"], sources=["Heyo"], references=["Howdy"])

Hi @zouharvi, This seems to be an issue on the evaluate package and not on unbabel-comet. I suggest you open and issue there...


add arm64 suffix to etcd image when CPU arch is arm64.

Fixes: cilium/cilium#17779
Signed-off-by: Derek Chen<EMAIL_ADDRESS>

Thanks @dereknex!

Shouldn't a "fat" manifest exist for the etcd image? This seems like it's a little backwards..

Shouldn't a "fat" manifest exist for the etcd image? This seems like it's a little backwards..

The etcd:v3.5.1 will solve this issue, but we need upgrade etcd to v3.5.


Added alternative to Sentry

TrackJS is an alternative that is more focused on front-end concerns.

hey @toddhgardner - TrackJS sounds neat! Sentry is pretty ubiquitous so we're going to keep this as-is but as best of luck on the project: if you make a Workers integration, let us know!

@signalnerve. Thank you, but this isn't a new project. We've been around just as long as Sentry, but we're more niche. We have customers using us on your workers today, who asked why we weren't referenced here.


zh-translation: docs/root/configuration/listeners/udp_filters/udp_filters.rst

ref: https://github.com/cloudnativeto/envoy/issues/343

[x] Docs

/review

/LGTM

/LGTM


PYTHON CODE TO IMPLEMENT MERGE SORT ON A LIST

Please provide a little description about the implementation, thanks.


(BOLT-633) Set modulepath, enable puppetdb functions during apply

Sets modulepath to Bolt's modulepath. Enables Bolt's puppetdb functions
when compiling a manifest block. Other functions continue to be
disabled.

CLA signed by all contributors.

Added tests.


Why cant we extract full playlist or all songs of an artist??

Yes, I did try to obtain individual lyrics and search playlists, but those tasks can be completed manually with just one click.

@RazyRo you can get all artist's songs from - http://localhost:3000/docs#tag/artists/get/api/artists/{id}/songs, playlist features is WIP


drone-cli: add package

Motivation for this change
I need to control a drone-server via the CLI.
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S unning nix path-info -S` before and after)
[X] Fits CONTRIBUTING.md.

Oups.I thought I already had added myself once in this repo, weird.
I added myself to maintainers/maintainer-list.nix as requested by @xeji .

@GrahamcOfBorg  build drone-cli

Success on x86_64-linux (full log)
Attempted: drone-cli
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/k9sirkg8mg6d80gl4d861k0fb6sa0fhd-drone-cli-0.8.6-bin/bin
patching script interpreter paths in /nix/store/k9sirkg8mg6d80gl4d861k0fb6sa0fhd-drone-cli-0.8.6-bin
checking for references to /tmp/nix-build-drone-cli-0.8.6.drv-0 in /nix/store/k9sirkg8mg6d80gl4d861k0fb6sa0fhd-drone-cli-0.8.6-bin...
shrinking RPATHs of ELF executables and libraries in /nix/store/b14944k3rfhjnbpjnr81dgv6k53zpjsi-drone-cli-0.8.6
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/b14944k3rfhjnbpjnr81dgv6k53zpjsi-drone-cli-0.8.6
/nix/store/b14944k3rfhjnbpjnr81dgv6k53zpjsi-drone-cli-0.8.6/share/go/src/github.com/drone/drone-cli/.drone.sh: interpreter directive changed from "/bin/sh" to "/nix/store/8zkg9ac4s4alzyf4a8kfrig1j73z66dw-bash-4.4-p23/bin/sh"
/nix/store/b14944k3rfhjnbpjnr81dgv6k53zpjsi-drone-cli-0.8.6/share/go/src/github.com/drone/drone-cli/vendor/github.com/docker/docker/hack/generate-authors.sh: interpreter directive changed from "/bin/bash" to "/nix/store/8zkg9ac4s4alzyf4a8kfrig1j73z66dw-bash-4.4-p23/bin/bash"
checking for references to /tmp/nix-build-drone-cli-0.8.6.drv-0 in /nix/store/b14944k3rfhjnbpjnr81dgv6k53zpjsi-drone-cli-0.8.6...
/nix/store/k9sirkg8mg6d80gl4d861k0fb6sa0fhd-drone-cli-0.8.6-bin

Success on aarch64-linux (full log)
Attempted: drone-cli
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/1zl2libr26glaawqmp3h2fhqd0nlsh2v-drone-cli-0.8.6-bin/bin
patching script interpreter paths in /nix/store/1zl2libr26glaawqmp3h2fhqd0nlsh2v-drone-cli-0.8.6-bin
checking for references to /build in /nix/store/1zl2libr26glaawqmp3h2fhqd0nlsh2v-drone-cli-0.8.6-bin...
shrinking RPATHs of ELF executables and libraries in /nix/store/p62l98p9hjl5k0nxdxdp88xjxki7v6yx-drone-cli-0.8.6
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/p62l98p9hjl5k0nxdxdp88xjxki7v6yx-drone-cli-0.8.6
/nix/store/p62l98p9hjl5k0nxdxdp88xjxki7v6yx-drone-cli-0.8.6/share/go/src/github.com/drone/drone-cli/.drone.sh: interpreter directive changed from "/bin/sh" to "/nix/store/p0vy17dp9jk2mvqsxsqnb14s3797lay7-bash-4.4-p23/bin/sh"
/nix/store/p62l98p9hjl5k0nxdxdp88xjxki7v6yx-drone-cli-0.8.6/share/go/src/github.com/drone/drone-cli/vendor/github.com/docker/docker/hack/generate-authors.sh: interpreter directive changed from "/bin/bash" to "/nix/store/p0vy17dp9jk2mvqsxsqnb14s3797lay7-bash-4.4-p23/bin/bash"
checking for references to /build in /nix/store/p62l98p9hjl5k0nxdxdp88xjxki7v6yx-drone-cli-0.8.6...
/nix/store/1zl2libr26glaawqmp3h2fhqd0nlsh2v-drone-cli-0.8.6-bin

Success on x86_64-darwin (full log)
Attempted: drone-cli
Partial log (click to expand)
/private/tmp/nix-build-drone-cli-0.8.6.drv-0
post-installation fixup
strip is /nix/store/7ddbq63v97nk8gkbf7gcsfmby37h6gbl-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/4hcfzyhnsirawq9fm6rlmcvj49w2153a-drone-cli-0.8.6-bin/bin
patching script interpreter paths in /nix/store/4hcfzyhnsirawq9fm6rlmcvj49w2153a-drone-cli-0.8.6-bin
strip is /nix/store/7ddbq63v97nk8gkbf7gcsfmby37h6gbl-cctools-binutils-darwin/bin/strip
patching script interpreter paths in /nix/store/0sbz9vc7w2jsvfrlb48416gm0kv7k446-drone-cli-0.8.6
/nix/store/0sbz9vc7w2jsvfrlb48416gm0kv7k446-drone-cli-0.8.6/share/go/src/github.com/drone/drone-cli/.drone.sh: interpreter directive changed from "/bin/sh" to "/nix/store/q2wqq1k20v8kc3vckapqf5nws30brnni-bash-4.4-p23/bin/sh"
/nix/store/0sbz9vc7w2jsvfrlb48416gm0kv7k446-drone-cli-0.8.6/share/go/src/github.com/drone/drone-cli/vendor/github.com/docker/docker/hack/generate-authors.sh: interpreter directive changed from "/bin/bash" to "/nix/store/q2wqq1k20v8kc3vckapqf5nws30brnni-bash-4.4-p23/bin/bash"
/nix/store/4hcfzyhnsirawq9fm6rlmcvj49w2153a-drone-cli-0.8.6-bin


Interactive

Menu driven program
It could start when ee -i is provided or could be made default behavior for ee or subcommands e.g. ee site

Won't be doing this anymore.


🚧 Swift 3

🚧 WIP: Not ready to merge 🚧
Started building the project to see if I can resolve #54, but the codebase is really out-of-date. So, figured I'd first convert the code to Swift 3. Then, if that goes well, migrate to Swift 4.  However, I don't see any recent activity on this project. If I can finish this up can someone even push out a release? Or, should I just build it for myself?

Any update on this? Is it still a WIP? @phatblat @tadeuzagallo

Yeah, I converted it to Swift 3 (which is now obsolete), but never got the embedded chrome plugin to work inside the macOS app. More recently, I updated the project to Swift 5, but I don't know what's going on inside the webview and haven't spent much time to debug it.
Anyway, since my initial PR #54 has been ignored for 2 years now, I figure this project is dead.


Update river.go

it looks that you have already support for multiple Primary key, but forget remove the code.

Thanks @WangXiangUSTC
Can you help me adding a multi primary key test?

BTW, CI failed

OK， how to do this?  I remove these code, and can sync mysql's table with  multi primary key  well


Add name overrides for Internal and External Load Balancers

When developing IaC for existing infrastructure and attempting to achieve parity between the two, the current naming assignments of multiple modules in CFF causes issues.
For example in net-lb-ext the name of the load balancer added additional items to the name if a forwarding_rules_config was passed.
name = (
  each.key == "" ? var.name : "${var.name}-${each.key}"
)

This can cause issues where a specific name is preferred, such as when bringing existing infrastructure under the control of IaC. Changing the name of a load balancer, backend service or health check would cause them to be replaced by Terraform.
This is also relevant to #2419 where peerings have a name set by combining the local and peering network names with a prefix. A similar override_name variable would resolve this issue.
Adding an optional override_name variable allows the user to specify a name if the generated name is not helpful for their use case. For users who would like to use the module as before and have names generated as before, leaving the override_name as null will use the default name assignment.

Checklist

I acknowledge that I have:

[x] Read the contributing guide
[x] Ran terraform fmt on all modified files
[x] Regenerated the relevant README.md files using tools/tfdoc.py
[ ] Made sure all relevant tests pass

It's probably easier if you run checks locally without waiting for the workflow to kick in. Details are in the contributing guide in the root of the repo. :)

I think that to be fully comprehensive, this requires allowing additional overrides in net-lb-ext for names of:
* google_compute_region_backend_service

* google_compute_service_attachment

If "fully comprehensive" is the goal, this could be expanded to adding override names across multiple other modules. This would also partially resolve #2419. We have encountered this across many areas of our deployment in taking existing click-ops configured infrastructure under control of Terraform but trying to use CFF to accelerate development. Overrides would be useful for many brownfield deployments.

This PR is getting messy. I extracted the changes to the peering module to #2459.

@cvanwijck-hub24, just checking in to see if you plan on finishing this PR. If not, we'll go ahead and close it

@cvanwijck-hub24, just checking in to see if you plan on finishing this PR. If not, we'll go ahead and close it

Yes I will resolve the outstanding issues this week if possible.


Add favicon to title

Fixes #196

@Panquesito7 @TechStudent11 Requested your review. Let me know if it looks good and will merge accordingly or if you have any other favicon which will look better. Thanks.

LGTM

Thanks, merging rn.

@all-contributors please add @KendallDoesCoding to the doc for their contribution


Very high memory usage with serde_json::Value

Unfortunately, due to Value being completely public I don't know how much can be done about this without breaking changes. However, a couple of times I've run into problems with exceptionally high memory usage when using a Value.
I don't think there's a bug here, just that common uses seem to be much more memory intensive than similar code in dynamic languages, where this kind of data is already heavily optimised.
I think it comes from several factors:

Each Value is 32 bytes on a 64-bit system even though the majority of Values will be the leaf nodes (numbers, strings, nulls, etc.) which don't need that much space. If Value were more highly optimized for leaf nodes, I think this could easily be halved.

Maps are optimized for access time rather than space efficiency, and this is made worse because there are lots of "empty" Value slots, each of which is another 32 bytes.

Strings are owned. When converting from a struct with to_value, object keys will all be known statically, and those strings will already be embedded in the program as static data, so using a Cow<'static, str> could dramatically reduce memory usage.

Strings are exclusively owned. When deserializing into a Value it's likely that there will be lots of duplicate strings, but there is no possibility for them to be shared with the current Value representation.

I think a more space-efficient Value type could be introduced. Keys could be stored as a pointer-sized union of &'static str, Arc<String> using a tag in the low bits to differentiate. The deserializer could automatically intern strings as they are deserialized. Value could be shrunk to 16 bytes, and store short strings inline. Maps could use a simple Vec representation for small numbers of elements to avoid any wasted space. The improved cache-coherency could also improve performance. All access to "compact values" should be done via methods to allow further optimisations in the future. There would also need to be a version of the json!() macro that produced this compact type.

Similar problem encountered.

Seems one of my comment has been deleted, if serde-rs doesn't want rust noobs like me to comment, please add one line in README.md so that rust noobs like me will be able to follow it.

@dtolnay I started working on a crate to address these issues:
https://github.com/Diggsey/ijson
https://docs.rs/ijson
It is functionally complete but needs a lot more testing, etc. to get to a point where I can recommend people actually use it. That said, it demonstrates that significant improvements are possible.
Is this something you'd be interested in bringing into serde-json some time down the line?

This came to me as a bit of an unpleasant surprise when my AWS Lambdas started running out of memory. I was sizing them based what is being retrieved from the DB. For example, ElasticSearch returns a 8,683KB document, I deser it into Value and the next RAM reading gives me delta of 98,484KB of RAM use. That's more than 10x the original size.
@dtolnay , David, is this high memory consumption a necessary price to pay for speed?
Is 561ms using from_slice() on an 8.6MB JSON string considered fast?

@rimutaka serde_json is much more efficient at deserializing into structs, compared to the Value type, so if that is possible for your usecase, then that's the best option.

@Diggsey , thanks for the suggestion. Do you know if Value is more compact if I deser into a struct and then convert it into Value?

It would only be more compact if some fields are dropped as part of the deserialization into a struct (if say they are not required).

Memory allocation log for processing 10MB of JSON data:

JSON as String of 10,812,199 bytes => +10,150KB allocated
JSON converted into struct => +63,996KB allocated
struct converted into Value => 188,162KB allocated

I can understand high memory consumption when JSON is converted into Value because the size of collections is not known, so more is allocated than needed to make it faster.
When a struct is converted into Value the size of collections is known in advance.
Why do we still get such large memory overhead? Is it inevitable or can be improved?

FWIW I think I encountered this on the current version of the google_sheets4 crate -- it uses serde_json::Value and my server goes OOM if I try to deserialize a large spreadsheet w/ multiple tabs with 20GB usage.


MemoryError in ubuntu, not mac

Hello, my script gets killed by MemoryError, running in an aws EC2 instance, with 7.5G memory.
However, it has no problem running in my Mac (8g memory) !!! they use same TF, but numpy version is slightly different.
Python 2.7.10 (default, Oct 23 2015, 19:19:21)
[GCC 4.2.1 Compatible Apple LLVM 7.0.0 (clang-7<IP_ADDRESS>)] on darwin
Type "help", "copyright", "credits" or "license" for more information.

import numpy as np
np.version
'1.11.1'

Python 2.7.6 (default, Jun 22 2015, 17:58:13)
[GCC 4.8.2] on linux2
Type "help", "copyright", "credits" or "license" for more information.

import numpy as np
np.version
'1.11.0'

I have absolutely no idea what happened... and my data is not likely to require that much memory either...  i did memory profiling in EC2, error occurs at line 290.
Line #    Mem usage    Increment   Line Contents
275  412.289 MiB    0.000 MiB       @profile
276                                 def test_naive(self, mx, yls, fn_model=None, W=None, b=None):
277                                     """load variables and calculate acc"""
278 5117.520 MiB 4705.230 MiB           mx_array = mx.toarray()
279 5117.520 MiB    0.000 MiB           if fn_model is not None:
280                                         reader = tf.train.NewCheckpointReader(fn_model)
281                                         logger.info("loaded saved param from " + fn_model)
282                                         W, b = reader.get_tensor("W"), reader.get_tensor("b")
283                                     else:
284 5117.520 MiB    0.000 MiB               assert (W is not None) and (b is not None)
285
286 5117.523 MiB    0.004 MiB           print 'read', type(W), type(b), mx_array.shape
287 5117.523 MiB    0.000 MiB           x, _ = self.init_placeholder()
288 5117.523 MiB    0.000 MiB           with tf.Session() as sess:
289 5117.523 MiB    0.000 MiB               tf.initialize_all_variables()
290 5130.176 MiB   12.652 MiB               y = sess.run(tf.nn.softmax(tf.matmul(x, W) + b), feed_dict={x: mx_array})
File "meshtags/model_bow.py", line 290, in test_naive
y = sess.run(tf.nn.softmax(tf.matmul(x, W) + b), feed_dict={x: mx_array})
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 372, in run
run_metadata_ptr)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 619, in _run
np_val = np.array(subfeed_val, dtype=subfeed_dtype)
MemoryError
any comments are welcome ! thanks a lot.

There were some memory problems that are fixed by running with tcmalloc -- http://goog-perftools.sourceforge.net/doc/tcmalloc.html Also, with tcmalloc you can turn on heap profiler and get a more detailed statistic of where the memory is going

@aimeida, please let us know if tcmalloc helps.

thanks a lot for the response ! The "sad" truth is that there is NO bug, the program simply runs out of memory :-) I didn't realize that EC2 instance doesn't have swap -- that's why it works on Mac.
Very sorry for the inconvenience :-(

Thanks for letting us know, glad you figured out the issue.


TOMEE-2963 Translate to Portuguese: examples/serverless-tomee-plus

Translate to Portuguese the example serverless-tomee-plus
link to Jira: https://issues.apache.org/jira/browse/TOMEE-2963

@Daniel-Dos I'll merge and any further Portuguese-related grammar can be fixed afterward.


Re-enable IncrementalTest as ignored.

See https://github.com/sbt/zinc/issues/152: this re-enables IncrementalTest as ignored for now. More work will be necessary in order to sync up the AnalysisCallback implementations, but there are some passing tests, and this will help to minimize compiler-checkable bitrot.

These are commented out because we opted for using scripted tests - https://github.com/sbt/zinc/tree/1.0/zinc/src/sbt-test

What's the status of this @eed3si9n ?

Yea. Let's close it for now.

@eed3si9n if we don't want this tests anymore can we remove it from codebase with all its utils? I don't think that having whole file commented out and some dead code will help us anywhere. We can keep link to this (or any other) revision in e.g. comment somewhere if for some reason this code would be needed (but IMHO we can just remove all dead code).
@jvican  @stuhood WDYT?

Yea. I'm fine with that. We can always dig it out of the git history.

+1
On Feb 9, 2017 12:58 PM, "eugene yokota"<EMAIL_ADDRESS>wrote:
Yea. I'm fine with that. We can always dig it out of the git history.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sbt/zinc/pull/153#issuecomment-278771669, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAC2lMj5q2ruTBqOsAtpNffHuL-WnQ6Wks5ra338gaJpZM4JMsca
.


Change the Tag component in kibbeh

This pr is related to the Atomic Components issue no. #1324. This changes the existing tag component and makes needful changes.

This one requires some changes to the config, let me handle it


Unable to add a Route to a Service

Hi Konga team,
First off. Kudos on a great interface! This has been a super useful tool in managing my Kong server.
I am running Kong and Konga in a Kubernetes cluster. I just updated to Kong 0.14.0 and Konga 0.12.0. I am trying out the Services/Routes feature for the first time. I was able to successfully create a new Service with no problem, but for some reason, I am unable to create a new Route on that Service.
Steps to reproduce:

Add a new Service
Select the Service and select Routes
Add a new Route
Populate the Hosts, Paths, and Methods fields (e.g. example.com; /api/v1; GET), and leave all other fields as default
Click Submit Route

An error message appears, and when I checked the logs on the Docker container, this is what I saw:
8/14/2018 9:33:55 AMKongProxyController:entity services
8/14/2018 9:33:55 AMKongProxyHooks:afterEntityList called() services
8/14/2018 9:34:15 AMKongProxyController:req.method POST
8/14/2018 9:34:15 AMKongProxyController:req.url /routes/
8/14/2018 9:34:15 AMKongProxyController:entity routes
8/14/2018 9:34:15 AMKongProxyController request error { fields:
8/14/2018 9:34:15 AM   { '@entity':
8/14/2018 9:34:15 AM      [ 'at least one of these fields must be non-empty: \'methods\', \'hosts\', \'paths\'' ] },
8/14/2018 9:34:15 AM  name: 'schema violation',
8/14/2018 9:34:15 AM  code: 2,
8/14/2018 9:34:15 AM  message: 'schema violation (at least one of these fields must be non-empty: \'methods\', \'hosts\', \'paths\')' }

It seems there may be an issue with the schema validation for creating a new Route, unless I am missing something. For the time being, I am okay to keep using the deprecated APIs feature, but it would certainly be nice to migrate to the latest and greatest. 😄 Any help on this would be greatly appreciated!

Hi James, try pressing enter after every method, host or path you type in the routes form. Theese are not text inputs but arrays visualized as chips

@pantsel ahhh... Yes, I see now. I was still used to the old style of API form input. Thanks for the quick response!

I ran into this as well.
Didn't know that i needed to hit "Enter" for it to work until seeing this.
thanks for https://medium.com/@tselentispanagis/managing-microservices-and-apis-with-kong-and-konga-7d14568bb59d btw


Istiod validationServer unrecognized type instance flood

Bug description
We are seeing constant message flood on istiod
2020-11-20T18:18:35.294707Z     info    validationServer        unrecognized type instance
2020-11-20T18:18:35.302958Z     info    validationServer        unrecognized type instance
2020-11-20T18:18:35.494957Z     info    validationServer        unrecognized type instance
2020-11-20T18:18:35.499724Z     info    validationServer        unrecognized type instance
2020-11-20T18:18:35.695297Z     info    validationServer        unrecognized type instance
2020-11-20T18:18:35.700524Z     info    validationServer        unrecognized type instance
2020-11-20T18:18:35.895377Z     info    validationServer        unrecognized type instance
2020-11-20T18:18:35.900836Z     info    validationServer        unrecognized type instance

r>kubectl get instance -A
NAMESPACE      NAME                   AGE
istio-system   attributes             519d
istio-system   requestcount           401d
istio-system   requestduration        401d
istio-system   requestsize            401d
istio-system   responsesize           401d
istio-system   tcpbytereceived        401d
istio-system   tcpbytesent            401d
istio-system   tcpconnectionsclosed   401d
istio-system   tcpconnectionsopened   401d

Is there any workaround for this?
[ ] Docs
[ ] Installation
[ ] Networking
[X] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[X] User Experience
[ ] Developer Infrastructure
[X] Upgrade
Version (include the output of istioctl version --remote and kubectl version --short and helm version if you used Helm)
client version: 1.8.0
control plane version: 1.8.0
data plane version: 1.8.0 (28 proxies), 1.7.4 (4 proxies)
How was Istio installed?
istioctl install
Environment where bug was observed (cloud vendor, OS, etc)
GKE 1.18.10-gke.2101

This sounds like you have configured your validating webhook (or, we have configured it) to send the legacy mixer types to the new 1.8 server which doesn't support mixer types anymore

Ah
    rules:
    - apiGroups:
      - config.istio.io
      - security.istio.io
      - authentication.istio.io
      - networking.istio.io

we should probably remove config.istio.io

Actually we may just want to just filter these out in the webhook code
@jasonwzm @istio/wg-policies-and-telemetry-maintainers any thoughts

Not sure if there is other configuration under config.istio.io, but if mixer resources are the only part, we should just remove it.
@cober Work around is just to delete those instances, they are not functioning any more in since 1.8.

@bianpengyuan did we ever add warnings for mixer config on upgrade? i thought that was part of the work.  was it just for non-customized config?

Not sure if there is other configuration under config.istio.io, but if mixer resources are the only part, we should just remove it.

I don't think we are
What is the expected behavior for 1.8? I would think we should ignore the mixer types, whereas right now we are rejecting them.

@bianpengyuan did we ever add warnings for mixer config on upgrade? i thought that was part of the work. was it just for non-customized config?

I think the warning is for mixer resources that user created themselves, but @richardwxn would know better.

What is the expected behavior for 1.8? I would think we should ignore the mixer types, whereas right now we are rejecting them.

I think we could just ignore it, which is less confusing.

+1 to remove it from the configuration. I don't think we should add more hacky logic in validation code.
If we declare in the configuration, it still means we are supporting validating the object which we are not.


Unable to play opus files in Windows 10 and Android 10.

Hello,
First of all thanks a lot for this amazing work.
I would like to ask for a help. I am able to convert my books from aax to opus files with this command bash AAXtoMP3 -o -A myAuthCode -t /home/marcel/Books/ ./audible_dir/Content/*
The problem is that I am unable to play them On Android 10 phone (tried different makes) with any player except VLC. However, VLC has problems to fast forward and rewind back. Any other apps like Booksonic or Smart AudioBook Player that I have purchased are unable to play the files and just pass through the files with the result that the book is finished.
In addition, I tested the files with these players:

VLC (Windows 10, problem with fast forward and rewind)
Totem (Pop OS 20.04, No problems at all)
Groove Music (Windows 10 default player, , problem with fast forward and rewind)

Opus file specs:

Container: Ogg
Codec: Opus
Channels: Stereo
Sample rate: 48000 Hz

An interesting thing is that if I use SoundConverter in Pop OS or LameXP in Windows 10 and set it to the same output parameters as the file has currently, the resulting output opus file works without any problem.
Here is one opus file that was converted with AAXtoMP3: https://drive.google.com/file/d/19tjDo2xWuycRUbdWOEsJ43f9QKOD2GzY/view?usp=sharing
Any help is very appreciated.

I also converted a book to opus files and am having similar issues.
Audacious player plays the files with no problem, but Deadbeef player will only play the first file, and returns vague "ERROR: could not load:" on all the rest.
I noticed that the README.md states "The extension is .ogg", but in fact the files that my copy of AAXtoMP3 (v1.3) output had ".opus" extensions.  I tested by changing one file's extension to "ogg" but it didn't enable deadbeef to play it.
I was also curious if the metadata could be causing problems.  I removed the VorbisComment tag from the file - no go.  Wasn't sure if there could still be other issues with the opus headers.
I also tried "converting" the file to same parameters as you did (using soundKonverter) and it worked for me too.
One additional bit: I opened one of the suspect files in MediaInfo and it reports that it has video content!
General
Complete name                            : /home/ryan/Books/Bakunin-God-State/opus/God and the State (Carl Manchester)/God and the State-2 Chapter 2-testing.opus
Format                                   : Ogg
File size                                : 1.94 MiB
Duration                                 : 2 min 6 s
Overall bit rate                         : 128 kb/s
Movie name                               : Album cover
Description                              : Cover (Front)
Writing application                      : Lavc58.134.100 libtheora

Video
ID                                       : 828721532 (0x3165497C)
Format                                   : Theora
Duration                                 : 2 min 6 s
Nominal bit rate                         : 200 kb/s
Width                                    : 1 214 pixels
Height                                   : 1 214 pixels
Display aspect ratio                     : 1.000
Frame rate                               : 25.000 FPS
Compression mode                         : Lossy
Bits/(Pixel*Frame)                       : 0.005
Writing library                          : Lavf58.76.100
Track/Position                           : 2

Audio
ID                                       :<PHONE_NUMBER> (0xE4AAF516)
Format                                   : Opus
Duration                                 : 2 min 6 s
Channel(s)                               : 2 channels
Channel layout                           : L R
Sampling rate                            : 48.0 kHz
Compression mode                         : Lossy
Writing library                          : Lavf58.76.100

The file that I converted with SoundKonverter only has audio content.
Something's going wrong in the conversion to opus.

Removing lines 872, 873, 875 & 876 fixed it for me. So presumably changing these to be conditional on not having the opus format selected will fix the issue. Its important the -map options are also removed or FFMPEG will give an error. I'm not amazing with bash nor super familiar with the script so I will leave the actual implementation of the fix up to someone else. I literally just deleted the lines in my own copy of the script as I will only use the opus format anyways.


Removed a broken link

I spotted a broken link while testing all links.

@debugleader Thanks for the PR, would it be possible before merging it to check if they changed URL or if we can replace it
with some other?

Welcome, I checked but their site is so broken, a lot of buttons don't work and it seems weird

Do you think it's a good idea to merge it and remove the site altogether?

I am not sure if they actually send you notifications or not so for safety reasons, I am gonna tag you @unseen1980

Yes, let's merge it and if you find some other similar API please add it. Thanks


Sidebar loses transition

I was structuring a dashbaord with three main components, (Sidebar, Topbar and Content)
I used HeaderContainer to share { isSideNavExpanded and onClickSideNavExpand } between Topbar and Sidebar.
Basically, Topbar is updating Sidebar's isSideNavExpanded but when i click the button in topbar, sidebar expands but without transition. Is there any workaround for that?
App.js
      <HeaderContainer         render={({ isSideNavExpanded, onClickSideNavExpand }) => (           <>             <TopBar               isSideNavExpanded={isSideNavExpanded}               onClickSideNavExpand={onClickSideNavExpand}             />             <SideBar isSideNavExpanded={isSideNavExpanded} />             <Content>
Dashboard
             </Content>           </>         )}       />
TopBar.js
`

<HeaderMenuButton
  aria-label="Open menu"
  onClick={props.onClickSideNavExpand}
/>

`
Sidebar.js
   <SideNav       isFixedNav       expanded={isSideNavExpanded}       isChildOfHeader={false}       aria-label="Side navigation"     > {/* test.........content */}    </SideNav>
Thanks in advance

Hi @theprogboy could you create a code sandbox replicating the issue? It'd be helpful for us to figure out what is going wrong. Thanks!


branch per version

Description
This PR splits the master branch into two branches:

master: contains only the 'master' docs
release-legacy: contains all docs except master

To accomplish this, it introduces a new script "netlify/build.sh" which clones different branches into temp dirs, builds the sites with special "baseurl"'s, and then rsync's the files together.
It is capable of pulling additional "release-vX.Y" branches in as well but that's unused for now since there are no branches cut from master using this new format.
A subsequent PR will move "release-v3.11" from the "release-legacy" branch to a "release-v3.11" branch and then update branches accordingly.

Related issues/PRs

Remove 'master' directory from release-legacy branch (https://github.com/projectcalico/calico/pull/3102)

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

Deploy preview for calico ready!
Built with commit 1af495be0324e3aa6ee907dbe90da02fd48d41aa
https://deploy-preview-3103--calico.netlify.com


pull request for e2e test at 12/08/2016 04:26:01

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Update Blender info

Signed-off-by: soniasingla<EMAIL_ADDRESS>Checklist

[x] I have read the Contributor Guide
[x] I have read and agree to the Code of Conduct
[x] I have added a description of my changes and why I'd like them included in the section below

Description of Changes
Updating Blender's information as it's not recently announced 🐌

I made a small change - I think it makes more sense to say "proud to be a sponsor" rather than "proud to announce" that we are a sponsor, since "announcing" implies that it is a recent thing.


headers

Hi @tdelabro can I work on this one ?

Hi @tdelabro , I'm not sure to fully undertstand the following (from https://btcinformation.org/en/developer-reference#headers):
A headers message can be empty.
Does it mean we can have the following:
A- A message header with a payload size of 0
B- A message header with a payload size of 1 and headers.count == 0:  payload will be [0x00]
I have assumed A and B but I'm not sure  to be right.
Thanks for you help.

It's B


VoteMVP Frontend: Remove hover message.

5/4/22 AirClaim Votemvp Frontend v0.5.2qa5 Link: https://tjrhs-gqaaa-aaaad-qa24a-cai.raw.ic0.app
On the 'Vote on Locking Threshold' page, remove the message ''Please enter a valid value. The two nearest values are 0 and 1" displayed when hovering over the amount entered. (Tom reckons it is not required).

Removed Validation tooltip v0.5.4

6/4/22 Retest in v0.5.4-qa5. Tooltip still showing when hovering over the amount entered. Not fixed.

Im not getting this now in the browsers Ive tested Mac Chrome & Safari, what is the browser your using?

11/4/22 Retest in v0.5.4-qa5 using Mac Chrome. Cleared cache. Not seeing the tooltip when hovering over the amount.   Looks like clearing the cache works.
Also tested in mac safari and it's working fine. Fixed. Close.

11/4/22 Reopening this issue as tooltip still showing when hovering over the amount entered in new rel 0.5.4-qa5ii
account-vivvestin, browser-chrome

.

19/4/22 Retest in new rel v0.5.5-qa5. Tooltip still showing when hovering over the amount entered in iteration 16.
Account-billbeaumont, browser-chrome.

20/4/22 Retest in new rel v0.5.6-qa5. Tooltip no longer displayed when hovering over the amount entered in iteration 17.
Account-alanappleton, browser-chrome. Fixed. Close.

.


Resolved Isssue with using wrong variable for glBindFrameBuffer

Use the created framebuffer object instead of a texture object when binding an OpenGL framebuffer.

LGTM, thanks for the fix

LGTM, thanks for the fix


Cube: overview diagram should show mobile devices accessing the REST/VPC environment

https://github.com/confluentinc/demo-scene/tree/master/ccloud-cube-demo
The image at https://github.com/confluentinc/demo-scene/raw/master/ccloud-cube-demo/images/rest-api-call.png should show how the mobile devices (= what the users do) interact with the REST proxy.  I read the intro paragraphs and kept wondering where, in the diagram, the users are represented.
Also, the intro paragraph talks about "the application" and "the moves", but lacks even a 1-liner description of what "the Cube" (the app, the moves) actually is:

This code contains a sample application called "The Cube" that allow users to interact with Kafka clusters running on Confluent Cloud by using their mobile devices. While users play with the application from their mobile devices, events containing their moves are emitted to a topic.

I found this info much later:

The application presents the user with a cube that moves as the user moves its hands and body. It relies on the device orientation API found in most modern devices.

And typo here:

Anyone capable of perfectly positioning the cube using the coordinates the represents the number three will be shown in the output of a KSQL query.

Closing as pretty sure this is stale/legacy issue now.


use package.json defined fields for application info

We currently store application description as part of warehouse application registration. I should be using information provided via package.json, and/or github repo so that developer doesn't have to keep up with separate sets of information.
The only metadata for app that we should store is github repo name.

With updated abcd spec, most app probably won't have package.json (only main is required now). I think I could load description from github repo for each app instead.


t265 relocalization feature unstable.

Recently I tested the export_localization_map and import_localization_map feature of t265, and there some problems about. Firstly, I set about 20 static node and export the localization map, then i imported the localization map and try to get all 20 nodes in a loop, once i got one node, then i would get all nodes, that's for sure, But here is the problem: even i set 20 nodes in the localization map, it is very hard to get even one static node in my environment, and when the localization map is to large, about 600,000, my  program will crash with LIBUSB_ERROR_TIMEOUT. Thanks~

Hi @ahuizxc,
Can you share some code snippets in order to better understand your issue?

@RealSenseCustomerSupport After loop closing does it optimize the whole map or just the last pose. Can you please provide some insight. And it does optimize the whole map/trajectory is it possible to get it ?

@harville , any comments about this?

Thank you for highlighting the issues with relocalization. We have moved our focus to our next generation of products and consequently, we will not be addressing this issue in the T265.

I am really disappointed to hear that. It gives me an impression that other line of products will end up having the same fate.
Sent from Yahoo Mail for iPhone
On Saturday, September 12, 2020, 2:44 AM, RealSenseSupport<EMAIL_ADDRESS>wrote:
Thank you for highlighting the issues with relocalization. We have moved our focus to our next generation of products and consequently, we will not be addressing this issue in the T265.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

The statement is targeted for the T265 product and not other products.


[PAM Password quality] change password allowed using wrong credentials

Describe the bug
I am using following rules to enforce strong password. Though it enforces strong password but it allows me to change the password using wrong current password.
security.pam.services.passwd.rules.password.pwquality = {
        control = "required";
        modulePath = "${pkgs.libpwquality.lib}/lib/security/pam_pwquality.so";
        # order BEFORE pam_unix.so
        order =  config.security.pam.services.passwd.rules.password.unix.order - 10;
        settings = {
          retry = 3;
          minlen = 8;
          difok = 6;
          dcredit = -1;
          ucredit = 1;
          ocredit = -1;
          lcredit = 1;
          enforce_for_root = true;
        };
      };

Steps To Reproduce
Steps to reproduce the behavior:

Use above configuration
Run passwd command
Enter wrong current password
Enter new password
Password changed successfully.

Expected behavior
I should not be allowed to change password using wrong current password.
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.47, NixOS, 23.05 (Stoat), 23.05.3085.2ab91c8d65c0`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.5`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Add a :+1: reaction to issues you find important.

I'm not really understanding what you mean by "wrong current password", that means you can just change the password using any password? So it is not checking if the password is correct?

Yes

I could confirm this behavior on NixOS 24.05

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/enforcing-strong-passwords-on-nixos-pam-pwquality-so-module-not-known/36420/7

I cannot confirm this behaviour on NixOS 24.11. Upon entering a wrong password, followed by a new password and the confirmation of the new password you will receive a token manipulation error.

That's weird, because the issue still exist on my system:
sudo adduser test
echo 'test:C51.kH9wGnwHwJVRkN19' | sudo chpasswd
sudo su test
passwd

Changing password for test.
Current password: NotTheRealPassword
New password: AnyThingThatFits-P0licy
Retype new password: AnyThingThatFits-P0licy
passwd: password updated successfully

[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.8.12, NixOS, 24.11 (Vicuna), 24.11.20240616.b60ebf5`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.22.1`
 - nixpkgs: `/nix/store/5w7nhyvrgsa81h0kyifmbl6j97hvlzh5-source`

You're right, I had another PAM rule that caused an override. Removing it would cause passwd to change passwords without the old password.
I think the line responsible is
password sufficient pam_unix.so nullok yescrypt # unix (order 10200)

in /etc/pam.d/passwd. Other distros have this set to required, but NixOS seems to be defaulting to sufficient since 19.05.

@Majiir as the maintainer of the pam module, how should we proceed? Is this an actual error in the module that needs to be investigated, or have we just made a mistake in the configuration from above?

@NotAShelf But without the pwquality pam module, the line you mentioned is also included in the /etc/pam.d/passwd. Without loading the above configuration the old password is still required, so I'm wondering why you think that this line causes the issue :thinking:

My guess was kinda random, making comparisons to my Arch box and some man pages of PAM. In my testing, I got to override that sufficient with required  - which lead to a different error.


Push route without BuildContext

Hey, I've been trying to dig into this for a bit now. What happens if I want to push a route programmatically, say from a listener setup in my main widget constructor.
class Global {
  static Router router;
}

void main() {
  runApp(new App());
}

class App extends StatelessWidget {
  Router router;

  App() {
    Global.router = new Router()
      ..define("home",
          handler: new Handler(handlerFunc: (BuildContext context, params) {
        return new HomePage();
      }))
      ..define("auth",
          handler: new Handler(handlerFunc: (BuildContext context, params) {
        return new AuthPage();
      }));

    someListener.listen((user) {
      if (user == null) {
        // Do logout, navigate to "auth"
      }
    });
  }

  @override
  Widget build(BuildContext context) {
    return new MaterialApp(
      onGenerateRoute: Global.router.generator,
    );
  }
}

I'm trying to find a way to access the root navigator from my listener, and push directly from there.
This is probably also related to Flutter's built-in navigation.
What would be a solution?

Unfortunately I don't believe there is a concept of an "application" or global context in flutter. You need to pass the context down to any helper, screen, widget, etc that needs access to the current context. That unfortunately means the Navigator also. You could possibly capture the Navigator instance when you initialize your main application (or first screen) and use that but I can't attest to the safety of that approach. You may end up a bit worse for wear.

For now I'm closing this as it is more a Stack Overflow type question. Feel free to post there using the tag fluro!

for someone struggling with the problem here is promising workaround -- why people try to be fast to close question? It is not keeping bug reports in nice order. nobody gets better from such bureaucracy.

In Flutter GlobalKeys can be used to access the state of a StatefulWidget and that's what we'll use to access the NavigatorState outside of the build context.

The issue was closed because this isn't the forum for these types of questions. This is the forum for issues.

I have been able to modify the function navigateTo, and instead of using BuildContext context as parameter, I am using NavigatorState navigatorState (which it would be the GlobalKey in MaterialApp).
Now, I can get the context from there. It works for me. Should I open a PR to be able to navigate in both ways?
Future navigateTo(NavigatorState navigatorState, String path,
                  {bool replace = false,
                      bool clearStack = false,
                      TransitionType transition,
                      Duration transitionDuration = const Duration(milliseconds: 250),
                      RouteTransitionsBuilder transitionBuilder}) {
    /* Get context for non-navigation-matters */
    BuildContext context = navigatorState?.context;
    RouteMatch routeMatch = matchRoute(context, path,
            transitionType: transition,
            transitionsBuilder: transitionBuilder,
            transitionDuration: transitionDuration);
    Route<dynamic> route = routeMatch.route;
    Completer completer = Completer();
    Future future = completer.future;
    if (routeMatch.matchType == RouteMatchType.nonVisual) {
        completer.complete("Non visual route type.");
    } else {
        if (route == null && notFoundHandler != null) {
            route = _notFoundRoute(context, path);
        }
        if (route != null) {
            /* Here's the change */
            if (clearStack) {
                future =
                        navigatorState.pushAndRemoveUntil(route, (check) => false);
            } else {
                future = replace
                        ? navigatorState.pushReplacement(route)
                        : navigatorState.push(route);
            }
            completer.complete();
        } else {
            String error = "No registered route was found to handle '$path'.";
            print(error);
            completer.completeError(RouteNotFoundException(error, path));
        }
    }

    return future;
}

https://github.com/theyakka/fluro/pull/148

Isn't there any interest of making this improvement?

You can use the default global router for global access.
I decided not to use Fluro for other reasons.


[Suggestion] It would be better if add the database's name in the popped up notification when deleting an attached PostgreSQL database

OS: All
Build Version: 20210525.1
Repro Steps:

Create a PostgreSQL server -> Create a new database "testdata" for it.
Attach the "postgres" and new created database "testdata" by connection string.
Navigate to the "Attached Database Accounts" node.
Delete the "postgres" and new created database "testdata".
Check the notification.

Actual:

Popped up notification doesn't contain the database's name for the new created database "testdata".

Popped up notification contains the database's name for the "postgres" database.

Suggestion:
It would be better if add the database's name in the popped up notification when deleting a PostgreSQL database, like below:

The error notification is coming directly from the postgres api, currently not planning to change this.


toolbox: fix header bar border

The border color was being overridden by border-bottom: 1px solid ThreeDShadow in browser-shared.css.
Current border color

Fixed border color

🤦


Relax the certificate chain requirement for wider use case coverage

To support additional use cases as outlined below, the certificate chain requirement imposed by the merge of https://github.com/notaryproject/notaryproject/pull/162 should be relaxed to allow for a certificate chains of any length Vs the current limit of 2.
Desired outcome
The notary signature-specification is updated to support the following use cases

Self Signed certificates

@iamsamirzon  Since #162 was  done by @priteshbandi and @gokarnm, I would suggest Milind to update the spec and Pritesh to implement the fix based on the spec.

Merged https://github.com/notaryproject/notaryproject/pull/193 and https://github.com/notaryproject/notation-core-go/pull/72

Merged #193 and notaryproject/notation-core-go#72

I think the 2nd PR should be https://github.com/notaryproject/notation-core-go/pull/77


⚠ ClamAV has degraded performance

In b270cb4, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 208 ms

Resolved: ClamAV performance has improved in 00ea9ae.


YZSwipe under a Navigation Bar

Hi,
Thanks for the great and simple library. I've hit a situation where I have a setup where I'd like the YZSwipe underneath a NavigationController rather than each VC be in a separate NavigationController inside YZSwipe because that swipes the navigation bar and it looks ugly.
But when I do put YZSwipe inside the NavigationController, the YZSwipe goes crazy (slides below navigation, starts to scroll horizontally/vertically). Is there a way around this?
Thanks,
Michael

Maybe you could use a UIPageViewController.

Try the latest version https://github.com/yichizhang/YZSwipeBetweenViewController/releases/tag/0.1.0


How to use in a project that is setup using Vue CLI@3 and TypeScript?

td;dr
How do I include vue-apollo in a project that already uses TypeScript?
I created a fresh vue project using vue cli@3 and using typescript.
Then I added the vue-apollo plugin which modified my main.ts file to add
apolloProvider: createProvider(),

in the vue instance creation.
But the compiler complains about this.

Argument of type '{ router: VueRouter; store: Store<{}>; apolloProvider: { provide: () => {}; }; render: (h: CreateElement) => VNode; }' is not assignable to parameter of type 'ComponentOptions<Vue, DefaultData, DefaultMethods, DefaultComputed, PropsDefinition<Record<string, any>>, Record<string, any>>'.

Object literal may only specify known properties, and 'apolloProvider' does not exist in type 'ComponentOptions<Vue, DefaultData, DefaultMethods, DefaultComputed, PropsDefinition<Record<string, any>>, Record<string, any>>'. [2345]

I can see that there is a types directory in the vue-apollo package I fetched from npm but not sure how to use them.
I also get the following

[ts] Could not find a declaration file for module '@/vue-apollo'. '/Users/praveen/code/voicezen/repos/voicezen-ui/src/vue-apollo.js' implicitly has an 'any' type. [7016]

for the following import in main.ts
import { createProvider } from '@/vue-apollo';

I get that this might be coming from the noImplicitAny rule but then changing the generated vue-apollo.js to vue-apollo.ts also doesn't solve the problems.
Changing it to .ts removes the compiler errors from main.ts about the above two but then I get the following.

Could not find a declaration file for module 'vue-cli-plugin-apollo/graphql-client'. '/Users/praveen/code/voicezen/repos/voicezen-ui/node_modules/vue-cli-plugin-apollo/graphql-client/index.js' implicitly has an 'any' type.

for
import {
  createApolloClient,
  restartWebsockets
} from 'vue-cli-plugin-apollo/graphql-client';

To fix this, I can add a module declaration in my typings like this but is that the right way?
declare module 'vue-cli-plugin-apollo/graphql-client';

and the onLogin or onLogout param apolloClient starts complaining about them being implicitly any type too.

@praveenpuglia
I guess we have to improve vue-cli-plugin-apollo to deal with it.
https://github.com/Akryum/vue-cli-plugin-apollo
For now, you can define like below codes on main.ts to fix error report.
// add module import and define apolloClient type
import { ApolloClient } from "apollo-client";
import { SubscriptionClient } from "subscriptions-transport-ws";
import { InMemoryCache } from "apollo-cache-inmemory";

type VueApolloClient = ApolloClient<InMemoryCache> & {
  wsClient: SubscriptionClient;
};

//... some codes ...

// set apolloClient type
export async function onLogin(apolloClient: VueApolloClient, token: string) {
  //...
}

// Manually call this when user log out
export async function onLogout(apolloClient: VueApolloClient) {
  //...
}

Thanks for the comment. This really helps!
I have one more small doubt.
for onLogin and onLogout, if I have multiple clients, do I need to call resetStore on all the clients?
If so, how do I pass all clients to say, onLogout while calling from a component or store?
I tried this.$apollo.provider.clients and this.$apolloProvider.clients both of which result in type errors even though the clients exist there.

Currently this is what I'm doing:
const clients = this.$apollo.provider.clients
for (const client of Object.keys(clients)) {
  await clients[client].resetStore()
}

@quentinus95 this works, but TS complains about this.$apollo.provider doesn't have clients.

@praveenpuglia
I have made PR to deal with that.
https://github.com/Akryum/vue-apollo/pull/447
For now, please ignore error report on typescript.
@Akryum
I'm sorry to bother you though, I would be grateful if you would check that PR. 🙇

Thanks so much! Really waiting for this to get merged.

I tried to change tsconfig
{
  "compilerOptions": {
     ..............
    "allowJs": true
  },

Please advise


Add integration to guidance library

Thanks a lot for implementing this ToT library, nice work!
I think it could benefit a lot from using Microsoft Guidance (https://github.com/microsoft/guidance), specially with smaller local models that have a hard time following the instructions exactly.
Is this something you would be interested in?
Sadly, I’m low on time this week (and the next) to help with the implementation, but I would eventually look into making this integration (if you have no interest in making it yourself).

tutorial : https://github.com/microsoft/guidance/blob/main/notebooks/tutorial.ipynb

@paolorechia Guidance actually supports loading local LLMs and its pretty versatile https://github.com/microsoft/guidance/blob/main/guidance/llms/transformers/_llama.py, but loading local LLMs will surely making ToT heavier.
I believe you can extend from that llm class a little bit wrapping your webui API call inside, this way,  your pr would work with guidance.

@yhyu13 thanks for looking into it :)
I already have a fork of guidance that uses the text-generation-web-ui api: https://github.com/paolorechia/local-guidance
I didn’t mention it before I was not sure if you wanted an integration to text-generation-web-ui in the first place.
I did something similar as you mention, but based on OpenAI class. Main problem here is that my code doesn’t support many features of guidance, but that’s definitely a way to go.
Maybe something better would be to stick to the more “official” integrations, like Huggingface, so it also supports better guidance?

@yhyu13 Not part of, but you'll get a much better support to use guidance if you use Hugging Face. There are several features implemented specifically for the HF API, e.g., token healing.

I thought token healing was just an bool flag, had never looked inside.
ModuleNotFoundError: No module named 'guidance'

tree-of-thoughts % /usr/local/bin/python3 /Users/defalt/Desktop/Athena/research/tree-of-thoughts/experiements/extremely_experimental/prompting/guidancePrompt.py
Using api_model text-davinci-003
Generated thoughts: Given the current state of reasoning: '{state_text}', generate {5} coherent thoughts to continue the reasoning process:{#each (gen 'thoughts' k=k)}
{this}{/each}
Traceback (most recent call last):
File "/Users/defalt/Desktop/Athena/research/tree-of-thoughts/experiements/extremely_experimental/prompting/guidancePrompt.py", line 434, in
solution = tree_of_thoughts.solve(input_problem, k, T, b, vth, timeout, confidence_threshold=confidence, max_iterations=max_iterations, convergence_threshold=convergence_threshold, convergence_count=convergence_count)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/defalt/Desktop/Athena/research/tree-of-thoughts/experiements/extremely_experimental/prompting/guidancePrompt.py", line 401, in solve
result = self.tot_dfs(x, k, T, vth, confidence_threshold=confidence_threshold, max_iterations=max_iterations, convergence_threshold=convergence_threshold, convergence_count=convergence_count)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/defalt/Desktop/Athena/research/tree-of-thoughts/experiements/extremely_experimental/prompting/guidancePrompt.py", line 387, in tot_dfs
dfs(x, 1)
File "/Users/defalt/Desktop/Athena/research/tree-of-thoughts/experiements/extremely_experimental/prompting/guidancePrompt.py", line 379, in dfs
for s_prime in sorted(self.model.generate_thoughts(s, k)):
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/guidance/_program.py", line 326, in getitem
return self._variables[key]
~~~~~~~~~~~~~~~^^^^^
KeyError: 0

@jabowery is that using guidance?

is that using guidance?

Almost certainly.  The only way I was able to get the example.py program to run was to include guidance in my conda environment via the provided environment.yml because without guidance installed, example.py would fail with ModuleNotFoundError: No module named 'guidance'

I have the same issue. Running the example prompted me to install guidance. After pip installing guidance and using this as the input_problem: using 1 2 6 9 us these numbers and basic arithmetic operations (+-*/) to obtain 24. The output should be an equation. I get some response like solution: (['Observation: All of the numbers provided can be combined to create 24.\nThoughts: We can use trial and error to find the combination of numbers and operations that produces the desired result of 24.'], 1.0)
Any ideas on how (with what settings) to get the example going are highly appreciated :)

@ALL https://github.com/kyegomez/tree-of-thoughts/pull/18/#issue-1723812572 guidance is still WIP at this moment, the prompt suggested from https://github.com/microsoft/guidance that uses system, user, assistant actually leads to GPT4 spitting out wrong output for evaluation
The example.py will not use guidance before this problem is fixed

https://github.com/kyegomez/tree-of-thoughts/issues/11#issuecomment-1560042264 @jabowery
text-davinci-003 is poor at this task, gpt4 and chatgpt mostly will speculate with actual mathematics formula

I have finally managed to solve this puzzle:
input_problem = "use 1 2 3 as available numbers, you can only use each number once and basic arithmetic operations (+-*/) to obtain 7. The output should be an equation."
with gpt-4 and V2 in the example file with these settings:
k = 5
T = 2
b = 3
vth = 0.5
timeout = 20
confidence = 1.0 #cmodel is confident on performance
max_iterations = 40 #tree branh nodes
convergence_threshold = 0.01
convergence_count = 5
search_algorithm = "BFS"
#cot or propose
strategy="cot"
value or vote
evaluation_strategy = "value"
Pretty awesome! Did cost quite a lot of tokens, though.


add site RSS feed icon to header social bar

This PR adds a icon for the site's RSS feed in the header alongside the other social link icons.  I realize this is a bit redundant with the sidebar subscribe section, but I think it is convenient to have a simple icon to click on in the header too.

The tricky thing here is that you're using .Site.RSSLink which is a little opinionated; there's some question about which RSS link to use (should be it for the whole site, which would include blog posts, or just the podcast episodes, which is a different link?)

To see a bit more of what I'm talking about, see how it's done in the sidebar:
https://github.com/mattstratton/castanet/blob/master/layouts/partials/sidebar.html#L29-L37

Good point. When I first thought of it I figured it could be just the episodes feed, but for my site I will be adding custom content with a new type like article so I can put in-depth tutorials that are companions to the podcast show notes.  So I would want my listeners to be informed of new content if they wanted to keep track of it via RSS.  Perhaps it's just as well that we keep any RSS links in the subscribe sidebar and it's not worth putting it in the header since it's not obvious which feed it represents until you click it.

I'm now thinking we should scrap this change and keep any RSS feed links in the subscribe sidebar and not the header.  On that note, I see that a "site-wide" RSS feed is only added when the user defines their own custom RSS feed via the rss_subscribe option in the config.  Suppose I don't want to override the RSS feed, yet I still want to have a site-wide RSS feed link appear below the episode RSS link in the sidebar.  How would you recommend I set the value of rss_subscribe if I don't use something like Feedburner and want  to stick with the Hugo version of the overall RSS feed.  I tried rss_subscribe = "{{ .Site.RSSLink }}" but that didn't extract the value like it did for the header trick in my PR.

I have some thoughts around wanting to get this set; in a way, #101 is tracking that kind of work.