Re-fetch actions when we move from offline to online in IOUModal.js

Details

Fixed Issues

$ https://github.com/Expensify/Expensify/issues/208196
Tests

Sign into NewDot
Open a chat with anyone
Click on the + button close to the text input
Verify that a GET Command is sent with rvl GetLocalCurrency is made via Chrome Dev Tools network tab
Go offline
Come back online
Verify from the screenshot below

PR Reviewer Checklist

[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] iOS / native
[ ] Android / native
[ ] iOS / Safari
[ ] Android / Chrome
[ ] MacOS / Chrome
[ ] MacOS / Desktop

[ ] I verified there are no console errors (if there’s a console error not related to the PR, report it or open an issue for it to be fixed)
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained “why” the code was doing something instead of only explaining “what” the code was doing.
[ ] I verified any copy / text shown in the product was added in all src/languages/* files
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named “index.js”. All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] Any functional components have the displayName property
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose and it is broken down into smaller components in order to separate concerns and functions

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn’t already exist
[ ] The style can’t be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.

QA Steps

[ ] Verify that no errors appear in the JS console
[ ] Sign into NewDot
[ ] Open a chat with anyone
[ ] Click on the + button close to the text input
[ ] Verify that a GET Command is sent with rvl GetLocalCurrency is made via Chrome Dev Tools network tab
[ ] Go offline
[ ] Come back online
[ ] Verify from the screenshot below

Screenshots

Web

Mobile Web

Desktop

iOS

Android

cc: @marcaaron

Please bring back the PR author checklist, as well as add screenshots showing you tested on each platform

Hey @alex-mechler , please this is a sample PR, that didn't require testing on other platforms(https://github.com/Expensify/App/pull/8817 and https://github.com/Expensify/App/pull/8817), can you please take a look?

I thought they were required for all PRs, asked in contributor management https://expensify.slack.com/archives/C01SKUP7QR0/p1652738900438809

Looks like this is linked to the wrong issue. Should be linked here https://github.com/Expensify/Expensify/issues/208196

I thought they were required for all PRs, asked in contributor management https://expensify.slack.com/archives/C01SKUP7QR0/p1652738900438809

Yeah, they were, but this is a special case if you'll.

I also don't agree that this is a special case. This is testable on every platform but iOS, and you can still check to ensure nothing has broken like Brandon mentioned.

Updated and awaiting another review

🚀 Deployed to staging by @alex-mechler in version: 1.1.63-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Generation of a downloadable zip

If there's one massive headache for macOS, it's the expectation that users will generally understand basic *nix skills in order to get a basic server running, it would kinda be cool if a zip file could be generated that would have a start.command script inside of it, with the executable bit set on the file within.
Contents would also have to marginally be modified for this, mainly, adding a call to cd in order to take us to the directory where the command file is in order to be able to start the server. Doing so would allow mac users to download the zip, extract the command file from within into the server dir, and be able to double click the file to run it;
cd "`dirname $0`"
java -jar server.jar

I don't have a mac to test this on, so once implemented, I'll need a little help testing whether I've done it correctly, but I'm all for this!
I've experimented with the idea of bundling the selected server jar as well, but I'm not sure about whether projects would be happy with this - it can be visited in the future.
However, this is such a simple change, and if it'll improve the user's experience, I'm a fan!


(linting): improve linting

Subject
Linting
Description
After having talked with @nikolovlazar about some code standards during reviews I think that adding a few linting rules would help to make contributing more streamlined. My proposition would be:

warning for console.logs
error for import order
warning for unused variables

This is only a first step which can be improved further in the future.

If that's something we want I can take care of this. Just assign me to this:)


Plot X axis resizing

When changing Bit Mode on the Levels Toolbar, the X axis will expand to accommodate a higher bit mode, however, if we change back to a lower Bit Mode it will not zoom back to accommodate.

This is intended behavior. You don't want the histogram to scale when switching between the sigma's.
But I can make it scale when you choose one of the 'bit' settings.

That will work for us, thank you


RequestContentDataExtractor#extractFromMultipartRequest duplicated url-parameter-value if content-type is multipart/form-data and value contains escaped characters

Bugreport:
The value of a URL parameter gets duplicated if content-type is multipart/form-data and the value contains escaped characters.
POST 'http://localhost/test?name=hello%20world'
results in
name = hello world,hello world
This happens in org.springframework.cloud.netflix.zuul.util.RequestContentDataExtractor#extractFromMultipartRequest.
There two lists should be compared but the first listOfAllParams contains unescaped values and the other listOfOnlyQueryParams contains escaped values.
Example application: https://github.com/s3erjaeh/example_for_spring-cloud-netflix-issue
Environment:
org.springframework.cloud:spring-cloud-netflix-zuul:2.0.1.RELEASE
included via

org.springframework.cloud; version: Finchley.SR1
org.springframework.boot; version: 2.0.5.RELEASE

I'm also facing the same issue. And the code is not working for non-English characters i.e. UTF-8 characters are shown as ????

@s3erjaeh Thanks for reporting it. We will fix it.


SVG Base64 - Unable to load image

Category

[ ] Enhancement
[ ] Bug
[ ] Question
[x] Documentation gap/issue

Version
Please specify what version of the library you are using: [ 3.10.0 ]
Please specify what version(s) of PowerPoint you are targeting: [ PowerPoint 2007 and up ]
Expected / Desired Behavior / Question
I've passed an SVG as Base64 to slide.addImage({ data: svgData }) and I would expect that to render, however it doesn't.
Observed Behavior
After passing in my Base64 SVG as data I receive an error: "ERROR! Unable to load image (image.onerror): data:image/svg+xml;base64,PHN..." I've gone on to confirm that the Base64 data really does render the SVG I want and there isn't any issues in its encoding. I've then gone on to test this with a different Base64 SVG, namely yours that you use in the demos. You've defined it in demos/modules/media.mjs and that data does work.
I did notice that in the returned error there the listing of the Base64 data had "png" appened onto it and wondered if there was something attempting to convert the svg into a png and failing. It turns out that in gen-media.ts there is a function: createSvgPngPreview that does just that and this is also where the Unable to load image error originates. My best guess right now is that my SVG data is too large (it's roughly 40kb in size) and failing this process somewhere along the way.
Can you tell me if there is something limiting size of Base64 SVG data?
Would I do better to convert my SVG into a PNG on my side and then send it as "image/png;base64"?
Steps to Reproduce
I'm generating a powerpoint file with this basic code: https://gist.github.com/matthew-davis/f5f5e74cfdc639e1628e1297d07674be
Then the Base64 SVG data I'm using is in this gist: https://gist.github.com/matthew-davis/c9970d0becad7725a66af171a2bc1683

@matthew-davis - the demo includes use of a base64 SVG, so i'll have to see what's different in this case.
The demo_image.mjs image demo uses this format:
export const SVG_BASE64 = "data:image/svg+xml;base64A...="

Ran into the same issue. Works fine in Chrome, but fails in FF. I believe this is running afoul of this bug:  https://bugzilla.mozilla.org/show_bug.cgi?id=700533
Firefox will report the dimensions of your SVG as 0/0, so it bombs here.  The png at the end of the base64 data is a red herring because it uses path instead of data in the error message.
If you fix your SVG using one of the workarounds described in the bug, before creating the data URI, then it works. I added this util function to my export code:
function fixSvgDimensions(svg) {
	// Workaround for FF bug: https://bugzilla.mozilla.org/show_bug.cgi?id=700533
	svg.width.baseVal.valueAsString = svg.width.baseVal.value.toString();
	svg.height.baseVal.valueAsString = svg.height.baseVal.value.toString();
}

Thanks @nbrendler !
@matthew-davis - if this issue is indeed limited to Firefox, then i'll close this issue.

Yep, go for closing the issue.

Hey,
I am facing the same issue but on chrome as well. the data seems large and png is appended at the end in the error. Any pointers on how to move forward?


Feature: Support HTTP/3 (+quic)

Is there possible to make loadbalancer for handling new protocol using transport HTTP/3 ?

See upstream issue https://github.com/kubernetes/ingress-nginx/issues/4760


Merge nneva multiple application changes

Attempting to resolve conflicts between nneva's multiple application branch and jleclanche.

Ok, I'm stuck here.  How do I merge migrations from two different commits when the library is only an app?

There's a lot of stuff here. I can't tell what any of it does.

I agreed with @jleclanche, it should be divided to small features and every feature should splitted to many small commtis to make CR more easier.
Also I think, applications should be instances of some model, because I prefer to have an ability to manage applications from admin. And if this approach will be used, we have to store APNS Certificates into database, because some projects uses S3 backend for static and user media files, and there is a bad idea make certificates public (which contains private keys)
Therefore, openssl should be replaced by pyopenssl library to implement possibility to connect to APNS server with certificate from string. But more better is implementing this ability in PyAPNS and use it as external dependency. But this is another issue

I definitely would like to offload the APNS logic to PyAPNS or a similar library. Same for GCM etc.


shorten qps worker timeouts and run remote workers under timeout

to make sure remote workers killed shortly after if ssh is killed

seeing new failures like:
/home/jenkins/workspace/gRPC_performance_pull_requests/src/node/performance/worker_service_impl.js:78
        switch (setup.payload_config.payload) {
                                    ^

TypeError: Cannot read property 'payload' of null
    at ServerDuplexStream.<anonymous> (/home/jenkins/workspace/gRPC_performance_pull_requests/src/node/performance/worker_service_impl.js:78:37)
    at emitOne (events.js:77:13)
    at ServerDuplexStream.emit (events.js:169:7)
    at readableAddChunk (_stream_readable.js:153:18)
    at ServerDuplexStream.Readable.push (_stream_readable.js:111:10)
    at readCallback (/home/jenkins/workspace/gRPC_performance_pull_requests/src/node/src/server.js:360:14)

and
/home/jenkins/workspace/gRPC_performance_pull_requests/src/ruby/qps/client.rb:86:in `initialize': undefined method `simple_params' for nil:NilClass (NoMethodError)
	from src/ruby/qps/worker.rb:79:in `new'
	from src/ruby/qps/worker.rb:79:in `block (2 levels) in run_client'
	from /home/jenkins/workspace/gRPC_performance_pull_requests/src/ruby/lib/grpc/generic/bidi_call.rb:213:in `block in read_loop'
	from /home/jenkins/workspace/gRPC_performance_pull_requests/src/ruby/lib/grpc/generic/bidi_call.rb:191:in `loop'
	from /home/jenkins/workspace/gRPC_performance_pull_requests/src/ruby/lib/grpc/generic/bidi_call.rb:191:in `read_loop'
	from src/ruby/qps/worker.rb:76:in `each'
	from src/ruby/qps/worker.rb:76:in `block in run_client'

guessing may be related to this shortened timeout

jenkins: test this please

got rid of update after realizing failure unrelated, fixed by #9580.
went back again to original shortened timeout
jenkins: test this please

python and performance failures which should be fixed now., re-running again.
jenkins: test this please

test failures:

mac PR: https://github.com/grpc/grpc/issues/9586 (fixed)
mac PR: https://github.com/grpc/grpc/issues/9560 (known , expected to fail)

interop PR failure was a timeout building python docker image:

grouping it with  python docker build failure in #9099


Failed to load the ng-classify-preview package

[Enter steps to reproduce below:]

...
...

Atom Version: 1.1.0
System: Mac OS X 10.11.2
Thrown From: ng-classify-preview package, v0.2.1
Stack Trace
Failed to load the ng-classify-preview package
At Cannot find module '/opt/homebrew-cask/Caskroom/atom/1.0.16/Atom.app/Contents/Resources/app.asar/src/editor'

Error: Cannot find module '/opt/homebrew-cask/Caskroom/atom/1.0.16/Atom.app/Contents/Resources/app.asar/src/editor'
  at Module._resolveFilename (module.js:334:15)
  at Function.Module._resolveFilename (/opt/homebrew-cask/Caskroom/atom/1.0.16/Atom.app/Contents/Resources/app.asar/src/module-cache.js:383:52)
  at Function.Module._load (module.js:284:25)
  at Module.require (module.js:363:17)
  at require (module.js:382:17)
  at Object.<anonymous> (/Users/chris/.atom/packages/ng-classify-preview/lib/NgClassifyPreviewView.coffee:5:16)
  at Object.<anonymous> (/Users/chris/.atom/packages/ng-classify-preview/lib/NgClassifyPreviewView.coffee:1:1)
  at Module._compile (module.js:428:26)
  at Object.defineProperty.value [as .coffee] (/opt/homebrew-cask/Caskroom/atom/1.0.16/Atom.app/Contents/Resources/app.asar/src/compile-cache.js:169:21)
  at Module.load (module.js:353:32)
  at Function.Module._load (module.js:308:12)
  at Module.require (module.js:363:17)
  at require (module.js:382:17)
  at Object.<anonymous> (/Users/chris/.atom/packages/ng-classify-preview/lib/ng-classify-preview.coffee:1:15)
  at Object.<anonymous> (/Users/chris/.atom/packages/ng-classify-preview/lib/ng-classify-preview.coffee:1:1)
  at Module._compile (module.js:428:26)
  at Object.defineProperty.value [as .coffee] (/opt/homebrew-cask/Caskroom/atom/1.0.16/Atom.app/Contents/Resources/app.asar/src/compile-cache.js:169:21)
  at Module.load (module.js:353:32)
  at Function.Module._load (module.js:308:12)
  at Module.require (module.js:363:17)
  at require (module.js:382:17)
  at Package.module.exports.Package.requireMainModule (/opt/homebrew-cask/Caskroom/atom/1.0.16/Atom.app/Contents/Resources/app.asar/src/package.js:725:34)
  at /opt/homebrew-cask/Caskroom/atom/1.0.16/Atom.app/Contents/Resources/app.asar/src/package.js:176:28
  at Package.module.exports.Package.measure (/opt/homebrew-cask/Caskroom/atom/1.0.16/Atom.app/Contents/Resources/app.asar/src/package.js:153:15)
  at Package.module.exports.Package.load (/opt/homebrew-cask/Caskroom/atom/1.0.16/Atom.app/Contents/Resources/app.asar/src/package.js:167:12)
  at PackageManager.module.exports.PackageManager.loadPackage (/opt/homebrew-cask/Caskroom/atom/1.0.16/Atom.app/Contents/Resources/app.asar/src/package-manager.js:396:14)
  at PackageManager.module.exports.PackageManager.activatePackage (/opt/homebrew-cask/Caskroom/atom/1.0.16/Atom.app/Contents/Resources/app.asar/src/package-manager.js:473:30)
  at /opt/homebrew-cask/Caskroom/atom/1.0.16/Atom.app/Contents/Resources/app.asar/node_modules/settings-view/lib/package-manager.js:444:29
  at exit (/opt/homebrew-cask/Caskroom/atom/1.0.16/Atom.app/Contents/Resources/app.asar/node_modules/settings-view/lib/package-manager.js:73:16)
  at triggerExitCallback (/opt/homebrew-cask/Caskroom/atom/1.0.16/Atom.app/Contents/Resources/app.asar/src/buffered-process.js:213:47)
  at /opt/homebrew-cask/Caskroom/atom/1.0.16/Atom.app/Contents/Resources/app.asar/src/buffered-process.js:220:18
  at Socket.<anonymous> (/opt/homebrew-cask/Caskroom/atom/1.0.16/Atom.app/Contents/Resources/app.asar/src/buffered-process.js:98:18)
  at emitOne (events.js:82:20)
  at Socket.emit (events.js:169:7)
  at Pipe._onclose (net.js:467:12)

Commands
     -0:44.3.0 core:backspace (atom-text-editor.editor.is-focused)
     -0:43.9.0 editor:delete-to-beginning-of-line (atom-text-editor.editor.is-focused)
  3x -0:43.7.0 core:backspace (atom-text-editor.editor.is-focused)
     -0:42.9.0 core:save (atom-text-editor.editor.is-focused)
     -0:38.1.0 command-palette:toggle (atom-text-editor.editor.is-focused)
     -0:37.2.0 editor:consolidate-selections (atom-text-editor.editor.mini.is-focused)
     -0:37.2.0 core:cancel (atom-text-editor.editor.mini.is-focused)
     -0:36.7.0 application:minimize (atom-text-editor.editor.is-focused)
     -0:27.2.0 autocomplete-plus:cancel (atom-text-editor.editor.is-focused.autocomplete-active)
     -0:26.9.0 command-palette:toggle (atom-text-editor.editor.is-focused)
  3x -0:20.7.0 core:backspace (atom-text-editor.editor.mini.is-focused)
     -0:19.4.0 editor:consolidate-selections (atom-text-editor.editor.mini.is-focused)
     -0:19.4.0 core:cancel (atom-text-editor.editor.mini.is-focused)
     -0:18.7.0 editor:consolidate-selections (atom-text-editor.editor.is-focused)
     -0:18.7.0 core:cancel (atom-text-editor.editor.is-focused)
     -0:18.2.0 settings-view:open (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "disabledPackages": [
      "symbols-view",
      "wrap-guide",
      "placeholder-text"
    ],
    "projectHome": "/Users/chris/code",
    "ignoredNames": [
      ".git",
      ".hg",
      ".svn",
      ".DS_Store",
      "._*",
      "Thumbs.db",
      ".tag*"
    ]
  }
}

Installed Packages
# User
Sublime-Style-Column-Selection, v1.3.0
aligner-ruby, v1.4.0
angularjs, v0.3.3
api-blueprint-preview, v0.6.0
atom-alignment, v0.12.1
atom-beautify, v0.28.17
atom-ctags, v4.4.0
autocomplete-ruby, v0.1.0
build, v0.48.0
color-picker, v2.0.13
emmet, v2.3.14
file-icons, v1.6.11
git-log, v0.4.1
grunt-runner, v0.11.0
highlight-line, v0.11.0
highlight-selected, v0.10.1
html-entities, v0.4.0
imdone-atom, v1.1.54
javascript-snippets, v1.0.0
language-api-blueprint, v1.0.2
language-nginx, v0.6.1
linter, v1.10.0
linter-coffeelint, v1.1.2
linter-csslint, v1.1.0
linter-jshint, v1.2.1
linter-less, v2.2.0
linter-ruby, v1.2.0
linter-scss-lint, v2.4.0
linter-tidy, v1.0.1
minimap, v4.16.0
ng-classify-preview, v0.2.1
pigments, v0.18.1
pretty-json, v0.4.1
project-manager, v2.6.5
rails-model-schema, v0.6.0
remote-atom, v1.2.9
ruby-block, v0.3.5
set-syntax, v0.3.0

# Dev
No dev packages

Got this error, too.

Hi @pelted and @penne12,
ng-classify is now supported in https://atom.io/packages/preview, which is a generic viewer of multiple file types, including ng-classify.
Hope this helps.
Thanks,
Cary

@CaryLandholt Ok, thanks! You might want to mark this package as deprecated so people don't load it. I still like the snippets it comes with, could you please export those for us?

Thanks @penne12 - I've deprecated the package.

THIS PACKAGE HAS BEEN DEPRECATED

Please use preview for previewing your ng-classify files
Please use ng-classify-snippets for convenient keyboard shortcuts

Thanks so much!


OSError when creating virtualenv symlinking in Vagrant

The issue
Using pipenv in a vagrant box is causing a little trouble with the symlinking.
It has been requested that --always-copy is added to pipenv when it creates the virtual environment see ( #1929) but it appears that is not gonna happen 😞
The error
Using /usr/bin/python3.6m (3.6.5) to create virtualenv…
Running virtualenv with interpreter /usr/bin/python3.6m
Using base prefix '/usr'
New python executable in /vagrant/project/thing/.venv/bin/python3.6m
Also creating executable in /vagrant/project/thing/.venv/bin/python
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/virtualenv.py", line 2349, in <module>
    main()
  File "/usr/local/lib/python3.6/dist-packages/virtualenv.py", line 712, in main
    symlink=options.symlink)
  File "/usr/local/lib/python3.6/dist-packages/virtualenv.py", line 927, in create_environment
    site_packages=site_packages, clear=clear, symlink=symlink))
  File "/usr/local/lib/python3.6/dist-packages/virtualenv.py", line 1395, in install_python
    os.symlink(py_executable_base, full_pth)
OSError: [Errno 71] Protocol error: 'python3.6m' -> '/vagrant/project/thing/.venv/bin/python'

Environment variables
export PIPENV_VENV_IN_PROJECT=1
export PIPENV_IGNORE_VIRTUALENVS=1

commands to fix it
pipenv sync # Fails with OSError listed above
virtualenv --always-copy .venv
pipenv sync # Success!

Seen in vagrant box: ubuntu/xenial64 and ubuntu/bionic64
I hope it helps someone who got stuck with this like I did

Seems like this is a problem with Vagrant running on Windows. Is this the case? This would be a legitimate use case worth looking into (for me).

@uranusjr It is indeed Vagrant running on windows
@techalchemy it works! thank you so much!
tested it doing:
export VIRTUALENV_ALWAYS_COPY=1
pipenv sync # success
pipenv --rm
export VIRTUALENV_ALWAYS_COPY=0
pipenv sync # fail as expected

Probably best to add this to documentation or wiki for future reference. You won’t be the last having this problem.

Possibly we should just turn this on for windows users...

Problem is you can’t detect Windows inside a Vagrant box (it’s report itself as Linux). Otherwise virtualenv would have detected it and switch to copy mode automatically.

In my setup I'm using Virtualbox as the virtualization layer, and I believe vagrant simply uses virtualbox to do the share.
I can confirm that I am making the virtualenv in the folder shared by the host OS.
The vagrantfile I use:
# -*- mode: ruby -*-
# vi: set ft=ruby :

Vagrant.configure("2") do |config|

  config.vm.define "testbox" do |dev|
    dev.vm.box = "ubuntu/xenial64"
    dev.vm.host_name = "testbox"
    dev.vm.network :private_network, ip: "<IP_ADDRESS>"
    config.vm.provider :virtualbox do |vb|
        vb.customize ["modifyvm", :id, "--memory", "4096"]
        vb.customize ["modifyvm", :id, "--ioapic", "on"]
        vb.customize ["modifyvm", :id, "--cpus", "2"]
        vb.linked_clone = true
    config.vm.synced_folder ".", "/vagrant",
        type: "virtualbox",
        mount_options: ["dmode=775,fmode=775"]
    end
    config.vm.provision "shell", path: "bootstrap_testbox.sh"
  end
end

I'm wondering if my mount options are causing the issues I see.

Yeah this is definitely an issue of the folder being shared by the host => the guest would need to know that somehow or you'd have to do the creation on the host itself or else you just need to keep that environment variable set anytime you do virtualenv creation in this folder


Serverless support

Hi there,
Is there any plans to have a serverless tool running on Rancher? This way, we'd be able to build functions as a service on top of orchestration tools like Cattle or Kubernetes. I don't want to rely on AWS or GCP just for this kind of features...
A Rancher tool for serverless, orchestration agnostic and connected to the Rancher secrets API (Vault) would be awesome.
Thanks.

There are some serverless frameworks like fission and kubeless for Kubernetes that you might want to check out. We don't have any current plans for a Cattle serverless framework yet, but we are interested in the idea.

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


Backup Truck Factor

As part of my PhD research on code authorship, we calculated the Truck Factor (TF) of some popular GitHub repositories.
As you probably know, the Truck (or Bus) Factor designates the minimal number of developers that have to be hit by a truck (or quit) before a project is incapacitated. In our work, we consider that a system is in trouble if more than 50% of its files become orphan (i.e., without a main author).
More details on our work in this preprint: https://peerj.com/preprints/1233
We calculated the TF for Backup and obtained a value of 2.
The developers responsible for this TF are:
Brian D. Burns - author of 85% of the files
Michael van Rooijen - author of 31% of the files
To validate our results, we would like to ask Backup developers the following three brief questions:
(a) Do you agree that the listed developers are the main developers of Backup?
(b) Do you agree that Backup will be in trouble if the listed developers leave the project (e.g., if they win in the lottery, to be less morbid)?
(c) Does Backup have some characteristics that would attenuate the loss of the listed developers (e.g., detailed documentation)?
Thanks in advance for your collaboration,
Guilherme Avelino
PhD Student
Applied Software Engineering Group (ASERG)
UFMG, Brazil
http://aserg.labsoft.dcc.ufmg.br/

I see you posted a lot of these kinds of issues on repositories, seems really spammy and off-topic. This is an issue tracker, not a place for a general discussion. An email or Gitter would have been better.
But anyway, since this is here now:

(a) Do you agree that the listed developers are the main developers of Backup?

No, Burns hasn't been contributing for a while now. I've taken over what he was doing.

(b) Do you agree that Backup will be in trouble if the listed developers leave the project (e.g., if they win in the lottery, to be less morbid)?

Backup shouldn't be in trouble just because the current developers get hit by trucks. It's an open source project, anyone can start contributing if they want to. It might not continue in it's current form, but people will find a way to keep using it.
I actually got hit by car recently and I didn't respond to issues for a while. I hope we will find someone else to help out if it had been more serious.
Me winning the lottery would actually help the project, because I wouldn't have to worry about money anymore and let me work on what I want.

(c) Does Backup have some characteristics that would attenuate the loss of the listed developers (e.g., detailed documentation)?

Backup has good documentation to solve most configuration problems. The code itself is readable and documented, and we have plenty of specs to detect problems.

Thank you.


Develop blockchain atomicswap tools

Issue migrated from https://api.github.com/repos/Jumpscale/lib9/issues/290, opened by @ahussein
We need a jumpscale tool that allow us to do atomicswap across different blockchain networks.
The API should be something like:
j.tools.atomicswap(tft_node_prefab, btc_node_prefab, amount_in_tft, amount_in_btc, tft_wallet_pass)

commented by @ahussein
code complete and code is merged to development. Will keep the ticket open to complete the documentation

commented by @ahussein
docs can be found here: https://github.com/Jumpscale/sandbox/blob/master/crypto_sandboxing/readme.md#atomicswap-process

commented by @ahmedelsayed-93
closed by mistake


Unable to revive connection while creating pelias schema index on ES

When I ran a Schema create pod it throws below error:
**http://pelias-staging-es-elb-XXXX.ap-southeast-2.elb.amazonaws.com:9200/_nodes => getaddrinfo EAI_AGAINpelias-staging-es-elb-XXXXX.ap-southeast-2.elb.amazonaws.com pelias-staging-es-elb-XXXXX.ap-southeast-2.elb.amazonaws.com:9200
at Log.error (/code/pelias/schema/node_modules/elasticsearch/src/lib/log.js:239:56)
at checkRespForFailure (/code/pelias/schema/node_modules/elasticsearch/src/lib/transport.js:298:18)
at HttpConnector. (/code/pelias/schema/node_modules/elasticsearch/src/lib/connectors/http.js:171:7)
at ClientRequest.wrapper (/code/pelias/schema/node_modules/lodash/lodash.js:4929:19)
at ClientRequest.emit (events.js:182:13)
at Socket.socketErrorListener (_http_client.js:392:9)
at Socket.emit (events.js:182:13)
at emitErrorNT (internal/streams/destroy.js:82:8)
at emitErrorAndCloseNT (internal/streams/destroy.js:50:3)
at process._tickCallback (internal/process/next_tick.js:63:19)
Elasticsearch WARNING: 2019-09-06T01:30:05Z
Unable to revive connection: http://pelias-staging-es-elb-XXXXX.ap-southeast-2.elb.amazonaws.com:9200/
Elasticsearch WARNING: 2019-09-06T01:30:05Z
No living connections
{ Error: No Living connections
at sendReqWithConnection (/code/pelias/schema/node_modules/elasticsearch/src/lib/transport.js:266:15)
at next (/code/pelias/schema/node_modules/elasticsearch/src/lib/connection_pool.js:243:7)
at process._tickCallback (internal/process/next_tick.js:61:11) message: 'No Living connections' }
please install mandatory plugins before continuing.**
here are the ES settings specified in values and passed it through configmap.tpl
elasticsearch:
host: "pelias-staging-es-elb-XXXXX.ap-southeast-2.elb.amazonaws.com"
port: 9200
protocol: "http"
keepAlive: false
DeadTimeout: 1200
maxRetries: 15
Do you have any idea how do i get workaround for this issue?

Hi @pravink,
This looks like an issue on your end with configuring Elasticsearch or your network, and not a bug report for this project.
If you need professional help setting up Pelias, we offer support plans at geocode.earth, you can email us at<EMAIL_ADDRESS>if that might be helpful to you.


Update tests

Refactor the template tests and add a few tests for parsing and actual compilation

Pull Request Test Coverage Report for Build 168

112 of 130 (86.15%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+13.6%) to 54.867%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pydrad/configure/tests/test_configure.py
3
4
75.0%

pydrad/parse/tests/test_strand.py
2
4
50.0%

pydrad/configure/configure.py
1
6
16.67%

pydrad/configure/util.py
5
15
33.33%

Totals

Change from base Build 165:
13.6%

Covered Lines:
310

Relevant Lines:
565

💛  - Coveralls


Patch 1

What current issue(s) does this address, or what feature is it adding?
Fixing description of ScriptBuilder after move to VM.

[ x ] Are you making a PR to a feature branch or development rather than master?

Please pull the latest development branch into your patch branch and make sure only your wanted change is in there. There seems to be additional unwanted changes in your PR (changing version numbers from 07.2-dev to 0.7.2)


[#5514] Fix DiskFileUpload and MemoryFileUpload equals(...) method.

Motivation:
DiskFileUpload and MemoryFileUpload.equals(...) are broken.
Modifications:
Fix implementation and add unit test.
Result:
Equals method are correct now.

Cherry-picked into 4.0 (f012d4c) and 4.1 (c735b3e)


How to disable unity view during development to be able to run on simulators

Things are fine, I just want to be able to develop on the simulator, what are you guy doing to solve this?

It works in the android emulator even with the UnityView.

Things are fine, I just want to be able to develop on the simulator, what are you guy doing to solve this?

Delete UnityExport folder
Delete libRNUnityView from Link Binary with Library
Change main.m in Xcode to
int main(int argc, char * argv[]) { @autoreleasepool {  return UIApplicationMain(argc, argv, nil, NSStringFromClass([AppDelegate class])); }

To run on the real device again just recreate the reverse process.


Unable to tag modules using a generic git repository as source

Describe the bug
Yor will not tag a module that is using a generic git repository as source:
module "my_module" {
  source = "git::https://internal.repo.com/my-terraform-module.git?ref=vX.X.X"
  var1   = var.var1
  var2   = var.var2
  ...
}

Module not tagged when running yor tag -d .
Output:
  __    __
  \ \  / /
   \ \/ /___  _  ____
    \  /  _ \| |/  __|
    | |  |_| |   /
    |_|\____/|__|v0.1.139
 Yor Findings Summary
 Scanned Resources:	  5
 New Resources Traced: 	  0
 Updated Resources:	  0

Expected behavior
Module to be properly tagged

Yor Version: 0.1.139

Hello @ruortega, thank you for raising this issue.
To reproduce I created this module
module "my_module" {
  source = "git::https://github.com/cloudposse/terraform-null-label.git?ref=tags/0.19.2"
}

And ran yor tag -d .
I could not detect a problem, and the module was tagged successfully:
    "summary": {
        "scanned": 1,
        "newResources": 1,
        "updatedResources": 0
    }

In your example I see that you're trying to tag internal.repo. Since tagging a remote module requires yor to install the module, are you certain that the local machine has the required permissions to perform such an action on the internal repo?

Hi @rotemavni,
Thanks for the answer, but I'm not having any issues when using github.com as a source, those are working fine and being tagged accordingly, it is just when referencing repositories hosted on our git server.

@ruortega Got it, and do you have access to the git server from the machine running yor?

Yes @rotemavni, no issues accessing the git server

@ruortega
In order for us to be able to tag a remote module we need to an indication that the module is taggable, either by having the module block already tagged or by having a tags input in the module.
Can you confirm that the module you are tagging does in fact contain a tags input?

Thanks @rotemavni, changing the input name in the module to "tags" made the trick.


Why the learning rate of  deformable_module is 0.1*base_lr

Hi @HuguesTHOMAS ,
https://github.com/HuguesTHOMAS/KPConv-PyTorch/blob/94b1d87e17ec339e2aad049ce5d5d175b9cd0db1/utils/trainer.py#L80
I was wondering why the learning rate of deformable_module is 0.1*base_lr? (less than other parameters)

It is the case in the original paper of Deformable convolution in images. From my experiment it helps a little bit.
My understanding of it is that it slows down the rate at which the deformation are learned (and thus modified) during training. Which helps the network weights "catching up" with the new dispositions that are learned, if that makes any sense to you.


Bump scala 2.13.4

refs #120

Bumping versions across the board. I'm expecting a few failures here and there. Let see what the travis build finds out.

[ ] play-java-streaming-example
[ ] play-scala-anorm-example
[ ] play-scala-chatrom-example
[ ] play-scala-compile-di-example
[ ] play-scala-fileupload-example
[ ] play-scala-forms-example
[ ] play-scala-hello-world-tutorial
[ ] play-scala-mavwire-di-example
[ ] play-scala-rest-api-example
[ ] play-scala-secure-session-example
[ ] play-scala-slick-example
[ ] play-scala-starter-example
[ ] play-scala-websocket-example

^

Thanks @dwijnand for the bump. I had to put this on hold and just today managed to squeeze in some time to look into it. 👩🏼🌾

^

Thanks @dwijnand for the bump. I had to put this on hold and just today managed to squeeze in some time to look into it. 👩🏼🌾

@ignasi35 Please have a look here as well, the seed templates need update:
https://github.com/playframework/play-scala-seed.g8/pull/149
https://github.com/playframework/play-scala-seed.g8/pull/150
https://github.com/playframework/play-java-seed.g8/pull/95
https://github.com/playframework/play-java-seed.g8/pull/96


Chained command with argument and option results in "No such command"

Chained command with argument and option results in "No such command"; according to the documentation,

Other than that [nargs=-1 being allowed only on the last chained command] there are no restrictions on how they [chained commands] work. They can accept options and arguments as normal.

Reproduced on Ubuntu 18.04 with Python 3.5, 3.6 and 3.7, for all tagged Click versions between 3.0 and 7.0 (chain= does not seem to be supported for the ones before that).
Minimal repro (removing the argument allows the option to be recognized):
import click

@click.group(chain=True, no_args_is_help=False)
def cli():
    pass

@cli.command()
@click.argument('argument')
@click.option('--option')
def command(argument, option):
    pass

if __name__ == '__main__':
    cli()

Results in:
$ python3 test.py command a --option x
Usage: test.py [OPTIONS] COMMAND1 [ARGS]... [COMMAND2 [ARGS]...]...
Try "test.py --help" for help.

Error: No such command "--option".
$ python3 test.py command --help
Usage: test.py command [OPTIONS] ARGUMENT

Options:
  --option TEXT
  --help         Show this message and exit.

OK, looking at the OptionParser instances used in the example above, I saw that they have allow_interspersed_args set to False (used to "to implement nested subcommands safely", which makes sense).
This, however, leads to inconsistent behavior:

non-chained commands allow both the  --option argument and the argument --option orders
chained commands only allows --option argument

While the help text clearly states that commands should be called like [OPTIONS] ARGUMENT, the fact that both orders work in most of the cases may be confusing to users (it was for me, at least).
If I understand correctly, it's unlikely argument --option will ever be possible for chained commands. Should I update the Multi Command Chaining section to reflect the behavior described above? Why not Argparse? implies it, but that's not the first place someone would look when encountering it.

Seems reasonable to add a line in the docs to say that you have to use argument followed by options. Although I recall this being mentioned several places in the docs.

https://github.com/pallets/click/issues/1323


Merge checks doesn't work on forked repos

See the example of PR where merge-checks weren't triggered: https://github.com/iakunin/codexia-bot/pull/118

[x] reproduce the problem
[x] fix the problem

Fixed. PR example: https://github.com/iakunin/codexia-bot/pull/263


[EPIC] Machine registry

This PR is a summary of possible ideas to improve the machine registry introduced with #4824.
### 🎰 Machine type improvements
- [ ] ❗ send notification on error status change or machines capture errors
- [ ] machine permissions with custom permission levels per machine type (e.g. so that only specific roles/users can print labels with a specific printer - e.g. for label printers there are the following permission levels: "print")
- [ ] Define icons for each machine type and show them along the UI

### 🚚 Driver improvements
- [ ] ❗ machine status polling function on driver (with option to disable status polling for all printers/for an individual printer)
- [ ] Termination hook for machines, (already tried at 6bb0ba719bef6f61f4395823f2ff8ee8f7097a3e but not that easy when run in gunicorn), then the machine restart could be a simple terminate + re init
- [ ] let driver auto discover physical machines in the network/computer and show them so that they can be added quickly (like homeassistant has it)
- [ ] machine actions (buttons on machine page that can trigger certain function on the driver)

### 🖨 Label printer improvements
- [ ] ❗Show the printer status badge in the machine selection dropdown (maybe by extending the ChoiceField with an optional status code property)
- [ ] integrated label printing queue for each machine
- [ ] label printer dialog sort machines based on priorities provided as tuple with `get_printers` (not sure if that makes sense or if last used is just enough, we have to see)

### 🎰 Ideas for other machine types
- [ ] PnP machine type (#4164)
- [ ] 3d printer machine type (#5191)

### 📝 Other things
- [ ] ❗ Settings only required if conditions are met (e.g. for a usb/network printer select the host, port, user, password only makes sense for network printers, for usb printer a single select is enough. => Dependent settings)
- [ ] Live Autocomplete Settings type key (Client posts current values to API and API responds with suggestion)
- [ ] add metadata to machine models
- [ ] How does scaling of the machine registry work if InvenTree runs on multiple nodes (maybe Kubernetes)
- [ ] Issue plugin events on machine load/finish load (ref: https://github.com/inventree/InvenTree/pull/4824#issuecomment-1574104681) (Already tried, but: enqueues to much offloaded events that are unnecessary on simple setups on startup => the event system first needs a fix)

I currently integrate my 2 PnP machines, conveyors and stencil printer with inventree using MQTT. Basically a program runs on the PnPs and sends info to inventree. The PnPs consume parts. Currently the way they create build orders / build units is a bit messy but I suspect I should transition over to this interface. I'd like to easily view status for the line and some stats would be handy too.

@loki42 sounds like a great candidate for a "machine" interface. Perhaps you could work with @wolflu05 to provide a demonstrator for this

@loki42 would you be open to do a short call/chat to understand your requirements better?

Yeah,  happy to do that.  Basically keen to have an easy way to see status
of all the machines and utilisation.
I currently have events triggered from the machines that run code that does
stuff to inventree.
A mis pick on the pnp machines consumes stock from that feeder,  a board
being finished runs a build order.  It would also be nice to see stencil
printer status so it can consume wiping paper, pcb panels on mis print and
maybe paste.
Seeing status of machines would also be good (waiting for upstream /
downstream,  reel empty error, oven not at temperatures etc). Would be nice
to know units per hour and other stats too. My air compressor is network
connected too so there's some out of line stuff as well.
On Tue, 12 Mar 2024, 5:59 am Matthias Mair, @.***>
wrote:

@loki42 https://github.com/loki42 would you be open to do a short
call/chat to understand your requirements better?
—
Reply to this email directly, view it on GitHub
https://github.com/inventree/InvenTree/issues/6486#issuecomment-1989215804,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAISQBE5WI6CZGRSURZRL6TYXX5H5AVCNFSM6AAAAABDINLJWOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBZGIYTKOBQGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

I've added my current integration of the PNP with Inventree here: https://github.com/polyeffects/poly_erp it's also got my code for Inventree sales orders to Xero invoices and a few other things. The next plan is to add shipping stuff using Karrio.


Keep initialized styling on -initialized class

This is a (multiple allowed):

[x] bug

[x] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 5.4.2

Platform/Target and Browser Versions: General

What you did
Added HTML markup for slider, to be initialized on some breakpoints (on desktop it has swiper markup, but it wont be initialized)
Expected Behavior
The swiper classes swiper-container, swiper-wrapper and swiper-slide has no/minimal styling, so non-initialized content won't behave like a bugged slider. All slider-related styles should then be attached on swiper-container-initialized - which in fact already gets added to the DOM.
Actual Behavior
The swiper classes swiper-container, swiper-wrapper and swiper-slide has slider-related styling. E.g swiper-container has some overflow styling, so even though the slider is not initialized, the slides are still overflown.

Swiper is designed to be used as initialised only! If you don't init it, remove those classes

This doesn't really resolve the issue where I want to initialize then slider on mobile, but not desktop. So it would be nice if I were able to keep the swiper markup ready for initialization.
Can't see how it should be a problem to best these styles in the initialized class?


Update for release 0.3.1 of libsemigroups

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Thanks @isuruf


chore: Update contributors across the project

Summary
This patch series includes some changes that are needed to address @Igarfinkle's comment concerning updating the contributors in places outside of the scope of the CODEOWNERS and current Active Maintainer section in our documentation.
I've decided to open this separately and remove the commits from #1968 as they're outside of the scope of the initial PR. This is currently in Draft as there may be other changes that need to be made to other files besides adding @Igarfinkle information.
Related Issues or PRs

#1968
https://github.com/trussworks/react-uswds/pull/1968#issuecomment-1092006258

How To Test
I'm not sure the best way to test this as I'm new to maintaining this project. If there's something I missed, please edit this comment or add an additional comment and I can edit this description.

Also, @Igarfinkle given the original intention of your changes here, should we be removing folks who are no longer at Truss from the .all-contributorsrc file as well?
diff --git a/.all-contributorsrc b/.all-contributorsrc
index fdfdf21..72c8758 100644
--- a/.all-contributorsrc
+++ b/.all-contributorsrc
@@ -111,15 +111,6 @@
         "code"
       ]
     },
-    {
-      "login": "christopherhuii",
-      "name": "Christopher Hui",
-      "avatar_url": "https://avatars.githubusercontent.com/u/8367504?v=4",
-      "profile": "https://github.com/christopherhuii",
-      "contributions": [
-        "code"
-      ]
-    },
     {
       "login": "jim",
       "name": "Jim Benton",

Agree - all contributors get to stay! In fact, I would vote we remove the contributors key entirely from the package.json and instead lean into the allcontributers file (which was added later on). It feels like there is unnecessary a duplication here.
Folks who contribute once are still contributors. Adjusting the CODEOWNERS when folks move on from Truss is the only thing we need to change.

I'm on the same page as @haworku here. I'm not aware of anything specific contributors in package.json gets us that allcontributors doesn't already convey. IMO, less duplication = less overhead.

I ran a force-push here to rebase my changes rather than merge them. But if that's not the right way to do things let me know. thanks!

@haworku sorry I think I dismissed the review you had made when I added the label 😕

Danger: Ignore "Do not make changes to package.json around contributors."

yeah @Igarfinkle I'm thinking of rebasing and removing those commits that adds things to package.json and see if that clears things up for Danger.

Danger: Ignore "Developer Specific file shouldn't be changed"

Danger: Ignore "Do not make changes to package.json around contributors."

@rogeruiz apparently there's an "allow a Fail just this once", which I'm trying to trigger

@Igarfinkle i wonder if this is supposed to be in my original PR body. You're free to edit my description for the PR.

@Igarfinkle i wonder if this is supposed to be in my original PR body. You're free to edit my description for the PR.

I tried adding it there first. Didn't seem to help, BUT it looks like danger just keeps referencing it's own first comment, so I edited that comment to remove the fail to see if danger will care/re-add it.

Nope, Danger, updated it's own comment, so the ignore is not working 😢

Trying again removing the  from the ignore message in the PR body in case that's messing with Danger's lookup


The Social Links

Describe the bug
The social links in the home tab beside ever social button there are button of up and down arrows
Expected behavior
I would like to work on it and like to remove them as they create a very bad user experience
Screenshots

Assign me this issue


Ability to add custom unicode characters to Quick Accent

Description of the new feature / enhancement
I have some unicode characters that I frequently copy paste from google. It'd be great to have ability to add let's say: U+0020, U+00A0, U+2007, ... as accents for regular space character. I think you get the idea.
Scenario when this would be used?
Sometimes we need to get some weird unicode characters that are not normally accessible through keybord for various reasons such as testing or just text styling. Having this ability baked into PowerToys would be so convenient I can't explain with words, since it also have ability to show the unicode name of the character when highlighted while selecting, making non-visibly differentiable characters easily differentiable.
Supporting information
There are LOADS of such apps floating on github, most of them are ok, but as said before having that functionality baked into PowerToys would be game-changing for some.

Hi I'm an AI powered bot that finds similar issues based off the issue title.
Please view the issues below to see if they solve your problem, and if the issue describes your problem please consider closing this one and thumbs upping the other issue to help us prioritize it. Thank you!
Open similar issues:

Add Quick Accent for whitespace characters (#24780),  similarity score: 0.79

Closed similar issues:

Ability to add custom accents (#29223),  similarity score: 0.81
Add unicode items of our chosing in Quick Accent (#24837),  similarity score: 0.76

Note: You can give me feedback by thumbs upping or thumbs downing this comment.

what's the point of showing me this if they are already a year old issues with no progress


Add statistics

Added statistics! Solves #38
More specifically, this adds functionality for the statistics button on Professor Bloom's home page. For gardeners, this will show their personal statistics and global statistics.
Global/personal statistics show:

All time welcomed
Welcomed this month
Welcomed this week to date (e.g. today is Wednesday, so it'll show Monday -> Wednesday)
Last 7 days (last Wednesday to this Wednesday)

Thanks!

@transcental pl review

Will review & test sometime in the next few weeks.


Require.ensure is awesome. But when I lazy load a huge javascript library, such as Kendo UI which is 3MB itself, it adds 10 seconds to my build time.
What I would like is to require.ensure a DLL bundle. I got my DLL bundle working fine, but webpack seems to expect to load it at runtime, and I havent figured out how to lazy load a DLL bundle.
I tried loading with script-loader
       require('script!../../../wwwroot/js/dlls/kendo.dll.bundle.js');
      SubscriberComponent.prototype.loadKendoUIGrid();

But kendo is undefined (which is normally set in the bundle)
I also tried scriptjs, which just appends the given script to the head at execution.
const loadScript = require('scriptjs');

      loadScript('/dlls/kendo_jquery.dll.bundle.js', () => {
        SubscriberComponent.prototype.loadKendoUIGrid();
    });

But kendo is still undefined.
If I do a normal require.ensure, it works perfectly
      // Async load KendoUI for jQeury, with webpack require.ensure
      require.ensure(['jquery'], function(require) {

         /** These are for the jQuery version of Kendo UI */
         // require('lib/kendoui/styles/kendo.material.min.css');
         require('../../lib/kendoui/styles/kendo.common.min.css');
         require('../../lib/kendoui/styles/kendo.default.min.css');
         require('../../lib/kendoui/js/kendo.web.min.js');
         require('../../lib/kendoui/js/kendo.core.min.js');
         require('script!../../lib/kendoui/js/kendo.grid.min.js'); // must pass through "script-loader"

         // Must call the prototype function, because 'this' is undefined inside the require.ensure during runtime
           HomeComponent.prototype.loadKendoUIGrid();

      }, "kendo.for.jquery") // 3rd parameter is the name of the chunk during compilation output - chunk.name.js

