Ensure socket owner always exits

Closes #265.

Pull Request Test Coverage Report for Build 91193c3b76e371ee95293aa25e67bb6e9dd4057c-PR-266
Details

0 of 6 (0.0%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.7%) to 85.356%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/redix/connection.ex
0
1
0.0%

lib/redix/socket_owner.ex
0
5
0.0%

Files with Coverage Reduction
New Missed Lines
%

lib/redix/connection.ex
1
76.74%

Totals

Change from base Build 4a8537a37deeb491ddf593f172cbd3c7543a5f1a:
-0.7%

Covered Lines:
612

Relevant Lines:
717

💛  - Coveralls


Fix floyd_warshall_numpy dispatcher

Fix bug found during #291 benchmarking.
floyd_warshall_numpy was dispatching to the wrong function and calling adjacency_matrix instead of graph_floyd_warshall_numpy and digraph_floyd_warshall_numpy.

Pull Request Test Coverage Report for Build 720550595

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 96.686%

Totals

Change from base Build 719210672:
0.0%

Covered Lines:
6155

Relevant Lines:
6366

💛  - Coveralls


Missing ca-certs in docker image

Describe the bug
ImageUpdater doesn't work with public docker images using the default configuration
To Reproduce
Deploy ImageUpdater without any registry configuration, add argoCD application with public docker hub image and annotate this app for imageupdater
Expected behavior
ImageUpdate should work with public docker hub images out of the box
Additional context
because ca-certificates are not in the docker images, docker hub ssl cert can't be verified
Version
v0.12.0
Logs
msg="Could not get tags from registry: Get \"https://registry-1.docker.io/v2/\": x509: certificate signed by unknown authority"

Same thing happend to me today


Daily Notes Folder Configuration

It would helpful to have a configuration to set a file location for daily notes instead of them always being created in the project directory.
ex: memo.dailyNotesDirectory: "file path"

hi, please take a look at link rules.


Update MyPlot.php

im lazy

[x] This PR actually submits something - don't use this system as a messaging service!
[x] This PR isn't duplicated - you can check if it is by scanning the small list - link is on the navigation bar for this repository.
[x] This PR includes appropriate markdown for sections - e.g. code blocks for suggested code.
[x] This PR description and comments in code is understandable - feel free to use your native language to write if you are not comfortable with English.
[x] This PR has a single branch for itself in your fork - don't just commit to the master branch of your repository!
[x] This PR has comments written in clear English - please don't write unreadable comments just for your eyes.

Pull Request description
{Type something here}

Optional information
{Type something here}

You can use this for yourself, but I'm not going to support it. Nor am I going to support any unreleased PocketMine builds


The @internal doc flag recognised by tsc should be an alias for @hidden

The @internal flag is used by tsc --stripInternal to exclude a particular property from a generated declaration file. If you're producing a documented library with public methods for consumption this is basically the same as the @hidden flag you'll want to use in your typedoc.
It would be great if I could just use @internal and have it work on both my .d.ts and typedoc output.
I realise PRs are needed to make progress on typedoc at the moment but I thought I'd put this up as an issue in case anybody has any better ideas!

Great idea! Looks like these lines are where the @hidden tag is recognized.
Should the @internal docs only be ignored with the --stripInternal flag?

Should consider adding @ignore to the possible aliases. See #198

I wrote a plugin for my project which allows me to mark specific code as @internal or @external.  The plugin also allows you to specify aliases for each.
https://github.com/christopherthielen/typedoc-plugin-internal-external
https://github.com/christopherthielen/typedoc-plugin-internal-external#annotation-aliases

@christopherthielen I noticed your plugin just this week. The aliases look like a great opt-in way to workaround this so thanks for sharing. I think it would be great to have some documentation pointing people to 3rd party plugins like this.
I've been hesitant to merge #525 because @internal was designed by Microsoft with a slightly different use case. My understanding of the TypeScript feature is limited and I want to avoid aliasing it to a concept which may not be the same. Additionally, TypeScript has a --stripInternal flag which I think TypeDoc should respect. Users may have the @internal doc flag used in their code but the strip internal flag turned off. Or users may want simpler declaration files but a more complete documentation website.
Actions steps I think should be taken

[ ] Symbols with @internal doc flag should be removed when the TypeScript --stripInternal flag is set
[ ] Look into allowing themes tracking/setting the visibility of internal apis (based on the @internal flag)
[ ] Review and document how external/internal code (as TypeDoc currently defines) is distinguished

TypeDoc doesn't produce declaration files, so I don't think it's a TypeScript bug.
The TypeScript compiler options page doesn't list all options, nor does tsc --help. If you want to see all options (which does list --stripInternal) you have to run tsc --help --all


Allow mocking methods with up to 22 parameters

Mocks and stubs are not limited with 9 parameter per method anymore, the "standard" 22 parameters are supported.
The change includes unit tests for traits and classes containing methods with up to 22 parameters.

This is awesome, many thanks!
One request before I merge this, could you add a copyright/license header to the new files you created (you can copy it from any existing file)

All done, thanks for accepting and let me know how can I get it from maven central ;)

Did this ship? I am using org.scalamock:scalamock-scalatest-support_2.10:3.2 and getting the limit 9 error. This is a show stopper for us using this framework.


No notification in iOS

Notification is not coming for iOS sample.Is there any restriction related to ios version.Tested with ios 10 and 11.

@pkiranprusty The existing API is deprecated for iOS 10+.

@siddeshpillai Is there any alternative API which will support all the ios version?

@pkiranprusty sorry for the delay in response. The newer APIs are https://developer.apple.com/documentation/usernotifications

@pkiranprusty The source is updated. Kindly verify whether it is working on iOS.

was not working on android, went to "QtMobileNotificationExample.pro" and added the "ANDROID_PACKAGE_SOURCE_DIR = $$PWD/android"
there it worked normal,
Thanks

@diegothome Can you submit a PR with the fix?

Fixed


OC monthly report issues

As a host (OC UK), I notice some issues with the monthly report that ensure the rec between the collectives fundings and expenses
1) Some funding/income have negative hostfee, platform fee
Other don't, so I don't understand the logic here.
example ID: 59194; 65288. Income of 200 GBP but former has positive fee, later negative.
2) missing transactions
For collective Agreelist, sum of monthly report (up to July 2018 included) indicates 9 expenses for a gross amount of 692.47GBP, although bank account statement recognises 16 expenses for a gross amount of 1110 GBP. The latter matches the collective webpage
I just did the rec for 1 collective. Other might be impacted too.
Required improvement :
1) Ability to download a date range of OC transaction
Host account do not end up in the same date (ie from Jan to Dec), so we should be able to download report for the data range on which the account must be submitted.
Also, currently we only receive monthly report, having a yearly one that matches accounting can be very useful
2) having a flag Income/ expense to identify the side of the transaction.
It will simplify the rec by filtering on it, instead of positive or negative amount
3) format field createdAt as date (DD/MM/YYYY)
This field can't be used for filtering or anything as the format is useless. Either create a new field for the valid date format, or change this one so filter can apply.
Another field might be required for the time of the transaction. Standard date field is really needed here
Thanks in advance for looking at improving the report
I really require the above change, so to simplify the accounting
201808 OC Report Rec.xlsx

[ ] Positive host / platform fees -> negative: done!
Missing transactions: they are in the DB. Need to double check export.

TODO

[ ] createdAt should be in the format YYYY-MM-DD
[ ] add DEBIT or CREDIT column
[ ] download transactions CSV at the host level for any date range

Not sure why this one is closed as in latest report, there are still missing transaction from 'a Man's world'.
Furthermore, there is no ability to a date range of OC transaction on OC UI for host


provider: add in rinkeby url

Description
This adds in the url for the public ropsten node so that the provider can be instantiated like:
const provider = new OptimismProvider('rinkeby')

This would set the hostname as rinkeby.optimism.io.
Also notes this in the README.md
Questions

What is the actual url that is going to be used?

Metadata
Contributing Agreement

[x] I have read and understood the Optimism Contributing Guide and Code of Conduct and am following those guidelines in this pull request.

CI failing on this step: https://github.com/ethereum-optimism/optimism-monorepo/blob/f443d93c50af3d37c6b8a140a469fb4477543b33/.github/workflows/integration-test.yml#L27


Example for operation on existing data buffer

I have a data buffer in memory (OpenCV Mat) that I want to run a Gaussian filter on. The example set currently doesn't give details on how to reuse existing buffer in memory. I plan to use ARM Compute Library side by side with OpenCV. Having to perform Tensor::allocator()->allocate() then copy over the data is slow.

