The Waits KQL provided does not work

The Waits KQL provided does not work.
The following always returns 0 as the column "Type_s" does not capture the Synapse DMV values
//Count of all queued queries
AzureDiagnostics
| where Category contains "waits"
| where Type_s == "UserConcurrencyResourceType"
| summarize totalQueuedQueries = dcount(RequestId_s)

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

ID: 763272bb-fbbc-f4b9-09b8-c0c086684a9e
Version Independent ID: 8ca11524-db18-3ede-4bef-2dafc9ef522e
Content: Monitor workload - Azure portal - Azure Synapse Analytics
Content Source: articles/synapse-analytics/sql-data-warehouse/sql-data-warehouse-monitor-workload-portal.md
Service: synapse-analytics
Sub-service: sql-dw
GitHub Login: @WilliamDAssafMSFT
Microsoft Alias: wiassaf

@veeragitmcs Thanks for the question. GitHub forum is dedicated for docs related issues. For any technical queries or clarifications, we encourage to utilise Microsoft Q & A platform. Kindly raise your query on Microsoft Q&A Platform

#please-open

#reassign:WilliamDAssafMSFT

#label:"doc-bug"


Add --validate option to amber/run_tests.py

This marginally helped my testing/debugging flow.

It looks like this changes the Amber bot to not run validation anymore. Suggest adding --validate to the build-amber.sh kokoro script.

that doesn't seem right?   Amber's -d option disables validation.  So before this change, the run_test.py never enabled validation.
But I'm checking with a local compile.


Refinery macro fails

The macro expansion of the migrations directory does not build on server/build.rs, thus causing the server build to fail entirely.
The observed error is:
error[E0084]: unsupported representation for zero-variant enum
  --> server/build.rs:11:1
   |
11 | refinery::embed_migrations!("migrations");
   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ zero-variant enum
   |
   = note: this error originates in the macro `refinery::embed_migrations` (in Nightly builds, run with -Z macro-backtrace for more info)

Fixed by commit fe986a3


Request for permission to publish a Rust port of this python module

Hi, I've made a Rust port of this module for my own experimental purposes and would like to publish for other to potentially use.
To ensure that the functionality is the same or at least similar, I've written ~40 unit tests for this python project (https://github.com/Mekadrom/rotary-embedding-torch/tree/add_unit_tests) and also converted those to Rust.
The repository is private for now, but I plan on publishing it with the same MIT license as this project if @lucidrains si alright with it. Otherwise, I'll keep it to myself.
Thanks for the consideration.

Also I can merge the unit tests in if you want.


Configure sst-env.d.ts generation

It would be great to be able to disable the generation of sst-env.d.ts for a specific package or a group of packages mostly likely by having a configurable exclude pattern.
For example my monorepo at it's root has:

sst.config.ts
packages/*
apps/*

By the current SST logic everything in packages/* gets given an sst-env.d.ts even though I would rather it not have one.
In practice I don't want anything in packages directly touching the infrastructure as they are designed to be self-contained packages such as what would potentially be publishable to npm. Not having the types helps to force separatation and cleans up the Git diff everytime a resource with links is changed in SST.
Broken out from my comment in #1017.

We have the same problem. What would be even better if SST was somehow able to generate sst-env.d.ts tailored for each package (I guess based on handler paths and their links? I can imagine it's quite tricky to do it right) - it's pretty annoying that every change in SST config results in like 20 changed files even though it realistically only affects one of packages.


68 paths tsconfig created

Path created:
    "paths": {       "@/*": ["./*"],       "components/*": ["./src/components/*"],       "styles/*": ["./src/styles/*"],       "images/*": ["./public/images/*"],       "types/*": ["./src/types/*"],     }

By convention, add @ at beginning.
remove the first one. '@/'
missing utils folder
missing data folder
missing asset folder

Complete to link your PR information please, and follow PR convention.

Also, Mark all devs to review!!!

All changes done.
Only the path from types that breaks when @ is added, so I skipped this one.


Add IP for hash in openvas parser.

If OpenVas found same vulnerability on different hosts, Dojo will ignore duplicates. Sample csv in attached file.

@devGregA I'm ok with this merge.

@aaronweaver  @pavkir  merged


chore(toolchain): allow hooking to existing game server process

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#564
#563  👈
#562
#560
main

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

Join @NathanFlurry and the rest of your teammates on  Graphite


Auth Bug: updating auth does not change results of catalog fetch

What happened:
I was editing a config and was unable to get new catalogs for new user information back. The only way to force this for me was to to delete the datasource and come back to it.
What you expected to happen:
I would expect us to make a new catalogs request anytime other user information changes.
How to reproduce it (as minimally and precisely as possible):

Go to config to add a new athena datasource plugin
Select the keys auth method
Type in an incorrect key and/or region and select Choose Datasource, it'll say no options found which is expected
now type in the correct keys and click Choose Datasource, you'll see there's no new results and no request is made in the network tab for /catalogs (don't touch the region)
I then refreshed the page and tried typing in the correct keys and tried selecting Choose Datasource and found the same bug
If I saved, I'd get an error because I didn't have a datasource selected, and on refresh it was still broken and threw an error in the network tab for /catalogs with the error message UnrecognizedClientException: The security token included in the request is invalid. status code: 400, request id: THE_REQUEST_ID
the way I was able to get around this was to delete the datasource and re-add it and type in the correct credentials the first time.

Screenshots

I'm not entirely sure why this is happening, it seems like a combo problem:

we don't refetch whenever we click on this Choose Datasource when we probably should
It seems like we don't make the request to /catalogs with the latest auth info but with whatever is stored in our session or something like that.

Anything else we need to know?:
Environment:

Grafana version: 8.4.6
Plugin version: 1.0.4

I think this is the same: https://github.com/grafana/athena-datasource/issues/141
Another fix here that i didn't write in this ticket would be to show a permissions error to the user as described in that ticket

FTR, how this should work is that if any change is made to the generic AWS info (e.g. the secret key), that should mark the config as "unsaved". That should trigger a new save and to re-fetch catalogs when clicking in the selector but seems that it's not working.

Hey, sorry just an update on this one before I head out of office for a bit. I had hoped to be able to finish this before I go, but it looks like I'm not going to be able to. If someone wants to pick this up I would understand, otherwise I'll get back to it when I get back in.
We merged and released a fix in the dependency package, here: https://github.com/grafana/grafana-aws-sdk-react/pull/20 the general gist is that ResourceSelectors have a list of dependencies that it refetches on, and secret key was missing from that list so we never made the backend call.
I was then going to install that package into athena, but in my testing I realized the bug still exists! While we now make the network call when we should, the response we get back is not accurate. This is is for 2 reasons as far as I can tell:
We cache sessions with a cache key that does not update when a secret key is updated:
https://github.com/grafana/grafana-aws-sdk/blob/3a212f1b20f9ea0f76f86a0c5595cb6ab14b44f2/pkg/awsds/sessions.go#L149
we store/load api's with a "connectionKey" that does not update when a secret key is updated:
https://github.com/grafana/grafana-aws-sdk/blob/main/pkg/sql/datasource/datasource.go#L50
I was still exploring ways to fix this. One thing I noticed and found interesting was that when we make an update to a datasource we get back a version property. It seemed like a potentially good thing to make a cache key/connection key on, but I did not find an obvious way to access that within sessions (but it's very possible I missed it somewhere). I have spent most of my exploratory work looking into this, but I haven't quite landed on a solution I like here.
A maybe easier fix would be to simply add the properties that could change to the cache key, but I feel like this is dangerous from a security perspective, tbh I already think it's weird we're using this properties at all in storage. We could do some kind of hash or something?
Another thought that occurs to me is why do we have 2 forms of caching at all? Maybe the idea of a both a cache key and a connection key is not necessary. I'm not entirely sure I see a meaningful distinction between the two but I'm also not as familiar as I'd like to be with our auth stuff and assume there are reasons.
Anyway, I think this bug is bigger than I had anticipated and might require a bigger look at caching in aws plugins overall, I sort of assume that whatever difficulties we've seen in athena could exist for all aws plugins, although I haven't heard anything to that affect, which makes me question my own analysis here. I think this could probably use another set of eyes to see if there's something simple and silly I'm missing.

We cache sessions with a cache key that does not update when a secret key is updated: https://github.com/grafana/grafana-aws-sdk/blob/3a212f1b20f9ea0f76f86a0c5595cb6ab14b44f2/pkg/awsds/sessions.go#L149

Agree, this one seems the culprit.

we store/load api's with a "connectionKey" that does not update when a secret key is updated: https://github.com/grafana/grafana-aws-sdk/blob/main/pkg/sql/datasource/datasource.go#L50

In theory (we may need to double check this), this cache depends on the api Mux, and this api is instantiated with the given settings. This mean that any change in the settings should create a new api and the value stored in the cache should be overriden (regardless of the key).

A maybe easier fix would be to simply add the properties that could change to the cache key, but I feel like this is dangerous from a security perspective, tbh I already think it's weird we're using this properties at all in storage. We could do some kind of hash or something?

I agree, I would generate a hash of the settings and use that one as the key.

hey @yaelleC gunna take this one back, if that's cool

update: https://github.com/grafana/grafana-aws-sdk

We looked at this together and this datasources aws-sdk-react package will need to be updated 0.0.37

I believe this has been fixed in version 2.0.1: https://github.com/grafana/athena-datasource/releases/tag/v2.0.1


Add bot announce

Fixes #10

Bot will announce its presence if HUBOT_ANNOUNCE environment variable is set, but take no additional action if it is not set.

Will you please bump the version as well?

Version done bumped and readme updated to include the new env var.

Actually, the version should be .13 (or .14 if you're superstitious).

Not sure what conflict this is reporting now.

Closing this - will resubmit this and a fix for #14 as one pull request, without the conflict.


YoLink integration with HomeBridge working, but devices don't show up in HomeKit

Describe Your Problem:
YoLink devices are not showing up in HomeKit.   They were in the past, but no longer.
YoLink hub and devices are all working if I use the YoLink app.
YoLink integration with HomeBridge appears to be working; I see all my YoLink devices and accurate status.
Every other HomeBridge integration and accessory (besides YoLink) appears in HomeKit and is working normally.
I do not know how to diagnose.
Troubleshooting steps I've tried:

I restarted the plugin child bridge.
I restarted HomeBridge
I restarted restarted the Raspberry Pi running HomeBridge.
I cleared cached devices from HomeBridge.
I attempted to re-add HomeBridge to HomeKit using the QR code (it was already added so this was a NOP).
I stopped running the plugin as a child bridge, and restarted HomeBridge.
I checked to ensure that I was running latest version of HomeBridge and the plugin
I tried forcing upgrade to node.js 22.5.1 but reverted after I got a message that the yolink plugin doesn't support it.

Logs:
attached

Plugin Config:
attached

Screenshots:

Environment:

Plugin Version: 1.5.7
Homebridge Version: 1.8.4
Node.js Version: 20.16.0
NPM Version: 10.8.1
Operating System: Raspbian Bullseye (Linux xxx.xxx.xxx 6.1.21-v8+ #1642 SMP PREEMPT Mon Apr  3 17:24:16 BST 2023 aarch64 GNU/Linux)

homebridge-config.json
homebridge.log.txt

I'm not sure what to tell you.  The logs show everything working as expected and as you note, the devices are appearing in Homebridge but for some reason not in HomeKit.  So the problem is somewhere in that link.
The brute force approach may be to remove the accessories from HomeKit and then re-pair them.  This is nasty because if you have any automations built around any of the devices then they will need to be redone.  If there are other non-YoLink devices part of the automation, then the automation will remain but the YoLink devices removed.  If only YoLink devices were part of the automation then the automation itself will be removed.  As I said, nasty.
If you go this path, then I would also recommend deleting the accessories from the cache in Homebridge.  I would start in Apple Home and remove each of the YoLink accessories and the YoLink bridge.  Then disable the YoLink plugin in Homebridge and make sure it shuts down (or restart Homebridge) then go to Homebridge settings and look for "Manage Cached Accessories" select "Remove Single Cached Accessory" and then delete each of the YoLink accessories. DO NOT select option to remove all cached accessories as that will really mess up your HomeKit automations.  Now re-enable the YoLink bridge and once it is fully initialized go through the steps to add each accessory back to Apple Home.
If you have automations for your YoLink devices this is all pretty ugly as you then have to recreate them.

Cool, thank you for your advice.   My intuition is that there is some state messed up in HomeKit but I don't have any way to verify.
Here's something related that might or might not have contributed: I turned off the HomeBridge YoLink plugin and enabled a YoLink plugin for HomeAssistant (which is also bridged to HomeKit) and perhaps HomeKit got confused.  That never worked so I turned it off and re-enabled this HomeBridge plugin.  But HomeKit might be remembering something and getting confused.

Just one follow up comment in case it helps you in the future.
I had the yolink plugin configured as a child bridge in both HomeBridge and HomeAssistant.   I didn't have both enabled at the same time, but HomeKit was getting confused since it saw two bridges.
I removed one of the bridges from HomeKit and everything started working immediately.
Thank you for your help and for a great plugin.
Eric

Thanks.


AttributeError: 'VispyCanvas' object has no attribute 'measure_fps'

📚 Documentation

I cannot reproduce anymore the tutorial on how to display fps in the main canvas by the current tutorial

I am receiving the following error (also displayed in the output of the tutorial):
Exception has occurred: AttributeError
'VispyCanvas' object has no attribute 'measure_fps'
  File "D:\Ruben\sandbox\napari-omaas\loctest_apdROIs_timestamp.py", line 19, in <module>
    o = napari_omaas.OMAAS(viewer)
AttributeError: 'VispyCanvas' object has no attribute 'measure_fps'

What is the recommended way to check and display the fps when rendering e.g time series of images?

My guess is that your using the main branch of napari and not v0.4.19, right?
In that case, you can look at the same example on main by looking at the dev docs: https://napari.org/dev/gallery/viewer_fps_label.html
which should work.
Looks like that needs private access now, which is unfortunate. I could imagine that we'd be open to publicly exposing measure_fps on the canvas (so that the previous example continues to work), but this is not an area of the code I know well.

Thanks for the fast reply.
This is the output of napari  --info
WARNING: pyqt5 already imported, cannot switch to pyside6
WARNING:vispy:pyqt5 already imported, cannot switch to pyside6
napari: 0.5.0a4
Platform: Windows-10-10.0.19044-SP0
Python: 3.10.14 | packaged by conda-forge | (main, Mar 20 2024, 12:40:08) [MSC v.1938 64 bit (AMD64)]
Qt: 5.15.8
PyQt5: 5.15.9
NumPy: 1.26.4
SciPy: 1.14.0
Dask: 2024.6.2
VisPy: 0.14.3
magicgui: 0.8.3
superqt: 0.6.7
in-n-out: 0.2.1
app-model: 0.2.7
npe2: 0.7.6

OpenGL:
  - GL version:  4.6.0 NVIDIA 536.23
  - MAX_TEXTURE_SIZE: 32768
  - GL_MAX_3D_TEXTURE_SIZE: 16384

Screens:
  - screen 1: resolution 2560x1080, scale 1.0

Optional:
  - numba not installed
  - triangle not installed

Settings path:
  - C:\Users\lopez\AppData\Local\napari\omaas_base_test_5e3ffa57acafb95873961019fba050cde5a9ccb0\settings.yaml
Plugins:
  - napari: 0.5.0a4 (81 contributions)
  - napari-console: 0.0.9 (0 contributions)
  - napari-macrokit: 0.0.1 (2 contributions)
  - napari-mat-file-reader: 0.0.2 (4 contributions)
  - napari-matplotlib: 2.0.1 (10 contributions)
  - napari-omaas: 0.1.3 (12 contributions)
  - napari-sif-reader: 0.0.2 (4 contributions)
  - napari-svg: 0.2.0 (2 contributions)

my env was created with micromamba with the following .yml  template:
name: omaas_base_test
channels:
  - conda-forge
dependencies:
  - python=3.10
  - pyqt
  - pip:
    - git+https://github.com/napari/napari.git
    - "napari-omaas[gpu] @ git+https://github.com/rjlopez2/napari-omaas.git@motion_correction_optimap"

the additional plugin (napari-omaas) is using similar logic as the tutorial to display the FPS when rendering.

Ah, you picked up the recently released alpha for v0.5 which has the change on main (whereas I assume the stable v0.4.19 does not).
To clarify, a quick fix is to use
viewer.window._qt_viewer.canvas._scene_canvas.measure_fps(callback=update_fps)

though that's using two private parts of the API (_qt_viewer, _scene_canvas), so may change in the future.
I guess we would need to provide viewer.measure_fps or similar for a long term public API. That doesn't seem too bad, but this is definitely not my area of the code - maybe @melonora, @brisvag, or others have thoughts?

I can pick it up

Hi @rjlopez2, I opened a PR to fix the problem. If you would like to test it for your application. The way to use it would be as in the updated example that you can see in the PR.

Thanks
It looks like an "easy" fix.
I will check this out soon.
I was just wandering (only out of curiosity) that your suggestion also access the private method (...)._scene_canvas . Is this approach stable in mid-long term?
Cheers and thanks for the lift.

It does, but it is from within the napari code base itself :) . The _scene_canvas was code that I wrote back then in #5432  for the reason that we may want to also have a different canvas backend at some point and we still needed a class to then still be able to deal with the mouse_events. Ultimately with _scene_canvas you access the instance of a NapariSceneCanvas which ultimately is the actual Vispy canvas with the measure_fps method.
One of the reason for this being private is because otherwise the user could easily do all kinds of things with the canvas which we don't control so we can't guarantee a user doing things then causing errors due to that.

Hi @rjlopez2, due to future changes it was opted not to make this api change at the moment, for the discussion please see the PR:) For now though the way you can still get the fps by is by doing the following:
viewer.window._qt_viewer.canvas._scene_canvas.measure_fps(callback=update_fps)
This will still give the warning which we understand is annoying, but otherwise it could be that in 1 version we have to change the api again. Let me know if you encounter any problems!

Alles Klar,
thank you very much for you support here, feel free to close this issue for the time being.
Cheers

No problem!


Support for kubernetes

Describe the solution you'd like
In a modern world, it would be cool if daytona can work with kubernetes, helm and kustomize.

@xavidop thanks for opening the issue.
We definitely plan on supporting Kubernetes through its own provider.
Added the issue to our Roadmap.
If you have any ideas on the direction that the provider should take, we're open to discussion.


Content Suggestion: 21Million Co Store

Link to Content: https://21million.co/
Briefly Describe Content: Bitcoin Apparel
Suggested Section: Spend Bitcoin > Stores

Merged


.gitignore update

That's a very reduced variation of #112.

This basically only removed .DS_Store. Any reason for that?

It's good that you ask, this only added *.pyc, but I actually wanted to add .eggs/, too.

Any reason for that?

Yes. .DS_Store has nothing to do with the repo nor with Python or any of the used tools. It's purely an artifact of one of many possible development environments.
This is not supposed to be in there for the same reason that not all editor-temp-files, autobackup, etc. are in there.
All those should be handled in a global ignore file which is specific to the given system (OS, text editor, ...).
See also:
https://help.github.com/articles/ignoring-files/#create-a-global-gitignore
https://github.com/github/gitignore/tree/master/Global
http://augustl.com/blog/2009/global_gitignores/

Fair enough.

Merge when you're ready.


error on client when it tries to connect

I just compiled from master on VS2017 and it works fine
Did you make any changes to the source before compiling? Which IDE did you use to compile, and did it fetch all of the dependency packages?
Keep in mind that this RAT is still in very early development stages and is basically just a client-server framework with basic GUI at this stage. There are virtually no useful features yet, so unless you plan to fork and develop your own extensions, it's better to wait for development and use a more complete RAT for your needs such as Quasar or a paid alternative

Using VS2019 running on Windows 10 64-bit
Client running on Windows 7 virtual PC running in VirtualBox
No changes made, and yes all dependencies fetched as it builds fine
Noted, was just testing several RAT's and liked your GUI

tried it again several times with same issue. here is full text of error message:-
[DEBUG]: Waiting for Server...
[DEBUG]: Connected!
[DEBUG]: Packet received.
Unhandled Exception: System.NullReferenceException: Object reference not set to
an instance of an object.
at Pace.Client.System.SystemInformation.GetProductName() in C:\Users\darre\Do
wnloads\remote\RATs\pace-master\pace-master\Pace.Client\System\SystemInformation
.cs:line 30
at Pace.Client.System.SystemInformation.Get() in C:\Users\darre\Downloads\rem
ote\RATs\pace-master\pace-master\Pace.Client\System\SystemInformation.cs:line 17
at Pace.Client.Handlers.SystemHandlers.HandleGetSystemInfo(PaceClient client,
IPacket packet) in C:\Users\darre\Downloads\remote\RATs\pace-master\pace-master
\Pace.Client\Handlers\SystemHandlers.cs:line 16
at Pace.Client.Network.PacketChannel.HandlePacket(PaceClient client, IPacket
packet) in C:\Users\darre\Downloads\remote\RATs\pace-master\pace-master\Pace.Cli
ent\Network\PacketChannel.cs:line 23
at Pace.Client.Program.Run() in C:\Users\darre\Downloads\remote\RATs\pace-mas
ter\pace-master\Pace.Client\Program.cs:line 45
at Pace.Client.Program.Main(String[] args) in C:\Users\darre\Downloads\remote
\RATs\pace-master\pace-master\Pace.Client\Program.cs:line 18

Problem fixed
In SystemInformation.cs you are assuming that the client is running on a 64-bit version of Windows and looking in a key which only exists in 64-bit
e.g.
private static string GetProductName()
{
var key = @"SOFTWARE\Wow6432Node\Microsoft\Windows NT\CurrentVersion";
return Registry.LocalMachine.OpenSubKey(key).GetValue("ProductName").ToString();
}
That is what causes the crash above
For testing I change it to below and it works, but it really needs changing to detect 32 or 64 and then look in correct key
private static string GetProductName()
{
var key = @"SOFTWARE\Microsoft\Windows NT\CurrentVersion";
return Registry.LocalMachine.OpenSubKey(key).GetValue("ProductName").ToString();
}

P.S. when you do "take screenshot" - where does it put it? as can't find where it is saving them or what it does with them.   This is one feature I am interested in as most of the others like Quasar don't seem to do screenshots

Hey nice work finding the bug for x86
I'm guessing the screenshot function isn't fully implemented yet.. also couldn't find the output myself xD
PS It's not my project I'm just subscribed to watch list updates etc ;)
I agree the interface is simplistic, modern, and clean. This project has great potential, hopefully we'll see it grow and evolve into something great :)


Add test case for issue #24

This is a test case for #24, but the tests are passing where they should be failing.

First off: thank you for being awesome and figuring out how to contribute a test in the absence of docs. I really appreciate it!
Complete coincidence — I was just looking at this in the last couple of days. (I have a test case that looks extremely similar to this.) I have the same problem: I'm unable to reproduce the issue. It looks like it “all works”. If anybody can figure out how to reproduce the issue, please let me know. I absolutely will fix it if I can reproduce it…but I just can’t seem to make it happen right now, no matter what I do.
I’m still thinking deeply about the YieldedObjectOutputter. I think it needs an overhaul, and I think I’m starting to figure out how to do it. The fundamental issue is that some helpers (e.g., link_to) semantically “should” output in Fortitude (you want to write link_to …, not rawtext(link_to …)), while others semantically shouldn’t (writing number_with_delimiter shouldn’t output, just return the string), but there’s zero programmatic difference between them. So there will always be some custom code required to specify which should be which. In Fortitude’s core, for Rails, this is this file.
I’m thinking of first making YieldedObjectOutputter more robust underneath, so it can easily know where it is in a stack of yielded blocks, and decide what to output and what not to. Then, creating some syntactic sugar so it’s really easy to define new complex helpers (like form_for, fields_for, and so on) that do the right thing.
My goal here is that when people want to use something like simple_form_for, it’s very easy to tell Fortitude what should be output and what shouldn’t — and allow the building of either a very general fortitude-contrib or tie-specific gem like fortitude-simple_form_for or whatever that shares this little piece of code. I don’t think it makes sense to put integration code like that for a gazillion third-party gems into Fortitude’s core, but I can do a much better job of making it really easy to create and share that code.

With regard to figuring out how to write a test: my pleasure!
I've also pinged Karl and Harlan in #24, so hopefully they can chime in and we can figure out if there's actually a bug here. I know that there's a reason I used the monkeypatch originally... but it might have had to do with simple_form.
Anyway, I've been thinking about some of the same stuff -- I was going to ask you about your preferred pattern for creating a compatibility gem like fortitude-simple_form.
I agree that the YieldedObjectOutputter is probably one of the more... perplexing parts of Fortitude, but that said, it's already an improvement over Erector, and I think some more documentation could go a long way towards making it less opaque. :smile:

Memory isn't too good on this one, I'll try and see if I can reproduce it again.
In the meantime, I did notice that in my code the only time this situation pops up is with a hash argument, e.g. label :for => :foo do ...

Weird, so the build failed on CI.

So I finally got my environment working (turns out its because I didn't have nodejs installed...), and your test does fail for me, here's a snippet:
   -/oop_rails_server_base_template/i
   +"{\"exception\":{\"class\":\"ArgumentError\",\"message\":\"wrong number of arguments (0 for 1)\",\"backtrace\":[\"/vagrant/lib/fortitude/rails/yielded_object_outputter.rb:9:in `block in wrap_block_as_needed'\",\"/var/lib/gems/1.9.1/gems/actionpack-4.0.13/lib/action_view/helpers/capture_helper.rb:38:in `block in capture'\",\"/var/lib/gems/1.9.1/gems/actionpack-4.0.13/lib/action_view/helpers/capture_helper.rb:200:in `with_output_buffer'\",\"/var/lib/gems/1.9.1/gems/actionpack-4.0.13/lib/action_view/helpers/capture_helper.rb:38:in `capture'\",

Which I think is exactly what we would expect?
If it helps, I set up a Vagrant box with the following config to test it:
Vagrant.configure("2") do |config|

  config.vm.box = "ubuntu/trusty64"

  config.vm.provider :virtualbox do |vb|
    vb.customize ["modifyvm", :id, "--natdnshostresolver1", "on"]
    vb.customize ["modifyvm", :id, "--memory", 4096]
    vb.customize ["modifyvm", :id, "--cpus", 4]
  end

  config.vm.provision "ruby", type: "shell" do |s|
    s.inline = <<-SCRIPT
      sudo apt-get -y install git libsqlite3-dev nodejs ruby-dev make
      su - vagrant -c "sudo gem install bundler"
    SCRIPT
  end
end

It looks like the failures are conditional on the version of Rails. I can give this another shot when I get a chance...

Finally getting back to this. Notes to myself:
The problem is triggered by code like this:
    form_for :person do |f|
      f.label(:name) do
        text 'Foo'
      end
    end

When Fortitude sees the call to form_for, it properly wraps the f being returned in an instance of Fortitude::Rails::YieldedObjectOutputter. The job of the YOO is to intercept calls like f.label, which normally just return text (i.e., in ERb, you say <%= f.label(:name) … %>, not just <% f.label(:name) … %>, and instead cause them to output text instead. And this works just fine.
The issue is the inner block. There is a bug in Fortitude without this patch that causes it to fail when trying to wrap the inner block — it’s expecting blocks to always have at least one argument, and that fails, which this PR fixes. But the much more important bug is deeper: when we call that inner block, text 'Foo' immediately outputs text. But this is wrong, because we’re still capturing the return value of f.label, and will output it only after that call is done. So we end up with Foo, which, of course, is completely incorrect. What we need is Foo".
What needs to happen is that the inner call to text doesn’t immediately output text, but, instead, appends it to whatever buffer f.label is going to return, so that it gets properly nested. I haven’t yet figured out how to do that, but that’s what needs to happen. I’ll tackle that as soon as I get a little more time…

So, after further digging — there were two issues here:

The YieldedObjectOutputter assumed that all methods that took blocks took at least one argument. This is not, in fact, true (and #label is a great example). I’ve fixed that.
Text from the label block showing up outside the label. This is actually a Rails bug in 3.0/3.1. A vanilla Rails install of either of these versions, without Fortitude even installed at all, when given the following code:

<%= form_for :person do |f| %>
  <%= f.label :name do %>
    Foo
  <% end %>
<% end %>

…ends up emitting something like this:
<form accept-charset="UTF-8" action="/foo/bar" method="post">

     Foo

     Foo
</form>

…which is clearly wrong.
Yes, this is wrong in a different way than Fortitude’s output is wrong, and that might make me suspect Fortitude. But (a) Fortitude isn’t doing anything particularly weird under the covers here, and (b) both Fortitude and ERb are broken on Rails 3.0/3.1, while (c) both Fortitude and ERb work perfectly on Rails 3.2 and above. As such, I don’t think there’s anything to fix in Fortitude here. I’ve added this example to the spec suite, with a skip for Rails 3.0/3.1.
The fix for the first issue outlined here is in 0.9.5, which I just released.


cartodb-logo toggle not working for password-protected maps

The CartoDB logo is appearing on password protected maps (embed view) even when it is disabled from the Editor.
Steps to reproduce

Create a map in CartoDB Editor
Disable logo in Options menu
Go to embed_map (click on Publish, click on 'Go to your map' link in 'Embed it' card)
Check the logo is not there
Go back to the map and set a password in privacy settings
CartoDB logo setting should still be disabled in options menu
Go to the embed map and hit refresh. The password should be asked
CartoDB logo is present, even when the option remains disabled in the editor
Go back to the map, set the privacy to 'with link'
Coming back to embed_map, refresh and check CartoDB logo is not there, it only appears on password protected maps

Expected result
The logo should appear or not depending on the option set, regardless of the map privacy setting
Current result
The logo is always appearing on password protected maps, regardless of the option set in the Editor

S/B: 9760169

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.


[Feature request] Conan recipe to build C++ client and publication to conan-center

Conan.io is a popular C++ package manager. It would be very handy to have a conan recipe to build the C++ client. It would make it easy to build even on Windows, even with MSVC.
Ultimately, you could pre-build conan packages for various platforms and compilers and publish them to conan-center so that it's easy to include the client lib in a conan-enabled project.

I checked and all dependencies used by the C++ client seem to exist in conan except log4cxx (but it is optional, isn't it ?):
Boost, OpenSSL, zlib, json-cpp, curl, gtest/gmock --> conan-center
Boost-Python, protobuf,  --> bincrafters
Also the only Linux-only dependency seems to be the crc32 lib. Couldn't it be possible to use a cross-platform CRC32 so that the client can be built on Windows ?

Related to #15 #2461 #2680

@cbornet I don't have any experience with Conan.io but it seems to be a good way to package the client lib.

log4cxx (but it is optional, isn't it ?):

Yes, log4cxx is optional and disabled by default

Boost-Python, protobuf --> bincrafters

boost-python is only needed at compile time (and we should refactor the CMakeList.txt to not require it at compile time as it's only used to build the python client wrapper).

Also the only Linux-only dependency seems to be the crc32 lib. Couldn't it be possible to use a cross-platform CRC32 so that the client can be built on Windows ?

We don't depend on crc32 libraries. We have a crc32-c implementation imported in our sources.

We don't depend on crc32 libraries. We have a crc32-c implementation imported in our sources.

Yes. My point was more that this crc32-c implementation depends on pthread which is not natively supported on windows. Sorry for not being clear enough.
But there's actually a conan package for pthread on windows so maybe it could be used.

Yes that's even better

ITNOA
__builtin_expect does not exist in Windows on MSVC, did you any plan to resolve this?

related to #3800


Menus / folders always at the bottom

Is there a reason why menus are always placed at the bottom of the menu. I now have a menu with four to five menu items, then the "exit" item, then the menus with subitems. How can I get the exit item to be all the way at the bottom?

If you use the addMenuItem(javafx.scene.control.MenuItem) API, it always inserts at the end of the menu item list. But, there is also a `insertMenuItem(javafx.scene.control.MenuItem, int) to insert at a menu item at a certain position.

Closing, considering the comment made by @dwalluck
I will consider implementing an index setting for the "Exit" item in future releases, though if you wanted to create your own custom "Exit" item, it simply calls Platform.setImplicitExit(true); Platform.exit();

@dlemmermann  - There is already a Builder method for automatically adding an Exit menu to the bottom of the trayIcon - note the .addExitMenuItem() in this build sentence:
trayIcon = new FXTrayIcon.Builder(stage, getClass().getResource("FXIconRedYellow.png"))
		.toolTip("An alternative tooltip!")
		.menuItem(menuItemTest)
		.menuItem(menuOptions)
		.applicationTitle("FXTrayIcon Builder Text")
		.separator()
		.addExitMenuItem()
		.build();```

I'm going to add the option of setting the label as desired in my next PR.

Maybe he wants additional methods that add a position index parameter.

@dwalluck The Builder does add the menuItems in the order that they are inserted into the build sentence. But the exitMenu can be added anywhere in the build sentence and it will always get placed at the bottom of the menu.
I also think that some kind of an indexing structure would be handy, I might start a new issue on that topic to open it up for some discussion as to how that should look from a usability perspective. I've got some ideas but it's always nice to get everyone's input on that kind of thing.


Hyperparameter Optimization

Implement (Bayesion) Hyperparameter Optimization for users

Lets maybe have an initial discussion on how to include HPO into the framework

V1.0.0 includes built-in bayesian HPO and is fully integrtable with sklearn hyperparameter optimizers


Release strategy for third-party apps

There needs to be a policy on how we release third party apps on our apps site in which the HydroShare core team has had no part.  Any strategy must balance user perception of the reliability of HydroShare against the capability advances that new apps can represent.
One proposed strategy is to label the app as "alpha", "beta" or "full release" as follows:

An "alpha" release is one in which there is at least one complete use case that works properly and reliably. This use case should be carefully documented and reproducible by users.
A "beta" release is one in which the variety of inputs to the app have been more fully explored, in the sense that there is high confidence that it works in a majority of intended uses. One promotes from "alpha" to "beta" either as the result of UI testing or as a result of evidence of usefulness from the community as an "alpha". One moves an app from "alpha" to "beta" status when one has some assurance that the remaining issues are peripheral to its basic intended function.
A "full" release is one in which the software has been sufficiently vetted that there is reasonably high confidence that all remaining issues do not affect intended use. Criteria for promoting to full include time in system as "alpha" or "beta", perceived risk of new defects, amount of use recorded in use tracking, as well as user ratings of the functionality of the app.

Combined with #1841 #1843


hugo new only accepts the .md extension in Hugo v0.24

This starts to fail in 0.24 and used to work in 0.23 and earlier versions of Hugo:
$ hugo new test.Rmd -v
INFO 2017/06/23 23:59:21 Using config file:
INFO 2017/06/23 23:59:21 attempting to create "test.Rmd" of "" of ext ".Rmd"
Error: open : no such file or directory

I don't know what this error message means. Only hugo new test.md is permitted now.

Duplicates #3626 -- it seems to be a pain for many, so I will push v0.24.1 today.

Very nice. Thanks!


[Fix] integrate_inputlinks if no published instance where found

integrate_inputlinks throw an error if nothing was published
Traceback (most recent call last):
  File "C:\Users\jerome.lorrain\PycharmProjects\OpenPype_build\.venv\lib\site-packages\pyblish\plugin.py", line 522, in __explicit_process
    runner(*args)
  File "C:\Users\jerome.lorrain\PycharmProjects\OpenPype_build\openpype\plugins\publish\integrate_inputlinks.py", line 85, in process
  File "C:\Users\jerome.lorrain\PycharmProjects\OpenPype_build\openpype\plugins\publish\integrate_inputlinks.py", line 121, in write_links_to_database
AttributeError: 'NoneType' object has no attribute 'data'

Fix to not throw an error (but still log a warning)

Alread yfixed with https://github.com/pypeclub/OpenPype/pull/2322


remove relationships and fields items from EntitySchema

After getting further along with the ZQL interface for join (pr coming soon) and seeing this suggestion for simplifying foreign key traversals, I don't think we need this information anymore.

Seems like a good step.


added JDK8/11 enforcement

Pull Request Description
This pull request closes strongbox/strongbox#
Acceptance Test

[X] Building the code of the strongbox-parent with mvn clean install still works.
[X] Building the code of the strongbox project with mvn clean install -Dintegration.tests still works.
[X] Running mvn spring-boot:run in the strongbox-web-core still starts up the application correctly.
[X] Building the code and running the strongbox-distribution from a zip or tar.gz still works.
[X] The tests in the strongbox-web-integration-tests still run properly.

Questions

Does this pull request break backward compatibility?

[ ] Yes
[X] No

Does this pull request require other pull requests to be merged first?

[ ] Yes, please see #...
[X] No

Does this require an update of the documentation?

[ ] Yes, please see strongbox/strongbox-docs#{PR_NUMBER}
[X] No

Code Review And Pre-Merge Checklist

[X] My code follows the coding convention of this project.
[X] I have performed a self-review of my own code.
[X] I have commented my code in hard-to-understand areas.
[X] My changes generate no new warnings.

Uhm... I'm sorry, but why is there a PR for this pointing to issues/1649 ? This is already enforcement is already in the master branch?

At the time, the issues/1649 had probably not been rebased.

@carlspring @steve-todorov I think we need to rebase instead of merging this

Okay.  In this case, I'll close it.


Remove LD metric because it is not relevent for genes (but for SNPs)

This metric can be removed or provided as a usable optional metric function

I think this is solved right?

Done


Pass through response

Fixes #97

Need to wait for #99 to be merged before re-opening this

Thanks once again @rupurt – good work!


Github action - build

Travis builds have been inconsistent over the past several months. Testing github actions to see if that works any better.

The last successful build was pr #75, which didn't complete for more than a week after new changes were merged to the branch.
I've cancelled the Travis checks for this PR.


Name collision with PyPI parcel

Ran into an issue today where your parcel package has the same name as this one from PyPI: https://pypi.python.org/pypi/parcel/

I have a similiar problem. NCI-GDC/gdc-client uses your package as a dependency and it seems problematic to be able ever pip install the gdc-client as long as this naming conflict exists.

Yes, that was exactly how I ran into this issue. pip installation of gdc-client gets really weird because of this package's name.

Yup, just ran into this myself installing gdc-client.


Add PrecompileTools

Temporary removed AMDGPU to make it run on Julia 1.11.
Although I haven't seen much improvement, unless I'm doing something wrong.
You can run julia benchmark/pipeline.jl from Nerf.jl directory to run the whole pipeline.
Haven't seen significant improvements with timings, I'm using master branch of GPUCompiler...
Without PrecompileTools:
Trainer benchmark
 17.861788 seconds (30.04 M allocations: 1.492 GiB, 2.76% gc time, 74.64% compilation time: <1% of which was recompilation)
 39.232965 seconds (4.11 M allocations: 124.968 MiB, 1.58% gc time)
Renderer benchmark
 13.524578 seconds (10.00 M allocations: 336.965 MiB, 3.08% gc time, 14.66% compilation time)
 51.170871 seconds (33.40 M allocations: 776.366 MiB, 3.85% gc time)

With PrecompileTools:
Trainer benchmark
 17.097459 seconds (29.72 M allocations: 1.478 GiB, 2.91% gc time, 73.60% compilation time)
 33.056091 seconds (4.09 M allocations: 124.626 MiB, 1.24% gc time)
Renderer benchmark
 11.619677 seconds (6.85 M allocations: 195.984 MiB, 2.69% gc time, 2.40% compilation time)
 50.162239 seconds (31.00 M allocations: 723.421 MiB, 3.02% gc time)

I wonder if enable the disk cache would do something, but it is odd that the trainer benchmark still spends 70% in compilation

Oh, indeed :) Now it is faster. Benchmark consist of:
@time trainer_benchmark(trainer, 10)
@time trainer_benchmark(trainer, 1000)

Where first line is to precompile all kernels mainly. So the time went down from ~18 seconds to ~4 seconds.


Doesn't support chats in binary format?

I tried uploading the attached chat but it did not appear to work. Perhaps this file type is not supported? Please forgive the absurdity of whatever my young self wrote at the time 😂
<EMAIL_ADDRESS>

No, sorry


 Don't do this. Create a cascade at the beginning and point at the context per query.
@claclark I assume you have no objections?

Fixed commit b5ab1cb735e78c1b5709a80958eda292d8f788d4


change default language

That was made for tests, should not be default

Also noticed default Portugal language. :)

Sorry, that wasn't planned


Docs Typo: blob server upload

Hi there 👋
I found a small typo in the vercel blob storage docs in the Create a server upload page section.

Here's the typo
const newblob = (await response.json()) as PutBlobResult;

setBlob(newBlob);

This would be the fix
const newBlob = (await response.json()) as PutBlobResult;

setBlob(newBlob);

To reiterate, the typo lies in the difference between newBlob and newblob.

PS: Would gladly open a PR, but wasn't able to find where the docs lived.

Thanks for reporting! The docs live in an internal repo, that's why you could not find them :) I've opened a PR to fix, which should go out later today

Thank you @dferber90


Corrections to render sphinx correctly

I took a look to see how they render via ReadTheDocs. It seems Sphinx got confused with how to render the Parameters and Status Codes.

Thx @ttmc, I should have checked the rendered output :/ (I did this time!)

Current coverage is 96.85% (diff: 100%)

Merging #819 into master will not change coverage

@@             master       #819   diff @@
==========================================
  Files            26         26
  Lines          1683       1683
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1630       1630
  Misses           53         53
  Partials          0          0

Powered by Codecov. Last update 57d9374...5ef5329

This time I checked out the branch and rendered it locally just to see what happens. The Parameters and Status codes render fine now.
I also noticed it fails to syntax-highlight the final example response (HTTP). Sphinx gave the warning:
/home/troy/repos/bigchaindb/docs/server/source/drivers-clients/http-client-server-api.rst:316: WARNING: Could not lex literal_block as "http". Highlighting skipped.

I honestly don't see what's wrong or different from previous example HTTP response blocks.  I'm not sure it matters that much though. I'd be okay with merging as is.


Switch to Zod

When normalizing we do a lot of work that would be otherwise be done by Zod. I think it's worth switching to it. I recently switched to it in Nuxt-VIntl and it was amazing, I think this plugin is also a worthy application. Bonus points: it will validate the options, something we don't currently do and rely solely on TypeScript.

Not worth it because Zod does not support JSDoc and bloats the declaration files.


Include attribute to target annotations / hotspot elements based on visibility

Resolves #989
In order to enable performant attribute updates on annotations, it was necessary to update the annotations feature to only update / render annotations when necessary, rather than on each tick.

@cdata I've created a new PR #1018 to replace this one


Grade Calculator?

SEQTA Grade Calculator

SEQTA Grade Calculator

Grade Calculator

Grade Calculator

Assignment 1 Score:

    Assignment 2 Score:
    <input type="number" id="assignment2" class="assessment">

    Exam Score:
    <input type="number" id="exam" class="assessment">

    Total Score:
    <input type="text" id="totalScore" disabled>

    Grade:
    <input type="text" id="grade" disabled>

</form>

[GradeCalcutaltor.txt](https://github.com/SethBurkart123/EvenBetterSEQTA/files/12911280/GradeCalcutaltor.txt)

Nice! That is a great script. One thing that I was thinking though, maybe a future idea. But some schools group assignments, exams etc. into separate groups where eg.

Average of exams = 40% of grade
Average of assignments = 60% of grade
Maybe we could push this even further to make it allow you to group your different assignments as like a advanced option or something! But this is a great start, once I have finished up with the background images and videos I'll drop this into the code.

For assessments, SEQTA doesn't know how much the assessment is weighted, the student will need to input that.

Yeah, definitely.

Grade Calculator

Assignment 2 Score:

    Exam Score:
    <input type="number" id="exam" class="assessment">

    Assessment Percentage (%):
    <input type="number" id="assessmentPercentage">

    Total Score:
    <input type="text" id="totalScore" disabled>

    Grade:
    <input type="text" id="grade" disabled>

</form>

For assessments, SEQTA doesn't know how much the assessment is weighted, the student will need to input that.

Hi, this might be a bit random but I think SEQTA might, well with my school it does. to find the weighting of an assessment follow:

Assessments tab
Select the desired assessment
Click on three dots at the top right of the page (should be next to the assessment date)
Print cover sheet
access document and assessment weight should be in there.

I don't know if this is what you meant but might be useful nonetheless. Thanks for all of your work on making such an outdated software modern and cool.

For assessments, SEQTA doesn't know how much the assessment is weighted, the student will need to input that.

Hi, this might be a bit random but I think SEQTA might, well with my school it does. to find the weighting of an assessment follow:

Assessments tab
Select the desired assessment
Click on three dots at the top right of the page (should be next to the assessment date)
Print cover sheet
access document and assessment weight should be in there.

I don't know if this is what you meant but might be useful nonetheless. Thanks for all of your work on making such an outdated software modern and cool.

That looks pretty cool (and super useful), because that would mean that it would be much more accurate by default. I tried it, But with my school it basically just errors out for some reason 😭.

I thought this would tell you the grade you are sitting on? If so, you could always just have the user enter the weight of the assessment. If not, then idk what this issue is about lol

What is our progress looking like on this?

What's the progress on this one?

Currently low priority. If somebody else would like to go and write up an implementation and add it to the UI with a setting feel free though.

It looks like not much has been done/is going to be done about this so I'll close this for the time being

GitHub, it's NOT done again.
Someone move it to not planned please.

Ladies and gentlemen. I present to you...

WOOOO

Although letter grades dont work...

she'll be right

she'll be right
huh


feat: 🎸 Added activity logger to close command

This PR implements a logger that increments a user's Setsudo =modactivity actions when closing a modmail.

Added database configuration option.


Views now deduplicate data from 2 days instead of 3

Coverage remained the same at 94.222% when pulling 1d42ddc6dd6925bf422c85cc732c8ad8cb44ffa1 on views_change into 40bc81596ba35eda85571bc1700e1ea3706bf912 on master.


in Chrome extension selection is always null

Hello there!
I tried to find what could cause this issue, but unfortunately I wasn't successful.
We develop extension with lexical, where lexical is in the content script.
The firefox extension with manifest v2 works perfectly, but chrome (with manifest 3) doesn't.
document.getSelection and window.getSelection works perfectly, so I think it is not related to permissions regard google chrome.
When I call root.select(), the selection is not null, but when I do anything it will always be null.
Could you give some clues, where to continue?
Thank you in advance!
Lexical version: 0.8.1
The current behavior
If I select a text, selection will be null always
The expected behavior
Selection should contain the same as in firefox.

Please share the full file where you try to get the selection.

We need a minimal reproducible example in order to help you figure out what the issue is

I obviously can't verify, but to me this seems more like a Chromium bug than a Lexical issue. We had a Developer Tools web extension for a little while (before deprecating in favour of the inline tools) which successfully showed the Lexical selection (#4023).

Thank you for your comments!
For future readers: I found out that the new manifest version causing the problem. Although the window.getSelection is not null, but is doesn't contain what it should. And it's pretty difficult to get the selection inside a chrome extension - and it requires some workaround with lexical too (I am still working on it)


Report error/warning when adding helm chart resource of type "dir" set to the bottommost directory.

What would you like to be added:
For a local helm chart resource of type dir, the option preserveDir is set to false by default.
For this to work, the helm chart directory has to be placed in an otherwise parent directory.
If the option preserveDir is not set to true explicitly, this parent directory has to be entered as the value of parameter path.
If this is not applied correctly, the landscaper cli will accept the local resource but the deployment in the landscaper will fail with a rather cryptic error message.
Example:
├── chart
│   └── ingress-nginx

This will fail
type: helm
name: ingress-nginx-chart
version: v0.1.0
relation: external
input:
  type: "dir"
  path: ./chart/ingress-nginx

This will succeed
type: helm
name: ingress-nginx-chart
version: v0.1.0
relation: external
input:
  type: "dir"
  path: ./chart/ingress-nginx
  preserveDir: true

For the user it would be helpful if the landscaper cli issues a warning or even an error when trying to add a helm chart resource with the  bottommost directory without setting option preserveDir to true.
Why is this needed:
Catching this error on the cli level will result in a more readable/understandable error message.
It also helps the user to immediately take action upon this error instead inspecting the landscaper error logs.

Hi @reshnm ,
the check that you propose would have to inspect the directory specified in the path field. Something like: if it contains a Chart.yaml file, then preserveDir should be true. Is that right?

Hi @reshnm ,
the check that you propose would have to inspect the directory specified in the path field. Something like: if it contains a Chart.yaml file, then preserveDir should be true. Is that right?

Exactly, yes.

Closed, as moved to new backlog.


[proposal] Update the devcontainer.json schema to be able to represent a 'mergedConfiguration'

The goal of this proposal is to:

Consolidate merged lifecycle hook and devcontainer.json schema
Keep track of the 'origin' of contributions to the merged config for purposes such as logging.

Is there still an appetite for this? In our SDK/CLI I'm tempted to implement something similar to help wrangle types (ours is written in Go, so I unfortunately can't leverage such rich types to keep everything in one place).
Something that would help additionally would be some test cases for how to merge configuration that implementing tools can run against, but that's a matter for another time I think.

Thanks for the nudge @avidal, this is definitely something I still want us to move forward with. I'll give this spec's comments another pass right now

Another question, since this is still in review: this proposal adds an $origin for lifecycle commands when they are merged, as well as $entrypoints to collect entrypoints from features. What about other properties that are merged, such as containerEnv, forwardPorts, or mounts? What about last-value such as containerUser or the specific semantics of init?
Do we think it's just not important to be able to report on the origin of those properties? Whereas lifecycle commands can use the origin for output logging?


Example: visible/hidden columns

Griddle major version
1.x
Changes proposed in this pull request
Update story to show <ColumnDefinition visible={false} />.

(I also fixed that &.)
Why these changes are made
Closes #682 (cc @mathetes).

This is awesome! Thank you!


[RFC] Adjust List.Item content

Bug Report
Content is too small and needs gap with the above header.

Is this in regard to the Teams theme?


about gravity

Hi i need change gravity of one object in simulation.
is possible on BodyCreationSettings with bcs.mGravityFactor = 0.5;
but after creation of body is not possible to change gravityfactor

You can't do it on the body directly but you can by using the body interface:
https://jrouwe.github.io/JoltPhysics/class_body_interface.html#a2171a36d0ff2b98c019ed4b0bcb0db39
So something like:
this.bodyInterface.SetGravityFactor(BodyId, 3);
It's been a hard habit to break for me but I'm learning that with Jolt many methods aren't on the object directly but managed through interfaces/systems/managers.

ok thanks is see
but is seem is not exposed in js version

There's also MotionProperties::SetGravityFactor (so body.GetMotionProperties().SetGravityFactor(...)). I've exposed both functions in the js version now (but you need to compile it yourself since I didn't create a release).

It's been a hard habit to break for me but I'm learning that with Jolt many methods aren't on the object directly but managed through interfaces/systems/managers.

Most of the functions on BodyInterface are convenience functions that will take a lock on the body and call a function, in this case they just mirror the function on Body or MotionProperties. Some of them (e.g. setting positions) are only available through the BodyInterface as the managing systems also need to be made aware that changes happened.


jobError switch

In this section here:
https://github.com/firstandthird/hapi-job-queue/blob/master/lib/index.js#L209
It appears that the callback should be err: jobError and not err: err which may be null.
Yes / no?

Oh good catch. We might want to have the job log that error itself since it's technically not an error caused by the job queue.


Laravel reverb config tls variables

In my dev I'm using a self signed certificate, so I have to put the cert path in .env.
In my production there's a nginx reverse proxy, so I don't need a certificate in my .env.
But the problem is if the options "tls" is NOT empty then the server will run in secure mode.
Servers/Reverb/Factory.php L54
$uri = empty($options['tls']) ? "{$host}:{$port}" : "tls://{$host}:{$port}";

My reverb.php config
'servers' => [
        'reverb' => [
           ...
            'options' => [
                'tls' => [ //tls is ALWAYS not empty
                    'local_cert' =>env('REVERB_CERT')
                ],
            ],
        ]
]

Thanks, I'll get a fix up for this as soon as I can.

This should be resolved in the next release: https://github.com/laravel/reverb/pull/85


filter

Is it important to use a median filter? There's a pretty decent speedup from using an average filter (~145ms to ~77ms on my pc) and the frogs still match (as well as the water mark picture lol)

Another spot for improvement could be just calculating the Y component instead of going through 0 {"1 RGB_to_YUV


[Additional Parameter Request] Codeunit 90 Purch.-Post - OnBeforePurchRcptLineInsert

Hi Team,
In codeunit 90 we need possibility to have additional checks in the procedure InsertReceiptLine. Standard publisher OnBeforePurchRcptLineInsert would suit us if it had additional parameter var IsHandled: Boolean.

local procedure OnBeforePurchRcptLineInsert(var PurchRcptLine: Record "Purch. Rcpt. Line"; var PurchRcptHeader: Record "Purch. Rcpt. Header"; var PurchLine: Record "Purchase Line"; CommitIsSupressed: Boolean; PostedWhseRcptLine: Record "Posted Whse. Receipt Line"; var IsHandled: Boolean)

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Created inverted-right-half

added code for inverted right half triangle

Hello
Thanks for contributing
Please create a issue first and link this Pull request to that issue .

Hi Thanks for contributing. We would like to highlight the following changes before merging the code : 1> Please rename the file in a format similar to the present files in our repo in this case it would be " InvertedRightHalfPyramid_Pattern " 2> Please create an issue first titled " Inverted Right Half Pyramid (Using *) "

Thanks for suggestions! I have changed it accordingly


Also i feel we should add a requirements.txt file.
Thanks :)

Hey, i would like to work on this issue. Also i feel we should add a requirements.txt file. Thanks :)

Hey @Pranjalmishra30 thanks a lot for bringing my attention to it .

Could you assign this issue to me so i can start working on it.

Could you assign this issue to me so i can start working on it.

Yup . Assigning this to you


Uninstall

i accidently installed it to my live eft when i meant to install it to my single player eft and i cant seem to figure out how to uninstall it

installer uninstall


Question: how to access the P-value as a variable from the model

Hope this is the right place to post this.
I'm using this model for campaign evaluation and comparing its ci.inferences.post_preds_means with actual daily metrics to evaluate the effect of the campaign.
I want to check the p-value in order to flag when the difference isn't significant. However the only way I can see the p-value is in the wrapped-up ci.summary() or ci.summary('report'). Is there any way I can just get the p-value itself?

You can access the p_value simply with ci.p_value. Notice though that the package already offers you a complete analysis of the impact and its statistical validity. For the most part you shouldn't concern yourself with finding posterior means or any other metric, the package already handles that for you.
Unless you are doing something more specific not already offered by the package, otherwise I'd recommend relying on the summaries implemented by default.


Read/write LandXML files

https://forum.dynamobim.com/t/landxml-export-from-civil-3d/57029

https://forum.dynamobim.com/t/landxml-package/75090


Removing small amounts of MVI boilerplate

Hi!
I've just recently started using Mosby/MVI within the past month and I have thoroughly enjoyed it so far. So first off I'd like to say thank you!
From the examples I've observed it seems there's some boilerplate that could be removed. All of the view interfaces have one thing in common: the render method. Is there a reason a method like this never made it into an interface? Perhaps one extending MvpView:
public interface MviView<VS> extends MvpView {
  void render(VS viewState);
}

Additionally, subscribeViewState seems to always be called at the end of bindIntents, so bindIntents could return the Observable that's passed as the first parameter to subscribeViewState and this call could instead be made internally. So my Presenter would look like the following:
public class MyPresenter extends MviBasePresenter<MyScreen, MyScreenViewState> {
  @Override protected Observable<MyScreenViewState> createViewStateObservable() {
    Observable<MyScreenViewState> obs1 = intent(...);
    Observable<MyScreenViewState> obs2 = intent(...);
    return Observable.merge(obs1, obs2);
  }
}

It doesn't remove tons of boilerplate code, but it removes enough where I think it's useful. Also, with this implementation, the mistake of forgetting to call subscribeViewState can't be made.
I'm sure there could be good reason why it hasn't been implemented this way. Any thoughts?

Hey,
thanks for your feedback! Indeed, there is some "boilerplate" if you implement MVI the way I am doing (and promoting) it. However, I would like to give the user of Mosby the freedom to use this library the way it works best for them and for example I don't want to limit them to use a MviView interface with a render(state) method.
We had this conversation some time ago:
https://github.com/sockeqwe/mosby/issues/209
Also, there are plans to add yet another way to implement MVI (MVVM-ish) where an MviView interface doesn't make sense:
https://github.com/sockeqwe/mosby/issues/247
So for now, I think it makes sense to let the developer (the user of this library) the choice how to implement MVI. Mosby should just offer some convenient classes around general problems like lifecycle and retaining presenters.  I think it is not too much work for any developer who wants to use Mosby for MVI based applications to introduce a MviView and AbstractMviPresenter (which is what you have described as MyPresenter in your comment above) in their own app, without having to introduce it in Mosby.
What do you think?

Makes sense :)
Indeed it takes no time at all to extend from MviBasePresenter and create a base version of the implementation discussed above.
Thanks for the insight!


Function to add single sticker | merge with existing collection

Great plugin. Thanks!
I am looking for an option to only add a single sticker to either an existing collection or by creating one.
Using sticker.ly you can only add the stickers you want, which makes it highly convenient for the user. I have an application where the user does not add a package but single stickers. Unfortunately, this package does not allow that and when trying to upload a single sticker I get the following error:
sticker pack sticker count should be between 3 to 30 inclusive, it currently has 1, sticker pack identifier:my_identifier

Or merging (using the same name package) I get:
XXX has been already added to WhatsApp

I would highly appreciate it if you could add this functionality. Thanks a lot

Hi @polarby. Did you find any way to do it? I saw in the documentation they said 'Each sticker pack must have a minimum of 3 stickers and a maximum of 30 stickers' https://github.com/WhatsApp/stickers/tree/main/Android#sticker-art-and-app-requirements

Unfortunately not. Please let me know if you find out a way though!


vdk-jupyter: add transform job UI button

What:
Add button to the VDK menu in the UI responsible for the Transform job action.
Why:
To provide the user with the option to do Transform job operation.
Testing Done:
Аdded jest tests for the component. + The plugin is built successfully.
Signed-off-by: Yoan Salambashev<EMAIL_ADDRESS>

Please make sure all UI related PRs have screenshots of relevant changes


refactor: renamed changelog, license and readme to uppercase for standardization

Part of PR #17

@OlivierLDff Wait this for CMakeLists.txt PR


Bar Chart and Stack bar charts are not rendering

I tried demo example from react-native-chart-kit-example1 , react-native-chart-kit-example and all charts are rendering correctly but bar and stack bar chart not rendering (showing blank instead) No error on command prompt.
"react-native": "https://github.com/expo/react-native/archive/sdk-34.0.1.tar.gz",
"react-native-chart-kit": "^3.6.1",
"react-native-svg": "^9.5.3",
Any idea?

@Hermanya bar chart are not showing up on latest release. Can you please take look at it?

Same here.

same here

@Hermanya any update on this?

https://github.com/react-native-community/react-native-svg/issues/1175
Issue is not the lib

It's working now!. Thanks!

@ni3galave How you make it working? I tried react-native-svg 9.9.2, but have "Unable to resolve "./React Native SVG" from "node_modules/react-native-svg/src/index.ts"" issue :(


fix fi overflow cause index out of range panic

sql: delete from db.table where col1 in(1) and col2=1;
Fingerprint(sql) got panic: index out of range

after fix

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.samuelzan seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Hello.
I have a question. What happens with the same query if there is no ; at the end?

Hello.
I have a question. What happens with the same query if there is no ; at the end?
The original code fails anyway?

it will panic too.


Pylance not detecting manim import

It might be something to do with the venv.

Hi, is there a manual workaround for that?
(I have cloned the repo into a Codespace but maim is not getting detected)

btw. great repo, this would make it much easier to get started with manim


When we install clusterpedia, it restarts for more than 5 times

What happened?

When we install clusterpedia, it restarts for more than 5 times
What did you expect to happen?
The Clusterpedia not restart, it wait for mysql ready.
How can we reproduce it (as minimally and precisely as possible)?
Install Clusterpeida using Kpanda
Anything else we need to know?
We can use initContainer to check mysql ready. Eg:

https://stackoverflow.com/questions/65646499/how-to-use-an-init-container-to-check-if-mysql-is-ready-for-connections
https://www.magalix.com/blog/kubernetes-patterns-the-init-container-pattern

Clusterpedia Version

# replace the pod name of clusterpedia apiserver
$ kubectl -n clusterpedia-system exec {clusterpedia-apiserver-pod-name} -- apiserver --version=raw
# paste output here

# replace the pod name of clusterpedia clustersynchro-manager
$ kubectl -n clusterpedia-system exec {clustersynchro-manager-pod-name} -- clustersynchro-manager --version=raw
# paste output here

Host Kubernetes version

$ kubectl version
# paste output here

@yankay Thanks for your feedback!
Since clusterpedia is not restricted to using mysql or postgres, and it is difficult to tell in the simplest manifest whether mysql or postgres is being used.
In the future, the type of storage component will be determined in the helm and the readiness of the storage component will be checked in advance. https://github.com/clusterpedia-io/clusterpedia/pull/53
It will also be considered that the storage layer will provide the checks on the storage components, and then the url for health checks will reflect the readiness of the storage components.

#134  has not been merged, and the issue does not affect the 0.3 release, ready to be fixed in 0.4

@Iceber @RuliXu  I think the pr  is better.  close the previous pr.


Question: "docker-storage-setup" and "is_atomic" var

Since https://github.com/openshift/openshift-ansible-contrib/pull/839 openshift.common.is_atomic variable should be defined in order to invoke docker-storage-setup role.
Is this variable should be provided manually by the user? or is there an automated way to provide it?
thanks.

@gbenhaim , the openshift.common.is_atomic variable is defined during the playbook's default pre-tasks when facts are collected on the hosts. The code that sets the variable is in one of the openshift-ansible rpms we have as a prerequisite. When that rpm is installed on the host you run your playbook from, it engages during the gather facts stage.

@cwilkers Thanks for the quick response.
I'm a bit confused now. How do I incorporate between the byo playbook and the docker-storage-setup role?

Are you seeing an error around the openshift.common.is_atomic variable?
For typical usage, I can offer the RHV playbook.

I'm using docker-storage-setup role with a custom playbook.
The following task was missing from my playbook:
- name: Gather facts
  openshift_facts:
    role: common

Do you think that it's worth to mention this in the docs of the role?

@gbenhaim Good catch--I'd missed that the rhsm-timeout role was calling that module.
I think rather than a documentation fix, it might be better to have a test for whether the openshift.common.is_atomic variable is defined, and if not, call that module at the start of the role.
I'll open this up for discussion with the team.
@e-minguez @dav1x @cooktheryan @markllama @pschiffe

Seems like openshift.common.is_atomic was dropped in the ansible modules of Openshift 3.9.

@gbenhaim Thanks for the heads-up, the existing test is trivial enough that we can duplicate it in our roles, but it will require some coordination.

 SPIKE Fix deprecation of is_atomic variable in ocp 3.9

I was running in to this problem trying to install openshift 3.9. I worked around the issue by modifying /usr/share/ansible/openshift-ansible/roles/openshift_facts/library/openshift_facts.py. In the function set_aggregate_facts I added facts['common']['is_atomic'] = False  right after the line that says facts['common']['all_hostnames'] = list(all_hostnames) .

With 3.9, the Docker setup is handled in OpenShift's prerequisites.yml playbook. See 'example/inventory' in the rhv-ansible subdirectory for variables to pass in.


Update pubspec.yaml

Quiver has released version 0.22, which is compatible with this package

I have disabled "Create merge commit" in the settings, so only squashing and merging is possible.  I think that is what you want for pull commits.

This issue also has a status line in pkg/pkgbuild.status that should be removed when the new version is rolled in DEPS.

Thanks, Bill!


Expand union type support

Following up on our discussion here: https://github.com/chanzuckerberg/sorbet-coerce/issues/71
My first attempt at getting something to work for this. One interesting case that came up while testing this is that String is able to coerce pretty much anything (a hash, an integer, etc.). It seems like this could cause some confusing behavior around T.any(String, Integer). I kind of cheated my around this by rearranging the test. Another option would be to always check T.any in the order of T.any(...String, NilClass) if needed.
I'm definitely open to feedback on this approach!

Thanks @billy1kaplan, I'll try to get to this today. Something else has broken CI so I've opened a PR to resolve that first! (#74)

Thank you for the review!
I'll try to create an iteration on this early next week. I also saw the comment here: https://github.com/chanzuckerberg/sorbet-coerce/issues/71#issuecomment-1220909337 and will make this an optional, opt-in feature.


Issues with Prices endpoints

From a user on Discord:

some scenarios where prices fails to return an array of prices and its resulting in me needing to retry.

Example given:
curl --location 'https://data.spiceai.io/v0.1/prices/ETH-CAD?preview=true&granularity=5m&start=1679169371' \
--header 'X-API-KEY: $API_KEY'

Docs mentioned: https://docs.spice.ai/sdks/node.js-sdk/api-reference

spice.js is only client with price documentation in https://docs.spice.ai/sdks/node.js-sdk/api-reference. They appear to still reference v0.1 (e.g. getPrice is a v0.1 only endpoint).

Closing as we are tracking the -CAD issue elsewhere.


[SPARK-23060][Python] New feature - apply method to extend rdd's functionality

What changes were proposed in this pull request?
Extend the RDD class with the method apply.
This method should be like the pipe operator, attached to the RDD class itself.
Example:
def foo(rdd):  return rdd.map(lambda x: x.split('|')).filter(lambda x: x[0] == 'ERROR')
rdd = sc.parallelize(['ERROR|10', 'ERROR|12', 'WARNING|10', 'INFO|2'])
result = rdd.apply(foo)
result.collect()
[('ERROR', '10'), ('ERROR', '12')]
The idea is to have something like this:
https://pandas.pydata.org/pandas-docs/stable/generated/pandas.DataFrame.pipe.html?
How was this patch tested?
Manual tests. Easy patch.
Please review http://spark.apache.org/contributing.html before opening a pull request.

I'm +1 to @srowen on this, I don't believe this is a change we're going to make to the API. @gianmarcodonetti please close this PR.


Temp Kubernetes configuration file is group-readable

Helm install (v3) produces following warning when used from within kubeswitch:
WARNING: Kubernetes configuration file is group-readable. This is insecure. Location: /Users/user/.kube/tmp/config_1603788223111952000

Thanks for reporting the issue. I’ll make the fix.

Fixed in latest v0.1.3 release.
https://github.com/trankchung/kubeswitch/releases/tag/v0.1.3


Publish 10/07/2019 10:30 AM PST

Docs Build status updates of commit 452c9a9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2012r2-ps/msdtc/Add-DtcClusterTMMapping.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/nettcpip/Get-NetTCPSetting.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Deletes unused test scripts

PR Type
Removes scripts that perform similar functionality to that in tests.unit.
I was looking through the folder and started refactoring legacy code when I realized that most of the stuff isn't doing anything that isn't provided by tests.unit and since they're not referenced anywhere within the actual project, I just went ahead and deleted them.
Why is this change necessary?
Reduces the overall weight of the project. The scripts are basically just copies of things found in tests.unit and don't belong in the project. There aren't any usages of them anywhere that I could find.
If they're for testing individually for something the daemon does at runtime, they should either be moved off the git tree and into local storage or in a directory tests/runtime-tests/.

Thanks for the PR @osilkin98! We love ones that delete unncessary lines of code :)

I don't use these scripts but I think others might. @jackrobison @shyba please chime in if you want to keep these.

If we're using the scripts then we should keep this. They also serve as examples on how to use the SDK / APIs.

@eukreign The scripts in this PR are very dated, using python 2 syntax and also don't provide any good example or insight as to how the internals work.

Can we delete them one at a time as equivalent functionality is added to daemon and/or updated versions of the scripts that do the same things are written?

There are already equivalent scripts in the tests directory. I do agree that we should have examples, so maybe we could create a separate examples directory at the root level and put new ones there? These should still be deleted though, they're not very explanatory and I wouldn't say that they constitute as good examples.


Tracking LAS phasediff issue

What happened?
From Neurostars user sabarnes I received a dataset with phasediff fieldmaps. This seemed potentially related to #325, but the BOLD/phasediff files have the same orientation/PE axis.

Docker
Is your data BIDS valid?
Yes
Are you reusing any previously computed results?
No
Please copy and paste any relevant log output.
230310-15:22:13,46 nipype.workflow INFO:
         [Node] Setting-up "fmriprep_23_0_wf.single_subject_020_wf.fmap_preproc_wf.wf_FM0.bs_filter" in "/scratch/fmriprep_23_0_wf/single_subject_020_wf/fmap_preproc_wf/wf_FM0/bs_filter".
230310-15:22:13,59 nipype.workflow INFO:
         [Node] Executing "bs_filter" <sdcflows.interfaces.bspline.BSplineApprox>
230310-15:22:13,201 nipype.interface INFO:
         Resampling image with resolution exceeding 'zooms_min' (2.5x2.5x2.5 → 4.0x4.0x4.0).
230310-15:22:13,312 nipype.workflow INFO:
         [Node] Finished "bs_filter", elapsed time 0.244008s.
230310-15:22:13,312 nipype.workflow WARNING:
         Storing result file without outputs
230310-15:22:13,313 nipype.workflow WARNING:
         [Node] Error on "fmriprep_23_0_wf.single_subject_020_wf.fmap_preproc_wf.wf_FM0.bs_filter" (/scratch/fmriprep_23_0_wf/single_subject_020_wf/fmap_preproc_wf/wf_FM0/bs_filter)
230310-15:22:14,676 nipype.workflow INFO:
         [Job 42] Completed (fmriprep_23_0_wf.single_subject_020_wf.anat_preproc_wf.anat_template_wf.anat_conform).
230310-15:22:14,676 nipype.workflow INFO:
         [Job 93] Completed (fmriprep_23_0_wf.single_subject_020_wf.fmap_preproc_wf.wf_FM2.phdiff_wf.compfmap).
230310-15:22:14,678 nipype.workflow ERROR:
         Node bs_filter failed to run on host ac5e290ca5c4.
230310-15:22:14,678 nipype.workflow ERROR:
         Saving crash info to /out/sub-020/log/20230310-152125_e9f102d3-4210-4f29-92a9-288ff8476345/crash-20230310-152214-root-bs_filter-9ed1d984-b646-4a9e-80c8-b947263a88a6.txt
Traceback (most recent call last):
  File "/opt/conda/lib/python3.9/site-packages/nipype/pipeline/plugins/multiproc.py", line 67, in run_node
    result["result"] = node.run(updatehash=updatehash)
  File "/opt/conda/lib/python3.9/site-packages/nipype/pipeline/engine/nodes.py", line 527, in run
    result = self._run_interface(execute=True)
  File "/opt/conda/lib/python3.9/site-packages/nipype/pipeline/engine/nodes.py", line 645, in _run_interface
    return self._run_command(execute)
  File "/opt/conda/lib/python3.9/site-packages/nipype/pipeline/engine/nodes.py", line 771, in _run_command
    raise NodeExecutionError(msg)
nipype.pipeline.engine.nodes.NodeExecutionError: Exception raised while executing Node bs_filter.

Traceback:
        Traceback (most recent call last):
          File "/opt/conda/lib/python3.9/site-packages/nipype/interfaces/base/core.py", line 398, in run
            runtime = self._run_interface(runtime)
          File "/opt/conda/lib/python3.9/site-packages/sdcflows/interfaces/bspline.py", line 187, in _run_interface                                                                                data -= np.squeeze(mode(data[mask]).mode)
        ValueError: operands could not be broadcast together with shapes (52,52,32) (0,) (52,52,32)

Additional information / screenshots
No response

The issue seems to be with extremely poor masking of the magnitude images, driven by a very low-SNR magnitude2 image. Deleting it passes this step. We'll see if the final result is sensible.

I have been very strongly considering synthstrip ... :sweat:

IMO we just don't need to merge magnitude images. Magnitude1, which is required for phase1/2 and phasediff, seems like it should be enough.


Implementation of DCLW

Hi,
I saw your implementation of the nominator of DCLW weight_fn, which uses element multiplication for the z1 and z2 calculation.
But in the paper, the nominator of DCLW weight_fn formula is: exp(<z1, z2> / sigma). Can you tell me why you use element multiplication for the z1 and z2 calculation instead of torch.mm(z1, z2) or dot multiplication?
Thanks.

Hi @wqtwjt1996, thanks for the interest and question. I believe this is the corresponding part in the paper:
As you can see, the nominator includes multiplication of each element in z1 into the corresponding element of z2.  Please let me know if this doesn't make sense. Otherwise, I'll close the ticket.

Hi @raminnakhli Thanks for the reply.

As you can see from the two pictures, the formulas are similar which all include: exp(<z_1, z_2>). But in your code, for formula(5) (6) you use matrix multiplication, and for the formula of w(z_1, z_2), you use element multiplication.
Could you please explain to me why? Thanks!

Hi @wqtwjt1996,
The matrix multiplication is for the calculation of the U variable that you can see in the denominator of the equation (5). As you can see, in line 27 of the loss file, I'm using the diag to get the diagonal elements which is equivalent to performing element-wise multiplication.
positive_loss = -torch.diag(cross_view_distance) / self.temperature
I believe this would answer the question. Therefore, I'd close the ticket. Feel free to follow up.


WFLY-11698 fixed max-delivery-attempts settings by TransactionTimeout…

…QueueSetupTask
https://issues.jboss.org/browse/WFLY-11698

Fixed, thanks.
Just a note that I omitted all the changes around SystemException handling as it looks it can't be hit after WFTC-58 change anymore.

@istudens one more thing, can you add one more change to your PR, please? It's what @bstansberry  asked at my original PR (https://github.com/jbosstm/narayana/pull/1403) which is make the ModelNode amqServerDefaultAdress private?

@ochaloup done


dspy.LM system prompt old behaviour

I'm trying to migrate to dspy.LM. I noticed that I now have to pass the system prompt in the messages that I send to the module. I'd like to suggest that we allow users to still pass the system prompt in dspy.LM, since we still support passing only the prompt, not just the messages list.
I think it'd be nice to make a small change in dspy.LM at ~ line 35
        cache = kwargs.pop("cache", self.cache)
        messages = messages or [{"role": "user", "content": prompt}]
        # ------------------------------------------------------------------------------------
        if self.system_prompt:
            assert messages[0]["role"] == "user", "System prompt was passed on model intialization and in messages."
            messages.insert(0, {"role": "system", "content": self.system_prompt})
        # -------------------------------------------------------------------------------------
        kwargs = {**self.kwargs, **kwargs}

These two lines will allows users who want to use system prompt to pass system prompt on dspy.LM and then only prompting the module with a prompt

Hey @MohammedAlhajji , this doesn't work anymore because DSPy builds a system prompt now.
What you may be trying to do is to build a new Adapter. See the migration guide: https://github.com/stanfordnlp/dspy/blob/main/examples/migration.ipynb
Let me know if this addresses your goal!

Thanks for your response. Sorry my bad. Should've read that migration guide


How to use PostgreSQL database as a MultitenantStore?

Is it possible to keep TenantInfo data in a custom database without using EF?

Hi again, it certainly is but you will have to implement IMultiTenantStore<T> yourself. It's not a complicated interface so you should be able to use it easily. You can also use EF Core with a Postgres database which works pretty well. The library for that is Npsql.


Proposal: Rename registers to mappings

The register fields are mostly used to translate from integer values to strings.
While often this is done for registers, sometimes it is not, for example with the RV32 fence instruction.
Renaming registers to mappings makes it maybe more clear that they can be used in other situations as well (e.g., translating a state value to a state name).

yeah, makes a lot of sense, since I changed the way that "registers" work to allow that sort of functionality, renaming them to something like mappings is the play


[Gekidou] pinned posts

Summary
Pinned posts 🚀
Ticket Link
https://mattermost.atlassian.net/browse/MM-44653
NONE

-->

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/mattermost-mobile/22965
https://circleci.com/gh/mattermost/mattermost-mobile/22964

Artifact links:
https://output.circle-artifacts.com/output/job/d998f50a-7ac9-418d-bd66-3a95cab89d6a/artifacts/0/tmp/artifacts/Mattermost_Beta.apk
https://output.circle-artifacts.com/output/job/7f31facc-cdb4-4d8a-81d3-920cd8ff2405/artifacts/0/tmp/artifacts/Mattermost_Beta.ipa

https://user-images.githubusercontent.com/456511/171761670-abc2b2e6-d51a-4ac3-9e4d-4f3a36aa399f.mp4
^^ Small bug - the Pinned Messages Count isn't being observed / updated when pinning / unpinning a message.


[minor] portico: Cleanups.

We added custom event handlers on anchor tags to show transitions
when switching between pages, a behaviour we have since removes in
commit a0dacea81193600983e55c60fe8265ed061abb04.
Our approach didn't respect the target attribute for links and other
defaults that browsers offer with links.
We can now safelhy remove the event handler and restore the default
behavior of anchor tags.
Also removed scroll-to attribute support.

Merged, after fixing a typo, thanks @aero31aero!


Update Flatpak Use & Install instruction.

Description
Since Flatpak is a chicken and egg situation that requires releases before testing, let's update the use and install instructions on the README.md.
Page URL Address
No response
Expected Presentation
Flatpak has install and use instruction.
Current Behavior
Flatpak does not have install & use instruction.
Associated Data Files
No response

Already updated.

Steps published.


[DO NOT MERGE] Refactor overall

New structure

uniflow

node
op

extract

load

pdf.py
txt.py
OCR.py

split

markdown_header_split.py
semantic_model_split.py

transform

gen_qa (openai, huggingface, ...)
summurize (openai, huggingface, ...)
etc.

autorate

flow

generalFlow([extract..., split..., transform...])
extract_pdf_gen_qa
extract_pdf_summerize
extract_txt_gen_qa
extract_txt_summerize

I will close this PR and reopen a new PR #59 for more clear code review.

Closed


Update backup/restore db scripts

The existing scripts need to be updated to take into account the swarm infrastructure

Not required anymore because we migrated to Mongo Atlas.


ClearTemporaryVideo process cannot access the file because it is being used by another process

[WinError 32] The process cannot access the file because it is being used by another process: 'E:\VideoTmp\2024-06-28_16-47-06\behavior\raw.harp\BehaviorEvents\Event_79.bin'
@XX-Yin Happening on ephys3

This happens regularly on computers with high speed cameras. @XX-Yin Can we make an SOP for opening/closing the camera?

Yes, I will also try to repeat this error.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Alex Piet @.>
Sent: Friday, July 19, 2024 8:36:55 AM
To: AllenNeuralDynamics/dynamic-foraging-task @.>
Cc: Xinxin Yin @.>; Mention @.>
Subject: Re: [AllenNeuralDynamics/dynamic-foraging-task] ClearTemporaryVideo process cannot access the file because it is being used by another process (Issue #576)
This happens regularly on computers with high speed cameras. @XX-Yinhttps://github.com/XX-Yin Can we make an SOP for opening/closing the camera?
—
Reply to this email directly, view it on GitHubhttps://github.com/AllenNeuralDynamics/dynamic-foraging-task/issues/576#issuecomment-2239475889, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A2CTX5W7N53C7EK6AUQVWGTZNEXBPAVCNFSM6AAAAABKGFYF6SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZZGQ3TKOBYHE.
You are receiving this because you were mentioned.Message ID: @.***>

@ellahiltonvano Can you add your camera SOP here

Camera SOP here, needs editing from scientists before implementation @XX-Yin @ZhixiaoSu

@ellahiltonvano Thanks for writing the camera SOP. Let's keep these steps until we update the preview.

I haven't seen this question recently. The error may be caused by my testing.

Happened alot this week:
1	[WinError 32] The process cannot access the file because it is being used by another process: 'C:\Users\svc_aind_behavior\Documents\temporaryvideo\2024-07-24_09-17-07\behavior-videos\bottom_camera.avi'	Dialogs.py	_ClearTemporaryVideo	line 1270	6B
1	[WinError 32] The process cannot access the file because it is being used by another process: 'C:\Users\svc_aind_behavior\Documents\temporaryvideo\2024-07-24_10-02-59\behavior\raw.harp\BehaviorEvents\Event_44.bin'	Dialogs.py	_ClearTemporaryVideo	line 1270	7B
1	[WinError 32] The process cannot access the file because it is being used by another process: 'C:\Users\svc_aind_behavior\Documents\temporaryvideo\2024-07-24_10-37-21\behavior-videos\bottom_camera.avi'	Dialogs.py	_ClearTemporaryVideo	line 1270	6B
1	[WinError 32] The process cannot access the file because it is being used by another process: 'D:\dynamic-foraging\temporaryvideo\2024-07-25_13-22-41\behavior-videos\bottom_camera.avi'	Dialogs.py	_ClearTemporaryVideo	line 1270	ephys1
1	[WinError 32] The process cannot access the file because it is being used by another process: 'E:\VideoTmp\2024-07-26_15-05-56\behavior-videos\body_camera.avi'	Dialogs.py	_ClearTemporaryVideo	line 1270	ephys3
1	[WinError 32] The process cannot access the file because it is being used by another process: 'E:\VideoTmp\2024-07-26_15-07-11\behavior-videos\body_camera.avi'	Dialogs.py	_ClearTemporaryVideo	line 1270	ephys3

Ok, this seems to happen when deleting the temporary video file (Preview). It will be fixed by the new preview methods.


feat(carousel): implement flexbox

Offload some JS computation onto CSS
Fix layout issues in Firefox

Coverage decreased (-0.09%) to 80.769% when pulling 642ed706b23bb1ecc18c0b1d9ea28b7a48be430b on feat/carousel/flexbox into a5e184b97700793c525e165892adc0b1ec5f9585 on master.


Can't seem to get it working

Hello, I can't seem to get this working properly. I installed multimc-git and multimc-curseforge from the AUR, but when I visit a modpack page on curseforge, the install button send me to download cursforge client. Is there some extra step I need to take? I would also be okay with somehow creating the curseforge:// link, but I have no idea how they are constructed.
Bellow is an image with the link of the Install button on curseforge.

Everything is correct, clicking thst button should prompt with a pop-up of which application to handle the custom protocol, if that's not happening try using a different browser, it might be a problem with the browser.

I have tried Firefox, Firefox in private window and Google Chrome (I have no addons in Chrome), but got the same result.

What desktop environment are you using, maybe it's not recognizing the protocol option in the desktop file

I'm not using a desktop environment. Maybe I need to import the file handler manualy?

You need xdg-open installed to handle protocols


Distinguish error cases for same tx vs other tx spent some inputs

This is just an idea for now...
It may be helpful for wallets (and similar clients) that need to re-submit txs for them to be able to get a distinction in the error they get when they submit a duplicate transaction.
Why can this happen? If a wallet notices a tx has been pending for too long, it can try to resubmit it. What outcomes can it expect? It may be that the tx is simply invalid, or that an input was spent by another transaction. It may be that the tx is still in the local mempool or that the tx has in fact made it into the chain. Wallets would like to distinguish the last two cases from the case that an input was spent by another transaction. Currently that is not possible, at least not just from the error reported by the ledger.
Right now, in Byron and Shelley ledgers, if you submit a duplicate tx then the failure you get is that one of the inputs has been spent already. Would it be possible -- without it being too expensive -- to establish if it's not just that some of the inputs were spent, but that all of them were spent and by the same txid as we are validating?
Perhaps this is not possible in general, because the UTxO does not keep that information. But perhaps there are common special cases where it is possible? Assume the special case that the tx being resubmitted is already in the ledger, and that not all of its outputs have themselves yet been spent (typically none yet spent). The UTxO is indexed by the (txid, ix) pair, but since it is an ordered map we can still do a map range lookup for any (txid, _) pair in the utxo, in log time. That lets us test if a specific txid is still in the utxo. If it is, we can confidently say that this is a duplicate, and not merely that some inputs were already spent.
Perhaps it does not want to be in the ledger validation itself, but available as a test that the mempool validation could use, just for the local client case, to more helpfully report distinguished failures.

We might consider moving this issue to ouroboros-network.


dyn/impl syntax (incl. old dyn-less trait objects) doesn't always allow trailing +.

Except for the case in #120, that #265 fixed, and a similar one with explicit dyn, there doesn't seem to be general support for trailing + in a type (after a path).
That is, Trait+, dyn Trait+ and impl Trait+ are all valid Rust types, but syn doesn't accept them.
This playground contains examples of the syntax being accepted by Rust, and also the failure of syn to parse the input.

This was discovered by creduce (likely) removing the RHS of the +, while doing an automated reduce (https://github.com/rust-lang/rust/issues/89195#issuecomment-932492732).
I tried using rust-reduce on the output of creduce (because of https://github.com/rust-lang/rust/issues/89195#issuecomment-932752698), and it failed with "expected identifier" (because it relies on syn). However, once rustfmt was ran on that file, rust-reduce was able to work again.
To track down the exact bug that was being found, creduce --not-c script.sh current.rs was used with:
#!/usr/bin/env bash

rustc -Z unpretty=normal current.rs > /dev/null || exit 50

timeout 1 rust-reduce true <(cat current.rs) 2>&1| grep "expected ident"

as script.sh, resulting in this reduction, after 65 seconds (down from a 13856 byte file):
mod a {
  mod b {
    impl c { fn d()->e + ; }
  }
}

See also #1073. I think #1074 only fixed this for impl Trait, not dyn Trait/bare trait objects.


Docker Image for armv7?

There were lots of difficulties in trying to package actions-runner inside a docker image for armv7.
What about this alternative? Is it doable? Has someone already done it?

I see so I came to this state:
running as user 'actions-runner'
System has not been booted with systemd as init system (PID 1). Can't operate.
Failed to connect to bus: Host is down
	WARNING: Lingering is not enabled for user 'actions-runner'. Lingering is needed to make user services start at boot and to prevent them from being stopped when user logs out. Enable lingering using command 'loginctl enable-linger'.
mkdir: cannot create directory '/home/actions-runner': Permission denied

So I should not use the systemd wrapper and use the commands from the table github-act-runner configure and github-act-runner run instead?

The entrypoint.sh now looks something like this:
github-act-runner configure --url ${GH_URL} --name "${name}" --token "${GH_TOKEN}" --labels "self-hosted,Linux,$TARGETARCH"
github-act-runner run &

It calls success than disconnects and immediately reruns no matter if there is the & or not