Add automatic linting and formatting requirements to code base

This is a continuation of #2478 to add some more substantial framing for how we handle code structure in the boto projects. This PR is currently a draft for comment and may be broken up into smaller parts for easier review. The current proposal adds 3 new tools to ensure we're cleaning up old code and keeping what we have currently consistent. This also helps expedite PR review by removing the need for iterations on minor style fixes.
Proposed additions:

~isort - This will order our imports and ensure consistent layout. This avoids duplicate imports by keeping everything in the same spot and makes it easier to find what's already available.~ (#2642)
pyupgrade - This will automatically "upgrade" existing code to use the feature set configured for the lowest supported version (in this case Python 3.6). This is a more significant change that most version bumps because it's cleaning up all of our Python 2.6, 2.7 and 3.0-3.5 code.
black -  This is the last one and maybe most contentious. We're adopting black with two caveats, we're not going to enforce quote normalization for now, and we're going to keep line length at 79 for the time being.

To jump into the conversation head first: Have you thought about adding a .pre-commit-config.yaml with pre-commit checks for e.g. isort and black, and enabling pre-commit.ci on the project? The former (yaml) would provide a standard way for users to keep their code formatted and linted consistently/locally, and the latter would use the same yaml to ensure pull requests painlessly maintain the same standards.
pre-commit.ci is free for open source projects, so I'd hope the administrative overhead of trying it out would be cheap. (Of course, I'd hope AWS would be willing to chip in and support that awesome project financially, too.)

This is great feedback, @kojiromike! We were talking about something similar last week but hadn't dove into options. We'll take a look at whether this will fit our needs. Thanks!

Cross linking with boto/boto3#2959.

Resolving this now that the last PR is merged in #2675.


Add Extra.find

Per @evancz's comment on my previous PR into core, here's a PR to add find, a useful function that can be found in several other standard libraries:

JavaScript
Ruby
Scala
Haskell