But again, it forces webpack to rebuild the same library every time.

Use the scriptjs approach. Try a different output.libraryTarget.

I tried all 6 of the options listed here and they didn't work for me. I got, "define is not defined", "module is not defined", "exports is not defined", or the same result as using default var "kendo is not defined".
Can you elaborate a little more about how to configure this correctly?

If that's the content of your dll bundle there is something incorrect with you dll build. The dll bundle should contain the webpack runtime.

I've been trying to do something similar. After digging a bit inside the bundled DLL file, I've come to understand that DLL bundles do not execute any modules by themselves. The execution of the first module in DLL bundle is being done (in my case) inside a progressive bundle.
I tried a diff of the bundle w/o DLL and with DLL. Here's what I found:
Bundle in normal build:
/* 0 */
/***/ function(module, exports, __webpack_require__) {

	__webpack_require__(1);
	module.exports = __webpack_require__(337);

/***/ },
/* 1 */

Same code in DLL bundled - manifest build:
/* 0 */
/***/ function(module, exports, __webpack_require__) {

	module.exports = __webpack_require__;

/***/ },
/* 1 */

And, somewhere in a progressively built bundle, I find this:
__webpack_require__(1)
I've some code in module 1 that helps me dynamically load different assets for my website. But, due to this, I've implemented a nasty workaround -- to load module 1 -- that I don't like at all.

