HOTFIX: fix failed cases in FeatureCommandTest

CI sample: https://github.com/apache/kafka/actions/runs/11051663030/job/30710832201?pr=17165
FAILED ❌ FeatureCommandTest > testDescribeWithKRaftAndBootstrapControllers [1] Type=Raft-Isolated, MetadataVersion=3.7-IV0,Security=PLAINTEXT
FAILED ❌ FeatureCommandTest > testDowngradeMetadataVersionWithKRaft [1] Type=Raft-Isolated, MetadataVersion=3.3-IV1,Security=PLAINTEXT
FAILED ❌ FeatureCommandTest > testDescribeWithKRaft [1] Type=Raft-Isolated, MetadataVersion=3.3-IV1,Security=PLAINTEXT

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

The failed test is caused by #17112, which added IBP_4_0_IV3, and the failure is due to the tool-module cache.
@mumrah This is the issue we discussed over the email thread. Perhaps we should always run certain modules to ensure the PR is covered by sufficient integration tests?

Looks like the tests were failing on the PR, so this should not have been merged.
https://github.com/apache/kafka/actions/runs/11027485627

@chia7712 I agree we should eventually have some "acceptance" tests which do some high level sanity checks. In this case, a change to MetadataVersion (located in server-common) will cause pretty much all tests to run except maybe client.

Looks like the tests were failing on the PR, so this should not have been merged.

