Problems panel lists 4 problems for jsconfig.json in my project

VS Code Version: 1.53.2
OS Version: Windows 10

Steps to Reproduce:

Create a new Gatsby project, or clone from my repo
Open VSCode
Create a new jsconfig.json file if it does not exist
Problems panel will randomly show errors on the jsconfig.json file

The jsconfig.json file contains this:
{
    "compilerOptions": {
        "module": "commonjs",
        "jsx": "preserve",
        "baseUrl": "./src"
    },
    "include": ["src"],
    "exclude": [
        "node_modules",
        "**/node_modules/*",
        "node_modules/**/*",
        "public"
    ]
}

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

/needsMoreInfo
Please run the Extension Bisect command to determine whether these are coming from an extension you added (you didn't list these in your report). If Bisect identifies an extension, report the issue to the extension author. The "Report Issue" dialog on the "Help" menu can assist with that. Then close this issue, and optionally link it to the one you created on the extension's repository.

Hi @gjsjohnmurray
I just ran it with Extension Bisect, and the issue is still there.

/assign @mjbvz - is this one for you, seeing as the problems appear to be reported by the builtin ts extension?

Please provide a minimal sample project and specific steps to reproduce the problem

Hi @mjbvz
You can use the project I in first comment. just create a new .js file inside src folder, that is already good enough to trigger the problem to appear. Or do you need something more simple than that?
In the repo I shared, you can use the branch "reboot" to reproduce the problem.

Run yarn install after checkout to reboot branch
If VSCode is already open, restart it
Create a new folder in components
Create a new file, with the same folder name

Then the problems will pop up. If creating 1 file does not trigger it, you can create more files in the folder you have created, or create new folders and new files to trigger it.

Please provide a minimal project. The project you linked to is quite large and this type of bug is often a project configuration issue. It's not a good use of our time to try debugging through it

Thanks the for reply, @mjbvz
I will try to provide one.
I actually specifically mentioned the reboot branch, because that branch is new, and it only has the bare minimum Gatsby startup code, with Airbnb's ESLint packages.
If you look at the main vs reboot, you will find that reboot is much smaller.

Hi @mjbvz
I have been able to reliably reproduce the problem using this new minimal project - lewisgauss/reproduce-jsconfig-error
I also have a much simpler way to reproduce it.

Open the project
Open any .js file, like pages/index.js
Open jsconfig.json
In jsconfig.json, edit it you can remove one line, like jsx: "preserve"
Click on the .js file that you opened, so Visual Studio Code can reinitialize the feature
Click on jsconfig.json again, add the line that you just removed back
Click on the .js file again, the problems will now show in the panel

I followed those steps but this does not reproduce for me in VS Code insiders with extensions disabled

@mjbvz did you do npm install before going through the steps?
You also have to wait for the initializing JS/TS feature to finish in between steps.

I have just tried this on a completely different computer, and I could reproduce it. (macOS and Windows 10) With all extensions disabled. :)

Another run with Extension Bisect.

Yes node modules are installed and initialization is done.

Does node_modules\defer-to-connect\dist\index.d.ts exist and are there any errors in it?
If it does, then please record a video showing the exact steps you are taking here so I can try to match them

Yes, the file is there, and TSLint does not show any error. You can see it from the recording below, which shows the reproducible steps and the file without error.
https://user-images.githubusercontent.com/8966669/108243721-5c6c6d00-7189-11eb-8303-993b96864aad.mp4

Thanks but I am still not able to reproduce following those steps. Can you try collecting the logs from reproducing this:

Set "typescript.tsserver.log": "verbose",
Restart VS Code and reproduce the problem
In VS Code, run the TypeScript: Open TS Server log command
This should open a large log file called tsserver.log

⚠Warning: The TypeScript log may include information from your workspace, including file paths and source code. If you have any concerns about posting this publicly on Github, just let me know and we can arrange something else. On our side, we only use these logs to investigate issues like this

Hello again, @mjbvz
The file is available here. Hope this helps!


Add correct topology of frames to broadcast TF2 information

The current implementation of drake_ros_tf2 provides correct transforms for all frames in a system, but they are all rooted in the world frame. Although this does not have an impact on use of the frames via tf2, it does not align with how frames are usually presented in ROS, which is as a tree with each frame being rooted in its real-world parent (i.e. a kinematic chain).
The problem is compounded when there are multiple robots in a scene. It can become confusing to find the correct frame for a particular robot except by naming - which assumes that the names of the frames are suitable for identifying their topological order.
There are some possible ways to resolve this. One is to get the topology information from the Multibody Plant. Another is to modify Drake (and push this modification upstream in a PR) so that the Scene Graph knows about topology of the objects in the graph. (See meshcat_visualizer.cc in Drake.)

Fixed by #91. Closing as completed.


Add an animation for Slider value change

Describe the Feature
A prop to enable animating the movement of the Slider since just by updating the value it looks very "grainy" and is subpar for anything presentable
Possible Implementations
There is the same functionality in https://github.com/jeanregisser/react-native-slider and it works well for iOS, but for me disables the Slider when testing in debug mode on Android.

Thanks for reporting that! It sounds like a nice feature.

Despite what all the Github stars may suggest, this library is really not usable for professional, public UI until this gets implemented.


AttributeError in scRNAseq clustering notebook with nbtools 19.09

I am trying to run the single cell clustering notebook locally in genepattern/genepattern-notebook:19.09.1. When the first cell with a UIBuilder auto-runs, I get this error:

The notebook runs fine on the public notebook server. I'm not sure what image version is running on the public server, but I did notice that my local image has nbtools==19.09 and the production notebook as nbtools==19.07.4.
Some light googling seems to indicate that otherwise nonsensical AttributeErrors can be caused by self-referential imports in some cases, and I did notice this line in a recent commit, maybe the issue is related to that?

Hi Alex,
19.09 is still in beta and Thorin is working on a fix for this, coming out of the testing I've been doing this week.
If you can revert to 19.07.1 (that is what is running in the production workspace) it should work.


App doesn't start - Couldn't register with accessibility bus: Did not receive a reply.

It's me again, well, I had to reinstall archlinux and now when I try to run twitch-indicator, it shows me this, I searched for the problem but any solution worked for me, so I was wondering if it was some kind of dependence.

The error shows that it can't import AppIndicator3 which is in girl1.2-appindicator3-0.1 package.

Well, sorry for the late response, but I couldn't find the solution at the moment and was Very busy.
But today, I had to reinstall f.lux (I don't know if you know this program) and it installed some packages as dependences: libappindicator-gtk2 libappindicator-gtk3 libappindicator-sharp.
After that, I remembered what was the problem with twitch-indicator, so I decided to give it a try, and it works normal now, pretty interesting.


documentation updates

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[X] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[x] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.

adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[X] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

Can one of the admins verify this patch?

/azp run automation - sdk

Most of the changes are from our documentation team to rewrite some descriptions.
Yet, there is 1 change that might affect SDK creation which is the change of operation from secureScoreControlDefinitions_List to SecureScoreControlDefinitions_List

@giromm2ms could you explain why you need change the operationId?

There was a casing issue and I wanted to align it to be consistent with all other operation ids
Sent from Outlook Mobilehttps://aka.ms/blhgte

From: Lei Ni<EMAIL_ADDRESS>Sent: Wednesday, May 27, 2020, 06:00
To: Azure/azure-rest-api-specs
Cc: Gideon Romm; Mention
Subject: Re: [Azure/azure-rest-api-specs] documentation updates (#9583)
@giromm2mshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fgiromm2ms&data=02|01|gideon.romm%40microsoft.com|5befa64f49fd477b17d608d801ea0c47|72f988bf86f141af91ab2d7cd011db47|1|0|637261452078491930&sdata=qHQusjm%2F0R7KDhuSFs4HXonEYzjasQSaRxlXsXp0l2w%3D&reserved=0 could you explain why you need change the operationId?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-rest-api-specs%2Fpull%2F9583%23issuecomment-634398089&data=02|01|gideon.romm%40microsoft.com|5befa64f49fd477b17d608d801ea0c47|72f988bf86f141af91ab2d7cd011db47|1|0|637261452078491930&sdata=90aKYd3jqty7HHQ06sFG7VmvA0BId7fSmEfA2FqiwBg%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAMP4G3ZWTK24RZJSHDT5DHDRTR63BANCNFSM4NKJA5AA&data=02|01|gideon.romm%40microsoft.com|5befa64f49fd477b17d608d801ea0c47|72f988bf86f141af91ab2d7cd011db47|1|0|637261452078501925&sdata=6%2Buhn6%2BUp23bJ5%2BB9Jxq1POGeWMg5rgPQVf6hJnnnMA%3D&reserved=0.

@giromm2ms got it. Which SDK are you going to release? I'd like to invite the sdk owners to review the breaking change.

All, mainly python so we can wrap it with CLI and PS
Sent from Outlook Mobilehttps://aka.ms/blhgte

From: Lei Ni<EMAIL_ADDRESS>Sent: Wednesday, May 27, 2020 7:50:16 AM
To: Azure/azure-rest-api-specs<EMAIL_ADDRESS>Cc: Gideon Romm<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Azure/azure-rest-api-specs] documentation updates (#9583)
@giromm2mshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fgiromm2ms&data=02|01|gideon.romm%40microsoft.com|cac7423548884060a7c508d801f9736a|72f988bf86f141af91ab2d7cd011db47|1|0|637261518180606942&sdata=y9Pp2lKhGFIwYmH6CU42F6ezmboorVoRANDu9zoQy4o%3D&reserved=0 got it. Which SDK are you going to release? I'd like to invite the sdk owners to review the breaking change.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-rest-api-specs%2Fpull%2F9583%23issuecomment-634426425&data=02|01|gideon.romm%40microsoft.com|cac7423548884060a7c508d801f9736a|72f988bf86f141af91ab2d7cd011db47|1|0|637261518180606942&sdata=f2%2FBgV%2BbMrEaerSdW7OaPZLZTXxefEMzNuZaMg6N29E%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAMP4G36ZKIGSB25U6H2N4ILRTSLYRANCNFSM4NKJA5AA&data=02|01|gideon.romm%40microsoft.com|cac7423548884060a7c508d801f9736a|72f988bf86f141af91ab2d7cd011db47|1|0|637261518180616936&sdata=R%2FjZ8%2BrTNj2KXZmWDr50PbmwDrUGklJ6PBTK%2BjflZDM%3D&reserved=0.

operationId secureScoreControlDefinitions_List to SecureScoreControlDefinitions_List

@zikalino could you confirm the breaking change is ok with you?

i don't think breaking changes here are caused by this PR, i think they were caused by another PR that was already merged. so this pr should be ok to merge

This rename:
operationId secureScoreControlDefinitions_List to SecureScoreControlDefinitions_List
Doesn't affect Python SDK, as these operations were not included in previous release


Feedback/Suggestions

Fantastic module! I've got a few ideas that would make it insanely perfect (for my very specific use anyway):
I'm running the Combat Utility Belt module, and I've rigged it with Triggler so when an NPC hits zero hit points, it triggers the "Generate Currency and Convert to Loot for Selected Tokens" macro. This is awesome, since once the NPC's health drops to <=0, it pops up a window asking if I wanna convert said NPC. Beautiful!
Feedback:
1.) One concern is it might be best if instead of triggering on the selected token, I wonder if there is a way to make it apply to "whichever token triggered the event". Just in case I'm selecting someone else when an NPC dies. Maybe as simple as a new macro/line of code in existing macro to get triggered instead?
2.) Settings wise, I would love a bit more granularity. For example. I never use platinum in my game, and would like the option to nix that from being handed out (similar to how I also don't use electrum, which I do appreciate being able to nix that one in the options!).
3.) I also would love the option to set a percentage of chance that armor/weapons are too damaged to be included in the loot. I can already imagine my players stockpiling weapons from the dead unnecessarily just because they can, lol.
4.) Finally, (and this probably could be solved with just a macro, or a commented out section in an existing macro that could be replaced), I'd like to be able to trigger a roll table to drop loot into the NPC's inventory. For instance, currently I have it set where CUB triggers your macro to make the NPC lootable. It simultaneously triggers a macro that I have set up that rolls a rolltable of rolltables (using the Loot Table by Better Roll Tables module). Currently all the resulting items from my rolltable show up in the chat window. I'd love it if I could make those items drop into the loot sheet automatically. I've attached an image of my rolltable set up for clarity. Actually, what I'd really love is for a window to pop up where I could approve/disapprove which items from the table will be dropped into the sheet from my rolltable result, but maybe I'm getting out of hand here!

