Advice for dealing with DX12 TDRs

Hello, i'm seeking for some advice dealing with DirectX12 device removals more gracefully.
This error occurs an one specific machine with my program, while it runs fine on a few other machines.
D3D12 ERROR: ID3D12Device::RemoveDevice:
Device removal has been triggered for the following reason (DXGI_ERROR_DEVICE_HUNG:
The Device took an unreasonable amount of time to execute its commands, or the hardware
crashed/hung. As a result, the TDR (Timeout Detection and Recovery) mechanism has been
triggered. The current Device Context was executing commands when the hang occurred.
The application may want to respawn and fallback to less aggressive use of the display
hardware). [ EXECUTION ERROR #232: DEVICE_REMOVAL_PROCESS_AT_FAULT]

I think this is a display driver issue(didn't update the driver to confirm yet because i want to dig more into this). I'm curious though whether there's way to capture this situation and prompt the user gracefully. Some Diligent Engine APIs start failing after the device removal but it's a little hard to recognize what's happening under the hood from the running program...
Could you give some suggestions? Thanks!

After device removal, all DX functions that return error codes will return DXGI_ERROR_DEVICE_REMOVED. However, like I said, many functions have no return values and silently fail. Usually if device gets removed, the program doesn't get to Present, while checking if the device has been removed after every single call is going to introduce extremely high overhead.
I guess it is better to figure out why the device has been removed and fix the problem in the first place rather than trying to handle the aftermath.

Fair enough, thanks!


Webview not passing header with Android 9

🐛 Bug Report
Webview not passing header with Android 9. it is working with lower android versions.
When I run, webview's header passes to webpage. It would everything working fine with android verison < 9. but not in android version 9 (Pie).
I wanna pass headers from android version 9 to webpage also.
<WebView
ref={REF => (this.WebViewRef = REF)}
bounces
source={{ uri: ${baseUrl.domain}${url}, headers: { 'is_app': 'true' }}}
onLoadStart={navState => this.onLoadStart(navState)}
onLoadEnd={navState => this.onLoadEnd(navState)}
onError={this.onLoadEnd}
key={this.state.key}
renderLoading={() => (
<View style={{ flex: 1, justifyContent: 'center', alignItems: 'center'}}>

)}
startInLoadingState
mixedContentMode="always"
thirdPartyCookiesEnabled
/>
Environment:
OS: Linux 4.15
Node: 8.11.4
Yarn: 1.12.3
npm: 5.6.0
Watchman: Not Found
Xcode: N/A
Android Studio: 3.3 AI-182.5<IP_ADDRESS>99772
Packages: (wanted => installed)
react: 16.3.1 => 16.3.1
react-native: 0.55.4 => 0.55.4

Ohh! yaa I updated my bug.

Do you mind opening the issue over at https://github.com/react-native-community/react-native-webview? The WebView component was forked to that repository, and has been deprecated in the core RN repository.

Now I update my react-native version but still having the same issue.
Environment:
OS: Linux 4.15
Node: 8.11.4
Yarn: 1.12.3
npm: 5.6.0
Watchman: Not Found
Xcode: N/A
Android Studio: 3.3 AI-182.5<IP_ADDRESS>99772
Packages: (wanted => installed)
react: 16.8 => 16.3.1
react-native: ^0.58.4 => 0.55.4

@hramos Actually I used https://github.com/react-native-community/react-native-webview but getting the same issue with it.

That's quite likely. You can report the issue over there. We'll remove WebView from the facebook/react-native repository soon.

@hramos Actually I want a solution regarding this bug does not want to remove Webview from facebook/react-native repository otherwise it would be more problematic.

See the Lean Core issue pinned to the top of the issues page. WebView was the first component to be forked and transferred to the RN community. All you need to do is import the new package and you will get access to the very same component that is available in core.
You may choose to continue using the component from core, but it’s not getting any new updates and will be removed in the next release or two.
Sent with GitHawk

@hramos  Thanks for answer! Actually I have solved my issue with another way now I am using query parameter except of header in Webview.


Add link button

Create link button
Used like flare.Row(button, flare.LinkButton("google.com", label="Your mom")

Question: Why doesn't a Component have a custom_id or cookie? Seems like those should be inherited from there even if it may not have params. Also probably good for future-proofing since these could be used for modal components like TextInputs too.

There is no cookie since link button components don't have one. I don't think modal components would have one either.
I didn't give it a custom id because I think it would get messy with link buttons wanting a url and everything else wanting a custom id. I can change it so that Link button has that attribute and it's the same as url thought.

Wondering if we need traits

Wondering if we need traits: Component, SupportsCallback, SupportsCookie

Yeah this might be a way. Component is the base cls, then we implement seperate traits on the ones that actually have a callback or cookie.

Wondering if we need traits: Component, SupportsCallback, SupportsCookie

Done


Git the error message

I install this Extension. And use the Agent pool is azure pipeline. I don't finish to run it. I got these error message.
BinSkim completed with exit code 1
##[error]Error running binskim job: 1 of 1
##[error]GuardianErrorExitCodeException: binskim completed with an Error exit code: 1. BinSkim failed. Verify the target(s) to be scanned. BinSkim targets must be a specific filename, or a pattern with a wildcard like *.dll, dir*.dll, or dir*
------------------------------------------------------------------------------
##[error]Error running tool 1 of 4: binskim
##[error]Error running binskim job: 1 of 1
##[error]GuardianErrorExitCodeException: binskim completed with an Error exit code: 1. BinSkim failed. Verify the target(s) to be scanned. BinSkim targets must be a specific filename, or a pattern with a wildcard like *.dll, dir*.dll, or dir*
##[error]MSDO CLI exited with an error exit code: 1

@edwardkuo
when you use the task in your pipeline it is best practice to provide categories input. Categories are nothing but the analyzers you want to run when the task is executed in pipeline.
The categories available are: secrets, code, artifacts, IaC, containers.
the categories input defaults to all & means it will run all analyzers available in the extension.
suggest you to understand what analyzer scans you need in your code base and mention them explicitly like below:
categories:secrets,code

the above input will let the extension execute secrets scan & code scan.
try it and let us know if this works fine.

The guidance provided by @lohithgn covers this, closing the issue. @edwardkuo if you continue to have problems with your pipeline, feel free to re-open or create a new issue.

Got, I will try it. thanks


Fix Bug 650: Remove reading warnings, add additional error handlers

Fix #650

Codecov Report

Merging #651 (514abf4) into develop (a57e65f) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 514abf4 differs from pull request most recent head 15badcc. Consider uploading reports for the commit 15badcc to get more accurate results

@@           Coverage Diff            @@
##           develop     #651   +/-   ##
========================================
  Coverage    64.58%   64.58%
========================================
  Files           43       43
  Lines         1423     1423
  Branches       240      240
========================================
  Hits           919      919
  Misses         504      504

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 afc4baf...15badcc. Read the comment docs.


[release/1.3] Fix repeated sending signal

Cherry pick of containerd/containerd#5021
Since our production environment still uses 1.3, we hope to add this patch

/ok-to-test

https://github.com/containerd/cri/pull/1626#issuecomment-785542433
I'm not sure if this is a bug, do you think it is necessary to cherry-pick it.


All connection settings lost when updating via Chocolatey

All connection settings are lost after running an update via Chocolatey. Have to recreate them all

One way to avoid this from happening is to set it to use the "User configuration file". Then the connection strings are stored in your user profile.

Thank you


GLES2 Android "Directional shadow framebuffer status invalid" on Google Nexus 7 / NVIDIA Tegra 3

Godot version:
caa42667e (includes recent 01a3dd3)
OS/device including version:
Google Nexus 7 (2012) with NVIDIA Tegra 3 (official 5.1.1 "nakasi" image)
Issue description:
Followup issue to #24952
Godot splash is ok, black screen afterwards due to invalid directional shadow framebuffer (see relevant logcat output below)
D/libEGL  (20580): loaded /system/lib/egl/libEGL_tegra.so
D/libEGL  (20580): loaded /system/lib/egl/libGLESv1_CM_tegra.so
D/libEGL  (20580): loaded /system/lib/egl/libGLESv2_tegra.so
I/OpenGLRenderer(20580): Initialized EGL, version 1.4
D/OpenGLRenderer(20580): Enabling debug mode 0
V/WindowManager(  510): Adding window Window{2c2d725f u0 SurfaceView} at 4 of 12 (before Window{31e53080 u0 com.example.gles2/org.godotengine.godot.Godot})
W/GodotView(20580): Trying ConfigChooser fallback
W/GodotView(20580): creating OpenGL ES 2.0 context :
I/godot   (20580): OpenGL ES 2.0 Renderer: NVIDIA Tegra 3
E/godot   (20580): **ERROR**: Directional shadow framebuffer status invalid
E/godot   (20580):    At: drivers\gles2\rasterizer_scene_gles2.cpp:3332:initialize() - Directional shadow framebuffer status invalid
I/ActivityManager(  510): Displayed com.example.gles2/org.godotengine.godot.Godot: +1s803ms
E/godot   (20580): **WARNING**: Could not create framebuffer!!
E/godot   (20580):    At: drivers\gles2\rasterizer_storage_gles2.cpp:4335:_render_target_allocate() - Could not create framebuffer!!

Steps to reproduce:
Open sample project and run on tablet
Minimal reproduction project:
Same as in #24952: GLES2.zip

@usbhell Could you check whether your Google Nexus still shows the same issue based on current master?

Still a problem, audio is playing in the background but screen is black.
I find this line of code returns null which is not correct, but perhaps unsurprising since the screen is black.
var img = get_viewport().get_texture().get_data()

btw I'm getting that "Directional shadow framebuffer status invalid error" in the console on my mac as well

Can you guys download this (or a similar one)
https://play.google.com/store/apps/details?id=com.realtechvr.glview&hl=en_US
And paste here the list of extensions supported by Tegra3? Evidently something is needed for 32 bit depth buffers in there.

I see this is closed, heres the info anyway.
bluetooth_content_share.txt

It still does not work, but it definitely sets config.depth_internalformat = GL_DEPTH_COMPONENT16; config.depth_type = GL_UNSIGNED_SHORT; in rasterizer_storage_gles2.cpp:4993 now.
The texture size seems to be irrelevant, I also tried with size 512 and 1024 (project settings).
Btw, the storage->config.use_rgba_3d_shadows condition evaluates to false in rasterizer_scene_gles2.cpp:3317, so this cannot not be the cause.
I tried to figure out the difference compared to godot 2.1, where rendering worked, but I did not find the relevant parts yet...

Btw, the storage->config.use_rgba_3d_shadows condition evaluates to false in rasterizer_scene_gles2.cpp:3317, so this cannot not be the cause.

That's actually a good hint, as it means you don't support GL_OES_depth_texture (and judging by the logs, neither does @usbhell), so all this logic using GL_DEPTH_COMPONENT in glTexImage2D cannot work.
glTexImage2D in OpenGL ES 2.0 doesn't support GL_DEPTH_COMPONENT format initially: https://www.khronos.org/registry/OpenGL-Refpages/es2.0/xhtml/glTexImage2D.xml (it does in OpenGL ES 3.0: https://www.khronos.org/registry/OpenGL-Refpages/es3.0/html/glTexImage2D.xhtml)
GL_OES_depth_texture adds support for it in OpenGL ES 2.0: https://www.khronos.org/registry/OpenGL/extensions/OES/OES_depth_texture.txt
So for your hardware, @reduz will have to come up with an alternative.
Finally the check added in 057d3e8 for GL_OES_depth24 is wrong (and broke rendering for me, reopening #24952), as this extension is only for storage, not for glTexImage2D: https://www.khronos.org/registry/OpenGL/extensions/OES/OES_depth24.txt
@reduz will fix it later today or tomorrow.

I am certain it does fix the issue, so closing unless it' s verified it happens again.

Great! It works! 👍 Thx a lot!

All working nicely here too, thanks.


Checking that a value is undefined and/or using the not operator does not translate to correct ASL

Example:

I would expect connections.Items to return false and not follow the return branch when connections.Items contains a valid array. However, when connections.Items is set, the step function follows the true branch.
Input to Step:

Generated ASL:
    "if(connections.Items == undefined)": {
      "Type": "Choice",
      "Choices": [
        {
          "And": [
            {
              "IsPresent": true,
              "Variable": "$.connections.Items"
            },
            {
              "IsNull": false,
              "Variable": "$$.Execution.Id"
            }
          ],
          "Next": "return"
        }
      ],
      "Default": "for(item of connections.Items)"
    },

This does work if I use if (connections.Items == null) as a workaround.
It does not work when I use if (!connections.Items).
I would expect all 3 to behave the same based on the behavior in javascript.

Interesting, didn't realize that == and === behave differently here.
const items = null
items === undefined //false
items === null // true
!items // true
items == undefined // true
items == null // true

const items2 = undefined
items === undefined // true
items === null // false
!items // true
items == undefined // true
items == null // true

Think I see the issue, will fix.

I also tried with === undefined which didn't work either. I switched to the double equals syntax after looking at the functionless docs. JavaScript isn't exactly the best role model when it comes to value comparison lol

it isn't...
Thoughts on what to do with == between number and string?
https://262.ecma-international.org/5.1/#sec-11.9.3

return the result of the comparison ToNumber(x) == y

Step Functions cannot convert a string to a number.

:grimacing:
I thought there was a way to do that roughly with intrinsic functions e.g. States.JsonToString?
Maybe its too disgusting but you could go the other way to get roughly the same thing:
States.JsonToString(num_value) == string_value
Otherwise I would say making the comparison illegal via your language service since it could lead to non-intuitive outcomes.

StringToJson will convert to number provided there are no quotes right? A digit is a valid json expression

Yeah that should work. I don't know why I forgot about the reverse operation lol

Good idea. We can use that to implement Number().
== will be more complex without synth time types. The algorithm depends on Type() in JS. We'll need to inject some nodes to create the correct logic.

Good idea. We can use that to implement Number().
== will be more complex without synth time types. The algorithm depends on Type() in JS. We'll need to inject some nodes to create the correct logic.

Need a Choice state with IsNumeric and unfortunate added transition cost.

isNumeric, isBoolean, isNull, isNotPresent, and the fallback to isObject.
One option is to use the typeof logic to output the types of each side and then decide how to compare.
Probably some optimization here as well.

I'm a bit naive on what is feasible to do with the language server, but I'm wondering if it might be better to limit surface area and not support https://262.ecma-international.org/5.1/#sec-11.9.3 explicitly for the time-being.
As far as I'm aware, the triple syntax is best practice anyways. If the language server could detect usage of ==, !=, etc. in a StepFunction context and relay a helpful error message, that might be a better solution.

We could emit an error or warning when == is used. Was thinking about that too. No matter what, the == will be less efficient and its unlikely someone should need it's weird logic if we expose Number() and String() (already have Boolean()) #446

@tvanhens you can try === undefined now once 0.21.19 releases.
The other issues have moved to other tickets. #445 #446


chore: promote react-spring to version 0.0.76

this commit will trigger a pipeline to generate the actual kubernetes resources to perform the promotion which will create a second commit on this Pull Request before it can merge

react-spring
Changes in version 0.0.76
Chores

release 0.0.76 (jenkins-x-bot)
add variables (jenkins-x-bot)

Other Changes
These commits did not use Conventional Commits formatted messages:

redeploy (iMckify)

[jx-info] Hi, we've detected that the pipelines in this repository are using a syntax that will soon be deprecated.
We'll continue to update you through PRs as we progress. Please check #8589 for further information.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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


"Congratulations!" comment is made two times

@MartinBasti reached out that "Congratulations!" comments appear two times after /packit copr-build commands.
And this made me remember that I've seen this when opening a PR, last Friday.
Related to #539.

Race condition?

Race condition?

Might make sense, Fedora 32 and Rawhide builds finished first, 1 second apart:
2020-04-21 14:45:59 fedora-32-x86_64
2020-04-21 14:46:00 fedora-rawhide-x86_64
2020-04-21 14:46:11 fedora-31-x86_64
2020-04-21 14:46:26 fedora-30-x86_64
2020-04-21 14:46:31 epel-7-x86_64
2020-04-21 14:46:41 epel-8-x86_64

this has happened to me recently as well; we should have a DB field for this so we can avoid the race

in this case, it commented only once: https://github.com/containerbuildsystem/koji-containerbuild/pull/182

We have a plan to turn-off the comments. We will probably don't need to fix it.
Let's keep this open until #539 is not fixed.

Still happening

We discussed this today and agreed that we will:

[ ] add the instructions to the result page
[ ] add a note about removal to the "Congratulations!" comment
[ ] remove the commenting after two weeks

I've updated the other issue (#539) and will close this one in favour of it. Let us know if you don't like our idea and plan for removal.


license

hello,
i'd like to add your tool into ArchStrike, but your tools doesnt have a license file.
is it possible for you to add one in so i can take in your tool?

hi xorond, I forgot to add a license, I will do that tonight. Do you have a recommendation of what license type I should choose?

oh, i think GPL or MIT would be fine but it's up to you honestly. i just found your tool interesting and wanted it on our project :)

License file specifying MIT added. Thanks for adding this into ArchStrike!

thanks! i'll add the package soon


Searching for plugin: MMCVModulatedDeformConv2d, plugin_version: 1, plugin_namespace

hello,
when I use the repo, I meet the error:

my tensorRT is "TensorRT-<IP_ADDRESS>", cuda is 11.3, and I use release/8.2 in this TensorRT plugin

solved

Greetings, @happyday-lkj
I met the same issue.
Can you explain how did you solve that ?

hello lkj! I have missed the same error. I still don't know how to solve this problem, could you please give me more details about how to find "MMCVModulatedDeformConv2d" plugin and add this plugin to my tensorRT version? Thanks for your answer!


How to pass an object to a constructor by reference

Hi !
I would like to achive the following workflow:
In Lua: instantiate a new object with a contructor who needs a reference to another obejct as a parameter.
Here's my code:
class Zalm {
public:
Zalm(Foo& bar);
void print()

private:
Foo& bar_;
}

Zalm::Zalm(Foo& bar) : bar_{bar} {}
Zalm::print() { std::cout << bar_.foo() << std::endl; }

class Engine {
public:
Engine(Foo& bar);
void Loadlibs();

private:
Foo& bar_;
}

Engine::Engine(Foo& bar) : bar_{bar} {}
void Engine::Loadlibs() {
    sol::state lua{};
    lua.open_libraries(sol::lib::base);
    lua.new_usertype<Zalm>("zalm", sol::constructors<Zalm(Foo&)>(), "print", &Zalm::print);
    lua.set("hBar", std::ref(bar_));
}

And here's my lua script:
test = zalm.new(hBar)
test:print()

As a result my program crashes and the script is not executed.
Can you tell me how should I pass my object by reference ? Obviously, the object should no be destroyed after the ending of the script because it is needed elsewhere.
Thank you for your help

I've solved my problem. cheers.


为什么要区分操作系统那，在使用过程发现window上面无法滚动

Prerequisites
Please answer the following questions for yourself before submitting an issue.

[ ] I am running the latest version
[ ] I checked the documentation and found no answer
[ ] I checked to make sure that this issue has not already been filed
[ ] I'm reporting the issue to the correct repository (for multi-repository projects)

Expected Behavior
Please describe the behavior you are expecting
Current Behavior
What is the current behavior?
Failure Information (for bugs)
Please help provide information about the failure if this is a bug. If it is not a bug, please remove the rest of this template.
Steps to Reproduce
Please provide detailed steps for reproducing the issue.

step 1
step 2
you get it...

Context
Please provide any relevant information about your setup. This is important in case the issue is not reproducible except for under certain conditions.

Device:
Operating System:
Browser and Version:

Failure Logs
Please include any relevant log snippets or files here.

(function() {
isWindows = navigator.platform.indexOf('Win') > -1 ? true : false;
if (isWindows) {
    // if we are on windows OS we activate the perfectScrollbar function
    $('.sidebar .sidebar-wrapper, .main-panel').perfectScrollbar();

    $('html').addClass('perfect-scrollbar-on');
} else {
    $('html').addClass('perfect-scrollbar-off');
}

})();


Filterable list control panel: Padding in the example image is incorrect

Which page is this about?
Filterable list control panel
Describe your issue
Currently, rather than a live example, the page uses images to show the component. The second image is incorrect. It shows that there's 30px of padding between the form fields and the "apply filters" button, when it should be 15 px. This issue is about updating that image to show the correct padding (and therefore be accurate).
Add any additional background info as needed

Not sure where the source file for this image is, it probably needs to be hacked in photoshop or reproduced.
Also, ideally this would be live code rather than an image. So, if this becomes live code #946 then this issue is no longer necessary.

Closing this - 30px is what's live on the site and looks like the appropriate amount of spacing


[WIP] Valgrind fixes

Currently some debug code to run against the autotester to help debug issue 15019:
https://issues.dlang.org/show_bug.cgi?id=15019

I thought the REX.W prefix bug was fixed and pulled. If there's another one, please let me know where.

I thought the REX.W prefix bug was fixed and pulled.

Yes, thanks. But you need to bootstrap through DMD head to take advantage of the fix. There are cases when you don't want to do that (e.g. Digger would need to compile DMD a third time when bootstrapping just to work around this), I hope no one objects to a few small patches in the DMD code.

In that case, I'd like to see the workaround wrapped in a version(all) { workaround } else { original } along with a comment referencing the bugzilla issue and the DMD version where the bug is fixed.

OK, will do. This pull request is still in early stages though (right now I want to use the autotester to check if a condition is ever triggered).
Off the top of your head, do you know if calling Outbuffer::setsize (from backend/outbuf.c) with a size that exceeds the buffer capacity is a valid operation?

Since it is implemented:
void Outbuffer::setsize(size_t size)
{
    p = buf + size;
}

I don't think so. Use Outbuffer::enlarge

I'm asking because some DMD code is doing it. I added an assert, and the test suite passes with the assert, so it has to be a bug.
It looks like addtofixlist it passing an out-of-bounds offset to Obj::bytes: https://issues.dlang.org/show_bug.cgi?id=15019#c13

I've cleaned up this PR and changed the Valgrind workarounds accordingly. It should be ready to review/merge. This doesn't fix issue 15019, but the changes here helped make the failure deterministic and reduce the test case, and are useful on their own.

grep: ./obj/freebsd/32/mytrace.log: No such file or directory
grep: ./obj/linux/32/mytrace.log: No such file or directory

Does anyone know what's with these test failures?

Removed https://github.com/CyberShadow/dmd/commit/d537ec8d06c0813dd8ba72f5ca2dc2048301e2cb because it is superseded by https://github.com/D-Programming-Language/dmd/pull/5074

Wrapped the memset in #ifdef DEBUG.

Auto-merge toggled on


Random Battle Item Changes

Dedenne gets Petaya Berry, since it always has Cheek Pouch
Dusclops no longer gets Eviolite, because it's covered elsewhere in code

I kinda want to introduce slightly more complicated Eviolite checks, since Scyther doesn't really like having Eviolite and would prefer an attacking item usually...

Choice Band is better if the set has four attacks.

Sure, for Scizor specifically, but I'm thinking of having a more generalized case (since Seasonals generators can roll NFE's as well)
Maybe NFE's get a Choice item/Life Orb if they have four attacks, and their speed and offenses are past a certain point?


IE11 not working / Promise

On IE11 it does not work, simply because there is no window.Promise.
I wonder why you list it as working on IE 10 because there is no window.Promise either

@Eun why don't you use Polyfill.io to add support for promises?

Actually I use a polyfill, now.
But the sentence on the Readme suggests that you don't need any additional stuff:
pageAccelerator doesn't depend on jQuery, Zepto or any other library to work. You need just to include it at the end of your HTML code...

@Eun I would guess that pageAccelerator does not need jQuery or Zepto in order to run, it is native javascript. This does not apply to browser features which would need to be polyfilled.

Whatever, as I said before (for me) it suggests that you do not need any other dependencies.
A simple hint that you need polyfills for IE would be good.

I just updated the readme. I think now it's more complete.
Thanks @Eun and @mrmartineau 👍


x/website/internal/dl: ?mode=json don't show the SHA1 hash when the SHA256 hash is not available

For releases older than Go 1.6, the SHA256 hash is not available, so the sha256 field in the generated JSON output is empty.
However, the SHA1 hash is available, but is not printed due to
Checksum       string    `json:"-" datastore:",noindex"` // SHA1; deprecated

A possible solution is:

Use the omitempty for both Checksum and ChecksumSHA256 fields
Remove "-" from the Checksum field tag
When ChecksumSHA256 is not empty, set Checksum to the empty string

It seems that the SHA1 checksums are not available on https://dl.google.com/go, and instead of adding them #45608 suggests to add the SHA256 checksum to old releases, instead.
@dmitshur, I think that this issue can be closed.
Thanks.


Installer requires Intl

$ php symfony.phar new foo

 Downloading Symfony...

    4.92 MB/4.92 MB ▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓  100%

 Preparing project...

  [Symfony\Component\Intl\Exception\MethodArgumentValueNotImplementedException]
  The Symfony\Component\Intl\Collator\Collator::__construct() method's argument $locale
  value 'fr_FR' behavior is not implemented. Only the locale "en" is supported.  Please install
  the "intl" extension for full localization capabilities.

new directory [version]

http://symfony.com/doc/current/reference/requirements.html — this page states that Symfony has optional dependency on Intl, why does installer require it?
Symfony Installer has version 1.1.3.

Well, it looks like BC break in either symfony-standard or symfony/symfony. I'm sure there was no such requirement in the past. Where I should create an issue?

This one is critical. Anyone having a look at it?

I can't reproduce this error with the same isntaller version you are using:

Could you please share more details about the error? Which operating system are you using? Which PHP version? Are you using a virtual machine? Thanks.

@javiereguiluz You just have intl installed and enabled, but we don't.

I've disabled intl extension and the problem seems to be related with this code of the SymfonyRequirements.php file:
        if (class_exists('Collator')) {
            $this->addRecommendation(
                null !== new Collator('fr_FR'),
                'intl extension should be correctly configured',
                'The intl extension does not behave properly. This problem is typical on PHP 5.3.X x64 WIN builds.'
            );
        }

Full error trace:
Exception trace:
 () at /Users/javier/Desktop/test_without_intl/vendor/symfony/symfony/src/Symfony/Component/Intl/Collator/Collator.php:80
 Symfony\Component\Intl\Collator\Collator->__construct() at /Users/javier/Desktop/test_without_intl/app/SymfonyRequirements.php:648
 SymfonyRequirements->__construct() at phar:///usr/local/bin/symfony/src/Symfony/Installer/DownloadCommand.php:237
 Symfony\Installer\DownloadCommand->checkSymfonyRequirements() at phar:///usr/local/bin/symfony/src/Symfony/Installer/NewCommand.php:65
 Symfony\Installer\NewCommand->execute() at phar:///usr/local/bin/symfony/vendor/symfony/console/Symfony/Component/Console/Command/Command.php:252
 Symfony\Component\Console\Command\Command->run() at phar:///usr/local/bin/symfony/vendor/symfony/console/Symfony/Component/Console/Application.php:874
 Symfony\Component\Console\Application->doRunCommand() at phar:///usr/local/bin/symfony/vendor/symfony/console/Symfony/Component/Console/Application.php:195
 Symfony\Component\Console\Application->doRun() at phar:///usr/local/bin/symfony/vendor/symfony/console/Symfony/Component/Console/Application.php:126
 Symfony\Component\Console\Application->run() at phar:///usr/local/bin/symfony/symfony:35
 require() at /usr/local/bin/symfony:10

@stof in case SymfonyRequirements is the actual error, do you think it's better to fix it upstream in https://github.com/sensiolabs/SensioDistributionBundle/blob/master/Resources/skeleton/app/SymfonyRequirements.php#L646 or should we "hack" the installer somehow? Thanks.

It seems like this bug has been around for awhile - probably introduced with the Intl refactoring in Symfony 2.6 (I can find people complaining about this error at least back to Jan 2015). The whole Collator thing originated in sensiolabs/SensioDistributionBundle#59
We should definitely fix it up-stream:
A) Fix it in SensioDistributionBundle
B) Update the composer.lock file in the SE to get the latest DistroBundle with the fix
C) Wait for a patch release
In DownloadCommand, we could temporarily wrap the new SymfonyRequirements() with a try-catch for the exception. If we did that, we could not tell them if they have any errors, but we could continue their installation. So, not too sure about this one.