you are right. I was checking the wrong CI :(

I verified that the test is passing locally. Merging now to un-break trunk

Thanks @FrankYang0529 for fixing this test!


Problems with applications using Material UI

 Issue by treyspiva
Tuesday Mar 07, 2017 at 14:45 GMT
Originally opened as https://github.com/storybooks/storyshots/issues/83

Our application uses Material UI.  I am trying to test the application UI Structure using Story Shots.  However, I am running into a few problems.  Some I have worked around, however one I have not.
Able to Work Around
In material-ui/src/internal/EnhancedButton.js they are doing a few things during the componentDidMount event.  Examples are:

call injectStyle, uses document.createElement and document.body.appendChild to create the style element.
call listenForTabPresses, which calls Events.on and passes in the window object.  The Events class calls the following methods addEventListener, removeEventListener, attachEvent, and detachEvent

However, since Story Shots does not have a DOM the window and document objects was not present.  I was able to get past these problems by providing a global mock.
global.document = {
    createElement: function () {
        return {}
    },
    body: {
        appendChild: function () {
        }
    },
    getElementById: function() {return null;},
    addEventListener: function() {},
    removeEventListener: function() {},
    attachEvent: function() {},
    detachEvent: function() {}
}
global.window = {
    addEventListener: function() {},
    removeEventListener: function() {},
    attachEvent: function() {},
    detachEvent: function() {}
};

Not Able to Work Around
Some components are using refs.  For example in the DropDownMenu.js they are using a ref to get the components rootNode.
<div ... ref={(node) => { this.rootNode = node; }}  ...>
   ...
</div>

Since Story Shots does not have a DOM the node is undefined.  Therefore this.rootNode is undefined and the component throws an error when rendering.
Does anyone know of any workarounds to the ref problem?

Hi @treyspiva -- if you are still having this problem, you may be able to get some way round this problem by providing a custom test function to storyshots and then using a custom createNodeMock.
Let me know how that goes


Scan issue & Saved limitations

1- when i scan, then block devices -> if any device close wifi and turn on it again, block not work on device
2- when scan again for a twice -> block & limitations i did would be canceled and dont't be saved !!!
--->  i wish to be able to limit with mac addr.. & operations i did will be saved

you can bypass scan if you have any other scanner or an access to the router, you can add new or invisible hosts manually, and they will be updated when you command "hosts" without losing the rules applied to the ones you already have.

Ty so much,
Solved
On Sat, Oct 28, 2023, 2:47 PM austrian-oak @.***> wrote:

you can bypass scan if you have any other scanner or an access to the
router, you can add new or invisible hosts manually, and they will be
updated when you command "hosts" without losing the rules applied to the
ones you already have.
—
Reply to this email directly, view it on GitHub
https://github.com/bitbrute/evillimiter/issues/141#issuecomment-1783800076,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AW32DPP6EGLEBI2BZJ23MZLYBT5EJAVCNFSM6AAAAAASAH5LA6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBTHAYDAMBXGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

you are welcome
and to keep tracking those who turn off wifi or ip change in general, you can add the to "watch add id" for specific ones or " watch add all " for everyone.
even if the turn off wifi and change their ip, they will be added automatically with the new ip.


chore(master): release 1.1.25

:robot: I have created a release beep boop
1.1.25 (2023-08-19)
Miscellaneous

deps: update dependency orchestra/testbench to v8.9.0 (b452688)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/laravel-webhook-client/releases/tag/v1.1.25 :sunflower:


Fix typo in README and use constant formatting.

Found a typo in README.rst which is fixed by this PR.
Also, the titles weren't formatted in a consistent way.

Coverage remained the same at 94.433% when pulling 05b590263d945b93f74e1f15927eb30fb067ef4b on goggle:readme into 27cebc56a0f6a900eeb897512879d545095f410b on gweis:master.

thanks


Calculation of available is wrong if creditline is smaller than balance.

Right now if the creditline is 0 and the balance is -1 it will calculate it to -1 as available which is wrong.

As discussed with gus this is the right behavior at the moment. We will make this part of a bigger discussion about the available.


Video plays blank with sound.

What version of WebTorrent Desktop? (See the 'About WebTorrent' menu)
Version 0.5.1 (0.93.4)
What operating system and version?
Ubuntu 14.04 LTS
What did you do?
Tried playing a MKV torrent
What did you expect to happen?
Play the video
What actually happened?
Video plays blank with sound

This is caused by using H265 (HEVC) or another unsupported video codec.
This is fixed now, thanks to https://github.com/feross/webtorrent-desktop/issues/564. You'll now see a message about using VLC when a HEVC video is detected.
This will go out in the next release of WebTorrent (v0.6).


Failing validation of Parameter Store type parameters

CloudFormation Lint Version
1.22.0
What operating system are you using?
Mac
Describe the bug
A template with Parameter Store type parameters fails validation when the default value is a parameter name (as it should be) not a value of the type that will eventually be read from Parameter Store.
Eg,
Parameters:
  vpcId:
    Description: "VPC for Lambdas"
    Type: "AWS::SSM::Parameter::Value<AWS::EC2::VPC::Id>"
    Default: "/network/vpc/primary/id"
  subnets:
    Description: "Subnets for Lambdas"
    Type: "AWS::SSM::Parameter::Value<List<AWS::EC2::Subnet::Id>>"
    Default: "/network/vpc/primary/subnets"
    ```

The first gets a `E1151 '/network/vpc/primary/id' is not a 'AWS::EC2::VPC.Id'`.
The second gets a `E3012 '/network/vpc/primary/subnets' is not of type 'array'`.

The error messages reference the lines where the parameters are referenced, not where the parameters are defined.

The values in parameter store are correct.

This issue is not present in 1.21.0 or earlier.

### Expected behavior

These parameters pass validation as in previous versions.

### Reproduction template

```yaml
Parameters:
  vpcId:
    Description: "VPC for Lambdas"
    Type: "AWS::SSM::Parameter::Value<AWS::EC2::VPC::Id>"
    Default: "/network/vpc/primary/id"
  subnets:
    Description: "Subnets for Lambdas"
    Type: "AWS::SSM::Parameter::Value<List<AWS::EC2::Subnet::Id>>"
    Default: "/network/vpc/primary/subnets"
    ```

Can you share how you are using the values?  Are you using the language extension?  Is this in a FindInMap?  Looking at possible PRs with that release that could have caused this.

The stack where I first found the problem after bumping the ver of cfn-lint the pipeline uses is a SAM template.
Transform:
  - AWS::LanguageExtensions
  - AWS::Serverless-2016-10-31

The subnets param is used in the VpcConfig:
Globals:
  Function:
    Architectures:
      - x86_64
    Runtime: python3.12
    Tracing: Disabled
    VpcConfig:
      SecurityGroupIds:
        - !Ref sg
      SubnetIds: !Ref subnets

The security group referenced above is where the vpcId param is used.
  sg:
   Type: "AWS::EC2::SecurityGroup"
   Properties:
     GroupDescription: "mySG"
     GroupName: "mySG"
     VpcId: !Ref vpcId
     ...

The language extensions transform is used elsewhere in the template.

Okay, yea, I know what the issue is then.  I'm on it.

Some people are using LanguageExtension into SAM to help with parameter resolution before running SAM.  This is part of https://github.com/aws/serverless-application-model/issues/2533.  We did this in #3861 but we shouldn't be doing this for SSM parameters.   Thanks for the quick turn around.


Create End Game Logic

Create EndGame Page
Create the basic template for the page and components. Pass through necessary props and state.

[x] Create structure for EndGame page and components
[x] Pass user info and score to page upon game completion
[ ] Display score on page

Create Score Model
Create the Score model that will be used to save all User scores. This is an individual model that will reference the user model via user-id.

[x] #9

[x] Type (Daily or Continuous)
[x] Date
[x] Number of Guesses
[x] User ID

[x] Create mutation to create a score based on logged in user
[x] Create a query to get all scores
[x] Create a query to get all of a users scores

Connect It
Put it all together!

[x] Run the Score create mutation at the end of game to save users score

Nice work on the Score model @maxh1231 🌟

Looking good @maxh1231

@maxh1231 killed it 🔫


Maint/adjust platform target

Modify projects to point to VS 2015 build tools
Fix a few compiler errors
Change .gitmodules to use SSH for build server.

Also: Updated Squirrel build to use nuspec. http://teamcity.techsmith.com/admin/editBuildParams.html?id=buildType:DotNetCommonWindows_Squirrel_CreateNuGetPackage2015


Add test case showing image with placeholder and failed src

Issue Description
In Profile Image tests, we have a test that displays a image with a placeholder when the img src fails to load. Now that we have moved this functionality into terra-image, we should add a test case for this scenario to terra-image.
Issue Type

[ ] New Feature
[x] Enhancement
[ ] Bug
[ ] Other

Expected Behavior
Test casing showing placeholder content for terra-image when src image fails to load.

Looked into this with @amichaelparker and we found there is already a wdio testcase for this. 👍


Files created while running in a container are created as root user

When the buildkite-agent runs a second time it cannot cleanup the git directory because it doesn't own the files created by the previous build in the docker container.
Is there a recommended way of dealing with this? Adding a hook that runs another container and does a chown of some kind would probably work, but I was hoping for something a little more clean.

We use a script that we allow via sudo for the buildkite-agent user to reset permissions as part of the environment hook: https://github.com/buildkite/elastic-ci-stack-for-aws/blob/master/packer/conf/buildkite-agent/scripts/fix-buildkite-agent-builds-permissions
It's a horrible hack though
If you are running your own docker's, I'd be inclined to try running docker with user namespace remapping: https://success.docker.com/KBase/Introduction_to_User_Namespaces_in_Docker_Engine
That way you can map the UID of root in your containers to that of the user that runs buildkite-agent on the host.

OK, I think I will look into the second option. Thanks!

Ended up going with the first option, don't have time myself right now but would be great to have that a documented thing.
Thanks for a great product and quick followup, much appreciated.


Add Brain Power by NOMA.

Song Name
Brain Power
Artist Name
NOMA
Source
Youtube
Youtube Link
https://youtu.be/mj-v6zCnEaw
Direct File Link
No response
Song ID
1299591
Start Offset [ms]
0
Extra
The existing version of the song in the mod is a remix.  This is the original. You can see it in NexusGD's video https://youtu.be/ThroGkahQFw
Created from Auto Nong.
Created for level: 104444534

UPD: edited song link

reject
already in the mod


[ci] Upload MMI files to GCP after Vivado build

See issue #13603
Signed-off-by: Dan McArdle<EMAIL_ADDRESS>

Hey Miles, PTAL! Could you recommend another reviewer who is also a committer?

I think I should also teach Bazel how to fetch the precomputed MMI files (maybe here) along the same lines as hw/bitstream:test_rom.

I think I should also teach Bazel how to fetch the precomputed MMI files (maybe here) along the same lines as hw/bitstream:test_rom.

Seems like that would be useful if we wanted to allow people to splice arbitrarily. That would probably be integrated into our local (non-GCP) bitstream cache (see rules/bitstreams.bzl and rules/scripts/bitstreams_workspace.py)

@timothytrippel Yes, go for it!


Fixed issue #541 in README.md

The Timestamp column is nullable by default but the README.md stated the opposite. This was fixed now.

Thanks, but why is the column actually nullable? How can there be a log line without a timestamp?

That's a good question. It can probably be answered only by the initial developers of this sink (which was not me :)). Now it is like it is and changing it would maybe break existing applications using the MSSQL sink. I agree with you, that a log entry without a timestamp is not quite useful.


closes #75 | add sidebar support

Codecov Report

Merging #89 (d4f2c0a) into master (f3cd493) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #89   +/-   ##
=======================================
  Coverage   99.15%   99.15%
=======================================
  Files          14       14
  Lines         119      119
  Branches        2        2
=======================================
  Hits          118      118
  Partials        1        1

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 f3cd493...d4f2c0a. Read the comment docs.

Codecov Report

Merging #89 (d4f2c0a) into master (f3cd493) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #89   +/-   ##
=======================================
  Coverage   99.15%   99.15%
=======================================
  Files          14       14
  Lines         119      119
  Branches        2        2
=======================================
  Hits          118      118
  Partials        1        1

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 f3cd493...d4f2c0a. Read the comment docs.


Lockpicking become incredibly hard

Dont know what happend but the lockpicking is ultra hard :D

actually don't know how it happens. maybe @kodkuce knows?

needs to be re-validated on my high end machine.


Addition in STL

Added a bit of data about STL
And code to use frequent methods with vector

Please go through CONTRIBUTING.md

So I should just go through a leet problem and submit solution here that is more easy. Sorry, I should have read it earlier.


from_columns errors when a column is blank

From email: "the from_columns function throws an error when any column is blank... Need to catch this and pass in some representation of Null."

If the column is blank, you'd rather have it implicitly fill with NaN or None? How would we determine if it's a numeric or text column? My initial thought was that raising an error is more explicit. Perhaps the error message could be better, explaining why an implicit action is difficult to decide on.

Currently no column can be blank:
if not all(mapping.values()):
    raise ValueError('all columns must have at least one element')

This seems like a fairly clear error message. At least one value is necessary for every column to infer whether the column should be numeric or text.


Zenith: Use class instead of top-level functions in template

Fixes #6728

The failure is from read the docs and is unrelated.


Discussion: switch to using Noto font

The great thing about using this font is that it supports many many languages (about 581 for the Noto sans IIRC) plus its made and supported by Google so we can bank on lots of improvements over time.
See https://www.google.com/get/noto/
Inspired by butterproject/butter-desktop#444

Currently we're using the font in our designs. For design related changes, we should design mockups and post them in the 'redesign' issue to get some feedback.

Closing because discussion should be moved to https://github.com/amilajack/popcorn-time-desktop/issues/145


Not sure if this was a typo or not, but possibly should be 'tagged' i…

…nstead of 'targed'?  If not, then I guess i'm not familiar with the term and disregard.

@kmansel Thanks for raising this. The original intent was actually targeted.

@samba this has been updated and ready to go . 👍

Looks good to me. 🚀


Support a goto file action in the explorer

VSCode Version: 1.0.0
OS Version: Mac

Please ignore if its suggested already. Ask is, would it be nice to have "search and filter" capability in project explorer. This will become handy especially for Node development, when we bring in some external module, that'll bring in boat load of its own dependencies as well and the list is so huge. If we have to verify something inside the node_modules folder, its a lot of scroll up and down. So capability something similar to "Visual studio solution explorer" search and filter will be nice.

There is the files.exclude setting, pls see  https://code.visualstudio.com/docs/editor/codebasics#_explorer. Does this cover your needs?

Seach is covered by Goto>File. Closing the issue both search and filter is supported.

@egamma, I understand its possible to get to the file I want without using any of these, just scrolling up and down inside node_modules. What I experienced was, I had problems with my own npm module not installing its own dependencies. Unfortunately it starts with "g" and Its right in the middle of 100s of other node_modules. Whenever I wanted to verify whether my module has its dependencies installed, it made more sense to stay within the folder explorer instead of using shift+cmd+f (search) or Goto-->File ..at some point, I started using console, it was very easy to write "tree" function in my profile to filter and show me the tree view of node_modules and modules that start with "g". So, my point is these capabilities might exist but not just useful for this context. IMO,  may be one of those many but console wins vscode over on something the editor could easily do.

@egamma It would also be great to on-demand filter the explorer pane to reduce the "noise" when working on a subset of files. For example, I usually work with VSCode open to the root of our repo (with many projects in it) and I am lately finding myself needing to update common things across the projects (say in package.json, project.json, etc), so right now I have to either scroll around in the explorer pane or use Goto->File -- which forces me to have to type the file name in over an over. If I could just filter the explorer pane to show all project.json files it would be quick and painless.

I've updated the title to capture the original feature request.
@eamodio your comment is a valid feature request, please file a separate issue for it.

@egamma Thanks! Added https://github.com/Microsoft/vscode/issues/9373

This feature request will not be considered in the next 6-12 months roadmap and as such will be closed to keep the number of issues we have to maintain actionable. Thanks for understanding and happy coding!


Use json instead of django simplejson (depreciated)

Hi there, thank you for this great lib.
I just wanted to suggest to switch from the django.utils.simplejson to the json module as the first one will be depreciated from Django 1.7 (and raises annoying warnings in 1.6) and is not really useful anymore since Python 2.7.

Please merge this!

Simple change - sorry we missed it!

Cool thanks. How long till its normally available via pip?

I'll try and get something out this weekend. :-D


HasCheckConstraint() strips out ( )'s inside the string

A check constraint like
(([Prop1] IS NULL AND [Prop2] IS NOT NULL) OR ([Prop1] IS NOT NULL AND [Prop2] IS NULL))

is reduced to
([Prop1] IS NULL AND [Prop2] IS NOT NULL OR [Prop1] IS NOT NULL AND [Prop2] IS NULL)
 ^                                      ^  ^                                        ^

Notice that the internal ( and ) are dropped around ^. We're concerned this can lead to ambiguity in evaluating the condition. Typically this is added to avoid problematic data, so we want to NOT have any ambiguity in either SQL engine evaluation or user readability.
Steps to reproduce
Just try the following in your OnModelCreating(..)
modelBuilder.Entity<FirewallRule>(entity =>  {
    entity.HasCheckConstraint("CK_MyEntity_OnlyOneType", "(([Prop1] IS NULL AND [Prop2] IS NOT NULL) OR ([Prop1] IS NOT NULL AND [Prop2] IS NULL))");
});

Further technical details
EF Core version:
Database provider: Microsoft.EntityFrameworkCore.SqlServer
Target framework: .NET Core 3.0
Operating system: Windows 10 x64 v1903
IDE: Visual Studio 2019 16.3.5

I also want to add before someone nit-picks ;)
For the specific example above, dropping the ( actually doesn't impact the logical outcome given the operator precedence of AND before OR.
We don't know if we're just lucky with the ( stripping "bug" or if you actually have some rules and strip off the ( based on the target SQL provider's capabilities/precedence.

@SidShetye Where do you get the reduced constraint from?
You can script the migrations to see what EF sends to the database: dotnet ef migrations script

@AndriySvyryd : The .sql file from dotnet ef migrations script does show the inner ( )'s.
But if using any other tool like SSMS or VS2019 SQL projects (for schema compare) to see what was written inside the database, they all return back that check constraint without the ( )'s.

Since the output from EF still has the intact constraint then any further manipulation is done by SQL Server


Target netstandard2.0 and add support for protecting keys with certificates on .NET Core

Resolves #80
Changes:

Retarget Microsoft.AspNetCore.DataProtection.* libraries to netstandard2.0. (NB: Microsoft.AspNetCore.Cryptography.* remain netstandard1.3/net451)
Make available APIs that were previously only .NET Framework.

IDataProtectionBuilder.ProtectKeysWithCertificate(string thumbprint)
DataProtectionProvider.Create(string applicationName, X509Certificate2 certificate)
DataProtectionProvider.Create(DirectoryInfo keyDirectory, X509Certificate2 certificate)
DataProtectionProvider.Create(DirectoryInfo keyDirectory,  Action<IDataProtectionBuilder> setupAction, X509Certificate2 certificate)
CertificateXmlEncryptor
ICertificateResolver

Possible breaking change:

Because env:LOCALAPPDATA is not normally defined on macOS/Linux, FileSystemXmlRepository would place xml key files in ~/.aspnet/DataProtection-Keys. In .NET Core 2.0, has implemented supprot for Environment.SpecialFolders.LocalApplicationData which would map to env:XDG_DATA_HOME or ~/.local/share/ASP.NET/DataProtection-Keys

TODO:

If we don't want to make the breaking change to FileSystemXmlRepository, then we need to update it's behavior to detect prefer ~/.aspnet/DataProtection-Keys to Environment.SpecialFolders.LocalApplicationData when running on .NET Core 2.0.

What are your thoughts on the breaking change to the location where FileSystemXmlRepository stores files?

Changing where the files are stored would be bad....

@natemcmaster why breaking change?

@pakrym in PR description.

In .NET Core 2.0, has implemented support for Environment.SpecialFolders.LocalApplicationData which would map to env:XDG_DATA_HOME or ~/.local/share/ASP.NET/DataProtection-Keys

What if we provided an "obsolete" warning about the old file location in 2.0, and 3.0 we moved the location?
Something like:
warning: Storing keys in $HOME/.aspnet/DataProtection-Keys is obsolete. In a future version, these files will no longer be loaded by DataProtection.
         The recommended location is $HOME/.local/share/ASP.NET/DataProtection-Keys.
         You can use the new behavior now by moving the files.

    mkdir -p $HOME/.local/share/ASP.NET/
    mv $HOME/.aspnet/DataProtection-Keys $HOME/.local/share/ASP.NET/DataProtection-Keys

@blowdart ?

Running apps which people never interact with would break.

Ok, I've updated this to preserve compatibility with 1.x. $HOME/.aspnet will keep precedence over $XDG_CONFIG_HOME/ASP.NET on Linux/macOS.

Please ensure the retargeting doesn't break the compat package.

What is it now? 4.5 needed?

Microsoft.AspNetCore.DataProtection.SystemWeb 1.x supported .NET Framework 4.5.1 and 2.0.0-preview1 lifted that to .NET Framework 4.6. I'm still not clear on PM requirements for desktop framework support.

Are we going to continue cross-compiling https://github.com/aspnet/DataProtection/blob/dev/src/Microsoft.AspNetCore.Cryptography.Internal/Microsoft.AspNetCore.Cryptography.Internal.csproj#L7?

Blocked on https://github.com/dotnet/standard/issues/355

Blocked issue was moved to https://github.com/dotnet/corefx/issues/20015.

@Eilon are we okay to merge this to unblock other repos lifting to ns2.0?

:bell: @pakrym @Eilon


Can't connect to raspberry interface

Hello,
I've already set up the Raspberry Pi on the PSV system and can get the device list on <IP_ADDRESS>.
However, on HA, it says it cannot connect.
When I run ifconfig on HA system. I can see the eth0 is at <IP_ADDRESS>, which is weird. The HA is on <IP_ADDRESS>.
In the HA system, It says the network interface it's using is End0. However, I can't see that on ifconfig.
I can confirm that they are on the same lan.

If you can explain your networks in a little more detail that might be helpful.
You have probably 4 systems at minimum:

Your Router / DHCP server (just guessing <IP_ADDRESS>) for your home network and looks like that network is <IP_ADDRESS>/24 (or maybe bigger netmask, but roughly that)
Your HA box <IP_ADDRESS> (is it running the full HA-OS + Supervisor or is it some other container management system?).  This is the system you are seeing eth0 is <IP_ADDRESS>?  (that sounds like it might be a docker virtual device if I'm not mistaken..).  If this is just the docker network on an HA-OS box it should bridge just fine..  if its some other container management system you might well need to allow access.
Your Raspberry Pi which it sounds like is living on both the PVS and your home network?  On your home network the RPI has an address of <IP_ADDRESS> (and presumably a 172 address from the PVS).  When you say you can get the device list endpoint 'on <IP_ADDRESS>' are you running the command from the Raspberry Pi?  It might well have a firewall on the RPI or not be setup to listed on the <IP_ADDRESS> interfaces.  Given the way you say 'from <IP_ADDRESS>' I'm assuming you are curling from a command line on the Raspberry Pi to http://<IP_ADDRESS>/ which means its at least opening the right TCP port on the right interface (so I'd check for a firewall).  Have you tried curling from a different host on the <IP_ADDRESS>/24 network?  does that work properly?
Last of all the PVS, I'm assuming its management interface is not 'on the same lan' because it always has DHCP running and that would conflict with the DHCP you are presumably running on your home network.  I assume there is a two device network of RPI & PVS and then another 'home network' (<IP_ADDRESS>/24 ish)

The first spot of debugging would be to curl the device list endpoint from another machine on the <IP_ADDRESS>/24 network.  If that doesn't work its probably a firewall on the RPI.  If that does work I'd look at the machine running HA-OS..  I'd TCPdump on the host itself (the very base system, not within a container).  See if any connections to <IP_ADDRESS> are leaving the system, investigate from there.

I also have a Raspberry PI (RPI 5 to be exact) and am having the same symptoms, unable to get it to recognize the IP.  Since OP is 6 weeks out and hasn't responded, here are my details.
My RPI is wired with IP <IP_ADDRESS>.  The PSV6 is wireless with IP <IP_ADDRESS>.  Network is Eero 6e.
From RPI Terminal (add-on to HA), I am able to ping <IP_ADDRESS>.  Running curl -v http://<IP_ADDRESS>/cgi-bin/dl_cgi?Command=Get_Comm, it just spins.  Is the URL wrong?
From my laptop, also wireless, I can ping <IP_ADDRESS> and the curl command continues to time out.
Is there any more information In can provide?

@krbaker , from more searching online, I think the issue is that the PVS6 doesn't provide an API over the home connected wifi.  Instead, people have connected an RPI to the PVS6's ethernet and used HA Proxy to forward the LAN connection across their wifi.
Please confirm that we cannot use the PVS6 for API calls over it's wifi connection to the home network.

Please confirm that we cannot use the PVS6 for API calls over its wifi connection to the home network.

This is correct. You cannot access the local API over the PVS6's connection to the home Wi-Fi if so-configured.
You can access the local API over the PVS6's built-in access point (SSID SunPower######) when available.

from more searching online, I think the issue is that the PVS6 doesn't provide an API over the home connected wifi.  Instead, people have connected an RPI to the PVS6's ethernet and used HA Proxy to forward the LAN connection across their wifi.

Correct. Based on the IPs you shared it looks like you either have subnets or a <IP_ADDRESS> <IP_ADDRESS> setup.
My PVS6 is currently showing it is handing out an address in the <IP_ADDRESS> range on it's local network.
The issue that is happening when trying to access via the home connected wifi is that the firmware is running a firewall on the wifi and wan ethernet/usb port.
Using the RPi to the Lan port, where a SunVault would connect for it's Internet connection, and using software like Nginx (HA Proxy) to forward requests from their home network through the RPi's WiFi and out the RPi's Ethernet to reach the LAN, SunVault Ethernet plug, allows the requsts to be answered. This bypasses the firewall.
Inside the SunVault is a communication device for the inverter that requires an internet connection that the PVS would provide through itself, in addition to any other communication methods being used for management of the inverter.

@koleson , I am getting a 403 on the API so I think that means I am out of luck to use this service. Please confirm.

@jgrant216 can you be a bit more specific about which URL you're trying to hit when you get a 403?

@koleson , I was getting a 403 against http://<IP_ADDRESS>/#/summary.
Now I see the integration uses http://<IP_ADDRESS>/cgi-bin/dl_cgi?Command=DeviceList which does indeed return my device list after about 8 seconds.
Sorry for the red herring as I think I'm OK to proceed.  My next step is to sort out my mini-router mapping but at least, from the router WIFI, I can connect to the dl_cgi URL.


Use eslint

Description
Eslint is a lot faster than jshint and friends. Also we had only enabled linting on some files. This time it's the other way around, all files in src/ and test/ will be checked.
Fixes #176
Purpose
Improve linting

changed my user.email to something that is covered by an individual CLA to make googlebot happier...

Might need a commit though

@samdutton PTAL and let me know if anything is out of whack.

@samdutton merging this now due to lots of other PR's are pending and I would like to get this in before. Please let me know in a comment here, in a new issue or PR if something is wrong. Thanks!


Resize on left and top edge

Hi Bokuweb,
I would like to be able to resize by clicking and dragging on the left (and top) edges.
Are there any plans to allow for this?

Hi @bdyetton
It's not the plan for now. I'm sorry.

Ok cool. Maybe Ill have a go adding it myself then

@bdyetton implement left and top resizer, please see demo.

Awesome thanks!!!


Tests to be added.

Tests are to be added.

Database part
Socket part
Authentication part

Added necessary tests


Jgit

ref: https://github.com/OpsMx/kork-oes/pull/106

closing due to merge conflicts
new PR:https://github.com/OpsMx/clouddriver-oes/pull/72


Add zagura.one

This PR is:

[x] Adding a new domain

[ ] Updating existing domain size

[ ] Changing domain name

[ ] Removing existing domain from list

[ ] Website code changes (512kb.club site)

[ ] Other not listed

[x] I used the uncompressed size of the site

[x] I have included a link to the Cloudflare report

[x] This site is not an ultra lightweight site

[x] The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

[x] Check to confirm

- domain: zagura.one
  url: https://zagura.one
  size: 47.39
  last_checked: 2024-02-7

Cloudflare Report: https://radar.cloudflare.com/scan/244d5548-849c-4997-9652-d6fdf98da3ab/summary

Thanks!


Weird Mod In Creative Server

Hey guys, I am reporting a bug for me in my creative server, it's a creative server as I said and people are spawning weird mods like domination pack and rainbow cars, I double checked my server workshop folder it's just no there, now everyone is spawning ntw which is a extremely op gun to have, it's a weird bug so I hop anyone can help me

The rainbow car is actually in the core unturned, as for the other things, there is no possible way they can spawn them if they are not added to your workshop content.

Nelson snuck in the flying car in this weeks update, it's not in the update notes. As for the other workshop mods, players can't spawn items from them if the server doesn't have the mods loaded on it.

Hey guys I figure out what actually happened, I download some mods from unturned mods.net or some websites like that I forgot about it after. Now when I make server it's still there. I corrected this thank you guys


Fix crash on iOS 13

This will fix crash on iOS 13 as iOS 13 no longer supports access to StatusBarWindow.

Can somebody please merge this?


Fix roundProgressLine setting for RingIndicatorView

The roundProgressLine setting wasn't working because is was applied to the wrong UIBezierPath

Good catch! Cheers


[19.03] neovim fails to build

Issue description
When I tried to update my system, the builder for luajit-2.1.0-beta3-env and in turn neovim failed because of a collision:
building Nix...
building the system configuration...
these derivations will be built:
  /nix/store/fwwf6ifi7q4iyxdf6hawasxzb967c8sg-luajit-2.1.0-beta3-env.drv
  /nix/store/6d9icyxmk2w9yppvi70svlaa5akzcykz-neovim-unwrapped-0.3.4.drv
  /nix/store/fzphjpp9w5rmnl9sazz757k8pb09lnlc-neovim-0.3.4.drv
  /nix/store/6z0hckbifg0mdyn8y6fdsz5kvigjrlwx-urn-rt-0.7.2.drv
  /nix/store/img713426wpjjlhz9jjrbb40f5y7gwis-source.drv
  /nix/store/s4wqkhvm3hi2rjfnr7p5z6x7hb8gg9vc-urn-0.7.2-pre.drv
  /nix/store/lrzmjzcdhnyma2n637srl6v1n1y30rdr-system-path.drv
  /nix/store/51pg5hzpd715amhddc91n6pz606nz8mz-unit-systemd-fsck-.service.drv
  /nix/store/l21f790d977ryhmfg7fgdjbpk0jvbvib-dbus-1.drv
  /nix/store/p2slwhmgf4a0vhvmf2lz7azp5x0q14r8-unit-dbus.service.drv
  /nix/store/ramd9wlh2bb6hzgpnkapwzyd7rzb4bjj-unit-accounts-daemon.service.drv
  /nix/store/x518xa3k39h6lbv37j6i78ybdl93x8kr-unit-polkit.service.drv
  /nix/store/0684rzjzg1d43l5jck1gdicn8xnn2gzp-system-units.drv
  /nix/store/bcrvap0ikpzvh965fbn15f6x5ybvm8hw-xsession.drv
  /nix/store/g13bcsc2h8wbbjb3dja50p2azzc88175-desktops.drv
  /nix/store/rf9ksclhnxjxb6f54025np3hvf0rkhpn-xsession-wrapper.drv
  /nix/store/7xxfsgqnv46g44zbs7bn5d81622704ak-lightdm.conf.drv
  /nix/store/fmnv7n77c88l2s4fbc792d0vl5z5d8w9-unit-dbus.service.drv
  /nix/store/jfd8ghhaz7hww3zzp49nk1sdvmzy729h-user-units.drv
  /nix/store/jzxbzqjqyyd7w2v021p6x3n6vvxaprgb-etc.drv
  /nix/store/whmvdnn5n3dc4f7cv0kqhmz6ai0b8ppv-nixos-system-xenon-19.03beta172025.91fa6990b25.drv
these paths will be fetched (32.97 MiB download, 106.75 MiB unpacked):
  /nix/store/06qhys2248snhhmi9mqgbrrbq4qqpyk0-openjdk-8u202bga-jre
  /nix/store/2q0qb7cynpxlwyfnikdrv5bsr8xibdc0-wf-recorder-unstable-2019-03-12
  /nix/store/hkkwld1h5qhd2mpjbgx7nnx7n1wybdl7-ccemux-1.1.1
copying path '/nix/store/2q0qb7cynpxlwyfnikdrv5bsr8xibdc0-wf-recorder-unstable-2019-03-12' from 'https://cache.nixos.org'...
copying path '/nix/store/06qhys2248snhhmi9mqgbrrbq4qqpyk0-openjdk-8u202bga-jre' from 'https://cache.nixos.org'...
building '/nix/store/6z0hckbifg0mdyn8y6fdsz5kvigjrlwx-urn-rt-0.7.2.drv'...
building '/nix/store/img713426wpjjlhz9jjrbb40f5y7gwis-source.drv'...
building '/nix/store/fwwf6ifi7q4iyxdf6hawasxzb967c8sg-luajit-2.1.0-beta3-env.drv'...
created 4 symlinks in user environment
collision between `/nix/store/7k2ln192zjm622pfk7p2h922v1c13pqy-lua5.1-luabitop-1.0.2-3/lib/luarocks/rocks-5.1/manifest' and `/nix/store/z6vry69b33w7rsz252wc0q5afdll15pi-lua5.1-mpack-1.0.7-0/lib/luarocks/rocks-5.1/manifest'
builder for '/nix/store/fwwf6ifi7q4iyxdf6hawasxzb967c8sg-luajit-2.1.0-beta3-env.drv' failed with exit code 25
cannot build derivation '/nix/store/6d9icyxmk2w9yppvi70svlaa5akzcykz-neovim-unwrapped-0.3.4.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/fzphjpp9w5rmnl9sazz757k8pb09lnlc-neovim-0.3.4.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/lrzmjzcdhnyma2n637srl6v1n1y30rdr-system-path.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/whmvdnn5n3dc4f7cv0kqhmz6ai0b8ppv-nixos-system-xenon-19.03beta172025.91fa6990b25.drv': 1 dependencies couldn't be built
error: build of '/nix/store/whmvdnn5n3dc4f7cv0kqhmz6ai0b8ppv-nixos-system-xenon-19.03beta172025.91fa6990b25.drv' failed

Installing luajit without installing neovim works fine, so the luajit package is not at fault.
Steps to reproduce
Update your system to the latest nixos-19.03 version with neovim in your system.environmentPackages. Alternatively, running nix-shell -p neovim will most likely also give the same issue.
Technical details
 - system: `"x86_64-linux"`
 - host os: `Linux 5.0.5, NixOS, 19.03beta171990.ef28995795d (Koi)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.2`
 - channels(root): `"nixos-19.03beta172025.91fa6990b25, nixos-unstable-19.09pre174415.d956f2279b8"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Thanks!  I missed this regression.

Thank you! :)


Remote Extension Host Fails on Codespaces off of main

Describe the bug
(As of main branch, today) After 5 minutes activity, the error “Remote Extension host terminated unexpectedly 3 times within the last 5 minutes.” Is displayed.
To Reproduce
Steps to reproduce the behavior:

Go to “Code”
Click on “Codespaces”
Click on “Create codespace on main”
wait till codespace is created
Wait 5 minutes - edit files, whatever
See error

Expected behavior
This error seems to mess with codespace setup. While not immediately clear, it looks like it interferes with the notebooks; they are not loaded properly.
Desktop (please complete the following information):

OS: codespaces
IDE: VS Code (via Codespace)
As per devcontainer folder
(Current hash) 134906aa3926e2f6515ec9e2bb9e5a32246b1650

Following up on this as we got a different, potentially related report over at #1891 - are you still seeing the same issue with the current main @gblikas?

Following up on this as we got a different, potentially related report over at #1891 - are you still seeing the same issue with the current main @gblikas?

@craigomatic nope! Doesn’t mean it’s fixed tho, just not reproducible.

@alliscode anyone on your team that is familiar with our codespaces setup that could confirm whether this is an issue?

@lemillermicrosoft played around with the codespace settings a bit and it seems to be related to the size/level of the codespace. For instance, upgrading to a 8-core seems to have resolved the issue sometimes, but using the default 2-core causes the issue more often.

Closing this issue since it was created several months back. Please reopen if it's still an issue.


Merge master to 3.0 epic try #2

Merge master to asdf 3.0 epic.
Closes #1080

I mistakenly opened this against master not the epic branch.


[INF] Make pyspark optional

PR Description
Please describe the changes proposed in the pull request:

Make janitor.spark optional as suggested in #551
Fix pytest mark

This PR resolves #(put issue number here, and remove parentheses).

PR Checklist

Please ensure that you have done the following:

[x] PR in from a fork off your branch. Do not PR from <your_username>:master, but rather from <your_username>:<branch_name>.

[x] If you're not on the contributors list, add yourself to AUTHORS.rst.

[x] Add a line to CHANGELOG.rst under the latest version header (i.e. the one that is "on deck") describing the contribution.

Do use some discretion here; if there are multiple PRs that are related, keep them in a single line.

Quick Check
To do a very quick check that everything is correct, follow these steps below:

[x] Run the command make check from pyjanitor's top-level directory. This will automatically run:

black formatting
pycodestyle checking
running the test suite
docs build

Once done, please check off the check-box above.
If make check does not work for you, you can execute the commands listed in the Makefile individually.
Code Changes

If you are adding code changes, please ensure the following:

[x] Ensure that you have added tests.
[x] Run all tests ($ pytest .) locally on your machine.

[x] Check to ensure that test coverage covers the lines of code that you have added.
[x] Ensure that all tests pass.

Relevant Reviewers

Please tag maintainers to review.

@ericmjl

The code coverage drops because I added the try-except clause to the import statements. I'm not sure what is the proper way to test that. I'm also not sure about the values of testing that. Any thoughts?

Hmm, right. And the issue is that you're testing if the package is installed. I'm not sure how you could write a test that would pretend that pyspark is not installed. It's probably fine unless you know how.

Not too worried about the code coverage :smile:. Going to merge!


[4179] Skip providers and lead schools that do not exist and report them at end

Context
https://trello.com/c/gvr8epcX/4179-payment-schedule-import-skip-providers-and-lead-schools-that-do-not-exist-and-report-them-at-end
Changes proposed in this pull request

Change Funding::PayablePaymentSchedulesImporter so it doesn't blow if a provider or lead school is missing. It instead tracks the missing organisations and reports them after the import

Guidance to review

Download a local copy of the prod DB
Open the each CSV and pick an ID (files are attached to the Trello card)
Open the console and find the provider and school for each ID
Change the ID so the import won't be able to find it
Run the follow rake tasks

rake funding:import_lead_school_payment_schedules["SDS_Profile_Apr.csv",1]

and
rake funding:import_lead_school_payment_schedules["SDS_Profile_Apr.csv",1]

After the import is finished, it should print out information about the missing organisations

Important business

[x] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

 Payment schedule import: skip providers and lead schools that do not exist and report them at end

@stevehook has opened a PR for the other import task and uses abort there so we should make this one consistent.


Ignored .editorConfig max_line_length' in Spring Boot Kotlin Project with Spotless

We need to set the 'max_line_length' in the editorConfig file and the editorConfigOverride function. Currently, the '.editorConfig' 'max_line_length' property is not being recognized. However, when we override the editorConfig property in the Gradle files, it works fine. Please address and fix this issue."
root = true

[*]
charset = utf-8
trim_trailing_whitespace = true
end_of_line = lf
max_line_length = 140

[*.{kt,kts}]
ij_kotlin_allow_trailing_comma = true
ij_kotlin_allow_trailing_comma_on_call_site = true
ij_kotlin_code_style_defaults = KOTLIN_OFFICIAL

    spotless {
    kotlin {
        target("src/**/*.kt")
        targetExclude("**/build/**/*.kt")
        targetExclude("bin/**/*.kt")
        ktfmt().kotlinlangStyle()
        // com.pinterest.ktlint:ktlint-cli
        ktlint("1.1.0").setEditorConfigPath("$rootDir/.editorconfig")
//            .editorConfigOverride(
//                mapOf(
//                    "max_line_length" to 140,
//                ),
//            )
        trimTrailingWhitespace()
        endWithNewline()
        licenseHeaderFile("$rootDir/spotless/spotless.license.kt")
    }}```

I have no knowdlegde of Spotless. What is the content of file $rootDir/.editorconfig?
Have you asked you question in the Spotless project first? They know how they have integrated with ktlint and are in best position to determine whether this issue is caused by Spotless or by Ktlint. Ktlint does respects the max_line_length when set.


Update mncvision.id.yml

Shift MNC Vision workflow schedule to 5:05 PM (0:05 AM UTC +7) since scraping takes long time based of recent workflows and it's slightly annoying to me when it's done around 2-3 PM, not counting it's delay due to other guides.
If you allow this, should i change the schedule for Transvision and Vidio too?

So, does this actually resolve the issue @freearhey? Or is more work required?

@ajanes3 I don't know yet.

So.. should i close it? This PR has stalled.
On Thu, Feb 2, 2023, 07:21 Aleksandr Statciuk @.***>
wrote:

@ajanes3 https://github.com/ajanes3 I don't know yet.
—
Reply to this email directly, view it on GitHub
https://github.com/iptv-org/epg/pull/1635#issuecomment-1412952212, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AV74QVHL72W7KNVVBNWNQODWVL4XVANCNFSM6AAAAAATUTTAGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Stalled PR


AIMetatrader.com

Hello,
i am a privat Person and i try to learn programming. I have a project to include Google AI latest to Metatrader4/5. Actually i want to inframe MT4/5 to a Domain called www.AIMetatrader.com and connect the MT4/5 to Python to include AlphaPy and open-source the result, as a free Web-Trading-AI.
This is difficult for me, because i am not yet a programmer. I eater don't have Money to pay a programmer so i want to asking you for Partnership on this Project. I hope you are interested and agree.

Duplicate of #19


Update dependencies (06/2016)

Here we go again :smile:

[ ] Check gems
[ ] Check stuff in vendor/

Closing and rolling into #1629


Update Ghost (blog) theme

As a developer I want to run Ghost locally because I need to update and test a theme
Background
The Open Targets blog uses a modified version of the Liebling Ghost theme. Unfortunately not all the changes are available in Github.
The newest version of the theme offers support for Ghost v5 so we need to update our fork of the theme with the latest changes available.
For testing/dev purposes we'll also probably want to run Ghost locally...
Tasks

[x] Merge new changes in original theme, fixing conflicts
[x] Run Ghost locally
[ ] Since we're at it, we could also rename default branch to main

Acceptance tests
How do we know the task is complete?

We can run the updated theme locally and/or
Check the theme with the Ghost theme app: https://gscan.ghost.org/

Hi @HelenaCornu adding you in for reference and in case of comments or ideas


docs: fixing typos and enhance fr translations

Hello @rrousselGit, hope you're doing well.
Description
This PR is an attempt to fix some typos and enhance fr translation for docs-v1.

Merci! :)


