Empty exports

Hi, thanks for your great work. It looks like it's exactly the tool I need right now.
Unfortunately my exports are always empty. It doesn't matter if PNG or PDF and it's also the same if I try to export an image from your demo @ https://printmaps.mpetroff.net/ (I'm running on macOS Mojave 10.14).
Do you have an idea?

What browser / browser version are you using?

I ususally use Chrome and it's also not working in Safari for me. But I just tried in in Firefox and it works well there.

It sounds like some sort of strange WebGL compatibility issue (something I'd expect from Safari, not Chrome).
Can you check to see if any error messages / warnings were printed to the browser's JavaScript developer console?

No, there are no errors in the console. I think it depends to the pixel convertion or the pdf export. I took the relevant pieces from your code, which I need to create custom PNGs and created my own script and it works also in Chrome and Safari (mainly the the toBlob -> saveAs part).

I was able to reproduce this with Safari 12 on macOS 10.13 via Sauce Labs, although I still don't know what's causing it.
I have a 2010 Mac Pro that I sometimes use for testing in Safari, but it tried updating itself to 10.14 despite not having a Metal-compatible graphics card, so it's out of commission until I get around to reinstalling 10.13 (or replacing the graphics card).

Seconded, Safari 12.0.1 on macOS 10.13.6 and google chrome.
canvas.toDataURL() seems to generate a blank image, and in this project, the toBlob also seems to be generating a blank image.

Apparently this is fixed by setting preserveDrawingBuffer option to true in mapboxgl.Map() initialization. Cheers!

It broke when I updated Mapbox GL JS a month ago in d18430c1d51598c1e4f7fa7a45e6ccf1700c73ad, and I apparently only verified the update with Firefox. Since preserveDrawingBuffer has been set to false for years in Mapbox GL JS, it's actually pretty amazing that it's worked up until this point.


database minCapacity: a floating-point number is not considered valid

The property "minCapacity" expected a value of type "int | null" but the provided value is of type "string".
Microsoft.Sql/servers/databases@2022-02-01-preview doesn't accept "float" type minCapacity setting, although the interface supports vCore values of 0.5 or 0.75.
Workaround: I add this parameter as string, but it would be nice to support float as well.

You can use the json() function to avoid the error. i.e. json('0.5'). We have a separate issue tracking supporting floats in #1386, so I'm going to close this one.

OK, thanks for the tip/heads-up!


Update historical allocations

It's been a while since we've pulled in Discord bot grants and folded them into the base allocations file for testnet genesis. Let's run the script to pull down history, and then deduplicate it manually by discord id to minimize filesize. No need to rewrite the CSV handling now, we'll just do a manual update to catch up.
Related: #1783, #1876.

Made some logic changes in https://github.com/penumbra-zone/galileo/pull/32, which unbroke the history subcommand.


ascend.js subfolder

Make ascend.js work with subfolder.

I can't figure out the best way to get line 35 to work (kill scripts) even after adding getFilePath to the imports for runCommand_Custom.


Initialize ctor from properties is being put before the namespace import

which is not ideal I think. Would be nice to have it come after them if possible.

Agree. Wonder if it's the order in which extensions are registered. Wonder
if @simsod knows
On Thu, 1 Dec 2016 at 19:45, Tugberk Ugurlu<EMAIL_ADDRESS>wrote:

[image: image]
https://cloud.githubusercontent.com/assets/328122/20809612/8473eee8-b7fe-11e6-8cd9-252a33bdc49a.png
which is not ideal I think. Would be nice to have it come after them if
possible.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jchannon/csharpextensions/issues/13, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAGapmGzvxy5rvPXTEQL2iCF6xYQsK2Lks5rDyPEgaJpZM4LBzxH
.

@jchannon I think I saw something somewhere in the docs about the order of codeactions, i´ll have a look at it.

I haven´t been able to find any information about how the order of the commands are sorted.
Since it´s a promise you return from the provideCodeActions method I´m guessing it´s just in the order that the promises are returned.
I guess we could do a setTimeout to make the codeActions return slower than omnisharp-vscode  delivers theirs, but that seems really really ugly.


Test/auth unit tests

@MildTomato  - as we spoke some time ago on Discord (sorry for being so late with this, but apparently moving to a different country can be more occupying than I thought :D), this PR introduces new tests for the Auth component. I've added MSW.js to create unit/integration tests, as similar as possible to the actual app behavior. There are still a few TODOS, but I think this is a good starter. Let me know if something doesn't make sense here, happy to discuss :)

Any news on that? :)

Oh yea, looks good. I'll merge it in soon.
I've been busy with the alpha branch, haven't had time to look at this yet.

Hey @MildTomato is this still valid?

@maciekgrzybek i didn't merge it in since we were moving auth component somewhere new.
https://github.com/supabase-community/auth-ui

@maciekgrzybek could you move your PR over there? 🙏

@MildTomato Sure thing, will do it this week :)

Supabase UI is being deprecated.
We are moving the components to the main mono repo at github.com/supabase/supabase.
The auth component has been moved to it's own repo and package.
github.com/supabase-community/auth-ui.
You can also read the docs for auth ui here.
@maciekgrzybek would you be ok to move your unit test spike over there?

sorry, just realized you've already tried to move it over there 🙈


flinkSQL-CDC configuration item

I would like to set some parameters in the flinkSQL-CDC configuration item, such as reading from the latest position of binglog, etc. Where can I find these similar parameter configurations @leonardBang

I'm sorry, I found it


How to get the value of the coverage variables (ef, ep, nf, np) ?

How can I get the values of the four covering variables (ef, ep, nf, np) to run the tarantula heuristic?

I am closing this issue. Feel free to reopen it (or open a new one) if you have further specific questions.

I am closing this issue. Feel free to reopen it (or open a new one) if you have further specific questions.


Umi/molecular_alignment_workflow to subworkflows

Figured it would be easiest to move molecular_qc_workflow.cwl at the same time as the other ones instead of waiting for this to merge


Manual Inclusion in __init__, an alternative to @postinit

Rather than forcing an implementation of __init__ on the user and having them opt into their own __init__ with @postinit, could you instead expose a function which has a method implementation for each module, and the user manually include that in their __init__? This is the approach that BinaryTraits.jl takes, reference here.
I think this would be much less disruptive to users workflows than having them restructure their existing __init__s per package.
I really like the idea that this package enforces the presence of fields as part of an interface, that's an issue I've not found a good solution for, and this looks promising. Thanks for the contribution!

Initially there was an option to not generate any init. But it's quite complicated to find a supertype's module in some situations. So I'm using init to setup the lookup table for the loaded module based on its fullname(...)
To allow manual checking, I'd need to build up a dynamic route map depending on how a set of modules was loaded.

I could have @postinit mangle the name of the function a bit, so all you'd need to do is prepend @postinit to your existing init with no other changes.

some other possible solutions if enough people are concerned about clobbering of init

solution
good
bad

let the user change the name of the verification function through an environment variable
the user has full control
if the function is not called by the user, client modules may not work correctly

the above but implement dynamic discovery of supertype module object
doesn't require init
complex. benefits of changing function name not quite clear

throw an exception when Inherit.jl detects presence of __init__
reduces silent clobbering
cannot eliminate all silent clobbering, i.e. user defines it after Inherit.jl macros execute. may mislead the user to think clobbering cannot happen.

factor out @postinit into a separate module that allows multiple init registrations. Verify that modules loaded after it do not have their own definitions of __init__ (using techniques from Revise.jl?)
have a central place to manage multiple inits. Prevents silent clobbering altogether.
complex


assign_moving_average function explodes when smoothing bounded inputs

System information

Problem encountered when running custom code
Linux Ubuntu 16.04 LTS
TensorFlow installed from source
TensorFlow version 1.8.0
Python 3.5.2
Bazel Version 0.11.1
GCC 4.9.4
CUDA 9.1.85, CuDNN 7.1.2
GPU: Gtx1070, 8GB memory

Describe the problem
In training a network consisting of the architecture described below, I encountered a problem with batch normalization layers. I found that the Keras batch normalization moving average parameters were unstable. While decreasing loss was observed over several iterations, it would regularly explode to a very high value. In observing the batch normalization weights in Tensorboard I notice that the moving mean in batch normalization layers explode at the same iteration of this loss explosion, while the batch statistics remain well behaved as shown in the picture below. This doesn't add up as the moving averages are a stable function of the batch statistics.
I noticed that the only Keras code employed in updating these parameters is the assign_moving_average function in tensorflow/tensorflow/python/training/moving_averages.py. Looking deeper this function in turn uses the assign_sub function in tensorflow/tensorflow/python/ops/state_ops.py which by default does not use any locking and hence its behaviour may be undefined. Perhaps this is the cause, and if so, should the default be to ignore locking? How certain is it that this wont run into undefined behaviour?
When I replace the Tensorflow assign_moving_average function with a simple handcrafted (1-self.momentum)*mean + (self.momentum)*self.moving_mean in the Keras BatchNormalization object, this problem never occurs.
Source code / logs
For reference I am training a three layer convolutional network with three max pooling layers and three batch normalization layers, followed by two dense layers and two batch normalization layers at the output. The network employs a triplet loss and so the weights are each employed three times prior to back propogation. I hope to get a chance some time in the future to reproduce this problem in a simple example that I can share here.
The decay parameter to the tensorflow assign_moving_average function in this case is 0.99.

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce
Mobile device

I guess these parameters need to be in the specific format provided. I've provided the parameters again below, adhering to the template format:
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 16.04 LTS
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): Source
TensorFlow version (use command below): 1.8.0
Python version: 3.5.2
Bazel version (if compiling from source): 0.11.1
GCC/Compiler version (if compiling from source): 4.9.4
CUDA/cuDNN version: CUDA 9.1.85, CuDNN 7.1.2
GPU model and memory: Gtx1070, 8GB memory
Exact command to reproduce: N/A

@brainnoise Is there enough information here to go on?

Nagging Assignee @poxvoculi: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @fchollet: It has been 29 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @fchollet: It has been 44 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @fchollet: It has been 59 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @fchollet: It has been 74 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!

@MCMcCallum Is your handcrafted update fix still working?

I am facing the name problem. The moving_mean and moving_variance go to nan at training time and the effect of this can only be seen during the evaluation time. I am not using keras I am coding in tensorflow APIs and I am using tf.layers.batch_normalization. Is there a workaround?


Remove scripts used to verify and build the UI in the midstream

