Allow Team owners to manage/add/remove members

Splitting this off from https://github.com/gratipay/gratipay.com/issues/3545.
https://github.com/gratipay/gratipay.com/issues/3545#issuecomment-136154542
Notify

[ ] https://gratipay.freshdesk.com/helpdesk/tickets/3126

@mattbk Are you thinking we'll allow adding Team members before we bring back payroll itself?

It could be possible, although it would require a lot of explaining.
Is there a ticket for this technical aspect of bringing back Payroll? I mostly wanted to distinguish this from "edit teams" (which is done).

It could be possible, although it would require a lot of explaining.

Agreed.

Is there a ticket for this technical aspect of bringing back Payroll? I mostly wanted to distinguish this from "edit teams" (which is done).

We have #3433 but that's pretty general. I think having a ticket for this is fine, but I'm not sure about having it on the Team Basics milestone. Why'd you go for that instead of Payroll?

+1 from @stevepiercy at https://github.com/gratipay/gratipay.com/issues/3681#issuecomment-135148451:

when I log in as ~stevepiercy and view my account profile, I expect to see a UI for me to manage Team KnopProject, but cannot find one. I couldn't manage the Team, so I assume one of the Gratipay staff assigned me as Owner on my behalf. Is some public Team management UI is forthcoming?

Closing this ticket now that we have #3994, where this idea is captured as part of our plan to bring back payroll for Team Gratipay. Looks like the "notify" here is already cross-listed over on #3433.


Server Function Extended HTTP Verb Support

I request that server function encodings be expanded to allow user input of verbs, even if not recommended.
Why not PUT or DELETE? Why URL/form encoding, and not JSON?

    These are reasonable questions. Much of the web is built on REST API patterns that encourage the use of semantic HTTP methods like DELETE to delete an item from a database, and many devs are accustomed to sending data to APIs in the JSON format.

    The reason we use POST or GET with URL-encoded data by default is the <form> support. For better or for worse, HTML forms don’t support PUT or DELETE, and they don’t support sending JSON. This means that if you use anything but a GET or POST request with URL-encoded data, it can only work once WASM has loaded. As we’ll see [in a later chapter](https://leptos-rs.github.io/leptos/progressive_enhancement), this isn’t always a great idea.

    The CBOR encoding is suported for historical reasons; an earlier version of server functions used a URL encoding that didn’t support nested objects like structs or vectors as server function arguments, which CBOR did. But note that the CBOR forms encounter the same issue as PUT, DELETE, or JSON: they do not degrade gracefully if the WASM version of your app is not available.

I understand that WASM takes a while to load, I'm following a certain.. trend.. and have been looking into using leptos and axum with htmx. Htmx as a javascript library can possibly be embedded into an app and load faster than wasm, potentially allowing for form puts and deletes.
https://htmx.org/docs/#ajax
I'm not sure the stack is entirely the best idea, but as it's setup I'll be having to make my own delete endpoints separately from leptos to fully be able to build-out with htmx.
Maybe I'm really asking for an hxPut, hxPatch, and hxDelete encoding to be built in and autoload the js for me as-needed. ;-P
I would even wish for ActionForms that could hxDelete with a fallback to a post path that handled deletes or even if it just did nothing/showed-a-css-validation-error if the js didn't load somehow, but that starts getting messy. A lot can be integrated at the project level, but I think there might be some use for a leptos htmx cargo feature or something that adds some traits / features and a way to embed the htmx library automatically.

I took a look at htmx myself some time ago and while is is an interesting approach on how to handle frontend building, I don't see a reason why I would would use it together with leptos. You simply just don't need it and I think it makes your stack over complicated and does not provide a real benefit.
I would either use just axum + httpx + askama (very important to have a good templating engine with it), or simply just leptos. And yes wasm takes some time to load, but it "feels" a lot faster, since it can do a lot on the server already before the hydration happens on the client side.
I tried both above mentioned solutions and I prefer "just leptos" a lot.
Btw you can do PUT and DELETE with htmx too, just use POST endpoints for that purpose. Basically, POST for everything that does any modification in the backend or should not be cached by a CDN or something, GET otherwise.

See #1159 for additional discussion.
If you want to integrate HTMX and use other HTTP verbs it's perfectly possible to do that by creating PUT, DELETE, etc routes in Actix/Axum and using HTMX to make requests to them.
I will not include this in Leptos itself unless the HTML spec itself one day expands to include forms or other HTML elements that make requests with verbs other than GET or POST.
From my perspective this is quite simple: graceful degradation. As far as I can tell the HTMX approach here simply fails, silently, if HTMX's JS has not loaded for any reason (JS is off, device doesn't support it, you are on a subway and lost connectivity between receiving HTML and receiving JS, etc.)
I just don't see the reason to make it easier for people to ship fragile apps, in exchange for more of a maintenance burden and additional runtime complexity/binary size. Sounds like a lose-lose-lose to me.


Crashes on wasm_import_memory modules

wasm-gc panics with no memory section when run on a module that has:
#![feature(wasm_import_memory)]
#![wasm_import_memory]

Such a module imports memory as env.memory (eg (import "env" "memory" (memory $env.memory 17))) instead of exporting it (eg (memory $memory (export "memory") 17)). It manifests as an ImportEntry in import_sections: ImportEntry { module_str: "env", field_str: "memory", external: Memory(MemoryType(ResizableLimits { initial: 17, maximum: None })) }

Heh yeah I've run into this from time to time and just haven't had a chance to fix it yet! I suspect there's just a stray unwrap somewhere or something like that which should be easy enough to remove.

The unwrap location is not a mystery - I linked it in the OP. The issue is that the code assumes in a few different ways that memory is an export and all imports are functions.
For example LiveContext::add_function assumes any index that belongs to an import entry is an imported function, even when the index points to an imported memory and is actually the index of an internal function (should compare against ImportSection.functions() instead of ImportSection.entries().len(), which requires bumping the parity-wasm dep in Cargo.lock).
I'm fixing these as I go along.

Oh awesome! I'd happily take a look at a PR to fix this up :)


JDP Remediation - CSOM

Adding JDP Remediation - CSOM to the github

thx. Let's get this merged right away. This was missing still readme.md file, but we can agree who will do that later. It did had docx for the additional documentation, but readme.md is always needed to ensure that we have proper descriptions on how to use the code.


Text lost in IE9

Please help to check this PDF: https://drive.google.com/file/d/0B5sTqArpFYNmY2tTM3BxQUo3RUU/view?usp=sharing
All the text lost in IE9. But it works fine in IE10 or FireFox.

Thanks!

Snuffleupagus, do we have plan to fix this IE9 issue? My application is in IE9 compatible mode, so it is really an issue for us. Thanks1

IE9 is not supported anymore in PDF.js 2.0, so I'm closing this issue as won't fix.


Shortest path is not the shortest

func TestShortestPath(t *testing.T) {
	g := graph.New(graph.StringHash, graph.Weighted(), graph.Directed())

	g.AddVertex("start")
	g.AddVertex("a")
	g.AddVertex("b")
	g.AddVertex("fin")

	g.AddEdge("start", "a", graph.EdgeWeight(6))
	g.AddEdge("start", "b", graph.EdgeWeight(2))
	g.AddEdge("a", "fin", graph.EdgeWeight(1))
	g.AddEdge("b", "a", graph.EdgeWeight(3))
	g.AddEdge("b", "fin", graph.EdgeWeight(5))

	path, _ := graph.ShortestPath(g, "start", "fin")

	wanted := []string{"start", "b", "a", "fin"}
	if !reflect.DeepEqual(path, wanted) {
		t.Errorf("shortest path failed, result: %v, wanted: %v", path, wanted)
	}
}

shortest path failed, result: [start b fin], wanted: [start b a fin]

Nice catch, thank you.

The fix has been released in graph v0.16.2.

The fix has been released in v0.16.2.


Error: invalid index file

It appears that somewhere in the mapnik-3.4 series the index file format has changed. We're unable to use .indexes that were created with shapeindex-0.3.0 using mapnik-3.4.16 (https://github.com/mapnik/node-mapnik-sample-code/issues/6 shows it happening in 3.4.15 as well). We were previously, but I'm not sure how far back at the moment.
We managed to solve it by using the shapeindex alternative that now ships with node-mapnik to reindex: node_modules/.bin/mapnik-shapeindex.js

I'm sorry about this change and not documenting it well. I've added details now at https://github.com/mapnik/node-mapnik/blob/master/CHANGELOG.md#3413

We managed to solve it by using the shapeindex alternative that now ships with node-mapnik to reindex: node_modules/.bin/mapnik-shapeindex.js

That is the intended upgrade process/workaround. In other words, when you upgrade to node-mapnik >= 3.4.13 then you must regenerate .index files. The benefit is then you can use the experimental --index-parts option which can lead to faster read for polygons with many geometry parts that are outside the query extents.


The website provided is no longer accessible

The website provided in the Readme file: http://siis.cse.psu.edu/ic3/, has become inaccessible. I was unable to get more information about this tool.

You can access it through Wayback Machine: https://web.archive.org/web/20190401232754/http://siis.cse.psu.edu/ic3/index.html#banner

Thanks


How to have a bundle specific loader configuration?

I need to make two versions of a bundle using different loaders.
Is there a way to define bundle specific loaders?
The only way I can think of that this can be done at the moment is two have two separate webpack configurations and run webpack build twice. This is far from ideal since this duplicates compilation of all assets.

As far as I understand, nope. You could do module.exports = [{...}, {...}] and then use a utility like webpack-merge or similar to manage duplication.
Once the configuration is in a format like that, you can use parallel-webpack to speed up execution. There will be overhead, but at least you'll get multiprocessing.


Look at doing tract tracing in original space

We talked about this forever ago...I bet we could try this really quick now.  (Basically just need to apply registration to each point of each tract in original space.)
Sephira brought this up in the talk today.  I think essential for m2g, and important and fun.

