Feature: add subcommand to convert an existing QGIS profile into a QDT config

Codecov Report

Merging #269 (57cb293) into main (d27ba93) will decrease coverage by 0.02%.
The diff coverage is 64.51%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #269      +/-   ##
==========================================
- Coverage   64.70%   64.68%   -0.02%
==========================================
  Files          35       36       +1
  Lines        2037     2067      +30
  Branches      429      430       +1
==========================================
+ Hits         1318     1337      +19
- Misses        605      616      +11
  Partials      114      114

Impacted Files
Coverage Δ

qgis_deployment_toolbelt/profiles/qdt_profile.py
76.59% <33.33%> (-1.93%)
:arrow_down:

qgis_deployment_toolbelt/commands/extract.py
65.00% <65.00%> (ø)

qgis_deployment_toolbelt/cli.py
85.71% <100.00%> (+0.93%)
:arrow_up:

qgis_deployment_toolbelt/commands/__init__.py
100.00% <100.00%> (ø)


Error: query failed: synchronization failed: Operation timed out (Mac OSX 10.10.4, 0.8)

Running projects under Example folder (following the steps in the docs) starts the app but shows the following red screen after the initial launch screen:

Looking for JS files in  /Users/henry/Code/react-native
React packager ready.

[18:01:19] <START> fs crawl
[18:01:26] <START> request:/Examples/Movies/MoviesApp.includeRequire.runModule.bundle
[18:01:26] <START> find dependencies
Error: query failed: synchronization failed: Operation timed out

at BunserBuf.<anonymous> (/Users/henry/Code/react-native/node_modules/sane/node_modules/fb-watchman/index.js:93:23)
at emitOne (events.js:77:13)
at BunserBuf.emit (events.js:169:7)
at BunserBuf.process (/Users/henry/Code/react-native/node_modules/sane/node_modules/fb-watchman/bser.js:289:10)
at /Users/henry/Code/react-native/node_modules/sane/node_modules/fb-watchman/bser.js:244:12
at doNTCallback0 (node.js:408:9)
at process._tickCallback (node.js:337:13)

However, this React Native app, https://github.com/iSimar/HackerNews-React-Native, runs and starts just fine.
Watchman has been installed per brew and is version 3.7.0:
{  "error": "invalid command (expected an array with some elements!)",
    "cli_validated": true,
    "version": "3.7.0" }

This may sound really lame but i did get this working by just restarting the packager once.

Restarting the packager didn't work for me. However, restarting my computer did. Thanks @chirag04

glad it worked for you. This could be watchman or some cache. @ide ?

Same error here with Node 4.1.0, Watchman 3.7.0, react-native 2.14.3 on my Macbook pro yosemite.
detailed errors:
react packager output:
[10:14:05 PM] <START> request:/index.android.bundle?platform=android
[10:14:05 PM] <START> find dependencies
Error: query failed: synchronization failed: Operation timed out

    at BunserBuf.<anonymous> (/Users/path_to_user_and_app/testapp/node_modules/react-native/node_modules/sane/node_modules/fb-watchman/index.js:93:23)
    at emitOne (events.js:77:13)
    at BunserBuf.emit (events.js:169:7)
    at BunserBuf.process (/Users/path_to_user_and_app/testapp/node_modules/react-native/node_modules/sane/node_modules/fb-watchman/node_modules/bser/index.js:289:10)
    at /Users/path_to_user_and_app/testapp/node_modules/react-native/node_modules/sane/node_modules/fb-watchman/node_modules/bser/index.js:244:12
    at doNTCallback0 (node.js:407:9)
    at process._tickCallback (node.js:336:13)
Error: query failed: synchronization failed: Operation timed out

    at BunserBuf.<anonymous> (/Users/path_to_user_and_app/testapp/node_modules/react-native/node_modules/sane/node_modules/fb-watchman/index.js:93:23)
    at emitOne (events.js:77:13)
    at BunserBuf.emit (events.js:169:7)
    at BunserBuf.process (/Users/path_to_user_and_app/testapp/node_modules/react-native/node_modules/sane/node_modules/fb-watchman/node_modules/bser/index.js:289:10)
    at /Users/path_to_user_and_app/testapp/node_modules/react-native/node_modules/sane/node_modules/fb-watchman/node_modules/bser/index.js:244:12
    at doNTCallback0 (node.js:407:9)
    at process._tickCallback (node.js:336:13)
[10:17:41 PM] <START> request:/index.ios.bundle
[10:17:41 PM] <START> find dependencies
Error: query failed: synchronization failed: Operation timed out

    at BunserBuf.<anonymous> (/Users/path_to_user_and_app/testapp/node_modules/react-native/node_modules/sane/node_modules/fb-watchman/index.js:93:23)
    at emitOne (events.js:77:13)
    at BunserBuf.emit (events.js:169:7)
    at BunserBuf.process (/Users/path_to_user_and_app/testapp/node_modules/react-native/node_modules/sane/node_modules/fb-watchman/node_modules/bser/index.js:289:10)
    at /Users/path_to_user_and_app/testapp/node_modules/react-native/node_modules/sane/node_modules/fb-watchman/node_modules/bser/index.js:244:12
    at doNTCallback0 (node.js:407:9)

output from watchman log: (username.log)
2015-09-20T22:12:13,892: [client:fd=4] path /Users/peter/sites/private/mobile/react/testapp is on filesystem type hfs
2015-09-20T22:12:17,894: [client:fd=31] send_error_response: fd=31 failed to parse query: must use ["allof", expr...]
2015-09-20T22:13:36,708: [notify /Users/peter/sites/private/mobile/react/testapp] kFSEventStreamEventFlagRootChanged /Users/peter/sites/private/mobile/react/testapp, cancel watch
2015-09-20T22:14:36,778: [client:fd=4] sync_to_now: /Users/peter/sites/private/mobile/react/testapp/.watchman-cookie-PeterPro.lan-1254-6-4 timedwait failed: Operation timed out
2015-09-20T22:14:36,778: [client:fd=4] send_error_response: fd=4 query failed: synchronization failed: Operation timed out

Reinstalling everything didn't help, removing and recreating the project neither, restarting idem
Redoing everything in node 4.0 instead of 4.1.0 works neither.
Android emulator, my nexus 5 and ios emulators give the same problem
anyone here any idea what I can do to get stuff working=

Try killing the watchman daemon or restarting your computer.

@ide, did not working.

upgrading to watchman 3.8.0 seemed to do the trick

https://github.com/facebook/react-native/issues/2191#issuecomment-143528115

same error running the Hello World example with node 4.21 and watchman 3.9.0.
can't find any solution so far... :(

Not sure if this is applicable for anyone else, but I was having the same issue.  Had tried upgrading / uninstalling and reinstalling watchman with no luck.
It's a little strange, I'm using Laravel Homestead for some other projects, which is basically a pre-packaged Vagrant box, during the set up for that you map a directory to the vagrant box in my case ~/Code. Which is where I had my react native tutorial code so ~/Code/AwesomeProject
I was able to get rid of this error by moving my code base out of that directory, that is being mapped, into a different directory.
Everything seems to be working fine now and I'm not too sure why that was causing an issue...
Hopefully that is useful for someone.  Good luck!

Same issue here, I am running the UIExplorer example inside react-native project

Same issue here, watchman 4.6.0 and OS X 10.11.2. Tried reinstall, restart, reboot computer...

I Did what facebook recommends us to do !
watchman shutdown-server
brew update
brew reinstall watchman
brew didn’t want to reinstall watchman, did chown and I did
sudo chown root /usr/local/bin/brew
react-native run-ios
Build succeeded,
but still got the dependency graph error and Watchman not running )
jmals-MacBook-Pro:~ ajmal$ ps -ef | grep watchman
501 5268 1 0 10:28PM ?? 0:00.95 /usr/local/Cellar/watchman/4.6.0/libexec/bin/watchman --foreground --logfile=/usr/local/var/run/watchman/ajmal-state/log --log-level=1 --sockname=/usr/local/var/run/watchman/ajmal-state/sock --statefile=/usr/local/var/run/watchman/ajmal-state/state --pidfile=/usr/local/var/run/watchman/ajmal-state/pid
0 5364 1 0 10:28PM ?? 0:00.01 /usr/local/Cellar/watchman/4.6.0/libexec/bin/watchman --foreground --logfile=/usr/local/var/run/watchman/root-state/log --log-level=1 --sockname=/usr/local/var/run/watchman/root-state/sock --statefile=/usr/local/var/run/watchman/root-state/state --pidfile=/usr/local/var/run/watchman/root-state/pid
501 5530 5502 0 10:39PM ttys000 0:00.00 grep watchma
So, Watchman is running in the foreground, where may be lying the ISSUE ?????

I Did what facebook recommends us to do !
watchman shutdown-server
brew update
brew reinstall watchman
brew didn’t want to reinstall watchman, did chown and I did
sudo chown root /usr/local/bin/brew
react-native run-ios
Build succeeded,
but still got the dependency graph error and Watchman not running )
jmals-MacBook-Pro:~ ajmal$ ps -ef | grep watchman
501 5268 1 0 10:28PM ?? 0:00.95 /usr/local/Cellar/watchman/4.6.0/libexec/bin/watchman --foreground --logfile=/usr/local/var/run/watchman/ajmal-state/log --log-level=1 --sockname=/usr/local/var/run/watchman/ajmal-state/sock --statefile=/usr/local/var/run/watchman/ajmal-state/state --pidfile=/usr/local/var/run/watchman/ajmal-state/pid
0 5364 1 0 10:28PM ?? 0:00.01 /usr/local/Cellar/watchman/4.6.0/libexec/bin/watchman --foreground --logfile=/usr/local/var/run/watchman/root-state/log --log-level=1 --sockname=/usr/local/var/run/watchman/root-state/sock --statefile=/usr/local/var/run/watchman/root-state/state --pidfile=/usr/local/var/run/watchman/root-state/pid
501 5530 5502 0 10:39PM ttys000 0:00.00 grep watchma
So, Watchman is running in the foreground, where may be lying the ISSUE ?????


how to replace default Redactor with TinyMCE ?

in version 1.12.7 Redactor is default, but i want to use free tinymce(with image uploading if possible) in my project.
how can i do that without "badcode"?
Thanks for answers :)