If that's the content of your dll bundle there is something incorrect with you dll build. The dll bundle should contain the webpack runtime.

I separated the webpack runtime out of dll bundle into commons.bundle.js which is loaded before any dlls. Like this:
   <script type="text/javascript" src="js/commons.bundle.js.gz?8609335d43fa570a3897"></script> <--- (webpack runtime is here)
   <script type="text/javascript" src="dlls/polyfills.dll.bundle.js.gz?8609335d43fa570a3897"></script>
   <script type="text/javascript" src="dlls/vendors.dll.bundle.js.gz?8609335d43fa570a3897"></script>
   <script type="text/javascript" src="js/app.bundle.js.gz?8609335d43fa570a3897"></script>

How did you go about solving this? I'm facing a related (but fairly different) issue.
Have you tried placing your module imports, that are supposed to be resolved by the DllReference, inside a require.ensure block?

I have yet to solve this, and I do not know how to proceed at the moment. I'm hoping we can get some more input from a webpack wizard who can shine some light on this. Hint hint @sokra :-)

Related issue: #2592

I had the same basic problem which was due to loading order. Once I moved require.js, which defines 'define' and 'require', into the header I was able to require my modules using Webpacks libraryTarget of 'amd'.


SystemProfiler: ensure perf collects samples

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] I have updated the relevant documentation.
[ ] I have added tests for new logic.