Still working on getChatDetails

I've added reject because if the user has no chats, the whole function fails with an unhandled error

@adamlui what exactly you want me to add/fix?

except for the docs, I'll make a commit to userguide.md once the method is completed

@adamlui what exactly you want me to add/fix?

the logic you described in the last pr details

I can check if idx is a number and use it as a list index, else I do a for-loop to match the title or id with the respective title or id in the list items

(but with the right sub-obj)

I don't know how you can work with stuff in light mode, I personally can't 💀

Some sites dark mode like GitHub really don't work for me, also there's no rush since the method is minimally viable already, you can add it as a task if I enable the Projects tab and you can get familiar with project management if you want, lmk and I'll activate it


[BEAM-9288] Not bundle conscrypt in gRPC vendor

Exclude conscrypt from the vendored grpc for now, more detail of discussion can be found in https://issues.apache.org/jira/browse/BEAM-9288.

R: @kennknowles @lukecwik

Run CommunityMetrics PreCommit

Run PythonLint PreCommit

Run Python2_PVR_Flink PreCommit

R: @lukecwik @kennknowles

@lukecwik I have checked that conscrypt is only used internally by ConscryptAlpnSslEngine, so I agree with you that it should be a runtime dependency.
@kennknowles  If we change conscrypt to runtme dependency, the concern you have will not be a problem any more.

