Connect runs after componentWillUnmount

4.4.4 has a change that breaks my current solution.
I'm cleaning the state dispatching the action in the componentWillUnmount. It never caused Connect method invocation after unmount happend.
Next is how I'm using connect:
export default connect(state => ({
    state1: state.state1
    state2: state.state2
  }
}))(Component);

Tested on 4.4.1, 4.4.2, 4.4.3, 4.4.4

Ah, good point, thanks.

Hmm, no, I can’t reproduce it.
I added a test confirming setState() doesn’t get called when you dispatch from componentWillUnmount: 85982f64ed891c6e191e46a1d076edf569670f2e.
Do you mind providing a reproducible test case showing otherwise?

The issue can be reproduced when you have a child component which fetched by parent.
Initial state: data.profile
Clean state: data = null
Connects to data.profile
I feel that it's not good way to use nullable object properties. But this was missed during code writing. With 4.4.4 this issue appeared. So do we have an issue on both sides? Or only my side?
@connect(null)
export default class Parent extends React.Component {
  componentWillMount() {
    this.props.dispatch(fetch());
  }

  componentWillUnmount() {
    this.props.dispatch(clean())
  }

  render() {
    return <Child />
  };
}

@connect(state => ({ profile: state.test.data.profile }))
class Child extends React.Component {
  render() {
    return (

        Hello {this.props.profile.test1} {this.props.profile.test2}

    );
  };
}

Simple actions:
export function fetch() { return { type: 'fetch' } }
export function clean() { return { type: 'clean'} }

Reducers:
function initializeState() {
  return {
    data: {
      profile: null
    }
  }
}

export default function test(state = initializeState(), action = {}) {
  switch (action.type) {
    case 'fetch':
      return Object.assign({}, state, {data: {profile: {test1: 'John', test2: 'Little'}}});
    case 'clean':
      return Object.assign({}, state, {data: null});
    default:
      return state;
  }
}

I’d really appreciate if you could fold this into a standalone project on GitHub. Thanks!

I tried adding this test based on your sample:
    it.only('whatever', () => {
      function reducer(state = { data: null }, action) {
        switch (action.type) {
          case 'fetch':
            return { data: { something: 42 } }
          case 'clean':
            return { data: null }
          default:
            return state
        }
      }

      @connect(null)
      class Parent extends React.Component {
        componentWillMount() {
          this.props.dispatch({ type: 'fetch '})
        }

        componentWillUnmount() {
          this.props.dispatch({ type: 'clean' })
        }

        render() {
          return <Child />
        }
      }

      @connect(state => ({ something: state.data.something }))
      class Child extends React.Component {
        render() {
          return null
        }
      }

      const store = createStore(reducer)
      const div = document.createElement('div')
      ReactDOM.render(
        <ProviderMock store={store}>
          <Parent />
        </ProviderMock>,
        div
      )
      ReactDOM.unmountComponentAtNode(div)
    })

For me, it fails consistently in 4.4.4, 4.4.3, and 4.4.0.

Able to reproduce with a slightly different test case. Looking into it.

Should be fixed in 4.4.5.
Thank you for reporting.

Thank for the quick fix. Sorry that I was not able to make a test projects in time and you did it by yourself.

No problem. I almost gave up but then figured out how to do it without giving up perf improvements in 4.4.4. :smile:

I don't understand what was changed, but 4.4.5 breaks my current solution :)
render function is calling when i dispatch setLocale, but FormattedMessage doesn't change the message. I works on previous releases
I will investigate the bug late, but for now here is my component
import React, { Component } from 'react';
import { connect } from 'react-redux';
import { setLocale } from '../../actions/app';
import CSSModules from 'react-css-modules';
import { FormattedMessage, injectIntl } from 'react-intl';
import styles from './styles/index.local.css';

@CSSModules(styles, {
  allowMultiple: true
})
class SettingsPage extends Component {
  render() {
    const { locale, setLocale,
      intl: { formatMessage }
    } = this.props;

    return (

          <FormattedMessage id="pages.settings.title"></FormattedMessage>

        <form className="form-horizontal">

            <lable className="col-sm-1 control-label">
              <FormattedMessage id="pages.settings.language"></FormattedMessage>
            </lable>

              <select ref="select" className="form-control"
                onChange={ () => setLocale(this.refs.select.value) }
                value={ locale }
              >
                <option value="en">English</option>
                <option value="ru">Русский</option>
              </select>

            <lable className="col-sm-1 control-label">
              <FormattedMessage id="pages.settings.theme"></FormattedMessage>
            </lable>

              <select className="form-control">
                <option>
                  { formatMessage({id: 'pages.settings.theme.dark'}) }
                </option>
                <option>
                  { formatMessage({id: 'pages.settings.theme.light'}) }
                </option>
              </select>

        </form>

    );
  }
}

const mapStateToProps = (state) => {
  return {
    locale: state.getIn(['app', 'locale']),
  }
}

export default connect(mapStateToProps, {
  setLocale,
})(injectIntl(SettingsPage));

Ok i get it, i need add ownProps arg. Oh but it's a little confusing

const mapStateToProps = (state, own) => {
  console.log(state.getIn(['app', 'locale']))
  return {
    locale: state.getIn(['app', 'locale']),
  }
}

and
render() {
    const { locale, setLocale,
      intl: { formatMessage }
    } = this.props;
console.log('locale', locale)

in both locale is 'ru'.

@gaearon I am facing this in v7.2.0
I am injecting reducers only when particular module is getting opened and removing the reducer in componentWillUnmount
But connect() is getting invoked even after componentWillUnmount


ReferenceError: crypto is not defined on node

In order to use CAPI on node I must have the following code at the very beginning:
import crypto from 'crypto'
(globalThis as any).crypto = crypto

Otherwise, I get the following error:
        Object.defineProperty(this, "id", { value: marker + crypto.randomUUID() });
                                                            ^

ReferenceError: crypto is not defined

What version of node are you running? Capi requires node 20+

What version of node are you running? Capi requires node 20+

Thanks! It does work on Node 20.
I was using node 18 (which is currently the Active LTS). It feels a bit much to require the very latest "current" release, but I guess that it's fair game. Although, I didn't see any mention of this in the docs? Perhaps I missed it, though. However, if I didn't miss it, then I think that it would be a good idea to document this. I think that most developers assume libraries to work with the Active LTS releases, unless it's stated otherwise, ofc.

We had a similar issue before https://github.com/paritytech/capi/issues/1003, i think it has to do with the web crypto apis being experimental or stable. Its not our goal to force users to use the current release of node, but it is a circumstantial pain point since we are going from Deno -> Node -> Browser and there are transpilational issues along the way.

However, if I didn't miss it, then I think that it would be a good idea to document this

https://github.com/paritytech/docs.capi.dev/pull/17


Appears to login but AWS CLI and Boto3 won't work

Hi, gimme-aws-creds appears to sign me into the correct role:
........
........
Saving arn:aws:iam::xxx123456xxx:role/MyAccountAdministrator as xxx123456xxx-MyAccountAdministrator
Written profile xxx123456xxx-MyAccountAdministrator to /Users/halingg/.aws/credentials
... But I get errors with Aws-Cli and Boto3:
botocore.exceptions.ClientError: An error occurred (AccessDeniedException) when calling the GetRestApis operation: User: arn:aws:sts::xxx123456xxx:assumed-role/MyAccountAdministrator/halingg is not authorized to perform: apigateway:GET on resource: arn:aws:apigateway:us-east-2::/restapis
In my config I'm using:
gimme_creds_server = appurl
In the credentials file, it's writing the profile with access keys and all but when I use the profile, I get the errors above.
Any advice on debugging this? I feel stuck.
Thank you,
Halin

Are you using MacOS Catalina by any chance?
Have a look at your homedir and see if you find a ~ (tilde) directory. E.g. do you see this in your terminal?
/Users/Halin/~
I found the same problem. Basically gimme-aws-creds was writing my credentials to /Users/Me/~/.aws/credentials instead of to /Users/Me/.aws/credentials so the profile keys didn't exist.
It went away after performing a clean wipe of my MacBook.
The issue isn't specifically to do with gimme-aws-creds btw, I never really got to the bottom of it. Both Python3 and GoLang performed the same natively when calling ~ from my code or a terminal.
--
Jonathon Taylor
<EMAIL_ADDRESS>On Wed, 8 Jan 2020, at 12:37, HalinGG wrote:

Hi, gimme-aws-creds appears to sign me into the correct role:

........
........
Saving arn:aws:iam::xxx123456xxx:role/MyAccountAdministrator as xxx123456xxx-MyAccountAdministrator
Written profile xxx123456xxx-MyAccountAdministrator to /Users/halingg/.aws/credentials

... But I get errors with Aws-Cli and Boto3:

botocore.exceptions.ClientError: An error occurred (AccessDeniedException) when calling the GetRestApis operation: User: arn:aws:sts::xxx123456xxx:assumed-role/MyAccountAdministrator/halingg is not authorized to perform: apigateway:GET on resource: arn:aws:apigateway:us-east-2::/restapis

In my config I'm using:

gimme_creds_server = appurl

In the credentials file, it's writing the profile with access keys and all but when I use the profile, I get the errors above.

Any advice on debugging this? I feel stuck.

Thank you,

Halin

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/Nike-Inc/gimme-aws-creds/issues/161?email_source=notifications&email_token=AGHRZ6RFQPI2RWZNUNF7FYTQ4U4ALA5CNFSM4KEB4POKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IEUUKKA, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGHRZ6XF5BIBKYIOO3L5HDDQ4U4ALANCNFSM4KEB4POA.

Thanks @JDTAY ! It had to do with the credentials file mapping like you said.


Show pkg name and url when installing istio on VM

fixes https://github.com/istio/istio/issues/3600

/assign @ldemailly
The test you referred to in the issue seems stable on Prow now. I added more information in the VM installation script so users know what is being downloaded from where.

but how come it didn't download it in that prow job ?

@chxchx: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
754caa3d84504db9bb4bb8d603f70427d513eaf9
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Import trash data of Jabbeke IVBO

Is the info available in the generic provider? If not, we received the files. They are uploaded to the Trash Data folder.

On hold. Jabbeke didn't order a trash calendar integration.


Improve deployments

Fixes

Models for ApplicationSettings and ProjectSettings are created on hook.
Private Keys belongsTo on Git App based models.

Features

Coolify version config.
Private GitHub deployments.
Expose, labels for compose files.
Use application debug settings for debug logs.

cc @ijpatricio

I don't understand the JWT stuff much, but I guess we can't avoid it, right?? 😄
This is going really well!
I'm adding a test pipeline now, so we can start focusing on that also

I don't understand the JWT stuff much, but I guess we can't avoid it, right?? 😄
Haha, I do but we cannot. 😃


Refactor Spring integration modules

Currently common java files for Spring integration are copying to the gen-src directory of the other modules. Definitely it's not beautiful situation, so we need to refactor them in some other way, for example creating a module for common code.

I also agree. 👌


LUN-389: Upgrade tools dependencies

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword LUN.

Have you tested this?

Not every command but some, yes.


Sim swap subscriptions alignement with commonalities 0.5

What type of PR is this?
Add one of the following kinds:

enhancement/feature

What this PR does / why we need it:
Alignement of sim_swap-subscriptions.yaml & sim_swap-subscriptions.feature with commonalities 0.5 rules.
See detail in release note.
Which issue(s) this PR fixes:

Fixes #161
Special notes for reviewers:
This is breaking change.
Changelog input
 release-note
sim_swap-subscriptions.yaml:
- Change error structure definition to normalize error & status
- Update error code to introduce 429 & 422 - removed 5xx, 410 and 415 error
- Follow principle of data minimization by removing sinkCredential in POST & GET response
- Adapt request for 3-legs usage featuring identification of the phoneNumber
- Improve documentation part
- Add 3-legs examples

sim_swap-subscriptions.feature:
- Align test scenario with commonalities 0.5 improvements

Additional documentation
This section can be blank.
docs

Hello @rartych
I will need your help for the megalinter issue.
Megalinter is unhappy because in the 3LEGS examples the subscriptionDetail is null.
This attribute is mandatory but in these examples we will have an empty object.
I tried
subscriptionDetail: null
or
subscriptionDetail:
but neither work. Any idea?
Thanks for you help.


[DNM] Driver testing

Cross-repo testing for https://github.com/apple/swift-driver/pull/474

https://github.com/apple/swift-driver/pull/474
@swift-ci please smoke test

@swift-ci please smoke test Linux


Continuous Integration

Set up CI on Travis CI with a simple Rake task to start. Add aptible-tasks to your Gemfile, and add a default .travis.yml with
rvm:
  - "2.1"

which will run the default rake task.
To avoid having to reconfigure Travis CI, I'd do this after the repo has been made public.

Closed by #13.


[EOSi Frontend] #21 All these footer links should go...

All these footer links should go to the lead form page:
https://eosi-frontend.netlify.app/lead-form.html
Source url: https://eosi-frontend.netlify.app/why-eosi.html#
Reported by: Chris ()
Reported at: 07 Jul at 14:49 UTC
Type: Feedback
Location: US, Massachusetts, Buzzards Bay
Browser: Firefox 102.0
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0) Gecko/20100101 Firefox/102.0
OS: macOS 10.15
Screen: 2560x1440
Viewport: 1341x726
Referrer: https://eosi-frontend.netlify.app/industry-template.html
Screenshot:

For more details please visit report page on Ybug:
https://ybug.io/dashboard/reports/detail/xgsvhhzzhxn98hwrwh2q

I got this one


Release with tab dropdown fix

I just stumbled over the issue fixed in https://github.com/thednp/bootstrap.native/commit/35622a0ff3a0ea161f89c2a6921126256addb3b6 and I cannot use the library because of that. Is it possible to do a hot fix release 1.0.1? Would appreciate that a lot. Thanks! :)

@thednp Did you patch package.json yet? if so ill do a npm publish. ^^

@IngwiePhoenix does it use the lib files or the dist files? I don't quite remember. If so, I'll do an update to the json file for patch 1.0.1.

@thednp I use the libfiles in the require-calls.

Alright, you can push the 1.0.1 I think.

Done.

Thanks guys, that was quick! Don't forget the git tag.

@neothemachine
What do you men with git tag? Are you referring to 1.0.1 release?

Yep, creating a 1.0.1 tag in git.

@neothemachine
Read here https://github.com/thednp/bootstrap.native/releases/tag/1.0.1
also the 1.0.1 CDN will be available within a few hours.


feat: unique entity id requirement emphasis (#1027)

UEI is now required for entities of type "Contractor" and "Subrecipient". Update our validation logic to reflect this.
Ticket #1027
Description
Updates our validation logic to reflect new treasury rules.
Screenshots / Demo Video
Coming soon post confirmation of logic over on https://github.com/usdigitalresponse/usdr-gost/issues/1027
Testing
Upload a sheet with a Subrecipient row that is missing a UEI but is of type "Subrecipient" - you should see a validation failure
Automated and Unit Tests

[x] Added Unit tests

Manual tests for Reviewer

[x] Added steps to test feature/functionality manually

Checklist

[x] Provided ticket and description
[x] Provided screenshots/demo
[x] Provided testing information
[x] Provided adequate test coverage for all new code
[x] Added PR reviewers

NOTE
I am not sure I did the logic correctly for the db update -- see questions over at https://github.com/usdigitalresponse/usdr-gost/issues/1027

Thanks for the comments, @as1729 !
based on Joe's comments over at https://github.com/usdigitalresponse/usdr-gost/issues/1027 I've got to change the logic slightly.
Moving back to draft for now to reflect this

Made a number of changes to accommodate correct logic based on Joe's latest comment; simplified and wrote unit tests. Re-opening for review.
Need to confirm that this works with a working sheet, but at least logic should be correct, and wanted to open up for review to double check


JSON decode error

As of 2.1.x I get this error.
The integration is not functional at all.
Seems like all data fetched isn't decoded.
Not sure if it's the request of response mimetype that's wrong, either way, should be an easy enough fix.
Logger: homeassistant.components.websocket_api.http.connection
Source: custom_components/olarm_sensors/olarm_api.py:497
Integration: Home Assistant WebSocket API (documentation, issues)
First occurred: 6:34:32 AM (1 occurrences)
Last logged: 6:34:32 AM
[547636758368] 0, message='Attempt to decode JSON with unexpected mimetype: text/plain; charset=utf-8', url=URL('https://apiv4.olarm.co/api/v4/devices')

Please refer to this issue. Update to Version 2.1.1. Then restart and enter your new api key under the Configure button. Your API key has been blocked by Olarm.

The error above was after upgrading to 2.1.1 and also I've reset / regenerated my API key more than once.
I suspect the only long term solution here is to use webhooks. Do you have any plans yet to implement a webhook solution, this will also make the service much more responsive I would imagine.


Maybe should have set cookie options "secure"  to false as default?

function shouldSaveSession(request, cookieOpts) {
if (cookieOpts.secure !== true) {
return true;
} ...
}
The default options "secure" for cookie makes it's impossible to save encrypted cookies, which makes it's impossible to keep user as logged-in status.
Awesome plugin BTW, thanks 😆

Sorry I didn't realize this issue was duplicated


Scripts for negative data

These are the scripts for the negative data.

@gnsrivastava
I have several suggestions:

Could you please put every thing under /docs/data_curation/  ?
Could you please remove the numbers before the python scripts? For example, 2.renumber.py can be renamed as renumber.py.
Could you please remove the unnecessary comments in the python scripts?


Removed the option to hide so-called "low-coverage" species

The information is not provided any more on recent Vertebrates assemblies. I think it's (almost) never been set in EG either.
:warning: I haven't tested this on my sandbox !
Then you can also remove the line in public-plugins/ensembl/conf/ini-files/DEFAULTS.ini that has "LOWCOVERAGE"

Thanks for the pull request @muffato. I have merged it now.


Add WindowSettings::draw_size

See https://www.reddit.com/r/rust_gamedev/comments/3ejcy2/using_piston_how_can_i_configure_a_set_game/
Currently the window back-end decides the resolution of the frame buffer.

One problem is that PistonWindow automatically resizes the frame buffer.
Probably need some way to tell whether one should resize. Could be AdvancedWindow::should_resize_frame_buffer.

The Iterator is implemented for PistonWindow with the Window constraint, so it needs to be Window::should_resize_draw_buffer.

Doesn't seem like Glutin support this.

rust-sdl2 doesn't support this either, it seems.

It seems that window mode automatically sets the frame buffer. The only way to fix the resolution is to open the window in full screen.

It remains to figure out whether PistonWindow should resize the buffer or not.

I believe this is a viewport/transformation issue. It might be possible to do this without needing a setting.

I haven't managed to find a good solution for this. Reopen if you have better ideas.


BK new async actions

Bet, new server seed and client seed change actions are now async
Tested with a local install of BK.

When will these changes be live?


Search results - bold the entity names in BEL statement

Ref name
authors
comments

Closing for now as I've no idea what I meant here.


[Question] Using Picture In Picture not for video playback

Thank you for the awesome library!
As far as I understand, Apple supports Picture in Picture in the following scenarios:

For video content playback
For video calls

Did anyone succeeded in using the picture-in-picture mode for anything except video playback for App Store apps?

Hi.
Several apps, including a memo app, can be found in the App Store.
I don't think they will be rejected unless there is a direct disadvantage to the user.
Also, of course, the PiP in this library may not be available in future iOS updates.

@uakihir0 Understood, thank you for the samples!
I have a rather unique idea (displaying the screen translation in a PIP window) and evaluating the probability if such a use will be accepted. We’ll see.


Contents links don't correctly link to article sections

Clicking the links at the top of the article don't navigate you to those sections of the article

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: e9e9c055-6a8e-5e81-00c7-e57c8c912a8b
Version Independent ID: 28ac7239-a6c2-e31e-d423-b5268fd95c65
Content: Xamarin Profiler - Xamarin
Content Source: docs/tools/profiler/index.md
Service: xamarin
GitHub Login: @topgenorth
Microsoft Alias: toopge

Thanks, @PureWeen. Since the contents is listed on the right-hand margin of the document, I went ahead and removed this contents section. The fix should be published soon. Appreciate the help!


Port over addressbook features to trivia

I have created a PR for the ported list and select command, please do take a look.

clear command will be removed.


Search Box doesn't work

I just did a pull and something in the update brakes the search box.  I did roll back the change on the live site so the work would work again.
The last working pull for the beta site was on March 6th

Zach,
Could you check now? It should be working fine with the new clean urls.

Git did the pull again and searching is not working in Firefox or Chome.  I am leaving it so you can see

Can you verify that
1> config file has the ''http://beta.empireadc.org' without the last '/'  as the base url and
2> there exists a .htaccess file in the root directory of eaditorsearch.

Zach,
In controller>eaditorsearch.php I forgot to remove line 38 code: echo $resultsLink; that will display the code for testing.
Thanks,
Monish

This issue has been resolved


Reworked ambilight layer and added a vertical flip option

Fixes #442
This pull request proposes the following changes:

Adds an option that flips the image vertically. This improves the illusion that th keyboard is reflecting the screen.
Changed the way the cropping is done. Before, the image was cropped differently for each screen capture mode. Now, the target screen region is calculated from the options and then applied to the layer. Should do the same thing but is a bit cleaner imo. Code review appreciated.
Adds the ability to display the ambilight image in a keyboard region instead of covering the whole canvas.
Reworks the 2 implementations of ambilight into an interface + 2 classes for better clarity.

Known issues/To do:

The access lost exception is still not fixed, not sure what causes this exactly,
Only one DX desktop duplicator layer should be active at any time. There needs to be a system trhat prevents then user from turning on the "experimental mode" if any other layer on the aplication has it turned on already.

Instead of adding the flip option just to the ambilight layer. I would suggest to add a mirror vertically and horizontally option to the layer options here:

In my opinion the Monitor selection should start at 1 instead of 0
Also Wibbles process picker should be used?


Get rid of illegal reflective access warning in ec2 admin

The ec2 admin raises an illegal reflective access message when it reports any other problem such as trying to delete something that does not exist or runs into a permission problem.
Terminating the selected instances in region 'eu-central-1' ... WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by com.amazonaws.util.XpathUtils (file:/home/anyone/projects/loadtest/xlt-5.3.0/lib/aws-java-sdk-core-1.11.762.jar) to method com.sun.org.apache.xpath.internal.XPathContext.getDTMManager()
WARNING: Please consider reporting this to the maintainers of com.amazonaws.util.XpathUtils
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release
Failed: No instances specified (Service: AmazonEC2; Status Code: 400; Error Code: InvalidParameterCombination; Request ID: 23234-3bd3-4157-b365-4bf7b4432221ec7)

The Illegal access warning is caused by 3rd-party library aws-sdk-java.
According to their issue tracker (https://github.com/aws/aws-sdk-java/issues/1603), it won't be fixed for v1 but should be fixed in v2 of the library. Until we update to v2, there isn't anything we can do here.

At least we can suppress these warnings by passing the appropriate command line options to java, the same way we did for mastercontroller.sh, agentcontroller.sh, and agent.sh.


Debug adapters should get the current open folder

I think there used to be a 'cwd' property automatically sent over set to the current directory. Why was it removed? I'd like to make my launch config simpler by globbing for files inside the user's open folder.

@roblourens Initially we had a 'cwd' attribute as part of the protocol for 'launch' and 'attach'. But later we decided to remove all 'standard' attributes from the the 'launch' and 'attach' requests because they are highly debugger specific. So every debug adapter can introduce a 'cwd' in their package.json.
I suggest that you introduce a mandatory attribute 'cwd' and set its default value to ${workspaceRoot}. Add this to your default launch config as well.

What do you mean 'highly debugger specific'? Seems like it would be applicable to any other client implementing the debug protocol. I'm trying to make the config simpler, but adding a mandatory property that should have the same value 100% of the time has the opposite effect.

@roblourens with 'highly debugger specific' I was referring to the whole set of attributes that we had initially. I agree, 'cwd' is probably one of the generic attributes.
My reluctance to introduce any 'side channel' attributes not visible to the user in his launch config, comes from my bad experience with any implicit, behind-the-scene magic: if I see a launch config, I see the whole picture. If you want to glob for files automatically, why not making this explicit?
I've seen many project folders with nested sub projects that had their own launch configs. Globbing at the root folder will probably not exclude the sub projects, which might be confusing. Having an explicit attribute where a user can specify from where to start the globbing seems to me a more transparent and flexible approach.
Do you consider to have an optional attribute where the user could override the default globbing behaviour?

I agree in principle but it doesn't seem like magic to me because the user explicitly opened that folder. I would definitely allow them to override it for subprojects. I understand if you don't want to add it, at least specifying 'cwd' would still be simpler than figuring out 'webRoot' which is the property I want to get rid of.

@roblourens I'm not sure that I understand 'cwd' any better than 'webRoot' in the context of a web project. But you can use/introduce whatever you want.
What I don't understand is why you think that 'figuring out webRoot' is difficult? If a user is asked to create a new launch config, she will pick the Chrome Debug type and that will create launch configs that have the webRoot attribute already set to ${projectRoot}. So she does not even have to review the launch config but can just press F5. And if she later wants to narrow down the web root, the attribute is already there and she does not have to find (and understand) it first.

In a really simple web project, you have some files that get built to root/out/script1.js and root/out/componentA/script2.js, referenced by your .html or whatever, and they're served as localhost/out/script1.js so webRoot is your project root, or a simple path from the root. In more complex configs, it's not uncommon for the url to not totally match the path on disk. Like your node_modules are served as one path, some scripts from another path, other scripts are bundled in memory and don't exist on disk. So I have a task to support a more complex webRoot config which can mirror the server config. But I can also make lots of cases "just work" by doing some fuzzy matching.
And then even in simple cases, I've found that some people are confused about what it should be set to, and I want to make it as simple as possible.

@weinand @isidorn - It would be nice to use https://github.com/Microsoft/vscode/issues/8645 to start the debug adapter with the workspaceRoot path via vscode.workspace.rootPath. This would let us eliminate cwd from the default launch config and simplify the launch config by a bit. Maybe we could go back to resolving a relative path for 'program' as well.
I probably wouldn't do this for webRoot like I said in June - like Andre said,

And if she later wants to narrow down the web root, the attribute is already there and she does not have to find (and understand) it first.

Changing the webRoot is somewhat common, and seeing the webRoot is necessary to understanding the behavior of chrome-debug. But I think that changing 'cwd' is more common, and people will already understand it if they need to change it.

If the extension contributions a startSessionCommand it can manipulate the launch.json as it sees fit, thus adding a vscode.workspace.rootPath would not be a problem.

@roblourens since we can manipulate the launch.json in the extension's startSessionCommand in any way we need, I think we can close this issue?


Script Install-VMInsights.ps1 is still using AzureRm module

Hi Team,
We already moved to Az module, so please help modify the script to Az module. Thanks.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d385c06a-f7e4-8875-835e-8983b2fd5f9c
Version Independent ID: 89f1c9ce-4f52-0a10-b464-e544a859b25d
Content: Enable Azure Monitor for VMs (classic) with PowerShell or templates - Azure Monitor
Content Source: articles/azure-monitor/insights/vminsights-enable-at-scale-powershell.md
Service: azure-monitor
GitHub Login: @bwren
Microsoft Alias: bwren

@MSNina123456 Thanks for your comment! We will review and provide an update as appropriate.

@MSNina123456 This issue is duplicate of #35575. Hope the update in #35575 helps!

@MSNina123456 Closing this issue for now. If there are further questions regarding this matter or further feedback on the documentation, please do let us know and we will gladly continue the discussion.

checked #35575: I've modified the statement and the change should be reflected later this morning. #please-close.
But I just checked the latest version of this script, it's still using AzureRm cmdlets for all!
and more #35575 is for special cmdlet issue, but this issue is for Az module update!

@MSNina123456 The comment was more about "While the script is using the old AzureRM module cmdlets, our guidance is to encourage customers to upgrade from that module to the new Az module and enable compatibility mode for these circumstances while leveraging the improvements of the new module with your custom scripts or scripts that the different product groups have updated using cmdlets from the new module. The note clearly states this at the beginning of the article" but content owner @MGoedtel has reworded the paragraph that was called out."
Tagging current content owner @bwren for visibility, review and to provide your insights on @MSNina123456's feedback and consider document enhancement.

#label:"CreateAzMonWIFeb2021"

Thank you for your contribution to the Azure Monitor documentation. We have created a workitem for this issue in our internal backlog database. We will prioritize it and make any necessary changes as we are able.
#please-close


handle bucket flush

Saw this hint from Aliaksey K (alk) from some related email thread...
"Any client ... can and should (as far as I'm aware) detect flush via vbucket uuid change. I.e. for well behaved DCP client it means it will work automatically. Because flush is same as data-loss-full failover followed by rebalance (i.e. when some or all vbuckets are lost during failover and has to be re-created during rebalance)."
Not sure if cbft client does that right, so this issue is a reminder to double-check.

flushing a bucket seems to bring cbft to a state where it closes the index, but cbft doesn't seem to reopen it.

Following up on Feb 13, 2015 comment, this is reconfirmed even in latest cbft/cbgt circa Dec 2015.
After a FLUSH, a cbft node can get into a looping state of...
2015/12/10 15:20:06 feed_dcp: OnError, name: bs_be8575242aa17215: bucketName: beer-sample, bucketUUID: , err: sendStreamReq, err: bleve: BleveDest already closed
2015/12/10 15:20:06 feed_dcp: OnError, name: bs_be8575242aa17215: bucketName: beer-sample, bucketUUID: , err: pkt.Receive, err: read tcp <IP_ADDRESS>:11210: use of closed network connection
2015/12/10 15:20:26 feed_dcp: OnError, name: bs_be8575242aa17215: bucketName: beer-sample, bucketUUID: , err: sendStreamReq, err: bleve: BleveDest already closed
2015/12/10 15:20:26 feed_dcp: OnError, name: bs_be8575242aa17215: bucketName: beer-sample, bucketUUID: , err: pkt.Receive, err: read tcp <IP_ADDRESS>:11210: use of closed network connection
2015/12/10 15:20:46 feed_dcp: OnError, name: bs_be8575242aa17215: bucketName: beer-sample, bucketUUID: , err: sendStreamReq, err: bleve: BleveDest already closed
2015/12/10 15:20:46 feed_dcp: OnError, name: bs_be8575242aa17215: bucketName: beer-sample, bucketUUID: , err: pkt.Receive, err: read tcp <IP_ADDRESS>:11210: use of closed network connection
2015/12/10 15:21:06 feed_dcp: OnError, name: bs_be8575242aa17215: bucketName: beer-sample, bucketUUID: , err: sendStreamReq, err: bleve: BleveDest already closed
2015/12/10 15:21:06 feed_dcp: OnError, name: bs_be8575242aa17215: bucketName: beer-sample, bucketUUID: , err: pkt.Receive, err: read tcp <IP_ADDRESS>:11210: use of closed network connection
2015/12/10 15:21:26 feed_dcp: OnError, name: bs_be8575242aa17215: bucketName: beer-sample, bucketUUID: , err: sendStreamReq, err: bleve: BleveDest already closed
2015/12/10 15:21:26 feed_dcp: OnError, name: bs_be8575242aa17215: bucketName: beer-sample, bucketUUID: , err: pkt.Receive, err: read tcp <IP_ADDRESS>:11210: use of closed network connection
2015/12/10 15:21:46 feed_dcp: OnError, name: bs_be8575242aa17215: bucketName: beer-sample, bucketUUID: , err: sendStreamReq, err: bleve: BleveDest already closed
The latest theory is that the feed_dcp (and it's datasource) is just holding onto its old receiver; or the feed_dcp instance isn't getting stopped / shutdown and restarted with fresh, brand new pindex instance, but perhaps is just holding onto the old (unfortunately, already closed) pindex.

Proposed fix in cbdatasource here: http://review.couchbase.org/57740


KAFKA-16780: Document the new RLMM#nextSegmentWithTxnIndex API in upgrade notes

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Could you share the instructions to preview the upgrade notes?

Could you share the instructions to preview the upgrade notes? Let me know if the note need to be improved.

You can use the instructions here https://cwiki.apache.org/confluence/display/KAFKA/Setup+Kafka+Website+on+Local+Apache+Server to setup website locally and verify your change.

Verified the change, by following the README in the kafka-site:


prevent heilbronn crash

Fixes a complete crash at heilbronn if GeoJSON and scraped information have different datasets. Adds error messages to the result, too, in order to use / log them later.

LGTM


Reduce HAAQI times

Running HAAQI in full-length song is very slow.
This affects both tasks  of this first round of Cadenza

Hi @ns-rse, @jonbarker68
I've been trying to speed up the HAAQI code.
It is pretty slow because they have several (around 8) full autocorrelations of very long signals (about a 3-min song).
In the code, we are using scipy which is not supported by numba but suppose to be faster than np.correlate for long signals.
But, I did some profiling using both and couln't find a significant difference.
We can't just add @jit(nopython=True) on top of this function because Numba only supports the default of numpy.correlate, i.e., correlation with mode = 'valid' and not 'full'
https://numpy.org/doc/stable/reference/generated/numpy.correlate.html
https://github.com/numba/numba/blob/07f824e49e5e2f158b79e7ba787674272baf1da0/numba/np/arraymath.py#L4199

I tried to overwrite the numba's numpy.correlate overload but I wasn't able to do it successfully.
Have you ever tried something like that?
Is it worth trying to define our own overload to try to accelerate the code?
https://numba.pydata.org/numba-doc/dev/extending/overloading-guide.html?highlight=overload

Thank you for your help.

@ns-rse, @jonbarker68
After the release of pyclarity 0.3.0 I want to start working on improving HAAQI.
There are many changes we can do to make it more pythonic and improve its readability and performance.
For example:

We can cache most of the functions. This would help in all cases when the references need to be processed more than one time. And/or,
We can refactor the code to an abstract class. Then haaqi, haspi and hasqi can implement or overwrite the methods. Some functions in haspi.eb are common like ear_model that should be a method under the abstract class, and others functions are metric specific like eb.melcor9 which is only used by haaqi and only need to be a method in haaqi class.
We need to add type hints.

What are your thoughts? What else would you do to improve the metrics?

These all seem like eminently sensible ideas @groadabike
Obviously vectorising loops are going to offer good speed ups and restructuring into abstract classes are really useful.
Has your profiling covered more than just numpy vs scipy for correlation?
If not it might be worth using profiling against the code base to determine which areas might benefit most from careful refactoring.

The profiling I am running evaluates the time of each function when running haaqi.
35% - 40% of the time is in function eb.bm_covary which has 6 correlations in it and nested for loops. (eb.bm_covary)(https://github.com/claritychallenge/clarity/blob/16e0472ac7f04f19066c85ef396b580262d11f40/clarity/evaluator/haspi/eb.py#L1748).

@ns-rse , @jonbarker68
In a quick review, I can see 5 possible classes
              HA_METRICS
                   |
               __ _|_______
              |           |
      HA_QUALITY        HASPI
HAAQI         HASQI

we could have a HA_PERCEPTUAL as the father of haspi but I can't see a benefit of doing that.
I have a class ha_quality as parent for haaqi and haspi in order to have the common methods in an abstract class.
obviously, we could have the common methods as part of the parent ha_metrics
@ns-rse, how would you organise these classes?
Any advise would be great.
Thank you
Below, I have 2 tables,

The functions each metric is calling
The constructor parameters

These have to be reimplemented in a way that doesn't affect the previous use.
Table one
Functions each metric is using

Haaqi
Hasqi
Haspi

eb.ear_model
eb.ear_model
eb.ear_model

eb.env_smooth
eb.env_smooth

eb.melcor9

eb.spectrum_diff
eb.spectrum_diff

eb.bm_covary
eb.bm_covary

eb.ave_covary2
eb.ave_covary2

eb.mel_cepstrum_correlation

ebm.env_filter

ebm.cepstral_correlation_coef

ebm.fir_modulation_filter

ebm.modulation_cross_correlation

ip.get_neural_net

ip.nn_feed_forward_ensemble

Table Two
Constructor Parameters

reference: np.ndarray,
reference,
reference: np.ndarray,

reference_freq: int,
reference_freq,
reference_freq: int,

processed: np.ndarray,
processed,
processed: np.ndarray,

processed_freq: int,
processed_freq,
processed_freq: int,

hearing_loss: np.ndarray,
hearing_loss,
hearing_loss,

equalisation: int,
equalisation=1,

level1: int = 65,
level1=65,
level1: int = 65,

silence_threshold: float = 2.5,
silence_threshold=2.5,

add_noise: float = 0.0,
add_noise=0.0,

segment_covariance: int = 16,
segment_covariance=16,

f_lp: int = 320,

itype: int = 0,

Hi @groadabike ,
I've not masses of experience with design patterns (I've no formal training in computer science so pick things up as I go along) and usually refer to Refactoring Guru when encountering them/needing to implement them.
It sounds like you're proposing using Abstract Factory and so ha_metrics would decide which constuctor to return and in turn, ha_quality would be deciding which quality metric (haaqi or hasqi) constructor to return.
Another possible design pattern to use is Builder which allows construction of complex interfaces but they don't always have to the same construction processes, so maybe that would be preferable here given there are some differences in what each is required.
Not sure if any of that helps at all I'm afraid.

For reference, there is also a python HASPI translation here
https://github.com/nii-yamagishilab/NELE-GAN/tree/master/pyHASPI

HAAQI refactoring was done in #364
Closing this issue


Support rightButton width calculation based on image if it exists

Slightly related to #54, when using an image instead of text for the right button, it skews the image because the width is based on the title text. Would you be open to a patch that checks for the presence of an image, and uses the image width instead of the text?

Can you please merge that to the master branch? I would really want that ability to be presented in your library:)


Transition the code to Rust 2018

Rust 2018 is about to be released and it's already the default for new crates on nightly. So we should update this crate and the blog posts to the 2018 edition of Rust.
After transitioning:

[ ] Use #![cfg_attr(not(test), no_std)] instead of #[cfg(test)] extern crate std as proposed here.

With Rust 2015, you can use #![cfg_attr(not(test), no_std)] just by adding
#[cfg(test)]
extern crate core;

I took my copy of the source, removed the 2018 edition, and added that one snippet (to lib.rs and to bin/test-exception-double-fault-stack-overflow.rs) and it compiled.

Yeah, but this wouldn't really be an improvement over the current situation (#[cfg(test)] extern crate std and unconditional #![no_std]).


Flake8 E741 ignore flag removal

This PR removes E741 ignore flag and makes the required adjustments.

This addresses half of the #267 .


BugFix - Final event never called depending on maxParallel size

This is SUPER critical. Can you please help me resolve this?
Take a look at the tests and you can gather what's going on...
Basically, the high water mark being set to the max concurrency seems to cause problems and if you hit the high water mark, the stream is paused and never resumed. (Or something I'm not quite clear).
The issue was reported on the parallel-transform Repo back in 2015.
https://github.com/mafintosh/parallel-transform/issues/1

I think, somehow, we need to detect if we've hit the high water mark and then listen for the even when you fall below that and then resome.

It appears to be fixed.
As you can see in the code. Now I'm reading the ready state returns by this.push and setting up listeners that will resume and drain when the buffer opens up.

Beautiful! So we have to stop reading when it's full and resume on the next readable event. I made a trivial comment, waiting for your reply.

@tbreeding Thanks. I published 0.2.0 which includes your change 🎉

Thanks you for such a quick response.


Umbraco 9

Do you have any plans to upgrade this package for Umbraco 9?

Hi!
Definitely I'm not sure when I can find the time but for sure during the next couple of weeks.
If you have an urgent need for it I would be happy to accept a PR. If you are up for it, let me know before you start and we can figure out the best way to do it. Otherwise I'll keep you posted here.
// m

No urgent need. I've been using SeoVisualizer on Umbraco 8 for a while now and I really like it. With the Umbraco 9 release coming up I was just wondering what your plans were for v9. :-)

Sweet! I'll make sure to update you here when the release is out!
Cheers!

Just pushed a new version to NuGet: https://www.nuget.org/packages/UmbracoSeoVisualizer/9.0.0-beta1
@creativesuspects Could you give it a try and see if it works for you as well?
Cheers!

Awesome! I'll be starting a new v9 project next week (currently working on the front-end), so I'll be able to test the beta version then.

Great! Keep me posted =D

It's maybe me but when I use the beta package, it doesn't copy automatically the content from package to App_Plugins.
If I copy it manually everything seems working great.
I'll try with a fresh install of Umbraco 9 and keep you posted.

Hi!
I’ll see if I can reproduce this, it used to work “on my machine” :)

I think it's just because the target file is not alligned with the packageId and then the clean & copy actions does not run.
I've made a simple change, and now it

too 😉


379 updated meta description

Fixes #379
Creator

[ ] Build relevant tests if any apply
[ ] Ensure there are no new warnings
[ ] Upload an animated GIF showcasing the solution (if it applies)
[ ] Set a relevant reviewer

Reviewer

[x] Run test and make sure all tests pass
[x] Review function and form on iOS
[x] Review function and form on Android
[x] Review function and form on Web
[x] Review new test logic

@cdanwards FYI - I forgot to run yarn test for this change. #equality


Updates client server image

inspired by gdisf/teaching-materials issue #45

@jkarnowski There is an unclosed quote on the alt text. Can you fix that?
alt="Diagram showing computer connecting to server height="510"

Sorry @jkarnowski, managed to accidentally close this one. Too many tabs!


%loading every reload on waveform

Hello, great plugin ! Is there the possibility to save the waveform to not reload it every time ? Thanks :)

This plugin only provides two methods. Can you please review the documentation of those two methods? It shouldn't take long.

If you need help finding the API documentation, I can't be the person to help you with that, but maybe someone on stack overflow can help.


Develop

Coverage decreased (-16.9%) to 69.905% when pulling d28364a0f2fea891f60ef0ba6e648259aeffdbb0 on develop into d436a2c5307d0907c5842b7ec7bbf1f1bb6736c9 on master.

Coverage increased (+1.8%) to 88.658% when pulling 2c530ee0f49d1229a36910851cf117b1e1190bef on develop into d436a2c5307d0907c5842b7ec7bbf1f1bb6736c9 on master.


offer to add **/.venv to exclude if user has explicit exclude but .venv is not part of it and pylance think it is part of user workspace

This issue is related to https://github.com/microsoft/pylance-release/issues/4762#issuecomment-1696151733
when user has explicit exclude and not include .venv in that, we can't just auto exclude since if users really wanted to do it (for whatever reason), there is no way to opt out from it (or we need to provide yet another option to opt out again - since explicit exclude is a way to opt out from auto exclude)
instead, we can offer suggestion saying we see .venv part of workspace and see user has explicit exclude, whether user wants us to put .venv in their explicit exclude?

I am guessing this is why I am getting:

"/Users/anentropic/dev/myproject/.venv/lib/python3.11/site-packages/typing_extensions.py" is overriding the stdlib module "collections.abc" Pylance[reportShadowedImports]
(https://github.com/microsoft/pyright/blob/main/docs/configuration.md#reportShadowedImports)

it happens intermittently... if I restart VS Code it goes away for a while but always comes back after using it for a while (which is a bit odd in itself)
I have this in my pyproject.toml:
[tool.pyright]
venvPath = "."
venv = ".venv"
exclude = [
    "website/**/migrations/*.py",
    "cdk.out",
]
reportIncompatibleVariableOverride = "warning"

I would not have guessed to add .venv path to exclude as I would still want those packages to contribute to type checking
if this is generally desirable then I wonder if it's possible to do something with the venv = ".venv" already specified
maybe an exclude_venv boolean flag that defaults true, but you can exclude_venv=false if you really wanted the exclude to replace the useful default


Added a Navbar

Added navbar

The navbar has a bunch of baggage in the way of changes to other files that are not needed for this pull request. I will work with Lissa to create a new pull request that loses all this excess.
Travis

See previous comment


raise ExtractorError('No video formats found') youtube_dl.utils.ExtractorError: No video formats found

Checklist

[ ] I'm reporting a broken site support issue
[ ] I've verified that I'm running youtube-dl version 2021.06.06
[ ] I've checked that all provided URLs are alive and playable in a browser
[ ] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ ] I've searched the bugtracker for similar bug reports including closed ones
[ ] I've read bugs section in FAQ

Verbose log

PASTE VERBOSE LOG HERE

Description

WRITE DESCRIPTION HERE

For actual help, check 6 things.
Then
PASTE VERBOSE LOG there
and
WRITE DESCRIPTION there.

Problem solve


Can you support building containerd in a virtual machine

I want to run a k8s amd64 cluster because my host is an arm64 architecture

Duplicate of #602 and #465.

seccomp is enabled, but not supported in emulated x86 machines. You can subscribe to #465 for updates, but it's probably not feasible.
You should run amd64 images in an arm64 machine/container engine instead, or use cri-dockerd. I haven't tested it but OrbStack's Kubernetes cluster should also be able to run amd64 images, although the arch labels won't be correct.


Glassdale minor typos

GLASSDALE_DELETE_NOTES.md: fixed example code at the end of the event listener
GLASSDALE_CRIMINAL_FACILITIES.md: fixed typo in fetch request in the getFacilities function

:shipit:


Scaling and Window Resizing

The window can become huge but the game says the same size (see image bellow)

Maybe center the objects?

2. Can't resize the window

You can use the maximize and minimize
You can't use the window edges to drag the border (could by the system I am running it on?)

Not being able to resize the window was intentional, I wanted a fixed cell size. But I will disable the maximize button. I might consider adding an option to resize the cells in the future though.

how about center alignment?

Maybe. I'll think on that. Leaving open for now.

:bomb: Yay

Thought on it, I don't think it is really that big of a feature.

Well that is disappointing :cry:


Update saving data use case

AC:

Update statement 1 of saving data use case to be less implementation-focused.
Statement 1: User enters a valid command that alters data (client add, client delete, client note add, etc).
Add numbering for use cases.

While updating this, could you add in numbering for the use cases as well? Then this statement can refer to other use cases as an example.

Okay, I have updated the issue description.


[Import] Product Spec

Product specifications should be imported/uploaded as a sheet and are references by product pages & category pages.

Imported into https://adobe.sharepoint.com/sites/HelixProjects/Shared Documents/Forms/AllItems.aspx?ga=1&id=%2Fsites%2FHelixProjects%2FShared Documents%2Fsites%2Fmoleculardevices%2Fproducts%2Fspecifications&viewid=124ad5f1-60ba-476c-ade8-7c1df274ee95


ACS Chat - Data retention policy beta

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

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


Feature request: customizable generation folder

Correct me if I'm wrong, but I don't see an option to generate the monosize results in any other location than dist/bundle-size/*.
We use Rush for our monorepo which for its caching layer needs a clear 1-1 mapping of commands to output locations. Since /dist is already being generated by the "build" cmd, we need a different location for the output of monosize.
We can workaround it with a quick script to copy from A to B, but being able to specify a custom target folder would simplify our integration. Thanks!

@benkeen yes, this indeed is not supported. Would be nice to have.

@layershifter let me know what you think of this PR please?
https://github.com/microsoft/monosize/pull/66

FYI fixed in #66, I triggered a release, so it will be on NPM in ~10 minutes.


tweak merging errors to propagate correct error in non-null situations

When we merge errors - instead of blindly merging at the given path in the error - traverse the path and check the data at each segment. If a given segment is null in the data, merge at that current path instead of potentially deeper.
This yields propagation of errors to the "outer" graphql execution regardless of the order that fields are encountered by that "outer" graphql execution

closing for now - going to combine this with removing proxy resolvers


Fix brew_expand_alias for non-standard packages

For example:
brew info brew-cask 2>/dev/null | head -1 | awk '{gsub(/:/, ""); print $1}'
caskroom/cask/brew-cask
brew info brew-cask 2>/dev/null | head -1 | awk '{gsub(/.*\//, ""); gsub(/:/, ""); print $1}'
brew-cask

brew list -1 only has the shorter name listed so it will try to re-install packages that are already installed

Thanks!

I think this is obviated by https://github.com/thoughtbot/laptop/pull/403 or https://github.com/thoughtbot/laptop/pull/451 ?


user's "site" doesn't require protocol

If I set my site (edit:site) to twitter.com/frozenpandaman, clicking on the URL in my sidebar will simply append the text to my dat:// URL, i.e. dat://1327c...69838/twitter.com/frozenpandaman, giving a 404.
It seems if a protocol, e.g. https://, is given, it hides it in the HTML but keeps it in the rendered a href tag, i.e. just shows twitter.com/frozenpandaman but the link still works correctly. (I like this!)
The site field should support dat:// URLs too, of course, but when setting/editing it, it should probably default to http:// if no protocol is given.

That should do it, the idea of default to http makes sense and that's what it does now.

looks good @underscorediscovery. :) thanks!


Adding t_start and t_unit

This PR does what #215 lists. Important to note is that the new tutorial sets some design considerations that will be useful for implementing the methods needed to deal with time in the model.

Largely agreeing with @dylanhmorris's comments. Also wanted to say that I would be open to shorter names, since these will be referenced so often. Some ideas: time_start/unit, ts_start/unit, or t_start/unit.

Largely agreeing with @dylanhmorris's comments. Also wanted to say that I would be open to shorter names, since these will be referenced so often. Some ideas: time_start/unit, ts_start/unit, or t_start/unit.

Agree, at some point I was using ts_start/ts_unit. Will return there!

I think the s is superfluous; I vote for t_start/unit

Ready for last pass, @damonbayer @dylanhmorris

One non-critical issue with how the rendered docs look. @gvegayon, your call whether to fix here or merge and make a separate issue to fix.

Fixed the code in the PR and added an issue referencing this discussion. I think it would be a good thing to review the website and check how the docstrings are rendered.


Put Check Calibration Code into Base

This code should be released by adding it to the base code

What's the status on this? We would like to know what the quality of calibration will be and having your code to look at will help us.

@mike-matera In the past we have simply run the calibration checker separately. I imagine we will still do that, whether or not we have copied the checker to this repository.
If I'm not mistaken we have actually added it to the swarmie hard drive image we are distributing/using.


error when training using one GPU when multiple GPUs are available

PLEASE FOLLOW THESE INSTRUCTIONS BEFORE POSTING

Please search existing open and closed issues in case your issue has already been reported
Please try to debug the issue in case you can solve it on your own before posting

After following steps above and agreeing to provide the detailed information requested below, you may continue with posting your issue
(Delete this line and the text above it.)
Expected results
What did you expect to see?
Actual results
What did you observe instead?
Detailed steps to reproduce
E.g.:
The command that you ran

System information

Operating system: ?
CUDA version: ?
cuDNN version: ?
GPU models (for all devices if they are not all the same): ?
python version: ?
pytorch version: ?
Anything else that seems relevant: ?

I want to specify GPU id when training, the command is
CUDA_VISIBLE_DEVICES=2 python tools/train_net_step.py --dataset coco2017 --cfg configs/e2e_faster_rcnn_R-101-FPN_1x.yaml
In train_net_step.py, I comment the line
cfg.NUM_GPUS = torch.cuda.device_count()
to enable the use of "NUM_GPUS: 1" in the yaml file
And I got the following error:
INFO train_net_step.py: 361: Training starts !
INFO net.py:  72: Changing learning rate 0.000000 -> 0.006667
Exception ignored in: <bound method _DataLoaderIter.del of <torch.utils.data.dataloader._DataLoaderIter object at 0x7f229d730ba8>>
Traceback (most recent call last):
File "path/miniconda3/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 349, in del
self._shutdown_workers()
File "path/miniconda3/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 328, in _shutdown_workers
self.worker_result_queue.get()
File "path/miniconda3/lib/python3.6/multiprocessing/queues.py", line 337, in get
return _ForkingPickler.loads(res)
File "path/miniconda3/lib/python3.6/site-packages/torch/multiprocessing/reductions.py", line 70, in rebuild_storage_fd
fd = df.detach()
File "path/miniconda3/lib/python3.6/multiprocessing/resource_sharer.py", line 58, in detach
return reduction.recv_handle(conn)
File "path/miniconda3/lib/python3.6/multiprocessing/reduction.py", line 182, in recv_handle
return recvfds(s, 1)[0]
File "path/miniconda3/lib/python3.6/multiprocessing/reduction.py", line 153, in recvfds
msg, ancdata, flags, addr = sock.recvmsg(1, socket.CMSG_LEN(bytes_size))
ConnectionResetError: [Errno 104] Connection reset by peer
INFO train_net_step.py: 425: Save ckpt on exception ...
INFO train_net_step.py: 135: save model: Outputs/e2e_faster_rcnn_R-101-FPN_1x/May13-21-57-31_tnt14_step/ckpt/model_step0.pth
INFO train_net_step.py: 427: Save ckpt done.
Traceback (most recent call last):
File "tools/train_net_step.py", line 407, in main
net_outputs = maskRCNN(**input_data)
File "path/miniconda3/lib/python3.6/site-packages/torch/nn/modules/module.py", line 491, in call
result = self.forward(*input, **kwargs)
File "path/CODE/Detectron.pytorch/lib/nn/parallel/data_parallel.py", line 114, in forward
return self.gather(outputs, self.output_device)
File "path/CODE/Detectron.pytorch/lib/nn/parallel/data_parallel.py", line 142, in gather
return gather(outputs, output_device, dim=self.dim)
File "path/CODE/Detectron.pytorch/lib/nn/parallel/scatter_gather.py", line 96, in gather
return gather_map(outputs)
File "path/CODE/Detectron.pytorch/lib/nn/parallel/scatter_gather.py", line 71, in gather_map
return {key: gather_map([d[key] for d in outputs]) for key in out}
File "path/CODE/Detectron.pytorch/lib/nn/parallel/scatter_gather.py", line 71, in
return {key: gather_map([d[key] for d in outputs]) for key in out}
File "path/CODE/Detectron.pytorch/lib/nn/parallel/scatter_gather.py", line 71, in gather_map
return {key: gather_map([d[key] for d in outputs]) for key in out}
File "path/CODE/Detectron.pytorch/lib/nn/parallel/scatter_gather.py", line 71, in
return {key: gather_map([d[key] for d in outputs]) for key in out}
File "path/CODE/Detectron.pytorch/lib/nn/parallel/scatter_gather.py", line 65, in gather_map
return Gather.apply(target_device, dim, *outputs)
File "path/CODE/Detectron.pytorch/lib/nn/parallel/_functions.py", line 54, in forward
ctx.input_sizes = tuple(map(lambda i: i.size(ctx.dim), inputs))
File "path/CODE/Detectron.pytorch/lib/nn/parallel/_functions.py", line 54, in
ctx.input_sizes = tuple(map(lambda i: i.size(ctx.dim), inputs))
RuntimeError: dimension specified as 0 but tensor has no dimensions

The problem is caused by the new bug in gather(outputs, target_device, dim=0) in nn.parallel.gather in pytorch 0.4.  [https://github.com/pytorch/pytorch/issues/6983]
There are two ways to solve it.

Wait for the bug fix in pytorch 0.4
unsqueeze the scalar loss to be of 1 dimension, like adding the following in model_builder.py

for loss_name, loss in return_dict['losses'].items():
    return_dict['losses'][loss_name] = loss.unsqueeze(0)
return_dict['metrics']['accuracy_cls'] = return_dict['metrics']['accuracy_cls'].unsqueeze(0)


Window with native dialog breaks when quitting

Mac

Cause native dialog to be shown (eg start "attach" debug config when it will fail)
Switch to another vscode window
cmd+q
All windows close except the one with the dialog open

Open extension viewlet, see that it can't load extensions or search. Also can't quit from the menu.

Not a regression, but a good one we should fix, here is what happens:

you hit Quit
Electron starts to close all opened windows except the one with a modal dialog
the shared process is a window and closes
=> nothing works anymore with regards to extensions management and telemetry

The shared process should override the close event and prevent it (see here how we do it for Code windows). It should only close upon explicit disposal from here.

Great catch! Thanks @bpasero for the pointers!


Added options={"expose"=true}

I added expose options for possible use with FOSJsRoutingBundle

Hi @kormanyuri. As route parameters like this are a matter of personal preference I will not add these to the library. So please try to set the parameter in another way. You can try overriding the route and parameter via YAML or set the parameter via a listener.
Maybe this post helps you in the right direction.
http://stackoverflow.com/questions/22510569/override-router-and-add-parameter-to-specific-routes-before-path-url-used
Good luck!

Please also note that the bundle is moved to inside the library and this repository is abandoned. Instead use the bundle provided here https://github.com/endroid/QrCode/tree/master/src/Bundle. Thank you.


AWS S3 SDK does not work on Unity 2019.1 Android

AWS S3 SDK does not work on Unity 2019.1
Expected Behavior
AWS S3 SDK works on Unity 2019.1
Current Behavior
After update to Unity 2019.1 AWS s3 sdk does not work. After call UnityInitializer.AttachToGameObject(this.gameObject); I see in logs exception:
04-17 22:51:29.910 4895-4917/? E/Unity: ArgumentException: Object of type 'System.Object[]' cannot be converted to type 'UnityEngine.AndroidJavaObject[]'.
                                          at System.RuntimeType.CheckValue (System.Object value, System.Reflection.Binder binder, System.Globalization.CultureInfo culture, System.Reflection.BindingFlags invokeAttr) [0x00000] in <00000000000000000000000000000000>:0
                                          at System.Reflection.MonoMethod.ConvertValues (System.Reflection.Binder binder, System.Object[] args, System.Reflection.ParameterInfo[] pinfo, System.Globalization.CultureInfo culture, System.Reflection.BindingFlags invokeAttr) [0x00000] in <00000000000000000000000000000000>:0
                                          at System.Reflection.MonoMethod.Invoke (System.Object obj, System.Reflection.BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00000] in <00000000000000000000000000000000>:0
                                          at System.Reflection.MethodBase.Invoke (System.Object obj, System.Object[] parameters) [0x00000] in <00000000000000000000000000000000>:0
                                          a

Your Environment

AWSSDK.Core version used: AWSSDK.S<IP_ADDRESS>.4
Operating System and version: Android
Targeted .NET platform: 4.x

How are you targetting .NET 4.X with unity?  My understanding is that Unity is only compatible with netstandard2.0 and PCL.  Could you take a look at the dll that your application is loading and identify which platform Unity is resolving?

Regarding .NET 4.x - https://docs.microsoft.com/en-us/visualstudio/cross-platform/unity-scripting-upgrade?view=vs-2019

Any platform

With Unity 2018.3.1f1 it works fine. But after updating of Unity to 2019.1.0f2 it stopped working

Are you using NuGet package to resolve your dependencies?  If so, could you take a peek at the DLL(such as dotpeek) and make sure it's either BCL45 or netstandard2.0 variant?   I am guessing it's resolving to PCL dll.

No, I don't. Just downloaded and import to Unity from https://github.com/aws/aws-sdk-net/blob/master/Unity.README.md -> http://sdk-for-net.amazonwebservices.com/latest/aws-sdk-unity.zip

Unity 2018.1 and going forward should support netstandard2.0 out of the box.  Instead of using the unity SDK build, could you try taking dependency on  NuGet packages?  Are there APIs in the unity flavor of the SDK that's not in vanilla AWS SDK you need?

I added NuGet for Unity (https://github.com/GlitchEnzo/NuGetForUnity), installed AWSSDK.Core.<IP_ADDRESS> and AWSSDK.S<IP_ADDRESS>.5 (latest from NuGet) and adopted code.
When called s3Client = new AmazonS3Client(accesKey, secretKey, RegionEndpoint.EUWest1);  throws  04-19 18:16:02.375 11494-11514/? E/Unity: NotSupportedException: System.Configuration.ConfigurationManager::get_AppSettings                                             at System.Configuration.ConfigurationManager.get_AppSettings () [0x00000] in <00000000000000000000000000000000>:0                                              at Amazon.AWSConfigs.GetConfig (System.String name) [0x00000] in <00000000000000000000000000000000>:0                                              at Amazon.AWSConfigs..cctor () [0x00000] in <00000000000000000000000000000000>:0                                              at Amazon.Runtime.ClientConfig..ctor () [0x00000] in <00000000000000000000000000000000>:0                                              at Amazon.S3.AmazonS3Config..ctor () [0x00000] in <00000000000000000000000000000000>:0                                              at Amazon.S3.AmazonS3Client..ctor (Amazon.Runtime.AWSCredentials credentials, Amazon.RegionEndpoint region) [0x00000] in <00000000000000000000000000000000>:0                                              at RemoteContent.InitS3 () [0x00000] in <00000000000000000000000000000000>:0                                              at Game.InitRemoteContent () [0x00000] in <00000000000000000000000000000000>:0                                              at Game+<Init>d__40.MoveNext () [0x00000] in <00000000000000000000000000000000>:0                                              at UnityEngine.SetupCoroutine.InvokeMoveNe
I think not just the unity package created. I see specific AWSConfigs.unity.cs configuration for Unity (https://github.com/aws/aws-sdk-net/blob/master/sdk/src/Core/_unity/AWSConfigs.unity.cs)

I tried to add System.Configuration.ConfigurationManager from NuGet but exception was the same

It looks like this is an issue of how NuGet incorrectly resolves DLLs.  Could you try one more thing?  Could you try unpacking the NuGet package(it's just a zip file) and grab the dlls under netstanard2.0 ?  Reference those DLLs directly?

Packages are already unpacked.
AWSSDK.Core.<IP_ADDRESS>/lib/net45/AWSSDK.Core
AWSSDK.S<IP_ADDRESS>.5/lib/net45/AWSSDK.S3

There are not any nestandard2.0 folders

I found AWSSDK.Core.<IP_ADDRESS>.nupkg, renamed it to AWSSDK.Core.<IP_ADDRESS>.zip and unpacked. There are a lot of folders in lib folders and nestandard2.0 folder also exists

I removed net45 and paste nestandard2.0 and it works fine.
@sstevenkang Thanks you very much.
But I think you should simplify installing sdk for Unity users.

Your feedback is totally warranted. Thanks for doing some investigating with us.  We plan on sharing news regarding how we are going to better support Unity going forward.

@sstevenkang The solution here is a little unclear to me.  Just so I'm following correctly, the proposed solution is to depend on the nuget package instead of the unitypackage, and then open said package as a zip file, and remove a directory in lib called net45. Is this correct?
Also, I would be interested in learning how to achieve the same goal using your GitHub repo, as a solution that is a little easier to maintain long-term.

I'm also facing the same issues and wondering over the solution. @Troshchuk Maybe you can give us a a little more detailed steps for solving the issue?
@sstevenkang And I am also interested achieve the same goal using your GitHub repo.

Unfortunately nuget doesn't allow to consistently choose a specifc target framework (in this case nestandard2.0) out of packages that contains multiple versions of the libraries.
The workaround we currently suggest is to go to the nuget page for the SDK packages (https://www.nuget.org/packages/AWSSDK.S3, https://www.nuget.org/packages/AWSSDK.Core), download the packages and open them as .zip files. You can extract the content of the lib\netstandard2.0\ folder of the packages you plan to use (always including AWSSDK.Core) and then directly reference the extracted DLLs from your project. Because the SDK DLLs target NetStandard 2.0 you can use them across multiple platforms (https://docs.microsoft.com/en-us/dotnet/standard/net-standard).
We are currently investigating ways improve the experience for Unity users.

Sorry for a late response. I have just had the time to test the solution. By using this solution there is issues like
error CS0103: The name 'UnityInitializer' does not exist in the current context
error CS0117: 'AWSConfigs' does not contain a definition for 'HttpClient'
error CS0117: 'AWSConfigs' does not contain a definition for 'HttpClientOption'
Also when will you be able to share some news for the Unity developers?

Hi guys,
I added NuGet for Unity (https://github.com/GlitchEnzo/NuGetForUnity) and installed:
AWSSDK.Core
AWSSDK.S3
In Packages folder I found 2 folders with AWSSDK.Core.<IP_ADDRESS>.nupkg and AWSSDK.S<IP_ADDRESS>.5.nupkg . I unpacked them and copied netstandard2.0 from here into lib folder. And removed net45 from lib folder. And everything works fine

@Troshchuk  thanks for the answer. But did you not get any errors regarding the AWSConfig? for me it seems like I get tons of errors.

@JakobJJ When I tried to use AWSSDK with net45 I had
NotSupportedException: System.Configuration.ConfigurationManager::get_AppSettings
but after replacemnt to netstandard2.0 everething is ok

@JakobJJ Make sure you reference the netstandard2.0 SDK assemblies and that you use the API reference at https://docs.aws.amazon.com/sdkfornet/v3/apidocs/Index.html. The nestandard2.0 assemblies don't contain the UnityInitializer class and the AWSConfigs class doesn't contain the HttpClient or HttpClientOptions properties. As you can see here, all of those are defined here in a unity-specific folder. If you are porting existing code using the Unity packages to the new netstandard2.0 assemblies, you will likely have to do some refactoring.
I am not able to comment on unannounced features but, if you have suggestions on what would make it easier to use the netstandard2.0 SDK in Unity, feel free to open an issue on this repo and tag me directly.
@Troshchuk @JakobJJ @asoule If you want a simpler way to get all the netstandard2.0 assemblies to use in your Unity project, you can use something like this PowerShell script. Obvioulsy you won't need all downloaded assemblies, just AWSSDK.Core.dll and those for the services you use (like AWSSDK.S3.dll for example).

Yes then i see.. Thank you for clearing this up. I am porting a existing unity project that were using the old aws unitypackage. And that is why  I'm getting all of these issues. I will open an issue and post some suggestions. Thank you @matteo-prosperi

Solved by exporting my project as a unity package and importing it into a new project made in Unity 2018.3.14f1...after that you only have to write your tags again and delete the following files: androidx.browser.browser-1.0.0
androidx.browser.browser-1.0.0.aar
androidx.core.core-1.0.0
androidx.core.core-1.0.0.aar
from \YoureProjectLocation\Assets\Plugins\Android

Yes then i see.. Thank you for clearing this up. I am porting a existing unity project that were using the old aws unitypackage. And that is why I'm getting all of these issues. I will open an issue and post some suggestions. Thank you @matteo-prosperi
@JakobJJ I am also trying to port my existing project,
were you able to do it?
if so can you please explain how?

Not able to resolve it. I have downloaded the Nuget package, and converted it into zip. Now, If I delete AWSSDK.Core.dll from Unity, and copy paste the AWSSDKCore.dll from netstandard2.0 folder, then UnityInitializer file is not found. When should we expect a fix from AWS?

@sahilgoyal123 This is expected. The AWS SDK for .NET is not API-compatible with the AWS Mobile SDK for Unity. You will have to rework some of your code in order to transition to the .NET Standard 2.0 libraries.
In case there are particular Unity-specific or mobile-specific features you are missing, please open an issue on this repo and we will consider the feature request for development.

I am having the same issue as the OP. I used the AWS Moble SDK for Unity.  Following the steps described above, I was able to make my code compile except the UnityInitializer. If I am using now the AWS SDK for .NET, do I still need to attach the Unity Initializer or can I make it work using the .NET methods?
Refactoring the code to make the Unity Initializer work seems to be a tedious task.

I am having the same issue as the OP. I used the AWS Moble SDK for Unity. Following the steps described above, I was able to make my code compile except the UnityInitializer. If I am using now the AWS SDK for .NET, do I still need to attach the Unity Initializer or can I make it work using the .NET methods?
Refactoring the code to make the Unity Initializer work seems to be a tedious task.

For me using only AWS Simple Email Service, I didn't need a Unity Initializer.
However steps in the Unity.README, with link.xml are still required.

Hello,
in case you decide to use the .NET Standard 2.0 version of the AWS SDK for .NET we have made available a download in .zip format that should be more convenient for Unity users (we also have version specific downloads using a URI like this one).
We have also continued our testing and, whenever using IL2CPP, we recommend to add a file named link.xml in your project assets and configure it so that IL2CPP doesn't strip any of the AWSSDK assemblies.
It should look like this:
<linker>
       <assembly fullname="AWSSDK.Core" preserve="all"/>
       <assembly fullname="AWSSDK.DynamoDBv2" preserve="all"/>
       <assembly fullname="AWSSDK.Lambda" preserve="all"/>
</linker>

Please make sure to include all the AWSSDK dlls that you are using.
The usage of link.xml is described here.

Solved by exporting my project as a unity package and importing it into a new project made in Unity 2018.3.14f1...after that you only have to write your tags again and delete the following files: androidx.browser.browser-1.0.0
androidx.browser.browser-1.0.0.aar
androidx.core.core-1.0.0
androidx.core.core-1.0.0.aar
from \YoureProjectLocation\Assets\Plugins\Android
got aws training and devops trainingby expert who has resolved and explained the above issue to me

@inventateq @matteo-prosperi
When I use the  Dll's that came with the UnitySDK Found here, and with the one you added the Android build fails.
Failed to Build:

/gradleOut/unityLibrary/src/main/java/com/amazonaws/unity/Utils.java:11: error: cannot find symbol

location: class AWSUnityGCMWrapper

Temp/gradleOut/unityLibrary/src/main/java/com/amazonaws/unity/AWSUnityGCMWrapper.java uses or overrides a deprecated API.

This is for AWS SNS for Unity.
When I replace the dll's of sns,AWSSDK.SecurityToken, and some others, that I have found here I get compile errors. They are the new netstandard2.0 dll's.

The name 'UnityInitializer' does not exist in the current context

Cannot convert lambda expression to type 'CancellationToken' because it is not a delegate type

Cannot convert lambda expression to type 'CancellationToken' because it is not a delegate type

For more information please see here
but if there is any updates on this please help. Thank you so much.

Cannot convert lambda expression to type 'CancellationToken' because it is not a delegate

It's been a while since I've used the AWS SDK in Unity, but I think you no longer need to use UnityInitializer anymore. Just follow the fundamentals here.
As for CancellationToken, I guess you need to specify the namespace System.Threading.

Cannot convert lambda expression to type 'CancellationToken' because it is not a delegate

It's been a while since I've used the AWS SDK in Unity, but I think you no longer need to use UnityInitializer anymore. Just follow the fundamentals here.
As for CancellationToken, I guess you need to specify the namespace System.Threading.

Thank you for the response, but I followed the instructions as seen and when I use the dll's given here it cant build apks.
And when I use the updated dll's I get the compiler errors.
Is there a newer sdk for Unity, if so how do I put it in?
See here for more info

Why was this issue closed when there are only hacky work arounds and no fix!

Using the provided .zip of netstandard2 .dlls did not prove any more fruitful than using the SDK as unity packages. I also spent a lot of time fiddling with downloading packages manually from nuget and extracting DLLs into my Unity project, only to find a ton of version discrepancies.
Ultimately, the only thing that worked was stealing the Assets/ dir .dlls from this kind hero's example repo: https://github.com/rms80/awscognito-unity-accounts-demo
The fact that that folder is 2 years old and I still need it to operate in Unity (and I am using Unity 2020) says some attention needs to be paid to the Unity SDK. Ideally, there would be a unity AWS SDK for each major version of Unity, and a "latest" option for people trying to push the boundaries. Being that I am using a beta version of Unity, I expected to encounter some issues, but this really was quite a difficult operation even for an experienced developer.

Any updates to this issue?

Still a lot of views about this issue.
You may watch this video to solve that step by step:
https://youtu.be/uRNUJzQzNTk

Yes this is very difficult to set up for a noob like me. There were no clues that Unity Mobile SDK was deprecated until I happened on some forum posts looking for help. An up-to-date unity-centric tutorial provided by AWS would be really appreciated.

Amazon, can you either formally state that you no longer support Unity with the latest AWS SDK for .Net v3+ or provide a solution for the missing UnityInitializer? Or at least explicitly declare whether UnityInitializer is required anymore or not?
Hello?

I agree with @karunateam! Please provide a solution for missing UnityInitializer....

To know more about AWS visit our site - aws with bigdata training

Solved by exporting my project as a unity package and importing it into a new project made in Unity 2018.3.14f1...after that you only have to write your tags again and delete the following files: androidx.browser.browser-1.0.0 androidx.browser.browser-1.0.0.aar androidx.core.core-1.0.0 androidx.core.core-1.0.0.aar from \YoureProjectLocation\Assets\Plugins\Android
got aws training and devops trainingby expert who has resolved and explained the above issue to me

just go through this devops training in chennai
|| data science course in chennai

Very Informative blog thank you for sharing. Keep sharing.
SAS Training in Chennai

ohh thank you for this man. good job keep it upppp...
https://devopsacademy.co/
https://devopsacademy.co/devops-training/
https://devopsacademy.co/terraform-training-in-bangalore/
https://devopsacademy.co/aws-training-in-bangalore/
https://devopsacademy.co/top-5-devops-tools-in-2021/
https://devopsacademy.co/welcome-to-uni-education/
https://devopsacademy.co/about-us/

Very informative blog, thanks for sharing. Tosca Training in Bangalore


CBOR deserialize checks and error handling

I'm personally unclear if there's error checking that needs to be added to CBOR deserialize methods. If the response is an invalid cbor object, does the code know and propagate that? Should it?
sibling issue with #274

:link: zboto Link

When deserializing responses from actors, we should use assert as it should never fail. If something fails, it means there is a bug. That is why there is no error propagation (like require or revert). We have rusts tests as integration tests where serialization and deserialization is tested.


Add query text search documentation

What
Update query documentation to include details for text search support.
Why
Text search support was recently added to Query. We need to provide developers with documentation that assists them in using Query Text Search.
How
Add text search content to query.md
reviewer @mikerhodes
reviewer @gadamc
BugID: 46163

:+1:

+1


Allow unique notification icons for separate notifications

Modeled after Facebook / Google Hangouts ability for each notification to have a different icon depending on the message (i.e. if you get a message from Jan, Jan's picture shows up as the notification icon, if you get a message from Bill, Bill's picture shows up as the notification icon, etc.).
Just pass the url of the image you want to show up for that particular notification as the variable "icon" in your GCM request. Works like this in the GCM request:
(modeled after http://stackoverflow.com/questions/11242743/gcm-with-php-google-cloud-messaging)
  extras.getString("message"); */

$message = "Life is incredibly exciting ....";
$title="You have a message.";

// Set POST variables.
$url = 'https://android.googleapis.com/gcm/send';
$icon='http://www.example.com/BillsProfilePicture.jpg';

$fields = array(
  'registration_ids'  => $registrationIDs,
  'data'              => array( "message" => $message, "title" => $title, "notId"=>'300', "icon"=>$icon )
);

$headers = array(
     'Authorization: key=' . $apiKey,
     'Content-Type: application/json'
);

// Open connection
$ch = curl_init();

// Set the url, number of POST vars, POST data
curl_setopt( $ch, CURLOPT_URL, $url );

curl_setopt( $ch, CURLOPT_POST, true );
curl_setopt( $ch, CURLOPT_HTTPHEADER, $headers);
curl_setopt( $ch, CURLOPT_RETURNTRANSFER, true );
curl_setopt ($ch, CURLOPT_SSL_VERIFYHOST, 0);
curl_setopt( $ch, CURLOPT_POSTFIELDS, json_encode( $fields ) );

// Execute post
$result = curl_exec($ch);

// Close connection
curl_close($ch);

// Echo success or failure
echo $result;

?>

Also, FYI, I merged the code from TheNewMR from here to allow for stacked notifications depending on different notification ID's:
https://github.com/thenewmr/PushPlugin/commit/728e975442e45668b7d809aec028396a9b023dfa
Instructions for how that works are on that page, however in case it gets deleted for some reason I'll copy/paste here:
"This allows for multiple notifications to be displayed on Android.  Simply add data with key "notId" and an integer value when sending via GCM.  Differentiate between different notifications by setting different notIds.  Defaults to 0 if not key supplied."

+1

+1

+1

+1

+1

+1 :)

