Close button freezes the application - injected input from RawInput (unpackaged app)

Describe the bug
Not working:

The close button doesn't react to injected input (MOUSEEVENTF_LEFTDOWN/ MOUSEEVENTF_LEFTUP) by SendInput triggered from RawInput/ RIM_TYPEMOUSE) within the WinUI 3 app.
I've tried a workaround but sending WM_Close by PostMessage does nothing on the close button.

Note: Everything (the mentioned points) works fine in other applications or on other controls within the app except the close button.
Working:

The close button reacts and closes the app, if you send injected input for the right key (MOUSEEVENTF_RIGHTDOWN/ UP).
The close button reacts, if you send the same injected input (MOUSEEVENTF_LEFTDOWN/ MOUSEEVENTF_LEFTUP) directly from a keyboard hook triggered by a keyboard key.
The WM_Close event in the subclass only gets executed, if you interrupt the freezed process.

Steps to reproduce the bug

Create a mousehook and rawinput.
Block all mouse input within the mousehook and trigger your own injected input from rawinput.
Additionally test the same directly from the keyboard hook.

Expected behavior
The close button should normally behave by injected input for the left mouse key as it does in any other application; it should trigger the WM_Close/ close the application window.
Screenshots
No response
NuGet package version
No response
Windows app type

[ ] UWP
[x] Win32

Device form factor
No response
Windows version
No response
Additional context
No response

Sorry, can we pause this here. I have to test a bit. Maybe it's because of this:
https://stackoverflow.com/questions/26558755/mouse-pointer-freezes-after-calling-sendinput-on-minimize-maximize-close-window


High resolution export?

Hi, I'd like to export the canvas at high resolution for printing. I've tried using this library but the output doesn't look sharp (see crop at 100% zoom):

Figured it out! Replacing innerWidth/innerHeight values did the trick.


fixfix hive catalog miss partition that have special characters. #42906

cherry pick from #42906

run buildall


Support React.RefObject in React >=16.3

Handle children ref created using the new React.createRef. This is an object containing a current property pointing to the HTML element. E.g.
import Observer from '@researchgate/react-intersection-observer';

class MyComponent extends React.Component {
  targetRef = React.createRef();

  handleClick = () => {
    console.log(this.targetRef.current); // HTMLDivElement
  }

  render () {
    return <Observer>

Foobar

    </Observer>
  }
}

I've raised this with the React team before. See: https://github.com/facebook/react/issues/13029.
We need to extend here to support this new format.
Expected behavior
☝ this.targetRef.current should be defined and should match HTMLDivElement.
Current behavior
☝ this.targetRef.current is null.
Context (environment)
React >= 16.3. See https://github.com/facebook/react/issues/13029.

Alright, so the main issue I see here – whatever the intent we have – is that if we manage make that TS prop non-readonly, we can't re-assign props.children.ref.current, because props are frozen (or should be treated as such).
Correct me if I'm wrong but seems like the only future-proof solution would be to have a new prop that you could optionally provide.
<Observer innerRef={reactRef} onChange={...}>

 {/* <-- gets innerRef assigned if provided */}
     {...}

Pardon me for not mentioning the obvious approach to innerRef 🙊. We would rather use React.forwardRef 💯

@Rendez no, it should be ok. The less pervasive solution is to simply assign to ref.current if it's an object - like in here - as it's their recommended approach (also what they do under the scenes with the ref). Given the source of this module is not TS, there shouldn't be any difference or problem with the types.
Now, tapping on the children passed in the component may be seen as a bad practice (as you mentioned). I've seen other patterns for doing this exact same thing (get hold of a DOM node for measuring, etc.) - such as react-measure. Though, that's a much bigger breaking change.


Other Powershell Modules for utilization of webcommander

i have been messing with Webcommander for a while now getting used to it and how it functions. This tool is going to be great i can see it.
I am having an issue trying to use other Powershell Modules for lets say Netapp Filers or the Active Directory Module. and still running everything in 32 bit mode still functions the way i need it to but when trying to run the same scripts from webcommander it Runs a success but doesn't seem to run the script at all.
Keep in mind i am trying to run different modules in powershell an example of what i go it below.
Param (
[parameter(
Mandatory=$true,
HelpMessage="Which OU to change the accopunts passwords?"
)]
[string]
$OU,
[parameter(
    Mandatory=$true,
	HelpMessage="Set the password you want the accounts to be changed to"
)]
 [string]
	 $Password

)
foreach ($paramKey in $psboundparameters.keys) {
$oldValue = $psboundparameters.item($paramKey)
$newValue = [system.web.httputility]::urldecode("$oldValue")
set-variable -name $paramKey -value $newValue
}
. .\objects.ps1
Import-Module ActiveDirectory

start-sleep -s 5

Get-ADUser -filter "enabled -eq 'true'" -SearchBase "OU=XXXXX,OU=XXXX,DC=XXXX,DC=XXXX,DC=XXXX,DC=com"
any assistance and or help with this is appreciated

After runing the command, right click on the page and select "view source".
Let me know if you see any error message there.
BTW, I tested running the following command on my server
get-aduser -filter 'samAccountName -like "*"'
It worked well.
So I believe what you met is a configuration issue.

WARNING: Error initializing default drive: 'Unable to contact the server. This may be because this server does not exist, it is
currently down, or it does not have the Active Directory Web Services running.'.
Get-ADOrganizationalUnit : Unable to contact the server. This may be because this server does not exist, it is currently down,
or it does not have the Active Directory Web Services running.
I do see this as an error but i have checked the Domain serves themselves making sure ADWS is there and running. not too sure now
but thank you for the help

That sounds an authentication issue. What authentication method you configure on IIS?
If anonymous authentication is used, set it run as administrator.

It was set for windows Authentication. but enabling anonymous Authentication and the specifying a user it worked just fine.
thank you for the simple solution
thanks again
Jordan Mannor
Netapp system administrator

My pleasure.


Add instructions to revert a release

Related Issue or Background Info
We have been operating under a fix and move forward model for buggy releases. In preparation for rolling out an on-call schedule, to increase the stability of our application and to ensure no one has to "fix and move forward" at 3am we should move to rolling back buggy releases and fixing in a non-prod env
Changes Proposed

Add instructions for rolling back to a previous version

Additional Information

It seems that the create release form doesn't seem to let you target a tag or arbitrary commit (only allows "recent" commits)
To test the purposed flow I checked out v29, created and published a branch and confirmed that the branch is select able in the create release form.
Would like to test this process out during business hours with a trivial change

Screenshots / Demos
None
Checklist for Author and Reviewer
UI

[x] Any changes to the UI/UX have been approved by design
[x] Any new or edited error messages have been approved by design

Testing

[x] Includes a summary of what a code reviewer should verify

Changes are Backwards Compatible

[x] Database changes are submitted as a separate PR
[x] GraphQL schema changes are backward compatible with older version of the front-end

Security

[x] Changes with security implications have been approved by a security engineer (changes to  authentication, encryption, handling of PII, etc.)
[x] Any dependencies introduced have been vetted and discussed

Cloud

[x] DevOps team has been notified if PR requires ops support

We should include the much-simpler process of rolling back using the staging slot.

My understanding is that the staging slot will only work for the backend. We would still need to redeploy the frontend. If that is true then I would like to go forward with this strategy and we can follow up with a V2 to use the staging slot and redeploy the frontend

