Allow AND/OR-searches for multiple tags

When I search SteamDB for multiple tags, I can currently only do an AND-search, e.g. Roguelite AND Roguelike, not Roguelite AND/OR Roguelike. It would be helpful to be able to do AND/OR searches, as well.
Example use case for an AND/OR search: suppose I'm a Roguelike afficionado and want to search for games with aspects of roguelikes, permadeath, exploration, or similar. Unfortunately, Steam spreads that info out over many tags, so I'd need an AND/OR search for tags like these: Roguelite, Roguelike, Action Roguelike, Perma Death, Metroidvania, Souls-like, Traditional Roguelike, Roguevania.
(One service I'm aware of which allows AND/OR searches is Steam250 (here's the aforementioned search for all these roguelike tags), but their search otherwise lacks important features like searching for discounts or hiding games in one's library.)

On which page?

Ah, I meant the search feature and its tag filtering on this page. Quote from the tooltip: "If multiple tags are included, all of them have to match".
Consider this example search: there seem to be ~10 Traditional Roguelikes and ~138 Action Roguelikes. When I search for both tags via SteamDB, it displays games which have both tags (i.e. 2 games). The AND/OR search I want would find games with either tag and display ~146 games (138+10 minus the 2 games which have both tags).

Thanks a ton for this update! As a very tiny issue, due to this update, the tag search tooltip is no longer up-to-date:


ArgumentOutOfRangeException from ReverseExpressionParser while editing

Migrated from: http://pytools.codeplex.com/workitem/2915
Bug found originally 2015 February 23.
PTVS 2.2 CL1419060 (2015 February 20)
VS 2013 ULT UPDATE 4
You can see where my cursor is in this picture, I had just hit spacebar when I got the activity log dialog.