See: https://github.com/comfy/comfortable-mexican-sofa/wiki/Replacing-default-wysiwyg-editor-with-redactor
It applies for replacing editor with whatever you want. I need to change Wiki entry though.
TinyMCE doesn't have image manager out of the box. This is something they sell as an add-on. So I never had to integrate image uploading using it. I'm sure it's not that hard. Might need to override a few things.

I think the path for the file he has to overwrite has changed from /app/assets/javascripts/comfortable_mexican_sofa/admin/application.js.coffee to /app/assets/javascripts/comfy/admin/cms/custom.js.coffee in a recent commit.

Disclaimer: I am using cms-fortress
Thanky you very much @GBH for including Redactor. This is really awesome.
First, a question
I got this working so far, as I replaced TinyMCE. So now I see the Redactor editor. YAY ;-). But either uploading an image or a file does not work. The JS console says when clicking the "Insert image" button:
GET http://localhost:3000/cms-admin/sites/1/pages/13/undefined?
source=redactor&type=image&_=1424561594889 404 (Not Found)

Choosing an image for upload is for sure also not working:
POST http://localhost:3000/cms-admin/sites/1/pages/13/undefined?source=redactor&…
Bo5O6qGhreDo2Om7Ql3K6BZgtcaDMF3ZJdEiCFWkKJ61D4PFk39tOz0KSGReK%2FHdEw
%3D%3D 404 (Not Found)

I am not sure what the value for undefined should be so I debugged the JavaScript a bit and found out, that CMS.file_upload_path is undefined. I am wondering where I have to define that?
Secondly,
I assume that the images will be uploaded in the same directories server side when using the Redactor editor as when I upload them in the Upload section directly. Correct?
Thanks you very much!

UPDATE:
My analyses with CMS.file_upload_path was correct. I changed it back to:
imageUpload:      $('.cms-files-modal').data('iframe-src') + "?source=redactor&type=image&#{params}"
imageManagerJson: $('.cms-files-modal').data('iframe-src') + "?source=redactor&type=image"
fileUpload:       $('.cms-files-modal').data('iframe-src') + "?source=redactor&type=file&#{params}"
fileManagerJson:  $('.cms-files-modal').data('iframe-src') + "?source=redactor&type=file"

and now it is working. I placed everything in app/assets/javascripts/comfy/admin/cms/custom.js.coffee ...

It's this file: https://github.com/comfy/comfortable-mexican-sofa/blob/master/app/views/layouts/comfy/admin/cms/_head.html.haml
If it's replaced by cms-fortress, that would explain missing js variables

@GBH perfect! Thanks for the tipp. This is now running.
I have some ideas for improving the Image and FileManager popup. I guess it's best to open a new issue to discuss it?

There's already one in progress: #593


[WIP] Add generation for GREYInteraction

This PR moves most iOS calls over to generated code (detox_selectElementWithMatcher is the only missing one and it's a bit of a special case.

Is this still a WIP?

Yes, but close to being finished

I now rebased this PR. The original goal was more ambitious, but I think incremental steps forward are a nicer way to work together 🙈 I think this might help with #618 maybe only by us having a conversation about what these instance methods mean for us in detox 👍

Of course this needs to pass e2e before merge (from travis log)
     Error: Error: target is invalid
      at Client.execute (/Users/travis/build/wix/detox/detox/src/client/Client.js:75:13)
      at <anonymous>

@rotemmiz I refactored the things you mentioned and got a working version up (let's see if CI agrees). I could only finish one of the calls, but left TODO items on the others. They "only" need a bit of fine tuning, but I thought this progress might already be helpful for you, especially in regards of #618

Awesome! Will we be able to generate non static invocations now ? Can't see an example in the new generated code, can you please point me to it ?

Hey @DanielMSchmidt , anything to be done on your side ?

Not yet, trying to get to it next weekend

@rotemmiz This is done now 👍 Let's get it into master ;)

@rotemmiz ping, could you review this PR? 🙏 😇

@rotemmiz Addressed your comments / replied. Thank you for taking the time to review <3

Fun fact: This PR moves us from 47 to 40 open .call invocations

💪

@DanielMSchmidt , I'm getting a weird behaviour when running ci.ios.sh locally.
generation script generates something a bit different and breaks tests. Can you please take a look ?

I now get it, I switched to EarlGrey 1.13, there is some breakage, not sure what.

@LeoNatan, NS_REFINED_FOR_SWIFT macro was added to a few of EarlGrey's functions.
What should we do if we want to invoke this code through DTXMethodInvocation? should we just ignore it in the method signature ?

Refined means there is a better alternative for Swift. It has no influence on ObjC.

So
method: "performAction:"
not
method: "performAction:NS_REFINED_FOR_SWIFT"
right?

Yes, of course.
It's an attribute.
# define CF_REFINED_FOR_SWIFT __attribute__((swift_private))


Issues with building Apache-OpenOffice on FreeBSD

Currently editors/openoffice-4 cannot be built with either tmpfs or ccache enabled.
1- synth caps tmpfs to 12Gb which is plenty for most ports but insufficient for OpenOffice. The build requirements for this port are 11Gb of disk space and 2Gb of memory. With synth this results in a 'no space left on device' error late in the build if tmpfs is enabled. (I have 8Gb memory + 17Gb of swap space configured on this system running FreeBSD 11.1-RELEASE)
2- This port also have its own 'CCACHE' option (off by default). Enabling this option works fine with poudriere, but with synth it results in a failure early in the build phase. (I have ccache enabled in synth)

on the second issue, I'd clear the ccache directory.  After that, I would not be surprised to see ccache work fine with apache openoffice.  Occasionally ccache uses previous work that should be considered invalid, and I think that's what is happening with you.

on the first issue, you can try rebuilding synth with a patch to file src/replicant.adb, line 1083
change integer from "12" to something higher.
However, the last time I checked, 12 is greater than 11, so in theory 12Gb allocation for work directory should be sufficient.  Obviously the 11Gb value is wrong.

On the first issue, my understanding is that the tmpfs requirement would be 11 + 2. I'm going to patch this as you suggested and report back later.
For the second issue, I already tried to fix it by clearing the cache but that didn't solve the issue.

no, the 2G memory requirement doesn't count against tmpfs

on ccache issue, I'd just check that you didn't clear your default ccache directory instead of the ccache directory specified by the synth configure screen.  that's easy to do if you forget to set CCACHE_DIR In the environment for the command.

I was able to build apache-openoffice with a tmpfs cap of 16Gb. The usage of tmpfs in the work area grows to almost 15Gb at the end of the build phase.
Concerning ccache, I confirm that the build still fails even with a clean cache directory.
I think there's a problem with the port. CCACHE_DIR is set to /ccache but there's this errror  the log file:
ccache: error: Failed to create directory /root/.ccache/tmp: Operation not permitted
dmake:  Error code 1, while making '../unxfbsdx.pro/obj/ifparser.obj'

well, the /root directory is read only, so that would be the error.
I'm not sure how the port can influence how ccache works though.
Ports are not allowed to write to $HOME or /root though, so that's an issue with the port.

I've created a PR for this port. https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=224276
Thanks for the help

the tmpfs cap of 16G is only required for the tmpfs_workdir right?
meaning 12G is still okay for tmpfs_localbase.  I assume that's the case.

Yes, that is correct. The 16Gb is only for the work area.

wait, I just looked up your PR and the port itself.
This is insane.
You don't set CCACHE on at the port level.  You set it on at the poudriere / synth level.
So the port authors are doing something extremely non-standard.
However, it should not affect you.  You just leave the CCACHE option off and set ccache on at the synth configure screen.
I misunderstood before.  I thought you were using ccache for all ports (as most people do).
So to iterate:

make sure inane CCACHE option is OFF for openoffice
set up ccache properly at the synth level.

That should solve the issue.

I use ccache at the synth level, but this port won't honor that (it's the same thing with poudriere).   Surely only simple code is needed to detect ccache is being used and the makefile can then adjust CONFIGURE_ENV according (option should be needed here)
The "CCACHE_DIR=/ccache" in /etc/make.conf is being ignored, so this hack the openoffice port maintainers have come up with isn't complete.

Yes, this looks wrong, but I don't think these 2 files are used anymore. They're not referenced in any of the makefiles and svn log says it's some old support scripts from the OpenOffice-3 days.

well, delete them and see if openoffice still builds.
That will let you know if they are used or not. :)

put it this way: if they are used, they will definitely break the build.

I just completed a build without those files, so that answers the question.

might be worth recommending their removal in the PR then.   Nobody needs cruft.
Regardless, something similar is causing the breakage: ccache is being invoked without overriding the default definition of CCACHE_DIR.  And it's not in the man makefile, because CCACHE_DIR is defined in the jails /etc/make.conf file.

Actually, I've figured out a fix. CCACHE_DIR need to be exported to the shell environment prior to calling build.pl in the Makefile.
I'll do some tests and add a patch to the PR later today. I'll also request that they drop the ccache option entirely and just follow WITH_CCACHE_BUILD.

FYI, the issues were fixed in the FreeBSD ports tree with r459465 and r460047.

The ccache option has been removed and the port now follow the framework settings.
The port now builds without the crashdump option and this reduces tmpfs usage to 8Gb.

So I'm closing this ticket.


feat: implement rich text widget

the rich text widget should be able to display formatted static text. The supported syntax should html, markdown or similar

editorjs is rich enough for now


Authorization error

Hello,
When trying to reach this endpoint :
https://api.kraken.com/0/public/Assets
I have this error :
{"error":[{"severity":"E","errorClass":"Auth","type":"Authorization error"}]}
Here is my server code :
var host = '<IP_ADDRESS>';
var port = 7000;
var cors_proxy = require('cors-anywhere');
cors_proxy.createServer({
removeHeaders: ['cookie', 'cookie2'],
//requireHeader: ['origin', 'x-requested-with']
//setHeaders: {'origin':'https://api.kraken.com','refer':'https://api.kraken.com', 'Authorization': 'Basic'}
}).on('request', function (req, res) {
var date = new Date();
console.log('/' + req.method + " on "+req.url +" - "+date.toLocaleString());
var ip = req.headers['x-forwarded-for'] || req.socket.remoteAddress;
console.log("FROM " + ip)
}).listen(port, host, function() {
console.log('Running CORS Anywhere on ' + host + ':' + port);
});
BR,
Franck

