Replace redunant calls to getValue

This PR:

removes redundant calls to getValue with a single call + variable

Merged with 97a80883f8a1724613e4a44c3bae4d6180c91191.


Deprecated cypher options used

Openvpn server is throwing this warning on the logs:
DEPRECATED OPTION: --cipher set to 'AES-256-CBC' but missing in --data-ciphers (AES-256-GCM:AES-128-GCM). Future OpenVPN version will ignore --cipher for cipher negotiations. Add 'AES-256-CBC' to --data-ciphers or change --cipher 'AES-256-CBC' to --data-ciphers-fallback 'AES-256-CBC' to silence this warning.
There do not seem to be options to set up this on the module.

You can set the cipher to AES-256-GCM - that's a supported option. But data-cipher isn't supported other than through openvpn_addl_server_options.
Reading https://community.openvpn.net/openvpn/wiki/CipherNegotiation it seems like the best path forward would be to let OpenVPN use the builtin defaults, but that's not backward compatible.
I'm changing the role to use AES-256-GCM:AES-128-GCM:AES-256-CBC as the default data-cipher because that overlaps with the current default of AES-256-CBC.


Useragentstring.com throws 404

Hi @hellysmile!
Thank you for this nice library, it is a really nice addition to the community! Today I tried to use the library but got stuck when it tried to connect to http://useragentstring.com/pages/Chrome/.
It seems as if that page throws a 404. I have checked through the Wayback Machine and it seems as if that page existed in the past. My guess is that useragentstring.com changed their website architecture.
If you believe the site was down temporarily and everything works as expected then please mark this issue as solved.
Thank you so much!

pip install -U fake-useragent

That was solved. It seems You using old version...

Alright, error on my side, sorry!
Thanks for your quick response!


Reverse QR code reading on Android

Hi, first of all, thank you for the plugin, it works great and its also free. I have a problem when scanning reverse QR codes. I need to scan white QR codes on dark backgrounds, and the plugin is not able to do so currently. I saw an issue on another fork where they fix this with some minor changes, but its only for IOS, and I'm using it on Android. Could you please take a look at it? I'm thinking it could also be a small change for Android, but I'm not very good with Java. Here's the original link: phonegap#88
Thank you for your help

Hi @flagohuvelle, have you found a way to achieve this?
Thanks!


chore: cleanup unused files & npm audit fix

The changes to package.json are re-arranging the sections to be consistent with all of our other README's, and an npm audit fix.

Now the tests are all green, and so I'm going to land this.


Suggestion: “Updating” subsection in “Install” is unclear

There’s no description for the point: should there be?

Thanks. I edited this.


[VS Code] Could not connect to the server. Check that you are connected to the internet.

LiveShareError.zip
Error:
Could not connect to the server. Check that you are connected to the internet. I have VS code on my windows and mac.
Live share works on my windows but when I tried to sign in using my Mac it gave me this error.
I have reinstalled VS Code, uninstalled and reinstalled the extension, tried to use the code to sign in (results in a never-ending 'signing in' message) and retry with both my GitHub account and my Microsoft account. None of this works and I get the same error.
Running on MacOs Sierra version 10.12.5

Version Data

extensionName
VSLS

extensionVersion
0.3.379

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.25.0

platformName
MacOS

platformVersion
16.6.0