In the next release we'll add the possibility to import existing allocations, however, correct me if I'm wrong, but I don't think OpenCV matrices usually use any padding ?
In which case you will be able to use OpenCV allocations for some computer vision algorithms (But the pixels near the borders will not get processed) and you will not be able to use it for machine learning algorithms (As it doesn't make sense to process only part of a matrix for machine learning)

OpenCV matrix does use padding to make the read of each row aligned. Its cv::Mat class doesn't have the concept of offset-from-begin though. Each header already points to where the data starts.

I guess you'll have to try and see if it works for you then, it's hard for us to work around this.

I will give it a try on my own. Will providing an implementation of arm_compute::core::ITensor that wraps an existing buffer (OpenCV's Mat being the prime example) still be on the roadmap for next release?

Yes, this will be part of the next release (At some point this month)

Feature added: closing


Why is there no license for the repo?

What is this repo licensed under?
Thank You.

Its licensed under MIT license.

https://github.com/slimphp/Slim-Skeleton/blob/master/composer.json#L6

Thanks guys.


Burp ask me for activation for systemctl when it was already activated for my linux user.

I create a daemon via systemctl to run burp-rest-api, but when I make systemctl start daemon.service and check it status, i see, that a Burp ask me for activation when it was already activated for my linux user. What I do wrong?..
Config:
[Unit]
Description=Burp RestAPI Service
[Service]
Type=simple
ExecStart=/usr/local/BurpRest/burp-rest-api.sh
Restart=on-failure
[Socket]
ListenStream=--host--:--port--
Accept=yes
[Install]
WantedBy=default.target
I could not map https://github.com/vmware/burp-rest-api/blob/master/FAQ.md to my situation. I have ~/.java/.userPrefs/burp/prefs.xml but it not works. Or this question for guys from PortSwiger?..

This daemon is probably executing as root. As such, that user doesn't have the license file.


Size incompatibility

Thank you so much for sharing
I am trying to retrain your network on the PBVS dataset, and I tried running your code without any modification, but I get this error:
Traceback (most recent call last):
File "main.py", line 185, in
main()
File "main.py", line 96, in main
L1_loss = tf.compat.v1.losses.absolute_difference(y, t)
File "C:\Users\moaaz\anaconda3\envs\TherISuRNet\lib\site-packages\tensorflow_core\python\ops\losses\losses_impl.py", line 253, in absolute_difference
predictions.get_shape().assert_is_compatible_with(labels.get_shape())
File "C:\Users\moaaz\anaconda3\envs\TherISuRNet\lib\site-packages\tensorflow_core\python\framework\tensor_shape.py", line 1115, in assert_is_compatible_with
raise ValueError("Shapes %s and %s are incompatible" % (self, other))
ValueError: Shapes (4, 192, 192, 3) and (4, 256, 256, 3) are incompatible
Trying to find what is the cause of the problem, I found that y and t in main.py Line#92 are of different sizes. Digging further into it, I found that in model.py Line#75, you use the img_size3, img_size3 , where in model.py Line#95 you use size=[h * 4, w * 4] , and that is why y and t are resulting in different sizes.  Is that how it is supposed to be?
I wonder if you can help me with it.

Thanks for your query.
Actually, one correction is there. I have corrected it. In the main path, I have upscale the feature maps with pixel shuffle operation using factor 2 (i.e., line number 81 from model.py file) while in the global residual learning path (i.e., line number 95 from model.py file), I have upscale the feature maps with factor 4.
So in order to solve this, two options are there.

You just change the upscale factor value from 2 to 4 (i.e., at line number 81 from model.py) as
x = tf.depth_to_space(x, 4)

or

Just use that command two time as listed below (line number 114 -115 from updated model.py file):

xUP = self.Upsample2xBlock(x4, kernel_size=3, filters=64, strides = 1)
xUP = self.Upsample2xBlock(xUP, kernel_size=3, filters=64, strides = 1)
so that the feature maps from the main path are upscaled with factor 4.
Another correction is from main.py file. It was created for upscale factor 3. So I have changed few sentences as listed below:

line number 75,  t = tf.placeholder(tf.float32, [bs, img_size4, img_size4, 3])
line number 149,  X_ = cv2.resize(X_,(img_size24,img_size24),interpolation = cv2.INTER_CUBIC)
(earlier the img_size was multipled with 3)


Uncaught Error: spawn E2BIG

[Enter steps to reproduce:]

...
...

Atom: 1.16.0 x64
Electron: 1.3.13
OS: Mac OS X 10.11.6
Thrown From: context-git package 0.15.8
Stack Trace
Uncaught Error: spawn E2BIG
At /Applications/Atom.app/Contents/Resources/app.asar/src/buffered-process.js:376

Error: spawn E2BIG
    at exports._errnoException (util.js:1026:11)
    at ChildProcess.spawn (internal/child_process.js:313:11)
    at Object.exports.spawn (child_process.js:392:9)
    at BufferedProcess.spawn (/app.asar/src/buffered-process.js:305:51)
    at BufferedProcess.start (/app.asar/src/buffered-process.js:111:14)
    at /app.asar/src/buffered-process.js:96:12)
    at /packages/context-git/lib/git-cmd.js:21:15
    at Object.cmd (/packages/context-git/lib/git-cmd.js:19:10)
    at Object.add (/packages/context-git/lib/git-cmd.js:111:15)
    at /packages/context-git/lib/commands.js:49:18

Commands
     -2:33.2.0 save-workspace:open (input.hidden-input)
     -2:25.4.0 context-git:commit (span.name.icon.icon-repo)
  2x -1:26.1.0 core:confirm (input.hidden-input)
     -0:50.9.0 context-git:commit (span.name.icon.icon-repo)

Non-Core Packages
busy-signal 1.4.1
context-git 0.15.8
git-control 0.9.0
git-log 0.4.1
intentions 1.1.2
linter 2.1.4
linter-js-standard 3.9.1
linter-ui-default 1.2.4
php-server 0.7.0
pretty-json 1.6.3
process-palette 0.12.0
save-workspace 0.4.0
termination 0.5.1
tree-view-git-status 1.3.1

This happens when committing too many files at once because it has to list all of the files on the command line
same as #16

there are about 3 files.  not a problem though, I removed the package and am using a different one.  The problem went away


Added screenshots

@MichaelPesce I tried adding sensitivity analysis but BSM2 is very finicky.
Screenshot in next comment...


Fancy zones show on all monitors while dragging window

Before fancyzones showed only on one monitor while dragging window.
Added support for showing on all monitos

Please do this into master, not stable

@crutkas
we closed this and open the new one for master.
I was implemented from stable to not having to worry for possible instability from the latest commits in master.


convert from nuget- read target framework from packages.config file -

references #609 #759
Not sure if this is enough, couldn't find anywhere info re the targetFramework attribute possible values.
Tested on a simple packages.config with net451 requirement

@Thorium interested in beta-testing this?

I'll test...

If you have multiple projects having package.config files, some of them use .Net 4.5, some Portal, some Silverlight, etc.
And you use technologies like Castle-IoC or Rx, that are used in many frameworks.
Now it just parses one of the frameworks and forgets other ones.
nuget Castle.Core 3.3.0 framework: sl50
This is wrong, I would say this is not ready yet.
But what should actually happen in multi-case?
a) remove the target totally
b) somehow include list of different targets
c) add multiple lines to paket.dependencies and paket.lock

I'm voting for a)

It forgets the others because it takes only the one with highest version of the package - there's a yellow warning when you have multiple versions for same package

I have same versions, but still:
c:\MyApp\UI\package.config:

c:\MyApp\Logics\package.config:

result of c:\MyApp\paket.dependencies is:
nuget Rx-Core 2.2.5 framework: sl50

Ok so Then in case of multiple different targetframework should we omit the framework restriction?

I also vote for a)
as it is simple and clear solution.

ok so here it comes, rebased on master - @Thorium is this acceptable now?

LGTM

Thanks Tomasz!

 ;)

Ok, one more problem by this conflicts of nuget targetframework and the real TargetFrameworkVersion.
If I make a new C#-class library and add reference to for example NServiceBus.Log4Net, by-default Visual Studio will make packages.config have:
    <package id="NServiceBus.Log4Net" version="1.0.0" targetFramework="net45" userInstalled="true" />

Then if people update the .NET-version of their projects, by re-targetting to .NET 4.5.1, Visual Studio will modify the .csproj-file to have <TargetFrameworkVersion>v4.5.1</TargetFrameworkVersion>  but the packages.config is not updated. The project still builds and works ok.
But now, if I convert to Paket, it seems to be ok, but the build will fail. And the reason is that Paket will add to .csproj-file only a condition:
    <When Condition="$(TargetFrameworkIdentifier) == '.NETFramework' And $(TargetFrameworkVersion) == 'v4.5'">

...which won't hit, because of the conflicting 4.5 vs 4.5.1.
How could these mismatches between csproj TargetFrameworkVersion and nuget targetFramework detected?

how is the paket.dependencies file for this case?
The idea is that if we have targetframework 4.5 then Paket should install >= 4.5
So the bug might be elsewhere

Generated paket.dependencies (Paket 0.40.0):
    source https://www.nuget.org/api/v2
    source https://www.myget.org/F/particular

    nuget log4net 2.0.0 framework: net45
    nuget NServiceBus 5.0.0 framework: net45
    nuget NServiceBus.Log4Net 1.0.0 framework: net45

and paket.lock:
    NUGET
      remote: https://www.nuget.org/api/v2
      specs:
        log4net (2.0.0) - framework: net45
        NServiceBus (5.0.0) - framework: net45
        NServiceBus.Log4Net (1.0.0) - framework: net45
          NServiceBus (>= 5.0.0 < 6.0.0)
          log4net (>= 2.0.0 < 3.0.0)

we should generate >= net45 as described in http://fsprojects.github.io/Paket/nuget-dependencies.html#Framework-restrictions
care to send a PR?

Actually, if we have restriction ">= net45" and ">= net451" then the later one containing the another should be enough. So minor version conflicts could be just merged. But "net45" and "net40-client" would be better to not be merged.

we already have a function which can do this. see https://github.com/fsprojects/Paket/blob/master/tests/Paket.Tests/DependencySetSpecs.fs#L28

Great, I was thinking of some kind of difficult folding, but I don't have to do it then!
Added PR #768.


Support None as a type for the typed-list

This is an intial, hacky, version of a None-typed typed-list.

This was extracted from #4883 to ease review.

The List[None] support is currently very limited. Here are some additional tests that are failing: https://github.com/sklam/numba/commit/ae804c0e561ef71087a6521a32a41cbb4d855399

This PR will be closed as the commits are part of #4883 and will continue to live there instead. As discussed OOB with @sklam we will keep the List[None] support but it will be severely limited and only a small subset of operations will be allows to support the list comprehension use case:
@njit(disable_reflected_list=True)
def foo():
    l = typed.List()
    [l.append(i) for i in range(10)]
    return l


Append Regexp.CaptureNames method for reading names of capturing groups

It useful for extracting values by dynamic or unknown capturing group names.
Using:
re := MustCompile("(?<name2>a)(b)(c)(?<n1>d)", 0)
names := re.CaptureNames()
// names == []pcre.CaptureName{{Index: 4, Name: "n1"}, {Index: 1, Name: "name2"}}

Hi @ukolovda
Sorry for ignoring you for so long, It seems like the GitHub notification has been lost in the spam folder, or I've just missed it.
Your PR looks useful, I'll take some time to thoroughly review it soon.


Add final cross country model

This PR adds the cross-country model and pkl file as per this issue. We have an overall cross validated R^2 value of 0.61 and std of 0.03.

One more thing is, in line with our planned Sprint 3 results sharing format, let's create a new folder instead like "2023-02-22-cross-country-rollouts" and place this final model training notebook (and pkl file) under this. The rollout notebooks for the 5 SEA countries will be placed here as well.


Card rarity colors sometimes show basic cards as rare

I love the card rarity coloring, by the way!
It seems right most of the time, but I have noticed some cases where basic cards are being colored blue.
In my games with Hunter and Mage, I noticed the following cards displaying incorrectly: Arcane Missiles, Arcane Intellect, Fireball, Houndmaster, Oasis Snapjaw.
I haven't walked through all my decks or anything, but looking at those two decks rare cards always seemed colored correctly, as did epic and legendary. I just had those few basic cards showing up as rare. I've enclosed a screenshot of my mage deck to show the issue.

Also perhaps worth noting... the "Rare" cards have a blue gem and a blue border, but the "Basic" cards only have the blue gem.

Yep, and the problem is that the "free" cards does not have a "color", so, to keep them coherent with the border, here is the color for those cards

The first one is a free, the second is a basic


Comment flag in cell not displayed when there's a comment but no score