Run Java PreCommit

Run Python PreCommit

Run Python PreCommit

Run Java PreCommit

Run Java PreCommit

Run Java PreCommit

@sunjincheng121
This PR seems not stop adding conscrypt native libraries as part in META-INF.
$ jar tvf beam-vendor-grpc-1_26_0-0.3.jar | grep conscrypt
2218176 Mon Sep 17 10:36:24 CEST 2018
META-INF/native/libconscrypt_openjdk_jni-linux-x86_64.so
1720832 Mon Sep 17 10:36:24 CEST 2018
META-INF/native/conscrypt_openjdk_jni-windows-x86.dll
2637496 Mon Sep 17 10:36:24 CEST 2018
META-INF/native/libconscrypt_openjdk_jni-osx-x86_64.dylib
2557952 Mon Sep 17 10:36:24 CEST 2018
META-INF/native/conscrypt_openjdk_jni-windows-x86_64.dll

I am not familiar with this part of build system but I am willing to help. Is there a known resolution in your mind that you can share to remove  conscrypt from META-INF?


Fix usage of NEO4J_TYPE environment variable to match documentation

Fixes #1234
This pull introduces/changes:

Correctly check the value of NEO4J_TYPE environment variable

Pings:
@cheerfulstoic
@subvertallchris

Awesome, thanks!
I generally prefer the NEO4J_URL or NEO4J_PATH variables, but this should definitely also work.  Will cherry-pick to a release