These are all good ideas. Let me respond to each one individually.

This should be doable. I'll probably do a check to see if a token is passed in and if not, use the selected one. I just need to figure out the syntax for how the arguments are passed.
This is easy enough, but I would like some more clarity on the level of granularity. Let me know exactly what settings would be useful for you, and I can translate that. For instance, do you want some check boxes that allow you to decide which currencies are included or something like that?
This one is very easy. I'll just add a couple checks to ensure we're not deleting any magic items.
I had considered doing it in the past, but Loot Sheet NPC 5e already has this functionality built into the sheet itself. Meaning, you could manually switch it over to a Loot Sheet, generate the items, and then switch back to a regular sheet. However, you're the second person who has asked for this, so I'll look into it more. Maybe I can hook into something Loot Sheet exposes and populate it that way. But yeah, we'll see for it. If I am able to figure it out, it won't be for a while.

Excellent! As for the granularity of #2, checkboxes would work perfectly, 1 for each denomination of coin.
Too bad about #4, I tried the idea of swapping back and forth on Loot Sheet, but it was pretty clumsy, since I'd have to choose the roll table and adjust the formula each time. My current macro is smoother where it rolls automaticallyand I drag from the chat window into the sheet. I'll keep that up for now.
Glad they're helpful ideas. Thanks for creating such a great module. I love letting my player loot, but it's always been clumsy. This module kicked my butt into gear to make a good roll table, and now I can implement it far better than before. Looking forward to the update!

 It's great hearing that type of thing. 👍
I went ahead and created four separate issues based on your requests (#8, #9, #10, and #11) as I will probably be including these in four separate releases. Feel free to add any other important details to the corresponding issue. I'm going to close this one as a result.


undefined FLAGS

I'm not 100% sure but i think, looking at the documentation for discord.js that "Intent" should be "GatewayIntentBits" in this file, but i'm not sure so I'm doing a bug report instead of PR
https://github.com/Markoudstaal/node-red-contrib-discord-advanced/blob/main/discord/discordMessage.js

https://discordjs.guide/popular-topics/intents.html#privileged-intents

seems like discord pushed some broken update yesterday for their API
https://status.tickettool.xyz/incidents/v5lvlrkfsmdv

I'm not 100% sure but i think, looking at the documentation for discord.js that "Intent" should be "GatewayIntentBits" in this file, but i'm not sure so I'm doing a bug report instead of PR https://github.com/Markoudstaal/node-red-contrib-discord-advanced/blob/main/discord/discordMessage.js

Maybe you installed a new specific version of discord.js from console or you are using a function node with a discord.js reference. The current version of the node-red-contrib-discord-advanced use a discord.js version does not have GatewayIntentBits.
I updated the discord.js version to 14.7.1 in alpha version.
#64

@geekgarage Try now with the last version of the node

Works as it should THANK YOU


Error: connect EACCES <IP_ADDRESS>:80

I copied pasted example 4 to send an email and here is what I got:
Error: connect EACCES <IP_ADDRESS>:80
at Object._errnoException (util.js:1022:11)
at _exceptionWithHostPort (util.js:1044:20)
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1198:14)
Is it possible to change the port? I guess that's where the problem comes from, port < 1024
EDIT: I'm trying to to send an email in an application hosted on Azure

Hello @JasonBBelcher,
is it possible to add the brief reason why this closed? is it some kind of misconfiguration or what?

I found the misconfiguration. I provided the protocol in 'host' property - only domain was specified. This is a cause of redirect to localhost.


refactor(rpc): prefer monitored_connect over connect

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

#18635  👈
main

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

Join @BugenZhao and the rest of your teammates on  Graphite


Update and Move eng/common

Update and Move eng/common workflow to internal Wiki

/azp run azure - sync - eng - common

/azp run azure-sdk-tools - sync - eng - common

/azp run azure-sdk-tools - sync - eng-common

/check-enforcer override


WA hits python error under 32-bit ubuntu OS with python2.7

2016-06-20 15:16:09,738 DEBUG    android: adb -s <IP_ADDRESS>:5555 pull '/sdcard/wa-working/cpufreq.tar.gz' 'wa_output/idle_big_big_idle_state0_1_1/cpufreq.tar.gz'
2016-06-20 15:16:10,042 ERROR    instrumentation: Error in insturment cpufreq
2016-06-20 15:16:10,044 ERROR    instrumentation:   File "/usr/local/lib/python2.7/dist-packages/wlauto/core/instrumentation.py", line 240, in call
2016-06-20 15:16:10,044 ERROR    instrumentation:     self.callback(context)
2016-06-20 15:16:10,044 ERROR    instrumentation:   File "/usr/local/lib/python2.7/dist-packages/wlauto/instrumentation/misc/init.py", line 166, in update_result
2016-06-20 15:16:10,044 ERROR    instrumentation:     tf.extractall(context.output_directory)
2016-06-20 15:16:10,044 ERROR    instrumentation:   File "/usr/lib/python2.7/tarfile.py", line 2079, in extractall
2016-06-20 15:16:10,044 ERROR    instrumentation:     self.extract(tarinfo, path)
2016-06-20 15:16:10,044 ERROR    instrumentation:   File "/usr/lib/python2.7/tarfile.py", line 2116, in extract
2016-06-20 15:16:10,044 ERROR    instrumentation:     self._extract_member(tarinfo, os.path.join(path, tarinfo.name))
2016-06-20 15:16:10,044 ERROR    instrumentation:   File "/usr/lib/python2.7/tarfile.py", line 2209, in _extract_member
2016-06-20 15:16:10,044 ERROR    instrumentation:     self.utime(tarinfo, targetpath)
2016-06-20 15:16:10,044 ERROR    instrumentation:   File "/usr/lib/python2.7/tarfile.py", line 2330, in utime
2016-06-20 15:16:10,044 ERROR    instrumentation:     os.utime(targetpath, (tarinfo.mtime, tarinfo.mtime))
2016-06-20 15:16:10,044 ERROR    instrumentation:
2016-06-20 15:16:10,044 ERROR    instrumentation: OverflowError(Python int too large to convert to C long)
2016-06-20 15:16:10,044 DEBUG    instrumentation: Disabling instrument cpufreq
Even if we write standalone demo python script under the same environment ( copy cpufreq.tar.gz to local folder),  there will be the same error.
Traceback (most recent call last):
File "test.py", line 25, in
main()
File "test.py", line 20, in main
tf.extractall("./")
File "/usr/lib/python2.7/tarfile.py", line 2079, in extractall
self.extract(tarinfo, path)
File "/usr/lib/python2.7/tarfile.py", line 2116, in extract
self._extract_member(tarinfo, os.path.join(path, tarinfo.name))
File "/usr/lib/python2.7/tarfile.py", line 2209, in _extract_member
self.utime(tarinfo, targetpath)
File "/usr/lib/python2.7/tarfile.py", line 2330, in utime
os.utime(targetpath, (tarinfo.mtime, tarinfo.mtime))
OverflowError: Python int too large to convert to C long

Hi,
This seems to be an issue with python rather than WA. It looks like the timestamp of the files is set beyond 2038-1-19 (which would put it over the maximum value for a 32-bit integer). Can you check the date on the target device?

Yes, this issue can be resolved under 64-bit Ubuntu OS.

Please could you confirm what date is being set on the target device?

It's something like:
root@juno:/s # date
Fri Apr  9 22:51:07 GMT 2066

Ah then there is your problem. A 32-bit integer cannot represent a date past 2038-01-19. This is a fundamental limitation of 32-bit operating systems not an issue with WA. I would recommend either setting the correct date on the target device and/or switching to a 64-bit operating system.


PR: Add Jedi as a new dependency for users to be aware of its right version

This PR also reorganizes our list of dependencies in alphabetical order.
Fixes #11047.

Hello @ccordoba12! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file spyder/dependencies.py:

Line 109:80: E501 line too long (81 > 79 characters)
Line 117:80: E501 line too long (84 > 79 characters)
Line 146:80: E501 line too long (80 > 79 characters)
Line 204:80: E501 line too long (91 > 79 characters)
Line 236:80: E501 line too long (83 > 79 characters)