Here is the exception from the logs:
[2018-07-09 12:27:33.383 Agent.Proxy W] Connecting to <https://insiders.liveshare.vsengsaas.visualstudio.com/f322148c> via proxy failed with exception System.Net.Http.HttpRequestException: An error occurred while sending the request. ---> System.Net.Http.CurlException: Couldn't connect to server
   at System.Net.Http.CurlHandler.ThrowIfCURLEError(CURLcode error)
   at System.Net.Http.CurlHandler.MultiAgent.FinishRequest(StrongToWeakReference`1 easyWrapper, CURLcode messageResult)
   --- End of inner exception stack trace ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.ConfiguredTaskAwaitable`1.ConfiguredTaskAwaiter.GetResult()
   at System.Net.Http.HttpClient.<FinishSendAsyncBuffered>d__58.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.Cascade.Agent.DefaultWebProxy.<ConnectToUriAsync>d__8.MoveNext() in E:\A\_work\33\s\src\Agent\DefaultWebProxy.cs:line 103
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.Cascade.Agent.DefaultWebProxy.<EnableConnectionViaProxyOrResetAsync>d__7.MoveNext() in E:\A\_work\33\s\src\Agent\DefaultWebProxy.cs:line 73

@NotAkash Thanks for reporting the issue!
Can you please check to see whether you have Auto Proxy Discovery enabled in your System Preferences -> Network settings (underneath Advanced, then Proxies)? This appears to be a common cause for this error, and turning it off may resolve it for you. We're tracking this issue via #447, so I'm going to close this as a duplicate, but let me know if this unblocks you or not. Thanks!

@IlyaBiryukov Thank you for the prompt response, followed your instructions, everything works fine now. Sorry for the duplicate.


Material Optimization recommendation not matching actual results for work order

The material optimization window recommended mats that should give me Rank 3 results. However, when plugged into the actual crafting interface, it results in a Rank 2 item.

does the heradly give difficulty?

Yes, it's an embellishment with +25 difficulty for the rank 1.

I found a related case, where the work order material optimization is completely wrong without any optional reagents.
In this case, there is no possibility of reaching rank 5. The recommended materials only give rank 4. Using all rank 3 mats also gives only rank 4. This character is mail-specialized and has zero points in leather.

Actually I was only part right. The Forged Aspirant's Heraldry (top slot) adds +50 recipe difficulty. That might be the offset. Testing in simulation mode with the different heraldry options, it's not changing the recipe difficulty.

Fixed with the pvp reagents fix


If I try to join first and then host, I get an error with Failed to listen on port

The use case is that I connect to Noray with both game instances.
Then, with one of them I do client connect(join) to the other instance(the other instance isn't yet started as host).
Then, with other instance I start it as a host.
I get the following error:
Failed to connect to <IP_ADDRESS>:51200 with status 1
After the client fails to connect, I can host normally. I wonder why though. Is that port in use while the client is connecting? I am trying with the ForestBrawl demo.

If I try to join first and then host

If I understand your question correctly, that's the intended behaviour. Before you click the host button, the game host is not listening for connections, thus noone can connect.
Maybe the point of confusion is the buttons on the Noray UI? The Connect button only connects to the noray server, and registers its address, so noray can forward traffic there.
Think of it like announcing your address to the post office before moving in to your new place. They can send you letters, but you won't be there to read them. Only once you moved can you read the letters. Moving in in this example is starting the host, i.e. listening for connections.
Let me know if that clears it up, and/or if there's a specific point in the documentation that you feel didn't provide enough info on this!

I see now! The issue was that when the host receives the handshake command from noray, it has no role yet. Roles are set when you press the host or the join button.
Without a role, it assumes that you're joining a game, so it tries to connect. So it creates a UDP connection, binds the registered local port, and the connection fails, since both clients are trying to connect to eachother, with neither of them listening.
Created PR with fix.


Powershell crash?

Issue Type: Bug
Hi,
This issue is somewhat random, but happens very often. I open my simple VueJS+babel+node project. Start working, open terminal run any command (here it has 50/50 chance of crashing, sometimes just opening terminal crashes it) . Error looks like this person's I found online:
https://user-images.githubusercontent.com/23094645/30272291-444ee39c-970d-11e7-9ac2-e4a761f79589.png
Then it is stuck in infinite error loop, because restarting the app causes the console and this error to come up immediately. The only way I figured out to stop it is to press Ctrl+` to close console split second before the error pops up on restart of app.
Hopefully this can be replicated somehow. I dont have any extension except VueJS, but disabling it didnt help.
VS Code version: Code 1.21.1 (79b44aa704ce542d8ca4a3cc44cfca566e7720f1, 2018-03-14T14:47:13.351Z)
OS version: Windows_NT ia32 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700K CPU @ 4.00GHz (8 x 4008)

Memory (System)
31.96GB (19.99GB free)

Process Argv
C:\Program Files (x86)\Microsoft VS Code\Code.exe

Screen Reader
no

VM
0%

Extensions: none
Reproduces without extensions

@inemtsev could you try get a performance profile before the crash happens using devtools (Help > Toggle Developer Tools)?

Not sure what I'm looking for but here is the screenshot of the result

@inemtsev hmm, I'm a little stumped on this one. Could you try reproducing in the Insiders build which is built from the latest code? https://code.visualstudio.com/insiders/

This is interesting indeed, I uninstalled my VS Code and installed from the link you provided...The crash still happens

@inemtsev some more follow ups:

Could you clarify the characteristics of the crash?

Does a dialog come up saying the window is not responding?
Is the UI responding but just laggy?
Is it just the powershell process that's crashing?

Does it happen when you run powershell.exe outside of VS Code?
Does it happen when you set the default shell to cmd.exe inside VS Code?


NullPointerException when you click on image in MediaDetailFragment

Summary:
App crashes when you click on the image in MediaDetailFragment.
Steps to reproduce:

Click on one of the images in Contributions tab
When MediaDetailFragment loads the particular media click on the image again. Do it immediately before the other fields such as license, description, depictions are loaded.

System logs:
USER_COMMENT=
APP_VERSION_CODE=561
APP_VERSION_NAME=2.12.3-debug-master
ANDROID_VERSION=9
PHONE_MODEL=AOSP on IA Emulator
STACK_TRACE=java.lang.NullPointerException: uriString
	at android.net.Uri$StringUri.<init>(Uri.java:490)
	at android.net.Uri$StringUri.<init>(Uri.java:480)
	at android.net.Uri.parse(Uri.java:452)
	at fr.free.nrw.commons.media.MediaDetailFragment.launchZoomActivity(MediaDetailFragment.java:231)
	at fr.free.nrw.commons.media.MediaDetailFragment_ViewBinding$1.doClick(MediaDetailFragment_ViewBinding.java:48)
	at butterknife.internal.DebouncingOnClickListener.onClick(DebouncingOnClickListener.java:18)
	at android.view.View.performClick(View.java:6597)
	at android.view.View.performClickInternal(View.java:6574)
	at android.view.View.access$3100(View.java:778)
	at android.view.View$PerformClick.run(View.java:25885)
	at android.os.Handler.handleCallback(Handler.java:873)
	at android.os.Handler.dispatchMessage(Handler.java:99)
	at android.os.Looper.loop(Looper.java:193)
	at android.app.ActivityThread.main(ActivityThread.java:6669)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:493)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:858)

IS_SILENT=false
USER_EMAIL=
USER_CRASH_DATE=2020-05-05T10:12:02.219+05:30
REPORT_ID=c543b132-25f8-4e69-ba28-5168cb41e410

Device and Android version:
PHONE_MODEL=AOSP on IA Emulator (Pixel 2 emulator)
ANDROID_VERSION=9
Commons app version:
APP_VERSION_NAME=2.12.3-debug-master
FLAVOUR=betaDebug
Would you like to work on the issue?
I would like to work on this issue.

Assigned to you, go for it @gvaibhav1734 . :)
I suggest you start by verifying that the problem also exists on prodDebug.

Problem occurs on prodDebug as well.
The line of code in onClickListener for the ImageView causing the issue:
startActivity(Intent(ctx,ZoomableActivity.class).setData(Uri.parse(media.getImageUrl())))

media.getImageUrl() depends on an a network call to be populated. Would a simple null check be sufficient?

A null check should be fine, what do you say @misaochan ?

If we just don't do anything when media.getImageUri() returns null, will it still be able to display the image after the network call succeeds?

We can, but that might not be a good thing,
User could be interested in browsing other  things once he realises image has not loaded yet and we not knowing his intention would take him to the zoom activity even though he might not be interested in that at that moment

Good point. Let us go with the simple null check for now, and see how it goes. :)


Inconsistent Form State Due to Missing Return in Expression

I encountered an issue with the form related to the data_pensione field. The expression for hiding the field is currently implemented as follows:
{
          key: 'data_pensione',
          type: 'date',
          className: 'col-md-3',
          templateOptions: {
            translate: true,
            label: 'd1_txt16',
            required: true,
          },
          expressions: {
            hide: (field: any) => {
              if ((field.model.specif_misura_ridotta !== 'D3A' && field.model.specif_misura_ridotta !== 'D3B') || (field.model.flag_misura_ridotta === 0 || field.model.flag_misura_ridotta === false)) {
                return true;
              }
            },
          'templateOptions.required': (field: FormlyFieldConfig) => {
              return !field.hide;
            }

The expression does not explicitly return a value when the condition is not met, leading to an inconsistent state in the form without any error messages. As shown in the attached figure, there is a field error, but the form is still marked as valid.

To resolve this issue, we need to ensure that the function always returns a boolean value. I modified the expression to explicitly return false when the conditions are not met, and it works as expected:
expressions: {
            hide: (field: any) => {
              if ((field.model.specif_misura_ridotta !== 'D3A' && field.model.specif_misura_ridotta !== 'D3B') || (field.model.flag_misura_ridotta === 0 || field.model.flag_misura_ridotta === false)) {
                return true;
              }
              return false;
            },
           'templateOptions.required': (field: FormlyFieldConfig) => {
              return !field.hide;
            }

This issue has been fixed and released as part of v6.3.8 release.
Please let us know, in case you are still encountering a similar issue/problem.
Thank you!

@aghosh-evertz I think its an issue with Typescript somehow, the main goal here is to warn user to return a boolean which does for regular case such as the following example:

I believe this error started coming post 6.3.8 release. Prior to that, we didn't have such issues. For now, I had to explicitly provide a default return statement at the end of the 'hide' expression to make it work.
This might break all the existing hide expressions which has conditional return statements only.

Return undefined for hide expression may provide an inconsistant result thats why we push that fix in a minor version. Anuway I'll try to revert it in the upcoming release and move the fix to v7


Component does not render, blocks render.

The component does not seem to render at all, and in our case actually blocks any other components after it from rendering:
import 'flatpickr/dist/flatpickr.material_green.min.css';

import React, { Component, PropTypes } from 'react';
import Flatpickr from 'react-flatpickr';

/**
 * Date field
 *
 * @author Andrew McLagan<EMAIL_ADDRESS> */
export default class DateField extends Component {

  static propTypes = {
    value: PropTypes.any.isRequired,
    onChange: PropTypes.func.isRequired,
    onBlur: PropTypes.func.isRequired,
    name: PropTypes.string.isRequired,
    placeholder: PropTypes.string,
  }

  render() {

    const { name, placeholder } = this.props;

    return (
      <div className="date-field clearfix">
        <Flatpickr
          data-enabletime
          onChange={() => console.log('123')}
        />
      </div>
    );
  }
}

The issue is you are not importing react:
import { Component, PropTypes } from 'react'
to
import React, { Component, PropTypes } from 'react'

Not everyone has react as a global webpack import.

Yeap!


Create release workflow

This workflow automates the release for this gem on push to main. https://github.com/newrelic/newrelic-telemetry-sdk-ruby/issues/16
This will build the gem, determine the version, tag the version on github, and push to rubygems.
I copied the scripts over from the agent for releasing to use in this workflow as well.
The secrets used have already been added to the repo.

FYI, I cherry-picked the Snyk bundler fix from the documentation branch into this one.


cmd/compile: possible PGO miscompilation in biogo-igor benchmark

The biogo-igor benchmark part of Sweet has been failing regularly at tip in the end-to-end test. It specifically fails when being run under PGO.
Broken out from #56958.

CC @golang/compiler

Example failure looks like:
error: piling: record on line 0; parse error on line 1, column 3: strconv.ParseInt: parsing "16243440": value out of range

2024-09-20 15:27 x_benchmarks-gotip-linux-amd64-longtest benchmarks@b8e725b7 go@c208b913 x/benchmarks/sweet/cmd/sweet.TestSweetEndToEnd (log)


Remove standard signature from models and move it to servers

WIP
You can check this with code (assuming you have a mlem model at "sklearn")
from mlem.api import serve
from mlem.contrib.fastapi import FastAPIServer
from mlem.core.metadata import load_meta
from mlem.core.model import Argument, Signature
from mlem.core.objects import MlemModel
from mlem.runtime.interface import InterfaceDescriptor

def main():
    serve("sklearn", "fastapi", standardize=True)
    #
    serve("sklearn", "fastapi", standardize=False)

    model = load_meta("sklearn", force_type=MlemModel)
    predict = model.model_type.methods["predict"]
    desciptor = InterfaceDescriptor(version="", methods={
        "kek": Signature(name="kek",
                         args=[Argument(name="lol", type_=predict.args[0].type_)],
                         returns=predict.returns)
    })
    serve("sklearn", FastAPIServer(interface=desciptor), standardize=False)

if __name__ == '__main__':
    main()

Need to check how this will work for builds/deployments
closes #445

Is this still WIP? If yes, it is better to indicate in title :P

It's a WIP because I need your feedback to continue :)

Will review today

@mike0sv, thanks for the example - it was VERY helpful.
Tried your script, how it works looks good to me.

Unifying endpoints is not something we got requests for yet, so keeping it simple (2 endpoints instead of 4 makes sense to me). I know I suggested this in the very beginning 😅 , so maybe strange to hear this from me, but simple and straighforward is better here IMO - and then we can improve.
For this very reason I don't have any feedback about InterfaceDescriptor - let's wait until someone use it and then we can improve.
Will serve("sklearn", "fastapi", standardize=True) rename the endpoint as well? (not only arguments)
serve("sklearn", FastAPIServer(interface=desciptor), standardize=True) - it uses custom interface even standardize=True - confusing to me. Should throw away some error maybe? Or standardize stuff (was assuming the flag gets priority here TBH, but maybe someone else can assume otherwise).

Yes
I use "more granular setting takes preference", so flag is overrided by custom server signature. We can add this to the docstring
Actually, it might be better to move this field to Server class so it can be configured "permanently"

Actually, it might be better to move this field to Server class so it can be configured "permanently"

Yes, that makes sense to me!

Hmm, tests pass, but when trying to run your code snippet from the first message, I get this:

INFO:     <IP_ADDRESS>:59115 - "GET /openapi.json HTTP/1.1" 500 Internal Server Error
ERROR:    Exception in ASGI application
Traceback (most recent call last):
  File "/Users/aguschin/.local/share/virtualenvs/mlem-Utz6DvOn/lib/python3.10/site-packages/uvicorn/protocols/http/h11_impl.py", line 404, in run_asgi
    result = await app(  # type: ignore[func-returns-value]
  File "/Users/aguschin/.local/share/virtualenvs/mlem-Utz6DvOn/lib/python3.10/site-packages/uvicorn/middleware/proxy_headers.py", line 78, in __call__
    return await self.app(scope, receive, send)
  File "/Users/aguschin/.local/share/virtualenvs/mlem-Utz6DvOn/lib/python3.10/site-packages/fastapi/applications.py", line 270, in __call__
    await super().__call__(scope, receive, send)
  File "/Users/aguschin/.local/share/virtualenvs/mlem-Utz6DvOn/lib/python3.10/site-packages/starlette/applications.py", line 124, in __call__
    await self.middleware_stack(scope, receive, send)
  File "/Users/aguschin/.local/share/virtualenvs/mlem-Utz6DvOn/lib/python3.10/site-packages/starlette/middleware/errors.py", line 184, in __call__
    raise exc
  File "/Users/aguschin/.local/share/virtualenvs/mlem-Utz6DvOn/lib/python3.10/site-packages/starlette/middleware/errors.py", line 162, in __call__
    await self.app(scope, receive, _send)
  File "/Users/aguschin/.local/share/virtualenvs/mlem-Utz6DvOn/lib/python3.10/site-packages/starlette/middleware/exceptions.py", line 75, in __call__
    raise exc
  File "/Users/aguschin/.local/share/virtualenvs/mlem-Utz6DvOn/lib/python3.10/site-packages/starlette/middleware/exceptions.py", line 64, in __call__
    await self.app(scope, receive, sender)
  File "/Users/aguschin/.local/share/virtualenvs/mlem-Utz6DvOn/lib/python3.10/site-packages/fastapi/middleware/asyncexitstack.py", line 21, in __call__
    raise e
  File "/Users/aguschin/.local/share/virtualenvs/mlem-Utz6DvOn/lib/python3.10/site-packages/fastapi/middleware/asyncexitstack.py", line 18, in __call__
    await self.app(scope, receive, send)
  File "/Users/aguschin/.local/share/virtualenvs/mlem-Utz6DvOn/lib/python3.10/site-packages/starlette/routing.py", line 680, in __call__
    await route.handle(scope, receive, send)
  File "/Users/aguschin/.local/share/virtualenvs/mlem-Utz6DvOn/lib/python3.10/site-packages/starlette/routing.py", line 275, in handle
    await self.app(scope, receive, send)
  File "/Users/aguschin/.local/share/virtualenvs/mlem-Utz6DvOn/lib/python3.10/site-packages/starlette/routing.py", line 65, in app
    response = await func(request)
  File "/Users/aguschin/.local/share/virtualenvs/mlem-Utz6DvOn/lib/python3.10/site-packages/fastapi/applications.py", line 225, in openapi
    return JSONResponse(self.openapi())
  File "/Users/aguschin/.local/share/virtualenvs/mlem-Utz6DvOn/lib/python3.10/site-packages/fastapi/applications.py", line 200, in openapi
    self.openapi_schema = get_openapi(
  File "/Users/aguschin/.local/share/virtualenvs/mlem-Utz6DvOn/lib/python3.10/site-packages/fastapi/openapi/utils.py", line 423, in get_openapi
    definitions = get_model_definitions(
  File "/Users/aguschin/.local/share/virtualenvs/mlem-Utz6DvOn/lib/python3.10/site-packages/fastapi/utils.py", line 45, in get_model_definitions
    m_schema, m_definitions, m_nested_models = model_process_schema(
  File "pydantic/schema.py", line 580, in pydantic.schema.model_process_schema
  File "pydantic/schema.py", line 621, in pydantic.schema.model_type_schema
  File "pydantic/schema.py", line 254, in pydantic.schema.field_schema
  File "pydantic/schema.py", line 461, in pydantic.schema.field_type_schema
  File "pydantic/schema.py", line 922, in pydantic.schema.field_singleton_schema
KeyError: <class 'pydantic.main.DataFrameRow'>

pydantic==1.10.2
This is for both old models (4 methods for iris classifier) and new (2 methods).

just fyi, the problem is still there)

That was wrongdoings of #469 :)


Provide a running configuration example

Hi,
Thanks for this great project.
I had some hard time today trying to get this running. My main concern was surely the use of a version not compatible with my elasticsearch setup.
I think a running docker example could be useful to understand how to setup the whole thing.
Hope it helps,

Wow, fantastic!

Thanks :)


[Work in progress] Don't use arial font

I don't find the error message except the failed message :-(

The full log gives more details: https://s3.amazonaws.com/archive.travis-ci.org/jobs/169144129/log.txt
org.mapfish.print.ExamplesTest > testAllExamples FAILED
    java.lang.AssertionError at ExamplesTest.java:142
...

org.mapfish.print.PrintApiTest > testCreateAndGetReport_Success_NoApp FAILED
    java.lang.AssertionError at PrintApiTest.java:379
...

org.mapfish.print.PrintApiTest > testCreateReport_Success_NoApp FAILED
    java.lang.AssertionError at PrintApiTest.java:296
...

org.mapfish.print.ExamplesTest > testAllExamples FAILED
    java.lang.AssertionError at ExamplesTest.java:142

I see that but witch one...

I see that but witch one...

Usually the failed examples are listed. It passes for you locally?

No It failed locally, but no more useful informations ...

It failed locally, but no more useful informations ...

And if you run the tests with a debugger? See also: https://github.com/mapfish/mapfish-print/blob/master/examples/src/test/java/org/mapfish/print/ExamplesTest.java#L36

=> all examples failed :-(
java.util.concurrent.ExecutionException: java.lang.RuntimeException: java.lang.RuntimeException: java.lang.RuntimeException: org.apache.http.conn.HttpHostConnectException: Connect to localhost:8080 [localhost/<IP_ADDRESS>] failed: Connexion refusée

Did you start the test server with GeoServer as described here? https://github.com/mapfish/mapfish-print/blob/master/examples/src/test/java/org/mapfish/print/ExamplesTest.java#L36

Better thanks, now I can start to debug :-)

With my last commit I don't have any error locally but it steel failing :-(

I completely blocked, actually  the test pass locally, the test failed on Travis,
We have relay too many logs on travis, and there isn't possible to have a useful log:

System.out.println => nothing.
LOGGER.error => nothing.
fail => nothing

2 examples failed

* printwms_background_layer (requestData-no-data-layer.json) -> similarity difference between images is: 50.22925622494165 which is greater than the max distance of 50.0

* printwmts_tyger_ny_EPSG_900913 (requestData-scale-wmts1_0_0-full-kvp.json) -> similarity difference between images is: 63.28584611807446 which is greater than the max distance of 50.0

Thanks @tsauerwein :-)

@tsauerwein for me it's ready to review :-)


Fix pip install with older versions of pip

Description
This PR fixes #1009, in which pip installation of napari-0.2.12 failed with older versions of pip.
A file-naming strategy I had used to test plugin discovery in #908 seems to have been incompatible with older versions of pip.  But it was testing a pretty fringe case (where a dist-info entry point points to another existing module), which apparently is so fringe that it used to actually raise an exception. I just renamed a couple files in the plugin test fixtures and modified one test slightly and it works again.
I tested locally by running python setup.py sdist, pip installing that sdist in a clean environment with pip versions as far back as 10.0.1, and running pytest --pyargs napari
Sorry for the problems!
Type of change

[x] Bug-fix (non-breaking change which fixes an issue)

How has this been tested?

[x] all tests pass with my change

The chances that someone else would have done this is next to nothing.  Basically, I created a module napari_test_plugin that was in-itself a valid napari plugin via naming convention.  Then, to also test discovery of a working plugin via entry points, I created a mock dist-info package with an entry_point pointing to that other napari_test_plugin (rather than to it's own module, which is how probably 100% of packages would be shipped).  Pip found it while installing napari and older versions of pip didn't like it.  So I created a new module working_plugin.py, and had the entry_point point to that instead.
I think the main challenge here is that we ship all of our tests with the release.  That's easy enough to do when you have a library that has easily generated test cases, but the goal here is to test plugin discovery with "real looking" mock plugin packages.  Apparently, pip also found those mock packages during installation and got confused due to the very unconventional entry point usage (which was never intended to be used for a real pip installation anyway).


ERROR in ./src/app.jsx 10:9

Pre-flight checklist

[X] I have read the contribution documentation for this project.
[X] I agree to follow the code of conduct that this project uses.
[X] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Electron Forge version
7.3.0
Electron version
29.0.1
Operating system
macOS sonoma 14.3
Last known working Electron Forge version
none
Expected behavior
build succeed while run script npm run make
Actual behavior
const { resolve } = require("./pathUtils");
module.exports = [
// Add support for native node modules
{
// We're specifying native_modules in the test because the asset relocator loader generates a
// "fake" .node file which is really a cjs file.
test: /native_modules[/\].+.node$/,
use: 'node-loader',
},
{
test: /[/\]node_modules[/\].+.(m?js|node)$/,
parser: { amd: false },
use: {
loader: '@vercel/webpack-asset-relocator-loader',
options: {
outputAssetBase: 'native_modules',
},
},
},
{
test: /.jsx?$/,
include: resolve("src"),
use: [
{
loader: "babel-loader",
options: {
exclude: /node_modules/,
presets: ["@babel/preset-react","@babel/preset-env"],
plugins: ["@babel/plugin-transform-react-jsx"]
}
},
],
},
{
test: /.html$/,
use: [
{
loader: "html-loader",
},
],
}
];

<EMAIL_ADDRESS>make
electron-forge make

✔ Checking your system
✔ Loading configuration
✔ Resolving make targets
› Making for the following targets:
❯ Running package command
✔ Preparing to package application
❯ Running packaging hooks
✔ Running generateAssets hook
❯ Running prePackage hook
❯ [plugin-webpack] Preparing webpack bundles
✔ Preparing native dependencies [0.4s]
✖ Building webpack bundles
› Compilation errors in the renderer: asset main_w…
runtime modules 0 bytes 1 module
./src/preload.js 158 bytes [built] [code generat…
webpack 5.90.3 compiled successfully in 290 ms
assets by status 5.53 KiB [cached] 2 assets
runtime modules 796 bytes 5 modules
orphan modules 1.11 KiB [orphan] 1 module
cacheable modules 11 KiB
modules by path ./node_modules/ 8.58 KiB
modules by path ./node_modules/style-loader/dist…
./node_modules/style-loader/dist/runtime/injectS…
./node_modules/style-loader/dist/runtime/styleDo…
+ 4 modules
modules by path ./node_modules/css-loader/dist/r…
./node_modules/css-loader/dist/runtime/sourceMap…
./node_modules/css-loader/dist/runtime/api.js 2.…
modules by path ./src/ 2.38 KiB
./src/renderer.js + 1 modules 1.15 KiB [built] […
./src/app.jsx 323 bytes [built] [code generated]…
./node_modules/css-loader/dist/cjs.js!./src/inde…
ERROR in ./src/app.jsx 10:9
Module parse failed: Unexpected token (10:9)
You may need an appropriate loader to handle thi…
|   });
|
>   return hello world
| }
|
@ ./src/renderer.js 1:0-18
webpack 5.90.3 compiled with 1 error in 621 ms
◼ Packaging application
◼ Running postPackage hook
◼ Running preMake hook
◼ Making distributables
◼ Running postMake hook
Steps to reproduce
const { resolve } = require("./pathUtils");
module.exports = [
// Add support for native node modules
{
// We're specifying native_modules in the test because the asset relocator loader generates a
// "fake" .node file which is really a cjs file.
test: /native_modules[/\].+.node$/,
use: 'node-loader',
},
{
test: /[/\]node_modules[/\].+.(m?js|node)$/,
parser: { amd: false },
use: {
loader: '@vercel/webpack-asset-relocator-loader',
options: {
outputAssetBase: 'native_modules',
},
},
},
{
test: /.jsx?$/,
include: resolve("src"),
use: [
{
loader: "babel-loader",
options: {
exclude: /node_modules/,
presets: ["@babel/preset-react","@babel/preset-env"],
plugins: ["@babel/plugin-transform-react-jsx"]
}
},
],
},
{
test: /.html$/,
use: [
{
loader: "html-loader",
},
],
}
];

<EMAIL_ADDRESS>make
electron-forge make

✔ Checking your system
✔ Loading configuration
✔ Resolving make targets
› Making for the following targets:
❯ Running package command
✔ Preparing to package application
❯ Running packaging hooks
✔ Running generateAssets hook
❯ Running prePackage hook
❯ [plugin-webpack] Preparing webpack bundles
✔ Preparing native dependencies [0.4s]
✖ Building webpack bundles
› Compilation errors in the renderer: asset main_w…
runtime modules 0 bytes 1 module
./src/preload.js 158 bytes [built] [code generat…
webpack 5.90.3 compiled successfully in 290 ms
assets by status 5.53 KiB [cached] 2 assets
runtime modules 796 bytes 5 modules
orphan modules 1.11 KiB [orphan] 1 module
cacheable modules 11 KiB
modules by path ./node_modules/ 8.58 KiB
modules by path ./node_modules/style-loader/dist…
./node_modules/style-loader/dist/runtime/injectS…
./node_modules/style-loader/dist/runtime/styleDo…
+ 4 modules
modules by path ./node_modules/css-loader/dist/r…
./node_modules/css-loader/dist/runtime/sourceMap…
./node_modules/css-loader/dist/runtime/api.js 2.…
modules by path ./src/ 2.38 KiB
./src/renderer.js + 1 modules 1.15 KiB [built] […
./src/app.jsx 323 bytes [built] [code generated]…
./node_modules/css-loader/dist/cjs.js!./src/inde…
ERROR in ./src/app.jsx 10:9
Module parse failed: Unexpected token (10:9)
You may need an appropriate loader to handle thi…
|   });
|
>   return hello world
| }
|
@ ./src/renderer.js 1:0-18
webpack 5.90.3 compiled with 1 error in 621 ms
◼ Packaging application
◼ Running postPackage hook
◼ Running preMake hook
◼ Making distributables
◼ Running postMake hook
Additional information

Hi, I suggest you should follow this guide on "How to create an Electron app with React and Electron Forge".
Maybe this will help fix your errors.


Allow users to view available teams and edit active teams

User facing
This change adds a "View my teams" button to the user welcome card as well as the matching notification cards. Previously, a user was only able to pause or resume matches for all of their Icebreaker-installed teams. The "View my teams" button will send a card which shows all of the user's teams on which Icebreaker is installed.

The user can select teams to pause from this list, and teams which are already paused are automatically selected. Once saved, the user will receive a card listing their active (not paused) teams. This card also includes an "Edit active teams" button, which functions the same way as the "View my teams" button.

Database
In the previous implementation, users were only saved in the database if they paused their matches. In this change, we store the Icebreaker-installed teams for each user. Thus all users on an Icebreaker team are stored, and user information is added/updated when Icebreaker is added to a team that they are on or when they are added to a team with Icebreaker installed. Instead of storing a boolean value for whether a user is opted into matches, we store a dictionary mapping the team ID for a team that a user is on to a boolean value for whether a user is opted in for that team.
Follow-up work
It could be better to combine some buttons into an "Options" button, especially since we will be adding a button to customize bio as well as the ones listed above. Also, unit tests will need to be updated to reflect the database structure changes. With the new implementation, current users of Icebreaker may have to delete and re-install Icebreaker on their teams due to the database changes, but the only information that would be lost is opt-in status.

@alisonfang16  Has this been merged also into the current main version or is this only in v2/main?
Concerned as v2/main seems much older with some last changes made 2 months ago while the main one has the updated deployment powershell script that we need and was updated 15 days ago.

Why is there a "v2/main" in which this is merged into??
Was this created before the current main has been updated to v2, since the description refers to version 2 (March 2021). @alisonfang16


dev-Sentinel-2022-06-01-preview

Please note all PRs to this branch have already been reviewed and approved by Swagger team and ARM. No need for addition ARM review.

/azp run

/azp run


Cannot pull images when logged in to Docker Desktop (Status 500: unauthorized: incorrect username or password)

Test Containers 1.16.3 on Docker Engine 20.10.12 with Docker Desktop using WSL2 Backend on Windows 21H1.
When I am logged in to my account in Docker Desktop, Test Containers can no longer pull images from Docker Hub. Once I log out of my account, Test Containers is able to pull images as expected.
com.github.dockerjava.api.async.ResultCallbackTemplate - Error during callback
com.github.dockerjava.api.exception.InternalServerErrorException: Status 500: {"message":"Head \"https://registry-1.docker.io/v2/testcontainers/ryuk/manifests/0.3.3\": unauthorized: incorrect username or password"}
	at org.testcontainers.shaded.com.github.dockerjava.core.DefaultInvocationBuilder.execute(DefaultInvocationBuilder.java:247)
	at org.testcontainers.shaded.com.github.dockerjava.core.DefaultInvocationBuilder.lambda$executeAndStream$1(DefaultInvocationBuilder.java:269)
	at java.base/java.lang.Thread.run(Thread.java:833)

I think my Docker installation is fine, as I can pull images from the terminal when I am logged in to Docker Desktop. I don't want to log out every time I use Test Containers. How can I fix this issue?
Logs (on INFO): https://pastebin.com/fyExxi76
Logs (on DEBUG): https://pastebin.com/b4H283fL

I have exactly the same issue with Test Container 1.16.2.

Problem still exists with Test Containers 1.16.3 and Docker Desktop with versions: 4.2.0, 4.3.2, 4.6.1 and 4.7.0
Actually, it happens regardless if I'm logged in or not.
12:21:49.321 [main] INFO  org.testcontainers.utility.ImageNameSubstitutor - Image name substitution will be performed by: DefaultImageNameSubstitutor (composite of 'ConfigurationFileImageNameSubstitutor' and 'PrefixingImageNameSubstitutor')
12:21:49.368 [main] INFO  org.testcontainers.dockerclient.DockerClientProviderStrategy - Loaded org.testcontainers.dockerclient.NpipeSocketClientProviderStrategy from ~/.testcontainers.properties, will try it first
12:21:49.854 [main] INFO  org.testcontainers.dockerclient.DockerClientProviderStrategy - Found Docker environment with local Npipe socket (npipe:////./pipe/docker_engine)
12:21:49.854 [main] INFO  org.testcontainers.DockerClientFactory - Docker host IP address is localhost
12:21:49.888 [main] INFO  org.testcontainers.DockerClientFactory - Connected to docker:
  Server Version: 20.10.14
  API Version: 1.41
  Operating System: Docker Desktop
  Total Memory: 51188 MB
12:21:55.569 [docker-java-stream--301690517] ERROR com.github.dockerjava.api.async.ResultCallbackTemplate - Error during callback
com.github.dockerjava.api.exception.InternalServerErrorException: Status 500: {"message":"Head \"https://registry-1.docker.io/v2/testcontainers/ryuk/manifests/0.3.3\": unauthorized: incorrect username or password"}

Image pulled manually via docker pull testcontainers/ryuk:0.3.3 works just fine. It's just a workaround though.

Thanks for bumping this issue. I was recently running into this error on Docker Desktop as well.
Can you please try out to logout and then perform the Docker login on the command line instead (docker login)?

I tested on 4.3.2 on macOS(12.1 (21C52)) and it's looking like a charm, but for Windows I encountered the same issue as @awojcik64

I don't know how I did that, but now containers are pulled correctly even if I am logged out of everything.
I was only juggling logging in using GUI and command line. (tested on Docker Desktop 4.7.0)

@awojcik64 How I perceive the error so far with regards to pulling public images with Testcontainers:

Docker Desktop did the login through the GUI: not working ❌
Docker login through CLI and username/pw (docker login): working ✅
Docker logged out: working ✅

For further triangulation, may I ask you @SiwyDym if you are sure that on macOS, you recently did a login through the Docker Desktop GUI and it still worked?

I can confirmed that issue occurs when I login by Docker Desktop GUI :|
Docker Desktop: 4.7.0 (77141)
OS: 12.1 (21C52)

@SiwyDym Can you confirm that performing the login through CLI mitigates the issue?

I cannot confirm that, the only working mitigation for me is manually pulling the image testcontainers/ryuk:0.3.3

@SiwyDym I see. Can you instead confirm that it works if your Docker daemon is unautheticated? (after docker logout)

Thanks a lot for sharing your experiences @jodastephen, this really helps in triaging this issue. I am currently in the process of debugging and triaging as well.
What seems to help is to, in addition to performing a docker logout, also remove the Docker config file:
`rm USER_HOME\.docker\config.json

I currently suspect this is caused by changes to the Docker Hub registry, but the permutation in which this is currently failing are unfortunately not 100% clear to me yet.

I am also having this exact issue. I am setting up a new development environment on a M1 Max MacBook Pro, with latest Docker (darwin/arm64). I have tried all the suggestions in the previous comments; i.e. logging out of docker, logging in via CLI (with and without registry.hub.docker.com), reset Docker to factory defaults, delete .docker/config.json.
The only thing that ever works for me is to manually pull the image testcontainers/ryuk:0.3.3
Docker Desktop 4.7.0 (77141)
Docker version 20.10.14, build a224086
macOS 12.3.1 (21E258)

Would someone affected by this issue like to try out this PR from Jitpack?
https://github.com/testcontainers/testcontainers-java/pull/5263
https://www.testcontainers.org/jitpack_dependencies/

This appears to work for me, thanks.

That worked for me on M1 Macbook Pro. It added an {"auths":{"index.docker.io":{}}} entry in my ~/.docker/config.json file. Thereafter the workaround persists across, restarts, etc.

i tried resetting docker windows setup to defaults, login / logout with docker (and specific registry urls) nothing helped.
The only thing that helped me was pulling every image needed manually.
But there was something odd: The first image needed was ryuk (one project 0.3.3 another 0.3.1) but after pulling only this and run the project again i get too many failed login requets for the other images to pull (for ryuk i always got unauthorized).

@dermoritz So you say, login with exactly docker login index.docker.io did not help?

@kiview I can also confirm the command fixes this for me.

You are now getting this error from registry:
toomanyrequests: too many failed login attempts for username or IP address"

That's why the workaround did not work. You have to wait until the block times out on Docker Hub.

You are now getting this error from registry:
toomanyrequests: too many failed login attempts for username or IP address"

That's why the workaround did not work. You have to wait until the block times out on Docker Hub.

Still waiting for several (5?) days, so...

Which is the workaround you have currently applied? Also, maybe the block does not time out if you are connecting from the same IP address.

@kiview I just tried the command line fix on macOS 12.3.1 (M1 Pro) running Docker Desktop 4.8.1 and confirmed it works here too.
Thank you!

The fix that worked for me on an M1 Mac was:
remove credsStore:desktop from ~/.docker/config.json, run docker login in terminal

I'm running into this problem in our CI environment.
We're running testcontainers via maven and this is the command line we use to run maven:
mvn verify -DskipUnitTests -Dregistry.url=${CI_REGISTRY}  -Dregistry.username=${CI_REGISTRY_USER}  -Dregistry.password=${CI_REGISTRY_PASSWORD}
For some reason, this seems to break testcontainers in our tests:
119153 [main] WARN  🐳 [testcontainers/ryuk:0.3.4]  - Retrying pull for image: testcontainers/ryuk:0.3.4 (2s remaining)
124727 [docker-java-stream--909396508] ERROR com.github.dockerjava.api.async.ResultCallbackTemplate  - Error during callback
com.github.dockerjava.api.exception.InternalServerErrorException: Status 500: {"message":"Get https://registry-1.docker.io/v2/testcontainers/ryuk/manifests/0.3.4: unauthorized: incorrect username or password"}
	at org.testcontainers.shaded.com.github.dockerjava.core.DefaultInvocationBuilder.execute(DefaultInvocationBuilder.java:247)
	at org.testcontainers.shaded.com.github.dockerjava.core.DefaultInvocationBuilder.lambda$executeAndStream$1(DefaultInvocationBuilder.java:269)
	at java.base/java.lang.Thread.run(Thread.java:833)

Since the container that we're running the Maven tests from within doesn't have the docker executable, I can't run docker login in our CI scripts.
Does anybody have anything I could try out to get this working?

@StFS Can you please ask about this in Slack, or open a dedicated discussion/issue?
This issue is about Docker Desktop.

@StFS Can you please ask about this in Slack, or open a dedicated discussion/issue? This issue is about Docker Desktop.

Good point. Sorry about that.

Had the same issue. Logout, and than login via console. Helped for me.

docker logout
docker login --username

I'm running into this problem in our CI environment.
We're running testcontainers via maven and this is the command line we use to run maven: mvn verify -DskipUnitTests -Dregistry.url=${CI_REGISTRY} -Dregistry.username=${CI_REGISTRY_USER} -Dregistry.password=${CI_REGISTRY_PASSWORD}
We need to use a private registry to get the containers that are being tested, but for some reason, this seems to break testcontainers ability to fetch public containers:
119153 [main] WARN  🐳 [testcontainers/ryuk:0.3.4]  - Retrying pull for image: testcontainers/ryuk:0.3.4 (2s remaining)
124727 [docker-java-stream--909396508] ERROR com.github.dockerjava.api.async.ResultCallbackTemplate  - Error during callback
com.github.dockerjava.api.exception.InternalServerErrorException: Status 500: {"message":"Get https://registry-1.docker.io/v2/testcontainers/ryuk/manifests/0.3.4: unauthorized: incorrect username or password"}
	at org.testcontainers.shaded.com.github.dockerjava.core.DefaultInvocationBuilder.execute(DefaultInvocationBuilder.java:247)
	at org.testcontainers.shaded.com.github.dockerjava.core.DefaultInvocationBuilder.lambda$executeAndStream$1(DefaultInvocationBuilder.java:269)
	at java.base/java.lang.Thread.run(Thread.java:833)

Since the container that we're running the Maven tests from within doesn't have the docker executable, I can't run docker login in our CI scripts.
Does anybody have anything I could try out to get this working?

This is the exact situation I have as well. Has anyone found a workaround or solution?

I am also encountering this issue while running Github Actions locally using Act. I have tried all the suggestions above, and none seem to work. When running my tests in CI (Github Actions), it works fine though.

I have exactly the same issue with Test Container 1.16.2. This issue only occurs if the Docker image has to be pulled. As long as I have a local image everything is fine.

sorry this isnt a valid solution, somebody must run this in a pipeline for example, the image must be pulled

@angelomelonas We never tested TC with Act, but essentially, this will lead to Docker-in-Docker scenario, so the issue you are facing here, is very likely independent from the issue that is discussed here. I would suggest opening a dedicated issue or discussion for using TC with Act and Docker authentication.

Ah that does makes sense. I will create a separate issue for that. Thanks @kiview!


Post-Scroll Re-Assessment of Elements Not Updating

Meta -
Watir Version: 7.2.2
Selenium Version: selenium-webdriver 4.8.0
Browser Version: Chrome 109.0.5414.119
Browser Driver Version: 109.0.5414.74%
OS Version: MacOS Ventura 13.2
Expected Behavior -
I'm expecting the count of elements to augment upon scrolling down for more comments (the elements I'm looking for).
Actual Behavior -
It scrolls properly, displays 'jump' (see code), assesses element count as equal to the previous count, and therefore exits that loop.
Steps to reproduce -
s = 'https://www.youtube.com/watch?v=u9DPBxiZZfo&ab_channel=America%27sTestKitchen'
Watir.default_timeout = 10
browser = Watir::Browser.new :chrome
browser.goto s
sleep 6
comments = []
mycount = 0
while mycount < browser.div(id: "contents").divs(id: "comment-content").size do
  thiscount = browser.div(id: "contents").divs(id: "comment-content").size
  puts "#{ mycount } : #{ thiscount }"
  break if mycount == thiscount
  mycount = thiscount
  browser.driver.execute_script("window.scrollBy(0,500)")
  puts 'jump'
  sleep 1
  puts "#{ mycount } vs #{ browser.div(id: "contents").divs(id: "comment-content").size }"
  puts browser.div(id: "contents").divs(id: "comment-content").last.text
  sleep 1
end
sleep 2
browser.div(id: "contents").divs(id: "comment-content").each do |comment|
  comments << comment.text
end
sleep 1
p comments
browser.close

That answer posted doesn't address the problem.  The issue is definitely with Watir.  Read what I posted there.  It's not a solution.  The issue still persists.


Explore switch to Dirichlet-Multinomial likelihood for comps

Dirichlet-Multinomial (D-M) likelihood proposed by Thorson et al. (just added to https://github.com/cgrandin/hake-assessment/blob/master/doc/all.bib as "ThorsonEtAl2017Dirichlet" and uploaded to Google Drive "papers" folder) is worth considering over status-quo multinomial likelihood.
The key advantage is that data-weighting is done automatically via an estimated parameter, rather than via manual iteration as in the McAllister-Ianelli approach. Having automated data-weighting means that as you explore sensitivities and retrospectives, each model is automatically tuned rather than potentially out of balance. This is probably most important when exploring alternative setups for time-varying selectivity.
The D-M option is available in SS version 3.30, but the r4ss output needs improvement to apply it. I'm working this morning on addressing this this morning (see https://github.com/r4ss/r4ss/issues/129)  (though I need to take a break from now until about 10am).

Quick comparison of results of 12-million MCMC chain indicates nothing strange in comparison with MLE values.
The median weights given to the fishery and survey age comps are 0.36 and 0.99, respectively, which is closer to the 1:3 ratio from the McAllister-Ianelli weights applied to this model (though still higher in overall scale).


Backdoor accuracy can't reach 100%

I have some problems when i am trying to reproduce the experiment results in this paper.
I pull the source code and run it with params.yaml after setting is_poison=True and switching some report loss options to True. After finishing training i found that the test accuracy on backdoor tasks cant reach 100% as it is said in the paper, it usually remains to be around 70% sometimes lower. I am wondering if there is something need to be set particularly or something wrong with my params.yaml. Could someone plz help me?
here are my params.yaml and the final results.

type
image

lr
0.1

momentum
0.9

decay
0.0005

batch_size
64

no_models
10

epochs
10100

retrain_no_times
2

number_of_total_participants
100

sampling_dirichlet
True

dirichlet_alpha
0.9

eta
1

save_model
True

save_on_epochs
[10, 100, 500, 1000, 2000, 5000]

resumed_model
False

environment_name
ppdl_experiment_Jul.13_13.34

report_train_loss
True

report_test_loss
True

report_poison_loss
True

track_distance
False

track_clusters
False

modify_poison
False

poison_type
wall

poison_images_test
[330, 568, 3934, 12336, 30560]

poison_images
[30696, 33105, 33615, 33907, 36848, 40713, 41706]

poison_image_id
2775

poison_image_id_2
1605

poison_label_swap
2

size_of_secret_dataset
200

poisoning_per_batch
1

poison_test_repeat
1000

is_poison
True

baseline
False

random_compromise
False

noise_level
0.01

poison_epochs
[10000]

retrain_poison
15

scale_weights
100

poison_lr
0.05

poison_momentum
0.9

poison_decay
0.005

poison_step_lr
True

clamp_value
1.0

alpha_loss
1.0

number_of_adversaries
1

poisoned_number
2

results_json
False

s_norm
1000000

diff_privacy
False

fake_participants_load
False

fake_participants_file
data/reddit/updates_cifar.pt.tar

fake_participants_save
False

current_time
Jul.13_14.56.08

adversary_list
[0]

Hey, I don't really support the code anymore, so it might not work well. Maybe try to test memorization first, i.e. mix test and train together.

Hello, I have encountered a similar problem as you. When I was training with param.yaml, I did not see the convergence as shown in the picture, but kept a straight line (about 10%). But I found out that your training rounds reached 10k, which is too big! I am using a personal computer with limited computing resources. Is there any way to reduce the number of rounds to reach convergence?

I think there may be some problems with your parameter setting, such as the obvious: "adversary_list" is 0. In the code for adversaries, if "NUMBER_of_telecom" is 1 or "random_copromise," there may be some problems in the backdoor training. However, the picture you provided is indeed converging, and the accuracy rate of the back door increases sharply after 10k, which makes me feel very confused. If you still have an impression on this reproduction, please help me review it, thank you very much!


Help needed.

Hi. Could you make an example of using a button? Something like if up pressed do something. That would help me understand how this works

The "avrshock2_button_t" values are defined here: https://github.com/dhustkoder/avrshock2/blob/master/src/avrshock2.h#L12
In this example it lists the state of the buttons:
https://github.com/dhustkoder/avrshock2/blob/master/src/example.c#L78
You could do something like this using the "buttons" variable:
        if (buttons&AVRSHOCK2_BUTTON_CROSS) {
                // do stuff if button is pressed
        } else {
                // do stuff if button is unpressed
        }

Thank you :D This is gonna really help me

Maybe I need a little more help. How to define ports to make it work?

@NiHuShu You can define ports in the makefile: https://github.com/dhustkoder/avrshock2/blob/master/makefile#L17
Use your micrcontroller's  datasheet  to find the ports names

I've tried but I think that I don't have enough knowledge to make this project my self. Could you help me to merge your project with https://github.com/bootsector/XBOXPadMicro so that I could make PS2 controller to XBOX adapter?

@NiHuShu sorry I have been working and I coulnd't reply to your issue. I'll close this but if you're still interested in this project we can talk about it.


Custom validation

I would like to add custom validation to some fields of the document (and optionally return error), but I don't know what methods should I override (Resource.create_object or ResourceView.post?). It's not just validation, I'm looking for a way how to run custom code in POST requests. Is this even possible? Thanks!

We support validation schemas using cleancat (https://github.com/elasticsales/cleancat). If this isn't flexible enough you can also override save_object in the resource and  post in the view.  Check out the latest cleanup branch: https://github.com/elasticsales/flask-mongorest/pull/62


Update osx_image to xcode12

This solves #161. Additionally, it fixes some of the warnings thrown by Travis CI during the Build config validation.

Hi @dnerini. This is cool.
I was thinking that it could be more complex than just upgrade Xcode.

Thanks Carlos for having a look!


chore(ci): exprimental lualint integration

We have developed a tool called Lualint, which checks for basic formatting errors in user-submitted code, thereby reducing the workload of reviewers.

It can not work as expected. See: https://github.com/Kong/lua-resty-events/actions/runs/4954194392/jobs/8862463745?pr=28


fixed TestErrorInHTTPRequest

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[x] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 44.05%. Comparing base (f9cff99) to head (50eced8).
Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1348      +/-   ##
==========================================
+ Coverage   42.34%   44.05%   +1.70%
==========================================
  Files         331      267      -64
  Lines       34441    24895    -9546
==========================================
- Hits        14584    10967    -3617
+ Misses      19384    13480    -5904
+ Partials      473      448      -25

Flag
Coverage Δ

go-1.20.x
44.05% <ø> (-0.64%)
:arrow_down:

go-1.21.x
?

go-1.22.x
?

macOS
44.05% <ø> (+2.19%)
:arrow_up:

ubuntu
?

unit
44.05% <ø> (+1.70%)
:arrow_up:

windows
?

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[WIP] Use nested style for class and module children in API

Purpose or Intent
To avoid having to do ManageIQ::API for every dependency within the
API module, using a nested style allows us to drop the
namespace. Constant lookup should start in the API module and move
upwards if it can't find what it's looking for.
@miq-bot add-label api, refactoring, wip
@miq-bot assign @abellotti
@abellotti once https://github.com/ManageIQ/manageiq/pull/10496 is merged I'll encorporate that into this PR too. Marking as WIP for now and skipping CI to avoid wasting a run until this is ready.
/cc @jrafanie

@imtayadeway ApiUserTokenService PR got merged.  Thanks.

@miq-bot rm-label wip

Very Nice !!  :musical_score:

🎉 👍


Issue with CdkWorkshop typescript

:bug: Bug Report
What is the problem?

Workshop url: https://cdkworkshop.com/20-typescript/40-hit-counter/200-handler.html
The Typescript CdkWorkshop uses the following import statement:
// file: lambda/hitcounter.js
import { DynamoDB, Lambda } from 'aws-sdk';

However, this produces the following error (from the deployed lambda): See Verbose log
I googled the error and found the following stackoverflow page:
https://stackoverflow.com/questions/56859727/why-do-i-get-a-usercodesyntaxerror-when-i-have-no-syntax-error-in-my-code
The guide uses an ES6 style import, which is not supported by the aws node.js 10.x runtime. I changed it to a standard module require syntax and the function worked as expected.
Reproduction Steps

cdk deploy with the lambda/hitcounter.js file and look in the cloudwatch logs
Verbose Log

2020-02-16T12:47:27.490Z	undefined	ERROR	Uncaught Exception
{"errorType":"Runtime.UserCodeSyntaxError","errorMessage":"SyntaxError: Unexpected token {","stack":["Runtime.UserCodeSyntaxError: SyntaxError: Unexpected token {","
at _loadUserApp (/var/runtime/UserFunction.js:98:13)","
at Object.module.exports.load (/var/runtime/UserFunction.js:140:17)","
at Object.<anonymous> (/var/runtime/index.js:45:30)","
at Module._compile (internal/modules/cjs/loader.js:778:30)","
at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)","
at Module.load (internal/modules/cjs/loader.js:653:32)","
at tryModuleLoad (internal/modules/cjs/loader.js:593:12)","
at Function.Module._load (internal/modules/cjs/loader.js:585:3)","
at Function.Module.runMain (internal/modules/cjs/loader.js:831:12)","
at startup (internal/bootstrap/node.js:283:19)"]}

Environment

CDK CLI Version:  1.23.0 (build 01f326e)
Section:  https://cdkworkshop.com/20-typescript/40-hit-counter/200-handler.html
Browser:  Firefox
Language:  TypeScript

Other information

For those of use (like me), who never used node before.
"changed it to a standard module require syntax" means changing
import { DynamoDB, Lambda } from 'aws-sdk';

to
const { DynamoDB, Lambda } = require('aws-sdk');

This did the tick for me

@TheKangaroo I just talked to one of our JS people and unfortunately it looks like Lambda does not support ES6 syntax at all.
Good catch!
If you would like to put in a PR to fix this, feel free, otherwise I will take care of it soon.
😸

@NGL321 I was just about to start when I noticed @ravanapel already put in a PR  🎉


no running container in docker ps after hyper started the pod

{
    "name": "myweb",
    "containers" : [{
        "image": "httpd",
        "ports": [{
            "containerPort": 80,
            "hostPort": 8080
        }]
    }],
    "resource": {
        "vcpu": 1,
        "memory": 512
    }
}

root@debian:~# hyper run -p pod-1
POD id is pod-WXBmlQWfEf
Time to run a POD is 4577 ms

But no running container found in `docker ps`.

Hello Henry,
Thanks for trying hyper.  You can use hyper list container command to check the running containers if you want.
More details regarding with list sub-command are following:
# hyper list --help
Usage:
  hyper list [pod|container]

list all pods or container information

Help Options:
  -h, --help  Show this help message

@carmark thanks, i have got this container via hyper list container. One more problem: the running container did not run as a web server, expected in pod file. I have gone through your document, it seems no kind of tips for the troubleshooting. Besides, i have used "hyper exec" and tried to execute some command in my container, but the outuput was nothing. Could you please help me on this problem?

@henrysher Hi, try to run hyperd manually with  parameter -v=3, this will give you full debug message.

@gao-feng i got the error message, but no idea as the document said, if no tty configured, by default turned on.
[HYPER INFO  0531 20:54:^@55 26892 qemu_process.go] [:53] [console] setup pty device /dev/null for exec
[HYPER INFO  0531 20:54:^@55 26892 qemu_process.go] [:53] [console] ioctl pty device for execcmd failed: Inappropriate ioctl for device
[HYPER INFO  0531 20:54:^@55 26892 qemu_process.go] [:53] [console] setup tty failed
[HYPER INFO  0531 20:54:^@55 26892 qemu_process.go] [:53] [console] hyper send type 10, len 0
[HYPER INFO  0531 20:54:^@55 26892 qemu_process.go] [:53] [console] wait for container started failed
[HYPER INFO  0531 20:54:^@55 26892 qemu_process.go] [:53] [console] container 2e60d916bb16655746830d89e1503c7067ef13560883a6ba62a77f11c44cadc8 init exit code -1

@henrysher Thanks for your helpful information, this is a bug in hyperStart. you can clone hyperStart from https://github.com/hyperhq/hyperstart.git and recompile it, change the Initrd in /etc/hyper/config to the path of hyperStart/build/hyper-initrd.img.
NOTE: you should use '#' to comment the Cbfs in hyper config too, since the cbfs-qboot.rom doesn't contain new hyper-initrd.img. if you want to use cbfs-qboot.rom, please use cbfstool to generate new cbfs-qboot.rom
#dd if=/dev/zero of=boot.bin bs=4096 count=1
#cbfstool cbfs-qboot.rom create -s 4096k -B boot.bin -m x86  0x1000
#cbfstool cbfs-qboot.rom add -f /prefix/hyperStart/build/kernel -n vmlinuz -t raw
#cbfstool cbfs-qboot.rom add -f /prefix/hyperStart/build/hyper-initrd.img -n initrd -t raw
#echo "console=ttyS0,panic=1" > cmdline
#cbfstool cbfs-qboot.rom add -f cmdline -n cmdline -t raw

@henrysher the port-map and inspect command are on the way, will come soon. Thanks!

@henrysher the install script updated too, you can reinstall it by curl -sSL https://hyper.sh/install | bash now. thanks!

looks no further problems, close this issue


Interpret http-get feedback as utf-8 (issue #495)

I tried the proposal from DokuroKM in issue #495, and this helped.
I think the root cause is, that google changed the format of the information which the input stream helper uses to check the latest available widevine version.
On my system after the change of the local file ~/.kodi/addons/script.module.inputstreamhelper/lib/inputstreamhelper/utils.py the widevine update dialog was shown correctly and I was able to update. After the update everything worked as intented.
Can please someone review that and check that it has no side effects?

From my testing this change should not affect other calls of http_get and also seems to work well on Python 3. If the tests finish fine, I'll merge this tomorrow and probably do a release right after.

This will fix #495


Issue 130 - fixed hyperlink format

Signed-off-by: Marhababanu Chariwala<EMAIL_ADDRESS>fixes https://github.com/open-horizon/devops/issues/130

@johnwalicki please review


Add details on how to contribute

Readers may want to request new recipies, or contribute fixes. Add details to navigation / README to cover this

Could do this if it still needs doing

Awesome, thank you :) The place for this is https://geek-cookbook.funkypenguin.co.nz/support/ I think (support.md) - what we want is to make it obvious that PRs for feedback, new recipies etc are welcome, and than further interaction is available in the discord channels

https://github.com/funkypenguin/geek-cookbook/blob/contribute/contribute.md
https://github.com/funkypenguin/geek-cookbook/tree/contribute
Done
Closing ticket


Don't allow explicit drive selection on the Etcher Pro

There will still be a drive step, but instead of allowing a drive selection modal, we will directly show the summary of selected available drives. So basically, if the user puts a drive inside one of the ports, then the drive is selected by default.

This "mode" should be enabled by an environment variable specific for this feature


Missing logs when using systemd source

what
We have fluentd-kubernetes-sumologic to our nodes and are providing an additional source to pull in from a custom systemd unit. We have found that some of the logs from the service show up in Sumologic, but not all. Ive included the config below. How can we go about troubleshooting where  the logs are being dropped off?
environment
fluentd-kubernetes-sumologic release: v1.6
Kubernetes: 1.8.x
my.service: performs a docker run with no custom logging
Config:
      <source>
        @type systemd
        path /mnt/log/journal
        filters [{"_SYSTEMD_UNIT": "my.service"}]
        <storage>
          @type local
          persistent true
          path /mnt/pos/my.log.pos
        </storage>
        tag my
      </source>

      <filter my.**>
        @type kubernetes_sumologic
        source_category system/my
        source_name my
        source_category_prefix "#{ENV['SOURCE_CATEGORY_PREFIX']}"
        exclude_host_regex "#{ENV['EXCLUDE_HOST_REGEX']}"
        exclude_priority_regex "#{ENV['EXCLUDE_PRIORITY_REGEX']}"
        exclude_unit_regex "#{ENV['EXCLUDE_UNIT_REGEX']}"
      </filter>

Update
We traced the issue to fluentd running out of memory and constantly restarting. This was not immediately apparent as the pod was not exiting, but if you look in the logs you would see an event like:
2018-03-20 15:58:45 +0000 [info]: Worker 0 finished unexpectedly with signal SIGKILL

In the host logs you would see an event like:
Mar 20 15:58:44 ip-172-20-60-200 kernel: Memory cgroup out of memory: Kill process 22663 (ruby2.3) score 1971 or sacrifice child
Mar 20 15:58:44 ip-172-20-60-200 kernel: Killed process 22663 (ruby2.3) total-vm:5557248kB, anon-rss:350160kB, file-rss:50628kB

With help from @frankreno  we traced the issue to the flush interval being defaulted to 30s when installing via the chart. This is because the chart does not override the value, its commented out: https://github.com/kubernetes/charts/blob/c0e593513184490f895b3a82375934545677c309/stable/sumologic-fluentd/values.yaml#L44. The default of 30s is then inherited from the Dockerfile: https://github.com/SumoLogic/fluentd-kubernetes-sumologic/blob/6ba540de36b28425bbd1cc41ba5aa0412d023ab2/Dockerfile#L26
On hosts with any volume of traffic this buffer can easily require more than the default 256MB than the chart limits fluentd to: https://github.com/kubernetes/charts/blob/c0e593513184490f895b3a82375934545677c309/stable/sumologic-fluentd/values.yaml#L154
We reduced the flush interval to 5s and increased the memory limit to 512mb and collection has been stable.
Possible Improvements

Default the flush interval to a lower value within the Dockerfile and chart
Allow the container to exit in the event of OOM. The current approach is nice when running fluentd outside kubernetes. However when run in k8s, its better to allow it exit and be restarted. This type of activity is far more likely to be noticed


Update names of parameter for SVM

Renamed next params:
k->scale
b->shift
coefficients->coeffs

/intelci: run

@rlnx , @michael-smirnov please, review this changes

I will merge with master, after review or approve.

/intelci: run

/intelci: run


Prometheus pull consumer metrics are labeled as gauges when they should be labeled as counters.

Environment

Telemetry Streaming Version: 1.12 to 1.24
BIG-IP Version: 13.1.x to 15.1.x

Summary
Prometheus pull consumer metrics are labeled as gauges when they should be labeled as counters.
Steps To Reproduce
Steps to reproduce the behavior:

Submit the following declaration:

{
    "class": "Telemetry",
    "My_Poller": {
        "class": "Telemetry_System_Poller",
        "interval": 60
    },
    "Custom_Endpoints": {
    "class": "Telemetry_Endpoints",
        "items": {
             "detailedCPU": {
                 "name": "detailedCPU",
                 "path": "/mgmt/tm/sys/host-info"
             },
             "systemHostname": {
                 "name": "system",
                 "path": "/mgmt/tm/sys/global-settings"
            }
        }
    },
    "Custom_System_Poller1": {
        "class": "Telemetry_System_Poller",
        "interval": 60,
        "endpointList": "Custom_Endpoints"
    },
    "My_Pull_Consumer": {
        "class": "Telemetry_Pull_Consumer",
       "type": "Prometheus",
        "systemPoller": ["My_Poller", "Custom_System_Poller1"]
    },
    "My_System": {
        "class": "Telemetry_System",
        "enable": "true",
        "systemPoller": ["My_Poller", "Custom_System_Poller1"]
    }
   }

Observe the following when the PULL consumer endpoint is queried:
GET /mgmt/shared/telemetry/pullconsumer/My_Pull_Consumer

# HELP f5_numberReqs numberReqs
# TYPE f5_numberReqs gauge
f5_numberReqs{httpProfiles="/Common/http"} 5402028

# HELP f5_serverside_bitsOut serverside.bitsOut
# TYPE f5_serverside_bitsOut gauge
f5_serverside_bitsOut{pools="/Common/http_pool"} 8470816
f5_serverside_bitsOut{pools="/Common/http_pool",members="/Common/x.x.x.x:80"} 8470816

Expected Behavior
Any metric that is a number that only increases (unless there is a counter reset) should be labeled as a counter. Counter types
Actual Behavior
It seems all metrics provided by the Prometheus pull consumer are labeled as gauges. Some of the metric should be labeled as gauges however there are a large number of metrics that should be labeled as counters.
Prometheus interprets the data differently for counters. There are certain functions that should not be used on gauges such as rate().
The customer I am working with is seeing massive artificial graphing anomalys when their dashboards running rate() function against an F5 metric (currently gauge) experiences a counter reset. The customer has opened TS issue for this in the past here and here.
We believe this is the first step to fixing the massive graphing anomalys because the underlying metric should be classified as a counter rather than gauge.

Please prioritize this issue in next release, it is very impactful to my use case.

We are looking into this. Stay tuned.

Upon further testing it appears simply changing the label of these metrics to counter wont resolve our underlying issue.
It is more accurate to label counter type metrics as counters so this git issue is still valid; however this does not change the outcome when rate() function is applied.
1.) Example when a metric (f5_2xx) labeled as counter experiences a dip:

2.) The rate function still shows a massive comsmetic issue:

We do not yet understand why TS Prometheus PULL consumer is intermittently showing a "counter" metric as decreasing (but not zero) and then returning to normal on the next scrape interval (see screenshot number 1).
We have a support SR open for this. Any tips on debugging TS would be appreciated. Is there a way to validate what data is returned on a PULL consumer request to TS?

@megamattzilla Do you see the same data anomalies when forwarding data through an Open Telemetry Collector?

https://clouddocs.f5.com/products/extensions/f5-telemetry-streaming/latest/setting-up-consumer.html#opentelemetry-exporter-experimental
https://opentelemetry.io/docs/collector/getting-started/

If you experience this issue try implementing the workaround described in https://support.f5.com/csp/article/K21811305


"Debug Test" and "Run Test" buttons no longer appear

Environment data

VS Code version: 1.43.2
Extension version (available under the Extensions sidebar): 2020.3.69010
OS and version: macOS 10.15.4
Python version (& distribution if applicable, e.g. Anaconda): 3.7.6
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): venv
Relevant/affected Python packages and their versions: NA
Relevant/affected Python-related VS Code extensions and their versions: NA
Jedi or Language Server? (i.e. what is "python.jediEnabled" set to; more info #3977): Jedi
Value of the python.languageServer setting: Microsoft

Expected behaviour
The "Run Test" and "Debug Test" buttons which usually appear inline with code no longer appear since updating to the latest version. Discovering tests works as expected, and I can see the tests in the side bar and run and debug from there. The buttons just don't appear inline. If I disable Jedi, reload the window, then re-enable it without reloading the window, then the buttons appear and work as expected. But disappear as soon as the window is reloaded.
Actual behaviour
See above.
Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

Applies with all Python tests (run using pytest). Config options:

    "python.testing.pytestArgs": [
        "olmbuild"
    ],
    "python.testing.unittestEnabled": false,
    "python.testing.nosetestsEnabled": false,
    "python.testing.pytestEnabled": true,

Logs
Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)
> ~/Microsoft/client-cocoa/venv/bin/python3 ~/.vscode/extensions/ms-python.python-2020.3.69010/pythonFiles/testing_tools/run_adapter.py discover pytest -- --rootdir ~/Microsoft/client-cocoa/scripts -s --cache-clear olmbuild
cwd: ~/Microsoft/client-cocoa/scripts
> ~/Microsoft/client-cocoa/venv/bin/python3 ~/.vscode/extensions/ms-python.python-2020.3.69010/pythonFiles/testing_tools/run_adapter.py discover pytest -- --rootdir ~/Microsoft/client-cocoa/scripts -s --cache-clear olmbuild
cwd: ~/Microsoft/client-cocoa/scripts

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help; turn on source maps to make any tracebacks be useful by running Enable source map support for extension debugging)
[Extension Host] Info Python Extension: 2020-03-30 14:40:15: getActivatedEnvironmentVariables, Class name = C, completed in 1ms, has a truthy return value, Arg 1: <Uri:/Users/dalemy/Microsoft/client-cocoa/scripts>, Arg 2: undefined, Arg 3: false
console.ts:137 [Extension Host] Info Python Extension: 2020-03-30 14:40:15: > ~/Microsoft/client-cocoa/venv/bin/python3 ~/.vscode/extensions/ms-python.python-2020.3.69010/pythonFiles/testing_tools/run_adapter.py discover pytest -- --rootdir ~/Microsoft/client-cocoa/scripts -s --cache-clear olmbuild
console.ts:137 [Extension Host] Info Python Extension: 2020-03-30 14:40:15: cwd: ~/Microsoft/client-cocoa/scripts