pjsip: 2.5.5 -> 2.6

Motivation for this change
Needed to bump the version for another package I want to add
Things done

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

[X] NixOS
[ ] macOS
[ ] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Thanks!


It keeps asking to install .NET Core

It shows the message, I press ok and nothing happens.
I also already have it installed, but the message still pops up.
Thank you.

I am having the same issue here. Got my new laptop today, installed the .NET Runtime, still getting the message saying it isn't installed.
My installation on my desktop computer is still working without issue.

solution: install .NET Core 3.1 Desktop Runtime
link: https://dotnet.microsoft.com/en-us/download/dotnet/thank-you/runtime-desktop-3.1.32-windows-x64-installer

was facing the same problem after uninstalling multiple .NET cores and then installing the latest one, and it complaining about not finding .NET core and the "yes to download" button not working.
decided to through code to find it needed .NET core 3.1, accidentally downloaded the entire .NET 3.1 SDK, still giving me issues running the app, but running through command line ($ dotnet "ChromiumForWindows Settings.dll") the CLI complained and gave me the exact link to download the runtime which is the one above


Combined PR for Issue Resolution / New Features

This combined pull request includes several significant changes to the project, focusing on enhancing logging, improving argument parsing, and updating package analysis heuristics. This also implements additional defensive coding and a few bug fixes. The most important changes are summarized below:
Logging and Configuration Improvements:

Added logging functionality to Combobulator, along with command line arguments for a log file and logging level. This also supports the silent mode (called quiet mode) from Issue #12

Fix syntax errors and defensive coding:

Added defensive coding to address PR #21
Fix multiline string formatting in combobulator.py
Better error handling in returns from API calls
Added pydoc and general code formatting enhancements
Fixed failure to add devDependencies on NPM package
Safety checks on dict usage from API results to gracefully handle unexpected API responses
Added short sleep on API calls to avoid throttling. Note, this can in some cases slow down execution a bit, but is "nicer" on the public APIs

Added source scanning for PyPi:

Added source scanning for PyPi using the requirements module - resolves issue #15

Package upgrades:

Upgraded commons-io:commons-io
Upgraded requests - multiple high priority vulnerabilities remediated

Added new functionality:

Added JSON output option to resolve #6
Added new exit codes and an option to exit with an error code when heuristic warnings present, to better enable integration with CI/CD
Added recursive scan option to scan sub-packages within a repository
Added new fields onto CSV (and JSON) for boolean condition checks on heuristic failures to avoid need to parse program output to determine failures

Argument Parsing and decoupling Enhancements:

Refactored argument parsing into a new src/args.py file, introducing new arguments for logging configuration, recursive scanning, and error handling.
Updated README.md to reflect the new argument options and supported package types.
Created new constants.py to avoid hardcoded and duplicate strings

Package Analysis Enhancements:

Improved heuristics in src/analysis/heuristics.py by adding detailed docstrings, refactoring function names for clarity, and incorporating additional risk checks.

Configuration Updates:

Updated .pylintrc to include an init-hook for setting the Python path. This allows for running PyLint against the project.
Added Python-specific settings to .vscode/settings.json for better development environment configuration.

And more:

See diff for full change set