As discussed here: https://github.com/redhat-developer/app-services-ui/pull/106 the midstream scripts shouldn't be in the upstream repo

This PR will also end up deleting the same scripts from the midstream, so we will have to create another PR in the midstream to immediately restore these scripts there

Looks good

It looks good to me too. Ping @pmuir


✨ Add per-port rx and tx metrics for switches

@charlie-haley I've got another small tweak to add a name label to the switch port metrics. I can add it to this PR or make another PR, according to your preference.

Feel free to add it to the same PR


Suggestion list

Hi, there is my suggestion for this plugin:

Move this project on spigot.
Make a better system of detecting IPs (actual is very bad). So this new detect system will check EVERY words wrote in a text of a player. For Example: "Hi, join in Mc.servername.it!", your plugin will search if this ip works, and if work, the plugin will stop the text from its send. If the text is: "Hi, Join in Mc.serverthatnotexist.it" the plugin will see that this ip is not real and it doesn't stop the send of the text.
Make a new config: whitelist ip, blacklist ip, custom command on detect, and other.
Make a system that understand when a player spam an ip. Example: "Join in mc server it (with space)" or "Mc|server|it" or "Mc-server-it" or also "Mc0 servername0 it". So the plugin will understand that the player tried to advertise.
Make ALL messages custom in the config.
Make a system that block the visibility of the messages when a player try to advertise. Example: "Join in server.it", so the plugin report this to all staff and save it on the file in the folder, all this, while all player can't see the message of the player that tried to advertise.
System that detect words also in books, signs, chat and under commands (config as Commandtocheck: (list of commands)).
System that execute a custom commands (also multiple commands) when a player advertise, from config.
Feature that replace words with other words, like this example:
#OriginalWord - Word to replace
Hello - Hi
Add a singular check for AntiSpam, AntiCommandSpam and AntiCaps with custom commands for each one, like this example:
#AntiSpam
EnableCheck: true
SecondsFOrEachMessages: 3
CommandsOnPunish:

Mute %player Spam is not allowed!
Broadcast %player was muted for spam!
And same thig with other 2 checks.

In this mode this plugin has not false positive, ad hi, i'm ...player .. (points go detect as advertise), and it will be the best.

Move this project on spigot.

I am not sure what you mean? it is already on spigot - follow the link on the readme.

Make a better system of detecting IPs (actual is very bad). So this new detect system will check EVERY words wrote in a text of a player. For Example: "Hi, join in Mc.servername.it!", your plugin will search if this ip works, and if work, the plugin will stop the text from its send. If the text is: "Hi, Join in Mc.serverthatnotexist.it" the plugin will see that this ip is not real and it doesn't stop the send of the text.

The ip checker is already really powerfull, if you have any issues with the current ip checker please create a new issue with what you posted.
If this is to actually verify that there is a minecraft server at the IP, I am note sure that is the best idea - why should it check the ip?

Make a new config: whitelist ip, blacklist ip, custom command on detect, and other.

We already make you whitelist certain domains.
In which case would you blacklist a  ip address which already should be blocked because it is in the whitelist?

Make a system that understand when a player spam an ip. Example: "Join in mc server it (with space)" or "Mc|server|it" or "Mc-server-it" or also "Mc0 servername0 it". So the plugin will understand that the player tried to advertise.

That is already in place - fell free to make a new issue with which cases isn't found.

Make ALL messages custom in the config.

That is on the roadMap to do at some point, as stated on the README we had multiple different languages, if you don't like a transaltion please state so.

Make a system that block the visibility of the messages when a player try to advertise. Example: "Join in server.it", so the plugin report this to all staff and save it on the file in the folder, all this, while all player can't see the message of the player that tried to advertise.

That is already in place, if this does not work create an issue.

System that detect words also in books, signs, chat and under commands (config as Commandtocheck: (list of commands)).

I am not sure what you mean, if you mean Detected-Comman then we already have than in the config.

System that execute a custom commands (also multiple commands) when a player advertise, from config.
Feature that replace words with other words, like this example:
#OriginalWord - Word to replace
Hello - Hi

This is made in #81

Add a singular check for AntiSpam, AntiCommandSpam and AntiCaps with custom commands for each one, like this example:
#AntiSpam
EnableCheck: true
SecondsFOrEachMessages: 3
CommandsOnPunish:
Mute %player Spam is not allowed!
Broadcast %player was muted for spam!
And same thig with other 2 checks.

Why have it as one?

I Think that you should redevelop the plugin with these suggestion, because your plug-in has lots of false positive .-.

Saying I should redevelop it doesn't really help anything -
What is that a comment to?
please use markdown to show which comment you are refering to.

No No, i only said that you should develop a plugin with my suggestions, to see if the plugin created is better than yours

The code fi by how project has not really been touched in 2 years, I am no longer in bukkit/spigot so it is only of intesrt I have came back to it

So none for my suggestion?

I have added comments on all of them, and most are waiting on more information from you

Can you resume your plugin on spigot and not on bukkit? It will have more followers
IpChecker have lots of bugs, this because this is not an efficent blocking system.
3 \
4 \
5 \
6 \
7 \
8 \
I mean that you can create a system that detect an ip wrote on books and signs.
Because if owners use an Anticheat plugin (that already block spam) you can disable spam protection from here.

The code is the same, as described in the last issue https://github.com/antiAD/AntiAd/issues/77#issuecomment-370253113 there was a prerelease.

I need something precise.

you can already do that.

already possible

2, most important
A NEW ip blocker
I spam an ip, so AntiAd will check if the spammed ip exist, if yes, it will be execute the command.
If not, none will happen. Only this.

as said examples and make a issue on it.

sorry, how i can use the whitelisted ips?
Can you send me an example of the format by adding 3 random ip?

use the command /antiad add

Mhm, /reload and command detection list doesn't work :\

I need more info - link a config and log

None, 0 errors and default config.
It works only if i Restart all server

share a config and the server log, so ican see what the problem is.


[stable/spinnaker] Specify --wait-for-completion on hal deploy apply

Is your feature request related to a problem? Please describe.
I use helm template to create a Spinnaker manifest then deploy it with kubectl apply. I'd like my build process to wait for all Spinnaker services deployment finish so that we can run automated test after that. Halyard's hal deploy apply command has --wait-for-completion for this but I don't see a way to pass this flag through the helm chart.
Describe the solution you'd like
I'd like to be able to give --wait-for-completion to hal deploy apply when helm template.
Describe alternatives you've considered
Can't think of a nicer way to achieve it...
Additional context
Add any other context or screenshots about the feature request here.

PR #19205 to address this feature request hasn't been reviewed yet...


⚠ Bofrost.de has degraded performance

In 5d1e9e7, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1155 ms

Resolved: Bofrost.de performance has improved in 94500fd after 6 minutes.


File Locksmith not showing up in context menu

Microsoft PowerToys version
0.76.0
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
File Locksmith
Steps to reproduce
Having enabled File Locksmith and right clicking a file
✔ Expected Behavior
"What's using this file" context menu option in right click menu
❌ Actual Behavior
No such option is presented in context menu
Other Software
No response

Hello and thanks for your message. Can you tell us the setting for "Show in default menu" or "Show in extended menu"? And maybe a screenshot of the menu that pops up would be helpful. Thanks :) /needinfo