The comment for a particular cell is only retrieved from the service if the cell has a score.  This means the comment flag for an empty cell is not displayed even if a comment has been added for a particular grade.
To replicate:

Find a grade cell that has never had a score set before
In the cell's menu, select Edit Comment
Enter a comment and hit "Save Comment"
Cell will render with the speech bubble flag within the cell
Reset the tool via the breadcrumb
The cell no longer shows the speech bubble flag

@jeffpasch, @kyleblythe, @steveswinsburg - I just rediscovered this issue. Not sure it's a priority , but if we let it go perhaps we should hide the "Edit Comment" menu option until there's a grade entered?

Hi Payten. I believe we have at least some instructors that use gradebook for providing seating assignments for exams, using the comments field in a grade item without inputting an actual grade. Maybe other schools do similar things...so probably a good one to try to get in if possible.

Confirmed!


[Event Request] Codeunit 80 - Sales-Post - OnBeforePostJobContractLine with JobContractLine

A follow up to #1457: the global boolean JobContractLine cannot be set from outside. It will be initial filled in PostJobContractLine. Sadly the existing event OnBeforePostJobContractLine is not able to set it. So you cannot Handle the code in that procedure as you cannot set that variable.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Can not find any references.

Describe the bug
No new connections found while they do exist.

To Reproduce
Click on find visual references.
Additional context
public class TestQuery : ITestResourceQuery<PagedResult<Test>>, IPagingOptions
{
        public bool? TestBool{ get; set; }

Hi - I'm sorry but I need a bit more context - what are you clicking on in the source code? Where do you expect to see references? Is there a simple repro that demonstrates the problem?

Given the lack of detail, there's not much I can do about this. Please feel free to re-open the issue with more details if you can find a repro 👍


Are there plans to update this package?

Are there any plans to update this project to be compatible with Guzzle 6 and PHP 7.2+?

Hi @bkuhl. Unfortunately, there are no plans to update this package. If it is still useful to people, I am looking for a new owner to give it the attention it needs.


OverfeatTransformer

Hi, I am using this link:http://sklearn-theano.github.io/auto_examples/plot_asirra_dataset.html, but get this error: I do not know why?
tf = OverfeatTransformer(output_layers=[-3])
Traceback (most recent call last):
File "", line 1, in
File "/home2/ift6ed05/anaconda2/lib/python2.7/site-packages/sklearn_theano-0.0.1-py2.7.egg/sklearn_theano/feature_extraction/overfeat.py", line 298, in init
detailed=detailed_network)
File "/home2/ift6ed05/anaconda2/lib/python2.7/site-packages/sklearn_theano-0.0.1-py2.7.egg/sklearn_theano/feature_extraction/overfeat.py", line 253, in get_fprop
fprop = theano.function([input_var], expressions)
File "/home2/ift6ed05/.local/lib/python2.7/site-packages/theano/compile/function.py", line 317, in function
output_keys=output_keys)
File "/home2/ift6ed05/.local/lib/python2.7/site-packages/theano/compile/pfunc.py", line 479, in pfunc
output_keys=output_keys)
File "/home2/ift6ed05/.local/lib/python2.7/site-packages/theano/compile/function_module.py", line 1777, in orig_function
defaults)
File "/home2/ift6ed05/.local/lib/python2.7/site-packages/theano/compile/function_module.py", line 1641, in create
input_storage=input_storage_lists, storage_map=storage_map)
File "/home2/ift6ed05/.local/lib/python2.7/site-packages/theano/gof/link.py", line 690, in make_thunk
storage_map=storage_map)[:3]
File "/home2/ift6ed05/.local/lib/python2.7/site-packages/theano/gof/vm.py", line 1037, in make_all
no_recycling))
File "/home2/ift6ed05/.local/lib/python2.7/site-packages/theano/gof/op.py", line 965, in make_thunk
no_recycling)
File "/home2/ift6ed05/.local/lib/python2.7/site-packages/theano/gof/op.py", line 879, in make_c_thunk
output_storage=node_output_storage)
File "/home2/ift6ed05/.local/lib/python2.7/site-packages/theano/gof/cc.py", line 1207, in make_thunk
keep_lock=keep_lock)
File "/home2/ift6ed05/.local/lib/python2.7/site-packages/theano/gof/cc.py", line 1152, in compile
keep_lock=keep_lock)
File "/home2/ift6ed05/.local/lib/python2.7/site-packages/theano/gof/cc.py", line 1602, in cthunk_factory
key=key, lnk=self, keep_lock=keep_lock)
File "/home2/ift6ed05/.local/lib/python2.7/site-packages/theano/gof/cmodule.py", line 1174, in module_from_key
module = lnk.compile_cmodule(location)
File "/home2/ift6ed05/.local/lib/python2.7/site-packages/theano/gof/cc.py", line 1513, in compile_cmodule
preargs=preargs)
File "/home2/ift6ed05/.local/lib/python2.7/site-packages/theano/gof/cmodule.py", line 2233, in compile_str
return dlimport(lib_filename)
File "/home2/ift6ed05/.local/lib/python2.7/site-packages/theano/gof/cmodule.py", line 331, in dlimport
rval = import(module_name, {}, {}, [module_name])
ImportError: ('The following error happened while compiling the node', CorrMM{valid, (4, 4)}(Elemwise{Composite{(i0 * (i1 + i2))}}.0, Subtensor{::, ::, ::int64, ::int64}.0), '\n', '/home2/ift6ed05/.theano/compiledir_Linux-2.6-el6.x86_64-x86_64-with-redhat-6.1-Carbon-x86_64-2.7.11-64/tmpXsngiA/0abe6c2454e59eab7778e6c645d664c5.so: undefined symbol: sgemm', '[CorrMM{valid, (4, 4)}(<TensorType(float32, 4D)>, <TensorType(float32, 4D)>)]')

Can you give me some more information about your platform / install setup?


Cleaner tests.py file, tests stills are failing

Related to #2 , looks like there was a erroneous "paste" in the test file, so i've removed unneded lines. Still there is two tests failing
======================================================================
ERROR: test_all (main.BitcoinCore_TransactionValid)
Traceback (most recent call last):
File "test.py", line 829, in test_all
for i, (po, tx, flags) in enumerate(load_tvv()):
File "test.py", line 822, in load_tvv
prevouts[prevout] = parse_script(json_prevout[2])
File "test.py", line 798, in parse_script
elif word in OPname:
NameError: global name 'OPname' is not defined
======================================================================
ERROR: test_all_testnet (main.TestBIP0032)
Traceback (most recent call last):
File "test.py", line 424, in test_all_testnet
bip32_ckd(mk, path),
File "/home/reiven/own/github/forks/pybitcointools/bitcoin/deterministic.py", line 251, in bip32_ckd
return bip32_serialize(raw_bip32_ckd(bip32_deserialize(data), i))
File "/home/reiven/own/github/forks/pybitcointools/bitcoin/deterministic.py", line 91, in raw_bip32_ckd
i = int(i)
ValueError: invalid literal for int() with base 10: 'm/'
======================================================================
FAIL: test_all (main.TestTransaction)
Traceback (most recent call last):
File "test.py", line 249, in test_all
self.assertTrue(verify_tx_input(tx1, 0, mscript, sig1, pubs[1]), "Verification Error")
AssertionError: Verification Error
======================================================================
FAIL: test_all (main.TestTransactionSignVerify)
Traceback (most recent call last):
File "test.py", line 169, in test_all
"Verification error"
AssertionError: Verification error

Ran 25 tests in 21.307s
FAILED (failures=2, errors=2)

Thx @reiven , I'm on it now :+1:

Pythonista is causing me grief, so let me know @reiven if this commit fixes test issue


Error installing elm-json on Travis CI - Windows container

I have a Windows (1809) container on Travis CI running with elm-test. But I get the following error from elm-tooling:

Failed to download:
< https://github.com/zwilias/elm-json/releases/download/v0.2.10/elm-json-v0.2.10-x86_64-pc-windows-msvc.tar.gz > C:\Users\travis\AppData\Roaming\elm\elm-tooling\elm-json\0.2.10\elm-json.exe
tar exited with exit code 2:
tar: C:\Users\travis\AppData\Roaming\elm\elm-tooling\elm-json: Cannot open: No such file or directory
tar: Error is not recoverable: exiting now

Hi!
This is what happens:

elm-tooling downloads https://github.com/zwilias/elm-json/releases/download/v0.2.10/elm-json-v0.2.10-x86_64-pc-windows-msvc.tar.gz. That succeeds.

elm-tooling tries to extract elm-json.exe using tar. That seems to work well until tar tries to write the output to C:\Users\travis\AppData\Roaming\elm\elm-tooling\elm-json\0.2.10\elm-json.exe.

Earlier elm-tooling has run fs.mkdirSync("C:\\Users\\travis\\AppData\\Roaming\\elm\\elm-tooling\\elm-json\\0.2.10") so that directory should exist, but somehow it doesn’t!
Can you help debug this? Travis CI lets you SSH into failed builds or something, don’t they? If so you could have a look around and see how much of the C:\Users\travis\AppData\Roaming\elm\elm-tooling\elm-json\0.2.10\elm-json.exe path exists, and what happens if you manually create directories there?

Hi.
Sorry, I don't know how to do that.
What I did notice is this the error was on C:\Users\travis\AppData\Roaming\elm\elm-tooling\elm-json, which doesn't seem to be the whole path. But I'm not sure if that's relevant.

I managed to reproduce it. The issue is that Travis CI Windows runs in Git Bash. Git Bash ships with GNU tar, which works differently from the built-in (BSD) tar that comes with Windows.
The good thing is that I can reproduce on my own Windows machine when using Git Bash, so I should be able to fix it.

Fixed in v1.2.1. Thanks for reporting!


Database task mapping doesn't handle nvarchar(MAX) in SQL server

Description
I have a remote db with columns of type nvarchar(MAX). I want to copy it in a local db using a task (because I often do it once a week).
The problem is that DBeaver does not specificy the MAX in the target mapping. The default size is 1 so I get errors looking like String or binary data would be truncated in table 'mytable', column 'SALESMEMO'. Truncated value: 'T'.
DDL of remote table :

DDL generated by DBeaver :

Note: the "Columns mapping" data in the "Configure metadata structure" window seems fine, but the generated DDL is not.

DBeaver Version
Community Version <IP_ADDRESS>301060755
Operating System
Linuxmint 20.3
Database and driver
Remote server : Microsoft SQL Server 2019 (RTM-CU15) (KB5008996) - 15.0.4198.2
Local server : Microsoft SQL Server 2022 (RC0) - 16.0.900.3
Driver : com.microsoft.sqlserver:mssql-jdbc:RELEASE 9.2.0.jre8
Steps to reproduce