this was split to 2 PR's


Copy-editing and fixed some typos

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Docs update

What is the current behavior? (You can also link to an open issue here)
There are some typos and English grammar that can be improved

What is the new behavior (if this is a feature change)?
I fixed the typos and improved the grammar

Other information:
I am happy to do the same to the other docs to improve readability, but I will wait until this PR is approved and merged to do so (in case the help is not wanted).

Thanks much @myadhdoutlet, any help with the project is much appreciated!


Colisions does not respect coordinate system

What happened?
The collisions seems to be offsetted when having a collision in different coordinate systems.
What do you expect?
Convert all positions to global ones to have the same coordinate system.
How can we reproduce this?
Create a collider inside camera.viewport and one in world
What steps should take to fix this?
Maybe use localToGlobal or something like this on all colliders
Do have an example of where the bug occurs?
https://dartpad.dev/?id=931319d09967e36386690e817afb0baa
click on run and then you see that it collides if the cursor is on the top left corner.
Relevant log output
No errors reported

Execute in a terminal and put output into the code block below
[✓] Flutter (Channel stable, 3.24.0, on Fedora Linux 40 (KDE Plasma) 6.10.4-200.fc40.x86_64, locale de_DE.UTF-8)
• Flutter version 3.24.0 on channel stable at /home/codedoctor/Tools/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 80c2e84975 (3 weeks ago), 2024-07-30 23:06:49 +0700
• Engine revision b8800d88be
• Dart version 3.5.0
• DevTools version 2.37.2
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
• Android SDK at /home/codedoctor/Android/Sdk
• Platform android-34, build-tools 34.0.0
• Java binary at: /home/codedoctor/.local/share/JetBrains/Toolbox/apps/android-studio/jbr/bin/java
• Java version OpenJDK Runtime Environment (build 17.0.11+0-17.0.11b1207.24-11852314)
• All Android licenses accepted.
[✓] Chrome - develop for the web
• CHROME_EXECUTABLE = /usr/bin/chromium-browser
[✓] Linux toolchain - develop for Linux desktop
• clang version 18.1.6 (Fedora 18.1.6-3.fc40)
• cmake version 3.28.2
• ninja version 1.12.1
• pkg-config version 2.1.1
[✓] Android Studio (version 2024.1)
• Android Studio at /home/codedoctor/.local/share/JetBrains/Toolbox/apps/android-studio
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 17.0.11+0-17.0.11b1207.24-11852314)
[✓] IntelliJ IDEA Ultimate Edition (version 2024.2)
• IntelliJ at /home/codedoctor/.local/share/JetBrains/Toolbox/apps/intellij-idea-ultimate
• Flutter plugin version 80.0.2
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
[✓] VS Code (version 1.92.2)
• VS Code at /usr/share/code
• Flutter extension version 3.94.0
[✓] Connected device (2 available)
• Linux (desktop) • linux  • linux-x64      • Fedora Linux 40 (KDE Plasma) 6.10.4-200.fc40.x86_64
• Chrome (web)    • chrome • web-javascript • Chromium 127.0.6533.99 Fedora Project
[✓] Network resources
• All expected network resources are available.
• No issues found!
Affected platforms
All
Other information
Also reported on discord: https://discord.com/channels/509714518008528896/1274083884451102762.
I would like to help with this but I have 0 idea how this all works. I'm currently using a workaround by moving the collider in the world space if it isn't interacting with camera viewport elements
Are you interested in working on a PR for this?