Hello!
Setting is Show in default and extended menu(but I tried with both options with no result). So, I opened a .jpg file and then right clicked it, and sorry for my context menu not being in English (:
aaand if i am being correct, the option on my language is going to be something like that - "Что использует этот файл?", but there's none

I also had this issue (on Win10), but upgrading to v0.77.0 solved it. (Not sure of my previous version. Maybe v0.75.0 based on the release notes it showed after upgrade?)

@blackkriger could you please update to 0.78, restart your pc, then check if this is still an issue? /needinfo

Still no option from File Locksmith :(

@blackkriger could you submit a /bugreport

Here:
PowerToysReport_2024-02-13-20-41-07.zip

/dup #31347

On windows11.

same issue with 0.85.1 in W10PRO 64 22H2

Same here with LockSmith and PowerRename. Win11 22H2, now I'm trying to update at 23H2; Pro, latest PowerToys version!


Segfault on INDO calculation

Describe the bug

forrtl: severe (174): SIGSEGV, segmentation fault occurred
Image              PC                Routine            Line        Source
libmopac.so.1      00007FD16C637B69  for__signal_handl     Unknown  Unknown
libc.so.6          00007FD16B1EF520  Unknown               Unknown  Unknown
libmopac.so.1      00007FD16B6B4C40  scfmat_               Unknown  Unknown
libmopac.so.1      00007FD16B749A13  iter_                 Unknown  Unknown
libmopac.so.1      00007FD16B6936F9  compfg_               Unknown  Unknown
libmopac.so.1      00007FD16B668F8D  run_mopac_            Unknown  Unknown
mopac<PHONE_NUMBER>401320  Unknown               Unknown  Unknown
mopac<PHONE_NUMBER>4012B2  Unknown               Unknown  Unknown
libc.so.6          00007FD16B1D6D90  Unknown               Unknown  Unknown
libc.so.6          00007FD16B1D6E40  __libc_start_main     Unknown  Unknown
mopac<PHONE_NUMBER>4011BE  Unknown               Unknown  Unknown

To Reproduce
I'm running INDO calculation on system with 300 - 500 atoms. I have successfully run on below ~350 atoms. Above that, I started to see the segfault immediately after execute the mopac operation.
MOPAC executable

[x] Using pre-built executable
[ ] Compiled my own executable

Operating system

[x] Linux
[ ] Mac
[ ] Windows
[ ] Other (specify below)

Additional context
Add any other context about the problem here. (e.g. Linux distribution if that might be relevant)

As a preliminary note, I have reproduced this INDO problem with random large molecules that I have lying around, so it does appear to be generic and only dependent on the size of the molecule.

I am now having a hard time reproducing this problem in a debug setting that would allow me to fix it. I reproduce your crash with the pre-built executable on Mac, but I don't get a crash with the pre-built executable on Linux. I don't get crashes on either OS with debug flags. Could you provide some examples, so that I can check if they crash on my Linux test environment?
While it isn't impossible to fix this without debug flags, it is much less efficient. More information may help me narrow this down faster.

Thanks for looking into this. These are some of the samples: samples.zip. I am able to run on a somewhat small molecule (AAA.mop). I got segfault for the rest of the samples.

Unfortunately, I lost track of this issue last year (got sick & took a long vacation right after), and I'm trying to clean up the list of issues right now.
On further inspection, these are probably out-of-memory crashes. I get crashes on the larger examples in restricted-memory environments, but they are running (slowly) in environments with a large amount of memory. The larger examples here are using over 3 GB of memory. You can reduce the size of the problem by restricting the set of active orbitals in the excitation space using the MAXCI or C.A.S. keywords. Please feel free to reactivate this issue if you are still getting crashes even when the memory usage is reduced or you are in a large-memory environment.
There isn't really a graceful way to handle out-of-memory errors in Fortran unless the program is made aware of the memory limits in advance. MOPAC has sensible defaults and barriers to prevent extremely large amounts of memory from being used, but large CI calculations can get very memory intensive.


Not correctly handling properties with : characters?

Attempting to convert this:
apiVersion: v1
kind: Secret
metadata:
  name: pulumi-api-secret
type: Opaque
stringData:
  accessToken: "<REDACTED: PULUMI_ACCESS_TOKEN>"
---
apiVersion: v1
kind: Secret
metadata:
  name: pulumi-aws-secrets
type: Opaque
stringData:
  AWS_ACCESS_KEY_ID: "<REDACTED: AWS_ACCESS_KEY_ID>"
  AWS_SECRET_ACCESS_KEY: "<REDACTED: AWS_SECRET_ACCESS_KEY>"
---
apiVersion: pulumi.com/v1alpha1
kind: Stack
metadata:
  name: s3-bucket-stack
spec:
  accessTokenSecret: pulumi-api-secret
  envSecrets:
    - pulumi-aws-secrets
  stack: joeduffy/s3-op-project/dev
  initOnCreate: true # noop if stack already exists
  projectRepo: https://github.com/joeduffy/test-s3-op-project
  commit: cc5442870f1195216d6bc340c14f8ae7d28cf3e2
  config:
    aws:region: us-east-2

Fails with this:
Error: Missing attribute separator

  on pcl-859051780.pp line 40:
  40: config = {
  41: aws:region = "us-east-2"

Expected a newline or comma to mark the beginning of the next attribute.

Seems to be a problem with the key being named aws:region (which YAML supports just fine, but perhaps one of our conversion steps doesn't?).
Aside - it's a bit of a shame that we surface pcl-859051780.pp here - since it's not something the user has any control over or understanding of.

Yup, there is some logic that checks for some special characters when deciding whether or not to quote a key: https://github.com/pulumi/kube2pulumi/blob/master/pkg/yaml2pcl/yaml2pcl.go#L352
This needs to be updated for all of the special characters that aren't in the PCL spec. I imagine there's a regex or something in the codegen that could be referenced.

Was looking into the issue, and realized that the Stack defined is a Custom Resource. The declaration isn't being recognized as it is an "unknown resource type", and that causes further issues with parsing while validating the intermediate PCL


fix: multiple UI styling changes

What does it do?

Field label style fixed
Tertiary Icon button color: neutral500 and hover-color: neutral600 updated as per design
Simple menu with tag: IconButton should have tertiary style applied
Select carret down color: neutral500 fixed.
Combobox with no options provided should show default "No results found" value.

Related issue(s)/PR(s)
Fixes: https://github.com/strapi/design-system/issues/1626

@lucasboilly Could you please check deployment link for changes: https://design-system-3lcbqtedh-strapijs.vercel.app/?path=/docs/getting-started-welcome--docs

@madhurisandbhor could you add a 4px spacing above the popover for all variants in the SimpleMenu? Except that, all changes of this PR look good 👍

@lucasboilly @remidej could you please provide your approvals if its good for you, thanks!


Allow user to provide options object in constructor

the last object should be an options object.
Originally posted by @frozeman in https://github.com/ERC725Alliance/erc725.js/pull/12#discussion_r615704473

This was introduced via PR #20


net: ipStackCapabilities.probe creates sockets that can escape into child process

Perhaps I am mistake. But ipStackCapabilities.probe calls socketFunc (which is just a syscall.Socket). Returned socket is inheritable by child process on Windows, so it can escape, if we call Windows CreateProcess at the same time.
Maybe this is expected, but I decided to report. Just in case.
Thank you.
CC @ianlancetaylor @mikioh

@ianlancetaylor you marked this with OS-Windows label, but this affects pretty much everyone.
Alex

Ah, so it does.  Sorry.


Fix net driver response loss on createEndpoint

Fix related to bug: https://github.com/docker/for-linux/issues/348
We should perform updateToStore(ep) after n.addEndpoint or do update twice,
otherwise response from network plugin will not be written to KV storage.
This results in container creation with broken network config.
Signed-off-by: Siarhei Rasiukevich<EMAIL_ADDRESS>

@fcrisciani @ctelfer note that this PR is against the 18.03 bump branch; the equivalent for master is https://github.com/docker/libnetwork/pull/2205. See the discussion on https://github.com/docker/for-linux/issues/348#issuecomment-400265829; this PR should likely be closed, in favor of a cherry-pick if this fix should go out in a 18.03.x patch release

@thaJeztah , thanks. Closed


Did the official demo use better model than vit_h model?

When I provide multiply points for the model sam_vit_h_4b8939.pth, bad result occurs. It's different from the official demo 'https://segment-anything.com/demo'. In official demo, better segment result occurs with more points prompt. So I wonder whether the official demo used better model than sam_vit_h_4b8939.pth.

I also have this question

I also have this question https://github.com/facebookresearch/segment-anything/issues/346

In my case, when I prompt it iteratively (add one point a time and use old mask as prompt), it's result is more like the web demo.
for i in range(1,len(samPoints)):
masks, scores, logits = predictor.predict(
point_coords = np.array(samPoints[0:i]),
point_labels = np.array(pointFlag[0:i]),
mask_input = prev[None, :, :] if i > 1 else None,
multimask_output = False,
)
prev =  logits[np.argmax(scores), :, :]

I'm confused too!!
The later is the demo result.

I use the iterative prompt,and got this.
The result has improved slightly, but it still doesn't meet the requirements compared to the demo.

It seems like less point bright better results??

In my case, when I prompt it iteratively (add one point a time and use old mask as prompt), it's result is more like the web demo.
for i in range(1,len(samPoints)): masks, scores, logits = predictor.predict( point_coords = np.array(samPoints[0:i]), point_labels = np.array(pointFlag[0:i]), mask_input = prev[None, :, :] if i > 1 else None, multimask_output = False, ) prev = logits[np.argmax(scores), :, :]

Hello, may I ask how the result looks after this processing? If you could provide a visual representation, I would greatly appreciate it!

following

@AlexLee1235

In my case, when I prompt it iteratively (add one point a time and use old mask as prompt), it's result is more like the web demo.
for i in range(1,len(samPoints)): masks, scores, logits = predictor.predict( point_coords = np.array(samPoints[0:i]), point_labels = np.array(pointFlag[0:i]), mask_input = prev[None, :, :] if i > 1 else None, multimask_output = False, ) prev = logits[np.argmax(scores), :, :]

Hi friend, thank you very much for your guidance and advice. However, I have a few questions I'd like to ask:
Are the samPoints something you customized?
If they are customized, how should I modify the code to iteratively feed the points into the model? I'd appreciate your guidance, and I would be very grateful if you could provide it.


Add vesting wallet implementation

This adds a vesting wallet implementation based on the OpenZeppelin upgradeable VestingWallet, and a factory contract that creates cloned instances of the base implementation contract for gas optimisation.
The approach of cloning a base implementation, rather than naively deploying a new contract copy on every factory call, reduces gas costs by more than 4x:
Gas Costs (wei)
---
Naive Deployment:    638726
Cloned Deployment:   135756

Resolves #4.

This all looks good to me. I can't think of any other capabilities we might want that aren't supported here. @bghughes Do you have any thoughts?


DATAJDBC-453 - DbActions and AggregateChange are effectively immutable.

Removed the Interpreter and replaced it with AggregateChangeExecutor and  AggregateChangeExecutionContext.
The latter handles the mutable data like ids and versions.
https://jira.spring.io/browse/DATAJDBC-453

That's merged and polished now.


Plugin Unresponsive in Home App - Works in HomeBridge

Alarm.com accessories become unresponsive shortly after adding bridge to Home app. Accessories work in HomeBridge interface. Is anyone else having this issue?
I tried restarting modem, router, and primary home hub. Also, I have removed and added the plugin/bridge multiple times. Last, I tried with both my Hoobs box and then my Mac Mini using HomeBridge. Note these are all on the latest software versions.
This issue is specific to the Alarm.com plugin and not the other plugins that I have.
From my standpoint, there is nothing in the logs that indicate any issues.
This issue started for me a few days ago and I went down the rabbit hole trying to diagnose it. Tried my best to get the required logs, I am a bit of an amateur when it comes to this. If you need anything else, please walk me through it.

Node.js Version:  v20.9.0
Homebridge Version: v1.8.3
Operating System: MacOS

Thanks

It seems like I was able to fix it by reverting to 1.8.0. Strange.

I did not touch any of the settings. It just was not working. I seemed to get everything working somehow. Not sure what happened.

Did you get it working on the latest version or on the downgraded version you mentioned?ChasePlease excuse any grammatical errors as this was sent from my phone.On Jul 11, 2024, at 8:19 AM, weissk93 @.***> wrote:
I did not touch any of the settings. It just was not working. I seemed to get everything working somehow. Not sure what happened.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

No worries on grammar. Excuse my lack of technical understanding.
I think my Hoobs box was on its last legs. When I unplugged it and plugged it back in I could not get it to start back up. Rather than reflashing the SD Card or replacing it, I just decided to rebuild my entire setup on HomeBridge on my Mac mini.
Things were working at first on my Mac mini. Then I noticed that it would stop working randomly just like the Hoobs box.
I tried downgrading the plugin and removing DNS settings on my Eero router. It seems the culprit was making sure my Mac mini did not fall asleep and "stayed on". First time using Homebridge on my Mac mini so rookie mistake.
However, not sure why it stopped working on the Hoobs box to begin with. I do not want to conflate the issues. In any case, things seem to be working fine. Homebridge runs alot better on more capable hardware.


Refactoring change: IsLoopFusion() etc.

Create IsLoopFusion(), IsInputFusion(), IsOutputFusion() and IsCustomFusion() methods.
Tested using //tensorflow/compiler/xla/service/gpu/tests:gpu_fusion_test and //tensorflow/python/compiler

@sana-damani thanks for your contribution , is it possible to combine this PR #26301 with this.

@sana-damani thanks for your contribution , is it possible to combine this PR #26301 with this.

I'd like to keep independent changes separate unless you think there is some benefit in combining the two.

@sana-damani thanks for your contribution , is it possible to combine this PR #26301 with this.

I'd like to keep independent changes separate unless you think there is some benefit in combining the two.

@sana-damani combining helps to fasten the process of review and merging internally, let me know

@sana-damani thanks for your contribution , is it possible to combine this PR #26301 with this.

I'd like to keep independent changes separate unless you think there is some benefit in combining the two.

@sana-damani combining helps to fasten the process of review and merging internally, let me know

@rthadur  Hi, I've been working with Sana and recommended to keep PRs small and focused. I'm doing the internal code reviews and prefer this approach. Does it create overhead for you when triaging PRs, for instance? If so, is there anything we could do to avoid this? I'd like to make sure the process works smoothly for all of us.

@sana-damani thanks for your contribution , is it possible to combine this PR #26301 with this.

I'd like to keep independent changes separate unless you think there is some benefit in combining the two.

@sana-damani combining helps to fasten the process of review and merging internally, let me know

@rthadur Hi, I've been working with Sana and recommended to keep PRs small and focused. I'm doing the internal code reviews and prefer this approach. Does it create overhead for you when triaging PRs, for instance? If so, is there anything we could do to avoid this? I'd like to make sure the process works smoothly for all of us.

@thomasjoerg no issues , i was trying to combine related PR so that it would be easy for everyone. @sana-damani could you please rebase the branch

I had to resolve some conflicts and merge new changes.

Nagging Reviewer : You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 14 days with no activity and the awaiting review label has been applied.


Name changes to include 'dot' in Gruntfile not actually done

After a much too long time I found a strange commit (c27271f) that was supposed to add 'dot' to the name of some renderers. The current master has the changes in the Gruntfile for the build process, but the files have not been changed?

I am not sure to fully understand. Can you make a PR to fix this?

Will do.


Migrate images in body

Based on https://github.com/bm-sms/daimon-news-multi-tenant/pull/208

Closing in favor of https://github.com/bm-sms/daimon-news-multi-tenant/pull/215


Add scam data queries.

Signed-off-by: Joe Shimkus<EMAIL_ADDRESS>

Would it make sense to prefix the scam checks with something like scam?  For example, scam_albireo instead of albireo  There isn't a conflict now, but in the event that there is in the future, it could help avoid it.  It could also help keep the queries together in the doc.


Adding Python test for ReadFromBigQuery transform

This test case will be useful for running comparative tests with BQSource and REadFromBq.
r: @kamilwu

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.


Allow non-ip addr and resolve during flush

This PR allows a hostname:port string to be specified for Addr instead of a net.TCPAddr.
At first, I tried allowing both a net.TCPAddr and optionally a string for the server hostname, but it got messy, and I decided it wasn't worth it.

@kibra, thanks for your PR! By analyzing the blame information on this pull request, we identified @cyberdelia and @dt to be potential reviewers

@kibra this looks like what we were thinking about. least it will re-resolve which is what we want.

:+1: :shipit: :rocket:

I would rather not be notified unless you're planning to merge this upstream.

👍
thanks @kibra
sorry @cyberdelia about our mention-bot pinging you. We do plan to submit this upstream as a separate PR.


chore: remove repetitive words

Description

Related Issue

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Contract Breaking Change (fix or feature that would cause the system to no longer work with contracts currently deployed on live networks)
[ ] Documentation update

Motivation and Context

How Has This Been Tested?

Example Sphinx Deployment Script (if appropriate):

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


Confirm change to canvas size and tell user their canvas will be reset

What really should be happening is the user should be creating a new "file" with a given size, and there should be a confirmation that they want to leave their previous work unsaved.


Cannot create multiple hosts from same AMI

I cannot create more than one host using custom AMI in Rancher.
I have deleted /var/lib/cloud and /etc/ssh/ssh_host* in order to force different fingerprint among different hosts.
Even if I do it manually, it conflicts if both machines come from customized AMI.

If the host was added to Rancher before creating the AMI, you'll need to wipe out /var/lib/rancher/state too.

it worked.


blank home directory in docker container

Here is my .hodorfile
containers:
  littlesis:
    background: false
    image: littlesis/littlesis
    ports:
      - 80:80
      - 81:81
    onetime: true

tasks:
  littlesis:
    sync_project_to: /var/www/littlesis
    cmd: "cd /var/www/littlesis && bash"
    container: "littlesis"

When I run hodor littlesis it launches my container, but everything in my container's home directory (/root) is gone. I only see .bash_profile, .unison, etc.

Could you provide some more details. What OS are you running on, are there other files in your project directory, full hodor output.

Wait, are you saying that /root directory is empty inside of the container? How about /var/www/littlesis, do you see you project files there?


Documentation improvement

The knife vsphere datastore file command doesn't make it clear that you pass the data store as a positional arg, and the file path as a named arg. An example would also be helpful.

Also, the entry for knife vsphere vm find should make it clear what the required arguments are (--pool ??)


vm.expectRevert unable to use custom errors declared outside of contract

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (fb9bc90 2022-08-15T16:31:02.329212Z)
What command(s) is the bug in?
forge test
Operating System
macOS (Apple Silicon)
Describe the bug
To my knowledge, there is no way for vm.expectRevert() to detect a custom error that has been declared outside of a contract although this is a common place to declare them: https://docs.soliditylang.org/en/v0.8.11/contracts.html#errors

Example:
When I define a custom error outside a contract:
pragma solidity ^0.8.13

error CustomError();

contract MyContract {}

and I call
vm.expectRevert(MyContract.CustomError.selector);
I get the following error:

error[9582]: TypeError: Member "CustomError" not found or not visible after argument-dependent lookup in type(contract MyContract).

Only declaring the error inside the contract fixes the error:
pragma solidity ^0.8.13

contract MyContract {
    error CustomError();
}

this error occurs because the CustomError is no longer defined in the scope of the contract, but you are trying to access it as such.
if you are importing the file with the directive in a form of import "some-path.sol", it imports all global symbols (like MyContract & CustoError) into the current scope. the solution is to call vm.expectRevert(CustomError.selector);

i am closing this issue as irrelevant to foundry. otherwise, happy to elaborate more on solidity imports


This File

https://github.com/IlluxDev/Oxide/blob/main/websocket/src/server/ConnectionCloseCodeRange.ts

@EmeraldMike Please recrate the SkylixDev github org. Then rename it to something else such as SkylixDist
When your done please let me know, this may free up the name from github so i may be able to transfer all the repos correctly


Release notes content not loading after application update

Issue Type: Bug
I attempted to update VS Code by clicking on the Code dropdown, selecting Check for Updates, and waiting for VS code to restart.  I expected that after the restart the Release Notes would load in a separate tab and allow me to scroll through the release notes to see the new features and changes as it worked in the previous version. Instead what happens is the Release Notes tab will load, but none of the content is loading in the tab and VS Code will freeze or hang. I wait a few minutes for the tab to finish loading and then I close the tab after it does not load any content.
VS Code version: Code 1.26.0 (4e9361845dc28659923a300945f84731393e210d, 2018-08-13T16:20:44.170Z)
OS version: Darwin x64 17.7.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-5257U CPU @ 2.70GHz (4 x 2700)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
3, 3, 5

Memory (System)
8.00GB (0.03GB free)

Process Argv
/Applications/Visual Studio Code.app/Contents/MacOS/Electron -psn_0_1077511

Screen Reader
no

VM
0%

Extensions (25)

Extension
Author (truncated)
Version

project-manager
ale
8.0.0

color-info
bie
0.5.0

bracket-pair-colorizer
Coe
1.0.59

jshint
dba
0.10.19

vscode-eslint
dba
1.4.12

githistory
don
0.4.2

gitlens
eam
8.5.4

prettier-vscode
esb
1.6.1

code-runner
for
0.9.4

beautify
Hoo
1.3.2

color-highlight
nau
2.3.0

indent-rainbow
ode
7.2.2

polacode
pnp
0.2.2

vscode-css-peek
pra
2.1.0

java
red
0.29.0

vscode-icons
rob
7.25.0

open-in-browser
tec
2.0.0

highlight-matching-tag
vin
0.4.4

vscode-java-debug
vsc
0.11.0

vscode-java-pack
vsc
0.3.0

vscode-java-test
vsc
0.8.0

vscode-maven
vsc
0.10.0

vscode-wakatime
Wak
1.2.3

ReactSnippets
xab
2.2.0

html-css-class-completion
Zig
1.17.1

(1 theme extensions excluded)

Did you have any Python or PowerShell files opened when VS Code is restarted?

I did not have any other files opened when I restarted VS Code.

Are you behind a proxy? Does Code run fine now? Can you run Show Release Notes from the command palette?

I am not behind a proxy. Code updated to 1.2.1 and has a tab in the editor for release notes but there is no text. I ran the command Show Release Notes and it produced the same result.

The screenshot shows the fully expanded Problems panel. Can you close it and screenshot it again?

So the issue was that I dd not close the Problems panel and the release notes were hidden from view.


Should we not allow publisher == advertiser?

currently it's possible for an advertiser to be their own publisher
in effect, they're basically doing a no-op, other than paying the validators
while it looks like something that we maybe shouldn't allow, this may have usecases actually, like if the same entity is acting both as a publisher and an advertiser, but nominates a third party validator to keep their data

not relevant with the new protocol anymore; there's just a channel deposit/creator and an arbitrary balance tree


Rethinking the gftools/fontbakery boundary

This is inspired by something @m4rc1e said elsewhere:

When we finally get gftools imported into fb as a dependency, we'll have easy access to the GF glyph sets/encodings.

This creates a situation where:

gftools depends on fontbakery for running QA tests (gftools qa)
fontbakery depends on gftools for data

A mutual dependency is obviously a Bad Thing from a practical point of view. But from an architectural point of view, it's actually a Good Thing because it forces us to re-evaluate the boundary between the two packages!
Let's step back and think about what is in each package. I'm reminded that @graphicore has often argued (see here amongst other places) that fontbakery is a very generic check runner which provides cool things like dependency injection and profiles and so on, and that nothing should happen to the fontbakery check runner that makes it more font-specific. And that's good. I agree!
But it means we currently have:
fontbakery

generic check runner
knowledge about diagnosing font problems
some miscellaneous font metadata (e.g. fontbakery.constants includes style names, licensing text, name/encoding IDs, GF Latin core glyph list...)

gftools

knowledge about how to fix font problems (often the same problems diagnosed by fontbakery)
assorted font-related tools
some miscellaneous font metadata (e.g. gftools.util.styles includes style names, gftools.constants includes name/encoding IDs, gftools.encodings includes GF Latin core glyph list...)

How can we arrange these different things to encourage code reuse and reduce mutual dependencies?
Here's my suggestion. The suggested package names are terrible but you'll get the idea:
thebakery

generic check runner

gfdata

a "data-only" module similar to opentypespec which contains all the font metadata we use.

fontdoctor
(requires gfdata)

knowledge about how to diagnose font problems.
knowledge about how to fix font problems.

fontbakery
(requires fontdoctor and thebakery)

Font QA engine (diagnosis/testing only)

gftools
(requires fontbakery, fontdoctor and gfdata)

assorted font-related tools
front-end CLI utilities to call fontbakery
front-end CLI utilities to call fontdoctor fixes

"GF Latin core glyph list" is currently included in fontbakery.constants exactly because we did not want to include the entire gftools as a dependency

I think I'm going to put my refactoring work in #397 on hold until we've got this issue sorted. It may make sense to talk about it in tomorrow's meeting. Overall, I like Simon's dependency tree but we could make some tweaks which are worth discussing.

"GF Latin core glyph list" is currently included in fontbakery.constants exactly because we did not want to include the entire gftools as a dependency

Right. If two modules want to have access to the same thing (data, in this case), you can either (a) make one module a dependency of the other, (b) duplicate the thing, or (c) refactor the shared thing into a third module that both can use. You don't want to do (a) (right!), so you did (b). But the idea of Don't Repeat Yourself suggests that (c) is a better approach.

sounds good!

@graphicore has often argued (see here amongst other places) that fontbakery is a very generic check runner which provides cool things like dependency injection and profiles and so on, and that nothing should happen to the fontbakery check runner that makes it more font-specific. And that's good. I agree!

Kindly I disagree, since if its constitutionally a fully generic check runner, that begs the question why we didn't just use one of the 100s of existing fully generic check runners - and my answer to that question has been that we (will) have font specific needs that I want to be "batteries included" with.
If we refactor out thebakery, then I wonder if there are other check runners that will be comparable, and if we do some comparisons, are there things we can improve to be superlative? :)

But it means we currently have:

fontbakery
gftools

How can we arrange these different things to encourage code reuse and reduce mutual dependencies?

I think there's a 3rd piece of this - https://github.com/googlefonts/gf-docs - and since in a way the googlefonts profile is a coded version of the GF Spec, which both includes a lot of technical super detail that isn't in the doc, and lacks a lot of stuff which either can't easily be coded or just hasn't been yet, then, I'd like to advocate that any re-architecting include the human readable docs.
Given topically-unrelated efforts by related people - to make better structured docs for the font format spec, OT features, fontbakery check dogma, etc - perhaps there's something we can do to deduplicate and pattern things so there's a common approach that makes contribution skills more transferrable

I've started drafting a UML diagram for this issue, https://drive.google.com/file/d/1C84Ghabaz7hI1bHlnhCbcOWmhA3O2WQl/view?usp=sharing. Let's work on it in today's meeting.

Kindly I disagree, since if its constitutionally a fully generic check runner, that begs the question why we didn't just use one of the 100s of existing fully generic check runners - and my answer to that question has been that we (will) have font specific needs that I want to be "batteries included" with.
If we refactor out thebakery, then I wonder if there are other check runners that will be comparable, and if we do some comparisons, are there things we can improve to be superlative? :)

I looked at unit testing tools before writing the check runner, good luck finding a better fit. (Doesn't mean there isn't room for improvements.) I'd like to see the comparison.
Batteries included and separation of concerns should not be contradictory. I think of the "generic" property of the check runner as best practices approach to not mess things up and thereby go to spaghetti code hell — again.

If we refactor out thebakery, then I wonder if there are other check runners that will be comparable, and if we do some comparisons, are there things we can improve to be superlative? :)

It wouldn't be much of a refactor, so far concerns are separated (99%). I wonder, if we make thebakery: Are there other domains
where it would be useful, finding a bigger community for it?

I’m writing something now (a language/shaping tester) that would benefit from a standalone check runner...

I've just solved a significant part of this puzzle!
Now all glyphset info is separated in a new reusable python module called glyphsets available at:

https://github.com/googlefonts/glyphsets
https://pypi.org/project/glyphsets/

Please read the details at: https://github.com/googlefonts/gftools/pull/453


move to github.com/fclairamb/ftpserver package

github.com/fclairamb/ftpserver  already have all needed stuff


ci: downgrade tfcmt for test

Test #1229

Looks good.

I expect tfcmt will be updated by Renovate.


take group info from profile when parsing

Signed-off-by: Rick Marry<EMAIL_ADDRESS>:nut_and_bolt: Description
When posting a profile to be ingested, we need to take the group information from the profile itself.

I think the danger with this, like @vjeffrey mentioned in standup, is that there's a lot of squirrelly data around profiles and we've messed things up "fixing" things in the past. See chef/a2#3158 for the example of when I caused a regression by trying to fix things in here.

Fixes #209


command is generated from previously executed command

Things to think about

How to specify a rule/command without typing Python on command line

Grammar for defining where to match (script/stdout/stderr), what to match, and logical combinations
Grammar for command or just take command string?
easier to just accept Python from command line?

Automatically call 'fuck' after 'fuck this'?
Ditch the args except for rule name & open default editor with generated boilerplate rule with script/stdout/stderr included in comments at top of file?

Big +1 from me.. This is first thing our team thought about.

I would imagine we could design a simple algorithm that compares the last command with the correct command given as an argument and auto generate a regex and rule that matches.  If the new rule matches an existing rules command it can append the new rule.

Closing this one for now.


Publish modules to npm

Aurelia CLI seems to be solely reliant on npm and was hoping you could publish the latest set to npm

Sure; I'll try to get it done as soon as I'm able.

I've pushed 0.4.1 to with the correct dependency; if it doesn't work I doubt I'll have time to fix it before I go on holiday; on friday

D:\test>npm install aurelia-resize --save
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files\nodejs\node.exe" "D:\Users\anon\AppData\Roaming\npm\node_modules\npm\bin\npm-cli.js" "install" "aurelia-resize" "--save"
npm ERR! node v5.10.1
npm ERR! npm  v3.8.6
npm ERR! code E404
npm ERR! 404 Registry returned 404 for GET on https://registry.npmjs.org/aurelia-resize
npm ERR! 404
npm ERR! 404  'aurelia-resize' is not in the npm registry.
npm ERR! 404 You should bug the author to publish it (or use the name yourself!)
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.
npm ERR! Please include the following file with any support request:
npm ERR!     D:\test\npm-debug.log


Java: Prepare demo materials for new consumers of Java Semantic Kernel

Prepare new demo materials for onboarding/showing new users how to use the Java Semantic Kernel. Build on existing demo materials, and watch for end to end bugs and enhancements required.

In progress, have assets ready but needs some publishing finesse.

We have some demos ready to use, but will always be getting refined. Closing this generic work item, we'll add further tasks as specific TODOs going forward.


Feature/missing quick link settings

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


Refactor _getPageFromOffset method

In the file home_bloc.dart
Refactor the function _getPageFromOffset(int offset) by adding +1 inside the function

Hey, I would like to work on this issue.
Could you elaborate on how you need refactoring?

It is very simple.. you can check that home_bloc file and change that function.

Okay, so you want _getPageFromOffset(offset) to be equivalent to _getPageFromOffset(offset) + 1.
At first, I couldn't help but fail to notice it.. don't know why. 😆
okay, I'll do that. 👍


Please build a Release 0.10.8. from current HEAD

Hi,
I just wanted to ask you politely if it would be possible to build a new release with the current fixes (so batarang can be used with angularjs 1.6.3+) and upload it to the Chrome Web Store.
Yesterday, I built the current HEAD myself (named it 0.10.8) and verified that it is working with angular 1.6.3 and distributed the package within the company for all the other developers - who now have to install the extension from a ZIP file.....
It would be so much nicer (and more trustworthy) if everybody could download the extension from the Chrome Web Store.
Thank you very much :)