Solid point about the front end—we talked about that issue but didn't conclude anything very useful. I think there should be a way to snap back the front end as well, but there isn't now. (And staging slots wouldn't be it.)

Solid point about the front end—we talked about that issue but didn't conclude anything very useful. I think there should be a way to snap back the front end as well, but there isn't now. (And staging slots wouldn't be it.)

Currently I think we arn't cleaning up any of the static resources. It may be possible to change just the index.html file back to the old version


No way to zoom in for hiDPI screens

I'm pretty used to things not looking right on a 4k monitor in linux, but it would be nice if i could do a 'ctrl +' to zoom in a bit manually.

closed via #453


Why TextDocumentSyncKind.Incremental is implemented but not used?

I notice that the handler is capable to process incremental changes in textDocument/didChange event.
https://github.com/sourcegraph/go-langserver/blob/885ad3639de0e1e6c230db5395ea0f682534b458/langserver/fs.go#L154-L159
However, in the initial parameters, the sever declares its capability as TextDocumentSyncKind.Full
https://github.com/sourcegraph/go-langserver/blob/885ad3639de0e1e6c230db5395ea0f682534b458/langserver/handler.go#L205-L210
What's the reason for prefering Full than Incremental?

The server doesn't process notifications necessarily in the order they are received, since we spin up a goroutine for each received jsonrpc2 request/notif. Which means we may apply the incremental patches in the wrong order depending on the whims of the go runtime. Once that is fixed we can start using the incremental patches again.

I see, but now the server uses jsonrpc2.HandlerWithError(a sync handler) instead of jsonrpc2.AsyncHandler, and lspHandler wraps LangHandler so that the requests that mutate the typecheckers (e.g. textDocument/didChange) are blocking. Am I correct to assume that all changes to the document are processed serially?

@Contextualist you are right, I forgot I "fixed" that :) We should be fine with using incremental.


[CWS] ensure the btfhub combine step is using the correct python version

What does this PR do?
The job is currently failing because of unsupported type hints. This PR fixes it by using python 3.11, as other github actions steps are already doing.
Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


permalinks for headers not showing for asciidoc

clickable permalinks next to headers appear for markdown-generated html, but not for asciidoc. See example and commit.

Thanks for your report. I'm very sorry about this, but that because the asciidoc support in Hugo itself is very limited -.- We use markup render hooks to customize the rendering of images links and headlines. You might see other formatting issues for images and links as well.
As there is not really a good way to handle this for asciidoc I have to mark this issue as wontfix.

That's rather unfortunate, but I appreciate the response. Will take this under advisement.


[function] need format to compare two strings

Formats the number X to a format like '#,###,###.##', rounded to D decimal places, and returns the result as a string. If D is 0, the result has no decimal point or fractional part.
please refer to   https://dev.mysql.com/doc/refman/5.6/en/string-functions.html#function_format
mysql> SELECT FORMAT(12332.123456, 4);
-> '12,332.1235'
mysql> SELECT FORMAT(12332.1,4);
-> '12,332.1000'
mysql> SELECT FORMAT(12332.2,0);
-> '12,332'

@mchades hi could you please reply to this issue so I can assign it to you?

@mchades hi could you please reply to this issue so I can assign it to you?

OK, I will do this

I see the assign of this issue has been a while, can I also try to complete this, I want some simple issue to cut into starrocks to learn better @wangsimo0

@iiiuwioajdks Hi buddy, are you still working on this issue？ If you have any questions, feel free to join WeChat for discussion: https://ma3you.com/articles_qrcode_show/b12e90J.png?le_time=220907203008013206

@kateshaowanjou I want to try this task, can you assign it to me?

@wangsimo0 Could you please assign this to me, I would like to contribute to this project.
Thanks in advance

sorry for the delay. @Jin-H @LakshmiGayathri19 hi are you still have interest to try this task? sorry again for the delay

@wangsimo0 Plz assign it to me, thanks.

hi @wangsimo0
Is anyone still working on this? If no, please re-assign it to me and I will start implementing it now

Sorry, I originally planned to deal with it in these two weeks, but I have been busy with work recently and have not had time to do this task. At present, I don’t expect to start doing it until after New Year’s Day in 2024. If you particularly want to try this task, Then do it. @silly-carbon

Sorry, I originally planned to deal with it in these two weeks, but I have been busy with work recently and have not had time to do this task. At present, I don’t expect to start doing it until after New Year’s Day in 2024. If you particularly want to try this task, Then do it. @silly-carbon

@Jin-H thanks mate, OK I'll implement this


Proposal: Json Field Naming strategy

Is it possible to have something similar like Jackson kind of configuration options in json library

Naming Strategy
Ignore null
Accept single value as array
Fail on unknown properties.
Map unknown properties in a known json to different section.

These all are very useful handy configuration when we deal with serializations and de-serializations.
Can we get this supports added to the existing json library.

I think this is a dupe of #16426. You can already do 2 and 3 by implementing the Unmarshaler interface, and you can do 1 with struct tags. 4 and 5 are part of #16426.


TestOnly and DescribeOnly

Some ambiguity regarding behavior:
Jest's behavior:
test.only

applies on a per file basis

test.only in one file does not affect other files
skips all tests not in a describe.only or also declared as test.only

describeOnly (only runs this test suite - may not make as much sense)

also applies on a per file basis (e.g. one file with describe.only does not stop other test files from running)
effectively replaces test with test.only inside the describe block

e.g. if you have a describe.only block and a normal describe block and manually add test.only to the normal block, it will still run

The Rely analogue would replace "file" with "top level describe (e.g. the describe exposed by a TestFramework module or as a result of calling extendDescribe).
Implementation would require separation of "test discovery" and "test execution"

presently the "most nested tests" are run first. We would want to gather all tests before executing any of them. This will require some non trivial code changes, but I've run into the necessity to do this a couple of times already anyway so I think it should be done.

Not necessarily. We could still run every test and just hide the output when necessary. The main point of "only" is to focus the output, doesn't really matter if we run extra stuff.

IMO, it would be preferable to not run every test. I would love this feature for Onivim 2 (only, filtering tests), because we have some slow tests - and they are slow not due to any deficiency of Rely, but slow because they are talking to other services (like, spinning up a node process and exercising a protocol).
I want the coverage of these slower tests, but don't necessarily want to run them all the time - especially when I'm iterating on new, fast tests. A flow where I could re-run just the tests I'm iterating on would be awesome for productivity!

 IE, esy x MyTestRunner --filter SomeTest - I think, if we had a 'test discovery' method, it could just be used as a filter there.

I actually just finished the work to separate discovery and execution, adding testOnly and describeOnly should be pretty straightforward (and I'll definitely build in such a way that supporting --filter in the future is also easy).
I think we are planning on releasing a CLI library shortly so I will probably wait on that to implement filter, but I imagine testOnly/describeOnly will be out soon. Feel free to file a separate issue for filter!

This has been released in Rely 2.1.0!


Revert "Decompress and ecode config always but when not supported by the CPO"

What this PR does / why we need it:
This reverts commit 6948f4f4a71406a2aa13762e46a113fe633275d2.
The above commit results in a broken ignition-server in some cases. Reverting so we can have more time to look at it.
Checklist

[x] Subject and description added to both, commit and PR.

This shouldn't actually be blocking CI itself, but does manifest in user created clusters

/hold

/close


Presentation of documentation for completion proposals

VSCode Version: Code - Insiders 1.13.0-insider (ae5c28306b8db90d0b9d7c4af976c2fa76237bb5, 2017-05-08T15:28:02.591Z)
OS Version: Darwin x64 16.5.0

With the documentation being displayed to the right:

The two boarders add-up to a strong line between the list and the documentation. It should only display as a single border.

Reverted the fix for this issue as the fix causes weird 3 side border as described in #27651


Add cron job to auto-update pixi lock file

This way there will be a PR that updates the lock file every month

Looks good to me! So we can start this job manually as well right? Would be good to give it a shot right away.

Yes, we can. Happy to test it manually, I am pretty sure it works though. Just added the same for the imod docs :)
https://github.com/Deltares/iMOD-Documentation/blob/main/.github/workflows/pixi_auto_update.yml


drivers/sensor/lsm6dsl: Fix read/write bus API in shub code

The ce0cc3a7 commit (lsm6dsl: make the driver multi-instance) changes
all bus API signature, which now requires a "const struct device *dev"
and not a "struct lsm6dsl_data *data". By mistake the shub part was
left unchanged. Fix: #28565
Not tested yet!
I just built it and verified it is compiling properly.
I would like to test it on x_nucleo_iks01a2 shield, but I require to extend the current available
sample. For the moment just take a look at it and see if I have done any mistake.
For the moment it is a DNM. I do not think the shub will have any problem. So, we may want to
skip the test if v2.4.0 has to be released soon.
EDIT:
Removed the DNM!
Tested on x_nucleo_iks01a2 shield in shub mode. Tested both LIS2MDL and LPS22HB sensors connected in shub mode.
The shield and sample changes will be provided in a separated commit as some documentation change is required.

@erwango @pabigot
I addessed your requests. And I finally tested it on shield, so I removed now the DNM.
Board and sample changes for shub mode will be provided in a separate commit later on (to be planned).


Simplify MSYS2 CI jobs further

Current: ucrt64 static job takes 10m 17s and
  Total Download Size:   154.66 MiB
  Total Installed Size:  977.88 MiB

Proposed is 9m 27s and no additional download
  Total Installed Size:  653.59 MiB

Also, I don't think the CI failure is specific to this PR, as it already (intermittently?) happens on master already: https://github.com/mm2/Little-CMS/actions/runs/4386006557/jobs/7679480015


Error reading .eslintrc - Syntax error

Starting with a recent update, I get this:

I've seen recently that there have been some commits relating to JSON parsing of .eslintrc, is this error because I have a YAML .eslintrc?
Snippet:
---
  parser: "babel-eslint"

  env:
    browser: true
    mocha: true
    node: true
... etc

Yes. I assume it is Json. I need to do some more work in parsing in general to cope directory recursion, yaml,  js files and extends.  I can put yaml parsing in quickly. In the meantime, you can either turn off the package settings or command toggle auto indenting of jsx or rename it to .yaml and add jsx indent rules as json.

I was also getting this problem, good to know the source of it now.
For the moment I just converted the YAML .eslintrc files in various projects back to JSON.

@gandm Okay great, thanks for your help.
For now I've just downgraded the plugin to 2.5.4 (picked a random old version) and it works fine.

Whoops. Removed JSON5 in error Just pushed 2.11.1to fix that!
More haste less speed.


[Feature Request] Deployable to Heroku

Can you Please make it deployable to Heroku 😇

Are you affiliated with the site SamFirm ?

Are you affiliated with the site SamFirm ?

No


[release-v1.11] Force kafka broker class for ApiServerSource tests

is SO we force Kafka broker class
https://github.com/openshift-knative/serverless-operator/blob/b60d84aaf8b095102b86bd45381c364bbdf581d1/hack/lib/vars.bash#L36
but that's not enough since the configuration should match the expected one for Kafka brokers

/lgtm


Add Screenpipe Filter

Implemented as an Open WebUI function as well, but part of the Filter class
This serves as an inlet + outlet to modify the messages array before/after it passes through Pipe.pipe()

Here is the current pipe method in screenpipe_search_function.py:
def pipe(self, body: dict) -> Union[str, Generator, Iterator]:
    """Main pipeline processing method"""
    try:
        self.initialize_settings()
        messages = self._prepare_initial_messages(body["messages"])

        # Get search results
        results = (self._tool_response_as_results_or_str(messages)
                    if self.native_tool_calling
                    else self._json_response_as_results_or_str(messages))

        if isinstance(results, str):
            return results

        # Process results
        sanitized_results = PipeUtils.sanitize_results(
            results, self.replacement_tuples)
        print("Sanitized results:", sanitized_results)
        messages_with_data = self.get_messages_with_screenpipe_data(
            messages,
            json.dumps(sanitized_results)
        )

        # Generate final response
        return self._generate_final_response(body, messages_with_data)

User message -> Filter.inlet (render) -> Pipe.pipe (live render) -> Filter.outlet (render) -> END
New logic would move some of the steps into the Filter class. Here's the nerd sauce:

User sends query
Open WebUI stores messages + other keys in a dictionary called "body." This dict is passed downstream.
Enter the Filter.inlet method
Initialize settings
Prepare initial messages (preserving original user message)
Get tool formation. (Dict or str)
If invalid, add an error key to body and return body to pass downstream to Pipe.pipe. Otherwise, store value as self. search_construct.
Perform the screenpipe search
If invalid, add error key to body and return body. Otherwise, sanitize and set the value as self.search_results.
Add values for body["user_message"], and body["search_results"]
Append the successful search_construct to the user message in body["messages"]
Return body and head to Pipe.pipe(body).
Enter the Pipe.pipe method
Use the user_message and search_results in body to re-format the messages array.  Then, call the equivalent of generate_final_response in current implementation, returning that stream/string.
Enter the Filter.outlet method (with body)
Remove search construct from user message, and move to Assistant response, along with other stats like resulting chunk count.
END! :D

This is now fully implemented, including a full-filter.py that doesn't rely on the open_webui subfolder!


[Request feature] Add setting menu for Disable/Enable debug message on developer tool

Hi , Thank for nice extension.
[Request feature] Please add setting menu for  "Disable/Enable"  show the debug message on developer tool
It's disruptive for website development.
Thanks.

Sure we will add this feature after Feb release to stable. Currently we are focusing on fixing issuw which are there in DEV and later deploy to stable version


Adding eosio structure attribute

Change Description

Adding eosio::structure custom attribute. Can be used for retrieving metadata. Example usage: struct [[eosio::structure]] fee_payer { /* fields */ };
API Changes

[ ] API Changes

Documentation Additions

[x] Documentation Additions

This PR is part of Transaction Sponsorship feature

I was waiting for https://github.com/EOSIO/clang/pull/40 to be approved, so I can  update the submodule changes to this PR. @larryk85, can you please approve it. Thanks.

The file structure_test.json is missing.


FFM-7502: adds set evals section in JS SDK, style tweaks

Find the new section on this path: /docs/feature-flags/ff-sdks/client-sdks/java-script-sdk-references#evaluate-a-flag > (Optional) Provide a set of evaluations

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Forwarded http client to EventSource

We need this otherwise we get a unauthorised exception if we use this functionality with BlockDaemon. It will otherwise not use the JWT we configure in the StellarSDK instance.

Hello @jopmiddelkamp, thank you for extending EventSource. Looks good!


more way to create connection and add useful when fetch path

add more method to create connection and separate nodes edges when fetch path

Hello, kevintruong
Sorry for late reply, I've been on long term holiday recently.
And thank you for pull request.
I think 'connect_new' is awesome,
but making nodes and edges list in the constructor (init) of Path object consumes unnecessary resources(Memory, CPU).
I think it's better to add  functions 'getAllNodes', 'getAllEdges' separately.

hi @rhizome-ai
I rework on your comments.
I believed that we should support to get nodes and edges native from driver level
As your concern about CPU/RAM. I think that let application level have responsible about it
By using property decorator it will avoid your concern.
Thanks

It looks awesome.
Thank you for contribution.


es模块导出方式不正确，导致项目构建编译报错[!] Error: 'default' is not exported by node_modules/highlight.js/lib/index.js, imported by node_modules/highlight.js/es/index.js

es模块引用了lib/index.js，而lib/index.js是commonjs的导出方式

Yet default is exported you can see it right there... how are you trying to use this in your project?

Oh, I had this issue too (but had written it off to an oddity of our setup).
For some more colour; our project is a mono repo which has a "platform" and a "ui lib" (which is used by the platform with a package entry of "ui": "file:../ui").
Then we use Vite.
My working theory had been that the highlight.js package.json uses exports which alias require and import definitions, and maybe those are getting lost?

Perhaps it's a vite issue?

Got here b/c I ran into this trying to use highlight.js with @web/dev-server(for dev - which gave me a The requested module './../core.js' does not provide an export named default error) and rollup(for build - which gives the same error as in the title - just core.js instead of index).
Neither tool handles commonjs module conversion by default anymore. Have to setup @rollup/plugin-commonjs and - for dev-server at least - explicitly tell it about the highlight.js package.
Can't speak to vite, I'm not using it. The docs say that it handles commonjs module conversion by default, but I wouldn't be surprised if it didn't include node_modules in that. Might need to tell it to convert the package.

这个问题有解决方案嘛

Have to setup @rollup/plugin-commonjs

That sounds like a reasonable solution in the interim.   This should also be fixed when we hit version 12 and only ship ES builds.
Closing this as #wontfix.


Invalid path error when using Export-DbaDacpac to the network share

Before submitting a bug report:

[x] Ensure you are able to reproduce it on the latest released version (we release often)
[x] Verified this bug is not already reported in an issue
[x] Verified errors are not related to permissions
[x] Can reproduce in a clean PowerShell session (clean = powershell -NoProfile)

Steps to Reproduce
Export-DbaDacpac -SqlInstance myInstance:1234 -Database MyDBName -Path \\some\network\path

WARNING: [15:58:54][Export-DbaDacPackage] Standard output - *** Argument 'TargetFile' has an invalid value:
'Microsoft.PowerShell.Core\FileSystem::\\some\network\path\myInstance,1234-MyDBName.dacpac'.
*** The given path's format is not supported.

Expected Behavior
Create file as expected
Actual Behavior
Recent update to the Path handling in the function broke the ability to use network path as a destination, since Resolve-Path \nw\path | % Path returns a path including a provider. Using ProviderPath would probably be a better approach:
Resolve-Path \\nw\path | % ProviderPath

Environmental data

PowerShell:
PSVersion                      5.1.15063.1266
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.15063.1266
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

@jasonmcboyd please let me know if the solution works on your end as expected - I'll PR it later.

@nvarscar Looks good from my side.


Add review policy.

Coverage remained the same at 81.795% when pulling c1b60197ee8e6924a9c6f75b1fb67dd1ce6da41e on tdaede:reviews into df6451aadf6c1d19a487aab64549ba743d003d95 on xiph:master.

I am a bit uneasy about including URLs in commit messages, since they will probably eventually become inaccessible (and AWCY URLs can be quite long). Why are they needed specifically?

Probably enough to have them in the PR

It's so you can jump and look at more detailed results. I could make it optional I suppose.

In addition to rebasing, should there be a comment about squashing commits? Do we enforce squashing commits?

We do not enforce squashing, sometimes makes sense if you push additional
refinements as commits instead of reading and pushing an update.
On Mon, Aug 5, 2019, 17:30 Josh Holmer<EMAIL_ADDRESS>wrote:

In addition to rebasing, should there be a comment about squashing
commits? Do we enforce squashing commits?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/xiph/rav1e/pull/1529?email_source=notifications&email_token=AAB2LJBC27PUYUOPXVEN35LQDBBQPA5CNFSM4IJBPKFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3SF2PI#issuecomment-518282557,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB2LJC3MZTQSBWSPZRWCMDQDBBQPANCNFSM4IJBPKFA
.

I originally enforced squashing, everyone hated it so I stopped. The main issue is that CI doesn't run on each individual commit, so by not squashing we have some revs that were never CI tested.

On the one hand yes, but on the other hand I just had bisection pinpoint me to the exact small change leading to #1513 and that's super helpful to have.

@lu-zero please re-review

To clarify, the Contributing chapter has moved to CONTRIBUTING.md. This section should be added to that document.

This might need to be manually reapplied to the new file but it would be great to finally merge this.


create record_set if it does not exist

Currently, the plugin tries to update the Route53 DNS record, but it fails if the record does not exist. Ideally if all of the permissions are correct the record should be created if it does not exist.

I agree. Working on a pull request.


Different spellings of common names

Currently, alias generator does not predict common spellings. For example:
Josef / Joseph
Philipp / Phillip / Philip
Especially in multilingual contexts, it is common for one person to use multiple such variants.

A related example from social media feedback:

My name is Fränkel. In German-speaking countries, it would be written Fraenkel to avoid issues with diacritics. Depending on the country and the language, there might be lots of additional variations


Run migration 760 on compendium JSON

That said the migration knocking out that much text does seem to be a bug.


crash (raspberry pi)

Traceback (most recent call last):
File "/home/pi/./PlaceNL.py", line 693, in
run()
File "/home/pi/./PlaceNL.py", line 689, in run
asyncio.run(main())
File "/usr/lib/python3.9/asyncio/runners.py", line 44, in run
return loop.run_until_complete(main)
File "/usr/lib/python3.9/asyncio/base_events.py", line 642, in run_until_complete
return future.result()
File "/home/pi/./PlaceNL.py", line 685, in main
await asyncio.gather(*tasks)
File "/home/pi/./PlaceNL.py", line 628, in reddit_client
await place_client.load_full_map()
File "/home/pi/./PlaceNL.py", line 454, in load_full_map
canvas4 = await self.load_canvas(3)
File "/home/pi/./PlaceNL.py", line 393, in load_canvas
async with self.session.ws_connect(PLACE_WEBSOCKET, protocols=["graphql-ws"], headers=headers) as ws:
File "/home/pi/.local/lib/python3.9/site-packages/aiohttp/client.py", line 1138, in aenter
self._resp = await self._coro
File "/home/pi/.local/lib/python3.9/site-packages/aiohttp/client.py", line 776, in _ws_connect
resp = await self.request(
File "/home/pi/.local/lib/python3.9/site-packages/aiohttp/client.py", line 559, in _request
await resp.start(conn)
File "/home/pi/.local/lib/python3.9/site-packages/aiohttp/client_reqrep.py", line 898, in start
message, payload = await protocol.read()  # type: ignore[union-attr]
File "/home/pi/.local/lib/python3.9/site-packages/aiohttp/streams.py", line 616, in read
await self._waiter
aiohttp.client_exceptions.ServerDisconnectedError: Server disconnected

Hmm I believe that happens occasionally, the latest commit should be more robust to such errors.


Move PublicDataProof ERC draft to another repository

All codes, docs, tests for PublicDataProof ERC draft have been moved to a separate repository PublicDataProof.
All subsequent discussions on ERC draft and proof algorithms will in that repository.
old ERC draft related files(doc/erc) should be seen as archived.
@fulldecent
@waterflier

Got it ~


After phx.gen.auth

#178  👈
main

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

Join @acco and the rest of your teammates on  Graphite

Merged in c6fd34e


Implement sudo password via CLI

Ability to securely collect the sudo password and use it in various
playbooks.
Fixes: #992

I've added a --sudo flag to the base Molecule command (molecule --sudo), this will prompt for your sudo password, and make a MOLECULE_SUDO_PASSWORD environment available to Molecule.
Now for you to use this, you will need to configure your playbooks which use docker, lxc, whichever as follows.
- name: Destroy
  hosts: localhost
  connection: local
  gather_facts: False
  become: True
  no_log: "{{ not lookup('env', 'MOLECULE_DEBUG') | bool }}"
  vars:
    ansible_sudo_pass: "{{ lookup('env', 'MOLECULE_SUDO_PASSWORD') }}"
    molecule_file: "{{ lookup('env', 'MOLECULE_FILE') }}"
    molecule_yml: "{{ lookup('file', molecule_file) | molecule_from_yaml }}"
  tasks:

PR 1000 LOL
~1000

It's over :nine: :nine: :nine:! Congratulations.

hey @retr0h - this is working great on the destroy and create but is bombing out on the actual converge playbook. Whats the best way I can pass this environment variable to the runner that is calling this playbook?

    TASK [Gathering Facts] *********************************************************
    fatal: [gnomeshell_role_f26]: FAILED! => {"failed": true, "msg": "Docker version check (['/usr/bin/docker', 'version', '--format', \"'{{.Server.Version}}'\"]) failed: Got permission denied while trying to connect to the Docker daemon socket at unix:///var/run/docker.sock: Get http://%2Fvar%2Frun%2Fdocker.sock/v1.26/version: dial unix /var/run/docker.sock: connect: permission denied\n"}

hey @retr0h - this is working great on the destroy and create but is bombing out on the actual converge playbook. Whats the best way I can pass this environment variable to the runner that is calling this playbook?

You can't just add the inventory var to your playbook.yml as well?
Maybe this is an incomplete solution to the actual problem.

You can't just add the inventory var to your playbook.yml as well?

No :( Honestly, I think I have to do more research. This is a great start though! I suspect I could be hitting issues with upstream ansible and how they make docker connection calls. Maybe I'll need to run ansible as sudo during the converge playbook call ...

Yah, I can just seeing there being all sorts of weird edge cases.  For example if testinfra needs to talk to docker and requires sudo as well, then this solution really wont work.
Can backout to a Molecule on pypi and simply run sudo molecule test instead and let the OS prompt you for your sudo password and run molecule itself in privileged mode?

Uggghh totally I can see that cropping up with testinfra and friends. Dang it. Thanks for trying to tackle this. I do really appreciate it and think this is an important issue that needs to get solved but definitely outside the scope of molecule.
I think I'll just use docker-machine to run docker under a VM to solve my woes. However,  I still think this is a valuable PR to leave in. I can't think of any use-case off-hand but I can image a scenario spin-up/spin-down task requires sudo but the rest can be run as regular user. Would just have to add a big disclaimer to the docs.

For shits 'n giggles does sudo molecule test work?

errrrr @retr0h - i'll try running that in a disposable VM later, makes me nervous to run the entire molecule environment as root 😬

Closing


[Bug]: Issue with new body consitency - wrapper with only metadat properties is not used

import "@typespec/http";

using TypeSpec.Http;

model Wrapper {
  @header extra: string;
  ...Base;
}

model Base {
  // Add header to make it work.
  // @header other: string;
  code: string;
}

op get(): Wrapper;

the return type used here is Base instead of Wrapper
Playground Link

I think this is actually expected, here because Base only has body params and Wrapper just adds header it makes sense for Base be used as the Body model. This used to be the case when this pattern was used in a request but not a response.
Body consitency PR made it so both are handled the same.


Chinese Edition Fixing

Fazri was right. The Chinese docs are really messed up.
This is the first fix, please let me check the result on the site.

Hi Coco, thanks for this.
Some notes about the Chinese docs:

Whenever you see code wrapped using a triple backslash (```) we should have line breaks to make sure the formatting looks like a block rather than "inline" in the generated HTML file. (You can compare the ssh<EMAIL_ADDRESS>command from Chinese and English versions)
We have special formatting for "notes", "tips" and "warnings" - in the current Chinese version they are slightly problematic. I think if you search for !!! you should find all of them

!!! note "Title of the note"
    Contents of the note

!!! tip "Title of the tip"
    Contents of the tip

!!! warning "Title of the warning"
    Contents of the warning

Please let me know if you have any questions. Cheers!

Now all md formats should be corrected. (polish needed)

Thank you @CocosGames - can we merge this PR, then? 👀

Thank you @CocosGames - can we merge this PR, then? 👀

I think so. They should be displayed correctly. But they need more work on translation.

Thanks a lot for all the work here @CocosGames 🚀 , if you feel more work/tidying-up is still necessary feel free to open a new PR please 😇


Allow documented components not to have a script block

closes #61

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


remove unused add_stream() function from Daemon

Just removing this unused function add_stream() from Daemon

          1 Error

:no_entry_sign:
Please include a CHANGELOG entry.

          1 Message

:book:
See http://keepachangelog.com/en/0.3.0/ for details on good changelog guidelines

  Generated by :no_entry_sign: Danger

this doesnt need to be a PR, you can just merge it


Need a method to get the channel ID using the current channel name

We need a way to get the channel ID when provided the current channel name as an input. Previously this could be done using the channels.info API method, but now with that method fully deprecated and disabled, we don't have any other reliable method of doing this. We could theoretically post a new message to the channel using chat.postMessage, which returns the channel ID, then remove that message, but this seems like overkill.
The use case is that a user provides a channel name as input, and we want to convert that channel name to an ID and store that ID for later use when posting messages. Obviously the name may change over time, so using the channel ID is ideal.
Category

[X] slack_sdk.web.WebClient (sync/async) (Web API client)
[ ] slack_sdk.webhook.WebhookClient (sync/async) (Incoming Webhook, response_url sender)
[ ] slack_sdk.models (UI component builders)
[ ] slack_sdk.oauth (OAuth Flow Utilities)
[ ] slack_sdk.socket_mode (Socket Mode client)
[ ] slack_sdk.audit_logs (Audit Logs API client)
[ ] slack_sdk.scim (SCIM API client)
[ ] slack_sdk.rtm (RTM client)
[ ] slack_sdk.signature (Request Signature Verifier)

HI @angrychimp thanks for writing in 💯
This is a less then ideal scenario, I do not know a simple workaround for this, but since this is a backend issue I would suggest asking for a fix via<EMAIL_ADDRESS>

To add to this, we also are struggling with this issue and we typically run into the rate limit prior to getting the result we need. For us, this would reduce the api calls from hundreds down to one.

I absolutely wish to have this looked at. This is an ongoing concern and I routinely run into situations where an easy method to convert channel name to and ID would be far easier than scanning a full list of channels in a workspace

Commenting again to prevent this from being closed. Still hoping this gets looked at.

Hi @angrychimp 👋  This is a popular feature request that I agree would be useful, but we can only recommend paginating the conversations.list method and filtering outputs from the SDK side of things. I understand all of the concerns with rate limits and that this isn't an ideal approach, but I can only echo messaging<EMAIL_ADDRESS>😓
I do think it's worth keeping this issue open too so others can +1 👍  in agreement, but we don't have other plans to share at the moment.
📚 For more reference, this has been mentioned in issues like: #405, #1066, #1158, #1495, and a few other times in other repos.

+1 to this.
For our usecase, we post messages and files to multiple channels (public and private). The conversation.list method returns only public channels (ref: https://github.com/slackapi/python-slack-sdk/issues/1312 and https://github.com/slackapi/python-slack-sdk/issues/1494). We've ended up just statically mapping the channels name to the id in config, which is far from ideal... And a method to provide the channel name as input and returns channels ID sounds pretty barebone / simple.
@zimeg You're saying this has been requested multiple times (since 2019...) and still no improvments or even just a timeline? This is even worst considering that this feature existed in channels.info which has been deprecated (not sure I understand the reason why BTW).

You're saying this has been requested multiple times (since 2019...) and still no improvments or even just a timeline?

This is correct. I kindly ask for your understanding that myself, zim and other Slack employees that engage in these public GitHub issues with y'all do everything we can to raise your feedback, but ultimately these decisions and prioritizations are not ours to make. The only recourse I can suggest is to have your voice be heard by emailing<EMAIL_ADDRESS>If I could make the decision, I would provide such an API for you and the countless other developers who have asked for this feature. Unfortunately, I cannot make that decision.


Update unit2 - Typo in code

 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.


add check for GET_VERSION in session.

Fix https://github.com/DMTF/libspdm/issues/1987

This also applies to (V)CA.

I think CA cannot success because of the connection_state check.

I think CA cannot success because of the connection_state check.

Ok.


Feature Array has size of 0?

Hello, I'm running the program for the first time in Python and there is a ValueError for the basic running of the algorithm becaus of a size 0 feature array.

Can you help identify what went wrong?

Hi, it seems the error lies in configuring file location. Can you double confirm the 'config_path' is the right place for storing all the files?

Config has all the correct pathways to all of the file names

Try running scripts in 'phenosv/setup.py', which basically change the file paths in the csv files of feature_files and scaler_file.

@Karenxzr that worked thanks!
I'm trying to replicate this in the terminal, and I get the following error?

The notebook cell worked fine, and the csv file is formatted correctly based on the sample datasets provided.

Hi team, bumping this in case it was missed.

Can you try to score the given sampledata.csv? It works well on my end.

@mjang2000 The error might be caused by SVs on chromosomes as mentioned in #9 . I fixed the bug, please pull and try it again.


CI is not running on PRs

Hi @Grommash9, can you please check why the CI is not running on PRs?
I think that https://github.com/fastuuid/fastuuid/blob/master/.github/workflows/ci-cd.yml#L9 is the issue since our main branch's name is master and not main.

Yes, it the reason for sure, good shout, thanks!


Suggestion to change from sha1 to sha2 or sha3, in the PackageInfoPlugin class.

Plugin
package_info_plus
Use case
Suggestion to change from sha1 to sha2 or sha3, in the PackageInfoPlugin class.
Proposal
SHA-1 (Secure Hash Algorithm 1) is a cryptographic hash function that takes an input and produces a 160-bit hash value. As of 2005, SHA-1 was considered insecure, being formally banned by the NIST (National Institute of Standards and Technology) in 2011. Currently, attacks against SHA-1 hashes are considered trivial for adversaries with collection and processing power. of collision attacks. As such, NIST recommends removing the use of SHA-1 from products, replacing its use with hashes from the SHA-2 or SHA-3 families, which still
are considered cryptographically secure. It was identified that the informed application makes use of SHA-1 hashes, being susceptible to breach of confidentiality and data integrity.
References
• CWE
– CWE-327: Use of a Broken or Risky Cryptographic Algorithm
Recommendation
It is recommended that you change and implement a secure cryptographic hash algorithm, such as the SHA-2 and SHA-3 families.
More information
• OWASP - Testing for Weak Encryption
• What is a collision attack?
I look forward to hearing from you about the possibility of the change.
Ricardo Magno da Silva
Nostrum Consultoria e Projetos.

Where do we even use encryption?

Good Morning. Well, thank you very much for the extremely quick response from you.
This alert was generated by MobSF and I thought it prudent to inform them, I didn't do a deeper code analysis of the plugin, so I didn't come to that same conclusion, looking better at the class, I fully agree and we will put it on the false positive list.
Ricardo Magno da Silva
Nostrum Consulting and projects.


Add README.md

We need a comprehensive Readme file which makes the aim and functionality of the project clear to anyone who choses to view the repository and help them get started.

README added


fix(e2e): handle wrapped coin intra-chain transfer (#798)

Closes #798
Checklist:

[x] I have performed a self-review of my own code
[x] I have documented my code in accordance with the documentation guidelines
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have run the unit tests
[x] I only have one commit (if not, squash them into one commit).
[x] I have a descriptive commit message that adheres to the commit message guidelines

Please review the CONTRIBUTING.md file for detailed contributing guidelines.

@andrii-kl , could you please check if the changes in this branch work for your case ?

We have a lot of hardcoded values, it would be nice to refactoring it to get all these values from a config file.


Resolve "level" env var at runtime

Resolves #413, #444
Currently, the log level is read when the container built. This means we cannot change a log level at runtime using env vars.
With this change, log level can be modified for a specific command run: LOG_LEVEL=debug bin/console ...
~I would like to get some feedbacks before updating tests. I don't know yet what is the best strategy to check the resolution of the values.~

Did you check if this is even needed?
Last time I checked, every handler already casts the log level.
I see no reason for this bundle to cast the level at all.

@GromNaN do you have an idea on when this will be released?


Change ModelNotFoundException to NotFoundHttpException

This is needed since ModelNotFoundException throws error code 500. A change to NotFoundHttpException is needed in order for it to throw error code 404 instead.
ModelNotFoundException happens when using Laravel's explicit model binding and whenever the Id of the Model being queried doesn't exist.

Strongly against this, as it dramatically changes the semantics. A ModelNotFoundException could mean you want to throw a NotFoundHttpException, but you might also handle it differently.
Also, it is easy enough to implement yourself on application level.

:-1: 100% agree with @mrgrain
The fact that a model is not found has nothing at all to do with http

Correct, handle the exceptions how you want within your own application.


Create new project

Since we are touting this as a beginner's tool that makes leiningen invisible, we need to be able to run lein new mies or some variant to create projects, not just adding existing projects.

Still a few punchlist items before this is complete:

[ ] put focus on new project input field when going to that step
[ ] press enter on new project input field should work
[ ] input validation of new project folder

Needs testing on Mac and Windows.

On a Mac: pressing enter and input validation working. I am getting a cursor in the new project input field, but no outline.

Works on Windows


Bosh::Clouds::VMCreationFailed' with message 'Server `4ccca6e9-5dd7-41da-8568-20469d3680d3' state is error, expected active

Deploying
Are you sure you want to deploy? (type 'yes' to continue): yes
Director task 57
Deprecation: Ignoring cloud config. Manifest contains 'networks' section.
Started preparing deployment > Preparing deployment. Done (00:00:01)
Started preparing package compilation > Finding packages to compile. Done (00:00:00)
Started compiling packages
Started compiling packages > cli/60a741596e7fc803f45c9628364b21d1b247550b
Started compiling packages > nginx/8d0c80e9594988b792c1f83a8c1598a3f4d02b7a
Started compiling packages > haproxy/f061c9f4854fab702991cf42653b9b09d711c9c1
Started compiling packages > ruby-2.3/ca5b2d16eac54f907ab6c1f454d9c81560f311a2
Started compiling packages > libmariadb/90d7836132e02bb570d91f08c9e116c394888d75
Started compiling packages > dotnet-core-buildpack/dc2da8240e6831e5769ebab26340bb8ae3768630
Failed compiling packages > ruby-2.3/ca5b2d16eac54f907ab6c1f454d9c81560f311a2: CPI error 'Bosh::Clouds::VMCreationFailed' with message 'Server 4ccca6e9-5dd7-41da-8568-20469d3680d3' state is error, expected active Exceeded maximum number of retries. Exceeded max scheduling attempts 3 for instance 4ccca6e9-5dd7-41da-8568-20469d3680d3. Last exception: It is not allowed to create an interface on external network dc8385e3-3771-481d-ba98-4abf1f21c30b' in 'create_vm' CPI method (00:01:06)    Failed compiling packages > cli/60a741596e7fc803f45c9628364b21d1b247550b: CPI error 'Bosh::Clouds::VMCreationFailed' with message 'Server 624c8c2f-909f-400d-be25-7780b62d4a23' state is error, expected active
Exceeded maximum number of retries. Exceeded max scheduling attempts 3 for instance 624c8c2f-909f-400d-be25-7780b62d4a23. Last exception: It is not allowed to create an interface on external network dc8385e3-3771-481d-ba98-4abf1f21c30b' in 'create_vm' CPI method (00:01:10)
Failed compiling packages > libmariadb/90d7836132e02bb570d91f08c9e116c394888d75: CPI error 'Bosh::Clouds::VMCreationFailed' with message 'Server 050e6c84-856f-45b6-95c4-be2b603f2ef3' state is error, expected active Exceeded maximum number of retries. Exceeded max scheduling attempts 3 for instance 050e6c84-856f-45b6-95c4-be2b603f2ef3. Last exception: It is not allowed to create an interface on external network dc8385e3-3771-481d-ba98-4abf1f21c30b' in 'create_vm' CPI method (00:01:10)    Failed compiling packages > haproxy/f061c9f4854fab702991cf42653b9b09d711c9c1: CPI error 'Bosh::Clouds::VMCreationFailed' with message 'Server 5bf1fbc6-75df-409c-b961-dfc7ad8d2473' state is error, expected active
Exceeded maximum number of retries. Exceeded max scheduling attempts 3 for instance 5bf1fbc6-75df-409c-b961-dfc7ad8d2473. Last exception: It is not allowed to create an interface on external network dc8385e3-3771-481d-ba98-4abf1f21c30b' in 'create_vm' CPI method (00:01:12)
Failed compiling packages > dotnet-core-buildpack/dc2da8240e6831e5769ebab26340bb8ae3768630: CPI error 'Bosh::Clouds::VMCreationFailed' with message 'Server f558b95c-0b1f-4482-8e61-a20423cabd9e' state is error, expected active Exceeded maximum number of retries. Exceeded max scheduling attempts 3 for instance f558b95c-0b1f-4482-8e61-a20423cabd9e. Last exception: It is not allowed to create an interface on external network dc8385e3-3771-481d-ba98-4abf1f21c30b' in 'create_vm' CPI method (00:01:13)    Failed compiling packages > nginx/8d0c80e9594988b792c1f83a8c1598a3f4d02b7a: CPI error 'Bosh::Clouds::VMCreationFailed' with message 'Server cf2f444c-7988-4ed1-93f5-5f3124adb0d4' state is error, expected active
Exceeded maximum number of retries. Exceeded max scheduling attempts 3 for instance cf2f444c-7988-4ed1-93f5-5f3124adb0d4. Last exception: It is not allowed to create an interface on external network dc8385e3-3771-481d-ba98-4abf1f21c30b' in 'create_vm' CPI method (00:01:14)
Error 100: CPI error 'Bosh::Clouds::VMCreationFailed' with message 'Server `4ccca6e9-5dd7-41da-8568-20469d3680d3' state is error, expected active
Exceeded maximum number of retries. Exceeded max scheduling attempts 3 for instance 4ccca6e9-5dd7-41da-8568-20469d3680d3. Last exception: It is not allowed to create an interface on external network dc8385e3-3771-481d-ba98-4abf1f21c30b' in 'create_vm' CPI method
---
director_uuid: e23b5aa1-d934-4176-a9c4-680c74c4b1f0

meta:
  environment: dev

  floating_static_ips:
  - <IP_ADDRESS>
networks:
  - name: floating
    type: vip
    cloud_properties:
      net_id: dc8385e3-3771-481d-ba98-4abf1f21c30b
      security_groups: []
  - name: cf2
    type: manual
    subnets:
    - range: <IP_ADDRESS>/24
      gateway: <IP_ADDRESS>
      dns:
      - <IP_ADDRESS>
      reserved:
      - <IP_ADDRESS> - <IP_ADDRESS>
      static:
      - <IP_ADDRESS> - <IP_ADDRESS>

      cloud_properties:
        net_id: ee216d00-1adb-4b00-a3f8-5d0a5514c142
        security_groups: ["cf"]

  - name: cf1
    type: manual
    subnets:
    - range: <IP_ADDRESS>/20
      gateway: <IP_ADDRESS>
      dns:
      - <IP_ADDRESS>
      static:
      - <IP_ADDRESS>
      - <IP_ADDRESS>
      - <IP_ADDRESS>
      - <IP_ADDRESS> - <IP_ADDRESS>

      cloud_properties:
        net_id: dc8385e3-3771-481d-ba98-4abf1f21c30b
        security_groups: ["cf"]

error It is not allowed to create an interface on external network dc8385e3-3771-481d-ba98-4abf1f21c30b' indicates you have put the external network's UUID into your manifest instead of the private network UUID – might that be the case?

Yes!! I was trying with external network afterwards i tried with private network as well. It worked. But i needed to manually assign static_ips in cf deployment manifest.

@voelzmo After assigning static ips in cf deployment manifest i am able to perform bosh deploy. and launch 16 vms successfully.  So slowly but steadily moving forward towards cloud foundry deployment with your help 👍
Appreciate your knowledge


Local images cannot be inserted in jupyter notebook.

In the markdown part, online images can be inserted well, but the local images cannot be recognized.

Hi, I'm using VS Code Insiders and the Python "dev" extension for insiders, and I'm having this same problem. Local images (on the same folder and folders above etc) doesn't render on python notebooks at all.
I tried to copy the suggested solution into settings.json "python.dataScience.useWebViewServer: true" or :false, but my vscode says this command isn't associated with anything.
So, what do I do? Thanks!

Hi, I'm using VS Code Insiders and the Python "dev" extension for insiders, and I'm having this same problem. Local images (on the same folder and folders above etc) doesn't render on python notebooks at all.
I tried to copy the suggested solution into settings.json "python.dataScience.useWebViewServer: true" or :false, but my vscode says this command isn't associated with anything.
So, what do I do? Thanks!

@AndrewLaganaro On insiders we've moving over to a new UI provided by the VSCode notebook interface. Could you please open up a new issue for this? This old issue was just for our old webview UI and we won't reactivate it.

@AndrewLaganaro On insiders we've moving over to a new UI provided by the VSCode notebook interface. Could you please open up a new issue for this? This old issue was just for our old webview UI and we won't reactivate it.


Replace use of requests in examples with httpx

E.g. https://gidgethub.readthedocs.io/en/latest/sansio.html#requests .

Yeah I think this is straightforward change. It'll be a matter of "Find and replace" requests with httpx.
httpx docs: https://www.python-httpx.org/quickstart/


Updated getInitialProps to getServerSideProps

Updated dependencies
With nextjs 9.3 getInitialProps has been replaced by getServerSideProps.
Prismic logo image path fixed

Shouldn't this use getStaticProps? Rationale here.

You're right @osamaaamer95, getStaticProps makes more sense here.
Want to take care of it?

@shaswatsaxena how about it?

@osamaaamer95 Sure, go ahead. I couldn't get the current previews setup to work with getStaticProps.

I think this ressource could be of great help in your situation:
https://nextjs.org/docs/advanced-features/preview-mode
See in particular the "Fetch preview data" section, where the preview logic would be specific to Prismic. I'll take a look in the coming days but feel free to experiment 👍

I found a working example in the Next.js examples folder: https://github.com/zeit/next.js/tree/canary/examples/cms-prismic
It shouldn't be too hard to add the api routes

Hey again 👋
I made the new getStaticProps work with Prismic but it clashes a bit with the Toolbar. I'm trying to understand what can be done there but will push something in the meantime. Thank you for your patience 😊


Adding Thread Safe ability on HaeinsaTransaction for Put / Delete / Get with different hTable

Currently we cannot run the following scenario:

Create manager.// one thread
Begin transaction. // one thread
Run:
Put / Delete / Get. // with the same transaction with different threads.
On success of all Threads -> commit // one thread.

The fix I am suggesting will not change the current API at all, it will only add the ability to use some locks on mutation collections (using java.concurrent library).
I added a test to illustrate the scenario.
We would appreciate it if you would pull the attached code. Furthermore, any feedback would be welcome.

How do you think about this scenario?

MainThread: tx=BeginTransaction
Thread1: value1=get(tx, key1)
Thread1: newValue1=value1+1
Thread2: value1=get(tx, key1)
Thread2: newValue2=value1+1
Thread1: put(tx, key1, newValue1)
Thread2: put(tx, key1, newValue2)
MainThread: On success of all Threads, tx.commit()

I think user would expect that value stored in key1 should be value1+2. But actual value might be value+1. In my opinion, to support multi-threaded transaction, this problem should be solved. How do you think?
I'm now looking for how other transaction system handle this problem.

Yes this is obviously a problem, but as I see it this is a wrong usage of the library.
I made this ability to  enable running a "command" where each command only works on it's own rows, and each command can run in a different thread.
So maybe the definition of "thread safe" is not the best, but the point is working on multiple rows. As long as the rows are different you should be thread safe.
The mutli-threaded ability we added was to optimize the bounding between I/O and CPU. We are optimizing the concurrency on the I/O.
When running the example you describe, you actually do the opposite since it is preferable for you to get  the same row with the same thread. The other direction is not efficient.
So this scenario is not possible in our system , but as I see it, this scenario is also not relevant.
Maybe we should add better documentation or change the name of the Flag.
Anyhow, we appreciate your advice.

I added some code to protect this case.
I am aware that using this code, I prevent the option of mutate the same row more than once, even with the same thread.
What do you think about this ?


ClearOnSelect evaluated as true when multiple is false

Hi,
I'm trying to set some properties conditionally on the multiple prop.
Setting the clear-on-select on false when multiple is false doesn't work. It seems to me that clear-on-select always is evaluated as true when the property is set.
Docs of the ClearOnSelect prop note the following:
Clear the search input after select(). Use only when multiple is true.
So I think this is the expected behavior.
Which I find fairly odd, any chance this behavior can be updated?
Component:
<template>
    <div>
        <multiselect-element
                :options="options"
                :track-by="trackBy"
                :label="label"
                :allow-empty="allowEmpty"
                :disabled="disabled"

                :value="value"
                v-model="hiddenValueObject"

                :multiple="multiple"
                :close-on-select="!multiple"
                :hide-selected="multiple"
                :clear-on-select="multiple"
        ></multiselect-element>
        <input type="text" :name="formName" :value="hiddenValue">
    </div>
</template>

<script>
    import Multiselect from 'vue-multiselect';
    _ = require('lodash');

    export default {
        components: {
            'multiselect-element': Multiselect
        },

        data () {
            return {
                hiddenValueObject: '',
                hiddenValue: ''
            }
        },

        mounted () {
            this.setHiddenValue(this.value);
        },

        watch: {
            hiddenValueObject: function(value) {
                this.setHiddenValue(value);
            }
        },

        props: {
            options: {
                type: Array,
                default: function () {
                    return [];
                }
            },
            value: {
                default: null
            },
            trackBy: {
                type: String,
                default: 'id'
            },
            label: {
                type: String,
                default: 'name'
            },
            formName: {
                type: String,
                default: null
            },
            multiple: {
                type: Boolean,
                default: false
            },
            allowEmpty: {
                type: Boolean,
                default: true
            },
            disabled: {
                type: Boolean,
                default: false
            }
        },

        methods: {
            setHiddenValue(value) {
                if (!_.isArray(value)) {
                    value = [value];
                }

                this.hiddenValue = _.map(value, 'value').join(',');
            }
        }
    };
</script>

And the usage:
<multiselect
        :options='{!! json_encode(array_values($choices)) !!}'
        label="label"
        track-by="value"
        form-name="{{ $options['real_name'] }}"

        @if (!empty($values))
            :value='{!! json_encode($values) !!}'
        @endif

        @if ($options['multiple'])
            :multiple="true"
            :close-on-select="false"
            :hide-selected="true"
            :clear-on-select="true"
        @endif

        @if (!isset($options['attr']['required']))
            :allow-empty="true"
        @endif

        @if (isset($options['disabled']) && $options['disabled'])
            :disabled="true"
        @endif
    ></multiselect>

clearOnSelect is used to clear the search input. If you’re not using the multiple option, then the search input value is set to the selected option label. Thus it only makes sense when using together with multiple="true". Otherwise the input would not show the value at all.
I will investigate how does it behave in cases where multiple is set to false.

Been looking into it but it seems that everything is fine. When you add :clear-on-select="false" when you also have :multiple="false" it results in a bug just as expected, as it’s due to a wrong configuration.
Adding a warning for that. :)


Add a man page for Schema

Example (It goes on for much longer) -

to test, do man -l schema.1

@Lathie this looks great. Before merging, we should move schema.1 into a directory called man/ or man_pages/. I know I said that it's fine to put it in the base directory earlier, but I didn't realize that it's called schema.1. Some people wouldn't be able to recognize that it's a man page just by its filename if it's in the base directory.

@thomasdfischer created man_pages/


Lock Aavegotchis when listing

We discussed locking Aavegotchis when they are listed for sale, and unlocking when the listing is removed. This should be implemented before launch.

This has been implemented.

Great, closing.


[Question]  Production code deployment

Is there any way to roll back the changes?
Is there any way to do the zero downtime deployment with or without rolling releases?

Rollback is by redeploying your previous tag. Use the -t or --tag option. Zero downtime deployment via rolling deployments is explained in the README. You need to provide a few configuration options and tune them for your load balancer check intervals. We do zero downtime deploys 100% of the time.


pending label not showing up for initial liquidity

similar design
https://www.figma.com/file/bZh7tOiLm6Ls9Hl4RL6qMC5X/Augur-Redesign

@phoebemirman what is this? when I go to figma I dont see anything so not sure what the fix is, that I am looking for

just the pending label on initial liquidity orders was not showing up previousy


Add babel polyfill for IE11

Overview
The client code is currently missing polyfills for IE11, which means it won't work there:

Add the required polyfill so the site works in IE11+
Expected Behavior
Site should load in IE11
Actual Behavior
Site fails to load because of some ES6/7 syntax IE11 can't parse.
Additional context
We typically add the polyfill in our webpack configuration file, but since this is set up in CRA we will have to figure out an alternate way to handle it.

Is this a good opportunity to use https://babeljs.io/docs/en/babel-preset-env?

Oh, cool -- that's already available in the package.json file but not yet configured, so I'll see about adjusting package.json to use it.

I looked into this a bit more and it seems like the immediate problem is a missing polyfill for String.prototype.startsWith, which is required by redux-act.


The file is locked: nio:./crawler/crawlstore/mvstore/crawler//mvstore [1.4.196/7]

Hi Pascal,
we have a cronjob which is starting the crawling in a scheduled manner.
Sometimes we get the following error and the crawler does not start the next scheduled crawl run.
I found this issue https://github.com/Norconex/collector-http/issues/336 and you mention it was already fixed, any further suggestions?
java.lang.IllegalStateException: The file is locked: nio:./crawler/crawlstore/mvstore/crawler//mvstore [1.4.196/7]
        at org.h2.mvstore.DataUtils.newIllegalStateException(DataUtils.java:765)
        at org.h2.mvstore.FileStore.open(FileStore.java:173)
        at org.h2.mvstore.MVStore.<init>(MVStore.java:347)
        at org.h2.mvstore.MVStore.open(MVStore.java:395)
        at com.norconex.collector.core.data.store.impl.mvstore.MVStoreCrawlDataStore.<init>(MVStoreCrawlDataStore.java:57)
        at com.norconex.collector.core.data.store.impl.mvstore.MVStoreCrawlDataStoreFactory.createCrawlDataStore(MVStoreCrawlDataStoreFactory.java:49)
        at com.norconex.collector.core.crawler.AbstractCrawler.createCrawlDataStore(AbstractCrawler.java:243)
        at com.norconex.collector.core.crawler.AbstractCrawler.doExecute(AbstractCrawler.java:204)
        at com.norconex.collector.core.crawler.AbstractCrawler.startExecution(AbstractCrawler.java:184)
        at com.norconex.jef4.job.AbstractResumableJob.execute(AbstractResumableJob.java:49)
        at com.norconex.jef4.suite.JobSuite.runJob(JobSuite.java:355)
        at com.norconex.jef4.suite.JobSuite.doExecute(JobSuite.java:296)
        at com.norconex.jef4.suite.JobSuite.execute(JobSuite.java:168)
        at com.norconex.collector.core.AbstractCollector.start(AbstractCollector.java:131)
        at com.norconex.collector.core.AbstractCollectorLauncher.launch(AbstractCollectorLauncher.java:95)
        at com.norconex.collector.http.HttpCollector.main(HttpCollector.java:74)

Best regards
Sascha

Which version are you using? Is that file shared by more than one running instance?  Are you relaunching a crawler in the same JVM?  If yes, it is possible java did not release the file handle before you requested it again.

Hi Pascal,

version 2.8.1
the file is not shared by more than one running instance
our cron job is running the following command sh collector-http.sh -a start -c project/crawler-configuration.xml

Best regards
Sascha
Best regards
Sascha

Can you share a config that reproduces it?
This could happen if the crawler tries to open the crawlstore while there is still a lock on it.   Is it possible when you start the crawler that the other one has not yet terminated? You should normally see a different error if so, but I am looking for patterns.
So maybe forcing a delay between two runs could fix this?   If you restart after that exception, will it work eventually or does it always fail after that exception is thrown?   If it works fine after a while, it would tend to confirm there is sometimes not enough delay between two runs and the lock from the previous run is still there.
One thing to try is to make sure one execution does not start until the previous one is terminated and/or modify the collector launch script to insert a wait time before starting the JVM.
There might be a way to fix this natively, but we need to find out what causes this first.

Hi Pascal,
The cronjob itself will restart and the crawler then work as expected (sometimes it requires a few runs till the crawler starts again). As recommended I added a delay of 15 minutes to the cronjob but it can happen that the crawler runs longer and again is very close to the next crawl run. (Which leads to the the same error message).
I tried to find the pattern in the log files,  here you can see the time frames are overlapping. The file is locked exception is thrown while the crawler is still running. Though the log file itself is written after the previous crawl run is completed.
Logfile 1 - /2019/07/01/logs/201907010545060000__obfuscated.log
2019-07-01 05:30:08 INFO - Starting execution.
2019-07-01 05:49:21 INFO - Running: END

Logfile 2 - /2019/07/01/logs/201907010549210895__logistics.log
2019-07-01 05:45:48 ERROR - Execution failed
java.lang.IllegalStateException: The file is locked: nio:./obfuscated/crawlstore/mvstore/obfuscated//mvstore [1.4.196/7]

How can I make sure one execution does not start until the previous one is terminated? (When using cronjob).
Best regards
Sascha

Normally you do not have to do anything since your new process would throw an exception and you should have in your logs something that starts with "JOB SUITE ALREADY RUNNING".   In other words, it should detect a job is already running and abort (letting the current job finish normally).
I do not get why you have this error instead.
If you have your own code around the execution, you can have a look at
com.norconex.jef4.status.JobSuiteStatusSnapshot#newSnapshot(File)

The file is the suite .index file that gets created under your progress folder.  Once you have that snapshot object created, you can invoke #getRoot() to obtain a IJobStatus instance.  From that last object, you can use the methods only to get the status of the last/current job.

Hi Pascal,
Google meanwhile reduced the error most likely due to an error with Norconex itself.
See https://github.com/google-cloudsearch/norconex-committer-plugin/issues/15
Any ideas how we can solve this behavior?
Best regards
Sascha

Since this one appears to be related to #634, shall we close as a duplicate?

Hi @essiembre
yes absolutely


Ensure role assignment name is unique

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Provisioning


[RFC] Merge Raneto and Raneto-Core

I'd like to hear your opinions on this, hence the RFC-Tag.
Would it make sense to merge Raneto and the Raneto-Core package into one, providing not an example server implementation but one that is working out of the box? I had to make changes to the core package a few days ago and realized it is pretty impractical to npm-update that way. Besides, that would make for a much cleaner directory layout.

You are right, thanks for bringing this up.
I had this on a list to discuss with Gilbert and doing it here is an even better idea.
I see this going 2 ways so far:

We merge Raneto Core into Raneto and make a separate example repository for viewers
Underlying functionality of Raneto gets moved to Raneto Core, adding tests along the way and the Raneto repo become the example repository.

Open to others as well

I'd propose merging Raneto Core into Raneto and creating a new Raneto-Server (or whatever) repository containing a properly named and ready-to-use app with Raneto listed in the package.json as a dependency. The repo could also contain a developer branch with the debug module enabled etc.
What do you think?

Yep, we can extract the example/ directory as a starting point for the ready-to-use demo including Raneto's own documentation contents.
So long as the example repo contains "example", "demo" or something similar, and a link back to the main repository, that should all work fine

Maybe a Yeoman generator would help?

Yeoman sounds like a perfect match, though I'd rather see it as an alternative beneath the demo repo, considering installing Yeoman is an additional step that seems maybe too complicated for some people.

Having both a repo and Yeoman as options would work.
I'm in the camp of wanting a regular repo and wouldn't mess with Yeoman.

The reason I originally split out Raneto and Raneto Core was because I imagined that down the line Raneto Core could be the basis for all sorts of "flat file" apps, not just a knowledgebase (e.g. Blog, CMS etc.). It also meant the core could be isolated and have better tests, then you could include it as a dependency into this repo, similar to the structure of Laravel. I'm aware this makes development slightly harder but I still think it's a good solution.

I'd propose merging Raneto Core into Raneto and creating a new Raneto-Server (or whatever) repository containing a properly named and ready-to-use app

I don't see how this is any different to what we have just now>

Underlying functionality of Raneto gets moved to Raneto Core, adding tests along the way, and the Raneto repo become the example repository.

I'm not necessarily adverse to this. My feelings are that this repo should always be the "ready-to-use app" and include any "knowledgebase" specific code. Any functionality that has a more general application (e.g. folder structure, categories, file parsing etc.) should go into Core.
One other way we could structure it is to split out the "knowledgebase" specific code into it's own repo and make this repo have two dependencies: raneto-core and raneto-knowledgebase. That way core can remain generic and raneto-knowledgebase can build/extend the functionality as needed.

I've got a newbie question :smiley: I have two repositories within a folder: raneto and raneto-core. How to make raneto use raneto-core from neighbour repository and not its own node_modules?

@yaruson The easiest thing to do is make the ranteo package.json use a local path to raneto-core.

Any update on this? I'd like to help with some bugs but wanted to wait to see if there was any movement on this?

Marked the Raneto-Core repo as deprecated a month or two ago, and made it clear no new Issues/PRs would be accepted while I reached out to resolve remaining issues.
I'd say it's fine to move on now, without responses, due to the length of time.
The opportunity was there, but we have to keep moving.
So, I'll merge Raneto-Core into Raneto tonight and create a maintenance release so we can move forward on locking down a v1.0

Raneto-Core has now been merged and will be part of the next release
This should happen within a few days as PRs close out.
Still not sure how I feel on a separate example repository, so let's keep it all in one for now

Completed.
Closing


changed fail color to orange and added encouragement phrases

As suggested here.
Example:

Closing as it doesn't seems relevant anymore.


Add arm64 support for binaries and docker images

Which problem is this PR solving?

Platform support for arm64.

Resolves #1656
Short description of the changes

This commit adds building support for binaries and a Dockerfile for all-in-one container image.

Signed-off-by: Henry Wang<EMAIL_ADDRESS>

Hi @yurishkuro sorry to bother but I think the CI machine currently has come issues with the GPG key?
As it gives me output in https://travis-ci.org/github/jaegertracing/jaeger/jobs/676105369
gpg: key 86E50310: public key "Yarn Packaging<EMAIL_ADDRESS>imported
gpg: Total number processed: 1
gpg:               imported: 1  (RSA: 1)
gpg: Signature made Mon 09 Mar 2020 03:52:13 PM UTC using RSA key ID 69475BAA
gpg: BAD signature from "Yarn Packaging<EMAIL_ADDRESS>> GPG signature for this Yarn release is invalid! This is BAD and may mean the release has been tampered with. It is strongly recommended that you report this to the Yarn developers.

Hi @yurishkuro, this PR has been a while after my "workaround" of this binary naming issue, anything about the "renaming the binaries consistently" or we leave it as it is? Could you please review again? Thanks!

I put it on the agenda for the next project meeting, but let me ping ppl directly to see if we can resolve faster: @pavolloffay @objectiser

@yurishkuro I'm fine with the rename, to avoid duplication.

@MrXinWang do you want to (a) rebase and (b) change Makefile to have a single name format for every binary?

Hi @yurishkuro.
All done :)) Please review again, thanks!

Hi @yurishkuro ! I tested on both my x86_64 and arm64 machine with following methods:

Build all binaries and docker images by:

git clone https://github.com/jaegertracing/jaeger.git && cd jaeger
git submodule update --init --recursive
make install-tools
make docker

Use file command to check if the binary has the right architecture. Use docker run command to test if all docker images can be successfully started.

Could you also try these steps?

./scripts/travis/build-docker-images.sh
make build-all-platforms && ./scripts/travis/package-deploy.sh

Also, please add a change to the CHANGELOG describing the breaking changes to binary names in the tarball distributions.
We probably also need to change some docs, e.g.

https://www.jaegertracing.io/download/ only links to amd64 tarballs, we may want to mention that there are other arch's available
https://www.jaegertracing.io/docs/1.18/windows/ shows examples with concrete binary names, which may not be correct anymore.

@yurishkuro Tested these two commands, both worked well.
I have also added the related change to CHANGELOG.md

minor suggestion: in the future, try to avoid force-pushing a single commit, it makes it impossible to see incremental changes, e.g. in the last one the whole Makefile is shown as changed again. We squash commits before merge anyway.

minor suggestion: in the future, try to avoid force-pushing a single commit
Thanks for the suggestion, yes that makes sense. I did some rebase work and to make the commit tree clean, I used squash and force push again....sorry about that...

@MrXinWang just a couple of nits. Overall it looks good.

Thanks @pavolloffay. Rebased and added 90b5c3f66f1a71197f478a817a82d7d5999db2c9 to fix your request. Please take a look.

thanks @MrXinWang

Hi, can anyone direct me to the Arm based Agent Docker image? we would like to run on AWS Graviton.


checkNumberStatus

is the checkNumberStatus Working? when i try check number status i get this error below.
number_status = self.wapi_functions.checkNumberStatus(number_id)
File "/home/ubuntu/WebWhatsapp-Wrapper/webwhatsapi/wapi_js_wrapper.py", line 45, in getattr
wapi_functions = dir(self)
TypeError: 'NoneType' object is not iterable
please solved and let me know.

i'm using the function driver.check_number_status(number_id) and working well

just used it a couple of hours ago:
    `print(driver.check_number_status(phone_whatsapp).status)`

i dont think your problem is created by the function checkNumberStatus

i'm using the function driver.check_number_status(number_id) and working well

can you tell me where i shoud change in code.

I have the same problem :/


How to get depth and extrinsic parameter?

Hi, thanks for your great job. I want to use this data to train a 3D policy where point-cloud and extrinsic parameter are needed but unfortunately I cant see this information in the raw data. So can you provide this information or tell how to get these？
Thanks for your attention and wait for your kind response!

Hi!
Thanks for your interest! The extrinsics and depth information is currently not stored in the RLDS dataset unfortunately.
For both, you need to process our raw dataset, which we also uploaded. The extrinsic information is stored in each episode's metadata file.
For depth, we don't store the actual depth data (since it's tricky to compress and would blow up storage usage by a lot), but instead our raw data contains stereo video, which you can convert into depth images of the desired resolution, eg using ZED's stereo-to-depth model.
We are also working on releasing depth generated by a new TRI depth model, but the ETA on that is not super clear...