Remove hard coded roll_torques and roll_forces plot

Due to hard implementation of roll torque and roll force plots it's not possible to solely simulate cooling processes.
Remove hard dependencies and move plots to plugins.

Would not remove it, just check if the list of forces or torques is empty.


git component first push throws error of "timed out waiting for the condition"

[kind/bug]

What versions of software are you using?

In OpenShift CI on two test node and these test runs on each node with complete isolation simultaneously.
Actual behavior
Flaky
Expected behavior
Should not be flaky
Any logs, error output, etc?
odojsonoutput odo machine readable output on project nodejs is deployed
  should be able to list url
  /go/src/github.com/openshift/odo/tests/integration/json_test.go:94
 Created dir: /tmp/107843751
Creating a new project: pisruzhzmt
Running odo with args: [project create pisruzhzmt -w -v4]
[odo] I0728 05:56:04.245893   24345 preference.go:116] The configFile is /tmp/107843751/config.yaml
[odo] I0728 05:56:04.245994   24345 occlient.go:486] Trying to connect to server api.ci-op-zlj5gmpl-f09f4.origin-ci-int-aws.dev.rhcloud.com:6443
[odo] I0728 05:56:04.261297   24345 occlient.go:493] Server https://api.ci-op-zlj5gmpl-f09f4.origin-ci-int-aws.dev.rhcloud.com:6443 is up
[odo] I0728 05:56:04.327344   24345 occlient.go:416] isLoggedIn err:  <nil>
[odo]  output: "developer"
[odo] I0728 05:56:04.327380   24345 context.go:367] Ignoring error as it usually means flag wasn't set: flag accessed but not defined: context
[odo] I0728 05:56:04.327388   24345 context.go:367] Ignoring error as it usually means flag wasn't set: flag accessed but not defined: context
[odo] I0728 05:56:04.327437   24345 context.go:367] Ignoring error as it usually means flag wasn't set: flag accessed but not defined: app
[odo] I0728 05:56:04.327442   24345 context.go:367] Ignoring error as it usually means flag wasn't set: flag accessed but not defined: project
[odo] I0728 05:56:04.341654   24345 context.go:367] Ignoring error as it usually means flag wasn't set: flag accessed but not defined: app
[odo] I0728 05:56:04.341680   24345 context.go:367] Ignoring error as it usually means flag wasn't set: flag accessed but not defined: output
[odo] I0728 05:56:04.341687   24345 context.go:367] Ignoring error as it usually means flag wasn't set: flag accessed but not defined: component
[odo]  •  Waiting for project to come up  ...
[odo]  ✓  Waiting for project to come up [332ms]
[odo]  ✓  Project 'pisruzhzmt' is ready for use
[odo] I0728 05:56:04.687589   24345 odo.go:70] Could not get the latest release information in time. Never mind, exiting gracefully :)
[odo]  ✓  New project created and now using project : pisruzhzmt
Current working dir: /go/src/github.com/openshift/odo/tests/integration
Setting current dir to: /tmp/107843751
Running odo with args: [create nodejs nodejs --app myapp --project pisruzhzmt --git https://github.com/openshift/nodejs-ex]
[odo]  •  Validating component  ...
[odo]  ✓  Validating component [355ms]
[odo] Please use `odo push` command to create the component with source deployed
[odo]
Running odo with args: [url create myurl]
[odo]  ✓  URL myurl created for component: nodejs
[odo]
[odo] To create URL on the OpenShift Cluster, please use `odo push`
Running odo with args: [push]
[odo] Validation
[odo]  •  Validating component  ...
[odo]  ✓  Validating component [347ms]
[odo]  •  Checking component  ...
[odo]  ✓  Checking component [16ms]
[odo]
[odo] Configuration changes
[odo]  •  Creating component  ...
[odo]  ✓  Initializing component
[odo]  •  Triggering build from git  ...
[odo]  ✓  Triggering build from git [79ms]
[odo]  •  Waiting for build to finish  ...
[odo]  ✗  Waiting for build to finish [30s]
[odo]  ✗  Failed to create component with name nodejs. Please use `odo config view` to view settings used to create component. Error: unable to wait for build nodejs-myapp-1 to run: timed out waiting for the condition
[...]

@mik-dass Don't you think this issue is somehow related with #1782 ?

@amitkrout #1782 is a update issue. From the logs, this looks like a git component creation issue.

@amitkrout #1782 is a update issue. From the logs, this looks like a git component creation issue.

Thanks for confirming :)

/state in-analysis

@amitkrout are we still facing this error?


driver: sensors: convert lsm9ds0_gyro to use information coming from dts

Convert lsm9ds0_gyro driver to get the device name as well as
i2c slave informationnd gpio info for triggers from device tree.
Updates the build_all test accordingly.
Signed-off-by: Armando Visconti<EMAIL_ADDRESS>

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@bfe0c2e). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #12183   +/-   ##
=========================================
  Coverage          ?   48.06%
=========================================
  Files             ?      284
  Lines             ?    43384
  Branches          ?    10398
=========================================
  Hits              ?    20851
  Misses            ?    18390
  Partials          ?     4143

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 bfe0c2e...b6b7ae0. Read the comment docs.

Code changes look good, but please fix the typo in the commit message. informationnd
should be information and

Ooops... thx!


Question: How to set Cache-Control on upload?

I've tried:
signingUrlQueryParams={{ 'x-amz-meta-Cache-Control': 'max-age=31536000'}}
signingUrlHeaders={{ 'x-amz-meta-Cache-Control': 'max-age=31536000'}}
uploadRequestHeaders={{ 'x-amz-meta-Cache-Control': 'max-age=31536000'}}
The first two allow the upload to go through, but not setting the cache-control meta data.  The last returns a 403 error.  Any insight?
Much appreciated.

I haven't done this, so not sure, but the backend that creates a signed URL may need to be responsible for adding the headers (or proxying them from the front-end request).  I imagine these headers would be set the same way 'x-amz-acl' is set, so maybe follow that through the code.

@sators Do you have a solution?

I did not @kuka until you came around!  Thanks!


VnV Plan review: Functional requirement

Your section 5.1.1 to 5.1.3 are too confusing.
Reasons:

[x] - I guess you wrote two subcategories of 5.1.1 are 5.1.2 and 5.1.3. - So, it should be either sub sub categories.
[x] - Also, same table for both subcategories make it even more confusing. - I think you should separate it for different subcategories.
[x] - How testing for "testing output with non-trivial inputs" is performed? - It's not clear.

You can work on it to make it better.

Thank you, I have clarified it a bit more! 726e3e23c


Introducing Xml Serializer Generator for .NET Core

Introducing Xml Serializer Generator for .NET Core
Like the Xml Serializer Generator (sgen.exe) on desktop, Microsoft.XmlSerializer.Generator NuGet package is the solution for .NET Core and .NET Standard Libraries. It creates an Xml serialization assembly for types contained in an assembly in order to improve the startup performance of Xml serialization when serializing or de-serializing objects of those types using XmlSerializer.
You can start using the tool today by following the instructions.
Discussion
Please share your thoughts and have any discussions with us at:

dotnet/corefx #25138

Details
When the Xml Serializer Generator is not used, an XmlSerializer generates serialization code for each type every time when an application runs. You can improve the startup performance of Xml serialization by simply adding the reference of Microsoft.XmlSerializer.Generator NuGet package to your project. Xml Serializer Generator will generate the serialization code and compile it into an assembly next to your output assembly at build time in advance. This assembly can then be deployed and ran with your application.
Please create an issue with your feedback at dotnet/corefx. We are actively looking to improve the tool and your input is a valuable part of the process.

Xml Serializer Generator for .NET Core v1.0.0 Released
Stable version (v1.0.0) of Microsoft.XmlSerializer.Generator NuGet package has just been released. Instructions is also updated. Please share your feedback or have any discussions at dotnet/corefx#25138.


Windows 8 Support

From Reddit: https://www.reddit.com/r/rust/comments/ene9mm/introduction_roguelike_tutorial/
This is really great!
I'm having some difficulty getting the "hello world" example running. After setting up the TOML / SRC / resources folders and files I just get a Windows error that says "The procedure entry point AdjustWindowRectExForDpi could not be located in the dynamic link library "My path to hellorust.exe""
I get a Rust error / panic that says "Exit code: 0xc0000139, STATUS_ENTRYPOINT_NOT_FOUND"
I've tried in both release and debug mode.
I'm on Windows 8, if that matters.

This has been merged, though I am not sure if it has been verified.


Aggregates and Grouped Daily methods

Are you planning on adding methods for aggregates and grouped daily?
https://polygon.io/docs/#get_v2_aggs_ticker__ticker__range__multiplier___timespan___from___to__anchor
https://polygon.io/docs/#get_v2_aggs_grouped_locale__locale__market__market___date__anchor

Hey @zenichanin
Currently built this wrapper SDK to make the API easier to use, If you want to fork and implement the additional features, send in a pull request and we can build it up. At the moment I've deep into a few big projects, I don't have the time to do it. I do have a few upgrades I want to make eventually, been using this within my own platform, and have been great so far.
Let me know if you have any more suggestions.


Remove RegistryAuthenticator.Initializer

@loosebazooka I don't know if you tried something on the Initializer, but I think it's textually redundant in that it's duplicated in other methods.

@loosebazooka @briandealwis we talked about starting afresh about auth, and I think this PR may actually help that by eliminating the weird Initializer out of the picture. This looks easier to understand. WDYT?


Provide webdriver support for non-Linux LUCI bots

Currently webdriver tests can only be run on Linux configurations. We should run on OSX and Windows when possible.

@bkonyi What prevents you from doing so? The work done for https://github.com/dart-lang/sdk/issues/42417 should work on OSX and Windows as well.

I thought you had mentioned that this would only work on Linux for some reason? :-) I created this issue to tie to an approval for failing SSE tests on Windows / OSX which I assumed were due to lack of support. If that's not the case I'll update this issue to reflect the actual problem and assign to myself.

Pretty sure this was done a long time ago. Closing.


Add HexaCalc by @AnthonyH93