@dalemyers, thanks for letting us know about this.  When the Python extension's code lenses stop working it is usually due to another extension, like the gitlens extension.  So the first thing to check is if you have any other extensions installed and if any of them provide code lenses.  Please check on that, let us know, and we can proceed from there.  Thanks!

I don't have any that provide lenses as far as I'm aware. My extensions are:

Azure Account 0.8.8
Azure Functions 0.21.0
Docker 1.0.0
Prettify JSON 0.0.3
Property List 0.0.2
Python 2020.3.71113 (Looks like this has been updated - I'll run another check to confirm if the issue is still there)
Syntax Xcode Project Data 0.1.3

After the latest update the issue appears to be resolved.

Thanks for letting us know, @dalemyers.  If you see any further problems with the please re-open this issue.  Thanks!


Watchdog 1.0.0 drops dependency on pathtools

We use pathtools for pattern matching in a couple of places, and should be easy to replace with an implementation based on pathlib, same as watchdog now does. Pathtools is essentially unmaintained and it would be good to get rid of it.
At the very least we should add an explicit dependency on pathtools in the meta.yaml and setup.cfg files for now until all traces of this package have been removed from our codebase.
For inspiration have a look at https://github.com/gorakhargosh/watchdog/issues/556 and links therein.

Closed in #155

Closed in #155


WPS: support multiple languages

Add a language parameter to the owslib.wps.WebProcessingService class.
This parameter is used in GetCapabilities, DescribeProcess and Execute requests to include the language attribute in the query string and the xml POST requests.

This PR is related to multiple language support in PyWPS:
https://github.com/geopython/pywps/pull/510

test failures not related to this PR.

See wps 1.0.0 spec, table 6.

@davidcaron Thanks again :)