(fix) Revert email address back to<EMAIL_ADDRESS>

Trello card URL:
https://trello.com/c/NlR6ykbK/606-switch-support-email-address-as-the-new-email-address-is-now-available#
Changes in this PR:
Email<EMAIL_ADDRESS>is now available to use

 Switch support email address as the new email address is now available


Docker-ize

We intend to address "Docker-ization" tonight, at the meeting. (Get ready!)

I spent a lot of time documenting our environment on the backend and how it interacts with docker. I strongly suggest using some of what I have written to help pump-up your README :). Take a look at ms-email for a good example.
I also suggest using our node-js base image and our Dockerfile as they will suit your scenario pretty darn well dog. Make sure to note that the base image is built using alpine so it does not come with bash installed. You can still use bash with it for development as detailed in the README I linked earlier.

As an update to above, the README's no longer suggest using bash for a development environment. They instead use /bin/sh.

The front end is still not working with docker, at least not with docker-compose. We can look into this tomorrow


Support Faraday's timeout options

I would like to control Faraday's timeout options, so this pull request supports both open_timeout and timeout options.
This pull request is based on #46's branch.
So I will rebase master branch after #46 was merged If this pull request was accepted.

Maybe this is actual diff from #46 ?
https://github.com/ma2gedev/sbpayment.rb/compare/feature/control_retry...ma2gedev:feature/timeout_options
👍 Code looks good to me. cc @banyan @kachick

