Graphics::Context concept for advanced rendering scenarios

In conversations surrounding #739, several advanced scenarios came up which revealed deficiencies in the abstractions currently available in the Graphics component. These deficiencies center around problems that arise during multi-render scenarios.
The "canonical" Babylon Native application is columnar in that, at any given time, it only has one instance of each of its major concepts.

One Graphics object.
One JavaScript thread/runtime/Napi::Env.
One NativeEngine.

In this simple case, the current abstractions are sufficient. However, for more complex scenarios, new abstractions will be needed to safely and usably expose rendering capabilities. In particular, not every app is guaranteed to have only one JavaScript thread/runtime/Napi::Env; many such instances can exist, and it must be possible for them to all independently and simultaneously run and render to their own (potentially even more numerous) NativeEngines. Under these circumstances, some of the behavior currently encapsulated in the FrameBufferManager becomes problematic. A key responsibility of the FrameBufferManager is to manage the relationships between bgfx's View concept and Babylon's FrameBuffers so that everything gets drawn to the correct targets in the correct order. If this is done improperly, bgfx will reorder rendering commands so that they can be batched by View, which can cause commands to execute out of the intended order and produce incorrect results. For example, consider a simple scene that renders a single object which casts a shadow using shadowmapping. The high-level Babylon commands when rendering this scene might be as follows:

Clear the main render target.
Clear the shadowmap render target.
Render the object from the light's perspective to the shadowmap render target.
Render the object from the camera's perspective to the main render target, using the shadowmap to render shadow.

With only two bgfx Views (one per render target), it is not possible to execute these commands in the specified order. (In this case, no visible artifact would result from doing step 1 after step 3, which would allow this to be done with only two Views; however, this is not the case in general as it is possible, for example, for step 2 or 3 to take a dependency on the state of the main render target.) To render in the correct order, then, the FrameBufferManager must spread these operations out over three Views.

View 0 targets the main render target and entails only a clear.
View 1 targets the shadowmap render target and entails a clear and a draw call.
View 2 targets the main render target and entails no clear and a draw call.

This, however, is made much more complicated if there are multiple NativeEngines drawing simultaneously across multiple threads. For example, consider a scenario where NativeEngine A is rendering the shadowmap scene described above using high-level Babylon commands A.1, A.2, A.3, and A.4, while a separate NativeEngine B on a different JavaScript thread/runtime/Napi::Env is rendering a simple scene with four objects using high-level Babylon commands B.1, B.2, B.3, B.4, and B.5 (a clear and four draw calls). Because of threading, it is possible for these commands to happen in a wide variety of orders including the following:
B.1, A.1, B.2, A.2, B.3, A.3, B.4, A.4, B.5

Because the current FrameBufferManager is a Graphics-unique object, all these commands would operate on the same FrameBufferManager. This framebuffer manager would thus be forced to spread these operations out across nine bgfx Views because the View must be incremented each time the target is changed to preserve command order. However, command order does not actually matter between NativeEngines A and B -- they exist in separate contexts and so their commands cannot take cross-context dependencies -- so this scenario really only needs four views: three for A and one for B. There are other problems with having multiple threads contending to interact with a single FrameBufferManager, too, as well as complications with having two NativeEngines in the same JavaScript thread/runtime/Napi::Env manipulating what they think of as two graphics contexts (for example, what on the Web might be two different canvases) and possibly taking dependencies on the states they "leave" them in.
To solve these problems, it seems likely that a Context concept must be added to the Graphics component. This concept will be logically descended from the Graphics::Impl/FrameBufferManager and will likely replace those as the point of interaction for consuming components. Contexts will be acquired from the Graphics::Impl (or whatever name the replacement for that takes) and will be responsible for brokering all interactions which require consistent state. There can be arbitrarily many of them -- you can have many owned by/used within a single JavaScript thread/runtime/Napi::Env -- but their relationship with threading may be complex, and I haven't fully comprehended it yet. On the one hand, they seem like they want to be fundamentally unsafe because they encapsulate state which includes inviolable ordering. On the other hand, however, they also seem like they want to be the source for UpdateTokens, which frequently must be acquired on one thread, then moved to another. Perhaps the Context is a mixed bag? Or perhaps UpdateTokens should evolve such that they can be acquired on any thread, but they can only be used (to get the bgfx::Encoder) on the thread to which their Context is bound? I'm not sure; I haven't fully comprehended it yet. This, and likely many other questions, will need to be answered in the course of adding this new Context concept to the Graphics component.

Conversation happening on PR #1028


Both


TypeError: init() got an unexpected keyword argument 'skip_link_detection'

For context and log file please see: https://github.com/mu-editor/mu/issues/1525
Mu user @Ardo9602 reports:

Adafruit RP2040 with Circuitpython 6.2.0 beta4 and release
Microbit with Micropython
Editor starts fine with no board connected, crashes with any of these connected, connecting before or after program start makes no difference.

Stack trace from Mu logs (provided by user)
2021-05-01 23:04:35,139 - root:173(excepthook) ERROR: Unrecoverable error
Traceback (most recent call last):
File "C:\Users\Admiq\AppData\Local\Programs\Mu Editor\Python\lib\site-packages\mu\logic.py", line 711, in check_usb
detected = mode.find_devices(with_logging=False)
File "C:\Users\Admiq\AppData\Local\Programs\Mu Editor\Python\lib\site-packages\mu\modes\base.py", line 423, in find_devices
device = self.compatible_board(port)
File "C:\Users\Admiq\AppData\Local\Programs\Mu Editor\Python\lib\site-packages\mu\modes\circuitpython.py", line 286, in compatible_board
for comport in circuitpython_serial.repl_comports():
File "C:\Users\Admiq\AppData\Local\Programs\Mu Editor\Python\lib\site-packages\adafruit_board_toolkit\circuitpython_serial.py", line 61, in repl_comports
port for port in comports() if port.interface.startswith("CircuitPython CDC ")
File "C:\Users\Admiq\AppData\Local\Programs\Mu Editor\Python\lib\site-packages\adafruit_board_toolkit\circuitpython_serial.py", line 46, in comports
ports = _list_ports_windows.comports()
File "C:\Users\Admiq\AppData\Local\Programs\Mu Editor\Python\lib\site-packages\adafruit_board_toolkit_list_ports_windows.py", line 458, in comports
return list(iterate_comports())
File "C:\Users\Admiq\AppData\Local\Programs\Mu Editor\Python\lib\site-packages\adafruit_board_toolkit_list_ports_windows.py", line 342, in iterate_comports
info = list_ports_common.ListPortInfo(port_name_buffer.value, skip_link_detection=True)
TypeError: init() got an unexpected keyword argument 'skip_link_detection'

Apologies @dhalbert - I missed your comment on the Mu issue. Closing this because it seems to be a PySerial / local user packaging problem.


fix(logger) default logger.js location

Currently a custom logger.js file is expected to be found in the anvil-connect node package root - if a logger.js file is added there it's deleted every time node_modules is reinstalled.
As discussed in the original commit here: https://github.com/anvilresearch/connect/commit/68ba379e4355ae70b36ea2d1a0f1ddf18b90a562 this PR changes the behaviour to look relative to where anvil is being run from.

Coverage remained the same at 79.743% when pulling 6b18c7c2b2d4effa29f59e26d691e1d9ebcfccd6 on hedleysmith:logger-js-path into b31264d76fd71158ab7b54db576a9cf8dafc0918 on anvilresearch:master.

@hedleysmith that's a great catch. Thanks!


Rotation state labels are invisible on the service instance details page

This is because the CSS class is "label label-".

Problem is that ResourceAssembler.doSpecialNonPersistentAssociations() is using the flat projection, which ignores the status type and hence leaves us without a label. Solution is to use the default projection.


Change main for develop in to sync forked repos

✨ Codu Pull Request 💻

👉 Please remove the below and replace with your own values, leaving the headers where they are. 👈
Pull Request details:
Changing the word main for develop- when using the Git CLI way to sync forked repos
Closes #58
Any Breaking changes:

My fork says I'm 22 commits ahead but that I'm okay to merge automatically

Associated Screenshots:
( Welcome file extensions include gifs/png screenshots of your feature in action )

None

We chatted offline - I'm going to close this one so you can open one and add the rebase command instead of merge. Since we can see how much fun using the merge command led to:


Monitor encrypted DNS (DOH and DOT)

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
New Monitor
🔖 Feature description
I would like to be able to monitor DNS-Over-TLS and DNS-Over-HTTPS.
Both are encrypted forms of DNS.
I have a dns server that only responds to encrypted queries and I would like to monitor if it's up.
✔ Solution
Add a DOT and DOH field to the DNS monitor.
❓ Alternatives
Theoretically I can monitor the ports, but that's not enough as the service could be intermitent
📝 Additional Context
No

+1

@StefanescuCristian
Please refrain from posting +1 / requests for updates things on issues, as this makes issue-management harder.
Issues are for discussing what needs to be done how by whom.
We use 👍🏻 on issues to prioritise work.
PRs are always welcome for bugfixes like this

👍

DoH at least should be possible. If I am not mistaken, the dns2 module is used by the dnsResolve function here:
https://github.com/louislam/uptime-kuma/blob/538f83205d75fdfc437bece61764e864ae90a5b0/server/util-server.js#L292-L293
And this module has baked-in DoH support. Here is the example code:
https://github.com/song940/node-dns/blob/master/example/client/doh.js
I might try to tackle this and make a PR sometime later if I get the time to brush up on nodejs, but no promises.

This would be nice to have.

The DNS monitor is currently quite barebones and not really useful.
If someone wants to improve upon this, our contribution guide can be found here https://github.com/louislam/uptime-kuma/blob/master/CONTRIBUTING.md

I'd love to have a way to monitor my Adguard Home DNS and see if it's still connectable.


Support for Http/2 Client and Mock Server

Provides support for Http/2
Client support for Http/2 messages is provided via integration with Apache Async client API. This pull request updates to all Karate modules to use the Apache Async client however the original Apache client remains available for legacy support if required.
Mock server is updated to support Http/2 by updating the use of the Netty API.
Supports HTTP/1.1 and HTTP/2 requests including

Cleartext upgrade and prior-knowledge and
Over SSL with upgrade and prior-knowledge (including supporting ALPN)

Relevant Issues : https://github.com/intuit/karate/issues/474
Type of change :

[ *] New feature

wow that's a lot :| thanks ! one thing that stood out, HttpConfig got moved to a different package https://github.com/intuit/karate/blob/master/karate-core/src/main/java/com/intuit/karate/Config.java
and also I spotted FeatureServer - can you double check while I figure out how to review all of this.
certainly won't make it into this weeks 0.9.2