Create a table on server S0
Add a column with type nvarchar(MAX)
Add a row with content "foo"
Create a db task to copy table from server S1 to S2. Keep the default mapping
Run the task. It should fail with String or binary data would be truncated in table...

Additional context
No response

Ok it seems the data are still inserted in the table if I click on "Skip all". And the table DDL is actually of type nvarchar(MAX)... :thinking:

Seems familiar to #17244

thanks for the report. closed as a duplicate of #17244


Make the scroller element support the scoped css

My implementation ideas is: As you know, the scrollView element is the parent node of scroller element, and they are in the same css scope. We can get the hash attribute just like 'data-v-xxxx' from the scrollView element, and create a new one the scoller element.
The concrete implementation will show in my code, and I send a pull request.
Looking forward to your reply

#11 Good idea, I suppose. By the way, 我以为你是个老外！

哈哈，同性交友平台遇同胞。


[WIP] Concurrent safe batch infra managers

Changes needed to allow for the concurrent_safe_batch saver strategy on InfraManager tables.
This is a work in progress
Depends on: https://github.com/ManageIQ/manageiq/pull/15941

This will need unique db indexes, closing until we add those


[SERV-812] Ignore empty metadata elements

Make harvester fail
Filter out empty elements :hole:

I don't have a very good answer to that question in the general sense, but for date in particular: we have been provided records without dates in the past, and those have been harvested to the current system; it's not ideal, but it doesn't break anything in a disastrous way, IMO. So my inclination would be to continue not requiring it.

Yes, I was thinking more in the general sense, given our conversation -- do we know what things are required, in addition to which values are required to match certain patterns (which I know you've done), etc. (e.g., are there things that are required if another one/element is present, etc.?)

Each metadata field is independent in that the presence of one doesn't have any bearing on whether another must be present. However, we do have a metadata guidelines document which specifies certain fields as required and others as optional. According to it, date is a required field, however there was never any specification as to what the behavior of the system should be if it is absent. The current implementation (the one we are replacing) takes the stance: allow it to be missing.

Is the current system inheriting the stance of the previous system?

The other "required" fields are title and creator.

I don't see why we should exclude records that don't have a date. Perhaps there is other metadata in such a record that is valuable to someone. The "PM" way back when wanted a fancier date facet UI than what we got (some sort of timeline with sliders or something), and I think that may have been the motivation behind "requiring" date.

Let's discuss this at the next team meeting?


codecov.io reports fishy behavior

/kind bug

What versions of software are you using?

Send a pr
Actual behavior
Most of the time watch.go is reported as a culprit though the file is not touched in the pr. For example - https://github.com/openshift/odo/pull/2572

Expected behavior
Coverage should only be calculated on those file which are part of go list  ./... | grep -v github.com/openshift/odo/vendor | grep -v github.com/openshift/odo/tests and if the pr doesn't touch the file within the list then coverage report should not report any decrease or increase in coverage for the same pr.
Any logs, error output, etc?

Similarly, I also noticed this with @kadel's PR to update the OWNERS file: https://github.com/openshift/odo/pull/2556#issuecomment-582055553
Codecov reported a 0.49% increase in code coverage for the OWNERS file update, which doesn't make sense haha.

/area testing


Importing OAS3

While inporting OAS3 spec with your plugin enabled all the path parameters are still converted to Insomnia Environment Variables.
Maybe there's a way to implement conversion from path parameters to tags?

As noted in #1 , I don't play around with Insomnia anymore, so I don't remember if this is possible or not...
It feels like if one wants to do that today, they'd have to define their own Insomnia importer for each possible import document type (OAS2, OAS3, all the different Postman collections). Not necessarily, but surely if OAS3 is supported, then someone would ask for other formats.
I base my judgement on these:

Insomnia UI has this link in it hinting that previously it was possible to define your own importer.

I have a vague feeling it was previously possible to be done by a user provided plugin, but alas, the link leads here: https://docs.insomnia.rest/insomnia/import-export-data
And it says:
If you want to contribute new formats, feel free to submit a pull request to the insomnia-importers on GitHub.
I cannot find the docs to define an importer any longer.

Back when I was playing with Insomnia, what I did, was I imported the collections I wanted, then modified imported collections with some shell scripts.
Also, the exporting would be a new symmetrical problem now to solve, where I'd need to teach Insomnia to treat the path parameters from this plugin as path parameters. And this is something that Insomnia simply doesn't agree with.
Again, as I mentioned in #1, I think the larger community wants per-request variables.
There's no concept of "path parameters" in Insomnia, and as such there's no leverage for plugins to do anything special with path parameters (be it import/export/using them in requests in a handy way).
And this is why there are many path-parameters related plugins each doing their own thing, but ultimately each having their tradeoffs. If I'm not mistaken, if the plugin made by the original Insomnia maintainer has the same OAS3 import problem that you describe


Add survey results

Part of #1505
Rendered
Repo for all data and statistics: https://github.com/janhohenheim/rust-gamedev-statistics/tree/main/jan-hohenheim-2024

Answered all the feedback. @AngelOnFira, ready for another round :)

Mirroring my comments from Discord to here.

Of note is that the verbal feedback we got indicated that a lot of readers did not fully understand what exactly was being proposed. People worried that we would start generating a majority of the newsletter or entire sections with AI, which is not something any of us wants. Some readers also thought we already started using LLMs. The actual idea was to use AI to generate summaries of articles that were already hand-picked by the editors but not summarized yet because of time constraints. The summaries would then be edited and verified by the editors. The extent to which AI would be used would be limited to up to two sentences per late article. Any confusion in this regard is our fault. We will try to be more clear on such questions in the future.

Although this misunderstanding might have skewed the results, we have reason to believe that the effect is not too large. Anecdotally, when we properly explained the proposal to readers who were against it, most did not change their mind and cited more principled reasons for their stance. Chief among these was solidarity with the large number of creatives who recently lost their jobs due to AI-generated content, inside and outside the game development industry.

I don't think this is a particularly fair or accurate summary of the situation.

In particular, it frames the negative feedback as coming from people who didn't understand it in the first place - I, as well as many others, understood that the proposal involved AI summaries of other articles. In addition, I think that saying that all pushback received was moral is very misleading when a number of practical objections were raised, and that the primary source of pushback was not job loss.

Anecdotally, I feel that the article attempts to frame pushback on LLM content as coming from pearl-clutching luddites, who did not actually understand the issue at hand, when in reality, the majority of pushback came from people who understood what was being proposed very well. I think this is neither fair, nor accurate.

While a majority of readers (65%) are at least okay with AI-generated summaries, a significant minority (35%) are not okay at all with this proposal.

This is burying the lede that negative responses (Not Okay) far outnumber positive responses (Good, Love). Furthermore, the fact that there were two positive responses and only one negative response will likely skew results in the positive direction. You can't just lump the neutral responses (Okay, Don't Care) in with the positive responses in your analysis. It is, at the very least, misleading.
I'm a bit frustrated by the article in many ways. I think it's inevitable that, when the people doing the review of the survey were the people pushing for the thing that everyone told them not to do in the survey, fair and accurate explanation of the reasons people disagreed with the idea are kind of impossible.
One final point:

The actual idea was to use AI to generate summaries of articles that were already hand-picked by the editors but not summarized yet because of time constraints. The summaries would then be edited and verified by the editors. The extent to which AI would be used would be limited to up to two sentences per late article.

Some points to address:

"the people doing the analysis" is just me in this case. If there's anything wrong with the survey, analysis or summary, that is entirely my fault and no one else of the working group or involved in the newsletter should be blamed.
I did my best to be objective for the raw data. "Most people are at least okay with AI" is a fact. "Most people would prefer to not have AI" is also a fact. I mentioned both. The colors I used in the plot are chosen intentionally so that the "okay" option is orange, not green.
The summary of reasons people had might not be true for you, and I respect that. This is entirely a summary of the feedback I personally got in private conversations, from the survey answers and in public discussions. There is nothing wrong with people mostly not wanting to have AI in the newsletter out of principle. I personally disagree with it, but I would not call anyone a "perl-clutching luddite" over it. People have different ethical frameworks, that's okay. In the end, it doesn't matter much why readers might be opposed to AI, it matters more that they are. I included my summary because I probably have received more messages concerning the topic than anyone else, so it seemed interesting to summarize them.
 I do not wish to chose anything based on who happens to be the most vocal over an issue. If we had an army of AI techbros in this server, it would not make it better to use LLMs either. It is my belief that a survey was the right thing to do. Considering that I now know that most readers would prefer AI not to be involved, I want to respect that wish.
The questions about AI, excitement and ease of contribution did not have optimally phrased answers. In hindsight, I should have used a Likert scale, which is a widespread standard for ordinal data. No sleight of hand here, I just never worked much with ordinals.

The last two commits address criticisms raised by people who did not feel represented by the summary given. I hope it's better now :)


Examples not working

I've come across a few difficulties that I'll document here:

Installing via pip completes, but o2wb-exp is not recognized by the command-line. Running from within the installed directory doesn't appear to work either.
There is no indication of a config file in the README, it really should be added as a step to take in the instructions. There is also no config file included when installing via pip.
I've begun the install by cloning from GitHub; unfortunately the requirements.txt does not appear to work so I changed it to be the following:

antlr4-python3-runtime == 4.9.3
rdflib == 6.2.0
wikibaseintegrator == 0.12.3
argparse == 1.4.0

Running as python export.py does not work, because export.py does not ever call main(). I'm attempting to fix this now and will update if I get further along or discover further issues.

The config file has to be included in the arguments, but the README doesn't mention this.

data has to be returned from the printProgressBar method; otherwise it leads to a NameError.


.cpp line 274

I am currently translating this library for nRF52832
Could anyone explain the reason for the request for 24 bytes followed by a read for 28 bytes, is this an error?
Wire.requestFrom(addrBME280, 24);
for (count = 0; count < 28; count++) {  // first 28 bytes we can process like this
compParams.compArray[count] = Wire.read();
}

I'm also interested in, because the buffer compArray is declared as 20 bytes
union compParams_u
{
   uint8_t        compArray[20];
   compParams_ts  compStruct;
};

but compArray is not used anywhere so I think this part of code is useless

