fix: hyphened name not auto-completed by the ls

Fixes #28905

Hi @Jack-Works - this is a great PR and I'd like to get it in for 3.9.
Any chance we could also get some tests? It looks like your first PR, so I'd recommend editing this test to have a hypen'd attribute and verify that it turns up in the completions at the bottom. Then it doesn't accidentally get broken in the future.

I added the test, sorry to steal that but getting it in to 3.9 had a bit of a deadline

Thanks!


Modify the buildoption-test and add to Travis CI as a job

JerryScript-DCO-1.0-Signed-off-by: Zsolt Borbély<EMAIL_ADDRESS>

LGTM

LGTM

LGTM


Special characters in page title

Main sections with special characters such as &, " and the Swedish å, ä and ö is shown as HTML characters in the page title.
Example:
// Färger & "typsnitt"
//
// Description...
//
// Styleguide 1.0
Shows correctly on the page but as "1 Färger & &qout;typsnitt" - Styleguide" in the page title when I'm browsing that section.

Just updated my project against the dev branch and the issue seems to be fixed, thanks for being so quick!

This issues is now fixed also in the NPM package (0.3.23)


Websockets constantly reconnect after exactly 1 minute

https://github.com/tornadoweb/tornado/issues/1070#issuecomment-45210635

As I said this was being done by something carefully and not abnormally. I am using Nginx as a Websocket proxy and has a default value of 1 minute for proxy_read_timeout setting. Increased that to a sufficiently large value and my problem was fixed. :)

Just saving this in case I need to edit the nginx config:
https://serverfault.com/questions/108261/how-to-make-a-modification-take-affect-without-restarting-nginx

It got fixed in a1d11e6.


Upgrade to V3 push token API

Description

Upgrade android SDK to upcoming V3 push tokens API. This feature requires the metadata work from #76 so is branched off that.
Check List

[x] Are you changing anything with the public API? - NO
[x] Are your changes backwards compatible with previous SDK Versions? YES
[ ] Have you tested this change on real device?
[x] Have you added unit test coverage for your changes? YES
[x] Have you verified that your changes are compatible with all Android versions the SDK currently supports? YES

Changelog / Code Overview

Converted from the V2 client/profiles endpoint that supported push tokens, to new V3 push tokens API exclusively meant for registering push tokens and device metadata
Added environment detection (debug/release)

Test Plan

[x] Unit tests against a stub of JSON from API spec
[ ] Manual tests on a physical device

Related Issues/Tickets

For review but do not merge -- it is still pointing to an alpha revision of the V3 APIs

closing to consolidate PRs


Extend peer dependencies to Angular 7

npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| ^6.0.0 but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| ^6.0.0 but none is installed. You must install peer dependencies yourself.

Oh man, it's already time for the next version! If you notice any issues, let me know. I'll get this updated.

@jaysaikia wait....I do need to do this but I think you have the wrong repo

@dockleryxk nope ... I think this is the right one. I will check anyway.

I pasted the wrong warn message. Lol.
I was about to put the request at agm too.
On Wed, 24 Oct. 2018, 2:27 pm Richard Jeffords<EMAIL_ADDRESS>wrote:

The console warning message is referring to<EMAIL_ADDRESS>—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dockleryxk/ng-simple-slideshow/issues/46#issuecomment-432499422,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGfzMiC8s-XyRS8rVssbIOlEbaCMgQOGks5un94TgaJpZM4X269K
.


Release 1.1.0

After #71 and #72, we have a new packaging method. I want to try it on PyPI, and btw, make the new features public!
TODO:

[x] Update the changelog
[ ] Cut the GitHub tag
[ ] Publish the release to PyPI

All done!

Cut the GitHub tag

https://github.com/DiddiLeija/diddiparser2/releases/tag/1.1.0

Publish the release to PyPI

https://pypi.org/project/diddiparser2/1.1.0/

Update: We had a little issue with uploading 1.1.0, and it's now broken. We made 1.1.0.post1, which can be safely used :)


Storage queue message encoding support

How can we enable the newer extensions support on the host? For instance the latest Microsoft.Azure.WebJobs.Extensions.Storage.Queues beta provides support for queue messages that don't use the base64 string encoding, such as queue messages from Azure Event Grid.
I'm looking to get this same setup / support in the isolated worker but so far, adding the package and modifying the host.json as was required in netcoreapp3.1 isn't working. I'm using the latest dotnet worker packages:
<PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.Storage" Version="4.0.4" />
<PackageReference Include="Microsoft.Azure.Functions.Worker.Sdk" Version="1.0.4" OutputItemType="Analyzer" />
<PackageReference Include="Microsoft.Azure.Functions.Worker" Version="1.4.0" />
Repo steps:

Add package reference: <PackageReference Include="Microsoft.Azure.WebJobs.Extensions.Storage.Queues" Version="5.0.0-beta.5" />
Set host.json extensions.queues.messageEncoding property to none
Use a QueueTrigger against the storage queue and append string content from Storage Explorer as String encoding instead of the Base64 encoding.

Error:
System.Private.CoreLib: Exception while executing function: Functions.ProcessStorageQueueMessage. Microsoft.Azure.WebJobs.Host: Exception binding parameter 'message'. System.Private.CoreLib:  The input is not a valid Base-64 string as it contains a non-base 64 character, more than two padding characters, or an illegal character among the padding characters.

Can you try to use the Microsoft.Azure.Functions.Worker.Extensions.Storage.Queues package? It looks like right now you're referencing the WebJobs one...

@brettsam had to upgrade the Microsoft.Azure.Functions.Worker.Extensions.Storage to 5.0.0-beta.4 ... and that appears to work... the 4.0.4 version does not.

Great!


Paralize the corpus processing script

The single-core processing can take several minutes. A scaling multi-core approach would increase the speed.

@Bachstelze Are you planning on implementing this part? or should we close this issue?


Broken link found (/docs/handbook/about-chatwoot/history)

Broken link found
Broken Link Checker found a broken link on https://www.chatwoot.com
Target: /docs/handbook/about-chatwoot/history

HTTP_404

View Actions Results

related: https://github.com/chatwoot/docs/issues/414


Receive notification from Azure Hub

Hi,
This plugin works so well, in few minutes I set it up and I can use it.
Is it possible to receive notifications from Azure cloud with this plugin?
Thanks,
Sorry because this is not an issue

@slimee Probably. The Azure cloud just ends up sending to GCM, APSN and WNS. I haven't tested it though.


[WINDOWS]: ERROR: C++ shared or static library 'atomic' not found

I want to generate visual studio solution folder, but got

$ meson setup build_vs --backend=vs2019
The Meson build system
Version: 0.59.0
Source dir: D:
Build dir: D:\build_vs
Build type: native build
Project name: webrtc-audio-processing
Project version: 1.1
C compiler for the host machine: cl (msvc 19.29.30038.1 "���� x64 �� Microsoft (R) C/C++ �Ż������� 19.29.30038.1 ��")
C linker for the host machine: link link 14.29.30038.1
C++ compiler for the host machine: cl (msvc 19.29.30038.1 "���� x64 �� Microsoft (R) C/C++ �Ż������� 19.29.30038.1 ��")
C++ linker for the host machine: link link 14.29.30038.1
Host machine cpu family: x86_64
Host machine cpu: x86_64
Executing subproject abseil-cpp
abseil-cpp| Project name: abseil-cpp
abseil-cpp| Project version: 20210324.1
abseil-cpp| C++ compiler for the host machine: cl (msvc 19.29.30038.1 "���� x64 �� Microsoft (R) C/C++ �Ż������� 19.29.30038.1 ��")
abseil-cpp| C++ linker for the host machine: link link 14.29.30038.1
abseil-cpp| Compiler for C++ supports arguments -Wno-sign-compare: NO
abseil-cpp| Compiler for C++ supports arguments -maes: NO
abseil-cpp| Compiler for C++ supports arguments -msse4.1: NO
abseil-cpp| Checking if "GCC atomic builtins" links: NO
subprojects\abseil-cpp\meson.build:36:2: ERROR: C++ shared or static library 'atomic' not found

Could you please give some suggestion?

Have you updated your version with  meson wrap update? This should have been fixed a few days ago.
Also, this is a legacy repo. All bugs should be filed against the mesonbuild/wrapdb repo. Closing this. If the issue is not fixed by an update, please file a new bug report on that repo.

Boss, how did you solve the problem？


Insert Title Here

Describe the issue in detail here. Attach any relevant screenshots or files. The section below was automatically generated. Remove any information that you do not wish to include in the report.

Debug Data

Property
Value

Name
"Wolfram/Chatbook"

Version
"1.4.0"

ReleaseID
"00db891fc2d1dd712145aa46464ded3b71633f75"

EvaluationEnvironment
"Session"

FrontEndVersion
"Desktop: 14.0 for Microsoft Windows (64-bit) (2023å¹´12æ12æ¥)"

KernelVersion
"14.0.0 for Microsoft Windows (64-bit) (December 13, 2023)"

SystemID
"Windows-x86-64"

Notebooks
True

DynamicEvaluation
False

SynchronousEvaluation
False

TaskEvaluation
False

Settings

Property
Value

StyleSheetVersion
"<IP_ADDRESS>13547000"

Stack Data
throwFailure
trimURL

Failure Data
<|
  "Evaluation" :> Wolfram`Chatbook`Actions`EvaluateChatInput[CellObject["137bc08c-8017-3f42-b2e1-2689427ac018", "05a59060-4373-7347-8efc-64443eabdb6f"], NotebookObject["71ad5d9b-f118-9a4f-9c6c-c7bae5bf9e07", "6d155112-2bba-ba47-846b-538510796de2"], Wolfram`Chatbook`Settings`currentChatSettings[NotebookObject["71ad5d9b-f118-9a4f-9c6c-c7bae5bf9e07", "6d155112-2bba-ba47-846b-538510796de2"]]]
  "Arguments" -> {}
  "Stack" :> Wolfram`Chatbook`Common`Private`$stack$34562
|>

This should be fixed since version 1.4.5. Please reopen if you can reproduce the issue in 1.5.0 or later.


Add index.html practice

adding the index.html file given to us by the bot so that we can start editing the file for our website (mine)

adding stuff


Receive new item

Hello,
First of all, thank you for your wonderful directive. Just wondering any plans to make it work in conjunction with jQuery draggable? I'm speaking about situation when sortable list receives new element. http://jqueryui.com/draggable/#sortable
Thank you.

I had no plans to work on this directive anymore as it was intended to support Angular 1.1.x versions. Do you still use 1.1? If not please take a look at https://github.com/angular-ui/ui-sortable. Folks there do pretty good job and might have what you need. If not let me know, I'll try to help you somehow.

Thank you for your answer. I've achieved similar result by using cloning feature. http://codepen.io/thgreasi/pen/qmvhG

Good to hear that, so I'm closing this one.


Loyalty Program landing page button that triggers login modal

This should pop up the login modal, before taking the user to the next page (where they view old inquiries).
Does this mean we should build not just the button as a component here, but also the whole login modal?

https://github.com/artsy/collector-experience/issues/86#issuecomment-278797200 (Authentication section)