This response is from kraken, not my library.
Without having wasted more time at looking into the specific issue, my guess is that your client-side code didn't specify the right credentials in the request header.


cleanup(build): enable cmake CMP0135 policy

Enabling this changes the behavior of timestamps for downloaded/extracted file timestamps, but more importantly avoids a ton of cmake warnings during build.

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area API-version

/area build

/area CI

/area driver-kmod

/area driver-bpf

/area driver-modern-bpf

/area libscap-engine-bpf

/area libscap-engine-gvisor

/area libscap-engine-kmod

/area libscap-engine-modern-bpf

/area libscap-engine-nodriver

/area libscap-engine-noop

/area libscap-engine-source-plugin

/area libscap-engine-savefile

/area libscap-engine-udig

/area libscap

/area libpman

/area libsinsp

/area tests

/area proposals

Does this PR require a change in the driver versions?

/version driver-API-version-major

/version driver-API-version-minor

/version driver-API-version-patch

/version driver-SCHEMA-version-major

/version driver-SCHEMA-version-minor

/version driver-SCHEMA-version-patch

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: gnosek
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [gnosek]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: gnosek, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [gnosek,jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: d953f22aca40743303bb42c00d4a43cbb7e6c4ce

Hi @gnosek ! Can you rebase?

@gnosek can we rebase this?

yeah, yeah, you don't need to remind me every month :D
seriously though, thanks for the ping, just pushed it (even though it's a clean rebase with no conflicts)

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, gnosek, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,gnosek,jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 1591bcd1b2189cc5877a5a27a5f434992da7baf3

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, gnosek, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,gnosek,jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@Andreagit97 wanna unhold?

/unhold


Tag  7.4.0 points to the wrong commit

https://github.com/dependency-check/dependency-check-gradle/compare/v7.3.2..v7.4.0 shows no changes
Tag shows no commits since 7.3.2: https://github.com/dependency-check/dependency-check-gradle/commits/v7.4.0
Tag is "This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository."
please force push v7.4.0 as this commit: https://github.com/dependency-check/dependency-check-gradle/commit/93114b9e98c8f29763ae4d74b46d88e6ef574afe

No - the 7.4.0 tag points to the actual release code. As documented in #295 there was a mistake during the release. 7.4.1 will be released shortly.

Note: if there were no changes in 7.4.0 then it's equivalent to 7.3.2, so by semver you should increment 7.5.0. but I guess then you'll get out of sync with the core library.


【SSL設定】証明書の作成とインストール

Macでは以下の設定で認証局介さない証明書をテスト用に発行するが、同じような操作をopensslを使って行うDockerfileを用意したい。
$ brew install mkcert
Running `brew update --auto-update`...
==> Auto-updated Homebrew!
（省略）

$ mkcert localhost
Created a new local CA 💥
Note: the local CA is not installed in the system trust store.
Note: the local CA is not installed in the Firefox trust store.
Note: the local CA is not installed in the Java trust store.
Run "mkcert -install" for certificates to be trusted automatically ⚠

Created a new certificate valid for the following names 📜
 - "localhost"

The certificate is at "./localhost.pem" and the key at "./localhost-key.pem" ✅

It will expire on 9 July 2025 🗓

$ mkcert -install
Sudo password:
The local CA is now installed in the system trust store! ⚡
Warning: "certutil" is not available, so the CA can't be automatically installed in Firefox! ⚠
Install "certutil" with "brew install nss" and re-run "mkcert -install" 👈
The local CA is now installed in Java's trust store!

本番用の設定は別途必要。


Webpack 4

Close #619

I think this is not relevant.
https://github.com/acdlite/recompose/blob/master/.size-snapshot.json#L16-L19


Removing comments icon WRONG sizing .css classes + Typo fixes

Comments icon already has sizing coming from 'src/definitions/elements/icon.less' this rules here overwrite it and display WRONG comments icon sizing.

I've fixed this in next, please check readme for guide on how to contribute a PR for this project in future ( should be next branchandsrc/` files


Overdocumented var

Fails with:
Error: CiviForm environment variables are not correctly documented. See <placeholder> for documentation instructions. Issues:
The following vars are documented but not referenced in /home/runner/work/civiform/civiform/server/conf/application.conf:
{'MY_NEW_VAR'}


Javascript filename missing variable references.

What steps will reproduce the problem?
Use copyright or character tags. (possibly others).
What is the expected behavior? What do you get instead?
a string of the copyrights or characters.
How often does this problem occur? On which sources?
Always, latest release.
What version of the program are you using? On what operating system?
5.5.1
Windows 10
Please provide any additional information below
Javascript I am using:
search_1 + '/' +
copyright+'/' +
character + '/' +
score + '' +
md5.slice(0,4) + '' +
id + '_' +  '.' +
ext
Javascript I am attempting to use (end goal):
search_1 + '/' + (
copyright.split('+').contains(search_1) ?
copyright.split('+').filter(function(x){
return x!== search_1 && x.indexOf(search_1) >= 0 ;
}):
'misc/'
) +
character + '/' +
score + '' +
md5.slice(0,4) + '' +
id + '_' +  '.' +
ext

Indeed, I could reproduce the missing variables in Javascript.
I just pushed a fix to add them back. And I also saw you doing copyright.split('+') so I added a copyrights variable for an already-split array. It should land in nightly soon, or you can wait for the 5.5.2 release. :smile:

I'll have to wait as I can't seem to get it to build.. -.-

I'll have to wait as I can't seem to get it to build.. -.-

If you're on Windows, the nightly built successfully and is available in the downloads automatically.
On Linux, I just built it 1h ago, so I don't think the build is broken (both CI are green after all).

Also thanks for the pre-split array. does it use whatever delimiter is specified in the settings?

For the moment, because of some old bad design issues, I'd say yes. But in the future it should simply be the "unmerged" array instead of "merged then split again", therefore the delimiter won't be relevant at that time.


[Build] Support sanitizers on all platforms

What
Support asan, and potentially ubsan sanitiers on macOS and Windows equivalents.
Why
We presently only support these on Linux. It's possible that there may be platform specific issues.

Sanitizers on other platforms would help devs who prefer those platforms; and might catch very platform-specific issues. But it's hard to justify spending the time to implement this, given the small amount of problems that wouldn't get caught on a linux build.


[AIRFLOW-3085] Bug fix to allow log display in RBAC UI

Make sure you have checked all steps below.
Jira

[x] My PR addresses the following Airflow Jira issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-3085

Description

[x] Here are some details about my PR, including screenshots of any UI changes:
Currently due to a change in method name, none of the roles beside Admin is capable of viewing logs in Airflow. Log viewing should be part of of the default rbac configuration for all roles (except public).

Tests

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

Commits

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

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

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[ ] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

Codecov Report

Merging #3916 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3916      +/-   ##
==========================================
- Coverage   77.55%   77.53%   -0.02%
==========================================
  Files         198      198
  Lines       15847    15847
==========================================
- Hits        12290    12287       -3
- Misses       3557     3560       +3

Impacted Files
Coverage Δ

airflow/www_rbac/security.py
91.27% <ø> (ø)
:arrow_up:

airflow/jobs.py
82.48% <0%> (-0.27%)
:arrow_down:

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 2f50083...16d3c23. Read the comment docs.

Codecov Report

Merging #3916 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3916      +/-   ##
==========================================
- Coverage   77.55%   77.53%   -0.02%
==========================================
  Files         198      198
  Lines       15847    15847
==========================================
- Hits        12290    12287       -3
- Misses       3557     3560       +3

Impacted Files
Coverage Δ

airflow/www_rbac/security.py
91.27% <ø> (ø)
:arrow_up:

airflow/jobs.py
82.48% <0%> (-0.27%)
:arrow_down:

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 2f50083...16d3c23. Read the comment docs.

LGTM!


ath9k-htc-blobless-firmware: @SuperSandro2000's suggestions

Description of changes
https://github.com/NixOS/nixpkgs/pull/208685#issuecomment-1380922562
Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Okay, @SuperSandro2000, I think I have implemented five of your nine comments.  Regarding the remaining four:

For your last two comments I need some clarification before proceeding
For your third comment asked why I had two update attributes; if my explanation is satisfactory please resolve the issue.  If not, please explain further.
Regarding your sixth comment, I have explained the situation here

Note that this PR also includes one additional tiny change: the unstable package changed from:
pname = "ath9k-htc-blobless-firmware";
version = "unstable-2022-05-22";

to
pname = "ath9k-htc-blobless-firmware-unstable";
version = "2022-05-22";

This made the logic slightly simpler.


Fix tmt import --dry and Polarion import file name

tmt import --dry was never respected, let's fix that
Also fixed polarion import filename to use default main.fmf when there is only a single test case and name is not forced with argument --polarion-case-id
Pull Request Checklist

[x] implement the feature

/packit test --identifier full

/packit test --identifier full

Looks good, thanks for adding that fix as well.

/packit test -i full


UAT: Case Filing - verify Identity

During filing a case, there is an option to upload documents for identity verification. need to relook at the format supported, like why are we supporting TXT format?
Why are we not supporting PNG/JPG etc?

@atulgupta2024 As per the figma we have added the extensions, if you want we can change the supporting format. Please confirm.

@Taherabharmal please comment on the file types. I don't see any TXT should be supported. PNG/JPEG/JPG should be supported.


Merge isValidForm and isValidatingForm

@christianalfoni  In the helpers folder isValidForm and isValidatingForm only has one diff on line 12. Maybe we could remove one of them and merge?

Or pull out the isValidArray as a function.

That sounds like a good idea :) Let me add my latest contribution here as well. Pushing it now

@christianalfoni Just reviewed the code and it is still there. This is not an issue because the code works just fine. Should I close it?

@edgesoft Leave it open. We need to clean it up at some point


Resumable upload - incorrect behavior if first chunk fails and needs resuming

When no bytes have yet been uploaded to the server, and the uploaded length is requested, the server returns no "Range" header at all (probably because a zero-length result is not possible to represent with the inclusive range of a Range header)
This is not handled properly in the ResumableUpload class.

Fixed


Vumi Bridge transport can't access Vumi Go over HTTPS

Vumi Go's SSL certificate relies on a weak (SHA1) intermediate certificate. This fails to be verified by a lot of server side software (including Twisted) but is accepted by current browsers. Vumi Go's certificate will be updated in October at the latest, but until then the bridge transport needs to accept weaker certificates.

@JayH5 @imsickofmaps Ready for review.

And @jerith ^^^

@morgs Since this involves SSL, I thought I'd ask you whether you think it's okay to use the weaker CA certificates for now?

:+1: although we should test it against some real HTTPS servers before releasing.


Undefined names can raise NameErrors at runtime

flake8 testing of https://github.com/mkocaoglu/CausalGAN on Python 2.7.13
$ flake8 . --count --select=E901,E999,F821,F822,F823 --show-source --statistics
./causal_dcgan/utils.py:147:16: F821 undefined name 'x'
        return x.astype(np.uint8)
               ^
./causal_dcgan/utils.py:149:18: F821 undefined name 'x'
        return ((x+1)/2*255).astype(np.uint8)
                 ^
./figure_scripts/sample.py:157:51: F821 undefined name 'list_labels'
    for name, lab, dfl in zip(model.cc.node_names,list_labels,list_d_fake_labels):
                                                  ^
./figure_scripts/sample.py:157:63: F821 undefined name 'list_d_fake_labels'
    for name, lab, dfl in zip(model.cc.node_names,list_labels,list_d_fake_labels):
                                                              ^
./figure_scripts/sample.py:442:47: F821 undefined name 'nsamples'
            outputs[k]=np.vstack(outputs[k])[:nsamples]
                                              ^
./figure_scripts/sample.py:507:41: F821 undefined name 'nsamples'
                if len(completed_rows)>=nsamples:
                                        ^
./figure_scripts/sample.py:585:40: F821 undefined name 'nsamples'
                outputs[k]=outputs[k][:nsamples]
                                       ^
./figure_scripts/sample.py:592:32: F821 undefined name 'nsamples'
            values=outputs[k][:nsamples]
                               ^

@cclauss  I'm not familiar with flake8. It seems to check for style consistency. I don't doubt that it would throw an error.
Does this result in a concrete problem I can help with?

--select=E901,E999,F821,F822,F823

These five errors have nothing to do with style.  These are about running the code through the Python abstract syntax tree process looking for syntax errors and other inconsistencies that can halt your program at runtime.
The first error is saying that the xis not defined in this context so the code is dangerous.  This can be seen by doing:

$ cd causal_dcgan/utils.py
$ python -c "import utils ; utils.make_gif(None, None)"  # NameError raised


Fix KeyError when removing a non-existent command/plugin

Checklist

[x] I have run nox and all the pipelines have passed.
[ ] I have made unittests according to the code I have added/modified/deleted.

I forgot to remove those redundant if statements by the way

I'll sort it dw

Alrighty


Copying changes made in PR https://github.com/Qiskit/qiskit-ibmq-provider/pull/1102

…

Summary
Copying changes made in PR https://github.com/Qiskit/qiskit-ibmq-provider/pull/1102
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1756

Relevant Lines:
4691

💛  - Coveralls


Add basic auth

This should fix #4

Nice. Please use const headers instead of var myHeaders and fix indentation (use tabs).


update-path not working

Describe the bug
tenv update-path is not including tenv in the path output.
To Reproduce
Steps to reproduce the behavior:

tenv tf use 1.7.5
tenv update-path

Expected behavior
I'd expect the output of tenv update-path to include the path to Terraform so that executing terraform results in the correct binary being invoked.
Screenshots
❯ tenv version
tenv version v1.7.1

❯ tenv tf use 1.7.5
Terraform 1.7.5 already installed
Written 1.7.5 in /Users/xxxx/.tenv/Terraform/version

❯ tenv update-path |grep tenv

❯ which terraform
terraform not found

Environment (please complete the following information):

OS: MacOS 14.4.1
tenv version 1.7.1

Additional context
Ideally this is like other *env tools where I can "inject" the path programmatically.  Obviously I could do something like to inject the path myself:
export PATH="/Users/xxxx/.tenv/Terraform/$(cat /Users/xxxx/.tenv/Terraform/version)/:$PATH"

However, that is not as clean as having the tool do this as ideally the tool should adjust pathing, for example if I change projects which have differing Terraform versions.

How did you install tenv ?
which terraform should indicate the proxy version installed with tenv

export PATH="/Users/xxxx/.tenv/Terraform/$(cat /Users/xxxx/.tenv/Terraform/version)/:$PATH"

That's not the correct way to use tenv, you should add to your PATH the location of tenv project which include 5 binaries : tenv to manage the tools, and the proxies tofu, terraform, terragrunt and tf which are used to call the version of the tool corresponding to your configuration.

I'm installing using binaries provided by the releases:
❯ curl -Ls -o tenv.tar.gz https://github.com/tofuutils/tenv/releases/download/v1.7.1/tenv_v1.7.1_Darwin_arm64.tar.gz
❯ tar -xvzf tenv.tar.gz -C /Users/xxxx/.local/bin tenv
x tenv
❯ rm tenv.tar.gz

❯ which tenv
/Users/xxxx/.local/bin/tenv

Installation using the release binaries allows parity between developer environments and our pipelines...
I'd assume my shell needs to be configured such that tenv is able to "present" the correct version of Terraform.  Other *env tools I've used in the past use shims or other mechanisms, but based on the README, I read it as the tenv update-path was the way to do this.  As it stands, nothing is manipulating my shell.

lol, and looking in the tarball I now see where I might have misstep..  😵💫
❯ tar -tvzf tenv.tar.gz
-rw-r--r--  0 runner docker      0 Apr  9 03:05 CHANGELOG.md
-rw-r--r--  0 runner docker  11349 Apr  9 03:05 LICENSE
-rw-r--r--  0 runner docker  46446 Apr  9 03:05 README.md
-rwxr-xr-x  0 runner docker 12683682 Apr  9 03:09 tenv
-rwxr-xr-x  0 runner docker  9101618 Apr  9 03:10 tofu
-rwxr-xr-x  0 runner docker  9084114 Apr  9 03:10 terraform
-rwxr-xr-x  0 runner docker  8677538 Apr  9 03:10 terragrunt
-rwxr-xr-x  0 runner docker  9119330 Apr  9 03:10 tf

I suspect, the terraform binary provided by the release is some sort of "redirect" to tenv and able to pick up the various versioning information it provides?

Yep, that's exactly what it's doing.. https://github.com/tofuutils/tenv/blob/main/cmd/terraform/terraform.go

The tenv update-path have been requested as one feature among those making easier to use tenv with GitHub Actions.
More globally that command can help when the original tool appear in PATH before the corresponding proxy, when :
$ which tenv
/Users/xxxx/.local/bin/tenv

Then it should look like :
$ tenv update-path
/Users/xxxx/.local/bin:$PATH

So export PATH=$(tenv update-path) would place the tenv location first, ensuring the proxy is found before any corresponding tool.
However, in most case you doesn't need it (I don't think many system have one of the managed tool installed by default), when using tenv installing the tool in other way seem strange...

