Remove integer port requirement for Facebook

PaaS hosting offerings like Heroku and Azure App Service do not have a standard numerical port, as it's randomly generated when the app is re-started.
The code currently has a check that ensures the port specified is a number. This will always fail and throw an exception on Heroku and Azure App Service, causing the app the crash.
Example port from Azure App Service: "\.\pipe\a59322f5-0005-4a3b-9242-24625191a3f2"
My fix removes the number check as ExpressJS, which is how the bots are served, has no requirement that ports are a number.
Closes howdyai/botkit#369

Nice catch!


Onfastpath

What's in this pull request?

Resolved bug number: (SR-)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci Please test

What is this for?

Basically it's to tell the inliner that such a code is worth inlining. It's similar to @inline(__always) but not as strict and it's effect is not limited to the containing function but also to it's callers (see also my comment on the first commit).

Makes sense, thanks. Is it worth giving the builtin an adjustable "bias" parameter, to adjust how "hot" the path is?

It's definitely possible to add a bias parameter, and it's easy to implement. But this does not answer the question if it's worth. I suggest we add it as soon as there is a real need for it.

Fair enough.


webpack test css question

I'm submitting a ...
[X ] question about the decisions made in the repository

Hi, I see we need to exclude index.html in config , webpack.test
        {
          test: /\.css$/,
          loader: ['to-string-loader', 'css-loader'],
          exclude: [helpers.root('src/index.html')]
        },

I'm looking to setup CSS modules for our app and I see that when import CSS via common js import it will break Karma tests.
Thanks

ignore this, please.


move text about how to contribute into it's own doc

I really appreciate the PR! but I think the downside here is that you can't view this information by immediately landing on https://github.com/pcottle/learnGitBranching, and have to find the (somewhat small) link to jump over to contributing.
I think having the high-level information on how to build the project and get going is useful in the readme -- maybe we could reserve the contributing doc for guidelines on how to format the pull request proposal (given that you really only see it when you start to make a PR).
Going to close for now but appreciate the effort!


Automatically update benchmarks results on wiki

[ ] Create scripts for automatic uploading of 'make log' information onto wiki page with comparison to other engines.
[ ] Create bot, running this scripts for each commit.

The build system underwent heavy modifications lately, therefore this old and abandoned issue is not applicable anymore.


Wrong code in [7] to plot the Rosenbrock function ?

For statements in [7] of https://github.com/SMTorg/smt/blob/master/tutorial/SMT_Tutorial.ipynb, the shapes for x, y are different from z, and it will get following figure which is different from the figure shown on that web page.
ax.scatter(xt[:, 0], xt[:, 1], yt, zdir='z', marker='x', c='b', s=200, label='Training point')
ax.scatter(xtest[:, 0], xtest[:, 1], ytest, zdir='z', marker='.', c='k', s=200, label='Validation point')

Maybe it should be revised as following, but it looks like that there's still a little different.
ax.scatter(xt[:, 0], xt[:, 1], yt[:, 0], zdir='z', marker='x', c='b', s=200, label='Training point')
ax.scatter(xtest[:, 0], xtest[:, 1], ytest[:, 0], zdir='z', marker='.', c='k', s=200, label='Validation point')

Thank you very much for noticing the problem and proposing a fix! It most likely is related to the release of numpy 2.0.0.

Installed numpy on my windows: 1.26.4.

I can not reproduce the problem in my environment. What are your Python, matplotlib versions?

WIN10
python: 3.12.4
matplotlib: 3.9.2
numpy: 2.1.1

import numpy as np
from matplotlib import cm
import matplotlib.pyplot as plt
from smt.problems import Rosenbrock
from smt.sampling_methods import LHS

########### Initialization of the problem, construction of the training and validation points

ndim = 2
ndoe = 20  # int(10*ndim)

# Define the function
fun = Rosenbrock(ndim=ndim)

# Construction of the DOE
# in order to have the always same LHS points, random_state=1
sampling = LHS(xlimits=fun.xlimits, criterion='ese', random_state=1)
xt = sampling(ndoe)
# Compute the outputs
yt = fun(xt)

# Construction of the validation points
ntest = 200 #500
sampling = LHS(xlimits=fun.xlimits, criterion='ese', random_state=1)
xtest = sampling(ntest)
ytest = fun(xtest)

# To plot the Rosenbrock function
x = np.linspace(-2, 2, 50)
res = []
for x0 in x:
    for x1 in x:
        res.append(fun(np.array([[x0, x1]])))
res = np.array(res)
res = res.reshape((50, 50)).T
X, Y = np.meshgrid(x, x)
fig = plt.figure(figsize=(6, 4))
ax = fig.add_subplot(projection="3d")
surf = ax.plot_surface(
    X, Y, res, cmap=cm.viridis, linewidth=0, antialiased=False, alpha=0.5
)

# Bad statements
ax.scatter(xt[:, 0], xt[:, 1], yt, zdir="z", marker="x", c="b", s=200, label="Training point")
ax.scatter(xtest[:, 0], xtest[:, 1], ytest, zdir="z", marker=".", c="k", s=200, label="Validation point")

# Updated statements
# ax.scatter(xt[:, 0], xt[:, 1], yt[:, 0], zdir='z', marker='x', c='b', s=200, label='Training point')
# ax.scatter(xtest[:, 0], xtest[:, 1], ytest[:, 0], zdir='z', marker='.', c='k', s=200, label='Validation point')

plt.title("Rosenbrock function")
plt.xlabel("x1")
plt.ylabel("x2")
plt.legend()
plt.show()

Note: Same result for Numpy 1.26.4

I've upgraded from matplotlib 3.8.3 to 3.9.2 and succeeded in reproducing the bug. Your change fixes it, thanks. We leave this issue opened till the notebook is fixed.


Multiple Select Dropdown does not display values in memory grid dialog.

Hi
I have added Multiple Select Dropdown on the dialog which is opened from memory grid.
Save is working and all selected values are going in table but selected values are not coming when editing the dialog.
Multi Select Dropdown is empty when editing the dialog.
Here is the screenshot for reference -
https://drive.google.com/file/d/0B9UGZZFXh-e6eWJZNHRKTEpXQms/view?usp=sharing
Please help to resolve this.

There are no easyUI those controls

Pff, see docs on MinSelectLevel

@107295472 stop commenting in all issues. Or i'll report it.

@volkanceylan
I have already tried MinSelectLevel but this is not working.
Multi Select Dropdown remains empty when editing the dialog.
Here is my code -
[LookupEditor(typeof(BillsRow), Multiple = true), ClientSide]
[LinkingSetRelation(typeof(VoucherTransactionRowBills), "TransactionId", "BillId")]
[MinSelectLevel(SelectLevel.List)]
public List<Int32> Bills
{
get { return Fields.Bills[this]; }
 set { Fields.Bills[this] = value; }
}

Sorry understood now. This is similar to two level master detail. You need to load these values in your repository list handler OnReturn method.

Thanks! It working now by overriding  OnReturn() in MyRetrieveHandler repository.

@ramveersgh How did you resolve this issue?

@swordz36
Here is my code to override onreturn method to add bills -
private class MyRetrieveHandler : RetrieveRequestHandler {
        protected override void OnReturn()
        {
            base.OnReturn();
            if (Row.TransactionList != null && Row.TransactionList.Count > 0)
            {
                for (int i = 0; i < Row.TransactionList.Count; i++)
                {
                    if (Row.TransactionList[i].Id.HasValue)
                    {
                        int transactionId = Row.TransactionList[i].Id.Value;
                        if (Row.TransactionList[i].Bills == null)
                        {
                            Row.TransactionList[i].Bills = new System.Collections.Generic.List<int>();
                            var query = new SqlQuery()
                                             .From("Accounts_VoucherTransactionBills")
                                             .Select("BillId")
                                             .Where("TransactionId = " + transactionId);
                            Row.TransactionList[i].Bills = Connection.Query<Int32>(query).ToList();
                        }
                    }
                }

            }
        }
    }

After upgrading to 2.0+, this stopped working again. I am assigning Bills information at described above but that does not show in dropdown on UI.
This was working before upgrading to 2.0+.
Do i need to do any changes to make it work?

Do you have any error in console? 2.0 didnt change something about master detail but can be side effect.

@volkanceylan
No i am not getting any error in console.

Need more details if you still couldn't resolve.


⚠ Grafana has degraded performance