Should be 24 from 0x88 to 0x9F, it's also missing the last humidity calibration H6 byte should be 8 total from 0xE1 to 0xE7
  Wire.requestFrom(addrBME280, 24);   for (count = 0; count < 24; count++) {  // first 24 bytes we can process like this    compParams.compArray[count] = Wire.read();   }
`  Wire.requestFrom(addrBME280, 8);
compParams.compStruct.dig_H2 = Wire.read();
compParams.compStruct.dig_H2 |= (uint16_t)Wire.read() << 8;
compParams.compStruct.dig_H3 = Wire.read();
compParams.compStruct.dig_H4 = (uint16_t)Wire.read() << 4; // bits 11:4
regE5 = Wire.read();
compParams.compStruct.dig_H4 |= regE5 & 0b00001111; // bits 11:4
compParams.compStruct.dig_H5 = regE5 >> 4;
regE6 = Wire.read();
compParams.compStruct.dig_H5 |= regE6 << 4;
compParams.compStruct.dig_H6 = Wire.read(); `


Finish check_datapoints_categories

This still needs to check mode and class and the category order

Finished. When category metadata is off you get a dataset that does a side by side comparison with f and q category metadata.


Popover opened inside a Dialog closes immediately under certain circumstances

Bug report
Current Behavior
A Popover opened inside a Dialog closes immediately when:

@radix-ui/react-dialog version is 1.0.4;
@radix-ui/react-popover version is 1.0.6;
Popover.Trigger re-renders according to the Popover.Root state;
Popover.Portal is present;

https://github.com/radix-ui/primitives/assets/125764/9df60724-6096-45c2-9b75-797a7701e575
Expected behavior
The Popover should stay opened when the trigger is clicked.
Reproducible example
Code Sandbox
Additional context

Fails on Chrome, Safari and Firefox;

Your environment

Software
Name(s)
Version

Radix Package(s)
@radix-ui/react-dialog; @radix-ui/react-popover
1.0.4 and 1.0.6 respectively

React
n/a
18.2.0

Browser
Chrome
115.0.5790.170 (Official Build) (arm64)

Assistive tech
n/a

Node
n/a
16.19.0

npm/yarn
yarn
1.22.19

Operating System
MacOS
13.4.1 (c) (22F770820d)

It looks like to be https://github.com/radix-ui/primitives/issues/2180. Upgrade to RC version should work
Codesandbox

@joaom00, any idea when the official release will be?

Not sure

@leofavre facing the same issue too – looks like downgrading to react-dialog to 1.0.3 and react-popover to 1.0.5 is a good temporary fix – this bug doesn't happen in those version :)

Hey @steven-tey! Have you tested this fix on Safari? We are having the same issue with popovers and dialogs with safari browsers in https://github.com/open-sauced/app/issues/1144. Chrome and firefox works as expected.
Thanks in advance 🙏

We have this problem also. Implemented an ugly temporary fix by using a cooldown like this:
const [cooldown, setCooldown] = useState(false);
  const [open, setOpen] = useState(false);
  function handleSet() {
    if (cooldown) return;
    console.log(‘rex’);
    setOpen(!open);
    setCooldown(true);
    setTimeout(() => {
      setCooldown(false);
    }, 200);
  }

Can confirm that the fix from @CarelessCourage works as well! Ended up needing to upgrade react-dialog and react-popover to the latest version so this fix came in handy – thank you!

Am seeing this as well. In our case we have an onclick handler inside a DropdownMenu item (from react-dropdown-menu) setting an isPopoverOpen state variable. But as soon as the PopOver is rendered, the Popover is triggering onOpenChange(false)

Running into this as well, but haven't yet been able to figure our a workaround that works for us.

My issue was caused by the fact that closing the DropdownMenu auto focuses the DropdownMenu.Trigger.
Since the DropdownMenu.Trigger is outside of my Popover the focus of the Trigger causes the Popover to close.
To resolve this I added the following:
  <DropdownMenu.Content
    onCloseAutoFocus={(e) => {
      e.preventDefault();
    }}

@leofavre facing the same issue too – looks like downgrading to react-dialog to 1.0.3 and react-popover to 1.0.5 is a good temporary fix – this bug doesn't happen in those version :)

Thanks!! For me working in both browsers firefox and chrome

Same happened for me. I have a popover command dropdown UX inside an alert dialog.
<AlertDialog...
<Popover...
<Command...
and basically my popover was auto closing.
Solution is to set <Popover modal={true} ... > AND <Popover.Content onOpenAutoFocus={e => e.preventDefault()} ... >
@nzjoel1234's solution worked but then triggered infinite recursion like https://github.com/radix-ui/primitives/issues/2390 as something fought over auto-focus. I had to prevent that by stopping the auto focus on the popover content 🤷♂.

Should this be closed? What if one wants to focus inside the popover in a modal?


Chef Server needs to have an API to indicate if an user is part of server-admin group

A Chef server user can be added to server-admin group
chef-server-ctl grant-server-admin-permissions alice
Reference:
https://docs.chef.io/server/ctl_chef_server/#server-admins
The Chef Server API for user /organizations/NAME/users/NAME does not provide any indication about whether the user
is granted server-admin or not.
Addition of a field is required:
{
  "username": "paperlatte"
  "email": "latte",
  "display_name": "Ms. Latte",
  "first_name": "Paper",
  "last_name": "Latte",
  "public_key": "-----BEGIN PUBLIC KEY----- ... "
 "is_server-admin": true
}

I'd rather we implement a user_type field here so we don't end up wth a handful of these boolean values.

Closing this out since we're moving away from users in the server and there's no real value in building this now for that migration.


Create HeightOfBT.cpp

Added program to calculate height of the given Binary Tree.

@Romi904 kindly review my PR


Document annotations used in PV controller

This is a Bug Report

Problem:
Annotations used by PV controller and other storage controllers should be documented here:
https://kubernetes.io/docs/reference/labels-annotations-taints/
Proposed Solution:
Document annotations that are currently missing from the doc.
Page to Update:
https://kubernetes.io/...

/sig storage

/triage accepted

/lifecycle frozen

I am taking this issue as still relevant, but please let me know if that isn't the case @xing-yang
/triage accepted

/sig docs


cypress: init at 3.2.0

Motivation for this change
Missing package.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Hey @acyuta108 , thanks for making this package. I'm getting the following error:
EACCES: permission denied, open '/home/igm/.config/Cypress/cy/production/browsers/chrome/interactive/CypressExtension/background.js'

It looks like it's chmodded to 0700, and it belongs to my user. Any ideas? Thanks!

Yes, I am getting the same error. Unfortunately I believe is a Cypress issue. I discovered it last Friday, when I last tested right after my push and got the error.
Try switching the browser to Electron and let me know if you still have the issue.
Unfortunately if We don't figure this out before this package is to be merged, I'll need to downgrade to Cypress 3.1.5, which worked just fine.
I have created a ticket in Cypress to see if they can help.
@thorstenweber83 and @erosennin, you guys are far more experience than me, would you kindly take a look and see if something might have changed from 3.1.5 => 3.2.0 that we need to address when unpacking/unzipping and linking the contents of the deb package from Cypress?
Thanks

Yes it works with Electron. I can also confirm the same error happens when
using Chromium.
On Sun, Mar 31, 2019 at 21:18 Acyuta<EMAIL_ADDRESS>wrote:

Yes, I am getting the same error. Unfortunately I believe is a Cypress
issue. I discovered it last Friday, when I last tested right after my push
and got the error.
Try switching the browser to Electron and let me know if you still have
the issue.
Unfortunately if We don't figure this out before this package is to be
merged, I'll need to downgrade to Cypress 3.1.5, which worked just fine.
I have created a ticket
https://github.com/cypress-io/cypress/issues/3852 in Cypress to see if
they can help.
@thorstenweber83 https://github.com/thorstenweber83 and @erosennin
https://github.com/erosennin, you guys are far more experience than me,
would you kindly take a look and see if something might have changed from
3.1.5 => 3.2.0 that we need to address when unpacking/unzipping and linking
the contents of the deb package from Cypress?
Thanks
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/58423#issuecomment-478429938, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAYfb1eLapTDurJMgFhfveUVl2RHIkzAks5vcYilgaJpZM4cM5cT
.

Any progress on this?


can't init java runtime

can't access https://api.bintray.com/search/packages/maven?repo=fnproject&g=com.fnproject.fn&a=fdk, even use some public cloud provider , like aws EC2, oracle public cloud (OCI-classic)
I have question why java runtime dependent bintray api service ?

when I set FN_JAVA_FDK_VERSION=1.0.59 it's work, but I think may be we can give  default FN_JAVA_FDK_VERSION,   even I set  "latest"  ,  it's also not work, it's not friendly use

Sorry for the delay In replying
we use bintray to fixate the latest version of the FDK when you fun fn init. This preserves repeatable builds as you end up with the same FDK lib and base images as when you defined the function.
It looks like bintray was down at the point you tested this.
We use bintray for convenience now but I’ll leave this open as appreciate that this is not ideal and we could host this repo some where we control


apachectl[16187]: Cannot define multiple Listeners on the same IP:port

when apache is restarted after configuring and activating the site.conf file this error appears that prevents apache2 from starting

The problem is that port 80 is listened to twice, once in the /etc/apache2/ports.conf file and once in the /etc/apache2/sites-available/site-example.conf file.
Solution:

version < 0.3.1: remove line 1: Listen 80 from file /etc/apache2/sites-available/site-example.conf.
version >= 0.3.1: fixed.


Pull Request Template: add reviewer checklist

What this PR does / why we need it:
This pull requests edits this template to add a checklist, which we said at retro might be helpful.
Which issue(s) this PR fixes:
action item from retro
Does this PR introduce a user-facing change?
nah
Additional Notes for your reviewer:
There would be a checklist here!
I think good questions for reviewing this change include:

[ ] do we really want a checklist here?
[ ] would you remove anything from the checklist in this PR?
[ ] would you add anything to the checklist in this PR?

Additional documentation e.g., Proposal, usage docs, etc.:
this pull request is self-documenting ;-)

cc @danielhelfand @neil-hickey

lgtm
Test failures seem like a flakey run

I went ahead and re-ran the things so hopefully next one will let us merge in :)

I went ahead and re-ran the things so hopefully next one will let us merge in :)

Did it fail again? That's the second time then as I also reran it.


Switching order of colors in tint/shade example functions:

It seems tint and shade examples don't work as intented.
Let's say I want to darken by 10%, I have to write shade($color, 90%) if using the given example.
Changing order of passed colors to mix fixes it and let me write shade($color, 10%);
Thanks for the guidelines :)

Thank you very much. :)


Duplicating a map previously edited with the new editor and opening the copy in the new editor fails

It gives an error on the maps api saying that SQL is missing.
Create a map using the new editor.
Open the map on the actual editor.
Duplicate the map.
Go to the new map in the new editor.
(ERROR)
If you clear view on actual editor, and go back to the new editor it works

Duplicated -> #7657


{CI} test network, vm for onboarding pytest on CI for latest profile

Description
Test incremental test logic basd on pytest and new azdev for network and vm.
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.

test network


Doc for ACL Renderer behaviour

Microsoft Reviewers: Open in CodeFlow