Maybe we need to make that clearer in the README.

While it's not exactly built-into to the system, some popular docker-oriented images like Alpine Linux includes Terraform.
Or at least they used, until Hashicorp's license change.


CI: Use new syntax for setting github output

What does this PR do?
Same as https://github.com/Lightning-AI/lightning/pull/15415.
weekend chore 🧹
Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[n/a] Did you make sure to update the documentation with your changes?
[n/a] Did you write any new necessary tests? [not needed for typos/docs]
[n/a] Did you verify new and existing tests pass locally with your changes?
[n/a] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review

[x] Is this pull request ready for review? (if not, please submit in draft mode)

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Hi, @akihironitta - Thanks for your PRs to Flash. The CI should be fixed once #1479 is merged. Post that, let's get this one merged as well. Thank you for your patience.


new topic about node.js

There was an error while validating job 67459890-2ce3-466c-a1d3-3624b3a15dc4 Message:
Result status: Failed
There was a problem accessing Github repository 'https://github.com/Microsoft/cordova-docs.git'.
Details:Microsoft.LightweightJobEngine.Core.Language.ContextCommandBase+ContextCommandException:
Result status: Failed
There was a problem accessing Github repository 'https://github.com/Microsoft/cordova-docs.git'.
---> System.AggregateException: One or more errors occurred. ---> System.ArgumentException: Error clonning repository ---> LibGit2Sharp.NotFoundException: Branch 296 not found. Available branches: origin/master
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.GetBranch(Repository repo, String name)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.<>c__DisplayClass30.<>c__DisplayClass34.b__29(KeyValuePair2 branch)    at System.Collections.Generic.List1.ForEach(Action1 action)    at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.<>c__DisplayClass30.<CloneRepository>b__28()    at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.<>c__DisplayClass1.<ExecuteAction>b__0()    at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.ExecuteAction[TResult](Func1 func)
at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.ExecuteAction(Action action)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository(String url, IDictionary2 branches)    --- End of inner exception stack trace ---    at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository(String url, IDictionary2 branches)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository()
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.GetVersionAsync(String path)
at Microsoft.DPS.Commands.IdentifyDocumentsToPublish.CheckExternalServicesCommand.Do(IDictionary2 context)    --- End of inner exception stack trace ---    --- End of inner exception stack trace ---    at Microsoft.LightweightJobEngine.Core.Language.ContextCommandBase.ThrowIfErrorsLogged()    at Microsoft.DPS.Commands.IdentifyDocumentsToPublish.CheckExternalServicesCommand.Do(IDictionary2 context)
at Microsoft.LightweightJobEngine.Core.Language.JobWorkflow1.RunCommands(Guid jobId, IDictionary2 context, CancellationTokenSource cancelSignal)


Add Constraints

Added Constraints classes.
I could not do RadialConstraint because it requires lambda and while_loop  support.

Hi @JimClarke5 , looks like there is now a conflict in this PR. Also, is there any preference of order for review between this PR and the Regularizers #198 ?

@karllessard Constraints and Regularizers should be a independent of each other and all other framework packages. I’ll check the conflict.

Closing this PT to link to a new branch built off the latest master checkin.


WebGL: Invalid Operation errors on second map

Hi,
Sorry we've been a few dot releases behind in our testing and just catching up.  In our application we have two maps, one on a dashboard page (occupying only part of the window) that's created on startup.  There's a second, full window map that's created when the user selects that tab.  We create it on tab selection so Leaflet correctly obtains the window dimensions and builds the map properly.  This has been working fine up to and including 0.11.0.
We tried installing 0.11.4 today and it has some problems (and turns out all the way back to 0.11.1).  When we try and substantiate the second map now we get a flood of WebGL INVALID_OPERATION errors mainly complaining about 'the object does not belong to this context'.  Do you have any idea how we can create the second map now as we really want to keep Tangram updated.
Thanks