There is the issue of transforming the tracts,  rather than the image.
It is perhaps non trivisl
Another option is doing tensor estimation in the original space.
Then we have to morph the tensor.
Rene has some work on this.
This seems potentially useful,  to explore once we have preliminary results
on the other stuff?
On Aug 21, 2015 12:45 PM, "William Gray"<EMAIL_ADDRESS>wrote:

We talked about this forever ago...I bet we could try this really quick
now. (Basically just need to apply registration to each point of each tract
in original space.)
Sephira brought this up in the talk today. I think essential for m2g, and
important and fun.
—
Reply to this email directly or view it on GitHub
https://github.com/openconnectome/m2g/issues/121.

migrated


feat: add isEnterpriseInstall to Context

Summary
#1510 - Add isEnterpriseInstall to aid in processing events.
Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

I've just seen that events have access to body.authorizations[0].is_enterprise_install and actions have access to body.is_enterprise_install.  However, given that Bolt has already parsed this information I think it'd be nice to have it available on context :)

@rockingskier looks like some of the tests are failing, I believe because some unrelated tests are using empty mock objects for the context property, but due to this PR, context now requires the isEnterpriseInstall property in it. Do you think you can update these so that the tests pass?

Thanks, @filmaj.  That was embarrassingly obvious :D
All fixed up and working locally now.

Thank you very much @rockingskier ! Much appreciated 🙇


File List In FilePicker Not Scrollable

Summary
The file list is not scrollable initially. The user needs to click inside it to activate the scrolling behavior.

Might need to reopen this one. It seems that this fix only works when the user has clicked or used the mouse wheel before.
One might even ask if this even was the default behavior and the fix didn't do anything at all...

No, it's fine. The behavior is different from the current live version. The scroll pane just needs to have an inital user input to work.

Sometimes getting a NRE for getStage() though. Reproduction is hard.

Debug output: FilePicker::show.
From Left To Right:

[Enter] Stage@53b8d12e, From Actor: FilePicker$9
[Exit] Stage@53b8d12e, To Actor: List
[Enter] Stage@53b8d12e, From Actor: ScrollPane
[Exit] Stage@53b8d12e: To Actor: ScrollPane

From Bottom To Top:

[Enter] Stage@53b8d12e, From Actor: List
[Exit] Stage@53b8d12e, To Actor: FilePicker$9
[Enter] Stage@53b8d12e, From Actor: FilePicker$9
[Exit] Stage@53b8d12e, To Actor: FilePicker$9


Quality Assurance Improvement

Goal

Set accessibility identifier of AVFoundationPlayback to identify when a video is playing in UI tests;
Rename LoadingContainerPlugin to SpinnerPlugin to follow plugin name pattern;
Hide SpinnerPlugin view instead of only make spinning wheel disappear;
Increase test coverage of SpinnerPlugin;

How to test
Run all tests.

 [Nova UI] Novo spinner

Hi @leandroalonso , I fixed the problem you commented about.


Better hxCodec versions compatibility