Docs Build status updates of commit c23eccf:
:warning: Validation status: warnings

File
Status
Preview URL
Details

AdaptiveCards/rendering-cards/renderer-behaviour.md
:warning:Warning

Details

AdaptiveCards/content/column_width_image_width_2.png
:white_check_mark:Succeeded

AdaptiveCards/content/height1_text_wrap_off.png
:white_check_mark:Succeeded

AdaptiveCards/content/height2_text_wrap_on.png
:white_check_mark:Succeeded

AdaptiveCards/content/image_size_3.png
:white_check_mark:Succeeded

AdaptiveCards/content/image_size_auto_stretch.png
:white_check_mark:Succeeded

AdaptiveCards/content/imagesize2.png
:white_check_mark:Succeeded

AdaptiveCards/content/spacing1.png
:white_check_mark:Succeeded

AdaptiveCards/content/spacing2_not_affected.png
:white_check_mark:Succeeded

AdaptiveCards/content/spacing3_seperator.png
:white_check_mark:Succeeded

AdaptiveCards/content/spacing4_with_seperator.png
:white_check_mark:Succeeded

AdaptiveCards/content/spacing5_no_effect.png
:white_check_mark:Succeeded

AdaptiveCards/content/spacing_no_effect.png
:white_check_mark:Succeeded

AdaptiveCards/content/style1.png
:white_check_mark:Succeeded

AdaptiveCards/content/style2_withBleed.png
:white_check_mark:Succeeded

AdaptiveCards/content/width3_auto_stretch_auto.png
:white_check_mark:Succeeded

AdaptiveCards/content/width5_w50_p100_w50_auto.png
:white_check_mark:Succeeded

AdaptiveCards/content/width6_all_auto.png
:white_check_mark:Succeeded

AdaptiveCards/content/width_1_auto_stretch.png
:white_check_mark:Succeeded

AdaptiveCards/content/width_2_stretch_stretch.png
:white_check_mark:Succeeded

AdaptiveCards/content/width_4_w50_p100_w50.png
:white_check_mark:Succeeded

AdaptiveCards/rendering-cards/renderer-behaviour.md

Line 1, Column 1: [Warning-author-missing] Missing required attribute: 'author'. Add the current author's GitHub ID.
Line 1, Column 1: [Warning-title-missing] Missing required attribute: 'title'. Add a title string to show in search engine results.
Line 1, Column 1: [Warning-ms-author-missing] Missing required attribute: 'ms.author'. Add the current author's Microsoft alias.
Line 1, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.
Line 61, Column 1: [Suggestion-duplicate-alt-text] Alt text 'Columns with weightage and pixel width combination' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.
Line 70, Column 1: [Suggestion-duplicate-alt-text] Alt text 'Columns with weightage and pixel width combination' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.
Line 88, Column 1: [Suggestion-duplicate-alt-text] Alt text 'Columns with weightage and pixel width combination' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.
Line 95, Column 1: [Suggestion-duplicate-alt-text] Alt text 'Columns with weightage and pixel width combination' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.
Line 122, Column 1: [Suggestion-duplicate-alt-text] Alt text 'Columns with weightage and pixel width combination' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.
Line 180, Column 1: [Suggestion-duplicate-alt-text] Alt text 'Column width and image width combination' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.
Line 185, Column 1: [Suggestion-duplicate-alt-text] Alt text 'Column width and image width combination' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit ce377f2:
:warning: Validation status: warnings

File
Status
Preview URL
Details

AdaptiveCards/rendering-cards/renderer-behaviour.md
:warning:Warning

Details

AdaptiveCards/content/action_submit_behaviour.png
:white_check_mark:Succeeded

AdaptiveCards/content/column_width_image_width_2.png
:white_check_mark:Succeeded

AdaptiveCards/content/height1_text_wrap_off.png
:white_check_mark:Succeeded

AdaptiveCards/content/height2_text_wrap_on.png
:white_check_mark:Succeeded

AdaptiveCards/content/image_size_3.png
:white_check_mark:Succeeded

AdaptiveCards/content/image_size_auto_stretch.png
:white_check_mark:Succeeded

AdaptiveCards/content/imagesize2.png
:white_check_mark:Succeeded

AdaptiveCards/content/spacing1.png
:white_check_mark:Succeeded

AdaptiveCards/content/spacing2_not_affected.png
:white_check_mark:Succeeded

AdaptiveCards/content/spacing3_seperator.png
:white_check_mark:Succeeded

AdaptiveCards/content/spacing4_with_seperator.png
:white_check_mark:Succeeded

AdaptiveCards/content/spacing5_no_effect.png
:white_check_mark:Succeeded

AdaptiveCards/content/spacing_no_effect.png
:white_check_mark:Succeeded

AdaptiveCards/content/style1.png
:white_check_mark:Succeeded

AdaptiveCards/content/style2_withBleed.png
:white_check_mark:Succeeded

AdaptiveCards/content/width3_auto_stretch_auto.png
:white_check_mark:Succeeded

AdaptiveCards/content/width5_w50_p100_w50_auto.png
:white_check_mark:Succeeded

AdaptiveCards/content/width6_all_auto.png
:white_check_mark:Succeeded

AdaptiveCards/content/width_1_auto_stretch.png
:white_check_mark:Succeeded

AdaptiveCards/content/width_2_stretch_stretch.png
:white_check_mark:Succeeded

AdaptiveCards/content/width_4_w50_p100_w50.png
:white_check_mark:Succeeded

AdaptiveCards/rendering-cards/renderer-behaviour.md

Line 1, Column 1: [Warning-author-missing] Missing required attribute: 'author'. Add the current author's GitHub ID.
Line 1, Column 1: [Warning-title-missing] Missing required attribute: 'title'. Add a title string to show in search engine results.
Line 1, Column 1: [Warning-ms-author-missing] Missing required attribute: 'ms.author'. Add the current author's Microsoft alias.
Line 1, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.
Line 61, Column 1: [Suggestion-duplicate-alt-text] Alt text 'Columns with weightage and pixel width combination' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.
Line 70, Column 1: [Suggestion-duplicate-alt-text] Alt text 'Columns with weightage and pixel width combination' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.
Line 88, Column 1: [Suggestion-duplicate-alt-text] Alt text 'Columns with weightage and pixel width combination' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.
Line 95, Column 1: [Suggestion-duplicate-alt-text] Alt text 'Columns with weightage and pixel width combination' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.
Line 122, Column 1: [Suggestion-duplicate-alt-text] Alt text 'Columns with weightage and pixel width combination' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.
Line 180, Column 1: [Suggestion-duplicate-alt-text] Alt text 'Column width and image width combination' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.
Line 185, Column 1: [Suggestion-duplicate-alt-text] Alt text 'Column width and image width combination' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit cd8f2ab:
:warning: Validation status: warnings

File
Status
Preview URL
Details

AdaptiveCards/rendering-cards/renderer-behaviour.md
:warning:Warning

Details

AdaptiveCards/content/action_submit_behaviour.png
:white_check_mark:Succeeded

AdaptiveCards/content/column_width_image_width_2.png
:white_check_mark:Succeeded

AdaptiveCards/content/height1_text_wrap_off.png
:white_check_mark:Succeeded

AdaptiveCards/content/height2_text_wrap_on.png
:white_check_mark:Succeeded

AdaptiveCards/content/image_size_3.png
:white_check_mark:Succeeded

AdaptiveCards/content/image_size_auto_stretch.png
:white_check_mark:Succeeded

AdaptiveCards/content/imagesize2.png
:white_check_mark:Succeeded

AdaptiveCards/content/spacing1.png
:white_check_mark:Succeeded

AdaptiveCards/content/spacing2_not_affected.png
:white_check_mark:Succeeded

AdaptiveCards/content/spacing3_seperator.png
:white_check_mark:Succeeded

AdaptiveCards/content/spacing4_with_seperator.png
:white_check_mark:Succeeded

AdaptiveCards/content/spacing5_no_effect.png
:white_check_mark:Succeeded

AdaptiveCards/content/spacing_no_effect.png
:white_check_mark:Succeeded

AdaptiveCards/content/style1.png
:white_check_mark:Succeeded

AdaptiveCards/content/style2_withBleed.png
:white_check_mark:Succeeded

AdaptiveCards/content/width3_auto_stretch_auto.png
:white_check_mark:Succeeded

AdaptiveCards/content/width5_w50_p100_w50_auto.png
:white_check_mark:Succeeded

AdaptiveCards/content/width6_all_auto.png
:white_check_mark:Succeeded

AdaptiveCards/content/width_1_auto_stretch.png
:white_check_mark:Succeeded

AdaptiveCards/content/width_2_stretch_stretch.png
:white_check_mark:Succeeded

AdaptiveCards/content/width_4_w50_p100_w50.png
:white_check_mark:Succeeded

AdaptiveCards/rendering-cards/renderer-behaviour.md

Line 2, Column 1: [Warning-ms-author-missing] Missing required attribute: 'ms.author'. Add the current author's Microsoft alias.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.
Line 175, Column 1: [Suggestion-duplicate-alt-text] Alt text 'Column width and image size combination' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.
Line 190, Column 1: [Suggestion-duplicate-alt-text] Alt text 'Column width and image size combination' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Initial yocto git based recipe

Added a base recipe that fetches latest version from git

Don't we also need layer.conf? I see many examples such as [1].
[1] https://github.com/wolfSSL/meta-wolfssl/blob/master/conf/layer.conf

I didn't yet add a layer. only a recipe so you don't see layer.conf.
Generally you don't create a layer for a single  simple. I can add one though for the sake of the example

I didn't yet add a layer. only a recipe so you don't see layer.conf. Generally you don't create a layer for a single simple. I can add one though for the sake of the example

yes, for the sake of example as I've done for the Gentoo, I think it will be great.

Layer is included

I also added bit of documentation and squashed it all.
Comment is here

I just wounder why all the examples and in core layer they are using ??= while we ?=...
For the layer.conf I ask what happens if I checkout the myencfs git repository directly as a layer, will bitbake will be able to use it directly?
For example in Gentoo there is no way to checkout the https://github.com/alonbl/myencfs git repos and use it as a repository as it must seek the metadata/layout.conf at the root, if not found it is not a repository. They should have added a way to look into a subdirectory...


Confirm DPG v1.0 and DPG v2.0 parity with regard to resiliency

DPG v1.0 and DPG v2.0 parity
This issue tracks the work to show that the DPG v2.0 generator can generate the same client output from a Cadl file as the DPG v1.0 generator can from a swagger file.
Cadl Inputs

dev-driven
srv-driven-1
srv-driven-2

Expected Outputs
The generated clients should match those generated from the corresponding autorest.testserver swagger files:

dpg-customization.json
dpg-initial.json
dpg-update1.json

Test Coverage
The tests written for the DPG v1.0 clients should work against the DPG v2.0 clients as well.

I think this could be part  of the effort of Test server support.
@annelo-msft, what is the priority of this item?

@annelo-msft looks like we only need to do the comparison, there is no coding effort, correct?
I would still put these three test scenarios into #2467

Current gap:

Fixed enum is not in the August scope: https://github.com/Azure/cadl-ranch/blob/fa1f20e21bf8dfa6316f7b2025cb7670ffbff068/packages/cadl-ranch-specs/http/resiliency/dev-driven/main.cadl#L26
paging and lro cannot be filtered now. Tracked by https://github.com/Azure/autorest.csharp/issues/2565

Will move to 2.0 beta as srv-driven2 is not a correct CADL file.

@pshao25, seems only 9&10 left and they are not the scope for beta2,   I plan to move this item to 2.0GA.
Is there anything else left for this item?

None of the cald files in cadl-ranch is ready. Not sure if @iscai-msft would update them?


Feature/tshaiman/refactor to vi client

move Function based calls  into OOP approach with Video Indexer Client

@ika-microsoft  thanks for the great Review , all comments except the cache got resolved.


Question about target deps

https://github.com/idelsink/qtcreator_sub_dir_example/blob/38f099015a30ff5d63efd04bbe6e4eadafd6cffb/SubTestProject/common.pri#L70
Hey, I've got a question. Shouldn't it be the exact path to the output library rather than a path to the directory? Otherwise, parent project won't be rebuilt when dependencies are updated. Working example for unix:
PRE_TARGETDEPS += $${LIB_PATH}/lib$${TARGET_NAME}.a

You make a good point.
Even the documentation specifies it: http://doc.qt.io/qtcreator/creator-project-qmake-libraries.html
You can make a pr to fix it if you want so you get the credit? :-)


dashboard: remove dragonfly-amd64-5_8 builder

This builder is no longer needed since 6.0 is out already and DragonFly
BSD can only support current RELEASE and bleeding-edge (tip). This was
discussed in golang.org/issue/46351. For ongoing work to migrate the
DragonFly BSD builders to GCE, see golang.org/issue/23060.
Fixes golang/go#46351.

Closing this PR because golang.org/cl/332129 has been submitted.


Added update with network partition bug

Update with two sparsely-syncing peers will hang.

added and fixed in 8.8.3


CA-170240: Bump API version number to 2.5

Apart from the above bug, everything else seems to be complete and correct.

Thanks, merging now.


Add support for multiple samples in SimBA.generate

Signed-off-by: Beat Buesser<EMAIL_ADDRESS>Description
This pull request adds support for multiple samples in SimBA.generate.
Fixes #1407
Type of change
Please check all relevant options.

[ ] Improvement (non-breaking)
[x] Bug fix (non-breaking)
[ ] New feature (non-breaking)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Codecov Report

Merging #1422 (45a2bae) into dev_1.9.0 (210e003) will decrease coverage by 68.04%.
The diff coverage is 0.00%.

@@              Coverage Diff               @@
##           dev_1.9.0    #1422       +/-   ##
==============================================
- Coverage      90.43%   22.39%   -68.05%
==============================================
  Files            234      234
  Lines          18993    18982       -11
  Branches        3107     3108        +1
==============================================
- Hits           17177     4251    -12926
- Misses          1050    14681    +13631
+ Partials         766       50      -716

Impacted Files
Coverage Δ

art/attacks/evasion/simba.py
9.25% <0.00%> (-75.25%)
:arrow_down:

art/evaluations/evaluation.py
0.00% <0.00%> (-100.00%)
:arrow_down:

art/preprocessing/audio/__init__.py
0.00% <0.00%> (-100.00%)
:arrow_down:

art/estimators/object_detection/utils.py
0.00% <0.00%> (-100.00%)
:arrow_down:

art/evaluations/security_curve/__init__.py
0.00% <0.00%> (-100.00%)
:arrow_down:

art/preprocessing/audio/l_filter/pytorch.py
0.00% <0.00%> (-100.00%)
:arrow_down:

art/attacks/poisoning/perturbations/__init__.py
0.00% <0.00%> (-100.00%)
:arrow_down:

...essing/expectation_over_transformation/__init__.py
0.00% <0.00%> (-100.00%)
:arrow_down:

...rmation/natural_corruptions/contrast/tensorflow.py
0.00% <0.00%> (-100.00%)
:arrow_down:

...formation/natural_corruptions/zoom_blur/pytorch.py
0.00% <0.00%> (-93.94%)
:arrow_down:

... and 176 more

Hi @beat-buesser thank you for making this fix and inviting me to review.
I would be happy to but I'm working a short week because of a US holiday.
Would it work for you if I tested early next week?

@Embeddave Thank you very much, next week would be perfect. Happy Thanksgiving!

Hi @Embeddave What do you think about this pull request?

Hi @beat-buesser was going to leave you a message today
I found a major bug in my own code in the process of testing this and needed to put out that fire first
I can check early this week
we typically run with batch_size=1 as the two-six teams lab does when running GARD evals so I haven't needed this specific functionality

@Embeddave Thank you for the update, that should work. I was sure you had not forgotten.

I have some separate feedback but I will reply on #1407 -- it's off-topic for this PR

👍
the additional commit changing the variable name and using the validator LGTM
I tested with my code that the attack is working as before when batch size = 1.
Seems to be working, AFAICT.
I would share so you can see how I'm testing, but the code involves some in-house libraries that are not public (nothing special, just not installable by anybody).
I am testing the batch size > 1 functionality now.

@Embeddave Thank you, I have added you to the list of collaborators in the repo which upon accepting the invite in your email should allow you to make inline comments.
I think the batch size still has to be 1 only because the algorithm processes one sample at a time and does not support processing in larger mini-batches. But with this version it is now processing all samples of x.

Hi again @beat-buesser I'm not sure the ability to run multiple images at a time is working as expected.
If I pass in a single x with multiple images, I get a different answer compared to if I pass in each image one at a time.
Here's a hopefully minimal reproducible example
from functools import partial
from art.estimators.classification import PyTorchClassifier
from art.attacks.evasion import SimBA
import numpy as np
import torch
import torch.nn as nn
from torchvision import transforms, datasets
from tqdm import tqdm

def run_batch(model, x, y, normalizer=None):
    """runs a batch of images through model to get
    predictions, compute which are correct, and
    from that compute accuracy

    Parameters
    ----------
    model : torch.nn.Module
        instance of a neural network model
    x : torch.Tensor
        input to network
    y : torch.Tensor
        ground truth that predictions should match
    normalizer : robustness.Normalizer
        used to normalize inputs to network.
        Default is None, in which case no normalization is applied.

    Returns
    -------
    y_pred : list
        of class predictions, same size as y
    y_prob : list
        of float, the probabilities that the model
        assigned to the classes in ``y_pred``.
        I.e., the scalar value from softmax(output)
        indexed by argmax(softmax(output)),
        for each input in ``x``.
    acc : float
        number correct / total number of samples in batch
    """
    if normalizer is not None:
        x = normalizer(x)
    with torch.no_grad():
        out = model(x)
        y_pred = out.argmax(dim=1)
        y_prob = torch.nn.Softmax()(out)[:, y_pred].flatten()
    correct = (y_pred == y).cpu().numpy().tolist()
    acc = sum(correct) / len(correct)
    return y_pred.cpu().numpy().tolist(), y_prob.cpu().numpy().tolist(), acc

# ... set up model etc
# for simba attack, need to make model output probabilities
model.fc = nn.Sequential(
    *[model.fc, nn.Softmax(dim=1)],
)

log_or_print('setting up adversarial attacks')

# OMITTED custom code that uses `robustness` library
# to get dataloaders etc
# we're using Imagenette dataset from fastai

# constants from torchvision references training script
mean = np.asarray(constants.IMAGENET_MEAN).reshape((3, 1, 1)).astype(np.float32)
std = np.asarray(constants.IMAGENET_STD).reshape((3, 1, 1)).astype(np.float32)
    nb_classes = 1000
    input_shape = (3, 224, 224)

clf = PyTorchClassifier(
    model=model,
    clip_values=(0., 1.),
    loss=criterion,
    input_shape=input_shape,
    nb_classes=nb_classes,
    preprocessing=(mean, std)
)

attack = SimBA(classifier=clf,
               attack='dct',
               max_iter=5000,
               order='random',
               epsilon=0.9,
               freq_dim=28,
               stride=7,
               )

MAX_GOOD_BATCHES = 4

good_batches = []
n_good_batches = 0

run_batch_partial = partial(helpers.run_batch,
                            model=model,
                            normalizer=normalizer,
                            y=y)

for batch in test_loader:
    x, y = batch['img'].to(device), batch['target'].to(device)
    source_img = batch['source']
    y_true = y.cpu().numpy().tolist()

    # ---- unattacked
    log_or_print('\trunning unattacked batch')
    y_pred, correct_unat, acc_unat = run_batch_partial(x=x)

    log_or_print(
        f'\t\tbatch accuracy: {acc_unat:.3f}'
    )
    if acc_unat > 0.99:
        good_batches.append(x)
        n_good_batches += 1
        if n_good_batches > MAX_GOOD_BATCHES:
            break

x = good_batches[2]

with torch.no_grad():
    y_probs = model(normalizer(x))
y_probs = y_probs.cpu().numpy()
x_np = x.cpu().numpy()
x_at = attack.generate(x=x_np, y=y_probs)

x_at_tensor = torch.from_numpy(x_at).to(device)
y_pred_at, correct_at, acc_at = run_batch_partial(x=x_at_tensor)
log_or_print(
    f'\t\tbatch accuracy: {acc_at:.3f}'
)

# gives me batch accuracy: 0.667

for x_img in x:
    x_img = torch.unsqueeze(x_img, 0)
    with torch.no_grad():
        y_probs = model(
            normalizer(x_img)
        )
    y_probs = y_probs.cpu().numpy()
    x_img = x_img.cpu().numpy()
    x_at = attack.generate(x=x_img, y=y_probs)

    x_at_tensor = torch.from_numpy(x_at).to(device)
    y_pred_at, correct_at, acc_at = run_batch_partial(x=x_at_tensor)
    log_or_print(
        f'\t\tbatch accuracy: {acc_at:.3f}'
    )

# gives me
# batch accuracy: 0.000
# batch accuracy: 0.000
# batch accuracy: 0.000

I can go as far as writing a whole script if you're not able to reproduce this
AFAICT SimBa is working without running multiple batches so if you don't need to add that functionality I would be fine with it the way it is, not sure about other users
I still think the other changes in this PR like the check for probabilities and the variable name change would be useful regardless