Setup scripts do not guard against folders with whitespace in

E.g. if PROJECT_DIR contains whitespace then the git-crypt unlock command will not work.
https://github.com/nestauk/ds-cookiecutter/blob/6baa97bdd602b756e818b69a27a6919dd64c4df7/{{ cookiecutter.repo_name }}/bin/install_metaflow_aws.sh#L4

Closed by #71


Display link targets of nodes in a regular flow, for Link Call nodes inside a subflow

fixes #3248
Types of changes

[x] New feature (non-breaking change which adds functionality)

Proposed changes

Make link call display targets both inside its own subflow and on regular flows
Ensure link in & link out nodes on a subflow ONLY show targets inside the same subflow
Ensure link targets inside a subflow are not displayed for link nodes on regular flows

Checklist

[x] I have read the contribution guidelines
[x] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[x] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage remained the same at 68.746% when pulling b9919b0a9c78817415420041891f596525b5e5e2 on Steve-Mcl:link-list-in-subflow into a7932da2072cbe3af62b52a9f98b1090c66fba82 on node-red:dev.


Submit Draft Inventory RFC

Signed-off-by: N8Shrader<EMAIL_ADDRESS>This is a draft RFC for discussion purposes with the Grid Community.

Putting this draft on 'pause' as we continue to dig in and do further research.  This inventory rfc will be re-opened when ready.