This mainly focus in helping any hxCodec  related scripts work on any psych engine build with different hxCodec versions easily
And it allows you to use VideoSprite just fine with hscript and runHaxeCode(beffor i made this i couldn't call the playVideo function so ig this fixed it)
I tested this with 2.6.0 I'm not sure about 3.0 compatibility and above...

If you see any dumb typos, misspelling or shitty code please know I'm still a beginner in flixel :)

and pls do return of video instance from startVideo

ehh i feel like this is pretty unnessecary
also you can't access VideoSprite currently due to it not being used in the source code, not a psych engine issue

ehh i feel like this is pretty unnecessary also you can't access VideoSprite currently due to it not being used in the source code, not a psych engine issue

nah lol, it used in set var in hscript

ehh i feel like this is pretty unnecessary also you can't access VideoSprite currently due to it not being used in the source code, not a psych engine issue

nah lol, it used in set var in hscript

not in the official src

i mean in playstate

i mean in playstate

error: Void should be backend.VideoHandler
error: backend.VideoHandler should be Void

replacereturn; with return null;`

ye like this

and i suggested change upper

Upstream?

where TheLeerName started a review

Btw when #13304 is merged i will replace set() with setVar()

here's a example of a VideoSprite in HScript

one of the only things that should be done out is adding PlayBackRate support for  VideoSprites which i have no idea how to do it

now fuck off

NONONONONONONONO

bro put his port in there💀💀💀💀💀💀💀💀💀💀💀💀

Bahahah, what the hell is that?


  This PR adds logic to test for both the readability of the pidfile and that its content is a positive integer.
Jenkins was still failing when the pidfile didn't exist: http://jenkins.corp.apptentive.com/job/Platform_Parameterized_Build/991/console

It should do nothing:
$ irb
irb(main):001:0 > require 'fileutils'
  true
irb(main):002:0 > FileUtils.rm_f('not-a-file')
  ["not-a-file"]
irb(main):003:0 > $?
  nil

rm_f(s) == rm(s, force: true) force true suppresses errors http://ruby-doc.org/stdlib-2.2.2/libdoc/fileutils/rdoc/FileUtils.html#method-c-rm

:+1:


NoReverseMatch at /admin/plans/userplan/19/

I'm using django-suit and I'm getting this error when I try to modify any user's plan:
Reverse for 'auth_user_change' with arguments '(3,)' and keyword arguments '{}' not found

The user exists, the plan exists, etc. The bug, if any, seems to come from this line in django-plans code. Any ideas on how I could adapt this to work with django-suit? A workaround would be to remove that feature... but wouldn't like to do it that way.

Closing due to inactivity


CyclomaticComplexity and "default" switch Case

I believe that just like else blocks, the default case in switch should not be added to the CCN. In http://phpmd.org/rules/codesize.html#cyclomaticcomplexity it should have a CCN of 11 without counting the the default case.

function foo($num)
{
    switch ($num) {
        case 1:
            echo 1;
            break;
        /*...*/
        case 10:
            echo 10;
            break;
        default:
            echo 'aa';
    }
}

Gives me the same complexity (11) than:
function foo($num)
{
    switch ($num) {
        case 1:
            echo 1;
            break;
        /*...*/
        case 10:
            echo 10;
            break;
    }
}

So this has been fixed yet.


Feature request: narrow:select file

Hello @t9md, just found narrow and it's awesome as all your projects. Thanks
It would be awesome if narrow can be used as file selector, like Ctrl-p and unite. What do you think?

Narrow already have narrow:select-file but this for different purpose.
Used for filter items on narrow-editor by filePath basis.

You can start narrow:select-file by s key if you are vim-mode-plus user
Or click folder icon on control-bar of narrow-ui

For your request, I don't want enhance narrow as open everything package.
I'm intentionally limiting the area narrow cover.
I defined narrow as code-navigation tool.
This is design decision.
Why not use Atom's fuzzy-finder?

I'm using fuzzy-finder, and advanced-open-file, and now narrow. And all of them has different interface, it means I need to do different actions to achieve same result: open file I'm thinking about. Less interfaces - less thinking.
I saw select-file mode and I thought it would be great to reuse it for opening files by name.
I appreciate your design decision.

Thanks for understanding.
I understand your motivation, and agree that unified interface to interact is better than many different interfaces.
But I intentionally make narrow purpose-limiting to make primal function(= code navigation) further better.
This issue is memo at very early stage of narrow.
But I changed my mind.
https://github.com/t9md/atom-narrow/issues/21


In Mobile product showing only on click

Bagisto Version: Master
Steps To reproduce--
1. Open shop on mobile view
Expected result: The products should be visible without clicking on the screen on mobile.
Actual result: In Mobile product showing only on click

@jitendra-webkul
Now, this issue is fixed. so I am closing this issue.


chore: update shiki

Closes #180

Sorry of the late accepting of CI, did this build when you tested it?

Thanks for giving this a shot, I've just ran through it locally and shipped the new versions!


Devise: session not invalited serverside

I am trying to follow the official way to use devise.
The problem I see is that after calling 'invalidateSession' you are properly logged out even after refreshing the page - clientside.
But...if you try then to login with random user & password '/sign_in' will return your user data, just like if you were still logged in. And log you in.
EDIT:
Soo...I see that readme mentions this problem.
But disabling cookies is not a proper fix.
https://github.com/plataformatec/devise/blob/master/app/controllers/devise/sessions_controller.rb#L16
We just need to override more, adding store: false to authenticate() call

Actually....here's my fully working solution. README's update would be welcome
class SessionsController < Devise::SessionsController
  respond_to :html, :json

  def create
    super do |user|
      if request.format.json?
        data = {
          token: user.authentication_token,
          email: user.email
        }
        render json: data, status: 201 and return
      end
    end
  end

  def auth_options
    super.merge! store: false
  end
end

I think that should be
def auth_options
  if request.format.json?
    super.merge(store: false)
  else
    super
  end
end

Can you verify that works and submit a pull request?

@marcoow no, actually  the
  before_filter :authenticate_user!

part needs to be removed too together with all other stuff relating to html

@ninja- I'm pretty sure the only thing you need to make sure is that no cookie gets set for JSON requests.

@ninja- any updates?

@marcoow my setup works great for me. are you asking about PR for documentation's update or something else?

I noticed one thing, without cookies the session doesn't stay  after page refresh. How to solve it?

I remmember a similar discussion here https://github.com/simplabs/ember-simple-auth/issues/201 and here https://github.com/simplabs/ember-simple-auth/issues/204

@marcoow no, actually the
before_filter :authenticate_user!
part needs to be removed too together with all other stuff relating to html

if that's the case it probably means that your server is responding with a token for JSON session creation requests as well as issuing a session cookie which it shouldn't. Please check whether after logging in you have a Rails session cookie in the browser.

@marcoow, what are you referring to? Yes I do indeed have a session cookie, but I am using session for other stuff. The 'logged in' status is not saved in the session.

@ninja-: I see. However, you don't actually want to use a session for REST APIs as you want that to be stateless. Also if authenticate_user! redirects users that are not logged in to a specific page also for JSON requests that's actually wrong. You should instead only do that for HTML requests.

For logout, why not just submit a DELETE /logout?
I'm currently dealing with a Spree implementation. the Admin requires the token to be stored. No if, maybes, or buts. I've already tried.
If you do the DELETE /logout, then you're left with one last issue: refreshing the page and discovering you've been "logged out", when you're not...
How does devise store the session? I believe its a cookie. What if we searched instead for the applications session cookie instead of hacking around to set an authentication token.
I'm willing to work with the current workflow, but the application I'm working on is a Spree app. I currently do not have the luxury to manipulate the Spree::UserSessionsController or the Devise::SessionsController even to the level that Ember Simple Auth requires of me.

@BenMorganIO: if you have a server session that's stored in a cookie you don't actually need most of ESA (especially the authorizer that adds the Authorization header etc.) as the cookie is sent with each request anyway.
Usually when implementing a REST backend you don't want to use a session cookie though as that adds state to the API. When using token based authorization instead for logging out it's sufficient to delete the token client side as that effectively logs the user out as no more authorized requests can be made as the authorization secret isn't known anymore.
What you and @ninja- seem to be trying to do is to make Ember Simple Auth work with a classic server session based backend which is just not what the library is built for. Of course you can extend the authenticators etc. to make it work but essentially the library is built to work with stateless (= no session cookie) REST backends.

@marcoow merci. This makes sense.
I don't think I have enough Ember knowledge to roll my own Devise + Spree + Ember library. You wouldn't happen to be able to provide a simple example?

@BenMorganIO: check out the examples folder - there's a very simple example for the devise package as well. Basically the one thing you have to make sure on the server side is that you don't create a session cookie (and no session in general) for any API requests and also only use the Authorization header to authorize API users.

I didn't manage to make any of the solutions above work on my side: the session just keeps being valid. I'm converting to Ember a Rails app where I was heavily using the session, and I don't want to get rid of authenticate_user! hooks at this stage.
What i did was adding a before_filter to sign out the user when the Authentication header is not correct. This seems to work well at this stage. +1 for better solutions to ember-simple-auth for making sure that devise doesn't persist the current user in the session (other than just getting rid of the whole session).
class ApplicationController < ActionController::Base
  before_filter :authenticate
  before_filter :authenticate_user!

  (..)

  private

    def authenticate
      authenticate_user_from_token || render_unauthorized
    end

    def authenticate_user_from_token
        authenticate_with_http_token do |token, options|
          user_email = options[:email].presence
          user2= user_email && User.find_by_email(user_email)

          if user && Devise.secure_compare(user.authentication_token, token)
            sign_in user, store: false
            @current_user=user #for devise current_user helper
            return true
          end
        end
      end
      def render_unauthorized
        sign_out(current_user) if !current_user.blank?
      end
    end
end


Our Nvidia Driver is CUDA9.0

Hello,
Thanks a lot for the excellent job from Princeton-VL, our system is based on the Nvidia-GPU driver of CUDA-9.0.
Can I just directly try the open source code on my system with the configuration file change.
Or should I upgrade my driver?
Thanks a lot!

I only tested it with CUDA10 so I am not sure if it would run on CUDA9. I would recommend upgrading your driver if possible.

Hello. Did you successfully reproduce it？I also used cuda 9.0 ，but it made a mistake.

@yinyuhangi  @momo1986  I have run the project on CUDA9 correctly with the following steps:

conda create --name CornerNet_Lite --file conda_packagelist.txt --channel pytorch
source activate CornerNet_Lite
conda uninstall pytorch  cuda100
conda install pytorch=1.0.0 torchvision -c pytorch
perfom the 4th step will install  the following modules.
pytorch            pytorch/linux-64::pytorch-1.0.0-py3.7_cuda9.0.176_cudnn7.4.1_1
torchvision        pytorch/noarch::torchvision-0.2.2-py_3
and you can run python demo.py to check !
hope to help you!

@chiminghui Thank you very much. I have run the project on cuda 10.1 correctly.

Thanks a lot!!!


BugFix: Use one of the loaded keys.

It seems after a recent change none of the loaded keys are being used. I have been getting the error:
No authentication methods available
Traceback (most recent call last):
  File "C:\Users\paverma\AppData\Local\Continuum\Miniconda3\lib\site-packages\azure\cli\main.py", line 36, in main
    cmd_result = APPLICATION.execute(args)
  File "C:\Users\paverma\AppData\Local\Continuum\Miniconda3\lib\site-packages\azure\cli\core\application.py", line 201, in execute
    result = expanded_arg.func(params)
  File "C:\Users\paverma\AppData\Local\Continuum\Miniconda3\lib\site-packages\azure\cli\core\commands\__init__.py", line 417, in _execute_command
    reraise(*sys.exc_info())
  File "C:\Users\paverma\AppData\Local\Continuum\Miniconda3\lib\site-packages\six.py", line 686, in reraise
    raise value
  File "C:\Users\paverma\AppData\Local\Continuum\Miniconda3\lib\site-packages\azure\cli\core\commands\__init__.py", line 399, in _execute_command
    result = op(client, **kwargs) if client else op(**kwargs)
  File "C:\Users\paverma\AppData\Local\Continuum\Miniconda3\lib\site-packages\azure\cli\command_modules\acs\custom.py", line 690, in k8s_get_credentials
    _k8s_get_credentials_internal(name, acs_info, path, ssh_key_file)
  File "C:\Users\paverma\AppData\Local\Continuum\Miniconda3\lib\site-packages\azure\cli\command_modules\acs\custom.py", line 711, in _k8s_get_credentials_internal
    '.kube/config', path_candidate, key_filename=ssh_key_file)
  File "C:\Users\paverma\AppData\Local\Continuum\Miniconda3\lib\site-packages\azure\cli\command_modules\acs\acs_client.py", line 47, in SecureCopy
    ssh.connect(host, username=user, pkey=pkey)
  File "C:\Users\paverma\AppData\Local\Continuum\Miniconda3\lib\site-packages\paramiko\client.py", line 381, in connect
    look_for_keys, gss_auth, gss_kex, gss_deleg_creds, gss_host)
  File "C:\Users\paverma\AppData\Local\Continuum\Miniconda3\lib\site-packages\paramiko\client.py", line 623, in _auth
    raise SSHException('No authentication methods available')
paramiko.ssh_exception.SSHException: No authentication methods available

@pareshverma91,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Codecov Report

Merging #3102 into master will increase coverage by 0.04%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #3102      +/-   ##
==========================================
+ Coverage   70.75%   70.79%   +0.04%
==========================================
  Files         358      358
  Lines       22951    22996      +45
  Branches     3529     3531       +2
==========================================
+ Hits        16238    16280      +42
- Misses       5617     5620       +3
  Partials     1096     1096

Impacted Files
Coverage Δ

...li-acs/azure/cli/command_modules/acs/acs_client.py
50.45% <0%> (+0.45%)
:arrow_up:

...-cli-lab/azure/cli/command_modules/lab/commands.py
90.37% <0%> (+0.48%)
:arrow_up:

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 a6b8fd8...e283e4c. Read the comment docs.

LGTM.


Add min css file

For faster than

Thanks! We will track minification in #125 so I'll close this.


Failure on tone tag --meta-remove-property recordingdate file.m4b

The following command does not work:
tone tag --meta-remove-property recordingdate file.m4b

Probably a Problem with atldotnet or a misuse of the internal API

This has to do with #17
Something is fishy with Track.RecordingDate and with Track.Description when writing or removing mp4 (m4a, m4b)

I've tested --meta-remove-property for several other properties like album & description for mp3/m4b and it doesn't work.
tone tag --meta-remove-property="album" file.m4b

output shows it was removed
tone dump file.m4b

album property remains unchanged

You could also allow setting meta tags to empty strings.

Yeah unfortunately I found multiple bugs in spectre.console (my console library for C#), reported them and was mostly not responding, which frustrated me a bit... recently someone else also thought that these issues should be fixed and that there is a major misbehaviour in the argument parsing, which I agree with.
So, until these issues are all fixed and the argument parsing works as expected, it is unreliable to use command line parameters in some edge cases. That's why I invented the tone.json format, which works reliably in all cases I've tested. Using tone.json you can also try to set album: null, which should delete the field (but that is something I did not fully test yet).
Nevertheless, that issue is definitely gonna be paid attention in the the very near future...

What about the --meta-remove-property? The spectre.console issues are not related to that bug right?

What about the --meta-remove-property? The spectre.console issues are not related to that bug right?

Correct. I'll try to fix this next release. same for #32

Mmh I added a new issue describing the problem in atldotnet (see above). The problem is, that I did not find a way to really remove the property in every case, but only setting it to an empty value in case of string.
I implemented the set to empty, but I think it would be more accurate, if remove really means remove. Would you mind if I wait with the next release until I get response?

No problem, thanks. The other one that is helpful for using in abs is #29


[Backport release-22.05] adl: init at 3.0.1

(cherry picked from commit a885d43d61a501ffc777809a70ca55c1ce4f29d3)
Description of changes
Backport of #179942

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 180406 run on x86_64-linux 1

1 package built:

adl


⚠ Processors has degraded performance

In 6ce6a35, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 59 ms

Resolved: Processors performance has improved in e840131.


adapter-node env.path config is ignored

Describe the bug
Feels free to point it out if I understand something wrong. https://github.com/sveltejs/kit/pull/2048 added the support for adapter-node to listen with path. However, because the port still defaults to be 3000 now. The path option would be ignored by node.js.
https://github.com/sveltejs/kit/blob/9080b2aa92ae4617e0dbd1489e5d6b3e57cb7db4/packages/adapter-node/index.js#L65
the overload currently using is the options object one:
https://github.com/sveltejs/kit/blob/9080b2aa92ae4617e0dbd1489e5d6b3e57cb7db4/packages/adapter-node/src/index.js#L11
As specified in the node.js docs when the port option is specified, the path option would be ignored. Which means it would listen on port 3000 instead of the specified path. Moreover, it would break existing apps using the PORT environment variable to listening with path. Because in this overload the option.port option would be verified if it's actually a port instead of a path.
The hosting service I am using uses a proxy approach and it uses the PORT variable to pass the named pipe. So honestly, I don't even know the difference between path and port until it breaks. I tried changing the config to let the node adapter to read the PORT variable as a path but it doesn't work.
Reproduction
use a svelte kit app and change the config to this.
// svelte.config.js
import adapt from '@sveltejs/adapter-node';
export default {
  kit: {
       adapter: adapt({
            env: {
                path: 'SOME_PATH'
            }
        })
  }
}

build the app and run `SOME_PATH=foo node build. Go to localhost:3000 in the browser.
Logs
No response
System Info
System:
    OS: Windows 10 10.0.19043
    CPU: (16) x64 AMD Ryzen 7 4800HS with Radeon Graphics

    Memory: 4.37 GB / 15.42 GB
  Binaries:
    Node: 16.2.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.10 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 7.13.0 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.19041.1023.0), Chromium (92.0.902.73)    Internet Explorer: 11.0.19041.906
  npmPackages:
    @sveltejs/adapter-node: next => 1.0.0-next.42
    @sveltejs/kit: next => 1.0.0-next.151
    svelte: ^3.34.0 => 3.38.2