Hi @Embeddave I have created a similar script based on ART's get_started_pytorch.py and it achives the same accuracy with calling generate on all images or on each image separately:
import torch.nn
import torch.nn as nn
import torch.nn.functional as F
import torch.optim as optim
import numpy as np

from art.estimators.classification import PyTorchClassifier
from art.utils import load_mnist

# Step 0: Define the neural network model, return logits instead of activation in forward method

class Net(nn.Module):
    def __init__(self):
        super(Net, self).__init__()
        self.conv_1 = nn.Conv2d(in_channels=1, out_channels=4, kernel_size=5, stride=1)
        self.conv_2 = nn.Conv2d(in_channels=4, out_channels=10, kernel_size=5, stride=1)
        self.fc_1 = nn.Linear(in_features=4 * 4 * 10, out_features=100)
        self.fc_2 = nn.Linear(in_features=100, out_features=10)
        self.softmax = torch.nn.Softmax(dim=1)

    def forward(self, x):
        x = F.relu(self.conv_1(x))
        x = F.max_pool2d(x, 2, 2)
        x = F.relu(self.conv_2(x))
        x = F.max_pool2d(x, 2, 2)
        x = x.view(-1, 4 * 4 * 10)
        x = F.relu(self.fc_1(x))
        x = self.fc_2(x)
        x = self.softmax(x)
        return x

# Step 1: Load the MNIST dataset

(x_train, y_train), (x_test, y_test), min_pixel_value, max_pixel_value = load_mnist()

# Step 1a: Swap axes to PyTorch's NCHW format

x_test = x_test[0:100]
y_test = y_test[0:100]

x_train = np.transpose(x_train, (0, 3, 1, 2)).astype(np.float32)
x_test = np.transpose(x_test, (0, 3, 1, 2)).astype(np.float32)

# Step 2: Create the model

model = Net()

# Step 2a: Define the loss function and the optimizer

criterion = nn.CrossEntropyLoss()
optimizer = optim.Adam(model.parameters(), lr=0.001)
# optimizer = optim.SGD(model.parameters(), lr=0.01)

# Step 3: Create the ART classifier

classifier = PyTorchClassifier(
    model=model,
    clip_values=(min_pixel_value, max_pixel_value),
    loss=criterion,
    optimizer=optimizer,
    input_shape=(1, 28, 28),
    nb_classes=10,
)

# Step 4: Train the ART classifier

classifier.fit(x_train, y_train, batch_size=64, nb_epochs=1)

# Step 5: Evaluate the ART classifier on benign test examples

predictions = classifier.predict(x_test)
accuracy = np.sum(np.argmax(predictions, axis=1) == np.argmax(y_test, axis=1)) / len(y_test)
print("Accuracy on benign test examples: {}%".format(accuracy * 100))

from art.attacks.evasion import SimBA

attack = SimBA(classifier=classifier,
               attack='dct',
               max_iter=5000,
               order='random',
               epsilon=0.9,
               freq_dim=28,
               stride=7,
               verbose=False,
               )

# attack `all`

x_test_adv_all = attack.generate(x=x_test, y=y_test)

predictions = classifier.predict(x_test_adv_all)
accuracy = np.sum(np.argmax(predictions, axis=1) == np.argmax(y_test, axis=1)) / len(y_test)
print("Accuracy on adversarial test examples - all: {}%".format(accuracy * 100))

# attack `single`

count_correct = 0

for i in range(x_test.shape[0]):

    x_test_adv_single = attack.generate(x=x_test[[i]], y=y_test[[i]])

    prediction = classifier.predict(x_test_adv_single)
    accuracy = np.sum(np.argmax(prediction, axis=1) == np.argmax(y_test[[i]], axis=1))
    if accuracy:
        count_correct += 1

print("Accuracy on adversarial test examples - single: {}%".format(count_correct / x_test.shape[0] * 100))

Thank you @beat-buesser for providing this script.
I will test using it with my images and models this week.
If I still get the same answer where attack success depends on the number of images passed in, I will upload a minimal reproducible example.

Hi @Embeddave Thank you for your review and testing. To prepare for the ART 1.9 release, I'll merge this PR to include the exception and the processing of mini-batches. In case there are still bugs left we can fix them with a new PR for 1.9.0 or 1.9.1.

Understood, thank you @beat-buesser


travis stages sequence problem

to fix travis stages sequence

:tada: This PR is included in version 0.0.7 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Get R from RStudio managed apt packages

Instructions at https://docs.rstudio.com/resources/install-r/
Simpler than getting it from CRAN apt repositories, as it's a single deb that is
downloaded on a per-version basis rather than an entire apt repository with
all the quirks that entails.
RStudio provides debs all R versions for Ubuntu 20.04, 18.04 and 22.04 -
making it much easier for us to switch to a different base Ubuntu image (see
#909)
The packages are built from https://github.com/rstudio/r-builds, so even if RStudio
stops providing them in the future, they can be built.
Better compatibility with binary builds from packagemanager.rstudio.com (compared to
R from conda), as these are appropriate deb builds for particular Ubuntu versions that
also come from rstudio.
We just pass through full versions to the RStudio CDN - so new R versions will now
be available as soon as RStudio hosts them! No more changes required in repo2docker
when a new R version is available. This means this PR also adds support for R 4.2.
This is awesome!

Nah, RStudio < 4.0 was already broken lol. Fixed by https://github.com/jupyterhub/repo2docker/pull/1148.
Had to tell rstudio exactly where R is by setting R_HOME: https://github.com/jupyterhub/repo2docker/pull/1161/commits/0fed94cd6a4cbac973723cd8f165f7e35344e92e
Also, it rstudio will fail to start if the uid is < 1000. And on my mac, my uid was 502! So I had to test repo2docker with a --user-id=1000 flag to get it to work. Took me a long time.

Thanks a lot for the review, @manics! I think I've addressed your comments now

@consideRatio ty, I've accounted for both your suggestions i think.

How risky do you think this change is to users? Are you happy for it to go into a tag/release in the near future, or do you think it needs extended testing on mybinder.org first?

@consideRatio done!
@manics in general i always like testing in repo2docker for a week or two before a release :D

@consideRatio @manics anything else i can do to help get this merged? :D

I saw no unresolved comments and this has been open for a long time so i figure a merge is in order! Thanks Yuvi and good that you pinged!!

Yay thanks a lot for the merge, @consideRatio!


Add Haxe support

Very cool extension!
Regex literals in Haxe are very similar to those in TS / JS, so it was relatively simple to add support for them. The biggest difference is that they start with ~ (~/regex/flags instead of /regex/flags).
There are two other changes I made to the regexRegex:

changed the flags to the ones accepted by Haxe
accept a trailing dot for cases like this:

Seems to work quite well so far, here it is in action:

If you want to test it, the vshaxe codebase is a good test bed (quite a few regex literals) - just search for ~/ with find in files.

Wow, how cool is this!

@chrmarti Is there anything about this PR that needs to be fixed before it can be considered for merging?

@Gama11 Thanks for notifying me, this had slipped my attention. Merged and released. Thanks!


Doc attachments are not available in the Symbol

Description:
Cannot find the documentation attachments in the BSymbol
Affected Versions:
v0.973.0

Closing the PR, Since the fix #8992 has been merged


Use getaddrinfo instead of inet_aton in Client::Connect

This following are now supported, in addition to IPv4 addresses:

hostnames
IPv6 addresses
service names instead of port numbers

Also fixes a couple of deficiencies:

Initializes the serializer, allowing the ct tool to deserialize v1 SCTs.
Loads the system CA roots by default, which allows most certificates to be verified.

Done with this round, just some changes to the description, really.

LGTM


Add Subgroup unittest

What type of PR is this?
Add cluster subgroup unittest
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

#545

Codecov Report

Merging #546 (7ffede1) into main (3db7cc8) will decrease coverage by 0.31%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #546      +/-   ##
==========================================
- Coverage   13.83%   13.52%   -0.32%
==========================================
  Files          65       66       +1
  Lines        7186     7477     +291
==========================================
+ Hits          994     1011      +17
- Misses       6117     6386     +269
- Partials       75       80       +5

Impacted Files
Coverage Δ

...rnet/clusternet/pkg/scheduler/algorithm/generic.go
5.84% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


New version: Bokeh v0.4.1

Registering package: Bokeh
Repository: https://github.com/cjdoris/Bokeh.jl
Created by: @cjdoris
Version: v0.4.1
Commit: 5de0bab3912e6ec6e575226f7ae97742fdc7e0f9
Reviewed by: @cjdoris
Reference: https://github.com/cjdoris/Bokeh.jl/commit/5de0bab3912e6ec6e575226f7ae97742fdc7e0f9#commitcomment-85285175
Description: Interactive plotting made easy

Sorry, I made an error, please don't merge.


Handle bad response

Empty
Non-conformant
... etc

I think this might be related to #4 as it would involve checking the response. If we tailor the request properly, I think for this assignment we should assume a proper response.


Update delete IP docs

Update docs around IP deletion for https://github.com/tumblr/collins/pull/596

Thanks Michael. Sorry about leaving this sitting for so long

No problem :) Thanks for merging.


Feature/alphabetize groups

A change to the Groups.json to alphabetize the listing by group name, and add a description to most groups.
Also updated the groups page to display the description, if available, to help users find groups that may suite their needs easier.

@perfectloser Thanks for this! :heart:


Update the manifests to use the AlwaysPull policy

We need to update our manifests to use the AlwaysPull image policy so that users can simply re-deploy the Portainer application and upgrade to the latest version.

@deviantony this is already the case in recent versions of the chart : https://github.com/portainer/k8s/blob/master/charts/portainer/values.yaml#L18

@deviantony this is already the case in recent versions of the chart : https://github.com/portainer/k8s/blob/master/charts/portainer/values.yaml#L18

@funkypenguin Not for the manifests deployment though?

@funkypenguin Not for the manifests deployment though?

Closing as this is already the case.

Closing as this is already the case.


Queue consumer improvements

Unifies codepaths, improves performance for the batch consumer.
Testing Plan: Existing and modified tests.

I added a comment as @mateuszmandera suggested and merged the series to master.  Thanks @alexmv!  The performance improvement for the batched work in the UserActivity worker is likely significant to Zulip's overall scalability without sharding that queue.
One follow-up thing we should investigate is whether this set of changes has been successful in achieving the goal of LoopQueueProcessingWorker not spinning CPU when idle; I'm not sure if the inactivity_timeout of 1 second is a problem there.  (Maybe we want a different timeout if we've collected 0 events vs. if we've collected at least 1 so that it can sleep a long time when there are none?  I'm not confident of the mechanics)