Ah, thanks. It's likely a result of optimizations to reuse GL shader and program objects. These are bound to specific canvas contexts. I thought these were being reset, but this may be a case where not.
To clarify, Tangram (unfortunately) currently doesn't properly support displaying multiple maps at the same time on the same page. It is however valid to switch between maps on the same page (remove/replace one with another). Is that what you are doing here?

Brett,
Our app has a number of tabs with hidden content as a single page application.  The first tab is a dashboard, which has the initial map created at startup.  When the user clicks on the map tab, the dashboard div is hidden and the map div is shown.  This is where the second map is created in the dom.  The two can never be shown at the same time but both end up existing in the dom.  Hope that helps and thanks for being so responsive as always.
Cheers
Rob

Hey @rgillan, could you give this build a try and let me know if it resolves the issue?
https://precog.mapzen.com/tangrams/tangram/context-id/dist/tangram.debug.js

Brett,
Fantastic, solved the problem.
Cheers
Rob

Great, will include it in the next release (in the next day or two).

Fix in https://github.com/tangrams/tangram/commit/00d0f15c34ecd16b0a7b06450dcfa6115c5859ab

Fix was released in v0.11.5.


Contact

Hello, me and a friend of mine are developing an advanced PC Growtopia trainer in C#, and seeing your knowledge in proxies and in the past the C# GrowBrew proxy we would need help with it.  We are wondering if you'd be interested. If so, contact me (preferrebly) via Discord, username+tag is nukkkiz#4400. If you can't contact through Discord, you can contact me some way you can. Let me know as a response, please. If you choose to decline you can just close this issue (or preferrably comment on this that you aren't interested :D)

Hello nukiz! i am sorry, i am still little knowledge about proxies, i dont know i can help you or not

and i am sorry, i am not interested

That's sad, glad for the response though. Have a nice day!

That's sad, glad for the response though. Have a nice day!

im sorry 🙏🏽


Android Basics: ViewModel

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-training-viewmodel#11
In which task and step of the codelab can this issue be found?
Step 12
Describe the problem
The textview that display the score on the screen is never updated. The score text is only changed on onViewCreated method from the fragment.
Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version:
API version of the emulator:
Additional information
Include screenshots if they would be useful in clarifying the problem.

Maybe this could be a temp solucion for this codelab:
 private fun onSubmitWord() {
        val playerWord = binding.textInputEditText.text.toString()
        if(viewModel.isUserWordCorrect(playerWord)) {
            setErrorTextField(false)
            binding.score.text = getString(R.string.score, viewModel.score) // **Adding this line**
            if(viewModel.nextWord()){
                updateNextWordOnScreen()
            }else{
                showFinalScoreDialog()
            }
        }else {
            setErrorTextField(true)
        }
    }

@evolquez
Yes that is right!
https://developer.android.com/codelabs/basic-android-kotlin-training-viewmodel#10


Generating a Gradle project without adding any extra extensions adds duplicate entry in build.gradle

After generating a Gradle project without any extra extensions added using the Quarkus: Generate a Quarkus project command:

In the newly generated project, implementation 'io.quarkus:quarkus-resteasy' appears twice:

It should only appear once.
This issue probably happens because the RESTEasy JAX-RS extension (the required extension) exists in state.extensions here: https://github.com/redhat-developer/vscode-quarkus/blob/f38f4caaf218cf9c6ce91e64a0d9cd632314a483/src/wizards/generateProject/ExtensionsPicker.ts#L150

I can't reproduce this issue; it must have been fixed upstream, either in the Quarkus Gradle extension or the Quarkus project generator service/website. I am looking for the corresponding PR.


Bump kafka-clients to 2.3.0

Codecov Report

Merging #77 into develop will not change coverage.
The diff coverage is 25%.

@@            Coverage Diff             @@
##             develop      #77   +/-   ##
==========================================
  Coverage      82.92%   82.92%
  Complexity       298      298
==========================================
  Files             80       80
  Lines           1031     1031
  Branches          11       11
==========================================
  Hits             855      855
  Misses           169      169
  Partials           7        7

Impacted Files
Coverage Δ
Complexity Δ

...fka/configuration/KafkaContainerConfiguration.java
89.28% <25%> (ø)
8 <0> (ø)
:arrow_down:

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 c36d904...4b7402d. Read the comment docs.


fix!: upgrade miku to net7 & fix issues

Depends on https://github.com/Sekoree/NicoNicoNii/pull/2

LGTM

LGTM

Working with #59 instead


Adding CI support for examples

I updated the project at https://codeship.com/projects/103513 to use @bfosberry's GitHub authentication, so we should see PR status from now on.

Is there any way for us to use the services and steps files from the examples directly and not having to duplicate all the information into the top level codeship-services.yaml and codeship-steps.yaml files? I fear that they will get out of sync otherwise.
Should we add the top level codeship.aes to .gitignore?
And what about missing examples (e.g. 06, do we have plans to add those as well)
Other than that, awesome work and :shipit:

There is currently no way to use child steps and services files, we should add the key and tmp, and 6 looks like it runs for me, what do you see?

You're right, example 6 is running fine, I didn't see that it was using the containers from example 5.
Examples 12 and 13 don't exist (which is what I was going to ask next, as I mostly looked at the services and steps file).
All the rest is :+1:

Not sure why those examples dont exist :P

:shipit:


License discussion

are we sure MIT license is the best ?

As per our discussion, I think we all agreed that as our package is open source and is created for the wider use of the public, we want the license that allows people to freely use, adapt, and build on our source code. In that sense MIT license, I think suits us the best.
The other licenses to consider are

GPL: gives freedom to the users to use, build on, adapt, share, but any derivative work must be distributed under the same license. For example, they cannot use our code and make it not open source.
BSD: MIT license is compatible with BSD two clause license with an attribution clause. The other versions of BSD, namely three clauses, and four clause ones, I think are irrelevant for us, because our package is simple and our intentions were to make it as publicly available as possible, thus the commercial use is actually even desirable. Thus keeping it simple and without restrictions is my suggestion.
MIT: grants free access of usage of our package, as long as attribution is given when used.

I believe, as per our discussion, MIT still remains the best license for us. Let me know, what you think guys, and do you confirm the initial thoughts and discussions. @imtvwy @vtaskaev1 @johnwslee

I definitely agree. I believe that MIT license is the one that fits the purpose of our package best.

I agree we should keeping using the MIT license.
The vision of our project is to allow more people to use, build and contribute on our package.   Both MIT and BSD license should fit our purpose.  A bit research on the difference between the two is MIT license has explicitly included the documentation files in the license as "this software and associated documentation files" , while BSD includes "source and binary forms" only
Given that and both license are simple for the users to use, I will opt for MIT license.


How to improve the scanning speed of PDF417?

How to improve the scanning speed of PDF417?

If you haven't already, try the latest version of ZXingObjC as it contains some PDF417 improvements. Other than that though, I'm afraid I can't support PDF417 beyond porting changes from ZXing or reviewing pull requests.


Container export doesn't save world

Hello, I was trying to export container and importing in another machine.

run commands:
docker run -d -e EULA=TRUE -e TZ=Europe/Rome -p 25565:25565 --name mc itzg/minecraft-server

export command:
docker export -o minecraft_2020_06_28_01_04.tar mc

I then scp the file to another machine (save OS, same docker version)
docker import minecraft_2020_06_28_01_04.tar minecraft_bacup:1.0

Run command
docker run -d --restart unless-stopped -e EULA=TRUE -e TZ=Europe/Rome -p 25565:25565 --name mc minecraft_bacup:1.0 /start
(With command as last argument either ways it would work from start)

First error: type cannot be ""  while reading the TYPE value.
Ok, I override this with: -e TYPE=VANILLA
Second error: java memory value error... Ok, I override this too with: -e MEMORY=2G
Now the run commands is:
docker run -d --restart unless-stopped -e EULA=TRUE -e TZ=Europe/Rome -e TYPE=VANILLA -e MEMORY=2G -p 25565:25565 --name mc minecraft_bacup:1.0 /start
It runs! I connect and... this is not my world. Or my spawn is somewhere totally different from before?
I still have the original server and would like to know of a safe way to export this, because even beeing familiar with docker and haveing done this before with other containers (without any issue) I really can't understand.
This is the docker inspect of Imported Backup Image:
[
    {
        "Id": "sha256:53835f3d130e43a8ba81ab6c4bf4961fd7659a7118b4c30b41353bbc632e97a3",
        "RepoTags": [
            "minecraft_bacup:1.0"
        ],
        "RepoDigests": [],
        "Parent": "",
        "Comment": "Imported from -",
        "Created": "2020-06-28T10:56:30.421321746Z",
        "Container": "",
        "ContainerConfig": {
            "Hostname": "",
            "Domainname": "",
            "User": "",
            "AttachStdin": false,
            "AttachStdout": false,
            "AttachStderr": false,
            "Tty": false,
            "OpenStdin": false,
            "StdinOnce": false,
            "Env": null,
            "Cmd": null,
            "Image": "",
            "Volumes": null,
            "WorkingDir": "",
            "Entrypoint": null,
            "OnBuild": null,
            "Labels": null
        },
        "DockerVersion": "19.03.12",
        "Author": "",
        "Config": {
            "Hostname": "",
            "Domainname": "",
            "User": "",
            "AttachStdin": false,
            "AttachStdout": false,
            "AttachStderr": false,
            "Tty": false,
            "OpenStdin": false,
            "StdinOnce": false,
            "Env": null,
            "Cmd": null,
            "Image": "",
            "Volumes": null,
            "WorkingDir": "",
            "Entrypoint": null,
            "OnBuild": null,
            "Labels": null
        },
        "Architecture": "amd64",
        "Os": "linux",
        "Size": 294076806,
        "VirtualSize": 294076806,
        "GraphDriver": {
            "Data": {
                "MergedDir": "/var/lib/docker/overlay2/f3460ebd26e8664863ccc7a2febd019e6700358a21d979fab92979022956528d/merged",
                "UpperDir": "/var/lib/docker/overlay2/f3460ebd26e8664863ccc7a2febd019e6700358a21d979fab92979022956528d/diff",
                "WorkDir": "/var/lib/docker/overlay2/f3460ebd26e8664863ccc7a2febd019e6700358a21d979fab92979022956528d/work"
            },
            "Name": "overlay2"
        },
        "RootFS": {
            "Type": "layers",
            "Layers": [
                "sha256:876c289ea6080fe331c8b2e8feb052b842a8935860594039490b60f9191b8dfc"
            ]
        },
        "Metadata": {
            "LastTagTime": "2020-06-28T12:56:30.427913204+02:00"
        }
    }
]