[x] Project URL: https://github.com/AnthonyH93/HexaCalc
[x] Update contents.json instead of README
[x] One project per pull request
[x] Screenshot included
[x] Avoid iOS or open-source in description as it is assumed
[x] Use this commit title format if applicable: Add app-name by @github-username
[x] Use approved format for your entry

Thx!


Detecting Extensionless C++ Headers

C++ libraries have an unfortunate convention of creating headers without extensions.
VSCode then doesn't detect those headers are C++, breaking tooling. It's be awesome if VSCode could perform some simple heuristics to detect those extensionless headers as being in the C language family.
More details:

Examples of extensionless headers:

Think the standard library's <iostream>.
Another good example I just ran into is Eigen, a popular linear algebra library. An example header would be "Eigen/Dense".

Some STL implementations already work because VSCode detects a magic comment (see https://github.com/microsoft/vscode/issues/23873), but many headers, like Eigen's, don't have such a magic comment.
The reason this is a bummer for usability is that it stops tooling for C++ from being automatically applied to the file.

Manually telling VSCode to mark the file as C++ by clicking in the lower right does seem to fix the issue, but it'd be nice if it happened automatically.
Even just marking no-extension headers that contain #include or #if at the start of a line would do the trick--and be a better default than plain text!
I'd originally brought this up wrt the clangd plugin (https://github.com/clangd/vscode-clangd/issues/139). While they have a bit more information about the include graph with which to infer headers, I now think this is more properly handled at the VSCode level. Other C++ extensions would have the same issue, I imagine. Looks like the Microsoft C++ extension ran into it, too.

Thank you for your consideration and a most incredible piece of software!

Since #23873 and #1800 have been resolved, I believe now it's the extension authors' responsibility to implement any algorithm they need to detect file types. (the Modelines extension does this, as mentioned in your linked issues)
A large portion of Code users don't code in C++ and won't want every file they opened to be checked for C++ syntax.

Ah! My apologies. If that's indeed general enough to check more than modlines, let's close this down.

Yes, #1800 added a vscode.languages.setTextDocumentLanguage API to let extension switch editor language mode dynamically.
Extensions can scan each opened file, detect their types, and call this API to tell Code the correct language. A quick glance of how Modelines does this:
https://github.com/ctlajoie/vscode-modelines/blob/b7b20dde15f3ab57c11c044f76366a49f18f5bd9/src/modelines.ts#L188-L215
export function applyModelines(editor: vscode.TextEditor|undefined): void {
	if (!editor || !editor.document || editor.document.isUntitled)
		return;
	try {
		let searcher = new ModelineSearcher(editor.document);
		let options = searcher.getModelineOptions();
		console.log('[modelines] setting editor options: ' + JSON.stringify(options));
		let language = translateLanguageName(options.language);
		if (options.language)
			delete options.language;

		extend(editor.options, options);
		// assignment is necessary to trigger the change
		editor.options = editor.options;

		if (language && language.length > 0) {
			vscode.languages.getLanguages().then(codelangs => {
				let codelang = codelangs.find(codelang => codelang.toLowerCase() === language.toLowerCase());
				if (codelang) {
					console.log('[modelines] setting language to '+codelang);
					vscode.languages.setTextDocumentLanguage(editor.document, codelang);
				}
			});
		}
	} catch (err) {
		console.error(err);
	}
}

"... A large portion of Code users don't code in C++ and won't want every file they opened to be checked for C++ syntax...."
A few C/C++ lucky ones can use the ever-excellent (and free) Visual Studio.
VSCode is slowly but surely approaching a point of implosion due to the "jack of all trades" policy seemingly applied.

@yume-chan thanks a lot for answering

Thanks all! Had mistakenly figured VSCode wanted to determine the languages itself.
FWIW, I think VSCode's ability to support a wide range of languages and extensions to be a real plus. We've certainly appreciated it! It lets us edit 10+ languages seamlessly in the same place.

"... It lets us edit 10+ languages seamlessly in the same place..." ... Bless


Create a mentor handbook

I have a copy of the one for MOL7 that we could use as example

To add (from cohort call):

Starting your Mentoring Relationship Checklist

Matching
Before the meeting
The first meeting
Share your story
Discuss Communication Styles
Set Goals
Establish the Flow

The commitment - Before embarking on a mentoring relationship, be sure that you have the time and energy to dedicate to your mentee.

Provide mentors with opportunities to reflect on whether or not they have the capacity to do this before and during selection, before the program begins.

This exists :)


Write PDS job uuid as meta data

Situation
Currently we have only mapping of PDS jobs and SecHub jobs inside our Logfiles.
Wanted
An administrator shall be able to fetch PDS job uuid
Solution
We write the PDS job uuid into adapter metadata when job has been created.
When  #799 is implemented, the administrator will be able to fetch the sechub job uuid by reading adapter meta data for given sechub job.

After # 643 has been implemented, this implementation will greatly simplify the handling and runtime analysis of PDS jobs. We still need two admin accounts - for security reasons - but after his a lot can then be automated.

@de-jcup is this done?


fix code block

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers: rootsongjc, sdake
If they are not already assigned, you can assign the PR to them by writing /assign @rootsongjc @sdake in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content/OWNERS
content_zh/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/assign @rootsongjc @sdake

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: rootsongjc
If they are not already assigned, you can assign the PR to them by writing /assign @rootsongjc in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content_zh/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@yangchuansheng  I would like to see some descriptin why this is needed, such as you can add some commit message in description.


feat(object): prop returns the argument if not an object

I think this handling of non-object arguments is more useful than just returning undefined. It makes it similar in use to invoke, which returns the argument if its not a function.

Bad idea


Error handling

This is really great work, @rubenarakelyan – but could I suggest one small improvement, that is in error handling? Currently, errors internal to your class are handled using die(), whereas errors coming back from the TWFY services are returned as array('error'=>'Invalid postcode').
When die is used, it's hard to capture and process them sensibly. I'd like to suggest handling them in a similar way to TWFY and returning an array with the element error.
I'll fork the project and make the changes and issue a pull request. Let me know if you think it's sensible...

Thanks very much for your work on this and sorry for the massive delay in getting it merged in!

That's alright, my pleasure! I'm the one that should thank you for the
original SDK – it helped get my project off the ground.
On 19 March 2015 at 20:32, Ruben Arakelyan<EMAIL_ADDRESS>wrote:

Thanks very much for your work on this and sorry for the massive delay in
getting it merged in!
—
Reply to this email directly or view it on GitHub
https://github.com/rubenarakelyan/twfyapi/issues/3#issuecomment-83753394
.


[BEAM-1218] Rename google_cloud_dataflow and google_cloud_platform

R: @aaltay PTAL
Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

LGTM

Changes Unknown when pulling 94cec999d7a30b211345adf9be879dfb2bfada06 on sb2nov:BEAM-1218-rename-directories into ** on apache:master**.


Log peer reputation score on disconnect

Add peer reputation score to disconnect message
We want to know if we're booting good peers, or if they are booting us.

this is covered by #3665. no need for a separate ticket.


glfw does not distinguish between numpad keys based on num lock state

glfw returns the exact same key and scan codes regardless of whether num lock is on or off. This makes it impossible to distinguish between KP_7 and KP_HOME for example.
Also glfwGetKeyName() returns NULL for numpad keys.
The list of glfw keys should be extended to add KP_HOME, KP_END, KP_INS, KP_DEL, KP_PG_DN and KP_PG_UP and glfw should return the appropriate key depending on what the underlying window system reports.
On X11, for example, xev is able to distinguish between KP_7 and KP_HOME, reporting them correctly.

This is definitely a hole in the API. However, key tokens like GLFW_KP_7 represent physical keys and not the function assigned to them by the OS, and changing that during 3.x would break existing applications.
Instead, I can add queries for lock key states and you can choose the behavior of the key based on the Num Lock state. Would that be sufficient for your purposes?

Being able to query num lock state is certainly a first step, however, it is a hack -- since the behavior of num lock is configurable at the OS level, and applications should follow the OS as much as possible. Otherwise, glfw based applications will behave differently than all other applications on systems where the behavior of num lock has been changed.
The ideal solution would be a key callback API that reports logical key names as reported by the OS after any keyboard layout/lock key mapping etc takes place. This will not break existing applications, which could continue to use the old API. The new API could either use a new callback function or a switch that changes the behavior of the existing one at runtime.

Yup, it won't work on systems with home-made keyboard layouts. Normally I'd argue that this is extremely rare, but you're writing a terminal emulator so that probably isn't true.
The input system for GLFW is by design a simplified abstraction of the OS APIs geared towards games and related tools. I won't add another key callback but once it's time for 4.0 we can look into improving the existing key event. For now, with many higher priority features still to be implemented, lock key state is what I can offer.

Sure, I wont refuse the ability to probe the lock key state :)
And let me just say thanks for glfw, which apart from a few minor issues, works very well.

Either solution would be useful. I'm currently hacking around this by using a Windows system call.

I'm currently considering one of the following solutions:
Option 1: A function to query the current lock state:
int glfwGetKeyLock(GLFWwindow* window, int key);

Where key is one of GLFW_KEY_*_LOCK. The returned value would be up-to-date wrt the event currently being processed.
Option 2: Additional bits in the modifiers passed to the key and charmods callbacks:
#define GLFW_MOD_CAPS_LOCK 0x0010
#define GLFW_MOD_NUM_LOCK  0x0020

I vote for option 1 on the grounds that it is less likely to cause compatibility issues with code that checks the masks incorrectly.

You read my mind. Option 1 it is.

FWIW seeing this in GLFW 3.3 would be amazing!!

The modifier bit path was so much cleaner internally and conceptually simpler externally (no potentially out-of-date cached state) that I had to give it a try.
What do you think of f3de10d297582dac1b986088524c84d5f1435091?

Fine by me.

I'd argue that this is extremely rare

I'm another guy who needed it for https://vcvrack.com/ to implement a togglable "virtual computer keyboard MIDI device". Much thanks!

@elmindreda is this also released, like #1098 ?

@AngelDeaD Sort of. The 0e8c4ea7ce653cc7346bfe9fd5e5ab3933961934 patch above is merged and if you turn lock key modifiers on (see GLFW_LOCK_KEY_MODS) then you can tell them apart by GLFW_MOD_NUM_LOCK. Having actual separate symbols for the different uses of the same physical keys has to wait until 4.0.


How to do pagination with this package ?