Maybe this is actual diff from #46 ?

Yes! Thank you for your concern.

Thanks for merging #46. I'll execute git rebase master, please wait.

Just rebased!

👍 Thank you for your contribution @ma2gedev !

@ma2gedev
Released v0.8.0
https://github.com/quipper/sbpayment.rb/releases/tag/v0.8.0
https://rubygems.org/gems/sbpayment/versions/0.8.0

Thank you!! 👍


[Bug]: Query.data is undefined for API calls that return an error code but no body

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When I complete a GET request to https://httpbin.org/status/400, Query1.data is undefined.

I'm only showing the data and responseMeta properties of the query object here:

I expect Query1.data to contain the error so I can react accordingly.
Steps To Reproduce

Create a GET request to https://httpbin.org/status/400 using the API integration.
Run the query.
Observe that a response is not displayed in the result pane below.
Log Query1 to the console and observe that the data property is undefined.

Public Sample App
No response
Environment
Production
Issue video log
No response
Version
Cloud - 1.9.53-SNAPSHOT
 Front conversations

I've also found that the response pane doesn't show the response body even if there is one. You can reproduce this with a GET request to https://www.toptal.com/developers/postbin/api/bin/ASDF. It will return as 404 with a JSON object containing more info, but the response pane will still display as if it was a total failure. You have to go to the Logs pane to see the body in the error message there.