From activity log:
System.ArgumentOutOfRangeException: Specified argument was out of the range of valid values. Parameter name: end
 at Microsoft.VisualStudio.Text.SnapshotSpan..ctor(SnapshotPoint start, SnapshotPoint end)
 at Microsoft.PythonTools.Intellisense.ReverseExpressionParser.GetStatementRange()
 at Microsoft.PythonTools.Intellisense.VsProjectAnalyzer.TrySpecialCompletions(IServiceProvider serviceProvider, ITextSnapshot snapshot, ITrackingSpan span, ITrackingPoint point, CompletionOptions options)
 at Microsoft.PythonTools.Intellisense.VsProjectAnalyzer.GetCompletions(IServiceProvider serviceProvider, ITextSnapshot snapshot, ITrackingSpan span, ITrackingPoint point, CompletionOptions options)
 at Microsoft.PythonTools.Intellisense.PythonAnalysisExtensions.GetCompletions(ITextSnapshot snapshot, IServiceProvider serviceProvider, ITrackingSpan span, ITrackingPoint point, CompletionOptions options)
 at Microsoft.PythonTools.Intellisense.CompletionSource.AugmentCompletionSession(ICompletionSession session, IList`1 completionSets)
 at Microsoft.VisualStudio.Language.Intellisense.Implementation.CompletionSession.Start()
 at Microsoft.VisualStudio.Language.Intellisense.Implementation.CompletionBroker.TriggerCompletion(ITextView textView)
 at Microsoft.PythonTools.Intellisense.IntellisenseController.TriggerCompletionSession(Boolean completeWord)
 at Microsoft.PythonTools.Intellisense.IntellisenseController.HandleChar(Char ch)
 at Microsoft.PythonTools.Intellisense.IntellisenseController.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
 at Microsoft.VisualStudio.Editor.Implementation.CommandChainNode.InnerExec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
 at Microsoft.VisualStudio.Editor.Implementation.CommandChainNode.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
 at MadsKristensen.EditorExtensions.CommandTargetBase`1.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
 at Microsoft.VisualStudio.Editor.Implementation.CommandChainNode.InnerExec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
 at Microsoft.VisualStudio.Editor.Implementation.CommandChainNode.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
 at MadsKristensen.EditorExtensions.CommandTargetBase`1.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
 at Microsoft.VisualStudio.Editor.Implementation.CommandChainNode.InnerExec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
 at Microsoft.VisualStudio.Editor.Implementation.CommandChainNode.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
 at MadsKristensen.EditorExtensions.CommandTargetBase`1.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
 at Microsoft.VisualStudio.Editor.Implementation.CommandChainNode.InnerExec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
 at Microsoft.VisualStudio.Editor.Implementation.CommandChainNode.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
 at MadsKristensen.EditorExtensions.CommandTargetBase`1.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
 at Microsoft.VisualStudio.Editor.Implementation.CommandChainNode.InnerExec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
 at Microsoft.VisualStudio.Editor.Implementation.SimpleTextViewWindow.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
 at Microsoft.VisualStudio.Editor.Implementation.VsKeyboardFilter.SendCommand(Guid cmdGroup, UInt32 cmdID, Object inParam)
 at Microsoft.VisualStudio.Editor.Implementation.VsKeyboardFilter.TextInput(TextCompositionEventArgs args)
 at Microsoft.VisualStudio.Text.Editor.Implementation.KeyProcessorDispatcher.<DispatchTextInputEvents>b__0(KeyProcessor p, TextCompositionEventArgs args)
 at Microsoft.VisualStudio.Text.Editor.Implementation.KeyProcessorDispatcher.<>c__DisplayClass1e`1.<Dispatch>b__1a()
 at Microsoft.VisualStudio.Text.Utilities.GuardedOperations.CallExtensionPoint(Object errorSource, Action call)

Hugues can't repro this anymore but the bits of the SDK aren't the same as they were so shrugs.
Some looking into history shows a change on the 18th.  Bits are from the 20th and issue was reported on the 23rd.
Might be worth looking into this commit to see any possible issues.
https://github.com/Microsoft/PTVS/commit/25e8327beaed91ac953a0fb493d581a1e8af3641


Post failed with mobile browsers

400 Bad Request
"This file is too large. It is 4.82MB but the maximum size is 976.56KB."

see https://scrapbox.io/inajob-inline/blueskyへのURL投稿が失敗する問題


exe-config: Fix underflow in ghcjs config lookup (Fixes #465)

Fixes #465

Yikes. Is there no way to traverse the collection directly without messing with indexes?
Failing that, maybe we should extract/upstream this fix into an util?

Should we make this impl match what seems to be the same bug fixed elsewhere?
https://github.com/obsidiansystems/obelisk/blob/86a9584c6d7648bd5324ef57d62421fed1bf1978/lib/frontend/src/Obelisk/Frontend.hs#L113

Your fix avoids fromIntegral so maybe you should actually port your fix over to that line as well. Also a comment would be great.

Is there a version of fromIntegral that throws/Maybes result so that we could "safely" cast to convert to signed type? I would much prefer simply getting rid of the unsignedness altogether.

Interesting, didn't even notice that one. Looks like collToList itself should be factored out

@tomsmalley This bug is bad enough that finding the perfect solution to abstracting out this function is definitely not worth it if it keeps us from getting rid of it. Let's just make the two implementations the same for now with a big TODO on both of them. I'll approve this and hopefully we can get it merged. I'll try to make the change if you don't beat me to it.


deepdiff

Pull Request Checklist:

[ ] Positive Test Case Written by Dev

[ ] Automated Testing

[ ] Sufficient User and Developer Documentation

[ ] Passing Jenkins Build

[ ] Peer Code review and approval

INFO:
Deepdiff upgraded on 4/10/22
https://github.com/seperman/deepdiff/commit/9c4dade2deae01307f4fd4ecf8a917d7fdbc5375
That breaks something in our build
Pinning at deepdiff==1.7.0 resolves the issue for now...

METRIC
VALUE

https://sonarqube.cuahsi-workstation.com:9000/dashboard?id=hydroshare-4586


Add: reorder_log_files

Codecov Report

Merging #98 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #98   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          40     41    +1
  Lines         448    465   +17
=====================================
+ Hits          448    465   +17

Impacted Files
Coverage Δ

problems/reorder-log-files/reorder_log_files.go
100% <100%> (ø)

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 9ce86d5...ef2bc0f. Read the comment docs.


Simplify travis-ci setup

Hi there,
I'm not going to barge into your project, but we are getting notifications about your project passing(/failing) in the #leiningen IRC channel every time a travis-ci run completes. So I updated the .travis.yml file to not notify the #leiningen channel, as well as not doing Leiningen-specific work.
Enjoy the Tiger book! :tiger:

I was too careless...


Issue: #147  Minor error on website: Realtime line chart example

See: https://github.com/fastly/epoch/issues/147

cc @fastly/ux

+1


ROX-21053: roxctl netpol generate - allow users to set custom dns port

Description
Adding the optional flag --dnsport to roxctl netpol generate, allowing users to set a custom DNS port for egress rules in the generated NetworkPolicy resources.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change
Added a test to tests/roxctl/bats-tests/local/roxctl-netpol-generate-release.bats and to tests/roxctl/bats-tests/local/roxctl-netpol-generate-development.bats. This test uses the new flag and verifies that the new port is indeed being used in the output NetworkPolicies.
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

/ok-to-test

/retest

@zivnevo I think this PR needs a rebase to solve some of the CI failures

@zivnevo I think this PR needs a rebase to solve some of the CI failures

Done

I am running CI checks in a clone-PR: #10597

It looks like now all unit tests from TestGenerateNetpol are failing:
=== RUN   TestGenerateNetpolCommand/TestGenerateNetpol/not_existing_inputFolderPath_should_raise_'does_not_exist'_error
    generate_test.go:137:
        	Error Trace:	/__w/stackrox/stackrox/roxctl/netpol/generate/generate_test.go:137
        	            				/go/pkg/mod/github.com/stretchr/testify@v1.9.0/suite/suite.go:115
        	Error:      	Received unexpected error:
        	            	illegal port number: must be between 1 and 65535, inclusive
        	Test:       	TestGenerateNetpolCommand/TestGenerateNetpol/not_existing_inputFolderPath_should_raise_'does_not_exist'_error

(src in my clone of this PR: https://github.com/stackrox/stackrox/actions/runs/8555988379/job/23444700984?pr=10597)
I think, we must check if the flag is correctly setting the default value, or whether the validation works well when --dnsport is not specified.

Thanks for the report. Definitely my mistake.
Unit testing bypasses parsing command-line flags, and so default was not set properly.
Now properly setting the default DNS port in unit-testing too.

This has been merged now in #10597.
The problems with CI checks for PRs from external contributors have been reported and I hope they will be fixed soon. For now, we need do make copies of such PRs (so that ACS member is the author) and merge them internally.


Access ex-mode on empty workspace

Currently it's not possible to toggle ex-mode panel if Atom has no active editors (empty workspace)
'atom-workspace:not(.editor)':
  ':': 'ex-mode:open'

while the following keymap works fine
'atom-workspace:not(.editor)':
  ':': 'application:open'

Am I missing something here? How can it be accomplished?

As of today, ex-mode is tied to an editor. Vim does not have this behaviour since it always has a buffer/window open, plus the fixed location makes it appear as if it were global. The impact is limited as you quickly have a file opened, so for the sake of simplicity this will not be implemented for now. See also #22.


Restructure CONTRIBUTING file

Our previous CONTRIBUTING file was somewhat terse and mean. (See #58.)
This PR contains work towards restructuring the CONTRIBUTING file and making it more inclusive and better-worded.
Closes #58.

Waiting on #60.

@cg505 can you approve also?


cmake and qt6

https://github.com/valbok/QtAVPlayer/pull/177

https://github.com/valbok/QtAVPlayer/pull/176


[rust] migrate to beta toolchain

Migrate from using the nightly toolchain to the beta toolchain now that async_await has been stabilized.

@metajack Yeah these commits aren't anywhere close to really being ready for review, which is why i left the PR in draft mode. In the end everything needs to be squashed because the commits don't compile individually (and as you can see CI is still broken).

:umbrella: The latest upstream changes (presumably 178fc36554ffb74181ffac798c54cd8cbd22afbd) made this pull request unmergeable. Please resolve the merge conflicts.

@bors-libra r+

:pushpin: Commit d624ebb has been approved by metajack

:hourglass: Testing commit d624ebb8ce27c27c328cc13a6d4c3a46ed030f43 with merge 6020c8f67f4785a66024aedf3f1340623ec192ed...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: metajack
Pushing 6020c8f67f4785a66024aedf3f1340623ec192ed to master...


Uncaught EvalError: Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: "script-src 'self'".

[Enter steps to reproduce below:]

Make an empty web application

Atom Version: 0.194.0
System: Mac OS X 10.10.3
Thrown From: omnisharp-atom package, v0.4.6
Stack Trace
Uncaught EvalError: Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: "script-src 'self'".
At events.js:141

EvalError: Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: "script-src 'self'".

  at Function (native)
  at Function.template (/Users/dfowler/Downloads/Atom.app/Contents/Resources/app.asar/node_modules/cheerio/node_modules/lodash/dist/lodash.js:6305:22)
  at underscore (/Users/dfowler/.atom/packages/omnisharp-atom/node_modules/generator-aspnet/node_modules/yeoman-generator/lib/util/engines.js:23:14)
  at [object Object].actions.engine (/Users/dfowler/.atom/packages/omnisharp-atom/node_modules/generator-aspnet/node_modules/yeoman-generator/lib/actions/actions.js:216:32)
  at [object Object].template (/Users/dfowler/.atom/packages/omnisharp-atom/node_modules/generator-aspnet/node_modules/yeoman-generator/lib/actions/actions.js:191:19)
  at [object Object].yeoman.generators.Base.extend.writing (/Users/dfowler/.atom/packages/omnisharp-atom/node_modules/generator-aspnet/app/index.js:146:18)
  at /Users/dfowler/.atom/packages/omnisharp-atom/node_modules/generator-aspnet/node_modules/yeoman-generator/lib/base.js:409:16
  at processImmediate [as _immediateCallback] (timers.js:369:17)

Commands
     -4:22.6.0 editor:newline (atom-text-editor.editor)
     -4:18.6.0 editor:select-to-first-character-of-line (atom-text-editor.editor)
  2x -4:18.1.0 core:backspace (atom-text-editor.editor)
     -4:17.6.0 core:save (atom-text-editor.editor)
     -3:44.1.0 core:move-up (atom-text-editor.editor)
     -3:43.8.0 editor:move-to-end-of-screen-line (atom-text-editor.editor)
     -3:43.6.0 editor:newline (atom-text-editor.editor)
  2x -3:41.3.0 editor:select-to-first-character-of-line (atom-text-editor.editor)
  2x -3:40.9.0 core:backspace (atom-text-editor.editor)
     -3:40.5.0 core:save (atom-text-editor.editor)
     -0:22.3.0 omnisharp-atom:new-application (atom-text-editor.editor)
     -0:15.9.0 editor:consolidate-selections (atom-text-editor.editor.mini)
     -0:15.9.0 core:cancel (atom-text-editor.editor.mini)
     -0:07.6.0 omnisharp-atom:new-application (atom-workspace.workspace.scrollbars-visible-when-scrolling.theme-atom-dark-syntax.theme-atom-dark-ui)
  2x -0:06.4.0 core:move-down (atom-text-editor.editor.mini)
  2x -0:05.0 core:confirm (atom-text-editor.editor.mini)

Config
{
  "core": {
    "themes": [
      "atom-dark-ui",
      "atom-dark-syntax"
    ]
  }
}

Installed Packages
# User
atom-yeoman, v0.1.5
autocomplete-plus, v2.12.0
language-csharp, v0.5.0
linter, v0.12.1
omnisharp-atom, v0.4.6

# Dev
No dev packages

This is probably an issue for atom-yeoman

this is due to a new change in the latest aspnet generator... and it's nasty...

fixed in atom-yeoman 0.2.0, we should bump omnisharp-atoms package.json.
See the dirty dirty hack here

Closing as this should be fixed in the latest atom-yeoman.


Problem converting ResNeXt-50 MXNet -> Keras

Platform (like ubuntu 16.04/win10):
ubuntu 16.04
Python version:
3.6
Source framework with version (like Tensorflow 1.4.1 with GPU):
mxnet 1.2.0 (cpu)
Destination framework with version (like CNTK 2.3 with GPU):
keras (cpu) 2.1.5 (tensorflow 1.7)
Pre-trained model path (webpath or webdisk path):
imagenet1k-resnext-50 (mxnet networks zoo)
Running scripts:
I follow the issue #58 but got the errors while converting the model:
$ python3 -m mmdnn.conversion._script.convertToIR -f mxnet -n ./resnext-50-symbol.json -w ./resnext-50-0000.params -d mxnet_resnext50 --inputShape 3 224 224
$ python3 -m mmdnn.conversion._script.IRToCode -f keras --IRModelPath mxnet_resnext50.pb --dstModelPath converted_resnext50.py --IRWeightPath mxnet_resnext50.npy
$ python3 -m mmdnn.conversion.examples.keras.imagenet_test -p imagenet1k-resnext-50 -s mxnet -n converted_resnext50 -w mxnet_resnext50.npy -i ./seagull.jpg

Local variable 'labels' referenced before assignment here - missed indentation level

Test fails

File "/media/D/anaconda3/lib/python3.6/site-packages/mmdnn/conversion/examples/imagenet_test.py", line 329, in test_truth
assert np.isclose(this_truth[index][1], i[1], atol = 1e-6)
AssertionError

I converted model as it is using:
mmconvert -sf mxnet -in resnext-50-symbol.json -iw resnext-50-0000.params -df keras -om resnext50.h5 --inputShape 3 224 224
And print sum of outputs of first model layers
Script
from keras.models import load_model
from keras.models import Model
import numpy as np

model = load_model('../resnext50/resnext50.h5')

def get_output_at_layer(model, layer, x):
    input = model.input
    output = model.layers[layer].output
    new_model = Model(input, output)
    return new_model.predict(x)

x = np.random.random((1,224,224,3)) * 255

for i in range(50):
    y = get_output_at_layer(model, i, x)
    print(model.layers[i].name, np.sum(y))

Output
data 19219074.0
bn_data 94388.93
zero_padding2d_1 94388.92
conv0 49383.504
bn0 443316.25
relu0 644856.3
zero_padding2d_2 644856.4
pooling0 253928.1
stage1_unit1_conv1 -197160.0
stage1_unit1_bn1 -81262.35
stage1_unit1_relu1 56259.15
zero_padding2d_3 56259.15
lambda_1 2100.377
lambda_2 2100.377
lambda_3 2100.377
lambda_4 2100.377
lambda_5 2100.377
lambda_6 2100.377
lambda_7 2100.377
lambda_8 2100.377
lambda_9 2100.377
lambda_10 2100.377
lambda_11 2100.377
lambda_12 2100.377
lambda_13 2100.377
lambda_14 2100.377
lambda_15 2100.377
lambda_16 2100.377
lambda_17 2100.377
lambda_18 2100.377
lambda_19 2100.377
lambda_20 2100.377
lambda_21 2100.377
lambda_22 2100.377
lambda_23 2100.377
lambda_24 2100.377
lambda_25 2100.377
lambda_26 2100.377
lambda_27 2100.377
lambda_28 2100.377
lambda_29 2100.377
lambda_30 2100.377
lambda_31 2100.377
lambda_32 2100.377

Thats seems to be strange to have the same output from all Lambda layers, where previous layer has different values
Script
i = 11
y = get_output_at_layer(model, i, x)
print(model.layers[i].name, np.sum(y))

for i in range(32):
    print('slice {}:{} \t'.format(i*4, (i+1)*4), np.sum(y[..., i*4:(i+1)*4]))

Output
zero_padding2d_3 56259.15
slice 0:4 	    3857.7256
slice 4:8 	    1897.7053
slice 8:12 	 46.061497
slice 12:16 	 4812.0205
slice 16:20 	 2283.8503
slice 20:24 	 54.295998
slice 24:28 	 20.24092
slice 28:32 	 1976.638
slice 32:36 	 662.6427
slice 36:40 	 2185.7913
slice 40:44 	 333.3028
slice 44:48 	 3391.289
slice 48:52 	 4329.0356
slice 52:56 	 431.72855
slice 56:60 	 0.0
slice 60:64 	 663.5157
slice 64:68 	 1484.2971
slice 68:72 	 753.3141
slice 72:76 	 594.09753
slice 76:80 	 4688.849
slice 80:84 	 353.21588
slice 84:88 	 0.0
slice 88:92 	 457.56436
slice 92:96 	 3652.3367
slice 96:100 	 818.3705
slice 100:104 	 5672.6484
slice 104:108 	 0.0
slice 108:112 	 1170.7343
slice 112:116 	 5348.7695
slice 116:120 	 2218.7273
slice 120:124 	 0.0
slice 124:128 	 2100.377

I dont know why, but all Lambda layers have output as the last 4 feature maps of previous layer.

Hi @qubvel
 python3 -m mmdnn.conversion.examples.keras.imagenet_test -p imagenet1k-resnext-50 -s mxnet -n converted_resnext50 -w mxnet_resnext50.npy -i ./seagull.jpg
``
It's a script for test the accuracy of the conversion, not a must. Error "Local variable 'labels' referenced before assignment" is because that there is no file 'mmdnn/conversion/examples/data/imagenet_1000.txt'. You need to check the newest code.

Lambda layer is for group convolution, you can check the converted code file. I test the accuracy of the conversion from resnext50 mxnet to keras. The error is e-6. It's right.

Hi @namizzz
Thanks for comment, could you please share converted model?
I just want to clarify following:

Should not this piece of code(lines 310-316) corrected as follows to avoid the Error Local variable 'labels' referenced before assignment (added indentation for for cycle), so in case i don`t have the file I will not get the error:

now:
                import os
                if os.path.exists(self.args.label):
                    with open(self.args.label, 'r') as f:
                        labels = [l.rstrip() for l in f]

                for i in top_indices:
                    print (labels[i - offset], i, predict[i])

corrected:
                import os
                if os.path.exists(self.args.label):
                    with open(self.args.label, 'r') as f:
                        labels = [l.rstrip() for l in f]

                    for i in top_indices:
                        print (labels[i - offset], i, predict[i])

I convert the model even without testing conversion error, but still have incorrect results.
My model (link) has result:
[(380, 0.48661867), (630, 0.32678068), (257, 0.065482974), (392, 0.045178358), (463, 0.029203746)]
correct result:
[(396, 0.7104751), (398, 0.122665755), (438, 0.06391319), (440, 0.029796895), (417, 0.019492012)]

converting script:
$ mmdownload -f mxnet -n imagenet1k-resnext-50 -o ./
$ mmconvert -sf mxnet -in resnext-50-symbol.json -iw resnext-50-0000.params -df keras -om resnext50.h5 --inputShape 3 224 224

Hi @qubvel,
Fixed the bug you mentioned. Thanks very much.
Reproduce the conversion with newest MMdnn(0.2.2) (the bug is not merged in newest pip package :(  )
$ mmdownload -f mxnet -n imagenet1k-resnext-50 -i mmdnn/conversion/examples/data/seagull.jpg
(396, 0.9810116)
(398, 0.008996904)
(409, 0.0026355518)
(438, 0.0025629073)
(440, 0.0015916944)

$ mmtoir -f mxnet -n ./resnext-50-symbol.json -w ./resnext-50-0000.params -d mxnet_resnext50 --inputShape 3 224 224
IR network structure is saved as [mxnet_resnext50.json].
IR network structure is saved as [mxnet_resnext50.pb].
IR weights are saved as [mxnet_resnext50.npy].

$ mmtocode -f keras -in mxnet_resnext50.pb -iw mxnet_resnext50.npy -d converted_resnext50.py
Parse file [mxnet_resnext50.pb] with binary format successfully.
Target network code snippet is saved as [converted_resnext50.py].

$ python3 -m mmdnn.conversion.examples.keras.imagenet_test -p imagenet1k-resnext-50 -s mxnet -n converted_resnext50 -w mxnet_resnext50.npy -i mmdnn/conversion/examples/data/seagull.jpg
(396, 0.9810091)
(398, 0.008996932)
(409, 0.0026355372)
(438, 0.0025629275)
(440, 0.0015917055)

slightly different from MXNet result.
But after saving to keras .h5 file and load it again, the prediction result is same with yours. The IR weights file is 145MB, but the .h5 file is only 99M. No idea about this problem, maybe cause by Lambda layer. We will try to figure it out.
Thanks for reporting the problem again!

Hi @kitstar
Thanks for the answer!
I checked it works correctly with loading weights from the *.npy and saving them to *.h5 just as weights (not as the whole model).
Thanks again, will wait for further updates with Lambda layer!


Add support for LabelPlacement of sld:TextSymbolizer

adds support for LabelPlacement: PointPlacement and LinePlacement.

Relevant tests pass in FF10 and Safari 5.1.3, TIA for any review.

I think lib/OpenLayers/Feature/.Vector.js.swp should not be part of the patch.

You're totally right.

This is ready for review again. Relevant tests are still passing.

@ahocevar any chance if you give a final look at this?

Looks good now. Thanks @bartvde. Please merge.


Move IOS_DEPLOYMENT_TARGET and MAC_DEPLOYMENT_TARGET to the command body

Summary:
This diff moves IOS_DEPLOYMENT_TARGET and MAC_DEPLOYMENT_TARGET definitions to the command body.
Changelog: [Internal]
Differential Revision: D55478563

07f38ea9f12aa4e4681829410abe052999120fe7


App Crashes when following the How to create your first chart

Followed the Beta 2 how to page and got a crash.
Description
Console Message:
SwiftUI/EnvironmentObject.swift:70: Fatal error: No ObservableObject of type ChartStyle found. A View.environmentObject(_:) for ChartStyle may be missing as an ancestor of this view.
Expected Behavior
See a line/ bar chart
Actual Behavior
App crashes
Possible Fix

Steps to Reproduce
Follow this:  https://github.com/AppPear/ChartView/wiki/How-to-create-your-first-chart
Your Environment

Version of this package used:  exact 2.0.0-beta.2
Device: iPhone 12 Pro
Operating System and version: 12.1 (21C52) - Xcode 13.2.1 (13C100)
Link to your project:

@YoelL I see that you closed this issue. What was the resolution? I am having the same crash.

I have the same crash situation ...

@YoelL I see that you closed this issue. What was the resolution? I am having the same crash.

I think the .chartStyle( ) modifier must need  to add.

Hi, yes it was probably that.


Make Latitude and Longitude evaluated

Closes #88

Since Latitude and Longitude are lazily evaluated, we have the issue described in the issue. 😅

please review


Make experimentalCompileTsWithBabel compatible with vue.config.js

The schema of vue.config.js is very specific in what it can accept.
For instance, it can accept custom options only in the pluginOptions param.
I put the experimentalCompileTsWithBabel back in it's place

Thanks for the PR - however, it is currently intentionally omitted from the main schema because this is by no means stable yet. We are waiting on the Babel side to stabilize before exposing this.

For now, if one chooses to use the experimental decorators, she will get an error.
Please try choosing options TypeScript > experimental decorators and then try to run yarn serve.
This PR makes it work without changing the vue.config.js.

I don't understand. The experimental flag has nothing to do with decorators and it's not exposed in CLI options. If you are talking about the class-syntax components, that seems to work as intended. Either way, if there is a bug please open an issue with reproduction.

Ooooops! I am so sorry, I guess writing pull request at 10 PM is not good for me.
I completely missed the process.env.VUE_CLI_EXPERIMENTAL?.
Brain fart.
Thank you for your time.


[component, core, rpc] Output retry timings in QueryTrace

Note: master currently doesn't pass 'verify' tests (JvmClusterQueryIT::distributedQueryTraceTest), so I haven't been able to verify 100% that this PR doesn't break anything.

@udoprog: Thanks for the code review. I've now improved the areas you mentioned, as well as fixing the AbstractClusterQueryIT::distributedQueryTraceTest test which had broken. What do you think?

Current coverage is 45.41% (diff: 54.54%)

Merging #178 into master will increase coverage by 0.01%

@@             master       #178   diff @@
==========================================
  Files           604        605     +1
  Lines         15631      15674    +43
  Methods           0          0
  Messages          0          0
  Branches       1601       1604     +3
==========================================
+ Hits           7096       7118    +22
- Misses         8074       8095    +21
  Partials        461        461

Powered by Codecov. Last update 0647212...db75805

@udoprog: Thanks again, have done some fixes now.


Missing Auth Headers in Success Response

I'm having an issue wherein the expected auth headers aren't being set, even though the request has been authenticated and there are no model validation errors (as reported in #884). I've determined the cause of the issue, but I'm not sure it's a bug. I wanted to open this issue to discuss potential workarounds or just note this behavior in case someone else runs into this in the future.
Roughly, here's what I'm seeing:

I have a controller action (#show) which can be accessed by an admin or an end-user
There are two before_actions which ensure that either an end-user or an admin is making the request. Handling this differently could potentially prevent the problem I'm reporting, so I'll include these below:

before_action :authenticate_admin!, only: [:show, :update], if: -> { current_client.nil? }
before_action :authenticate_client!, only: [:show, :update], if: -> { current_admin.nil? }

I have a conditional inside #show (if admin_signed_in?) which checks to see if an admin is making the request - in which case it uses a different serializer to render the response.

The culprit seems to be the admin_signed_in? helper call, which ends up unsetting the response auth headers (because it eventually calls warden.authenticate). As I mentioned above, that behavior would make sense in isolation, so I'm not going to call this a bug. In a perfect world, there'd be some warning about what was happening or there'd be a non-destructive helper I could call to determine which user is making the authenticated request.
Environment:

Ruby 2.4.2
Rails 5.1.3
Devise Token Auth 0.1.42
Devise 4.3.0

Thanks for reporting @ethagnawl . I'd definitely be down to include a warning at this stage!

@zachfeldman Cool. I will have a look at the codebase to see if that's feasible.
Otherwise, are you aware of any workarounds? I realize I should create a separate AdminClients controller, but I was hoping to find a temporary solution.

Ha! Look at that. That's great. I think we should do a warning and a link to this issue if you have a moment. Otherwise we could close this.


fix(cms): bypass form-data gql 2FA check

Root Cause
During file uploads, the GraphQL request includes a Content-Type header set to multipart/form-data but with an empty request body. This empty body leads to a parsing error in the gql-parser.
Solution
For multipart/form-data uploads, we bypass middleware that relies on req.body to avoid conflicts. Additionally, we don't use multer for parsing as it might disrupt the request processing.

Question:
我們會需要解析 gql query/mutation，並且檢視 query/mutation 內容的原因為何？
除了登入時用到的 gql mutation authenticateUserWithPassword 和 gql query authenticatedItem 之外，是不是只要檢視 request 有沒有帶合格的 keystonejs-session 和 keystonejs-2fa cookie 呢？

LGTM 👍 👍


Join problem when using Skip and Take

I have a query like this:
var data = context.Letters
	.Include(f => f.Referrals)
	.ThenInclude(f => f.Post)
	.Include(f => f.Attachments)
	.ThenInclude(f => f.Signs)
	.Include(f => f.Registerar)
	.Include(f => f.LetterChains)
	.Include(f => f.Secretariate)
	.Include(f => f.Zonecans)
	.ThenInclude(f => f.Zonecan)
	.Where(letter => !letter.Deleted && !letter.IsTemp);

This query returns a SQL command, something like following:
SELECT DISTINCT [r].[Id], ... , [r.Registerar].[UserName]
FROM [Letters] AS [r]
LEFT JOIN [Secretariates] AS [r.Secretariate] ON [r].[SecretariateId] = [r.Secretariate].[Id]
LEFT JOIN [AspNetUsers] AS [r.Registerar] ON [r].[RegisterarId] = [r.Registerar].[Id]
INNER JOIN (
    SELECT [f].*
    FROM [LetterToZonecans] AS [f]
    WHERE ([f].[Deleted] = 0) AND ([f].[ZonecanId] = 14)
) AS [t] ON [r].[Id] = [t].[LetterId]
WHERE (([r].[Deleted] = 0) AND ([r].[IsTemp] = 0)) AND (([r].[SecretariateId] = 1) AND ([r].[YearId] = 1))
ORDER BY [r].[RegisterDateTime] DESC, [r].[Id]

As you see, it joins to AspNetUsers and loads its columns. But when I add Skip and Take methods to the above  linq query, it doesn't join to the AspNetUsers table and finally, I get null values when I want to access to the Registrar properties of the Letter. Linq query with Skip and Take is as the following:
data = data.Distinct().Skip(paginationFrom).Take(paginationSize).OrderByDescending(f => f.RegisterDateTime);

SQL output of the above command:
SELECT DISTINCT [t0].[Id], ... , [t0].[YearId]
FROM (
    SELECT [r].[Id], ... , [r].[YearId]
    FROM [Letters] AS [r]
    INNER JOIN (
        SELECT [f].*
        FROM [LetterToZonecans] AS [f]
        WHERE ([f].[Deleted] = 0) AND ([f].[ZonecanId] = 14)
    ) AS [t] ON [r].[Id] = [t].[LetterId]
    WHERE (([r].[Deleted] = 0) AND ([r].[IsTemp] = 0)) AND (([r].[SecretariateId] = 1) AND ([r].[YearId] = 1))
    ORDER BY (SELECT 1)
    OFFSET 0 ROWS FETCH NEXT 100 ROWS ONLY
) AS [t0]
ORDER BY [t0].[RegisterDateTime] DESC

Am I missing something or there is an issue withSkip and Take?

blocked by #15611

@AminEsmaeily - We don't have enough code here to reproduce the error and see if it is fixed in latest release. Though in EF Core 3.1 we have redesigned whole query pipeline and believe that this issue should go away. I am closing this issue as fixed. If you still see the error after upgrading to EF Core 3.1, we will reopen and investigate.

@maumar - If you want to try repro this and add a regression test, feel free to reopen.


Inline Time picker stop working

The time picker stop working for inline model.

Input changes not set the value
The up / down error stop to response

https://jsfiddle.net/jerry153fish/Lc4s6n20/
Your Environment

flatpickr version used: 4.5.2
Browser name and version: Version 72.0.3626.109 (Official Build) (64-bit)
OS and version: MacOS High Sierra 10.13.6

Confirmed (functionality stopped working between v4.3.2 and v4.4.0)
Bug appears with the following config:
{
    inline: true,
    enableTime: true,
    noCalendar: true,
}

With the calendar shown, everything works as expected


Fix: Fixed an issue where an invalid Git Remote would cause a crash

Resolved / Related Issues

[x] Were these changes approved in an issue or discussion with the project maintainers? In order to prevent extra work, feature requests and changes to the codebase must be approved before the pull request will be reviewed. This prevents extra work for the contributors and maintainers.
Closes #13048

Possible Improvements
At the moment this PRs only prevent the crash.

Should we show a warning when there's an invalid remote?
If the invalid remote is the current head, what should we do? Checkout to the first valid branch? Do not show any git info at all? (If the head is invalid we can't get its name and all the other infos [we do not know which branch is the head])

Validation
How did you test these changes?

[x] Did you build the app and test your changes?
[ ] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[x] Did you implement any design changes to an existing feature?

[x] Was this change approved?

[x] Are there any other steps that were used to validate these changes?

Open app
Invalidate a git repo (I changed the last component of ref: refs/... to a . of the file .git/HEAD  )
Open that git repo

Possible Improvements
At the moment this PRs only prevent the crash.

Should we show a warning when there's an invalid remote?
If the invalid remote is the current head, what should we do? Checkout to the first valid branch? Do not show any git info at all? (If the head is invalid we can't get its name and all the other infos [we do not know which branch is the head])

@yaira2 what do you think?

If the remote is invalid, do the local branches still work?

If the remote is invalid, do the local branches still work?

At the moment I can only reproduce the bug when the HEAD branch is invalid. In this case you can't do anything: you have no active branch (if the branch is invalid you can't check if it's the HEAD) and so git status bar is not shown

@yaira2 I did some more testing:

if the head is invalid we can't do anything (except showing a warning) [see above]
if there are issues with a remote, after handling the exception, everything works well local-side (you can switch branch and the repo seems fine, but you can't get ahead by/behind by). I'll check what happens when you try to fetch/push/pull

if the head is invalid we can't do anything (except showing a warning) [see above]
if there are issues with a remote, after handling the exception, everything works well local-side (you can switch branch and the repo seems fine, but you can't get ahead by/behind by).
I'll check what happens when you try to fetch/push/pull ->It fails and shows git pull/push timeout error, we should add a specific warning

Do you have any better ideas?

Let's go with handling the exception without displaying a warning. If it becomes confusing in the future, we can add a warning.

@ferrariofilippo can you fix the merge conflict?


Replace apparently unsupported ~~ MD strike with

What's changed, or what was fixed?

Target Live Date: any
CC: @petele

Whoops!
There were 1 critical errors that broke the build and prevented it from being automatically deployed.
There were 5 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
ERRORS
src/content/en/updates/2015/04/cut-and-copy-commands.md - No 'wf_blink_components' found, see https://goo.gl/VXmg9e
WARNINGS
src/content/en/updates/2015/04/cut-and-copy-commands.md#L148 - Consider adding a "was this page helpful?" widget to your page: https://developers.google.com/web/resources/widgets#helpful
src/content/en/updates/2015/04/cut-and-copy-commands.md#L148 - Updates post is missing RSS feed widget: '{% include "web/_shared/rss-widget-updates.html" %}'
src/content/en/updates/2015/04/cut-and-copy-commands.md#L5 - WF Tag 'wf_updated_on' must be within the last 7 days.
src/content/en/updates/2015/04/cut-and-copy-commands.md#L130 - Line must be at most 100 characters
src/content/en/updates/2015/04/cut-and-copy-commands.md#L142 - Line must be at most 100 characters

:tada: This has been pushed live to https://developers.google.com/web/


no tests for filters, which are broken

I think I broke filtering when I switched from docopt to argparse for plumbum. The filter fix is pretty simple, and a test to prevent repeating this breakage would be great.

I have a fix & tests in the pipeline for as soon as #42 is merged.

Addressed with merge of #46.


The requested route [/eventstream/clip/v2] has not been mapped in Spark for Accept: [text/event-stream]

I recently installed the HA Bridge and I am getting "The requested route [/eventstream/clip/v2] has not been mapped in Spark for Accept: [text/event-stream]" written to the log every 5 seconds. Is there a way to stop this?
I know that this question has been asked before but I do not see a solution.

Some device/program is sending a request to the habridge that it does not know about. You will need to find the offending program but it does not hurt the habridge nor is it an error, just informational.

Home Assistant's Hue integration causes this for one - I've seen it literally 5 minutes ago, when I made a quick test to see if HA connects to HA Bridge properly.


discussion about extracting backend output json file

There is a discussion to extract the contents from the json file that is the backend output and visualize it.

Is it visualizing by extracting the contents of the json file by importing the json path from the .cfg file?
Is it possible to directly open the json file and extract the contents?

We need to discuss which direction to go in.

CC @llFreetimell

@yesfordev
If I understand correctly, "visualizing by extracting the contents of the json file by importing the json path from the .cfg file" was what I first thought.

Get json path from .cfg
read content in json
Visualize or do something other

But I am still curious what is "directly open the json file and extract the contents" T.T

Get json path from .cfg

Just curious... in which part of .cfg do we get JSON path?

@seanshpark
I am thinking something like picture in https://github.com/Samsung/ONE-vscode/issues/8#issue-976300410.

I am thinking something like picture in #8 (comment).

Thanks, path gave me some misunderstanding :)
For me, it seems the file name. path = folder + filename , where we need to ask to system to get the folder...

When I open a file and run extension, I have the function to read open files in extension. So I was wondering if I should load the contents with the json file open.
But now I know it's from a .cfg file, so I'll do it! :)

@yesfordev
I think we can close this issue :)
For further discussion, they can be handled at #28 !

Closing as not active. Feel free to reopen if needed.


RELATED: BB-2381 drill to url

Currently destined for release R208.
This release is zero downtime release.
Code drop: 2020-07-02 00:00.

Story BB-2381 Allow to post message with URL instead of redirecting the frame in embedded mode
Status: Hacking
Priority: Blocker
Assignee: Jan Lipnican

Build failed (check pipeline).

https://checklist.intgdc.com/job/client-libs/job/gooddata-typings-pull-request-compliance-check/117/ : SUCCESS in 8s
https://checklist.intgdc.com/job/client-libs/job/gooddata-typings-unit-tests-zuul-docker-el7/201/ : FAILURE in 59s

Build succeeded (check pipeline).

https://checklist.intgdc.com/job/client-libs/job/gooddata-typings-pull-request-compliance-check/118/ : SUCCESS in 5s
https://checklist.intgdc.com/job/client-libs/job/gooddata-typings-unit-tests-zuul-docker-el7/202/ : SUCCESS in 1m 13s

Build succeeded (check pipeline).

https://checklist.intgdc.com/job/client-libs/job/gooddata-typings-pull-request-compliance-check/119/ : SUCCESS in 6s
https://checklist.intgdc.com/job/client-libs/job/gooddata-typings-unit-tests-zuul-docker-el7/203/ : SUCCESS in 2m 34s

Build succeeded (gate pipeline).

https://checklist.intgdc.com/job/client-libs/job/gooddata-typings-pull-request-compliance-gate/42/ : SUCCESS in 7s
https://checklist.intgdc.com/job/client-libs/job/gooddata-typings-unit-tests-zuul-docker-el7/204/ : SUCCESS in 49s
https://checklist.intgdc.com/job/compliance/job/gooddata-typings-fossa-scanning-zuul-el7/6/ : SUCCESS in 13s (non-voting)


Azure Function App - Monitor Logs - App Service Plan

For Function Apps there is the new diagnostic settings where you can send User Logs to Log Analytics.
Is this available for Azure Functions using an App Service Plan or only Consumption plan?

@brianmander
Please help with the link of document on which above document feedback request is created.
In case your query in not related to document content, open a new forum thread in MSDN forum or Stack Overflow as it could be better place to get help on your scenario. These forum community members could provide their expert guidance on your scenario based on their experience. Thanks.

This is in reference to the following link:
https://docs.microsoft.com/en-us/azure/azure-functions/functions-monitor-log-analytics?tabs=csharp

Closing as duplicate of #53638


Support compression type "SNAPPY" in Tensorflow Python API

System information

TensorFlow version (you are using): 2.5.0
Are you willing to contribute it (Yes/No): Yes

Describe the feature and the current behavior/state.
From https://www.tensorflow.org/api_docs/python/tf/io/TFRecordOptions?hl=en, it does not appear that TensorFlow (up till v2.6.0) supports "SNAPPY" as a compression type in the Python API. However, in c++ source code https://github.com/tensorflow/tensorflow/blob/master/tensorflow/core/lib/io/record_writer.h#L44, this is already supported. For my use case, "SNAPPY" is preferred over "ZLIB" or "GZIP". Therefore, having "SNAPPY" as an option in Python TFRecord API is a desirable feature.
Will this change the current api? How?
Yes. The compression_type field of TFRecordOptions will have valid values of "GZIP", "ZLIB", "SNAPPY"(new) or "" (no compression).
Who will benefit with this feature?
Users who want to use snappy as their compression algorithm in order to take advantage of its speed.
Any Other info.

Hi!@zxiu2049 !We  see that the issue template has not been filled, Could you please do so as it helps us analyse the issue .

Hi!@zxiu2049 !We see that the issue template has not been filled, Could you please do so as it helps us analyse the issue .

Hi @mohantym ! Thanks for the response. Modified the description based on the feature request template. Please let me know if that works.

Hi @Saduf2019  ,Could you please look into this issue ?

Something like this works in the meantime until it's officially added:
def enable_snappy_compression():
    """
    Workaround Snappy not being a public option yet.
    """
    tf.io.TFRecordOptions.compression_type_map[3] = "SNAPPY"

and then just:
tf.data.TFRecordDataset(file, compression_type="SNAPPY")

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


ng serve keep on building

Note: for support questions, please use one of these channels:
https://forum.ionicframework.com/
http://ionicworldwide.herokuapp.com/
Short description of the problem:
ng serve keep on building....
And browser reload each time.
However i don't modify files.
What behavior are you expecting?
Steps to reproduce:

ng serve

insert any relevant code between the above and below backticks
C:\Applications\Node\exotica>ionic start
? What would you like to name your project: front
? What starter would you like to use: blank
[INFO] Fetching app base
       (https://github.com/ionic-team/ionic2-app-base/archive/master.tar.gz)
V Downloading - done!
[INFO] Fetching starter template blank
       (https://github.com/ionic-team/ionic2-starter-blank/archive/master.tar.gz
)
V Downloading - done!
V Updating package.json with app details - done!
V Creating configuration file ionic.config.json - done!
[INFO] Installing dependencies may take several minutes!
> npm install
V Running command - done!
> npm install --save-dev --save-exact ionic@latest
V Running command - done!
> npm install --save-dev --save-exact @ionic/cli-plugin-ionic-angular@latest
V Running command - done!
> npm dedupe
V Running command - done!
> git init
V Running command - done!
> git add -A
V Running command - done!
> git commit -m "Initial commit" --no-gpg-sign
V Running command - done!

? ♫ ? ♫  Your Ionic app is ready to go! ? ♫ ? ♫

Run your app in the browser (great for initial development):
  ionic serve

Run on a device or simulator:
  ionic cordova run ios

Test and share your app on a device with the Ionic View app:
  http://view.ionic.io

Next Steps:
Go to your newly created project: cd .\front

C:\Applications\Node\exotica\front>ionic serve
[INFO] Starting app-scripts server: --port 8100 --p 8100 --livereload-port 35729

       --r 35729 --address <IP_ADDRESS> - Ctrl+C to cancel
[15:55:52]  watch started ...
[15:55:52]  build dev started ...
[15:55:52]  clean started ...
[15:55:52]  clean finished in 5 ms
[15:55:52]  copy started ...
[15:55:52]  transpile started ...
[15:55:56]  transpile finished in 3.62 s
[15:56:05]  preprocess started ...
[15:56:05]  deeplinks started ...
[15:56:05]  deeplinks finished in 78 ms
[15:56:05]  preprocess finished in 82 ms
[15:56:05]  webpack started ...
[15:56:05]  copy finished in 13.22 s
[15:56:13]  webpack finished in 8.26 s
[15:56:24]  sass started ...
[15:56:25]  sass finished in 1.18 s
[15:56:25]  postprocess started ...
[15:56:25]  postprocess finished in 5 ms
[15:56:25]  lint started ...
[15:56:25]  build dev finished in 33.31 s
[15:56:26]  watch ready in 33.51 s
[15:56:26]  dev server running: http://localhost:8100/

[INFO] Development server running
       Local: http://localhost:8100

[15:56:27]  build started ...
[15:56:27]  transpile update started ...
[15:56:27]  transpile update finished in 144 ms
[15:56:27]  deeplinks update started ...
[15:56:27]  deeplinks update finished in 46 ms
[15:56:27]  webpack update started ...
[15:56:28]  webpack update finished in 924 ms
[15:56:32]  lint finished in 6.62 s
[15:56:33]  build finished in 6.34 s

[15:56:33]  build started ...
[15:56:33]  transpile update started ...
[15:56:34]  transpile update finished in 482 ms
[15:56:34]  deeplinks update started ...
[15:56:34]  deeplinks update finished in 43 ms
[15:56:34]  webpack update started ...
[15:56:38]  webpack update finished in 4.19 s
[15:56:38]  build finished in 4.78 s

[15:56:38]  build started ...
[15:56:38]  transpile update started ...
[15:56:38]  transpile update finished in 339 ms
[15:56:38]  deeplinks update started ...
[15:56:38]  deeplinks update finished in 40 ms
[15:56:38]  webpack update started ...
[15:56:39]  webpack update finished in 617 ms
[15:56:42]  build finished in 3.60 s

[15:56:42]  build started ...
[15:56:42]  transpile update started ...
[15:56:42]  transpile update finished in 488 ms
[15:56:42]  deeplinks update started ...
[15:56:42]  deeplinks update finished in 151 ms
[15:56:43]  webpack update started ...
^CTerminer le programme de commandes (O/N) ?

Which @ionic/app-scripts version are you using?
"ionic/app-scripts": "2.1.3",
{
  "name": "front",
  "version": "0.0.1",
  "author": "Ionic Framework",
  "homepage": "http://ionicframework.com/",
  "private": true,
  "scripts": {
    "clean": "ionic-app-scripts clean",
    "build": "ionic-app-scripts build",
    "lint": "ionic-app-scripts lint",
    "ionic:build": "ionic-app-scripts build",
    "ionic:serve": "ionic-app-scripts serve"
  },
  "dependencies": {
    "@angular/common": "4.1.3",
    "@angular/compiler": "4.1.3",
    "@angular/compiler-cli": "4.1.3",
    "@angular/core": "4.1.3",
    "@angular/forms": "4.1.3",
    "@angular/http": "4.1.3",
    "@angular/platform-browser": "4.1.3",
    "@angular/platform-browser-dynamic": "4.1.3",
    "@ionic-native/core": "3.12.1",
    "@ionic-native/splash-screen": "3.12.1",
    "@ionic-native/status-bar": "3.12.1",
    "@ionic/storage": "2.0.1",
    "ionic-angular": "3.6.0",
    "ionicons": "3.0.0",
    "rxjs": "5.4.0",
    "sw-toolbox": "3.6.0",
    "zone.js": "0.8.12"
  },
  "devDependencies": {
    "@ionic/app-scripts": "2.1.3",
    "@ionic/cli-plugin-ionic-angular": "1.4.0",
    "ionic": "3.6.0",
    "typescript": "2.3.4"
  },
  "description": "An Ionic project"
}

Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)

Antivirus issue


INSTALL script not installing java in Debian

Installing java for Debian in Linux installer.
Fixes issue #1420
Brief summary of changes
The installer was not installing Java in Debian. Not it installs it.
Testing I've completed
Tested locally in Debian 11.
CHANGELOG.md (choose one)

no need to update because this is an installer issue.

I have tested this again and it is working in both Ubuntu and Debian.


Create open_redirect_phoenixartstudio.yml

Description
Newly Observed Open Redirect
Associated samples

Sample 1

Associated hunts

Hunt 1

/update-test-rules

on telemetry yet, but has been observed ITW.  ready for review.


Fix display of sidetext gates with conditions

Summary
Fixes #7532
Details and comments
This PR fixes a problem where the gates with sidetext, such as CPhaseGate and RZZGate did not line up properly in the text drawer when a condition was set on the gate.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

6 of 6 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 83.331%

Totals

Change from base Build<PHONE_NUMBER>:
0.002%

Covered Lines:
52216

Relevant Lines:
62661

💛  - Coveralls

mpl and latex tests added in https://github.com/Qiskit/qiskit-terra/pull/7673/commits/b706676938b208a97748bf126ac81b5210939712.

Sorry Ed I actually meant can you add some more text drawer tests but for different types of sidetext gates. All of these test the CPhase gate but would be great to test some of the others (e.g. those mentioned in the other issue) as well

@javabster I added text drawer tests for cu1 and rzz. I left the mpl and latex tests in even though they work fine, since we didn't have any sidetext with conditions tests for those drawers


Typo fix to 'server basics' doc.

Removed non-existing selection annotations.
Fixing code errors (parentheses and braces).
Adding a missing import statement.

@krisjey Gentle ping

@krisjey Geentle ping

Superceded by #1041


Updates to data import/export steps

Allow import/export of users.owner model
Allow export of API tokens (optional, default=False)
Fixes https://github.com/inventree/InvenTree/issues/6127

This has now broken unit tests for postgresql. I guess that is why we exclude users.owner from the exported dataset.. will need a patch to fix


feat: Including new approved_date and user_id into Seedlot table and Audit table and trigger

Including approved_timestamp and approved_userid into seedlot table and seedlot_audit table.
Updating trigger to include some missing fields to be updated in the audit table
Description
Closes #850
Changelog
New

Changed

Removed

How was this tested?

[ ] 🧠 Not needed
[ ] 👀 Eyeball
[x] 🤖 Added tests

What gif/image best describes this PR or how it makes you feel?

Thanks for the PR!
Any successful deployments (not always required) will be available below.
Frontend
Backend
Oracle-API
Once merged, code will be promoted and handed off to following workflow run.
Main Merge Workflow

Hi @craigyu . I think it would be nice to tackle this in a bug issue. I could use it to do more tests over audit table.
Not sure if is this case, but if you try to update the same data twice in SEEDLOT table, it will be generate 2 new rows in seedlot_audit table, as it will store the whole history of the record, even if the update statement is for the same data (for audit purposes it is needed to identify system failures/unexpected behaviours, for example, when we have a double update in a process or 2 different processes updating the same data in a record).


fix(issues): Fixed some visual issues

Fixed the visual issues that were introduced with the latests changes

:tada: This PR is included in version 1.1.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


When running npx terminalgpt chat command  in

When running npx terminalgpt chat command in command prompt or PowerShell terminals,  get "Something went wrong!!  error: request failed with status code 401 Something went wrong!" after entering API key

I am having the same error as well. I have tried changing the API secret key but to now avail.

Thanks for the feedbacks. I don't have a way to test on powershell atm, but I'll create a way to create a check log where users can copy and paste here. I believe it would add more context and make it able to fix the issue.
In the meanwhile , any PR solving this problem will be very welcome .

This happens in zsh shell on macOS as well. Maybe it's not a Powershell problem

I think I found the problem. Inside apiKeyPrompt we are returning the encrypted key, not the original one. I will open a PR for this

Ok thanks
Dennis McMahon
CEO, MCM GreenMan Group / Global Buildtech
www.global-buildtech.weebly.com
Ph/ WhatsApp +60169839715
E: @.***
LinkedIn: www.linkedin.com/in/denniskl
On Sat, 7 Jan 2023, 18:52 Son Nguyen, @.***> wrote:

I think I found the problem. Inside apiKeyPrompt we are returning the
encrypted key, not the original one. I will open a PR for this
[image: Screenshot 2023-01-07 at 10 51 32 a m]
https://user-images.githubusercontent.com/28614996/211146580-c48de037-18b5-426a-8fac-bdc54110afde.png
—
Reply to this email directly, view it on GitHub
https://github.com/jucasoliveira/terminalGPT/issues/15#issuecomment-1374437285,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJTUYPYNDAQTF4DGTEOBQVLWRFDG3ANCNFSM6AAAAAATTSAIEQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

I think I found the problem. Inside apiKeyPrompt we are returning the encrypted key, not the original one. I will open a PR for this

Thanks for find this! I'll be reviewing your PR asap:)


eslint: Is yarn workspace supported?

Description
After banging my head on the wall several times. eslint still refuses to work inside a yarn workspace ;)
In the project there is no .eslintrc in the root instead config resides inside their own package.
Neovim version
NVIM v0.6.0-dev+509-g09e96fe60
Build type: Release
LuaJIT 2.1.0-beta3
Nvim-lspconfig version
c7cef85
Operating system and version
Linux 5.14.10-1-MANJARO
Affected language servers
eslint
Steps to reproduce

Clone the repo https://github.com/numToStr/eslint-issue
Run nvim -nu minimal.lua
Open packages/server/index.ts and wait for a bit.
Now you'll have the error.

Also adding output of :LspInfo
 Language client log: /home/hello/.cache/nvim/lsp.log
 Detected filetype:   typescript

 2 client(s) attached to this buffer:

 Client: tsserver (id: 1, pid: 357842, bufnr: [1])
 	filetypes:       javascript, javascriptreact, javascript.jsx, typescript, typescriptreact, typescript.tsx
 	autostart:       true
 	root directory:  /home/hello/Documents/code/eslint-issue/packages/server
 	cmd:             typescript-language-server --stdio

 Client: eslint (id: 2, pid: 357848, bufnr: [1])
 	filetypes:       javascript, javascriptreact, javascript.jsx, typescript, typescriptreact, typescript.tsx, vue
 	autostart:       true
 	root directory:  /home/hello/Documents/code/eslint-issue/packages/server
 	cmd:             vscode-eslint-language-server --stdio

 Configured servers list: tsserver, eslint

Actual behavior
Getting false diagnostics error on a project bootstrapped with yarn workspaces.
Screenshot

Expected behavior
No diagnostics errors. As a side note, I was using Coc.nvim before there were no errors reported.
Minimal config
local on_windows = vim.loop.os_uname().version:match("Windows")

local function join_paths(...)
	local path_sep = on_windows and "\\" or "/"
	local result = table.concat({ ... }, path_sep)
	return result
end

vim.cmd([[set runtimepath=$VIMRUNTIME]])

local temp_dir
if on_windows then
	temp_dir = vim.loop.os_getenv("TEMP")
else
	temp_dir = "/tmp"
end

vim.cmd("set packpath=" .. join_paths(temp_dir, "nvim", "site"))

local package_root = join_paths(temp_dir, "nvim", "site", "pack")
local install_path = join_paths(package_root, "packer", "start", "packer.nvim")
local compile_path = join_paths(install_path, "plugin", "packer_compiled.lua")

local function load_plugins()
	require("packer").startup({
		{
			"wbthomason/packer.nvim",
			"neovim/nvim-lspconfig",
		},
		config = {
			package_root = package_root,
			compile_path = compile_path,
		},
	})
end

_G.load_config = function()
	vim.lsp.set_log_level("trace")
	if vim.fn.has("nvim-0.5.1") == 1 then
		require("vim.lsp.log").set_format_func(vim.inspect)
	end
	local nvim_lsp = require("lspconfig")

	local flags = {
		allow_incremental_sync = true,
		debounce_text_changes = 200,
	}

	local function mappings(bufnr)
		local function buf_set_keymap(...)
			vim.api.nvim_buf_set_keymap(bufnr, ...)
		end
		local function buf_set_option(...)
			vim.api.nvim_buf_set_option(bufnr, ...)
		end

		buf_set_option("omnifunc", "v:lua.vim.lsp.omnifunc")

		-- Mappings.
		local opts = { noremap = true, silent = true }
		buf_set_keymap("n", "gD", "<Cmd>lua vim.lsp.buf.declaration()<CR>", opts)
		buf_set_keymap("n", "gd", "<Cmd>lua vim.lsp.buf.definition()<CR>", opts)
		buf_set_keymap("n", "K", "<Cmd>lua vim.lsp.buf.hover()<CR>", opts)
		buf_set_keymap("n", "gi", "<cmd>lua vim.lsp.buf.implementation()<CR>", opts)
		buf_set_keymap("n", "<C-k>", "<cmd>lua vim.lsp.buf.signature_help()<CR>", opts)
		buf_set_keymap("n", "<space>wa", "<cmd>lua vim.lsp.buf.add_workspace_folder()<CR>", opts)
		buf_set_keymap("n", "<space>wr", "<cmd>lua vim.lsp.buf.remove_workspace_folder()<CR>", opts)
		buf_set_keymap("n", "<space>wl", "<cmd>lua print(vim.inspect(vim.lsp.buf.list_workspace_folders()))<CR>", opts)
		buf_set_keymap("n", "<space>D", "<cmd>lua vim.lsp.buf.type_definition()<CR>", opts)
		buf_set_keymap("n", "<space>rn", "<cmd>lua vim.lsp.buf.rename()<CR>", opts)
		buf_set_keymap("n", "gr", "<cmd>lua vim.lsp.buf.references()<CR>", opts)
		buf_set_keymap("n", "<space>e", "<cmd>lua vim.lsp.diagnostic.show_line_diagnostics()<CR>", opts)
		buf_set_keymap("n", "[d", "<cmd>lua vim.lsp.diagnostic.goto_prev()<CR>", opts)
		buf_set_keymap("n", "]d", "<cmd>lua vim.lsp.diagnostic.goto_next()<CR>", opts)
		buf_set_keymap("n", "<space>q", "<cmd>lua vim.lsp.diagnostic.set_loclist()<CR>", opts)
	end

	local function disable_formatting(client)
		client.resolved_capabilities.document_formatting = false
		client.resolved_capabilities.document_range_formatting = false
	end

	-- Add the server that troubles you here
	-- Typescript
	nvim_lsp.tsserver.setup({
		flags = flags,
		on_attach = function(client, buf)
			disable_formatting(client)
			mappings(buf)
		end,
	})

	-- Eslint
	nvim_lsp.eslint.setup({
		flags = flags,
		on_attach = function(client)
			disable_formatting(client)
		end,
		settings = {
			useESLintClass = true, -- Recommended for eslint >= 7
			packageManager = "yarn", -- I use `yarn` btw
			-- run = 'onSave', -- Run `eslint` after save
		},
		root_dir = require('lspconfig.util').find_git_ancestor,
	})

	print(
		[[You can find your log at $HOME/.cache/nvim/lsp.log. Please paste in a github issue under a details tag as described in the issue template.]]
	)
end

if vim.fn.isdirectory(install_path) == 0 then
	vim.fn.system({ "git", "clone", "https://github.com/wbthomason/packer.nvim", install_path })
	load_plugins()
	require("packer").sync()
	vim.cmd([[autocmd User PackerComplete ++once lua load_config()]])
else
	load_plugins()
	require("packer").sync()
	_G.load_config()
end

Health check
lspconfig: require("lspconfig.health").check()
========================================================================
  - INFO: tsserver: configuration checked.
  - INFO: eslint: configuration checked.

LSP log
https://gist.github.com/numToStr/cade1192c61965071a1069fdabf98ab7

This issue is not neovim specific. vscode-eslint has special option eslint.workingDirectories to address it.
This worked for me with your test repo
  lspconfig.eslint.setup {
    settings = {
      format = false,
      workingDirectory = {
        mode = 'location'
      }
    },
    root_dir = lspconfig_util.find_git_ancestor
  }

@vatosarmat should that be workingDirectories instead of workingDirectory? Should this be a default setting?

Also seems like this should be the default?
 instructs ESLint to uses the workspace folder location or the file location (if no workspace folder is open) as the working directory. **This is the default** and is the same strategy as used in older versions of the ESLint extension (1.9.x versions).

This issue is not neovim specific. vscode-eslint has special option eslint.workingDirectories to address it. This worked for me with your test repo
  lspconfig.eslint.setup {
    settings = {
      format = false,
      workingDirectory = {
        mode = 'location'
      }
    },
    root_dir = lspconfig_util.find_git_ancestor
  }

According to vscode-eslint readme workingDirectories suppose to be an array. I also tried adding workingDirectories but didn't get it to working.
  lspconfig.eslint.setup {
    settings = {
      format = false,
      workingDirectories = {
        { mode = 'location' }
      }
    },
    root_dir = lspconfig_util.find_git_ancestor
  }

What's the version of eslint-language-server you are using?

It was latest vscode-languageservers-extracted.
(Can't check exact version, not on my desk)

My version
  "name": "vscode-langservers-extracted",
  "version": "3.0.0",

And it actually has workingDirectory option
~/.asdf/installs/nodejs/16.10.0/.npm/lib/node_modules/vscode-langservers-extracted/dist/eslint-language-server$ rg workingDirectory
eslintServer.js
733:        const settings = Object.assign({}, configuration, { silent: false, library: undefined, resolvedGlobalPackageManagerPath: undefined }, { workingDirectory: undefined });
740:        const hasUserDefinedWorkingDirectories = configuration.workingDirectory !== undefined;
741:        const workingDirectoryConfig = (_a = configuration.workingDirectory) !== null && _a !== void 0 ? _a : { mode: ModeEnum.location };
742:        if (ModeItem.is(workingDirectoryConfig)) {
744:            if (workingDirectoryConfig.mode === ModeEnum.location) {
752:            else if (workingDirectoryConfig.mode === ModeEnum.auto) {
761:                settings.workingDirectory = { directory: candidate };
765:            settings.workingDirectory = workingDirectoryConfig;
782:        if (moduleResolveWorkingDirectory === undefined && settings.workingDirectory !== undefined && !settings.workingDirectory['!cwd']) {
783:            moduleResolveWorkingDirectory = settings.workingDirectory.directory;
1281:        if (settings.workingDirectory) {
1282:            newOptions.cwd = normalizeDriveLetter(settings.workingDirectory.directory);
1283:            if (settings.workingDirectory['!cwd'] !== true && fs.existsSync(settings.workingDirectory.directory)) {
1284:                process.chdir(settings.workingDirectory.directory);

Is the upstream documentation wrong maybe? We can make this config the default for now.

I think this is just a mismatch between the extension and server documentation. @numToStr can you try with @vatosmart's settings? We can make this the default.

It is more likely that I(and<EMAIL_ADDRESS>am just having outdated version and they changed workingDirectory to array in the latest one. If OP's version is also old he should also use Directory instead Directories I think.

It seems like this should be the default that https://github.com/microsoft/vscode-eslint/blob/cbb35d03497bb39043033defbad7fd80b2a89698/server/src/eslintServer.ts#L1093

See:
❯ rg workingDirectory
client/src/extension.ts
219:	workingDirectory: ModeItem | DirectoryItem | undefined;
1144:							workingDirectory: undefined,
1173:							let workingDirectory: ModeItem | DirectoryItem | undefined = undefined;
1195:									workingDirectory = entry;
1232:									if (workingDirectory === undefined || ModeItem.is(workingDirectory)) {
1233:										workingDirectory = { directory: itemValue, '!cwd': noCWD };
1235:										if (workingDirectory.directory.length < itemValue.length) {
1236:											workingDirectory.directory = itemValue;
1237:											workingDirectory['!cwd'] = noCWD;
1242:							settings.workingDirectory = workingDirectory;

server/src/eslintServer.ts
233:	workingDirectory: ModeItem | DirectoryItem | undefined;
238:	workingDirectory: DirectoryItem | undefined;
1084:			{ workingDirectory: undefined}
1092:		const hasUserDefinedWorkingDirectories: boolean = configuration.workingDirectory !== undefined;
1093:		const workingDirectoryConfig = configuration.workingDirectory ?? { mode: ModeEnum.location };
1094:		if (ModeItem.is(workingDirectoryConfig)) {
1096:			if (workingDirectoryConfig.mode === ModeEnum.location) {
1102:			} else if (workingDirectoryConfig.mode === ModeEnum.auto) {
1110:				settings.workingDirectory = { directory: candidate };
1113:			settings.workingDirectory = workingDirectoryConfig;
1127:		if (moduleResolveWorkingDirectory === undefined && settings.workingDirectory !== undefined && !settings.workingDirectory['!cwd']) {
1128:			moduleResolveWorkingDirectory = settings.workingDirectory.directory;
1653:		if (settings.workingDirectory) {
1654:			newOptions.cwd = normalizeDriveLetter(settings.workingDirectory.directory);
1655:			if (settings.workingDirectory['!cwd'] !== true && fs.existsSync(settings.workingDirectory.directory)) {
1656:				process.chdir(settings.workingDirectory.directory);

and
❯ rg workingDirectories
client/src/extension.ts
581:	private workingDirectories: MigrationData<(string | DirectoryItem)[]>;
592:		this.workingDirectories = MigrationData.create(this.eslintConfig.inspect<(string | DirectoryItem)[]>('workingDirectories'));
722:		record(this.workingDirectories.global);
723:		record(this.workingDirectories.workspace);
724:		record(this.workingDirectories.workspaceFolder);
731:			MigrationData.needsUpdate(this.workingDirectories)
779:			await _update(this.eslintConfig, 'workingDirectories', this.workingDirectories.global, ConfigurationTarget.Global);
780:			await _update(this.eslintConfig, 'workingDirectories', this.workingDirectories.workspace, ConfigurationTarget.Workspace);
781:			await _update(this.eslintConfig, 'workingDirectories', this.workingDirectories.workspaceFolder, ConfigurationTarget.WorkspaceFolder);
1171:						const workingDirectories = config.get<(string | LegacyDirectoryItem | DirectoryItem | PatternItem | ModeItem)[] | undefined>('workingDirectories', undefined);
1172:						if (Array.isArray(workingDirectories)) {
1175:							for (const entry of workingDirectories) {

workingDirectories appears to only be used in the extension (vscode) not the server.

Github code search confused me. For some reason, it showed only 2 code results in microsoft/vscode-eslint for workingDirectory query

Yes, Github code search is laughably bad for something you'd expect to be core to its mission. You might give https://sourcegraph.com a try.

I think this might be why it needs to be sent in settings and the default fallback results in undefined behavior:
https://github.com/microsoft/vscode-eslint/blob/cbb35d03497bb39043033defbad7fd80b2a89698/server/src/eslintServer.ts#L1092

@williamboman @entropitor does this make sense to make a default?
 settings = {
      workingDirectories = {
        { mode = 'location' }
      }

To be honest, I don't know exactly what that option does. I've been using it in yarn workspaces just fine

@williamboman @entropitor does this make sense to make a default?

Hmm yes, but I think workingDirectory is still what the field is called though. workingDirectories seem to be a VSCode client thingy. Also, might as well omit the entry altogether from lspconfig, because it location seems to be the default setting :shrug:?
diff --git a/lua/lspconfig/server_configurations/eslint.lua b/lua/lspconfig/server_configurations/eslint.lua
index 71a6b38..84cabf5 100644
--- a/lua/lspconfig/server_configurations/eslint.lua
+++ b/lua/lspconfig/server_configurations/eslint.lua
@@ -81,10 +81,6 @@ return {
       -- nodePath configures the directory in which the eslint server should start its node_modules resolution.
       -- This path is relative to the workspace folder (root dir) of the server instance.
       nodePath = '',
-      -- Automatically determine working directory by locating .eslintrc config files.
-      --
-      -- It's recommended not to change this.
-      workingDirectory = { mode = 'auto' },
       codeAction = {
         disableRuleComment = {
           enable = true,

After revisiting the ESLint implementation, I think it's as follows:

location - Will assume eslint will be able to be resolved (via node module resolution) directly in the specified workspaceFolder (we set this setting in on_new_config currently)

auto - Will traverse the file tree upwards from each opened file to find the working directory. It will scan for the existence of the following files:

package.json
.eslintignore
.eslintrc
.eslintrc.json
.eslintrc.js
.eslintrc.yaml
.eslintrc.yml

It will however immediately stop traversing once it finds package.json or .eslintignore. Maybe this is what's causing some issues with monorepos?

Yes, it falls back but see https://github.com/neovim/nvim-lspconfig/issues/1427#issuecomment-980781094, that variable is unset if it is not passed in settings.

Yes, it falls back but see #1427 (comment), that variable is unset if it is not passed in settings.

Hmm the more I read this code the more confused I get. Might be better to add the setting explicitly then

@mjlbach Following config seems to fix the issue. I've also tested this with my work project and there were no diagnostics errors like before.
lsconf.eslint.setup({
    settings = {
        workingDirectory = { mode = 'location' },
    },
    root_dir = lsp_util.find_git_ancestor,
})

NOTE: find_git_ancestor is also required.

I merged the change (which matches the upstream default. You'll still need to override your root_dir because of the monorepo, but I think the defaults are reasonable for most people (until I add per project configs which will obviate the issue).

I merged the change (which matches the upstream default. You'll still need to override your root_dir because of the monorepo, but I think the defaults are reasonable for most people (until I add per project configs which will obviate the issue).

Works for me. Thanks!


Bumping helm chart dependency versions: support (again)

In short, this PR is a new attempt of #2067 which is reasonable to make now that carbonplan's k8s version is upgraded from 1.19 to 1.24, which makes all of our k8s clusters be versioned 1.21 or higher - where the only cluster stuck at 1.21 is openscapes.
"In long", this reverts commit d680896f1ba32a80643861a72650fffc6c1dc400, that was the merge commit for #2079, that in turn reverted #2067 after the incident in carbonplan #2080.

Related action points

[x] Review #2074 which I'd like to merge before this
[x] Review this PR
[ ] Review a subsequent PR I'll open removing the patch to the deployer script in this PR

@GeorgianaElena @sgibson91 does this look okay to you? Deploying this yesterday night caused the outage in carbonplan, but I've now checked and seen that all k8s clusters use v1.21 or higher, so I think this is safe to go for.
Yet, I'd still like to do it now before people in USA wake up and I have time to babysit it.

Two failures ocurred, and I think they relate to temporary broken connections by upgrading ingress-nginx. They did not cause an outage in the hubs. I've re-deployed the support charts manually and things work well!

Scratch that, there was one failure with pangeo-hubs restarting prometheus-server - I presume it relates to a lack of memory to startup. I'll resolve this but this is not related to the upgrade here.


Extra arguments for jtreg execution

The original plan for this branch was to add a series of options onto the jtreg command line.
However, the debug output from an earlier job shows us that these options are all used by default.
In the process of learning this, I discovered that there appears to be a variable defined entirely for the purpose of allowing the user to set jtreg command-line options without having to directly edit the makefile (which is what we used to do, using sed).
So now I'm moving us from sed to this environment variable.

Make documentation appears to imply that a variable passed into a makefile via an environment variable can be manipulated internally, unlike command-line arguments.
If that is the case, the test here should have the concurrency setting for jtreg set explicitly to 2.
https://ci.adoptopenjdk.net/job/adam_test/6/
If this is not the case, the concurrency variable will not be explicitly set to 2, and we'll need to add -concurrency:2 to the EXTRA_JTREG_OPTIONS environment variable consumed by the makefile.

@Bluevisitor Changes look fine although https://ci.adoptopenjdk.net/job/adam_test/6/console show test run has broken

I see 2 points for concern:

The test fails due to the 2 concurrency options on the jtreg command line.
The concurrency option used to be set to "7", despite the fact that we were setting it to 8. Now it's still setting the concurrency option (both of them) to 7, despite the fact that I changed it to 2 in the environment variable.

Will investigate.

The 2 concurrency options may be caused by bug: https://bugs.openjdk.java.net/browse/JDK-8157605
It's fixed in jdk9, but the request to backport is to 8 has gone unanswered. Also had difficulty finding why/how it takes the concurrency value and subtracts from it before passing the final value to the jtreg command.
Right now, my best plan is to replace the concurrency environment variable with a direct -concurrency option inside the EXTRA_JTREG_OPTIONS variable.

Here's a jenkins run to test that the aforementioned change works: https://ci.adoptopenjdk.net/job/adam_test/7/

Didn't work. Will investigate further.

with regard to 3 we probably don't want the test to fail if no tests are found otherwise it will fail every time we build without the --jtreg flag

It seems the only way around this is to get the fix for the bug (link below) backported to jdk8. Will raise a new bug to cover this.
https://bugs.openjdk.java.net/browse/JDK-8157605
As for this PR, the only change we can make now is the change to the jenkins job config to cause a job failure if there are no results files. Since that doesn't need a PR, I'm removing this PR and deleting the branch.

Sorry George, I didn't spot your comment. Will leave the jenkins option alone.


Getting error on k8s but working fine on locally

Error Response :
Value cannot be null. (Parameter 'asset')    at SkiaSharp.HarfBuzz.BlobExtensions.ToHarfBuzzBlob(SKStreamAsset asset)
Severatity : Critical
From :
var drawer = new ZplElementDrawer(printerStorage);
drawer.Draw(labelInfo.ZplElements)
Thanks & Regards,
Anand Singh(Coder)
India

Do you have an example label that produces the error?

Thanks for your response @primo-ppcg. We have resolved the issue on prod as well. Due to some linux related font on k8s & docker nuget packages it was creating an issue.

Thanks for your response @primo-ppcg. We have resolved the issue on prod as well. Due to some linux related font on k8s & docker nuget packages it was creating an issue.

What resolved your issue? We're experiencing the same...


Implemented dynamic arguments and JSON support

I finally implemented dynamic arguments, so instead of --nvr being hardcoded, now you can only use it if you declare it first with --build-config-find "changelog.xml:nvr:/build/nvr" for example (but you can define as many arguments as you want).
All the build filtering is now done using the ConfigFinder class, as I intended earlier. The finding of the job result (SUCCESS, UNSTABLE, ABORTED etc.) is now also done using the ConfigFinderClass. The default place the tool is looking in is the build.xml file and /build/result xpath, however the user can set it to whatever they want, using the --build-config-find argument, like so: --build-config-find "myConfig.properties:result:buildResult".
I also implemented a parsing of JSON config files into the ConfigFinder class.
Last thing, I updated the help message:
Test Variant Comparator Usage: java -cp <classpaths> <mainclass> [arguments...]

Where mainclass is:
    io.jenkins.plugins.report.jtreg.main.comparator.VariantComparator

And arguments include:

    --path <path/to/jenkins/jobs>
                  A system path to a directory with your jenkins jobs. This argument is mandatory

    You can choose one of these to filter the jobs by their name:
    --query <querystring>
                  Filtering of the jobs by a query string (the syntax is described below).
    Query string syntax:
    The tool splits every job name by . or - characters and compares each
    split part with the query string. The query string consists of N parts
    separated by spaces (or other whitespace) and each of these parts
    corresponds with 1st..Nth part of the split job name.
    Example with explanation:
     "jtreg~full jp17 * {f37,el8} !aarch64 !{fastdebug,slowdebug} * * *"
        jtreg~full - specifies that the job's first part should be exactly
                     jtreg~full.
        jp17 - specifies that the job's second part should be exactly jp17.
        * - asterisk is a powerful wildcard that matches everything, so in
            this example, the job's parts on the 3rd, 7th, 8th and 9th don't
            matter = the tool takes everything on these positions.
            To stop the tool from draining system resources by looking at
            all jobs, if more than half of the query elements are asterisks,
            or the query has less than 4 parts, you must combine it with the
            "--force" switch.
        {f37,el8} - this is a set of possible matches, so the jobs's part on
                    4th position can be either f37 or el8. There can me as
                    many elements as you want, but they must be split by
                    commas with no spaces between them.
        !aarch64 - matches everything, BUT aarch64.
        !{fastdebug,slowdebug} - matches everything, but the elements in
                                 the set.

            --exact-length <number>
                          Meant to be used in combination with --query. It filters only the jobs that have this specified length (number of elements in its name).
    --regex <regex>
                  A regular expression to filter the jobs by their names.

    You need to choose ONE operation from these:
    --print
                  Print all jobs and their builds that match the rest of arguments, without actually doing any operation on the builds or tests.
    --enumerate
                  Print lists of all variants of jobs (that match the rest of arguments).
    --list
                  Print a table of matched jobs with their builds and the tests that failed in the build.
    --compare
                  Print a table of all failed tests (of matched job builds) and the builds where they failed.
    --virtual
                  Print a table of all matched jobs' builds and their result (e.g. SUCCESS, UNSTABLE, etc.). Can be used as standalone operation or combined with any other operation. Probably should be run with --skip-failed=false switch.

    Other arguments:
    --help
                  Print this help message.
    --skip-failed <true/false>
                  Specify whether the comparator should skip failed tests (only take successful and unstable) or take all. The default value is true.
    --history <number>
                  To specify the maximum number of builds to look in.
    --only-volatile <true/false>
                  Specify true to show only non stable tests with the arguments list and compare (shows only tests, that are NOT failed everywhere). The default value is false.
    --force
                  Used for forcing vague requests, that could potentially take a long time.
    --formatting <plain/color/html>
                  Specify the output formatting (plain, color or html). The default is plain.
    --exact-tests <regex>
                  Specify (with regex) the exact tests to show only. The rest of tests will be ignored.
    --use-default-build <true/false>
                  If set to true and no matching build with given criteria was found, the tool will use the latest (default) build instead. Default value is false.

    Dynamic arguments:
        Another type of arguments you can use are dynamic arguments. They are used
        for further filtering the jobs/builds to compare by any value in their config
        files.
        First, you must define the value to look for in a config file. The general syntax is:
          --X "configFileName:whatAreYouLookingFor:queryToFindIt"
            Instead of --X, you use one of these arguments:
    --build-config-find configFileName:whatAreYouLookingFor:queryToFindIt
                  Argument used for declaring dynamic arguments. Looks for the specified config file inside the BUILD directory.
    --job-config-find configFileName:whatAreYouLookingFor:queryToFindIt
                  Argument used for declaring dynamic arguments. Looks for the specified config file inside the JOB directory.
            configFileName is the name of the config file in the chosen working directory.
            whatAreYouLookingFor is the name of the config value you are looking for.
                You later use this name as an argument for the filtering itself.
            queryToFindIt is the query, to find the value in the config file. Currently,
                XPath is supported for XML files, Json Query for JSON files and plain value
                for properties files.
        After defining the value, you can proceed to the filtering itself. For that, you
        use the whatAreYouLooking part from the definition as an argument and this syntax:
            It either takes RegEx to match the value with or multiple RegExes
            in curly brackets, separated by commas. (e.g. {nvr1.*,nvr2.*})
        Example:
        --build-config-find "changelog.xml:nvr:/build/nvr"
        Then you can use this argument: --nvr "java-17-openjdk.*" to filter only builds,
        that have changelog.xml file in the build directory, and the value on XPath /build/nvr
        matches java-17-openjdk.*.
    Additional info:
        When taking the build results, the tool defaults to using the results from build.xml
        file with /build/result XPath, but you can set it to different value with
        --build-config-findargument. (The whatAreYouLookingFor part of the syntax
        should be set to "result".)

What do you think?

the help looks great. The only missing part is  that queries are XOR -or not? That you can youse only one, but you must use at least one. Or you can really use both?

break for now. Will continue on 6f44fa53c97775ad627c5281e2bbb6d1330b1060 after it

it loks great. It is necessary to fix the consumed exceptiosn (not easy here!!). Good luck!

The consumed exceptions are now logged (printing the stack trace) and it gives the user a warning that an exception occurred.

LGT


Fix typos and other documentation errors

Summary

This PR addresses some typos and miscellaneous errors discovered in multiple Install and Configure documentation pages.

Thanks for participating in Hacktoberfest! You can claim your sticker set here: https://get.printfection.com/hacktober21/4144583267 @michizhou


Typescript?

Hey is there any raison why this is not in Typescript? Thanks

It's on our to do list. We wanted to make sure we got the JS implementation right before we moved on to the TS implementation of the app.

Is there any ongoing effort? I think building this in TS will yield  some type issues that are related to the @shopify/shopify-app-remix package types. So would really be interested on how they are resolved.


⚠ Bofrost.be has degraded performance

In cfea89f, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2736 ms

Resolved: Bofrost.be performance has improved in 9279117.


Recursions not causing StackOverflow

I was positively surprised to discover that this code was not causing a stack overflow
(define (factorial-rec n)
  (if (= n 1)
      1
      (* n (factorial-rec (- n 1)))))

(factorial-rec 10000)

How could it be?

I think biwascheme implement a tranpoline.

Oh. Nice!
Is it required by Scheme standard to prevent stack overflow in deep recursions?

Is it required by Scheme standard to prevent stack overflow in deep recursions?

The answer is yes, if the function is "tail-recursive".
The requirement is called TCO(tail-call optimization). A Scheme implementation must not grow stack frame for tail calls.
http://stackoverflow.com/questions/310974/what-is-tail-call-optimization
However your factorial-rec is not tail-recursive, so it is implementation dependent that how big n could be.


How bad is it if my apiSecret is not so secret?

Hi all,
I am a developer of a browser extension to subscribe and listen to podcasts (https://github.com/podStation/podStation).
As it is a client side application, the API secret would be exposed to people using the software (could be obfuscated, but not really secret).
How bad is it to disclose the API secret?
Best Regards,
Guilherme.

I've pondered this myself, with no good conclusion yet.  It seems unavoidable for a no-server app.  My best advice currently, would be to use a proper User-Agent header, and make sure you aren't exposing a read-write key.
The keys serve two functions at present.  As a way to pinpoint something abusive and kill it.  And, to restrict functionality with permission levels, like write access.  When talking about read-only keys, it's not hurting us really.  It's just the risk that if someone started hammering the API with your key and we block it, it would hurt you by accident.
This is a good question for the larger group, since it's a "problem" I'd like a solution for.  I'll leave this issue open for a while, if anyone wants to chime in.  I'm open to any ideas...

Related issue:
How if I am developing a self-hosted open source podcast player? It seems silly to me to ask users to set up their own API key.
Possible issues with including a read-only API key in the (open) source code are, as you mention, someone taking that key and starting to hammer the API resulting in you guys blocking the key. Honestly, I don't see why anyone would do that, except as a way to annoy users of my app (which are none at the moment), but it does seem like a potential issue and it would be nice to have a way to deal with it.
Anyway, no big deal, I guess, but I thought I'll mention it here.

Yeah, it’s still an issue really. I’m open to any ideas you might have.

Would it be possible to include in apps/software a system that automatically requests and gets assigned a read-only API secret? It could be stored in the app/software and reused. That secret could have a special status so it gets deleted e.g. 12 months after not being used (to keep the key list somewhat clean).
Btw I believe that for AntennaPod the secret is hard-coded for free builds so that F-Droid can pick it up (not sure about this one). And we have not yet had to rest the secrets because of misuse.

Yes I think that’s a good way to go. I’m ok with building that. I’ll ping here when it’s ready. Give me some time, I’m pretty backed up atm.


sudo and sssd: Enable sssd support in sudo and building of sudo library in sssd

Motivation for this change
Sudo has support to poll SSSD for its sudoers rules, but it has to be compiled into sudo and it requires a sudo plugin, libsss_sudo.so, from SSSD to function.
Added optional arguments to sudo and sssd to enable this functionality and have tested it, via packageOverrides, on a baremetal nixos machine. Both sssd and sudo was built with nix.useSandbox = true and I tried the functionality it brings and it works, I have sudoers rules in an openldap directory which I could apply after appending sudoers: files sss to nsswitch.conf.
Things done

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

[x] NixOS
[ ] macOS
[ ] Linux

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

Hi, could you move this PR to a separate feature branch? It seems you are using your master branch, and now you have added some non-relevant commits to this PR. (All the "sheepdog" stuff.)

Definitively, was a while since I did the PR workflow, so completely missed that

Seems like I can't change from what branch I want to merge, but I can change to which branch I want to merge, so I'll close this PR and open up a new with the correct branch.

Thanks!


Support RETURNING functionality on DELETE queries

I'm working on a client side cache/data store front-end to sit in front of postgrest, and this functionality would save me a lot of trouble in terms of cache invalidation logic.
The prefer: return=representation header with a select request arg (like insertion) would be fine in my opinion.

Could you provide more context please about how having the information you just deleted helps you in implementing cache?

Specifically, if I want to intelligently invalidate the data cache after a delete query, without being able to see specifically what was deleted, I have to re-implement the selection logic in the client, and basically duplicate the delete query locally.  This isn't a big deal if the predicate matched was "IN" but this blows up with full text search, for instance.  In that case I would have to query the table again just for ids, but that requires special case code, and that query is more expensive on the server side than just returning matched ids from the delete query, not to mention requiring another client round trip.
By getting a list of ids that matched a delete query in the response, I can use the same logic to flush deleted entries from the cache regardless of what predicates were used.

I was thinking you were deleting records one by one which is usually the case hanse my confusion related to this feature.
The good news is that DELETE (like INSERT/UPDATE) supports RETURNING (which is the base for implementing &select for mutate queries), so this is possible.

i think this has been implemented (&select works on delete also) and can be closed

Doesn't look like there is regression test yet for this feature, we should add one before closing. https://github.com/begriffs/postgrest/blob/master/test/Feature/DeleteSpec.hs


HTTPS not being used for redirect_uri

I apologize if this isn't the right place to outline the issue we're experience, but after creating our Application and setting the Callback urls to https://example.com/zooniverse-callback/ (trailing slash included 😉)

the id + secret are then used without our oauth implementation, but we get this error page when returning from signing into Zooniverse:

that we believe originates from the https protocol not being preserved in the redirect. This was not an issue when testing locally with https://localhost:8000/zooniverse-callback/ for example.
(Screenshots intentionally removed information)
Is this suggestive of a different error that's originating because of a setting our our end?
Thank you for your time and consideration,

Max

The return_url in that last screenshot is a parameter provided by the calling application, so I think it's probably your code that's not sending in an HTTPS url, and then our app rejects it because it doesn't match one of the configured URLs..


Attach file (e.g. pdf) to a page

I would like to attach one or more files to a created page. These files should be for example pdf or word documents.
Describe the benefits this feature would bring to BookStack users
This would make bookstack even more versatile. It would be for example possible to add official manuals of products to a page

That's already possible.
You can add them via the side menu:

Edit the Page

3. Drag and Drop your files
They will show up when you view the page in the left top corner As Attachments

Cool. Thanks a lot. I was searching this feature and did not find it.


Update developer guide for range command

Codecov Report

Merging #214 (6bc1612) into master (3608eee) will increase coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #214      +/-   ##
============================================
+ Coverage     59.41%   59.42%   +0.01%
- Complexity      670      671       +1
============================================
  Files           130      130
  Lines          2582     2588       +6
  Branches        326      326
============================================
+ Hits           1534     1538       +4
- Misses          901      903       +2
  Partials        147      147

Impacted Files
Coverage Δ

.../java/seedu/condonery/logic/parser/ParserUtil.java
62.29% <0.00%> (-2.12%)
:arrow_down:

...ry/logic/commands/property/AddPropertyCommand.java
56.09% <0.00%> (ø)

.../java/seedu/condonery/model/property/Property.java
70.27% <0.00%> (+0.40%)
:arrow_up:

...ogic/parser/property/AddPropertyCommandParser.java
85.71% <0.00%> (+0.71%)
:arrow_up:

...property/utils/ParsePropertyInterestedClients.java
72.00% <0.00%> (+2.43%)
:arrow_up:

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


Keyboard module is not listening keyboardWillShow on android

Is this a bug report?
yes
Environment

react-native -v: latest
platform: android

Steps to Reproduce
The following code on componentWillMount works on ios but doesn't work in android
this.keyboardWillShowListener = Keyboard.addListener(
  "keyboardWillShow",
  () => console.log("do something")
);

Expected Behavior
keyboardWillShow native event is triggered on android
Actual Behavior
It doesn't trigger so I had to use keyboardDidShow event instead.
Reproducible Demo
https://snack.expo.io/r1eRxoNvb

upps,
Reading at the docs again I found that keyboardWillShow is not available on Android since there is no native corresponding event.
https://facebook.github.io/react-native/docs/keyboard.html

Closed #15387