Introduced a fix for the SensioDistributionBundle at sensio/SensioDistributionBundle#207.
@javiereguiluz can you test that?

@weaverryan thanks for working on this. Here it is the result of the test:
Before

The patch
To test the patch you provided for SensioDistributionBundle, I added the following in the checkSymfonyRequirements() method:
protected function checkSymfonyRequirements()
{
    $originalRequirements = file_get_contents($this->projectDir.'/app/SymfonyRequirements.php');
    $newRequirements = str_replace(
        "if (class_exists('Collator')) {",
        "if (class_exists('Collator') && extension_loaded('intl')) {",
        $originalRequirements
    );
    file_put_contents($this->projectDir.'/app/SymfonyRequirements.php', $newRequirements);

    // ...
}

The result
The original error has gone, but now I get the following:

Thanks Javier! Can you try it again now? You can probably also test this by downloading my new SymfonyRequirements.php, just putting that into a project (post using the installer), and then running php check.php. But try that first to make sure you get the error :).

@weaverryan I can confirm that after applying your last patch, the error is gone (and the app/check.php fiel works too):

excellent - thanks!

Fixed now in the bundle.

how do I get this update?
$ symfony self-update
Symfony Installer is already up to date.

We should wait for release of symfony/symfony-standard.

@fabpot This will only be included in the next builds of the Framework Standard Edition, right? So, don't we still have to find a solution for the installer for all older versions?

