Document options.hash

…or provide some other way of specifying the hash directly. Some idiot developer who most definitely was not myself from a few years ago decided to use user's email addresses to generate geopatterns, and now I need to either change the patterns for all users or compute the hashes server-side. It would be nice if this use case was handled/documented.

Can you help me understand what you're trying to do? Before, you were calculating the pattern on the fly from the user's email address, but now you want to calculate the pattern server-side? It doesn't sound like I have that right.

Roughly. I don't want to calculate the pattern server-side, but I want to calculate the SHA1 server-side so that I don't leak everyone's email address, as that's a Bad Thing.
Before:

Client requests /api/users/1.
Server responds with {name: 'Alice', email<EMAIL_ADDRESS>Client calls<EMAIL_ADDRESS>
Goal:

Client requests /api/users/1
Server computes hash of<EMAIL_ADDRESS>and responds with {name: 'Alice', patternHash: 'fc2398a73dd54d6237c4fdb58fd7d75347cf5af3'}
Client calls Geopattern.generate(null, {hash<EMAIL_ADDRESS>

Ah I see. If you're trying to protect users' email addresses, unsalted SHA-1 isn't going to be very resistant to a rainbow table (certainly better than plaintext of course), especially when a handful of extensions (@gmail.com and friends) ought to cover the majority of cases. Do you have a hard requirement that you can't change the pattern? If you can change the pattern, could you use the users' IDs instead?


Update integration docs

To resolve https://github.com/Azure/azure-sdk-for-java/issues/25935, update spring integration ref docs for event hubs, service bus and storage queue.

Spring Integration: consumer -> channel

Processor.BlobCheckpointStore is not implementing the CheckpointStore interface, it's a configuration properties class.


edited some wording in AO intro and link text

Changed wording around experienced teacher and also shortened providers offering assessment only QTS to international teachers
Trello card
https://trello.com/c/EPcL36d6/1859-revisit-the-content-at-the-top-of-the-assessment-only-page

I think the heading needs changing as well @tcannon3 - 'international' doesn't make sense as 'a region of the provider'?
Perhaps 'Select a region' would be better?


Attributes for tools, 2.0

This RFC proposes a temporary solution to the problem of letting tools use
attributes. We outline a (partial) long-term solution and propose a step towards
that solution for tools which are part of the Rust distribution.

rendered

Could a special prefix be used for tools? I'm thinking of an analogue to the different proposals for crates:

[crate]::foo::bar
@crate::foo::bar
extern crate::foo::bar
...

So, #[<prefix> tool::foo].

Note that clippy uses very few attributes directly. Instead almost all of the #[cfg_attr...] that we see in code using clippy is #![cfg_attr(clippy, warn(clippy))] or #![cfg_attr(clippy, allow(some_overzealous_lint))].

@oli-obk I have wanted many times to had attributes for clippy but the uglyness of #![cfg_attr(clippy, …)] stopped me :smile:

#![cfg_attr(clippy, allow(some_lint))]

Should these be migrated from "optional language lints" to "clippy lints", and use syntax like the following?
#[clippy::allow(some_lint)]

#[allow(clippy::some_lint)] fits better in the current model. Clippy does not manage allow/warn/deny by itself, and just relies on rustc to do that. But if rustc knew that some lints are from clippy, it could just skip the unknow lint message.

This sounds good. Per the unresolved question, I think we should definitely triage all existing attributes and give them better names---I wouldn't a lang prefix for builtin non-rustc-specific stuff either.
Also, I'd like #[test] and friends to eventually be procedural macros instead of baked into the compiler---it would be wonderful if we can pave the way for that with this.

Regarding lints, it sounds like a potential extension is allowing tool prefixes on lints, so rather than lints being just a single identifier (some_lint) they can be a path (foo::some_lint). Furthermore, that using a known tool's name as the root of the path (e.g., clippy::foo::some_lint) would prevent the compiler giving an 'unknown lint' error.
I think this is a good idea!
It seems a bit orthogonal to this RFC, so I'm not sure whether we should add it. Ideally, it should be a separate RFC, but it would be quite a small addition to this RFC, and I would like to avoid clippy moving from cfg(attr) to clippy::allow(some_lint) to allow(clippy::some_lint), rather than doing that in a single step.
Clippy people (@Manishearth, @oli-obk, @ilogiq) thoughts? Specifically, does anyone want to write the scoped lints proposal up as an RFC in the near future? (I'd be happy to help out). Does anyone think it is essential or terrible to include here? And do you have an idea for how hard the implementation would be?

I think it's fine to include here, I would phrase it as #[allow(clippy::foo)]. The basic concern is that there should be an easy mechanism to say "ignore clippy::* when clippy isn't on instead of unknown-lint warning it"

I've added tool-scoped lints to the RFC. I expect these to be implemented separately and not depend on one another.
Ping @rust-lang/lang this is a fairly significant extension to the RFC, but logical, I think.

@rfcbot fcp merge
There's not been much discussion, but it is a fairly simple RFC and a stepping stone to a general solution that leaves nearly every sensible possibility open. It hasn't been open long, but the previous RFC (#1755) has been around for much longer. Therefore I think it is OK to FCP this now.

@rfcbot reviewed

@rfcbot reviewed

@rfcbot reviewed

@rfcbot reviewed

my only concern is to make sure that this uses the same syntax as the #2126. There is some discussion (although it isn't gaining much ground) of requiring use extern::foo::bar for external crates. If that were the case, this feature should also use #[extern::clippy::the_lint]


Forgot login email update

@jsimpson-cc  this looks good to me. If the automated tests pass, feel free to merge this it in.


appends sapce to the replacement item if 'appendSpace' option is added to typeaheadOpts

+1
I would even make it the default.


Click position issue with touch screen on Raspbarry PI

Software Versions

Python: 3.7
OS: lastest Rasbian
Kivy: 2.0.0
Kivy installation method: PIP

Describe the bug
I built a simple application on the PI, which only has 3 buttons placed horizontally on the screen.
If i use mouse to click on buttons on the screen, each button click is triggered correctly. However, when I test the button clicks on the touch screen, the left and right buttons got mixed up, if I click on certain locations of the buttons. it feels like the screen has been horizontally flipped as the mid button is always fine.
Initially, i thought it was the problem with the touch screen. but when I test with other UI frameworks (same UI layout), e.g. Tinkter or AppJar. they all behave correctly.
So I came back to the real basic: have a UI that has only one custom component, and upon click, print out the clicked position.
please see the code snippets below
as a result, when I tap the left side of the screen, the program prints out TWO lines instead of one
pressed at [407.4432234432234,<PHONE_NUMBER>7106228]
pressed at [62.00000000000001, 221.0]
The screen is a 3.5 inch GPIO touch screen, reoslution at 640*480, the print out suggested that Kivy picked up two clicks and the two clicks are from the (almost) mirrored horizontal postions (407 vs 62)
with the same code at the same environment, if i click with a mouse, Kivy only picks up one event with correct coordinates.
Expected behavior
Kivy should only pickup or process the correct position,
To Reproduce
setup a PI with touch screen and run the code below
Code and Logs and screenshots
class RootWidget(BoxLayout):

    def __init__(self, **kwargs):
        super(RootWidget, self).__init__(**kwargs)
        cb = CustomBtn()
        self.add_widget(cb)

class CustomBtn(Widget):

    pressed = ListProperty([0, 0])

    def on_touch_down(self, touch):
        if self.collide_point(*touch.pos):
            self.pressed = touch.pos
            return True
        return super(CustomBtn, self).on_touch_down(touch)

    def on_pressed(self, instance, pos):
        print ('pressed at {pos}'.format(pos=pos))

class TestApp(App):

    def build(self):
        return RootWidget()

if __name__ == '__main__':
    TestApp().run()

Additional context
Add any other context about the problem here.

the screen in question is not damaged rht ?

i'll be compiling kivy on rasp in a bit,
i'll post my findings then

the screen in question is not damaged rht ?

I am sure its fine, i tested on multiple Pi setups, with different screens. it is functioning. and it works with other apps

Any luck reproducing the issue?

@paul-li-pinery forgot to do :(. first thing to do tomorrow morning

These types of issues always seem to be caused by configuration issues.  @p0lygun @ydydxhg Did you manage to resolve this issue?  See: https://kivy.org/doc/stable/installation/installation-rpi.html#using-official-rpi-touch-display


asked what i'm trying to debug when I have a test file open with breakpoints set

Testing #22081
It would be better if it could be assumed that the active file (with breakpoints) is what I'd like to debug

the problem is that if this is part of a e.g. Django app,  we don't know if users want to debug it as a script or as part of the Django app. The key thing is that only the debug configurations are different for the scenarios listed in the drop down -- they could all end up debugging that active Python file (we've actually seen too many users get confused with selecting "Python file" when in fact they wanted a different configuration). I've wondered if replacing "Python file" with "Script" would make the decision making there less confusing, but not sure 🤔
@paulacamargo25  @cwebster-99 would love to hear your thoughts!


perf(sidenav): avoid extra repaints while scrolling

Avoids repaints while scrolling inside of a sidenav. This isn't an issue on desktop browsers, but can potentially be on mobile. This can be checked by going to the dev tools > console fold-out menu > Rendering > Scrolling Performance Issues.

LGTM


Add recaptcha implementation for stripe transactions

@manuelurenah tried this out locally, it's good to go


Assets/RealSenseSDK2.0/Scripts/AlignImages.cs(2,7): error CS0246: The type or namespace name `Intel' could not be found. Are you missing an assembly reference?

when I Open the Unity demo in librealsense-2.10.3 on windows10 connecting d435,there are some errors.Most errors are missing an assembly reference.how can I issue these problems?

Same problem here, after I get the binaries and put in the correct folder on my unity project, everything looks to work but I can´t compile (build) I get this error :Assets/RealSenseSDK2.0/Scripts/RealSenseDevice.cs(8,7): error CS0246: The type or namespace name `UnityEditor' could not be found.

[Realsense Customer Engineering Team Comment]
Hi @wxp0821, @superaldo,
Did you follow below instructions to successfully build the Unity app?
https://github.com/IntelRealSense/librealsense/tree/master/wrappers/unity
For the namespace issue, you can refer to below. Try to add a new script like ‘rsEditor.cs’ and move below code in RealSenseDeviceInspector.cs to this new script.
public class RealSenseDeviceInspectorEditor : Editor
{
…
}
https://answers.unity.com/questions/316805/unityeditor-namespace-not-found.html

[Realsense Customer Engineering Team Comment]
Hi @wxp0821,
Please try librealsense 2.12.0. This should resolve this issue.

[Realsense Customer Engineering Team Comment]
Hi @wxp0821,
Do you have any update?

[Realsense Customer Engineering Team Comment]
Hi @wxp0821,
I will close this. If you have any further question, you still can re-open or create another ticket.


[SNatives] Update account balance

Hello, I created a SNative contract to update the account balance, but the stateWriter.SubtractFromBalance is not persisting the state change (I.e. the account balance don't change).
File changed: https://github.com/hyperledger/burrow/blob/develop/execution/evm/snative.go
SNative Contract description:
NewSNativeContract(`
* Interface for gas consumption.
`,
  "Balance",
  &SNativeFunctionDescription{Comment: `
  * @notice Test decreaseBalance
  * @return result a bool
  `,
    Name:      "decreaseBalance",
    PermFlag:  permission.Send,
    Arguments: reflect.TypeOf(decreaseBalanceArgs{}),
    Returns:   reflect.TypeOf(decreaseBalanceRets{}),
    F:         decreaseBalance},
)

Function implementation:

type decreaseBalanceArgs struct {
	Account crypto.Address
	Amount  uint64
}

type decreaseBalanceRets struct {
	Result uint64
}

func decreaseBalance(stateWriter Interface, caller crypto.Address, gas *uint64, logger *logging.Logger,
	a interface{}) (interface{}, error) {
	args := a.(*decreaseBalanceArgs)

	exists := stateWriter.Exists(args.Account)
	if !exists {
		return false, fmt.Errorf("unknown account %s", args.Account)
	}

	stateWriter.SubtractFromBalance(args.Account, args.Amount)

	return decreaseBalanceRets{Result: args.Amount}, nil
}

The state cache is updated accordingly, but the final state is not persisted.
I can read the final state using SNative's, but I cannot write a new state.
Something is missing?
 In Solidity interface I had decreaseBalance with the modifier view, making Burrow execute CallSim and not Call transaction.


SL Review: Ability to search within BC gov by name

User Story
As an ADM filling out the SL review form, I want to be able to search for anyone in the government with his/her name so that i cant easily select the person I want to nominate for the process
Tasks

[ ] Hook API endpoints to ODS for idir/employee full name to a text field in the SL review form

[ ] Any input to the field will have drop-down options to select from

[ ] Access need to be restricted

Acceptance Criteria

This ticket will be considered complete when ADMs are able to search and select someone by inputting a name into the "Employee Name" field

@HasanSifat @PatrickAHeath
Explore the below API to fetch employee details.
https://analytics-testapi.psa.gov.bc.ca/apiserver/api.rst#Datamart_PECSF_employee_demo
question: Can we apply filer on name, first/last name here?
In this API we can apply filters as per below.
Search is a query param, it applies search in all the text fields. like name, supervision name, email, city, position, etc.
Let's see how it works,
Request URL: https://analytics-testapi.psa.gov.bc.ca/apiserver/api.rsc/Datamart_Telework_employee_demo?$search='mark'
Result: Mark L Bakusko, L. Darcy Houston, Wendy Knott
Notes:
Here, the first result(Mark L Bakusko) looks good.
The second and 3rd names do not match with the keyword 'mark' still we see those in the result because their supervisor name is Mark Carter & Mark Van Tassel respectively.
question: Can we apply filter based on supervison_email?
In this form ADM/Supervisors add details for employee who reports them.
we can run API https://analytics-testapi.psa.gov.bc.ca/apiserver/api.rsc/Datamart_Telework_employee_demo?$filter=supervisor_email eq<EMAIL_ADDRESS>to get employee whose supervisor email match with the provided.
It helps to eliminate security risks like any user trying to access employees who are not reporting to them.

Using the $filter function of the API service allows better control on what fields to filter vs. the search function.  As you have discovered, it will search for all fields and return records that matches the criteria.

@T854236 filter does not allow a LIKE kind of query on the string value. And if possible then I want to know that keyword(that search in the field as LIKE in SQL)
it works with a short form like eq (equal), or gt (greater than).
In employee search, we will have not a full name to match using eq. We have to pass a partial name and expect to autocomplete result on it(LIKE query in SQL).

Please try this: https://analytics-testapi.psa.gov.bc.ca/apiserver/api.rsc/Datamart_Telework_employee_demographic?$search='insert partial name'
This endpoint has two differences in comparison to Datamart_Telework_employee_demo:

Only 1 row is returned per employee (even if they have concurrent positions).
Only the name column is searchable.  Using the $search filter function translates to a LIKE in SQL.

If you have any other questions, please email me.

we've got the ok signal from the PIA people to hook up the end points & test

tested successfully in DEV. One issue I noticed, once a selection has been made, it's difficult to cut through to re-edit the field. There is a small cross button that needs to be clicked to remove that name and select a new one. Being able to press backspace would be more friendly from an accessibility point of view.

@HasanSifat
For now, it enables you to start a new search without clicking on the x icon. When you click on the name again it will open a search popup and you can begin typing a new name.
The same control is not allowed in the selected name itself. It's a select component hence value cannot be changed.
To enable backspace in the name, we must introduce a new component in which we will add a text box, and attach a search popup that connects with a remote API. It seems a little bit bigger task here as we are trying to replicate features from one to another component. @simensma-fresh @iman-jamali-fw Let's connect and see if its fesible and try to estimate efforts.

@bhumin-fw @simensma-fresh That makes sense. The team has agreed to stick with the drop-down field. Also I tested the feature to day this morning. It's not working right now
@EduardoRamirezBC @JodiLittle please review and confirm if this feature is working correctly and as per requirements for you.

@bhumin-fw the enter name field is still not working in Dev. Did you have a chance to look at it? the pre-population fields are also not filling out with that selected person's info

@HasanSifat
I'm trying to generate the same scenario on dev or local, But no luck, It works fine for Simen, Iman, and mine login. I did check issues in the form-io library but no such issue logged so far.
I'll keep you posted if I'll find anything that we can note down or fix in the form itself.
Meanwhile, could you please check with anyone else on your side if its working fine or not?

Tested successfully in DEV. Ready to be transferred to PROD

One bug I noticed, the search field does not refresh if someone made a typo. The search feature does not give out any options if i remove anything i typed before

@HasanSifat Fixed the issue of the typo and the search just stop loading new data. Now it will start searching as soon as you type minimum 3 characters in the search box.

Looks good in DEV. Ready to be transferred to TEST/PROD

confirming the endpoint used is deployed to PROD for the BI API service.

Okay to move test

After initial UX testing, we identified a bug with the search field where it stops giving options after users type in something wrong. @bhumin-fw i can debrief you on this on monday
Video recording to show the bug- [Uploading... Recording_1665787283509.webm]

@HasanSifat This happens if you type special characters in the search field (the ' in O'leary), it's going to take a bit of effort to get this fixed, so it will not be fixed for the testing tomorrow.

Thanks @simensma-fresh at least we now know what is causing the search feature to break. We can address this in our test tomorrow by informing the ADMs if needed and it looks like a refresh of the page makes the search work again.

Works fine except those special characters notes above - Bhumin working on this one.

@HasanSifat Fix the special character break search functionality issue. Also, handle errors so search functionality will not be blocked.
It's available to test on DEV.

Tested successfully in DEV. Ready to be transferred to TEST/PROD

Approved

tested successfully in TEST


How can I know the latest version number?

You can see this badge.
https://github.com/castorflex/SmoothProgressBar#integration

Thank you !


Migrate Travis CI to GH workflows

What kind of change does this PR introduce?
CI Update
What is the current behaviour?
CI running both on Travis and GH workflows
What is the new behaviour?
Move whole CI to GH workflows
Does this PR introduce a breaking change?
No
Other information:
Please check if the PR fulfills these requirements:

[ ] Tests for the changes have been added
[ ] Docs have been added / updated

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@bb7ac7b). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             master       #33   +/-   ##
==========================================
  Coverage          ?   100.00%
==========================================
  Files             ?        18
  Lines             ?       284
  Branches          ?        33
==========================================
  Hits              ?       284
  Misses            ?         0
  Partials          ?         0

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 bb7ac7b...333981b. Read the comment docs.


[ZH-CN] Feedback

Check out a review request at GitLocalize

@lxwzy and @yuxizhe Can you sign the CLA?

@agektmr I suggest we close this one as this is an old PR, and the old version of GitLocalize messed up the script part.


SubDag Support

Hey!
I was testing some use cases to use it at our company, and was wondering if there is any support for SubDag operator?
I played around a bit and couldn't make it work.
Congrats on the great lib.
Cheers

I haven't tried to use SubDags withdag-factory yet, but I don't see why it couldn't be extended to handle them if it doesn't work now. I'll see if I can tackle it soon.

SubDags have been deprecated since Airflow 2.0 and will be removed in AIrflow 3, so we should not invest in this area.


Support HLG decoding for models trained using CTC loss

A few days ago wenet merged a PR about using HLG decoding from k2. However, it supports only Python.
I think we should also support HLG decoding with C++ in sherpa.
People can either use sherpa or wenet for decoding if there have a conformer-ctc model or any model trained with CTC loss.
It provides another alternative for users.

Closing as it is completed.


Redirect on catalogue API going to localhost:8888

James noticed that https://api.wellcomecollection.org/catalogue/v2/works/m66pav3d should redirect to https://api.wellcomecollection.org/catalogue/v2/works/pusr37n3.
Instead it is redirecting to https://localhost:8888/catalogue/v2/works/pusr37n3

Fixed in https://github.com/wellcometrust/catalogue/pull/17, needs deployment

I've deployed this to stage and it's 500'ing; I'll dig into it more later.

I have a newer fix for this in https://github.com/wellcometrust/catalogue/pull/20, and it’s deployed to live.

@alexwlchan So can this be closed now?

@pollecuttn Not until the code is reviewed and merged.


[HUDI-6448] Improve upgrade/downgrade for table ver. 6

Change Logs
When downgrade from table version 6, record-index partition should be removed from metadata table if exists
Impact
Upgrade/downgrade flows
Risk level
Medium.

[ ] UT and e2e verification.

Documentation Update

[ ] should be highlighted in the release notes as part of the migration guide

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

manually verified the flow 0.13.1 -> 0.14.0-SNAPSHOT (this PR)
before upgrade
hoodie.table.version=5
hoodie.table.metadata.partitions=files

upgrade
./hudi-cli.sh
connect --path /tmp/hudi_trips_13_1_to_14_0_COPY_ON_WRITE
upgrade table --toVersion 6 --sparkMaster 'local[2]'

after upgrade
hoodie.table.version=6
hoodie.table.metadata.partitions=files

write data with RLI enabled
hoodie.table.version=6
hoodie.table.metadata.partitions=files,record_index

RLI partition and hfiles created
downgrade
downgrade table --toVersion 5 --sparkMaster 'local[2]'

after downgrade
hoodie.table.version=5
hoodie.table.metadata.partitions=files

RLI partition is removed
➜ ll /tmp/hudi_trips_13_1_to_14_0_COPY_ON_WRITE/.hoodie/metadata/record_index
ls: /tmp/hudi_trips_13_1_to_14_0_COPY_ON_WRITE/.hoodie/metadata/record_index: No such file or directory

Fixed a failing test and have pushed an update.


Update cargo for 1.0 beta

@brson: no appropriate reviewer found, use r? to override

r? @alexcrichton


Draft: consume js inputFileSystem on the rust side

Summary
resolves #5091

Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

Hi guys, I have been struggling to investigate the CI "Test Node 18" failure for 2 days. I feel I really need your help.
I believe my implementation can pass all the tests if the tests are run separately.
However, the tests will always be stuck in the config/builtin-swc-loader/swc-plugin case, when they are run concurrently.
My findings so far are that it's stuck in https://github.com/web-infra-dev/rspack/blob/937153100e644b068aef9a78d52844ef9324d9a0/packages/rspack-test-tools/src/runner/runner/cjs.ts#L142
even if the output is the same.

thanks ,it's may related to vm support
I think the biggest blocker now is  rspack_resolver doesn't use async_fs, which cause it has to block_on on fs binging call, which will introduce huge performance regression https://github.com/web-infra-dev/rspack-resolver/issues/34.

thanks it maybe related to performance regression, block_on fs binding will cause huge performance regression
I think the biggest blocker now is rspack_resolver doesn't use async_fs, which cause it has to block_on on fs binging call, which will introduce huge performance regression web-infra-dev/rspack-resolver#34.
but we can still merge this pr first if we don't enable input_fs binding by default and when we solve the rspack_resolver async_fs problem, we can enable inputfs binding

yeah, I also suspect the performance regression, but cannot prove it.
I'm afraid we cannot merge this PR now, because compiler.inputFileSystem has been set by default.
https://github.com/web-infra-dev/rspack/blob/5ad5803a0990e4b153c2e9ece0e8933406f4d6cc/packages/rspack/src/rspack.ts#L53-L62
and
https://github.com/web-infra-dev/rspack/blob/9e1205cd17937f624dc4dc4d419641f3b58afa9c/packages/rspack/src/node/NodeEnvironmentPlugin.ts#L53
if you feel we can use the refactors in this pr, I can create a new pr and cherry pick the refactor commits.

yeah, I also suspect the performance regression, but cannot prove it.
I actually tried it before and revert cause huge performance regression introduced, you can test it in project with large modules like https://github.com/hardfist/performance-compare-ng/tree/master/apps/10000

I'm afraid we cannot merge this PR now, because compiler.inputFileSystem has been set by default.
that's easy, you can just ignore the inputFileSystem in the rust side, it's actually current's behavior

I disabled inputFileSystem on the js side in this PR.
I also created another branch and cherry-picked the refactor commits. maybe #8654 is exactly what you expect.

I disabled inputFileSystem on the js side in this PR.
I also created another branch and cherry-picked the refactor commits. maybe #8654 is exactly what you expect.

yes, good job!


Differentiate between empty lines at beginning and end

To me, the spacing between the class definition and the first method makes this a little difficult to read.
class MyAwesomeClass < MyAwesomeModule::MyAwesomeParentClass
  def my_awesome_method(awesome_parameter)
    ...
  end

  def another_awesome_method
    ...
  end

  def awesome?
    ...
  end
end

Rubocop provides the Layout/EmptyLInesAroundClassBody style, which allows me to add spaces inside the class.
class MyAwesomeClass < MyAwesomeModule::MyAwesomeParentClass

  def my_awesome_method(awesome_parameter)
    ...
  end

  def another_awesome_method
    ...
  end

  def awesome?
    ...
  end

end

However, that extra space at the bottom feels unnecessary to me. At the top, it helps with the readability of the method name and lets me group those two things together. At the bottom, it's just inserting extra whitespace.
What I really want to do is set the line spacing separately for the top and bottom of the class. My ideal would look like this:
class MyAwesomeClass < MyAwesomeModule::MyAwesomeParentClass

  def my_awesome_method(awesome_parameter)
    ...
  end

  def another_awesome_method
    ...
  end

  def awesome?
    ...
  end
end

I could also see adding these types of options for other Layout/EmptyLine styles as well, such as Layout/EmptyLineAroundBlockBodyor Layout/EmptyLineAroundModuleBody.

I consider the Rubocop rule and behaviour of EmptyLInesAroundClassBody to be correct and sensible.
But I also agree that, for whatever reason, if someone really really wants to use the above, then why not?
I assume it may be trivial to add, since one could use the existing cop rule, and just not add a newline to the bottom part.
You may probably have to convince bbatsov either way.
PS: I think it may be easier if you just focus on one suggestion in this issue request; the other three could be added at a later time. Sometimes it takes a little while for things to change.
What name would you give the new rule by the way? Coming up with a new name is often difficult; old name is EmptyLInesAroundClassBody (probably misspelled? EmptyLinesAroundClassBody .. but let's ignore the typo anyway)

Thanks for the quick reply! I also tend to use the above style in specs, because it helps the file to be more readable without the end getting too spaced out.
describe "MyAwesomeClass" do

  describe "#my_awesome_method" do

    context "when it's awesome" do

      context "and when the parameter is awesome" do

        it "returns something else that's awesome" do
          ...
        end
      end
    end
  end
end

But I totally understand I might be in the minority in liking this coding style.
As far as naming suggestions, you split this property into Layout/EmptyLinesBeforeClassBody and Layout/EmptyLinesAfterClassBody, which seems consistent with the other Layout rules. However, that might cause more pain for other people updating their Rubocop guides. You could also add an empty_lines_only_at_beginning style, which wouldn't be as flexible, but might be simpler. (Or, you could call it landons_funky_empty_line_style. 😉)

I like the idea of adding configurations to Layout/EmptyLinesAroundClassBody rather than splitting this into 2 cops. However, the layout cops for spacing have individual cops for before and after so that would be more consistent with the rest of the code.

I've spiked out a solution, by adding an additional SupportedStyle for this cop:
Layout/EmptyLinesAroundClassBody:
  EnforcedStyle: no_empty_lines
  SupportedStyles:
    - empty_lines
    - empty_lines_except_namespace
    - empty_lines_special
    - no_empty_lines
    - beginning_only

I originally also had ending_only but after looking at the source, I skeptical of that layout.
The actual changes would happen in lib/rubocop/cop/mixin/empty_lines_around_body.rb, so it would be easy to add this option to similar cops, like Layout/EmptyLinesAroundModuleBody.
This is a middle-ground approach compared to 1) splitting into 2 cops, and 2) introducing another config option(s) like allow_beginning_empty_lines, etc.
WDYT?

That works for me! It solves my problem.

:+1:

@LandonSchropp I don't think the solution as proposed will work for RSpecs.

Late to the party, but as long as we're considering modifying this Cop, I'd love it if I could exclude include and extend from the empty line required cop. That is, I like the empty line between class and the first def, but I prefer no line between class/module and the first include/extend.
Example:
class MyThing
  extend Forwardable

  attr_reader :thing
  delegate %i( a b c ) => :thing

  def initialize(thing)
    @thing = thing
  end

end

I don't know if other class methods should also be excluded from the cop, like attr_reader or delegate, but I do feel that include and extend are "special", and should be permitted w/o the empty line.

Nevermind, now I see that was added as empty_lines_special in #3624.


Use a pure Ruby implementation for Growl instead of RubyCocoa

Instead of skipping the Growl code entirely on non-RubyCocoa enabled Rubies we probably want to use: https://github.com/drbrain/ruby-growl.

+1 — RVM doesn't seem to install the RubyCocoa libraries.

The downside of this is that people need to run the Growl server ("Listen for incoming notifications"). I haven't found a way around it, I assume there is none.

Does it break kicker if Growl isn’t listening for notifications? Otherwise I’d say it’s good enough.

Would relying on the growlnotify cli be even simpler?

I guess we could have notification support with multiple backends and figure which one is usable.


[Package Issue]: update Publisher Name/ID for packages of NewTek to NDI

Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue
Other
Brief description of your issue
The 2 apps published under the NewTek Name are now under a new publisher name : NDI
Discussion originating this issue : https://github.com/microsoft/winget-pkgs/discussions/130027
Justification & Context for name Change :

pre-2022 : Newtek created a tech called NDI, this what later made available on winget as Newtek.NDI5Tools & Newtek.NDI5Runtime.
2022-2023 : Newtek got bought out by "Vizrt Group", but kept its name for a while, then in 2022/2023, NDI became it's own fully independant group within the "Vizrt Group" (aka being disconnected from the Newtek brand).
Since (at least) April 2023 : the Licence Agreement is solely referencing the new entity (Branded "NDI" only), and since Sept. 2023 (at least) the white paper and document published reference exclusively the new publisher Name (aka NDI)
Sept. 2023 : The Newtek brand is being retired and ceased to exist (https://www.vizrt.com/vizrt/press-center/vizrt-unites-portfolio-to-strengthen-customers-and-partners/)
2023 onwards : The package available should carry the "NDI" Name as a published as the old name is no longer in use / valid.

April 2024 they release a new version of their tool as "NDI 6" this would also break all the update/names / ID, so all the new version have been submitted without the version number for longer-term use.
Will submit PR that will proceed to "move" all the older NDI V5  from NewTek to NDI publisher.
There will be 2 PR (as there are 2 apps) linked to this issue.
Steps to reproduce

Actual behavior
Publisher of app under NewTek shoul dbe moved /redirected to NDI
Expected behavior
NewTek folder/naming and app dissapear fromt he repo in favor of the NDI ones.
Environment
-

Screenshots and Logs
No response

The publisher name change is painful. I will not push further for it and let the package be deleted in couple weeks / months once the new package with the proper name will have more adoption.


JSON serialization with .NET Core 3.0

.NET Core 3.0 Preview 5 implemented a full JSON serialization stack. We should document how to use the new JSON serializer to serialize and deserialize objects. Objects can be customized with attributes to control serialization.
More information: https://github.com/dotnet/corefx/blob/master/src/System.Text.Json/docs/SerializerProgrammingModel.md
[EDIT] This issue will cover the initial publication of an overview and how-to doc. Other issues will be created as the doc set is developed.

Can the default be to not mangle property names on serialization (keep whatever case the properties have)? Or at least make it really easy to set without fidling with varied option objects

@devel112

Can the default be to not mangle property names on serialization (keep whatever case the properties have)? Or at least make it really easy to set without fidling with varied option objects

That is in fact the default (keep the same case).  Feel free to review the initial docs PR and let me know if there are any other questions you have that the new how-to doc doesn't answer.


Fix ALMA antenna configuration

The ALMA configuration should not have this many antennas. This needs fixing.

Devin Crichton Today at 3:13 PM
Regarding the presentation, why does ALMA have antennas and stations?
12 replies
Marc Audard 2 hours ago
If I may intervene here, perhaps to help, not sure I understand the question.
Rohit Sharma 2 hours ago
Yes, that is one issue we need to sort it out. So, there is only one station.
Marc Audard 2 hours ago
OK, as I am not involved I may not catch the issue. ALMA has many antennas with many stations, with the goal of having different configurations.
Devin Crichton 2 hours ago
To clarify the question, it looks like ALMA is being simulated like an SKA-low-like system but with the ALMA array layout for stations.
Mark Sargent 1 hour ago
Stations = antenna foundations (there are 192 of them, which can accommodate the 66 antennae in a range of different configurations)
:+1:
1
Devin Crichton 1 hour ago
Yeah, but the image had the orange points as stations with many blue antennas per if I saw correctly.
Mark Sargent 1 hour ago
Would you be able to post the image here?
Christoph Vögele 1 hour ago
Of course. I have the image here. It may very well be it needs fixing.
alma.png
alma.png
Rohit Sharma 1 hour ago
Yes Devin, at the moment the implementation of a dish is equivalent to a station with one antenna with modified beams. (edited)
Devin Crichton 1 hour ago
Ok, so that's not a grid of antennas per station, just one.
Devin Crichton 1 hour ago
(Sorry I think there is some overlapping terminology here, I assumed the SKA-low terms since OSKAR is being used)
Rohit Sharma 1 hour ago
The terminology is explained here: https://ska-telescope.gitlab.io/sim/oskar/telescope_model/telescope_model.html

We can clarify this in the documentation.

@rohitcbscient any updates on this?

The documentation needs to be updated. Not done yet.

Manual is here https://i4ds.github.io/Karabo-Pipeline/development.html#update-documentation

A brief readme for telescope directory structure is added.


Enum to String

not properly handling string input?

Still wasn't working...  New solution requires converting incoming data into variant....and then dealing with a variant inside a variant:


Several Weird Issues with the firefox driver

So, I randomly get this error, if the process takes ~4 seconds to do its thing: Failed to decode response from marionette, and then I also can't seem to rescale the resolution and its stuck on 720p. Any idea what might be causing these things?

This isn't a problem with this package, but likely with Geckodriver:
https://github.com/mozilla/geckodriver/issues?utf8=✓&q=is%3Aissue+is%3Aopen+"Failed+to+decode+response+from+marionette"
Please contribute to one of those bugs or open a new one in that repository with debugging logs.


[Adobe Flash & Connect: bug?]

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2019.03.18. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[ x] I've verified and I assure that I'm running youtube-dl 2019.03.18

Before submitting an issue make sure you have:

[x ] At least skimmed through the README, most notably the FAQ and BUGS sections
[ x] Searched the bugtracker for similar issues including closed ones
[ x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2019.03.18
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
Link available for the next 30 days (except on March 30th due to
maintenance)
Login/subscription required for watching?? Perhaps with this url it is NOT
needed, because I simply pasted the url in ANOTHER browser without login nor
a previous stored login.
Anyway...I could simply send to you my ID and passw via PM!
https://asppinext.adobeconnect.com/verso-lesame-di-stato_w1_reali_turazza/?session=em2breezwnfpchek9n2zksg9
In my browser:
https://postimg.cc/McTSWxv0
Nobody, according with the chat, was able to download this Flash embedded
content that remains 30days available after the live streaming. A mystery?
LOG:
C:\Users\admin>youtube-dl -v https://asppinext.adobeconnect.com/verso-lesam
e-di-stato_w1_reali_turazza/?session=em2breezwnfpchek9n2zksg9
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://asppinext.adobeconnect.com/verso
-lesame-di-stato_w1_reali_turazza/?session=em2breezwnfpchek9n2zksg9']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2019.03.18
[debug] Python version 3.4.4 (CPython) - Windows-Vista-6.0.6002-SP2
[debug] exe versions: ffmpeg N-92765-g2744d6b-Reino, ffprobe N-92765-g2744d
6b-Reino
[debug] Proxy map: {}
[generic] ?session=em2breezwnfpchek9n2zksg9: Requesting header
WARNING: Falling back on generic information extractor.
[generic] ?session=em2breezwnfpchek9n2zksg9: Downloading webpage
[generic] ?session=em2breezwnfpchek9n2zksg9: Extracting information
ERROR: Unsupported URL: https://asppinext.adobeconnect.com/verso-lesame-di-
stato_w1_reali_turazza/?session=em2breezwnfpchek9n2zksg9
Traceback (most recent call last):
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpc
s52imf5\build\youtube_dl\YoutubeDL.py", line 794, in extract_info
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpc
s52imf5\build\youtube_dl\extractor\common.py", line 529, in extract
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpc
s52imf5\build\youtube_dl\extractor\generic.py", line 3320, in _real_extract
youtube_dl.utils.UnsupportedError: Unsupported URL: https://asppinext.adobe
connect.com/verso-lesame-di-stato_w1_reali_turazza/?session=em2breezwnfpche
k9n2zksg9

Where is its duplicate?

#20283 obviously its link has expired and should be replaced by this 20463.

Here an updated link available for the next 29 days.
Please fix the extractor.
Webinar is very important both for students and professors
https://asppinext.adobeconnect.com/verso-esame-stato-diff-neri-olivieri-w2/?session=em2breezxevy2czzd3tn9ug8


Vector2: Added static methods for angleDeg and angleRad

Added two static methods to Vector2.
public static float angleDeg (float x, float y)
public static float angleRad (float x, float y)

This makes it possible to calculate an angle without having to use Vector2 instances or the like and is in line with the other static methods in Vector2:
public static float len (float x, float y)
public static float len2 (float x, float y)
public static float dot (float x1, float y1, float x2, float y2)
public static float dst (float x1, float y1, float x2, float y2)
public static float dst2 (float x1, float y1, float x2, float y2)

@tommyettinger if you're not opposed, I'd say we go ahead and merge this.

@tommyettinger if you're not opposed, I'd say we go ahead and merge this.

I think so too. We may need to fix the changelog for this, and the GWT 2.10.0 PR that was merged today as well.


[WFCORE-4207]: Add capabilities in the Thread subsystem resource definitions.

Adding capabilities and registering services with the new names.

JIRA: https://issues.jboss.org/browse/WFCORE-4207

Core - Full Integration Build 8226 outcome was UNKNOWN using a merge of c53be004fd3f356f45c5eef3f43f857358410722
Summary: Canceled (Error while applying patch; cannot find commit ebc9320e15f87a0b091dc095480e47d047827e7f in the https://github.com/wildfly/wildfly-core.git repository, possible reason: refs/pull/3623/merge branch was updated and the commit selected for the ... Build time: 00:00:22


For master branch, mybatis-parent 28-snapshot isn't available in maven repo

So if I clone the project, I can't build correctly

Its in the snapshot repository.  Look at the mybatis parent pom.  It shows the distribution local.  By default maven is configured to see both releases and snapsots.  If you think it missing from snapshots location after checking please let us know.  However im sure you will see it otherwise travis ci should be failing.
Get Outlook for Androidhttps://aka.ms/ghei36
On Thu, Aug 18, 2016 at 5:03 AM -0400, "Rasefon Wang" <notifications<EMAIL_ADDRESS>wrote:
see https://mvnrepository.com/artifact/org.mybatis/mybatis-parent

You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/mybatis/mybatis-3/issues/767#issuecomment-240665438, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AA7ho7YsjwxgYAqegiK5HlpEVki9ujOaks5qhB_TgaJpZM4JnR5R.

Ok, thank you.


GEOMETRY-60: Bumping Jacoco Version

Bumping Jacoco version to 0.8.4 for compatibility with JDK 13.

Fixed in commit 687412e982d4d088716090a310159c782f0c09be


I want a function to BlockService calculating merkleComponentHash from TransactionHashe and cosigner PublicKey.

Expected behavior and actual behavior.

If I want to use BlockService.validateInBlock , merkleComponentHash must be calculated from TransactionHash in advance.
I think it would be useful if validateInBlock works just by specifying transactionInfo.

Steps to reproduce the problem.

I have confirmed that I can calculate merkleComponentHash by inserting the following logic.
However, I have no idea what kind of interface I should use.
var leaf = transactionHash;
if( cosignatures !== undefined && cosignatures.length > 0){

	const hasher = jssha3.sha3_256.create();
	hasher.update(Buffer.from(transactionHash, 'hex'));
	for (cosignature of cosignatures ){

		hasher.update(Buffer.from(cosignature.signer.publicKey, 'hex'));
	}
	leaf = hasher.hex().toUpperCase();
}

I'm in no particular hurry.
Please consider this.thanks.

Related to #481


Adds pods property to configuration

This PR adds a new property pods to the configuration class. This is helpful when you want to add new pods that are not yet on your Podfile to the synchronisation.

There is no CI here?

@pietbrauer there is. I think you were too fast... probably because push notifications while using Git2Go? :wink:

@Ruenzuo Right, sorry. The new UI is confusing. But nevertheless I like the additions :wink:

@pietbrauer thank you Piet :blush:


e2e/txsource.go: Fix golangci-lint G204 error

fix for newer golangci-lint versions

also updating golangci would make sense :)

Replaced make lint's --deadline with --timeout flag.

also updating golangci would make sense :)

Should I just bump the version here or is there something else I missed? https://github.com/oasislabs/oasis-core/blob/master/docker/development/Dockerfile#L7

New oasislabs/testing:0.3.0 and oasislabs/development:0.3.0 docker images are now pushed.


How to cause the client browser reload if I update the next version of viewer?

I Find if I update the next version of viewer, the client browser don't reload js script because of the browser's cache mechanism. I should press Ctrl+F5 or clear the browser's cache data then cause client browser reload.
I've tried to change the version number of version.js in a directory '\platform\viewer\src', but can't cause all reloads

@JiangWeian, the service worker should auto-update if it detects a new version. Are you setting cache headers on the server? Do you have a service worker that won't uninstall? What version do you have? What version are you unable to update to?

@dannyrb, what's you said 'service worker'? Is a software? It can be installed in an environment without network connection? Because I have some clients whose server machines are not connected to the Internet

https://developer.mozilla.org/en-US/docs/Web/API/Service_Worker_API


Ingestion: Airflow 2.5.1 SourceUrl, throws 404

Affected module
Does it impact the UI, backend or Ingestion Framework?
Ingestion
Describe the bug
A clear and concise description of what the bug is.
Airflow SourceUrl for airflow 2.5.1 is different, throws 404 with current implementation
Actual
https://xxxxxx.airflow.amazonaws.com/home/tree?dag_id=pipeline__analytics__claim_transactions
Expected
https://xxxxxx.airflow.amazonaws.com/dags/pipeline__analytics__claim_transactions/grid
OM version: 1.2.3
https://openmetadata.slack.com/archives/C02B6955S4S/p1704944260291739

thank you

It looks like AWS is doing its own instead of using the OpenSource Airflow URL format.
The issue here is we are using Airflow connector and we are forming the URLs based on their Airflow UI link format

hi team, I tested with version 1.3.0 (after upgrade) however issue remain same with version 1.3.0

hi team, I tested with version 1.3.0 (after upgrade) however issue remain same with version 1.3.0

i am also facing the same


schema: multi-tenancy

Summary
The pr allows the users to pass schema as tenant in headers if the users wants multi-tenancy into their application .
Also it removes Form() reading from /basic/user and uses request.json() to read from body as pydantic does not reset the pointers and causes stream consumed error

Test Plan

[ ] PR has an associated issue: #1122
[ ] make check passes
[ ] make test shows 100% unit test coverage

Deployment Plan

I have updated the implementation to only route to different schema and remove creation of schema code as discussed with @samredai

Wondering if this is close to finalization? Anything we can do to help ?

@ockhamlabs sorry for the late reply here. Let's revisit this after this refactor for the query service goes in which should make this ultimately much easier to achieve. I'm planning to catch up with @sadath-12 soon, either during the community meeting or async.

(after talk with @samredai )
Note: Whenever we switch across schemas make sure u use existing connection pool

@sadath-12 super sorry for losing track of this. I think we're close to figuring this out--this PR is doing the right thing by parsing the tenant value from the request header, but let's save any of the changes in the datajunction-query directory or to the alembic/env.py file for a follow up PR. If you could update this PR to just include the changes to datajunction_server/config.py and datajunction_server/utils.py, we can iterate on that and get this PR merged pretty quickly.


Handle variadic and not declared arguments in stack traces

This pull request fixes #1645.
Short summary: SDK wasn't sending some function arguments to Sentry, namely default, variadic, not declared arguments and arguments with value null. This pull request reworks function responsible for handling parameters in order to send all of the above.
I tested my changes with all inputs variations I could think of, it seems to be working. However, I don't quite understand how to add tests for it. Testing this method requires using stack trace with real function or real class and method. Where do I create them? Current FrameBuilder tests don't do anything similar, so I'm not sure what would be the right way to do it.
Also, PHPStan gives me Unreachable statement - code above always terminates. error on lines 254 and 267. Those statements are definitely reachable, and I confirmed it with manual testing. I tried changing method parameter from mixed[] to array<array-key, mixed> but it didn't help. Do you have an idea what could be causing this?

Creating this as draft before all mentioned issues are addressed.


Deploy reports "Invalid Id" but not service is noted

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

App Id
d1d2nf54jf2ui0
Region
us-east-2
Amplify Hosting feature
Manual deploys
Describe the bug
With no changes to my code or infrastructure I've gotten the following output in my builds. I'm not sure what ID the output is referring to or which service is involved. I've checked S3 and the deployment and hosting buckets are showing updated documents (so my thinking is that the files are at least partially updated).
CLI version 6.3.1
frontend build command exited with code 0
--
1778 | Publish started for S3AndCloudFront
1779 | - Uploading files...
1780 | ✖ Error has occurred during file upload.
1781 | An error occurred during the publish operation: Invalid id
1782 |
1783 | [Container] 2022/07/29 17:05:45 Command did not exit successfully amplify publish -y -a -c exit status 1
1784 | [Container] 2022/07/29 17:05:45 Phase complete: BUILD State: FAILED
1785 | [Container] 2022/07/29 17:05:45 Phase context status code: COMMAND_EXECUTION_ERROR Message: Error while executing command: amplify publish -y -a -c. Reason: exit status 1
1786 | [Container] 2022/07/29 17:05:45 Entering phase POST_BUILD
1787 | [Container] 2022/07/29 17:05:45 Uploading S3 cache...
1788 | [Container] 2022/07/29 17:17:38 Phase complete: POST_BUILD State: SUCCEEDED
1789 | [Container] 2022/07/29 17:17:38 Phase context status code:  Message:

The build script uses the following buildspec

phases:
  install:
    runtime-versions:
      nodejs: 14
  pre_build:
    commands:
      - npm install -g<EMAIL_ADDRESS>--unsafe-perm=true
      - npm install
  build:
    commands:
      - AMPLIFY="{\"envName\":\"dev\"}"
      - amplify init --amplify $AMPLIFY --providers "{\"awscloudformation\":{\"configLevel\":\"general\"}}" --yes
      - amplify publish -y -a -c

cache:
  paths:
    - node_modules/**/*
    - amplify/backend/api/wbbugs/build/**/*

Expected behavior
Error should be more descriptive. Root of problem is very difficult to ascertain.
Reproduction steps
Run deploy with CLI @ 6.3.1 perhaps. Reproduction is difficult as this worked and no changes have been made.
Build Settings
No response
Additional information
No response

Hello @Endlessline apologies for the delay, are you still running into this issue?

This stopped without any changes. Possibly some internal issue?

We recommend upgrading to the latest version of the Amplify CLI. If this issue reoccurs please open a new issue. Thank you!


How to change permissions on wix external database collection?

Hi,
I was able to link my SQL database to a wix collection. My issue now is that even though I have a permissions secret giving read permissions to everyone, the wix collection still shows it for admin only...can someone please point me in to the right direction? Tks!!!

Hello,
This repo is deprecated, please follow this article for connecting external database - https://support.wix.com/en/article/velo-integrating-external-databases-with-your-wix-site
If your SQL database is on GCP, go directly to this article - https://support.wix.com/en/article/velo-integrate-your-google-cloud-mysql-postgres-and-bigquery-databases-with-your-wix-site


curveadm client status failed when one client is down

If one client down and status of all clients will not work.

It seems to have been fixed in the latest code.


acesss and refresh tokens can be reused after logout

I am using express-stormpath with the angular-sdk. On logout, express-stormpath clears the access and refesh tokens from the cookie. If the old token values are manually applied e.g. via the browser console, it is then possible to perform any request that requires authentication without needing to re-login.

Looked into JWT tokens and it seems this is how they work in the absence of an explicit revocation list

In theory yes, in practice: Stormpath does maintain that revocation list for you :)
This integration should be deleting the tokens on logout, I'm flagging this to go in a release as a fix.
For me information about Stormpath's token management features, please see:
http://docs.stormpath.com/guides/token-management/


Can't evaluate values on console at breakpoint

Long time user, first time issue poster. Thanks very much for this wonderful application.
Lately I've not been able to evaluate expressions on the console during a breakpoint. I've been aware of and understand similar-feeling Buffer-related issues for some time, but lately I've not been able to evaluate any expression at all. The problem is similar in feel to the Buffer slowness, except the expressions never turn up at all, yet are available for inspection by hovering over the symbol in the code and in Scope Variables on the right:

You see evaluating path in this screenshot did not produce any results. It's a primitive string.
Not sure at all how to debug this or what information I could provide to assist. I tried downgrading with no luck (0.7.3 -> 0.7.2). Searched this issue tracker and Google with no results.
Thank you.

I know this issue has been closed, but I'm experiencing the same on<EMAIL_ADDRESS>Also my bar seems pretty messed up.
I'm using Chrome on OSX to reach Node-debug. Not sure if I should check/decheck some filter


dashboard: improve fs bugs routing

This set of patches should enable us to change syzbot routing rules to the following:

If the guilty file of the crash is fs/*.c, don't report it until we have found a reproducer. Otherwise (even if the guilty file is in e.g. fs/*/*.c) do as we do now.
Once the reproducer is found, determine the exact filesystems from the syz_mount_image$ calls.
Use this information to augment the Cc list and the title (e.g. [syzbot] [ntf3?] WARNING in ...).

For now, use the hard-coded Cc values. Some time later I'll publish a PR that parses the fresh version of the MAINTAINERS file, so it'll not be needed.
Also note that this PR contains tiny parts of the subsystem management infrastructure that's planned for implementation in the future, so the changes to database entities are more global than seems necessary. The reason for that is that it'll be much harder to alter the DB structure later, so it's better to err on the side of flexibility.

lgtm

Added one more commit and fixed the previous comments.
@dvyukov  @tarasmadan  PTAL


I can't open links

let's say that from the telegram ap pc send me a link. if I click on the link they sent me (so all the links), it does not open me ungoogled chromium (no there are no other browsers on the pc); nothing just happens

What platform does this happen on? I have ungoogled-chromium set as the default browser on linux and its working just fine

What platform does this happen on? I have ungoogled-chromium set as the default browser on linux and its working just fine

i use windows 11. (but I still had this problem on windows 10)

well this is clearly not an issue with the patchset then


feat: Chroma 0.5.x support

Closes #58

#59  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tazarov and the rest of your teammates on  Graphite


Make the rebuild command more reliable

This takes the existing vast rebuild command, and rewrites it from scratch with some significant changes (that are all backwards cmpatible):

The orchestration logic of the partitions scheduled for rebuilding moved from the vast rebuild client process into the server process. To that end, the REBUILDER actor is now a singleton component inside the NODE.

The new vast.automatic-rebuild option allows for automatically rebuilding undersized partitions in the background. This option defaults to 1, i.e., the automatic rebuilding is enabled by default.

The vast rebuild command was renamed vast rebuild start, although the old command forwards to avoid an unnecessary breaking change. The command no longer displays an indicator with the current progress; instead, the progress moved into metrics (under the keys rebuilder.partitions.{remaining,rebuilding,completed}) and the status output (vast status rebuild).

Only a single rebuild can now run at a given time. The newly introduced vast rebuild stop command allows for stopping the current run. Manually started rebuilds automatically stop automatically started rebuilds.

The vast rebuild {start,stop} commands gained a --detached option that causes the commands to exit as soon as the start and stop are dispatched instead of waiting for the run to finish.

The vast rebuild start command gained a --max-partitions option that allows for limiting the number of partitions to rebuild.

The --undersized option for the vast rebuild start command now implies --all. Partitions are now considered undersized if they are below 80% of the desired vast.max-partition-size (up from 50%).

:memo: Checklist

[x] All user-facing changes have changelog entries.
[x] The changes are reflected on vast.io, if necessary.
[x] The PR description contains instructions for the reviewer, if necessary.

:dart: Review Instructions
@tobim: Try to follow the flow of the code per command. I deployed this to one of our testbed servers, so this should already be observable on our internal dashboard.
@mavam: Changelog, docs, and draft blog post.

Code wise this looks fine. I approve that part because I didn't get around to do any manual testing yet, and maybe someone else can inspect the company internal test setup.

This has been running for 1 and 4 days respectively on our two largest setups, and the metrics dashboards show that it's running as expected.


#comment class rename TieTime => TeaTime

Hi @chaturanganamal  thanks for your help, but I am sorry to say that this was fixed with #6
We will create more subtasks soon 😃


Cannot load phonlp model

I have a problem when loading model using this scripts:
phonlp.download(save_dir='/content/phonlp')
model = phonlp.load(save_dir='/content/phonlp')

The error occurs:

Is there anything wrong with file phonlp.pt

I just updated the model URL. Please try again. Thanks!


Q client for odbc

Recreated file interfaces/q-client-for-odbc.md and added it to the yaml. This file was was erroneously deleted in https://github.com/KxSystems/docs/commit/df7298e30255b88c346ea051e17fed11e336cff4#diff-03cb1d95e1e6b34b2c971402ef[…]50eff9c10687ddc64845bfb037e7583b

looks good, thanks!


:bug: Fix base url for the new repo

Editing pages and linking to their source now works! Good catch @mudler !


Box plot

I want to visualize the box plot of 54 anatomical structures of LPBA40, can you provide the code？ Thank you！

yes. I want this LPBA40 dataset plotting script, can you publish it? Thank you very much for your work, I need it very much.

Ok. Good luck with your research. Thank you.

@1135451439 Sorry for the delay. I have uploaded the code plot/visualize.py

Thank you very much for your work. I still have a problem at the moment, which is how to save the csv file of the different anatomical structures to visualize boxplot during the inference phase. I see that lku has the ixi data set, but I don't know how to save the lpba40 data set, can you add this part of the code to your work? Thank you.


Generating certs for TLS and using them for public endpoints (DO NOT MERGE)

Ingress certs are generated for each new deployment and are
different from airship default certs.
Certs are generated via cfssl (CloudFare Utility). Airship promenade
uses same tool to generate SSL certs which are bundled in airship
treasuremap sample sites.
No TLS for public endpoints in ucp.
Added TLS for keystone, cinder, nova, neutron, hoirzon public endpoints
in openstack namespace.
Added flag for disabling TLS for ucp and osh namespace.
Need to change ingress domain suffix from 'svc.cluster.local' to
'svc.dustycloud.local' so that tls secret reference is added in k8s ingress
definition for public endpoints. This is different from upstream
treasuremap airskiff site reference. Ingress k8s definition is generated
by helm-toolkit _ingress.tpl template and should have domain different
from svc.{cluster_domain_suffix} value to add tls section.

really outdated and with conflicts, closing and we can reopen later.


API for setting initial cursor position?

It is unclear how, when a post has content and an editor enters "isEditable" mode, how to set the initial cursor position.
I see in the ... cursor.js code there is an API for selectRange(...), which is pretty cool as I can programmatically create selections in an editable post... However, I'd like to also be able to programmatically set the cursor position...
Does this mean that I should be using editor.cursor.selectRange() with a collapsed range in order to set the cursor position?
I'm just trying to clarify, and would be willing to create some documentation about this if it turns out that this is the right API usage for the functionality I'm looking for.

@mysterlune Thanks for the question! Your instinct is absolutely right here: Setting a range that is collapsed (headPosition equals tailsPosition) will move the cursor to that spot. When creating a range the tailPosition is optional and defaults to the headPosition's value, so you would create one like so:
let headPosition = ...;
let rangeToSelect = new MobiledocKit.Range(headPosition);
// or, equivalent:
// let rangeToSelect = new MobiledocKit.Range(headPosition, headPosition);
editor.selectRange(rangeToSelect);

The API I strongly suggest you use for the above is:
let headPosition = ...;
let rangeToSelect = headPosition.toRange();
// or, equivalent:
// let rangeToSelect = headPosition.toRange(headPosition);
editor.selectRange(rangeToSelect);

On a similar note, how would I get the position for the start and end of an atom?
let atom = editor.insertAtom('my-atom', '...');
let range = atom.section.toRange(); // seems to give a bigger range then I expect, like the whole line.


Set v5.0.0

What does this PR do?
Sets the version to 5.0.0.
How should it be tested manually?
yarn test
# or
npm test

Pull Request Test Coverage Report for Build 208

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 206:
0.0%

Covered Lines:
1908

Relevant Lines:
1908

💛  - Coveralls


something wrong with my code? i just following your code,3ks

ValueError                                Traceback (most recent call last)
 in ()
5                     callbacks=callbacks,
6                     validation_data=valid_generator(),
----> 7                     validation_steps=np.ceil(float(len(ids_valid_split)) // float(batch_size)))
/home/yang/anaconda3/lib/python3.5/site-packages/keras/legacy/interfaces.py in wrapper(*args, **kwargs)
86                 warnings.warn('Update your ' + object_name +      87                               ' call to the Keras 2 API: ' + signature, stacklevel=2)
---> 88             return func(*args, **kwargs)
89         wrapper._legacy_support_signature = inspect.getargspec(func)
90         return wrapper
/home/yang/anaconda3/lib/python3.5/site-packages/keras/engine/training.py in fit_generator(self, generator, steps_per_epoch, epochs, verbose, callbacks, validation_data, validation_steps, class_weight, max_q_size, workers, pickle_safe, initial_epoch)
1875                                          'a tuple (x, y, sample_weight) '
1876                                          'or (x, y). Found: ' +
-> 1877                                          str(generator_output))
1878                     if len(generator_output) == 2:
1879                         x, y = generator_output
ValueError: output of generator should be a tuple (x, y, sample_weight) or (x, y). Found: None

Have you converted the train masks to .png?

@petrosgk i have changed the 'png' to 'gif' in your code instead of converted the mask to .png. Wondering if it works.

Yes OpenCV doesn't support .gif, that's why I say you must convert the masks to .png in the readme. Glad you solved the issue.


add mysql image for m1

add mysql image for m1

 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.


Connection errors found when use listening addess '<IP_ADDRESS>'.

Bug reporting
A good bug report has some very specific qualities, so please read over our short document on reporting bugs before submitting a bug report.
To ask a question, go ahead and ignore this.

Hey,
I'm trying to run etcd with listening address '<IP_ADDRESS>', then etcd server ends up repeatedly producing the error 'transport: dial tcp [::]:3379: connect: network is unreachable'.
I'm not sure where the transport is started, but I guess you want to use the advertised addr instead of listening addr.

Outout:
$ ./etcd --listen-client-urls 'http://<IP_ADDRESS>:3379' --advertise-client-urls 'http://<IP_ADDRESS>:3379' --listen-peer-urls 'http://<IP_ADDRESS>:3380'
2017-05-22 12:01:42.648920 I | etcdmain: etcd Version: 3.2.0-rc.1+git
2017-05-22 12:01:42.648961 I | etcdmain: Git SHA: 4cd5e7e
2017-05-22 12:01:42.648966 I | etcdmain: Go Version: go1.8
2017-05-22 12:01:42.648971 I | etcdmain: Go OS/Arch: linux/amd64
2017-05-22 12:01:42.648976 I | etcdmain: setting maximum number of CPUs to 8, total number of available CPUs is 8
2017-05-22 12:01:42.648985 W | etcdmain: no data-dir provided, using default data-dir ./default.etcd
2017-05-22 12:01:42.649014 N | etcdmain: the server is already initialized as member before, starting as etcd member...
2017-05-22 12:01:42.649116 I | embed: listening for peers on http://<IP_ADDRESS>:3380
2017-05-22 12:01:42.649190 I | embed: listening for client requests on <IP_ADDRESS>:3379
2017-05-22 12:01:42.650673 I | etcdserver: name = default
2017-05-22 12:01:42.650684 I | etcdserver: data dir = default.etcd
2017-05-22 12:01:42.650689 I | etcdserver: member dir = default.etcd/member
2017-05-22 12:01:42.650694 I | etcdserver: heartbeat = 100ms
2017-05-22 12:01:42.650699 I | etcdserver: election = 1000ms
2017-05-22 12:01:42.650704 I | etcdserver: snapshot count = 100000
2017-05-22 12:01:42.650712 I | etcdserver: advertise client URLs = http://<IP_ADDRESS>:3379
2017-05-22 12:01:42.651090 I | etcdserver: restarting member 8e9e05c52164694d in cluster cdf818194e3a8c32 at commit index 4
2017-05-22 12:01:42.651118 I | raft: 8e9e05c52164694d became follower at term 2
2017-05-22 12:01:42.651129 I | raft: newRaft 8e9e05c52164694d [peers: [], term: 2, commit: 4, applied: 0, lastindex: 4, lastterm: 2]
2017-05-22 12:01:42.702847 W | auth: simple token is not cryptographically signed
2017-05-22 12:01:42.755776 I | etcdserver: starting server... [version: 3.2.0-rc.1+git, cluster version: to_be_decided]
2017-05-22 12:01:42.756689 I | etcdserver/membership: added member 8e9e05c52164694d [http://localhost:2380] to cluster cdf818194e3a8c32
2017-05-22 12:01:42.756811 N | etcdserver/membership: set the initial cluster version to 3.2
2017-05-22 12:01:42.756880 I | etcdserver/api: enabled capabilities for version 3.2
2017-05-22 12:01:43.651339 I | raft: 8e9e05c52164694d is starting a new election at term 2
2017-05-22 12:01:43.651525 I | raft: 8e9e05c52164694d became candidate at term 3
2017-05-22 12:01:43.651595 I | raft: 8e9e05c52164694d received MsgVoteResp from 8e9e05c52164694d at term 3
2017-05-22 12:01:43.651656 I | raft: 8e9e05c52164694d became leader at term 3
2017-05-22 12:01:43.651699 I | raft: raft.node: 8e9e05c52164694d elected leader 8e9e05c52164694d at term 3
2017-05-22 12:01:43.652030 I | etcdserver: published {Name:default ClientURLs:[http://<IP_ADDRESS>:3379]} to cluster cdf818194e3a8c32
2017-05-22 12:01:43.652127 I | embed: ready to serve client requests
2017-05-22 12:01:43.652565 N | embed: serving insecure client requests on [::]:3379, this is strongly discouraged!
2017-05-22 12:01:43.652762 I | etcdserver/api/v3rpc: grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp [::]:3379: connect: network is unreachable"; Reconnecting to {[::]:3379 <nil>}
2017-05-22 12:01:43.652860 I | etcdserver/api/v3rpc: grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp [::]:3379: connect: network is unreachable"; Reconnecting to {[::]:3379 <nil>}
2017-05-22 12:01:43.652946 I | etcdserver/api/v3rpc: grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp [::]:3379: connect: network is unreachable"; Reconnecting to {[::]:3379 <nil>}
2017-05-22 12:01:43.653028 I | etcdserver/api/v3rpc: grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp [::]:3379: connect: network is unreachable"; Reconnecting to {[::]:3379 <nil>}
2017-05-22 12:01:43.653106 I | etcdserver/api/v3rpc: grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp [::]:3379: connect: network is unreachable"; Reconnecting to {[::]:3379 <nil>}
2017-05-22 12:01:43.653189 I | etcdserver/api/v3rpc: grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp [::]:3379: connect: network is unreachable"; Reconnecting to {[::]:3379 <nil>}
2017-05-22 12:01:43.653266 I | etcdserver/api/v3rpc: grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp [::]:3379: connect: network is unreachable"; Reconnecting to {[::]:3379 <nil>}

/cc @xiang90

Probably related to the json gateway. I can partially reproduce this on osx, but not the exact error:
$ nc -l localhost 3379 &
$  ./bin/etcd  --listen-client-urls 'http://<IP_ADDRESS>:3379' --advertise-client-urls 'http://<IP_ADDRESS>:3379' --listen-peer-urls 'http://<IP_ADDRESS>:3380' &
...
2017-05-22 11:13:42.231696 I | etcdserver/api/v3rpc: grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp [::]:3379: getsockopt: connection reset by peer"; Reconnecting to {[::]:3379 <nil>}
2017-05-22 11:13:42.231762 I | etcdserver/api/v3rpc: grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp [::]:3379: getsockopt: connection reset by peer"; Reconnecting to {[::]:3379 <nil>}
2017-05-22 11:14:02.231361 I | etcdserver/api/v3rpc: grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp [::]:3379: i/o timeout"; Reconnecting to {[::]:3379 <nil>}
2017-05-22 11:14:02.231429 I | etcdserver/api/v3rpc: grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp [::]:3379: i/o timeout"; Reconnecting to {[::]:3379 <nil>}
2017-05-22 11:14:02.231458 I | etcdserver/api/v3rpc: grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp [::]:3379: i/o timeout"; Reconnecting to {[::]:3379 <nil>}
2017-05-22 11:14:02.231683 I | etcdserver/api/v3rpc: grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp [::]:3379: i/o timeout"; Reconnecting to {[::]:3379 <nil>}

No luck on Linux, get etcdmain: listen tcp <IP_ADDRESS>:3379: bind: address already in use instead. What linux distro is this / is there any special configuration or anything else running on the system?

On linux machine, I've also tried both master branch and 3.2.0-rc.1, and works fine?

@heyitsanthony You may try 2379 instead, I'm using 3379 only because I have another etcd instance occupied 2379.
@gyuho It seems that to reproduce the error depends on system or network configuration somehow. I've also noticed on some system/machine it's ok to dial '<IP_ADDRESS>' to establish a connection. But I think the problem is we should always dial the advertised address instead of <IP_ADDRESS>.

https://github.com/coreos/etcd/blob/f75e33326406a4f05bd2a92ddf214ec4018e5b0b/embed/serve.go#L167 may be related to this issue. It uses the listener's addr to register some handlers.

@disksing yes, it's from the grpc json gateway dial-out. I'd like to have a way to reproduce this bug instead of blindly using the advertise address and saying it's fixed.

@heyitsanthony If you can not easily reproduce the problem, perhaps you may fix it blindly then let me to help the test.

@disksing I really need to know the root cause here. I don't think using advertise address is the right fix-- the machine could be NATed in such a way that going through the advertise address would need more hops than serving directly off the listen address or possibly not even be able to connect at all.

1
Which version is the last version that is working? Is there a estimation when you will fix this issue?

Thanks

+1
It would be really required to get this fix done

@ghovat @carlitos26 is there a way to reliably reproduce this?

Got this bug with running etcd on alpine:edge testing repository, Don't know if was fixed in 3.2.1 version or not, but here's the way how to reproduce it:
etcd --version
etcd Version: 3.2.1
Git SHA: 16b3950000
Go Version: go1.8.3
Go OS/Arch: linux/amd64

Dockerfile.alpine (nfox/etcd:alpine docker image):
FROM alpine:edge

RUN set -x \
    && sed -e 's/main/testing/' -i /etc/apk/repositories \
    && apk update \
    && apk add etcd etcd-ctl

CMD ["etcd"]

Docker container run script:
HOST_IP=<IP_ADDRESS>

docker run -d -p 4001:4001 -p 2380:2380 -p 2379:2379 \
  -v /usr/share/ca-certificates/:/etc/ssl/certs \
  --name etcd nfox/etcd:alpine etcd \
  --name etcd0 \
  --advertise-client-urls http://${HOST_IP}:2379,http://${HOST_IP}:4001 \
  --listen-client-urls http://<IP_ADDRESS>:2379,http://<IP_ADDRESS>:4001 \
  --initial-advertise-peer-urls http://${HOST_IP}:2380 \
  --listen-peer-urls http://<IP_ADDRESS>:2380 \
  --initial-cluster-token etcd-cluster-1 \
  --initial-cluster etcd0=http://${HOST_IP}:2380 \
  --initial-cluster-state new

@tarvitz this is fixed in 3.2.2


User-define imagePullPolicy

When deploying on certain types of clusters, i.e. bandwidth limited edge clusters, it's preferable to not pull images unless absolutely necessary.
It would be great to be able to define a global imagePullPolicy for the operator that all ArgoCD instances and their component services respect.
It would also be suitable, though tedious, to be able to pass this config along in the various component config sections of an ArgoCD object (e.g. controller.imagePullPolicy, dex.imagePullPolicy, etc.)
For context, here's what I'm seeing as "actual" imagePullPolicy of or various components:
argocd-application-controller                    Always
argocd-dex-server-xxx                      Always
argocd-operator-xxx                       Always
argocd-redis-ha-haproxy-xxx               IfNotPresent
argocd-redis-ha-server                           IfNotPresent,IfNotPresent
argocd-repo-server-xxx                    Always
argocd-server-xxx                          Always

Would be fantastic if the default for all of these were set to IfNotPresent


Being prompted twice for password and sudo-password when they are the same value is annoying

Use Case
Users often want to use the same value for password and sudo-password, and want to use the prompt plugin to get the value. Right now that means using the prompt plugin in 2 places, and having to enter the same value twice, which is annoying.
Describe the Solution You Would Like
I have a few idea for how we could solve this:

Provide a config option to use the password value for sudo-password, if sudo-password is needed.
Create a new plugin or sigil that will load another config value as the value for that option:

password:
  _plugin: prompt
  message: Please enter your password:
sudo-password:
  _plugin: reuse
  config_option: password

Add a name attribute to the prompt plugin and only prompt once for each named prompt. This would mean that a prompt could be used throughout the inventory.

Additional Context
Slack convos: 1, 2, 3

I don't think it's possible to create a new plugin that references other values without opening up a lot of unwanted complexity to the plugin API.
I do think we could allow named references as a general extension to the resolve_reference functionality. Given that references are lazy and can be recursive it would be easy for users to create broken inventories that are difficult to debug but it should be possible.
I'm open to suggestions but the best way I can think to implement this is probably a new _reference(bikeshed) special key.
- config:
   password:
     _plugin: prompt
     _reference: password
    message: "enter passworf
  sudopassword:
    _reference: password

A better interface for this would allow users to define references outside any section of the inventory and then use them anywhere.
It makes sense intuitively for references to be scopeable to a section but I suspect this will make the error cases too.

It would be really nice to reference other config values, without it we have a lot of copy/paste in our inventory files:
groups:
  - name: windows
    config:
      transport: winrm
      winrm:
        user<EMAIL_ADDRESS>        password: xxx
        ssl: true
        ssl-verify: false
    groups:
      - name: windows_servers_a
        vars:
          patching_order: 1
        targets:
          - _plugin: wsus_inventory
            host: 'sql01.domain.tld'
            database: 'SUSDB'
            username: 'DOMAIN\bolt_sql_svc'
            password: 'xxx'
            group: Servers_A

      - name: windows_servers_b
        vars:
          patching_order: 2
        targets:
          - _plugin: wsus_inventory
            host: 'sql01.domain.tld'
            database: 'SUSDB'
            username: 'DOMAIN\bolt_sql_svc'
            password: 'xxx'
            group: Servers_B

      - name: windows_servers_c
        vars:
          patching_order: 3
        targets:
          - _plugin: wsus_inventory
            host: 'sql01.domain.tld'
            database: 'SUSDB'
            username: 'DOMAIN\bolt_sql_svc'
            password: 'xxx'
            group: Servers_C

So, if this service account password changes, i now have 3x places i need to update it.


Integrate eclipse 2022-06, click the database interface to prompt the error message could not accept changes

System information:

Operating system (distribution) and version
Windows 10
DBeaver version
22.1.1
Additional extensions

Connection specification:

Database name and version
mariadb 10.6.8
Driver name
jdbc:mariadb-java
Do you use tunnels or proxies (SSH, SOCKS, etc)?

Describe the problem you're observing:
Integrate eclipse 2022-06, click the database interface to prompt the error message could not accept changes (the currently displayed page contains invalid values)
Steps to reproduce, if exist:
eclipse window -preferences-Database-User Interface
Include any warning/errors/backtraces from the logs

Hello @lyh061619,
Can you please provide more information regarding your issue?
Is DBeaver installed as a standalone application or as an Eclipse plugin?

Hello @lyh061619,
Can you please provide more information regarding your issue?
Is DBeaver installed as a standalone application or as an Eclipse plugin?

Install as an eclipse plugin plug-in


Example has no Navigation Bar Buttons

The demo gif shows the navigation component with Back buttons and has several properties that indicate  customization of left and right navbar buttons. I was unable to get these to work locally, and after pulling the example code, it is not working there either.

Feel free to improve it by sending PR.

The issue is not with the example itself, it's that I believe the underlying code isn't working as intended. In neither the example nor my own use can I actually get navbar buttons to display — back or otherwise — using any of the exposed functions.

@wkirby Did you try adding onRight to the props in <Route>?

I am having a problem like this aswell. I can only seem to get the buttons to render using renderRightButton if they are on a <Route... /> that is a direct child of the main <Router .../> component. Any ideas?

I wonder if this is at all related to my issue #304?
I've just downloaded and run the demo, and feel its quite misleading - I've spent the last couple of hours trying to understand why my navbar does not show back buttons etc and now I see the example app does not do that either. The animation shows back buttons etc so the example code should be updated or the example animation changed.

@jeduan Yes. So far, nothing I've been able to do with this library adds navigation buttons to the navbar. It sounds like @RealOrangeOne  is having the same problem.
I haven't had time to debug this, but for now we've decided to switch away from this library in favor of the native navbar.

Are you guys on the latest version? I had a similar problem and updating to latest (2.3.12) sorted it all out for me.

I see the NavBar.js in the Examples but can't find it used anywhere. How do I hook it up. I thought this could be done by setting header={NavBar} in the main Router?
Anyone has a working example I can look at?
Thanks :)

Check 3.0 release, now the component uses new React Native Navigation API and I've added nav bar buttons to Example. However it has a little bit different API comparing with 2.x release. Anyway I'm closing this ticket because 2.x release will not be supported except critical bugs.


Change default behavior to generate only the list of packages used, with further dependencies optional

Feature Request
As a user i would rather have the main, default behaviour of this tool to be one that generates a list of the bare minimum
requirments for my code. In short, let the packages handle their own dependencies, and just give me what is needed in
imports in the code.
What it would look like
if i had a package like this:
mysource/mysource.py
import foo
from bar import baz

foo(baz)

Where we know that foo has a dependency on spam
Then running:
realreq -s mysource

would output:
foo==1.0.0
bar==1.0.0

but something like:
realreq -s mysource --freeze

would output:
foo==1.0.0
bar==1.0.0
spam==1.2.1

Merged to dev


No module named 'holobot'

Describe the bug
I am trying to run the fish training. When I launch the program, a compilation error appears.
Error Messages
Traceback (most recent call last):
  File "train_hand.py", line 19, in <module>
    from replay_buffer_hand import ReplayBufferStorage, make_replay_loader, make_expert_replay_loader, make_ssl_replay_loader
  File "./FISH/FISH/replay_buffer_hand.py", line 13, in <module>
    from holobot.robot.allegro.allegro_kdl import AllegroKDL
ModuleNotFoundError: No module named 'holobot'

Environment(for bug reports)

[x] Operating System: Linux version 4.15.0-147-generic (buildd@lcy01-amd64-028) (gcc version 7.5.0 (Ubuntu 7.5.0-3ubuntu1~18.04))
[x] Conda: All environment dependencies are consistent with the provided file conda_env.yml

Steps to Reproduce the Issue
I followed the instructions in README
conda env create -f conda_env.yml
conda activate fish
python train_hand.py agent=bc suite=hellogym suite/hellogym_task=dooropen num_demos=1

I also tried to install using
conda install holobot
pip3 install holobot

but both of the results show that the package cannot be found.
Would you know how to solve this problem ?

It seems like holobot is somehow related to the Holo-Dex project and this might be an internal package to the CILVR lab. @siddhanthaldar can you please verify this and let us know what the steps forward are?

Hi,
Apologies for the late reply. @varunagrawal is correct that holobot was an internal package at the time and had been derived from the Holo-Dex project. You can follow the instructions there to set it up.
We recently released an improved teleoperation system, Open Teach which has the VR-based teleop code and all robot controllers public. You might want to take a look since it also supports other robots and has a detailed documentation about setting it up.
Hope that helps!


How to update polyline according to the user current location?

Call this library with the new location then redraw the polyline on the map

@akexorcist This approach will consume a lot of API requests which will cost more.

@islamarr Absolutely yes. But no way to update the new route when user is outside the route direction with only single request from Direction API

@akexorcist I think we can! by using polyline.setPoints(points);
check this
https://stackoverflow.com/a/17038499/7826372
But still I can't use it through this library.

This library is great, but can I get the polyline created, so I can update just the polyLine. So that I can avoid calling the API request again


If runs are executed in local, results should allow to save screenshots on same zafira reporting server.

Pre-requisite :

Tests are build using carina framework and executed from local machine
Tests are having capability to send results to the Zafira reporting server.
Zafira reporting server will be treated as hub for all reports.

Steps:
Whenever we execute tests on local results are reflected on the Zafira reporting server.
But, screenshots related to tests are available only on a local machine.
Enhancement: Screenshots of existing test run should be available on the Zafira reporting server. So all test artifacts(test run logs and screenshots) are available at the same place.

@prasadjshimpi as of right now S3 integration should be enabled, otherwise it won't work (this is expected flow).
Closing this right now. We might come up with some sort of alternative flow for screenshots but that should be tracked as separate item.
cc: @vdelendik


direct messaging to broadcast user specific details (which differs between users), to a closed group

I am aware we can broadcast an already posted photo to a closed circle using InstagramDirectShareRequest as shown in readme.
instagram.sendRequest(InstagramDirectShareRequest.builder(ShareType.MEDIA, recipients).mediaId(mid).message(message).build());
Is it possible to make use of direct messaging to broad cast user specific details ( which differ from one user to another ) to a closed user group?

Are you able to execute the code you posted?
instagram.sendRequest(InstagramDirectShareRequest.builder(ShareType.MEDIA, recipients).mediaId(mid).message(message).build());
because I can't
Instagram responds with "Oops, an error occurred"


InTheHand.Devices.Bluetooth any complete examples ?

I could find any examples related to the new API

https://archive.codeplex.com/?p=32feet

@pirhano, I'm also looking at trying to get a working sample up and running, and what I've found in the documentation does not match up with what I'm seeing in the nuget package.  It's very confusing.
I'm trying to follow this example on device discovery:
var cli = new BluetoothClient();
BluetoothDeviceInfo[]() peers = cli.DiscoverDevices();
BluetoothDeviceInfo device = null;//= ... select one of peer()...
BluetoothAddress addr = device.DeviceAddress;

But I can't find any of the classes in the nuget package.  I can find them in the repository, but it seems to be a completely different API than the one exposed on nuget.
The code looks like exactly what I'm looking for, but I've no idea where the classes are located.
😕

To @matthewjones555's comment, the documentation available all seems to be for the old API.. Both the BluetoothListener and BluetoothClient classes have been replaced. The v4 package is still in preview, but but it would be nice to have an article/blog post about what exactly is changing, and how to update from using these two classes.

Sorry for the delay on this. I've stopped work on the "new" InTheHand.Devices.Bluetooth API and instead have a new version modelled on the original API but adding .NET Standard support and better support for NuGet, Unity and eventually other platforms. I hope to have the new code merged in soon along with a new v4.0 NuGet package.

Thanks for the update @peterfoot.  I'll keep an eye out for any updates.

Hello, thanks for the package. Is there any update on Unity support? Thanks!

InTheHand.Net.Bluetooth is out on NuGet. It uses Win32 Socket APIs to get around the issue where Mono Sockets don't allow the Bluetooth namespace/address family. The API is closely modelled on the original 32feet.NET v3.x API with some simplifications.

@peterfoot thanks for your answer! I used the library you recommended, but it does not process LE devices. When I tried to use the LE library in my project it would not compile as it was missing "Windows.Foundation.UniversalApiContract". I really like the BluetoothPicker approach, but unfortunately it only discovers standard BT devices.
This is my code:
var pick = new BluetoothDevicePicker();
pick.PickSingleDeviceAsync().ContinueWith(task =>
{
            var dev = task.Result;
            Debug.Log(dev.DeviceName);
});

For Bluetooth LE there is a separate library - InTheHand.BluetoothLE. I need to do a new release and update documentation/samples as it's a completely different API

Awesome! Can’t wait to try it in unity! Will make our life so much easier ;)

When i try to connect using IntheHand.net.bluetooth Bluetoothclient. I get an exception.
client.BeginConnect(endPoint, new AsyncCallback(ConnectCB), client)
I get null reference exception.
What might be causing the exception?


[Search] Update code with semantic search changes

API changes have been detected in Azure.Search.Documents. You can review API changes here

API changes have been detected in Azure.Search.Documents. You can review API changes here

cc: @tg-msft

The customization I have currently is a rename on the properties of PrioritizedFields.
PrioritizedContentFields -> ContentFields
PrioritizedKeywordsFields -> KeywordsFields

API changes have been detected in Azure.Search.Documents. You can review API changes here

API changes have been detected in Azure.Search.Documents. You can review API changes here