We have a need of pagination in vertical list view .  2-way actually both top and bottom ..
How this can be achieved ? I don't see any scroll change listener in the code like Google's ListView package has ..

Please refer https://github.com/robert-luoqing/flutter_list_view/blob/master/example/lib/pull_to_refresh_list.dart. It integrates with pull_to_refresh to implement pagination

@robert-luoqing  that's not I meant by pagination .. I need to make next page page once list is at bottom/top.
Anyways I found the answer. I am using the code below . and it works fine..
 `listViewController.sliverController.onPaintItemPositionsCallback =
    (double widgetHeight, List<FlutterListViewItemPosition> positions) {
  var firstVisiblePosition = positions.first;
  var lastVisiblePosition = positions.last;
  // here we can build logic around above position to call the next/prev page

};`

Thanks for this amazing package man ...


use async-graphql over juniper

async-graphql is another gql framework for rust and overall, it seems to be better than juniper according to their comparison. I'ved used both and async-graphql seems better to work with

Interesting! I actually wasn't aware of this async-graphql crate. I cannot promise anything however, as there are many projects that need attention and this would be s fairly substantial rewrite. I'm happy to review+accept a PR however!

Interesting! I actually wasn't aware of this async-graphql crate. I cannot promise anything however, as there are many projects that need attention and this would be s fairly substantial rewrite. I'm happy to review+accept a PR however!

I might PR this for a better DX with the app I'm currently working on as duplicating types is really annoying rn. Subscriptions could also be very helpful with the app. I'll start making a PR soon

But the current version of the plugin has subscriptions too, just FIY. It's just hidden behind the subscriptions flag bc it adds another dependency.

The current version of the plugin has subscriptions too, just FIY. It's just hidden behind the subscriptions flag bc it adds another dependency.

I know, I just prefer async-graphqls impl over juniper


Implements abstracted send/recv functions for KEM public keys and ciphertexts

Please note that while we are transitioning from travis-ci to AWS CodeBuild, some tests are run on each platform. Non-AWS contributors will temporarily be unable to see CodeBuild results. We apologize for the inconvenience.
Issue # (if available): N/A
Description of changes:
This lays the preliminary groundwork for PQ-TLS 1.3.

Adds shared_secret to the PQKEM structure in s2n_crypto_parameters

Also renames the struct: s2n_kem_keypair -> s2n_kem_params

Adds generic send/receive functions for KEM public keys and ciphertexts
Adds function to retrieve a KEM from an extension ID
Integrates the generic KEM send/receive functions with the PQ-TLS 1.2 KEX functions

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1863 into master will increase coverage by 9.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1863      +/-   ##
==========================================
+ Coverage   81.46%   90.47%   +9.00%
==========================================
  Files         226      148      -78
  Lines       16902    10790    -6112
==========================================
- Hits        13770     9762    -4008
+ Misses       3132     1028    -2104

Closing - will break this up into several smaller PRs for easier review.


user12986714: Watch eldwar.com

user12986714 requests the watch of the watch_keyword eldwar\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

eldwar\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Makyen in Charcoal HQ


Helmfile charts deletion order is not correct if we have more than 6 files in sub-charts folder

Hello!
Helmfile chart deletion (from the last chart to first) order is not correct if we have more than 6 files in sub-charts folder. For example, for 7 files we have the next deletion order - first file charts, next 6th. 5th .. 3. 2. and finally 7th.
Correct order should be 7,6,5,4,3,2,1
Correctly working only if we have less than 7 files.
Same situation if files are divided between subfolders.
helmfile version v0.125.5
Commands to run
# Creation
helmfile --file charts/helmfiles/helmfile.yaml --environment dev sync
# Deletion
helmfile --file charts/helmfiles/helmfile.yaml --environment dev destroy

Helmfile.yaml
bases:
  - common/helm-config.yaml
  - common/cluster_templates.yaml
---
helmfiles:
  - path: sub-helmfiles/01-boot.yaml
  - path: sub-helmfiles/02-core.yaml
  - path: sub-helmfiles/03-addons.yaml
  - path: sub-helmfiles/04-services.yaml
  - path: sub-helmfiles/05-backup.yaml
  - path: sub-helmfiles/06-test.yaml
  - path: sub-helmfiles/07-final.yaml

What could be the reason for this issue? Thanks.

@mkruliv Thanks for reporting. This is kind of embarrassing... but there was a terrible bug that resulted in random deletion order, rather than the reverse order. I've already submitted the fix at #1451, and it should be released as the part of upcoming Helmfile v0.126.2!

@mumoshu  Thanks for the help and for the quick response.


Augury impact is excessive when using the Performance Profiler plugin.

Problem:
When looking at the timeline, it's clear that Augury's impact when collecting data is too large.
Augury's CPU usage is many times that of the example app. When using Augury to profile an app with performance issues, it makes the application freeze.
The core is quite lean, and the UI uses a timed repaint strategy to avoid repainting every time it gets new data.
Possible bottleneck:
performance-profiler-plugin does some pretty heavy tree traversals every time it gets new data (which happens extremely often). These could probably be optimized or avoided.
Thoughts?

This plugin looks like it needs a cleanup anyway, I could do a PR?

Any update on this? Is this project dead?

Been meaning to jump on it.. Will do this weekend.

FYI - We're doing some refactoring that should address existing performance concerns. Stay tuned.


Importing components when using SSR

I'm not sure if this is something that works right now, but I'm having trouble getting a component to import other components when using the ssr compiler.
For reference, this is the structure of the code I'm working with in a minimal environment:

index.js is just this:
import { config } from "@vue-email/compiler";
const vueEmail = config("./components", {
    verbose: false,
    options: {
        // baseUrl: "https://vue-email-demo.vercel.app/",
    },
});

const html = vueEmail.render("Email.vue").then(console.log);

The setup for Email.vue looks like this:
<script setup>
import { EHtml, EHead, EBody, ETailwind } from 'vue-email';
import EApiTestWorkflowTwoTemplate from<EMAIL_ADDRESS></script>

Running this results in Error: Cannot find module<EMAIL_ADDRESS>Before trying to do this with SSR, I wanted to prove out this template within a live Vue app and I didn't have any issues like this, but the needs for our project require the SSR utility to produce the rendered html string. Is this possible right now? Am I just not understanding correctly how SSR needs imports to be defined?
This is using "^0.8.0-beta.4".

Same problem

@eabworks @linspw currently import from outside templates it's not possible, this is a feature to add to the next releases, thanks!


Feature request: traditional view for timeline and article

Hoping "Layout" option(s) might be implemented, as in NNW3. (as in attached screenshot)

Seconded - the "Traditional View" in NNW3 is one of the main reasons I still use it over the other Mac RSS readers out there.

Definitely second that. If you implement this, I'll leave my Vienna exile and return to NNW.

Ditto.  Came here to search open issues just looking for this one.  Even though I'm already sold on NNW5, this would be really great IMHO.

+1 Yes, this is necessary.

+1
The usability of a layout is highly subjective. The current 3 column "modern" one is not the best for everyone.

Actually, I think that this is a duplicate of https://github.com/brentsimmons/NetNewsWire/issues/102.

Duplicate of #102.


Feature: Need to delete things and channels

Is your feature request related to a problem? Please describe.
I can add and update things and channels, but I can't find a way to delete them, it's not right
Describe the feature you are requesting, as well as the possible use case(s) for it.
I can add and update things and channels, but I can't find a way to delete them, it's not right
Indicate the importance of this feature to you.
Must-have
Anything else?
No response

i can't find APIs for them

Hello, @andychao217. Currently, we do not support it due to the complex implication removal may cause in the system. You can, however, disable things: https://docs.api.magistrala.abstractmachines.fr/?urls.primaryName=things.yml#/Things/disableThing

Also, please check here: https://docs.api.magistrala.abstractmachines.fr/?urls.primaryName=things.yml#/Things/delete_things__thingID_, we even support the actual hard removal.


[recent history] Recent devcontainer display string corrupted on Get Started page

VSCode Version:.
Version: 1.74.0 (user setup)
Commit: 5235c6bb189b60b01b1f49062f4ffa42384f8c91
Date: 2022-12-05T16:38:16.075Z
Electron: 19.1.8
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19045
Sandboxed: No

Local OS Version: Windows 10 22H2

Remote OS Version: Debian Unstable Linux x86-64

Remote Extension/Connection Type: SSH + Containers (docker is on the remote SSH host)

ms-vscode-remote.remote-ssh version: v0.92.0

ms-vscode-remote.remote-containers version: v0.266.1

Steps to Reproduce:

Open a devcontainer on a remote host via SSH (so the tunnel is: Windows Host ==> SSH Remote Linux ==> Docker Container)
Close the container and open the Get Started page (via menu Help => Get Started)

The devcontainer name in the recent list is somewhat corrupted:

I expect the blue link to be streamlink-debian [Dev Container: Alpine @ ssh://debian64] instead of debian64].
The cause is that VS Code unconditionally use the part after the last / as the blue link:
In Get Started page:
https://github.com/microsoft/vscode/blob/9d1c2920c0d4a433af493b3777b0e8d45088299b/src/vs/workbench/contrib/welcomeGettingStarted/browser/gettingStarted.ts#L858-L864
In quick pick:
https://github.com/microsoft/vscode/blob/9d1c2920c0d4a433af493b3777b0e8d45088299b/src/vs/workbench/browser/actions/windowActions.ts#L219-L230
https://github.com/microsoft/vscode/blob/9d1c2920c0d4a433af493b3777b0e8d45088299b/src/vs/base/common/labels.ts#L441-L450
A workaround is to remove the slashes after ssh:.
Does this issue occur when you try this locally?: No
Does this issue occur when you try this locally and all extensions are disabled?: No

Needs to be fixed in the label provider code.

I would also like to add that this issue affects the Windows "jump" lists that you can access by right clicking the VSCode icon in your taskbar.  I assume its the same code for both issues, but I wanted to mention it just in case it wasn't.

If the solution is too complex for a simple fix, can the label be changed to avoid ssh:// and instead just use ssh: for example ?
This would fix this issue before the complete solution (which may involve more API changes).

I tried to test with Windows Host ==> SSH remote Windows ==> Docker , but it looks like connecting to a container from a Windows SSH remote doesn't work. I will leave for someone else to test.

I've tried on Linux, but I can't get the full remote name and only get foldername [Dev Container] even when connecting to a container on a remote host via SSH (on VSCode Insider).
I've opened https://github.com/microsoft/vscode-remote-release/issues/8578 for this.


Quickfix to workflows

Why
Quick hack to push package to pypi
What
Commented out if statement
Risk and impact

Minimal

What is the impact if something does go wrong with this PR?

lgtm


Fix main.go generation when executing on a windows machine

When normalizing the importPath it will try to write on main.go files with imports
like github.com/caddyserver/xcaddy/c:\xcaddy
The fix is basically changing from path.Join(currentModule, strings.TrimPrefix(cwd, filepath.ToSlash(moduleDir))) to path.Join(currentModule, filepath.ToSlash(strings.TrimPrefix(cwd, moduleDir)))

I just rebased it

Hey @mholt
Do you mean that this testcase:
args{
	currentModule: "github.com/caddyserver/xcaddy",
	cwd:           "c:\\xcaddy\\subdir",
	moduleDir:     "c:\\xcaddy\\subdir",
}

should be:
github.com/caddyserver/xcaddy/subdir
?

I mean we need to add at least 2 more test cases.

If I understood correctly, something like this?
diff --git a/cmd/xcaddy/main_test.go b/cmd/xcaddy/main_test.go
index af9f0de..23975c4 100644
--- a/cmd/xcaddy/main_test.go
+++ b/cmd/xcaddy/main_test.go
@@ -88,11 +88,21 @@ func TestNormalizeImportPath(t *testing.T) {
 			cwd:           "/xcaddy",
 			moduleDir:     "/xcaddy",
 		}, "github.com/caddyserver/xcaddy"},
+		{"linux-subpath", args{
+			currentModule: "github.com/caddyserver/xcaddy",
+			cwd:           "/xcaddy/subdir",
+			moduleDir:     "/xcaddy",
+		}, "github.com/caddyserver/xcaddy/subdir"},
 		{"windows-path", args{
 			currentModule: "github.com/caddyserver/xcaddy",
 			cwd:           "c:\\xcaddy",
 			moduleDir:     "c:\\xcaddy",
 		}, "github.com/caddyserver/xcaddy"},
+		{"windows-subpath", args{
+			currentModule: "github.com/caddyserver/xcaddy",
+			cwd:           "c:\\xcaddy\\subdir",
+			moduleDir:     "c:\\xcaddy",
+		}, "github.com/caddyserver/xcaddy/subdir"},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {

Yes I think that's right! (My brain's all over the place today but see if that works)

Thank you!
Done

Hmm, and it fails on Linux for windows subpaths.
Should I split up the test for linux and windows and run windows' one only on it?

Well, Windows paths shouldn't come up on Unix at all, so running tests for Windows path on Unix does seem a bit unnecessary.
Or, maybe something like this?
	return path.Join(currentModule, filepath.ToSlash(strings.TrimPrefix(cwd, filepath.ToSlash(moduleDir))))

I guess for that o make work I need to make the cwd slashed as well.
I will try in a few and commit if it works

I guess for that to work I need to make the cwd slashed as well.

Oh, yes, that's probably true!


Set Calendar Colors Manually

Hey is there a way to set the color used for each of my calendars manually?
Currently colors are set by read/write authorization/ownership. I would like to add a line to my irc like
--setcolor 'School Calendar' 'RED'
--setcolor 'Work Calendar' 'BLUE'
etc
I looked at the source and think I could edit things to add that functionality but I wanted to check and make sure that this was not already simple to do. I noticed there is an array that seems to map Calendars to colors using the _CalendarColor command which seemed like the place to start.

I've updated the color handling in master recently.  I don't know if you want to review the code changes to see if there's something you'd like to modify to achieve your goals.
FWIW, I think it works to do something like
$ gcalcli<EMAIL_ADDRESS><EMAIL_ADDRESS>calw

And of course you can set up that kind of stuff in a config file as well.  I don't know if that solves what you're trying to do with your IRC client or not.


Projecting the perspective images to the semantic mesh (Semantic.obj)

Hi,
I am trying to project the available 2D perspective images (using their corresponding poses in folder area_x/data/pose) onto the provided semantic mesh (area_x/3d/semantic_obj) but I can't find the correct camera parameters to do that. The documentation is not clear regarding this issue . I tried different combinations using the so called "camera_rt_matrix", "final_camera_rotation", "camera_original_rotation"... still can't find the correct pose. I am using the mve environment to visualize both mesh and the camera poses. For instance this is an illustration when using the pose provided in camera_rt_matrix

Any help would be much appreciated @ir0
Thanks

Hi, thanks for the question! The correct extrinsic parameters corresponding to the perspective image can be found in matrix form via camera_rt_matrix, or alternatively by using location + Euler angles via final_camera_rotation (XYZ Euler) and the camera_location (XYZ location). These parameters should be the same for all meshes (i.e. identical in semantic.obj, semantic_pretty.obj, and rgb.obj).
I just confirmed the process by doing this manually in Blender 2.79 for the following pair (left: RGB image, right: screen cap of a visualization of the camera in Blender, using pose info from the corresponding JSON file):

area_3/data/rgb/camera_842facf372454735b7fd2880e14de97f_lounge_2_frame_4_domain_rgb.png
area_3/data/pose/camera_842facf372454735b7fd2880e14de97f_lounge_2_frame_4_domain_pose.json

Here are the steps that I used.

Load Area 3's semantic.obj file into Blender using the default setting. Note that Blender loads meshes using the following convention, which may be different from the software that you are using:

Set the camera location to [9.506696, -2.915594, 1.420641] and the Euler angles to [1.8459843397140503,  0.007588082458823919,  1.4350531101226807] (using the XYZ convention)

ALTERNATIVELY, I did step 2 by using the camera_rt_matrix. In this case I set camera.matrix_world =  camera_rt_matrix.invert(). I also needed to decrease the camera's elevation by 180 degrees, and then the the two methods (Euler, RT matrix) were equivalent.
After this I used the screen capture to produce the images shown above. Does this solve your issue?

I'm closing this for now, but please reopen if this did not solve your issue.

I'm closing this for now, but please reopen if this did not solve your issue.

Hello, could you be more specific on the second method which adopts the camera_rt_matrix? This camera_rt_matrix represents the rotation plus translation, right? So you just calculated this matrix inverse. Why you decrease the camera's elevation by 180 degree? Could you be more specific about this method? Really appreciate.


fix: added inherited string keys check on subsetEquality method

Summary
As a result of #13810, this fixes the subsetEquality method to consider also an object's inherited string keys.

@SimenB What do you think a good regression test could be in this case?

@joshkel thoughts?

Hi @joshkel ✋
Following your recent fix (#13810), I pushed this PR that will use the getObjectKeys function in subsetEquality.
I need to add a regression test as well.
Can you give me an example where getObjectKeys produces a different output than Reflect.ownKeys?


Add Excel-specific metadata

SheetCount
SheetNames (list of strings in page order)

This PR is targeted at the develop branch, which was created from the 1.22 tag. Upstream's master is quite far ahead of 1.22, so we need to decide at some point whether we want to update to that, but currently I'm just taking 1.22 because that's the released version that we were using previously.


Naming for API calls/interface parts

I am starting this because of @glassnotes's comment on #3 that we could call the QRAM type that a user can call to access the ram a 'LooqUpTable / LooqUpBit, or just the general qROM.'

Also moving some notes from the source files here:
 BB has a register of hardware qubits - need to know which of those are
 set to 1; easiest way to do that is to send a string like "0011" to indicate that
 this register is in the state

 |0> ------   (qubit 0, address 00)
 |0> ------   (qubit 1, address 01)
 |1> ------   (qubit 2, address 10)
 |1> ------   (qubit 3, address 11)

 This is a 2-bit address; so QRAM(00) = 0, QRAM(01) = 0, QRAM(10) = 1, QRAM(11) = 1

The black box has to perform this operation ^^ given the inputs, based on the contents
 of the hardware register it starts with

This has come a long way, we will eventually add an explicit API document in the docs for the library to address this.


[Packager] Fails when package doesn't contain pure javascript

The packager fails when the required package does not contain an javascript entry point with an exported module.exports. It looks like the feature is missing / other packagers support it.
Example: dfcreative/color-name is a module that only provides a json file.
Error: "cannot find package main file for package"

What version of React Native are you using?

0.5

This is fixed in master as we implement the full node resolution algorithm.


GEOMESA-1747 Arrow geometry vectors, Arrow-based data store

Signed-off-by: Emilio Lahr-Vivaz<EMAIL_ADDRESS>

uses a snapshot build of arrow - will update to 0.3 once it's released and available in maven central.

Looks great.  I'm happy to merge now or wait until a few more bits of functionality.  Additional things we need: command line tools, WFS mime/type, iterator push-down. What do you think?

I'd say merge this so the PR doesn't become too huge. I've got the iterator push-down queued up, but I was waiting for this PR first.

It is going to break master though, since it requires an arrow snapshot

they're looking at a release in the next 2 weeks possibly. Given that, I'll just keep stacking PRs for now I guess

replacing with #1457


Add support for X-TOKEN header for gRPC connection

Services such as QuickNode support an x-token header for grpc authentication. It would be useful to use a QuickNode specific endpoint for the EVM gateway instead of the public endpoint.

Have you tried using the --address-header  flag for EVM GW startup? eg:
--address-header=X-TOKEN \
https://github.com/onflow/flow-evm-gateway/blob/6ccaf586f0e5f9c01b5fbf5159fce53e1e8443d5/README.md?plain=1#L200

--address-header is something different. If I were to use grpcurl, it would look like this:
./grpcurl \
  -H "x-token: <token>" \
...

something like this needs to be added to https://github.com/onflow/flow-evm-gateway/blob/d003e8ab9938fe755c855b4879b0972b7ec67e5b/bootstrap/bootstrap.go#L28:
	if *token != "" {
		md := metadata.New(map[string]string{"x-token": *token})
		ctx = metadata.NewOutgoingContext(ctx, md)
	}

It would be useful to use a QuickNode specific endpoint for the EVM gateway instead of the public endpoint.

Can you elaborate on this? What is the desired outcome/usage of this?

We want to be able to set an auth token on our QuickNode endpoint so that we can use a configuration like this:
--access-node-grpc-host=my-quicknode-endpoint.flow-testnet.quiknode.pro:9000
--access-node-grpc-token=2ff605389410fd74640be87d73b4a62782e0eed5


Doesn't Work In Gulp 4

I am currently trying to get gulp-connect-php running on a gulp4.0 version. I have tried with my existing gulp file and with a standalone project, Neither to much joy.
[10:07:09] Using gulpfile C:\development\gulp4\gulpfile.js
[10:07:09] Starting 'default'...
[10:07:09] Starting 'connect-sync'...
[10:07:09] The following tasks did not complete: default, connect-sync
[10:07:09] Did you forget to signal async completion?

This tends to be the issue that appears with the following gulpfile.js:
var gulp = require('gulp'),
    connect = require('gulp-connect-php');

gulp.task('connect-sync', function() {
  connect.server({});
});

gulp.task('default', gulp.series('connect-sync'));

Is there any possibility of getting this working with the new gulp4.0?

@KITSDominicWhite so the official line would be "Gulp4 has not been released in a final form", but I hate it when people respond like that... so I'll do the best I can to assist.
Gulp 4 has a bunch of breaking differences from Gulp 3 one of which is that tasks are supposed to return a promise or implement the 'done parameter callback pattern'. Failure to do so will cause the task to timeout with that warning if it is being held open by anything it invoked internally.

@KITSDominicWhite any luck?

Sorry @grmartin I went on holiday for a few days. I have given the first one a try with a few variants of my own, and unfortunately I haven't been able to get it working.
I am going to dig a little deeper into it and see if I can get some more information / figure it out :)