This looks handy, thanks @rtfeldman.
P.S. perhaps I can give you (and anyone else you'd like at NoRedInk) push access to the *-extras packages? Since we're a small bunch of commercial users I think it might worth keeping on eye on each other's projects to see if there's any overlap.

Published as 3.1.1

@rehno-lindeque Sounds good! :+1:


[DAT-16006] Fixing v8 Checksum calculation issue when using sql dbms attribute

Impact

[X] Bug fix (non-breaking change which fixes expected existing functionality)
[ ] Enhancement/New feature (adds functionality without impacting existing logic)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
Checksums (v8) are calculated differently for XML changesets with the dbms attribute on the change itself (instead of the changeset) after 4.22.0 .
Fixes #4665
Fixes #4457

As noted in https://github.com/liquibase/liquibase/issues/4665#issuecomment-1741689649 this seems to resolve the issue in #4665 🙏


"Error while parsing JSON body" with "responseType: 'auto'" if No Content is successfully returned

Hello,
If the sucess response of a GET method is

HTTP/1.1 204 No Content

Qwest throws the following exception:

Error while parsing JSON body : SyntaxError: Unexpected end of input

Example code:
var url = 'http://crunchbase.linkurio.us/api/auth/logout';

qwest.get(url, null, {
  cache: true,
  responseType: 'auto'
})
.catch(function(e) { throw e; });

I'll investigate this ASAP ;)

Fixed withe next release


⚠ Bofrost.at has degraded performance

In 5afe73f, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1051 ms

Resolved: Bofrost.at performance has improved in 5d89b7f.


AWS Phone System Should Not Listen on Public Pages

Describe the bug
This is a part of a larger
To Reproduce
Steps to reproduce the behavior:

Go to any public, unauthenticated page.
Open the console. The AWS Connect API is pinging every 5 seconds, even though you're not logged in.

Expected behavior
Pinging should NOT occur on the public website.
Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Previously (I dont believe) this was possible due to the way the connect streams integration works, but I tried it again and it's working fine now. Pretty sure SAML auth "fixed" it.

...even though you're not logged in.

Just for reference, the "pinging" is actually connects way OF logging in (or at least, checking if the current user has a live session to resume).

Ok, that makes sense, but there is no reason to do it on unauthenticated pages, or with unauthenticated users, right?

@aarontitus Previously I could only seem to get the connect API to work when initializing it from the root domain, so I needed to do it as early as possible. But somewhere along the line since then the issue no longer appears to be the case, and I suspect it has something do with the SAML auth that replaced the original, connect centric auth.

Let me also clarify that's it's not "us" persay that do the pinging. Once the connect API is initalized it automatically begins pinging until a user is authorized.


Unable to run ps1 scripts that are in path if they start with a number

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce

Name the script starting with a number.
E.g. 20220611-robocopy.ps1, where 20220611 signifies creation date.
Add the script's path to path.
Run ./20220611-robocopy.ps1 from a different location.

Expected behavior
No errors.

Actual behavior
./20220611-robocopy.ps1: The term './20220611-robocopy.ps1' is not recognized as a name of a cmdlet, function, script file, or executable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

### Error details

```console
PS > Get-Error

Exception             :
    Type        : System.Management.Automation.CommandNotFoundException
    ErrorRecord :
        Exception             :
            Type    : System.Management.Automation.ParentContainsErrorRecordException
            Message : The term './20220611-robocopy.ps1' is not recognized as a name of a cmdlet, function, script file,
or executable program.
                      Check the spelling of the name, or if a path was included, verify that the path is correct and
try again.
            HResult : -2146233087
        TargetObject          : ./20220611-robocopy.ps1
        CategoryInfo          : ObjectNotFound: (./20220611-robocopy.ps1:String) [], ParentContainsErrorRecordException
        FullyQualifiedErrorId : CommandNotFoundException
        InvocationInfo        :
            ScriptLineNumber : 1
            OffsetInLine     : 1
            HistoryId        : 1
            Line             : ./20220611-robocopy.ps1 -Restore
            PositionMessage  : At line:1 char:1
                               + ./20220611-robocopy.ps1 -Restore
                               + ~~~~~~~~~~~~~~~~~~~~~
            InvocationName   : ./20220611-robocopy.ps1
            CommandOrigin    : Internal
        ScriptStackTrace      : at <ScriptBlock>, <No file>: line 1
    CommandName : ./20220611-robocopy.ps1
    TargetSite  :
        Name          : LookupCommandInfo
        DeclaringType : System.Management.Automation.CommandDiscovery, System.Management.Automation,
Version=<IP_ADDRESS>0, Culture=neutral, PublicKeyToken=31bf3856ad364e35
        MemberType    : Method
        Module        : System.Management.Automation.dll
    Message     : The term './20220611-robocopy.ps1' is not recognized as a name of a cmdlet, function, script file, or
executable program.
                  Check the spelling of the name, or if a path was included, verify that the path is correct and try
again.
    Data        : System.Collections.ListDictionaryInternal
    Source      : System.Management.Automation
    HResult     : -2146233087
    StackTrace  :
   at System.Management.Automation.CommandDiscovery.LookupCommandInfo(String commandName, CommandTypes commandTypes,
SearchResolutionOptions searchResolutionOptions, CommandOrigin commandOrigin, ExecutionContext context)
   at System.Management.Automation.CommandDiscovery.LookupCommandProcessor(String commandName, CommandOrigin
commandOrigin, Nullable`1 useLocalScope)
   at System.Management.Automation.ExecutionContext.CreateCommand(String command, Boolean dotSource)
   at System.Management.Automation.PipelineOps.AddCommand(PipelineProcessor pipe, CommandParameterInternal[]
commandElements, CommandBaseAst commandBaseAst, CommandRedirection[] redirections, ExecutionContext context)
   at System.Management.Automation.PipelineOps.InvokePipeline(Object input, Boolean ignoreInput,
CommandParameterInternal[][] pipeElements, CommandBaseAst[] pipeElementAsts, CommandRedirection[][]
commandRedirections, FunctionContext funcContext)
   at System.Management.Automation.Interpreter.ActionCallInstruction`6.Run(InterpretedFrame frame)
   at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)
TargetObject          : ./20220611-robocopy.ps1
CategoryInfo          : ObjectNotFound: (./20220611-robocopy.ps1:String) [], CommandNotFoundException
FullyQualifiedErrorId : CommandNotFoundException
InvocationInfo        :
    ScriptLineNumber : 1
    OffsetInLine     : 1
    HistoryId        : 1
    Line             : ./20220611-robocopy.ps1 -Restore
    PositionMessage  : At line:1 char:1
                       + ./20220611-robocopy.ps1 -Restore
                       + ~~~~~~~~~~~~~~~~~~~~~
    InvocationName   : ./20220611-robocopy.ps1
    CommandOrigin    : Internal
ScriptStackTrace      : at <ScriptBlock>, <No file>: line 1

Environment data
PS > $PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.2.4
PSEdition                      Core
GitCommitId                    7.2.4
OS                             Microsoft Windows 10.0.22000
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

Run ./20220611-robocopy.ps1 from a different location.

This is current directory.

This is most likely because PowerShell is trying to treat is as a command since it has a hyphen. You can workaround this by doing one of the following things.
# Run using the full path name instead of 20220611-robocopy.ps1
 ./20220611-robocopy.ps1

# Place a non-number character before the hyphen
20220611a-robocopy.ps1

# Remove the hyphen
20220611_robocopy.ps1


ready to support http client timeout feature

I am supposing that @bbansal is little bit busy. So, I have done some changes as per @vincentwoo 's suggestions on @bbansal pull request. We need HTTP client support as soon as possible.

Made a couple doc-related comments, seems good to go once you fix those.

I have added description. I am supposing now it's ready to merge.

These docs are really bad and the tests don't even pass (did anyone even run them?). I'm closing this for now, but I'll try to make my own commit for passing args to httpclient.

I haven't run test cases. I was tried to full-fill your requirement over
@bbanasl pull request.
Sure do it your own and publish soon.
Thanks
On 10-Nov-2015 12:53 pm, "Vincent Woo"<EMAIL_ADDRESS>wrote:

Closed #48 https://github.com/oscardelben/firebase-ruby/pull/48.
—
Reply to this email directly or view it on GitHub
https://github.com/oscardelben/firebase-ruby/pull/48#event-459784500.


Houdini: Move get_current_context_template_data function to a proper place

Changelog Description
Moving get_current_context_template_data to Houdini lib as it's Houdini specific.
This PR was made in response to this comment https://github.com/ynput/OpenPype/pull/5651#discussion_r1377949974
Testing notes:

everything should work the same in Houdini settings, specifically:

Update Houdini Vars on context change
Shelves Manager

I suspect this PR tries to fix the same as this PR: https://github.com/ynput/OpenPype/pull/5845

I suspect this PR tries to fix the same as this PR: https://github.com/ynput/OpenPype/pull/5845

Thanks!
tbh, I hate Github notifications! I really don't why I didn't see that yesterday!

Closing this PR in favor of https://github.com/ynput/OpenPype/pull/5845 .


Specific metrics do not appear to be stored

We are working on a validation tool for teams to use that ensures metric are being emitted properly. Most metrics for our source can be found in the read endpoint, but health.check.cliCommand.probe.count does not appear in v1.4.1. It does appear in the firehose. The value of this metric is 0 in our specific environment which may be related.
In the screen shot we are using cf tail on the left and cf nozzle on the right.

@joerodriguez could you confirm that you're using the latest version of the CLI

We are using the newest version. Additionally we checked the /read and /promql endpoints directly.

@joerodriguez We were assuming that this had something to do with zero-value metrics, but we were still able to see zeroes coming through in our testing. Are you free for some XTP so that we can observe this in your environment and try to figure out where the issue is originating?
/cc @pianohacker

@toddboom @colins we ran into a similar issue on Loggregator with gauge metrics on the RLP Gateway. The RLP Gateway is not using the grpc-gateway however was converting protobuf envelopes to JSON with jsonpb which omits zero values default.
If you are able to set the marshaler for the grpc-gateway you should be able to turn on EmitDefaults which will emit the zero values. You can reference our fix for the RLP Gateway here.

@bradylove thanks for the tip. i thought we had already done this here a while ago:
https://github.com/cloudfoundry/log-cache/commit/df05c6457305c8b633f00cc9e4a8cbfa89e2cf03
but i will take a look and see if it's happening somewhere else that we missed.

This is fixed in master.


Outline of atbash-cipher exercise.

Got the runner working with a few unit test inputs. Example implementation is currently empty.
This is still a work-in-progress. I think the runner should work fine, but I haven't put any implementation in the example file yet. @ozan feel free to have a go at an implementation if you want :)
Need to add more unit test input from here: https://github.com/exercism/x-common/blob/master/atbash-cipher.json
And need to decide whether to implement decode function as well.
Haven't updated config.json yet either.

Added an implementation that passes the test. It's not particular pretty though... I'm sure it could be improved!

Very cool! @ozan do you have time to review this?

Yup, just heading back from holidays so can review it in a day or two
On Sunday, September 4, 2016, Katrina Owen<EMAIL_ADDRESS>wrote:

Very cool! @ozan https://github.com/ozan do you have time to review
this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/exercism/xmips/pull/15#issuecomment-244632190, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAG6z9I81Lfk2KNBtYyBNBcGn8LJP8Vdks5qm0U9gaJpZM4JyDu1
.

@mpriestman thanks for this! Reviewed and looks great, merging.


[FEATURE] Persistent choices in the dialogs

When uploading a batch of videos and checking on the results, it would be wonderful if my selections for fps, number of rows to display in the data view, etc., could be persistent, rather than resetting each time.

Context: for example, for DIVA, it's always 30 fps, so Roddy wants a way to set that user-wide.

Other model is: when a user updates this value once, that value becomes "sticky" for the next time.

Looking into this and doing some planning:

Desktop and Web could both use persistent annotationFPS settings as well as a data display row number setting.  Gives me justification for using the dive-common/store/settings
This could be stored in the existing clientSettings, just need to make the generic watcher for all of clientSettings occur outside of the specific setup function used by Viewer.vue which is clientSettingsSetup which is passed the types for the unknown category to be updated.
For DataBrowser settings will require some modification on girder-web-components to create a .sync with initialItemsPerPage within the data browser.  This will allow changes to the itemPerPage within that browser to be stored and saved in the localStorage.  Once that is done we should have persistent settings across everything.  Need to check about how others feel with having added update:initalItemsPerPage to girder-web-components.
We should probably unify all of the dropdown row-number settings between SharedDatasets, Dataset Browser (web and desktop) so they all use the same dropdown list (10, 30, 50)? or (10, 50, All)?  I believe they are using different numbers now which may make sense (given that Shared may be lower), but if we want to persist the setting I think we need to make them the same.


Predicted ME search center for bottom-up partition rdo

AWCY results for 5 frames at speed 0

PSNR
PSNR Cb
PSNR Cr
PSNR HVS
SSIM
MS SSIM
CIEDE 2000

-1.8632
-1.7386
-1.7199
-1.8155
-1.8189
-1.7893
-1.8325

If anyone objects to landing this patch, pls let me know in 1 hour! Otherwise, it will land!

Copying derf's response from #daala IRC, since he seems not using gihub and assume LGTM from him
1:34 PM  derf: It seems predicted ME search center also works fine for bottom-up (i.e. speed 0) partition search. Got 1.86% for 5 frames.
2:33 PM <+derf> codeview: Bottom-up propagation or top-down propagation in a bottom-up search?
2:34 PM  top-down propagate in bottom-up earch
2:34 PM  bottom-up propagate is what I expected would not work so deferred.
2:41 PM <+derf> codeview: Glad to hear it works.

Yes, r=me.
P.S., I block all e-mail from Github since it spams me way too much for projects I'm not even contributing to, so if you want me to see something, please say so.


Badge properties in details component not honoured

Current Behavior
When the badge link text badgeLinkText for example is set on a details component, it is no longer set correctly in componentProperties. Instead, the page title gets set which is not how it was previously functioning.
Expected behaviour/code
badgeLinkText should use the value at the highest order which in the case of a list is the details component.
Environment

AEM 6.5.7
Core Components v2.2.0
Java 11

Turns out this specific issue is caused by https://github.com/aem-design/aemdesign-aem-core/blob/master/aemdesign-aem-services/src/main/java/design/aem/models/v2/details/ContactDetails.java#L50.
Debugged the props loop and everything checks out so happy to close this and fix Contact Details.


Interceptor path is hardcoded!

def interceptor(request):
    if request.url == 'https://server.com/some/path':
        request.create_response(
            status_code=200,
            headers={'Content-Type': 'text/html'},  # Optional headers dictionary
            body='<html>Hello World!</html>'  # Optional body
        )

driver.request_interceptor = interceptor
driver.get(...)

# Requests to https://server.com/some/path will have their responses mocked

This is documentation about mock response, line:
if request.url == 'https://server.com/some/path'
is hardcoded!
A lot of requests have dynamic values in them. In my case, the request has node_id coded into it, and my test gets that node_id value in runtime!
Your documentation, in my opinion, has knowingly hardcoded paths in every place!
I need to find a request with a dynamic value in it, and mock the response, to check if the browser properly displays data, that normally takes a month to get. Tried to use a lambda expression, but your validator refuses any other arguments in the interceptor method! For me, thats a wall I cannot pass now.

Hi, this issue tracker is for issues with the codebase behind https://pypi.org itself, not the projects hosted on PyPI.
You should report this issue to the tracker for the project in question instead.


o1-preview support？

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

I think it's because o1-preview is not yet returned from this endpoint:
https://api.openai.com/v1/models

yes, mine got an error as well
Error message: Oops! There was a problem connecting to the O1-mini.
External: 400, message='Bad Request', url='https://aihubmix.com/v1/chat/completions


Support for StmtCursor and CursorPlan

cursorPlan.sqlplan.txt
Hello Justin,
First of all, I have to say - great idea! This is a very useful tool, especially since the only way of viewing a SQLPLAN is using SSMS.
The question I have for you is whether you're planning to support the whole array of elements in the the schema: http://schemas.microsoft.com/sqlserver/2004/07/showplan/
In the attached file, you can see elements such as StmtCursor and CursorPlan, which don't seem supported in your XSLT.
Thanks,
Costin

This is mostly completed, just need icons, plus description tooltips for Keyset and SnapShot cursor types, as well as for PopulateQuery and RefreshQuery operation types.


Add Jenkins build scripts

Let's see if we can make this work...

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please


Support filtering unread conversations WIP

This makes it so if the filter starts with @unread we'll limit the filtering to convos with unread messages. I had to tweak markAsRead handling so we don't mark them as read while arrowing through the list. Hooks up cmdKey+R to focusing + prefilling the filter with @unread . r? @keybase/react-hackers

Sorry @zanderz I should've commented, this is gonna go a different direction after a talk irl. Instead of @unread and a new shortcut it'll default to showing unread conversations first if no filter is set.

Done by inbox search


feat(consensus): clique

Implementation of Clique proof-of-authority consensus protocol.
This PR includes:

Snapshot voting
Header and block validation
Clique header seal

This PR does not include:

Forward downloader
Clique instantiation
Forckchoice tracker

Where do we need this & how?

hive doesn't test clique explicitly, but it's required to be able participate in rpc test suite

The main way I'd be convinced into merging this as a large chunk would be if we actually had a Hive test which did not pass w/o this PR and passed with this PR. So let's try to get there?

@gakonst we did agree on it, just didn't finalize the scope

closing as stale and not planned for now

as assumed we need this


Fixes 2 bugs regarding connections

NetworkServer.cs was not clearing connections on stopping. This meant when then starting again it was trying to communicate with old connections. Simply clear the list in DisconnectAllConnections. Also put in InternalListen for just in case as there should never be any connections at that point.
NetworkClient.cs was unable to load the scene on connect under certain circumstances. This was because NetworkClient.pauseMessageHandling had been set to false on a previous failed connection attempt. On a subsequent connect it would then never accept messages after it connected successfully so didnt know to load the scene. Simple fix to set NetworkClient.pauseMessageHandling to false when connecting. It should never be true at this point.

Codecov Report

Merging #128 into mirror will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           mirror    #128      +/-   ##
=========================================
- Coverage    6.48%   6.48%   -0.01%
=========================================
  Files          29      29
  Lines        6257    6260       +3
  Branches      867     867
=========================================
  Hits          406     406
- Misses       5838    5841       +3
  Partials       13      13

Impacted Files
Coverage Δ

Mirror/Runtime/NetworkServer.cs
0% <0%> (ø)
:arrow_up:

Mirror/Runtime/NetworkClient.cs
0% <0%> (ø)
:arrow_up:

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 0b64797...bcb6a60. Read the comment docs.


Payout  Pending Mouse Over needs to be changed to 60 confirmations

Currently says 100 confirmations. This is changed in 4.0
Please have it say 60 confirmations and ensure looking for spendable inputs with 60 and not 100.

This is completed on the SportsBook Explorer Middleware

This is completed on the SportsBook Explorer Middleware

Is not complete on front end

Is not complete on front end


data unzip error

thanks for your great job!
I  got some error when I untar data package  FOREGROUND_6NN_WITH_DPETHS_sweeps.tar.gz.
Here is my command,
tar xvf FOREGROUND_6NN_WITH_DPETHS_sweeps.tar.gz
After minutes, there is something wrong happened. Could you repack data set?
gzip: stdin: invalid compressed data--crc error
tar: Child returned status 1
tar: Error is not recoverable: exiting now

You can try the following two solutions. 1. Use the command tar -zxvf FOREGROUND_6NN_WITH_DPETHS_sweeps.tar.gz to unzip the downloaded data. 2. Check the md5 code of your downloaded data in case you have not downloaded the complete data. The correct md5 code for file "FOREGROUND_6NN_WITH_DPETHS_sweeps.tar.gz." should be 16c837a175c90e8bad9c6c2c5d9b4786  FOREGROUND_6NN_WITH_DPETHS_sweeps.tar.gz. Hope it helps.

when I use tar -zxvf FOREGROUND_6NN_WITH_DPETHS_sweeps.tar.gz，it still got same error。
Here is my md5 check code. It's different from md5 code you provided.
I use baidu_networkdisk-client and it download FOREGROUND_6NN_WITH_DPETHS_sweeps.tar.gz completely.

PS C:\Users\linux> certutil.exe -hashfile  F:\msmd\new\FOREGROUND_6NN_WITH_DPETHS_sweeps.tar.gz
SHA1 的 F:\msmd\new\FOREGROUND_6NN_WITH_DPETHS_sweeps.tar.gz 哈希:
1f92384d34e98cbb4b5b81985e3e997ea89ec47b
CertUtil: -hashfile 命令成功完成。

The wrong md5 code means that the downloaded file is not complete or damaged due to network reasons. You can try to download it again.

thanks for your great job! I got some error when I untar data package FOREGROUND_6NN_WITH_DPETHS_sweeps.tar.gz. Here is my command, tar xvf FOREGROUND_6NN_WITH_DPETHS_sweeps.tar.gz
After minutes, there is something wrong happened. Could you repack data set?
gzip: stdin: invalid compressed data--crc error tar: Child returned status 1 tar: Error is not recoverable: exiting now

Hello, have you trained this model? I'm using A800 here but still not enough to train. If you've trained, can you give me an answer to the reason for my lack of memory here, thank you


Portable VS Code stores settings in the roaming profile

Does this issue occur when all extensions are disabled?: Yes

Version: 1.66.2
Commit: dfd34e8260c270da74b5c2d86d61aee4b6d56977
Date: 2022-04-11T07:46:01.075Z
Electron: 17.2.0
Chromium: 98.0.4758.109
Node.js: 16.13.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19044
Steps to Reproduce:

Portable installation.

Locate the settings file. It is not in the portable-editor tree, but rather in the roaming profile.

This is clearly a problem: I can't take the settings with the portable editor. They are (a) left behind on the computer I have used, and (b) I have to recreate them on any computer on which I use the editor again and again.

/needsMoreInfo
Have you followed all steps from https://code.visualstudio.com/docs/editor/portable ?
In particular section about data folder and it's location https://code.visualstudio.com/docs/editor/portable#_windows-linux

Oops, I did not create that folder.
On that note: Why is it up to the user to "opt in" by creating that folder. After all, the user
downloaded a portable version, so the expectation would be for it to be set up as portable
from the get to. The data folder should have been present in the zip!
Is there a rationale?

After all, the user downloaded a portable version, so the expectation would be for it to be set up as portable from the get-go.

We don't distribute a portable version. We just distribute a ZIP archive which can be made portable. The rationale is to reduce the amount of different distributables.


Default paketo image for amd64 only

Describe the bug
Default epinio packeto image paketobuildpacks/builder:full is available for amd64 only, so this image won't be usable for arm64 nor s390x.
I haven't found any replacement on dockerhub.

We should document this limitation somewhere

At the moment we are not going to provide the builders for other architectures, but someone can still do this.
Let's close it and keep an eye about the future on the other issue.


https://docs.readthedocs.io/en/latest/getting_started.html to show text using more screen real estate.

Details

Project URL: https://docs.readthedocs.io/en/latest/getting_started.html
Build URL (if applicable):n/a
Read the Docs username (if applicable): n/a

Expected Result
A description of what you wanted to happen
https://docs.readthedocs.io/en/latest/getting_started.html show text using all screen real estate.
Actual Result
A description of what actually happened
https://docs.readthedocs.io/en/latest/getting_started.html show text using half screen real estate.
Suggestion

Would be nice if a using could drag the text part to the right, bigger, if wanted.
Or, in the Read the docs v: lastest dropdown/expander add the view size option

This has been brought up with our theme a number of times. Current plan is to size font depending on screen width. Closing here, this isn't a problem with RTD core.

@agjohnson
When will

Current plan is to size font depending on screen width.

be active/work?


Reverse proxy support?

Hi,
I'm prototyping this with our reverse proxy. However, I'm running into an issue where coredns (exposed) is returning the internal IP of reverse proxy entrypoint, whereas I'd like it to return an external IP so we can have our main DNS delegate to it. Is this currently possible?
Thanks for your help!

Hi @neilmfrench , can you elaborate more about the specifics of your setup? What kind of ingress controller do you use and how is it integrated with reverse proxy?

Hi @neilmfrench , can you elaborate more about the specifics of your setup? What kind of ingress controller do you use and how is it integrated with reverse proxy?

Sure, I am using Traefik (other cluster uses nginx, but testing on cluster with Traefik first). Life of a request flows like this
request -> Cloudflare proxy -> external IP (189.XX.XX.XX)-> internal load balancer for Traefik (172.16.XX.XX) -> cluster nodes (not exposed)
I need coredns to respond with the external IP rather than the internal load balancer IP it currently responds with:
Status:
  Healthy Records:
    .....................
      <IP_ADDRESS>

I tried creating my own DNSEndpoint with the correct external IP as the target, but it didn't seem to get picked up.

I think the simplest solution would be to allow the reading of the "real" external IP from an annotation on the ingress if it exists, rather than relying only on status.loadBalancer.ingress. Health checks would still use the internal IP, but for building out the local targets list it would use whatever ip the annotation has.

@neilmfrench, thanks a lot for the suggestions! That sounds reasonable and should be relatively straightforward to add. We will put it into the roadmap 👍

Hello, is there any update on fixing this issue ?

Hi, we haven't implemented it yet. I will make sure we will discuss it on the next community meeting. Thanks for  bringing it up

Thank you for your prompt response, i'm looking forward this feature implemented !!


Yaml to Json Conversion is using comma instead of period symbol on float values

Current behavior
When converting  a yaml content with float props values, the json output value uses the wrong decimal separator

How to reproduce it (as minimally and precisely as possible)
Accessing DevToys, going on Converters and then JSON <> Yaml option, insert a value line floatvalue: 1.0
Expected behavior
Expected that the json output to follow the RFC 8259
Screenshots

Workaround
No response
Affected platforms
Windows
Affected DevToys kind
DevToys (app with GUI)
DevToys Version
Versão <IP_ADDRESS> | X64 | RELEASE | 2706c6c2 | 2706c6c2 | OS Build 22631.3007
Relevant Assets/Logs
No response

Yaml to Json Conversion is using comma instead of period symbol on float values
Current behavior
When converting a yaml content with float props values, the json output value uses the wrong decimal separator
How to reproduce it (as minimally and precisely as possible)
Accessing DevToys, going on Converters and then JSON <> Yaml option, insert a value line floatvalue: 1.0
Expected behavior
Expected that the json output to follow the RFC 8259
Workaround
No response
Affected platforms
Windows
Affected DevToys kind
DevToys (app with GUI)
Relevant Code and Potential Causes
The issue with the decimal separator in float values during YAML to JSON conversion may be related to the handling of float parsing in the JsonYamlToolViewModel.cs file. The FloatParseHandling property of the JsonSerializerSettings is set to Decimal, which may not account for locale-specific decimal separators:
private readonly JsonSerializerSettings _defaultJsonSerializerSettings = new()
{
    FloatParseHandling = FloatParseHandling.Decimal
};

This setting is used in both JsonHelper.cs and YamlHelper.cs when converting between JSON and YAML. If the system's locale expects a comma as the decimal separator, this could lead to the observed behavior where the conversion process uses a comma instead of a period.
To address this issue, the conversion logic in JsonHelper.cs and YamlHelper.cs may need to be updated to ensure that the correct decimal separator is used based on the system's locale settings or to enforce the use of a period as per the RFC 8259 standard regardless of locale.
Disclaimer
Hey there,
I'm Doug, I'm a SWE working on a ticket specification bot to help get folks understanding specific bugs faster We are trying to test out our product on various open source repos like this one.
Here is what our bot churned out for this ticket. If you found it helpful or have feedback just reply to this comment, I'll be watching it. Otherwise, I just hope this is helpful. Our thing isn't ready to sell yet, but you can find more info on our website: www.AI-Maintainer.com
In the meantime, I haven't been paid to write C# in a looooonnngg time and the bot still gets stuff wrong, but looking at the code this seems at least like a decent starting point. If you could give feedback good and bad I'd love you for it. I genuinely hope this is just useful though.
As a side note, it seems like the default in Brazil, where you are from @LuscasLeo , uses a comma in the scenario you described. So this might be hard for the devs to reproduce until they switch their local to Brazil or similar. Hence this bug not being caught previously. So you may need to adjust your local settings locally, or the devs need to enforce consistent usage between conversions.

This issue is also present in Version 2.0-preview.1.
I just updated for mainly this feature and it is broken now :-/

Hi there,
I got a fix ready here: https://github.com/DevToys-app/DevToys.Tools/pull/42
Thanks for your patience. I'm hoping to publish the fix this weekend or early next week.


[APIM 4.2.0] - API Name is exceeding limit

Description
We have a requirement from the customer that they need to exceeding the limit of the API Name which is currently 60 and this is the curl that customer is using in order to create the API.
Getting this error

{
	"code": 400,
	"message": "Bad Request",
	"description": "Validation Error",
	"moreInfo": "",
	"error": [
		{
			"code": "400_name",
			"message": "name: size must be between 1 and 60",
			"description": null
		}
	]
}

Steps to Reproduce
Do a PUT request to update the api
curl --request PUT --url 'https://localhost:9443/api/am/publisher/v4/apis/
With an existing api that the name exceed more than 60 characters. And we cannot update that particular API.
Affected Component
APIM
Version
4.2.0
Environment Details (with versions)
No response
Relevant Log Output
No response
Related Issues
No response
Suggested Labels
No response

Fixed via https://github.com/wso2/carbon-apimgt/pull/12224/


Create Split-HostsFile function

Imported from Bitbucket issue #48, created on 2015-12-02

Create a function that will parse the hosts file and return objects for each line.  Each object should have IPAddress, HostName, Comments properties.
bb-issue-48

Originally created on 2012-09-03
Add answer to http://stackoverflow.com/questions/12187388/how-to-read-a-specific-line-in-a-text-file when finished.
bb-issue-comment-1910706

Originally created on 2012-12-27
Decreasing priority. Current project focus is releasing v1.0. I'm prioritizing all issues that won't be part of the 1.0 released as trivial. They will be re-prioritized after the 1.0 release.
bb-issue-comment-2708037

Here, I knocked this up in about an hour - most of that was playing around with regex!!  :)
https://github.com/My-Random-Thoughts/Various-Code/blob/master/Get-HostsFile.ps1


Provided label column 'IsPurchased' was of type String, but only type Single is allowed.

Problem encountered on https://dotnet.microsoft.com/learn/ml-dotnet/get-started-tutorial/data
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.
My Data is

IsPurchased
Country
Age
Salary

0
France
44
72000

1
Spain
27
48000

0
Germany
30
54000

0
Spain
38
61000

1
Germany
40
57000

1
France
35
58000

0
Spain
37
52000

1
France
48
79000

0
Germany
50
83000

1
France
37
67000

i m getting the following error

Provided label column 'IsPurchased' was of type String, but only type Single is allowed.

i selected sql database with the column IsPurchased of type int. I also tried with a column of type bit but getting same error

I tried in multiple ways and conclude that i have selected wrong scenerio.
i had selected Value Prediction
but when i tried with Text classification used for sentiments, it worked.


Create an otPlat API to handle vendor / platform specific Spinel properties.

Occasions exist where a host needs to send commands that are specific to its platform. Such commands should not be implemented by Openthread but should be handled by code that is technically separate from Openthread but still part of the NCP image.  Because Spinel and ncp_base get first opportunity to parse incoming frames from the host, it becomes necessary for ncp_base to have a way to route vendor specific requests to an appropriate handler.
For this reason it is proposed to create a new otPlat API that ncp_base will call for any Spinel property frames that it cannot handle internally.  Calls to this API shall be configured at build time via a core config variable that shall be disabled by default.

Resolved by #2117.


original errors get swallowed due to retries

specifically, hostname nonexistence is being swallowed in my case, but because a retry is always attempted if a conn is None in urlib3 (it appears), then after invocation the retry is aborted if tries is too large, then a max connection exception is thrown (losing the original exception)
>>> import logging
>>> logging.basicConfig(level=logging.DEBUG)
>>> import requests
>>> r = requests.get('http://doesntexist.foo')
INFO:requests.packages.urllib3.connectionpool:Starting new HTTP connection (1): doesntexist.foo
WARNING:requests.packages.urllib3.connectionpool:Retrying (0 attempts remain) after connection broken by 'gaierror(8, 'nodename nor servname provided, or not known')': /
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "venv/lib/python2.6/site-packages/requests/api.py", line 78, in get
    return request('get', url, **kwargs)
  File "venv/lib/python2.6/site-packages/requests/api.py", line 65, in request
    prefetch=prefetch
  File "venv/lib/python2.6/site-packages/requests/sessions.py", line 191, in request
    r.send(prefetch=prefetch)
  File "venv/lib/python2.6/site-packages/requests/models.py", line 453, in send
    raise ConnectionError(e)
requests.exceptions.ConnectionError: Max retries exceeded for url: /
>>>

note: the above output was generated testing against the current release version 0.8.2.

Is a ConnectionError not specific-enough?

I suppose it would be, but the message I get is about failing retries (even when retry is set to 0). I have no visibility at the api level into the actual cause (name not found).
On Nov 26, 2011, at 7:31 AM, Kenneth<EMAIL_ADDRESS>wrote:

Is a ConnectionError not specific-enough?

Reply to this email directly or view it on GitHub:
https://github.com/kennethreitz/requests/issues/289#issuecomment-2882078

I can certainly see your point. It was just a surprising departure from how the code used to behave.
Didn't it used to return a result object with the 'ok' attribute set to true or false, and an error object possibly present?
I was using something like this with an earlier version (0.7.x maybe?)... A bit ugly but it was functional (provided a means to discern the underlying issue).
        resp = requests.get(
            url, headers=req_headers, timeout=2, allow_redirects=False,
            hooks={'pre_request': no_follow_redirect_hook})
        if resp.error:
            logging.debug('response error: %s', resp.error)
            if isinstance(resp.error, requests.exceptions.Timeout):
                raise HTTP504Exception('Timeout')
            elif isinstance(resp.error, urllib2.URLError):
                if hasattr(resp.error, 'reason') and \
                    isinstance(resp.error.reason, socket.gaierror):
                    raise HTTP404Exception('Host name not found')

My main use-case for knowing a bit more about the underlying connection error, is a proxy service scenario. Knowing the underlying problem can give insight into a proper http response code to the requesting client, as well as a friendly and hopefully helpful error message.
I can certainly work around it and return a more generic message to the requesting clients, but I was wondering if this was an intended change to error handling or an unforeseen one due to the urllib3 code.
Thanks.

Yes, it was absolutely an intended change — mostly due to the multitude of requests that were made by the community.
There's a safe_mode parameter that will surpress the errors, as before.
Here's the docs on the new exceptions: http://docs.python-requests.org/en/latest/user/quickstart/#errors-and-exceptions


NOT TO MERGE - Testing GHA working fine

This PR is a copy of another branch to test the GHA workflow is working as expected. It is not easy to test as workflow file must bu updated on master so that it affects PR.

/render

/render

normal comment with the keyword


Use public pool and PAT for publishing public symbols

Change the publish symbols task to use the Reunion pool and a PAT for credentials.

/azp run


Gentx submission - Sydai

Please make sure your node is up and running now. Join #validators on Discord for help: https://discord.gg/QeJWCrE.


Add docs for eth_getFilterLogs

Description:

Missing records in openrpc.json, rate-limiting.md and rpc-api.md for eth_getFilterLogs.
Related issue(s):
Fixes #1638
Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Codecov Report
Patch and project coverage have no change.

Comparison is base (407a271) 77.45% compared to head (199dca8) 77.45%.

:exclamation: Current head 199dca8 differs from pull request most recent head 610899f. Consider uploading reports for the commit 610899f to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1639   +/-   ##
=======================================
  Coverage   77.45%   77.45%
=======================================
  Files          36       36
  Lines        2710     2710
  Branches      543      543
=======================================
  Hits         2099     2099
  Misses        443      443
  Partials      168      168

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


Add a sensu_check resource, closes #10

This will generate the object files necessary for a single check definition and
executes sensuctl create -f with the generated file. Includes :create and :delete actions.
Includes unit/integration tests.

This is deliberately simple to start.
Some ideas for future improvements:

validate cron property syntax, ideally reusing what can be from builtin cron resource and provider
check if command property exists
validate if org/environment properties exist (sensuctl shell out)

IMO in order to close #10 we need to support every attribute the sensu check spec supports. I think for now we can hit the most important ones for now and add an additional that allows passing an arbitrary hash of key value pairs so that you can add non spec specific config such as a runbook link.

I added the rest of the documented attributes from: https://docs.sensu.io/sensu-core/2.0/reference/checks/#check-attributes but let me know if you see anything missing

Updated based on feedback. I agree about the tests being too generous, they're quite simple now and not comprehensive enough for what I'd like to see eventually

I'm not sure about something as generic as additional at this time. I think with sensu2 that's going to be handled by extended_attributes which can store arbitrary json. I'll go ahead and add extended_attributes and it can be used this way if someone wants.

Makes sense I was just calling it additional because that is what the sensu 1.x cookbook called it. That being said I think I view them as potentially 2 different properties. Rationale:

when a new check attribute is added if there is not some generic hash you can add keys to then it would require someone to come to the cookbook to add it and wait for review + release before they could leverage it. By having a generic hash they can immediately leverage it and then get it to become a first class property in the cookbook.
extended attributes seems more like the generic add arbitrary metadata about the instance but I could be wrong as there is no real description yet in the docs.

I updated the properties that are tested and verified. Focused solely on inspec test at this point, chefspec testing will be mildly useful but I feel like it can be enhanced later.
I don't mind adding additional as a property if that's a blocker right now. I'm also a little unclear about it's relationship to extended_attributes and would probably need @portertech  to weigh in on it to be sure.

I don't consider adding additional or whatever we call it to be a blocker just something we should be thinking about and put in shortly. Let's hold off on that until we understand the relationship of extended_attributes as I don't think it serves a different use case but I'd like to confirm.

I have opened #16 to address the questions around extended_attributes and how we want to allow people to add keys before they exist in the cookbook.


星期格式

可以提供个接口设置星期几的格式吗，目前只能是默认是周一、周二、最好可以设置显示格式，比如：一、二；星期一、星期二

@JW-chenjingwei use English please, by google translate, for example.
Custom week format
Can provide a format to set the day of the week format, currently only the default is Monday, Tuesday, it is best to customise the display format, such as: one, two;

1


Evaluation test

Hi @andras-sereny I was trying to integrate the spark-evaluation module using mahout example and zeromq interface.
What I've seen running the idomaar-demo is the following error:
INFO     [bg:reco-manager] 2015-07-14 14:09:29,295 (PollableSourceRunner-KafkaSource-r1) [INFO - eu.crowdrec.flume.plugins.interceptor.IdomaarRecommendationInterceptor.intercept(IdomaarRecommendationInterceptor.java:111)] Received recommendation [{}	[{"id":"1392170","rating":"10.0","rank":"1"},{"id":"1649419","rating":"10.0","rank":"2"},{"id":"975645","rating":"9.0","rank":"3"},{"id":"1623205","rating":"8.715197","rank":"4"},{"id":"2023587","rating":"8.6278","rank":"5"}]]
INFO     [bg:reco-manager] 2015-07-14 14:09:29,296 (PollableSourceRunner-KafkaSource-r1) [INFO - eu.crowdrec.flume.plugins.interceptor.IdomaarRecommendationInterceptor.intercept(IdomaarRecommendationInterceptor.java:121)] Sending event [recommendation	47<PHONE_NUMBER>	{"reclen":5}	{"subject":"user:28"}	19	{}	[{"id":"1392170","rating":"10.0","rank":"1"},{"id":"1649419","rating":"10.0","rank":"2"},{"id":"975645","rating":"9.0","rank":"3"},{"id":"1623205","rating":"8.715197","rank":"4"},{"id":"2023587","rating":"8.6278","rank":"5"}]]
INFO     [bg:reco-manager] 2015-07-14 14:09:29,296 (PollableSourceRunner-KafkaSource-r1) [INFO - eu.crowdrec.flume.plugins.interceptor.IdomaarRecommendationInterceptor.intercept(IdomaarRecommendationInterceptor.java:75)] Received end of recommendation file
INFO     [bg:reco-manager] 2015-07-14 14:09:29,312 (PollableSourceRunner-KafkaSource-r1) [INFO - eu.crowdrec.flume.plugins.interceptor.IdomaarRecommendationInterceptor.intercept(IdomaarRecommendationInterceptor.java:78)] Sent 'FINISHED' to orchestrator, waiting for reply ...
INFO     [datastream] Evaluator: no recommendations received for 60 secs, stopping.
INFO     [datastream] Exception in thread "Thread-6" java.lang.ArithmeticException: / by zero
INFO     [datastream] at eu.crowdrec.idomaar.evaluation.StreamingEvaluator.run(StreamingEvaluator.java:325)
INFO     [datastream] at java.lang.Thread.run(Thread.java:745)
INFO     [datastream]
INFO     [datastream] Evaluation results
INFO     [datastream] ==================
INFO     [datastream] 69950 [ProducerSendThread-evaluator-producer] INFO kafka.client.ClientUtils$  - Fetching metadata from broker id:0,host:<IP_ADDRESS>,port:9092 with correlation id 0 for 1 topic(s) Set(output)
note that we have changed the output data adding info about duration (see https://github.com/crowdrec/idomaar/wiki/DATA-FORMAT) but it should not be the cause cause the problem was present before the modification.
Did you see this error behaviour before? It seams to me that the interceptor is sending the correct message perhaps there is some misconfig in topic names?

Hi @andras-sereny, I've made some minor modification (added new flume) and added the creation of the recommendation-result topic but it still doesn't work.
Do you have time to have a look on that before september?

I hope to have some time next week. Will take look.

Hi @andras-sereny,
we've modified the classpath for flume 1.6 and now the full process seams to work, I'm now getting a correct recommendation results in /tmp/recommendations, like the following:
recommendation	40<PHONE_NUMBER>	{"reclen":5, "expected": {"ratings": [{"rating":1, "subject": {"type": "user", "id": 2}, "subject": {"object": "movie", "id": 3} }
]}}	{"subject":"user:7"}	55	{}	[{"id":"1702439","rating":"7.5554733","rank":"1"},{"id":"1623205","rating":"7.3246436","rank":"2"},{"id":"1024648","rating
":"7.1299243","rank":"3"},{"id":"1351685","rating":"6.6666665","rank":"4"},{"id":"1371111","rating":"6.464102","rank":"5"}]
recommendation	41<PHONE_NUMBER>	{"reclen":5}	{"subject":"user:10"}	18	{}	[{"id":"1840417","rating":"8.0","rank":"1"},{"id":"1024648","rating":"7.0"
,"rank":"2"},{"id":"1790885","rating":"7.0","rank":"3"},{"id":"1907668","rating":"7.0","rank":"4"},{"id":"1235522","rating":"7.0","rank":"5"}]
recommendation	42<PHONE_NUMBER>	{"reclen":5}	{"subject":"user:11"}	23	{}	[{"id":"1707386","rating":"10.0","rank":"1"},{"id":"1623205","rating":"4.9
629755","rank":"2"},{"id":"1024648","rating":"3.1594896","rank":"3"},{"id":"1428538","rating":"1.0","rank":"4"},{"id":"1045658","rating":"1.0","rank":"5"}]
recommendation	43<PHONE_NUMBER>	{"reclen":5}	{"subject":"user:15"}	27	{}	[{"id":"1547234","rating":"10.0","rank":"1"},{"id":"1840417","rating":"8.0
","rank":"2"},{"id":"1276104","rating":"8.0","rank":"3"},{"id":"1235522","rating":"7.7758107","rank":"4"},{"id":"1790885","rating":"7.725913","rank":"5"}]
recommendation	44<PHONE_NUMBER>	{"reclen":5}	{"subject":"user:16"}	5	{}	[{"id":"1907668","rating":"7.0","rank":"1"}]
the evaluator is still getting the following error:
INFO     [datastream] Evaluator: no recommendations received for 60 secs, stopping.
INFO     [datastream] Exception in thread "Thread-6" java.lang.ArithmeticException: / by zero
INFO     [datastream] at eu.crowdrec.idomaar.evaluation.StreamingEvaluator.run(StreamingEvaluator.java:325)
INFO     [datastream] at java.lang.Thread.run(Thread.java:745)
INFO     [datastream]
INFO     [datastream] Evaluation results
INFO     [datastream] ==================
could you take a look on that?

Fixed with flume classpath update


Render default highway shield as fallback

I see there is a default shield in the code, but I'm not seeing any render, for example, this route relation which lacks the network tag, but is otherwise a valid road route relation and thus should (in my opinion) render. It will make it easy to spot routes which should probably be supported in the future.
It's also very helpful for development purposes as it makes it easy to change the default style when trying to add support for a route network that is not yet tagged with network.

This seems like a reasonable change, and it would also help users identify routes which are missing network tags.

Probably also a tile change since these don't appear in the route_1 properties:

Maybe we should make the default something other than the black and white rounded rectangle so it's clearer when the default fallback is being used vs routes that intentionally use that style.

The issue comes from OpenMapTiles:
https://github.com/openmaptiles/openmaptiles/blob/da7b1b164552445fc6b1f326af7df3b08f3fe4ee/layers/transportation_name/update_transportation_name.sql#L311-L316
Basically, if the network is null, it doesn't populate the ref.


preg_match(): Compilation failed warnings

Expected behavior
No warnings.
Actual behavior
Got multiple warnings:
( ! ) Warning: preg_match(): Compilation failed: \k is not followed by a braced, angle-bracketed, or quoted name at offset 11 in [...]\vendor\timber\timber\lib\ImageHelper.php on line 246

Stack trace:
1   0.0009  442848  {main}( )   ...\upload.php:0
2   0.4023  21430736    wp_delete_attachment( string(4), ??? )  ...\upload.php:161
3   0.4053  21442560    do_action( string(17), string(4) )  ...\post.php:4868
4   0.4063  21442904    Timber\ImageHelper::Timber\{closure}( string(4) )   ...\plugin.php:525
5   0.4070  21515304    Timber\ImageHelper::delete_generated_files( string(77) )    ...\ImageHelper.php:178
6   0.4071  21515888    Timber\ImageHelper::process_delete_generated_files( string(20), string(3), string(52), string(13), string(24) ) ...\ImageHelper.php:221
7   0.4090  21518600    preg_match ( string(105), string(85) )  ...\ImageHelper.php:246

Steps to reproduce behavior
Delete multiple images in the Media library.
What version of WordPress, PHP and Timber are you using?
WordPress 4.5.3, PHP 7.0.3, Timber 1.0.3
How did you install Timber? (for example, from GitHub, Composer/Packagist, WP.org?)
Composer

@kirkov is this on a multisite or regular install?

Regular install - and forgot to mention, on Windows 10.

We put the "Win!" in "Uninstall Windows!"
.... sorry

@kirkov Is this still something you're actively working on or need addressed? Please respond in two weeks or we'll close this thread

No worries 💯


konlet ignores ENTRYPOINT defined in the Dockerfile

I am having some issues running a Docker image that defines both an ENTRYPOINT and a CMD (for example: https://hub.docker.com/r/prom/prometheus) on GCE running Container-Optimized OS 71-11151.71.0 stable.
Here is the interesting part of the Dockerfile (https://github.com/prometheus/prometheus/blob/release-2.7/Dockerfile):
ENTRYPOINT [ "/bin/prometheus" ]
CMD [ "--config.file=/etc/prometheus/prometheus.yml", \
      "--storage.tsdb.path=/prometheus", \
      "--web.console.libraries=/usr/share/prometheus/console_libraries", \
      "--web.console.templates=/usr/share/prometheus/consoles" ]

When I run a VM on GCE and specify Prometheus as the Docker image to use, konlet is not able to start the container. It seems like konlet is ignoring the ENTRYPOINT and tries to run the CMD in a shell. Here are the interesting logs:
Feb 05 23:38:11 my-instance konlet-startup[2472]: 2019/02/05 23:38:11 Error: Failed to start container: Error response from daemon: {"message":"OCI runtime create failed: container_linux.go:348: starting container process caused \"exec: \\\"--config.file=/etc/prometheus/prometheus.yml\\\": stat --config.file=/etc/prometheus/prometheus.yml: no such file or directory\": unknown"}

If I specify both the command and the command arguments using the Google Cloud Console GUI, the container is started correctly (see screenshot below).

Sorry, my fault, I was using a bugged Terraform module. Sorry to bother.


Use manual encoders/decoders

Asked by @forki on Gitter

I wonder if you could send a pr to bookstore that actually uses Toth decoders and removes the generic ones

Possible improvements:

Use an alias for Encode.toString. Like that in dev mode we have readible JSON and in production they are optimizedmodule Encode =
    let inline toString (value: Value) : string =
    #if DEBUG
    Encode.toString 4 value
    #else
    Encode.toString value
    #endif

Because we can't set compiler directive for intellisense at least in Ionide we need to reference Thoth.Json.Net for some special place in the client. Perhaps, there is another way to do it but I don't know it. If you want to try something here is the place where I needed this:

Decode model from SSR
Manually access the localStorage: needed here because BrowserLocalStorage.load is a Fable library.

About this limitation some people in the past mentioned that we could perhaps create a Common library to use the same "type references". for example. However, I don't know how to write such a common layer.
I decided to go fully with using manual decoders but a solution to avoid this intellisense problem could be to use AutoDecoders only for the SSR part. And use manual decoders for the rest of the Project. It can be a solution and don't impact too much the bundle size because we would need to generation the reflection data only at one place in the Client.

LGTM, if this is the way that we want to bookstore to go :-) The only question is that the indentation for some |> looks a bit atypical to me - but that could just be the github diff.

@isaacabraham Feel free to correct it (you have the right to commit to my branch as a maintainer) or make a review of with annotations I will update accordingly.

And yes for F# to F# JSON deserialization/serialization it's probably not needed to use manual encoders/decoders as long as you share the same Type definition between both projects.
However, if you are interacting with an external JSON provider then using manual encoders/decoders is the way to go so you don't need to do a 1 to 1 mapping between the JSON and F# types.
For example, at work, I decode the JSON data directly into a structure that is meaningful for my Model which isn't following the JSON structure. I flatten the data structure or add "state information" like (isLoading, isEditing, etc.) which isn't part of the JSON.

@Krzysztof-Cieslak could you please look at this an saturnize it?! that would be really great


Heroku toolbelt ubuntu install

When trying to install the heroku toolbelt on ubuntu using the method described on the site
wget https://toolbelt.heroku.com/install-ubuntu.sh | sh

An error occurs with the following message
Resolving toolbelt.heroku.com... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>
Connecting to toolbelt.heroku.com|<IP_ADDRESS>|:443... connected.
ERROR: certificate common name `www.heroku.com' doesn't match requested host name `toolbelt.heroku.com'.
To connect to toolbelt.heroku.com insecurely, use `--no-check-certificate'.