build errors installing and running on ios

Starting out with vanilla app using latest react-cli (2.0)
react-native init Foo && cd Foo
yarn add react-native-lock
react-native link
react-native run-ios

error:
 The following build commands failed: 	Ld build/Build/Products/Debug-iphonesimulator/Foo.app/PlugIns/FooTests.xctest/FooTests normal x86_64 (1 failure)

@mypark Please provide the full log since that error provides no useful info

Closing this one, if the issue still persists please add more info. Also please check #123 comments


Slider does not go to first slide when changing route with react-router

SEE: https://github.com/akiran/react-slick/issues/85
I am changing the route using <Link> (provided by react-router), but when doing so it keeps the slider on the currentSlide (Even though the slider's data changes).
SEE: https://github.com/akiran/react-slick/issues/100

Navigate to item A page
Show a list of related items in the slider
Click on related item B
Navigate to item B page
Show a list of related items in the slider (They may or may not be the same items in step 2—The current page will reflect the previous list in step 2)

Perhaps there is a middle ground between all of these related issues to enable reseting currentSlide: 0?

Yep, I am facing the same issue here.

+1
is there no one still solve this problem?

+1
imo currentSlide should reset to 0 when the Slider component updates


Scatter Gather Operations

Hi, I read the wiki about scatter gather operations and how a mset is divided and then weaved to return a single result, but when I try to use it I get a org.apache.commons.lang.NotImplementedException.
Are there any plans to implement this soon? We could really use this feature as mset is essential for us.
Thanks!

any update on this one?
it looks like on the server side mget listed as supported, but driver has no implementation.

@maksimlikharev I anticipate having this ready in the next couple of weeks.


Rails 4.2

[x] Fixes #90 #98
[x] Simplifies default static serving (removed engine)
[x] uses appraisals and pointed this out in readme
[x] rails 4.2 working specs

/cc @Strech

Changes Unknown when pulling c3b6030812d7eb401b83fb84cba5010c0832f48e on rails-4.2 into ** on master**.

Changes Unknown when pulling c3b6030812d7eb401b83fb84cba5010c0832f48e on rails-4.2 into ** on master**.

Changes Unknown when pulling c3b6030812d7eb401b83fb84cba5010c0832f48e on rails-4.2 into ** on master**.

@razum2um :+1: :heart:

Changes Unknown when pulling c3b6030812d7eb401b83fb84cba5010c0832f48e on rails-4.2 into ** on master**.

Changes Unknown when pulling c3b6030812d7eb401b83fb84cba5010c0832f48e on rails-4.2 into ** on master**.

Changes Unknown when pulling c3b6030812d7eb401b83fb84cba5010c0832f48e on rails-4.2 into ** on master**.


Connected Chroma, chroma/sync-workspace fails

Installed in Docker and connected to local docker Chroma instance. Can see my Workspaces. Clicked on Workspace and then 'Sync Workspace Data'. This queues a job that failes.
Job response:
{
"canRetry": true,
"message": "Job failed with error",
"error": "Cannot read properties of null (reading 'title')",
"details": {}
}
Chroma 0.4.4

How was this data embedded? That seems to indicate that it cannot find a document title for the embedding in the metadata. Tools like Llama Index and Langchain should attach the appropriate metadata

The data was embedded using Chroma with a local embedder. There is no metadata with the embedding, just document and ids.

Can confirm that rebuilding the collection with the metadatas: [{"title":"title"}] does work, although it does surprise me that this breaks the import of data, as it could use id instead of title if title is not present?

Can confirm that rebuilding the collection with the metadatas: [{"title":"title"}] does work, although it does surprise me that this breaks the import of data, as it could use id instead of title if title is not present?

Absolutely, will mark this as a bug because that's a very specific thing to expect. We will eventually not enforce any specific metadata at all and the table will be customizable to which columns you would want to display at all. Also add/update existing metadata fields as well

moved to #40


Indexing zipped epubs

This would probably require some refactoring ?
The simplest way as I see it would be to enable unzip in EpubMetaDataParser.getMetaData to temporary directory
Though if someone would point writeToIndex to a large set of zipped files, that would be very suboptimal, so the better way would be to create a list of dirs that are unziped epubs and zipped epubs, pass it down and the act on each entry accordingly - if it's zipped: unzip, extract metadata. index, if it's unziped: extract metadata, index


incorrect Row ID in "Pagination Controlled" react example

Describe the bug
The row ID and index are not changing when switching pages while using manual pagination with react query data.
This bug means each row after the first page is receiving an incorrect key because

This could be related to #4812 and #4771
This is the example from the docs and below is the modified version to highlight the incorrect row IDs in the console.
Your minimal, reproducible example
https://codesandbox.io/s/throbbing-wildflower-byp0ht?file=/src/main.tsx
Steps to reproduce

open console
switch between pages
notice the row IDs are not changing

Expected behavior
The row IDs should change when switching pages.
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform

OS: Linux
Browser: Chrome
Version: 113

react-table version
v8.9.1
TypeScript version
No response
Additional context
No response
Terms & Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

I fixed this by using the getRowId function to provide a custom unique ID provided by the api response.
		getRowId: (row, relativeIndex, parent) => {
			// console.log(row);
			// In row object you have access to data.
			// You can choose parameter. In this example I used uniqueId
			return row.id;
			return parent ? [parent.id, row.uniqueId].join(".") : row.id;
		},

Your getRowId function has two returns, could be mistake or mis-typing here? Second return is not reachable !
For the issue related with
, I don't understand why do you need a unique id for all records for all pages even if the next pages are not visible in the DOM. The rows in the same page may need unique key but do not need for the records in the other pages. The key is mostly related with current render of the table component. I suppose, you don't need to produce unique ids. The row.id is sufficient.

Are you sure this wouldn't cause any problems?
I think the example should at least include a way to set a unique ID like I did to avoid potential problems.

I am almost sure, the row.id is sufficient for the key; it will not produce any problem.

@OleksandrHubarEK check out my previous comment, it has the solution.

@hichemfantar
I tried the above method, although the rowSelection state problem has been resolved, but when using table.getSelectedRowModel() it still gives wrong results. Have you encountered this situation?

@hoangtrung99 It's been a while since I've visited this.
Are you sure you're using the correct id?
Otherwise it could be a new bug.

@hichemfantar
When I select 2 rows on 2 different pages, the state is correct, but table.getSelectedRowModel() is wrong, I think this is an error.


Update input-method-editor-error.md

This issue was fixed in 9C and 10B. However, the requesting customer has asked that we update documentation to reflect that the issue was addressed in Windows 10+ and remains in Windows 8.1 and below.

@tyadon : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Fix note about .spec.ipFamilies

This fixes #40728

With reference to https://github.com/kubernetes/website/issues/40728, this LGTM.
Thanks for the contribution.
/lgtm

This PR uses the API reference docs as the base to change the Note about .spec.ipFamilies
https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.27/#servicespec-v1-core
I'm good with the change
/approve


Update badge components

Following bitstyles new designs. Update bistyles phoenix badge components.
https://bitcrowd.github.io/bitstyles/?path=/story/atoms-badge-overview--page

Done in https://github.com/bitcrowd/bitstyles_phoenix/pull/127


ADBDEV-4726-35 Check that scalarVal is not null

Check that scalarVal is not null
appendValue and appendKey dereference scalarVal without checking that it's not
null. We can't guarantee that scalarVal is always valid. This patch adds Insist
for release builds and Assert for debug builds

update PR description

update PR title and description according to all changes

@RekGRpth they already contain all the changes

appendElement was changed too... and fix title

@RekGRpth title seems fine

@RekGRpth changed commit body


Can this example be updated to use the new Azure.Identitiy library, and latest Azure.Storage libraries?

[Enter feedback here]

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

ID: d02de0bf-09d7-a7fc-1b4a-213426e96b2d
Version Independent ID: 36f6c8fe-6bc1-6643-4b66-2c38bcd2e5f0
Content: Acquire a token from Azure AD for authorizing requests from a client application - Azure Storage
Content Source: articles/storage/common/storage-auth-aad-app.md
Service: storage
Sub-service: common
GitHub Login: @tamram
Microsoft Alias: tamram

@EdwinStraver Thanks for the question! We are investigating and will update you shortly.

@EdwinStraver Apologies for the delay in responding here!! Your feedback has been shared with the content owner for further review.

@EdwinStraver - We're working on this and it will be done soon.

@EdwinStraver - The doc has been updated to use the latest libraries.
#please-close


[Feature Request] Augementation config file

It would be great to be able to define augmentation using some kind of config file. I imagine YAML would for well.  For example a config file could look like:
sequential:
    fliplr:
        arg: 0.5
    flipud:
        arg: 0.2
    sometimes:
        cropandpad:
            percent: (-0.05, 0.1)
            pad_mode: all
            pad_cval: (0, 255)
...

@aleju I actually did pretty much like this with my text generator. Here is my code (which is definitely very messy :P), hope this could help
def process_list_augment(input_list, cval, random_state, is_binary):
    """Convert dictionary augmentation configs into augmentation object"""
    current_pipeline = []
    for _, each in enumerate(input_list):
        function_name = next(iter(each))
        augment_param = each[function_name]
        if isinstance(augment_param, list):
            prob = 1
            for each_element in augment_param:
                if isinstance(each_element, dict) and 'prob' in each_element.keys():
                    prob = each_element['prob']
                    augment_param.remove(each_element)
                    break
            big_object = None
            if function_name == 'Sequential':
                for each_element in augment_param:
                    if isinstance(each_element, dict) and 'random_order' in each_element.keys():
                        random_order = each_element['random_order']
                        augment_param.remove(each_element)
                        big_object = iaa.Sequential(process_list_augment(
                            input_list=augment_param, cval=cval,
                            random_state=random_state, is_binary=is_binary), random_order=random_order, random_state=random_state)
            elif function_name == 'SomeOf':
                for each_element in augment_param:
                    if isinstance(each_element, dict) and 'n' in each_element.keys():
                        n = each_element['n']
                        augment_param.remove(each_element)
                        big_object = iaa.SomeOf(literal_eval(n), process_list_augment(
                            input_list=augment_param, cval=cval, random_state=random_state, is_binary=is_binary), random_state=random_state)
            elif function_name == 'OneOf':
                big_object = iaa.OneOf(process_list_augment(
                    input_list=augment_param, cval=cval, random_state=random_state, is_binary=is_binary), random_state=random_state)
            if prob:
                big_object = iaa.Sometimes(
                    prob, big_object, random_state=random_state)
            current_pipeline.append(big_object)
        elif isinstance(augment_param, dict):
            prob = augment_param.pop('prob', None)
            for key, value in augment_param.items():
                if isinstance(value, str) and value[0] == '(' and value[-1] == ')' and len(value) > 2:
                    augment_param[key] = literal_eval(value)
            try:
                augment_function = getattr(iaa, function_name)
            except AttributeError:
                print(function_name, 'not found in imgaug lib, using custom')
                augment_function = globals()[function_name]
            required_param = list(inspect.getargspec(augment_function))[0]
            if 'cval' in required_param:
                augment_param['cval'] = cval
            if 'is_binary' in required_param:
                augment_param['is_binary'] = is_binary
            augment_param['random_state'] = random_state
            augment_object = augment_function(**augment_param)
            if prob is not None:
                augment_object = iaa.Sometimes(
                    prob, augment_object, random_state=random_state)
            current_pipeline.append(augment_object)
    return current_pipeline

Also, here is what my yaml config file looks like:
augmentation_pipeline:
    - Sequential:
        - random_order: False
        - ItalicizeLine:
            prob: 0.3
        - RotateLine:
            prob: 0.3
        - OneOf:
            - Pad:
                percent: ((0.05, 0.2),(0.05, 0.2),(0.05, 0.2),(0.05, 0.2))
                pad_mode: constant
                pad_cval: 1
            - Pad:
                px: ((0, 50), (0, 300), (0, 50), (0, 300))
                pad_mode: constant
                pad_cval: 1
        - TransformPerspective:
            prob: 0.3
            scale: (0.1, 0.25)
            keep_size: False
        - ElasticTransformation:
            prob: 0.3
            alpha: (0, 1.5)
            sigma: (0.4, 0.6)
        - Skeletonize:
            prob: 0.02
        - Invert:
            p: 0.1
            max_value: 1

I can submit a PR, but I don't know where to start injecting my function yet @aleju  :P

Writing a proper config parsing method (or a few of them, one per augmenter) is not that straight forward and has quite some requirements, mainly:

The method should require as few unittests as possible. If its requires one method per augmenter then that will require implementing many new unittests, which costs a lot of time.
The method should not require adding an abstract method to imgaug.augmenters.meta.Augmenter as that would break all augmenters implemented by users.
The method should automatically be able to parse configs for new augmenters without any changes. Requiring changes would (a) create frequent bugs by forgetting to change these things when adding new augmenters, (b) make it harder for users to implement their own augmenters and (c) create frequent conflicts between different PRs.
The method should be able to handle new augmenters by users, even when their name is unknown.
The method must be able to deal with augmenters having children (possibly in multiple different lists). These augmenters again must be allowed to have their own children.
The method should be able to handle parameters from imgaug.parameters.
The method should be able to handle the various shortcuts for parameters, such as (0.0, 1.0) for Uniform(0.0, 1.0).
The method should be able to handle non-augmenter classes. E.g. Voronoi accepts classes for point sampling that aren't augmenters.
The method should be able to handle renaming of augmenters and parameters between versions of the library, as well as removals/additions of parameters without crashing.
The method shouldn't only work with yaml as some people might use other config file formats.
The method shouldn't have many augmenter-specific parsing rules as nobody could memorize these.
If at any point the library adds GPU support, the method should allow to add this to the parsing without breaking previous config files.

It's not that hard to quickly come up with a parsing method. But it is equally easy to violate one or more of the above points and then e.g. suddenly spend ages of time writing unittests for dozens of augmenters.

Damn :P. Sound kind da overwhelming for me lul

You could break your code down into 2 or more modules, one of them being your configuration data (ie, the constructed sequence), and then import it into your main script.

I think I did something that fulfills the requirements (proof of concept, not PR right now) ...
Right now it is able to generate the example labelled "Heavy Augmentations" in the documentation (https://imgaug.readthedocs.io/en/latest/source/examples_basics.html) by using an equivalent yaml file: https://gist.github.com/jspaezp/6152733a644b8f6fe9b985b58c856997#file-complex_yaml_001-yaml only requiring to define the sometimes lambda function in the python call environment.
It more accurately parses the yaml and given a series of locations (similar to a PATH on the system level), looks for the names that match callable elements and greedily does the call.
https://gist.github.com/jspaezp/6152733a644b8f6fe9b985b58c856997#file-yaml_parser-py
(right now it has a lot of print statements for debugging purposes... but they should be easy to remove or make conditional on a debug/verbose argument).
right now I'm testing it with pytest and works fine, there is definitely some more work to do regarding unit tests, since I don't really know how to test if two augmenters are equivalent
let me know what you think.
Best,
Sebastian

Any updates on this issue?