v0.10.8 is out, but it's broken. Tracking in #310.


New Resource: AutoScaling AutoScalingGroup

To ensure feature parity with AWS Config Advanced Query we need to support: AutoScaling AutoScalingGroup

Completed in release 0.16.0


ctrl+q can't still be configured to not close vscode

this report exists because the original issue was mistakenly closed
see https://github.com/Microsoft/vscode/issues/62372
due to my reply here:
https://github.com/Microsoft/vscode/issues/62372#issuecomment-435314823

/duplicate of #62372


Ionic 2 Menu doesn't always work

Ionic version:  (check one with "x")
[ ] 1.x
[x ] 2.x
I'm submitting a ...  (check one with "x")
[ x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
Having some consistency issues when clicking the "hamburger menu."  As you can see here

So what happens is you will click the menu and sometimes it will work sometimes it doesn't.  In my app first click always gets swallowed.  After that it looks like it randomly registers the click to open the menu.
Expected behavior:
Every time I click the "hamburger" the menu opens.
Steps to reproduce:

gulp e2e.serve
navigate to http://localhost:8000/dist/e2e/menu/basic/
click the hamburger
click navbar 2 times
click hamburger

It's not 100% consistent but you will be able to replicate it.
Other information:
Console from when you click and it doesn't open:
menu gesture, onSlideBeforeStart left
ionic.system.js:12690 keyboard close()
ionic.system.js:23510 menu gesture, onSlideEnd left distance 0 delta 0 velocity 0 min 0 max 304 shouldCompleteLeft false shouldCompleteRight false currentStepValue 0
ionic.system.js:2045 Animation, progressEnd, shouldComplete false currentStepValue 0
ionic.system.js:24166 menu, swipeEnd left
app-module.js:97 Menu is closed true

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Cordova CLI: 6.3.1
Gulp version:  CLI version 1.2.2
Gulp local:
Ionic Framework Version: 2.0.0-rc.2
Ionic CLI Version: 2.1.4
Ionic App Lib Version: 2.1.2
Ionic App Scripts Version: 0.0.45
ios-deploy version: 1.8.6
ios-sim version: 5.0.8
OS: Mac OS X Sierra
Node Version: v6.2.1
Xcode version: Xcode 8.1 Build version 8B62

My guess is somehow gestures are stealing the click.  I'm looking into it now.

Hello, thanks for using Ionic! Would you mind trying with the latest nightly? I am not able to repro this issue on that nightly. To install the Ionic 2 nightly you can run npm install ionic-angular@nightly --save.

Hello, I have the same problem only by iOS version. (and I use swipeBackEnabled="false")

@jgw96  Just tested and I'm having the same issue.  I'm assuming "nightly" is the same as pulling down the latest master branch.

@ncapito I'm not abel to reproduce this using latest. Are you still seeing this issue?

Hello all! As it seems it has been a while since there was any activity on this issue and we are not able to reproduce this issue i will be closing it for now. Feel free to comment if you are still running into this issue. Thanks for using Ionic!


Add teacher brief for The Hack Series

Waiting on site link.

@paked Looks like https://series.hackclub.io is up

Looks like a great first draft!
I revised it a bit. What do you think of the following?

Hey!
Thanks so much for taking the time to speak with me on the phone earlier today
-- I'm from the nonprofit that called. Below are the full details for the
workshops we're running this weekend. I'd really appreciate it if you could
forward it to the teachers and ask them to announce it during class (it'd be
best if they put the website on projectors, if they have them) :-).
The Hack Series, hosted by Hack Club
Never written code before? Build your first website this weekend at The Hack
Series, a free workshop hosted in downtown San Francisco by Hack Club for
absolute beginners.
Website: https://series.hackclub.io
Location: 1547 Mission St, San Francisco, CA 94103
We'll be running ten 10-15 person sessions this Friday, Saturday, and Sunday.
Each session lasts 2.5 hours with a 30 minute break. Spots are limited and
filling up, so if you're interested go to our website and sign up now! It's
free.
What will you learn?
You'll leave the workshop with two things built:

A personal website
A website that calls and texts your phone

We'll also be providing you with a series of workshops to do in your free time
that'll teach you the basics of game development.
Who are we?
Hack Club is a San Francisco-based nonprofit that brings coding clubs to high
school around the globe. Hack Clubs are in 52 schools across 12 states and 5
countries. If you're interested in starting a club, apply at
https://hackclub.io/apply.

bit

;). I like your version better.

Here's a PDF to attach to the email
hack_club_hack_series.pdf

@zachlatta Not sure if we should be asking if we should be telling the teacher to sign up for a Hack Club?

Also I'd go with

Hack Club is a San Francisco-based nonprofit that brings coding clubs to high school around the globe. We are in 52 schools across 12 states and 5 countries. If you're interested in starting a club, apply at https://hackclub.io/apply.

Ah, let's cut that from the teacher email (but include it in the PDF).

@zachlatta yeah sure

Okay, let me comment a revised version (and revised PDF). One sec.

Hey!
Thanks so much for taking the time to speak with me on the phone earlier today -- I'm from the nonprofit that called. Below are the full details for the workshops we're running this weekend. I'd really appreciate it if you could forward it to the teachers and ask them to announce it during class (we've had the most success when they put the website on a projector, if they have one, and print out the below details to pass around to students (I've attached a PDF)) :-).
The Hack Series, hosted by Hack Club
Never written code before? Build your first website this weekend at The Hack Series, a free workshop hosted in downtown San Francisco by Hack Club for absolute beginners.
Website: https://series.hackclub.io
Contact email<EMAIL_ADDRESS>Location: 1547 Mission St, San Francisco, CA 94103
We'll be running ten 10-15 person sessions this Friday, Saturday, and Sunday. Each session lasts 2.5 hours with a 30 minute break. Spots are limited and filling up, so if you're interested go to our website and sign up now! It's free.
What will you learn?
You'll leave the workshop with two things built:

A personal website
A website that calls and texts your phone

We'll also be providing you with a series of workshops to do in your free time that'll teach you the basics of game development.
Who are we?
Hack Club is a San Francisco-based nonprofit that brings coding clubs to high school around the globe. We are in 52 schools across 12 states and 5 countries. If you're interested in starting a club, apply at https://hackclub.io/apply.
hack_club_hack_series.pdf

@zachlatta I'd also highlight that it is free
Do you want to create a PR for this? Or shall I just copy it into mine

I just made a few changes—namely adding a "How much do I need to know?" section and highlighting that it's free near the email. What are your thoughts?

Also, it'd be awesome if you could edit your PR to include this.

@zachlatta lgtm

Awesome, here's an update PDF as well (it'd be great if you could include this in the PR).
hack_club_hack_series.pdf