Also added CONTRIBUTERS.md file (referenced in issue #4 ) - Note, probably should have more info still as I didn't know how you wanted to handle contributions overall.


add subject areas to the bidding console

this PR needs https://github.com/openreview/openreview-web/pull/1357 to be deployed first. No needed for the release of today.


Update bindings.md with 'x_k8s_get()' example

Explanation

Added a commented example on how to bind other objects with the x_k8s_get() function, then refer and assert against.
Related issue

Resolves #1897.
Proposed Changes

Documentation only, with an example test and contextual comments.
Checklist

[x] I have read the contributing guidelines.
[x] I have read the PR documentation guide and followed the process including adding proof manifests to this PR.
[ ] This is a bug fix and I have added unit tests that prove my fix is effective.

Further Comments

There is currently no documentation anywhere that I could find to describe how the x_k8s_get() function behaves, or what inputs it would take.

Yes, of course, we can split it up and rearrange however you see fit.
I just wanted to capture some working samples for this functionality, because I think it's yet another example of how powerful Chainsaw can be. 💪

@eddycharly I moved the code sample to its own page, and pointed a couple of links there. Is that structure OK?

@jlucktay you can't really do that in the functions reference docs, it is a generated page.
Can I push in your PR ?

Can I push in your PR ?

Sure, please go ahead.
I don't really have any context on the tooling used for generating and maintaining these docs, and am glad to defer to someone who does.

Ok, let's do that, thanks!

/cherry-pick release-0.2


New package: CFTime v0.0.1

Registering package: CFTime
Repository: https://github.com/JuliaGeo/CFTime.jl
Created by: @Alexander-Barth
Version: v0.0.1
Commit: ab81c7e383991081c5852d125d15bc829dd34ac8
Reviewed by: @Alexander-Barth
Reference: https://github.com/JuliaGeo/CFTime.jl/commit/ab81c7e383991081c5852d125d15bc829dd34ac8#commitcomment-34035664

Yes, thanks a lot.


Fix dependencies

Fixes #51
Am I right in assuming that only the js dependencies are required and not the types? My thinking here is that the extension appear to be compiled from ts down to js and thus the types are not needed.
To (hopefully) verify that the only thing needed are these two I ran npm install mime and npm install compare-versions inside my installed extension. Is this sufficient or is there something else to check?

Yeah that's the idea. There's not really an easy way to do it--just have to have some state on what exactly each dependency is for (is it used within the app/tool/extension/etc OR is it used as part of some sort of build/compilation step).
I should have caught this on the audio PR #48 so my apologies.

Just as a heads-up, I am currently in the process of moving and about all my computer stuffs are scattered about. My dev server is one of the few pieces of the puzzle I haven't gotten put together yet. I should hopefully be able to release a new version in a bit more than 24 hours from now, however.

Don't stress yourself too much. A move is already stressful enough.
I hope it all went well  :)


feat(strategy-sheet): 自定义趋势分析表 tooltip

👀 PR includes

✨ Feature

[x] New feature

📝 Description
自定义 趋势分析表 行/列/数值单元格 的 tooltip

修复 自定义 tooltip content 后, operation 区域 不显示的问题
🔗 Related issue link

🔍 Self Check before Merge

[ ] Add or update relevant Docs.
[ ] Add or update relevant Demos.
[ ] Add or update relevant TypeScript definitions.

:tada: This PR is included in version @antv/s2-v1.4.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-react-v1.4.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-v1.5.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-react-v1.5.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-v1.6.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-react-v1.6.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-v1.7.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-react-v1.7.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-v1.8.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-react-v1.8.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Bug Report: Compilation error

C:/HaxeToolkit/haxe/lib/linc_luajit/git//linc/linc_lua.h(34): note: see declaration of 'linc::lual' ./src/LuaCharacter.cpp(468): error C3861: 'error': identifier not found #2375
How to fix it?

haxelib remove linc_luajit
haxelib remove hxvm-luajit
haxelib git linc_luajit https://github.com/nebulazorua/linc_luajit.git
haxelib git hxvm-luajit https://github.com/nebulazorua/hxvm-luajit
have ya tried this?


pull request for e2e test at 12/24/2016 08:45:10

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Experimental

All object parsing is correctly done, just needed to adapt Handy main library to the news inputs/outputs


formatNumberConfig decimals > precision

What does this PR do and why?
Previously the formatNumber util assumed the number to format used 18 decimal places. Although this is standard, not all ERC20s use 18 decimal places, and formatting non-standard token balances requires specifying the number of decimal places to use.
The decimals property has been renamed to precision, and a new decimals property is now used to specify the decimal length of the number to be formatted.
For consistency, the WAD_PRECISION constant is now named WAD_DECIMALS.
Currently the only place this new decimals property is used is in the ERC20TokenBalance component, which reads the decimal places from the ERC20 contract.
Acceptance checklist

[x] I have evaluated the Approval Guidelines for this PR.
[ ] I have tested this PR in all supported browsers.

@peripheralist great PR, thanks for adding this clarity. Left one question.


Firefox tab-switch issue

Preconditions:
Browser: Firefox.
(Element A) - any existing element from 1-st tab
(Element B) - any existing element from 2-nd tab
Steps to reproduce:

Open 2 tab-s.
Navigate to 1-st tab search (wait_for_element_present) Element A
Navigate to 2-nd tab, search Element B
Expected result: Element found
Actual result: Element not found
Search existing element from 1-st tab by xpath
Expected result: Element not found
Actual result: Element not found
Refresh tab 2.
Search existing element from 2-nd tab by xpath
Expected result: Element found
Actual result: Element found

DOM is not loaded after switching to the tab.

Hi @Meghrikyan. It looks like the problem is Firefox/geckodriver does not wait for the new tab to load. This is a known issue or a different implementation of webdriver between Chrome and Firefox. See the following issues:

https://github.com/mozilla/geckodriver/issues/308
https://github.com/mozilla/geckodriver/issues/878
https://github.com/SeleniumHQ/selenium/issues/4570

As an example, the following test works only if I "wait_for_window_present_by_title" before switching. You ou should check that the tab finished loading as well. Should this be patched from the Golem side? probably not.
def test(data):
    navigate('https://www.web-playground.net/tabs/')
    assert_title('Web Playground - Tabs')
    click('#openTab')
    wait_for_window_present_by_title('Tab')
    assert_amount_of_windows(2)
    switch_to_next_window()
    assert_title('Tab')
    assert_element_present('#button-one')
    switch_to_next_window()
    assert_title('Web Playground - Tabs')
    assert_element_present('#openTab')

Thanks, will take into consideration for this type of tests


Add documentation regarding Namespaces field

Add doc and sample yaml in antrea-network-policy.md to describe namespaces field in ACNP.

Codecov Report

Merging #2271 (78bf230) into main (d9ec097) will decrease coverage by 20.36%.
The diff coverage is n/a.

:exclamation: Current head 78bf230 differs from pull request most recent head dc6df46. Consider uploading reports for the commit dc6df46 to get more accurate results

@@             Coverage Diff             @@
##             main    #2271       +/-   ##
===========================================
- Coverage   61.62%   41.26%   -20.37%
===========================================
  Files         279      141      -138
  Lines       21329    17375     -3954
===========================================
- Hits        13145     7170     -5975
- Misses       6811     9573     +2762
+ Partials     1373      632      -741

Flag
Coverage Δ

kind-e2e-tests
?

unit-tests
41.26% <ø> (ø)

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


[EBPF-384] migrate tracer to use ErrorsTelemetryModifier

What does this PR do?
removed ebpfTelemetry reference in tracer
Motivation
ebpftelemetry is deprecated now
Additional Notes
This change effectively adds another default modifier (printkmodifier) to the list of applied modifiers of tracer's manager
Currently, I left the ebpfTelemetry exported in the ErrorsCollector, I will make it private in the following PR, since I need to fix the tracer integration tests that still use some functions of the ebpfTelemetry struct.
Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


feat: use kuma version when installing it

In Kuma addon builder we allow setting a version, but we don't use it when calling helm install.

Codecov Report
Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (bf671d1) 60.19% compared to head (bfaba2d) 31.28%.

Files
Patch %
Lines

pkg/clusters/addons/kuma/addon.go
0.00%
3 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #949       +/-   ##
===========================================
- Coverage   60.19%   31.28%   -28.91%
===========================================
  Files          50       50
  Lines        4032     4034        +2
===========================================
- Hits         2427     1262     -1165
- Misses       1300     2591     +1291
+ Partials      305      181      -124

Flag
Coverage Δ

integration-test
31.28% <0.00%> (-28.91%)
:arrow_down:

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

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

When your PR will be merged and this https://github.com/Kong/kubernetes-testing-framework/pull/946 too I think we can release a new version


Feature request: Support for AsyncAPI

Is your feature request related to a problem?
I'd like to have support for AsyncAPI schemas
Describe the solution you'd like
It'd be nice when having both: openapi and asyncapi solutions when doing websockets and rest apis
Describe alternatives you've considered
No response
Additional Context
No response

Thanks for your feedback :raised_hands:
I'm assuming this was meant for the https://github.com/HiDeoo/starlight-openapi repo, not sure how it ended up here.
Supporting both the OpenAPI & Swagger specifications is already pretty heavy, and by looking at the AsyncAPI spec (that I'm not familiar with), it seems like it would be a lot of work to support it as well considering the differences between the three. I'm leaning towards the idea that AsyncAPI could be an entirely dedicated project/plugin at the moment.
I'll keep it in mind for the future though and depending on the interest, I might consider it. Thanks for the suggestion!


Readme should instruct on how to install certs

Hi Aurelien,
When attempting to download the housing data for chapter 2, I ran into a Python SSL error.  It wasn't too difficult to resolve, but it might save others some time and frustration if you were to include in your Readme that certs need to be installed for Python.
This is when running on MacOS, that is.
One need only run the below command:
open "/Applications/Python 3.6/Install Certificates.command"

Good point, thanks.  I added a comment in the README.md, as you suggested.


Python Example Syntax Error

Please use '#' to comment first lines of python example

Thanks. It will be fixed in the next update.

Perfectly launched.
// also works

Fixed in 1.3.0.


Fail to download objects 101-1000

When I tried to download objects 101-1000 using wget https://download.cs.stanford.edu/viscam/ObjectFolder/ObjectFolder101-1000.tar.gz, I got a 404 error: HTTP request sent, awaiting response... 404 Not Found.

There were some issues with sharing them altogether. It will be shared 100 objects by 100 objects as soon as they are ready.


naming coherence

Small change, but looking to change a few things such as this to be faster to understand. Another term I think can be changed is field, maybe call it schema, since it's where the graphQL schema is created. Let me know what you think

Thanks for the PR! Indeed, it helps! Please follow the commit message conventions we use. We forgot to include it in our CONTRIBUTING.md.

Let me know if this follows it since I've never commited with this convention

Yes! You are free to change the module name too. It's a good idea.


Error: Argument  is mandatory

Hi all and thanks for this amazing project for i3 enamorados :D
I am following on-window-detected.
What I would like to achieve is to move the mouse to newly created windows. So I attempted to put this in my aerospace.toml:
[[on-window-detected]]
if.during-aerospace-startup = false
run = ['move-mouse']

I am pretty new to toml and aerospace but when I reload the config I receive the following message (haven't tried to restart yet cause I don't want to break stuff).

What am I doing wrong?
Checklist

[X] I've searched for duplicates. My search queries were (please don't forget to search in closed issues too):