Does this mean we should build not just the button as a component here, but also the whole login modal?

Nah, I’d like to use the existing one. Can’t we just send the user there and set a referrer URL that the user gets redirected back to after authentication?

Yea, but we would need to require the user to do a whole other page load, whereas I think ideally the button just triggers the modal.
What we could do w/o building it here is have the button, if not logged-in, link you to something like: https://www.artsy.net/log_in?redirect-to=/artists (replace the redirect-to with whatever the next step route will be). That will load the home page, and pop up the log in modal, which would also then redirect you.

I'm not sure how else to trigger the modal besides that server-side thing ^, so while it would work, it is a whole other page load, as well as having the home page in the background, so I think it's probably not ideal.

switched this to 'needs detail' given the conversation above

Done, we have the redirect/standalone page.


fix for issue #68

Its a fix for issue: https://github.com/vpulim/node-soap/issues/68
if the element is not defined then it throws error on line lib/wsdl.js:469.

Can you please merge this as I am running into issues without this line.

Thanks @Himanshu4 :+1:


Please support multi-module

Terraform Version
latest / not using it right now
Affected Resource(s)

rediscloud_subscription

Panic Output
n/a
Expected Behavior
We ought to be able to enable multiple modules (Search, Graph, JSON) so we can search graphs and JSON.
modules:  ['redisgraph', 'redisearch', 'redisjson']
ideally the details of order would be handled internally (i.e. if redisearch needs to come before others to index them), then this list could be re-sorted accordingly
Actual Behavior
the max number of items in the list is set to 1 so you can't do this right now
also, it's called "module" when we'd ideally call it "modules" plural
Steps to Reproduce
Please list the steps required to reproduce the issue, for example:

terraform apply

Important Factoids
https://github.com/RedisLabs/terraform-provider-rediscloud/blob/6895d80cec470d670b2eb4cb476dc35b36b8e7be/internal/provider/resource_rediscloud_subscription.go#L356
References
"module": {
	Description: "A module object",
	Type:        schema.TypeList,
	Optional:    true,
	**MinItems:    1,
	MaxItems:    1,**
	Elem: &schema.Resource{
		Schema: map[string]*schema.Schema{
			"name": {
				Description: "Name of the module to enable",
				Type:        schema.TypeString,
				Required:    true,
			},
		},
	},
},

Thanks for raising the issue. This is not a terraform issue, currently in our cloud offering, you can only use one module. we are working on it, and the terraform provider will be updated accordingly.


Dragging not working properly when the containment is inside an Iframe.

Test case: https://codepen.io/desandro/pen/yyRapr

I'm sorry to see you're having trouble with Draggabilly. Could you provide a reduced test case? See Submitting Issues in the contributing guidelines.


Swift Compiler Error (Xcode)

Swift Compiler Error (Xcode): Type 'MapCoreOSMView' does not conform to protocol 'OSMUserLocationHandler'
/Users/carapacik/.pub-cache/hosted/pub.dev/flutter_osm_plugin-1.0.0-rc.6/ios/Classes/map_view/osm_map.swift:12:6

Swift Compiler Error (Xcode): Incorrect argument label in call (have 'controlMapFromOutSide:', expected 'configuration:')
/Users/carapacik/.pub-cache/hosted/pub.dev/flutter_osm_plugin-1.0.0-rc.6/ios/Classes/map_view/osm_map.swift:144:58

Swift Compiler Error (Xcode): Cannot convert value of type 'Bool' to expected argument type 'TrackConfiguration'
/Users/carapacik/.pub-cache/hosted/pub.dev/flutter_osm_plugin-1.0.0-rc.6/ios/Classes/map_view/osm_map.swift:144:82

Swift Compiler Error (Xcode): Missing argument for parameter 'configuration' in call
/Users/carapacik/.pub-cache/hosted/pub.dev/flutter_osm_plugin-1.0.0-rc.6/ios/Classes/map_view/osm_map.swift:168:55

change min ios sdk to 13 if you are rc version

My MinimumOSVersion and IPHONEOS_DEPLOYMENT_TARGET is 14, but it returned the same error.

i updated ios sdk without publish new version for out plugin because im still need to work on some staff
i will update plugin this weekend so for now
try to force the version in pod file
of OSMFlutterFramework to 0.4.9

in the Podfile in ios folder  add this line
pod 'OSMFlutterFramework' , '0.4.9'

@Carapacik we publish new version 1.0.0


feat(documentation): add page structures of stories we still need to write

Added TODOs on the pages.
Changesets should be created as soon as one or more of the stories gets written. And therefore are not part of this PR.

Preview environments are getting posted here, as soon as they are ready!


Windows?

You can use krokodil directly downloading the official .exe file CLICK HERE or build from source running the command make win and open the .exe file built in the build/krokodil.exe directory of the project.
Sincerely -usbo


Some option(?) to auto-add radar-detected sites.

Still mulling this over, and still pondering how to add this to the code.  I think I see the code side.  I just haven't added it yet... since I'm still unsure how to balance this.  Probably a higher piece of research... hmm, don't know if a mod can add modules.  Also, I'd want this to be there early.  Dang, decisions, decisions... ;-)

The problem I have with this is that the more entities you're tracking, the slower your game will run; it's really easy to drop below 60 UPS with YARM.
The other problem is interface: there's no pagination yet, so if you create many sites they're going to scroll off the bottom of your screen.

If you'd like to implement it, you're more than welcome; I'd agree that it seems like a blue-science level research, so some kind of upgrade to the regular resource monitor.
It will also require a settings interface, and probably some way to merge sites. Finally, you'd want to take a close look at the automatic naming scheme and try to avoid generating sites with the same name (automatic suffix would be acceptable), which is a rare but very unfortunate event that can happen currently.

As an idea, I'd probably approach this with the mindset of "don't track them until the player asks to, but just be aware of where they are so we can easily add them" - so radar discovery would auto-add ore patches to a tagged list (which would ideally be filterable by distance, direction, size, ore type), and the player could then 'promote' patches from the tagged list to the active list. Again, though, it'd need some sort of management UI.
You'd have to be a bit careful how you wrote the auto-add code, though - if you have a tagged ore patch that spreads over both a discovered chunk and an undiscovered chunk, when the radar exposes the undiscovered chunk that'll change the tagged patches' extent (and then potentially the name, which is the index into the sites table).

Yeah, it's non-trivial for sure, both interface-wise and code-wise. Performance probably isn't as much of a worry as it was before, though. Still have to wonder about what happens when you place a radar, but that's chunkable, too, I imagine.

This is probably no longer necessary (or at least much less necessary) as of 0.15, since the map will show the quantities of ore as soon as they're discovered.

With the new miner support for reading an ore patch to the circuit network, plus the aforementioned map improvements, I'm thinking YARM itself can be reduced to a more generic "estimate time to depletion" tool that could work on anything circuit-publishable.
Regardless, automatically creating sites is something I was never keen on, and I much prefer that the base game has rendered it moot.


Starting huginn with OpenRC?

Hi,
is there a possibility to run huginn via OpenRC?

I'm sure it's possible since it's just a normal Rails app, but I don't think it's something we've tried.

I dont know if this is the best way, but i solved it on my Gentoo Linux as follow:
I've installed huginn in to "/opt/huginn".
I created a 3 little files to fire up Huginn:
/opt/huginn/run_huginn.sh
#!/bin/bash

source /etc/conf.d/huginn

cd $HUGINN_DIR
exec $RUBY $RAILS >> $LOGFILE 2>&1

/etc/conf.d/huginn
# Config file for /etc/init.d/huginn

HUGINN_DIR="/opt/huginn"
source $HUGINN_DIR/.env

PIDFILE="/run/huginn.pid"
RUBY="/usr/bin/ruby22"
RAILS="bin/rails server -p $PORT -b <IP_ADDRESS>"
LOGFILE="/log/huginn.log"
HUGINN_RUN="$HUGINN_DIR/run_huginn.sh"

/etc/init.d/huginn
#!/sbin/openrc-run

depend() {
        need net
}

start() {
        ebegin "Starting Huginn Server"
        start-stop-daemon --start -m -p $PIDFILE --quiet -b --exec $HUGINN_RUN
        eend $?
}

stop() {
        ebegin "Stopping Huginn Server"
        start-stop-daemon --stop -p $PIDFILE --quiet --exec $HUGINN_RUN
        eend $?
}

Now i can fire up and shutdown Huginn via:
/etc/init.d/huginn start

and
/etc/init.d/huginn stop

You also need a script for the background processes (bin/threaded.rb).
It looks like gentoo also has a package for runit, since you already have all the system package dependencies installed the manual installation guide should also work on gentoo.

You also need a script for the background processes ( bin/threaded.rb ). ...

Thanks!
I changed the files as follow and it seems working:
/opt/huginn/run_huginn.sh
#!/bin/bash

source /etc/conf.d/huginn

cd $HUGINN_DIR
bundle exec rails runner bin/threaded.rb &
exec $RUBY $RAILS >> $LOGFILE 2>&1

/etc/init.d/huginn
#!/sbin/openrc-run

depend() {
	need net
}

start() {
	ebegin "Starting Huginn Server"
	start-stop-daemon --start -m -p $PIDFILE --quiet -b --exec $HUGINN_RUN
	eend $?
}

stop() {
	ebegin "Stopping Huginn Server"
	start-stop-daemon --stop -p $PIDFILE --quiet --exec $HUGINN_RUN
	ps aux |grep "bin/threaded.rb" |grep -v grep |awk '{print $2 }' | while read i ; do kill $i; done
	eend $?
}

@3PO I'm going to close this, but it would be awesome if you were able to copy your steps over to the wiki or similar, in case it helps others in future!


You are not authorized to access SendGrid, please contact

I'm trying to login since yesterday and i got an error "You are not authorized to access SendGrid, please contact" and the Contact Support don't give me a response. I need use the service.

Hello @djsoul86 ,
We're sorry you're having this issue. Unfortunately, the documentation team is unable to assist with account and login Issues. We have passed your request to the support team, and they should be reaching out soon. Again, we apologize for the delay and understand that the wait is less than ideal.
Twilio SendGrid Docs Team


help me about the DIDpayload

I can't find why the user_data change ?
and I want to know how to change my code?

this is my code,and use the python-udsoncan.

Your DIDCodec says f190 is a 15 byte ASCII string. According to your log, the ecu responded with an incomplete DID response. udsconcan kept polling hoping to find the rest of the bytes, but the ECU never sent more. Hence the two timeouts.
I believe udsoncan and doipclient are behaving correctly and your ecu is wrong or the DID definition is wrong

I don't think so. I recheck this bug,and find the requestid"0x22" don't struck.back to use,maybe this is a question about udsoncanCodeC.I'll try to change my way. Thankyou .

@pyle

this question maybe is the request always sends the b'"' instead of "\x22". I don't know how to solve it. Could you give me a hand? Thank you.