I made a final change to the last comment and below is the final version of the PDF.
hack_club_hack_series.pdf

@zachlatta looks ready to merge


new download location of azdata.msi

This PR fixes #12405

Pull Request Test Coverage Report for Build 261675034

0 of 6 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.004%) to 38.221%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/azdata/src/azdata.ts
0
6
0.0%

Totals

Change from base Build 261566852:
-0.004%

Covered Lines:
19128

Relevant Lines:
45748

💛  - Coveralls


Accept Socks4/Socks5 proxies aswell as HTTP

Make bots have the ability to use socks4/socks5 proxies aswell as http proxies. Perhaps less socks4/socks5 proxies are blocked from agar, than http proxies.

Yeah this would be really good

done


internal/goglfw: use platform agnostic functions for context_windows.go

What issue is this addressing?
Updates #2546
What type of issue is this addressing?
refactor
What this PR does | solves
context_windows.go is used for all the platforms. However, it used syscall.Syscall which behaves differently on macOS and Windows as well as windows.BytePtrToString which isn't available on Linux or macOS.
This commit replaces syscall.Syscall with purego.SyscallN and windows.BytePtrToString with a copied version called bytePtrToString.

Do we have a plan to implement pruego.SyscallN for Linux by the way?

purego.SyscallN is implemented for Linux here

Oh I forgot that. Thanks!


CMAKE_CXX_FLAGS handling fails when variable is empty

Fixes #3227

Hi @zaddach , thanks for your fix!
Could you please take a look at our CLA and add “I have read and agree to the terms under CLA.md” to your PR description if you agree with the CLA?

I have read and agree to the terms under CLA.md


tracker very slow

Hi
I would like to have the option to change the engine to fzf that will speed my search
because I have many files it not usable today because it very slow
and another thing I was wondering how to set the dirs that I want it to search on
because of today I don't which dirs it search, but I know that it takes forever
BTW I'm using Regolith 3, with ubuntu 22.04 on a Dell gaming laptop (16 cores, 32 GB RAM and very large disk)

Hi @engdorm , sure.  You can override the launcher program by specifying what you'd like to execute in the Xresources key wm.program.launcher.app ([further details]( You can see all of the keys you can override here: )).
Learn more about overriding Xresources here: https://regolith-desktop.com/docs/howtos/override-xres/


Remove pip_base

I managed to detect if it is a pipe without using the tag pipe_base.
To do so, it detects if the class has a method named onReceive which can have 2 arguments or more.
But there are two drawbacks:

CompositePipe has now a useless onReceive method to meet the IsAPipe requirements
Because generic_pipeline has a method onReceive with variadic arguments it also meets the IsAPipe requirements so in IsAPipe it also check that the pipe does not inherit from pipeline_base.

The first drawback does not seems to be a real probrem. The second bothers me, I tried to check on has_onReceive_method if the last argument must meet IsAPipeline requirements but for now I failed.
I will be happy to have a review on it even if you don't want to accept the pull request.

Thanks for your PR, and for its very clear explanation.
I understand the issues you're mentioning, and they may reveal that the distinction between a pipe and a pipeline is not clear enough in the library. Should they both take a method called onReceive?
Or perhaps the SFINAE expression should detect that the last argument is a pipeline?
That would have the advantage of removing the base class, but that would lock the form of onReceive.
Indeed, I think we should further clarify the design of the library before moving on. But your PR helps bring forward those design questions. Thanks for that.

You're welcome.
I think the name onReceived is good for both pipe and pipeline because it is a really explicit name.
The differences I see between a pipeline is also an output iterator. A pipe can send data with operator >>= and we can create a CompositePipe from it.
Detect a pipeline using its capabilities as an output iterator (onReceive, ++ and *) instead of its base type (pipeline_base) would be a nice idea, it would still be possible to use pipeline_base as a helper to implement a pipeline but something else could fit.
Maybe CompositePipe should not be a pipe but something else. Define it could help to clarify the distinction between a pipe and a pipeline.

If that's ok with you I'll close this thread for the moment, as I think we need more hindsight on the design of the library to move forward on this topic.


add college

.

@thelunartear Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


update to 0.15.0

Changes:

update to 0.15.0
update xsimd https://github.com/AnacondaRecipes/xsimd-feedstock/pull/9 to 12.1.1 with patch to match commit pinned in pythran.
run tests against openblas on linux-64 - due to an issue locating mkl-sdl

https://github.com/serge-sans-paille/pythran/tree/0.15.0

I see that they now use xsimd 12.1.1, see https://github.com/serge-sans-paille/pythran/blob/0.15.0/pythran/xsimd/config/xsimd_config.hpp#L15.

It looks like we should: they updated from 11.0.0 to 12.1.1 to fix a scipy build error

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_9d30lipwoa/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:53: python_build_tool_in_run: The python build tool setuptools is in run depends
===== Final Report: =====
0 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_2b3cyyc3w4/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:53: python_build_tool_in_run: The python build tool setuptools is in run depends
===== Final Report: =====
0 Errors and 1 Warning were found


⚠ Bofrost.es has degraded performance

In 69380e1, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1625 ms

Resolved: Bofrost.es performance has improved in 260e17b after 7 minutes.


chore(ci): run snapshot deployment on nexus in a dedicated job to avo…

…id blocking nightly deployment
fixes AM-1020

Skip Integration tests as the PR is related to CI settings after a merge action on a PR


Backbone producing nan

During training on a custom dataset, I noticed that loss_reg was nan. After looking into it further, I realized that when calling the backbone to extract features was resulting in tensors filled with nan for the last 3 layers. This is occurring across all available backbones and detectors.
The problem seems to be here (taken from mmdet.models.backbones.ResNet):
    def forward(self, x):
        x = self.conv1(x)
        x = self.norm1(x)
        x = self.relu(x)
        x = self.maxpool(x)
        outs = []
        for i, layer_name in enumerate(self.res_layers):
            res_layer = getattr(self, layer_name)
            x = res_layer(x)
            if i in self.out_indices:
                outs.append(x)
        return tuple(outs)