In aadba14, Grafana (https://grafana.hydeos.fr) experienced degraded performance:

HTTP code: 200
Response time: 1727 ms

Resolved: Grafana performance has improved in f9e94f0 after 1 hour, 17 minutes.


Feature/adp 1208 resubmit transactions on rollback

Context
Retry rollbacked transactions.
Proposed Solution

Emit rollback transactions along with the historical transactions
Store confirmed transactions that are not yet stable
Retry confirmed transactions that were rolledback

Important Changes Introduced

Unit tests and refactoring will follow

Changes made fixups complicated and unreadable, so I force pushed the requested changes.
Still pending:

Unit tests for new code
Fix some new problems with the existing unit tests

Rebased on top of master and fixed existing unit tests

On squash, I will add a better commit message and include the breaking changes.


Can't load image from smartphone

Description
When try to add image from phone to chat, sometime it's work, sometimes dosn't.
From PC it's work properly.
Devices and Versions
Your Rocket.Chat.IOS version: (e.g. 3.1.0)
Your Rocket.Chat Server version: (0.69.2)
Steps to reproduce
Choose image to add from library to chat
Choose new one, and it didn't work

Can you check what is the filesize of each image? If it only fails for big images you may have to change client_max_body_size value in nginx config.


time: Minor documentation correction for pkg/time/#Time.Format

The documentation should mention that the example for http://golang.org/pkg/time/#Time.Format prints the same value for t.Format() and t.UTC().Format(). Reader may assume Format() always prints UTC. I think it behaves that way only because play.golang.org server runs in UTC. May be you should do one of

set play.golang.org to a different timezone because UTC is so special (for the rest of the world)
or document this near / inside the example.
or change the example to use a different timezone instead of time.Local

Interesting point.  The example is clear enough, and if you run it yourself the expected thing will happen, but if you run it by clicking on the button on golang.org it will indeed always print in UTC.
I guess my vote is to expand the comment in the example to mention what happens on golang.org.

At tip the example has been made much more complete, and I don't think it suffers from the same muddiness as the original pithy one:
http://tip.golang.org/pkg/time/#Time.Format

Fixed at tip.


Update webpack to latest

Fixes:
WARNING in webpack: Using NoErrorsPlugin is deprecated.
Use NoEmitOnErrorsPlugin instead.

ERROR in multi webpack/hot/poll?1000 ./src/server.js
Module not found: Error: Can't resolve 'babel' in 'webpack-hot-server-example'
BREAKING CHANGE: It's no longer allowed to omit the '-loader' suffix when using loaders.
                 You need to specify 'babel-loader' instead of 'babel',
                 see https://webpack.js.org/guides/migrating/#automatic-loader-module-name-extension-removed
 @ multi webpack/hot/poll?1000 ./src/server.js

@ericclemmons can you please disable greenkeeper for this project and close all its PRs?
Thank you!


chore: upgrade kokoro test to use latest go version

Kokoro tests should pass now that https://github.com/googleapis/google-cloud-go/pull/10484 is merged. Just need Go to pick up the new HEAD ref


Gfycat 403 error

I just downloaded the new version, but I'm still having gfycat issues. On here and Gonewilder.
On RipMe, this is what comes up:
2015-01-10 18:52:02,719 147259580 [Thread-5234] DEBUG utils.RipUtils  - Checking http://gfycat.com/NeglectedMellowKoi
2015-01-10 18:52:02,719 147259580 [Thread-5234] INFO  ripper.AbstractRipper  - Retrieving http://gfycat.com/NeglectedMellowKoi
2015-01-10 18:52:02,845 147259706 [Thread-5234] WARN  ripper.AbstractRipper  - Error while loading http://gfycat.com/NeglectedMellowKoi
org.jsoup.HttpStatusException: HTTP error fetching URL. Status=403, URL=http://gfycat.com/NeglectedMellowKoi
at org.jsoup.helper.HttpConnection$Response.execute(HttpConnection.java:449)
at org.jsoup.helper.HttpConnection$Response.execute(HttpConnection.java:424)
at org.jsoup.helper.HttpConnection.execute(HttpConnection.java:178)
at com.rarchives.ripme.utils.Http.response(Http.java:130)
at com.rarchives.ripme.utils.Http.get(Http.java:111)
at com.rarchives.ripme.ripper.rippers.video.GfycatRipper.getVideoURL(GfycatRipper.java:67)
at com.rarchives.ripme.utils.RipUtils.getFilesFromURL(RipUtils.java:45)
at com.rarchives.ripme.ripper.rippers.RedditRipper.handleURL(RedditRipper.java:186)
at com.rarchives.ripme.ripper.rippers.RedditRipper.parseJsonChild(RedditRipper.java:153)
at com.rarchives.ripme.ripper.rippers.RedditRipper.getAndParseAndReturnNext(RedditRipper.java:86)
at com.rarchives.ripme.ripper.rippers.RedditRipper.rip(RedditRipper.java:61)
at com.rarchives.ripme.ripper.AbstractRipper.run(AbstractRipper.java:316)
at java.lang.Thread.run(Unknown Source)
Sometimes, I get the gfy, most of the time not. I generally get the 403 error. I've tried switching to a different network via VPN, and even switching computers. I still get 403 errors.
I searched around, and it seems like it's a common issue (?), but I'm not savvy enough to know if it's on my end or not.
Thanks!

Kind of reproduced by ripping /r/nsfw_html5
I updated the user agent & now send the referrer when ripping gfycat links from reddit.
Haven't gotten a 403 since the change, so... maybe that fixes it?
Some people on /r/gfycat mentioned that these errors just suddenly started happening [1]. Maybe it's due to server load balancing & ignoring automated requests?

Hoping this fixes it.

So far, no 403 errors. I'm kind of busy right now, but I will check again in the next few days and test out some users with gfys to see if I get any 403s.
I know this is for RipMe, but any idea why 403 errors coming up for Gonewilder as well? It's also having trouble with vidme videos. Downloading very slow or not at all. I'm wondering if it's similar issues that seem to have just been fixed in the latest RipMe patch you committed.

So after a bit more testing it seems to work... and also that commit you also made to Gonewilder seems to have worked as well for gfycat 403 issue. The only thing that it says (for Gonewilder) is this:
[2015-01-17T05:24:30Z] ImageUtils: gfycat.com: getting http://www.gfycat.com/BabyishGroundedBoutu
[2015-01-17T05:24:32Z] Gonewild: ShesAWitch0: process_url: downloading #1 https://zippy.gfycat.com/BabyishGroundedBoutu.webm
[2015-01-17T05:24:40Z] Gonewild: ShesAWitch0: process_url: #1 cannot identify image file
But I think that has to do with the thumbnail.
I will try to test more later on.
Thanks so much!


View/ Update User Profile

actually when i relook at it - for this issue - please change getUsers() to not return the user password with the response


LLVM objmode compilation error: Instruction does not dominate all uses

Reproducer:
from numba import float64, njit, objmode
import numpy as np

def cbk(arr):
	print(arr)

@njit
def test(flag):
	arr = np.zeros((10,), float64)
	if flag:
		with objmode():
			print("test")
	with objmode():
		cbk(arr)

test(1)

With output:
RuntimeError: Instruction does not dominate all uses!
  %.69 = bitcast i8* %.68 to { i8*, i8* }*
  %.141 = getelementptr inbounds { i8*, i8* }, { i8*, i8* }* %.69, i32 0, i32 1

Thanks for the report, can reproduce this bug on Numba main.


User impersonation

Allow BO users with certain rights (user administration ? or a brand new "impersonation" right ?) to impersonate other BO users : easily check what they see / have access to ; and change password if email access is blocked.
For audit purposes, all actions taken while impersonating should mention both the impersonated account, and impersonating account.

Implementation could use this approach : https://www.dontpanicblog.co.uk/2015/08/20/user-impersonation-with-spring-security/


Merge dev into production

So the POs have a stable branch for testing

Coverage remained the same at 100.0% when pulling 4e3ae866952251f07625de389a9963d5241383cc on dev into b9b8afb45893c6effdc754b946077fd17ba5beb0 on production.

Coverage remained the same at 100.0% when pulling 4e3ae866952251f07625de389a9963d5241383cc on dev into b9b8afb45893c6effdc754b946077fd17ba5beb0 on production.


Error: The requested accessory 'GPIODevice' was not registered by any plugin.

Hi Guys don't work, pls help.
Error loading accessory requested in your config.json at position 1
Error: The requested accessory 'GPIODevice' was not registered by any plugin.
at PluginManager.getPluginForAccessory (/usr/lib/node_modules/homebridge/src/pluginManager.ts:198:15)
at /usr/lib/node_modules/homebridge/src/server.ts:347:37
at Array.forEach ()
at Server._loadAccessories (/usr/lib/node_modules/homebridge/src/server.ts:330:29)
at Server.start (/usr/lib/node_modules/homebridge/src/server.ts:156:12)
at cli (/usr/lib/node_modules/homebridge/src/cli.ts:80:10)
at Object. (/usr/lib/node_modules/homebridge/bin/homebridge:17:22)
at Module._compile (internal/modules/cjs/loader.js:1138:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1158:10)
at Module.load (internal/modules/cjs/loader.js:986:32)

My way to fix this problem, install steps:
update pi
sudo apt-get update && sudo apt-get upgrade -y
homebridge
sudo apt install git
git clone https://github.com/fantomnotabene/homebridge_setup_script
cd homebridge_setup_script
bash install.sh
plugin
curl -sL https://deb.nodesource.com/setup_10.x | sudo bash -
sudo apt install nodejs
cd /tmp
wget https://unicorn.drogon.net/wiringpi-2.46-1.deb
sudo dpkg -i wiringpi-2.46-1.deb
sudo usermod -a -G gpio pi
sudo npm install -g homebridge-gpio-device --unsafe-perm

Same problem here, I'll try your steps at another time.
This is my error log:
`pi@raspberrypi:~ $ sudo npm install --unsafe-perm -g homebridge
npm WARN deprecated<EMAIL_ADDRESS>bonjour-hap is no longer maintained. Refer to the Notice section of the README.md. The library should not be used anymore
/usr/bin/homebridge -> /usr/lib/node_modules/homebridge/bin/homebridge

<EMAIL_ADDRESS>updated 1 package in 20.037s
pi@raspberrypi:~ $ sudo npm install -g --unsafe-perm homebridge-relays
npm WARN deprecated<EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142

<EMAIL_ADDRESS>install /usr/lib/node_modules/homebridge-relays/node_modules/rpio
node-gyp rebuild

make: Entering directory '/usr/lib/node_modules/homebridge-relays/node_modules/rpio/build'
CXX(target) Release/obj.target/rpio/src/rpio.o
CC(target) Release/obj.target/rpio/src/bcm2835.o
../src/bcm2835.c: In function ‘bcm2835_gpio_pad’:
../src/bcm2835.c:487:3: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation]
if (bcm2835_pads == MAP_FAILED)
^~
../src/bcm2835.c:490:5: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the ‘if’
volatile uint32_t* paddr = bcm2835_pads + BCM2835_PADS_GPIO_0_27/4 + group;
^~~~~~~~
../src/bcm2835.c: In function ‘bcm2835_gpio_set_pad’:
../src/bcm2835.c:500:3: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation]
if (bcm2835_pads == MAP_FAILED)
^~
../src/bcm2835.c:503:5: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the ‘if’
volatile uint32_t* paddr = bcm2835_pads + BCM2835_PADS_GPIO_0_27/4 + group;
^~~~~~~~
SOLINK_MODULE(target) Release/obj.target/rpio.node
COPY Release/rpio.node
make: Leaving directory '/usr/lib/node_modules/homebridge-relays/node_modules/rpio/build'

<EMAIL_ADDRESS>added 52 packages from 68 contributors in 67.646s`

pi@raspberrypi:~/.homebridge $ homebridge  [7/11/2020, 14:03:10] Loaded config.json with 1 accessories and 0 platforms. [7/11/2020, 14:03:10] --- [7/11/2020, 14:03:13] ==================== [7/11/2020, 14:03:13] ERROR LOADING PLUGIN homebridge-relays: [7/11/2020, 14:03:13] Error: The module '/usr/lib/node_modules/homebridge-relays/node_modules/rpio/build/Release/rpio.node' was compiled against a different Node.js version using NODE_MODULE_VERSION 64. This version of Node.js requires NODE_MODULE_VERSION 72. Please try re-compiling or re-installing the module (for instance, using npm rebuildornpm install).     at Object.Module._extensions..node (internal/modules/cjs/loader.js:1188:18)     at Module.load (internal/modules/cjs/loader.js:986:32)     at Function.Module._load (internal/modules/cjs/loader.js:879:14)     at Module.require (internal/modules/cjs/loader.js:1026:19)     at require (internal/modules/cjs/helpers.js:72:18)     at bindings (/usr/lib/node_modules/homebridge-relays/node_modules/bindings/bindings.js:112:48)     at Object.<anonymous> (/usr/lib/node_modules/homebridge-relays/node_modules/rpio/lib/rpio.js:17:34)     at Module._compile (internal/modules/cjs/loader.js:1138:30)     at Object.Module._extensions..js (internal/modules/cjs/loader.js:1158:10)     at Module.load (internal/modules/cjs/loader.js:986:32) [7/11/2020, 14:03:13] ==================== [7/11/2020, 14:03:13] Loading 1 accessories... [7/11/2020, 14:03:13] Error loading accessory requested in your config.json at position 1 [7/11/2020, 14:03:13] Error: The requested accessory 'Relay' was not registered by any plugin.     at PluginManager.getPluginForAccessory (/usr/lib/node_modules/homebridge/src/pluginManager.ts:198:15)     at /usr/lib/node_modules/homebridge/src/server.ts:347:37     at Array.forEach (<anonymous>)     at Server._loadAccessories (/usr/lib/node_modules/homebridge/src/server.ts:330:29)     at Server.start (/usr/lib/node_modules/homebridge/src/server.ts:156:12)     at cli (/usr/lib/node_modules/homebridge/src/cli.ts:80:10)     at Object.<anonymous> (/usr/lib/node_modules/homebridge/bin/homebridge:17:22)     at Module._compile (internal/modules/cjs/loader.js:1138:30)     at Object.Module._extensions..js (internal/modules/cjs/loader.js:1158:10)     at Module.load (internal/modules/cjs/loader.js:986:32)


nali: 0.7.0 -> 0.7.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/zu1k/nali/releases.
meta.description for nali is: An offline tool for querying IP geographic information and CDN provider
meta.homepage for nali is: https://github.com/zu1k/nali
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.7.1 with grep in /nix/store/ncv2hxkd0z6gi0ssf41k9x6x3b7s0jnr-nali-0.7.1
found 0.7.1 in filename of file in /nix/store/ncv2hxkd0z6gi0ssf41k9x6x3b7s0jnr-nali-0.7.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
nali

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ncv2hxkd0z6gi0ssf41k9x6x3b7s0jnr-nali-0.7.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A nali https://github.com/r-ryantm/nixpkgs/archive/d884d3d4cac0be623c0d2bfedde56702f9e7ea62.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ncv2hxkd0z6gi0ssf41k9x6x3b7s0jnr-nali-0.7.1
ls -la /nix/store/ncv2hxkd0z6gi0ssf41k9x6x3b7s0jnr-nali-0.7.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

nali

Maintainer pings
cc @Diffumist @XYenon for testing.

/nix/store/ncv2hxkd0z6gi0ssf41k9x6x3b7s0jnr-nali-0.7.1/bin/nali <IP_ADDRESS>
2023/02/09 08:36:47 文件不存在，尝试从网络获取最新纯真 IP 库
<IP_ADDRESS> [澳大利亚 APNIC/CloudFlare公共DNS服务器]


Fixed port leads to problems on Windows 10

On version 1.23.1, the port of the tika server is set to "9998" in line 175 of tika.py.
However, this port is already in use by a system process on my Windows 10 system. It would be nice, if there would be an option in the library to change this port.
Might be related to the closed issue https://github.com/chrismattmann/tika-python/issues/206

Hi @BorisWiegand see the documentation in https://github.com/chrismattmann/tika-python/blob/master/tika/parser.py#L23 you can provide your own custom Tika Server end point which should take care of it.


Device setup: Phone - device Wi-Fi connection failure.

Hi,
I have an issue in the "Choose your XX device" screen during the device setup. When I select the device network to connect my phone into it, the app pops up the progress dialog, but it will never leave this screen. I have been going through your code to see what is causing this problem and finally, I figured out that the phone is connecting/disconnecting until the variable discoverProcessAttempts reaches "MAX_NUM_DISCOVER_PROCESS_ATTEMPTS". Then, the phone gets blocked on this screen.
This effect is caused by the phone itself. In this case, the phone detects that the device network does not have an internet connection and then, the phone decides to connect to any other registered existing network that really does. If one explicitly disables all the registered existing Wi-Fi networks, this problem vanishes.
So, having a look in your code from the version (io.particle:devicesetup:0.6.3), I found that you are using the WifiConfiguration class and the priority field, which is set to 999999 giving maximum priority to device's network (ApConnector.java). However, I have seen that this field is deprecated (https://developer.android.com/reference/android/net/wifi/WifiConfiguration.html#priority).
Could the problem be here perhaps? Maybe the phone does not really prioritize the device's network after all and that's why it is connecting/disconnecting all the time.
This issue happens on a Xiaomi Mi 9T.
I am looking forward to your response.
Thanks in advance!
Nacho

This effect is caused by the phone itself. In this case, the phone detects that the device network does not have an internet connection and then, the phone decides to connect to any other registered existing network that really does. If one explicitly disables all the registered existing Wi-Fi networks, this problem vanishes.

We already disable the network we're currently connected to.  Disabling all networks is an option I suppose (I'm guessing one would actually have to disable just the registered networks currently in range), but that seems like it runs a high risk of side effects and would require a good amount of testing.  As such, I can't give any kind of timeline for fixing this right now.

