Enum binding when it shouldn't

If we have the following:
public enum TestEnum {
    Val1 = 1,
    Val2 = 2
}

[HttpGet("test/{testEnum}")]
public IActionResult Get(
    [FromRoute(Name = "testEnum")] TestEnum test,
{
    return Ok(test);
}

GET test/10 will return 10, and likewise GET test/invalid returns 0 when I should not expect either to bind.
How can I configure it to do so? I assume I need to set allowBindingUndefinedValueToEnumType some how? https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNetCore.Mvc.Core/ModelBinding/Binders/EnumTypeModelBinder.cs

Ah.. I see this was added here https://github.com/aspnet/Mvc/pull/6812
And there's this potentially related issue https://github.com/aspnet/Mvc/issues/6858
Will this be the correct way to configure this?

@kichalla can you take a look?

This issue is a duplicate of the issue: https://github.com/aspnet/Mvc/issues/6329. (Issue https://github.com/aspnet/Mvc/issues/6858 is a side effect of the fix that was made for it)
@Mardoxx The fix you suggested is part of 2.1.0-preview1 and is only available on MyGet though.

Great thanks!


Timezone issues

Fix trigger when using different timezone

I have manually merged this into a branch that lets me create a pre-release (<IP_ADDRESS>-timezone-issues).  Can you test this, please, and ensure that the behaviour you observed is correct?  If so then I will merge into the main branch!


Tuya water leak sensor

Before opening a new issue, please read the following
Are you reporting a successful flash?
No
Do you have a question about configuring your newly flashed device?
No
Did you have issues getting a device to flash?
Yes
Please remember to close your issue when the problem has been addressed!

I can not help if you do not provide information.

Hello everybody!
I'm having issues in flashing Tuya water leak sensor based on TYWE3S chip through Tuya-converter (OTA).
Log files are the following:
smarthack-psk.log
new client on port 443 from <IP_ADDRESS>:63396
could not establish sslpsk socket: [SSL: HTTPS_PROXY_REQUEST] https proxy request (_ssl.c:1056)
new client on port 443 from <IP_ADDRESS>:63401
could not establish sslpsk socket: [SSL: HTTPS_PROXY_REQUEST] https proxy request (_ssl.c:1056)
new client on port 443 from <IP_ADDRESS>:29845
ID: 0242416f68626d6436614739314946523189171e11c10b33ac675ac84173c862e8cf2101eb31025a140ed4b3190120fea2
PSK: a78407407c485620c80ee44da304ba68ee026b43eb2598cbb38de60c995513b0
could not establish sslpsk socket: [SSL: DECRYPTION_FAILED_OR_BAD_RECORD_MAC] decryption failed or bad record mac (_ssl.c:1056)
new client on port 443 from <IP_ADDRESS>:121
ID: 0242416f68626d6436614739314946523189171e11c10b33ac675ac84173c862e8cf2101eb31025a140ed4b3190120fea2
PSK: a78407407c485620c80ee44da304ba68ee026b43eb2598cbb38de60c995513b0
could not establish sslpsk socket: [SSL: DECRYPTION_FAILED_OR_BAD_RECORD_MAC] decryption failed or bad record mac (_ssl.c:1056)
new client on port 443 from <IP_ADDRESS>:7455
ID: 0242416f68626d6436614739314946523189171e11c10b33ac675ac84173c862e8cf2101eb31025a140ed4b3190120fea2
PSK: a78407407c485620c80ee44da304ba68ee026b43eb2598cbb38de60c995513b0
could not establish sslpsk socket: [SSL: DECRYPTION_FAILED_OR_BAD_RECORD_MAC] decryption failed or bad record mac (_ssl.c:1056)
new client on port 443 from <IP_ADDRESS>:43427
ID: 0242416f68626d6436614739314946523189171e11c10b33ac675ac84173c862e8cf2101eb31025a140ed4b3190120fea2
PSK: a78407407c485620c80ee44da304ba68ee026b43eb2598cbb38de60c995513b0
could not establish sslpsk socket: [SSL: DECRYPTION_FAILED_OR_BAD_RECORD_MAC] decryption failed or bad record mac (ssl.c:1056)
new client on port 443 from <IP_ADDRESS>:42231
ID: 0242416f68626d6436614739314946523189171e11c10b33ac675ac84173c862e8cf2101eb31025a140ed4b3190120fea2
PSK: a78407407c485620c80ee44da304ba68ee026b43eb2598cbb38de60c995513b0
could not establish sslpsk socket: [SSL: DECRYPTION_FAILED_OR_BAD_RECORD_MAC] decryption failed or bad record mac (ssl.c:1056)
^CTraceback (most recent call last):
File "./psk-frontend.py", line 113, in
main()
File "./psk-frontend.py", line 106, in main
r,, =  select.select(readables, [], [])
KeyboardInterrupt
smarthack-wifi.log
===========
Configuring AP interface...
RTNETLINK answers: File exists
RTNETLINK answers: File exists
Starting DNSMASQ server...
Starting AP on wlan0...

Thank you in advance!

Unsupported, please see pinned issue #483 and the wiki for more information.


all github commands cannot be found.

i've installed github pull requests from within vs code.
i received no github authentication prompts.
whenever i try to create a pull request, i get the following error: command 'pr.create' not found
trying to run any other GPR command gives the same type of error (not found error).

Extension version: 0.7.0
VSCode Version: 1.34.0
OS: Windows 10 (Windows_NT x64 10.0.17763)

Steps to Reproduce:

Install Github Pull Repository
Click on "+" to create a new pull request

I have had this same issue for a few says also. I was just about to report it. I have tried enabling/disabling and installing/uninstalling. There are still no commands found.
I have also tried the option to Manually Provide an Authentication response. This gives an error

command 'auth.inputTokenCallback' not found

I have also tried setting the githubPullRequests.hosts setting. This does not solve the problem.

@Pooslice81 @MattWThomas What do you see in the PR tree?

The extension starts initializing some pieces on startup, but doesn't finish registering commands until a folder is open that has git. We should remove the "+" button and commands until this registration is finished.

@RMacfarlane , initially, the PR tree was as empty as your screenshot.
I then manually opened a folder that has git.
this added some categories into the PR tree.


Can't resolve all parameters for LogProvider

I integrated the plugin to my project. But when I am building it, it throws following error message:
Uncaught Error: Can't resolve all parameters for LogProvider: ([object Object], [object Object], [object Object], ?).     at syntaxError (vendor.js:108172)     at CompileMetadataResolver._getDependenciesMetadata (vendor.js:124725)     at CompileMetadataResolver._getTypeMetadata (vendor.js:124618)     at CompileMetadataResolver._getInjectableTypeMetadata (vendor.js:124840)     at CompileMetadataResolver.getProviderMetadata (vendor.js:124849)     at vendor.js:124787     at Array.forEach (<anonymous>)     at CompileMetadataResolver._getProvidersMetadata (vendor.js:124747)     at CompileMetadataResolver.getNgModuleMetadata (vendor.js:124466)     at JitCompiler._loadModules (vendor.js:131750)
I am using Ionic Version 4.12.0.

Hi,
Can you show me how you call the LogProvider constructor in your app.module file? It should look something like this:
`export function provideLogger(file: File, platform: Platform, datePipe: DatePipe) {
/**

Provider for persistent file logging services
*/
return new LogProvider(file, platform, datePipe, new LogProviderConfig({/your config here/}));
}
`

Based on the error message, it seems like you might not be providing a LogProviderConfig object to the constructor correctly.

Hello again,
I have just released version 2.0.3 which I expect will fix this issue. Note that there are changes to the way the LogProvider should be incorporated into your project; please consult the README for details.

I am having same issue.
Just added to my app.module
import {DatePipe} from '@angular/common';
import {LogProvider, LogFileAppenderModule} from 'ionic-log-file-appender';
providers
File,
DatePipe,
LogProvider
Then on app.components
constructor( private log: LogProvider...
return log.init({
/* Your config here */
});
Just did ionic serve and was getting this error
Do I need to pass a config to the Init?
Please provide an example

I integrated the plugin to my project. But when I am building it, it throws following error message:
Uncaught Error: Can't resolve all parameters for LogProvider: ([object Object], [object Object], [object Object], ?). at syntaxError (vendor.js:108172) at CompileMetadataResolver._getDependenciesMetadata (vendor.js:124725) at CompileMetadataResolver._getTypeMetadata (vendor.js:124618) at CompileMetadataResolver._getInjectableTypeMetadata (vendor.js:124840) at CompileMetadataResolver.getProviderMetadata (vendor.js:124849) at vendor.js:124787 at Array.forEach (<anonymous>) at CompileMetadataResolver._getProvidersMetadata (vendor.js:124747) at CompileMetadataResolver.getNgModuleMetadata (vendor.js:124466) at JitCompiler._loadModules (vendor.js:131750)
I am using Ionic Version 4.12.0.

I was also faced same error, but i am able to fix it by making following changes. Note I am using ionic 5
File to update :
node_modules/ionic-log-file-appender/log.service.d.ts
node_modules/ionic-log-file-appender/fesm2015/ionic-log-file-appender.js
In both of above files I have made following changes:
import { File } from '@ionic-native/file/ngx';
import { Platform } from '@ionic/angular';
I have update "from" part in above imports
Thats all.


Fix the Hello World exercise so it passes with 'make test' on CI

I do not know if this is failing locally, but on Travis CI we are getting the following failure:
===> Compiling exercises/hello-world/src/hello-world.lfe failed
/home/travis/build/exercism/lfe/exercises/hello-world/src/hello-world.lfe:9999: unbound function: #(from 1)
make: *** [test] Error 1
The command "make test" exited with 2.

Glancing over it, it was introduced in #79.  If it were it would have catched the problem.


[ENH] - Enable hot reloading when developing

Feature description
Currently, development with docker compose up --build can be slow because making a small change requires the developer to docker compose down, then docker compose up --build, which redownloads the entire environment.
uvicorn has support for hot reloading (although there are caveats), and if we can mount the current filesystem into the container read/write, we should be able to develop and have the server hot reload when changes are made.
Value and/or benefit
Faster development.
Anything else?
No response

CondaStoreServer inherits from traitlets.config.Application. Currently we have

class CondaStoreServer(traitlets.config.Application):
...

    def start(self):
        fastapi_app = self.init_fastapi_app()

        uvicorn.run(
            fastapi_app,
            host=self.address,
            port=self.port,
            workers=1,
            proxy_headers=self.behind_proxy,
            forwarded_allow_ips=("*" if self.behind_proxy else None),
            reload=self.reload,
            reload_dirs=(
                [os.path.dirname(conda_store_server.__file__)]
                if self.reload
                else []
            ),
        )

When hot reloading, uvicorn requires the user to pass a string that returns a FastAPI app i.e.:
    uvicorn.run(
        "conda_store_server._internal.server.app:CondaStoreServer.create_webserver", # <-- this factory command instantiates the object which knows the server address and port
        host=server.address, # <-- There's no way for the new CondaStoreServer instance to pass the address and port here
        port=server.port,
        workers=1,
        proxy_headers=server.behind_proxy,
        forwarded_allow_ips=("*" if server.behind_proxy else None),
        reload=server.reload,
        reload_dirs=(
            [os.path.dirname(conda_store_server.__file__)]
            if server.reload
            else []
        ),
        factory=True
    )

However in our case, the traitlets application contains the server address, port, and other settings. There's no way to pass that to the uvicorn.run call without instantiating the traitlets application twice.

So I tried that: if I break out the running of the webserver into conda_store_server._internal.server.__main__:
import uvicorn
import os
import conda_store_server
from conda_store_server._internal.server.app import CondaStoreServer

main = CondaStoreServer.launch_instance

if __name__ == "__main__":
    server = main()
    uvicorn.run(
        "conda_store_server._internal.server.app:CondaStoreServer.create_webserver",
        host=server.address,
        port=server.port,
        workers=1,
        proxy_headers=server.behind_proxy,
        forwarded_allow_ips=("*" if server.behind_proxy else None),
        reload=server.reload,
        reload_dirs=(
            [os.path.dirname(conda_store_server.__file__)]
            if server.reload
            else []
        ),
        factory=True
    )

and define
class CondaStoreServer(Application):
...
    @staticmethod
    def create_webserver() -> FastAPI:
        server = CondaStoreServer.launch_instance()
        return server.init_fastapi_app()

then from what I can see, running conda_store_server._internal.server.__main__ would

Create an instance of CondaStoreServer, which would read configuration values
Call uvicorn.run, which would instantiate another CondaStoreServer via CondaStoreServer.create_webserver():

CondaStoreServer.create_webserver()
CondaStoreServer.launch_instance() (which would then call CondaStoreServer.start(), initializing the FastAPI app that would be used by uvicorn.

In any case the webserver never seems to come up.

I might be wrong, but AFAIK, to enable hot reload, it would be needed to use bind mounts on our docker-compose.yamland perhaps run this with docker compose up (might be ok with the --build flag)
Using bind mounts would basically enable hot-reloading. Right now, we are copying the code and installing it in editable mode at build time (for the Docker container).
so basically we'd need to add something like:
volumes:
  - type: bind
    source: .
    target: /opt/conda-store (I think?)

without setting those bind mounts uvicorn wouldn't be aware of any changes and the uvicorn.run(..., reload=True) becomes useless which I believe is what is being described in your findings above

👆 Bind mounts are enabled (with shorthand syntax):
- ./conda-store-server/:/opt/conda-store-server/:delegated

Fortunately @soapy1 has had some insight into getting this working, although recent changes have a few issues that we're still working out.


Accessible setting (text, icons..)

One of the tester complained about the small text in the notes and icons behind the mobile menu. How about we offer a setting for people for example with visual impairment.

*Christian: Is actually adjustable via the browser settings. We would only have to check if it also works for Scrumlr.


Lastgenre plugin whitelist

After reading http://beets.readthedocs.org/en/latest/plugins/lastgenre.html I decided to try to implement the genres whitelist.
I opted for a list entered by user in the config file. Because genre name can contain spaces, genres must be separated by comma. That's not homogeneous with the plugins names option of the [beets] section where separator is space, do you see a better solution ?
[lastgenre]
genres_whitelist: blues,acid jazz,afro-beat,afro-cuban jazz

Merged your patches along with a new change that makes the config parameter point to a text file (rather than inlining the whitelist). The default whitelist is included with beets.
Let me know if you make progress on the "canonicalization" idea. Seems interesting.


Cannot join testnet

Expected Behavior
Should sync with peers
...
Actual Behavior
Cannot connect to peers
01:49:59.514 [info] aec_peers started for "pp_TEshw19...eTkJ4t1"
01:50:00.029 [warning] lager_error_logger_h dropped 0 messages in the last second that exceeded the limit of 50 messages/sec
01:50:04.591 [info] Connection handshake failed - timeout was from <<"<IP_ADDRESS>">>
01:50:04.600 [info] Connection handshake failed - timeout was from <<"<IP_ADDRESS>">>
01:50:04.603 [info] Connection handshake failed - timeout was from <<"<IP_ADDRESS>">>
01:50:04.769 [info] Connection handshake failed - timeout was from <<"<IP_ADDRESS>">>

...
Steps to Reproduce the Problem

Download official binaries( I tried l.0, 1.0.1, RC4,rc5,rc6 and 0.25
configure epoch.yaml. I added the following lines below to epoch.yaml for testnet and verified using ./bin/epoch check_config epoch.yaml

peers:
    - "aenode://pp_QU9CvhAQH56a2kA15tCnWPRJ2srMJW8ZmfbbFTAy7eG4o16Bf@<IP_ADDRESS>:3015"
    - "aenode://pp_2vhFb3HtHd1S7ynbpbFnEdph1tnDXFSfu4NGtq46S2eM5HCdbC@<IP_ADDRESS>:3015"
    - "aenode://pp_27xmgQ4N1E3QwHyoutLtZsHW5DSW4zneQJ3CxT5JbUejxtFuAu@<IP_ADDRESS>:3015"
    - "aenode://pp_nt5N7fwae3DW8Mqk4kxkGAnbykRDpEZq9dzzianiMMPo4fJV7@<IP_ADDRESS>:3015"

Start node using ./bin/epoch foreground

Logs, error output, etc.
01:49:59.514 [info] aec_peers started for "pp_TEshw19...eTkJ4t1"
01:50:00.029 [warning] lager_error_logger_h dropped 0 messages in the last second that exceeded the limit of 50 messages/sec
01:50:04.591 [info] Connection handshake failed - timeout was from <<"<IP_ADDRESS>">>
01:50:04.600 [info] Connection handshake failed - timeout was from <<"<IP_ADDRESS>">>
01:50:04.603 [info] Connection handshake failed - timeout was from <<"<IP_ADDRESS>">>
01:50:04.769 [info] Connection handshake failed - timeout was from <<"<IP_ADDRESS>">>

...
Specifications

Virtualization: NONE
Hardware specs:

RAM:  32GB
CPU: Intel(R) Xeon(R) CPU E3-1230 v3

OS: Ubuntu 18.04
Node Version:

Fixed now. Network id should be set to ae_uat network_id: ae_uat


Various changes

Coverage decreased (-0.09%) to 93.617% when pulling 6f3da043ebdb48768e0684ba6a9dd047e036b7d2 on freq2 into<PHONE_NUMBER>c687c34ec024af3c1d1ca0c4ac4eb3 on frequencies.

Coverage decreased (-0.09%) to 93.618% when pulling<PHONE_NUMBER>8527b7998d5c7c24a14a73a4a414de on freq2 into<PHONE_NUMBER>c687c34ec024af3c1d1ca0c4ac4eb3 on frequencies.


Stuck at "Waiting For Spotify" - Spotify changed the way you get the song metadata AGAIN

With the new Spotify update, Spotispy can't go any further than the "Waiting For Spotify..." screen.
Spotify changed again the port/security/way you retrieve the song metadata, and this affects other apps that used to use it like Musixmatch in Windows 10.
I suspect Musixmatch will roll an update someday, maybe it is helpful.

It looks like the only way to access the currently playing track now is through their OAuth web API. Let me know if you find anything else about Musixmatch.

@tma02 as far as I know, they are aware the app stopped working due to the last Spotify update and are working on a new update that should come soon.
However, they have direct contact with the Spotify team because they recently released a new feature of connecting both apps, so I wouldn't take for granted the existence of a way for third party apps to access currently playing track. Maybe they are restricting it to partners (Although if that was the case, I suppose they would tell it to Musixmatch instead of surprising them with this change).
Is there a way to obtain the tokens for the OAuth API?

Thanks for the info. And yes, you can obtain API tokens but just authenticating with your Spotify account. It's just a barrier I don't think users would want to deal with on a local app.

well I sure would. Logging in with an account is not a problem for me. I understand it may be hard to code from scratch though.

I would certainly be fine with logging in, if you wanted to include that. I think as an optional feature it would be a nice backup in case Spotify messes with their metadata again.

I would be fine with signing in, It's almost like they change it on purpose.... heh

It doesn't seem like a lot of people are actually opposed to signing in to access spotispy. If this remains the case, I will work on an update to comply with the API change.

@tma02 Musixmatch was updated today. Their app remains the same, no Spotify Login required nor restarting of any kind. It just works. Unfortunately I can't get the code for you or the lines the update changed because it is a W10 App and I don't know how those work, but maybe you're able to reverse engineer it?

Interesting, thanks for the update. Maybe they're not using the Spotify Web API then. I'll have to investigate that.

I started reverse engineering Musixmatch today, it doesn't look like a lot has changed actually. I should be able to write a version that doesn't require user sign in.


Disable COMPILER_INDEX_STORE_ENABLE on CI

📲 What
Adds a CircleCi.xcconfig file which disables COMPILER_INDEX_STORE_ENABLE for a small performance gain on CI.
🤔 Why
@dusi mentioned a tweet that suggested this would offer a small performance gain so we figured we'd give it a try.
Results that I've seen on an initial run are:

Test suite
Before 🐛
After 🦋

Library
5:51
5:37

Kickstarter-Framework
9:20
8:31

KsApi
1:22
1:35

LiveStream
1:03
1:09

Curiously, on KsApi and LiveStream we saw a slight decline in speed, so it's hard to say whether the gain of nearly a minute for Kickstarter-Framework is worth it?
🛠 How

Added a CircleCI.xcconfig file which inherits from Base.xcconfig.
Implemented the use of CircleCI.xcconfig in our Makefile for tests and Fastfile for gym.

👀 See
https://twitter.com/steipete/status/1100404024224804871
✅ Acceptance criteria

[ ] There is a tangible performance gain

 Disable compiler indexing flag on CircleCI

Wonder whether we should try running this couple times (10-50 times) and try collecting the metrics? Might be an overkill but would be interesting to see if these times are consistent or not.

Yeah that makes sense @dusi, I'll re-run the workflow a couple times, and let's see if we can get some averages 👍

Yeah that makes sense @dusi, I'll re-run the workflow a couple times, and let's see if we can get some averages 👍

Closing this for now - I couldn't reliably reproduce significant improvements 🤷♂


Remove stale and UB winapi compatibility layer

Fixes #255
With the windows migration we introduced a winapi compatibility layer to make it easier to use winapi types with gpu-allocator.  As most of the ecosystem has switched to windows it is now time to sunset these helpers which are otherwise tedious to test and maintain, not to mention contain long(er) standing unsoundness bugs.

Yikes this new presser code is causing some new clippy lints... and as far as I remember didn't even resolve our original soundness issues :sweat_smile:


[Fix][UI Next][V1.0.0-Alpha] Fix the upstream Tasks field UI display extra long.

Purpose of the pull request
close #9113

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Codecov Report

Merging #9148 (12ec2fa) into dev (73ca831) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #9148      +/-   ##
============================================
- Coverage     40.32%   40.28%   -0.04%
+ Complexity     4370     4367       -3
============================================
  Files           808      808
  Lines         32569    32569
  Branches       3650     3650
============================================
- Hits          13135    13122      -13
- Misses        18205    18220      +15
+ Partials       1229     1227       -2

Impacted Files
Coverage Δ

...er/master/processor/queue/TaskResponseService.java
49.43% <0.00%> (-13.49%)
:arrow_down:

...dolphinscheduler/remote/future/ResponseFuture.java
81.96% <0.00%> (-1.64%)
:arrow_down:

...r/plugin/task/sqoop/parameter/SqoopParameters.java
56.33% <0.00%> (ø)

Continue to review full report at Codecov.

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


test case implementation example for JavaScript Validators

test case implementation example for JavaScript Validators.
redpen.js : utlity script for testing JavaScript validators
validator-js-test.js : tests validator.js

Thank you a lot for the contribution @yusuke as always! I have verified the changes are fine. I just think the directory name does not exist in the REDPEN_HOME as I commented.


Grouping devices in AutoPilot deployment groups

We have multiple logical groups of Corporate owned devices, I want to assign specific policies and content to a an azure dynamic group of devices that was enrolled using a specific AutoPilot deployment profile. A Dynamic group in Azure only allows to query the deployment profile that works for Apple Devices, please make it work for Windows devices as well.

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

ID: cc947476-4fc3-cbb1-26c2-1e7dedb7892f
Version Independent ID: 16bf5a1f-00d8-ad55-f66f-a9a38f115bb6
Content: Manage Windows device deployment with Windows AutoPilot Deployment
Content Source: store-for-business/add-profile-to-devices.md
Service: unspecified
GitHub Login: @TrudyHa
Microsoft Alias: TrudyHa

@TrudyHa -- Can you take a look at this issue?

@koeneijken - thank you for engaging with the community here for the docs.
I know the product teams are always looking for user feedback on how to make Windows 10 better. They keep a close tab on user feedback and it would be awesome if you could add your voice in
User Voice https://support.microsoft.com/en-za/help/4021566/windows-10-send-feedback-to-microsoft-with-feedback-hub
You might find others have made a similar request that you can add to, or you can open a new one. Let us know here if you do so we can up-vote your request.
Thank you for your contribution to make the docs better! Much appreciated!

@AndreaBarr - This issue can be closed. Thank you.


Kansas

data

assigned to argolb

@tarakc02 KS is ready for review.

I've created PR #39, we'll review the two requested updates on Tuesday morning, but overall this looks good to me! should be able to merge during our meeting.

@ayyubibrahimi data is processed, uploaded to dropbox, and gets downloaded in preprocess/download. let us know if there's anything else we need to do here. thanks


Noise module

implementation of GAN noise module with config files

Have a quick skim first?

Sure!

Added one comment. I think the paths related issues might crop up again. An ideal solution would be to have an .env file that stores certain paths as environment variables. Let's handle this change later/in another PR.


Add tests that we receive expected events from the shim for pod/container

This work adds tests to ensure we see expected events when a pod or container is created, started, exited, and deleted. A containerd event service is needed in the tests to subscribe to targeted events. This required  adding new or updating existing go modules in the project.
Signed-off-by: Kathryn Baldauf<EMAIL_ADDRESS>

I think we should also check that the ID on each task matches what we expect it to be (e.g. for the container or sandbox). Otherwise the container events test could just be seeing events from the sandbox container.
You can look at how IDs are extracted from events here for a reference: https://github.com/containerd/cri/blob/master/pkg/server/events.go#L138

@katiewasnothere - Please dont do merge commits they make things messy. For a PR if you want to update to the latest just rebase on current master so its still just 1 commit

ping @microsoft/containerplat

Hey @katiewasnothere - Can you update the PR including the vendor versions to match containerd 1.3.2 and then we can take this. Thanks!


dont match only part of the node type

I noticed while developing a node that I was seeing the wrong data-help-name. My node type was 'timer' and I was seeing the help text for 'bigtimer'. I figured it had to be a substring match someplace.  It seems it's in the editor itself.
There may be a good reason there is a partial attribute match done. I couldnt really think of it, and tested a bit and everything seems ok. So if anything, this might help you think of other solutions to the problem.

Coverage remained the same at 87.269% when pulling 173e1e556e8fe6059dccb9a05d48979cef64946d on corbosman:master into b756a8edef78280bcfe6010cc5ea3147ba69c78e on node-red:master.

Ah - this is already fixed in the 0.17 branch - I tightened up the matching when doing some other work on the info sidebar.
Will merge this as a bug fix in case we do a 0.16.x service release from master - before 0.17


HADOOP-17988. Disable JIRA plugin for YETUS on Hadoop

Description of PR
The jira-json goes missing all of a sudden and we get the following error in the Jenkins CI run -
[2021-10-27T17:52:58.787Z] Processing: https://github.com/apache/hadoop/pull/3588
[2021-10-27T17:52:58.787Z] GITHUB PR #3588 is being downloaded from
[2021-10-27T17:52:58.787Z] https://api.github.com/repos/apache/hadoop/pulls/3588
[2021-10-27T17:52:58.787Z] JSON data at Wed Oct 27 17:52:55 UTC 2021
[2021-10-27T17:52:58.787Z] Patch data at Wed Oct 27 17:52:56 UTC 2021
[2021-10-27T17:52:58.787Z] Diff data at Wed Oct 27 17:52:56 UTC 2021
[2021-10-27T17:52:59.814Z] awk: cannot open /home/jenkins/jenkins-home/workspace/hadoop-multibranch_PR-3588/centos-7/out/jira-json (No such file or directory)
[2021-10-27T17:52:59.814Z] ERROR: https://github.com/apache/hadoop/pull/3588 issue status is not matched with "Patch Available".
[2021-10-27T17:52:59.814Z]

The hadoop-multibranch pipeline doesn't use ASF JIRA, thus, we're disabling the jira plugin to fix this issue.
How was this patch tested?
In progress.
For code changes:

[x] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

@aajisaka could you please review this PR?

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
9m 46s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-2.10 Compile Tests _

+0 :ok:
mvndep
2m 24s
Maven dependency ordering for branch

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 15s
Maven dependency ordering for patch

+1 :green_heart:
shellcheck
0m  1s
There were no new shellcheck issues.

+1 :green_heart:
shelldocs
0m 15s
There were no new shelldocs issues.

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

_ Other Tests _

+1 :green_heart:
asflicense
0m 28s
The patch does not generate ASF License warnings.

15m 22s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3623/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3623

Optional Tests
dupname asflicense shellcheck shelldocs

uname
Linux 436c8abd3c59 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-2.10 / f7ec214

Max. process+thread count
52 (vs. ulimit of 5500)

modules
C:  U:

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3623/1/console

versions
git=2.7.4 maven=3.3.9 shellcheck=0.3.7

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Clone can't checkout commit pointed to by tag, but Git works

If a tag is created pointing to a commit that hasn't been pushed yet, then the checkout will error. However, git works, and the commit is browse-able on GitHub. However, you can see its not in the commit history at all.
The only reason this feels like a bug to me is really because git show <tag> and git checkout <tag> somehow work despite the commit not being easily viewable in any other way.
I created a test repository to reproduce this: https://github.com/mitchellh/go-git-test-failure
Notice the tag v0.1.0 points to a commit (you can click it and see it), but if you go to the commit history it doesn't exist, and there are no other branches either. This is a lightweight tag. But clone the repository manually and you'll be able to use that tag with git.
And some code you can run with go run to reproduce this:
package main

import (
	"fmt"
	"io/ioutil"
	"os"

	"gopkg.in/src-d/go-git.v4"
	"gopkg.in/src-d/go-git.v4/plumbing"
)

func main() {
	// Create the temporary directory for cloning
	dir, err := ioutil.TempDir("", "test")
	if err != nil {
		panic(err)
	}

	// Clone it into the target directory getting only the tag commit
	repo, err := git.PlainClone(dir, false, &git.CloneOptions{
		URL: "https://github.com/mitchellh/go-git-test-failure.git",
	})
	if err != nil {
		os.RemoveAll(dir)
		panic(err)
	}

	// Get the working tree so we can change refs
	tree, err := repo.Worktree()
	if err != nil {
		panic(err)
	}

	tagName := "v0.1.0"

	// Checkout our tag
	err = tree.Checkout(&git.CheckoutOptions{
		Branch: plumbing.ReferenceName("refs/tags/" + tagName),
	})
	if err != nil {
		panic(err)
	}

	fmt.Println(dir)
}

The problem is the following:
By default when you clone a repository, only the tags that belong to any of the branches being clone are created in the local repository.
Since this tag points to a commit that is not in the history of the branch master, the tags doesn't exists locally.
I though that this was the behavior of git, but apparently isn't. I will try find this code in git and then I will fix the issue.

Ok I found the logic on git.
Making a PR.

Thanks! I'll look for the v4 gopkg update and grab it. :)

https://github.com/src-d/go-git/releases/tag/v4.0.0-rc15

Thanks, confirmed it works great.


Add JSX support

This will allow parsing JSX-containing (e.g. React) files

This still doesn't quite work on React files written in full ES2015 style (particularly import aliases are lost and I can't seem to find them in the AST) but it gets farther along :smile:


Removed shallowequal, simplified transitionMiddleware

@stevoland, this look okay?

Yeah looks good, I wonder what I was thinking at the time.


useSelector hook seems to break hook parsing in React Developer Tools

What version of React, ReactDOM/React Native, Redux, and React Redux are you using?

React: 18.2.0
ReactDOM/React Native: 18.2.0
Redux: 4.2.1
React Redux: 8.0.1

What is the current behavior?
When I use the useSelector hook in my component, the Components tab of the React Developer Tools fails to parse the hooks:

Steps to reproduce:

Install the React Developer Tools extension in Firefox.
Open an app that contains a component that uses the useSelector hook.
Open the Developer Tools and click on the "Components" tab.
Select the component in the tree and look at the "hooks" section of the right panel.

What is the expected behavior?
I expect the React Developer Tools to be able to parse the hooks without error and show the names of the variables:

Which browser and OS are affected by this issue?
Firefox 123.0.1, WIndows 10 Pro
Did this work in previous versions of React Redux?

[ ] Yes

Looks like this is an upstream bug: facebook/react#27889
It may be fixed by React Dev Tools 5.0.1 or 5.0.2, either that or a uSES fix. Either way, it's not something we can fix ourselves.

@timdorr Thanks for the link to the relevant issue! I'm glad to see it's been fixed. Hopefully there will be a new release of the extension soon.


CB-11070 Document SplashScreenBackgroundColor

Also documented Windows-specific transparent BackgroundColor
Jira issue

LGTM. @rakatyal, @omefire for FYI

ACK!


timeouts, retries and circuit breakers for Envoy

Jenkins job manager/presubmit passed

Jenkins job manager/presubmit passed

Added. Although, I have not set them up such that they are triggered, as it would be equivalent to testing if Envoy's implementation of circuit breaker is correct or not. My aim is to see that these things get parsed and end up in the config file.
The YAML as such looks very bad (thanks to the 100 levels of nesting we have :( ). For a simple timeout, we have 3 levels of nesting.

Thanks. I'll push a unit test to make sure we don't regress on those
samples. It's a good start.
On Thu, Mar 2, 2017, 6:42 PM Shriram Rajagopalan<EMAIL_ADDRESS>wrote:

Added. Although, I have not set them up such that they are triggered, as
it would be equivalent to testing if Envoy's implementation of circuit
breaker is correct or not. My aim is to see that these things get parsed
and end up in the config file.
The YAML as such looks very bad (thanks to the 100 levels of nesting we
have :( ). For a simple timeout, we have 3 levels of nesting.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/istio/manager/pull/249#issuecomment-283849329, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJGIxhCsVgyk9IXZXSJZyIWeT6SAj_psks5rh34NgaJpZM4MRuRM
.

Not sure why test failed. Logs look clean, but the test failed to get restart epoch.

Yep. Will do.
On Sat, Mar 4, 2017 at 4:09 PM Kuat<EMAIL_ADDRESS>wrote:

@kyessenov commented on this pull request.
In proxy/envoy/config_test.go
https://github.com/istio/manager/pull/249#discussion_r104297190:

)

func TestMockConfigGenerate(t *testing.T) {
ds := mock.Discovery
r := mock.MakeRegistry()
+

rTemp, err := ioutil.ReadFile(routeRule)

This is a good idea - was going to do that once your PR is in. I'd suggest
adding another golden config with all rules applied. That way it's easier
to see problems with core TCP/HTTP routing vs policies vs filters.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/istio/manager/pull/249#pullrequestreview-25132513,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH0qdwMG8J1B5MCllTMkVXdpCZSWSLadks5ridL9gaJpZM4MRuRM
.

--
~shriram

So this test is failing. Despite adding logic for not generating the
default catch all cluster in build function (if the route already had a
catch all route). Any idea why?
On Sat, Mar 4, 2017 at 4:56 PM Shriram Rajagopalan<EMAIL_ADDRESS>wrote:

Yep. Will do.
On Sat, Mar 4, 2017 at 4:09 PM Kuat<EMAIL_ADDRESS>wrote:
@kyessenov commented on this pull request.
In proxy/envoy/config_test.go
https://github.com/istio/manager/pull/249#discussion_r104297190:

)

func TestMockConfigGenerate(t *testing.T) {
ds := mock.Discovery
r := mock.MakeRegistry()
+

rTemp, err := ioutil.ReadFile(routeRule)

This is a good idea - was going to do that once your PR is in. I'd suggest
adding another golden config with all rules applied. That way it's easier
to see problems with core TCP/HTTP routing vs policies vs filters.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/istio/manager/pull/249#pullrequestreview-25132513,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH0qdwMG8J1B5MCllTMkVXdpCZSWSLadks5ridL9gaJpZM4MRuRM
.
--
~shriram

--
~shriram

Jenkins job manager/presubmit passed

Jenkins job manager/presubmit passed

Jenkins job manager/presubmit passed

Jenkins job manager/presubmit passed


"IndexError: list index out of range" not causing install to fail

I'm seeing errors like the following but the install doesn't seem to be failing;
# Install OSU T18 ls digital standard cells.
set -f ; ../common/foundry_install.py -std_format -source  -target /ssd/gob/foss-eda-tools/skywater-pdk-libs-sky130_fd_bd_sram/third_party/open_pdks/sky130/sky130A \
        -techlef sky130_osu_sc_18T.tlef rename=sky130_osc_sc_18t_ls.tlef \
        -spice 18T_ls/cdl/*.spice compile-only exclude=*.*.spice \
        -lef 18T_ls/lef/*.lef \
        -lib 18T_ls/lib/*.lib \
        -gds 18T_ls/gds/*.gds compile-only \
                options=custom/scripts/gds_import_setup.tcl \
        -library digital sky130_osu_sc_18t_ls 2>&1 | tee -a sky130A_make.log
Traceback (most recent call last):
  File "../common/foundry_install.py", line 430, in <module>
    sourcedir = option[1]
IndexError: list index out of range

Also having this issue. Any tips would be great!

It's not a fatal issue in any way, but there are parts of the Makefile that should be completely ignoring libraries that have not been enabled during configure, and instead it's attempting to install them.  Since it's not affecting the rest of the install, I've put the issue on the back burner until I have some time to look into it (along with a lot of other unnecessarily alarming error and warning messages).

I believe this was fixed by https://github.com/RTimothyEdwards/open_pdks/pull/186
There is also #189 which takes it the next step further.


Update protocol-params with costModels

once dbsycn12 is in production

https://github.com/blockfrost/blockfrost-cardano-cli/commit/eb163a78524ab15c8e53822d4548593079b04d3e


Changed gas siphon to check controller slots

Just realized I forgot to run spotless. In a few hours I'll amend the commit and push it.


Update microsoft365-maturity-model--cognitive-business.md

Updated governance 300 section as this seemed to have text from another competency!
Please use this template to ensure easier processing of your pull request
Category

[x] Content fix
[ ] New article

Related issues
Contents of the Pull Request
Updated governance 300 section as this seemed to have text from another competency!
Guidance

Thanks, @SimonJHudson!


Ketcher doesn't trigger change event in macromolecule mode

How can I subscribe to Ketcher change event when using macromolecule mode.
I have used the demo stand, tried to subscribe to 'change', but it didn't work.
Please see the gif below.

Tested. Bug fixed.
https://github.com/user-attachments/assets/c6285c1e-6f4f-4b43-995c-50fed95ad5fb
Desktop:

OS: Windows 11
Browser Chrome
Version 128.0.6613.120 (Official Build) (64-bit)

Ketcher version
[Version 2.28.0-rc.1]
Indigo version
[Version 1.27.0-rc.1]


docs: update internal-documentation for batching and export

The documentation is already upto date from the recent changes.


Upgrade node packages

What is the motivation for adding / enhancing this feature?
Running yarn audit  on clean installation of vue-storefront-api on branch master outputs 19 vulnerabilities. 2 with critical severity.
Running yarn outdated states most of the packages current version being behind wanted semver version and way behind latest versions, thus do not apply minor or major updates.
What are the acceptance criteria

[ ] yarn audit shows 0 vulnerabilities
[ ] yarn outdated shows packages apply latest version

Can you complete this feature request by yourself?

[ ] YES
[x] NO

Which Release Cycle state this refers to? Info for developer.
Pick one option.

[ ] This is a normal feature request. This should be available on https://test.storefrontcloud.io and then after tests this can be added to next Vue Storefront version. In this case Developer should create branch from develop branch and create Pull Request 2. Feature / Improvement back to develop.
[ ] (Pick this option only if you're sure) This is an important improvement request for current Release Candidate version on https://next.storefrontcloud.io and should be placed in next RC version. In this case Developer should create branch from release branch and create Pull Request 3. Stabilisation fix back to release.
[x] (Pick this option only if you're sure) This is a critical improvement request for current Stable version on https://demo.storefrontcloud.io and should be placed in next stable version. In this case Developer should create branch from hotfix or master branch and create Pull Request 4. Hotfix back to hotfix.

Additional information

Thanks, we'll take care of it. @ResuBaka can you take a look please?

I can do it. I only would look into it in the develop branch as we don't really have a released version on master, when that is okay?


Day 96

Good morning! Here's your coding interview problem for today.
This problem was asked by Microsoft.
Given a number in the form of a list of digits, return all possible permutations.
For example, given [1,2,3], return [[1,2,3],[1,3,2],[2,1,3],[2,3,1],[3,1,2],[3,2,1]].

https://en.wikipedia.org/wiki/Heap's_algorithm


Added OCR typo transformation

Description

Added OCR typo transformation. I have only added replacement and deletion as they seem to be the most common type of error in OCR as mentioned here
Related Issue

#1565
Type of Change

[ ] 📚 Examples / docs / tutorials / dependencies update
[ ] 🔧 Bug fix (non-breaking change which fixes an issue)
[ ] 🥂 Improvement (non-breaking change which improves an existing feature)
[x] 🚀 New feature (non-breaking change which adds functionality)
[ ] 💥 Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🔐 Security fix

Checklist

[x] I've read the CODE_OF_CONDUCT.md document.
[x] I've read the CONTRIBUTING.md guide.
[ ] I've updated the code style using make codestyle.
[x] I've written tests for all new methods and classes that I created.
[ ] I've written the docstring in Google format for all the methods and classes that I used.

@andreybavt I apologize for the late fix, I had gone on a new year vacation. Happy New Year. I have fixed the error (I was using another random seed in testing). And added the things you recommend. Kindly let me know if I should do anything else, Thank you.

Thank

@andreybavt I apologize for the late fix, I had gone on a new year vacation. Happy New Year. I have fixed the error (I was using another random seed in testing). And added the things you recommend. Kindly let me know if I should do anything else, Thank you.

Thanks @goyalpramod and happy new year! No worries about it, hope enjoyed your vacation!


fix: HTTPClientWithHeaders

ensure it supports nil client as input

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #138 (b5a6e71) into main (a4ec3b3) will decrease coverage by 5.72%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #138      +/-   ##
==========================================
- Coverage   45.51%   39.79%   -5.73%
==========================================
  Files          44       44
  Lines        3950     3950
==========================================
- Hits         1798     1572     -226
- Misses       1818     2095     +277
+ Partials      334      283      -51

Flag
Coverage Δ

2.0.5
39.79% <0.00%> (ø)

2.1.4
39.62% <0.00%> (ø)

2.2.2
?

2.3.3
39.62% <0.00%> (ø)

2.4.0
?

2.5.1
39.62% <0.00%> (ø)

2.6.0
39.62% <0.00%> (ø)

2.7.0
39.62% <0.00%> (ø)

community
39.79% <0.00%> (ø)

enterprise
?

enterprise-<IP_ADDRESS>
?

enterprise-<IP_ADDRESS>
?

enterprise-<IP_ADDRESS>
?

enterprise-<IP_ADDRESS>
?

enterprise-<IP_ADDRESS>
?

enterprise-<IP_ADDRESS>
?

enterprise-<IP_ADDRESS>
?

enterprise-<IP_ADDRESS>
?

integration
39.79% <0.00%> (-5.73%)
:arrow_down:

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

Impacted Files
Coverage Δ

kong/utils.go
78.69% <0.00%> (-1.31%)
:arrow_down:

kong/mtls_auth_service.go
0.00% <0.00%> (-63.74%)
:arrow_down:

kong/workspace_service.go
0.00% <0.00%> (-53.45%)
:arrow_down:

kong/test_utils.go
32.78% <0.00%> (-37.71%)
:arrow_down:

kong/exists.go
50.00% <0.00%> (-25.00%)
:arrow_down:

kong/admin_service.go
0.00% <0.00%> (-19.12%)
:arrow_down:

kong/credentials.go
85.71% <0.00%> (-14.29%)
:arrow_down:

Continue to review full report at Codecov.

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


4 feature/authentic-characters-in-game

Why is this merge request being submitted?
Character model has been completed and can be use for the game
What does this merge request add functionality wise?
A character is crucial element in the game
How has the code been tested?
[ ] Automated tests
[ ] Manual tests (specify what you've done)
[ ] New feature (never existed before)
[x] Feature update (enhancing already existing feature)
[ ] Breaking feature (changes the already existing way of how the process works)

If you have encountered an issue when viewing the model, please let me know before merging it back to the dev branch. Thank you


docs(README): add semantic release badge

Current coverage is 100% (diff: 100%)

Merging #12 into master will not change coverage

@@           master   #12   diff @@
===================================
  Files           1     1
  Lines          14    14
  Methods         0     0
  Messages        0     0
  Branches        0     0
===================================
  Hits           14    14
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update c23e736...0c92bb3


Add the possibility to install tfsuit through brew (for MAC)

Add the possibility to install tfsuit through brew (for MAC) and for Linux (using some package manager).

For Linux as well.


Fixes for airfields in Belgium

Beerse has been closed in 2007 and is unlandable
Tournai Maubray is located in Wallonia, not in Flanders

Thanks - I have updated the region of Tournai Maubray to Wallonia. Deleting an airport is not possible at the moment (because there might be flights in the past). We might have to consider adding a field 'closed' to the schema.


thread safety

Dear Mr. Nehab,
In the past, I have used Lua successfully for multithreaded server architecture, where each thread runs in its own Lua state, and some memory demanding resources (created in a C lib, accessed as userdata) are kept in shared memory  and used by all threads.  Server IO is done using lua-socket; while I know that lua-socket is not guaranteed to be thread safe, its debian package version 2.0.2-8  worked fairly well,  and crashes were really rare.
However, after installing Jessie with lua-socket 3.0~rc1-3, socket IO become totally unstable in multithreaded execution. Lua segfaults or crashes instantly by various memory corruption issues.
Would it be possible to develop lua-socket  towards thread-safety?  In my opinion, absence of basic IO features under multithreading cripples otherwise promising possibility to use Lua in performance critical applications requiring use of multiple cores, provided by modern computers.
Thank you for your valuable work!
Vesa Meskanen

Maybe you should consider migrating your server infrastructure to OpenResty where you have what you want:

shared resources
pre-loaded modules / libraries
configurable worker-threads
modern and high performance (socket) IO using coroutines

On top of that:

LuaJIT
lots of additional data-bindings

Just an idea... Nowadays OpenResty could be considered battle proven and stable

Thanks for an interesting link! OpenResty seems to be worth knowing even if it it would not solve my current problem conveniently.
Currently I solved the problem by placing all http and socket.connect calls behind a mutex lock. Luckily, socket select, send and receive seem to work OK asynchronously. This is a decent solution as my service is mainly computation, not IO, intensive.

So long as there is a single thread accessing a socket, you should be fine. Is there a good scenario where you'd need multiple threads calling socket functions on the same socket object?

At least the following POSIX functions used by LuaSocket are not required (by POSIX) to be thread-safe:

strerror
gethostbyaddr
gethostbyname
inet_ntoa

It is indeed not sensible to use the same socket from several threads simultaneously :) In my server architecture, each thread (they are worker threads serving separate clients) uses its own socket.

Then the problem is most likely in the use of non-threadsafe POSIX functions. Connect should be threadsafe except for the strerror functions, right?

I believe so. gai_strerror is thread-safe in GLIBC (but not on Windows), inet_ntoa is as well (the GLIBC-version uses thread-local storage). gethostbyaddr and gethostbyname are not called from connect, and getaddrinfo/freeaddrinfo are thread-safe. POSIX has strerror_r as a thread-safe replacement for strerror (but that will require some code changes). The usual socket functions (socket, connect, bind, accept, etc.) are all thread-safe.

I did some further testing to see where the problem could be. Again, I noticed that only socket.connect and http calls seemed to cause failures. To be exact, my setup is lua 5.1.5, debian jessie and lua-socket 3.0~rc1-3, which I compiled with debugging information. At the end of this message is a backtrace of a typical crash.
Interestingly, I noticed that when I built a bsd version of lua 5.1.5 for debugging purposes, by linking it with pthreads ( by setting in the Makefile  MYLIBS="-Wl,-E -pthread -ldl" ), crashes do not occur. If I remove the pthread option, crashes show up. So, maybe the bug is my project configuration after all: I do use pthreads for launching the server threads, which run Lua code, from a C module. So, maybe standard Lua will not do, and I have to use a special build of it. I really do not know Linux kernel well enough to understand why this kind of incompatibility issue could arise.
Anyway, I hope this information is useful. Thanks for your attention!
backtrace:
Program received signal SIGSEGV, Segmentation fault.
[Switching to LWP 3266]
__GI_fgets_unlocked (buf=0x7ffff0f47cd0 "0", n=n@entry=880, fp=0x7ffff0f47cd0) at iofgets_u.c:53
53      iofgets_u.c: No such file or directory.
(gdb) bt
#0  __GI_fgets_unlocked (buf=0x7ffff0f47cd0 "0", n=n@entry=880, fp=0x7ffff0f47cd0) at iofgets_u.c:53
#1  0x00007ffff05404ca in get_contents (stream=, len=880, linebuf=0x7ffff0f47cd0 "0") at nss_files/files-XXX.c:201
#2  internal_getent (result=result@entry=0x7ffff0f48190, buffer=buffer@entry=0x7ffff0f47cb0 "0", buflen=buflen@entry=912,
errnop=errnop@entry=0x7ffff0f49690, herrnop=herrnop@entry=0x7ffff0f48174, af=af@entry=2, flags=flags@entry=0)
at nss_files/files-XXX.c:246
#3  0x00007ffff0540db9 in _nss_files_gethostbyname3_r (name=0x7fffd40271e8 "milou", af=2, result=0x7ffff0f48190, buffer=,
buflen=912, errnop=0x7ffff0f49690, herrnop=herrnop@entry=0x7ffff0f48174, ttlp=ttlp@entry=0x0, canonp=canonp@entry=0x0)
at nss_files/files-hosts.c:141
#4  0x00007ffff0541444 in _nss_files_gethostbyname2_r (name=, af=, result=,
buffer=, buflen=, errnop=, herrnop=0x7ffff0f48174) at nss_files/files-hosts.c:372
#5  0x00007ffff73db609 in __gethostbyname2_r (name=0x7fffd40271e8 "milou", af=af@entry=2, resbuf=resbuf@entry=0x7ffff0f48190,
buffer=buffer@entry=0x7ffff0f47cb0 "0", buflen=buflen@entry=912, result=result@entry=0x7ffff0f48188, h_errnop=0x7ffff0f48174)
at ../nss/getXXbyYY_r.c:266
#6  0x00007ffff73b5c72 in gaih_inet (name=, name@entry=0x7fffd40271e8 "milou", service=,
req=req@entry=0x7ffff0f48410, pai=pai@entry=0x7ffff0f48268, naddrs=naddrs@entry=0x7ffff0f48264) at ../sysdeps/posix/getaddrinfo.c:586
#7  0x00007ffff73b72cd in _GI_getaddrinfo (name=0x7fffd40271e8 "milou", service=0x7fffd40dfa28 "60617", hints=0x7ffff0f48410,
pai=0x7ffff0f483d8) at ../sysdeps/posix/getaddrinfo.c:2406
#8  0x00007ffff5c24cf1 in inet_tryconnect (ps=0x7fffd40ee2b8, family=0x7fffd40f0330, address=0x7fffd40271e8 "milou",
serv=0x7fffd40dfa28 "60617", tm=0x7fffd40f0318, connecthints=0x7ffff0f48410) at inet.c:390
#9  0x00007ffff5c26fc9 in meth_connect (L=0x76bd10) at tcp.c:255
#10 0x00000000004080c0 in ?? ()
#11 0x00000000004115b2 in ?? ()
#12 0x000000000040851d in ?? ()
#13 0x000000000040780b in ?? ()
#14 0x000000000040867b in ?? ()
#15 0x0000000000405dba in lua_pcall ()
#16 0x00007ffff5c25f54 in protected (L=0x76bd10) at except.c:107
#17 0x00000000004080c0 in ?? ()
#18 0x00000000004115b2 in ?? ()
#19 0x000000000040851d in ?? ()
#20 0x000000000040780b in ?? ()
#21 0x000000000040867b in ?? ()
#22 0x0000000000405dba in lua_pcall ()
#23 0x00000000004167a8 in ?? ()
#24 0x00000000004080c0 in ?? ()
#25 0x00000000004115b2 in ?? ()
#26 0x000000000040851d in ?? ()
#27 0x0000000000405d25 in lua_call ()
#28 0x000000000041dbea in ?? ()
#29 0x00000000004080c0 in ?? ()
#30 0x00000000004115b2 in ?? ()
#31 0x000000000040851d in ?? ()
#32 0x0000000000405d25 in lua_call ()
#33 0x00007ffff6eb95fb in ?? () from /usr/lib/x86_64-linux-gnu/lua/5.1/thread0.so
#34 0x00000000004080c0 in ?? ()
#35 0x00000000004084d4 in ?? ()
#36 0x000000000040780b in ?? ()
#37 0x000000000040867b in ?? ()
#38 0x0000000000405e45 in lua_cpcall ()
#39 0x00007ffff6eb9635 in ?? () from /usr/lib/x86_64-linux-gnu/lua/5.1/thread0.so
#40 0x00007ffff6cb5c6b in ?? () from /usr/lib/libsdf0.so.0
#41 0x00007ffff6a9e0a4 in start_thread (arg=0x7ffff0f49700) at pthread_create.c:309
#42 0x00007ffff73ca04d in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:111

Some further notes: Lua's thread safety is not discussed much in the Internet, but I managed to find the following link: https://lwn.net/Articles/654762/ . So, while the error is not in Lua itself, but in the C library, it can be addressed by suitable build options.
Consequently, I submitted a bug report to Lua's Debian maintainers, requesting that future packages would be built using -pthread flag.


[SPARK-41421][UI] Protobuf serializer for ApplicationEnvironmentInfoWrapper

What changes were proposed in this pull request?
Add Protobuf serializer for ApplicationEnvironmentInfoWrapper
Why are the changes needed?
Support fast and compact serialization/deserialization for ApplicationEnvironmentInfoWrapper over RocksDB.
Does this PR introduce any user-facing change?
No
How was this patch tested?
New UT

@techaddict LGTM except for two minor comments. Thanks for working on it.

@gengliangwang Thanks for the reivew, addressed comments.

Thanks, merging to master


markdown_timestamp: Remove incorrect space

The space wasn’t visible because it was narrower than the padding on .rendered_markdown time, but it showed up in copy-and-paste.

Looks good, marked to merge once CI passes.


NodePort health checks should be disabled when kube-proxy is installed

See commit descriptions.

test-me-please

ConformanceAKS is failing with known flake https://github.com/cilium/cilium/issues/16292, k8s-1.19-kernel-5.4 with https://github.com/cilium/cilium/issues/15455. Reviews are in. Marking as ready to merge.


fix:restic#292 Prevent concurrent processing of the same blob

Prevent concurrent processing of the same blob by first adding a preliminary index entry: Adding a preliminary index entry fails if another goroutine is already working on the same blob. In contrast to my proposal in restic#292, index.Store has not been changed. Instead a new method index.StoreInProgress has been added.

I tested the patch by making a backup of the data set described in restic#292, that initally made me discover the issue addressed by the pull request. During that test restic check did not find any errors. Besides the analysis in restic#292, this is a strong indicator, that the issue is fixed by the patch. However, due to the non-deterministic nature of the problem, it's no guarantee.

Awesome, thanks a lot! I'll have a look on the weekend.

Thanks again for your contribution, I added my comments. These may seem to be only small nitpicks, but it's important for us to keep the code quality high (this includes the comments).
In addition to the comments above there are only two things that should be changed before I will merge this PR:

Please reformat the commit message, following the format described here: https://github.com/restic/restic/blob/master/CONTRIBUTING.md#git-commits
Add a test for StoreInProgress() that tries to add the same ID multiple times.

Please reformat the commit message, following the format described here: https://github.com/restic/restic/blob/master/CONTRIBUTING.md#git-commits
Mmh. Now I'm in trouble, somewhat: Should I rewrite history in a public repository?

Yes, you should rewrite history. This is okay (and common practice) in a topic-branch for a PR, it is never acceptable in e.g. the master (or release or stable) branch.

This looks good, are you finished? :)

Well, I should have addressed all your comments... so, yes :)

ok, cool, thanks for the contribution!


refactor(ast_codegen): consistent import order

#4764
#4763
#4762
#4761  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @overlookmotel and the rest of your teammates on  Graphite


Create package "exposed"

Takes a hackage package, merges other-modules to exposed-modules and releases the package with suffix -exposed for all versions.
This would solve Elm package overrides in stack.yaml.

No longer needed for #14


ChebyKAN benchmarks

ChebyKAN uses Chebyshev polynomials instead of B-splines.
https://github.com/SynodicMonth/ChebyKAN
Would be interesting to see it benchmarked against other KANs

I added it to the benchmark. I chose degree 9 so that the number of parameters is roughly the same. Its performance is between that of efficient kan and fourier kan.


[opt](routine load) optimize routine load timeout logic

If IO/CPU resources are tight, routine load task is likely to timeout. The current method is self-adaption backoff https://github.com/apache/doris/pull/32227, but the problem is it will do some ineffective work to match proper timeout. For one routine load task, a better way to handle task is finishing executing instead of retry when  resources are tight.


Fails when 'UnlockWallet' section is filled in config.json

Version <IP_ADDRESS> silently fails on startup (Windows 10) when I put some data to UnlockWallet. If remove this data then it starts normally again

What error did you receive? and please, show me a sample of your UnlockWallet data

I cannot reproduce it today. There was no error shown, only event was written to Windows Application Log. UnlockWallet very simple:
"UnlockWallet": {
"Path": "myPrivNetWallet.json",
"Password": "pwd",
"StartConsensus": false,
"IsActive": true
}
Also I notices that neo-cli application stopped to sync itself with private network (until I removed its databases)

I think that this has been solved. No similar problem has been reported and this variable has been used without issues.
If you still have any problem in latest version please reopen this issue.


Visibility is not declared on constants.

Description
Visibility MUST be declared on all constants if project PHP minimum version supports constant visibilities (PHP 7.1 or later).
PSR-12: https://www.php-fig.org/psr/psr-12/#43-properties-and-constants

Thanks for suggestion, I think we should refer to PSR-2, that also includes this as MUST, since Laravel itself does not follow PSR-12.
Other than that I think it's a welcome change


Squash all deprecations reported in 0.190.0

Included fixing:

[x] Stylesheets directory rename
[x] .editor class name
[x] Use forks of React and Reactionary
[x] Use new editor callbacks.

This fixes #78 and fixes #77

@alexcorre are there any issues with this? would love to squash the deprecations.

@alexcorre It would be nice, if you review an merge this pr.
@blimmer thanks for fixing stuff for the new api.

sorry guys I must have had notifs off on this. I'll check this out. now. Sorry @ruabmbua @blimmer

Publishing a new release, too, would be helpful.. ;-)

I asked for that in #80

publishing now.

Hi @alexcorre / @blimmer
Seems this broke the package due to atom reactionary fork not working anymore with the module caching and no longer maintained I think.
See #83

That's weird. I just was playing around with it this weekend and it still seemed to work. Since reactionary is dead, we should probably file a story to move off of it entirely.


Flag to disable Singularity cache

Add a no_cache flag (or similar) to the run command to tell Singularity not to cache SIFs.
Should not be necessary in most cases (given https://github.com/Computational-Plant-Science/plantit/issues/94) but some deployment targets with very limited disk quotas or high job volume might require it.

Resolved by 9ff6a73


AriesFrameworkError: Missing required dependency: 'Wallet'

Hey, I am trying to use @hyperledger/aries-askar-react-native with the following configuration, please have a look
Versions
  "@aries-framework/anoncreds": "^0.4.0-alpha.127",
    "@aries-framework/anoncreds-rs": "^0.4.0-alpha.127",
    "@aries-framework/askar": "^0.4.0-alpha.127",
    "@aries-framework/indy-vdr": "^0.4.0-alpha.127",
    "@aries-framework/react-native": "^0.4.0-alpha.127",
    "@hyperledger/anoncreds-react-native": "^0.1.0-dev.18",
    "@hyperledger/aries-askar-react-native": "^0.1.0-dev.11",
    "@hyperledger/indy-vdr-react-native": "^0.1.0-dev.15",
    "@aries-framework/core": "0.4.0-alpha.127",
    "react": "17.0.2",
    "react-native": "0.66.1",
    "react-native-fs": "^2.16.6",
    "react-native-get-random-values": "^1.8.0"

I am getting the following error while initialising the agent
[AriesFrameworkError: Missing required dependency: 'Wallet'. You can register it using one of the provided modules such as the AskarModule or the IndySdkModule, or implement your own.]

Code
const init = async () => {
    console.log("INIT CALLED")
    const agent = new Agent({
      config: {
        label: 'testing-askar',
        walletConfig: {
          id: 'alice',
          key: 'pw',
        },
        // logger: new ConsoleLogger(LogLevel.debug),
      },
      dependencies: agentDependencies,
      modules: {
        askar: new AskarModule({
          ariesAskar,
        }),
        mediationRecipient: new MediationRecipientModule({
          mediatorPickupStrategy: MediatorPickupStrategy.PickUpV2,
          mediatorInvitationUrl:c
        }),
        credentials: new CredentialsModule({
          autoAcceptCredentials: AutoAcceptCredential.ContentApproved,
          credentialProtocols: [
            new V2CredentialProtocol({
              credentialFormats: [new LegacyIndyCredentialFormatService()],
            }),
          ],
        }),
        proofs: new ProofsModule({
          autoAcceptProofs: AutoAcceptProof.ContentApproved,
          proofProtocols: [
            new V1ProofProtocol({
              indyProofFormat: new LegacyIndyProofFormatService(),
            }),
            new V2ProofProtocol({
              proofFormats: [new LegacyIndyProofFormatService()],
            }),
          ],
        }),
        anoncreds: new AnonCredsModule({
          registries: [new IndyVdrAnonCredsRegistry()],
        }),
        anoncredsRs: new AnonCredsRsModule({
          anoncreds: anoncreds,
        }),
        indyVdr: new IndyVdrModule({
          indyVdr: indyVdr,
          networks: [
            {
              genesisTransactions: trential,
              indyNamespace: 'trential-dev',
              isProduction: false,
            },
          ],
        }),
      },
    });
    const httpOutboundTransport = new HttpOutboundTransport();
    const wsOutboundTransport = new WsOutboundTransport();
    agent.registerOutboundTransport(wsOutboundTransport);
    agent.registerOutboundTransport(httpOutboundTransport);
    await agent.initialize();
  };

It seems you have @aries-framework/indy-sdk installed, which is on alpha 129, while the other packages are on 127, which causes the core package to be installed twice. Can you try removing indy sdk (As you're using askar/anoncreds/indy-vdr), and lock the versions to a specific alpha?
    "node_modules/@aries-framework/indy-sdk": {
      "version": "0.4.0-alpha.129",
      "resolved"<EMAIL_ADDRESS>      "integrity": "sha512-sr8rt+nDelUAFJICJbQ2ZsaWvFKAHsOIW4wbkx4r7iHf1Oefhi5eR5HN1g3fsePCplixOQYSw91DHUxa43XqKA==",
      "dependencies": {
        "@aries-framework/anoncreds": "0.4.0-alpha.129+25c76ae6",
        "@aries-framework/core": "0.4.0-alpha.129+25c76ae6",
        "@stablelib/ed25519": "^1.0.3",
        "@types/indy-sdk": "1.16.26",
        "class-transformer": "0.5.1",
        "class-validator": "0.14.0",
        "rxjs": "^7.2.0",
        "tsyringe": "^4.7.0"
      }
    },
    "node_modules/@aries-framework/indy-sdk/node_modules/@aries-framework/anoncreds": {
      "version": "0.4.0-alpha.129",
      "resolved"<EMAIL_ADDRESS>      "integrity": "sha512-9OUsC5ZmqSD8GAFVzLBZ9qE+uPgdkolGWyGPX5n1bsh5y1eMw4PtO9BFuXxWvzeXlRsGLVe9BZiC9hPPJR/rEQ==",
      "dependencies": {
        "@aries-framework/core": "0.4.0-alpha.129+25c76ae6",
        "bn.js": "^5.2.1",
        "class-transformer": "0.5.1",
        "class-validator": "0.14.0",
        "reflect-metadata": "^0.1.13"
      }
    },
    "node_modules/@aries-framework/indy-sdk/node_modules/@aries-framework/core": {
      "version": "0.4.0-alpha.129",
      "resolved"<EMAIL_ADDRESS>      "integrity": "sha512-Y6tI9RpMGlAZofk5DyRrDTLBFy/4xRvSTb19ucFlp3LKidsHtJXn9Te4VJe0pBj4BOjP8I0T5bNmVLmz5sR8Ig==",
      "dependencies": {
        "@digitalcredentials/jsonld": "^5.2.1",
        "@digitalcredentials/jsonld-signatures": "^9.3.1",
        "@digitalcredentials/vc": "^1.1.2",
        "@multiformats/base-x": "^4.0.1",
        "@stablelib/ed25519": "^1.0.2",
        "@stablelib/random": "^1.0.1",
        "@stablelib/sha256": "^1.0.1",
        "@types/ws": "^8.5.4",
        "abort-controller": "^3.0.0",
        "big-integer": "^1.6.51",
        "borc": "^3.0.0",
        "buffer": "^6.0.3",
        "class-transformer": "0.5.1",
        "class-validator": "0.14.0",
        "did-resolver": "^4.1.0",
        "lru_map": "^0.4.1",
        "luxon": "^3.3.0",
        "make-error": "^1.3.6",
        "node-fetch": "^2.6.1",
        "object-inspect": "^1.10.3",
        "query-string": "^7.0.1",
        "reflect-metadata": "^0.1.13",
        "rxjs": "^7.2.0",
        "tsyringe": "^4.7.0",
        "uuid": "^9.0.0",
        "varint": "^6.0.0",
        "web-did-resolver": "^2.0.21"
      }
    },

Oh Okay, removed @aries-framework/indy-sdk
Please check out the following package-lock.json file
https://www.transfernow.net/dl/20230530qZO8XRAe/JTXl0DOg


Address real-time-enforcer container registry/autobuild

With migration of real-time-enforcer module to new Forseti GitHub Organization we need to fix how the docker container is built.

Docker container is built and is working. Need to migrate over to use it.


Add ability to activate with fingers

I really like the extra immersion, but I wish I could brush things with my fingers. Perhaps the vibration intensity could even be lower when you touch with your fingers and then get stronger when you touch with more of your hand?
I don't know the proper etiquette for GitHub feature requests so I'll just say please and thank you!

Hi, I am not sure sure what do u mean by that, if u would want me to do like a kinda button for each user to open a menu it should not be hard tho the only issue with this game its that the ui does not get triggered by the game itself cause they have a weird type of mesh rendered html menu, and i've tryied few stuff like a colider behaind it and stuff like that but sadly that didn't work out, so I would probably need to do a new raycast to trigger my ui, tho im not in the country for few days and also I want do finish some other projects before I would do something like this.


Please have an up to date howto on at least how to open a webpage

After following this:
https://cat.ninja/using-selenium-with-headless-firefox-on-freebsd/
And many other tutorials I still cannot get what I need is that I want to get and save the content of a webpage using javascript, the way it looks like in the browser.
After spending hours on dead projects like console crawler and phantomjs I have found selenium, and it MIGHT be the solution for what I need but I still cannot get it to work.
OS: latest FreeBSD stable FreeBSD 11.2-RELEASE
pkg install py36-pip
pip-3.6 install selenium
pkg install geckodriver
pkg install firefox-62.0.2,1
Example1:
from selenium import webdriver
from selenium.webdriver.common.keys import Keys

driver = webdriver.Firefox()
driver.get("http://www.python.org")
assert "Python" in driver.title
elem = driver.find_element_by_name("q")
elem.clear()
elem.send_keys("pycon")
elem.send_keys(Keys.RETURN)
assert "No results found." not in driver.page_source
driver.close()

Traceback (most recent call last):
  File "open.py", line 1, in <module>
    from selenium import webdriver
ImportError: No module named selenium

I try:
python3.6 a.sel
Traceback (most recent call last):
  File "a.sel", line 5, in <module>
    br = webdriver.Firefox()
  File "/usr/local/lib/python3.6/site-packages/selenium/webdriver/firefox/webdriver.py", line 162, in __init__
    keep_alive=True)
  File "/usr/local/lib/python3.6/site-packages/selenium/webdriver/remote/webdriver.py", line 154, in __init__
    self.start_session(desired_capabilities, browser_profile)
  File "/usr/local/lib/python3.6/site-packages/selenium/webdriver/remote/webdriver.py", line 243, in start_session
    response = self.execute(Command.NEW_SESSION, parameters)
  File "/usr/local/lib/python3.6/site-packages/selenium/webdriver/remote/webdriver.py", line 312, in execute
    self.error_handler.check_response(response)
  File "/usr/local/lib/python3.6/site-packages/selenium/webdriver/remote/errorhandler.py", line 242, in check_response
    raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.SessionNotCreatedException: Message: Unable to find a matching set of capabilities

Example 2
from selenium import webdriver
from selenium.webdriver.common.action_chains import ActionChains
from selenium.webdriver.common.keys import Keys

br = webdriver.Firefox()
br.get('http://www.google.com/')

save_me = ActionChains(br).key_down(Keys.CONTROL)\
         .key_down('s').key_up(Keys.CONTROL).key_up('s')
save_me.perform()

Let's start it with that these are python, why on earth this is not mentioned anywhere that this is freaking python, why cannot at very least a simple example be provided for opening a god damn webpage?!?

Hi @daredevilbsd,
We do mention the languages each browser is made for. Also, this is just a listing, and the individual browsers can do a better job of explaining how they work, so including usage instructions is outside the scope of this project. But that is why we link to the browsers. I'll go ahead and close this issue.

After following this:
https://cat.ninja/using-selenium-with-headless-firefox-on-freebsd/
And many other tutorials I still cannot get what I need is that I want to get and save the content of a webpage using javascript, the way it looks like in the browser.
After spending hours on dead projects like console crawler and phantomjs I have found selenium, and it MIGHT be the solution for what I need but I still cannot get it to work.
OS: latest FreeBSD stable FreeBSD 11.2-RELEASE
pkg install py36-pip
pip-3.6 install selenium
pkg install geckodriver
pkg install firefox-62.0.2,1
Example1:
from selenium import webdriver
from selenium.webdriver.common.keys import Keys

driver = webdriver.Firefox()
driver.get("http://www.python.org")
assert "Python" in driver.title
elem = driver.find_element_by_name("q")
elem.clear()
elem.send_keys("pycon")
elem.send_keys(Keys.RETURN)
assert "No results found." not in driver.page_source
driver.close()

Traceback (most recent call last):
  File "open.py", line 1, in <module>
    from selenium import webdriver
ImportError: No module named selenium

I try:
python3.6 a.sel
Traceback (most recent call last):
  File "a.sel", line 5, in <module>
    br = webdriver.Firefox()
  File "/usr/local/lib/python3.6/site-packages/selenium/webdriver/firefox/webdriver.py", line 162, in __init__
    keep_alive=True)
  File "/usr/local/lib/python3.6/site-packages/selenium/webdriver/remote/webdriver.py", line 154, in __init__
    self.start_session(desired_capabilities, browser_profile)
  File "/usr/local/lib/python3.6/site-packages/selenium/webdriver/remote/webdriver.py", line 243, in start_session
    response = self.execute(Command.NEW_SESSION, parameters)
  File "/usr/local/lib/python3.6/site-packages/selenium/webdriver/remote/webdriver.py", line 312, in execute
    self.error_handler.check_response(response)
  File "/usr/local/lib/python3.6/site-packages/selenium/webdriver/remote/errorhandler.py", line 242, in check_response
    raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.SessionNotCreatedException: Message: Unable to find a matching set of capabilities

Example 2
from selenium import webdriver
from selenium.webdriver.common.action_chains import ActionChains
from selenium.webdriver.common.keys import Keys

br = webdriver.Firefox()
br.get('http://www.google.com/')

save_me = ActionChains(br).key_down(Keys.CONTROL)\
         .key_down('s').key_up(Keys.CONTROL).key_up('s')
save_me.perform()

Let's start it with that these are python, why on earth this is not mentioned anywhere that this is freaking python, why cannot at very least a simple example be provided for opening a god damn webpage?!?

Hi @daredevilbsd . Just like I mentioned on my website, it's important to provide the path to the firefox binary. For some reason it doesn't work otherwise on FreeBSD:
driver = webdriver.Firefox(firefox_binary='/usr/local/bin/firefox', ...)
That's what you're missing in your code.


'Incomplete string escaping or encoding warning'  from CodeQL

I'm running CodeQL on one of my GitHub Actions repositories that found a hit in the moment.js code for  Incomplete string escaping or encoding on this function:
https://github.com/moment/moment/blob/e96809208c9d1b1bbe22d605e76985770024de42/moment.js#L806
I think it is hitting the first string replace, since that is only escaping the first instance of the \\ characters, instead of all of them, but I wasn't sure. Is this something that needs to be fixed?
Reference to CodeQL query here
Referenced CodeQL message:

I'm not that well versed into Javascript, and wanted to let you know nevertheless :-).

Also curious about the best fix for this.  Upgrading fixed 5 issues, but left this new one remaining which CodeQL is considering high severity.

I think this is a false positive, and does not need any action.
The method that does regex-escaping is this one, which CodeQL does not have any issues with:
export function regexEscape(s) {
    return s.replace(/[-\/\\^$*+?.()|[\]{}]/g, '\\$&');
}


Report door status

Report 'door' status for capabilities Garage Door Control and Door Control

@fuzzysb let me know if there are any issues with the proposed changes to make it compatible with the capabilities for door control and garage door control


nixos/kernel.nix: ensure same kernel is used

Motivation for this change
Things done

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

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

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

I can't tell if this is doing the right thing because I get confused with the overriding mechanisms.
However, I was having the issue described in #86835, which for me meant NixOS wouldn't boot since I was using ZFS boot and root filesystems.
This PR fixed the issue for me.

Would it be possible to git commit --amend to explain why this is required, and how this solves the listed issue?

Yeah a better fix would probably be fixing the extend override, but I'm not sure it works.

This significantly changes the semantics of the boot.kernelPackages option, causing all the attributes of the linuxPackages package set to be ignored except for the kernel attribute. If a user specifies a package set with customization other than to kernel, those extra customizations are just ignored now.
See https://github.com/NixOS/nixpkgs/pull/111845 for a fix for the original problem with .extends.


Cannot synth go web-service example

Description of the bug:
When trying to synthesize the go web-service example it fails
Reproduction Steps:
# cdk8s import --language go
Importing k8s v1.17.0...
Importing resources, this may take a few moments...
k8s
# cdk8s synth
# example.com/web-service-go
./webservice.go:10:13: undefined: constructs.ConstructOptions
./webservice.go:18:24: undefined: constructs.ConstructOptions
Error: command "go run . " at /home/rogelio/dev/3rdparty/cdk8s/examples/go/web-service returned a non-zero exit code 2
    at ChildProcess.<anonymous> (/home/linuxbrew/.linuxbrew/Cellar/cdk8s/2.0.80/libexec/lib/node_modules/cdk8s-cli/lib/util.js:50:27)
    at Object.onceWrapper (events.js:520:26)
    at ChildProcess.emit (events.js:400:28)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:285:12)

Error Log:
Error: command "go run . " at /home/rogelio/dev/3rdparty/cdk8s/examples/go/web-service returned a non-zero exit code 2
    at ChildProcess.<anonymous> (/home/linuxbrew/.linuxbrew/Cellar/cdk8s/2.0.80/libexec/lib/node_modules/cdk8s-cli/lib/util.js:50:27)
    at Object.onceWrapper (events.js:520:26)
    at ChildProcess.emit (events.js:400:28)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:285:12)

This is :bug: Bug Report

👍

Hi @rogeliodh. I was able to replicate this issue by creating a new cdk8s go app and then copying the code exactly as it is in the webservice.go example.
The problem seems to be the first import in the webservice.go file:
import (
	"example.com/cdk8s-composition/imports/k8s"
        ...
)

The path before /imports/k8s should be the same as the module name in the first line of the go.mod file:
module example.com/cdk8s-composition

This module name will be different depending on the name of the directory you created the cdk8s app in. For example, if I create the cdk8s app in a folder called go-demo, my module name in the go.mod file will be example.com/go-demo. So, to use this in the webservice.go file I have to change the first import string to example.com/go-demo/imports/k8s. This should fix your issue.


Fedora 39 + Gnome 45.1 - Doesn't work

I upgraded to Fedora 39 with Gnome 45.1 a while ago, and the extension manager says the "current Gnome version is not supported" (mouseover on red clock in the screenshot). What's the best way of getting it to work again? I would prefer a method that doesn't involve loading the extension through a different system...

Try removing the extension. Logging out and logging in and installing it again.

https://extensions.gnome.org/extension/4470/media-controls/
There should be also the option to update

Try removing the extension. Logging out and logging in and installing it again.

That worked. Thank you!

https://extensions.gnome.org/extension/4470/media-controls/ There should be also the option to update

It just gives me an installation button and when clicking it the Extension Manager opens, but it did not resolve the issue.


Integration doc - LoginRadius with MySQL

Prepare integration document on how to integrate MySQL with LoginRadius
You will need to create an account here - https://accounts.loginradius.com/auth.aspx?return_url=https://dashboard.loginradius.com/login&action=register
Go to the integration section to explore
Reference for template - https://www.loginradius.com/docs/developer/guide/shopify/
Document should be end to end I.e. what all needs to be done in LoginRadius and MySQL

Hi @IArchna, I'd love to work on this issue becomes available.

This issue has been assigned to @Hard-Coder05 @JubayerJoy
We will review and merge PR on FCFS basis.


fail to retrieve "http://yaroslavvb.com/upload/notMNIST/nmn.png"

I encounter this error when udacimak tries to retrieve "http://yaroslavvb.com/upload/notMNIST/nmn.png". Maybe because it is not https?
2018-11-16T08:08:12.805Z [error]: ERROR [ERR_TLS_CERT_ALTNAME_INVALID]: Hostname/IP does not match certificate's altnames: Host: yaroslavvb.com. is not in the cert's altnames: DNS:*.accountservergroup.com, DNS:accountservergroup.com
CODE: ERR_TLS_CERT_ALTNAME_INVALID
STACK: Error [ERR_TLS_CERT_ALTNAME_INVALID]: Hostname/IP does not match certificate's altnames: Host: yaroslavvb.com. is not in the cert's altnames: DNS:*.accountservergroup.com, DNS:accountservergroup.com
    at Object.checkServerIdentity (tls.js:228:17)
    at TLSSocket.onConnectSecure (_tls_wrap.js:1059:27)
    at TLSSocket.emit (events.js:182:13)
    at TLSSocket._finishInit (_tls_wrap.js:631:8)

Thanks for reporting.
http or https doesn't matter.
If possible, could you email me the JSON file to<EMAIL_ADDRESS>so I can have a better look?

@jacky-ttt
I've fixed the issue according to here:

https://github.com/request/request/issues/1777#issuecomment-152323328

Enjoy!


Update js

Try pub get for this dep:
flutter_secure_storage: ^9.2.2
flutter_sound: ^9.16.3
Because no versions of flutter_secure_storage match >9.2.2 <10.0.0 and flutter_secure_storage 9.2.2 depends on flutter_secure_storage_web ^1.2.1, flutter_secure_storage ^9.2.2 requires flutter_secure_storage_web ^1.2.1.
And because no versions of flutter_secure_storage_web match >1.2.1 <2.0.0 and flutter_secure_storage_web 1.2.1 depends on js ^0.6.3, flutter_secure_storage ^9.2.2 requires js ^0.6.3.
And because flutter_sound >=9.16.3 depends on flutter_sound_web 9.16.3 which depends on js ^0.7.1, flutter_secure_storage ^9.2.2 is incompatible with flutter_sound >=9.16.3.
So, because iepes_app depends on both flutter_sound ^9.16.3 and flutter_secure_storage ^9.2.2, version solving failed.
You can try one of the following suggestions to make the pubspec resolve:

Consider downgrading your constraint on flutter_secure_storage: flutter pub add flutter_secure_storage:^4.2.1
Consider downgrading your constraint on flutter_sound: flutter pub add flutter_sound:^9.6.0

I found that there is a develop branch which updated the requirements for the web version but it is not published yet.
That's why I forked it and changed the versions temporary until they update the package.
Here is how I am currently doing
  # flutter_secure_storage: ^9.2.2   flutter_secure_storage:     git:         url: https://github.com/selimb86/flutter_secure_storage.git         path: flutter_secure_storage         ref: develop

Also conflicts with flutter_google_places_sdk 0.3.10


HDDS-9892. ozone sh --verbose snapshot info throws IllegalFormatConversionException

What changes were proposed in this pull request?
Fix IllegalFormatConversionException error in verbose snapshot info output -
bash-4.2$ ozone sh --verbose snapshot info vol1/buck1 snap1
Volume Name : vol1
Bucket Name : buck1
Snapshot info for snapshot: snap1 under o3://vol1/buck1
{
  "volumeName" : "vol1",
  "bucketName" : "buck1",
  "name" : "snap1",
  "creationTime" :<PHONE_NUMBER>747,
  "snapshotStatus" : "SNAPSHOT_ACTIVE",
  "snapshotId" : "cd856ae2-cea4-499c-82b0-b91ce07b9d2d",
  "snapshotPath" : "vol1/buck1",
  "checkpointDir" : "-cd856ae2-cea4-499c-82b0-b91ce07b9d2d",
  "referencedSize" : 0,
  "referencedReplicatedSize" : 0,
  "exclusiveSize" : 0,
  "exclusiveReplicatedSize" : 0
}

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-9892
How was this patch tested?
Tested on docker

@hemantk-12 @swamirishi @nilotpalnandi - Please review

@jyotirmoy-gh building is failing because of compile error caused by HDDS-9816 and HDDS-9303."
Please rebase after #5764 is merged.

Thanks @jyotirmoy-gh for the patch, @hemantk-12 for the review.


Relative positonal encoding for continuous positions

The current implementation of relative positional encoding (https://github.com/entity-neural-network/incubator/pull/139) only supports discrete positions (technically, you can give already it continuous positions, which then get truncated by tensor.long). The most basic version of relative positional encoding for vector spaces would apply a function that discretizes all positions, e.g. by dividing by a specified width and then rounding to the nearest integer. There are various extensions to this that might improve performance:

Discretize positions in a way that has a higher resolution for nearby positions and lower resolution for distant positions (only skimmed this very briefly, but https://arxiv.org/pdf/2107.14222.pdf seems to be doing something similar). The idea is that you might care a lot more about small changes in position for nearby entities, but only care about the rough location for more distant entities. Lots of different possibilities for the discretization function.
When an entity position doesn't fall exactly onto one of the discrete locations, interpolate between nearby positions.

This could be prototyped with the Minefield task (https://github.com/entity-neural-network/incubator/issues/50), on which relative positional encoding should achieve the same performance as translation.

One alternative to discretizing positions to a square grid would be to go with a radial grid.

Interpolation implemented in https://github.com/entity-neural-network/incubator/pull/203 and appears to work quite well.


kube-burner-ocp v1.5.1

Type of change

[ ] New feature
[ ] Bug fix
[ ] Optimization

Description
Some relevant changes are included
Related Tickets & Documents

Related Issue #
Closes #

/cc @paigerube14 @afcollins @jtaleric @josecastillolema @mohit-sheth @Harshith-umesh @mukrishn


fix: login settings

If "Do not authenticate existing WordPress-Users" checked, disable normal wp login.
Else, fallback to normal wp login after ldap login failed.

Sorry for the late reply. What were you trying to solve with this?
Currently the feature "Do not authenticate existing WordPress-Users" allows administrators to add new users via LDAP while users that exist with the same user-identifier in the WordPress-Installation as well as in LDAP will not be using the LDAP-Login. This is especially for installations where LDAP-Login is introduced after already some users were created directly in WordPress and where those users accounts could be taken over by someone with the same user-identifier in LDAP. So this is a very edge-casey flag that normal users shouldn't need to use.
So it's not about disabling normal WordPress login. On the contrary. SOme users still need to be able to login via the default WordPress means.
Additionally you would loose the possibility to use an admin-account in WordPress as fallback once the LDAP-Server is not reachable for whatever reason...


handling error escape xml

I've made changes so it would resolve this issue. Already passed test on Grunt

Could you add a unit test to describe what this fixes?
It should fail without the patch. This way we're sure that any future changes won't break this again.

there's no need for another test case, your test cases already covered it all.

Then why do we need this patch?

I've got an error, said that
/works/nolimit/cql-hit/node_modules/xlsx-writer/lib/index.js:224
    return str.replace(/&/g, '&').replace(/</g, '<').replace(/>/g, '&gt
               ^
TypeError: Object 5 has no method 'replace'
at XlsxWriter.escapeXml (/works/nolimit/cql-hit/node_modules/xlsx-writer/lib/index.js:224:16)
    at /works/nolimit/cql-hit/node_modules/xlsx-writer/lib/index.js:134:45
    at /works/nolimit/cql-hit/node_modules/xlsx-writer/node_modules/async/lib/async.js:551:21
    at /works/nolimit/cql-hit/node_modules/xlsx-writer/node_modules/async/lib/async.js:227:13
    at iterate (/works/nolimit/cql-hit/node_modules/xlsx-writer/node_modules/async/lib/async.js:134:13)
    at /works/nolimit/cql-hit/node_modules/xlsx-writer/node_modules/async/lib/async.js:145:25
    at /works/nolimit/cql-hit/node_modules/xlsx-writer/node_modules/async/lib/async.js:229:17
    at WriteStream.<anonymous> (/works/nolimit/cql-hit/node_modules/xlsx-writer/node_modules/async/lib/async.js:556:34)
    at WriteStream.g (events.js:180:16)
    at WriteStream.emit (events.js:117:20)

Here's one element of my array's object
{
    	original_id:"656258342147502080",
    	content:"RT @GingerLanier: Putin and the Shiite 'Axis of Resistance' http://t.co/eumnAIIrTV",
    	date_created:"2015-10-19 23:59:16",
    	from_display_picture:"http://pbs.twimg.com/profile_images/651504414998396928/hdoQSqiv_400x400.jpg",
    	from_id:"308130306",
    	from_name:"Novorossiyan",
    	url:"http://twitter.com/statuses/656258342147502080",
    	sentiment:5,
    	score:0,
    	social_media:"Twitter",
    	score_breakdown:[],
    	"extra":[]
    }

So whats your idea?

After that error I made a quickfix in lib/index.js as in prev issue (that's how I solved my problem).
Then I forked your repo, make some changes.
Could you just merge my unit test, object_test.coffee then see by yourself ?

Ok, now I see it. This is very strange, there's numeric_test which does seem to work without problems.
Need to look into this. Thanks for the test.

Same for me, that's strange after I saw your repo..hehe
Thanks again

Stuff has come up so I don't really have the time to dive into this, but I guess it can't hurt if things work for you.
Release 0.0.11 with this fix, thanks!


How to check if two different molecules represent the same molecule?

In the dataset ‘USPTO’，we can get some product smiles from reaction like:
[NH2:1][CH2:2][CH2:3][CH2:4][CH2:5][C@@H:6]([C:7]([O:8][CH3:9])=[O:10])[NH:11][C:12](=[O:13])[NH:14][c:15]1[cH:16][c:17]([O:18][CH3:19])[cH:20][c:21]([C:22]([CH3:23])([CH3:24])[CH3:25])[c:26]1[OH:27]
And its CanonSmiles is
COC(=O)[C@H](CCCCN)NC(=O)Nc1cc(OC)cc(C(C)(C)C)c1O
I try to use Chem.CanonSmiles() to solve it, but it doesn't work.

I'm closing this because it is a question, not a bug report or enhancement request.
Please ask questions using the "Discussions" tab.


[REQUEST]HTML entity encoder/decoder

I notice some security challenges like math questions numbers will be in HTML Entitiy in the source code, but currently, we dont have decode function in OB and cant continue with config. Would be nice to add them under functions

Please provide an example

Example code is below to parse CAPTCHA numbers  which they are like HTML Entity:
BYPASSCF "https://daofile.com/?op=login"

#CAP PARSE "<SOURCE>" REGEX "Enter code below:<\\/b><\\/td><\\/tr>\\s*
\\s*
([^<]*)<\\/span>([^<]*)<\\/span>([^<]*)" "[1][2][3]" CreateEmpty=FALSE -> VAR "CAP"

Done


Interpereter performance is too low

Bug Report
Interpereter performance is too low , less then 50%-70% of native build .
target app srouce code at https://github.com/NeuraLegion/wafalyzer.
when i it running with native build , it  only took 6  s .  but it runing with interperter took  5 minutes !. this is unacceptable during actual debugging
the http://<IP_ADDRESS> is my local apache2  server . it just provides a empty page .
os : ubuntu20  Linux  5.13.0-51-generic #58~20.04.1-Ubuntu SMP Tue Jun 14 11:29:12 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Crystal:Using Crystal 1.6.0 (2022-10-06)
LLVM: 10.0.0 Default target: x86_64-pc-linux-gnu
xxxx@xxxx-F117-V:~/Downloads/xxxx/wafalyzer-master/wafalyzer-master$ time /home/w/Documents/localgitserver/crystal/bin/crystal     src/cli.cr  http://<IP_ADDRESS>
Using compiled compiler at /home/xxxx/Documents/localgitserver/crystal/.build/crystal

No WAF detected

real    0m6.682s
user    0m6.474s
sys     0m0.562s

xxxx@xxxx-F117-V:~/Downloads/xxxx/wafalyzer-master/wafalyzer-master$ time /home/xxxx/Documents/localgitserver/crystal/bin/crystal     i src/cli.cr  http://<IP_ADDRESS>
Using compiled compiler at /home/xxxx/Documents/localgitserver/crystal/.build/crystal

No WAF detected

real    4m45.937s
user    4m45.578s
sys     0m0.294s

thank for crystal team . please fix this . this is very important for every   developer !

Did you compile the interpreter in release mode?

The purpose of the interpreter is that you are trading compilation time for execution time so that you can quickly start and step through your code for debugging purposes. By definition the code will run slower, however the debug/edit loop is significantly faster in human time.

Hi @acodervic , thanks for reporting, but we're still far from the point in which optimizing the interpreter is a good use of resources. We still need to work on to make it accessible to most platforms, and to polish its rough edges. Speed is a nice to have feature at this point.

I think @straight-shoota suggested somewhere to do make interpreter=1 to have the interpreter, but I can't remember where. It should include release=1 as it makes a huge difference.

Hi @acodervic , thanks for reporting, but we're still far from the point in which optimizing the interpreter is a good use of resources. We still need to work on to make it accessible to most platforms, and to polish its rough edges. Speed is a nice to have feature at this point.

ok , thank for you anser.

@acodervic are you using the interpreter compiled in release mode? It would save us all a discussion 😅


Influxdb Choking Config help

The scenario and the issue is similar to whats shown here
https://github.com/influxdata/influxdb/issues/375
Running on Linux Redhat Processor = 3, CPU Core = 4 and Memory = 16GB, Core(s) per socket = 4
However, im unable to find these config parameters to tweak. Im using the version v0.10.0. Please suggest
flush-after = 10000 # the number of writes after which wal will be flushed, 0 for flushing on every write
bookmark-after = 10000 # the number of writes after which a bookmark will be created
index-after = 10000

Those config options no longer exist in v0.10.0.  You can take a look at a v0.10.0 sample config or run influxd config to generate one.

Thanks for your reply, but what are the new options which can used for tuning, please advice.

Jason,
Thanks for your reply, but what are the new options which can used for tuning, please advice.
Cheerssraj
From: Jason Wilder<EMAIL_ADDRESS>To: influxdata/influxdb<EMAIL_ADDRESS>Cc: gitvegas<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Sent: Tuesday, April 26, 2016 9:47 PM
Subject: Re: [influxdata/influxdb] Influxdb Choking Config help (#6473)
Closed #6473.—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub


Removed class iris.fileformats.rules.CoordAndDims

This is to address issue #2672 . Wanted to see how this fairs against the Travis tests, further changes may be necessary.

This seems to solve issue #2672 and all of the tests pass. However I'd appreciate if somebody could check the changes I've made. For example are the comments I've re-written presented in an acceptable manner.

Is is good to go, just needs to be rebased in order to resolve the merge conflicts. @pp-mo - could you work with @LukeC92 on that please?

Closes #2672

@LukeC92 Good job!