@javiereguiluz @fabpot ping

Trying to simply start a new project. Before I've done any coding at all I already encounter this error...

Same for me...
tried to install the intl using the composer with no luck
any idea when the installer will be updated?

@weaverryan @xabbuh @stof what can we do from the installer point of view to avoid this issue? Should we wrap the requirements checker in a try ... catch block to fail silently?

Just check the requirements before trying to create a new project? Maybe
add 'symfony check' to explicitly call the requirements check and use it in
the 'new' method to check for missing reqs.
Tom Lankhorst

http://webhub.nl/
m. +31 (0)6 307 41 869
e<EMAIL_ADDRESS>Meeuwenstraat 54
7523 XV Enschede
2015-06-08 18:10 GMT+02:00 Javier Eguiluz<EMAIL_ADDRESS>
Reopened #163 https://github.com/symfony/symfony-installer/issues/163.
—
Reply to this email directly or view it on GitHub
https://github.com/symfony/symfony-installer/issues/163#event-325429570.

see #166


Fixed text around entity types

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Also: The text that follows the indication of shape in the docs could use revision. It still reads as if something other than Record is a reasonable choice. E.g., around line 149 it says "use type Record when ..." but of course Record is the only actual choice. Probably you will need to make a bunch of changes here, perhaps even deleting big chunks of text.