I contacted heroku support (Request #122616), they confirmed the issue and they pointed me to here.

Has this been resolved?


when i go /admin, redirected to /

Laravel Version: 5.4
Voyager Version: 0.11.14
PHP Version: 7.1.1
Database Driver & Version: mysql

Description:
i install clear laravel & voyager, register user with php artisan voyager:admin<EMAIL_ADDRESS>set admin name and password, then create laravel auth with php artisan make:auth
after i go to /admin/login, write my login and password, but after browser sends to url '/', if i go to '/home', i see message 'You are logged in!'
'/admin' not worked, but in routes/web.php i see Route::group(['prefix' => 'admin'], function () { 	Voyager::routes(); });
Steps To Reproduce:

php artisan voyager:admin<EMAIL_ADDRESS>--create

i have a admin user in database, I can't get to the admin panel, just go straight to the main page

Check issue #1217 and #1164 as it seems to be the same.

@trollsk1n, if you have direct access to your DB, please confirm that your user is in the "admin" role (likely id = 1).  Also, please confirm that that role has the browse_admin permission set.

I entered the command php artisan voyager:admin [myadmin] and get the answer The user now has full access to your site..
In database in table users i have the only user with role_id=1 and in the table roles the only record with id=1 and name=admin, other tables likely permissions, permissions_group, permission_role is empty

I found a solution, if u use MariaDB or old version of MySQL go to the app\Providers\AppServiceProvider.php and add to boot function string Schema::defaultStringLength(191); after enter the command php artisan voyager:install.

Yes that's certainly required per the laravel 5.4 docs due to an issue with the collation type used and a change in newer MySQL versions.
Rerunning artisan voyager:install is what fixed the symptoms you were describing. It ran the permissions seeder, which added the "browse_admin" permission and then added it to the admin role, which gave you access.


Please see the modified code

I just added the code so that it can work if number of player is more then five.

@ashwani8958 Thank you for the PR, there are minor conflicts to the code from the recent merge, let me know when conflicts are resolved.


nm device: Handle the race of device deletion

Before device deletion, the profile deletion might
also trying to delete the device.
To handle this race,
* Don't invoke device deletion if interface name is emtpy.
* Don't invoke device deletion if interface name became empty after
  mainloop started.
* When deletion failed, if source object is None, treat as deleted.
* When deletion failed, if source object interface name is emtpy,
  treat as deleted.
* When deletion failed, get the latest device by its name, if
  NM.Device is None, treat as deleted.

Removed the old dirty hack on error type.
Tested on bond and dummy interface deletion with 100 times.

This one fixed the issue on device deletion on NM 1.22.

Coverage increased (+10.8%) to 86.492% when pulling 20f3abda61a232d3e9c7f307dcb1ed47d495c1e2 on cathay4t:fix_dev_del into 055f39eac9043e6f6800e89c3359e13b979534ae on nmstate:master.

Coverage increased (+10.8%) to 86.492% when pulling 20f3abda61a232d3e9c7f307dcb1ed47d495c1e2 on cathay4t:fix_dev_del into 055f39eac9043e6f6800e89c3359e13b979534ae on nmstate:master.

In favor of PR #649 , closing this one.


nonpersistent_voxel_layer: 1.1.3-0 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository nonpersistent_voxel_layer to 1.1.3-0:

upstream repository: https://github.com/SteveMacenski/nonpersistent_voxel_layer.git
release repository: https://github.com/SteveMacenski/nonpersistent_voxel_layer-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.6.9
previous version for package: 1.1.2-0

Holding for sync


Basic create-react-app test throwing error

I've tried this locally as well and gotten the same error, but as you can see in from this basic code snippet in code sand box instance I'm getting this error:
Please load Stripe.js (https://js.stripe.com/v3/) on this page to use react-stripe-elements. If Stripe.js isn't available yet (it's loading asynchronously, or you're using server-side rendering), see https://github.com/stripe/react-stripe-elements#advanced-integrations

This is happening even though I'm simply including the StripeProvider with a key at root. I see that this is clearly not a problem in the jsfiddle example from the docs, which doesn't use create-react-app. Has anyone else run into this issue and found a solution?

https://medium.com/@ozluy/loading-stripe-script-if-necessary-189f228e0f34
how about this solution?


Webhook authorization

Hello. As it is said in your readme, I don't need to authorize in any way if I use only webhooks. But I tried to write a simple one, and it keeps coming back with error 401 (unauthorized). Can you help me with that?

hi @dsbatov ! can you show a code snippet describing how you send a request?

sure, but it is simply the copy of your code in readme
from pybitrix24 import Bitrix24
bx24 = Bitrix24('consiliummedicum.bitrix24.ru')
bx24.call_webhook('pu0qtn08t3e12eby', 'crm.lead.fields', {'ID': 17})
{'result': {...}}

@dsbatov thanks for your feedback! i missed that user id is required when making requests. just add appropriate argument during initialization of Bitrix24 like follows:
bx24 = Bitrix24('consiliummedicum.bitrix24.ru', user_id=1)

i'll add default user id in future versions

thanks, it worked!


Update quickstart.md

More explanations and additions to the prerequisites and background on the simple experiment.

I'm not sure how this got onto master with going through PR first, I reverted the changes on master so they can come through review instead.

I'm going to close this out, the simple example is being removed.


Minification Breaking Application

Since adding ngMeta it runs as you would expect it to but when minified it breaks throwing the following error.
Uncaught Error: [$injector:unpr] Unknown provider: eProvider <- e
I'm struggling to locate the cause. Some help would be appreciated.
Thanks!

Finally fixed this.
.run(['ngMeta', function(ngMeta) {   ngMeta.init(); }]);
Perhaps add this in the readme for anyone using it with minification?

Minification messes with implicit dependency injection.
If you're not using a build tool like ng-annotate (in your Grunt/Gulp/whatever-the-cool-kids-are-using-these-days process), you must explicitly handle dependency injection before minification for config, run and all components of AngularJS - controller, filter, service, factory etc.
I will add a note in the readme about this, thanks.

Cool, I wasn't aware of ng-annotate.
Thanks for the advice and getting back to me.
I'll close this now, cheers.

Hey, just checking on the this -- I didn't see anything in the readme and initializing ngMeta in our run block didn't solve the issue. We still get the following exception when we use minification, even with atostudios' run block suggestion:
Unknown provider: eProvider <- e <- ngMeta <- ngMeta
Is ng-annotate the only work around? If so it would be really helpful to list that as a requirement for anyone hoping to use ng-meta with minification.

ng-annotate is not a dependency of this module. It's just a helper that adds explicit dependency injection exactly as in @atostudios's run block code.
If this error seems to show up only while running minified code, I guess there's a function that has implicit dependencies. This is not an issue with ngMeta, but rather with all minified Angular code.
For full minification support, please replace
angular.config(function(ngMetaProvider, stateProvider) {
...
})

With
angular.config(['ngMetaProvider', 'stateProvider', function(ngMetaProvider, stateProvider) {
...
}])

You might need to do this for controllers as well. The dependency injection Angular doc has more information.
The run block in the README uses the minification-friendly code, but I will add a section for minification. Thanks.

We actually do use explicit dependency injection everywhere in our application, in order to make it safe for minification. So, our run block starts with:
.run(['$window', '$rootScope', 'ngMeta', function ($window, $rootScope, ngMeta) {     ngMeta.init()
and we still see the error eProvider <- e <- ngMeta <- ngMeta only when we minify our js. So maybe this is a different issue than @atostudios?
Thanks

I'd be able to help you better if you could show me the line that's throwing this error.
Please check if all uses of ngMeta are injected explicitly and that there's no mismatch (like ngMetaProvider being injected in place of ngMeta, or vice versa).

.run(['$window', '$rootScope', 'ngMeta', function ($window, $rootScope, ngMeta) is the line that's throwing the error -- I can remove all other references to ngMeta and I will still see that error.
I'll try to do some more investigation though -- maybe the way we're importing the package is incorrect or something.

Hi @prov-alexandra - how did you resolve this issue? I am getting the same error. I'm using @ngInject to handle dependency injection. It breaks following minification with error: eProvider <- e <- ngMeta <- ngMeta
Any help would be much appreciated :-)

Seems still an issue today. I minify and uglify with webpack using npm run build:prod:
rimraf dist && cross-env NODE_ENV=production && webpack -p
Running my application renders this error at startup:

Uncaught Error: [$injector:unpr] Unknown provider: eProvider <- e <- ngMeta <- ngMeta

Uncompressed my app works fine and ngMeta is working properly. This is the only occurrence of ngMeta in my source code:
    .run((ngMeta) => {
      'ngInject'
      ngMeta.init()
    })


Misc corrections

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

#sign-off


Required prop filters was not specified in DashboardPage. Check the render method of Connect(DashboardPage).

At the beginning i have 2 files with actions: dashboard.js and filter.js. On filters i have all my filter for my data from DB. And its works. But i decide to move all filter in dashboard.js where i have the main logic of my app. But after moved  i have next error:
Failed propType: Required prop filters was not specified in DashboardPage. Check the render method of Connect(DashboardPage).
dashboard.js
const initialState = {
    competitions: {
      draft: [],
      upcoming: [],
      active: [],
      past: [],
      complete: [],
      finished: []
    },
// filterStatuses  <--- this object i add from filter.js
    filtersStatuses: {
      draft: true,
      scheduled: true,
      started: true,
      ended: true,
      complete: true,
      finished: true,
      myOwn: true,
      subscribed: true,
      byTeam: true,
      lagging: true,
      onFire: true,
      endingSoon: true
    },
    loading: false
  };

export default function reducer(state = initialState, action = {}) {
  switch (action.type) {
    case DASHBOARD_LOAD:
      return {
        ...state,
        loading: true
      };
       ....
     // and more another actions
    default:
      return state;
  }
}

DashboardPage.js
import React, { Component, PropTypes } from 'react';
import { ButtonToolbar, Button, Row, Table, Glyphicon, ButtonGroup, Panel, Well, PageHeader } from 'react-bootstrap';
import { Link } from 'react-router';
import {bindActionCreators} from 'redux';
import {connect} from 'react-redux';
import _ from 'lodash';
import * as dashboardActions from '../redux/modules/dashboard';

@connect(
  state => ({
    competitions: state.dashboard.competitions,
    currentCompanyId: state.auth.companyId,
    startNowLoaded: state.dashboard.startNowLoaded,
    startNowLoading: state.dashboard.startNowLoading,
    startNowError: state.dashboard.startNowError,
    endNowLoaded: state.dashboard.endNowLoaded,
    endNowLoading: state.dashboard.endNowLoading,
    endNowError: state.dashboard.endNowError,
    loaded: state.dashboard.loaded,
    user: state.auth.user,
    companies: state.companies.companies,
    filters: state.dashboard.filtersStatuses,
    updated: state.competitions.updated,
    savedCompetition: state.competitions.savedCompetition
  }),
  dispatch => bindActionCreators({...dashboardActions, ...alertBarActions, ...filtersActions, ...competitionActions}, dispatch)
)

export default
class DashboardPage extends Component {
  static propTypes = {
    load: PropTypes.func.isRequired,
    competitions: PropTypes.object.isRequired,
    filters: PropTypes.object.isRequired   <----- on this moment i have error
  //Failed propType: Required prop `filters` was not specified in `DashboardPage`. Check the render     method of `Connect(DashboardPage)`.

  };

componentDidMount() {
    console.log(this.props); //<--in this object fitlers is present
    let {currentCompanyId} = this.props;
    this.props.load(currentCompanyId);
  }

render() {
    console.log("THIS IS FILTER : ", this.props.filters);  <--- and on this moment i have object
    // THIS IS FILTER :  Object {draft: true, scheduled: true, started: true, ended: true, complete: true…}
    const { filters } = this.props;
    return (
      <main className="content container dashboard">
       <button
           // BUT then i have error on this button
           // Unhandled promise rejection TypeError: Cannot read property 'draft' of undefined(…)(anonymous function) @ es6.promise.js?bc01:127
           className={ this.props.filters.draft ? 'btn btn-danger btn-xs' : 'btn btn-danger btn-xs filter-off'}
           onClick={this.switchFilter.bind(this,'draft')}>Draft
       </button>
</main>

Hi,
This is place isn't the right place to ask usage questions.
It is a bug and feature request tracker for React Redux.
It's hard to say from a glance what's wrong—a JSBin reproducing the issue would help.
It doesn't appear like an issue in React Redux so I'm closing but please feel free to ask this on StackOverflow.


TypeError: 'NoneType' object cannot be interpreted as an integer

Describe the bug
I load my  XGBoost classifier named xgb_model train on tabular data.
As in the examples I import necessary library:
from art.attacks.evasion import ZooAttack from art.estimators.classification import XGBoostClassifier import xgboost as xgb
the I train the art classifier :
classifier = XGBoostClassifier(model=xgb_model,nb_classes=2)
Then when  i trie to create an attack object with
attack= ZooAttack(classifier=classifier)
I got this typerror:
TypeError: 'NoneType' object cannot be interpreted as an integer
It point the line 164 of zoo.py but can't figure out whats wrong..

Hi @MajidAkh Thank you for using ART! Could you please print the value of model in cell 130 and classifier in cell 132?

Hello thank you for your answer in fact I figure out this issu and I am facing another one now.
When I use the method generate on my benign dataset it process normaly but I see no changes on the data set.
As you can see the score is the same for the two data set.

I think I miss something also I am working on tabular data.

Can you describe the tabular dataset in more detail? What kind of features does it contain?
Did you notice that the default values ZooAttack are set to run the attack fast rather than strong? Did you experiment with the arguments of the attack?

thank you for your answer.
The dataset is the 'Adult' dataset which has been preprocessing such that it contains only integer. The task is to perform a binary classification in order to predict instances which earn less or more than 50K.
No I do not experiment in fact I am reading your code and trying to understand it. What  arguments should I change according to you ?


Breaking Changes

The latest changes/updates seems to have broken the plugin.

The .git folder in previous version messed up the new display.
Deleting the .git folder resolved the problem.


Reset properties should be set in the class itself (prob. using decorators)

Reference:
methods.ts

Using transformable


Protect Transfer rpc with password

Let's add a password_hash string field to Transfer request message in order to protect this endpoint in such a sensitive operation.

For Ocean instead it may be interesting to move to 2fa/topt server as definitive solution.
Also some user may and other may not want it, so it should be enabled at will and maybe for all and specific RPCs

Not sure if requiring 2fa for generating a receiving address is a feature from user POV.
Tipically, wallets protect with 2fa the login, the transfer of funds and eventually some settings operation like changing the password for example. We could start by protecting these 3 RPCs initally.

Yes agreed, but my point was that if we have a whitelist/blacklist system to filter out RPCs would be easier to add/remove the protected route at will


Replace username with realname if they’re the same: show-names

Closes #2165
Before

After

Test

https://github.com/microsoft/TypeScript/issues/30442#ref-pullrequest-433925264
https://github.com/bfred-it/webext-options-sync/issues/13#issuecomment-497941181
Requires #2146 to use select<HTMLAnchorElement>('.author')!.pathname.slice(1)

https://github.com/sindresorhus/refined-github/issues/2165#issuecomment-504649392

to use (select('.author')! as HTMLAnchorElement).pathname.slice(1)

It might be worth looking for other places that would break because of this.
TS note: the code would be select<HTMLAnchorElement>('.author')!.pathname.slice(1)

I would check for the following:

username: (long and short) hyphen and dot.
real name: (long and short) hyphen, dot and space.

(tested #2185)

Can you pick this up again before #2229?

Oops, didn't notice you fixed this a while ago.
Tested and it works great both in discussions and newsfeed.


Get abfss file date modified

Hello,
This doc does not provide help to display the modified date of a file on a datalake. From my experiments, the os package does not work with abfss file system.
>>> dbutils.fs.ls("abfss://mypath")
[FileInfo(path='abfss://mypath', name='myfilename', size=myfilesize)]
>>> os.stat("abfss://mypath")
FileNotFoundError: [Errno 2] No such file or directory: 'abfss://mypath'

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

ID: f7dc6ed8-9e4f-4fa4-8ff8-e11e66ce26c2
Version Independent ID: ec675b15-edc7-4d38-edab-82a123280e0c
Content: Display file and directory timestamp details - Azure Databricks
Content Source: databricks/kb/python/display-file-timestamp-details.md
Service: azure-databricks
GitHub Login: @rrp-India-tech
Microsoft Alias: saperla

Hi @Neyri ,
In the documentation it shows how to Get File and directory timestamp details on data bricks file system /dbfs/.
In your case could you please try to mount your location to databricks file system and them use similar code?
Please Note, GitHub forum is dedicated for docs related issues. For any technical queries or clarifications, we encourage to utilise Microsoft Q & A platform. Kindly raise your query on Microsoft Q&A Platform.


Allow using examples in default-run

Problem
I have a library crate with examples under /examples. I want to be able to run cargo run to run an example automatically.
Proposed Solution
Allow something like
default-run = "./examples/my-example.rs"

in Cargo.toml.
Notes
This can be emulated by adding a [[bin]] target with your example:
[[bin]]
name = "my-example"
path = "./examples/my-example.rs"

However:

This is annoying
It gives you the following warning:

warning: file `~/lib-crate/./examples/my-example.rs` found to be present in multiple build targets:
  * `bin` target `my-example`
  * `example` target `my-example`

Can't that example be a bin instead?
Could you add an [alias] in .cargo/config.toml to run your default example, like:
[alias]
realrun = "run --example my-fake-example-realbin"

Could you share more about the workflow and scenario?

Going to close due to inactivity. Feel free to reopen if you have a chance to provide more info.


[WIP] testing doc-builder change

Testing https://github.com/huggingface/doc-builder/pull/214

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


waiting for load event using page lifecycle events

Fails for the below script with Error: Could not find node with given id as it trying to fetch using the wrong root id. Seems to be not waiting for load to update rootId after clicking "About".
const { browser, openBrowser, goto, click,comboBox } = require('taiko');

(async () => {
    try {
        await openBrowser();
        await goto("google.com");
        await click("I'm Feeling Lucky");
        await click("About");
        const inputfield =  await inputField({"placeholder":"Search Doodles"});
        await comboBox(near(inputfield)).select("हिन्दी");
        await click("Doodles संग्रह");
    } catch (e) {
        console.error(e);
    } finally {
        await closeBrowser();

    }
})();


Variable "console" is not defined!

Trying to setup a basic hello world program fails with the error Variable "console" is not defined!. I've created a repo for this example here: https://github.com/Yamboy1/hegel-test. How do I get the console to work?

@Yamboy1 your repo is private or does not exist at all...

oops, sorry. It should be visible now

Hi @Yamboy1. The browser environment still in development. Sorry about that. We will try to implement browser environment as soon as possible and will fix the problem. Thank you for the issue and your interest in the project ^_^.

Not directly related to the browser environment. Node also has console global variable. Any plans to release this support? Loving this tool so much :heart:

@richardaum It's already solved for Node.js, as I remember


Wrong lookup order in the loader

Hello!
We have been having an issue when we try to extend the class Loader into MY_Loader. The function load_class tries to load the children before the parent, therefore it loaded MY_Loader before CI_Loader and couldn't find the class CI_Loader, causing an error.
Reversing the order of paths in the following line fixes the issue, it works if we set BASEPATH, APPPATH instead of the one there: https://github.com/EllisLab/CodeIgniter/blob/develop/system/core/Common.php#L149
Not sure if it's the right solution but it works for us, would love to see your thoughts on this issue.
Pinging @victou to keep him informed about this issue.

How are you loading the class?
The order it would load the loader class is as follows (stoping as soon as it finds one):

APPPATH/libraries/CI_Loader.php
BASEPATH/libraries/CI_Loader.php

As soon as it finds a parent, it will attempt to load the child:

APPPATH/libraries/MY_Loader.php

So it is always looking for the parent first, then the child... how is your child setup/named/extending the parent?

Right.
It seems my coworker got this wrong, even though it allowed us to move forward and understand our issue it wasn't the problem at hand. We had to make a few changes here and there and had to redefine a few methods that were defined private because of it, and what I reported was actually unrelated and unneeded.
Sorry for the noise, and thanks for the clarification!


Enable compress of metrics payload

This would eliminate the limit of maximum payload size.

https://datadoghq.dev/datadog-api-client-typescript/classes/v1.MetricsApi.html#submitMetrics
The metrics end-point allows you to post time-series data that can be graphed on Datadog’s dashboards. The maximum payload size is 3.2 megabytes (3200000 bytes). Compressed payloads must have a decompressed size of less than 62 megabytes (62914560 bytes).

Issue

https://github.com/int128/datadog-actions-metrics/issues/521

It seems enabled by default.
https://datadoghq.dev/datadog-api-client-typescript/#disable-compressed-payloads


Creation of default system-tailored profile

e4s-cl should be able to create a default profile for the system MPI, along with a backend and image, for ease of use.

Added with e4s-cl profile init


Demo playlist view item can be draggable

As a user I want to reorder the playlist by moving item with touch event too.
Acceptance criteria

Playlist item reorder is still accessible
Playlist items are draggable to reorder the current playlist

Tasks

[ ] ...

The current solution provided by Compose doesn't seem to support Drap&Drop in LazyList (or in scrollable container) yet.
I've pushed my initial branch in the branch linked in the Development section for future reference, when Compose will support this use case.

I've updated the branch to include the recent changes in main and remove the Up/Down buttons from the playlist view.
Probably an other approach should be used to implement this feature. According to Compose's Drag & Drop documentation:


Getting a unnecessary semi-colon (;) node

Describe the bug
Getting a unnecessary text node with semi-colon(;).
Your minimal, reproducible example
https://codesandbox.io/s/aged-forest-3tyrpy?file=/pages/_app.jsx
Steps to reproduce

Create a Nextjs app with create-next-app
Install the latest @tanstack/react-query and @tanstack/react-query-devtools.
Setup QueryClientProvider with Hydrate in the _app.js  as per https://react-query-v2.tanstack.com/guides/ssr#using-hydration
Start the server and check the UI

Expected behavior
No unnecessary node should be rendered
How often does this bug happen?
Every time
Screenshots or Videos

Platform

OS - WIndows 11
Browser - Chrome, Firefox

react-query version
v4.0.10
TypeScript version
No response
Additional context
On commenting the QueryClientProvider and Hydrate code in _app.js, the node is not rendered.

Found the root cause. :|
Closing this.


Add support for on-premise Jira instances

Adding support for on-premise Jira instances. I made use of the Bearer PAT from the parent Jira package.
On self-hosted Jira instances the username is not needed for API calls.

Just FYI, this change would also apply to Steampipe+Confluence and Steampipe+BitBucket too

Just FYI, this change would also apply to Steampipe+Confluence and Steampipe+BitBucket too

Do you mean I should also create PRs there? I don't have any self-hosted instance of bitbucket or confluence to test :grimacing:

Welcome to Steampipe @juandspy and great to see the PR 👍!!
Few questions on the changes:

Do we need to pass the Bearer PAT in the token config argument? Or should we create a new config argument? IMO the latter sounds like a more feasible idea.
When using the PAT, are there any tables that would still require the username?

Welcome to Steampipe @juandspy and great to see the PR +1!!

Thanks! I'm loving Steampipe so far :)

Few questions on the changes:

Do we need to pass the Bearer PAT in the token config argument? Or should we create a new config argument? IMO the latter sounds like a more feasible idea.

I think this way is friendlier. What do you propose? :)

When using the PAT, are there any tables that would still require the username?

In the code, the username is just used for the connection. However, there are some tables like jira_user that cannot be queried (at least in my self-hosted instance):
> select * from jira_user;

Error: request failed. Please analyze the request body for more details. Status code: 404 (SQLSTATE HV000)

+--------------+------------+---------------+--------------+--------+------+-------------+-------------+-------+------+
| display_name | account_id | email_address | account_type | active | self | avatar_urls | group_names | title | _ctx |
+--------------+------------+---------------+--------------+--------+------+-------------+-------------+-------+------+
+--------------+------------+---------------+--------------+--------+------+-------------+-------------+-------+------+

I'm not sure if this is due to my instance or my permissions or if this would happen for all cases. Maybe we can just append to the error message a note about PAT tokens not supported for all tables (?)

Just FYI, this change would also apply to Steampipe+Confluence and Steampipe+BitBucket too

Do you mean I should also create PRs there? I don't have any self-hosted instance of bitbucket or confluence to test 😬

I just meant that the same problem is present in the other plugins too.
(I’m not associated with the steampipe project, by the way! Just someone who found the project over the weekend and was amazed to find a PR already open for an issue I was having 😄)
Happy work to port, and/or test, the fix to the other plugins once this one is merged 👍

Just FYI, this change would also apply to Steampipe+Confluence and Steampipe+BitBucket too

Do you mean I should also create PRs there? I don't have any self-hosted instance of bitbucket or confluence to test grimacing

I just meant that the same problem is present in the other plugins too.
(I’m not associated with the steampipe project, by the way! Just someone who found the project over the weekend and was amazed to find a PR already open for an issue I was having smile)
Happy work to port, and/or test, the fix to the other plugins once this one is merged +1

awesome! glad it helped :)

@juandspy Do you have an example/mock of what a bearer PAT looks like? When we add plugins to Turbot Pipes, we add validation to configuration arguments, so if the PATs have different regex patterns than the API tokens the plugin takes in today, that may be a good reason to have a separate config arg (and also more clear what users should pass in when creating connections).
Also, for the error message in:
> select * from jira_user;

Error: request failed. Please analyze the request body for more details. Status code: 404 (SQLSTATE HV000)

Does Jira give more details? Is the request body actually available and we can return that along with the error? This would be difficult for users to understand and if we add a note around PATs in this error message, that may not always be applicable since Jira is just throwing a 404 error here, which they may throw for other cases too.

@juandspy Do you have an example/mock of what a bearer PAT looks like? When we add plugins to Turbot Pipes, we add validation to configuration arguments, so if the PATs have different regex patterns than the API tokens the plugin takes in today, that may be a good reason to have a separate config arg (and also more clear what users should pass in when creating connections).

I can only say that I generated a couple of tokens and they have length of 44 characters, being numbers, letters and symbols. F.e "MDU0MDMx7cE25TQ3OujDfy/vkv/eeSXXoh/zXY1ex9cp" (this is not my token btw :laughing: ).

Also, for the error message in:
> select * from jira_user;

Error: request failed. Please analyze the request body for more details. Status code: 404 (SQLSTATE HV000)

Does Jira give more details? Is the request body actually available and we can return that along with the error? This would be difficult for users to understand and if we add a note around PATs in this error message, that may not always be applicable since Jira is just throwing a 404 error here, which they may throw for other cases too.

The request body is empty actually... You can see it's set to nil here.

@juandspy I just checked my API token and it had a format like wkNO4jUlNktmg4s1AJL1FAKE (though this may be an older token with fewer characters, I think the newer tokens are 100+ chars though, see https://jira.atlassian.com/browse/ID-8131), so they do appear to be different. Given this, I think I'd recommend creating another config arg called personal_access_token with a link to https://confluence.atlassian.com/enterprise/using-personal-access-tokens-1026032365.html as a comment.
If both the token and personal_access_token config args are set in a connection, the plugin should return an error with the message 'token' and 'personal_access_token' are both set, please only use one.
Can you please also update the comment line for token to be API token for Jira Cloud. See https://support.atlassian.com/atlassian-account/docs/manage-api-tokens-for-your-atlassian-account/.
In retrospect, I'd probably would have named token api_token instead if we knew about PATs before, but we shouldn't change it now since that'd be a breaking change (but we may add api_token later and deprecate token).
For the username config arg, given that Jira returns a vague error, maybe it's better to leave username mandatory? This will ensure the jira_user table will work, as long as a valid username is passed in.

OK, I'll do the changes. However, as of now, if the username is not defined, there is no error message in the plugin:
❯ steampipe query "select key, epic_key, created, updated, creator_display_name from jira_issue limit 10;"

Error: rpc error: code = Internal desc = hydrate function listIssues failed with panic runtime error: invalid memory address or nil pointer dereference (SQLSTATE HV000)

+-----+----------+---------+---------+----------------------+
| key | epic_key | created | updated | creator_display_name |
+-----+----------+---------+---------+----------------------+
+-----+----------+---------+---------+----------------------+

however if you check the plugin logs:
❯ cat ~/.steampipe/logs/plugin-2023-08-10.log | grep username
2023-08-10 05:50:18.253 UTC [ERROR] steampipe-plugin-jira.plugin: [ERROR]<PHONE_NUMBER>420: jira_issue.listIssues.searchWithContext: connection_error="'username' must be set in the connection configuration. Edit your connection configuration file and then restart Steampipe"

Is this the expected behavior?

If both the token and personal_access_token config args are set in a connection, the plugin should return an error with the message 'token' and 'personal_access_token' are both set, please only use one.

@cbruno10 alright, updated :)

@juandspy No that's not expected, the plugin should be showing the error instead of a nil pointer dereference. We could be trying to grab the error from a missing property or element.

@cbruno10 can we merge this one already?

> select * from jira_project

Error: invalid character '<' looking for beginning of value (SQLSTATE HV000)

I'm getting errors on some of the Jira tables when testing this branch, jira_project and jira_epic for example.
Tables which appear to work correctly are: jira_board and jira_issue
(We're on the Long Term Release version)

Thanks for testing @lorcanmcdonald !
@juandspy Were you able to query the tables @lorcanmcdonald had mentioned successfully? If so, are you on a different version?

Sorry @cbruno10 @lorcanmcdonald , I've been busy lately.

> select * from jira_project
Error: invalid character '<' looking for beginning of value (SQLSTATE HV000)

I've been investigating this issue. It looks like the jira_project table gets the API response  in HTML format, that's why we see the invalid character '<' looking for beginning of value error. If you look at the issues table, you may realize that a JQL query is used, not a raw query to the REST API. This is because the JQL is used just for getting issues, not projects. I'm afraid we will need to change the way we get the projects. I'm looking into it...

For me just these tables work:

table
works

jira_advanced_setting
❌

jira_backlog_issue
✅

jira_board
✅

jira_component
❌

jira_dashboard
❌

jira_epic
❌

jira_global_setting
❌

jira_group
❌

jira_issue
✅

jira_issue_type
❌

jira_priority
❌

jira_project
❌

jira_project_role
❌

jira_sprint
✅

jira_user
❌

jira_workflow
❌

Maybe we can put a disclaimer that for self-hosted Jira instances just these tables work and create a set of issues so that we fix the rest in the future.
BTW the code I've used to generate this:
#!/bin/bash

folder="docs/tables"

echo "table,works"

for file in "$folder"/*; do
    if [ -f "$file" ]; then
        table=$(basename "$file" ".md")

        cmd="steampipe query \"select * from $table LIMIT 5;\""
        eval "$cmd" &> /dev/null
        ret=$?
        if [ $ret -ne 0 ]; then
                echo "$table,❌"
        else
                echo "$table,✅"
        fi
    fi
done

I guess this can be reused in any plugin. So I'm sharing just for saving future work :laughing:

@lorcanmcdonald Also, the lazy approach for listing the Jira projects would be
SELECT DISTINCT project_key FROM jira_issue;

:stuck_out_tongue:

@juandspy apologies for the delayed response and thanks for sharing the table query results 👍!!
Could you please resolve the merge conflicts so that we can merge in the PR and make it available on the hub 👍!!

@juandspy apologies for the delayed response and thanks for sharing the table query results 👍!!
Could you please resolve the merge conflicts so that we can merge in the PR and make it available on the hub 👍!!

shall I add some notes about the tables that are working in Jira server vs Jira cloud? @misraved

Thanks @juandspy for pushing the changes 👍 !!
In the docs file, we could add a section that mentions the tables that work with PAT 👍. So yes, notes around Jira server vs Jira cloud would be great 👍.
We could merge and release the feature today 🚀 !!

Thanks @juandspy for pushing the changes 👍 !!
In the docs file, we could add a section that mentions the tables that work with PAT 👍. So yes, notes around Jira server vs Jira cloud would be great 👍.
We could merge and release the feature today 🚀 !!

Updated. Once you ship a new version of the plugin, I will create an issue to track this bug.

Hey @misraved , I cannot see https://github.com/turbot/steampipe-plugin-jira/pull/86/commits/6121a8a5faa17c186c58d78c2086a1a3de25da3c in docs/index.md. Any idea?