@KITSDominicWhite No worries! If i can assist or if you get it working, please let me know.

+1

I have gulp-connect-php + browserSync working perfectly fine using Gulp 4.0.0 on Windows 10.
Does this issue only occur on Linux / macOS?

Worked for me on Windows, after putting the php executable on the PATH. It just gave me a hard time, as the error message when PHP is not installed was quite hard to understand. No issues on Gulp 4.0 though.

it stopped working after requiring database connection (db.php)


updated_at field for Documents

We find that a good amount of users like to replace existing documents files with new versions. This allows them to keep the references and links from any page to the document. Bad news is that the model only keeps track of the first upload date created_at, causing a bit of confusion as they like to use the documents interface to keep track of the new uploaded versions.
It would be nice and useful to add a new datetime field to solve this.

Yeah, I can see revisions of binary assets being a very useful feature for some of our users.

This was touched on in #1420, although we'll probably use the SHA1 hash / Etag route for that...
To clarify, would you want this timestamp to update on any edit to the document record (e.g. editing the title), or only when uploading a new file? Or would this be a user-facing field for them to fill in themselves?
Also, would this replace the current 'uploaded' column in the documents listing, or be in addition to it?

In a broader sense this was touched on in #496 to, created because I thought that sites with a more complex structure, or a bigger body of contributors would require a better paper trail for alterations to each of the objects tracked by wagtail.

@gasman Just to clarify, we were more thinking of a timestamp that gets updated each time any field of the Document (file and/or title) is edited. From what we've learnt from our users, most of the time it's just the file is being updated. ie: a pdf monthly review gets updated each month, title stays the same Monthly review but file is replaced. We understand that other case scenarios are possible though.
The new datetime field wouldn't be editable but only visible, complementing or replacing the uploaded (potentially created) column in the admin interface.
However I also like where @davecranwell is going with the generic logging system. I agree with the images thing but imho I consider the documents a bit more critical as the user can't see the contents of the doc at first sight. They need to download and open the file to see what's in it. With images the user can see how they do look like, knowing what version is in the CMS at first sight.

With documents now being easily customised through WAGTAILDOCS_DOCUMENT_MODEL this can be easily achieved.
@davecranwell I'd like to know about the history feature you were mentioning above. Has some work been done on this? Cheers

A comprehensive logging system hasn't become any more real than the comments on these Issues and it's not on the roadmap yet either. If you're interested in taking it on, an RFC would certainly be welcomed.

Adding a +1 to this issue. Today I worked with a new Wagtail user who was very confused that the "Uploaded" column in the documents list view doesn't get updated when a document gets replaced with a new version. Their use case is very similar to what @jordij describes above; replacing a document with new versions periodically.
Labeling the created_at column "Uploaded" is thus a bit misleading. As an immediate UX fix short of any code changes, could this be changed to something like "First Uploaded" or maybe "Created"?

Closing in favour of #5658


R: seed parameter doesn't work, except set.seed()

The seed parameter in xgboost doesn't work when I set subsample or colsample_bytree,  for example
data(agaricus.train, package='xgboost')
data(agaricus.test, package='xgboost')

dtrain <- xgb.DMatrix(agaricus.train$data, label = agaricus.train$label)
dtest <- xgb.DMatrix(agaricus.test$data, label = agaricus.test$label)
watchlist <- list(train = dtrain, eval = dtest)

param <- list(max_depth = 2, eta = 1, silent = 1, nthread = 2, subsample = 0.5, colsample_bytree = 0.1,
  objective = "binary:logistic", eval_metric = "auc")
bst <- xgb.train(param, dtrain, nrounds = 5, watchlist, seed = 2018)

the results of eval-auc are different when you run the example, only when you set.seed(2018) in advance like this

data(agaricus.train, package='xgboost')
data(agaricus.test, package='xgboost')

dtrain <- xgb.DMatrix(agaricus.train$data, label = agaricus.train$label)
dtest <- xgb.DMatrix(agaricus.test$data, label = agaricus.test$label)
watchlist <- list(train = dtrain, eval = dtest)
param <- list(max_depth = 2, eta = 1, silent = 1, nthread = 2, subsample = 0.5, colsample_bytree = 0.1,
  objective = "binary:logistic", eval_metric = "auc")

set.seed(2018)
bst <- xgb.train(param, dtrain, nrounds = 5, watchlist, seed = 2018)

R sessionInfo
R version 3.4.4 (2018-03-15)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Ubuntu 16.04.4 LTS

Matrix products: default
BLAS: /usr/lib/openblas-base/libblas.so.3
LAPACK: /usr/lib/libopenblasp-r0.2.18.so

locale:
 [1] LC_CTYPE=en_US.UTF-8       LC_NUMERIC=C               LC_TIME=en_US.UTF-8
 [4] LC_COLLATE=en_US.UTF-8     LC_MONETARY=en_US.UTF-8    LC_MESSAGES=en_US.UTF-8
 [7] LC_PAPER=en_US.UTF-8       LC_NAME=C                  LC_ADDRESS=C
[10] LC_TELEPHONE=C             LC_MEASUREMENT=en_US.UTF-8 LC_IDENTIFICATION=C

attached base packages:
[1] parallel  stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] DBI_1.0.0         alpha_0.1         data.table_1.11.4

loaded via a namespace (and not attached):
 [1] TM_0.1.0            zoo_1.8-1           compiler_3.4.4      tools_3.4.4
 [5] yaml_2.1.19         Rcpp_0.12.17        grid_3.4.4          TA_0.1.0
 [9] RMySQL_0.10.15      TD_1.0              RhpcBLASctl_0.18-12 fst_0.8.2
[13] lattice_0.20-35

seed is not available for xgboost or xgb.train.
Yet it is confusing to see it appears in https://github.com/dmlc/xgboost/blob/master/doc/parameter.md.
The reason I believe is that the mechanism of the seed in params is different from R's set.seed, and the latter one is the standard way to go in R.

@hetong007 thx


heyy

this is a pull re

This has the search page fixes I was looking for!  When will this be merged to main?


'alert' type in config.yaml violated the latest schema

Hello there,
Comparing the master and v0.0.63, I can find there is a change at this line
https://github.com/Yelp/elastalert/blob/master/elastalert/schema.yaml#L156
This caused elastalert can't run test or start from command line with complaining alert types is not an arrayofObject. Given using the same config.yaml in v0.0.63, such as the example below:
  alert:
  - "email"

So I had another look between master and v0.0.63, and found in the master schema.yaml, the definition of this arrayOfObjects is identical to arrayOfStrings which might be where the problem comes from.
If so, please also update the rule example with this fixed arraOfObjects type in it.
Many Thanks
James

Ah yes this was broken in master by a recent merge. I've fixed it in master for now. I'm thinking of removing the JSON schema entirely soon as it's becoming unwieldy and not very useful.

Cheers Qmando


Import the Bitcoin.com wallet BCH seed words into telescope

This way users can just use one address for managing all their funds

(BIP39; derivation path 44/0/0/0)


Adding/removing relationships in the submission form does not update the view automatically

References

Fixes DSpace/dspace-angular#1671
Related to DSpace/dspace-angular#2252

Description
When the backend returns the submission form after an update, it will not only return all the current values but also create the relation.* metadata fields for all the relation fields (who don't have a <linked-metadata-field>) because otherwise when they don have an explicit field linked to them (using <linked-metadata-field>) the data is not sent back to the frontend
Instructions for Reviewers
By default the when the form is loaded for the first time the item metadata is used to populate the form data. This includes virtual metadata (so relationships).
When the form is updated, the submission form object is retrieved, and the metadata is added by the DescribeStep. This did not include virtual metadata. I’ve added it for the case when there is no linked metadata field to a relationship field.
The easiest way to test this PR is to use the linked angular PR and follow the steps on how to test it.
Checklist

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & integration tests). Exceptions may be made if previously agreed upon.
[x] My PR passes Checkstyle validation based on the Code Style Guide.
[x] My PR includes Javadoc for all new (or modified) public methods and classes. It also includes Javadoc for large or complex private methods.
[x] My PR passes all tests and includes new/updated Unit or Integration Tests based on the Code Testing Guide.
[x] If my PR includes new libraries/dependencies (in any pom.xml), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[x] If my PR modifies REST API endpoints, I've opened a separate REST Contract PR related to this change.
[x] If my PR includes new configurations, I've provided basic technical documentation in the PR itself.
[x] If my PR fixes an issue ticket, I've linked them together.

While testing this PR I also found out that the "read-only" for the <scope> tag is not working like it should. When "read-only" is specified you can still edit the fields in he UI and even save them

@alexandrevryghem @tdonohue
I've verified the problem with read-only fields exists with every input type. However we've already fixed on DSpace-CRIS and we can port the fix quickly

@tdonohue
we can open it