Unable to load DLL 'libsodium' on case sensitive file system

I have Windows 10 (21H1) and when I turn on case sensitivity on (fsutil.exe file SetCaseSensitiveInfo C:\folder\path enable) I got this error:
System.TypeInitializationException
  HResult=0x80131534
  Message=The type initializer for 'Sodium.SodiumCore' threw an exception.
  Source=Sodium.Core
  StackTrace:
   at Sodium.SodiumCore.GetRandomBytes(Int32 count)
   at Sodium.PasswordHash.ScryptGenerateSalt()

Inner Exception 1:
DllNotFoundException: Unable to load DLL 'libsodium' or one of its dependencies: The specified module could not be found. (0x8007007E)

FYI: I found what the problem is and reported it to .net: https://github.com/dotnet/runtime/issues/66608
It should be fixed in .net 7.


Performance of uvgRTP against other frameworks

Hi,
I've seen that you provide a quite nice benchmark comparing uvgRTP with other frameworks, in particular with Live555. I wonder if someone already tested uvgRTP versus others like GStreamer or JRTPLIB? I have a pretty good experience with the latter but I'm thinking about changing to a more modern framework like yours. Since it's an enterprise product, gstreamer is also in the short list considering its flexibility and maintainability (but it also looks more time consuming to implement in the first place)

Hi,
No, we have not tested against GStreamer or JRTPLIB. Overall, the performance differences are often not significant since the RTP process does not consume much CPU unless you are working in a low power environment such as embedded systems.
GStreamer probably has more features than uvgRTP, but uvgRTP is much easier to use and adopt.
BR, Joni


[FR] Hide the card if all bars are hidden

Consider this card:
  - type: custom:bar-card
    severity:
      - from: '0'
        to: '0'
        hide: true
      - from: '1'
        to: '40'
        color: '#007A00'
      - from: '41'
        to: '90'
        color: '#EA9000'
      - from: '91'
        to: '100'
        color: '#C41400'
    entities:
      - entity: input_number.test_level_1

When a bar is hidden - the card still is displayed:

This may be solved by card-mod, but I would like to suggest an option: when it is set to true:
-- if no bars are displayed - hide the whole card.

I would love to see this feature.  I'm currently using card-mod to do this; but, it would be much nicer to have a builtin feature!


Complete the constructors and assignment operators of Thing

Some code snippets in the TM library are using the move constructor of Thing, so it might be a good idea to actually speficy that.
I've made this PR over here instead of at https://github.com/seven1m/tm, since this repository contains all the other updates for TM as well, so this is the most up to date version that can be copied over to the other locations.

Related: https://en.cppreference.com/w/cpp/language/rule_of_three#Rule_of_five

The move operations now reset the source object, so it can be used in tests as well:
auto t1 = Thing(1);
Thing t2(std::move(t1));
assert_eq(0, t1.value()); // t1 has been reset
assert_eq(1, t2.value());


Stacktrace can ignore user-owned file when using tearoff

Repro:
import 'dart:async';

void main() {
  var controller = StreamController<int>();
  var controller2 = StreamController<int>();
  controller2.stream.listen((v) => controller.add(v));
  controller2.add(42);
  controller.close();
  controller2.add(42);
}

This code throws an error:
Connecting to VM Service at ws://<IP_ADDRESS>:34837/-D1zY7tPPbM=/ws
Connected to the VM Service.
Unhandled exception:
Bad state: Cannot add event after closing
#0      _StreamController.add (dart:async/stream_controller.dart:605:24)
#1      main.<anonymous closure> (package:bug/bug.dart:6:47)
#2      _RootZone.runUnaryGuarded (dart:async/zone.dart:1609:10)
#3      _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:366:11)
#4      _DelayedData.perform (dart:async/stream_impl.dart:542:14)
#5      _PendingEvents.handleNext (dart:async/stream_impl.dart:647:11)
#6      _PendingEvents.schedule.<anonymous closure> (dart:async/stream_impl.dart:618:7)
#7      _microtaskLoop (dart:async/schedule_microtask.dart:40:21)
#8      _startMicrotaskLoop (dart:async/schedule_microtask.dart:49:5)
#9      _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:118:13)
#10     _RawReceivePort._handleMessage (dart:isolate-patch/isolate_patch.dart:185:5)

Exited (255).

Focus on line 1 (not 0) where it tells me the file, function, line and offset of where the error was thrown.
If you use a tear off:
import 'dart:async';

void main() {
  var controller = StreamController<int>();
  var controller2 = StreamController<int>();
  controller2.stream.listen(controller.add);
  controller2.add(42);
  controller.close();
  controller2.add(42);
}

This is the output:
Connecting to VM Service at ws://<IP_ADDRESS>:35306/udzDEf1Qq0U=/ws
Connected to the VM Service.
Unhandled exception:
Bad state: Cannot add event after closing
#0      _StreamController.add (dart:async/stream_controller.dart:605:24)
#1      _RootZone.runUnaryGuarded (dart:async/zone.dart:1609:10)
#2      _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:366:11)
#3      _DelayedData.perform (dart:async/stream_impl.dart:542:14)
#4      _PendingEvents.handleNext (dart:async/stream_impl.dart:647:11)
#5      _PendingEvents.schedule.<anonymous closure> (dart:async/stream_impl.dart:618:7)
#6      _microtaskLoop (dart:async/schedule_microtask.dart:40:21)
#7      _startMicrotaskLoop (dart:async/schedule_microtask.dart:49:5)
#8      _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:118:13)
#9      _RawReceivePort._handleMessage (dart:isolate-patch/isolate_patch.dart:185:5)

Exited (255).