Comparing it to the original one, the differences are huges, and this never happened before to me.
Am I doing something wrong or is this a issue with Minecraft server docker?
Thanks and sorry to bother. Your work is AWESOME.

docker export does not include volumes and /data, which is where all of the minecraft content is maintained, is a volume:

Volume attachment of /data to a host directory would have been the easiest way to transport the content from one host to another; however, since you're using the auto-volume creation you can instead use docker cp to copy the /data directory out of the old container, move that directory to the new host and attach that directory as the data volume.

I'm used to create whatever I want inside a container, export and import somewhere else and find the new things I created.
Is /data  somehow different?
I will use external volume from now on to let me move this thing easily.
Thanks for the answer!

You can also find all your docker volumes under /var/lib/docker/volumes.

@raikoug again, /data is only "different" because it is declared as a volume:
https://github.com/itzg/docker-minecraft-server/blob/bbe930e55203f80a0cdf4791ce3a8f4ac04c6cd9/Dockerfile#L64


[2.x] Add unsetCache method

New method encapsulates cleanup logic in separate method.

Might be worth to write a test for it if you have some spare time.

@Flyrell sorry, I have real problems with spare time right now. Working hard to launch closed beta of the new startup project until 2020.

@antonkomarev Sure, I guess we're both in the same place 😄 I will try to write them when I have some time.

Closing and moving it to #42.


Created toxic-npcs plugin details

Plugin Name
Toxic NPCs on Player Death
Overview
Heavily inspired by IdylRS and his 'bad plugins' idea for toxic npcs that will be toxic to the player if they die.
When a nearby player dies, all nearby npcs will display overhead text that makes fun of them for dying.
Depending on the npc that displays overhead text, there is unique dialogue that will display for them, i.e. God Wars Dungeon bosses and Jad.

We have denied plugins similar to this before https://github.com/runelite/runelite/issues/12683 https://github.com/runelite/plugin-hub/pull/704


Update dtabs to use globs

Remove uses of io.buoyant.http.any* namers in favor of finagle's new wildcard support.

:star: 🎉 📇

I want to test part of this change on the kubernetes demo -- mind hold off on shipping for a few mins?

:star: 👣

⭐ 🔧


Move access and opening times into offices and organisations

Note: This runs a data migration automatically from the db/migrate folder. We discussed this, and it seems to be the way we usually perform this kind of migration. This means a user might lose their edit if the migration runs while they are doing work, but we think this is unlikely!