@tdonohue  @alexandrevryghem
here the PR for the read-only issue
https://github.com/DSpace/dspace-angular/pull/2297


Compute environment is only configured for default user

centos user (Centos 7) does not have permission to administer the cluster, no problem.

However, the root account is not configured at all.

This problem was previously closed here: https://github.com/aws/aws-parallelcluster/issues/1159
but I would like to suggest a solution.
I noticed that the environment configuration is done in /home/centos/.bashrc
If this configuration was moved to /etc/bashrc then there would be no additional steps necessary to administer the cluster. Additionally, this could make it possible to add multiple user accounts to the cluster (/home/ would need to be shared).

@RickoLacharity /home is already shared via NFS. This is an excellent feature request.

Closing this because scheduler profiles are in /etc/profile.d


Moving WebView2 to another window still routes window message to previous window

Describe the bug
There is a scenario where I need to stack two WebView2 controls on top of each other, and then when user clicks a button, one WebView2 pops up as a new window. Originally I asked the question here
I tried removing the WebView2 from a Grid in the old window and make it as a child to the Grid in the new window, and it seems to be working, except that when interacting with the new window, the old window gets activated and bought to the front of the new window.
My method might be wrong, but doing the same thing with other controls such as a ToggleButton gives expected behavior, and does not activate the previous window.
Steps to reproduce the bug

In MainWindow.xaml

<Window
    ...
    <Grid x:Name="RootGrid">
        <Grid.RowDefinitions>
            <RowDefinition Height="*"/>
            <RowDefinition Height="*"/>
        </Grid.RowDefinitions>

        <WebView2 x:Name="Browser" Source="https://www.google.com"/>
        <Button Click="Button_Click" Grid.Row="1">Move</Button>
    </Grid>

Create another window, call it BlankWindow

<Window
    ...
    <Grid x:Name="RootGrid" Background="Red"/>

In the button handler code,

void winrt::WinUI3Cpp2::implementation::MainWindow::Button_Click(winrt::Windows::Foundation::IInspectable const& sender, winrt::Microsoft::UI::Xaml::RoutedEventArgs const& e)
{
    winrt::WinUI3Cpp2::BlankWindow window;

    window.Activated([this, window](auto...)
        {
            static bool called = false;
            if (called)
                return;

            window.RootGrid().Loaded(
                [this, window](auto...)
                {
                    auto webview = Browser();
                    RootGrid().Children().RemoveAt(0);
                    window.RootGrid().Children().Append(webview);
                }
            );
            called = true;
        });
    window.Activate();
}

Expected behavior
The previous window should not get activated.
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.5 Experimental 1: 1.5.231202003-experimental1
Windows version
Windows 11 (22H2): Build 22621
Additional context
No response

Duplicate of #7335


Play a video a specified number of times

Hello --
Is there a way to play a video a specified number of times? i.e. 3 times and then stop?
If so, is there a way to provide a sample of how?
Much appreciated!
JF

You could use the onEnded callback prop to seekTo(0) and increment a count stored in local state. Then when the count gets to a certain number, just don’t call seekTo.

Why not build support for this ?
On Mon, May 2, 2022 at 5:52 PM Pete Cook @.***> wrote:

You could use the onEnded callback prop to seekTo(0) and increment a
count stored in local state. Then when the count gets to a certain number,
just don’t call seekTo.
—
Reply to this email directly, view it on GitHub
https://github.com/cookpete/react-player/issues/1446#issuecomment-1115407137,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWQAZBSDQOTJ6OORL7JJVBLVIBFBJANCNFSM5UTER64A
.
You are receiving this because you authored the thread.Message ID:
@.***>


Resource-derived type parameter doesn't respect being nullable

Bicep version
v0.25.53
Describe the bug
When you create an optional parameter with a default value, you'd expect that you don't have to pass it, Now you get an exception saying:  The specified "object" declaration is missing the following required properties: "myOptionalResourceDerivedTypeParameter".bicep(BCP035)
To Reproduce
Create a bicep module with a resource-derived type as an optional parameter. Use this module somewhere else and the bicep turns out to be invalid.
Additional context
Generated ARM snippet:
   "siteProperties": {
     "type": "object",
     "metadata": {
       "__bicep_resource_derived_type!": "Microsoft.Web/sites@2022-09-01#properties/properties"
     },
     "nullable": true
   },

VS Code shows (required):

Just want to clarify: this only happens when you load the module from JSON (either via a registry or just by compiling and pointing to the .json file), correct?

"This only happens when you load the module from JSON (either via a registry or just by compiling and pointing to the .json file), correct?" -> Yes.
Via bicep it works like expected. Via JSON it doesn't respect the optional parameter.


Assest not working with unity.

Hello,
I did not know where to post this. But I haven't been able to run a scene in unity using the leap motion. I followed the instructions from the official website and I also followed the instructions from a YouTube video, not successful. Anyone has any clue what I may be doing wrong?
I've tried different assets versions, but none is working.

Leap_Motion_CoreAssets_4.2.1
Leap_Motion_Core_Assets_4.3.4
Leap_Motion_CoreAssets_4.40

I also tried with Leapmotion V2 and the Orion.
Thanks.

Can you please provide more information on what specifically is not working?
e.g.

Have you tried the diagnostic visualizer. If you did, are hands showing up in the view?
Are you getting any errors in Unity console?
What version of Unity are you using?

HI! I am having the same problem! My hands are showing up in the diagnostic visualizer. I am getting this error "warning CS0618: 'PrefabType' is obsolete: 'PrefabType no longer tells everything about Prefab instance.'. I am using Unity Hub 2.0.3

I ended using another laptop. I Could not figure it out what was the problem. The one that gave me problems had the next specs:
Intel i7-6500U CPU @ 2.50GHz
64-bit Windows 10
I do not remember the problem I was dealing with using Linux. But It may be my computer. Once I switched everything worked like a charm at the first try.

Closing since the primary issue is resolved. I'm not sure what the issue was but I'd need more details to figure out what went wrong. If someone else is having an issue, please file a fresh ticket and we can dive in there.


Added 2.3, 5.11, 5.12.4, 5.32 test cases

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


TagHelper, RestrictChildren vs @if statement

Version: rc1-final
My taghelper definition is
[HtmlTargetElement("panel")]
[RestrictChildren("panel-heading", "panel-body", "panel-footer")]
public class PanelTagHelper : TagHelper
{
	...
}

The problem is that I'd like to show the footer depends on something in the model:
<panel>
   <panel-body>
   ...
   </panel-body>
   @if(Model.NeedsFooter)
   {
   <panel-footer>
      ...
   </panel-footer>
   }
</panel>

The error message:

The parent  tag helper does not allow non-tag content. Only child tag helper(s) targeting tag name(s) 'panel-heading, panel-body, panel-footer' are allowed.

Please let us to use code block inside a taghelper (with restricted childrens).
Thanks,
Istvan

Sadly this is currently a restriction of the RestrictChildren attribute. However, I agree that we should definitely allow code when a parent has restricted children.

Thanks Taylor. Is there any workaround?

You could bake the visibility functionality into the panel-footer tag helper or even build another VisibilityTagHelper.
That way it'd look something like:
<panel>
  <panel-body>
  ...
  </panel-body>

  <panel-footer visible="Model.NeedsFooter">
  ...
  </panel-footer>
</panel>

Thanks again but this solution is not too nice for me. The panel(-xxx) taghelpers are renders a bootstrap panel used around the whole project (and company). I don't want to create dummy attributes as workaround.
I commented out the restrictions and I hope I'll able to use code in the final version.

@DamianEdwards / @NTaylorMullen how critical is this behavior? Backlog?

Yeah this seems overly restrictive. @NTaylorMullen any idea how much work to make this supported?

@DamianEdwards @Eilon I took a quick look at the class responsible for restricting this behavior and it doesn't look like it'd require too much effort. Biggest time sink will be creating new tests for it.
Not too much effort equates to 2-3 days.

OK that's a little high for my comfort at this stage given the amount of other work we have remaining. Put it on the backlog.

Backlog it is.


Main

Overall Review of Changes:
A general description of the changes made that are being requested for merge
Issue Fixes:
Please list (using linking) any open issues this PR addresses
Enhancements:
Please list any enhancements/features that are not open issue tickets
How has this been tested?:
Please give an overview of how these changes were tested. If they were not please use N/A

hi @gbspop2
Ive just noticed the PR, this doesn't appear quite right to me.
thanks
uk-bolly

hi @gbspop2
I am closing as believed it is raised in error.
thanks
uk-bolly


Add an API to query for status of moveShard and cleanOutServer jobs.

This is so far intentionally undocumented, since we want to collect
experience with it first.

Tested works. LGTM.


VCS now reports the same signals as the other big3

Tested with VCS M-2017.03-1.

To be clear, this is unrelated to #884, right?
Not really sure there's any way of reviewing this other than taking your word for it - which seems fine given that this is only a test change anyway.

This is unrelated to #884, I only saw the issue when I tried the other PR.


Vue 3 compatibility

Hello Yoann,
We are using your library at Crisp, as we are migrating to Vue 3, so made the compatibility for vue-chartist :)

To anyone wanting to use this in a Vue 3 project you can temporary use:
"vue-chartist": "git+https://github.com/crisp-dev/vue-chartist.git#79e9559b94db493a522fbe675895e6934d4517bc"
(deployed on a branch)

Hello Baptiste,
Oh great !
Regrettably, I no longer have time to manage this repository :disappointed:
Do you want a manage access to this repository ? :pray:

We can handle it!

What is the current state of this??
Is vue-chartist compatible with Vue 3?

I published new version #58


New Design required for simulator

Is your feature request related to a problem? Please describe.
The current design of the simulator does not have the best look and feel. And includes a lot of third-party dependencies on the UI (which are unnecessary).
The current UI:

Describe the solution you'd like
A completely new design. This does not include the coding aspect, a few designs as to how the simulator UI should be.
Additional context
The design need not be too complicated and with rich features for starting. A basic design is required so that we can move the front end from Django to a much reliable front-end component.
Although there are certain dependencies that cannot be removed:

Chart.js, the graph will be generated by chart.js itself

Preferably keeping the charts as it is. (The representation of data using those specific charts)

Yes, thanks for reminding me, the graph will be generated by chart.js, so the design for the graph can be as it is. If their needs to be a change, it must adhrere to the chart.js framework.