Severity
blocking an upgrade
Additional Information
No response

Should we just change const listenOpts = { path, host, port }; to const listenOpts = path ? { path } : { host, port };?


As an author, I want to delete my own posts.

There is an endpoint implemented and tested for deleting posts. This deletion does not check any authentication credentials. I am pushing back this work item as a result.

Added check to ensure the user who is making the request is the user associated with Author who created the post. This means that if someone from a remote server creates a post on our server they won't be able to delete it. I am not sure how else to implement this otherwise though, because when you make a delete, you are just sending the id of the post. From the request I am able to get the django user, but for remote servers, that user is just the user we made for them to connect with our server, not the user of the Author who is trying to delete stuff.
Will ask other group.

Other groups are ignoring this. I am closing this.


Avoiding overwriting andWhere()/orWhere() with where() in ActiveQuery when scoping data

Although the result of using where() after andWhere() or orWhere() is known and documented, still it's one of the vulnerable areas that could easily fall through the cracks especially by new team members. It's easy to use where() in a child class to result in removing user scoping in the parent class and expose sensitive data.
To enhance this area, I'm suggesting the introduction of new method called alwaysWhere() (or fixedWhere() or scopeWhere() or mustWhere() ...etc) with the same signature of where() but its condition is not affected by where(), andWhere(), or orWhere(). Its condition would always be added to the WHERE condition in the resulting SQL statement. Taking this further, a parameter like accumulate could be used ( where($condition, $accumulate = true) ) so by default, any additional use of alwaysWhere() in the application will lead to adding the new condition to the old one unless accumulate is explicitly set to false by the developer to overwrite the scoping condition.

I don't get need of that. Could you provide an example with some data?