[ ] I want to work on this

Imo the collision is work as expected. All the collision checks are indeed performed in global coordinates space. It might be looking like an issue because the default settings of the camera renders the world's origin at the center of the viewport. If you just move the camera by half size of the viewport, it will look correct. Adding following line in the MyGame.onLoad, produces this output:
camera.moveTo(camera.viewport.virtualSize * 0.5);

https://github.com/user-attachments/assets/5ca81c76-9a6b-4fcd-9aca-08a739e6d6f7
Note: To make it work in the DartPad version, you might have to move the camera to size * 0.5 on the onGameResize.

On your end it looks okay. See the dartpad version for reference. This is the default behavior and for me as a user it's very confusing that with default rendering settings, the collisions are offsetted. Maybe this should get subtracted if the camera has this offset or at least be noted somewhere in the documentation.
I will try out your solution and reply if it works

Aaah right, @ufrshubham is correct of course. The docs should be improved and mention this though, I don't think they do now.

Hmm, I tested it myself but doesn't accept the solution right now.
My game looks like this:

You can move the camera by dragging with the mouse on the game table. Internally it uses moveBy to navigate through the world.
But when moving this way, the offset gets bigger and bigger. For me this makes no sense that these positions are moved with the world because it doesn't render like this. The camera position should be independent from the offset of the camera.
I think the camera offset should be considered in the collision system.