I have no more reference as to where this happened on my code. I think this should be better handled for better DX. I was struggling for more than half an hour on my Flutter project with many different files and pages when I realized where this could be happening. The uncaught exceptions also triggers inside the framework code and there is no way for you to go back to your own code.

That looks correct.
The controller2 is not a synchronous controller, so the call to the function argument to controller2.listen happens in a later microtask. At that point there is nothing on the stack except the internal Stream code that tries to call that function.
Then it calls the function, which is the tear-off controller1.add, which is exactly what shows up in the stack trace.
There is no code from main on the stack at this point, so the stack trace is correct.
In the first example, the argument function was (x) { controller1.add(x); }, not controller.add1. That adds one extra stack frame when called, which shows the location of controller.add(x) in main as that position that calls contoller1.add.
Errors that happen in asynchronous code do not have the original stack any more, that's just how event loops work.

I know this is the current behaviour. Sorry if I was not clear on my request.
I'd like to ask if there is any way for the call stack to know about the call being a tear off (at least when developing). If this is possible, we could possibly show more information to the programmer in cases like the above so it's easier to find errors even in asynchronous code.

I guess could be technically possible for an instance member tear-off to record where in the code the tear-off occurred, at least in development mode. It will still be equal (==) to another tear-off of the same function from the same object that happens in another place, but could maybe look different in stack traces.

Just to be clear, for the above case as an example, are you talking about controller1.add knowing where it got used or controller2.stream.listen knowing it received a tear off and where?
Just because both could help here but they would show slightly different things in the stack trace I believe.

I'm talking about controller.add knowing where it was torn off, so that can show up in stack traces.
I don't know how the tear-off works.
If there is no code in it, and calling it directly invokes the torn off method, there is nothing on the stack to recognize that it was involved.
If it contains any code, even if just a jmp to the real method, it should be possible to do something else in development mode, like changing jmp to call+ret, and have a stack entry.
(But I'll stop guessing now and leave this to people who actually know what the VM is doing.)

The VM does not record where a function was torn off from and we do not have plans to implement that.
There is possibly a workaround to use a closure instead of a tear off.


video_player在ios手机上视频资源释放不掉

Hi @ailibin
could you provide more details on the issue?
your flutter doctor -v, related logs, reproducible code sample if related

Closing as additional information isn't provided


Accept RegExp and Functions in filter options.

ref #1059.
ignoreErrors, includePaths, ignoreUrls, whitelistUrls now accept arrays of RegExp and functions.
The major change is that ignoreErrors now stay as array after config(), and the configuration must always be an Array. Which I think makes for even more consistent API.
joinRegExp is wrapped by optimizeFilters. And the test must be done using using matchFiltters function.
The provided test function will be tested before string and RegExp.
Call the check with matchFilters(ignoreUrls, typeString, type, message, fileurl,...), then test function will be called as function(typeString, type, message, fileurl, ...) (I keep typeString as first parameter for convenient).
Noted that I didn't pass any extra parameter to includePaths, ignoreUrls, whitelistUrls but I made them accept function anyway, for consistency.

All the test case on travis passes. I don't know why it's still listed as failed.

Thanks @pisinr, I'll review it Today. Travis is failing because of SauceLabs keys, we're aware of this issue, so it's fine :)

@kamilogorek @pisinr What is the status with this PR? I would be happy to continue the work if its stopped

Not relevant for sentry-javascript SDK v4.


Please expose the contents of the _type file to allow for better static type analysis

I am writing some wrapper/library code using Ollama, and I'm running into issues when trying to type hint. Not all of the types used in the library are public, and I would appreciate if that were to be remedied.
The simplest fix at present would be to include the contents of _type in the ollama __init__ file.
Thanks

More types will be exported in the next release. I have a change almost ready for a PR


xml2kvp: attribute configurations

Need a bit more granular control on inclusion of attributes in field name creation.
Currently have these configurations:
"exclude_attributes": []   <-- list of attributes to skip
"include_attributes": true  <-- boolean to turn on/off attributes entirely

Was never happy with their inverse names, but not quite inverse behavior.  What is also needed is a list of attributes to include, which might supplant include_attributes.  Could call the high level include_all_attributes?  But have the behavior be, if that is False, still listen to include_attributes list, the same way the inverse is true for True to all attributes, but exclude_attributes removes?

Done.


Integration with Lotus::Helper

[x] Make Lotus::Helper a dependency
[ ] Application generator should explicitly include helpers in views (see Example 1). This strategy allows devs to toggle this feature.

Example 1
require 'lotus/helpers' # << NEW

module Web
  class Application < Lotus::Application
    configure do
      # ...
      view.prepare do
        include Lotus::Helpers  # << NEW
      end
    end
  end
end

Hey @jodosha are you working on this? otherwise I'm happy to take it.

@tomkadwill Please :)

@tomkadwill @joneslee85 I think that lotus-helpers should be an hard dependency like lotus-view et all.

Closing as it was implemented.


Re-added a random primer when no melody or midi primer supplied in the command line

Address issue https://github.com/tensorflow/magenta/issues/121 recent update removed default behavior that supplied a random primer if a midi file or melody was not supplied in the command line.

@danabo  Good work getting it going. Do you want me to test your code. I had issues with the code dying if bpm was set in the command line 14
I think the issue is at
https://github.com/tensorflow/magenta/blob/master/magenta/models/shared/melody_rnn_sequence_generator.py#L91


Fix authentication issues when minting

How did we miss this to begin with 😮💨
Nice work

Approve? :)


Category counts

I have allowed the system to track the counts of each category rather than just a list. Like, instead of just listing the categories like [a,b,c], we can now internally have a way to count each category. The categories will return as before like [a,b,c], but inside, _categories variable is now something like [a:1,b:2,c:3].

Need to add some tests to ensure categorical data was counted as expected at initialization, update, and merge.

Refer to Ability to internally store counts of each category in a CategoricalColumn in #296


Prevent setting SERVER_NAME and SESSION_COOKIE_DOMAIN to an IP address

See #1946 and #1906. I think we should prevent setting those things to IP addresses in the first place and only allow None and domain names != localhost.

From #1955: We should also check that no port is contained.

Given that SERVER_NAME is used with url_for(..., _external=True) when not in a request context, preventing ports would break links.  Maybe there's a more correct way to do external links outside request contexts, but I couldn't find it at the time.

Given that SERVER_NAME is used with url_for(..., _external=True) when not in a request context, preventing ports would break links.

It rather seems to me that using ports there only accidentally worked and was never expected by the API authors. SERVER_NAME is passed in Flask, yes. However, I don't think Werkzeug's Map.bind expects a port either, since 1.) the parameter is also called server_name 2.) server_name is then idna-encoded, which is nonsensical for host + port combinations and actually breaks for certain hostnames:
>>> 'foobar:443'.encode('idna')
'foobar:443'
>>> 'fööbar:443'.encode('idna')
b'xn--fbar:443-n4aa'

However, apparently app.run derives its default port from SERVER_NAME.
This is so broken...

This and the propagate exceptions issue are both cases of one option doing more than it should.  We should probably do a more thorough review of what each option does and maybe add some more to disambiguate stuff before 1.0.

Something like:

Prevent ports in SERVER_NAME
Add SERVER_PORT
Change app.run behavior and url_for implementation accordingly.

Might be nice to make SERVER_NAME separate from bind, so that multiple domains / proxies don't cause 404s when you're really just trying to use SERVER_NAME for url_for.

I don't understand, SERVER_NAME is already the frontend hostname.
On 10 September 2016 20:17:51 CEST, David Lord<EMAIL_ADDRESS>wrote:

Might be nice to make SERVER_NAME separate from bind, so that
multiple domains / proxies don't cause 404s when you're really just
trying to use SERVER_NAME for url_for.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/pallets/flask/issues/2007#issuecomment-246127048

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

@untitaker yeah never mind, not sure what I was thinking with that one.

would you guys mind if I had a go at this one? I'd like to help and also squash my first bug. Thanks!

Ah, this isn't actually that beginner friendly since we still need to decide what to actually do.

I do think that adding SERVER_PORT would be correct, but somebody might disagree!

I didn´t fully undestand this bug. Please correct me if I'm wrong. I think that one solution  for this problem is check if the variables SERVER_NAME and SESSION_COOKIE_DOMAIN are an IP address and if so what should I do? Abort the program? Give a warning message? And do you want me to check if it is IPV4 or IPV6 or only IPV4?

@mferreira96 IP addresses don't work when used in cookies, only domains do, this is why setting SESSION_COOKIE_DOMAIN to an IP is nonsensical. I can't remember why I wanted to prevent setting SERVER_NAME to an IP though (IPs should work for url_for).
Warning message sounds actually more reasonable. Both v4 and v6.

i am having some difficults in find where i should call the function that i build to give a warning when the person use an IP address in SESSION_COOKIE_DOMAIN, can you help me?

I set the SESSION_COOKIR_DOMAIN as "mysite.com", but browser domain remain be ".mysite.com" and my subdomains can see my main site cookie.


ERRO[0000] [metrics]: .... bind: address already in use

Hello!
When I restart RoadRunner daemon it can not bind to the same port it was opened and returns errors like ERRO[0000] [metrics]: .... bind: address already in use
If i check thoses ports it says
tcp        TIME_WAIT   -
I googled the issues and looks like there's a issue with go lang when it does not reuse ports
https://github.com/nordicdyno/golang-tcp-port-reuse
Tested on linux \ ubuntu
Thank you.