Anyone encounter a similar problem? It's interesting because the first res_layer produces a output but subsequent layers produce all nans.

NaN is usually caused by the improper gradient, which will harm the network weights.

NaN is usually caused by the improper gradient, which will harm the network weights.

I'm seeing similar problem. The FasterRCNN-ResnetX was training alright in the first few hundreds steps, then once NaN starts to appear, it never recover. Any tips on things to try to get rid of NaN?
2019-06-19 08:17:31,414 - INFO - Epoch [1][75000/217878]        lr: 0.00300, eta: 29 days, 9:59:30, time: 0.589, data_time: 0.005, memory: 8595, loss_rpn_cls: 0.1343, loss_rpn_bbox: 0.0180, loss_cls : nan, acc: 27.5210, loss_bbox: nan, loss: nan Warning: NaN or Inf found in input tensor. Warning: NaN or Inf found in input tensor. Warning: NaN or Inf found in input tensor.

I am seeing Nans with retinanext as well

I did some more debugging by adding autograd.detect_anomaly() in mmdet/core/utils/dist_utils.py
``
from torch import autograd
...
def after_train_iter(self, runner):
with autograd.detect_anomaly():
runner.optimizer.zero_grad()
runner.outputs['loss'].backward()
allreduce_grads(runner.model, self.coalesce, self.bucket_size_mb)
if self.grad_clip is not None:
self.clip_grads(runner.model.parameters())
runner.optimizer.step()
``
Finally, I traced my problem to be that the bbox 'target' in smooth_l1_loss() contains NaN. As the bboxes annotation look alright on my data, so I'm not sure how the NaN got there after all the transforms. I worked around the problem by checking for NaN and setting them to 0.

@soon-yau Hi, I meet the similar problem. could you please tell me how to check the grad and how to set them to 0?

Hi, I am also having this problem. @analyticsbot how much higher did you set the grad_norm to be? I am doing 2 GPUs with batch size 20, learning rate 8e-5, 2000 warmup iters, and grad_norm 35 (type 2) and I run into NaNs after about 50 epochs. After the NaNs start the training does not recover. Worth noting my dataset is quite large (50k images) and I checked my annotations, they are correct.


Bluetooth: L2CAP: remove metadata requirement

This removes the dependency between CONFIG_BT_ATT_TX_COUNT and
CONFIG_BT_L2CAP_TX_BUF_COUNT since previously there was still a need
for an L2CAP context for every TX'd buffer.
~Stacked on https://github.com/zephyrproject-rtos/zephyr/pull/67532~

CI failure should be fixed by: https://github.com/zephyrproject-rtos/zephyr/pull/68104


Remove Meshery's Performance Gql Resolvers

Current Behavior

The Performance profile and result subscriptions were removed with this commit:9154b353bb081b2927c99c366e12de90c4ebec7c
So now the go-resolvers and grapqhQL's schema has to be removed as well.
Why removed:
The reason for removal is because the table data was fetched as the graphQL query and the same query was subscribed to changes, this suffices our use case.
Also, the two gql resolvers are not pumping any performance related data. The relay's browser environment doesn't receive any data as well.
Expected Behavior

Screenshots/Logs

Environment

Host OS: Mac Linux Windows
Platform: Docker or Kubernetes
Meshery Server Version: stable-v
Meshery Client Version: stable-v

Contributor Guides and Handbook

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

Is this whole file needed to be removed ?
https://github.com/meshery/meshery/blob/master/server/internal/graphql/resolver/performance.go

@suhail34 Actually that was a mistake removing the grapqhQL subscription from frontend, infact it was helping in fetching the latest release on time. So this issue is no longer relevant and thus closing it.


refactor optimize the array_distinct and array_sort fun…

Proposed changes

this pr is used to optimize the array_distinct and array_sort function.

Issue Number: close #10052
Problem Summary:
Checklist(Required)

Type of your changes:

[ ] Improvement
[ ] Fix
[ ] Feature-WIP
[ ] Feature
[ ] Doc
[x] Refator
[ ] Others:

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[x] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[x] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes
[x] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

There are still much duplicate code between function_array_sort.cpp and function_array_distinct.cpp. Is there any good idea to extract these code out?

ok，I will re-check again.

reopen this pr.


update depreciated methods

Notes: https://github.com/StronkMan/react-native-braintree-no-ui/pull/22


Fix: replace exp.StructKwarg with exp.ColumnDef in schema diff

This PR resolves the regression that will be caused due to StructKwarg being removed in https://github.com/tobymao/sqlglot/commit/7b09bffd7820748df5e9f1ce21f74883a7041ca8.
This depends on releasing a new SQLGlot version. Once that's done I'll bump its version here too.

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


repository: BTFHUB should contain BTF files for all kernel modules