I misunderstood this users issue. They have an API that is returning a body but it's not populating to the data field. This detail was lost in their conversation with our AI bot and later examples trying to recreate the issue. I'll file a new issue.


Add AIDs for ETSI/3GPP packages related to SIM toolkit

Can be assigned to myself for now.
I need to collect all the required information first.

Thank you, assigned to you


Make all file access 64-bit (uint64_t)

Forward port of #47254 to master. Draft for now as I haven't tested the changes yet, nor done a full pass to check potential type mismatches on calls to get_buffer() and similar.

This changes the types of a big number of variables.
General rules:

Using uint64_t in general. We also considered int64_t but eventually
settled on keeping it unsigned, which is also closer to what one would expect
with size_t/off_t.
We only keep int64_t for seek_end (takes a negative offset from the end)
and for the Variant bindings, since Variant::INT is int64_t. This means
we only need to guard against passing negative values in core_bind.cpp.
Using uint32_t integers for concepts not needing such a huge range, like
pages, blocks, etc.

In addition:

Improve usage of integer types in some related places; namely, DirAccess,
core binds.

Note:

On Windows, _ftelli64 reports invalid values when using 32-bit MinGW with
version < 8.0. This was an upstream bug fixed in 8.0. It breaks support for
big files on 32-bit Windows builds made with that toolchain. We might add a
workaround.