Yeah, similar issues are causing some tests to fail periodically. I'm not seeing quick way to fix it yet.


Create Login Page

Set up Mongo DB with Admin user and relations

User Stories G7 (1 point), G8 (2 points)


Case sensitivity for label names

Case sensitivity for label names

Create label name with the first letter of upper case
SELECT *
FROM cypher('graph_name', $$
CREATE (:Person{name: 'Andres', title: 'Developer'})
$$) as (n agtype);
Search by lowercase label name -> "label person does not exists" error
SELECT *
FROM cypher('graph_name', $$
match(a:person)
RETURN a
$$) as (a agtype);
-> label person does not exists
Is it a normal case ?

I am not sure if open-cypher has any mention about the casing of Labels in their reference, but in neo4j cypher and most of its variants, Labels are always case-sensitive.

The cypher used in AgensGraph and Neo4j is known to be case-insensitive for label names.
If case-sensitivity is required, the label can be identified by wrapping the label with a double quotation "".

@kysmou I do not know about Agensgraph or openCypher, but in Neo4j, Labels are case-sensitive.
It has been some time that i last used neo4j and i wasn't sure, so i decided to confirm in the docs.
From the Neo4j Cypher Manual v4.2:

Case-sensitive:
Names are case-sensitive and thus, :PERSON, :Person and :person are three different labels, and n and N are two different variables.

Some times the implementation can be different from the manual. I tried your queries in neo4j:
CREATE (:Person{name: 'Andres', title: 'Developer'})
> Added 1 label, created 1 node, set 2 properties,

MATCH (a:person) RETURN a
> (no changes, no records)

In the Match operation, it also gives me a contextual warning:

The provided label is not in the database.
One of the labels in your query is not available in the database, make sure you didn't misspell it or that the label is available when you run this statement in your application (the missing label name is: person)

Thanks for your point.
AGE is Since the Graph Data area is an extension module based on postgresql, I think we should follow the policy. In postgresql and AgensGraph, it is insansitity by default.

As for conventions about case-insensitive or not, i don't really have an opinion. i think the most important is for them to be clear and for everybody following them.
Databases is not really my cup of tea and i am not familiar with this repo's code or even writing postgresql extension, so my following comments are very "tentative".
I am preoccupied that case-insensitivity in Labels might have some performance impact on the extension in some extreme cases.
Labels in AGE are stored as a record in a table ag_label of the ag_catalog schema. And since searching records is case-sensitive, the code would have to make a full table search with LOWER or UPPER (and also possibly the two at the same time because, quoting another person, "some characters have different representations in upper case but the same representation in lower case") every time to detect possible duplicates?
To avoid this, the code could forcibly transform them to LOWER (for example) upon creation. But there might have more complications...
It seems that there is some kind of cache, but it is another thing where i am not sure of the implications.

Maybe this will be helpful here,...
AGE's team's philosophy, with regards to aligning towards either openCypher, Neo4j, or PostgreSQL, is as follows -
First, and foremost, we try to implement the openCypher specification as closely as possible. Where there is ambiguity or vagueness in the openCypher specification, we look to Neo4j for possible clarification. When there is still ambiguity or vagueness, we try to think of what our user base would expect - what makes sense.

@jrgemignani  Spot on, thanks John!


Clean up code in ESLint & pass Code Climate

Overview
We need to clean up code in ESLint & Code Climate so it can clear compilation issues.
Action Items

[ ] Review action steps with Jonathan/ Laurence
[ ] Clean up code in ESLint
[ ] Confirm it can pass Code Climate

Resources/Instructions
Useful command for auto-fixing via ESLint (low hanging formatting fruit):
"lint": "eslint 'src//*.js'",
"lint:fix": "eslint --fix 'src//*.js'",

Comment - Issue blocking code pull requests may be related to a github action (rather than codeclimate).

Update 1/3/2021:
Phase I (All flagged corrections which do not alter code logic) - 60%
Pull Request ETA 1/4/2021, no blockers, resolving questions with @mealthebear as they arise. Expecting a comprehensive test pass on my end as this touches most js files in the website.
Phase II (All flagged corrections involving minor changes to code logic) - 0%
No ETA yet
Phase III (All flagged corrections involving major refactors to code logic) - 0%
No ETA yet
NOTE: Please mention me on any pull requests affecting js files over the next week or so, so that I can update my own branch and avoid overwriting code

Update 1/3/2021:
Phase I (All flagged corrections which do not alter code logic) - 60%
Pull Request ETA 1/4/2021, no blockers, resolving questions with @mealthebear as they arise. Expecting a comprehensive test pass on my end as this touches most js files in the website.
Phase II (All flagged corrections involving minor changes to code logic) - 0%
No ETA yet
Phase III (All flagged corrections involving major refactors to code logic) - 0%
No ETA yet
NOTE: Please mention me on any pull requests affecting js files over the next week or so, so that I can update my own branch and avoid overwriting code

Note from 1/3/2021 Meeting:
Individual issues to be generated for each file to be edited for ESLint cleanup (including formatting fixes). List of files affected to follow. Further discussion of cleanup organization to follow at end of meeting.

Note from 1/3/2021 Meeting:
Individual issues to be generated for each file to be edited for ESLint cleanup (including formatting fixes). List of files affected to follow. Further discussion of cleanup organization to follow at end of meeting.

Files flagging ESLint problems:
src/components

/Data/* - ALL FILES CLEAN
/Footer/index.js - Import Alpha sort, 6 lines too long, new line at EOF
/Footer/styles.js - 1 Function too many lines (108/50), new line at EOF
/Header/index.js - Import Alpha sort, 1 Function too many lines (87/50)
/Header/styles.js - 1 Function too many lines (270/50), new line at EOF
/Dropdown.js - Indents too large, Import Alpha sort, Key Alpha Sort, props missing in validation, 1 line too long.
/DropdownThumbnail.js - indents too large, Key Alpha Sort, props missing in validation, 1 line too long, unexpected console statement, new line at EOF.
/GetStartedCard.js - Import Alpha Sort, Key Alpha Sort, props missing in validation, 2 lines too long.
/index.js - Key Alpha Sort, 1 line too long.
/NavBreadcrumbs.js - Import Alpha Sort, Key Alpha Sort, Comment Block Format
/NavButton.js - Indents too large, Import Alpha Sort, Key Alpha Sort, 1 line too long, new line at EOF.
/PictureCard.js - Import Alpha Sort, Key alpha sort, props missing in validation, trailing spaces, 2 lines too long, extra semicolon
/ProjectCard.js - Import Alpha Sort, const arrow syntax, missing key prop, 1 Function too many lines (and many lines too long), new line at EOF.
/SearchBar.js - Indents too large, Import Alpha Sort, props missing in validation, new line at EOF.
/TitleSection.js - Import Alpha Sort, Key Alpha Sort, props missing in validation, 1 line too long.

src/context

/reducer.js - unexpected tokens (ellipses on L7, L14)
/tagGeneratorContext.js - unexpected tokens (ellipses on L88, etc.)
/tagGeneratorContext1.js - Key Alpha Sort, owner prop missing in validation.
/tagGeneratorContextProvider1.js - Import Alpha Sort, Key Alpha Sort, props missing in validation, Comment Block Format, (Extra Blank lines?)

src/pages

/Contributors/index.js - Indents too Large, Import alpha sort, key alpha sort, Unexpected Token
/Contributors/styles.js - Indents too Large, Key Alpha Sort, 1 Function too many lines (114/50), new line at EOF.
/Donation/index.js - CLEAN
/Donation/styles.js - 1 Function too many lines (169/50)
/Home/sections/NotableUsersSection.js - Indents too Large, Import Alpha Sort, props missing in validation, new line at EOF.
/Home/sections/TrendingTopicsSection.js - Indents too large, Import Alpha Sort,

LIST TO BE CONTINUED

Files flagging ESLint problems:
src/components

/Data/* - ALL FILES CLEAN
/Footer/index.js - Import Alpha sort, 6 lines too long, new line at EOF
/Footer/styles.js - 1 Function too many lines (108/50), new line at EOF
/Header/index.js - Import Alpha sort, 1 Function too many lines (87/50)
/Header/styles.js - 1 Function too many lines (270/50), new line at EOF
/Dropdown.js - Indents too large, Import Alpha sort, Key Alpha Sort, props missing in validation, 1 line too long.
/DropdownThumbnail.js - indents too large, Key Alpha Sort, props missing in validation, 1 line too long, unexpected console statement, new line at EOF.
/GetStartedCard.js - Import Alpha Sort, Key Alpha Sort, props missing in validation, 2 lines too long.
/index.js - Key Alpha Sort, 1 line too long.
/NavBreadcrumbs.js - Import Alpha Sort, Key Alpha Sort, Comment Block Format
/NavButton.js - Indents too large, Import Alpha Sort, Key Alpha Sort, 1 line too long, new line at EOF.
/PictureCard.js - Import Alpha Sort, Key alpha sort, props missing in validation, trailing spaces, 2 lines too long, extra semicolon
/ProjectCard.js - Import Alpha Sort, const arrow syntax, missing key prop, 1 Function too many lines (and many lines too long), new line at EOF.
/SearchBar.js - Indents too large, Import Alpha Sort, props missing in validation, new line at EOF.
/TitleSection.js - Import Alpha Sort, Key Alpha Sort, props missing in validation, 1 line too long.

src/context

/reducer.js - unexpected tokens (ellipses on L7, L14)
/tagGeneratorContext.js - unexpected tokens (ellipses on L88, etc.)
/tagGeneratorContext1.js - Key Alpha Sort, owner prop missing in validation.
/tagGeneratorContextProvider1.js - Import Alpha Sort, Key Alpha Sort, props missing in validation, Comment Block Format, (Extra Blank lines?)

src/pages

/Contributors/index.js - Indents too Large, Import alpha sort, key alpha sort, Unexpected Token
/Contributors/styles.js - Indents too Large, Key Alpha Sort, 1 Function too many lines (114/50), new line at EOF.
/Donation/index.js - CLEAN
/Donation/styles.js - 1 Function too many lines (169/50)
/Home/sections/NotableUsersSection.js - Indents too Large, Import Alpha Sort, props missing in validation, new line at EOF.
/Home/sections/TrendingTopicsSection.js - Indents too large, Import Alpha Sort,

LIST TO BE CONTINUED

I've submitted a PR that (hopefully) improves ESLint settings. Fixing the linting errors as shown above in all the JS files should be equally doable with or without this PR, but I think going forward we'll be better served by my PR. My humble 2¢.
See PR #202

I've submitted a PR that (hopefully) improves ESLint settings. Fixing the linting errors as shown above in all the JS files should be equally doable with or without this PR, but I think going forward we'll be better served by my PR. My humble 2¢.
See PR #202

Update 1/10/2021:
Need to discuss cases where ESLint should be overridden when addressing difficult-to-refactor functions (particularly where max-lines-per-function is thrown). For discussion Tuesday (or Thursday) with @mealthebear, @shinjonathan.

Update 1/10/2021:
Need to discuss cases where ESLint should be overridden when addressing difficult-to-refactor functions (particularly where max-lines-per-function is thrown). For discussion Tuesday (or Thursday) with @mealthebear, @shinjonathan.

Shin ran the following commands on the repo during today's group coding session, which automatically resolved the bulk of the phase 1 issues across the project:
"lint": "eslint 'src//.js'",
"lint:fix": "eslint --fix 'src//.js'",

Shin ran the following commands on the repo during today's group coding session, which automatically resolved the bulk of the phase 1 issues across the project:
"lint": "eslint 'src//.js'",
"lint:fix": "eslint --fix 'src//.js'",

gfm seems to be taking your asterisks and mangling what you posted. I have it as:
"lint": "eslint 'src/**/*.js'",
"lint:fix": "eslint --fix 'src/**/*.js'",