The camera position should be independent from the offset of the camera.

Do you mean that the viewport position should be independent of the viewfinder? Because it is...

But when moving this way, the offset gets bigger and bigger. For me this makes no sense that these positions are moved with the world because it doesn't render like this.

That makes sense, since all collisions will be in world coordinates, not in screen/viewport coordinates (or any other local coordinate system).

I can understand the confusion, but we cannot just make all the collision calculation based on the viewport/viewfinders configurations. For a multi-camera system it would mean we will have to perform the collision step per camera. An alternate approach to solve your problem is to add the cursor to the world and manually update its position based on actually mouse position and the camera's configuration.
class MyGame extends FlameGame with PointerMoveCallbacks, HasCollisionDetection {

  @override
  Future<void> onLoad() async {
    ...
    world.add(_cursor = Cursor());
  }

  @override
  void onPointerMove(PointerMoveEvent event) {
    _cursor.position = camera.globalToLocal(event.localPosition);
  }
}

@ufrshubham the problem he had from the beginning was that the cursor needed to collide with components both in the viewport and the world, which is a bit tricky without moving the cursor component between the world and the viewport, which was what I sugggested in the discord thread.

Can those component be added to the viewfinder instead? If I am not wrong, collisions with viewfinder.children will work as expected, right?

Yeah I want to collide with both the camera and the world.
I'm not an expert in flame but my setup looks like this:
FlameGame (which has CollisionDetection)
-> World
-> BoardGrid
-> GridCell (which has a passive, solid Hitbox)
-> Camera
-> Viewport
-> Positioned with custom painter (to have the black background)
-> Multiple HandItem (which spawns a cursor when starting dragging)
Now I want to check if the hand item when dragged should be put on a different cell or it should be moved inside the hand (to the front for example).
I don't know what you mean with viewfinder. If I added components there, it should be added in the background right?
If you have questions to my setup, my code is opensource and can be found here: https://github.com/LinwoodDev/Quokka/blob/bd191bece663326c4c28ad62d879bced2321b1cc/app/lib/board/game.dart

Can those component be added to the viewfinder instead? If I am not wrong, collisions with viewfinder.children will work as expected, right?

Those components are tied to the worlds coordinate system, because the camera has to be able to move over the different rows of cards.
To explain a bit more what he has:

A world (bigger than the viewport) where you can place cards, and each place that you can put a card has a hitbox so that he can detect it.
A viewport which holds the hand of cards, which should be static when the viewfinder moves around.
A mouse component that moves with the mouse which should be able to take a card from the hand and drop it in the world on one of the selected spots.

I'll close this for now since it is working as expected.

Hi, why is it now working as expected?
It collides with something which isn't there. And I'm currently not seeing a real solution here?
Having two collidable objects isn't a real solution here, since you need to handle both viewports manually.
An in-built solution could help maybe other users who want to create a "world-gui".

why is it now working as expected?

Because it is working according to how it is designed, I don't think it would be feasible to have the same collision detection root for both the world and the viewport.
When thinking about your specific problem, do you really need collision detection in the viewport? Isn't it enough to have them draggable and just check where it is dropped?

Hmm, okay...
Maybe this could a good "feature" since a cursor system could be useful for many games.
In my specific problem, I wanted to show an outline where you put it.
Additionally componentsAtPoint has a really poor performance on my case (I render ~ 1000 components on the screen)

But are you doing components at point only inside the viewport? Because surely you're not having 1000 components inside of that? Inside the world you should still be able to use the collision detection system.

No, I'm currently using it on the game object. Here the position works as expected. It uses the correct coordinate system on both viewport and world.
https://github.com/LinwoodDev/Quokka/blob/f40ea11070063c5c82f265919b6d07da5763ca98/app/lib/board/hand/item.dart#L206-L219