https://trello.com/c/NFkJQYZw
Remove unused FormObject model
This FormObject model is not used anywhere (since #5965) and can safely be
removed.
It includes properties that were specific to the feature for which it was
introduced.
Add access and opening time columns to worldwide organisation and office
We want to remove the access_and_opening_times table to make it easier to
move the rendering of Worldwide Organisations and Worldwide Offices out of
Whitehall.
Create migrations to add new columns to worldwide_organisations and
worldwide_offices. Work will follow to use these instead of the
access_and_opening_times table.
Use access and opening times columns
We have added columns to worldwide_organisations and worldwide_offices so
that we can remove the access_and_opening_times table to make it easier to
move the rendering of Worldwide Organisations and Worldwide Offices out of
Whitehall.
This replaces the use of the AccessAndOpeningTimes model with the new
attributes of the WorldwideOrganisations and WorldwideOffices. There is no
difference in functionality with this change.
Remove AccessAndOpeningTimes model
This class is no longer used, and the table associated with the model is to be
removed.
Migrate Access and Opening Times data
We have added new columns to Worldwide Organisations and Worldwide Offices so
that we can remove the access and opening times table.
Copy the data over to the new columns. This specifically uses update_columns
instead of update so that ActiveRecord callbacks are skipped. Some of the
callbacks are not desired here, for example, we do not want to create an audit
trail entry.
The nested AccessAndOpeningtimes class allows us to use the ActiveRecord query
interface despite the fact that the model has been removed from the app.
Add pending migration to drop the access_and_opening_times table
We have added new columns to Worldwide Organisations and Worldwide Offices so
that we can remove the access and opening times table.
Add a pending migration to drop the access_and_opening_times table.

⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
Follow these steps if you are doing a Rails upgrade.

Unless I'm missing something, the first commit ("Remove unused FormObject model") seems quite orthogonal to the other changes in this PR. Although it's quite small, I think it would be better to move it into a separate PR. Also probably worth linking to the commit where it was last being used, which I think is this one: 2669db5.
I found the 3rd commit ("Use access and opening times columns") quite hard to follow - there's a lot going on. While I think the use of the model form is a significant improvement; I wonder if it could've been used to split the commit up, by introducing the model form in one commit without making any changes to the underlying models/database and then changing the model form to use the new models/database in another commit. I haven't thought this through in detail so it may not be practical and in any case I don't think it's worth re-working now; just something worth bearing in mind.
In the 4th commit ("Migrate Access and Opening Times data") I'm concerned that the AccessAndOpeningTimes model will have been removed from the codebase before the migration runs as per the 6th commit ("Remove AccessAndOpeningTimes model"). I'm not sure what the regular practice is in Whitehall/GDS, but a more typical way to handle this would be to either (a) define the required bits of the AccessAndOpeningTimes model class as a nested class within the migration; or (b) avoid depending on the model methods by using SQL to make the database changes. I imagine this probably worked out for you, because you will have run the migration before removing the model class. However, when this PR is merged and the changes are deployed, I think the migration will fail.
For the 5th commit ("Add pending migration to drop the access_and_opening_times table"), I'm not familiar with the idea of the "pending" migrations. Do these get run automatically at some point or is the idea that we'll create a new PR at some point to move the pending migration into the standard migration directory?
While I've made quite a few comments/suggestions, overall this looks good to me and I don't think you definitely need to make any changes. The one thing that's worth checking is that the migration will run OK without the model being in the codebase, so on that basis I'm going to hold off on approving for now.

I didn't even consider that the migration would fail when deployed, so thanks for raising that. I've used a nested class as you suggested which seems to work fine.
"Pending" migrations are explained here
Sorry that the 3rd commit is a bit large, I found it difficult to split up without making it too granular and hard to follow in the history!

I didn't even consider that the migration would fail when deployed, so thanks for raising that. I've used a nested class as you suggested which seems to work fine.

Cool 👍

"Pending" migrations are explained here

I saw that, but it was unclear to me how/when those pending migrations are run. Presumably it's a completely manual process of opening a new PR...? Anyway, I'm sure all will become clear!

Sorry that the 3rd commit is a bit large, I found it difficult to split up without making it too granular and hard to follow in the history!

No worries - it's definitely a challenge!
Given you've addressed the problem with the migration, I'm going to approve this, although it looks as if you've got some conflicts to resolve before merging!


Creating other boards is forbidden?

when I  create another other Board object in the code (Board a = new Board();), the mouse hover says "Create a new board. Note: this should not be used in the challange, use the board provided in the think method instead". So can I use other boards in my calculation code? Or did I miss something and its forbidden?

Duplicate of #48. Please check open and closed issues before creating a new issue.

Hi, I've added: public static Board CreateBoardFromFEN(string fen) to the board class in V1.1
Please note that this is quite slow, so I'd generally recommend using the MakeMove and UndoMove functions on the board provided in the think function instead.


Create README.vi-VN.md

Initial Vietnamese translation added

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


Add issue form templates

I'm adding issue form templates to this repo to make it easier for the community to create useful issues.

I didn't know which GitHub team to assign these issues to, so I commented on the line; this needs to be updated ASAP.


can i get the code

hey could i get the code i would like to host a bot list site

Code is now available


InvalidOperationException when validating a target that contains a relative Uri property

Related to #44.
When validating an object that contains a Uri property that specifies an absolute URI using MiniValidation @ 0.7.3, MiniValidator.TryValidate successfully validates the object. However, if you set the Uri property to specify a relative URI, MiniValidator.TryValidate throws an InvalidOperationException, ultimately caused by it getting Uri.Segments, which is not a valid operation on a relative URI.
Example code to reproduce:
static void Main() {
    var model = new Model() {
        Path = new Uri("./some-relative-path", UriKind.Relative)
    };

    if (!MiniValidator.TryValidate(model, out var errors)) {
        Console.WriteLine("fail");
    }
}

public class Model {

    [Required]
    public Uri Path { get; set; }

}

Example stack trace:
This operation is not supported for a relative URI.
   at System.Uri.get_Segments()
   at MiniValidation.PropertyHelper.CallNullSafePropertyGetter[TDeclaringType,TValue](Func`2 getter, Object target) in /_/src/MiniValidation/PropertyHelper.cs:line 129
   at MiniValidation.PropertyDetails.GetValue(Object target) in /_/src/MiniValidation/TypeDetailsCache.cs:line 208
   at MiniValidation.MiniValidator.TryValidateImpl(Object target, Boolean recurse, Boolean allowAsync, Dictionary`2 workingErrors, Dictionary`2 validatedObjects, List`1 validationResults, String prefix, Int32 currentDepth) in /_/src/MiniValidation/MiniValidator.cs:line 271
   at MiniValidation.MiniValidator.TryValidateImpl(Object target, Boolean recurse, Boolean allowAsync, Dictionary`2 workingErrors, Dictionary`2 validatedObjects, List`1 validationResults, String prefix, Int32 currentDepth) in /_/src/MiniValidation/MiniValidator.cs:line 334
   at MiniValidation.MiniValidator.TryValidate[TTarget](TTarget target, Boolean recurse, Boolean allowAsync, IDictionary`2& errors) in /_/src/MiniValidation/MiniValidator.cs:line 138
   at MiniValidation.MiniValidator.TryValidate[TTarget](TTarget target, IDictionary`2& errors) in /_/src/MiniValidation/MiniValidator.cs:line 64
   at UserQuery.Main(), line 6

Fix for this is available in 0.7.4 on NuGet: https://www.nuget.org/packages/MiniValidation/0.7.4
Thanks again for the reports!

Works like a charm now, thanks!


[Lightning:DB:ErrInitErrManager]init error manager error: create conflict view failed: Error 1146 (42S02): table doesn't exist

What did you do?
dm upstream switch job failed, see https://github.com/pingcap/tiflow/actions/runs/12035450530
What did you expect to see?
No response
What did you see instead?
No response
Versions of the cluster
DM version (run dmctl -V or dm-worker -V or dm-master -V):
master```
Upstream MySQL/MariaDB server version:
```console
(paste upstream MySQL/MariaDB server version here)

Downstream TiDB cluster version (execute SELECT tidb_version(); in a MySQL client):
(paste TiDB cluster version here)

How did you deploy DM: tiup or manually?
(leave TiUP or manually here)

Other interesting information (system version, hardware config, etc):
>
>

current status of DM cluster (execute query-status <task-name> in dmctl)
(paste current status of DM cluster here)

It's not caused by https://github.com/pingcap/tidb/pull/57480 . Please attach TiDB log in GitHub actions artifacts to know what's happened

close via #11904


Link photos

Have a look @guillermo-carrasco :small_red_triangle:

:+1:

So are you gonna merge this or what!

Haven't had much time today but I'll look over the comments first

Ready to merge for me :smile_cat:

Gogogo! :kimono:


intermittant IO error -- nothing produced in tempdir except empty src/lib.rs

If I change a target specificatiton JSON file, and then run xargo, and then revert everything back to a known working state.... the known working state doesn't work anymore.   I'm not mistyping these commands, I use a Makefile.
$ xargo build --target x86_64-unknown-os2018 --features="boot_multiboot2"
Finished release [optimized] target(s) in 0.0 secs
error: intermittent IO error while iterating directory `/tmp/xargo.7NQW9DkNHi8F/target/x86_64-unknown-os2018/release/deps`
caused by: IO error for operation on /tmp/xargo.7NQW9DkNHi8F/target/x86_64-unknown-os2018/release/deps: No such file or directory (os error 2)
caused by: No such file or directory (os error 2)

Note: similar behavior has been observed in closed issues where the user put a .json at the end of their --target argument.  I am not doing that.  Everything should be exactly as it was when it last worked, but it no longer works.  I've tried rebooting.  I've tried removing ~/.xargo/lib/rustlib/*.  Neither helps.

Ok, maybe I've updated my rustc version, and nothing bad happened even though the target.json file was no longer valid for that rustc version... and I only became aware of this when the target.json file was again needed by xargo.
I think two things should happen here:  (1) cargo/rustc should not just exit with $?=0 and no output, and (2) xargo should get a different error message.   I could do (2), but if (1) was done, (2) shouldn't really be needed.

I'm not at all certain what is going on.  I can't find a working configuration (rust version and target specification that work together).

If I change a target specificatiton JSON file, and then run xargo, and then revert everything back to a known working state.... the known working state doesn't work anymore.

I can't repro the problem from this statement. I tried:
$ cargo new --lib foo && cd $_
$ echo '#![no_std]' > src/lib.rs
$ rustc --target thumbv7m-none-eabi -Z unstable-options --print target-spec-json > foo.json
$ xargo build --target foo
$ vi foo.json # change 'vendor' from "" to "foo"
$ xargo build --target foo
$ vi foo.json # change 'vendor' from "foo" back to ""
$ xargo build --target foo

But I see no I/O problem. This was with:
$ rustc -V
rustc 1.21.0-nightly (c11f689d2 2017-08-29)

$ xargo -V
xargo 0.3.8
cargo 0.22.0-nightly (3d3f2c05d 2017-08-27)

Some ideas:

Do you have RAM available? Do you have permissions to write to the /tmp directory?
Do you have CARGO_INCREMENTAL enabled (set in the environment)? I have seen it cause some bizarre problems before.

When using Xargo with an invalid target specification I get:
$ xargo build --target foo
error: ~/tmp/foo/foo.json is not valid JSON
caused by: expected `,` or `}` at line 24 column 3
note: run with `RUST_BACKTRACE=1` for a backtrace

Thanks for your reply, tests and suggestions.
I found the problem.   I have a .cargo/config file with:
[build]
target-dir = "../../build/rust"

So the build output for sysroot is not ending up in the tmp directory.
Do you have any suggestions for how the main build output can be redirected, without breaking xargo's ability to build sysroot in the tmp directory?

It appears that CARGO_TARGET_DIR works.


Nomination Update view not working

Login as admin and try to edit a nomination.
Possible bug is due to a button which skips nomination Continue without creating Nomination. That button is not rendered in Update ModelForm

Fixed. Changed the redirect url to index . Earlier it was to post_view and required an already existing pk


BalanceStrategyRange does not behave like Java / librdkafka

Versions
Please specify real version numbers or git SHAs, not just "Latest" since that changes fairly regularly.

Sarama
Kafka
Go

0c4ba61b272cf7bb88872e5698aab5053637b8e5
7.0.1
1.17.2

Problem Description
From the name / description, it looks like sarama's Range balance strategy seeks to mimic the Java and librdkafka behavior.
The Java / librdkafka behavior is desirable because it allows a consumer to subscribe to multiple topics that have "co-partitioned" data (i.e. each topic has the same partition count, key format, and partitioning function) and be guaranteed to receive the same subset of keys across all subscribed topics.
BalanceStrategyRange (by design?) does not exhibit the same behavior; there is even a test case indicating as much. It looks like it's using a hash of the topic and consumer id to determine the order to dole out partitions, which leads to seemingly non-deterministic behavior, since consumer IDs are usually random. Here's an additional test case for TestBalanceStrategyRange that shows how changing consumer group names can, confusingly, cause the behavior to look like Java's:
func TestBalanceStrategyRange(t *testing.T) {
	tests := []struct {
		members  map[string][]string
		topics   map[string][]int32
		expected BalanceStrategyPlan
	}{
		{
			members: map[string][]string{"M1": {"T1", "T2"}, "M2": {"T1", "T2"}},
			topics:  map[string][]int32{"T1": {0, 1, 2, 3}, "T2": {0, 1, 2, 3}},
			expected: BalanceStrategyPlan{
				"M1": map[string][]int32{"T1": {0, 1}, "T2": {2, 3}},
				"M2": map[string][]int32{"T1": {2, 3}, "T2": {0, 1}},
			},
		},
                // New test case. Two new consumer groups with same lexicographical order different result
		{
			members: map[string][]string{"M3": {"T1", "T2"}, "M4": {"T1", "T2"}},
			topics:  map[string][]int32{"T1": {0, 1, 2, 3}, "T2": {0, 1, 2, 3}},
			expected: BalanceStrategyPlan{
				"M3": map[string][]int32{"T1": {0, 1}, "T2": {0, 1}},
				"M4": map[string][]int32{"T1": {2, 3}, "T2": {2, 3}},
			},
		},

If it would be problematic to update this strategy to accurately implement the Java library's behavior, it would be very helpful to at least have an alternative strategy that does and some warning about the discrepancy. If that seems reasonable, I'd be happy to contribute a PR in the near future.

@njhartwell 👋🏻 hi and thanks for raising this issue
Yes the intention should always be that the Sarama balance strategies match the Java/librdkafka implementations. We had a similar PR a year or so ago (https://github.com/Shopify/sarama/pull/1788) which corrected and replaced the implementation of round robin strategy, so I'd be happy to review and merge one which did the same for the range strategy

Thanks @njhartwell — closed as fixed by https://github.com/Shopify/sarama/pull/2245 and released in v1.34.1


Ads Are Back

As of the 2017.02.22 version, video/audio ads are playing again. Seems like they may have updated their API. Any assistance would be appreciated!

Alright, I'll take a look when I get the time (currently really busy with exams). Thanks for reporting.

Awesome! Thank you for not abandoning this project ^.^

Hmm, so I tried it out (with both 2017.02.22 and 2017.03.03) and I can't reproduce the problem, ad blocking seems to be working fine. Mind posting your Xposed logs and a logcat dump (adb logcat -s "SoundCloudAdAway")? Thanks!

I'm embarrassed to admit... I forgot that my primary phone no longer has xposed, and noticed it there. My 5X still has xposed and it's fine there. Sorry, my friend ;/

Lol, no worries. Nice to know that the module still works after a year without any updates :-p


export PATH so it's more accessbile during runtime

This mini change makes thePATH env var accessible to any sub-processes during runtime in the dyno.

The environment variable PATH is already exported to begin with. This change doesn't accomplish anything.

@sandfox Was there a problem you were trying to solve? The unix shell sources these profile.d scripts at startup so the PATH shouldn't need to be exported.

Oh dear, I'm very sorry, I've been debugging the wrong buildpack..... (appears the app I'm looking at is using a very modified fork for $reasons) so I need to check if the problems we're encountering are confined to our mod/fork.
Going to close unless/until I find out the problem is definitely related to the official buildpack
@kr just to clarify - do you mean that it's sourced  in the way freeformz mentions or that there is something thing happening somewhere that exports it?
Some background for whats it's worth :
We're also using a bash script inside the app to run somethings which seems like it wasn't seeing $PATH that .profile.d/go.sh was setting so we're having to run commands with path to the ./bin/our-binary rather then just our-binary. We noticed that some other buildpacks were exporting the vars inside their equivalent .profile.d/* and concluded that our subshell wasn't see the modified $PATH
bash script (very tl;dr version)
for file in .profile.d/*; do
    source $file
done
command="$@"
runner="bash -c"
exec $runner "$command"

@kr just to clarify - do you mean that it's sourced in the way freeformz mentions or that there is something thing happening somewhere that exports it?

Startup Process
When a dyno starts up, the heroku process manager creates its container with a basic environment (setting $HOME and $PATH etc) and executes a bash process with the flag -c and the command line from the procfile to run. As part of its startup process, bash executes ("sources") one or more shell scripts such as /etc/profile and $HOME/.profile (I don't recall the exact list). Heroku has arranged that one of these files contains a loop like the one you gave above, for file in /app/.profile.d/*; do source $file; done or something similar. Then, once all that startup stuff is finished, bash executes the command line it got in the -c flag.
Exported Variables
It's helpful for us to distinguish between shell variables and environment variables. Environment vars are provided by the operating system, and they're inherited by child processes when the parent calls fork. Shell variables exist only inside a shell script process – they are a local feature provided by bash.
Bash sort of tries to hide this distinction by exposing both shell variables and environment variables in the same namespace. So $FOO could be either a shell variable local to the process or an environment variable, and you can't tell which just from looking at it. But it does provide some knobs to control this. When you write export FOO that sort of "promotes" the variable FOO from a shell variable to be an environment variable (if it isn't already). From that point on, any changes to the value of FOO will also be copied to the environment, so for example:
$ FOO=x
$ env|grep FOO
$ export FOO
$ env|grep FOO
FOO=x
$ FOO=y
$ env|grep FOO
FOO=y

And writing export FOO=bar is just shorthand for export FOO; FOO=bar.
The last piece of the puzzle is that any variables that are already in the environment when bash starts are "exported" from the moment bash starts executing, even if you don't explicitly export them. And PATH is one such variable (as mentioned above). It's basically like the very first thing bash does is run export PATH HOME FOO BAR … to cover all the variables inherited in the environment, before executing any of the actual scripts or commands it was provided.
Path
Putting this all together means that, in just about any bash script (inside a heroku dyno but also just about anywhere else), writing export PATH=xyz is no different than writing just PATH=xyz.

@kr Thanks for the detailed response.


Simultaneous total deletion of related objects fails in the presence of multi links

EdgeDB Version: 2.9
EdgeDB CLI Version: Irrelevant
OS Version: Irrelevant

Steps to Reproduce:

Migrate to the schema below using start migration to ...; populate migration; commit migration;.
Run query A below.
Run query B below.
Step 3 will fail with ConstraintViolationError: deletion of default::B (xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx) is prohibited by link target policy

It's unclear to us if this is considered a bug (Query C succeeds), however a similar setup with single links succeeds regardless of the order of deletion. If this is not considered a bug, the burden of the topological sort falls to the end-user.
Schema:
module default {
  type A {
    required multi link x -> B;
  }

  type B;
}

Query A:
with
  a := (insert B),
  b := (insert B),
  c := (insert A {
      x := {a, b}
  })
select {
  c := c
}

Query B:
with
  a := (DELETE B),
  b := (DELETE A)
select {
  a := a,
  b := b
}

Query C:
with
  a := (DELETE A),
  b := (DELETE B)
select {
  a := a,
  b := b
}

I think it is debatable whether this is a "bug" but I think it would be an improvement if we fixed it, certainly.

The issue here has to do with how multi links (and properties) are represented and how we use triggers to enforce link target policies.
Triggers are used for several things in the implementation, including:

Enforcing link target policies. When an object is deleted, a trigger checks if any incoming links exist and fails if so. (Or allows it, or deletes the source, depending on the policy.)
Deleting entries from a link table on source object deletion. A multi link is stored as a table with (source, target) rows. When an object is deleted, a trigger fires that deletes all link table entries with the object as source.

These triggers all run after the query, and they are triggered in the order the objects were modified.
So for query B, triggers are run in the following order

On the deletion of each B, a link target policy checks the link table for A.x to see if the object is a target
On the deletion of each A, all the entries in A.x with each A as source are deleted

This fails, because when step 1 runs, the link table entries haven't been deleted yet. In query C, those steps are reversed and so it works. The single link case works because for single links, the link lives in the source row, which got deleted in the actual query.
I know of two possible fixes here:

Make the link target policy trigger join against the source object instead of just checking the link table. Since the source object is gone, it won't fail.
Delete entries from the link tables in the actual query, instead of relying on triggers to do it.

We should do option 2. Option 1 would make all link policy checks slower. Option 2 on the other hand would make most deletes faster, since we could get rid of the need for the triggers.
Option 1 is included really just in case there is some reason why option 2 won't work that I'm not thinking of.
I think it should be pretty straightforward to do.


[engSys] Update  dependencies

Packages impacted by this PR
@azure/dev-tool
@azure/monitor-opentelemetry
Issues associated with this PR

Resolves #31917
Resolves #31916

Describe the problem that is addressed by this PR
Updates winston-transport and arethetypeswrong/cli to their latest versions. I
looked at the changelog and did not see anything concerning
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR: (Applicable only to SDK release request PRs)
Checklists

[ ] Added impacted package name to the issue description.
[ ] Does this PR need any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here.)
[ ] Added a changelog (if necessary).

API change check
API changes are not detected in this pull request.

@microsoft-github-policy-service rerun

/check-enforcer evaluate


[prelude] Parse: readAspect + spaced + readWhile + by-name parsers

The build for Scala Native is failing with the same issue as https://github.com/getkyo/kyo/pull/839 because of the use of Aspect in Parse. I've moved ParseTest to JVM only as a workaround.


[zh-cn] minor update task/administer-cluster

Update EncryptionConfiguration configuration API (v1) link
Add task-tutorial-prereqs-node-upgrade.md translate & replace
feature-state replace feature_gate_name with state & for_k8s_version

It's required to squash your commits into a single one.

/lgtm


{Misc} Add NOTICE.txt for legal compliance

Description

Add NOTICE.txt for legal compliance according to https://docs.opensource.microsoft.com/content/using/required-notice-template.html
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

add NOTICE.txt

@fengzhou-msft @arrownj Please help verify whether the way I add NOTICE.txt is right or not.


changing enhancement levels resets stat pots values.

if stat pots are changed, then enhancement levels are changed, stat pots values are reset to max.

@lyrgard remember to close this, as the fix has already been merged in PR #140

@biovenger thanks. Closing !


Add GetBatch method.

Adding the method and the request class.
Needs to retrieve the publication details to see the status
and make sure that the publication is in the correct state.
Resolves: OLPEDGE-2048
Signed-off-by: Oleksii Zubko<EMAIL_ADDRESS>

Codecov Report

Merging #337 into master will increase coverage by 0.0%.
The diff coverage is 100.0%.

@@          Coverage Diff           @@
##           master    #337   +/-   ##
======================================
  Coverage    89.4%   89.5%
======================================
  Files          65      66    +1
  Lines        1788    1795    +7
  Branches      203     203
======================================
+ Hits         1599    1606    +7
  Misses        111     111
  Partials       78      78

Impacted Files
Coverage Δ

...dk-dataservice-write/lib/client/GetBatchRequest.ts
100.0% <100.0%> (ø)

Continue to review full report at Codecov.

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


Sort out size of lastReportTimeMs in reporting

Problem
lastReportTimeMs is 32-bit, but we are assigning 64-bit chip::System::Layer::GetClock_MonotonicMS() values to it.
Proposed Solution
Need to figure out what size lastReportTimeMs should really be.

There is no more lastReportTimeMs.


Go SDK insecure by default, does not allow for audience/issuers checks

Using Pomerium JS SDK clearly mentions that the audience/issuer fields of JWTs should be checked by backends before being trusted.
However, the Pomerium Golang SDK does not seem to be configurable to perform such checks.

cannot configure JWKSURL to retrieve the key (header value is trusted)
cannot enforce issuer and audience field match

If such is the case then the current Go SDK implementation is insecure against JWT tokens theft/reuse from one backend service to another.
The JS example clearly demonstrate how to do this:
    const jwtVerifier = new PomeriumVerifier({
      issuer: 'authenticate.localhost.pomerium.io:4443',
      audience: 'react.localhost.pomerium.io',
      expirationBuffer: 1000
    });

Given that Pomerium is using this verify Docker image in a lot of the documentation on how to properly demo Pomerium, it is reasonable to assume that the underlying SDK should support audience/issuer checks and that it could be configured.

Hi @calderonth, I believe the Go SDK does support both configuring a JWKS URL as well as requiring specific JWT claims. See the sdk.Options struct (specifically the JWKSEndpoint and Expected fields).
Maybe we should add a corresponding Go example to the documentation, just like the JS example you referenced? (I can follow up with @desimone about that.)
And just so I understand your last question about the verify service, are you asking to add a configuration option to enforce audience/issuer checks inside verify itself?

Yes, documentation should exist to make it clear how to integrate and do the validation properly as a backend service. I hadn't seen the Expected field in the options, the devil is in the details.
And also yes, having the ability to configure this in the verify service (ideally as environment variables) would be great as one could experiment (i.e. unset then current behavior (maybe with a warning rather than all green tick), when set then verify would reject a JWT when iss/aud fields are not what should be expected for instance.
Let me know if that doesn't make sense?

Thanks for the clarification, that makes sense to me. I'll look in to updating the Go SDK documentation as well as adding new environment variables for the verify service.

Hi @calderonth, pomerium/verify:latest should now honor the new environment variables EXPECTED_JWT_ISSUER and EXPECTED_JWT_AUDIENCE (see #155). Setting either of these should validate that the corresponding JWT claim matches the provided value.
I still need to update the documentation, but feel free to try out this change and let me know if you run into any issues. Thanks!


Adding the possibility to inactive image drag and drop

Adding the possibility to users configure if drag and drop is active or not.
$('element').mediumInsert({
  editor: mediumEditor,
  addons: {
    images: {
      useDragAndDrop: false
    }
  }
});

I'm not sure about setting in image tag draggable as false and setting onmousedown to return false.

@orthes you removed this config. Do you know if there is another way to achieve this?

You can disabled it in MediumEditor directly.

@j0k3r thanks. I already tried that but no luck.  I'm posting this under the issue that relates to exactly what I just described. Sorry.


MAINT: Restore @property decorator

This will keep opens, closes, early_closes, etc to the
same pattern.

Coverage decreased (-0.04%) to 86.935% when pulling 265c02b0c182b4db9a5f1c5994e57a5ab2a3563d on bernoullio:master into 285b5f7d77760afe79b47195c319d7c4cc352d49 on quantopian:master.

Good catch!  Thanks @keang .


Enum DisplayTemplate fixes

Fixes #5547.
We seperate the "view" of how we display a model from the model itself by using FormattedModelValue

:up::date:. The way MemberData works seems to preclude using the same TheoryData across multiple classes (it appears to use reflection to look for the member by name on the test class).

🆙📅

:up::date:

🆙:date: Also fixed a test.

@ryanbrandenburg This fix didn't make it in 1.1.2, was there a specific reason for, too much risk perhaps? Is there a chance this fix will move to 1.1.3?