Thanks Peter.
We've waited this long to get this in another little while won't harm too much. It'd be great to get into the next release after this week though.
HttpConfig was merged in error but the changes required to control this new feature were added to Config.java in any case.  HttpConfig.java was removed and pushed to develop branch on Openet-Labs/karate.
FeatureServer.java has been updated with SSL context creation primarily with the application protocol negotiation config. Not sure what your comment above about FeatureServer relates to.
Cheers

HI Peter,
Have you made any further considerations on accepting this request?
Cheers

@bmcnama I haven't, I've found it very hard to review this and the other commits you see have been priority. I thought I mentioned this but karate-netty was moved into karate-core but I don't see that taken care of. obviously now there are some conflicts that need to be synced.
since evidently this is not progressing as fast as you would like, do you think you releasing / maintaining a stand-alone module for async would be a better approach in the time being ?

Thanks @ptrthomas
I would really like to get this work mainstreamed as a karate feature and feel it's a worthwhile and valuable feature. However it is hard to review and I totally understand that other commits continued with a priority in the meantime. Is there something that can be done with the request to make it easier to consume? Would more documentation or examples be required?

@bmcnama if you use slack, please send me a mail at  at gmail dot com and I'll invite you to a karate slack channel - it may make sense to do this over a video conference, so let's explore that
I was about to suggest the option of "mock only first" thanks for bringing it up. but I see your point about tests. but if it makes it easier for me (yep I am a bottleneck) maybe we can try that
re: karate-netty I see un-necessary entries in karate-junit and karate-netty - but other than that agree the code is in core now. I was reacting to the pom.xml-s

Thanks Peter...brian.mcnamara at openet com
On Tuesday 28 May 2019, 13:58:20 IST, Peter Thomas<EMAIL_ADDRESS>wrote:
@bmcnama if you use slack, please send me a mail at at gmail dot com and I'll invite you to a karate slack channel - it may make sense to do this over a video conference, so let's explore that
I was about to suggest the option of "mock only first" thanks for bringing it up. but I see your point about tests. but if it makes it easier for me (yep I am a bottleneck) maybe we can try that
re: karate-netty I see un-necessary entries in karate-junit and karate-netty - but other than that agree the code is in core now. I was reacting to the pom.xml-s
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@bmcnama  closing this since we've discussed about re-starting this thread from a clean slate and collaborating closer, also we will experiment with tracking roadmap items here: https://github.com/intuit/karate/projects/3#card-22529196

Has Karate support for HTTP/2 now been abandoned? I'm not clear what happened to this pull request.

@ari this PR is out of date. the latest develop version is based on Armeria which supports http/2 so we estimate it should be very easy to get http/2 support but it needs some research and ideally code-contributions from the community. not currently considered a priority by the devs

Hello There
My team have faced with h2 problem long time ago. So we have written our own h2 client based on jetty. It was adjusted in accordance with 1.0.1 karat interface so it is fully up to date. We are happy to share our code so I am ready to open PR.
@ptrthomas what do you think? is it better to move jetty-h2-client to core or separate module

@ArtyomKravchenko jetty sounds like it would add more dependencies so definitely a separate module.
my preference is that if we can complete the implementation of ArmeriaHttpClient
so the question for you is that does your implementation support all the config that apache supports (ssl, timeout, etc) - there are a lot - but yes most people may not need them. the way to prove this is to run all the existing apache tests with the new client
anyway good point and I forgot to mention it: @ari you should be able to implement a custom http client and plug it in - this has become a lot more simpler in 1.X: https://github.com/intuit/karate/tree/master/karate-mock-servlet#switching-the-http-client

Yes, it has limited config support, but it still support CONNECT_TIMEOUT, READ_TIMEOUT, HTTP_PROXY. It definitely support ssl (the h2 works over ssl always). It also has wildcard trust manager and host verifier (allow self signed or expired certificates). In first order we surely satisfied our needs. But we are happy to share our experience all over the world and transfer ongoing support to karate team.
In case if you worried about extra dependencies it is good choice to move it to separate karate module

@ArtyomKravchenko okay. for now I suggest you open-source your implementation separately as a reference and I can update the docs here to link to it.
I'm just re-reading through this thread and the PR commits - and it sounds like maybe the best option now is to look at the apache httpcomponents client 5.X - and switch to it - and then we should get http/2 support
I think the API is similar to 4.X (which we use currently) so all the config will be supported and the migration will be simple
for server-side we should be already on http/2 because of armeria - but this is anyway lower priority

Hi Peter, Artyom,
It's great that you're still interested in adding Http/2 functionality to karate. For our part, we continue to use an extensive set of functional tests based on karate which use the separate http apache async module that was produced in this PR. So we are very interested in getting this functionality added back into the mainline. It's just a pity that we have not prioritised the effort to get this work over the line.
The later Apache components supported Http/2 with TLS and included support for ALPN (which was a problem with the earlier version). We have some work locally that aligned the async module with a beta version of this component however this could do with updating.
Let us know if we can still be of assistance in this regard or even with testing of any new karate http client component or other solution. I realise that this work has been hanging around for some time and want to get a solution in place.
Cheers
Brian
On Tuesday 15 June 2021, 07:56:51 IST, Peter Thomas<EMAIL_ADDRESS>wrote:

@ArtyomKravchenko okay. for now I suggest you open-source your implementation separately as a reference and I can update the docs here to link to it.
I'm just re-reading through this thread and the PR commits - and it sounds like maybe the best option now is to look at the apache httpcomponents client 5.X - and switch to it - and then we should get http/2 support
I think the API is similar to 4.X (which we use currently) so all the config will be supported and the migration will be simple
for server-side we should be already on http/2 because of armeria - but this is anyway lower priority
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

@bmcnama it is great to see you still active ! I apologize for not merging this PR - it is mostly my fault, over the course of time I have realized that merging large change-sets is a problem-area for me that I need to work on :| also I indeed lacked understanding of some of the things being discussed.
here's what I propose as the next step to keep things simple for me. it will be great if you can contribute from your code-base, which will certainly same me some time:

focus only on the http client
focus only on "classic" not async or reactive
ideal state is to replace http-client 4.X with 5.X. if my understanding is right it will give use HTTP 1+2 support with minimal code-changes - and all the work we did for config, ssl, proxy, cookies - can be retained - good thing is we have a lot of tests
I don't mind replacing the apache-http-client, so we don't need to waste time keeping the "old" one around.

does that make sense !

hello there again
https://github.com/ishgroup/oncourse/tree/server_api_test/api-test
it is our gradle based module for karate tests certainly
See
https://github.com/ishgroup/oncourse/tree/server_api_test/api-test/src/test/java/ish/oncourse/api/test/client
package for test client implementation


Unexpected key(s) in state_dict: "lstm_bert.bert.embeddings.position_ids"