For example, you could have a base class BaseModel that scopes all records by current user like where(['user' => Yii::$app->user->id]) or andWhere(['user' => Yii::$app->user->id]). You'd expect that any child class like Messages, Invoices, ...etc, to be scoped by the current user. If during execution at any point after scoping where() is used, user scoping would be lost. This is not a far-fetched scenario especially with new programmers who know nothing (yet) about the base class.
While where() would overwrite other where(), andWhere(), and orWhere() conditions too like andWhere(['read => false]) to show only new messages, such bug would result in showing both read and unread user messages instead of read only. A error yes but not disastrous. But imagine the same thing happening with user scoping condition in an email service or invoicing system you are building, user scoping would be lost and users would see each others' messages or invoices, this would be a disaster.
That's why a top level WHERE condition (e.g. fixedWhere()) not affected by and/or/where() during execution would provide a kind of safe zone or an extra safety layer to avoid such scenarios, making the framework safer in handling such scenarios. Of course fixedWhere() could be used again later during execution but I assume who does that would know what exactly he's doing and why. Furthermore, if the default behavior of fixedWhere() is accumulative (adding conditions instead of replacing them) unless forced to replace via an option as explained above, there still could be bugs but not as disastrous. For example no invoices displayed at all instead of mixing up users' invoices.
Finally, I see where() as a possible problem area as it could wipe out all previous conditions. Of course such function is required but with complex structures and inheritance, I'd prefer if it is handled in a way that makes using it less prone to errors.


MM-20264 - Fixing youtube crop in RHS

Summary
MM-20264 - Fixing youtube crop in RHS
Ticket Link
https://mattermost.atlassian.net/browse/MM-20264

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-4281.test.mattermost.cloud

Account Type
Username
Password

Admin
sysadmin
Sys@dmin123

User
user-1
User-1@123

Test server destroyed


Remove no longer existing OCI registry link to bundle.bar

LGTM. For these cases I'm also ok removing it only on next/.

Yes, it felt like re-writing history. I couldn't work out when bundle.bar ceased to be which may have provided more guidance. Anyway ... it improves the accessibility/quality score that's getting generated for the product docs.


Fix with expressions using tuple destructuring.

A with expression like the following caused the compiler to abort:
with (a, b) = some_tuple do
   error
end

This PR fixes those cases and adds some test coverage for desugaring with expressions in general.
Fixes #3581

@jemc @kulibali I don't feel qualified to review this one, can either of you give it a look?


add events.json checker

some suggestion on Split event.json up into days #26

single events.json look good
add a events.json checker to verify it as a good JSON file

anther way to keep events.json looks good is to check it by json_pp command
in Linux or macOS
$ cat events.json | json_pp

which could format the JSON file nicely and also check the problem

I think your right this is better way to do it. and because we want to keep support for windows id rather do it in node

The reason why we want to split it into days is to avoid the problems we had last year where we were deleting days as they went on (we didn't show Monday timetable on Tuesday) and we could either do it on the front-end or by removing days from the events.json file. Due to lack of time we were doing it the bad way of removing days. Splitting the events into days would add it a bit more structure.

Ive since added collapsed cards no not need removing it and just added a pr
for scema checking so I don't think this will be needed anymore
On Sun 2 Oct 2016 at 18:18, Wojtek Bednarzak<EMAIL_ADDRESS>wrote:

The reason why we want to split it into days is to avoid the problems we
had last year where we were deleting days as they went on (we didn't show
Monday timetable on Tuesday) and we could either do it on the front-end or
by removing days from the events.json file. Due to lack of time we were
doing it the bad way of removing days. Splitting the events into days would
add it a bit more structure.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/redbrick/techweek.dcu.ie/pull/29#issuecomment-250982788,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0-WkqlS2veVGR1EzuGrVTxcdEvBx7Lks5qv-d3gaJpZM4KL7VM
.

--
Cian Butler (butlerx)
Redbrick Secretary 2016-2017
Computer Applications and Software Engineering - Year 3

@VoyTechnology how about add an attribute, to indicate its visibility, e.g. show: true


Potential panic on unterminated CTCP messages.

I had a bot I wrote using this lib crash the other day with a slice bounds out of range error at irc_callback.go: 84.  Sadly, I didn't have verbosity on at the time, so I wasn't able to see the message that generated it, but I suspect it was something that started with \0x01 and did contain another \0x01.  This would mean that the slice would evaluate to
msg = msg[1:0]

Which panics.  I'll update the bug if I see it again, but it might be worth adding a check.

Thanks :+1:


spkl: Support for removing plugin steps not in the plugin assembly

Optionally allow removing of steps that are not defined in the plugin assembly. This will need to be done before the new plugin is deployed otherwise the deploy will fail.

@scottdurow - I just added a PR for this

@scottdurow  I think what you are after here is more the PluginType, not the step. The issue comes up when you remove a class from the plugin assembly and try to "update" it as it errors out when there is a PluginType registered that is not present on the new assembly. It would also be good to have an option to remove the entire Assembly.
Syncing the steps & images to what is present in the metadata tags of the updated Plugin is already in the PR I submitted.

See additional logic added to my PR. It now removes the orphaned PluginType when you have removed a class from the assembly to allow the Update of the assembly to work.


Undefined Control Sequence \theoremstyle

Description

I get the following error when building with kaobook:
! Undefined control sequence.
l.563     \theoremstyle
                       {plain}

This comes from the newpxtext package. It seems to happen whether or not I load kaotheorems.
I'm not totally convinced that this is necessarily a kaobook error, as it looks like newpxtext was updated pretty recently and it could be that it broke something with amsthm, but nothing I've tried to debug it works or really helps narrow down the issue at all so I'd appreciate some help on that. Also I'm not sure where to get an older version of newpxtext and there isnt an easy way to downgrade it using arch linux.
I'm using the texlive version from arch linux which is 2021 and was updated on the 27th presumably this includes the latest newpxtext which was updated on the 25th.
Minimal Working Example
The minimal_book example throws this error for me, I can put together an even minimaler one if you want I compile it with pdflatex -output-directory examples/minimal_book main.tex

The following minimal example without kaobook does work BTW:
\documentclass{article}

\usepackage{amsthm}
\usepackage[scaled=.97,helvratio=.93,p,theoremfont]{newpxtext}

\begin{document}

Name, date, Exercise X

\section*{Part a}

Put your answer to part a here

\section*{Part b}

etc

\end{document}

so I don't think it is necessarily newpxtext that is broken here

Have a look at https://github.com/fmarotta/kaobook/issues/193#issuecomment-992328930 and the following discussion

Yes I was able to fix it by following option (2) and loading amsmath in kao.sty thanks for the pointer, I should have searched the issues more thoroughly before opening a new one


i have idea

can you add max health control in world ?
p.s. sorry for my english. I am Russian

Do you mean to set the health higher than 20 hearts? I don't thing that's belongs in a world manage plugin.


Module build failed: Error: ENOENT webpack/buildin/global.js

Laravel Mix Version: 6.0.27
Node Version: 14.17.5
NPM Version: 6.14.14
OS: CentOS

Description:
Uncaught Error: Module build failed: Error: ENOENT: no such file or directory, open '/home/company/laravel-vue-crud-starter/node_modules/webpack/buildin/global.js'
at eval (webpack-internal:///./node_modules/webpack/buildin/global.js:1)
at Object../node_modules/webpack/buildin/global.js (app.js:6924)
at webpack_require (app.js:20)
at eval (webpack-internal:///./node_modules/vue/dist/vue.common.dev.js:12010)
at Object../node_modules/vue/dist/vue.common.dev.js (app.js:6867)
at webpack_require (app.js:20)
at eval (webpack-internal:///./node_modules/vue/dist/vue.common.js:2)
at Object../node_modules/vue/dist/vue.common.js (app.js:6878)
at webpack_require (app.js:20)
at eval (webpack-internal:///./resources/js/app.js:2)
Steps To Reproduce:
When initially compiling with npm run watch it works. However, after a couple saves it starts throwing ENOENT errors.

Can you provide a reproduction repo?

I'm facing this issue now  how to solve this?

I'm facing this issue now how to solve this?

how did you resolve this problem?


Unittest: support emptypoddirs

We expect quite a few components to have no pod files anymore (and thus empty target/doc/pod/).
This makes it easy for the 00-tqu test to check this, via
use Test::Quattor::Unittest qw(nopod);

@jrha will be necessary foir quite a few components to avoid unnecessary complicated code in 00-tqu.
https://github.com/quattor/configuration-modules-core/pull/1024 now uses this (and 1.52 buildtools 😁 )


improve UI Tests stability

click on status bar item with retry
assert there is no quarantineProperties
make move alert independent of preferred applications folder
refactor MoveAppHelper to use preferred applications folder
fix test timeout waiting for app to be ready for relaunch

Codecov Report

Merging #14 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #14   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            6         6
  Lines           74        74
=========================================
  Hits            74        74

Flag
Coverage Δ

#AppSpecs
100.00% <ø> (ø)

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 f3968af...4a477ef. Read the comment docs.


propagate errors out of zebra_network::init

fixes #408

What's the difference between the InitHandle and the impl Service? I mean, I know that the purpose of the InitHandle is to catch errors from background tasks, but as a user of the API, why do I have to manage another object and try to wait on it?  How does it relate to the impl Service I get when I call init and in particular its poll_ready implementation?
We should definitely propagate errors out of the background tasks of zebra_network, but I'm wondering if there's a way to put the error handling inside the poll_ready implementation, which callers are already required to call before making service requests.  This way an error in one of the service's background tasks would be reported as a failure of the service itself. WDYT?

I think that sounds like a good idea, I'll see what I can do.

Thinking through it a little more I think one obstacle might be that the init function sets up a bunch of stuff after the service is created, so connecting that stuff's errors back to the service might require some amount of plumbing.  We should probably come up with a plan on how to get the errors out.

I think with FuturesUnordered it should be possible to setup an empty container for the handles when constructing the service then push the JoinHandles into the service as they're constructed

I talked to eliza about this for a bit and it seems like the best option is to add a channel that is used to send the JoinHandle's to the peer_set and then have the peer set pop those from the channel and insert them into the FuturesUnordered before trying to wait for them.

TIL: tioli


App launches but immediately closes when an unavailability is manually edited to be invalid in data file

Expected behaviour would be to start with an empty file

Duplicate


Type family not injective in a RHS type variable

What happens if I say:
type family F a b = r | r -> a where
    F Int b = b

? Here b is not declared as injective but is mentioned in the RHS.

I think this is unnecessary.


Fixes #2961. v2 - WindowsDriver isn't render well on conhost terminal size.

Fixes #2961 - Two profile setting are needed for WindowsDriver, one for Windows Terminal (WT) and other for conhost (CH).
Pull Request checklist:

[x] I've named my PR in the form of "Fixes #issue. Terse description."
[x] My code follows the style guidelines of Terminal.Gui - if you use Visual Studio, hit CTRL-K-D to automatically reformat your files before committing.
[x] My code follows the Terminal.Gui library design guidelines
[x] I ran dotnet test before commit
[ ] I have made corresponding changes to the API documentation (using /// style comments)
[x] My changes generate no new warnings
[x] I have checked my code and corrected any poor grammar or misspellings
[x] I conducted basic QA to assure all features are working

https://github.com/gui-cs/Terminal.Gui/assets/13117724/4ef03a9e-0cef-434b-ab38-9b5d4f76bc48

@BDisp , if it's okay with you, I'd like to address this in #2610.

Closing this because will be addressed on the #2610.


Refactor of Mars APIs

In order to achieve the goals proposed in #882, we need to separate this into a few tasks.
First, implement all interfaces using actor client or actor context. Currently web API is the only way to talk to scheduler or worker, once this work done, client can directly connect to them using mars APIs. If network is unreachable, web API is the other choice, so we also need to adjust our web APIs to fit these interfaces.

[ ] Implement scheduler APIs and use it to replace the present API held by MarsWebAPI.

[ ] Implement worker APIs (part of them were done by #937).

[ ] Implement mutable tensors(dataframes is also needed if supported) APIs, make mutable tensors be able to write data directly into involved workers.

[ ] Implement something like WebContext, WebSession will hold it to send web requests.

[ ] Implement local APIs, only scheduler API is needed.

I think each of the above tasks is independence and won't affect the master branch. After all done, a new context and session are needed, there will only one class for context or session and no need to distinguish them by prefix like 'local' or 'web'.

[ ] Add a new context. Remove other context class like DistributedContext, context would hold different API object according to the environment.

[ ] Add a new session. Session would hold a context and using it to talk to scheduler.

Local scheduler needs to be taken into account as well.

Staled.


Add workaround for the karma test on WSL

Reference Issue
Related discussion: #2194
The problem this PR fixes
It doesn't seem that $ num run test:ci runs on WSL2.
$ npm run test:ci

> test:ci
> ./node_modules/.bin/lerna run test:ci --stream

lerna notice cli v3.22.1
lerna info versioning independent
lerna info Executing command in 4 packages: "npm run test:ci"
@google/model-viewer: ><EMAIL_ADDRESS>test:ci
@google/model-viewer: > npm run test
@google/model-viewer: ><EMAIL_ADDRESS>test
@google/model-viewer: > karma start --single-run
@google/model-viewer: (node:13498) [DEP0148] DeprecationWarning: Use of deprecated folder mapping "./dist/" in the "exports" field module resolution of the package at /mnt/c/Users/Takahiro/Documents/model-viewer/packages/model-viewer/node_modules/systemjs/package.json.
@google/model-viewer: Update this package.json to use a subpath pattern like "./dist/*".
@google/model-viewer: (Use `node --trace-deprecation ...` to show where the warning was created)
@google/model-viewer: START:
@google/model-viewer: 01 06 2021 18:12:18.813:INFO [karma-server]: Karma v5.2.3 server started at http://localhost:9876/
@google/model-viewer: 01 06 2021 18:12:18.864:INFO [launcher]: Launching browsers ChromeHeadless with concurrency 10
@google/model-viewer: 01 06 2021 18:12:19.160:INFO [launcher]: Starting browser ChromeHeadless
@google/model-viewer: 01 06 2021 18:19:19.262:WARN [launcher]: ChromeHeadless have not captured in 420000 ms, killing.
@google/model-viewer: 01 06 2021 18:19:19.284:INFO [launcher]: Trying to start ChromeHeadless again (1/2).
@google/model-viewer: 01 06 2021 18:26:19.386:WARN [launcher]: ChromeHeadless have not captured in 420000 ms, killing.
@google/model-viewer: 01 06 2021 18:26:19.389:INFO [launcher]: Trying to start ChromeHeadless again (2/2).
@google/model-viewer: 01 06 2021 18:33:19.489:WARN [launcher]: ChromeHeadless have not captured in 420000 ms, killing.
@google/model-viewer: 01 06 2021 18:33:19.493:ERROR [launcher]: ChromeHeadless failed 2 times (timeout). Giving up.
@google/model-viewer: Finished in 0 secs / 0 secs @ 18:33:19 GMT-0700 (Pacific Daylight Time)
lerna ERR! npm run test:ci exited 1 in '@google/model-viewer'

Platform: Windows10 + WSL2 + Ubuntu 18.04 LTS
It doesn't seem that Headless Chrome boots up correctly.
Solution
This PR fixes the problem by setting the following as workaround in karma.conf.js
    browsers: ['ChromeHeadlessNoSandbox'],
    customLaunchers: {
      ChromeHeadlessNoSandbox: {
        base: 'ChromeHeadless',
        flags: ['--no-sandbox']
      }
    },

The solution is from https://github.com/karma-runner/karma-chrome-launcher/issues/158#issuecomment-339265457
With this change $ npm run test:ci runs on my WSL2.
Request
If this change looks ok, just in case would you please check that this change doesn't break the tests on other platforms if you work on Mac, Linux, or others, before you merge this PR?

Thanks, runs on mac.

Hm, test:ci on Mobile Safari 13.0.3 (iOS 13.2) seems to fail...
https://github.com/google/model-viewer/runs/2749649928#step:9:841
@google/model-viewer: FAILED TESTS:
@google/model-viewer:   ModelScene
@google/model-viewer:     setModelSource
@google/model-viewer:       ✖ fires a model-load event when loaded
@google/model-viewer:         Mobile Safari 13.0.3 (iOS 13.2)
@google/model-viewer:       the xmlhttprequestprogressevent {"isTrusted":true} was thrown, throw an Error :)
@google/model-viewer:       promiseReactionJob@[native code]
@google/model-viewer:     with a model
@google/model-viewer:       ✖ "before each" hook for "can increase intensity and reset it to zero"
@google/model-viewer:         Mobile Safari 13.0.3 (iOS 13.2)
@google/model-viewer:       the xmlhttprequestprogressevent {"isTrusted":true} was thrown, throw an Error :)
@google/model-viewer:       promiseReactionJob@[native code]

Unfortunately we have some flaky tests, especially on Mobile Safari. I'm confident it's not your fault.


Weird dates

First of all, I can confirm this is working on a Volvo V60 T8 Polestar Engineered (2023)! So kudos for that!
The only thing that is a bit weird are the dates that I see on mqtt/HA:

Do you perhaps have an idea what could be causing this? I'm located in Belgium.
I use the following docker command to run:
docker run -d --restart unless-stopped --pull=always -e CONF_updateInterval=60 -e CONF_babelLocale='be' -e CONF_mqtt='@json {"broker": "xxx", "username": "xxx", "password": "xxx"}' -e CONF_volvoData='@json {"username": "xxx", "password": "xxx", "vin": "xxx", "vccapikey": "xxx"}' -e TZ='Europe/Brussels' --name volvo2mqtt ghcr.io/dielee/volvo2mqtt:latest

Also, the charging system status is empty so it seems

Please try CONF_babelLocale='fr_BE' for the date error.
Charging system status is empty until a cable is plugged in.

I used nl_BE, and that fixed it!

The cable is currently plugged in, so it was charging when I installed the add-on. I will try to remove and re-plug the cable :)

The "Charging System Status" is not updated so it seems :(

Maybe this isn't returned from the api...
I will build a debug version, wait a few minutes.

Seems indeed like no data is returned from the API

Please pull v1.0.9 and set env -e CONF_debug=true. This should give you the raw API results. If charging system state contains no data, there is nothing I can do, sorry!

{   "status" : 200,   "operationId" : "00d61ca7-8cbd-4382-9e6e-b600d1309b20",   "data" : {     "batteryChargeLevel" : {       "value" : "100.0",       "unit" : "percentage",       "timestamp" : "2023-06-20T11:28:54.122Z"     },     "electricRange" : {       "value" : "65",       "unit" : "kilometers",       "timestamp" : "2023-06-20T11:28:54.122Z"     },     "estimatedChargingTime" : {       "value" : "17",       "unit" : "minutes",       "timestamp" : "2023-06-20T11:28:54.122Z"     },     "chargingConnectionStatus" : {       "value" : "CONNECTION_STATUS_CONNECTED_AC",       "timestamp" : "2023-06-20T11:28:54.122Z"     },     "chargingSystemStatus" : {       "value" : "CHARGING_SYSTEM_CHARGING",       "timestamp" : "2023-06-20T11:28:54.122Z"     }   } }

v1.0.10 is on it's way

Issue resolved! Thanks a lot!
You can also add my car model to the supported list ;)

So whats you car model ?


Jump-to-error not working in nvim

Jump-to-error in nvim has been working sporadically for me. It's entirely possible that this is due to a config error on my part.
I have cargo-limit binaries and nvim-send in my path:
$ which nvim-send
/Users/rodarmor/.cargo/bin/nvim-send
$ which cargo-lcheck
/Users/rodarmor/.cargo/bin/cargo-lcheck

In my .vimrc I have:
Plug 'alopatindev/cargo-limit'

However, running something like cargo lcheck doesn't seem to invoke my g:CargoLimitOpen. I'm a bit of a loss as to how to debug this. Do you have any advice? The main question I'm trying to answer is: How does cargo-limit invoke nvim-send? Can I get it to print out logs? How does it find my nvim server?
I do seem to have an nvim server, :echo v:servername prints out:
/var/folders/b5/q2c4mxk534v6kwdtx2mpw91m0000gn/T/nvimoRduyt/0

Which when I pass to nvim-send:
$ nvim-send --servername /var/folders/b5/q2c4mxk534v6kwdtx2mpw91m0000gn/T/nvimoRduyt/0 --remote-send i

Does seem to work.

How does cargo-limit invoke nvim-send?

With default config (without custom g:CargoLimitOpen) it's invoked in the following conditions:

before running nvim, current directory was project (sub)directory
current nvim mode is normal
current buffer is either empty or contains some existing and unmodified (saved) file

How does it find my nvim server?

nvim may create several socket files but only /tmp/nvim-cargo-limit-${USER}/%path%to%project%directory is used by cargo lcheck and cargo-limit.vim.
Do you have this socket file while nvim is running?
Do you have this socket file when nvim is not running? If yes — try to remove this file and rerun nvim.
For now the only known buggy case when nvim-send is not invoked is if nvim crashed/killed. This results to not removing the socket file and next time nvim plugin thinks that server is already running, so it doesn't recreate the socket file.
If you run multiple instances of nvim in your project (sub)directory — only the first instance should jump to error lines, that's normal behavior.

Can I get it to print out logs?

For now there is no logging. I might will add it in future if it really makes sense.
If the issue still reproduces for you:

which OS and nvim version you're using?
which full project path do you have? (any special characters?)

Thanks.

Thank you for the detailed response!
Frustratingly enough, everything started working again, and I'm not sure what I did. I don't actually think I did anything, so I still don't know what went wrong 😭 If it happens again, I'll refer to your message to debug. What you wrote is so comprehensive, I'd consider adding it to the docs on the editor integration.


more CLGs

abstract
This goes towards #113
fixes
Please provide here the issues this pull requests fixes. You may want to make
use of Github's feature to automatically close issues via pull requests. See
https://github.com/blog/1506-closing-issues-via-pull-requests for more
information.
checklist

[x] Make sure you read and understood the CONTRIBUTING.md.
[x] Make sure you applied sufficient labels, milestone and assignee.
[x] Make sure all TODOs are addressed.
[x] Make sure there is sufficient documentation.
[x] Make sure there are sufficient tests.
[ ] Make sure all commits are squashed before merging.
[ ] Make sure to delete the feature branch after merging.

Current coverage is 97.19%

Merging #121 into master will decrease coverage by -0.53% as of 242d2d7

Powered by Codecov. Updated on successful CI builds.

LGTM


chore(main): release 1.7.0

:robot: I have created a release beep boop
1.7.0 (2024-10-07)
Features

Add MetalLB (59bfa70)
Migrate Traefik to ArgoCD (e712bc2)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.7.0
:sunflower:


Simplify the code to zoom in using a pinch gesture

/botio-linux preview

/botio-linux preview

It should be better now.


[G&M] Added a formula parser for indexof

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: Guillar1:x: akhileshYou have signed the CLA already but the status is still pending? Let us recheck it.


ArgumentError, "Missing secret_key_base for '#{Rails.env}' environment

Hello everyone,
After some changes to my Rails app and upgrading from Kamal 1.3.1 to Kamal 1.9.2 when I try to deploy the app I get the following message:
/usr/local/bundle/ruby/3.2.0/gems/railties-<IP_ADDRESS>/lib/rails/application.rb:658:in `validate_secret_key_base': Missing `secret_key_base` for 'production' environment, set this string with `bin/rails credentials:edit` (ArgumentError)

        raise ArgumentError, "Missing `secret_key_base` for '#{Rails.env}' environment, set this string with `bin/rails credentials:edit`"

I have in my .env the following env variables ( snippet )
RAILS_MASTER_KEY=xxxxxxx
SECRET_KEY_BASE=xxxx

Then kamal env push as always
But then when i fire kamal deploy I got that message. But ofcourse seems not possibile to edit the credential file inside a docker container. Or I missing something?
How can I set up correctly the secrete_key_base?
Also I don’t know if this can help you understand based on my configuration but I also share the deploy file, although it always worked.
# Name of your application. Used to uniquely configure containers.
service: xxxxxx

# Name of the container image.
image: xxxxxx/xxxxxx

# Deploy to these servers.
servers:
  web:
    hosts:
      - xxxxxx
    labels:
      {}

# Credentials for your image host.
registry:
  server: ghcr.io
  username: xxxxxx

  # Always use an access token rather than real password when possible.
  password:
    - GITHUB_TOKEN
# Inject ENV variables into containers (secrets come from .env).
# Remember to run `kamal env push` after making changes!
env:
  secret:
    - DATABASE_URL
    - RAILS_MASTER_KEY

# Configure builder setup.
builder:
  args:
    RUBY_VERSION: 3.2.1
  secrets:
    - GITHUB_TOKEN
  remote:
    arch: amd64
#     host: ssh://root@xxxxxx

# Configure custom arguments for Traefik
traefik:
  options:
    publish:
      - 443:443
      - 8080:8080
    volume:
      - "./acme_certs:/acme_certs"
  args:
    api.dashboard: true
    api.insecure: true
    providers.http.endpoint: http://xxxxxx/traefik/config
    providers.http.pollInterval: "5s"
    entryPoints.web.address: ":80"
    entryPoints.websecure.address: ":443"
    certificatesResolvers.letsencrypt.acme.email: "xxxx@xxx"
    certificatesResolvers.letsencrypt.acme.storage: "/acme_certs/acme.json" # Must match the path in `volume`
    certificatesResolvers.letsencrypt.acme.caServer: "https://acme-v02.api.letsencrypt.org/directory"
    certificatesResolvers.letsencrypt.acme.httpchallenge: true
    certificatesResolvers.letsencrypt.acme.httpchallenge.entrypoint: web
# Configure a custom healthcheck (default is /up on port 3000)
# healthcheck:
#   path: /
#   port: 3000

Edit: I managed to pass that error, i was not pushing the production.key.enc in my repo ( don't know how it was working before ).
Anyway now i have the following problem:
/usr/local/bundle/ruby/3.2.0/gems/activesupport-<IP_ADDRESS>/lib/active_support/messages/codec.rb:57:in `catch_and_raise': ActiveSupport::MessageEncryptor::InvalidMessage
        from /usr/local/bundle/ruby/3.2.0/gems/activesupport-<IP_ADDRESS>/lib/active_support/message_encryptor.rb:242:in `decrypt_and_verify'

I guess something is broken. Do I need to re create credentials?


[Excel Outlook] (events) update eventtype enum namespace

Docs Build status updates of commit 384863c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/tutorials/excel-tutorial.md
:white_check_mark:Succeeded
View

docs/tutorials/outlook-tutorial.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Compile issue [Win 10]

g++ -std=c++11 -Wsuggest-override  -fPIC -I../../../include -I../../../dep/include -MMD -MP -g -O3 -march=nocona -funsafe-math-optimizations -Wall -Wextra -Wno-unused-parameter -DARCH_WIN -D_USE_MATH_DEFINES  -c -o build/src/SidEx.cpp.o src/SidEx.cpp
src/SidEx.cpp:9:8: error: 'void MidiOutput::onMessage(uint8_t*, size_t)' marked 'override', but does not override
9 |   void onMessage(uint8_t bytes, size_t size) override {
|        ^~~~~~~~~
src/SidEx.cpp: In member function 'void MidiOutput::onMessage(uint8_t, size_t)':
src/SidEx.cpp:10:42: error: no matching function for call to 'MidiOutput::sendMessage(uint8_t*&, size_t&)'
10 |     midi::Output::sendMessage(bytes, size);
|                                          ^
In file included from ../../../include/rack.hpp:15,
from src/plugin.hpp:2,
from src/SidEx.cpp:1:
../../../include/midi.hpp:186:7: note: candidate: 'void rack::midi::Output::sendMessage(rack::midi::Message)'
186 |  void sendMessage(Message message);
|       ^~~~~~~~~~~
../../../include/midi.hpp:186:7: note:   candidate expects 1 argument, 2 provided
make: *** [../../../compile.mk:68: build/src/SidEx.cpp.o] Error 1

The output looks like you are trying to compile SidEx with the official Rack. At the moment my patched Rack is required, since the official branch doesn't support longer (sysex) midi messages.
You may want to try the compilation script that comes with SidEx. I haven't tested it in any of the unix environments of Windows (msys, cygwin, ...)
I just created a pull request for my Rack patch https://github.com/VCVRack/Rack/pull/1713
If you want to help speed things up, you may want to comment the pull request or one of the related Rack issues.

My Rack PR was rejected so I don't know when SidEx can be used with official Rack. @SteveRussell33 is there any other way I can help?

Having read the comments concerning this, I think in the mean time I will build another custom Rack EXE with your patch to test; I have a couple of other PR's added to it so this is not a problem for me as I pretty much build everything for Rack myself anyway.
I see that Andrew has indicated https://github.com/VCVRack/Rack/issues/1433#issuecomment-559625345 that this will be possible (i.e. PR not needed) in Rack v2 API. So for those that want this now, have to build Rack + PR themselves or wait for v2 to be released.
I'll close this for now and reopen when v2 gets closer and go from there.


Device.Print_Property() does not seem to be available from python

I tried to run the code in '2.2. Number of GPUs' of the documentation. However, Device.Print_Property() does not seem to be available from the Python API.?
I am using cytnx_cuda from conda (where the cytnx version is still 0.7.1a and python 3.8 must be used instead of 3.9).
In Python:
import cytnx
print(cytnx.Device.Ncpus)
print(cytnx.Device.Ngpus)
cytnx.Device.Print_Property();
Output>>
16
1

AttributeError                            Traceback (most recent call last)
Cell In[11], line 3
1 print(cytnx.Device.Ncpus)
2 print(cytnx.Device.Ngpus)
----> 3 cytnx.Device.Print_Property();
AttributeError: module 'cytnx.cytnx.Device' has no attribute 'Print_Property'

Similarly, Device.getname(0) cannot be called.
print(cytnx.Device.getname(0))

Fixed in 1f95e79.


US439108 - Updated OpenSUSE to version 15.4

Getting conflict issue at the time of building new environment using this environment. So updated OpenSUSE to latest v15.4.

CI Build Link:
http://sou-jenkins2.swinfra.net:8080/job/CAFapi/job/CAFapi~buildenv-dotnet-image~US439108~CI


Page timestamps / version?

(Do we want to make this optional?)
It'd be nice to get some basic page metadata:

I.e.:

timestamp of first revision (maybe a relative date but the absolute date as tooltip)
timestamp of most recent revision
rev count for this file (something like git rev-list --count --first-parent)

I'll transfer this issue over to the API repo, as it involves more work there. On the frontend, we just need to display a few more values sent by the API at the /api/Documents/ByLocation endpoint.


Add support for fields overriding

This PR enables support for both @override directive and --override CLI option.
Inspired by #17 and #135

@IvanGoncharov Do you have a minute to review? Thx.


Error 500 When Listing Products (non-empty body in GET requests)

Describe the bug
I am encountering an Error 500 when trying to list products using the Paddle PHP SDK in the Sandbox environment. The API key is validated, and the client seems to be implemented correctly. The issue arises when calling $client->products->list(); which triggers an error from the Paddle API.
{
  "type": "https://tools.ietf.org/html/rfc2616#section-10",
  "title": "An error occurred",
  "status": 500,
  "detail": "Paddle API can't process your request"
}

Steps to reproduce

Implement the Paddle PHP SDK.
Initialise client with API KEY and in Sandbox.
Execute the command $client->products->list();.

Expected behavior
A successful API call returning a list of products.
Code snippets
	// Initialize the API
	$this->api = new Client(
		$this->params->get('app.paddle')['api_key'],
		new Options(
			$this->params->get('app.paddle')['is_sandbox'] ?
				Environment::SANDBOX :
				Environment::PRODUCTION
		),
		$this->asyncClient,
	);

	$this->api->products->list();

PHP version
PHP 8.1
SDK version
0.1.0
API version
V1
Additional context
Investigation and Findings

Debugging revealed that the GET request to 'https://sandbox-api.paddle.com/products' was correctly formatted but included an unusual string "null" in the request body, which is atypical for GET requests.
Manually resending the request using Paw/Postman with the string "null" in the body reproduces the error, confirming its role in the issue.
I have traced the problem to this line of code: Line 179 in Client.php. It appears that even if the $payload is null, the serialization process converts it to a string "null", which is then incorrectly included in the GET request body.

Possible Cause
The method of serializing the payload in the SDK seems to convert a null value into a string "null", which is then included in the body of the GET request. This behavior is not standard for GET requests and could be causing the Paddle API to return an error.
Suggested Fix
Review and modify the serialization process in the SDK to ensure that a null payload does not result in a string "null" being included in the GET request body. It might involve adding a condition to check if the payload is null before serializing it. This change could prevent the inclusion of inappropriate data in the GET request body, potentially resolving the error.
===
P.s.: Thank you for this SDK. It comes just in time! I've already seen myself having to implement a PHP client myself in the next few weeks. 👍

Because I'm not sure after reading the Contributing.md:
Do you want Pull Requests on this repo?

Hey @Luehrsen thank you for the report and the time spent looking into the cause of this.
To help improve our coverage effort would you mind sharing what PSR-7 HTTP Client you provided to the library to fulfil the request?
Regarding contribution we will accept pull requests, though contributors must be wary that most of this repositories codebase will be or become generated code. Currently the Client is not generated so it wouldn't have been an issue.
I have a branch which should resolve the issue, PR will be up shortly, I'm happy for you to take a look before merging to ensure it resolves the issue for the client you're using 👍

To help improve our coverage effort would you mind sharing what PSR-7 HTTP Client you provided to the library to fulfil the request?

Hey @mikeymike,
thank you for fixing this so fast. I've tested both the standard http client of your library and the symfony http client (https://symfony.com/doc/current/http_client.html).
But if I trace both back, both rely on guzzle. So I think it does not make any difference.

@Luehrsen v0.1.1 has been tagged and published 👍
Thanks again for the report!


[GOBBLIN-1750] Add schemas for observability events in GaaS

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

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

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

Description

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

Users of GaaS have a difficult time monitoring their flows. Regular metrics do not have the cardinality needed to effectively determine the status of a pipeline, and GTE are too broad and are hard for users to create meaningful queries from.
This PR sets up the schema of an observability event which would be used as a monitoring baseline in GaaS
Tests

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

Commits

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

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

Codecov Report

Merging #3610 (979e819) into master (c3d1ba8) will decrease coverage by 3.14%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3610      +/-   ##
============================================
- Coverage     46.89%   43.75%   -3.15%
+ Complexity    10673     2062    -8611
============================================
  Files          2120      408    -1712
  Lines         83080    17618   -65462
  Branches       9254     2151    -7103
============================================
- Hits          38962     7709   -31253
+ Misses        40548     9051   -31497
+ Partials       3570      858    -2712

Impacted Files
Coverage Δ

...a/org/apache/gobblin/cluster/GobblinHelixTask.java
64.51% <0.00%> (-19.36%)
:arrow_down:

...lin/util/filesystem/FileSystemInstrumentation.java
92.85% <0.00%> (-7.15%)
:arrow_down:

...c/main/java/org/apache/gobblin/util/PathUtils.java
11.76% <0.00%> (-1.57%)
:arrow_down:

.../org/apache/gobblin/cluster/GobblinTaskRunner.java
62.22% <0.00%> (-1.41%)
:arrow_down:

.../org/apache/gobblin/metastore/MysqlStateStore.java
8.09% <0.00%> (-1.11%)
:arrow_down:

...rg/apache/gobblin/util/jdbc/DataSourceBuilder.java
0.00% <0.00%> (ø)

...g/apache/gobblin/util/jdbc/DataSourceProvider.java
0.00% <0.00%> (ø)

...ache/gobblin/metastore/MysqlDataSourceFactory.java
83.33% <0.00%> (ø)

...obblin/metastore/JobHistoryDataSourceProvider.java
62.50% <0.00%> (ø)

.../profile/ManagedIcebergCleanableDatasetFinder.java

... and 1711 more

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


[CodeCompletion] Show the enum type for enum element with assoc values

enum MyEnum {
    case null
    case str(String)
}

When completing elements for enum like this, the former shows MyEnum, but the latter shows (String) -> MyEnum as the display type. This is inconsistent with function/method call pattern which only shows the result type.
rdar://problem/48220244

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please smoke test OS X platform


[PROPOSAL] support rebuild or deep copy on SearchRequest.Builder

What/Why
What are you proposing?
Rebuild or deep copy is supported for SearchRequest.Builder. It seems tough but supported in opensearch rest client, but there seems to be no way to support it in opensearch java(refer).
What users have asked for this feature?
What problems are you trying to solve?
I want to run a query multiple times by adding only some values to an already prepared SearchRequest. For example, rerun the query by dynamically changing only the index or from field value.
What is the developer experience going to be?
Create a SearchRequest based on it. And for every query, SearchRequest.Builder is recreated and all field values set are put back in and built.
Are there any security considerations?
Are there any breaking changes to the API
What is the user experience going to be?
Are there breaking changes to the User Experience?
Why should it be built? Any reason not to?
What will it take to execute?
Any remaining open questions?
I wonder why you restricted SearchRequest.Builder to only build once.

I wonder why you restricted SearchRequest.Builder to only build once.

@kdh6429 The SearchRequest.Builder#build method does not make a deep copy from the builder to the built object, so it is not safe to reuse the builder.
This feature request seems reasonable to me. The general pattern I've seen is to add something like a toBuilder() method on a built SearchRequest instance, which will make a deep copy of all the data from the search request into a new mutable builder. This allows for the following pattern:
SearchRequest template = new SearchRequest.Builder()
    .index("my-index")
    // other common parameters
    .build()

SearchRequest request1 = template.toBuilder()
    // specific settings for request 1
    .build();

SearchRequest request2 = template.toBuilder()
    // specific settings for request 2
    .build();

Would that meet your use case?

Yes, this would be very satisfying.

We also have a similar use case. We need to dynamically add an additional query filter to an already build SearchRequest.
Adding a toBuilder() method is fantastic!
Can you please give us a rough estimation when this enhancement can be expected?

Can you please give us a rough estimation when this enhancement can be expected?

@assafcoh I don't think anyone is working on this, would you like to pitch in?

@dblock unfortunately I do not have time to work on this. However I believe this issue should be given higher priority since it is necessary for many projects (I found several issues with similar feature request). Thank you.

Hello,
I'm interested in this solution and tried the toBuilder() method on the BoolQuery.
But there is a limitation and it does not work for my use case which is:

Web application (restful API, developed with SpringBoot)
I am having a base BoolQuery.Builder setting up base filters on my opensearch documents (e.g. public documents only).
Then, for each user request, user specific filtering criteria are added, e.g. data between date d nd f, or author is John Doe... For doing that step, I copy the base BoolQuery.Builder by using .build.toBuilder() and add the user specific criteria. That works.
But for next user request I cannot call .build().toBuilder() on my initial baseBoolQuery.Builder since build() can only be called once. As a work-around, I currently call the .build() once, then I create 2 new Builders: 1 for my current user request, 1 to re-initialize the base BoolQuery.Builder. But this is not a production acceptable solution because it is not thread-safe: a new user request might arrive before the original base BoolQuery.Builder as been re-initialized. I've stressed test my server and that race condition occurs.

Any better idea on how to handle that case ? I was thinking that the most simple would be to have a copy/clone method directly on the BoolQuery.Builder. Any thoughts on that ?
I could manage in my own code all the attributes of the BoolQuery.Builder, e.g. filter, must, etc... so that I can re-use them whenever I need to, but it sounds silly that I cannot re-use the BoolQuery code more.
Thanks


dji_onboardsdk_ros: 0.1.7-0 in 'indigo/distribution.yaml' [bloom]

Increasing version of package(s) in repository dji_onboardsdk_ros to 0.1.7-0:

upstream repository: https://github.com/dji-sdk/Onboard-SDK-ROS.git
release repository: https://github.com/dji-sdk/Onboard-SDK-ROS-Release.git
distro file: indigo/distribution.yaml
bloom version: 0.5.21
previous version for package: 0.1.6-0

We're in a  quiet period holding indigo releases in preparation for a sync. If it's important for this to be in the sync please comment on the  ros-release thread titled Preparations for final releases for Saucy, Utopic, and Vivid for Indigo and Jade April 1st.

hi @tfoote how long will the period last?

All targets except ARM have already finished so likely one or two more days.

@dirk-thomas thanks, then I will wait.


Bluez

Please add a bluetoothctl command line to enable possibility to connect bluetooth devices

It`s on bluez package.
I tried to compile but I'm getting this error:
configure: error: D-Bus >= 1.6 is required

I tried with this:
./configure --prefix=/usr --with-udevdir=/usr/lib/udev --disable-systemd --disable-debug --disable-test --disable-nfc --disable-sap --disable-cups --disable-mesh --disable-android

Unless --disable-dbus can work here I don't think Dylan can add it.

Yeah, dbus is mandatory with Bluez sadly. I looked into this a while ago.

Do you known any other option to use bluetooth?

Sadly, no. Bluetooth on Linux require dbus. Even embedded devices are forced to run it to achieve working Bluetooth. I'd love it if somebody could prove me wrong here!

Should this issue be closed or are you investigating alternatives?

This could be an option for those wanting Bluez without the use of dbus: https://github.com/clbr/nobus

Should this issue be closed or are you investigating alternatives?

Regardless, this acts as an answer to the question of "Is Bluez packaged"?

Closing this as it won't ever be packaged. Users needing bluetooth can package both dbus and bluez themselves.


Push update to https://pkg.go.dev/github.com/teslamotors/vehicle-command or make a new release here?

Can we update to the newest version of vehicle-command at https://pkg.go.dev/github.com/teslamotors/vehicle-command or make a new release here?
Last release was Oct 9, 2023 and with all the recent updates that just took place this would make life easier when importing vehicle-command into our projects

Thanks guys!


Cannot read property 'keepImport' of undefined

happens when:
"plugins": ["css-modules-transform"]

vs.
"plugins": [
    ["css-modules-transform", {
      "keepImport": false
    }]
  ]

TypeError: src/button/index.js: Cannot read property 'keepImport' of undefined
    at PluginPass.ImportDefaultSpecifier (/Users/greada/Documents/workspace/s2/s2-component-library/node_modules/babel-plugin-css-modules-transform/build/index.js:194:43)
    at newFn (/Users/greada/Documents/workspace/s2/s2-component-library/node_modules/babel-traverse/lib/visitors.js:276:21)
    at NodePath._call (/Users/greada/Documents/workspace/s2/s2-component-library/node_modules/babel-traverse/lib/path/context.js:76:18)
    at NodePath.call (/Users/greada/Documents/workspace/s2/s2-component-library/node_modules/babel-traverse/lib/path/context.js:48:17)
    at NodePath.visit (/Users/greada/Documents/workspace/s2/s2-component-library/node_modules/babel-traverse/lib/path/context.js:105:12)
    at TraversalContext.visitQueue (/Users/greada/Documents/workspace/s2/s2-component-library/node_modules/babel-traverse/lib/context.js:150:16)
    at TraversalContext.visitMultiple (/Users/greada/Documents/workspace/s2/s2-component-library/node_modules/babel-traverse/lib/context.js:103:17)
    at TraversalContext.visit (/Users/greada/Documents/workspace/s2/s2-component-library/node_modules/babel-traverse/lib/context.js:190:19)
    at Function.traverse.node (/Users/greada/Documents/workspace/s2/s2-component-library/node_modules/babel-traverse/lib/index.js:114:17)
    at NodePath.visit (/Users/greada/Documents/workspace/s2/s2-component-library/node_modules/babel-traverse/lib/path/context.js:115:19)

What version of babel are you using? This is strange, because there is a test for this option and it's passing.

{
    "babel-cli": "^6.26.0",
    "babel-core": "^6.26.0",
    "babel-plugin-css-modules-transform": "^1.3.0"
}

I am also using env and react as presets
{
  "presets": ["react", ["env", {
    "targets": {
      "browsers": ["last 2 versions"]
    }
  }]]
}

Yes I found a bug, give me a moment.

Please try 1.3.1.

Success 👍
Works like a charm. Thx for the great work.

You are welcome 🙂


[BUG] XML Schema Violation: externalReferences.hashes included in XML BOM schema versions 1.1 and 1.2

Example BOM generation:
bom = Bom(components=[
        Component(
            name='toml', version='0.10.2', bom_ref='pkg:pypi/toml@0.10.2?extension=tar.gz',
            purl=PackageURL(
                type='pypi', name='toml', version='0.10.2', qualifiers='extension=tar.gz'
            ), hashes=[
                HashType.from_composite_str('sha256:806143ae5bfb6a3c6e736a764057db0e6a0e05e338b5630894a5f779cabb4f9b')
            ], external_references=[
                get_external_reference_1()
            ]
        )
    ])

Generates
...
<externalReferences>
                <reference type="distribution">
                    <url>https://cyclonedx.org</url>
                    <comment>No comment</comment>
                    <hashes>
                        <hash alg="SHA-256">806143ae5bfb6a3c6e736a764057db0e6a0e05e338b5630894a5f779cabb4f9b</hash>
                    </hashes>
                </reference>
            </externalReferences>
...

.externalReferences.hashes is not part of the schema prior to version 1.3.

Confirmed this doesn't affect 1.0 output as externalReferences is not supported full stop and is not included in 1.0.