[hex(ord(x)) fr xo in b'"\xf1\x90']
=> ['0x22', '0xf1', 0x90']
The ECU should respond with
DiagnosticMessagePositiveAcknowledgement
followed by
DiagnosticMessage with the user_data set to:
0x62 0xf1 0x90 xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx
Where xx's are the 15 bytes of your DID
The request is fine your ECU just isn't responding. If you know this works in a prior revision of the doipclient, we can compare. You can use wireshark to confirm if the ECU is just responding uncharacteristically late.

Thank you for your help, I solve the problem with change the session's P2 time and receive the data correctly.
Thankyou!!!!!!!!!


Add smoke tests to CI

Fixes #51442
But, maybe we want to run this in other places too like on release branches, pre-nightly, in AzDo...

Note: I should also pull the cases from #51474 too, because we don't want to regress those.

You sure you don't want smoke tests with our own tslib __importDefault and __importStar helpers, too?

I do, I just haven't added them yet. (Unless you mean something more than what I mentioned in the comment above.)


Explicit Resource Type

This PR makes the resource type explicit by importing the framework for resource types from cfn-certificate-provider, but defaulting to the backwards-compatible Custom::Custom.  Unlike cfn-certificate-provider (which has a provider in the else), I have added an explicit error branch.  I'm not sure if there's a compelling reason to not do this, but a typo in the Custom Resource name could land the user in the else branch.  If this is another provider, it could produce strange errors (e.g. parameters missing or even building wrong resources).
While I was in the file, I also changed the import strategy for providers.  This ensures that IDEs understand the imports even if they're nested in projects with different source roots.  I did this for a while to use a custom modification while it was PR'd.

thank you for your contribution!


Added example to use configMapGenerator from .env file

The examples in the website don't show how to use configMapGenerator with envs instead of files.
Since there is a clear difference in the result and  that kubectl does have both --from-file and --from-env-file, I consider this example extremely necessary to bring parity to the use of kubectl and kustomize.
It took me around 2hrs to notice this difference and it's not referenced anywhere in the docs.

Thanks for suggesting this @rojosinalma
I recommend revising the note to use a callout and with different wording. The shell script and remaining explanatory text looks fine (I also didn't test this).

Hey, thanks for the quick review.
Can you expand on "the note to use a callout and with different wording"?
I didn't get what you meant by that, specifically I don't really get what a "callout" is.

Thanks for the suggestion. I commited it now.

/approve
@rojosinalma I guess if you look at the page (or preview) you can see how that renders?


Double enter to open a file in terminal

This behavior comes with fzf. I use fzf.vim, it's based on terminal to display output through terminal, and I have to double to open a file. Trigger fzf, gvim's title become !cmd.exe --> choose file, still !cmd.exe, first enter--> still !cmd.exe, need to make another enter here to exit cmd and enter normal mode on file.
I think this problem caused by terminal itself, not fzf. Hope this is on your radar.
Gvim/windows 10

please give exact reproduction steps.

I run this command C:\Users\YYYYY>gvim.exe -u NONE --cmd "set nocp | set rtp+=C:\Users\YYYYY\vimfiles\pack\plugins\start\fzf | runtime! plugin/fzf.vim", please refer to this video to see the result.

and then? what do you do with fzf?

If I understand right, the command will load only fzf plugin without vimrc. It's pure fzf plugin to run in gvim that use terminal.
Hmm, now I'm not sure this issue from terminal itself, but what I can tell? The expected behavior should be choose file from fzf, press enter, open the file and ready to use in normal mode, the extra enter to bring the file to normal mode is not expected.

Maybe related to #1998.

please give simple reproducible steps starting from vim --clean. I am sorry, I can't hardly understand you.

Sorry, this issue happens when use vim with fzf.


Add a screenshot button in the examples header.

I don't know if we want this added to the examples, but it helps show off the feature, and is simpler than doing a proper screenshot widget.  If you think it is undesirable, we can get rid of the PR.
Another way to do this is to just show a save dialog with the screenshot (and not show a thumbnail of it).  In this version you can right-click on the image and copy it instead of being forced to download it.

Codecov Report

Merging #669 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #669   +/-   ##
=======================================
  Coverage   90.63%   90.63%
=======================================
  Files          84       84
  Lines        8723     8723
=======================================
  Hits         7906     7906
  Misses        817      817

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 7d2b572...d95f8ae. Read the comment docs.

I think the feature is okay.  Just a couple minor notes:

The button shouldn't appear in the header on the main examples page.
It should use a different pointer when mousing over the element.


Not a diagram file Microsoft Word error

Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
We have an existing Word document with embedded Visio objects, upon exporting them out to a Visio file and re-importing them using the Draw.io add-in for Microsoft Word I get an error message saying "Not a diagram file". However if I open the file using the stand alone Draw.io application, they open no problem.  Also, if I convert the Visio file to a drawio file type and then insert into Word via the add-in it works fine.
To Reproduce
Steps to reproduce the behavior:

Open a Word document with embedded Visio objects
Alt + double click on the diagram to open it Visio, then click Save to save it as a .vsdx file on your local computer
Click on the draw.io add-in and select Pick Device File  and select the recently saved file. embedded_drawing.zip
See error in screenshot

Expected behavior
I expect the drawing to be imported and I shouldn't have to open it first in the app and convert it.
Screenshots

draw.io version (In the Help->About menu of the draw.io editor):
<IP_ADDRESS> (desktop version)
<IP_ADDRESS> (Office add-in version)
Desktop (please complete the following information):

OS: Windows 11 Pro
Browser: Mozilla Firefox
Version: 113.0.2

Additional context
Using Microsoft Word for Microsoft 365 MSO (Version 2304 Build 16.0.16327.20200) 64-bit

The Draw.io add-in for Microsoft Word only supports draw.io files, while the standalone app automatically imports Visio files.


ImportFromStorageFeatureNotEnabled

azure-cli version: 2.54.0
When using the documented way to create an azure mysql flexible-server, after some checks, the CLI returns the error:
(ImportFromStorageFeatureNotEnabled) The import from storage feature is not enabled for the target server subscription id/resource group name/server name. Code: ImportFromStorageFeatureNotEnabled Message: The import from storage feature is not enabled for the target server subscription id/resource group name/server name.
There is no explanation on the page, nor can i find any information on this error on other pages. There should be a prerequisite section explaining how to get the feature enabled.
Full output:
Command group 'mysql flexible-server import' is in preview and under development. Reference and support levels: https://aka.ms/CLI_refstatus
Checking the existence of the resource group 'product-env-rg'...
Resource group 'product-env-rg' exists ? : True
IOPS is 396 which is either your input or free(maximum) IOPS supported for your storage size and SKU.
Creating MySQL Server 'product-env-mysql03' in group 'product-env-rg'...
Your server 'product-env-mysql03' is using sku 'Standard_E4ads_v5' (Paid Tier). Please refer to https://aka.ms/mysql-pricing for pricing details
(ImportFromStorageFeatureNotEnabled) The import from storage feature is not enabled for the target server subscription id/resource group name/server name.
Code: ImportFromStorageFeatureNotEnabled
Message: The import from storage feature is not enabled for the target server subscription id/resource group name/server name.

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

ID: 132d06a3-b6b5-2a3f-6a91-8deef636d131
Version Independent ID: a4e8d82c-5d61-6872-17eb-7b656c0980f4
Content: Migrate MySQL on-premises or Virtual Machine (VM) workload to Azure Database for MySQL - Flexible Server using Azure MySQL Import CLI
Content Source: articles/mysql/migrate/migrate-external-mysql-import-cli.md
Service: mysql
Sub-service: flexible-server
GitHub Login: @adig
Microsoft Alias: adig

The error is caused by missing to pass the --data-source-sas-token to the command. The error should be explained in the document and even better, the output of the command should be fixed to represent the missing argument

Hi @gasserp Thanks for your feedback, we will look into it and update as appropriate.

Article has been updated

This has now been resolved, with MySQL import CLI supporting Gen v2 storage architecture.


[FLINK-30517][log]Decrease log output interval while waiting for YARN…

… JobManager be allocated.
What is the purpose of the change
Flink Client will retrieve the application status every 250ms after submitting to YARN.
If JobManager does not start in 60 seconds, it will log "Deployment took more than 60 seconds. Please check if the requested resources are available in the YARN cluster" every 250ms. This will lead to too many logs.
We can keep the check interval at 250ms, but log the message every 1 minute.
Brief change log

Decrease log output interval while waiting for YARN JobManager be allocated

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)

@xintongsong Hi, Could you help take a look at this?


XmlPath behavior differs when namespaces are present

While using XmlPath, I noticed the behavior differs depending on whether I use namespaces. I'm unsure if this is user error, is expected or unexpected but thought I'd report it. Consider this example:
  public static void main(String[] args) {
    String someXml = "" +
        "<outer>" +
        "  <middle>" +
        "    <inner>1</inner>" +
        "  </middle>" +
        "</outer>";

    XmlPath xmlPath = XmlPath.from(someXml);
    System.out.println(xmlPath.get("outer.middle.inner")); // Prints 1
    System.out.println(xmlPath.get("middle.inner")); // print empty

    String moreXml = "" +
        "<n:outer xmlns:n=\"http://localhost\">" +
        "  <n:middle>" +
        "    <n:inner>2</n:inner>" +
        "  </n:middle>" +
        "</n:outer>";

    xmlPath = XmlPath.from(moreXml).using(xmlPathConfig().declaredNamespace("n", "http://localhost"));
    System.out.println(xmlPath.get("n:outer.n:middle.n:inner")); // prints empty
    System.out.println(xmlPath.get("n:middle.n:inner")); // print 2
  }

Yes I agree that it's strange but I think it works according to how XmlSlurper works in Groovy (which REST Assured is based on). It seems like the root element is just ignored when declaring a namespace. Don't ask me why (but you could ask on the Groovy mailinglist and then tell me :)). See for example here: http://stackoverflow.com/a/8670254/398441. I'm going to close the issue since REST Assured tries to be as close to Groovy as possible. If you find out that I'm wrong and Groovy works differently I'm happy to reopen the issue and fix it.

Gotcha, thanks


AppCenterTest task could be used on a Release Definition

I think the default should be System.DefaultDirectory or something that works with BD and RD
If a user doesn't changed this property right now using the task on a Release Definition will fail.
https://github.com/Microsoft/vsts-tasks/blob/fc17d6e2b05795936e685c12747f9bf511ed1daa/Tasks/AppCenterTest/task.json#L49

Discussing with PM

@rmarinho We acknowledge this issue, we'll revisit this on July 9th.

This issue is stale because it has been open for a year with no activity. Remove the stale label or comment on the issue otherwise this will be closed in 5 days


NIFI-3268 Add AUTO_INCREMENT column in GenerateTableFetch to benefit index

Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[X] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[X] Has your PR been rebased against the latest commit within the target branch (typically master)?

[X] Is your initial contribution a single, squashed commit?

For code changes:

[X] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[X] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