When I use predict.py to predict RiPPs, something is wrong:
Traceback (most recent call last):
  File "/my/soft/path/Software/TrRiPP/predict.py", line 161, in <module>
    model = Prediction.load_from_checkpoint(args.weight).to(DEVICE)
  File "/my/soft/path/trripp/lib/python3.9/site-packages/pytorch_lightning/utilities/model_helpers.py", line 125, in wrapper
    return self.method(cls, *args, **kwargs)
  File "/my/soft/path/trripp/lib/python3.9/site-packages/pytorch_lightning/core/module.py", line 1581, in load_from_checkpoint
    loaded = _load_from_checkpoint(
  File "/my/soft/path/trripp/lib/python3.9/site-packages/pytorch_lightning/core/saving.py", line 91, in _load_from_checkpoint
    model = _load_state(cls, checkpoint, strict=strict, **kwargs)
  File "/my/soft/path/trripp/lib/python3.9/site-packages/pytorch_lightning/core/saving.py", line 180, in _load_state
    keys = obj.load_state_dict(checkpoint["state_dict"], strict=strict)
  File "/my/soft/path/trripp/lib/python3.9/site-packages/torch/nn/modules/module.py", line 2189, in load_state_dict
    raise RuntimeError('Error(s) in loading state_dict for {}:\n\t{}'.format(
RuntimeError: Error(s) in loading state_dict for Prediction:
	Unexpected key(s) in state_dict: "lstm_bert.bert.embeddings.position_ids".

It maybe the version question of transformers. My installed version is 4.41.1. Which version was used to train models?

I used transformers=4.24.0 I will update the environment.yml according to this issue, thanks

Thanks, when I installed the version 4.24.0, it worked fine without any issues.


Error When Hashing Files

When attempting to import files, I'm getting the following error messages.
[2016-10-01 16:34:24:010] Error|BackgroundWorker.OnDoWork => CommandProcessorHasher.workerCommands_DoWork => CommandRequest_HashFile.ProcessCommand Error processing CommandRequest_ProcessFile: E:\Anime1\Trigun.Badlands.Rumble.2010.720p.BluRay.x264-THHT.mkv - System.NullReferenceException: Object reference not set to an instance of an object.
   at JMMServer.Commands.CommandRequest_HashFile.FillVideoHashes(VideoLocal v) in D:\My Files\Documents\GitHub\jmmserver\JMMServer\Commands\CommandRequest_HashFile.cs:line 378
   at JMMServer.Commands.CommandRequest_HashFile.ProcessFile_LocalInfo() in D:\My Files\Documents\GitHub\jmmserver\JMMServer\Commands\CommandRequest_HashFile.cs:line 248
   at JMMServer.Commands.CommandRequest_HashFile.ProcessCommand() in D:\My Files\Documents\GitHub\jmmserver\JMMServer\Commands\CommandRequest_HashFile.cs:line 61
[2016-10-01 16:34:37:336] Error|MainWindow.workerFileEvents_DoWork => Importer.RunImport_ImportFolderNewFiles => Importer.IsNetworkShare System.ArgumentException: Object must be a root directory ("C:\") or a drive letter ("C").
   at System.IO.DriveInfo..ctor(String driveName)
   at JMMServer.Importer.IsNetworkShare(String ImportFolderLocation) in D:\My Files\Documents\GitHub\jmmserver\JMMServer\Importer.cs:line 464

My import folders are on local drives, I do have a Google Cloud Account added but those files have already been imported.
@Baine has also gotten this error on 3.7 as well.

Already fixed but not on master.
On Sun, Oct 2, 2016 at 3:08 AM, ElementalCrisis<EMAIL_ADDRESS>wrote:

When attempting to import files, I'm getting the following error messages.
CommandProcessorHasher.workerCommands_DoWork => CommandRequest_HashFile.ProcessCommand
Error processing CommandRequest_ProcessFile: E:\Anime1\Trigun.Badlands.
Rumble.2010.720p.BluRay.x264-THHT.mkv - System.NullReferenceException:
Object reference not set to an instance of an object.
at JMMServer.Commands.CommandRequest_HashFile.FillVideoHashes(VideoLocal
v) in D:\My Files\Documents\GitHub\jmmserver\JMMServer\Commands\
CommandRequest_HashFile.cs:line 378
at JMMServer.Commands.CommandRequest_HashFile.ProcessFile_LocalInfo() in
D:\My Files\Documents\GitHub\jmmserver\JMMServer\Commands\
CommandRequest_HashFile.cs:line 248
at JMMServer.Commands.CommandRequest_HashFile.ProcessCommand() in D:\My
Files\Documents\GitHub\jmmserver\JMMServer\Commands\
CommandRequest_HashFile.cs:line 61
[2016-10-01 16:34:37:336] Error|MainWindow.workerFileEvents_DoWork =>
Importer.RunImport_ImportFolderNewFiles => Importer.IsNetworkShare
System.ArgumentException: Object must be a root directory ("C:\") or a
drive letter ("C").
at System.IO.DriveInfo..ctor(String driveName)
at JMMServer.Importer.IsNetworkShare(String ImportFolderLocation) in
D:\My Files\Documents\GitHub\jmmserver\JMMServer\Importer.cs:line 464

My import folders are on local drives, I do have a Google Cloud Account added but those files have already been imported.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<https://github.com/japanesemediamanager/jmmserver/issues/453>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABHDbQOgr0cMnIVc1cR-aCnlShJFkWyJks5qv0pigaJpZM4KL8bY>
.

Oh Awesome. :)
I'll close this issue once the fix is on master.

In master


Allow additional characters in Authorisation credentials

I'm getting an error when trying to use = inside my Authorisation value.
For example, using;
Authorisation: Basic YmQ0NT02NTM4YWE5ND03MzJkMTdlN0RiNTdkOGFj0TAwZjhkZj0mYTp5Qzc4aDgmZW85R1A=

results in:
Error executing Lua callback: .../share/nvim/lazy/rest.nvim/lua/rest-nvim/parser/init.lua:114: attempt to index local 'req_node' (a nil value)
stack traceback:
	.../share/nvim/lazy/rest.nvim/lua/rest-nvim/parser/init.lua:114: in function 'traverse_request'
	.../share/nvim/lazy/rest.nvim/lua/rest-nvim/parser/init.lua:432: in function 'parse'
	...al/share/nvim/lazy/rest.nvim/lua/rest-nvim/functions.lua:61: in function 'exec'
	...cal/share/nvim/lazy/rest.nvim/lua/rest-nvim/commands.lua:44: in function 'impl'
	...cal/share/nvim/lazy/rest.nvim/lua/rest-nvim/commands.lua:182: in function <...cal/share/nvim/lazy/rest.nvim/lua/rest-nvim/commands.lua:165>

This seems related to https://github.com/rest-nvim/tree-sitter-http/pull/23 where it mentions = being disabled for URLs.

I have the same problem when setting url as a header value. It seems like : and / are not allowed in value as well
Example:
###

POST {{BASE_URL}}/registration/registration
Content-Type: application/json
Origin: https://example.com

{
  "hello": "world"
}

Error messages:

[rest.nvim] ERROR: The tree-sitter node at the range [4:8 - 16] has a syntax error and cannot be parsed
[rest.nvim] ERROR: The tree-sitter node at the range [4:8 - 16] has a syntax error and cannot be parsed
Error executing Lua callback: .../share/nvim/lazy/rest.nvim/lua/rest-nvim/parser/init.lua:114: attempt to index local 'req_node' (a nil value)
stack traceback:
        .../share/nvim/lazy/rest.nvim/lua/rest-nvim/parser/init.lua:114: in function 'traverse_request'
        .../share/nvim/lazy/rest.nvim/lua/rest-nvim/parser/init.lua:430: in function 'parse'
        ...al/share/nvim/lazy/rest.nvim/lua/rest-nvim/functions.lua:61: in function 'exec'
        ...cal/share/nvim/lazy/rest.nvim/lua/rest-nvim/commands.lua:44: in function 'impl'
        ...cal/share/nvim/lazy/rest.nvim/lua/rest-nvim/commands.lua:182: in function <...cal/share/nvim/lazy/rest.nvim/lua/rest-nvim/commands.lua:165>


Rad auth breaking changes

To reproduce :

Clear the keys from ssh-agent
Go to radicle settings
Set RAD_HOME to a new directory
Click test

{exitCode=0, timeout=false, cancelled=false, stdout=
Initializing your radicle 👾 identity
✗ Authentication failed: The input device is not a TTY
, stderr=}

Here the problem is that the cli waits for the user to fill his alias. So i added the field alias in the settings

Result :
https://github.com/cytechmobile/radicle-jetbrains-plugin/assets/31673488/cabd04f9-4e83-4e9e-9dd4-4ce2a7422664


Add String resource mover CLI script

This adds a cli script to move String declarations from one module to another. This helps to strip the legacy code base or do some refactoring that includes touching string resources.
Usage:
./scripts/resource-mover --from <source-module-path> --to <target-module-path> --keys <keys-to-move>

This is a preparation for the app name provider implementation, that requires string resources to be moved.

The fact that source and target files are read multiple times for each key is making me a bit sad. But this is not mission critical code. It seems to work and performance is not an issue.

What about skipping such comments? This script is intended to help with refactoring the project, not to be a high-performance tool and neither to win any prizes. I'm prioritizing getting a job done and simplicity over performance concern and premature optimizations. Especially looking at the code base, there are bigger fishes to fry first.


samba: fix darwin

the Darwin sdk now includes the cups headers which configure detects but the build fails to link due to the missing cups library. use a configure flag to explicitly enable / disable cups (default is off).
there is no linux change (cups is not used by default) but the additional configure flags trigger a rebuild
https://hydra.nixos.org/build/276120820

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Uh, pushed as 2a44ea3ed4a8.  Git's rebasing of merges is... temperamental.


Merge pull request #1 from ory/master

sync

sync

Coverage remained the same at 47.131% when pulling dbaaeabbadfe72d638aabb0088c115ffc0a67bfe on paulkoanui:master into 959a190ae1aab52fce783f8bcb8aa8c51b591b4b on ory:master.


Implement legacy gas price strategy

Research has shown us that using 1.1 x eth_gasPrice was very cost efficient in getting fulfillments confirmed at a low gas cost (https://github.com/sassmilic/historical-gas-prices/)
We want the requester to be able to specify a reserved parameter (_gasPriceCoefficient? that corresponds to x1 when it's 10^18) that the node will multiply by eth_gasPrice to find the gas price it will use to fulfill a request.
This is a feature for when Airnode gets used in a time-critical way (e.g., for an asset price data feed).

Closed to be tracked by https://api3dao.atlassian.net/jira/software/projects/AN/boards/18/backlog?selectedIssue=AN-78


config : add renovate

Description
added renovate config following this guide renovate docs and installed renovate from github apps
Fixes #21
Type of change

[ ] 📦 Chore (config)

Added tests?

[ ] 🙅 no, because they aren't needed

will renovate bot works after merge ? because right now it's configured from forked repo
and also do we need dashboard while using it or github PR would be enough (created 2 dependency Pr on my forked one 😉)


content/archaeology: quote -- in option

Prevents from forming an em-dash instead of two double
for consistency do the same on other titles on the page

I did not do it previously because how Sphinx renders code in titles looks very ugly to me. But it is indeed more consistent.


Combining coverage results from multiple jobs in Python (with pytest)

I've setup my open source project to run CI with Azure Pipelines (see https://github.com/pydata/xarray/pull/3039), and am collecting code coverage following the example from the Azure pipelines docs on how to test Python apps.
This seems to work pretty well, but the code coverage statistics seem to only pick up a test results from a single job (at random). To get complete coverage for my project (e.g., for platform dependent code), I really need to aggregate coverage across all of the test jobs.
What's the right way to configure Azure to show this information?

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

ID: 2ca42420-9324-1988-16bd-501be69e278d
Version Independent ID: 400c0122-a0cc-bb52-b6cc-8aa93ff2c47a
Content: Build and test Python apps - Azure Pipelines
Content Source: docs/pipelines/languages/python.md
Product: devops
Technology: devops-cicd
GitHub Login: @thomps23
Microsoft Alias: dastahel

Thanks for your question. It looks like you're working through an issue with your scenario or implementation, rather than an issue with the documentation. Here are a few options where you might consider asking your question:

Azure DevOps on Stack Overflow
Azure DevOps Support Bot


[incubator/jaeger] hotfix for Cassandra TLS vaule

Signed-off-by: Pedro Silva<EMAIL_ADDRESS>Is this a new chart
No
What this PR does / why we need it:
Fixes
Which issue this PR fixes

fixes a bug created by #19376

Special notes for your reviewer:
@naseemkullah
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @naseemkullah

/lgtm


Fix prompt for s///c in Ex mode

Currently when doing s/foo/bar/c in Ex mode the substitution prompt ^^^ may shrink to single ^:
Entering Ex mode.  Type "visual" to go to Normal mode.
:print
  1 foo
:s/foo/bar/c
  1 foo
    ^^^x
  1 foo
      ^y

I wonder who uses Ex mode these days...
Anyway, can you add a test for this?  src/testdir/test_substitute.vim has Test_substitute_variants() that can probably used as an example.

Looks like src/testdir/test_ex_mode.vim already has Test_Ex_substitute(). I'll try to figure out how to improve it.


DRT-0418 "no pins on routing grid" for a pin that's covered by 6 routing tracks

Describe the bug
[WARNING DRT-0418] Term inst143924/Q1[3] has no pins on routing grid
but the pin is accessible from at least 6 routing tracks

Expected Behavior
no warning
Environment
M1 MacBook Ventura
OpenROAD v2.0-10072-g70db1404a (Aug 29 2023)

To Reproduce
read_lef ispd19_test6/ispd19_test6.input.lef.gz
read_def ispd19_test6/ispd19_test6.input.def.gz
set_thread_count 8
set_routing_layers -signal Metal2-Metal9 -clock Metal2-Metal9
set_global_routing_layer_adjustment Metal2-Metal9 0.5
#global_route -verbose
read_guides ispd19_test6/ispd19_test6.guide
detailed_route -output_drc 5_route_drc.rpt -verbose 1
write_db route.db
exit
Relevant log output
No response
Screenshots
No response
Additional Context
No response

I think the problem is that the warning is issued when the center of a pin doesn't lie on a routing track. The warning doesn't change any behavior of detailed route so it can safely be ignored, but it indicates that the check needs to be refined a little more.


Usage with pug

Is it possible to use this loader with pug-loader ? (formerly jade template engine)

Yes, you can use it with pug, sass and any other preprocessors. We only need to add rules for each file type. (Note that you should use pug-html-loader to use pug with this loader)
I just add an example using such preprocessors.
https://github.com/ktsn/vue-template-loader/tree/master/examples/complex


Some mocked OCI interfaces need to be updated every time OCI makes a backwards-incompatible release

Here is an arbitrary example:
https://github.com/helidon-io/helidon/blob/16ca27b5da8151a1e7241f76993ad75874ac71e3/integrations/oci/metrics/cdi/src/test/java/io/helidon/integrations/oci/metrics/cdi/OciMetricsCdiExtensionTest.java#L186
The Monitoring interface gained a few methods, which is a backwards-incompatible change. Currently, we solve this problem by manually adding "no-op" methods as appropriate.
Instead, for mock implementations of business interfaces such as these, we should use something like java.lang.reflect.Proxy or Mockito or something to implement only the parts we truly need to test.

I think I have an idea on what to do with this, hence moving to Sprint Scope.

Resolved via PR https://github.com/helidon-io/helidon/pull/8474.


feat: updating string literal to use "(double quote) and '(single…

…-quote) for identifiers
Related Issue
#64
Tasks

[X] I have signed the CLA.
[ ] I have written tests and verified that they fail without my change.

A few changes are pending

update test cases
the code currently passes the following expression \"foo. Fixing that.
antlr grammar has a duplicate rule RAW_STRING and STRING. Also some expressions are throwing parsing error via antlr.


Please merge revision faaff39aa9efea958f56e29ba6c9cecfb84d4668 into dev channel

@whesse @mit-mit @kasperl
This fixes how dart2js handles MediaStream#getAudioTracks and MediaStream#getVideoTracks. Before dart2js would crash if you tried to access a property on an element in the track list since it did not think that the element type MediaStreamTrack could be instantiated.
Please merge this commit:
faaff39aa9efea958f56e29ba6c9cecfb84d4668
This merged cleanly

Merged in 1.18.0-dev.4.4


How can I get train data?

I think the code of gpt3 should have a lot of room for improvement. Referring to the human brain, it doesn't need such a large amount of calculation cost, so I plan to try to modify the model so that gpt3 can achieve lightweight and efficient operation. Where can I get such a huge amount of training data?

This information is in the readme


refactor out all instances of path in the code

for instance the code still contains AcmPath

To clarify, all instances of “path” should be replaced with “team”. Related: #470.

Closing as duplicate of #470.


Frozen columns are shifted even if no scrolling is possible

Release number or git hash: v1.1.2
Web browser version and OS: Chrome v76.0.3809.87

Steps to reproduce

Set disableFrozenColumns: false in your LineUpFlags configuration
Set a column to frozen
Make sure your ranking does not scroll horizontally

Observed behavior
Even if the ranking is not shifted (no le-shifted class), the shift per column style is applied if they are frozen.
The frozen columns are implemented by setting left: ...px, which only works because le-shifted is active on the main lineup engine, as the style position: sticky; is applied to the column. If le-shifted is not present, the left: ...px is still applied, offsetting the column even if no shift is required. This causes a hiding of the column behind the others if no scroll is possible/the scrollbar is on the very left.

Additionally, it seems that the margin of 5px which is usually applied to separate each column is not included in the calculation.
Expected behavior

Apply the left: ...px only if le-shifted is present.

Possible Solution
The following lines cause the issue (I think): https://github.com/lineupjs/lineupengine/blob/v2.1.0/src/style/GridStyleManager.ts#L165-L173
When calling this.updateRule, one could prepend the CSS selector for le-shifted, such that this rule only applies if it is present. This would cause the left: ...px to not apply on non-shifted rankings.

For now, I fixed it in my app by applying this style:
.lineup-engine:not(.le-shifted) .lu-header.frozen {
  left: 0px !important;
}

For lineup#v4, the CSS class was renamed:
.lineup-engine:not(.le-shifted) .lu-header.le-frozen {
  left: 0px !important;
}


auto focus on passcode fields

What happened:
Auto focus on passcode entry. User should not have to tap on passcode field.
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:

@kyleleow  reopening due to not working on Android


Fix introspector endpoint in experiments.

Set the default endpoint on import.
Our experiment runs (from run_one_experiment) wasn't setting this.

CC @DavidKorczynski

PR: https://github.com/google/oss-fuzz-gen/pull/180
JOB: https://pantheon.corp.google.com/kubernetes/job/us-central1-c/llm-experiment/default/ofg-pr-180-ochang
REPORT: https://llm-exp.oss-fuzz.com/Result-reports/ofg-pr/2024-03-25-180-ochang-comparison/index.html
BUCKET: https://pantheon.corp.google.com/storage/browser/oss-fuzz-gcb-experiment-run-logs/Result-reports/ofg-pr/2024-03-25-180-ochang-comparison
BUCKET GS: gs://oss-fuzz-gcb-experiment-run-logs/Result-reports/ofg-pr/2024-03-25-180-ochang-comparison


CI-Actions: Security fixes

This PR:

updates the shared actions to run on a current version of node
defines the workflow permission for actions on the top-level to ensure OIDC tokens are issued with the lowest possible permissions

@matmair I believe that this PR has broken the translations workflow
You can see that the workflow has been broken since then
I assume it is due to the permissions change. Should we revert this for this file, or add an additional permission?


Nano server

Features

Syntax highlighting
Code snippets
IntelliSense for cmdlets and more
Rule-based analysis provided by PowerShell Script Analyzer
Go to Definition of cmdlets and variables
Find References of cmdlets and variables
Document and workspace symbol discovery
Run selected selection of PowerShell code using F8
Launch online help for the symbol under the cursor using Ctrl+F1
Local script debugging and basic interactive console support!

Hi @amhnews, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;


configuration question

Hi Brad,
Lately, We've been trying to analyze some SNP panels using bcbio. This is fastq data from a MiSeq tabletop sequencer. However, we're seeing a tremendous slowdown while analyzing these samples.
Do you have any recommendations on settings to speed things up a bit?
We're using the following config:
#bcbio-nextgen v1.0.5
---
#include an experiment name here
fc_name:
upload:
  dir: ../final
globals:
  analysis_regions: RefSeqExomeAndPanels_20171003.bed
  coverage_regions: RefSeqExomeAndPanels_20171003.bed
resources:
  tmp:
    dir: /tmp/bcbio
details:
  - analysis: variant2
    genome_build: hg38
    algorithm:
      aligner: bwa
      save_diskspace: true
      coverage_interval: amplicon
      coverage: coverage_regions
      mark_duplicates: true
      recalibrate: false
      realign: false
      variantcaller: gatk-haplotype
      variant_regions: analysis_regions
      jointcaller: gatk-haplotype-joint
      effects: false
      tools_off:
        - gemini
    # add the path to your files here
    files:

Note the regions file we're using. These are the same regions we use for our exome pipeline. Could this be the cause of the slowdown?
Thanks for the help
Cheers
M

Hi Brad,
Thanks for the reply. The analysis does seem to hang during variant calling, and especially during genotyping. In the time one amplicon run is genotyping I've analyzed about 10 full exomes on the same resources.

There's no region in particular that's taking a lot of time. Particularly, the Haplotyper and Genotyper take forever to process the data.

The BED file I use does contain all full exome regions, perhaps this is the cause of the slowdown.

I am using PCR amplicons, but since we're using these amplicons to confirm we don't have any sample mixups during the prep it's imperative the same methods are used. This means VarDict is not an option.

I'll check with our researchers if they can provide a more selective BED.
Thanks again
M


Get failedInitializations in pnpm

What can be the reason i've get this error when track()? there is no errors)
failedInitializations: ["Mixpanel", "Google Tag Manager"]
my code is (bundle with esbuild and usimg pnpm):
const [analytics] = await AnalyticsBrowser.load({
  writeKey: '...',
});

Maybe the package can't find the path when using pnpm?

This line failed, because i have no script in my HTML markup:
global.window.document.getElementsByTagName('script')[0]

@ballonura interesting! the code assumes that you have at least one script tag on the page. I'm Curios how the Analytics.js code is added to the page to run without adding a script tag?

@ballonura interesting! the code assumes that you have at least one script tag on the page. I'm curios how the Analytics.js code is added to the page to run without adding a script tag?

This cause l an error, no code is added when no script tag.


C56565: Show code for sample on whole page

@BillWagner,
Adding necessary breaklines to avoid localization issues.
Could you help to merge this PR?
Many thanks!

@carolinacmoravia localization should support the same MarkDown as English. Does this mean we'll have to add new lines in between samples for all of our topics?
/cc @nokura


Always repaint in search_git_status

I mentioned this earlier and forgot. When running the search_git_status function, repaint should always be used (like in search_git_log) because something will always be printed.

Thank you so much for another bug fix!


Fix to make shipping address from data passed from PayPal, per spec

Fix to issue #152

Can i forked this in my fork? I update many things from paypal, but this guys doesn't want merge it.

Please do!

Ok, i did.

it’s not if you are trying to see what is sent to paypal, as this view is usually only called by paypal.
-- excuse brevity, sent from my phone.

On 12 Apr 2019, at 07:05, Alexander Gaevsky<EMAIL_ADDRESS>wrote:
@sasha0 requested changes on this pull request.
In paypal/express/views.py:

     payload = urlencode(pairs)

   logger.debug(payload)

I think it's redundant, since we return payload in the response.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Ok, then the log message should be properly formatted.

That’s quite an open suggestion, I can’t find any documentation for django-oscar-paypal or oscar which specified a specific way to present log messages, so what are you expecting by ‘properly’ here? That there is some happy text informing what the string is,  eg logger.debug(Basket #%s - returning postage costs payload = ‘%s’  % basket.id, payload) following the format of other log messages in that view,  or that the whole list of pairs is dumped a more clearly formatted KVP list, or something else in between those extremes?
My personal experience is that I want to see exactly what is sent to PayPal without any additional formatting applied that may be confused as being part of the payload so I can easily extract that from a the log file and post-process as necessary if I need it to be more readable, so what I’ve done works for me.  I’m happy to make changes, but I’d just be blindly making change after change without clearer guidance.

On 12 Apr 2019, at 08:06, Alexander Gaevsky<EMAIL_ADDRESS>wrote:
Ok, then the log message should be properly formatted.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/django-oscar/django-oscar-paypal/pull/153#issuecomment-482463142, or mute the thread https://github.com/notifications/unsubscribe-auth/AAQvb0d4cjYfAKv4_JInUCCM4rdfuHNPks5vgDBngaJpZM4LCZEM.

--
theUsefulArts() { web design + creative technology }
{ web: theusefularts.org http://www.theusefularts.org/, twitter: @theusefularts https://www.twitter.com/@theusefularts, phone: +44 1294 823 600 }

That there is some happy text informing what the string is,  eg logger.debug(Basket #%s - returning postage costs payload = ‘%s’  % basket.id, payload) following the format of other log messages in that view

Yep, this should be enough.

I’m happy to make changes, but I’d just be blindly making change after change without clearer guidance.

Right. I'll be more explicit next time. Thanks for your efforts to update this PR and for the contribution.

Great, thanks very much for the feedback. Have committed that change now. I will make sure any future changes adhere to these kinds of standards, as best as I am able.

On 15 Apr 2019, at 10:12, Alexander Gaevsky<EMAIL_ADDRESS>wrote:
That there is some happy text informing what the string is, eg logger.debug(Basket #%s - returning postage costs payload = ‘%s’ % basket.id, payload) following the format of other log messages in that view
Yep, this should be enough.
I’m happy to make changes, but I’d just be blindly making change after change without clearer guidance.
Right. I'll be more explicit next time. Thanks for your efforts to update this PR and for the contribution.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/django-oscar/django-oscar-paypal/pull/153#issuecomment-483172401, or mute the thread https://github.com/notifications/unsubscribe-auth/AAQvbySl40hIqhMd9dm-oWAG4OKlMrhJks5vhEJygaJpZM4LCZEM.

--
theUsefulArts() { web design + creative technology }
{ web: theusefularts.org http://www.theusefularts.org/, twitter: @theusefularts https://www.twitter.com/@theusefularts, phone: +44 1294 823 600 }

All's good, except the lint warning. Would you have chance to look at it? Otherwise, I'll fix it by myself.
Thanks for your efforts and time to update this PR!

Happy to fix, but not sure about what lint warning you mean - I can’t see anything in my linter… Maybe a GitHub PR noob issue with me :-)

On 1 May 2019, at 10:37, Alexander Gaevsky<EMAIL_ADDRESS>wrote:
All's good, except the lint warning. Would you have chance to look at it? Otherwise, I'll fix it by myself.
Thanks for your efforts and time to update this PR!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/django-oscar/django-oscar-paypal/pull/153#issuecomment-488244620, or mute the thread https://github.com/notifications/unsubscribe-auth/AACC63532GHEP2WMBT6BPULPTFQHPANCNFSM4CYJSEGA.

--
theUsefulArts() { web design + creative technology }
{ web: theusefularts.org http://www.theusefularts.org/, twitter: @theusefularts https://www.twitter.com/@theusefularts, phone: +44 1294 823 600 }

E.g.:
flake8 paypal tests setup.py
paypal/express/views.py:408:9: F841 local variable 'country_code' is assigned to but never used
paypal/express/views.py:430:5: E303 too many blank lines (3)
paypal/express/views.py:485:69: W291 trailing whitespace

Great, thanks. I was relying on my Editors linter.

On 1 May 2019, at 11:35, Alexander Gaevsky<EMAIL_ADDRESS>wrote:
E.g.:
flake8 paypal tests setup.py
paypal/express/views.py:408:9: F841 local variable 'country_code' is assigned to but never used
paypal/express/views.py:430:5: E303 too many blank lines (3)
paypal/express/views.py:485:69: W291 trailing whitespace
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/django-oscar/django-oscar-paypal/pull/153#issuecomment-488253129, or mute the thread https://github.com/notifications/unsubscribe-auth/AACC636OVAVYTSN73MHZBLLPTFW55ANCNFSM4CYJSEGA.

--
theUsefulArts() { web design + creative technology }
{ web: theusefularts.org http://www.theusefularts.org/, twitter: @theusefularts https://www.twitter.com/@theusefularts, phone: +44 1294 823 600 }

Thanks. Done and pushed to fork.

On 1 May 2019, at 11:35, Alexander Gaevsky<EMAIL_ADDRESS>wrote:
E.g.:
flake8 paypal tests setup.py
paypal/express/views.py:408:9: F841 local variable 'country_code' is assigned to but never used
paypal/express/views.py:430:5: E303 too many blank lines (3)
paypal/express/views.py:485:69: W291 trailing whitespace
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/django-oscar/django-oscar-paypal/pull/153#issuecomment-488253129, or mute the thread https://github.com/notifications/unsubscribe-auth/AACC636OVAVYTSN73MHZBLLPTFW55ANCNFSM4CYJSEGA.

--
theUsefulArts() { web design + creative technology }
{ web: theusefularts.org http://www.theusefularts.org/, twitter: @theusefularts https://www.twitter.com/@theusefularts, phone: +44 1294 823 600 }

Alright, we're good now.


Weird javadoc redirecting

https://maven.fabricmc.net/docs/yarn-21w06a+build.5 redirects to
https://maven.fabricmc.net/yarn-21w06a+build.5/
while
https://maven.fabricmc.net/docs/yarn-21w06a+build.5/ (requires the / slash in the end)
works. same for http vs https.
same for the loader javadoc.
https://maven.fabricmc.net/docs/fabric-loader-0.11.1
redirects to
https://maven.fabricmc.net/fabric-loader-0.11.1/
https://maven.fabricmc.net/docs/fabric-loader-0.11.1/ works.
This is an interesting bug. Imo worth fixing as otherwise this is confusing. @modmuss50 since he is the admin of his maven.

@modmuss50 mind look into this weird phenomenom?

Humm, that is odd. Not sure if its an issue with the nginx proxy, or the service behind it


Drop Ruby 1.9.2 support, add testing on 2.3/2.4

Pull Request Checklist
RE https://github.com/sensu-plugins/sensu-plugins-feature-requests/issues/27
General

[x] Update Changelog following the conventions laid out on Keep A Changelog

[x] RuboCop passes

[x] Existing tests pass

Purpose
Drop Ruby 1.9.3 support as it no longer supported, add testing on 2.3/2.4
Known Compatablity Issues
Breaking change for users on Ruby 1.9.3

released: https://rubygems.org/gems/sensu-plugins-openstack/versions/1.0.0


mx.sym.constant

For symbol constructions, there is var, zeros, and ones. Can we also get a constant one that receives a list/numpy/mx.ndarray and makes a constant symbol out of it? Working it in via variables is a bit cumbersome, particularly because the Constant initialization doesn't work for any variable with a name not ending with the string 'weight'.

yeah that's something I wanted to do but didn't have time for.
Basically you serialize an ndarray into string and pass it as argument to an op. The op then decode it and copy it to output everytime.

I'm interested in this. @piiswrong
Can we just create a const op?

my friend has implemented a simple version of constant initializer @panzheyi

I implemented a possible solution.
A constant can be regarded as a variable which doesn't get gradient.
So you can firstly create a variable, and then block its gradient.
Here's my code:
import mxnet as mx
import numpy as np
@mx.init.register
class MyConstant(mx.init.Initializer):
"""Initializes the weights to a scalar value.
Parameters
----------
value : float
    Fill value.
"""
def __init__(self, value):
    super(MyConstant, self).__init__(value=value)
    self.value = value

def _init_weight(self, _, arr):
    arr[:] = mx.nd.array(self.value)

batch_size = 10
const_arr = np.ones((5, 5)).tolist()
a = mx.sym.Variable('a', shape = (5, 5), init = MyConstant(value = const_arr))
a = mx.sym.BlockGrad(a)
data = mx.sym.Variable('data')
loss = mx.sym.MakeLoss(mx.sym.broadcast_add(a, data))
mod = mx.mod.Module(loss, data_names = ['data'], label_names = [])
mod.bind(data_shapes = [('data', (batch_size, 5, 5)),])
mod.init_params(initializer = mx.init.Uniform())
mod.init_optimizer(optimizer = 'sgd', optimizer_params = (('learning_rate', 0.005),))
a = mx.nd.ones((5, 5))
data = np.ones((1000, 5, 5))
dataiter = mx.io.NDArrayIter(data = {'data': data}, batch_size = batch_size)
dataiter.reset()
for batch_id, databatch in enumerate(dataiter):
mod.forward_backward(databatch)
mod.update()
print mod.get_outputs()[0].asnumpy()

I implemented a possible solution.
A constant can be regarded as a variable which doesn't get gradient.
So you can firstly create a variable, and then block its gradient.
Here's my code:
'''python
import mxnet as mx
import numpy as np
@mx.init.register
class MyConstant(mx.init.Initializer):
"""Initializes the weights to a scalar value.
Parameters
----------
value : float
    Fill value.
"""
def __init__(self, value):
    super(MyConstant, self).__init__(value=value)
    self.value = value

def _init_weight(self, _, arr):
    arr[:] = mx.nd.array(self.value)

batch_size = 10
const_arr = np.ones((5, 5)).tolist()
a = mx.sym.Variable('a', shape = (5, 5), init = MyConstant(value = const_arr))
a = mx.sym.BlockGrad(a)
data = mx.sym.Variable('data')
loss = mx.sym.MakeLoss(mx.sym.broadcast_add(a, data))
mod = mx.mod.Module(loss, data_names = ['data'], label_names = [])
mod.bind(data_shapes = [('data', (batch_size, 5, 5)),])
mod.init_params(initializer = mx.init.Uniform())
mod.init_optimizer(optimizer = 'sgd', optimizer_params = (('learning_rate', 0.005),))
a = mx.nd.ones((5, 5))
data = np.ones((1000, 5, 5))
dataiter = mx.io.NDArrayIter(data = {'data': data}, batch_size = batch_size)
dataiter.reset()
for batch_id, databatch in enumerate(dataiter):
mod.forward_backward(databatch)
mod.update()
print mod.get_outputs()[0].asnumpy()
'''

+1

+1 good point

I have been searching for this solution for hours. Finally got one working :)

any official implementation?

I've looked everywhere for a proper solution, didn't like the idea of using the special constant initializer class since there should not be a difference between scalar constant to any shape tensor. Finally what worked for me is the following:
my_var = mx.symbol.Variable('myconst',shape=(),init=mx.init.Constant(value=(a).tolist())
if you also want to block gradients simply specify the variable name as constant variable when constructing the module.

@piiswrong Is there a plan to implement this simple feature?

@piiswrong a short example will be very helpful.


jax 0.4.13 does not provide the extra 'cuda12

error "jax 0.4.13 does not provide the extra 'cuda12'" occurred when I tried "dial-mpc --list-examples" or "dial-mpc --example unitree_h1_jog".
Should i change the version of jax?

What platform are you using? You might find this issue helpful!

My platform is ubuntu20.04.
now I have changed Cuda12.6+cudnn8.8+jax0.4.7
but still get the error "pkg_resources.UnknownExtra: jax 0.4.7 has no such extra feature 'cuda12' " when I tried "dial-mpc --list-examples"

Hmm we can try reproducing this on Ubuntu 20.04.
Your cudnn and jax version does seem a bit outdated. What python version are you using?
Meanwhile if possible can you try a conda environment which a recent python version (>= 3.10) and jax version (we have 0.4.33)?
Feel like this is more of a python install issue.

thanks for your reply
after changing my python3 version from 3.8 to 3.10, the origin problem has been solved .
I believe there's something wrong with the ubuntu20.04+python3.8.
By the way, it's hard to change the python3 version for me because i am not familiar with it. MANY MANY MANY issues happened !! So i strongly advice that other coders should follow the recommended environment !!!

Thanks for the update!


[hi] Localize /en/docs/reference/glossary/cluster-operator.md

[hi] Localize /en/docs/reference/glossary/cluster-operator.md
Instructions

Review a few example of localized glossary entries.
In the examples, notice which elements are localized and which ones are not.

Localizing the frontmatter
In the frontmatter, that is, between the --- markers that enclose the first dozen lines:

Localize the title and short_description values.
Do not localize the id, aka, tags values. We will systematically localize
those later on in a separate project.

Localizing the content body
Below the frontmatter, in the content body:

Localize almost everything.
If you encounter a glossary shortcode, for example, {{< glossary_tooltip  text="containers" term_id="container" >}}, localize the text value. But do not localize the term_id value.

Procedure:

In this issue, assign this issue to yourself by commenting: /assign <your_github_id>

Open a PR like this Eg:- https://github.com/kubernetes/website/pull/35942.

In the PR, select all the lines in the file and click + to comment.

In the comment, click the +/- suggestion icon (screenshot 2).

In the suggestion, localize the content to Hindi.

IMPORTANT: Click the Start a review button.

IMPORTANT: Click Review changes, Comment, and Submit review. Screenshot 3

Note:- By completing these steps, your GitHub profile will give you credit for performing a code review!

As follow-up steps, I will do the following:

Apply your suggested localization to the PR.
Ask a reviewer to review the localized content.
Update the content as needed.
Get the PR approved and merged.

This feature issue is part of the following project:
https://github.com/kubernetes/website/issues/35600

/help wanted
/good first issue

/help-wanted
/good-first-issue

/triage accepted

/assign

/language-hi

/language/hi

/language hi

/assign i-dipanshu

Hey @bishal7679 ,
I searched in the repository and found the file already exits refer

Can u please assign me on this?
i wanna work on it.

This issue has been solved, please consider closing.


[Feature] Keyboard shortcut to open the extension popup

maybe something like ALT+SHIFT+C ?

done


[SR-2660][Driver] Pass .swiftmodule to linker

The following two invocations of swiftc behave differently, despite their only difference being the -g option:
swiftc foo.swift bar.o baz.swiftmodule -o foo
swiftc -g foo.swift bar.o baz.swiftmodule -o foo

The first invocation compiles foo.swift, links it with bar.o, and passes the AST information from baz.swiftmodule to the linker. The second invocation results in the following error:
<unknown>:0: error: cannot load module 'baz' as 'foo'

The source of the problem is that the driver determines whether to generate a module based on the debug info level that has been requested, and merges all .swiftmodule inputs if a module is being generated.
Modify this behavior to instead pass .swiftmodule inputs directly to the linker if our output is to be linked. This results in both the swiftc and the swiftc -g invocations above succeeding.
Test Plan:

utils/build-script --test passes.
After cloning https://github.com/modocache/SR-2660 and modifying its build-driver.sh to point at the local Swift source build directory, running build-driver.sh succeeds, and lldb is able to print descriptions with accurate debug info.

Addresses one aspect of SR-2660.

@swift-ci please smoke test

Sorry for taking so long to update this one, @jrose-apple. I hope I addressed your comments, thanks for the review!

@swift-ci please smoke test

Looks like @swift-ci doesn't like me today... 😅

Great, thanks for the reviews!


Deprecate @postlight/mercury-parser in npmjs

Deprecate @postlight/mercury-parser in npmjs to point users to use the new @postlight/parser package.
Run something along the lines of:
npm deprecate @postlight/mercury-parser@"<= 2.2.1" "This project has been renamed to @postlight/parser. Install using @postlight/parser instead."

Deprecating the old package name relates to #688, changing the project name from Mercury Parser to Postlight Parser.
Npm package to deprecate: Mercury Parser

Package has been deprecated in NPM and will notify when a user installs it


Added @tryghost/metrics-server as a dependency

no issue

I apparently never added @tryghost/metrics-server as a dependency to ghost/core/package.json. It worked in most cases as a 'Ghost dependency' — yarn installs all node_modules in a flat structure, so even though it wasn't a dependency in package.json, it still resolved to the correct package, as long as the typescript packages were all built first.
This passed CI because we explicitly run ts:build on all packages before running tests, and it worked in production because we build the TS packages as part of the docker build. However, when trying to run tests locally, it would sometimes fail unless you explicitly ran nx run-many -t build:ts at the top level before running the tests.
Adding it as a dependency in package.json fixes this problem.

Somehow branch got messed up, closing in favor of https://github.com/TryGhost/Ghost/pull/21330


miner module, siad and siac miner apis, siac seed command

reworked this API a bit.  This means we don't need stateful /start and /stop endpoints.
I also switched to a single siac mine command, which can be used to mine indefinitely (stopping when killed) or to mine a specific number of blocks. You can also specify the address that will receive the block reward.
Haven't tested this; can you try it out? (I'm starting to feel the need for an API test suite...)

@lukechampine I gave this a try and it works great, having it as a siac command with limit and address is very convenient. Yep a test suite would be great, if we get that set up I can help add good test cases as I consume new endpoints during UI development.


circleci: run integration tests on multiple jobs

... in parallel on multiple instances.

Total execution time down to 8min: https://app.circleci.com/pipelines/github/Netflix/titus-control-plane/1141/workflows/b7f845ee-8145-49be-b310-1a4985533786


[W6.1h][F11-B1] Law Keng Ying

Implementation for interface

@kengying Some comments added. Please close the PR after reading comments.


fix(ui): pf5 visual glitches

Description

A change in where the global default font-family is applied results in our Tailwind font-family overriding the Red Hat font.
A global checkbox size override relied on CSS vars that are no longer valid (at least at the scope we were using them from). Removing the rules entirely fixes the issue.
The color prop passed to Icon components previously was set as the svg's fill attribute. Now, that attribute has a value of currentColor and a wrapper element with the style: color: var(--pf-v5-c-icon__content--Color, inherit);
The codemod changed a helperText prop into a passed child component, but by default passes the new child first. This results in bad form layout. Moving the child to the last position resolves this in most cases.
An extra padding override was causing layout shift of some form components. The newest version of PF makes this additional padding unnecessary.
Our Tailwind config sets all svgs to display: block, which is almost never what we want in PatternFly components. This was causing an issue in some dropdowns, but instead of adding yet another exception, I changed the default for all svg elements on PF pages.

Known Residue

There are many other instances where icons are not the correct color.
There are likely many other instances where error messages are displayed above input components
There are mostly minor issues with spacing and alignment throughout the UI (less of an issue, but makes the app look bad)

Checklist

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

If any of these don't apply, please comment below.
Testing Performed
Change 1
Before:

After:

Change 2
Before:

After:

Change 3
Before:

After:

Change 4
Before:

After:

Change 5
Before:

After:

Change 6
Before:

After:

style-check
ui/apps/platform/src/Components/ContainerSecretsInfo.tsx
40:45  error  Do not use Array index in keys  react/no-array-index-key
ui/apps/platform/src/Components/PatternFly/DayPickerDropdown.tsx
100:24  error  '_e' is defined but never used  @typescript-eslint/no-unused-vars
100:28  error  'v' is defined but never used   @typescript-eslint/no-unused-vars
ui/apps/platform/src/Containers/NetworkGraph/common/NetworkPolicies.tsx
135:5  error  Unexpected console statement  no-console

unit-tests needs update to snapshots for test files, which you might have done already on PatternFly 5 branch?

src/Components/TimelineGraph/EventsGraph/ClusteredEventMarker/ClusteredEventMarker.test.tsx
src/Components/TimelineGraph/EventsGraph/EventMarker/EventMarker.test.js
src/Components/ObjectDescriptionList.test.js


feat: net5 and net6 support

What this PR does / why it's submitted / why we need it:

added support for all Umbraco 9 versions (>=9.0.0) - more info in csproj
added net5 and net6 support
updated workflows: generic approach of project build process, net5 and net6 build process support, optimizations

Example of NuGet output:

Checklist:

[x] My PR has a descriptive title (not a vague title like Needed this)
[x] Title and all the commits follow commitlint guidelines
[x] My PR targets the develop branch or appropriate release branch
[x] My PR provides test coverage for my change (where applicable)
[ ] I've placed the link to the corresponding Trello card or issue which this PR fixes/address below ⤵

Fixes: ### ISSUE_OR_TRELLO_LINK ###

Build process can fail as on master and dev there are old workflows for NET Framework. The latest works.

For now Pull request validation workflow is not working, we need to add support for net5 and net6 and DotLiquid loggers.

For now Pull request validation workflow is not working, we need to add support for net5 and net6 and DotLiquid loggers.

done ✅
Workflow fail to comment due to permissions (fork based probably).


feat: add ouch-org/ouch

ouch-org/ouch: Painless compression and decompression in the terminal
$ aqua g -i ouch-org/ouch

How to confirm if this package works well
Reviewers aren't necessarily familiar with this package, so please describe how to confirm if this package works well.
Please confirm if this package works well yourself as much as possible.
Command and output
$ ouch --help
A command-line utility for easily compressing and decompressing files and directories.

Supported formats: tar, zip, gz, xz/lzma, bz/bz2, lz4, sz, zst.

Repository: https://github.com/ouch-org/ouch

Usage: ouch [OPTIONS] <COMMAND>

Commands:
  compress    Compress one or more files into one output file [aliases: c]
  decompress  Decompresses one or more files, optionally into another folder [aliases: d]
  list        List contents of an archive [aliases: l, ls]
  help        Print this message or the help of the given subcommand(s)

Options:
  -y, --yes
          Skip [Y/n] questions positively

  -n, --no
          Skip [Y/n] questions negatively

  -A, --accessible
          Activate accessibility mode, reducing visual noise

          [env: ACCESSIBLE=]

  -H, --hidden
          Ignores hidden files

  -q, --quiet
          Silences output

  -g, --gitignore
          Ignores files matched by git's ignore files

  -f, --format <FORMAT>
          Specify the format of the archive

  -h, --help
          Print help (see a summary with '-h')

  -V, --version
          Print version

I'm not sure how to fix this missing extension on Windows:
ERRO[0000] install the package                           aqua_version=2.16.0 env=windows/amd64 error="the asset isn't found: ouch-x86_64-pc-windows-msvc.exe" package_name=ouch-org/ouch package_version=0.2.0 program=aqua registry=standard

Thank you always!

0.2.0

https://aquaproj.github.io/docs/reference/registry-config/complete-windows-ext/
complete_windows_ext: false

v4.73.0 is out 🎉
https://github.com/aquaproj/aqua-registry/releases/tag/v4.73.0


It may make sense to upstream your spec into https://github.com/kaitai-io/kaitai_struct_formats/

Hi,
sure I will consider this in a moment.
For now, most of structures are being RE'ed so the contents are not reliable enough :)
I let this issue open to keep track of the action.


Chapter2 - select and train a model

When I try to run the code on my pc, I found that the rmse of linear model and its standard deviation under cross validation are both rather small (about 1e-10).
And for the model of decision tree, its standard deviation  is about 70.
For these 2 models, the performance is much better than it did on the book. Only for the model of random forest, it has a similar performance.
I'm wondering if anyone meets the same issue and curious about why this happened.


Azure Notification HUB Tag having issue

I want to use the email ID<EMAIL_ADDRESS>as Tag while registering Azure Notification Hub through my Xamarin iOS Project.
I found that notification hub registration is working fine and sending push notification successfully from Azure Notification Hub Test send console also but it is not receiving on my iOS device at any state of the Application. As per my observation if the tag ( email id) having more than two period/dot(.) characters it will not receive push notification to your iOS device. Is it a limitation or a known issue  ?
It is working fine if I'm setting the Tag as<EMAIL_ADDRESS>Environment
NuGet Package - Xamarin.Azure.NotificationHubs.iOS (2.0.4)
Xamarin.iOS
Version: <IP_ADDRESS> (Visual Studio Community)
Operating System
Mac OS X 10.15.4
Sample code is given below
string UserEmailID =<EMAIL_ADDRESS>                var tags = new NSSet(UserEmailID);
                    Hub.RegisterNative(deviceToken, tags, err =>
                    {
                        if (err != null)
                        {
                            Console.WriteLine("Error: " + err.DebugDescription);
                        }
                        else
                        {
                            Console.WriteLine("Success");
                        }
                    });

@prsreejith Thanks for the report, we'll investigate.

@prsreejith after working through this, we are unable to reproduce this bug.  Does this happen consistently?

Closing as per being inactive


Odd behaviour when taking second order derivatives through Interpolator.

This bug seems to be related to the nans returned from CircularAperture. Using config.update("jax_debug_nans", True) shows that both get nans when calling np.angle(wavefront).

So it appears that interpolating the magnitude and phase in a pupil plane will return second order nan-gradients, but will work fine when interpolating the real and imaginary components. However in a focal plane, interpolating the real and imaginary components gives nonsense answers (it seems that the resulting PSFs get interference from the interpolation, but this effect disappears for integer down-sampling interpolation). I assume this effect goes away if interpolating the PSF rather than the wavefront. For now the Interpolator class have been deprecated and split into InterpReIm and InterpAmPh for interpolations in different planes.


Moving cursor up and down, but it has creeping backward

Hey guys. Thank you for this awesome project.
When i moving cursor up and down, it has creeping backward.

Cursor movement is animated by default. You can disable the movement animation by typing :NvuiCursorAnimationDuration 0 into the command line.


[Datasets Sharepoint Issue] The file size exceeds the allowed limit.

Hi @LiheYoung,
I received an error when trying to download the ADE20K_Synthetic.zip file from OneDrive SharePoint.
The error message stated:

The file size exceeds the allowed limit.
CorrelationId: b3dc19da-f990-432e-95f5-5e3e26e50e35,
UTC DateTime: 11/08/2023 13:35:48

Would you be able to split the dataset into smaller collections and re-upload them?
Many thanks

Hi, thank you for your feedback. I will re-upload them after Nov 17 (CVPR deadline). Thank you for understanding.

Hi, I re-checked it and found the file size is not very large (~30G). It can be downloaded with the Chrome browser.


Add local & remote url sniffing to the kit

When using the IV & 2FA prototypes you send a link to the service so that you can complete 2FA & IV then come back to your prototype at a set point in the journey.
The Child Benefit prototype has some javascript that works out the domain (usually localhost or Heroku) of the referring URL and sends you back to the referring domain. You can then test locally and deploy without making any changes.
@feedmypixel @rpowis what do you think about including this?
@adamfenwick Could you point at the code in the Child benefit repo, please?

@gavinwye yes we have the success, failure url work created in the IV prototype earmarked to come in to the HMRC prototype kit in issue https://github.com/hmrc/hmrc_prototype_kit/issues/13
Specifics or working out of urls shouldn't be in the generic implementation. It sounds like you need a variable in one place (app.locals) which sends in the url and failUrl dependant on process.env.NODE_ENV equalling production or development. Then you explicitly set those in one place in your app and it propagates through your app (making it simple to change).
If the above isn't the case I'd like to fully understand the task you are trying to solve. Something for the #community-prototype room

To be honest, I just create a variable that's set at the start of routes.js and do it that way by adding it in the ULR as a nunjucks variable - perhaps that's the same way you do it?


Feature Request: Documentation for installing Zulip on Apple Silicone Chip/M1/M2 Architecture

In the past few months when trying to use Zulip through the desktop application, I realized installation was not supported for the M1/Apple Silicone Chips. I attempted to install Zulip through a Linux VM but was unable to until I found a program that was able to emulate x86 architecture. In the Zulip documentation I have read that pull requests are submitted through feature branches of open issues.
Although the browser version may be the best substitute, I created a video and written tutorial on how to install the desktop application I feel would be helpful for those looking to attempt to install to this new architecture.

@andersk FYI; I thought the Zulip desktop app did work on Apple M1 already; see https://github.com/zulip/zulip-desktop/issues/1076.

@andersk FYI; I thought the Zulip desktop app did work on Apple M1 already; see zulip/zulip-desktop#1076.

Ah I must have missed it then and tried to install the non silicon build. I see the option now on the mac installation page. Though now I am wondering if that option should have its own section or formatted equally with the other macOS download option as it was easy to miss (at least imo).

FYI, Apple silicon is different from silicon and very different from silicone.
Both the x86 build and the Apple silicon build work fine on M1. macOS will automatically emulate the former through Rosetta, and the latter will run natively. The automatic upgrader will switch you to the native build on supported hardware.


1.2 Analog Movement w/ PC Controllers

I'm working on analog movement based on the C stick support @fgsfdsfgs  added.
Creating this to track work tasks so I don't spam w/ emails:
Branch: https://github.com/NeonNyan/perfect-dark-fgsfdsfgs/tree/1.2_analog
Status: Walking / grabbing / biking working as expected.
Tasks:

[x] implement leaning
[x] implement  zooming
[x] implement menu movement
[x] implement camspy stick movement
[ ] implement slayer rocket airbrake
[x] test menu sliders

[ ] #42 : it makes more sense to implement this directly rather than implement aim + cstick crouching

[ ] cleanup ints that definitely should not have been changed


Cannot import systemjs-hot-reloader (weakee/Emitter is undefined)

With 0.5.1 I get:
Potentially unhandled rejection [2] TypeError: Error evaluating<EMAIL_ADDRESS>Super expression must either be null or a function, not undefined
    at exports.default (http://localhost/jspm_packages/npm/babel-runtime@5.8.20/helpers/inherits.js:7:11)
    at eval (http://localhost/jspm_packages/github/capaj/systemjs-hot-reloader@0.5.1/hot-reloader.js!eval:36:9)
    at execute (http://localhost/jspm_packages/github/capaj/systemjs-hot-reloader@0.5.1/hot-reloader.js!eval:272:9)
    at t (http://localhost/jspm_packages/es6-module-loader.js:7:19798)
    at v (http://localhost/jspm_packages/es6-module-loader.js:7:20180)
    at v (http://localhost/jspm_packages/es6-module-loader.js:7:20027)
    at u (http://localhost/jspm_packages/es6-module-loader.js:7:19856)
    at s (http://localhost/jspm_packages/es6-module-loader.js:7:19737)
    at http://localhost/jspm_packages/es6-module-loader.js:7:22064
    at O (http://localhost/jspm_packages/es6-module-loader.js:7:7439)

I found the super() TypeError is coming from
class HotReloader extends Emitter {

Emitter is undefined. As far as I can tell, I'm getting this error with a canonical setup (based on what's in the README and some of the examples). Is there some problem with weakee, or maybe some setup step that isn't in the docs?

Forgot to mention that this happens when trying to import the hot reloader:
<script src="/jspm_packages/system.js"></script>
<script src="/jspm-config.js"></script>
<script>
  System.trace = true;
  System.import('capaj/systemjs-hot-reloader');
</script>

@oinksoft what version of JSPM are u running?

The weakee package is installed? (This should happen automatically with installing the systemjs-hot-reloader). You can check if a /jspm_packages/npm/weakee... folder exists.

@capaj I'm running JSPM 0.15.7.
@peteruithoven The weakee package was installed, it shows up in the right places in jspm-config and can be imported with its unaliased name. It seems like only hot-reloader.js gets undefined for this module.

@peteruithoven If it weren't installed correctly, I believe there would be a 404 on the script URL and an "Error loading" message from SystemJS. Earlier I was able to see console.log statements that I added to weakee.js, so this file is definitely being loaded.

@oinksoft try updating to jspm 0.16.15 or newer and let me know if that fixes it.
I think I have had an issue from one guy with weakee problems who solved it like this.

@capaj Interesting, I had the same error with JSPM 0.16.18 but it seems to be working with 0.6.15 so I'll use that.

@oinksoft I just did a jspm install into clean repo and it worked even with 0.16.18 JSPM.
Well, anyway, closing this, because this is not an issue I can solve.


Add support for automatically translating Rust deprecations to GraphQL deprecations

Conceptually similar to what we do with descriptions (https://github.com/graphql-rust/juniper/issues/194), we should automatically translate any #[deprecated] field in a struct using the derives into GraphQL (https://graphql-rust.github.io/types/objects/defining_objects.html#deprecating-fields).

For anyone interested into this: wundergraph already does this, so feel free to "copy" the implementation from there.

This was merged in #269.

@willglynn thanks for the nudge. ;)

Is there a way to make this work for input object fields as well, a la https://github.com/facebook/graphql/pull/525?

Doesn't look like it is currently supported as it isn't in the spec. Feel free to file an enhancement (or a PR!)


[Draft] Initial work on JsonData property name casing

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Core.Experimental


Some tokens change sizes when loading into a map for the first time.

This is an issue with the most recent update to this mod. Any token with an effect placed on it with an ATL attribute will override the scale setting of the token, even if the scale it not noted in the module. It seems to revert the token to its default size in this instance.
For instance, a token with the Active Effect: ATL.dimLight, which as far as I know should have nothing to do with the scaling of the token, will revert the token when making the effect active/inactive, or even when loading into a map with that token on it. There are numerous reasons why I might want to temporarily change a token's scale, and this module now hinders the ability to do that. It also makes it conflict with a variety of other modules out there now.
Hopefully you can disable whatever is causing this to happen. Thanks!

This is (oddly) intentional. ATE now uses the prototype token as a default baseline to compare against. This means that it will apply those settings + any ATE effects. Im sorry if that change is undesirable for you, but there isnt any real way to avoid this issue that I can think of.
You are welcome to roll back to the previous version of ATE if that worked better for your use-case

So I definitely understand the rationale for this, but there seems to be a variety of what I assume are unintentional side-effects caused by this change. I think the biggest way I can define it is as a lack of consistency/control for a DM over a scene if they are using this module. This goes beyond incompatibilities with other mods. For instance, something as simple as token rotation, will now automatically get reset to the default values every time you load into a scene. So if I've set a group of characters up to be, say, sitting in chairs facing in various directions with respect to where the chairs are pointing, upon loading into the map, all those tokens, if they have any ATE effects present, will now drop to their default rotation.
If I want to change a token's disposition temporarily, just for the map they're on, perhaps to hide the fact that they're an enemy, that will automatically be reset as well. Tint Color, Z-Index, Token Opacity, Vision Settings, Lighting Settings, even Display Bar settings all get automatically reset, and none of this has anything to do with incompatibility with other modules, it just makes the DM's control over tokens and scenes severely handicapped.
If there's no way to avoid this issue then fair enough, but I think this introduces a lot of clunkiness that would be undesirable for a lot of people, and maybe deserves a second look at. Until then, I'm happy to use the previous version for as long as it's compatible with the most recent versions of Foundry. I appreciate what you've done either way!

I too believe this is change in behavior, forcibly changing everything back to the token prototype, was a bad idea. I have tokens that use limited cone lights sources (toggled via ATE) that automatically reset position.
It should change only aspects specified in the ATE-bearing effect, and not do anything else. Not to be too pointed, but it is no longer "Active Token Effects" it is "Smash The Token Back In The Box And Only Do What This Module Says".

I'm having a similar problem where a new preset or a copy of a base preset would also change a large token back to base size when the effect was turned on. A workaround was to make another copy of the preset with the large size though it's a bother to update multiple presets if I want to change something.


Default Sorting

How do i choose which row the gird defaults its sorting on? and once this is done how can i (in code) change the sorting based on a row name?
Thanks,
Andy.

Do you mean column sorting?

@lodine,
I have a similar issue where my grid has 4k+ rows and on clicking on any column, it goes and fetches new data again. Is there a way to just sort rendered rows?
Also is there a way to have a custom sort function? I read sort method, but it expects an object and not a function.

@RajeshDixitCF Would you mind making a new issue for the above?  I would like to be able to track it separately of this issue as they are unrelated


Handle Task commits (and Task/Project PR events) coming directly from GitHub.

Cute animal pic
Because everyone likes pictures of animals.

Link to Trello card (if applicable)
If this PR relates to a Trello card, don't forget to reference this PR on the card as well.
https://trello.com/c/mykQfEsb
Description
The commit messages say what you did; this should explain why and how.

[x] Description is in Trello card

Steps to test/reproduce
Please explain how to best reproduce the issue and/or test the changes locally.

Enable GH webhooks locally (see CONTRIBUTING.md)
Create a new Task.
Create an Org on that task to trigger branch-creation.
Go to the task branch (on https://github.com/oddbird/Metecho-Test/) and make a new commit directly from GitHub.
See the commit appear in Metecho, along with the ability to Submit for Review.
Create a PR for that branch, again directly from GitHub.
See the "Submit for Review" btn disappear, and the "View Pull Request" button appear.

Impacted areas of website
Provide URLs and/or a description of where to look.

Task detail page

Reviewer tasks:

[ ] Pulled branch and ran code locally (or viewed on preview deployment, if applicable)
[ ] Reproduced the issue to review in the browser
[ ] Reviewed code
[ ] Updated Trello card (if applicable)

@wlonk I'm sure this could use some cleanup (and definitely some tests), but I'd love your review on it first. ...And then your code cleanup and tests, if applicable! 😄
The basic goal is to allow Metecho to adjust has_unmerged_commits and pr_number accordingly when events come in via the GH webhook.

 Chore: Confirm that commits work even if coming directly from GH (not via ME).

Tests will be incoming.


Options binding from configuration should not have conditionally compiled code

Update: Covert.ChangeType() is now available in System.Runtime.Extensions and we can remove the conditional compilation.
Currently the whole body of OptionsServices.ReadProperties(object, IConfiguration) is in an #if and won't do anything if the functionality of Convert.ChangeType() presumably isn't available in the target platform. I believe we should decide more explicitly what we want to do with this, e.g.:

Throw an exception
Make sure the needed functionality is made available in CoreCLR and use it

I think @yishaigalatzer wanted to kill that whole type conversion feature for now and just do strings.
Do we have any components that truly rely on the type conversion behavior of Options / config? Does EF make use of it?

Updating title because Covert.ChangeType() is now available in System.Runtime.Extensions and we can remove the ifdefs.

@Eilon in general our Options classes are optimized for code configuration and they are not just for pulling stuff from configuration, so not sure it is a good idea to kill the type conversion. Was there any scenario in which it was dangerous or didn't work?

For code-based config you don't need type conversion right?
I think the only danger in the current state of the code is that it ignores stuff instead of either working or at least failing. I'm fine if we can do more basic conversions successfully, but then we should fail if we can't. That's probably a decent compromise and easy to implement, right?

Yep, we should not swallow exceptions from Covert.ChangType(). Frameworks that need more complex conversions/parsing should implement their own logic. @haok sure has more context on why the empty catch ended there.
On the other hand is by design that we skip properties not specified on configuration.

Skipping missing values is certainly fine (if I understood correctly). I mean ultimately we'll add new options and wouldn't expect "old" configs to start to fail.

I can take a look at this again, it was added very early on and since nothing was really using it, I didn't flesh the type conversion logic out much at all

My key feedback here was that the data should start with some well known subkey otherwise random collisions will start happenning

The user passes in an optional sub-key when they use config, no?

Yeah the caller is responsible for selecting the right parent config for the options to bind against, i.e.
services.ConfigureMvc(config.GetSubKey("mvc"))

Yeah I think that's fine by me. Forcing a parent key was discussed a long time ago and was determined to be far too difficult to use. If we had a default we'd need a way to un-set it, we'd need a way to document it, we'd still have collisions (though rare), and we'd still have to support custom sub-keys anyway.

Tangent - I wonder if GetSubKey should be GetKey?

The fact that you pass configuration rather than something like key or configuration node, confused me I want even aware of the pattern. From the shape of the API it looks like you just pass the configuration object proper. Consider making it more obvious, such that the API is self explanatory

Well the configuration itself is a node so it's just the nature of the new iconfiguration that makes it flexible.   Config.GetSubkey returns another iconfiguration representing the root of the subtree.


google login doesn't work if no session currently going on

if no session, should prompt some sort of sign in with google option, where you select email address ...

Getting around the popup blocker would require somehow setting the prompt not to open a popup, but to redirect the current page. I don't have an option with the current implementation of vue3-google-login. Would have to implement directly from Google's JS api for this stuff.

closing as I'm not sure there is currently an issue. In general, would be nice to redo auth, perhaps with a (free) vendor.


[SPARK-33396][SQL] Spark SQL CLI prints appliction id when process file

What changes were proposed in this pull request?
Modify SparkSQLCLIDriver.scala to move ahead calling the cli.printMasterAndAppId method before process file.
Why are the changes needed?
Even though in SPARK-25043 it has already brought in the printing application id feature. But the process file situation seems have not been included. This small change is to make spark-sql will also print out application id when process file.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
env
spark version: 3.0.1
os: centos 7

/tmp/tmp.sql
select 1;

submit command:
export HADOOP_USER_NAME=my-hadoop-user
bin/spark-sql  \
--master yarn \
--deploy-mode client \
--queue my.queue.name \
--conf spark.driver.host=$(hostname -i) \
--conf spark.app.name=spark-test  \
--name "spark-test" \
-f /tmp/tmp.sql

execution log:
20/11/09 23:18:39 WARN NativeCodeLoader: Unable to load native-hadoop library for your platform... using builtin-java classes where applicable
20/11/09 23:18:40 WARN HiveConf: HiveConf of name hive.spark.client.rpc.server.address.use.ip does not exist
20/11/09 23:18:40 WARN HiveConf: HiveConf of name hive.spark.client.submit.timeout.interval does not exist
20/11/09 23:18:40 WARN HiveConf: HiveConf of name hive.enforce.bucketing does not exist
20/11/09 23:18:40 WARN HiveConf: HiveConf of name hive.server2.enable.impersonation does not exist
20/11/09 23:18:40 WARN HiveConf: HiveConf of name hive.run.timeout.seconds does not exist
20/11/09 23:18:40 WARN HiveConf: HiveConf of name hive.support.sql11.reserved.keywords does not exist
20/11/09 23:18:40 WARN DomainSocketFactory: The short-circuit local reads feature cannot be used because libhadoop cannot be loaded.
20/11/09 23:18:41 WARN SparkConf: Note that spark.local.dir will be overridden by the value set by the cluster manager (via SPARK_LOCAL_DIRS in mesos/standalone/kubernetes and LOCAL_DIRS in YARN).
20/11/09 23:18:42 WARN Client: Neither spark.yarn.jars nor spark.yarn.archive is set, falling back to uploading libraries under SPARK_HOME.
20/11/09 23:18:52 WARN YarnSchedulerBackend$YarnSchedulerEndpoint: Attempted to request executors before the AM has registered!

Spark master: yarn, Application Id: application_1567136266901_27355775
1
1
Time taken: 4.974 seconds, Fetched 1 row(s)

Can one of the admins verify this patch?

Can one of the admins verify this patch?

cc @wangyum FYI

ok to test.

Test build #130979 has started for PR 30301 at commit 4d0d734.

Test build #130979 has finished for PR 30301 at commit 4d0d734.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/130979/
Test PASSed.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35585/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35585/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/35585/
Test PASSed.

Merged to master.


[PDI-13940] Installing Plugin from Marketplace Causes Spoon to Crash with SWT IllegalArgumentException

@e-cuellar same as before except now <IP_ADDRESS> branch