Fixes #44363.
Fixes godotengine/godot-proposals#400.

I made a test build for Windows (32-bit and 64-bit, both editor and templates) to help validate this PR: https://downloads.tuxfamily.org/godotengine/testing/4.0-dev-pr48768-windows.zip
See https://github.com/godotengine/godot/pull/47254#issuecomment-804145911 for suggestions on how to make a reproduction project.

I made a test build for Windows (32-bit and 64-bit, both editor and templates) to help validate this PR: https://downloads.tuxfamily.org/godotengine/testing/4.0-dev-pr48768-windows.zip

Did some testing with this build like in https://github.com/godotengine/godot/pull/47254#issuecomment-842171934
I have the same results, big PCKs are working properly, and this error also happens when trying to get too big a buffer:
ERROR: Condition "p_size < 0" is true. Returning: ERR_INVALID_PARAMETER
   at: resize (./core/templates/cowdata.h:260)
ERROR: Can't resize data to<PHONE_NUMBER> elements.
   at: (core/core_bind.cpp:1331)

Should be good to go as since it works in my tests (I just pushed an amend to handle a few missed get_len() and get_buffer() return types that could be uint64_t).


Update to Azure Data Factory Event (blob) Based Trigger breaks existing pipeline

Hi,
According to the documentation, an event trigger needs:
-an Azure Subscription
-a Storage Account Name
-either a blob starts with or blob ends with
Using these parameters, I have successfully created the pipeline I need, In that the pipeline looks for files dropped into any container of the storage account.
However it looks like the ADF code has been updated before the documentation because now my trigger requires the Container Name as a compulsory parameter.
This has broken my pipeline - please make the Container Name Dependency an option
Thanks
Mike