query 1
query 2
...

[X] I've read the guide https://nikitabobko.github.io/AeroSpace/guide
[X] I've searched in the documentation (https://nikitabobko.github.io/AeroSpace/guide & https://nikitabobko.github.io/AeroSpace/commands). My search queries were:

dvorak
monitor
...

aerospace CLI client version: 0.14.2-Beta 0cb8dbdfc5ee73b8cbc200f175f467ebead55201
AeroSpace.app server version: 0.14.2-Beta 0cb8dbdfc5ee73b8cbc200f175f467ebead55201

move-mouse window-lazy-center. Please see https://nikitabobko.github.io/AeroSpace/commands#move-mouse
But you won't be able to use move-mouse in on-window-detected anyway (yet) #20


Govee Integration Does Not Install

Version of the custom_component

0.2.2
Configuration
No config yet
Add your logs here.

Logger: homeassistant.config_entries
Source: config_entries.py:784
First occurred: 12:36:02 PM (3 occurrences)
Last logged: 12:43:38 PM

Error occurred loading configuration flow for integration govee: No module named 'govee_api_laggat.govee_errors'

Describe the bug
Hi, I downloaded the Govee integration via HACS, restarted, and when I go to integrations screen to add it, it does not install or prompt me for an API key.  This is on HA 2022.7.5.
Debug log

Nothing in debug logs other than message above

Update: I removed the HACS integration and Govee still shows when adding an integration...

Could this be coming from somewhere else incorrectly? I searched my filesystem for anything Govee and nothing comes up.

same for me.. using HA 2022.8.7

I have the same problem running HA 2022.9.1 and attempting to install govee 0.2.2; nothing happens when selecting Govee from integrations. It says it’s loading but eventually times out with an undefined error.

Same here, just keeps.om trying to open configuration wizard


Disable individual rules?

Is there any way to disable indivual analysis rules, without using pragma? In particular, I'd like to globally disable Serilog004, as I sometimes (purposely) use C#'s string interpolation.

Gah, I just realised I can disable them in the project properties!

Yes, you can use the standard roslyn ruleset files for that and just set the severity to none or so http://roslyn-analyzers.readthedocs.io/en/latest/config-analyzer.html

Btw. why would you want to do that, intentionally using interpolation, just for my curiosity?


chore(merge-back): 1.111.0

See CHANGELOG

AWS CodeBuild CI Report

CodeBuild project: codebuildgithubautobuildPro-fkVQbXRiQi6A
Commit ID: 60e57057127795690115fdf72c6bf1ec7d2c5437
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Add secondary authorized_keys file in sshd_config

Allows us to "reverse proxy" an ssh connection, as described in this blog post for GitLab. We can allow an un-managed public key access to ssh so as to allow the docker host user to pass the connection along.
A similar setup exists in GitLab, added in this issue.

This is already possible using https://docs.gitea.io/en-us/install-with-docker/#ssh-container-passthrough


feat: bumping version, adding prod definitions for alarms and s3 policy

Release #218, #221, #222, #223, #224 to production.
PRs are related to ElastiCache Redis alarm, S3 policy update (to not be public) and lifecycle rules for /tmp

You'll need to split it in 2 in production as well. Only release up to #221 (0.82.0 I think) to create the new output in production and then up to 0.84.0

Thanks @AntoineAugusti for the heads up and catching this.

This PR was split in two, the first part was released in https://github.com/cds-snc/notification-terraform/pull/226


acclerate support for IterableDataset

My problem is:
I have 512 JSONL files (each 1GB) and stored on the Amazon Cloud. I have 32 GPUs. I read this big dataset with IterableDataSet recommended by Huggingface so that I can train on the fly.
dataset = load_dataset("json", data_files=data_files, storage_options=storage_options, streaming=True)
But when doing training I found that the speed was very slow, especially when I use many GPU workers .
I found that the reason was because HuggingFace Trainer used DispatchDataloader to read Iterable DataSet.
It is the same with https://github.com/huggingFace/accelerate/issues/158.
Is there a good solution for my problem?
One of the solutions I think is to divide the 512 JSONL files to 32 GPU workers when building the dataset. So each GPU Worker only accesses the corresponding 16 JSONL file. it seems that I should shard the dataset manually in my code according to gpu worker id, and I should not pass the dataloader to accelerator.prepare().
Can HuggingFace Trainer currently support this way of data loading? Or is there any other way to deal with my problem.

cc @muellerzr

@muellerzr hello, looking forward to your reply.

...with something news?


Alligning with Martin Günther

It seems you are doing good work on porting these packages. I saw that @mintar (Martin Günther) is also working on a ros2 branch. He has been merging some of your commits, but have also contributed a good deal himself. Are you aware of his work, could it make sense to collaborate?

Thanks for your work Martin!
I have forked your repo and cherrypicked most of @relffok 's commits on her branch here:
https://github.com/AAU-RoboticsAutomationGroup/mir_robot
But it sounds like you have a lot local changes and plans, so I don't know if the work in my fork can help.

Hello! Thank you very much for your work on this project! I'm testing the repository that @kdhansen  presented on a physical robot, I'm having some difficulties and would like to know if it's possible to enable the issuis tab in the repository AAU- RoboticsAutomationGroup/mir_robot so that doubts are concentrated in the test repository.

I'm having some difficulties and would like to know if it's possible to enable the issuis tab in the repository AAU- RoboticsAutomationGroup/mir_robot so that doubts are concentrated in the test repository.

There you go :)


Simplify build-images.yml file to use the default docker args.

In a previous PR we resolved which default images should be using the correct base images for our outward facing docker files.
This change updates the build-images.yml to use the default args instead of secrets set on the repository. Since the action is to validate the images can be built using the default ubuntu:20.04 image from docker hub should be ok.
Please let me know if there are any issues with this change. If the change is acceptable, we can also remove some of the secrets set for GH actions from the repo.

Let's hold off merging this one until Bruno has a chance to take a look.
Thanks.


[Feature] Add existing to CloudinaryUploadWidgetInfo interface

Feature Request
Is your feature request related to a problem? Please describe.

I wouldn't say it is necessarily a problem it is just missing from the interface but appears in the cloudinary upload response.
Describe the solution you'd like
Add existing: boolean to CloudinaryUploadWidgetInfo

Describe alternatives you've considered

No alternatives considered.
Additional context

hey thanks @MateoWartelle - just to be clear
on this: https://github.com/colbyfayock/cloudinary-util/blob/main/packages/types/src/types/cloudinary-upload-widget.d.ts#L117
add:
existing: boolean;

correct?
can you show me an example response and how you got it? does this occur when you upload the same asset 2 times with overwrite: false? or perhaps by trying to set the public ID manually?


init.d scripts do not work on reboot

There seems to be a conceptual problem with the init.d scripts (using version 3.5.0 of the module):
The start script will store a cid-file at /var/run. However, this is a temporary filesystem (at least with Ubuntu) that will be recreated at reboot. This means that the cid-files will be lost.
The start scripts now have an if-else that checks for the presence of the cid-file. On a reboot, this goes to the else-part that tries to execute docker run. This will fail with a duplicate name error if the container is named and has only been stopped, not removed (which is quite a common case imho).
The init scripts should be smart enough to check for a stopped (named) container, even if the cid-file doesn't exist.

@pazoozooCH the cid-file approach turned out to be quite brittle for a number of reasons and has not been replaced in the 4.x series. I'll close this issue as I think it should be fixed now, but if you recreate on the latest release let me know.


0 reference(s) processed

Crawling a site, I get: 0 reference(s) processed, but I expect the initial url to generate some references.
Some information from the configuration file:
  #set($filterRegexRef  = "com.norconex.collector.core.filter.impl.RegexReferenceFilter")
...
  <urlNormalizer class="com.norconex.collector.http.url.impl.GenericURLNormalizer">
  <canonicalLinkDetector
    class="com.norconex.collector.http.url.impl.GenericCanonicalLinkDetector"
	  ignore="true">
  </canonicalLinkDetector>
...
      <startURLs>
        <url>http://www.bancodesangreextremadura.blogspot.com/p/</url>
      </startURLs>
...
      <referenceFilters>
        <filter class="$filterRegexRef" onMatch="include">http://www\.bancodesangreextremadura\.blogspot\.com/p/.*</filter>
      </referenceFilters>

I've tried the filter ending with p/.*and p/*
The result from the log is:
MC (crawler): - Running MC (crawler): BEGIN (Mon Aug 03 18:38:25 CEST 2015)
MC (crawler): - Initializing reference store C:\CRAWLER-MC\collectors\MC\work/crawlstore/mapdb/MC_32__40_crawler_41_/
MC (crawler): - C:\CRAWLER-MC\collectors\MC\work/crawlstore/mapdb/MC_32__40_crawler_41_/: Done initializing databases.
MC (crawler): - MC (crawler): RobotsTxt support: false
MC (crawler): - MC (crawler): RobotsMeta support: true
MC (crawler): - MC (crawler): Sitemap support: false
MC (crawler): - MC (crawler): Canonical links support: false
MC (crawler): -           CRAWLER_STARTED
MC (crawler): - MC (crawler): Crawling references...
MC (crawler): -         REJECTED_NOTFOUND: http://www.bancodesangreextremadura.blogspot.com.es/p/
MC (crawler): - MC (crawler): Crawler finishing: committing documents.
MC (crawler): - MC (crawler): 1 reference(s) processed.
MC (crawler): -          CRAWLER_FINISHED
MC (crawler): - MC (crawler): Crawler completed.
MC (crawler): - MC (crawler): Crawler executed in 2 seconds.
MC (crawler): - Closing reference store: C:\CRAWLER-MC\collectors\MC\work/crawlstore/mapdb/MC_32__40_crawler_41_/
MC (crawler): - Running MC (crawler): END (Mon Aug 03 18:38:25 CEST 2015)