gfm seems to be taking your asterisks and mangling what you posted. I have it as:
"lint": "eslint 'src/**/*.js'",
"lint:fix": "eslint --fix 'src/**/*.js'",

Dev is doing a clean up of the last final issues.

Dev is doing a clean up of the last final issues.

1/17/2021: No updates from assignee. Team work to clean and refactor code is ongoing. Timeline for completion to be discussed in meeting.

1/17/2021: No updates from assignee. Team work to clean and refactor code is ongoing. Timeline for completion to be discussed in meeting.

@Kevin will run eslint and make a new issue for each set of items per page that comes up.  Will also cross check with the list above.
Estimated ETA: a couple of days.

@Kevin will run eslint and make a new issue for each set of items per page that comes up.  Will also cross check with the list above.
Estimated ETA: a couple of days.

There are now 14 new issues for the 14 files with ESLint errors and warnings. The new issues will resolve the "Clean up code in ESLint" part of this large multi-file issue. However, it remains to be seen if they will all "pass Code Climate" -- but we will open new issues as needed, so I am closing this one for now.

There are now 14 new issues for the 14 files with ESLint errors and warnings. The new issues will resolve the "Clean up code in ESLint" part of this large multi-file issue. However, it remains to be seen if they will all "pass Code Climate" -- but we will open new issues as needed, so I am closing this one for now.


Feat: add the 'buflisted' options for the repl buffer.

By default, the repl buffer created by iron is not "buflisted".
I use the buffer functionality a lot, and I would like the REPL buffer
is nothing different with my other buffers.

Thank you very much for the PR!
I'm preparing a new release with a bunch of small improvements that ultimately should make iron more usable, stable and easier to maintain. Out of those I was planning to add something like this (https://github.com/hkupty/iron.nvim/blob/88812a3456489a3245abb346faadcd9e1d770363/lua/iron/lowlevel.lua#L68) but I can definitely merge your PR and work around the conflict in the future release.
Out of curiosity, would you mind if the buffer is no longer scratch when it is listed, like in my patch? Not for now, just so I know how to proceed for the next version.

I use 'buflisted' only because that I use ']b, [b' to navigate between buffers frequently, and I use a laptop so I don't split my screen unless necessary.
I don't see what would happen if setting "unscratch" for a terminal, is it means that I cannot use :q to quit it but have to use :q! to quit it?


From all possibilities of backbone atoms in res_atoms.dat - need to include logic for simple trims

RINRUS can't currently handle the situation where the only backbone atoms interacting with seed are the carbon alpha and it's hydrogen. If this is the case, then there are two outcomes:

If the sidechain is included in the model, we wouldn't want to add additional backbone, because the C_alpha trimmed to CH3R would be captured in the sidechain fragment
If the sidechain isn't included in the model, there needs to be some default trimming behavior based on if neighboring residues are in the model or will eventually be added.

Issue solved now - if only CA or HA selected, both sides of main chain added to anchor it


Time Detail Dimension View does not handle DST correctly

Describe the bug
There are issues with proper DST handling in the pivot table when opening the time detail dimension view and DST is included as part of the filtered time range. This issue will be further exacerbated if first_day_of_week is configured and set to a different value (e.g. Sunday), resulting in erroneous weekly columns on the wrong days to be displayed along with incorrect calculations. Originally reported by a customer here. We may want to extend the solution covered in #3494 (notion doc) to also apply to time detail dimension views as well.
To Reproduce
Steps to reproduce the behavior:

Clone the 311 Operational Metrics locally
Apply the filter to include Nov 5 (daylight savings in the US)
Drill into the time detail dimension view
Check the weekly columns (around Nov 5) with a non-UTC timezone selected (e.g. CST or EST)
Apply the first_day_of_week: 7 property to the dashboard YAML
Check the results and make sure the weekly columns (+ data) are correct

Expected behavior
The pivot table should display the correct weekly columns, both data and the date / day shown in the column header, regardless if UTC or a different timezone with DST is being used.
Actual Behavior
The pivot table displays incorrect weekly columns when a non-UTC timezone is being used where DST is a consideration.
Screenshots
DST issue with EST and no first_day_of_week set:

No DST issue when using UTC with no first_day_of_week set:

DST issue with CST and first_day_of_week: 7 set:

No DST issue with UTC and first_day_of_week: 7 set:

I checked main today, it works correctly.
First day - Monday

First day - Sunday

Timestamps are 7 days apart.