@michaelhambe Thanks for reaching out. We will review and provide an update as appropriate.

@michaelhambe Yes, you have correctly identified that the container name is now a mandatory field. Unfortunately it cannot be made optional.
I am assigning the issue to the content owner to take a look and update the doc as appropriate.
@djpmsft Can you please have a look at this ?

Hi,
Can you please explain why you broke the existing functionality? And why it cannot be made optional or accept a wild card?
My pipeline worked without issue until this latest change.
Thanks
Mike
Get Outlook for iOShttps://aka.ms/o0ukef

From: ChiragMishra-MSFT<EMAIL_ADDRESS>Sent: Thursday, July 4, 2019 6:57:13 PM
To: MicrosoftDocs/azure-docs
Cc: Michael Hambe; Mention
Subject: [EXTERNAL] Re: [MicrosoftDocs/azure-docs] Update to Azure Data Factory Event (blob) Based Trigger breaks existing pipeline (#34517)
@michaelhambehttps://github.com/michaelhambe Yes, you have correctly identified that the container name is now a mandatory field. Unfortunately it cannot be made optional.
I am assigning the issue to the content owner to take a look and update the doc as appropriate.
@djpmsfthttps://github.com/djpmsft Can you please have a look at this ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/34517?email_source=notifications&email_token=AKUWDEUFCQSOQL4IVRS7XTTP5W3OTA5CNFSM4H5S4CZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZGZMTQ#issuecomment-508401230, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKUWDERGZUZXJ4VY2ZB6YTTP5W3OTANCNFSM4H5S4CZQ.
global.weir http://www.global.weir/
Twitterhttp://www.twitter.com/weirgroup | Linkedinhttps://www.linkedin.com/company/the-weir-group | Facebookhttp://www.facebook.com/weirgroup
[WeirLogo]
CONFIDENTIAL:
The information contained in this email (including any attachments) is confidential, subject to copyright and for the use of the intended recipient only. If you are not the intended recipient please delete this message after notifying the sender. Unauthorised retention, alteration or distribution of this email is forbidden and may be actionable.
Attachments are opened at your own risk and you are advised to scan incoming email for viruses before opening any attached files. We give no guarantee that any communication is virus-free and accept no responsibility for virus contamination or other system loss or damage of any kind.

@michaelhambe Sorry to hear that your existing functionality broke.
Please wait for the content owner to comment on this with more information.

Thanks Chirac
Get Outlook for iOShttps://aka.ms/o0ukef

From: ChiragMishra-MSFT<EMAIL_ADDRESS>Sent: Thursday, July 4, 2019 19:04
To: MicrosoftDocs/azure-docs
Cc: Michael Hambe; Mention
Subject: [EXTERNAL] Re: [MicrosoftDocs/azure-docs] Update to Azure Data Factory Event (blob) Based Trigger breaks existing pipeline (#34517)
@michaelhambehttps://github.com/michaelhambe Sorry to hear that your existing functionality broke.
Please wait for the content owner to comment on this with more information.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/34517?email_source=notifications&email_token=AKUWDETE6HII545K3HUXOR3P5W4KXA5CNFSM4H5S4CZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZG2A4Y#issuecomment-508403827, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKUWDEQP5YUORNM4HPPKM5LP5W4KXANCNFSM4H5S4CZQ.
global.weir http://www.global.weir/
Twitterhttp://www.twitter.com/weirgroup | Linkedinhttps://www.linkedin.com/company/the-weir-group | Facebookhttp://www.facebook.com/weirgroup
[WeirLogo]
CONFIDENTIAL:
The information contained in this email (including any attachments) is confidential, subject to copyright and for the use of the intended recipient only. If you are not the intended recipient please delete this message after notifying the sender. Unauthorised retention, alteration or distribution of this email is forbidden and may be actionable.
Attachments are opened at your own risk and you are advised to scan incoming email for viruses before opening any attached files. We give no guarantee that any communication is virus-free and accept no responsibility for virus contamination or other system loss or damage of any kind.

Hey Mike,
We have recently introduced multiple improvements to the Event Based Trigger design and one of them, as you have experienced, is making container name a required property. The impetus behind this change was a large swath of users pointing their triggers at the root of a storage account and as a result, a very large number of pipeline runs would occur.
As noted, the documentation has not been updated and we will work to resolve that it the very near future.
I'm sorry to hear that you implementation was broken, but I am happy to work with you to make sure you reclaim your existing functionality.
Thanks,
Daniel

Hi Daniel,
Thanks for the reply,
I've actually raised a support request for this issue, and have a meeting Tuesday morning AEST with the product team to discuss.
The issue number is<PHONE_NUMBER>01070, and Jackie (copied here) has been helping.
Thanks
Mike
Michael Hambe
Data Scientist
The Weir Group PLC
1 Marden Street
Artarmon, NSW 2164
Australia
T +61 2 8081 7452
M +61 437 936 604
michael.hambe<EMAIL_ADDRESS>global.weir
Twitter http://twitter.com/weirgroup | Facebookhttp://www.facebook.com/WeirGroup | YouTubehttp://www.youtube.com/user/WeirMineralsDivision

From: Daniel Perlovsky<EMAIL_ADDRESS>Sent: Saturday, 6 July 2019 02:48
To: MicrosoftDocs/azure-docs
Cc: Michael Hambe; Mention
Subject: [EXTERNAL] Re: [MicrosoftDocs/azure-docs] Update to Azure Data Factory Event (blob) Based Trigger breaks existing pipeline (#34517)
Hey Mike,
We have recently introduced multiple improvements to the Event Based Trigger design and one of them, as you have experienced, is making container name a required property. The impetus behind this change was a large swath of users pointing their triggers at the root of a storage account and as a result, a very large number of pipeline runs would occur.
As noted, the documentation has not been updated and we will work to resolve that it the very near future.
I'm sorry to hear that you implementation was broken, but I am happy to work with you to make sure you reclaim your existing functionality.
Thanks,
Daniel
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/34517?email_source=notifications&email_token=AKUWDET676S6IA67ZF62GWLP553OTA5CNFSM4H5S4CZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZJ6DVI#issuecomment-508813781, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKUWDETW3CY5SJFA4TX6ZHTP553OTANCNFSM4H5S4CZQ.
global.weir http://www.global.weir/
Twitterhttp://www.twitter.com/weirgroup | Linkedinhttps://www.linkedin.com/company/the-weir-group | Facebookhttp://www.facebook.com/weirgroup
[WeirLogo]
CONFIDENTIAL:
The information contained in this email (including any attachments) is confidential, subject to copyright and for the use of the intended recipient only. If you are not the intended recipient please delete this message after notifying the sender. Unauthorised retention, alteration or distribution of this email is forbidden and may be actionable.
Attachments are opened at your own risk and you are advised to scan incoming email for viruses before opening any attached files. We give no guarantee that any communication is virus-free and accept no responsibility for virus contamination or other system loss or damage of any kind.

@michaelhambe We are adding this functionality back this week. You can expect to see it in your production environment either Wednesday or Thursday

closed #34517

#please-close


Setting UI?

Wil the settings for Scaled Cantrips ever be in UMM window instead of json?

This would be nice to have, indeed.

Added with latest commit.