Disabling all the networks is not a very efficient solution. The problem resides in the fact that android API for network preference has changed. The priority field is no longer used and deprecated (in fact, the whole WifiConfiguration class is). We have found the issue above in various phones from various manufacturers due to changes in Android and not due to changes made by manufacturers. From Android 8 onwards, wpa_supplicant is responsible for setting WiFi network priorities and unless the new network API is used, the device setup library will be facing this bug.
Independently of the issue of wpa_supplicant changing the selected network, I think it is important to be able to show an error to the user when this situation takes place, otherwise, the user will be forever waiting in a screen with a infinite progressbar that cannot cancel.
Thank you @jensck  for the support!

Hi,
Having the same issue with a Samsung Galaxy S9 with Android 10. The app is not able to connect to the device's WiFi. The phone remains connected to the WiFi network it was connected before starting the configuration process.
After a couple of minutes, the app prompts an error dialog which lets you retry the process.
Thanks in advance for your help!

Chiming in, this is a blocker on my Oppo Reno running Android 11 and Google Pixel 3 running Android 12 as well.


Handlebars PR author in the template?

Great work btw! auto-changelog is almost perfect for me. Last thing I kinda want for my changelog is the same behavior that changesets has with PRs. Was wondering if this is possible, or if I haven't read enough of the docs.
Basically it thanks the person who was responsible for the PR haha.

Closing since I figured it out.
This works: https://raw.githubusercontent.com/cookpete/auto-changelog/refs/heads/master/templates/conventional.hbs


[Q]when I run the sweep agent "wandb agent caimingxue/MagRobot_RL/i7b2ug3h", it occurs some following errors, however, when i run this script, it will ok.

Hey there, can you send the full stack trace and the script so we can repro the issue?


[Snyk] Upgrade dayjs from 1.8.28 to 1.11.11

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to upgrade dayjs from 1.8.28 to 1.11.11.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 38 versions ahead of your current version.

The recommended version was released on a month ago.

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Close this until we found a way to make this pass the DCO check


Show scrollback in same channel/query window

Rather than show as being from the "mattermost" user or in the "mattermost" query window, this shows scrollback text in the same window as the one we're requesting scrollback for.