@egor-ryashin So I can confirm that the issue is still reproducible on main (it's likely a corner case). See screenshots - tested on both v0.38.2 and main.
Steps to reproduce:

Use the Rill-311-Ops example project
Use the attached dashboard yaml to change the timezone to America/Chicago and first_day_of_week to Sun
Drill into Median Time to Resolve Ticket (Hours) from main dashboard view
Adjust date range to Oct 29 - Nov 30 and make sure that metric trends is set to week
Note that the column header in the TDD will still show Nov 4 (Sat) as first day of week instead of Nov 5 (Sun)


[add-resource] New Plugin/Theme

I would like to add my new plugin/theme to the Grav Repository.
Here are the project details: https://github.com/wernerjoss/grav-plugin-fullcalendar

Added to GPM


Hidden Ground Items Resetting

Describe the bug
When removing all items from the "Hidden" Ground Items list, after quitting and restarting the client, they are back.
To Reproduce
Go to Ground Items
Erase everything in "Hidden"
Close RuneLite
Open RuneLite
Check Ground Items
Expected behavior
Field should stay cleared after clearing.
Environment (please complete the following information):

OS: Windows 10
RuneLite version: 1.5.18
Launcher version: Unknown

FYI, I tested putting "Iron Daggers" in as a junk item, closed and restarted the client, and it saved it. For some reason, it is not saving the field when blank.

It resets to the default value when blank.

Which should be treated as a big, as all the other similar fields don’t act this way, and there is no reason for it to.

I’d also like to point out this wasn’t an issue before the last few releases.

Whatever release was 8/28 probably because that is when the default was added.

For anyone that is looking for a workaround for not hiding any items, just input anything in "Hidden Items". For example, I just put Nothing and it works fine.

For anyone that is looking for a workaround for not hiding any items, just input anything in "Hidden Items". For example, I just put Nothing and it works fine.

As of September 1, 2023, placing the text, "Nothing" in the Hidden Items field no longer works. It acts the same as leaving the field blank. Closing or resetting the Runelite app after using this method now results in example:
Hidden Items

Nothing,Green egg,Red egg,Blue egg,Hammer,Logs,Yellow egg,Crackers,Tofu,Worms

For anyone that is looking for a workaround for not hiding any items, just input anything in "Hidden Items". For example, I just put Nothing and it works fine.

As of September 1, 2023, placing the text, "Nothing" in the Hidden Items field no longer works. It acts the same as leaving the field blank. Closing or resetting the Runelite app after using this method now results in example:
Hidden Items

Nothing,Green egg,Red egg,Blue egg,Hammer,Logs,Yellow egg,Crackers,Tofu,Worms

This is incorrect. Adding "nothing", or anything else, prevents the field from being reset. The default items are also Vial, Ashes, Coins, Bones, Bucket, Jug, Seaweed.
Green egg,Red egg,Blue egg,Hammer,Logs,Yellow egg,Crackers,Tofu,Worms are specific to barbarian assault. Maybe you installed a barbarian assault related plugin which is overwriting your hidden items. You should uninstall and stop using such plugins.

For anyone that is looking for a workaround for not hiding any items, just input anything in "Hidden Items". For example, I just put Nothing and it works fine.

As of September 1, 2023, placing the text, "Nothing" in the Hidden Items field no longer works. It acts the same as leaving the field blank. Closing or resetting the Runelite app after using this method now results in example:
Hidden Items

Nothing,Green egg,Red egg,Blue egg,Hammer,Logs,Yellow egg,Crackers,Tofu,Worms

This is incorrect. Adding "nothing", or anything else, prevents the field from being reset. The default items are also Vial, Ashes, Coins, Bones, Bucket, Jug, Seaweed.
Green egg,Red egg,Blue egg,Hammer,Logs,Yellow egg,Crackers,Tofu,Worms are specific to barbarian assault. Maybe you installed a barbarian assault related plugin which is overwriting your hidden items. You should uninstall and stop using such plugins.

You are correct. Thank you for that information. The additional Barbarian Assault plugins were indeed causing the issue.

My friends, could we not resolve this 4 year old bug?

I think the simplest solution to this is to make a toggle for Hidden items (E.G. Hidden items enabled [ ]) and greying out the Hidden items area if that toggle is not on.

The same problem exists with resetting "Highlighted Items", it always reverts to a previous state, I'm functionally unable to make any changes to it.
The only solution that persists is creating a brand new Runelite profile (requires reinstalling desired plugins).
Is there a config file we can edit in Notepad or something?

The same problem exists with resetting "Highlighted Items", it always reverts to a previous state, I'm functionally unable to make any changes to it.
The only solution that persists is creating a brand new Runelite profile (requires reinstalling desired plugins).
Is there a config file we can edit in Notepad or something?

This is an issue tracker is not a support forum. Please join the discord if you wish to receive further support.

it always reverts to a previous state, I'm functionally unable to make any changes to it.

That's not the same problem, this issue is about how if the input box is empty, the client will revert to the default items, not just "be empty"
The issue you have is very likely a permission issue, the client is no longer able to update its file, which we can give you the steps to fix in the discord


Fix handling of empty gpu slices.

Remove overly strict assertion on zero size view.
Match slicing behavior to numpy's on out-of-bound slices (i.e. bounds are clip to valid values)

Codecov Report

Merging #2434 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #2434      +/-   ##
==========================================
- Coverage   86.44%   86.44%   -0.01%
==========================================
  Files         315      315
  Lines       60403    60403
  Branches     6583     6583
==========================================
- Hits        52218    52217       -1
- Misses       7137     7138       +1
  Partials     1048     1048

Merging since @stuartarchibald is out today.

Thanks @seibert


Vue2: missing Pinia tab with errors in console

Vue devtools version
6.4.5
Link to minimal reproduction
https://github.com/a-kriya/vue2-pinia-devtools-bug
Steps to reproduce & screenshots

clone repro repo
npm i
npm run dev
navigate to app (http://localhost:5173/)
open browser devtools
navigate to Vue.js devtools tab

What is expected?
Inspectable Pinia state in Vue devtools, as seen here with the same app in Vue3:

What is actually happening?
Pinia tab is unavailable with errors in browser JS console.

System Info
System:
    OS: Linux 5.4 Linux Mint
  Binaries:
    Node: 18.13.0 - /usr/bin/node
    npm: 8.19.3 - /usr/bin/npm
  Browsers:
    Chromium: 96.0.4664.93
    Firefox: 108.0.2
  npmPackages:
    vue: ^2.7.14 => 2.7.14

Removed some system info which I do not think is pertinent.

Any additional comments?
Ran into the issue when trying to reproduce another possible bug w/ Pinia and different Vue versions.

You forgot to install the plugin.
From the docs:

If you are using Vue 2, you also need to install a plugin and inject the created pinia at the root of the app:
Vue.use(PiniaVuePlugin)

This will also add devtools support.

Thank you.


CMD - service list command refactoring Fix #744

Hello!
A few changes here:

[x] Added text/tabwriter
[x] Changed the sort with the new options added in Golang 1.8.
[x] Added header labels

Around the compact format for backend address, I tried a different ways, but I didn't get any better option, if you have any suggestion I'm happy to change it.
Current output:
➜  cilium git:(744) ✗ ./cilium service list
ID   Frontend       Backend
2    <IP_ADDRESS>:90     1 => <IP_ADDRESS>:80
                    2 => <IP_ADDRESS>:80
                    3 => <IP_ADDRESS>:80
3    <IP_ADDRESS>:8080   1 => <IP_ADDRESS>:80
                    2 => <IP_ADDRESS>:80

Regards.

So the build is failing because the go version in the CI is go1.7.4.
@aanm shouldn't the VM be running at least go1.8.1 by now?
[Runtime Tests] ==> cilium-master: cilium/cmd/service_list.go:90: undefined: sort.Slice
[Runtime Tests] ==> cilium-master: Makefile:43: recipe for target 'tests-consul' failed
[Runtime Tests] ==> cilium-master: make: Leaving directory '/home/vagrant/go/src/github.com/cilium/cilium'
[Runtime Tests] ==> cilium-master: make: *** [tests-consul] Error 1

@aanm shouldn't the VM be running at least go1.8.1 by now?

@tgraf was the new image pushed to atlas?

@tgraf was the new image pushed to atlas?

It's currently building on my mac. I've updated the scripts a bit as well: https://github.com/cilium/packer-ubuntu-16.10/pull/4

It's currently building on my mac. I've updated the scripts a bit as well: cilium/packer-ubuntu-16.10#4

New vagrant box is uploaded and updated on Jenkins slaves. Please rebase master --ignore-date and push -f to trigger a new Jenkins run.

Hey,
I updated the branch from master. Looks like that there is a problem with the labels. Not sure if something related to this branch.
Do I need to do anything here?
Best regards

Do I need to do anything here?

No, this seems to be an issue with upgrading to 1.8.3, we're tracking this in #955

Hello!
Ok, I'll keep an eye on that issue. I forgot to check the tests suite and 06-lb.sh is affected. I changed a few lines, and I use the REST API instead of use the cilium list, not sure if It is correct, if you're not happy with that I'll change to the original behaviour ;-)

Ok, I'll keep an eye on that issue. I forgot to check the tests suite and 06-lb.sh is affected. I changed a few lines, and I use the REST API instead of use the cilium list, not sure if It is correct, if you're not happy with that I'll change to the original behaviour ;-)

The CI is back, you can push again after you've fixed the issues of this PR.

All looks good now, I think that is good to merge ;-)


Feat/cli dump

Implements https://github.com/ipfs-shipyard/go-dumpotron/issues/2

Minor, but it'd be nice if the archive was moved to the cwd when doing a local dump. I'm unlikely to remember where the temp folder is if i need to find it in the future, and I probably wont ever clean it up, nor will OSX, if it's hidden away in a temp.

Minor, but it'd be nice if the archive was moved to the cwd when doing a local dump. I'm unlikely to remember where the temp folder is if i need to find it in the future, and I probably wont ever clean it up, nor will OSX, if it's hidden away in a temp.

:+1: Will address in a PR


Proposal: Update to the Kaia Foundry Pyth Template

Description

This is an update to the pyth price oracle price conversion library

Bounty track

Developer tool

Please attach the link to the repo here.

Please attach the link to the repo here.

https://github.com/PaulElisha/kaiachain-foundryTemplate--pricefeedManager-Price-converter

Closing this issue because bounty has been paid. Open new one if you want to add updates.


Give users control over vector cache prefilling

This could either mean to make the prefill target configurable - or to allow a manual prefill through the APIs. Note that the current prefill implementation seems to be not ideal and potentially blocking. As part of this issue, this is something that would have to be improved as well.

The latest commits fix the blocking issue and optimize the loading of the cache. As a result, we can simply auto-prefill to the respective vectorCacheMaxObjects limit set on the class.
Will be part of the next release and the 1.8.0 stable release.
cc @ktoulgaridis


[FEATURE] Request adaptation to the Timeless and Classics Guns mod

In Timeless and Classics Guns mod, the guns aiming in over-the-shoulder view automatically switches back to first person, and the crosshair corresponds to the actual impact point of the firearm in over-the-shoulder view (which is not corresponding).

Thanks for the suggestion. This is a feature cannot be implemented as the player is never looking at that position with his head in the first place. However, you could add all weapons of Timeless and Classics Guns to adaptive_crosshair_items within Shoulder Surfings config file (.minecraft\config\shouldersurfing.toml). This would enable the dynamic crosshair or switch you to first person (depending on crosshair_type) when a weapon of Timeless and Classics Guns is held.