I'd appreciate some guidance

The problem is not with your filter. REJECTED_NOTFOUND happens because the page does return a 404 response code when accessing it.  See here the HTTP response I get when accessing http://www.bancodesangreextremadura.blogspot.com.es/p/:
HTTP/1.1 404 Not Found
Content-Type: text/html; charset=UTF-8
Cache-Control: no-cache, no-store, max-age=0, must-revalidate
Pragma: no-cache
Expires: Fri, 01 Jan 1990 00:00:00 GMT
Date: Mon, 03 Aug 2015 19:02:59 GMT
Content-Encoding: gzip
X-Content-Type-Options: nosniff
x-xss-protection: 1; mode=block
Content-Length: 11223
Server: GSE
Alternate-Protocol: 80:quic,p=0

You may contact the webmaster to find out why they throw a 404 there.  Maybe it is a strategy to fool crawlers?  If you want to crawl 404 pages, you can make it a valid code with this setting:
  <documentFetcher
      class="com.norconex.collector.http.fetch.impl.GenericDocumentFetcher">
      <validStatusCodes>200, 404</validStatusCodes>
      <notFoundStatusCodes>999</notFoundStatusCodes>
  </documentFetcher>

I have put 999 as the notFoundStatusCode just in case the crawler does not like it to be empty. It has no meaning.

Thank you for your answer. It's really my fault. I tried the URL with chrome and didn't realized the 404 error because I only saw on the page the different options to choose from, so I expected the crawler to generate references.


Return to updating istio and client-go

We are no longer blocked by cachito.

@jwendell: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_sail-operator_main
a2f324cc7526930b650104d187f26376cfba744c
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

/retest


Cache eviction should call resetStickyTaskList

Checked with @venkat1109 , and found the feature is already supported in the current system.


use Builder.byteString

Could you run
cabal run tie-tests -- --accept

This will generate the expected output for the golden tests


DirectXShaderCompilerDxc

RPM: https://github.com/nicholas-rh/o3de-rpm/tree/master/copr/directx-shader-compiler
Copr: https://copr.fedorainfracloud.org/coprs/nfrizzel/directx-shader-compiler/

The bundled O3DE DXC package has Ubuntu-specific linkage that the package manager complains about, so we have to build our own and link to it instead. The package I made to do so is linked above. There are also some changes necessary to work with the assumptions made by O3DE to symlink to and use our version of the DXC executables instead of the bundled version: https://github.com/nicholas-rh/o3de-rpm/blob/ba85072e74feeca867ef3637e3b536cec6d14f4f/copr/o3de/o3de.spec#L273
A patch to allow for using this system library is included here: https://github.com/nicholas-rh/o3de-rpm/blob/ba85072e74feeca867ef3637e3b536cec6d14f4f/copr/o3de/SystemPackages_linux.patch#L81


Convert to Python3

using : find . -name '*.py' | xargs 2to3 -w

Coverage decreased (-0.2%) to 99.813% when pulling ec6f6593789734bcaad9612f8c944a0f4f52151a on Python3pkg:master into e13869dec945f95287aad953038778ac038c9834 on glyph:master.

Coverage decreased (-0.2%) to 99.813% when pulling ec6f6593789734bcaad9612f8c944a0f4f52151a on Python3pkg:master into e13869dec945f95287aad953038778ac038c9834 on glyph:master.

Coverage decreased (-0.2%) to 99.813% when pulling ec6f6593789734bcaad9612f8c944a0f4f52151a on Python3pkg:master into e13869dec945f95287aad953038778ac038c9834 on glyph:master.


500 server response and app error, when updating a subscription with an invalid card number

When I create a subscription with a valid card number (eg:<PHONE_NUMBER>424242 - from stripe's test card numbers (https://stripe.com/docs/testing#cards)) and then go back and update the card details with an invalid card number (again, from stripe's test card number list eg:<PHONE_NUMBER>000002)... I get a 500 response and then an app error...
Here's the server logs:
method=POST path="/koudoku/owner/363/subscriptions/12" host=XXXXXX.herokuapp.com request_id=XXXXXXXX fwd="XXXXXX" dyno=web.1 connect=1ms service=2406ms status=500 bytes=1799

Hi @AlanDonohoe
can you please provide a stack trace?
How do you include koudoku in your Gemfile? Please copy&paste the line

no further details provided, closed


Activate TravisCI?

Hi,
I note that there's a .travis.yml in the repo, but navigating to TravisCI informs me that 'This is not an active repository.'
@nedbat : would you like to use Travis, and if so, would you mind activating it? Happy to help in any way I can.

I activated it for my Travis account, I think?

I'll admit to being confused about the difference between webhooks and integrations, travis-ci.org and travis-ci.com, etc.
Will this work?  https://travis-ci.org/nedbat/django_coverage_plugin

That sure looks like it works to me!
I get the impression Travis is consolidating .com and .org, but that's it's taking a ton of work. I expect everything to move to the new Github integration, but it doesn't work for OSS yet.
I'm closing the issue, as I think we're good to go, but I'll re-open it if tests don't run on PRs.
Thanks Ned!


Maybe improve design of application by using sync primitives

I'm a bit unhappy with the current structure: we pass an immutable reference of one module (not Rust module) to all other modules that need to use it. For example, the current main loop looks like:
try!(self.renderer.render(&self.world_view, &self.player.get_camera()));
let event_resp = self.event_manager
    .poll_events(vec![&mut CloseHandler, &mut self.player]);

We pass the world and the camera to the renderer, as well as all event receivers to the event manager. The renderer needs a whole lot more things in the future, like the weather system and the sky system; so we can expect the function signature of render() to increase in argument count. The event manager would also be nicer with methods like add_handler().
The problem here is that we, of course, can't pass immutable references (or in the case of the event manager: mutable ones) around while still mutating the object on our own. A possible solution would be to use an RwLock (taking into account that we will have threads in the future).
But the question is: what is more rusty, more efficient and the better design? The current design works and the borrow checker doesn't complain. The RwLock design also adds some complexity and introduces some overhead for locking. Maybe it's not that bad to have terrible long argument lists. I'm really not sure about this (?). Maybe at the very least we should wrap the World into an RwLock and give it to everyone?
I'm curious what you think, especially @jonas-schievink ...

If we really want to allow full access to everything from multiple threads, we can just use specs for the game. Using an ECS is overkill, though.
I don't think anything needs to be mutated by multiple threads, so an RwLock won't be necessary. When #57 is resolved, we'll just put a small loop that drains a Receiver and changes the World into the main loop. #58 can also be resolved by simply passing a &mut World.
Giving an add_handler method to the event manager also has the problem of running into borrow checker and lifetime inference problems, which I'd like to avoid for now, so we'd have to pass an Rc<RefCell<...>>. I think the current solution is good enough without requiring any major borrowck gymnastics.

Using an ECS is overkill, though.

I agree.

I don't think anything needs to be mutated by multiple threads, so an RwLock won't be necessary.

Whether or not we need an RwLock or Mutex does not depend on how many threads mutate the world, but how many threads have access to the world, right? If we only have one not-main thread (which doesn't access the world at all), then of course you're right and we can avoid the little overhead of atomic operations.

When #57 is resolved, we'll just put a small loop that drains a Receiver and changes the World into the main loop.

I agree, I also planned on doing exactly that.

#58 can also be resolved by simply passing a &mut World.

To what? I'm not sure I understand what you are suggesting. #58 is also about updating the view.
The world can potentially mutated at multiple spots in our program. One important example is the player's interaction with the world. Currently, when a click event is forwarded to the player via the EventHandler trait. The handle_event() methods receives &mut self, but nothing else. To mutate the world on a click we could

add a &mut World parameter to the handle_event() method
queue the click event within the player and pass &mut World to Player::update() to apply the event later
Save a RefCell<World> (or similar) within the player to allow mutating the world via &mut Player (even &Player ...)

I don't like (1.) because it again mixes different concerns and grows function signatures. I don't like (2.) because this makes our code more complex than it needs to be and everyone thinks Rust is annoying :stuck_out_tongue:
So again, I think that we should pass at least some inner-mutability-containers around to avoid those problems.

Well, we wouldn't need a queue of click events, just a bool flag (or a few)
would be enough. But other than that you're right.
Lukas Kalbertodt<EMAIL_ADDRESS>schrieb am Sa., 23. Juli 2016,
12:10:

Using an ECS is overkill, though.
I agree.
I don't think anything needs to be mutated by multiple threads, so an
RwLock won't be necessary.
Whether or not we need an RwLock or Mutex does not depend on how many
threads mutate the world, but how many threads have access to the world,
right? If we only have one not-main thread (which doesn't access the
world at all), then of course you're right and we can avoid the little
overhead of atomic operations.
When #57 https://github.com/OsnaCS/plantex/issues/57 is resolved, we'll
just put a small loop that drains a Receiver and changes the World into
the main loop.
I agree, I also planned on doing exactly that.
#58 https://github.com/OsnaCS/plantex/issues/58 can also be resolved by
simply passing a &mut World.
To what? I'm not sure I understand what you are suggesting. #58
https://github.com/OsnaCS/plantex/issues/58 is also about updating the
view.
The world can potentially mutated at multiple spots in our program. One
important example is the player's interaction with the world. Currently,
when a click event is forwarded to the player via the EventHandler trait.
The handle_event() methods receives &mut self, but nothing else. To
mutate the world on a click we could

add a &mut World parameter to the handle_event() method
queue the click event within the player and pass &mut World to
Player::update() to apply the event later
Save a RefCell (or similar) within the player to allow
mutating the world via &mut Player (even &Player ...)

I don't like (1.) because it again mixes different concerns and grows
function signatures. I don't like (2.) because this makes our code more
complex than it needs to be and everyone thinks Rust is annoying 😛
So again, I think that we should pass at least some
inner-mutability-containers around to avoid those problems.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OsnaCS/plantex/issues/56#issuecomment-234710712, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABtCRjoX5HJ93ow6Cs-pXkleOi0Z0vJNks5qYeikgaJpZM4JTUJJ
.

--
Gruß, Jonas

This was tackled in #71. While we might want to improve the design in the future, I'll close this issue for now.