Currently BTFHUB supports BTF files only for the main kernel image (vmlinux) but not for the included modules. This means that, if an eBPF program needs to kprobe a module function, for example, in a kernel without internal BTF information it would not be possible (as the program would have to load an external BTF file describing that module and those BTF files don't exist).
Things to observe:

We will need to extract BTF information from vmlinux and all the module objects and have a SINGLE BTF file containing all the BTF information. That will allow us to use "btfgen" (bpftool gen min_core_btf feature) to generate a minimum BTF file for 1 or multiple objects (even when the objects use type information from the kernel modules).

@brycekahle Please make sure to constantly rebase (if you are tackling this offline). If you can't work on this, let me know, then we can try to either split work or I can try to address it myself.
Cheers!

I'm going to start working on this next week. First, I need to establish the proper generation procedure. I think pahole can handle multiple files, but I need to verify.

In talking with cilium/ebpf maintainers, it seems that split kernel module BTF files are the preferred approach. This mirrors what a kernel does that natively exposes BTF.

For those following along, working through this on our fork: https://github.com/DataDog/btfhub/issues/7


View raster and mesh files from the Danesfield workflow

We need a way to associate Image, Mesh3D, and Tiles3D objects generated from a Dataset's output files with the Dataset itself.
@AlmightyYakob do you think this would be appropriate to upstream to OASIS? Then, we wouldn't need a separate model and could instead just put these ManyToManyFields directly on the Dataset model. I'm not certain that this would be in scope for OASIS though.

Just so I'm clear, the main reason for needing this is so that given a dataset, the ingested files can easily be accessed, not just the checksum files?

@AlmightyYakob do you think this would be appropriate to upstream to OASIS? Then, we wouldn't need a separate model and could instead just put these ManyToManyFields directly on the Dataset model. I'm not certain that this would be in scope for OASIS though.

I'd like to see how it shakes out here first, and then evaluate. I'm not against it necessarily, after all OASIS is using the RGD ecosystem, but I think using it here first is a better idea than jumping to upstreaming it.

Just so I'm clear, the main reason for needing this is so that given a dataset, the ingested files can easily be accessed, not just the checksum files?

Right. Now that I think about it, there's also an alternative to this that doesn't require a new model.
def get_ingested_file(file: ChecksumFile):
	try:
		return Image.objects.get(file=file)
	except DoesNotExist:
		try:
			return Mesh3D.objects.get(file=file)
		except DoesNotExist:
			try:
				return Tiles3D.objects.get(file=file)
			except DoesNotExist:
				return None

This approach has the advantage of not requiring any database changes. WDYT @AlmightyYakob ? I think this might be a better approach, the downside is we'd be making 2 and 1 extra queries for Mesh3D and Tiles3D, respectively, but since that's a constant number of queries I don't think it's a big deal.

@AlmightyYakob do you think this would be appropriate to upstream to OASIS? Then, we wouldn't need a separate model and could instead just put these ManyToManyFields directly on the Dataset model. I'm not certain that this would be in scope for OASIS though.

I'd like to see how it shakes out here first, and then evaluate. I'm not against it necessarily, after all OASIS is using the RGD ecosystem, but I think using it here first is a better idea than jumping to upstreaming it.

:+1:

Right. Now that I think about it, there's also an alternative to this that doesn't require a new model.
def get_ingested_file(file: ChecksumFile):
	try:
		return Image.objects.get(file=file)
	except DoesNotExist:
		try:
			return Mesh3D.objects.get(file=file)
		except DoesNotExist:
			try:
				return Tiles3D.objects.get(file=file)
			except DoesNotExist:
				return None

This approach has the advantage of not requiring any database changes. WDYT @AlmightyYakob ? I think this might be a better approach, the downside is we'd be making 2 and 1 extra queries for Mesh3D and Tiles3D, respectively, but since that's a constant number of queries I don't think it's a big deal.

I like this approach, We'd be making a max of 3 queries, and a min of one, which isn't too bad. I'd propose the following code suggestion though
def get_ingested_file(file: ChecksumFile):
    for model in [Image, Mesh3D, Tiles3D]:
        try:
            return model.objects.get(file=file)
        except DoesNotExist:
            pass

    return None

This would be hacky, but we also could make use of the ChecksumFile.description field, to add a tag that indicates which model it uses. That might introduce other issues though, so for now I think this is fine.

Right. Now that I think about it, there's also an alternative to this that doesn't require a new model.
def get_ingested_file(file: ChecksumFile):
	try:
		return Image.objects.get(file=file)
	except DoesNotExist:
		try:
			return Mesh3D.objects.get(file=file)
		except DoesNotExist:
			try:
				return Tiles3D.objects.get(file=file)
			except DoesNotExist:
				return None

This approach has the advantage of not requiring any database changes. WDYT @AlmightyYakob ? I think this might be a better approach, the downside is we'd be making 2 and 1 extra queries for Mesh3D and Tiles3D, respectively, but since that's a constant number of queries I don't think it's a big deal.

I like this approach, We'd be making a max of 3 queries, and a min of one, which isn't too bad. I'd propose the following code suggestion though
def get_ingested_file(file: ChecksumFile):
    for model in [Image, Mesh3D, Tiles3D]:
        try:
            return model.objects.get(file=file)
        except DoesNotExist:
            pass

    return None

This would be hacky, but we also could make use of the ChecksumFile.description field, to add a tag that indicates which model it uses. That might introduce other issues though, so for now I think this is fine.

Sounds good, I'll change this PR to do it this way instead

Changes I've made:

Add an additional endpoint that, given the name of a file in a Dataset, redirects the user to the RGD server-rendered page for the ingested version of that file (i.e. Rasters for 2D images and Mesh3D for 3D meshes, 3D tiles are not implemented yet).
Add logic to the frontend to use this endpoint to render the RGD page as an iframe

Adding an additional endpoint that redirects to another page on the server seems a little hacky, but I couldn't think of a cleaner way to do it; I'm open to suggestions if you can think of a better way.

Adding an additional endpoint that redirects to another page on the server seems a little hacky, but I couldn't think of a cleaner way to do it; I'm open to suggestions if you can think of a better way.

I don't think it's hacky, as that endpoint takes something known to us as a single type (a file in a dataset), and returns various things based on inspecting that file (which we have no real idea about from the client). Since we're dealing with files that map to various models, I think it's fine.

@mvandenburgh I guess we can just merge this PR now and create an issue for adding in 3d tiles support?

@mvandenburgh I guess we can just merge this PR now and create an issue for adding in 3d tiles support?

Sounds good to me.


Graphical glitch: might not be GP - untrusted player makes a broken door looks like half a door [<IP_ADDRESS>2]

Hello, Thank you for the great work.
I noticed today that when an untrusted player breaks a door by hitting the lower half of the door, to them it looks like half a door.
It still acts as a door, so it's fine, but just thought I'd mention it.
Let me know if there is anything else I can provide for help. Sorry if this is useless.
Peace.
Java version: 1.8.0_221-b11
minecraft server.1.12.2
forge 1.12.2 <IP_ADDRESS>38 universal
spongeforge 1.12.2 2838 7.1.7 RC 3900
GriefPrevention <IP_ADDRESS>2

This is a client-side visual glitch, leaving the area (far enough that chunks are unloaded) or logging out and back in fixes it.
Not a bug


Failed to send message: bot returned an error

We have developed a bot and deployed to azure and was working fine. Suddenly today we are facing issue while posting a message to the bot. Failed to send message: bot returned an error
Can anyone please advise?

Can you give more details on your type of deployment: Is this an Azure Bot Service, or an Azure App Service?
Do you have any specific exception details in your logs?

@maheshbabuvemulapati I'm going to close this issue for now. Please feel free to open a new issue with specific error messages you are seeing.


[Bug] DataFrame object has no attribute shape when running ExperimentAxis Query

Description:
When trying to fetch obsm_layers using ExperimentAxisQuery syntax, I get the error below:
  File ".../test_tiledb_pipeline_dummy copy.py", line 135, in <module>
    anndata = query.to_anndata(X_name="X", obsm_layers=("umap",))
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/somacore/query/query.py", line 295, in to_anndata
    return self._read(
           ^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/somacore/query/query.py", line 424, in _read
    obsm=obsm_ft.result(),
         ^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/concurrent/futures/_base.py", line 449, in result
    return self.__get_result()
           ^^^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/concurrent/futures/_base.py", line 401, in __get_result
    raise self._exception
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/somacore/query/query.py", line 385, in _read_axis_mappings
    return {key: fn(axis, key) for key in keys}
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/somacore/query/query.py", line 385, in <dictcomp>
    return {key: fn(axis, key) for key in keys}
                 ^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/somacore/query/query.py", line 571, in _axism_inner_ndarray
    _, n_col = axism[layer].shape
               ^^^^^^^^^^^^^^^^^^
AttributeError: 'DataFrame' object has no attribute 'shape'

To Reproduce
import shutil
import pandas as pd
import pyarrow as pa
import scipy.sparse as sp

soma_path = "./corpus"
sample_1 = {
    "barcode": [
        "A",
        "B",
        "C",
    ],
    "features": [
        "CD8",
        "CD4",
    ],
    "matrix": [
        [1, 0],
        [0, 1],
        [1, 1],
    ],
    "obsm": [
        [0.1, 0.2],
        [0.3, 0.1],
        [0.0, 0.8]
    ]
}

# SCHEMAS
obs_schema = [
    ("soma_joinid", pa.int64()),
    ("barcode", pa.large_string())
]
var_schema = [
    ("soma_joinid", pa.int64()),
    ("gene_name", pa.large_string())
]
obsm_schema = [
    ("soma_joinid", pa.int64()),
    ("umap_0", pa.float32()),
    ("umap_1", pa.float32()),
]

# Create experiment
exp = soma.Experiment.create(soma_path)
exp.close()

# Open experiment for reading
with soma.Experiment.open(soma_path, mode="w") as exp:
    ms = exp.add_new_collection("ms")
    rna = ms.add_new_collection("RNA", soma.Measurement)
    X_collection = rna.add_new_collection("X")
    X_collection.add_new_sparse_ndarray(
        "X",
        type=pa.uint32(),
        shape=(3, 2),
    )
    rna.add_new_dataframe("var", schema=pa.schema(var_schema))
    obsm_coll = rna.add_new_collection("obsm")
    obsm_coll.add_new_dataframe("umap", schema=pa.schema(obsm_schema))
    exp.add_new_dataframe("obs", schema=pa.schema(obs_schema))

    ms.close()
    rna.close()
    obsm_coll.close()
    X_collection.close()

current_obs_id = 0
current_var_id = 0
for i, dict_ in enumerate([sample_1,]):
    # Obs ingest
    sub_obs = pd.DataFrame(dict_["barcode"], columns=["barcode"])
    sub_obs["soma_joinid"] = range(current_obs_id, current_obs_id + len(sub_obs))
    pa_table = pa.Table.from_pandas(
        sub_obs,
        schema=pa.schema(obs_schema),
    )
    with soma.Experiment.open(soma_path, mode="w") as exp:
        exp.obs.write(
            pa_table
        )

    # Var ingest
    sub_var = pd.DataFrame(dict_["features"], columns=["gene_name"])
    sub_var["soma_joinid"] = range(current_var_id, current_var_id + len(sub_var))
    pa_table = pa.Table.from_pandas(
        sub_var,
        schema=pa.schema(var_schema),
    )
    with soma.Experiment.open(soma_path, mode="w") as exp:
        exp.ms["RNA"].var.write(
            pa_table
        )

    # X ingest
    matrix = sp.coo_matrix(dict_["matrix"], shape=(3, 2))
    with soma.Experiment.open(soma_path, mode="w") as exp:
        exp.ms["RNA"]["X"]["X"].write(
            pa.Table.from_pydict(
                {
                    "soma_dim_0": matrix.row + current_obs_id,
                    "soma_dim_1": matrix.col + current_var_id,
                    "soma_data": matrix.data

                }
            )
        )

    # Obsm ingest
    sub_obsm = pd.DataFrame(dict_["obsm"], columns=["umap_0", "umap_1"])
    sub_obsm["soma_joinid"] = range(current_obs_id, current_obs_id + len(sub_obsm))
    pa_table = pa.Table.from_pandas(
        sub_obsm,
        schema=pa.schema(obsm_schema),
    )
    with soma.Experiment.open(soma_path, mode="w") as exp:
        exp.ms["RNA"]["obsm"]["umap"].write(
            pa_table
        )

with soma.Experiment.open(soma_path) as exp:
    with exp.axis_query(measurement_name="RNA") as query:
        anndata = query.to_anndata(X_name="X", obsm_layers=("umap",))
        print(anndata)

Versions:

TileDB-SOMA version: 1.10.0
Language and language version (e.g. Python 3.8, R 4.2.2): Python 3.11.6
OS (e.g. MacOS, Ubuntu Linux): MacOS M1

[sc-46789]

@mdylan2 obsm entries should be SparseNDArray (or DenseNDArray) -- having DataFrame here should not be supported.
That said, we should have errored out at a much higher level, with a more intuitive error message. I'll fix this.

I think we can switch over to Sparse/Dense ND Array. There's no particular need for the DataFrame.
The only issue was dealing with empty rows but I think Dense/SparseNDArrays have row/column indices so we should be fine

The only issue was dealing with empty rows but I think Dense/SparseNDArrays have row/column indices so we should be fine

@mdylan2 indeed -- obsm array rows are keyed by the soma_joinid of obs, always

I think we can switch over to Sparse/Dense ND Array. There's no particular need for the DataFrame.
The only issue was dealing with empty rows but I think Dense/SparseNDArrays have row/column indices so we should be fine

Awesome, thanks @mdylan2 !


RTL support

Is it possible to avoid mirroring the component when app is RLT? (when app is RTL + button is at left side that is meaningless)

Hi @AlirezaAkbarix What does RTL or RLT stand for? Please share simple-stepper usage.

LTR: Left To Right (en, fr, du, ... languages)
RTL: Right To Left (he, ar, fa, ... languages) that renders view reverse.


URI segments passed in as method parameters skips segments with 0 value

Describe the bug
URI segments passed in as method parameters doesn't seem to work as expected if the segment contains 0 (integer zero).
CodeIgniter 4 version
4.0.0-beta3
Affected module(s)
Controllers & Routing
Expected behavior, and steps to reproduce if appropriate
If the URI segment contains zero as it's value, the segment is skipped when passed in as the method parameter.
class TestController extends Controller {
    public function segmentTest($param1 = 'x', $param2 = 'y') {
        var_dump($param1);
        var_dump($param2);
    }
}

If the URL called is something like
test/segmentTest/0/abc, (notice the third segment is zero, so $param1 should be set as 0)
but the result was:
string(3) "abc" string(1) "y"
whereas I was expecting something like:
string(1) "0" string(1) "abc"
Is this intentional? I couldn't find anything in the documentation. There are multiple situations where passing in 0 as the url param is valid.
Also please note that if no default values are given for the parameters, an error is encountered:
ArgumentCountError
Too few arguments to function App\Controllers\Test::segmentTest(), 1 passed in

Context

OS: Linux
PHP version - 7.2.17

Each section of the path between the slashes is a single segment. The URI class provides a simple way to determine what the values of the segments are. The segments start at 1 being the furthest left of the path.

Sorry if my question was not clear. I understand how segments work. My issue is that if the value of a segment is 0, then that value is not passed to the controller as a method parameter and the next segment if any is passed instead. The example I've given in the bug report illustrates the problem. I've edited the issue to try and make it a bit more clear...

So i've tracked down why this happens and the following line is where the 0 valued segments are filtered out:
https://github.com/codeigniter4/CodeIgniter4/blob/230c2abb65fa9c936391df3eb912ef00e4852602/system/Router/Router.php#L571
This line will filter out any values that evaluate to FALSE. So 0s will get dropped. I think that having 0s in the URL are valid and should not be filtered out.