+1

Guys, we have solution! Finally!
https://github.com/phonegap/phonegap-plugin-push


feat: More load-gen example options.

maxRequestsPerSecond

prevents throttling
log the limit so the user realizes it's the script, not the service

Change to time based instead of request based limits (see momentohq/client-sdk-dotnet#229)

totalNumberOfOperationsToExecute totalMsToRun
printStatsEveryNMs

Always print stats at the end.

This probably diverges from the other SDK's load generators.
Closes #123

@eaddingtonwhite What do you think of this implementation of momentohq/client-sdk-dotnet#229?

looks like there is a lint failure though.

FWIW if you run npm run build before committing I believe it will catch those.  And npm run format will fix them.


Close meetings from inside

Resolve #3622
Needs:

https://github.com/OpenSlides/openslides-backend/pull/2523
https://github.com/OpenSlides/openslides-meta/pull/128

Draft version with setting, dashboard, meeting list, committee detail changes, and disable setting for templates

client-side:

meeting access should be prohibited
delete dialog when deleting an account > the dialog should say when a meeting is locked from the inside
guards should not allow entry into meeting routes if the meeting is locked
organisation-level actions which would  change the meeting (add user to meetings) should not be usable
operator.service should be adjusted to match https://github.com/OpenSlides/OpenSlides/wiki/Permission-System

As far as I understand the PermissionGuard it forbids entering meetings, if you are not superadmin and not in the meeting (in one group).

So long, so good. We need a test, what is missing here. @Elblinator pls test and @bastianjoel pls review.

I worked on the CR. Introduced some changes for the committee-meeting-preview and the 'Set/remove meetings'
bulk action.

Okay, updated the PR to allow superadmins to see meeting-edit even if not in the meeting.

I have updated the way to get the meeting membership. I now use groups. So no need to load the long user_ids list.


Investigate OGG compression

I saw a few reports that the current OPUS/OGG encoding implementation results in a significant loss of fidelity compared to WAV. This is not ideal and needs to be investigated.
Also, investigate performance differences between x86 and arm architectures
Thoughts @blakesteve?

@lpatino10 Can you do some quick research to see if there is a way to encode the microphone audio to opus/ogg?

@lpatino10 are you find any solution for that ?

Sorry @haris15, I haven't gotten around to looking at this yet.


Packet Size EOF Bug / Problem

Currently I am using the ISO15118 software stack in a system that has an ethernet interface which cannot exceed 1500 MTU (Maximum transmission unit).
This means the interface can only receive a maximum of around 1500 bytes per packet.
I have found that when testing ISO15118-20 and certificates are sent between the EVCC and SECC that the packet(s) containing the certificate get split into for example two packets because of the size of the packets.
Only the first packet is read by the software stack and then rightfully so the Java codec complains about an unexpected EOF.

ERROR    2023-11-02 09:25:25,863 - iso15118.shared.comm_session (219): EXIDecodingError (Exception): Transformer Exception: java.io.EOFException: Premature EOS found while reading data.
Traceback (most recent call last):
File "/mnt/app_rw/data/SECC_SOFT_STACK/iso15118-master/iso15118/shared/exi_codec.py", line 292, in from_exi
exi_decoded = self.exi_codec.decode(exi_message, namespace)
File "/mnt/app_rw/data/SECC_SOFT_STACK/iso15118-master/iso15118/shared/exificient_exi_codec.py", line 63, in decode
raise Exception(self.exi_codec.get_last_decoding_error())
Exception: Transformer Exception: java.io.EOFException: Premature EOS found while reading data.
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/mnt/app_rw/data/SECC_SOFT_STACK/iso15118-master/iso15118/shared/comm_session.py", line 201, in process_message
decoded_message = EXI().from_exi(
File "/mnt/app_rw/data/SECC_SOFT_STACK/iso15118-master/iso15118/shared/exi_codec.py", line 294, in from_exi
raise EXIDecodingError(
iso15118.shared.exceptions.EXIDecodingError: EXIDecodingError (Exception): Transformer Exception: java.io.EOFException: Premature EOS found while reading data.
ERROR    2023-11-02 09:25:25,871 - iso15118.shared.comm_session (220): EXI message (ns=urn:iso:std:iso:15118:-20:CommonMessages) where error occured: 8

TCPDUMP OUTPUT OF TWO PACKETS BEING SPLIT
10:01:29.612806 IP6 fe80::d60f:b2ff:fe07:359b.56906 > X Flags [.], seq 111:1539, ack 97, win 507, options [nop,nop,TS val<PHONE_NUMBER> ecr 184799943], length 1428
10:01:29.612954 IP6 X > fe80::d60f:b2ff:fe07:359b.56906: Flags [.], ack 1539, win 501, options [nop,nop,TS val 184804426 ecr<PHONE_NUMBER>], length 0
10:01:29.615129 IP6 fe80::d60f:b2ff:fe07:359b.56906 > X Flags [P.], seq 1539:2297, ack 97, win 507, options [nop,nop,TS val<PHONE_NUMBER> ecr 184799943], length 758
10:01:29.615281 IP6 X > fe80::d60f:b2ff:fe07:359b.56906: Flags [.], ack 2297, win 501, options [nop,nop,TS val 184804428 ecr<PHONE_NUMBER>], length 0

I am wondering if this is the intended behavior or if there is a proper way for me to receive these two fragments to combine into one message. I have already dabbled with introducing a kind of "Number of fragments" addition to the code however I have been unsuccessful in its execution so far because of lack of knowledge.
Has anyone had issues similar to mine?
Thank you.

Okay so this is some very preliminary code. But this concept appears to resolve my issue. It is however not a very optimal solution
in iso15118/shared/comm_session.py rcv_loop()
                message = await asyncio.wait_for(self.reader.read(7000), timeout)
                message_len = len(message)

                # Fragments are likely...
                while (message_len > 1427): # TODO ENV MTU LIMIT, cap of 73 substracted from 1500 MTU for headers of packet
                        append_msg = await asyncio.wait_for(self.reader.read(7000 - message_len), timeout)
                        message = message + append_msg

                        message_len = len(append_msg)

Hmm...does replacing self.reader.read(7000) with self.reader.read() help?

Hmm...does replacing self.reader.read(7000) with self.reader.read() help?

Trying this solution did not go very well. After trying for some time make it work I could not get the stack to show normal function. It appears to keep reading indefinitely after receiving the first message. Anyone other ideas?
@shalinnijel2


Harden eap_proxy.sh script for use on USG

With a USG 3P a reboot of the device requires a subsequent reboot of the AT&T Router Gateway (RG) or unplugging/plugging the Ethernet cable on the USG to the AT&T Router Gateway (RG), which both require physical access. This is not ideal as the bypass setup with a USG would not survive a regular reboot.  Manually setting the interface towards the AT&T ONT with the MAC address of the AT&T Router Gateway (RG) will solve this problem.

This is exactly what the --set-mac option does, unless I'm misunderstanding something.

The --set-mac sets the ONT facing IF_WAN VLAN MAC (ether) address to the AT&T Router Gateway's (RG) MAC address. For that it needs to learn the RG's MAC address first, requiring traffic to be received by the USG.
It appears that when USG and RG loose power and have it restored at the same, the RG boots up quicker. Therefore the USG doesn't see relevant traffic for the eap_proxy to determine the RG's MAC address. Similarly when only restarting the eap_proxy.
This results in the eap_proxy trying to authenticate to the ONT with the wrong MAC address, unless the RG is restarted, allowing it to learn its MAC address.
The above manually sets the correct MAC address at start time of the eap_proxy.


RepeatableEntry on HasMany Relations not working

Package
filament/filament
Package Version
latest
Laravel Version
v3.2.115
Livewire Version
No response
PHP Version
PHP 8.3
Problem description
I have SalesOrderLines which is a HasMany relation. The RepeatableEntry doesn't show the data in infolist.
dd($infolist->record->SalesOrderLines()->get()); // I can confirm that there are items in it and it's not empty
public static function infolist(Infolist $infolist): Infolist
    {
        // dd($infolist->record->SalesOrderLines()->get());
        return $infolist
            ->schema([
                ...self::getSalesOrderInfoListSchema($infolist),
                \Filament\Infolists\Components\RepeatableEntry::make('SalesOrderLines')
                    ->columns(5)
                    ->schema([
                        Infolists\Components\TextEntry::make('ItemCode')
                            ->label(__('panel.itemcode'))
                            ->inlineLabel(),
                        Infolists\Components\TextEntry::make('ItemDescription')
                            ->label(__('panel.itemdescription'))
                            ->inlineLabel(),
                        Infolists\Components\TextEntry::make('Quantity')
                            ->label(__('panel.quantity'))
                            ->inlineLabel(),
                        Infolists\Components\TextEntry::make('AmountDC')
                            ->label(__('panel.amountdc'))
                            ->inlineLabel(),
                        Infolists\Components\TextEntry::make('AmountFC')
                            ->label(__('panel.amountfc'))
                            ->inlineLabel(),
                    ]),
            ]);
    }

Expected behavior
Show all related data
Steps to reproduce
public function SalesOrderLines():  HasMany
    {
        return $this->hasMany(SalesOrderLine::class, 'OrderID', 'OrderID');
    }

Reproduction repository (issue will be closed if this is not valid)
https://github.com/KamranBiglari/mattech-hub3
Relevant log output
No response

Donate 💰 to fund this issue

You can donate funding to this issue. We receive the money once the issue is completed & confirmed by you.
100% of the funding will be distributed between the Filament core team to run all aspects of the project.
Thank you in advance for helping us make maintenance sustainable!

@KamranBiglari the repo is not available..

@KamranBiglari the repo is not available..

Sorry, but it's not public!

I don’t think this is a Filament issue. There may be something wrong with your project. Please provide a minimal repository to reproduce the problem.


Added README and CDK Traffic Gen stack

Description
Set the repo up for CDK, added a CDK Stack to create a source of traffic using a AWS Fargate-backed Docker container that continuously curls some Alexa Top 20 sites, and added a README explaining what this repo is and how to use it.
Testing Done

Ran the steps in the README to create the traffic generator stack in my personal AWS Account
Confirmed the Fargate container was curling the websites as expected and the logs were ended up in CloudWatch

test comment


ByteBuffer: one fewer allocs to go to Data

Motivation:
When getting a Data from a ByteBuffer we currently allocate twice
(__DataStorage) and the closure for Data.Deallocator.
Modifications:
We can optimise that by making the closure capture exactly one
AnyObject which is already a reference counted object. The compiler
realises that (on Linux) and saves us an alloc.
Thanks @lukasa for the suggestion here: https://github.com/apple/swift-nio/pull/1836#issuecomment-830044155
Result:
Fewer allocs.

We should resurrect this


Full Review

A full review of the entire repository. When this PR is complete, be sure to manually merge its head branch into the main branch for this repository.

Great work so far! Very clear and idiomatic Go, well done.
It'll be nice to have some test that really makes HTTP requests, but not to the real API, won't it? Have you looked into the httptest package for ways to create simple local HTTP servers?

Have implemented this!


[SPARK-19028] [SQL] Fixed non-thread-safe functions used in SessionCatalog

What changes were proposed in this pull request?
Fixed non-thread-safe functions used in SessionCatalog:

refreshTable
lookupRelation

How was this patch tested?
N/A

cc @zsxwing @cloud-fan @yhuai

A related PR: https://github.com/apache/spark/pull/12915

Test build #70724 has started for PR 16437 at commit 722a0e2.

Test build #70724 has finished for PR 16437 at commit 722a0e2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70724/
Test PASSed.

LGTM, merging to master

shall we backport in to 2.1/2.0?

I think we need to backport it.

Please update the description in the JIRA ticket.  What is there now is simply not adequate, meaning that if anyone has to come back and address this issue some time on the future, what is there does not describe why a change was needed sufficiently for that person to avoid needing to look at the details in the code (which may well be quite different by that time.)

@markhamstra The JIRA is updated.

@markhamstra The JIRA is updated.

@gatorsmile it conflicts with branch 2.0, please send a new PR, thanks!


Update domains.json for iamganeshagrawal/goconfigenv

Added goconfigenv.ganeshagrawal.com for iamganeshagrawal/goconfigenv

 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.