@qfdk Thanks for your contribution! Excuse us for taking so long to respond.
Please note that when you submit a PR for a JIRA, push the "submit patch" button to move JIRA status forward to "Patch Available". This ensure other developers to know it's ready for PR.
https://cwiki.apache.org/confluence/display/NIFI/Contributor+Guide#ContributorGuide-CodeReviewProcess
I like the idea of using indexed column instead of offset.
I wanted to test this PR. However, the same GenerateTableFetch class is recently updated by #1407 and this PR is now conflicted to merge.
Would you update this PR by rebasing with the latest master branch and fix conflicts? And also add Expression support to the "AUTO_INCREMENT(index) column name" you added?

How database specific is this? I hadn't heard of it before, in looking around online it looks like it's for MySQL/MariaDB?
It sounds like a great idea for the systems that can support it, I'm just wondering where it belongs. Is this something that should be in the DatabaseAdapter layer?

@patricker Thank you for pointing that out. It can vary among databases. Peter, What RDB do you usually use? @qfdk I thought you're using PostgreSQL by reading the HCC question you posted. It would be great if we can review it together to see whether this PR can work with different databases.
@qfdk I saw you've committed several updates, is it ready to another review cycle?

@ijokarumawak yes you are right, It's my post :) Because the new master can't passer travis-ci. I tried to compile it with travis-ci so i committed several updates to launch travis-ci job :(

@qfdk I understand. It's ok to have Travis test failure with other USER_LANGUAGE than English for now. Since the test has passed with English locale, we can move forward. We're aware of several tests can fail with other locale, and actively fixing those tests right now, for example, NIFI-3466.
In order to add new capability to this generic processor, we need to test with variety Database engines as many user use it with different databases. If you verified it with PostgreSQL, then I'd try reviewing this with other database such as Oracle, SQL server or MySQL.

Hi @qfdk , excuse me for not providing further review comments. Today, I wanted to test this updated PR but unfortunately it is conflicted. Would you rebase it with the latest master, address merge conflict and squash commits into one? Please let us know when it gets ready to resume review.
It may not be ideal, but I think it will take time to get this PR fully reviewed with various DBMS to see if it works as expected and how it can improve performance. On the other hand, the NiFi community is discussing about releasing 1.2.0. To make this effort along with the release cycles, can we remove 1.2.0 as Fix Version from JIRA NIFI-3268? The Fix Version is used to track which version the issue is fixed, and should be empty until it's resolved.
Thanks for your contribution.

@qfdk I updated the JIRA and emptied Fix Version for now.
Please let me request one more thing, is it possible for you to provide test result with PostgreSQL (or whatever DBMS you're comfortable with) to illustrate how much this enhancement can improve performance? It'd be really helpful to shorten review cycle. Thanks!

@ijokarumawak Hey, i will take a look as soon as possible because i found some new modifications in the master so i will try to do that. PS: nifi-<IP_ADDRESS>.1.1.0-2 works with my modification, we have 26,360,516 documents (5.6 Gb), in 5 minutes we could finish reading et avron2json

@qfdk "26,360,516 documents (5.6 Gb), in 5 minutes" is amazing! A video would be very helpful, but a image or two might be enough. I'd like to know the difference between the time to read all those documents (records) with this proposed index column, and without it. Thank you!

@ijokarumawak I made a shot video :)  https://youtu.be/ZD-CXgfJ6Xg
this video works for nifi-<IP_ADDRESS>.1.1.0-2 PostgreSQL, I will recheck the latest version of nifi.

@qfdk Thank you very much for recording and sharing the video. I can see how it improves execution time of generated SQL. This enhancement is significant for the first execution of GenerateTableFetch processor if there are many records to fetch. The performance difference is huge!
For the 2nd time or later, GenerateTableFetch uses 'where' clause using 'Max value columns' so it will not be a problem after 1st run if the max value columns are properly indexed. But if there are so many records inserted or updated, between GenerateTableFetch runs, it will have the same issue if we use limit offset.
I understand how it should work now. Please let me know when you finish updating PR. I will review it as soon as possible. Thanks!

@ijokarumawak I'm very glad  to work in this code in my free time. Now there was some issues that i want to debug the prossor lunched but nothing happend ...  nifi-<IP_ADDRESS>.1.1.0-2 works verry well but the lastest version it can't work correctly .. I would change some code in this week-end and try to fix them.

@qfdk Sometimes, I encounter issue while debugging, such as forgetting terminate old IntelliJ debugger or having other thread being stopped at different break point, then NiFi jetty thread can't respond. If NiFi UI keeps showing a little running circle and doesn't update, probably that is the case.
Otherwise, I've been able to debug it with IntelliJ fine.

Closing due to inactivity.


Redirect sometimes fails in the go-swagger generated code

Problem statement
I used go swagger to generate the http handler code. One of the handler needs to call the http.Redirect(...) to redirect the request to another web page.
http.Redirect(wG, r, redirectURL, http.StatusFound)

In order to get the writer, I added the following code in the go swagger middleware part
var wG http.ResponseWriter

// The middleware configuration is for the handler executors. These do not apply to the swagger.json document.
// The middleware executes after routing but before authentication, binding and validation
func setupMiddlewares(handler http.Handler) http.Handler {
	ourFunc := func(w http.ResponseWriter, r *http.Request) {
		wG = w
		handler.ServeHTTP(w, r)
	}
	return http.HandlerFunc(ourFunc)
}

However, sometimes the redirect fails to redirect the client browser to the redirect URL. Instead, the browser stays on the same page. It occurs occasionally. In addition, I saw the following log in the API server
2018/07/20 17:03:46 http: multiple response.WriteHeader calls
2018/07/20 17:03:46 http: multiple response.WriteHeader calls

Could anyone help look at this issue? Thanks.
Swagger specification
version: 0.13.0
commit: 8135eb6728e43b73489e80f94426e6d387809502

I have hit this one too.
It is a warning issued by the http package.
It is because we always send a response in the end, even though the redirect status has already been set.
One should check for possible already redirected response before forcing a response.

@fredbi , thanks for your explanation. Do you have any code snippet that fixes the redirect issue? Thanks in advance.

not yet

The problem is that the http.Redirect method in Go does not provide any return value.
Also is it confirmed that the "http: multiple response.WriteHeader calls" causes the redirect failure?

do you have wg outside of the middleware like that?

In the above code snippet, wG is a global variable and the only one. Because I need to use wG in the http.Redirect method.

yeah but how does that work with 2 concurrent requests?

I thought the go-swagger generated code handles concurrent requests. For example, user 1 hits the redirect path of the server, which calls the redirect method to redirect to an external page.
Meanwhile, user 2 can access the same path.
Do you mean I need to handle concurrent access to the redirect write response?

yes but not when you store the response writer in a global variable. if you need to share in between functions for the same request you can put something on the context

@casualjim , could you provide some sample code ? Thanks.

I think I found some solution using the context library.
In the middleware part, I set the response writer value in the context. And then in the handler function, I can get that value and use it in the redirect call.

also if you just want to redirect, add the 301 or 302 response to your swagger spec
generate the server and when you want to redirect just use that response instead of what you're doing now
or return a responder from your go-swagger handler: https://github.com/go-openapi/runtime/blob/master/middleware/context.go#L55-L67

@huikang here some sample code with a responder saved in context:
https://github.com/go-swagger/go-swagger/blob/f9da9f1f9bb42571cfcc6c6a4a6323a5e1d0e115/examples/oauth2/restapi/implementation.go#L54

@casualjim @fredbi , thanks for your responses. The problem is solved by setting the correct response writer.


[Ionic V4-beta.12] Bug router-outlet destroyed

Bug Report
Ionic Info
   ionic (Ionic CLI)          : 4.1.2 (C:\Users\ariefolo\AppData\Roaming\npm\node_modules\ionic)
   Ionic Framework            : @ionic/angular 4.0.0-beta.12
   @angular-devkit/core       : 0.8.3
   @angular-devkit/schematics : 0.8.3
   @angular/cli               : 6.2.3
   @ionic/ng-toolkit          : 1.0.0
   @ionic/schematics-angular  : 1.0.1

Describe the Bug
In my app, i navigate between two pages.
My first page is a list of stations and my second page display details station.
For navigate to details station i use this:
this.router.navigate(['station'], {queryParams: station});

And for go back:
this.location.back();

In my detail page i get the station like this:
let stationObservable = this.route.queryParamMap.pipe(
      switchMap(params => {
        this.station = params["params"];
        console.log("STATION ========> "+this.station.name);
        return this.ginkoService.fetchTempsLieu(this.station.name);
      })
    );
    stationObservable.subscribe((stationAttente) =>{

    });

when I navigate several times between the two pages and change stations, sometimes the station does not change while the url has changed. At this moment i can see in the debug console
router-outlet destroyed

Steps to Reproduce
Steps to reproduce the behavior:

Navigate between two pages.
Change station
Sometimes the station doesn't change
Router-outlet destroyed

Additional Context
This is my route:
const routes: Routes = [
  { path: '', loadChildren: './tabs/tabs.module#TabsPageModule', canActivate: [AuthGuard] },
  { path: 'tabs', loadChildren: './tabs/tabs.module#TabsPageModule', canActivate: [AuthGuard] },
  { path: 'station', loadChildren: './pages/station/station.module#StationPageModule', canActivate: [AuthGuard] },
  { path: 'schedule', loadChildren: './pages/schedule/schedule.module#SchedulePageModule', canActivate: [AuthGuard] }
];

I also have this issue on v4.0.0-beta.11. It occurs only if I go from the initial module that loads for my app to a different module, and then try to navigate to a route from the initial module.
i.e.

App starts and loads Module A, showing Component 1
Press a button and be navigated to Module B, showing Component 2
Press the back button and come back to Module A Component 1
Press a button to be navigated to Module A, Component 3 - but this doesnt work. The URL changes but the screen remains the same.

Occurs in Chrome when running ionic serve, iOS 12 iPhone X simulator, and iPhone 6S iOS 12 real device.

+1

I think Angular Router in Ionic 4 Beta 12 requires a complete review its very unstable. And navigation is the core of any applications so Angular routing should be very robust.

Temporary i'm using  instead of . Don't know if it would be solution for you .

I temporarily resolved this by creating a StartupModule that is loaded when the app is opened. It shows a loading spinner whilst the user’s location is determined and then it navigates to my original start module.
This resolves the issue for me, but only really works because I need to determine the user’s location anyway. Perhaps as a temporary work-around you could just show the splash screen in the StartupModule for one second before navigating to the original module.
I believe you may also be able to use NavController.navigateRoute() to set the original module as the root screen to avoid the user being able to press back and find themselves back at the StartupModule.

Any update on this ??

Same issue, very unstable with multiple routers router-outlet destroyed is contant issue if pulling any data from api. If simple page loads static empty no issues.

this issue appear if you forgot to close some html tag. for example: ion-header

Just upgraded from beta.11 and now my app navigation is broken :(  Any suggestion for a temporary fix ? (instead of rolling back to beta.11 :-/ )

router outlet destroyed is a problem?

Same here...

+1

router outlet destroyed is no problem in general, it only shows that the component with the ion-router-outlet has been removed. This is normal behaviour if you use different general layouts (with sidebar / without sidebar etc).
So to take a deeper look into this issue, could someone create a repo to show the issue?

here we go:
https://github.com/sloschi/ionic-angular-routing-test

@urbanjoe could you plese tell me what I should look at? I am starting it and it's just a title there - no navigation or something like that

ups, just swipe from the right.. there is the menu:

@urbanjoe Alright: Your issue is because of two things:

You are using two ion-split-panes. Here it's normal that one will be destroyed if you leave the page with it (e.g. with the tasks).

You are using href, please always use routerLink and always absolute paths (this you already did)

If you doing step 2, it will work okay. For example I changed it for the object lists and then it's looking good (see gif). And if I leave the page, it's normal that router-outlet destroyed arrives because the objects list with the split-pane will be destroyed and so this log occurs

@paulstelzer Yes, u are right, the repo has no update to the beta 18 (routerLink) right now, but locally i already updated.
Please try to navigate between taks and object (both are childs), the the destroyed appears. If u go to a root page and then to another child page everything is ok :-/

@urbanjoe Ah okay, what you now describe is what I already created as seperate issue -> #16411

Perfect, i'll give it a shoot

@paulstelzer  Same there, router-outlet destroyed message in the console:

I talked with @manucorporat and he said that I am right. So I summarize it here and will close the issue because of the following:

"router-outlet destroyed" is not a bug, it's only shows you that the component with  <ion-router-outlet> inside is destroying
What are the reasons this happening? In general you are using more than one <ion-router-outlet>  or using ion-tabs. If a component will be destroyed (removed from the dom - ngOnDestroy() also calls) and there is  <ion-router-outlet> inside (or ion-tabs), the "router-outlet destroyed"  will be displayed in the console
Another reason: You are using different layouts (with sidebar, then a fullscreen layout) and every contains a <ion-router-outlet>. If you now destroy the parent component,  "router-outlet destroyed" will be displayed in the console

But also keep in mind:

If you have more than one <ion-router-outlet> and / or more than one <ion-split-pane> inside your code, this can lead to some issues, we are already aware of, e.g. #16411 or #15538
If you are using ion-tabs and you switch from tabs to another router, you will also get "router outlet destroyed" because inside the tabs, there is used another <ion-router-outlet> , but as said at the beginning, that is not a bad sign
If you have only one <ion-router-outlet> (and no tabs) and you still get "router outlet destroyed" then you should check your routing if you destroy the ion-router-outlet for some reasons.

I hope that I described it in detail so everyone knows what happens. And as I said that you get problems, has not to do with the logs, the root cause is something else :)
And because the root cause is not "router-outlet destroyed", it's better to open a issue for every problem. So if your problem is not #16411 or #15538, please make a short search (maybe there is already an issue for) and if not, please create a new issue and include a repo so we can check it!


Aligned the about us in home page

Fixed issue #434
Aligned the about us in center

Can you add a bit more space at center @Riddhi162

ok

is this okay?

gap 60px looks good, also make it responsive @Riddhi162

gap 60px looks good, also make it responsive @Riddhi162

made the changes

Good Work @Riddhi162


Ignore some Make output in errorformat

Problem: GNU Make issues messages such as
make[1]: *** [foo/bar/baz/Makefile:210: src/foo.o] Error 1

The default 'errorformat' value matches these messages because of the
%f:%l:%m pattern, and creates quickfix list entries that appear valid
but create a new buffer named make[1]: *** [foo/bar/baz/Makefile,
which is obviously incorrect.
Solution: Add entries to 'errorformat' to ignore these messages.
The entries added are:

%-Gg%\?make[%*\d]: *** [%f:%l:%m
%-Gg%\?make: *** [%f:%l:%m

This covers all of the following cases:
make[1]: *** [foo/bar/baz/Makefile:200: src/foo.o] Error 1
gmake[1]: *** [foo/bar/baz/Makefile:200: src/foo.o] Error 1
make: *** [Makefile:114: all] Error 2
gmake: *** [Makefile:114: all] Error 2

Problem: GNU Make issues messages such as
make[1]: *** [foo/bar/baz/Makefile:210] Error 1

The default 'errorformat' value matches these messages because of the
%f:%l:%m pattern, and creates quickfix list entries that appear valid
but create a new buffer named make[1]: *** [foo/bar/baz/Makefile,
which is obviously incorrect.
Solution: Add entries to 'errorformat' to ignore these messages.
The entries added are:

%-Gg%\?make[%*\d]: *** [%f:%l:%m
%-Gg%\?make: *** [%f:%l:%m

This covers all of the following cases:
make[1]: *** [foo/bar/baz/Makefile:200] Error 1
gmake[1]: *** [foo/bar/baz/Makefile:200] Error 1
make: *** [Makefile:114: all] Error 2
gmake: *** [Makefile:114: all] Error 2

Thanks, looks useful.
Can you add a test for this?  The option value is rather complicated,
further additions may break this again.  For an example look in
src/testdir/test_quickfix.vim, Test_efm1().  You can add a new function
to avoid cluttering the existing one.  Essential is to keep the
"CheckUnix" line, since your change only is in the Unix default value.
--
In a world without walls and borders, who needs windows and gates?
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

There is actually already a line in the Test_efm1 error file like this:
    make: *** [vim] Error 1

I changed this to
    make: *** [src/vim/testdir/Makefile:100: test_quickfix] Error 1

which causes the test to fail on master, but passes with this patch applied. Hopefully that is sufficient.

I'm very excited by the prospect of not having make: *** [ buffers` created all the time it's been driving me insane.
But does this ignore the messages? Wouldn't the sane default by to not ignore the error, but match it and properly locate the error to the appropriate location (in the makefile)?


Cargo build --all tries to build examples of dependencies

The meaning of --all is wider than I expected. I run cargo build --all in root of a workspace to build all crates in my workspace. I did not expect this to cause examples in 3rd party dependencies to be built as well (building examples of crates in the workspace is OK).
To reproduce, make a new crate with:
[dependencies]
rgb = "=0.8.2"

and run cargo build --all. It fails, because I've published this crate with [[example]] in Cargo.toml, but without a necessary file in include = […]. I did not expect examples to be relevant to packages on crates.io.

I think this is a bug, things like examples should only be built for member of the workspace.

@pornel could you give some more information about your setup? I could not reproduce with a member of the workspace depending on rgb = "0.8.2" and building the ws with --all.

Note the syntactic detail, rgb = "0.8.2" loads this or later version (and I've fixed it in the next version). Needs two =: rgb = "=0.8.2" to load the old broken version specifically.

That's what I did: A member of a workspace depends on rgb = "=0.8.2", I build the ws with --all, no examples get built. I've also added an out-of-tree dependency that has broken examples, they don't get built by cargo 0.23.0 as well.

I can't reproduce it any more either, so problem solved!


fetch response in IE 11 does not contain right info

It seems that window.fetch polyfill in IE 11 doesn't return the same info (https://developer.mozilla.org/en-US/docs/Web/API/Response)  as other browsers, i.e. Chrome when there happens a redirect in ajax call.
In the same scenario,
Chrome and Firefox returns such Response js object info:
Response {type: "basic", url: "http://localhost:8080/authfailed", redirected: true, status: 200, ok: true, …}

But IE 11 returns totally different Response js object:

[object Object]{_bodyBlob: Blob {...}, _bodyInit: Blob {...}, bodyUsed: false, headers: Object {...}, ok: true, status: 200, statusText: "OK", type: "default", url: ""}

Can someone shed the light on how to force IE 11 to return the same fetch Response as other browsers?
Thanks in advance!

https://github.com/github/fetch#obtaining-the-response-url

@dgraham Can i get redirected status in Response?


WIP: CMake build refactor

NOTE: More work needed on corner cases, please test!
Rework of the CMake build toolchain for singularity-eos. Some changes from singularity-opac carried over.
Highlights

Simplified "all end-environment" or "all submodule" selection; controlled with SINGULARITY_USE_SUBMODULES. If on, no environment checks for dependencies are carried out and all submodules are configured. If off, looks for required/requested dependencies and fails if not found. This felt most efficient, but if needed different use-cases can be appended.
Updated submodule inclusion; the (likely still underdeveloped) goal here is to modernize how submodules are included. For submodules dependencies where it is possible, after calling add_subdirectory, the CMake find_package call should behave as tho the package is present in the environment, e.g. like it was a loaded via a module or spack. This is part motivated the above, so that once it is determined if the project is built using it's own submods, or system packages, the rest of the code can forget about it.
Modernized cmake install options, e.g. used as find_package(singularity-eos) (Still needs work).

Todo:

Kokkos/EOSPAC test cases. I'll be looking for cracks but there's a lot of egde cases I'll prolly miss.
Clean up flags and options, there's some redundancy we can polish out.

@Yurlungur @dholladay00 @brryan

Comment on d031183
I may have gone overboard in redoing the directory structure, I had a lot of ideas struggling to find resolution. This is a draft of what I perceive as a simpler setup, but it's admittedly a bit muddied at the moment. If there's concern, it's not much work to revert.
On the plus side, with -DSINGULARITY_USE_KOKKOS=ON -DSINGULARITY_USE_CUDA=ON -DSINGULARITY_USE_KOKKOSKERNELS=ON -DSINGULARITY_BUILD_TESTS=ON, it builds and passes on darwin p9s. I've not yet rewrote attaching headers from subprojects, which is next on my todos

closing as I have redrafted most of this, see #30


Warning: Received true for a non-boolean attribute className.

#26

I believe this could be from the Log in page from Destiny's push.


Logging: Expand web interface to set log level, add log file rotation

Right now the logging web interface is limited to viewing the current log file.
We need to be able to:

[x] Change the log level
[x] Rotate the log files (when logging to a file)

This is now implemented thanks to @dsam82 and PR #30.


compact: Refactored compaction Planner for ability to extend planners. Added compatibility tests.

Signed-off-by: Bartlomiej Plotka<EMAIL_ADDRESS>

I'm joining very late the party, but LGTM.


consumeQueueExt directory not deleted when deleting topic

BUG REPORT

Please describe the issue you observed:

Set enableConsumeQueueExt to true.
After topic deletion,  the enableConsumeQueueExt file is deleted, but consumeQueueExt directory not.

And the cause is:

How would you fix it?

maybe. change getStorePathConsumeQueue() to getStorePathConsumeQueueExt()


APK availability

First thanks for working on this app! Did you consider making it available for those without access to Google's walled garden – e.g. by picking up tagging releases and attaching the APK files there? Thanks in advance!

Added standard version 2.7.2
https://github.com/ClassicDIY/ClassicMonitor/releases/tag/V2.7.2

Thanks a lot! Will become available here (in my F-Droid repo) with the next sync at around 7 pm UTC, updates fetched from your new tags within 24h of your making a new tag available with an APK attached. My updater is configured to skip pre-releases; if you want me to pick those up as well, just let me know. Be welcome to pick a badge to accompany the Google Play one and link to the new place :smiley:
That said: looks like your app meets the stronger inclusion criteria of F-Droid.org – so if you want your app listed there, I gladly help you with that.

PS: Having added it to my repo I today find out your app is already available at F-Droid (didn't see that before as your repo URL changed and F-Droid still had the old one). While I've picked the latest (stable) "release" F-Droid currently ships the experimental MQTT version ("prerelease") as newest and recommended version, not sure if that is intended. If you don't want F-Droid to pick up prereleases, please let me know and we can work that out (disclosure: I'm one of the F-Droid maintainers).

The pre-release has all the functionality of the production version plus the MQTT subscriber feature to allow it to used with the ClassicMQTT project so it's fine to include it in F-Droid.

Thanks! Just wanted to be sure. So shall I better remove it again from my repo then, or what would you prefer?

I think it's fine to leave it in
thanks
On Tue, Feb 7, 2023 at 10:22 AM Izzy @.***> wrote:

Thanks! Just wanted to be sure. So shall I better remove it again from
my repo then, or what would you prefer?
—
Reply to this email directly, view it on GitHub
https://github.com/ClassicDIY/ClassicMonitor/issues/26#issuecomment-1420955800,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABIYATIJCIMPUKBZIIKAJMTWWJSEZANCNFSM6AAAAAAUOAIQUQ
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

--
Graham A. Ross
(613) 769-1331

With the stable only, or pre included?

@graham22 ^^ shall I pick the pre-releases as well then?

Yes,
release V3.0.8 in releases

OK, thanks! Enabled "pre" then for update check. My updater looks at the latest tag only, though – so if you add two tags/releases the same day, it will only pick the last one. Just in case you wonder. If someone asks, I can of course always pull the other one manually should that happen :smile:

Aaah… I just noticed the two have different applicationIds, but the very same tag name patterns. Would it be possible to use a different tag naming, e.g. mqtt-<versionName> for the MQTT variant? I wonder why F-Droid picked that variant and bundled it as ca.farrelltonsolar.classic; I've just asked and a team mate promised to look ASAP (I suggested splitting it there as well, also naming the mqtt-<versionName> tag name pattern as being asked from you to keep them apart).
Meanwhile I'll set up the 3.x one in my repo with updates disabled until my updater has a way to tell the 2 apart.
Btw: Team mate just reported back, mentioning some issues with a JAR for the MQTT branch. He'll chime in here himself shortly.
Thanks a lot for your patience and support, @graham22!

There's a source for a jar is here: https://github.com/eclipse/paho.mqtt.java/releases/tag/v1.1.1 and it's in Maven Central afaics


rename modular_squareroot and add a_flag1 rule

modular_squareroot  -> modular_squareroot_in_FQ2
a_flag1: use the bit in y_re when y_im ==0.

What's the status here @ChihChengLiang @vbuterin

Closed in favor of BLS standardization in the future


Crashing on startup

Describe the bug
Whenever I launch the game I get a black screen with ENB watermark it loads for a moment and then crashes. I've tried on Dev and public builds
Is it already a known issue?
Does not look like it
Steps to reproduce
Launching the game
Environment

OS: Windows 11
Hardware
RTX 3080
i7 9700k
16GB RAM (With Page file)
Installed on NVME

Contact:
Discord: garrynards

crash-2024-04-04-23-56-28.log

crash-2024-04-04-23-56-28-AUTOSCAN.md

Have you modded the list at all or moved any mods around?


issue with "location"

name: msys

on:
  workflow_dispatch:

jobs:
  build:
    runs-on: windows-latest

    steps:

      - name: Set up MSYS2
        uses: msys2/setup-msys2@v2
        with:
          update: true
          install: >-
            base-devel
            mingw-w64-x86_64-toolchain
            mingw-w64-x86_64-yasm
            location: C:\msys

error: target not found: C:\msys
https://github.com/qwerttvv/test-tmp/actions/runs/7808725434/job/21299405932#step:2:247

The indentation of the last line is wrong


Exporting to TSV

What is this PR for?
Allow users to export data in a paragraph to a TSV file.
There is already a PR for this, but it uses DataTables, which requires Flash.
What type of PR is it?
Feature
Todos

[ ] - Hide/disable the button if it is not in table/chart view
[ ] - Find an icon for the new button
[ ] - Saved file name is not correct in Safari
[ ] - Test with large data set

Is there a relevant Jira issue?
ZEPPELIN-672
How should this be tested?

create a paragraph with data in %table view
select the "Export to TSV" item in the paragraph settings list

Screenshots (if appropriate)

Questions:

Does the licenses files need update?
NO

Is there breaking changes for older versions?
NO

Does this needs documentation?
NO

Tested after rebasing with master, no conflicts.
Let's rebase it and Merge it :)

@zhongneu Could you rebase?

@corneadoug are you sure you want to merge this? I thought we were working on #761?
There are still pending TODOs in this PR. I don't think we should merge it before finish them.

It was already discussed there.
Regarding the TODO, there is only one that is useful: Hide/disable the button if it is not in table/chart view, but it can always be done later. (not critical for now)
The icon is fine, the Safari problem is not related to this feature (it affect also the existing export notebook), the test with large dataset doesn't matter since if it was too big, the browser will crash.
If we get this done with, then we can work on improving it, and build around it.
Otherwise it just become a blocker

@corneadoug I think its not right/user friendly way to have download data functionality.
You liked approach of including download buttons in "settings".
But I believe "Settings" is something to provide configuration/setting options and not to provide any functionality.
Also @adejanovski #6 commented rightly that

viewers have to find out that they need to click on the gear, which is hidden in report mode and simply absent when you link directly to the paragraph as an iframe

So I believe it's not a good idea to include buttons in settings as many community member don't liked it.
Secondly if DataTables provides export functionality which works and is extensible, than we should use it instead of writing our own way to do data export, this I mentioned earlier as well... #6
Also about extra UI space, its not an issue for using datatable or any of its functionality, but its general issue and to fix it we need to have a different PR...
If I look different notebook implementation, than the way they handle this issue is like below...
They are providing all buttons at bottom of table, and zeppelin provides same on top of table.

Merging this if no more discussions.
I will take care of moving the button as suggested next to the graph buttons with a download icon, right after.


Color contrast for parameter titles is too low in dark mode

Search terms
color contrast
Expected Behavior
I expected parameter names to easily readable.
Actual Behavior

Steps to reproduce the bug
Open any page with functions that have parameters.
Environment

Typedoc version: 0.24.3
TypeScript version: 5.0.2
Node.js version: v16.11.1
OS: Win10

.. Colors hurt me :(


Update getting-started.md

Had issues at this step and debugging revealed that the Oauth login was attempting to redirect to http://localhost:8710/auth instead of http://localhost:8000/auth .
Made this change to the client credentials in GCP and everything worked.

Awesome, thanks!


Add yet more talks

Further talk, more to come
cc @will-moore @sbesson

okayed in https://openmicroscopy.slack.com/archives/C1PJHN2D7/p1590399335011200

Merging


templates: add tail argument to truncate_*

#5085
The way this is implemented, the tail length is not taken into account when finding the truncation position. That means that truncate_end(2, "abcdef", tail="...") will result in "ab...", a 5 character string.

Checklist
If applicable:

[x] I have updated CHANGELOG.md
[x] I have updated the documentation (README.md, docs/, demos/)
[x] I have added tests to cover my changes

The way this is implemented, the tail length is not taken into account when finding the truncation position. That means that truncate_end(2, "abcdef", tail="...") will result in "ab...", a 5 character string.

I don't think that's desired behavior because user would want to align columns. Can you check if we can copy the logic of elide_start()/elide_end()? We'll need a formatter version of these functions. (BTW, in order to just append ellipsis, we wouldn't need native support for tail. Something like truncate(...) ++ if(width(...) > 2, "...") should work.)
FWIW, I'm not sure if tail is a good name. It might be okay for truncate_end(), but it's definitely wrong name for truncate_start() (where the head is truncated.)

... length of tail not accounted for
I don't think that's desired behavior because user would want to align columns. Can you check if we can copy the logic of elide_start()/elide_end()? We'll need a formatter version of these functions.
Yeah, can take a look into that. Did it this way because it looked like less code to touch. I think elide_* is only used for some non-user-contrallable text, unsure what we should do if the user inputs a longer indicator than the truncate length.

(BTW, in order to just append ellipsis, we wouldn't need native support for tail. Something like truncate(...) ++ if(width(...) > 2, "...") should work.)
Hmm, so is this wanted at all? Doesn't require repeating the item to be truncated, but otherwise that looks like a simpler solution.

FWIW, I'm not sure if tail is a good name. It might be okay for truncate_end(), but it's definitely wrong name for truncate_start() (where the head is truncated.)
If we want to keep the same name for both, maybe indicator? Otherwise could do head and tail.

Yeah, can take a look into that. Did it this way because it looked like less code to touch. I think elide_* is only used for some non-user-contrallable text, unsure what we should do if the user inputs a longer indicator than the truncate length.

iirc, it trims ellipsis/indicator to the maximum width, but it wouldn't be super important.

(BTW, in order to just append ellipsis, we wouldn't need native support for tail. Something like truncate(...) ++ if(width(...) > 2, "...") should work.)

Hmm, so is this wanted at all? Doesn't require repeating the item to be truncated, but otherwise that looks like a simpler solution.

I would say we'll want this feature, but not in the way it is implemented in this PR.

FWIW, I'm not sure if tail is a good name. It might be okay for truncate_end(), but it's definitely wrong name for truncate_start() (where the head is truncated.)

If we want to keep the same name for both, maybe indicator? Otherwise could do head and tail.

I tend to use ellipsis (which was the name used in Qt), but maybe I'm biased. I'm not native either.


Updating extension results in AttributeError (null derefence)

Describe the bug
I tried to update an extension, which resulted in an AttributeError
To Reproduce
 az extension update --name azure-cli-iot-ext results in:
The command failed with an unexpected error. Here is the traceback:

'NoneType' object has no attribute 'get'
Traceback (most recent call last):
  File "...\knack\knack\cli.py", line 206, in invoke
  File "...\azure-cli-core\azure\cli\core\commands\__init__.py", line 351, in execute
  File "...\azure-cli-core\azure\cli\core\commands\__init__.py", line 409, in _run_jobs_serially
  File "...\azure-cli-core\azure\cli\core\commands\__init__.py", line 402, in _run_job
  File "...\six\six.py", line 693, in reraise
  File "...\azure-cli-core\azure\cli\core\commands\__init__.py", line 379, in _run_job
  File "...\azure-cli-core\azure\cli\core\commands\__init__.py", line 171, in __call__
  File "...\azure-cli-core\azure\cli\core\__init__.py", line 451, in default_command_handler
  File "...\azure-cli-core\azure\cli\core\extension\operations.py", line 258, in update_extension
  File "...\azure-cli-core\azure\cli\core\extension\__init__.py", line 115, in get_version
AttributeError: 'NoneType' object has no attribute 'get'

Expected behavior
Updating extensions should not result in exceptions.
Environment summary
Windows 10 1709, installed with MSI, version 2.0.61 (not updated  to 2.0.62 as I have read that it is broken).
Additional context
Issue encountered on corporate network with horrible network security settings. If this error is a result from that, I think at least the error message/Exception should reflect that in some way.

Possibly related to #6441

@oerpli what version of the IOT extension do you currently have?  Use az --version

0.3.7
>az --version
azure-cli                         2.0.61 *

acr                                2.2.3 *
acs                               2.3.20 *
advisor                            2.0.0
ams                                0.4.3 *
appservice                        0.2.16 *
backup                             1.2.2 *
batch                              4.0.0
batchai                            0.4.8
billing                            0.2.1
botservice                         0.1.9 *
cdn                                0.2.1 *
cloud                              2.1.1
cognitiveservices                  0.2.5
command-modules-nspkg               2.0.2
configure                         2.0.20 *
consumption                        0.4.2
container                         0.3.15 *
core                              2.0.61 *
cosmosdb                           0.2.9 *
dla                                0.2.5
dls                                0.1.8 *
dms                                0.1.3
eventgrid                          0.2.2 *
eventhubs                          0.3.4
extension                          0.2.4 *
feedback                           2.1.4 *
find                               0.3.1 *
hdinsight                          0.3.2 *
interactive                        0.4.2 *
iot                                0.3.7 *
iotcentral                         0.1.6
keyvault                          2.2.13 *
kusto                              0.2.1 *
lab                                0.1.6 *
maps                               0.3.4
monitor                           0.2.11 *
network                            2.3.5 *
nspkg                              3.0.3
policyinsights                     0.1.2
profile                            2.1.4 *
rdbms                              0.3.9 *
redis                              0.4.2
relay                              0.1.4
reservations                       0.4.2
resource                          2.1.12 *
role                               2.4.3 *
search                             0.1.1
security                           0.1.1
servicebus                         0.3.4
servicefabric                     0.1.15 *
signalr                            1.0.0
sql                                2.2.0 *
sqlvm                              0.1.1
storage                            2.3.2 *
telemetry                          1.0.2
vm                                2.2.17 *

As I've just noticed that 2.0.63 has been released a few hours ago I updated to that version.
Running az extension update --name azure-cli-iot-ext results in:

No updates available for 'azure-cli-iot-ext'. Use --debug for more information.
which seems fine. So either the issue only arises when there actually is an update that can be performed or the bug has been fixed somewhere between 2.0.61 and 2.0.63.

As far as I am concerned this issue can be closed - I may reopen it if I encounter the bug again.


server.bat does not set ORIENTDB_HOME environment variable

On a Windows 10 Pro 64 Bit machine, server.bat script does not set ORIENTDB_HOME variable. No difference whether the script was started with Administrator rights or not.
Machine has latest Java SDK installed.

Which version of ODB? Does the server start or you get an error message? thanks

ODB Community Edition 2.1.15
The server does not start. Error message is "The ORIENTDB_HOME environment variable is not defined correctly".
I found that this is a resulting error of java not being part of PATH environment variable. After adding the java.exe containing folder to the PATH variable it worked.


UWP window fails to fill client area

Describe the bug
UWP window will suddenly fails to fill client area when dragging the window to resize, usually after some extended period of system up time (something like 1 week of not restarting). And when this happens, all UWP window (including system settings) behaves the same. If you drag the window border to enlarge the window, new client area becomes transparent. Note: The app is NOT freezing, and is still interactive (even buttons in the transparent area is clickable if you try)
This issues happens universally, throughout any Windows 10 and 11 versions, and any hardware. (There are intel, amd, and nvidia gpu users reporting this)
WinUI3 window, win32 window and XAML island does not have such issue.
Restarting dwm.exe does NOT fix this issue.
Steps to reproduce the bug

Not sure how to reproduce, just keep the system running for some days. The longer it runs, it happens more likely.

Expected behavior
No response
Screenshots

Also this reddit

https://github.com/microsoft/WindowsAppSDK/assets/42881734/db993c36-96cf-4008-a09d-d2859fb2ca29
NuGet package version
None
Packaging type
Packaged (MSIX)
Windows version
No response
IDE
No response
Additional context
No response

the same issue here for my Microsoft Todo

version：
Microsoft Todo
2.114.7122.0
Windows 11 Professional
22621.3880


Fatal Exception after canceling AF.session

[x] I've read, understood, and done my best to follow the *CONTRIBUTING guidelines.

What did you do?
Called AF.session.invalidateAndCancel()
What did you expect to happen?
Requests cancelation
What happened instead?
Fatal Exception:
# Crashlytics - Custom logs
# Application: ----
# Platform: apple
# Version: 13.7 (3)
# Issue: 7759ceb36ff6e4fde9ce40ad8af738cc
# Session: 3c08685a931f4409895b31cc11f0518c_DNE_0_v2
# Date: Sun Oct 30 2022 12:49:48 GMT+0200 (Восточная Европа, стандартное время)

Fatal Exception: NSGenericException
0  CoreFoundation                 0x99288 __exceptionPreprocess
1  libobjc.A.dylib                0x16744 objc_exception_throw
2  CFNetwork                      0x297b4 CFURLRequestSetRequestPriority
3  Alamofire                      0x5ac00 (Missing UUID af451a8357e83a3bbb054e12b35499ca)
4  Alamofire                      0xbec98 (Missing UUID af451a8357e83a3bbb054e12b35499ca)
5  Alamofire                      0xcc4f0 (Missing UUID af451a8357e83a3bbb054e12b35499ca)
6  Alamofire                      0x42284 (Missing UUID af451a8357e83a3bbb054e12b35499ca)
7  libdispatch.dylib              0x1e6c _dispatch_call_block_and_release
8  libdispatch.dylib              0x3a30 _dispatch_client_callout
9  libdispatch.dylib              0xb124 _dispatch_lane_serial_drain
10 libdispatch.dylib              0xbcb4 _dispatch_lane_invoke
11 libdispatch.dylib              0xb000 _dispatch_lane_serial_drain
12 libdispatch.dylib              0xbc80 _dispatch_lane_invoke
13 libdispatch.dylib              0x16500 _dispatch_workloop_worker_thread
14 libsystem_pthread.dylib        0x10bc _pthread_wqthread
15 libsystem_pthread.dylib        0xe5c start_wqthread

Alamofire Environment
Alamofire Version: 5.6.2
Dependency Manager: Cocoapods

This appears to be a crash inside URLSession or other Foundation networking bits. If you can capture the exception being generated that might provide a clue, but right now I'd say it's an Apple issue.
As an aside, you really shouldn't interact with the underlying URLSession like this. You can do the same things from Alamofire's Session type, which will also ensure any underlying bookkeeping that is necessary is properly done.

Closing this, as it seems like an intermittent URLSession issue.


(chores) camel-test-infra-arangodb: update to ArangoDB 3.11.8 container

There's 3.12 out already, but it doesn't work w/ the component.


fipsld recursively invokes itself if FIPSLD_CC is unset and CC=fipsld

Our build tooling only allows us to select the compiler with CC (i.e., we cannot invoke fipsld directly), so we set FIPSLD_CC=/path/to/compiler and CC=/path/to/fipsld.
If both environment variables are set, this works just fine. But if FIPSLD_CC is unset, fipsld recursively calls itself and compilation seemingly hangs. Ideally, fipsld would exit with an error message instead of seemingly hanging.
curl -L https://www.openssl.org/source/openssl-fips-2.0.16.tar.gz | tar xf -
cd openssl-fips-2.0.16/fips/
cat << EOF > test.c
int main(void) { return 1; }
EOF
# Uncommenting `#set -x` in `fipsld` makes the bug much more apparent.
( export CC="$(pwd)/fipsld" ; env -u FIPSLD_CC "${CC}" test.c ) # hangs indefinitely

Don't do that, then.

@levitte With all due respect, that's a little flippant.
More specifically, it's easier said than done. If I could somehow ensure that nobody building our project ever forgets to set FIPSLD_CC, I would. Alas.

You tell folks not to use that script, but instead a wrapper that exists if both variables aren't defined.
Note that the FIPS 2 module is not supported, and 1.0.2 was end-of-life months ago.  Yes, this leaves you in a hard position.  But hey, you got it for free, right? :)

@richsalz we're ahead of you on that one :)
I understand what support is offered for FOM 2 and 1.0.2. Bug reports can still be useful, though. Perhaps fipsld is going to be repurposed for FOM 3? From the outside, it's impossible to know. Thus, the bug report.

Thus, the bug report.

Thanks.
FOM 3 is completely unrelated to FOM 2. It has been written from scratch. Unfortunately FOM2 is no longer receiving updates so this won't get fixed.

Sorry for coming off as flippant.
fipsld will not be repurposed for the new FIPS module. The integrity check mechanism is completely different.

@levitte no worries.
@mattcaswell @levitte awesome, I'll close this then. Cheers.


NETOBSERV-1875: Make enrichment indexes configurable

Description
For secondary interfaces, it's better to allow more configurable indexing for pods enrichment. For some network types, pods might be identified by their MAC address advertized in the network-status annotations, while in others it can be by IP, by interface name, or any combination of those.
This PR makes it all configurable in that way (example):
secondaryNetworks:
- name: my-network
  index:
    mac: true
    ip: true

More than one network can be configured that way.
Additionally, the api.K8sRule object needs to be configured with the Interface field as well. Breaking change: also the IP and MAC field names have been renamed for consistency. It's now, for instance:
    kubernetes:
      ipField:  "SrcAddr",
      macField: "SrcMAC",
      interfacesField: "Interfaces",

Additionally, a new metric is added to track which indexer is used for every enrichment hit: promql:
sum(rate(netobserv_secondary_network_indexer_hit[1m])) by (kind, network, error)
This metric can also report errors and unexpected multiple matches
Breaking change tl;dr
In k8s enrichment rule, rename <rule>.kubernetes.input into <rule>.kubernetes.ipField.
Dependencies
Operator: https://github.com/netobserv/network-observability-operator/pull/772
Checklist
If you are not familiar with our processes or don't know what to answer in the list below, let us know in a comment: the maintainers will take care of that.

[x] Will this change affect NetObserv / Network Observability operator? If not, you can ignore the rest of this checklist.
[x] Is this PR backed with a JIRA ticket? If so, make sure it is written as a title prefix (in general, PRs affecting the NetObserv/Network Observability product should be backed with a JIRA ticket - especially if they bring user facing changes).
[x] Does this PR require product documentation?

[ ] If so, make sure the JIRA epic is labelled with "documentation" and provides a description relevant for doc writers, such as use cases or scenarios. Any required step to activate or configure the feature should be documented there, such as new CRD knobs.

[ ] Does this PR require a product release notes entry?

[ ] If so, fill in "Release Note Text" in the JIRA.

[ ] Is there anything else the QE team should know before testing? E.g: configuration changes, environment setup, etc.

[ ] If so, make sure it is described in the JIRA ticket.

QE requirements (check 1 from the list):

[x] Standard QE validation, with pre-merge tests unless stated otherwise.
[ ] Regression tests only (e.g. refactoring with no user-facing change).
[ ] No QE (e.g. trivial change with high reviewer's confidence, or per agreement with the QE team).

/ok-to-test


Regex too big when pasting a somewhat big URL into the completion for org-insert-link

When I run the command org-insert-link and paste a URL like this:
https://REDACTED--fhcm2.um3.visual.force.com/apex/CollaborationPortalIndex?id=a1H580000038V5vEAE#/teammember/a1H580000038V5vEAE/services/a2k58000000Cnp8AAC/processes/a2r58000000GofOAAS/edit/current
I get this:
Error in post-command-hook (selectrum--minibuffer-post-command-hook): (invalid-regexp "Regular expression too big")

This makes it impossible to use selectrum-mode and insert this link, I had to disable selectrum-mode temporarily.

Thanks, on my setup I can create that link without getting an error. Do you get the error immediately after pasting the link? Can you provide your setup and a backtrace? See here how to get a backtrace.

Yes, I get this error immediately after pasting the link.
I don't have quite a bit of setup, but the selectrum related things are quite small. If you don't mind I will provide as is with use-package:
(use-package selectrum
  :hook
  (after-init . selectrum-mode)
  :general
  ("C-c SPC" 'selectrum-repeat)
  (:keymaps
   'selectrum-minibuffer-map
   "C-e" 'selectrum-move-end-of-line-or-insert-current-candidate)
  :preface
  (defun selectrum-move-end-of-line-or-insert-current-candidate (arg)
    "Move to end of line or insert current candidate.
   ARG lines can be used."
    (interactive "p")
    (if (eolp) (selectrum-insert-current-candidate) (move-end-of-line arg))))

(use-package selectrum-prescient
  :hook
  (selectrum-mode . selectrum-prescient-mode))

(use-package prescient
  :hook
  (after-init . prescient-persist-mode)
  (company-mode . company-prescient-mode))

The string-match looks suspicious. I wonder what is generating that huge regex, with similar letters, it basically have that for every character in the URL, with alternative characters...
Debugger entered--Lisp error: (invalid-regexp "Regular expression too big")
  string-match("\\(?:h[̧̣̮̱̂̇̈̌]\\|[hĥȟʰḣḥḧḩḫẖₕℎⓗｈ𝐡𝒉𝒽𝓱𝔥𝕙𝖍𝗁𝗵𝘩𝙝𝚑]\\)\\(?..." "ftp:")
  prescient-filter("https://REDACTED--fhcm2.um3.visual.force.com/apex/..." ("ftp:" "doi:" "news:" "http:" "help:" "file:" "shell:" "https:" "elisp:" "elfeed:" "mailto:" "file+sys:" "file+emacs:"))
  #f(compiled-function () #<bytecode -0x1b3d7de38ab9bc94>)()
  apply(#f(compiled-function () #<bytecode -0x1b3d7de38ab9bc94>) nil)
  (condition-case e (apply func args) ((debug error) (signal (car e) (cdr e))))
  force-debug(#f(compiled-function () #<bytecode -0x1b3d7de38ab9bc94>))
  apply(force-debug #f(compiled-function () #<bytecode -0x1b3d7de38ab9bc94>) nil)
  selectrum--minibuffer-post-command-hook()
  read-from-minibuffer("Link: " nil (keymap (5 . selectrum-move-end-of-line-or-insert-current-candidate) (9 . selectrum-insert-current-candidate) (10 . selectrum-submit-exact-input) (27 keymap (67108991 . backward-kill-sexp)) (remap keymap (previous-matching-history-element . selectrum-select-from-history) (kill-ring-save . selectrum-kill-ring-save) (end-of-buffer . selectrum-goto-end) (beginning-of-buffer . selectrum-goto-beginning) (minibuffer-beginning-of-buffer . selectrum-goto-beginning) (scroll-up-command . selectrum-next-page) (scroll-down-command . selectrum-previous-page) (exit-minibuffer . selectrum-select-current-candidate) (next-line-or-history-element . selectrum-next-candidate) (previous-line-or-history-element . selectrum-previous-candidate) (next-line . selectrum-next-candidate) (previous-line . selectrum-previous-candidate) (minibuffer-keyboard-quit . abort-recursive-edit) (keyboard-quit . abort-recursive-edit)) keymap (remap keymap (previous-matching-history-element . selectrum-select-from-history)) (3 keymap (39 . miniedit)) (menu-bar keymap (minibuf "Minibuf" keymap (previous menu-item "Previous History Item" previous-history-element :help "Put previous minibuffer history element in the min...") (next menu-item "Next History Item" next-history-element :help "Put next minibuffer history element in the minibuf...") (isearch-backward menu-item "Isearch History Backward" isearch-backward :help "Incrementally search minibuffer history backward") (isearch-forward menu-item "Isearch History Forward" isearch-forward :help "Incrementally search minibuffer history forward") (return menu-item "Enter" exit-minibuffer :key-sequence "\15" :help "Terminate input and exit minibuffer") (quit menu-item "Quit" abort-recursive-edit :help "Abort input and exit minibuffer") "Minibuf")) (10 . exit-minibuffer) (13 . exit-minibuffer) (7 . minibuffer-keyboard-quit) (C-tab . file-cache-minibuffer-complete) (9 . self-insert-command) (XF86Back . previous-history-element) (up . previous-line-or-history-element) (prior . previous-history-element) (XF86Forward . next-history-element) (down . next-line-or-history-element) (next . next-history-element) (27 keymap (60 . minibuffer-beginning-of-buffer) (114 . previous-matching-history-element) (115 . next-matching-history-element) (112 . previous-history-element) (110 . next-history-element))) nil org-link--history)
  selectrum-read("Link: " nil :default-candidate nil :require-match nil :history org-link--history :may-modify-candidates t :minibuffer-completion-table ("elfeed:" "file+sys:" "file+emacs:" "shell:" "news:" "mailto:" "https:" "http:" "ftp:" "help:" "file:" "elisp:" "doi:") :minibuffer-completion-predicate nil)
  selectrum-completing-read("Link: " ("elfeed:" "file+sys:" "file+emacs:" "shell:" "news:" "mailto:" "https:" "http:" "ftp:" "help:" "file:" "elisp:" "doi:") nil nil nil org-link--history nil nil)
  completing-read("Link: " ("elfeed:" "file+sys:" "file+emacs:" "shell:" "news:" "mailto:" "https:" "http:" "ftp:" "help:" "file:" "elisp:" "doi:") nil nil nil org-link--history nil nil)
  apply(completing-read ("Link: " ("elfeed:" "file+sys:" "file+emacs:" "shell:" "news:" "mailto:" "https:" "http:" "ftp:" "help:" "file:" "elisp:" "doi:") nil nil nil org-link--history nil))
  org-completing-read("Link: " ("elfeed:" "file+sys:" "file+emacs:" "shell:" "news:" "mailto:" "https:" "http:" "ftp:" "help:" "file:" "elisp:" "doi:") nil nil nil org-link--history nil)
  org-insert-link(nil)
  funcall-interactively(org-insert-link nil)
  call-interactively(org-insert-link nil nil)
  command-execute(org-insert-link)

If this is not enough, I will work on providing a minimal reproducible error later.

Thank you, this looks like the problem is on the prescient side where the regexp gets build. The repeated h like chars might have something to do with char folding, do you have some settings for this?

By default prescient.el enables char folding. I can reproduce the problem easily by:
(string-match-p (char-fold-to-regexp "https://REDACTED--fhcm2.um3.visual.force.com/apex/CollaborationPortalIndex?id=a1H580000038V5vEAE#/teammember/a1H580000038V5vEAE/services/a2k58000000Cnp8AAC/processes/a2r58000000GofOAAS/edit/current") "")

It is disappointing to me that Emacs can seemingly not handle a regexp of only 7,800 characters long. It's not as if it even has a fancy DFA-based implementation that would justify such a limitation. That said, the documentation of char-fold-to-regexp says explicitly
If the resulting regexp would be too long for Emacs to handle,
just return the result of calling `regexp-quote' on STRING.

which is clearly not happening in this case, so it seems like a bug in Emacs to me. I'm running 28.0.50 built from master on 2020-09-19, Manjaro Linux.
We can work around it in prescient.el, I suppose. Transferring the issue.


Missing the file "generation.json" for execution-based evaluation and Cannot load CodeT5

Hi, I cannot file generation.json in the data folder. Can you please check if it is available?
Also, the provided CodeT5 checkpoint doesn't match the model code (some key mismatches).

Hello,
If you want to use the pretrained model and generation.json files just download them from [Here] . Download the Codet5.zip file. Regarding the checkpoint I just did the same thing and the checkpoint was working fine. There are some extra keys that the transformer library adds but overall there are no key mismatches. I revised the readme with better instructions. Hope this helps.


Warning Command for different Protocol Class: %2RLMP Epson EB-L530U

Using 8 Epson EB-L530U projectors for just lamp on/off commands we get the following error after every device poll. Device is functional but just fills the log with "Warning Command for different Protocol Class: %2RLMP" errors.
It would be good to have the option to disable polling as I think Epson are just not complying with the standard.
Companion version 3.2.2
JBMIA: Projector; PJLink v2.2.0

That would also disable ALL of the other variables and feedback options.
Have you tried the recent 3.3 release of Companion?
There have been several updates to this module that are better at avoiding log spam.
Many models do not implement the complete protocol and whine about something they didn't/forgot to account for.
If it is still an issue, let me know and I'll work in some way to limit the amount of warnings.

I haven't tested 3.3 as I know it changes a few things and we are week 1 of a 3 week show. If its better 3.3 you can close this and I apologize for wasting your time.

No problem. Re-open this issue when you get a chance to test if the stable 3.3 release still spams the log.


Input events, select, drag, resize, do not work correctly in Safari (macOS & iPad)

I can live with it not working in desktop Safari, but I'd like it to work on iPad.
In Chrome & Firefox I can select items and move/resize them. Very nice feature.
In Safari 12.0.3 I cannot move or resize anything. I can select but after clicking to unselect, I can no longer select anything, even after regenerating the timeline. Input events seem to be handled differently in Safari.

If you are talking about the demo, it needs a bit of love, we haven't updated it in a while
I can confirm that the library in general does work in Safari though, a dev on our team that uses osx had it working last week. I can dig up some more information on specific versions if you like

Yes, I was referring to the demo
I also recreated the demo to see if it worked with Create-React-App (it does), by using the latest npm version: react-timeline-9000 v1.0.12
It works but has the same problems as the demo.
I hope the dev team gets it to work with Safari and can publish a new npm version.

There's only 1 guy on the team that has a mac, so no guarantees on when he'll get around to it :)
This problem might be related to interactjs, which is something I'd like to move away from in the future. Maybe a version bump of interact in the meantime might fix it?


Allow range_loc inside subscript expression index

Allow this kind of subscript expression: arr[^2].

Merged, thanks!


Specify UTF-8 encoding in PDO connect

Specify UTF-8 encoding in PDO connect in order to correct the resulting array for json_encode in > PHP7.1

This will fix an issue with json_encode that I ran into. I figured since it took me an hour to figure it out I would put the change here for review.

Hi - thanks for sending in this PR. However it is system specific, so it isn't something I'm going to pull in I'm afraid. For example if someone was using Latin-1 on their system by default, then this would cause problems. I prefer for the script to use the auto-negotiated character set from the database (setting the charset on our server isn't needed since it defaults to utf8 for example).