It works for channel, but I can't seem to get it to work for DMs. I think I need to use u.MsgSpoofUser with the receiver / sender thingy like in mm-go-irckit/userbridge.go:
    if event.Sender.Me {
            if event.Receiver.Me {
                    u.MsgSpoofUser(u, u.Nick, event.Text)
            } else {
                    u.MsgSpoofUser(u, event.Receiver.Nick, event.Text)
            }

@42wim, what do you think?

Okay, got something that works for both channel and DM/query.

very nice 👍


Alert Driver chart install not being registered by the UI

Setup

Rancher version: v2.6-head ID: 8245868
Browser type & version: Chrome Version 109.0.5414.119 (Official Build) (arm64)

Describe the bug

After installing the Alert Drivers chart, if you go back to the chart's page it doesn't appear to be installed as the action button still says Install and none of the versions listed show that they're the currently installed version. If you check the namespace that the chart was installed in, you can see that the pods exist and are running, but for some reason the UI is not registering that the chart is actually installed.
If you click the Install button, it'll take you through the install process again. If you try to install the chart in the same namespace again, you'll get an error stating that the chart is already installed.
To Reproduce

Set up a v2.6-head Rancher instance
In either the local or a downstream cluster, go to the Apps page
Install the Monitoring chart first
Go back to the Apps page
Open the Alert Drivers chart
Click install
Set the namespace to the cattle-monitoring-system namespace
Check either of the checkboxes (I left SMS checked)
Finish the install of the chart
See successful install in the shell
Go back to Apps > Alerting Drivers

Result
The UI shows that the Alerting Drivers chart isn't installed even though it was successfully installed, and therefore you're unable to upgrade or edit the install
Expected Result
The UI shows the chart is installed if it actually is (in this case it is, or at least appears to be from my checking)
Screenshots

Additional context
This is occurring on v2.7-head as well

@nwmac @nickwsuse this is as designed. The annotation should not be added as it is not a singleton:

Otherwise it is assumed that the Helm chart can be installed multiple times in a cluster.

This is true for Alerting Drivers.

Context
What is Alerting Drivers?
The Alerting Drivers Helm chart is intended to help users deploy non-native notification mechanisms such as those listed in the Alertmanager docs. Currently we support, Sachet for SMS or prom2teams for Microsoft Teams.
Once you have Alerting Drivers installed, updating your Alertmanager configuration to add a Webhook Receiver where the url points to the Service deployed by the chart is all you need to get any Alertmanager (cluster or project level) to point to any driver deployed by Alerting Drivers.
Why is Alerting Drivers Not A Singleton?
While our Rancher UI supports creating receivers on the cluster Alertmanager to point to the Alerting Drivers Helm chart in the cattle-monitoring-system namespace, it's a perfectly valid solution for a user to deploy a separate version of the Alerting Drivers Helm chart in a project-specific namespace (i.e. default) and configure their Project Alertmanager to point a webhook receiver towards that deployment.
In a multi-tenant environment where each project is owned by a different team, this allows individual tenants to configure Alerting Drivers independently with their Project Monitoring stack.

To upgrade an existing instance of Alerting Drivers, two possible paths should be tested (and addressed by UI if there are any issues):

You should be able to go to the Installed Apps page and click on the three dots, click Edit/Upgrade, and upgrade the namespace-specific installed app
If you selected the Alerting Drivers app on the catalog, clicked on install, and selected a namespace where Alerting Drivers is already installed into, you should reach a page where you can Edit as YAML the release or use the questions.yml to configure the chart. Here, two things need to be checked
a. When you trigger the install / upgrade, it should successfully work. Specifically, it should trigger a helm upgrade, not a helm install since UI should be detecting that the chart has already been installed onto this namespace
b. If you initially overrode values for the chart on an install, UI should be pulling in those overridden values in the Edit as YAML page. On an in-place upgrade, it should not be reverting back to defaults.

Moving tis to-test to ensure user experience is correct after install for the above cases mentioned in here https://github.com/rancher/rancher/issues/40483#issuecomment-1424962774

@aiyengar2 Thanks - when I try and install Alerting Drivers more than once in different namespaces, it fails with:
Error: INSTALLATION FAILED: rendered manifests contain a resource that already exists. Unable to continue with install: ClusterRole "rancher-alerting-drivers-admin" in namespace "" exists and cannot be imported into the current release: invalid ownership metadata; annotation validation error: key "meta.helm.sh/release-namespace" must equal "default": current value is "neil-test"

@mitulshah-suse Can your team take a look at this?

There are 2 parts to this issue.
Chart setup:
Based on the above discussion, it appears that the chart is expected to install successfully on multiple namespaces. If that is the case, the chart itself has multiple cluster scoped resources like ClusterRoles which would block the installation as seen in the comment above. https://github.com/rancher/rancher/issues/40483#issuecomment-1425473768
The possible solutions to this would be

either move the clusterroles and other cluster scoped resources to a separate chart which is marked as a prerequisite for this chart and only installed once.
or append namespace names to such resources, so that different ones are created each time and they dont conflict with each other.

UI setup
The UI also does not seem to recognize whether the chart is currently installed in a particular namespace or not. It simply seems to run helm install on all the namespaces even if its an upgrade for a particular namespace which is also causing failures.

Below screenshot has list of resources being created. As you can see there are a few cluster-scoped resources in the list.
Additionally there is another template for hardened clusters as well which has more such resources.

@prachidamle Please advise on which approach from the above https://github.com/rancher/rancher/issues/40483#issuecomment-1521535013 is preferable.

Checked with @aiyengar2 and he suggested converting cluster scoped resources to namespaced resources would be a good appraoch to take here @mitulshah-suse @doflamingo721


CONFSRVDEV-29911 update OgnlRuntime & ObjectPropertyAccessor to do ac…

OgnlRuntime.setFieldValue doesn't check member access rights via MemberAccess interface

Reason

Investigation shows that getMethodValue/ setMethodValue / getFieldValue are all updated with member access rights check but not setFieldValue, which cause ObjectPropertyAccessor#setPossibleProperty expose to security vuln.
ObjectPropertyAccessor#setPossibleProperty has a fallback mechanism using getWriteMethod which also lack member access rights check

Changes/ Solution

add field member access check to OgnlRuntime#setFieldValue that is controlled by parameter checkAccessAndExistence
add method member access check to ObjectPropertyAccessor#setPossibleProperty code block that uses OgnlRuntime#getWriteMethod

Result & Impact
now ObjectPropertyAccessor#setPossibleProperty will also check member access rights when fallback to use:

OgnlRuntime.setFieldValue
method invoke that is from OgnlRuntime.getWriteMethod

Demo

Loom video link
Struts atlassian test link

PS: I see that OgnlRuntime#getStaticField already has member access check, so I assume we dont need to update code.

PR is good ty, I'll discuss next steps with you tomorrow


How to remove image(Atomic block)

we add image like this:
  const Image = (props) => {
    return (
      <div className="relative transition-all w-fit hover:scale-105">
        <img
          src={
            props.src ||
            "https://m.media-amazon.com/images/I/6164bYTG-QL._AC_UF1000,1000_QL80_.jpg"
          }
          alt={props.alt || "image"}
          style={{ width: 200 }}
        />
        <RxCross2
          onMouseDown={(e) => {
            e.preventDefault();
            removeImage();
          }}
          cursor={"pointer"}
          size={24}
          className="absolute top-0 right-0 text-white bg-black rounded-full"
        />
      </div>
    );
  };

  const decorators = createDecorator([
    {
      component: Image,
      strategy: findEntitiesOf("image"),
    },
  ]);

  function addImageBlock(options) {
    addImage(editorState, setEditorState, options);
  }

But How to Remove Image + specific image

Hey @Karan09542
Generally, you can remove a block in draft-js by its entityKey. So these are steps you need to follow:

Store the entityKey of the atomic block, it can be a data attribute on the element as an example.
Then you can get the blockMap of the state and filter them to exclude the one you want to remove.
Set the filter result as the new state for the editor.

This could be a useful utility for the package as well, but unfortunately I need to find some free time for that.
Also there was a problem in previous version, I've checked the utility and understood that I did't return the entityKey in atomic related utilities like addImage function. Please update to v1.3.1.

Hey @Karan09542
Generally, you can remove a block in draft-js by its entityKey. So these are steps you need to follow:

Store the entityKey of the atomic block, it can be a data attribute on the element as an example.
Then you can get the blockMap of the state and filter them to exclude the one you want to remove.
Set the filter result as the new state for the editor.

This could be a useful utility for the package as well, but unfortunately I need to find some free time for that.
Also there was a problem in previous version, I've checked the utility and understood that I did't return the entityKey in atomic related utilities like addImage function. Please update to v1.3.1.

After doing the above see below Screenshot
const removeAtomicBlock = (entityKeyToRemove) => {
    const contentState = editorState.getCurrentContent();
    const blockMap = contentState.getBlockMap();

    // Filter out the block with the specific entityKey
    const newBlockMap = blockMap.filter((block) => {

      const entityKey = block.getEntityAt(0);
      console.log("block", block);
      console.log("entityKey", entityKey); // Get the entityKey of the block
      console.log("getType", block.getType());

      return entityKey !== entityKeyToRemove; // Keep blocks without the target entityKey
    });

    // Update the content state with the filtered blockMap
    const newContentState = contentState.set("blockMap", newBlockMap);
    const newEditorState = EditorState.push(
      editorState,
      newContentState,
      "remove-range"
    );

    setEditorState(newEditorState);
  };

entityKey is null
Also when this line Excecute Then WHOLE Text editor content clear:

also no matter what changeType :
| "adjust-depth" | "apply-entity" | "backspace-character" | "change-block-data" | "change-block-type" | "change-inline-style" | "move-block" | "delete-character" | "insert-characters" | "insert-fragment" | "redo" | "remove-range" | "spellcheck-change" | "split-block" | "undo";
also whene Backspace atomic block not remove


Extend the action to packages and registry

In some cases it may be beneficial to cache also packages (.julia/packages) and/or the registry (.julia/registries), not just artifacts.  For example I cache the registry in Measurements.jl, and today I enabled caching of packages in CI for the General registry.  Note that this would affect also the name of the repository/action.
Maybe we can have options to enable cachin of artifacts, packages and registy, with artifacts being the only one enabled by default?

Thanks for providing some extra details. Clear now. I'll add it to #1.

In the PR you advise against always caching the registry, especially on Windows. What would be your advice? If it is only Windows, then maybe we can enable all caching by default and disable caching the registry for Windows by default?
Or would you advice having inputs as follows:
inputs:
  cache-artifacts:
    description: 'Whether to cache artifacts'
    default: 'true'
  cache-packages:
    description: 'Whether to cache packages'
    default: 'true'
  cache-registry:
    description: 'Whether to cache the registry'
    default: 'false'

Note that this would affect also the name of the repository/action.

According to a GH code search, Rik is the only one using it so far, so I think we can still rename it to julia-actions/cache or so.


chore: add kyanos to emerging apps list

Add kyanos(https://github.com/hengyoush/kyanos) to emerging apps list.

Thanks for the addition, can you make the description a little shorter and focus on how it uses eBPF?

Updated.

Thanks for the addition!


Change TFA (2FA) to the new standard term MFA

MFA is now the preferred term
URL of RTD document: https://civicactions-handbook.readthedocs.io/en/fen-mfa/

@openprivacy and @alanna-casey does this close #741?

@dmundra yes, it closes 741


PUP-1253 Add systemd service masking support

This updates the systemd service type to add support for masking. If a
service is masked, it is deemed to also be disabled. If a service is
masked and changed to enabled, it will first be unmasked since the
standard 'systemctl enable' command does not properly unmask the command
first.

I can change this to a feature, that's easy enough and appropriate.
feature :maskable, :service_mask, something else?

Ooh naming ... I'd go with 'maskable' because the rest of the service features are 'xxxable's.
/cc @joshcooper both for the naming and because I'm not sure if I completely captured his concerns.

CLA signed by all contributors.

Ping @joshcooper

My $.02: it is surprising to allow mask on systems that don't have the maskable feature.

Mine: this is conceptually identical to ensure => purged, and we should treat them identically. I can see an argument for going either route, but from an explainability standpoint, I want it to be the same route. Inconsistency harms user brains, and there needs to be a compelling reason to introduce it.

Well, as surprising as it may be, here are your options:
Allowing Mask:
service { 'foo': enable => 'mask' }

Masks the service on systems that support mask, disables it otherwise because that's the obvious fallback (and documented!).
Not-Allowing Mask:
service { 'foo': enable => '.......Wait, no

if $::operatingsystem in ['RedHat','CentOS'] and $::operatingsystemversion in....Wait, no

<create custom fact, has_systemd>

if $has_systemd {
  service { 'foo': enable => 'mask' }
}
else {
  service { 'foo': enable => false }
}

I know where my vote lies.....

Ok, so now it's documented! I updated the documentation header to describe what happens when you use mask on systems without service masking.
Hopefully I didn't butcher Nick's expectations for formatting too much :-D

But my guess is that 99+% of sites won't need that special-case logic at all - they'll just keep saying ensure => stopped and it will behave as expected on systemd/upstart/etc.
And the rare site that wants masked behavior on systemd systems (but has a mix and wants the same code to work "the same" on all) might do something like the above. Optimizing for that corner case isn't making sense to me atm (and as Nick points out isn't consistent with how we've treated other provider-specific features).
I'll have to sleep on this :)

:-1: to treat mask equivalent to disabled where not supported; if we do that we're lying to the user and silently doing the wrong thing. If given the option between failing hard and silently doing the wrong thing, I've found you'll always regret going with the latter.

Well, of course I woke up and had some thoughts.

My implementation of ensure => 'mask' is incorrect! Thinking about this, the type should not be making implementation decisions for the provider. Should we have another provider that implements :maskable, my current implementation may not be what it should do. I will be changing this to be more appropriate.

@kylog and @adrienthebo are correct. Allowing :mask to happen on systems that do not support masking does not accurately reflect the state of the system.

I should change 'mask' to 'masked' since 'mask' does not fit with 'enabled|disabled'.

PR update coming soon.

As a side note, can we get FACT-853 pushed through since it will be necessary for figuring out whether or not we have systemd running on a system? I suppose that a more accurate fact would be has_maskable_services, but that's a bit overkill.

PR Updated with the adjustments mentioned above.

@kylog or @joshcooper have you had a chance to look this over?

I'm :+1: on this in its current state, but I'd like one of the earlier reviewers to weight in

I haven't yet, but I would like to take another pass through before merging.

@adrienthebo @kylog it looks like all your comments have been addressed on this PR. If you could take a look to give either a final thumbs up or additional comments, that would be great!

 We'll give you a few more days to approve or deny this before we merge it.

@kylog you mentioned you wanted to take another look at this

@kylog Any update on this PR?  Sorry for the repeated pings.

Damn. My apologies for becoming such a bottleneck on this PR.
:+1:

@kylog Are we good to merge this now or are we waiting for post 4.1?

@trevor-vaughan One last thing before we can merge this: could you amend the commit messages to be prefixed with "(PUP-XXX)" for consistency with our contribution guidelines.  The current commit messages are missing the parenthesis (it's a bit pedantic, I know, but consistency is a good thing when viewing git logs).  Thanks!

Weird, I thought I did this. I'll check again.
On Thu, Apr 23, 2015 at 2:55 PM, Peter Huene<EMAIL_ADDRESS>wrote:

@trevor-vaughan https://github.com/trevor-vaughan One last thing before
we can merge this: could you amend the commit messages to be prefixed with
"(PUP-XXX)" for consistency with our contribution guidelines. The current
commit messages are missing the parenthesis (it's a bit pedantic, I know,
but consistency is a good thing when viewing git logs). Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/puppetlabs/puppet/pull/3684#issuecomment-95684881.

--
Trevor Vaughan
Vice President, Onyx Point, Inc
(410) 541-6699
<EMAIL_ADDRESS>-- This account not approved for unencrypted proprietary information --

Rebased and done.
On Thu, Apr 23, 2015 at 2:58 PM, Trevor Vaughan<EMAIL_ADDRESS>wrote:

Weird, I thought I did this. I'll check again.
On Thu, Apr 23, 2015 at 2:55 PM, Peter Huene<EMAIL_ADDRESS>wrote:

@trevor-vaughan https://github.com/trevor-vaughan One last thing
before we can merge this: could you amend the commit messages to be
prefixed with "(PUP-XXX)" for consistency with our contribution guidelines.
The current commit messages are missing the parenthesis (it's a bit
pedantic, I know, but consistency is a good thing when viewing git logs).
Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/puppetlabs/puppet/pull/3684#issuecomment-95684881.

--
Trevor Vaughan
Vice President, Onyx Point, Inc
(410) 541-6699
<EMAIL_ADDRESS>-- This account not approved for unencrypted proprietary information --

--
Trevor Vaughan
Vice President, Onyx Point, Inc
(410) 541-6699
<EMAIL_ADDRESS>-- This account not approved for unencrypted proprietary information --

Thanks @trevor-vaughan! This will be released in puppet 4.1.


Install deps from git in docker container

I have a private git repository which I have in my Pipfile and it is marked as editable = true.
I can install this locally in my virtual environment no problem.
I am using Docker and inside my Dockerfile I run:
pipenv install --system --deploy
This installs my library into a src/ directory in the working directory. I don't know why it's doing that but is there something I am missing or is there a way to specify site-packages.

Pip also does this. You can set the environment variable PIP_SRC to change it.

Thank you 👍


设置 darkModeType 类型无效？

New Issue Checklist

[x] I have read the Documentation
[x] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
e.g. iOS / macOS

Platform Version
e.g. 12.0 / 10.14.0

TABAnimated Version
e.g. 5.0.0 / 4.4.0

Xcode Version
e.g. Xcode 11 / Xcode 10

Repro rate
e.g. all the time (100%) / sometimes x% / only once

Repro with our demo prj
e.g. does it happen with our demo project?

Demo project link
e.g. link to a demo project that highlights the issue

Issue Description and Steps
Please fill in the detailed description of the issue (full output of any stack trace, compiler error, ...) and the steps to reproduce the issue.

交流群的联系方式可以发下吗

上述问题已解决，closeCache = NO 即可生效


go-client example is not working on kubevirt go-client v0.27.0

Is this a BUG REPORT or FEATURE REQUEST?:
/kind bug
What happened:
listvms with go-client version v0.27.0 not working correctly. But it works fine in v0.26.0.
# kubevirt.io/client-go/log
../../../../pkg/mod/kubevirt.io/client-go@v0.27.0/log/log.go:72:25: undefined: log.Context

What you expected to happen:
listing vms.
How to reproduce it (as minimally and precisely as possible):
Just using kubevirt.io/client-go package with version v0.27.0
Anything else we need to know?:
NONE
Environment:

KubeVirt version (use virtctl version):
Kubernetes version (use kubectl version):
VM or VMI specifications:
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

Here is another report from a user that it does not work:
# kubevirt.io/client-go/kubecli../../pkg/mod/kubevirt.io/client-go@v0.27.0/kubecli/kubevirt_test_utils.go:27:2: cannot use MockKubevirtClientInstance (type *MockKubevirtClient) as type KubevirtClient in return argument:        *MockKubevirtClient does not implement KubevirtClient (missing DiscoveryV1beta1 method)

@woohhan how are you compiling the code? It works for me. What I did is the following on go 1.12.8:
git clone https://github.com/kubevirt/client-go.git
export GO111MODULE=on
cd client-go/
cd client-go/examples/listvms/
go build . # wait for go mod to be finally done
./listvms

@woohhan I ran into your problem too, it seems this lib: github.com/go-kit/kit unexported NewContext in v0.4.0 release
https://github.com/go-kit/kit/releases/tag/v0.4.0
I solved it by adding this in go mod github.com/go-kit/kit => github.com/go-kit/kit v0.3.0

@rmohr Thanks for replying. I built similarly, but I modified the client-go version to v0.27.0 in the go.mod file.

@Zizhen Thanks for helping! I added the replacement but looks not working. Could you please share your go.mod file?

@woohhan the steps above worked for me without any modifications. It just takes ages until the dependencies are resovled. Did you try it completely unmodified whith the steps above?

@rmohr It works fine with the above steps without any modifications :) But is the latest version of the kubevirt client not available? As far as I know, the latest version of kubevirt go-client is v0.27.0.

In my steps I checked out the v0.27.0 tag, so that should exactly be what you want :).
We don't update the master branch. We only do releases directly to tags. I think we need to improve the documentation here. I can understand that this can cause confusion.

In my steps I checked out the v0.27.0 tag, so that should exactly be what you want :).
We don't update the master branch. We only do releases directly to tags. I think we need to improve the documentation here. I can understand that this can cause confusion.

@rmohr I may be wrong, but git checkout v0.27.0 doesn't determine what module to be used in go build. When we type go build command, It downloads the packages what we needed(described in the go.mod file) and builds a binary with the packages. You can check it with the following commands. Please let me know if I am wrong :) Thanks
# Delete v0.19.0 client-go
sudo rm -r<EMAIL_ADDRESS>
# Build again (Same with above)
$ git clone https://github.com/kubevirt/client-go.git
$ export GO111MODULE=on
$ cd client-go/
$ git checkout v0.27.0
$ cd examples/listvms/
$ go build .
go: extracting kubevirt.io/client-go v0.19.0  # It downloads v0.19.0 pkg even though we checked out to v0.27.0

That looks strange. When I look at https://github.com/kubevirt/client-go/blob/v0.27.0/go.mod and https://github.com/kubevirt/client-go/blob/v0.27.0/go.sum I don't see any depency to kubevirt.io/client-go and also the go.mod file says that it is module kubevirt.io/client-go.

Oh now I see it. We have this extra go.mod file in the example directory. That is not right. Thanks! :+1:

I'm getting this with my Gomod as
replace k8s.io/client-go => k8s.io/client-go v0.27.2
Due to this bug  https://github.com/kubernetes/client-go/issues/874


nz-tooltip: [touch only] hover tooltip persists when a modal is opened

Reproduction link
https://stackblitz.com/edit/ng-zorro-tooptip-modal-persist?file=src/app/app.component.ts
Steps to reproduce
Go to responsive view in dev tools and make sure you use the touch emulation.
Place an element with a tooltip in hover mode.
Now make the element show a modal on click.
What is expected?
The tooltip should be discarded and only the modal should remain opened.
What is actually happening?
Both the modal and the tooltip show up.

Environment
Info

ng-zorro-antd
9.3.0

Browser
Chrome 85

ng-zorro-antd is now optimized for mobile devices. As a temporary solution, you can hide the tooltip when the button is blurred. You can try ng-zorro-antd-mobile.


Automatic commit: Move 'odata-05-data-model-service' from QA to Produ…

@qmacro FYI
@thecodester & @MichaelCzcz DJ is in contact with me and we work together on this PR and let you know once we are done.

@iwonahahn  @qmacro Any update on this?

@thecodester will close this
@qmacro I will open a new pull request with the changes we have agreed in our call.


dropAll() return immediately if no tables found

Small tweak to Schema Builder dropAll() - if no tables exists simply return immediately

Please add a test that runs this code.

Im actually not sure how to test this? Since dropAll() returns void, how would I test if it exited early due to there being no tables, or if it exited later because it found and dropped some tables?

I'm ok as long as all the code paths are covered.
So you can just run Connection::dropAllTables with no tables and just $this->assertTrue(true);.

added test_dropAllTablesEmpty(), that should cover it?

It appears to work when I run it, but I am not sure about a couple of things
Does that test_dropAllTablesEmpty() get run last in the file? If so it will be okay since the previous test_dropAllTables() will have emptied the db. Or do the test methods in the file get called in any order, in which case does each test method start with an empty database, or do artefacts from previous tests persist?

I think tests that depend on other tests to be run first is really bad.
So ideally, there should be a hook which deletes all created tables after every test, but that slows down the test significantly.
So as a compromise, I think you should drop all tables twice in this test. That way the 2nd try will always go through the code provided.

Done

Please add a changelog

Updated the changelog, not sure if its a new point release, please edit if needed
Also should we add #191 and #192 to the changelog too?

Also should we add https://github.com/colopl/laravel-spanner/pull/191 and https://github.com/colopl/laravel-spanner/pull/192 to the changelog too?

Yes but I'd like to have a different PR for them.

Yes but I'd like to have a different PR for them.

Do you want me to do anything or you got it?

I'd appreciate it if you can write the ones for the previous two. Thanks.

Which branch should I do it on?

master for all unreleased features.


Add generics to randomGenerator.element()

This change allows code like this, that would normally cause either cannot find declaration to go to or getter `values` isn't defined for the class Object, to resolve properly:
var items = [
  {'a': 'foo', 'b': 'bar'},
  {'a': 'zip', 'b': 'zap'}
]
var chosenItem = faker.randomGenerator.element(items).values;

@drager it looks like the failed check above is due to a bad configuration for TravisCI...

Thanks for the PR @KOGI! I think the code looks good! I will check out the Travis error.


implement abuse protection

This implements this part of the CloudEvent spec.
Really, this gateway doesn't work with fully-CloudEvent 1.0-compliant event sources (such as Azure Event Grid, for example) without this change, so I'm going as far as to count this as a bug fix.

Changing this to a draft because Azure Event Grid doesn't seem to like POST callbacks.
Note there's ambiguity in the spec for which I have raised an issue: https://github.com/cloudevents/spec/issues/1018

Changing this to a draft because Azure Event Grid doesn't seem to like POST callbacks. Will open another PR with an alternate strategy.

Note there's ambiguity in the spec for which I have raised an issue: https://github.com/cloudevents/spec/issues/1018

While this ambiguity does exist, I've determined through further troubleshooting that if we are too quick to execute the validation callbacks, Azure Event Grid responds with a 200, but doesn't actually complete the handshake.
A short delay resolves that problem for Event Grid without introducing problems for any other provider because the whole point of the callbacks in the first place is to implement an asynchronous handshake.
This makes #69 the superior alternative because the more custom implementation offers us the ability to incorporate that short delay. Closing in favor of #69.


Randomly failing UI tests, when upgrading to OC 1.5 (OSOE-483)

In some cases, the browser navigation results in HTTP 500. In this state, there are no logs under FaliureDump folder, and looks like the web app won't start.
Related:

https://github.com/Lombiq/UI-Testing-Toolbox/pull/225
https://github.com/Lombiq/UI-Testing-Toolbox/pull/225#discussion_r1019797990

Jira issue

After I collected the logs of many failed runs, I found errors only in the internal-nlog.txt, like below:
...
2022-11-24 20:18:24.4539 Error Failed to add type 'NLog.Web.LayoutRenderers.AspNetRequestConnectionIdLayoutRenderer'. Exception: System.InvalidOperationException: Operations that change non-concurrent collections must have exclusive access. A concurrent update was performed on this collection and corrupted its state. The collection's state is no longer correct.
   at System.Collections.Generic.Dictionary`2.TryInsert(TKey key, TValue value, InsertionBehavior behavior)
   at System.Collections.Generic.Dictionary`2.set_Item(TKey key, TValue value)
   at NLog.Config.Factory`2.RegisterDefinition(String itemName, Type itemDefinition, String assemblyName, String itemNamePrefix)
   at NLog.Config.Factory`2.RegisterType(Type type, String assemblyName, String itemNamePrefix)
   at NLog.Config.Factory`2.ScanTypes(Type[] types, String assemblyName, String itemNamePrefix)
2022-11-24 20:18:24.4540 Error Failed to add type 'NLog.Web.LayoutRenderers.AspNetRequestConnectionIdLayoutRenderer'. Exception: System.InvalidOperationException: Operations that change non-concurrent collections must have exclusive access. A concurrent update was performed on this collection and corrupted its state. The collection's state is no longer correct.
   at System.Collections.Generic.Dictionary`2.TryInsert(TKey key, TValue value, InsertionBehavior behavior)
   at System.Collections.Generic.Dictionary`2.set_Item(TKey key, TValue value)
   at NLog.Config.Factory`2.RegisterDefinition(String itemName, Type itemDefinition, String assemblyName, String itemNamePrefix)
   at NLog.Config.Factory`2.RegisterType(Type type, String assemblyName, String itemNamePrefix)
   at NLog.Config.Factory`2.ScanTypes(Type[] types, String assemblyName, String itemNamePrefix)
2022-11-24 20:18:24.4625 Error Failed to add type 'NLog.Web.LayoutRenderers.AspNetMvcActionRenderer'. Exception: System.InvalidOperationException: Operations that change non-concurrent collections must have exclusive access. A concurrent update was performed on this collection and corrupted its state. The collection's state is no longer correct.
   at System.Collections.Generic.Dictionary`2.TryInsert(TKey key, TValue value, InsertionBehavior behavior)
   at System.Collections.Generic.Dictionary`2.set_Item(TKey key, TValue value)
   at NLog.Config.Factory`2.RegisterDefinition(String itemName, Type itemDefinition, String assemblyName, String itemNamePrefix)
   at NLog.Config.Factory`2.RegisterType(Type type, String assemblyName, String itemNamePrefix)
   at NLog.Config.Factory`2.ScanTypes(Type[] types, String assemblyName, String itemNamePrefix)
...

I started from here and added a lock around Microsoft.AspNetCore.Mvc.Testing.WebApplicationFactory.CreateHost(IHostBuilder builder) in Lombiq.Tests.UI.Services.OrchardCoreHosting.OrchardApplicationFactory.CreateHost(IHostBuilder builder), because the CreateHost invokes Microsoft.Extensions.Hosting.HostFactoryResolver.CreateHost() which starts the web application in a new thread, and finally this can result in exceptions as I found in the internal-nlog.txt in case of parallel execution of tests.
After I added the change above, I wasn't faced with the issue again. I ran the tests locally and on GHA without any issues. See: https://github.com/Lombiq/Open-Source-Orchard-Core-Extensions/pull/307

Great work, Sherlock! 🧐

Fixed by https://github.com/Lombiq/UI-Testing-Toolbox/pull/240.


libxml2 requirement for pdf2xml on Linux

libxml2 appears necessary on Linux for having GROBID running. Let's try to reproduce the problem. If confirmed, we should recompile pdf2xml for Linux to avoid this dynamic dependency (and update the documentation in between ;).

How to install libxml2 on centos ? and  How to modify config libxml2 load path ?

yum install libxml2?


How to template this structure

How would my template look like if I have the following structure:
{
    "data": [
        {
            "firstname": "Test",
            "lastname": "Test",
            "email"<EMAIL_ADDRESS>            "departments": {
                "data": [
                    {
                        "id": "1234",
                    },
                    {
                        "id": "4321",
                    }
                ]
            }
        }
    ]
}

to
"user" : {
    "profile":{
        "firstname": "Test",
        "lastname": "Test",
        "email"<EMAIL_ADDRESS>        "departments": [
              {
                  "id": "1234",
               },
               {
                   "id": "4321",
                }
            ]
    },
    "departments": [
        {
             "id": "1234",
         },
         {
             "id": "4321",
          }
      ]
}

@jake142 I haven't used this in a long time -- did you figure this out?
Please update the README with any examples or notes if you did.
Thanks!


Add all team members into BDD repository

Add all team members into BDD repository

Added via https://github.com/RedHatInsights/insights-behavioral-spec/pull/77


[WIP] SPARK-9384: Easier setting of executor and driver classpath

Adding new property spark.common.extraClassPath for adding to both the driver and executor classpath.
More testing needs to be done.
Currently one can set extra jars in the driver classpath using spark.driver.extraClassPath property. Similarly, the executor's extra classpath can be set using spark.executor.extraClassPath property.
However, many users (myself) included have often suffered from the dogma of setting one and missing the other. Causing a lot of wasted time. And, this JIRA is an effort to see if the status quo can be made better.
--jars sets the classpath set by the classloader but there is no such way of setting the system classpath of the driver and the executor. This change would allow users to override libraries shipped with Spark, or add jars like some JDBC driver, that don't work unless they are adding to the system classpath.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Jenkins, ok to test

ok to test.

Merged build triggered.

Merged build started.

Test build #38832 has started for   PR 7739 at commit 5173f99.

Test build #38832 has finished for   PR 7739 at commit 5173f99.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

I will take a look at the failures.

Merged build triggered.

Merged build started.

Test build #38871 has started for   PR 7739 at commit 62498c3.

Merged build triggered.

Merged build started.

Test build #38876 has started for   PR 7739 at commit 0717f16.

Test build #38876 has finished for   PR 7739 at commit 0717f16.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

sorry, i broke the jenkins config and have since reverted my change.  allow me to retrigger your build...  :)
jenkins, test this please

Merged build triggered.

Merged build started.

Test build #38871 has finished for   PR 7739 at commit 62498c3.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Merged build finished. Test FAILed.

Still looks like a file missing in Jenkins workspace:
13:55:21.583 ERROR org.apache.spark.executor.Executor: Exception in task 0.0 in stage 11926.0 (TID 84068)
java.io.FileNotFoundException: File file:/home/jenkins/workspace/SparkPullRequestBuilder@3/target/tmp/spark-ce042421-1af0-42fd-a04a-892d72b3df72/tmp/part-00003 does not exist

Merged build triggered.

Merged build started.

Test build #39294 has started for   PR 7739 at commit 51a98b4.

Test build #39294 has finished for   PR 7739 at commit 51a98b4.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Build triggered.

Build started.

Test build #39398 has started for   PR 7739 at commit d905b92.

Test build #39398 has finished for   PR 7739 at commit d905b92.

This patch fails Scala style tests.
This patch does not merge cleanly.
This patch adds no public classes.

Build finished. Test FAILed.

Build triggered.

Build started.

Test build #39591 has started for   PR 7739 at commit 8ebf8e1.

Test build #39591 has finished for   PR 7739 at commit 8ebf8e1.

This patch fails Scala style tests.
This patch does not merge cleanly.
This patch adds no public classes.

Build finished. Test FAILed.

Build triggered.

Build started.

Test build #39598 has started for   PR 7739 at commit 163e8f6.

Build triggered.

Build started.

Test build #39603 has started for   PR 7739 at commit d81bfa3.

Test build #39598 has finished for   PR 7739 at commit 163e8f6.

This patch passes all tests.
This patch does not merge cleanly.
This patch adds no public classes.

Build finished. Test PASSed.

Test build #39603 has finished for   PR 7739 at commit d81bfa3.

This patch passes all tests.
This patch does not merge cleanly.
This patch adds no public classes.

Build finished. Test PASSed.

Hey @markgrover , could you rebase your local repository and then apply your change?
The lineage is messy.

Sure thing. Will do!
On Aug 3, 2015 10:18 PM, "Kousuke Saruta"<EMAIL_ADDRESS>wrote:

Hey @markgrover https://github.com/markgrover , could you rebase your
local repository and then apply your change?
The lineage is messy.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/7739#issuecomment-127482044.

Hi @sarutak, thanks again for taking a look, I've rebased and fixed up the history. I have also update the PR with some more info.

Merged build triggered.

Merged build started.

Test build #39716 has started for   PR 7739 at commit e33a404.

Test build #39716 has finished for   PR 7739 at commit e33a404.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Seems unrelated failure. I don't think I am on the whitelist but I will try this:
Jenkins, test this please.

If that doesn't work and some can retrigger the test run on my behalf, I'd appreciate that. cc @squito @sarutak

Jenkins, add to whitelist

Jenkins, retest this please

Merged build triggered.

Merged build triggered.

Thanks @squito !

Merged build started.

Merged build started.

Test build #231 has started for   PR 7739 at commit e33a404.

Test build #39860 has started for   PR 7739 at commit e33a404.

Test build #231 has finished for   PR 7739 at commit e33a404.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test build #39860 has finished for   PR 7739 at commit e33a404.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Jenkins, retest this please

Merged build triggered.

Merged build started.

Test build #40171 has started for   PR 7739 at commit e33a404.

Test build #40171 has finished for   PR 7739 at commit e33a404.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

I am taking a look.

Hmm, looks like PYTHONPATH is not being set up properly by some python tests. It turns out I am not the only suffering from this - the build before mine (https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40170/console) failed because of the same reason. I am working on something else at the moment but if it's not fixed by the time I get back to this, I will take a look at the orthogonal cause that's causing all these test builds to fail.

Jenkins, retest this please.

This looks pretty good. Could you also add some tests to each of the components (YARN/standalone and mesos to ensure the correct classpaths get passed in the correct order?)

Thanks @harishreedharan and @squito for your reviews!

Build triggered.

Build started.

Test build #41272 has started for   PR 7739 at commit 8740751.

Hey @harishreedharan about

Could you also add some tests to each of the components (YARN/standalone and mesos to ensure the correct classpaths get passed in the correct order?)
I added some tests with the original pull request. Could you please help me better understand where exactly in the code you'd like me to add new tests? Thanks!

Build triggered.

Build started.

Test build #41275 has started for   PR 7739 at commit 5f62d39.

Merged build triggered.

Merged build started.

Test build #41277 has started for   PR 7739 at commit bca4fa3.

Test build #41272 has finished for   PR 7739 at commit 8740751.

This patch fails Spark unit tests.
This patch does not merge cleanly.
This patch adds no public classes.

Build finished. Test FAILed.

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

Test build #41275 has finished for   PR 7739 at commit 5f62d39.

This patch passes all tests.
This patch does not merge cleanly.
This patch adds no public classes.

Build finished. Test PASSed.

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

Test build #41277 timed out     for PR 7739 at commit bca4fa3     after a configured wait of 175m.

Merged build finished. Test FAILed.

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

Jenkins, test this please.

Since most of the changes don't affect the cluster managers directly, and spark submit changes have tests, I think this is good (I was confused by the one change in YarnClusterSuite).

Thanks Hari! The YarnClusterSuite change is rather benign and not absolutely necessary. So, based on this I believe this request is ready to be merged.
And, for some reason, Jenkins is not listening to me, so if someone could trigger another test run, I'd appreciate that.

Looks sane but there's some room for cleanup; also, need to resolve conflicts.

I'm not super convinced that we should add something like this. This patch introduces a config that's not really necessary and has potentially unclear semantics. E.g. What happens if I specify different versions of some jar on both common and executor class path? Which one gets loaded first? What if I pass in --common-class-path and set spark.executor.extraClassPath at the same time, is it the same?
Either way this patch has gone stale over time and has not been active for more than 2 months. I would recommend that we close this patch for now and re-open it later if there is more interest.

Hey Andrew, thanks for your feedback.

What happens if I specify different versions of some jar on both common and executor class path? Which one gets loaded first? What if I pass in --common-class-path and set spark.executor.extraClassPath at the same time, is it the same?

The semantics are that common classpath is always added before the individual driver/executor classpath. I can clarify that in the documentation.
As far as your other point about two versions of the same jar goes, one could do that just by using spark.[driver|executor].extraClassPath too and the same logic would apply.
I was actually planning on updating this patch this week before the holidays roll in so I'd definitely vouch for not closing this. My motivation here is that many spark users, want to update both their driver and executor classpaths and either forget, miss or find cumbersome to update 2 different places for doing so. Common classpath would be one single go-to place for such users.
Would you be open to taking another look at this once I update this patch this week?

Sure, I wasn't sure if this would be updated any time soon.

I'm a bit on the fence about this.  Its kind of bringing us full circle to what SPARK used to have when they had env variables, most of those applied across everything.  I think we chose to do them separate so users had the option of setting different ones.   I see the case for this makes it easier for users in some cases but I also see it as possibly confusing and just more maintenance and complexity in the spark code. My other concern here is now this has different semantics then extraJavaOptions and extraLibraryPath as they won't have a common option.
cc @pwendell as I think he originally added the configs to see if he remember any discussion around common ones.

I see the case for this makes it easier for users in some cases but I also see it as possibly confusing and just more maintenance and complexity in the spark code.

That was exactly my concern as well. We had to go through the pain of deprecating most of those overly general environment variables but still keeping them around for backward compatibility.

I actually haven't seen a case of users needing different class paths for the driver and executor; in general, the executor class path is either the same or a subset of the driver's, so that using the same in both places leads to the desired behavior.
On the flip side, I've seen many people try to change just the driver class path and still run into errors because they needed the same change on the executor's configuration.

Eh ... I didn't see much of a resolution on this. Does anyone feel weakly, moderately, strongly for/against this?

I still think this is a good idea; just need to clearly define the semantics (in case they aren't). e.g. driver.extraClassPath should be added before common.extraClassPath so it's possible to override classes in the common config.

Thanks for bringing this up, Sean. Yeah, I'd like to see this go in too. I realize there are some merge conflicts here. I will make the semantics clear in documentation (they are already there, will make sure they are clear enough).
So, if someone is strongly against, let me know. Otherwise, I will be updated the code early next week.

I have seen cases where driver and executor needs different classpath, mostly in client mode.  gateways/launcher boxes can have different setup then hadoop nodes.  I have seen other people in jira/mailing lists mention this also.
I think I still have the same concerns.  Is the benefit here over possible user confusion worth it?  If the semantics are all clearly defined and its clear to the user and you can see this helping a bunch of your users then I'm OK with it.

Test build #66133 has started for   PR 7739 at commit bca4fa3.

Test build #66133 has finished for   PR 7739 at commit bca4fa3.

This patch fails SparkR unit tests.
This patch does not merge cleanly.
This patch adds no public classes.

Build finished. Test FAILed.

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


Ai90minlabs

@amar-ibm
Please review Pull Request and proceed to merge to Main.
Let me know if you find anything else.
Thanks.

@amar-ibm
FYI.  I added one missing file in the Topology lab.  ( 4th commit)
Thanks.


[SUGGESTION] include Patchouli support

I think It would be awesome if you could add Patchouli support for this mod, not saying that the already existing manual isn't cutting it, but Patchouli is more advanced and provides some really nit features that would really benefit this mode, plus it's a fully dedicated mod just for the purpose of providing the best user experience out there.

Mmmhh no

Well well well... look at you now :)


Add basic session log support

Hound already contains a helper to fetch browser logs (console.log output), but this doesn't cover more advanced use cases, like the performance logging implemented in Google Chrome exposed over the log feature trough ChromeDriver (https://sites.google.com/a/chromium.org/chromedriver/logging/performance-log) or the browser driver log
This PR adds two generic functions to fetch logs of a specific type and a list of all supported log types for a particular session.

Thank you :smile:


There is no field named onTap.

The widget, does not have a field called onTap. I need its. Please

use a focusNode


Breaking Change: Make Warehouse Optional

Queries like SHOW TABLES IN ACCOUNT are perfectly valid without a warehouse provided.

Thanks for getting stuff merged so fast last time, I will get to fix the tests soon!

thanks!


⚠ Bofrost.fr has degraded performance

In e5c43f0, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1577 ms

Resolved: Bofrost.fr performance has improved in 140e655.


Use leveldb / levelgraph / levelgraph-n3 / levelgraph-jsonld

Will allow for increased functionality by incorporating the complete MeSH descriptor records. Will leave out qualifier and supplementary concept records for now.

+1
Also, have you seen https://github.com/HHS/meshrdf ?

Will definitely look into it. My goal for restructuring MeSH into leveldb is to have a fast, persistent datastore to help support classification tasks. Using levelgraph and levelgraph-jsonld, we can better incorporate concept qualifiers, which will be huge - and possibly essential - for context disambiguation.

Using levelgraph-n3 and levelgraph-jsonld. MeSH as RDF n-triple file is ~1.8 GB. Note memory issue with levelgraph-n3 https://github.com/mcollina/levelgraph/issues/84. Will look into this further.
Will also look into linked data fragments.

No issues with levelgraph-n3 data import. Merged branch 3-leveldb into master


pthreadVC2.dll not found in Windows

I have sucessfuly made a build using Visual studio in my Windows 10.
You project Darknet is succesfully working.
But this project is giving dll file pthreadVC2.dll not found error. Do you have any solution?

same error

@USMANHEART @chiaqf Hi,
I added pthreadVC2.dll to the /bin/ directory

@AlexeyAB I found the pthreadVC2.dll in the /3rdparty/pthreads directory on my local darknet repository that shows the same error as the asker... should I re-pull and do the same process all over again (clone, CMAKE, VS build)?

@dreistheman
There is no CmakeList.txt in this repostiory. Just download the latest yolo2_light and build yolo_gpu.sln then run /bin/yolo_gpu.exe

@AlexeyAB Oh sorry! I thought this was in your Darknet repository. I am having this problem but in Darknet. Can you give a suggestion how to solve it?


.?serverwarns only for staffmembers

Proposal:
Proposal written by Dadi
What is the title of your proposal?: .?serverwarns only for staffmembers
Is it a suggestion for the website or for the bot?: Bot
Explain your proposal more accurately (It's best to give as much information as possible, so that we can implement the proposal better): .?serverwarns should be only for staffmembers with a certain permission. Like kick and ban perms
Why should we add this feature?: Why should normal users see the warns of other user?
ReportID: 110

➤ LenoxBot commented:
?serverwarns should be also moved from utility to moderation
Dadi#7808

➤ LenoxBot commented:
Approve:
good question (=good proposal)
Monkeyyy11#0001

➤ LenoxBot commented:
Approve:
agree with Monkey and Dadi
Profxy#5396

Command has been removed


add HTTP ranger support

I am adding support for HTTP range support so a database can be read from the HTTP server.
The benchmark shows it is "fast", but testing HTTP on the local networks doesn't make for realistic conditions.

Added. It is not realistic at the moment. On small files, maybe a 1MB, it works. On a 10GB file, things started to timeout.

The benchmarks seem to be misleading or I am reading them wrong.

Benchmarks are because it is done against an HTTP server running locally. When used in production, obviously, it is much slower. Closing.


No i2c communication with Adafruit Feather Board

When I run one of the example for adafruit feather board esp32 then nothing happen on the i2c lines.
I looked through the code (i'm working with visual micro) I couldn't find any differences to the example for RTC pcf8523, wich works perfectly.
Is this library not made for other boards?
I'm looking hard for a library for the lsm303c. This is the only one I could find.
Thanks for any help.

Hi,
I'm sorry for the delay in the reply. I was closing another issue when I noticed your question. Try redownloading the latest library. The Wire.begin() was in the .cpp file causing it to reset the bus. I remember seeing this in another issue. I do not have a Feather Board with ESP32 to verify. By moving the Wire.begin() into the setup, it fixed some of the issues that I saw in another library.
I'll close this issue for now but if you still have problems, we can re-open this issue.


Terrain data is missing

The terrain data seems to have been removed, and there are no alternate sources such as with image data.

Thanks Clint. Unfortunately I had to take down the data because somebody released an app that linked to it and it would have cost me a bundle to keep it up.
I could not find good notes on how I generated it, but I basically took the NASA DEM data, converted it to grayscale using GDAL, and then used MapBox's TileMill to tile and upload it.

@clintburgos  @RossAnderson  http://fortrangis.sourceforge.net/doc/globe.html  maybe this website helps.    and i write a python script to automate download this data and meta data https://gist.github.com/beyoung/7761608a3510e1ea24915eeb7b6a41fb