What I would do is to use the collision detection system for the world and then use componentsAtPoint for the viewport (if it's even needed, it looks like it would be trivial to calculate where in the row it is?).


Extend form docs with example form errors

Forms should show a error message whenever there is an error. This can be done nicely in bootstrap with the .text-help class. But until know people hat to figure that out by themselves.
This commit extends the example html with dummy error messages for the warning and danger cases. It also shows a case where a text-help paragraph is combined with an text-muted paragraph, wich requires an additional m-b-0 at the moment (see https://github.com/twbs/bootstrap/issues/18703).
In addition to that I extended the introduction to specify the .text-help class as the right way to show error messages in general. The .text-help is not explained in the form-docs yet, but it should.
This is what the new docs look like:

Hi @tordans!
Thanks for wanting to contribute to Bootstrap by sending this pull request!
Unfortunately, your pull request seems to have some problems:

Changes must be made to the original Sass or Less source code file(s), not just the compiled CSS file(s).

You'll need to fix these mistakes and revise your pull request before we can proceed further.
Once you've fixed these problems, you can either ask the maintainers to re-open this pull request, or you can create a new pull request.
Thanks!
(Please note that this is a fully automated comment.)


Fan with Light dimming not functional

Checklist

[X ] I have read the common issues wiki page
[ X] I have checked to make sure the plugin is up to date

Describe the bug
"devices": [
{
"type": "FanLight",
"name": "Bedroom Ceiling Fan",
"id": "id",
"key": "key",
"manufacturer": "NOPAARD",
"model": "60 inch ceiling fan",
"dpLightOn": 20,
"useBrightness": true,
"dpBrightness": 22,
"dpFanOn": 60,
"dpRotationSpeed": 62,
"fanDefaultSpeed": 8,
"maxSpeed": 100
}
]
With the above settings, the fan behaves as expected (0-100% speed operation), but the light dimming is non functional. The dimming slider shows full (like 100% brightness normally would) but the light sets itself to 10% brightness in reality. Home app reports the accurate 10% state in the summary/room view, but not the light detail view (where it reports 100%). If I ask Siri to set the light to 200%, it will set the light state to an actual 20%. See screenshots below
To Reproduce
The behavior is constant from start, and does not require steps to reproduce
Expected behavior
I expect to be able to set the light from 1% to 100% in 1% increments via the Home app and have that set the Fan Light bulb to the corresponding equal % state
Screenshots

Environment (please complete the following information):

OS: iOS
iOS version:
Homehubs: Homebridge
Node Version: 18.17.1
Plugin Version: 3.1.1
Accessory Type: Ceiling Fan with built-in dimmable LED Light bulb

Additional context
Add any other context about the problem here.

I have the same issue.
In homekit if I put 100%, it really puts at 10%.
But if I do from accesory tab of homebridge webui, I can put to 900, but if I put to 1000 it sticks to 900 (maybe the limit value is 999?).
Note that the values of brightness in tuya goes from 0 to 1000, in steps of 10, but in homekit goes from 0% to 100%.
I think there might be an scaleBrightness or something that multiplies the value from homekit per 10, and divide the value get from tuya by 10 to show it correctly.


mergesort in python

assign me this

add this in DSA/DSA-Python/Sorting Ago/..       directory

please labeled it hacktober fest accepted


Disable firestore integration tests temporarily

These tests are failing due to service-side changes
This mitigates #25851

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[x] Update CHANGES.md with noteworthy changes.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @Abacn

Java_GCP_IO_Direct back green. merge for now to get cron jobs back green


Routing by agreement with Transformer-based for NMT

Hello all :)
I’m trying to use Routing by agreement with TRANSFORMER-BASED for NMT task. The proposed idea is to use each output of head attention as an input capsule for a capsule network to fuse the semantic and spatial information from different heads to help boost the correction of sentence output. As below:

The implementation code is here, and Pytorch issue is here.
I have got so bad results. Kindly, I need and suggestion to work on.
I look forward to your feedback.

I just came here by random walk on Google.
The idea you proposed seems highly similar to this NAACL paper:
Li, J., Yang, B., Dou, Z.Y., Wang, X., Lyu, M.R. and Tu, Z., 2019, June. Information Aggregation for Multi-Head Attention with Routing-by-Agreement. In Proceedings of the 2019 Conference of the North American Chapter of the Association for Computational Linguistics: Human Language Technologies, Volume 1 (Long and Short Papers) (pp. 3566-3575).
If you're interested, I also have an NMT paper using capsules and a novel variant of dynamic routing to model better translation context.
Zheng, Z., Huang, S., Tu, Z., DAI, X.Y. and Jiajun, C.H.E.N., 2019, November. Dynamic Past and Future for Neural Machine Translation. In Proceedings of the 2019 Conference on Empirical Methods in Natural Language Processing and the 9th International Joint Conference on Natural Language Processing (EMNLP-IJCNLP) (pp. 930-940).

Yes sir, I have read both papers.
I have difficulty with the implementation and I'm looking for an example.


Update basic network related tests

What would you like to be added?

As per KEP-2578: Windows Operational Readiness Specification, the project needs to include Core.Network tests to ensure that a cluster can pass basic network related tests
As such, basic network tests need to be updated to ensure that they are conforming to the KEP specifications
Basic network tests were last updated in February 2022, and the following updates may be necessary

Update and identify new candidates for networking tests and ensure KEP is updated to reflect this
Ensure networking tests are successfully being executed successfully and where necessary, update upstream kubernetes e2e tests to ensure tests execute successfully

Why is this needed?

Refer to KEP-2578: Windows Operational Readiness Specification for motivation for the need for networking tests as part of the Windows Operational Readiness Conformance tests

/assign @pramitagautam

/assign @jayunit100

/assign @pramitagautam

/remove-lifecycle stale


Issue #11261: keep the search bar open

Reference:
Issue: https://github.com/adobe/brackets/issues/11261 - "keep the search bar open"
Based on this pull request that is waiting on the user: https://github.com/adobe/brackets/pull/11692 ( @adrianhartanto0 ). However I ended up taking a different approach; upon testing I found that the original approach had hiccups. The changes are influenced by the preference idea by @zaggino and @abose in the aforementioned pull request.
Changes:

Added in a menu option under View to allow find bar to stay open/automatically close
Add a close icon to the search bar (for completeness)
Fix escape key for in-editor searches
Remember last search phrase if no selection for in-editor searches
Go to the next search item if the bar is open and the user clicks Find or hits ctrl+F

Testing:

All tests pass for FindReplace-test.js after some changes to the code I wrote and one of the tests
Manual testing was briefly performed as such:

In-Editor - Find the same item by resetting the cursor position: Pass
In-Editor - Find a custom item by setting the cursor position: Pass
In-Editor - Find the first item by setting the cursor position after the last: Pass
In-Editor - Cycle through the items by hitting ctrl+F while still open: Pass
In-Editor - Closes when the "Automatically close search" option is checked and the editor is focused: Pass
In-Editor - Moving to a different editor closes the search: Pass
In-Editor - Pre-selecting text and re-opening the search shows the newly selected text: Pass
In-Editor - Pre-selecting nothing and re-opening the search shows the last query: Pass
In-Editor - Selecting the caps and regexp options functions as expected: Pass
View --> Automatically close search - Preference is remembered on application restart: Pass
In-File: Search functions as expected, and the same tests for In-Editor, where applicable, also pass.

:+1: Awersome PR!

Thanks @Denisov21 and @petetnt ! Will look into getting these fixed in the next few days :-)

Hey @petetnt (and/or @Denisov21) - I believe I've fixed all comments - what are the next steps on this pull request (assuming no further problems are spotted)? Thanks.

Made few small comments - haven't tried the implementation (yet)
//cc @swmitra @abose @nethip

@petetnt - I made it so that pressing ctrl+F or selection Find --> Find (in the top menu) will go to the next search result if the bar is already open. However! There is a 'Find Next' option in the Find top menu, so this new behaviour I added seems to be incorrect, and potentially confusing. What do you think?

Here was some discussion about similar thing here: https://github.com/adobe/brackets/pull/11869#issuecomment-153414305

Thanks, I think I'm going to undo that change I made to keep it in line with what has so far been done.

@petetnt - Should be ready to roll.

@pldilley Can you please resolve the the merge conflicts? I was trying to get the change list for some quick testing but saw the conflicts.
Please ping me once you have resolved.

@swmitra All resolved :)

LGTM (didn't retest the functionality after merging with master yet though, but I expected it to remain the same).

addresses https://github.com/adobe/brackets/issues/12280

Welp, how has this been left on the backburner for so long. Sorry about that @pldilley! If you could fix that one conflict so we can merge this in ASAP.
Sorry again!

@petetnt No problem, I have a bit of time this weekend, I'll retest it too to make sure that the behaviour hasn't changed.

Bump. Is anyone working on this PR now?

I'd appreciate this being incorporated if anyone has the time to iron out the conflicts

@ianshmean Totally agree with you. I hope to see this PR merged as well. @pldilley Can you please have a look at the review comments posted by @petetnt and incorporate the changes, if needed?.

Landed in #14141 with additional commit, thanks for your patience @pldilley, @ianshmean and @adrianhartanto0

P.S. Sorry I did not get to the test fixes. I hope this code change works out :)

Thanks @petetnt and @pldilley for your efforts.


make closeOnExit: true the default

This should take care of the easy part of #460.

 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.:x: lstefano71 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

where can i should start to help?


Ensure all methods in Context are mockable

Some methods in the Context class cannot be reliably mocked and therefore unit testing controllers is in many cases not possible.
The issue is, firstly, with all inline functions in Context as they cannot be mocked at all due to their inline nature. Inlining to the call-site and therefore getting called immediately.
Secondly, all methods with the same name and parameters as those inline functions. For reasons I could not figure out, they’re nondeterministically confused with one another. Using reified generics makes it possible in Kotlin to have a method with the same name and parameter in the same class. But one that makes MockK as well as Mockito confuse them. The issue could be Kotlin compiler, Java compiler, bytecode amendments by the Mocking frameworks.
The inline functions make for a great syntax but I’d value testability higher and therefore suggest removing the inline functions. At least they should be renamed to avoid the nondeterministic behavior when mocking.
Preliminary discussion is at #979.

Thanks for the summary @Zemke. For reference, this also relates to

https://github.com/mockito/mockito/issues/1943
https://github.com/mockk/mockk/issues/502
https://github.com/mockk/mockk/issues/55

Is there any way to work around this? Right now I see no way to write tests for code that use context (which is a lot of code). I suppose I could just hide context from my application by always wrapping it, but that's not idea.

https://github.com/tipsy/javalin/pull/1162 closes this.


TestOvsDriverDeinit occasionally fails

--- FAIL: TestOvsDriverDeinit (0.02s)
        ovsdriver_test.go:336: deinit failed. Error: %!s(<nil>) Output: _uuid               : 7b9c3193-a60e-49bb-a0fe-2fba400ad2ec
                controller          : []
                datapath_id         : "000092319c7bbb49"
                datapath_type       : ""
                external_ids        : {}
                fail_mode           : []
                flood_vlans         : []
                flow_tables         : {}
                ipfix               : []
                mirrors             : []
                name                : contivBridge
                netflow             : []
                other_config        : {}
                ports               : []
                protocols           : []
                sflow               : []
                status              : {}
                stp_enable          : false

Note that this only happens occasionally.

I believe this was fixed by adding retry logic in OVS bridge deletion.
Please reopen if you see it


Destroy transitions when resource id destroy

Fix this issue https://github.com/state-machines/state_machines-audit_trail/issues/14

@btrd  , Thank you for your PR.
I have some remarks about this change.
:destroy will load every single instance of the change and destroy them one by one.
If a record has thousands of changes , that could be a very expensive and slow process.
:delete_all will delete all the records with 1 SQL query.
The best approach i can think of now, is to delegate this to the database by have a constraint : on_delete: :cascade on the foreign key.
I still think we can add this feature if we can make it configurable.

The foreign key approach will work only in Relational databases. MongoDB will still need this IFAIK.


Draft: 3.1.3 from sdist

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

Jinja2 variable references are suggested to take a {{<one space><variable name><one space>}} form. See lines [9, 9].

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

Windows linker error is probably related to missing CMAKE_CXX_FLAGS (which was previously fixed by one of the git-source patches), although I don't know what flags are missing which, were they present, would prevent this ODR violation.
Still digging into it; it's a lot harder to debug server build problems in the sdist now that it relies on pulling perspective-server's sources from crates.io. Especially painful on Windows which doesn't have as nice of recipe debugging experience as Mac/Linux.

Got a lead by dumping CMAKE_CXX_FLAGS{,_DEBUG,_RELEASE} from a test cmake file in a Windows conda environment.
-- CMAKE_CXX_FLAGS_DEBUG: /MDd /Zi /Ob0 /Od /RTC1
-- CMAKE_CXX_FLAGS_RELEASE: /MD /O2 /Ob2 /DNDEBUG

without the patch, perspective's CMakeLists.txt resets the value of CMAKE_CXX_FLAGS_RELEASE to the empty string, which would mean the build is not getting  /Ob2, an MSVC flag which controls inlining behavior: https://learn.microsoft.com/en-us/cpp/build/reference/ob-inline-function-expansion?view=msvc-170
Planning to submit a fix to perspective to:

Delete the reset of CMAKE_CXX_FLAGS_RELEASE
Change functions in utils.h to be static inline which should prevent ODR to a C++ language lawyer's satisfaction

Will resubmit when 3.1.4 is released with perspective-server fixes


Workflows for creating releases with docs

Related #890
Motivation
This PR introduces two GitHub Actions Workflows:

create-release.yaml
rebuild-releases.yaml

create-release.yaml
This workflow runs anytime a tag is created that matches "effection-v*". It'll checkout the tag, generate docs with deno doc, create a tarball and create a release.
rebuild-releases.yaml
This workflow can only be triggered manually. It can be used to rebuild all of the releases. It has two jobs: setup & rebuild. The setup job gets a list of all tags that match the "effection-v*" pattern and sticks the result into outputs. The rebuild job uses outputes from setup as matrix. The actual workflow is same as create-release.yaml
TIL

GitHub has a new mechanism for managing tokens generated by users in an organization. If you want to interact with an organization's resources, you'll need to create a token specifically for that organisation. There are additional configuration options under TheFrontside github org settings
You can have dynamic matrixes in GitHub Actions at the bottom of the blog post there are good examples like this one
act -s GITHUB_TOKEN="<YOUR_TOKEN>" -W .github/workflows/rebuild-releases.yml workflow_dispatch can be used to run rebuild-releases.yaml on your machine

TODO

[x] How do I test this?
[x] How do I run this for all existing tags?

Using a release is a good idea and it might be easier. I'll do that

Do we need the rebuild releases? At the most, it seems like we would need them for 3.0.0, 3.0.1, and 3.0.2

I guess that's true. I was thinking in case we want to update the releases to a specific format. Like now, I need to rerun, to rebuild at least 3.0.0, 3.0.1 and 3.0.2. In the future, there will be more. I can change the filter to only include non-preview releases.
What do you think?

Do we need the rebuild releases? At the most, it seems like we would need them for 3.0.0, 3.0.1, and 3.0.2

I guess that's true. I was thinking in case we want to update the releases to a specific format. Like now, I need to rerun, to rebuild at least 3.0.0, 3.0.1 and 3.0.2. In the future, there will be more. I can change the filter to only include non-preview releases.
What do you think?

I'm torn, on the one hand, I feel like releases are releases, and that means a binary artifact corresponding to a specific git SHA. On the other, you're right. What if we need to update urls, or change style?
In that case, maybe it's best to have an executable task in the repo that we can do manually?

In that case, maybe it's best to have an executable task in the repo that we can do manually?

I will need to rewrite the job. The setup I have allows us to rerun the job via GitHub UI. Do you want me to rewrite it to Deno?


Gh 803 improve kryo serialisation in spark library2

Codecov Report

Merging #836 into develop will increase coverage by 0.02%.
The diff coverage is 91.83%.

@@              Coverage Diff              @@
##             develop     #836      +/-   ##
=============================================
+ Coverage      58.74%   58.77%   +0.02%
- Complexity      2669     2678       +9
=============================================
  Files            489      492       +3
  Lines          12220    12237      +17
  Branches        1341     1341
=============================================
+ Hits            7179     7192      +13
- Misses          4453     4457       +4
  Partials         588      588

Impacted Files
Coverage Δ
Complexity Δ

.../serialisation/kryo/impl/EntityKryoSerializer.java
100% <100%> (ø)
3 <3> (?)

...rk/serialisation/kryo/impl/EdgeKryoSerializer.java
100% <100%> (ø)
3 <3> (?)

...q/gaffer/spark/serialisation/kryo/Registrator.java
100% <100%> (ø)
2 <1> (ø)
:arrow_down:

...ation/kryo/impl/HyperLogLogPlusKryoSerializer.java
73.33% <73.33%> (ø)
3 <3> (?)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 50198c7...997a79c. Read the comment docs.


Implementation of "Sort" feature

Description
This is an initial implementation of the Sort feature.
It currently supports a Date type sort param.


facebook is not supported .. how to download videos from facebook then ?

Hello ,
I was trying to download an important personal  video from a previous facebook account that has been hacked via "youtube-dl" under "ubuntu 15.04" .
However , each time I put the link of the video I get the 'Unsupported URL' error .
What's the best solution to solve this problem ?

facebook IS supported...what is the URL?

Here is  a snapshot of my terminal output :

help if possible .

Post (as text, not an image) the output you get with the --verbose option.

this downloads OK...you might need to update your YouTube-DL version
C:>youtube-dl -v "https://www.facebook.com/RB2000ps/videos/vb.139448822740695/9
91016994250536/?type=2&theater"
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'https://www.facebook.com/RB2000ps/videos/vb
.139448822740695/991016994250536/?type=2&theater']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2015.05.20
[debug] Python version 2.7.8 - Windows-7-6.1.7601-SP1
[debug] exe versions: ffmpeg N-71346-gdf4fca2
[debug] Proxy map: {}
[facebook]<PHONE_NUMBER>50536: Downloading webpage
WARNING: unable to extract title; please report this issue on https://yt-dl.org/
bug . Make sure you are using the latest version; type  youtube-dl -U  to update
. Be sure to call youtube-dl with the --verbose flag and include its complete ou
tput.
[debug] Invoking downloader on u'https://video-sea1-1.xx.fbcdn.net/hvideo-xta1/v
/t42.1790-2/11199838_1435726020074183_158452998_n.mp4?efg=eyJybHIiOjQxMCwicmxhIj
o1MTJ9&rl=410&vabr=228&oh=de32f286a6714101075aa92dd88ee6b9&oe=55917C79'
[download] Destination:      ..       !-991016994250536.mp4
[download] 100% of 1.35MiB in 00:15

Still can't download from my side.
I can't figure out the problem either .. so here's a screenshot of my terminal while running the "youtube-dl"  command on that same URL , below.

You are using an outdated version of youtube-dl. Read our FAQ if you have troubles updating.

Sadly I've updated it to the latest version via the command specified .. but I don't know why I'm still getting an outdated version. I dont' think this is from my side.

youtube-dl in Ubuntu repos is usually outdated.

ok thanks for the support , I'm installing through the curl link then .
On Tue, Jun 30, 2015 at 4:16 AM, Sergey M<EMAIL_ADDRESS>wrote:

youtube-dl in Ubuntu repos is usually outdated.
—
Reply to this email directly or view it on GitHub
https://github.com/rg3/youtube-dl/issues/6121#issuecomment-116931047.


Text width

Make a CSS class (text-block) for controlling the width of long-form text on the website, set this to 62 characters, and apply it to tabs containing text.

LGTM