maximum likelihood as an objective function

In the documentation, the objective functions below are given. I wonder if Keras has maximum likelihood as an objective function. If not what is the easiest way to implement our own objective function?
mean_squared_error, mean_absolute_error, mean_absolute_percentage_error, mean_squared_logarithmic_error, squared_hinge, hinge, binary_crossentropy, categorical_crossentropy, sparse_categorical_crossentropy, kullback_leibler_divergence, poisson, cosine_proximity
Thanks.

Many objective functions built into keras are obtained by maximum likelihood. e.g. the mean squared error loss function is the maximum likelihood under the assumption that the data is normally distributed about its mean with variance the same everywhere. Maximum likelihood estimation yields a different objective function depending on how you assume (i) the mean is related to the feature $x$ and (ii) how the data is distributed about the mean.
That is, there is no such thing as a general "maximum likelihood objective function".
Implement your own loss as e.g.
def custom_loss(y_true, y_pred):
     return tf.square(y_true - y_pred)

and pass while compiling model.compile(loss=custom_loss, ...)

I disagree with @CorySimon; indeed, is evident that you can implement a maximum likelihood objective function for any important distribution (e.g. gaussian distribution).
Notice that the MSE is not the likelihood function of a gaussian distribution: it is the maximum likelihood distribution of a fixed variance, diagonal gaussian.
In many cases, you want to learn a distribution (e.g. gaussian mlp,  variational autoencoders) and you would like to have an objective function that has a single target (your data) and multiple outputs, e.g. in the case of a gaussian MLP mean and variance.
I don't think this is straightforward in keras, right now, if i'm not wrong. If I'm, please suggest a way to implement this in a clean way.
As far as I know, keras needs that targets and model outputs have the same dimensionality.
The only way I managed to solve this issue is either select the mean as output of the network and pass as class member the tensor representing the standard deviation, or to add a sampling layer as output, and pass both mean and standard deviation as class members.
Needless to say that on tensorflow it is easy to do so...

https://www.quora.com/What-are-the-differences-between-maximum-likelihood-and-cross-entropy-as-a-loss-function
"for a probability model to be trained on i.i.d training examples, maximize likelihood → minimize negative log likelihood → minimize cross entropy."

Is it even possible in principle with the Keras API? Since all you can give the objective function is y_pred and y_true while for MLE you need to give extra parameters. For example, Normal residual MLE requires y_pred y_true sigma for your current estimate of the standard deviation.
I really want to be able to make my own custom loss functions with distributions. Just imagine the possibilities!!!

@arose13 yes it is totally possible, as @liangbright has pointed out. Actually there's already package to do that called edward. Check out examples here: http://edwardlib.org/tutorials/supervised-regression

I guess you can do it like below
from keras.layers import Input, Dense, Layer
from keras.models import Model
import tensorflow_probability as tfp
tfd = tfp.distributions

class NegativeLogLikelihood(Layer):

    ...
    def call(self, inputs):
        y_true = inputs[0]
        y_pred = inputs[1]
        dist = tfd.Normal(loc=y_pred, scale=self.sigma)
        return -1.0 * dist.log_prob(y_true)
    ...

def identity_loss(y_true, loss):
    return loss

input_x = Input(shape=...)
input_y = Input(shape=(1,))
out = Dense(1)(net)
loss = NegativeLogLikelihood(1)([input_y, out])
model = Model(inputs=[input_x, input_y], outputs=[out, loss])
model.compile(loss=identity_loss, loss_weights=[0., 1.], ...)
model.fit([x, y.reshape((-1, 1))], [y, y]...)

Just calculate negative log likelihood in your network, pass it to loss function (does nothing)
If you don't like 2 outputs (out for predict / loss for fit), maybe 2 network (for fit/predict) with shared weight will work?


Migrate to requestAnimationFrame( ) from setInterval( )

Current Behavior
We use setInterval to drive animations in main.js
// Set up game loop
setInterval(update, 1000 / FPS)

Although this is an easy way to do it, it does not result in the cleanest animations. See this SO answer for more info.
I looked into the following resources, but I wasn't able to get a firm understanding of the best way to do it for GeoAsteroids
https://blog.webdevsimplified.com/2021-12/request-animation-frame/
https://developer.mozilla.org/en-US/docs/Web/API/window/requestAnimationFrame
https://dev.to/moyedx3/10-settimeout-setinterval-and-requestanimationframe-1io4
https://dev.to/suprabhasupi/requestanimationframe-in-javascript-n12
I tried
// Set up game loop
window.requestAnimationFrame(update)

and it results in nothing animating. I think we need to do more than a one-liner.

I figured this one out!
Implemented in this commit.
We are now keeping track of timestamps and only calling update() when 1 second / FPS time has elapsed.
Please take a look at gameLoop( ) inside events.ts for the logic.


Bug 2098508: Remove duplicate service manifest

#48 accidentally introduced a second service object, for some reason this wasn't picked up in my local testing.
Remove the additional manifest and set the correct port on the original so that the webhooks should then work

/bugzilla refresh

/approve
/override ci/prow/e2e-aws
/override ci/prow/e2e-aws-disruptive
/override ci/prow/e2e-aws-serial
/override ci/prow/e2e-aws-upgrade
This isn't actually in payload so waiting for these tests is pointless


snap() can't capture elements off initial page (Chromium has changed) [fixed]

Hi, I am looking to snap images that could be potentially off the initial screen and requires scrolling.
I am using t.hover to scroll to the image, then use t.snap to take a capture of the image. However it seems like the capture of the off screen element always turns up blank even though it has already scrolled to the element.
Refer to my sample code in Python:
t.url('https://www.google.com/search?q=test')
t.hover('(//div[@role="navigation"])[2]')
t.snap('(//div[@role="navigation"])[2]', 'test.png')

Is this a bug? Or is there a way to overcome this?

When you run the RPA process, it will open a new chrome browser, but I run your RPA process, I can't find your xpath.

So, you can try this xpath: //*[@id="tsf"]/div[1]/div[1], you will get a png like this:

Hi, I am looking to snap images that could be potentially off the initial screen and requires scrolling.
I am using t.hover to scroll to the image, then use t.snap to take a capture of the image. However it seems like the capture of the off screen element always turns up blank even though it has already scrolled to the element.
t.url('https://www.google.com/search?q=test')
t.hover('(//div[@role="navigation"])[2]')
t.snap('(//div[@role="navigation"])[2]', 'test.png')

Is this a bug? Or is there a way to overcome this?

Hello,
You are searching for the wrong xPath. The xPath is @.***="
navigation"])[2]
[image: image.png]
The snap works if the element is on the main screen, but if it needs
scrolling to reach the element, snapping does not work.
On Tue, May 30, 2023 at 10:07 AM Amanaway @.***> wrote:

When you run the RPA process, it will open a new chrome browser, but I run
your RPA process, I can't find your xpath.
[image: error]
https://user-images.githubusercontent.com/21170982/241829844-a1737bf5-927b-4b03-9cc4-142c4756a70e.png
So, you can try this xpath: @.***="tsf"]/div[1]/div[1], you will get a
png like this:
[image: test]
https://user-images.githubusercontent.com/21170982/241830155-659c9411-7a34-4ebb-9f47-8c041e7168b7.png
Hi, I am looking to snap images that could be potentially off the initial
screen and requires scrolling.
I am using t.hover to scroll to the image, then use t.snap to take a
capture of the image. However it seems like the capture of the off screen
element always turns up blank even though it has already scrolled to the
element.
t.url('https://www.google.com/search?q=test')
@.="navigation"])[2]')
@.="navigation"])[2]', 'test.png')
Is this a bug? Or is there a way to overcome this?
—
Reply to this email directly, view it on GitHub
https://github.com/tebelorg/RPA-Python/issues/479#issuecomment-1567669013,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOMVVFLKDEFXWE7JQVIJHATXIVI5RANCNFSM6AAAAAAYO2MU3Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

yes, scrolling will get this bug, I delete the reply.

Hello, You are searching for the wrong xPath. The xPath is (//div[@ROLE="navigation"])[2]  The snap works if the element is on the main screen, but if it needs scrolling to reach the element, snapping does not work.
…
On Tue, May 30, 2023 at 10:07 AM Amanaway @.> wrote: When you run the RPA process, it will open a new chrome browser, but I run your RPA process, I can't find your xpath. [image: error] https://user-images.githubusercontent.com/21170982/241829844-a1737bf5-927b-4b03-9cc4-142c4756a70e.png So, you can try this xpath: @.="tsf"]/div[1]/div[1], you will get a png like this: [image: test] https://user-images.githubusercontent.com/21170982/241830155-659c9411-7a34-4ebb-9f47-8c041e7168b7.png Hi, I am looking to snap images that could be potentially off the initial screen and requires scrolling. I am using t.hover to scroll to the image, then use t.snap to take a capture of the image. However it seems like the capture of the off screen element always turns up blank even though it has already scrolled to the element. t.url('https://www.google.com/search?q=test') @.="navigation"])[2]') @.="navigation"])[2]', 'test.png') Is this a bug? Or is there a way to overcome this? — Reply to this email directly, view it on GitHub <#479 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOMVVFLKDEFXWE7JQVIJHATXIVI5RANCNFSM6AAAAAAYO2MU3Q . You are receiving this because you authored the thread.Message ID: @.***>

@JS-code-dev @Vic-Lau I can replicate. I get the following from the logs. The log output looks correct, but somehow a blank image is returned. It's been a while since I created this function so I don't remember if it supports taking snapshot outside of the initial view. Likely this was working but along the way it might have stopped working with newer Chromium because below log output looks correct. I'll check more and update back.
[tagui] INPUT  -
[30] {"id":30,"method":"Page.captureScreenshot","params":{"format":"png","quality":80,"clip":{"x":180,"y":559,"width":652,"height":56,"scale":1},"fromSurface":true}}
[tagui] OUTPUT -
[30] {"id":30,"result":{"data":"iVBORw0KGgoAAAANSUhEUgAAAowAAAA4CAYAAAB396mcAAABL2lDQ1BTa2lhAAAokX2QPUsDQRCGH0NQonYqprC4RrBRc1FzBGzyocHCJlGIFsLlcgQxicflRHux80dYipWF/0SwsrDRWghYWPneXXEBjbPMzrPvDjM7C6kssnQOev3Ar9fKRvPo2Jh6Y0IrMtsZeIw3ZX29xLnPq//kjbNM2x04ip/ywFdzlWyL5zsxX4Xcivk25MvAC8R3IfsH9Yr4SbzSGeHWCDueH+a/ird73QsneTezbv+wodiUL1HjXKtDF5d1Gpxxii0qYVKkwIZ8l01FkwpVLEVL5wpldnRTIqe7KnntWxEXpVrhf8YtnRuwNGfmO9F6Q3icBmM/0ZZPIDsJ9x+JlvyxZ/t2JKXlKbcMwwWN8gBz7zBzLXUxqvz3rMavWQ326OOwJsrr3SaFHwYiSrMQwHDKAAABHElEQVR4nO3WMQHAIBDAwFL/nh8FZIXhTkHGrJmZDwAADv7bAQAAvM0wAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAMkwAgCQDCMAAGkDitEEbJUz5HAAAAAASUVORK5CYII="}}

@JS-code-dev @Vic-Lau I've fixed the issue. Can you try again with v1.49 to see if it works now, no need to do hover() first, it will auto-move to the web page element before taking screenshot.
There is some change in Chromium behaviour so the old snapshot code no longer works with the browser. In v1.49 I've updated logic to use absolute x,y coordinates instead of relative coordinates and it works now.

@JS-code-dev , this year I'm running a promotion if a user reports a new bug, there is 200 USD gift voucher. Could you pick a vendor you like so that I can get a gift voucher to appreciate reporting this bug and benefit other users of this package?
Thank you very much for raising this issue!!

Adding on, any online vendor will do, does not need to be in USD. Roughly 200 USD value will do.
If more it is also ok, you can cover the rest of the cost and I can cover the 200 USD. Thank you 🙏

@kensoh I have verified that snapping off-page works as intended without the need for hover! Should I go ahead and close this then? Also, that's very generous of you for the 200 USD! I will pm you regarding the details! Thank you :)

It's ok can leave issue open for a while so users with the same problem will know to upgrade their rpa package version.
You can email me at<EMAIL_ADDRESS>thank you for raising this issue! So I know Chromium has changed and broke the snap() function.

@JS-code-dev just give me a shout here after you've made your purchase and email me at<EMAIL_ADDRESS>(By the way do you have TransferWise / Wise? Paypal has additional 15% cost when I recently transfer to a China user for raising a good feature suggestion)

@kensoh I am situated in Singapore and have dropped you an email with a suggestion. We can discuss over there. Thanks again.

Oh great @JS-code-dev! I'm from Singapore and I can transfer you the equivalent value of 200 USD in SGD.

Oh! Cool! I have emailed you details for the transfer. Let me know if that works out. @kensoh

@kensoh snap() is good and it doesn't need hover().

(sent gift card payment to @JS-code-dev 🙌)


Composite unique index also marks a column unique=true

Example DDL:
CREATE TABLE "participations"(
    "user_id" BIGINT NOT NULL,
    "project_id" BIGINT NOT NULL,
    "team_id" BIGINT NOT NULL,
);
ALTER TABLE
    "participations" ADD CONSTRAINT "participations_team_id_user_id_unique" UNIQUE("team_id", "user_id");

Output:

from simple_ddl_parser import DDLParser
import json

parse_results = DDLParser("""CREATE TABLE "participations"(
    "user_id" BIGINT NOT NULL,
    "project_id" BIGINT NOT NULL,
    "team_id" BIGINT NOT NULL,
);
ALTER TABLE
    "participations" ADD CONSTRAINT "participations_team_id_user_id_unique" UNIQUE("team_id", "user_id");""").run()

[
    {
        "table_name": "\"participations\"",
        "schema": null,
        "primary_key": [],
        "columns": [
            {
                "name": "\"user_id\"",
                "type": "BIGINT",
                "size": null,
                "references": null,
                "unique": true,                 ------- This should probably be false
                "nullable": false,
                "default": null,
                "check": null
            },
            {
                "name": "\"project_id\"",
                "type": "BIGINT",
                "size": null,
                "references": null,
                "unique": false,
                "nullable": false,
                "default": null,
                "check": null
            },
            {
                "name": "\"team_id\"",
                "type": "BIGINT",
                "size": null,
                "references": null,
                "unique": true,            ------- This should probably be false
                "nullable": false,
                "default": null,
                "check": null
            }
        ],
        "alter": {
            "uniques": [
                {
                    "constraint_name": "\"participations_team_id_user_id_unique\"",
                    "columns": [
                        "\"team_id\"",
                        "\"user_id\""
                    ]
                }
            ]
        },
        "checks": [],
        "index": [],
        "partitioned_by": [],
        "tablespace": null
    }
]

It correctly identifies the composite uniques in alter.uniques. But for the individual columns, they also show "unique: true". I believe that's incorrect since there would be no way to know if the column itself has a non-composite unique index.
Looks like it's been recently fixed here: https://github.com/xnuinside/simple-ddl-parser/issues/236. Maybe the ALTER TABLE ... ADD CONSTRAINT UNQIUE were missed out there?

@hubg398 I think your words totally make sense, need to fix it

fixed in 1.5.0, really appreciate for opening this issue, test added https://github.com/xnuinside/simple-ddl-parser/pull/256, if will be needed anything else - feel free to open new one


Adopt the Microsoft Open Source Code of Conduct

Microsoft projects now have a unified code of conduct. This PR implements it by pointing to the document from the README.

:+1:

@EdMaurer

:+1:

:+1:


能否把演示环境的后台部分代码也开源出来？

Subject of the feature
Describe your issue here.
Problem
If the feature requests relates to a problem, please describe the problem you are trying to solve here.
Expected behaviour
What should happen? Please describe the desired behaviour.
Alternatives
What are the alternative solutions? Please describe what else you have considered?

演示页面的接口数据全部都在mock里


transfer hook cli: bump

In step with #5970, bump the transfer hook CLI!

Note: Rebased after latest crate version changes.


[Pods] - Update card designs for kits

We have decided to go with Option 1 for the design using the cards with logos (please see the team channel for the link to the figma file)
Acceptance criteria

[x] Make the card titles more prominent
[x] Add the showcases implementation from Figma including the link that jumps to the showcases list

We have some more requested feedback to handle for this ticket before the PR can be merged.
Homepage
All these tasks relate to the cards on the homepage

[ ] Increase font weight of showcase title to be bold (at least 500)
[ ] The title and icon list should both be left-aligned / in line with each other - can increase the icon size or spacing as needed to make it fit the card box better
[ ] Need to investigate how the order of the icons for each card on the homepage shows up. Ideally we want the key tech items to show first, then the smaller tooling / other items.

Logos and Tech Stacks

[ ] The Tanstack logo is too big for it's containing boxes and the edges bleed outside of the containing box - size needs to be shrunk
[ ] If we can, update/replace the BullMQ logo? It's a little hard to read / tell what it is
[ ] Solid and SolidStart - should use the full blue logo instead of the blue & green one it currently has
[ ] DenoDB has their own icon in their repo, get and replace the current one
[ ] Remove Oak from the tech stack list entirely (not really needed)

Kit Page

[ ] The card heading on mobile - the showcase and source links should be the full width of the card (will need to hide the spacer div)

[ ] On tablet sizes - when you click the "view showcases" link, the sticky header is over the section title, need to add scroll padding

Kit Page - Showcases Section (bottom of page)

[ ] GitHub text should have a capital H
[ ] Title should be base font size
[ ] Repo and App links should be small font size, no font weight
[ ] Remove the hr
[ ] Light mode: set card background to be gray-100 and border to be gray-400
[ ] Light mode: set icon border to be gray-400


code_of_conduct file added

resolve issue #33

Can you also add images to the readme?

sure @SohamKukreti


Master node crashed upon reboot

Attached Inspection Report
inspection-report-20220322_023344.tar.gz

It looks like some corruption happened to the dqlite data files perhaps due to an ungraceful shutdown.
Mar 22 02:28:37 calm-fox microk8s.daemon-k8s-dqlite[50658]: time="2022-03-22T02:28:37Z" level=fatal msg="Failed to start server: start node: raft_start(): io: load closed segment<PHONE_NUMBER>466626-0000000010467281: entries batch 594 starting at byte 7573496: entries count<PHONE_NUMBER>461211499 in preamble is too high\n"

I completely forgot how to resolve this particular issue.  I vaguely recall deleting some files and starting dqlite will suffice.
@ktsakalozos or @mathieubordere @neoaggelos  maybe able to help.

How do I find the offending file segment? Doesn't seem obvious to me here unless I missed something.

can you stop microk8s, make a backup of that folder, remove<PHONE_NUMBER>466626-0000000010467281 and try to restart?

Hey sorry to reopen this but this same issue happened again. I think this is the root cause, im not entirely too sure why this happened. I stepped away from my computer to attend to some other things and when i came back this is what I saw on my terminal.
Would you advise I repeat previous steps? Also anything I can do proactively to prevent this from happening again?

There are some interesting messages in dmesg as well:
...
[11699.243951] sd 1:0:0:0: [sda] tag#18 CDB: Write(10) 2a 00 01 0b 55 90 00 00 08 00
[11699.259926] scsi host1: uas_eh_device_reset_handler start
[11699.388640] usb 3-2: reset SuperSpeed USB device number 3 using xhci-hcd
[11699.411336] scsi host1: uas_eh_device_reset_handler success
[11735.596149] sd 1:0:0:0: [sda] tag#8 uas_eh_abort_handler 0 uas-tag 1 inflight: CMD
[11735.596167] sd 1:0:0:0: [sda] tag#8 CDB: Write(10) 2a 00 1d 17 b0 28 00 00 f8 00
[11750.956237] sd 1:0:0:0: [sda] tag#26 uas_eh_abort_handler 0 uas-tag 12 inflight: CMD
[11750.956252] sd 1:0:0:0: [sda] tag#26 CDB: Write(10) 2a 00 20 94 44 28 00 00 08 00
[11750.956262] sd 1:0:0:0: [sda] tag#25 uas_eh_abort_handler 0 uas-tag 11 inflight: CMD
[11750.956267] sd 1:0:0:0: [sda] tag#25 CDB: Write(10) 2a 00 20 94 31 68 00 00 08 00
[11750.956274] sd 1:0:0:0: [sda] tag#24 uas_eh_abort_handler 0 uas-tag 10 inflight: CMD
[11750.956279] sd 1:0:0:0: [sda] tag#24 CDB: Write(10) 2a 00 20 94 0c 70 00 00 08 00
[11750.956286] sd 1:0:0:0: [sda] tag#23 uas_eh_abort_handler 0 uas-tag 9 inflight: CMD
....

Maybe [1] is relative.
[1] https://forums.raspberrypi.com/viewtopic.php?t=303604


[ja] style update & translation fix

Review: @inductor

@inductor
https://www.jab.or.jp/files/items/common/File/NL5122015V1.pdf
https://physics.nist.gov/cuu/Units/checklist.html

@smaeda-ks I understand your point for the symbols but some of them are for Markdown and I don't think they need the blank for them. Your thought? :)

Resolved where the markdown modifications are. Please fix others and let me know 🙏

@inductor ! I actually pushed another commit to resolve your review comments right after this merge lol
https://github.com/smaeda-ks/http3-explained/commit/f7be9d95967500420a21033b580f1e4d9c0110df
Anyway, I'll add it to the next PR. Thanks!


handler drops connection

Hey great tool, however the reverse shell connection drops immediately.
I am not using the handler within the tool, I tried netcat listener and also metasploit exploit/handler and the same thing happens with both..
/wp-content/plugins/malicious/wetw0rk_maybe.php
-this file executes the reverse shell connection and it connects
/wp-content/plugins/malicious/QwertyRocks.php
-this file literally does nothing
I've tried it only visiting wetw0rk_maybe.php, and I've tried it visiting both consecutively as the tool hints at doing.
Either way the listener/handler catches the connection, but in MSF it automatically closes right after and with netcat as soon as i try to execute a command ("id") to test it; the connection closes.
What's the point of visiting both files to execute of wetw0rk_maybe.php executed the reverse shell and QwertyRocks.php does nothing?
Any ideas as to why the connection drops? Without assuming that it's the host's issue?
Using Kali 6.0.0 & zsh 5.9 - thanks!

Hard to say without the environment you're using. Is there a VulnHub you are working on where you tested this?
If you find out what's going on feel free to submit a pull request 😄
Love new contributors!
Thanks for using my implementation on the original module.
Did the original module work?


Will the local compiler for C/C++ be available in the future?

I’ve been using this app on my pad for a while, which is really convenient for us student. It’s awesome to see a local Python compiler available. However, some courses of mine require the c/c++ environment. I’m  wondering  if the local C/C++ compiler will be available in the future?

Yes. It would be possible and it's something I plan to add in the future.

Added in the latest build.


CMake Error

The error says, "CMake Error at CMakeLists.txt:47 (find_package):
By not providing "Findglfw3.cmake" in CMAKE_MODULE_PATH this project has
asked CMake to find a package configuration file provided by "glfw3", but
CMake did not find one.
Could not find a package configuration file provided by "glfw3" (requested
version 3.3) with any of the following names:
glfw3Config.cmake
glfw3-config.cmake

Add the installation prefix of "glfw3" to CMAKE_PREFIX_PATH or set
"glfw3_DIR" to a directory containing one of the above files.  If "glfw3"
provides a separate development package or SDK, be sure it has been
installed."
I made sure to update the .env file and I looked through the CMakeLists to make sure that all the paths were correct.

Did you rename the relevant env file from either envUnixExample.cmake or envWindowsExample.cmake to env.cmake? The call to find_package(glfw3 3.3 REQUIRED) on line 47 will only happen if you've not specified a value for GLFW_PATH and set it in the env file. You could also set it as a cache variable with -DGLFW_PATH=....
Since it looks like you're not setting a custom path, you could also install GLFW globally, which should allow find_package to locate the library correctly.


bug(#minor); monitor; Discord bot duplicating thread messages

The monitor is not reading the existing threads and creates messages for protocols that already have a thread.
The call to convert these messages to threads is failing.

https://discord.com/channels/953684103012683796/1002746564336431204/1089678586782421032

@bye43 I'll take this up

PR #1988 fixes this!
@MichaelC1999

@dhruv-chauhan merged/deployed!


Point to the correct type declaration file in package.json

Super minor quality of life improvement for typescript enjoyers. VSCode wasn't finding the type declarations in types/index.d.ts because of the missing .d. in the type dec filename.
Tested by linking locally w/ my fix, and intellisense was much happier.

@kroy thanks for catching that! Merging your PR now.
Thank you for contributing and being a part of open source 🙂

@kroy The most recent version 2.0.3 with these changes should now be available on npm.
https://www.npmjs.com/package/opendota.js
As always, if you find this package useful, consider leaving a ⭐ in the top right of the repository as it helps others find the package through increasing its popularity 🙂
Enjoy!

@wlamason I unfortunately was not a very good citizen here and changed the wrong filename 🙈 Something must have gotten mixed up while I was testing things. I fixed in https://github.com/wlamason/opendota.js/pull/21, but understand if you just want to revert and move on w/ your life.
Thanks for your time!


[feature] forbidden lower_case_table_names modify in my.cnf

Is your feature request related to a problem? Please describe.

Describe the solution you'd like

Describe alternatives you've considered

Additional context

In  MySQL 8.0 lower_case_table_names allow define in CRD  before MySQL cluster creating and  disallow modify when cluster is already running
But in MySQL 5.7 and lower  lower_case_table_names should allow modify any time
@acekingke


Trouble with addGraphQLSchema() and addGraphQLResolvers()

I'm having trouble with custom types and string syntax using addGraphQLSchema() and addGraphQLResolvers().
I'm logging in to a third party service--and I'm not sure if this is the correct approach--but I basically copied everything from the vulcan:voting package.
Error:

Uncaught (in promise) Error: GraphQL error: Cannot query field "login" on type "Mutation".
at new ApolloError

/lib/modules/authentication/graphql.js
import { addGraphQLSchema, addGraphQLResolvers, addGraphQLMutation, Utils } from 'meteor/vulcan:core';
import rp from 'request-promise'

const loginSchema = `
  type LoginResponse {
    sessionid: String
    success: String
    error: String
  }
`;

addGraphQLSchema(loginSchema);
addGraphQLMutation('login(email: String!, password: String!): LoginResponse');

const loginResolver = {
  Mutation: {
    login(args) {
      const options = {
        method: 'POST',
        uri: 'https://thirdpartyurl.com',
        body: {
          action: "login",
          password: args.password,
          username: args.email,
        },
        json: true
      };

      return rp(options)
        .then(function (parsedBody) {
          return parsedBody
        })
        .catch(function (err) {
          console.log(err);
          return err
        });
    }
  },
};

addGraphQLResolvers(loginResolver);

/lib/containers/withLogin.js
import React, { PropTypes, Component } from 'react';
import { graphql } from 'react-apollo';
import gql from 'graphql-tag';

const withLogin = component => {

  return graphql(gql`
    mutation login($email: String!, $password: String!) {
      login(email: $email, password: $password) {
        success
        error
        sessionid
      }
    }
`, { name: "login" })(component);
};

export default withLogin;

/lib/components/login.js
...
const email =<EMAIL_ADDRESS>password = "1234";
this.props.login({
    variables: {
        email: email,
        password: password
    }
})
...
export default withLogin(Login)

The code looks ok to me. Have you tried logging out your schema (Vulcan.getGraphQLSchema() in your meteor shell) to see what your mutation schema looks like?

Thank you, I was looking for that very command. The problem arose because I did not add my module to the index file (modules/index.js).


Fixing panic when non array value is passed to array type

Encountered a panic similar to issue https://github.com/vektah/gqlgen/issues/110 when passing an object to an array type.

@andrewmunro
can you share an example?

Thabks for the pr, a test would be the best way to give an example :)

👋 Added a test to the dataloader example to demonstrate this.
Essentially if you pass an object instead of an array in your query/mutation, gqlgen panics. Handling is not ideal as the field is now ignored with no errors, I imagine this will be fixed properly when @vektah introduces the new parser 🎉!
For now I hope this is sufficient to prevent app's from panic-ing with malformed input, without having some kind of middleware to recover the application...

Yeah that definitely should be caught by validation and give a good message to the user, but the runtime shouldn't panic.
:+1:


Clarifying return types in API

The current API documentation left me a little wondering what the actual return type is so I had to test even after reading the docs. This textual improvement would have helped me, maybe it helps others.

Thank you!


Bundle the font

For performance, offline support and privacy.
Cut down the CSS file from the package so that we only need to bundle
the one font file, like what Google was doing.

Thank you! Next time I'm ready to cut a release, I'll test this myself and merge

Hi @jlfwong, thanks for your response. @Krinkle and I are on the performance team at Wikimedia and we're looking at integrating speedscope with Wikimedia websites using our own sampling profiler for PHP. We're doing it in a way that should be usable by other PHP applications. The details are at https://phabricator.wikimedia.org/T291015

When I look at https://fonts.googleapis.com/css?family=Source+Code+Pro, it lists the woff2 formatted files rather than ttf. It looks like the npm package also contains these files: node_modules/source-code-pro/WOFF2/TTF/SourceCodePro-Regular.ttf.woff2. The woff files are smaller, which appeals to me. Is there a reason to use the ttf files instead of the woff2 files?
I'm also not sure what the licensing requirements of this become once it's bundled with the software.
From looking at https://github.com/adobe-fonts/source-code-pro/blob/release/LICENSE.md, it seems like I'm supposed to include licensing information somewhere in this project. If you're interested in having this merged, can you please look into that and see what, if anything, would need to change in this repository?

When I look at https://fonts.googleapis.com/css?family=Source+Code+Pro, it lists the woff2 formatted files rather than ttf. It looks like the npm package also contains these files: node_modules/source-code-pro/WOFF2/TTF/SourceCodePro-Regular.ttf.woff2. The woff files are smaller, which appeals to me. Is there a reason to use the ttf files instead of the woff2 files?

I didn't realise that it delivers different stylesheets depending on User-Agent. If you use curl, it only shows TTF.

I'm also not sure what the licensing requirements of this become once it's bundled with the software.
From looking at https://github.com/adobe-fonts/source-code-pro/blob/release/LICENSE.md, it seems like I'm supposed to include licensing information somewhere in this project. If you're interested in having this merged, can you please look into that and see what, if anything, would need to change in this repository?

Yes, I think you're right that a license notice is required. Although probably all of your runtime dependencies have this problem.

I didn't realise that it delivers different stylesheets depending on User-Agent. If you use curl, it only shows TTF.

Gotcha. As far as I can tell, WOFF-2 is very widely supported now (https://caniuse.com/?search=woff2), so I'd prefer this use that instead to keep things as compact as possible.

Yes, I think you're right that a license notice is required. Although probably all of your runtime dependencies have this problem.

This might be true, but AFAIK all of my other runtime dependencies have common software licenses like MIT, rather than a font-specific license. I've never heard of folks getting in trouble for bundling MIT licenses for npm dependencies (otherwise every single npm package would need this). But I know much less about legal issues around fonts.

Yes, I think you're right that a license notice is required. Although probably all of your runtime dependencies have this problem.

This might be true, but AFAIK all of my other runtime dependencies have common software licenses like MIT, rather than a font-specific license. I've never heard of folks getting in trouble for bundling MIT licenses for npm dependencies (otherwise every single npm package would need this).

Stuff about MIT and FLOSS licenses
I am not a lawyer, but my understanding based on working in open source at Wikimedia Foundation and in the jQuery Team (OpenJS/Linux Foundation) for over a decade, is as follows:
The MIT license is among the most liberal in open source. Practically the closest thing to being entirely in the public domain. You can do literally anything you want with it, as long as you attribute its authors/copyright line somewhere. Similar in spirit to the Creative Commons Attribution license in that sense. "Attribution", thus requires the copyright line to be preserved or otherwise presented in distributions that include copies of the software.
Most npm packages are not end-user applications or products, and thus don't have any practical involvement around this. For a typical library package, its only concern is its own license in the repo/package uploaded to npm, and anyting it uses or that others use it for, is handled transparently by npm-install later on when someone else uses it. When you upload your own package to npm, usually, you are not distributing your dependencies for they are not included in the package. package.json merely points to them by name, and it is later on, that someone else runs npm-install and then installs both it and the dependencies. Even for end-user applications like ESLint and Grunt, npm solves it automatically. Naturally, their dependencies are either not yet downloaded (you haven't ran npm-install yet) or both a copy of the code and its LICENSE file are present (after you run npm-install, in the node_modules directory). This requires no further action on your part.
If you do something special with the code and distribute it by other means outside npm, or if you attach artefacts to your npm package that bundle/embed/inline code from dependencies, then you have to preserve or present the attribution by other means yourself. For example:

If you distribute a tarball that includes the node_modules directory, you probably don't have to do anything. This naturally preserves any license header in source files and any separate license text files in directories.
Firefox (the finished binary on your computer) compiles and embeds lots of C++ and JS software under a variety of licenses. There is not really a way to view this in source form after compilation, and thus does not naturally preserve attribution. This is why Mozilla included a semi-automated way to present this in their UI.  You can go to Firefox > About > License information (or about:license) and you'll find a list there of each package they embedded/compiled, and the legally required attribution (e.g. Copyright John Doe). Most FLOSS licenses, also require a copy of (or web link to) the license terms. You'll find similar "about" views in many desktop apps, as well as mobile apps for iOS/Android often have such a license page.

In the case of Speedscope, Parcel is used to produce a single JS file and a single CSS file, and any comments are stripped. My best guess is that this is technically in violation of the MIT license for your dependencies. When I publish a copy of Speedscope on my own domain, I'm (probably?) not in violation of your license, since the Speedscope UI links to your GitHub repo, where your license file can be viewed. However, there's not really a clear path to the copyright statement of your dependencies (much less which ones were bundled, or which versions, or which copyright statement they had at that time). Parcel strips it all out. I could not find anything in the the Speedscope release tarball, e.g. the HTML, JS file, or other text files, that indicates what code was bundled or who holds their copyright. The source repo also does not distinguish in package.json between dev and prod dependencies (noting that your dev dependencies are generally not distributed. E.g. the tools you use for bundling, testing, minification are not being distributed in the release and do not require attribution). Instead, it seems your prod dependencies are listed among dev-only, and some of them are then bunded into the JS file during release. This makes sense for efficiency, but means that the npm distribution is probably technically in violation as well.
Font
As for the font, https://openfontlicense.org/ofl-faq/ does a good job of answering the hard questions. It is in essense an even more liberal license than the MIT license, as it presents plethora of scenarios where you do not even have to attribute its author or copyright holders. Unlike commerically licensed fonts where you may have to pay to download them, and then pay pay for permission to use the font in a brand or publication.
The OFL is much like the MIT license in that it permits copying, modification and re-distribution, so long as you preserve a copy of the original license text and/or other attribution. In addition, it grants very useful and practical exemptions for work that merely applies the font (e.g. a logo or a screenshot of Speedscope where the font is used), where it requires no attribution at all. Given that Speedscope releases would include a copy of the file as-is, atttribution would be required, however.
While by no means required, I would personally choose the following as the simplest approach:

Create the assets/source-code-pro directory with in it the SourceCodePro-Regular.ttf.woff2 file and https://github.com/adobe-fonts/source-code-pro/blob/release/LICENSE.md.
Check, or update as needed, the release step so that during or after the Parcel step, there are both SourceCodePro-Regular.woff2 and SourceCodePro-Regular.license.md in the dist directory. Alternatively, you could create your own "NOTICE.txt" file in Git with the two attribution lines of the license and make sure to copy that into the dist folder. You may want to expand that at some point to include the attribution of this repo and other dependencies as well.

Hey @Krinkle! Thanks for the thorough rundown, and especially for hunting down the licensing details of source-code-pro!
Regarding the technical MIT violations, I think you're probably right too. That sounds, frankly, like a pain to resolve, and certainly a separate concern from what this PR is addressing. So for now I think I'll punt on that.
The simplest approach you propose sounds good to me, and I'd be happy to accept a PR doing that.

Closing in favor of #472


Realtime not working

After message send destination side have to refresh tab not auto load chat like proper chat functionality. how to set auto refresh chat body.

I don't understand what you mean.

it means Real time chatting is not working. it means if user1 send message to user2 so user2 have to refresh page to view message of user1.

It is working as expected (with real-time chatting) over here. Probably something is wrong from your side.

Which thing you expect is wrong?

Probably your Pusher credentials. Open dev tools to see if there are any kind of errors.

Real time chatting not working.
To see the mesages i have to refresh the page everytime.

This happened to me too, it was because the pusher cluster was staticly set to 'eu' in public/js/app.js
After changing that it works.
@ammezie I recommend creating an environment variable in .env.example for the cluster to have that set dynamically in public/js/app.js and config/broadcasting.php

Thanks for pointing that out. As at the time of creating the app, Pusher cluster is not required to be explicitly set. I will update the code and readme as well.

I have updated the code and the readme, hence I'm closing this.

I have to refresh the page to see the difference. So i went to the console log and see this.
I don't understand the problem.


Windows 10 install: git-lfs: command not found (can't install esp-matter SDK using provided tools/commands)

Using the OFFLINE installation files for ESP-IDF (https://dl.espressif.com/dl/esp-idf/?idf=4.4), and installing the bundled 4.4.2 with IDE as well as the updated 4.4.3 standalone, I have a functional ESP-IDF/IDE installation. Now I'm attempting to install the ESP-MATTER SDK, and after success enabling long path names, I have encountered what looks to be my last problem: The git LFS plugin won't install, and it was never included in the embedded git that came with ESP-IDF 4.4.3. I downloaded git-lfs-windows-v3.2.0.exe (current as of today) and attempted to install it, only for it to tell me git is not installed on my computer. Of course it is, but it's embedded in ESP-IDF. On my computer it is at: C:\Espressif\tools\idf-git\2.34.2\cmd\
There does not seem to be any way to install lfs from the Windows command line and the installer doesn't find git as installed by esp-idf.
I considered reporting this in the esp-idf repository, but it doesn't require LFS, so here I am.
Here's what my console looks like:
``
// starts off fine
C:\Espressif\frameworks\esp-idf-v4.4.3>git clone --recursive https://github.com/espressif/esp-matter.git
Cloning into 'esp-matter'...
remote: Enumerating objects: 3926, done.
remote: Counting objects: 100% (577/577), done.
remote: Compressing objects: 100% (163/163), done.
remote: Total 3926 (delta 478), reused 487 (delta 412), pack-reused 3349
Receiving objects: 100% (3926/3926), 3.13 MiB | 9.09 MiB/s, done.
Resolving deltas: 100% (2740/2740), done.
Submodule 'connectedhomeip/connectedhomeip' (https://github.com/espressif/connectedhomeip.git) registered for path 'connectedhomeip/connectedhomeip'
// [...]
// then this small error
Cloning into 'C:/Espressif/frameworks/esp-idf-v4.4.3/esp-matter/connectedhomeip/connectedhomeip/third_party/mbed-mcu-boot/repo/boot/espressif/hal/esp-idf/components/bootloader/subproject/components/micro-ecc/micro-ecc'...
fatal: '$GIT_DIR' too big
fatal: clone of 'https://github.com/kmackay/micro-ecc.git' into submodule path 'C:/Espressif/frameworks/esp-idf-v4.4.3/esp-matter/connectedhomeip/connectedhomeip/third_party/mbed-mcu-boot/repo/boot/espressif/hal/esp-idf/components/bootloader/subproject/components/micro-ecc/micro-ecc' failed
Failed to clone 'components/bootloader/subproject/components/micro-ecc/micro-ecc'. Retry scheduled
// [...]
// then LFS identified as a problem
Cloning into 'C:/Espressif/frameworks/esp-idf-v4.4.3/esp-matter/connectedhomeip/connectedhomeip/third_party/ot-br-posix/repo/third_party/openthread/repo'...
remote: Enumerating objects: 95005, done.
remote: Counting objects: 100% (246/246), done.
remote: Compressing objects: 100% (149/149), done.
remote: Total 95005 (delta 119), reused 182 (delta 97), pack-reused 94759
Receiving objects: 100% (95005/95005), 95.14 MiB | 9.98 MiB/s, done.
Resolving deltas: 100% (74658/74658), done.
Submodule path 'connectedhomeip/connectedhomeip/third_party/ot-br-posix/repo/third_party/cJSON/repo': checked out 'cf97c6f066d81fdbba4ef722cfd327bbbba2365c'
Submodule path 'connectedhomeip/connectedhomeip/third_party/ot-br-posix/repo/third_party/http-parser/repo': checked out '4f15b7d510dc7c6361a26a7c6d2f7c3a17f8d878'
Submodule path 'connectedhomeip/connectedhomeip/third_party/ot-br-posix/repo/third_party/openthread/repo': checked out '2ce3d3bf0218566484be2e9943b95c755cefebe3'
Submodule path 'connectedhomeip/connectedhomeip/third_party/pigweed/repo': checked out '9235aeb653e684a6f0b7b563965d85c747281a0f'
Submodule path 'connectedhomeip/connectedhomeip/third_party/pybind11/repo': checked out '8de7772cc72daca8e947b79b83fea46214931604'
git-lfs filter-process: git-lfs: command not found
fatal: the remote end hung up unexpectedly
git-lfs filter-process: git-lfs: command not found
fatal: the remote end hung up unexpectedly
Submodule path 'connectedhomeip/connectedhomeip/third_party/silabs/matter_support': checked out '2425fb72cacd540df505f11e7de81e6270ed1c40'
// [...]
Submodule path 'connectedhomeip/connectedhomeip/third_party/zap/repo': checked out 'd4ce2c13916f1b1e066c6112dbee3e0dc5f5064f'
fatal: Failed to recurse into submodule path 'connectedhomeip/connectedhomeip/third_party/mbed-mcu-boot/repo'
fatal: Failed to recurse into submodule path 'connectedhomeip/connectedhomeip/third_party/openthread/ot-efr32'
fatal: Failed to recurse into submodule path 'connectedhomeip/connectedhomeip/third_party/openthread/ot-ifx'
fatal: Unable to checkout '8aadeb120922720aba2bf0be8d5230d979856a4e' in submodule path 'connectedhomeip/connectedhomeip/third_party/qpg_sdk/repo'
fatal: Unable to checkout '799200bb6788939aa0fef97d7fc4ee9aca57148d' in submodule path 'connectedhomeip/connectedhomeip/third_party/silabs/gecko_sdk'
fatal: Failed to recurse into submodule path 'connectedhomeip/connectedhomeip'
``

I found a hacky workaround for the git-lfs errors, but the "fatal: '$GIT_DIR' too big" errors persist.
For the git-lfs problem, find your git.exe in your embedded install folder, and paste the contents of the binary download (https://github.com/git-lfs/git-lfs/releases/download/v3.2.0/git-lfs-windows-amd64-v3.2.0.zip) in there and then run "git lfs install" on the command line. I found multiple git.exe files, and instead of trying to figure out which one is the right one, I just pasted those LFS files into each folder. An unintended side effect of this hacky solution is that the esp-matter folder showed up in the ../.git/ subfolder. Since it still doesn't work, I guess that's not an issue at the moment.

At the moment supported development hosts are:

macOS 10.15
Debian 11 (64 bit required)
Ubuntu 22.04 LTS

Please find more details here
Although esp-matter can be setup on Windows using WSL, this is not documented anywhere (will add this to docs)

Enable the Windows Subsystem for Linux 2 (WSL2) following instructions here: https://docs.microsoft.com/en-us/windows/wsl/install-win10
Install Ubuntu from the Windows App Store here: https://www.microsoft.com/store/productId/9MTTCL66CPXJ
Start the Ubuntu (search into start menu)
Running uname -a from within WSL(Ubuntu installed from Microsoft store) should report a kernel version of <IP_ADDRESS> or later. You’ll need to be running a WSL 2 distro. To upgrade the kernel, run "wsl --upgrade" from Windows Power Shell (ref: https://askubuntu.com/a/1384458)
Install usbip-win : https://github.com/espressif/vscode-esp-idf-extension/blob/master/docs/WSL.md. Required to map the COM ports on windows into WSL. (another ref: https://github.com/dorssel/usbipd-win/wiki/WSL-support)
Setup esp-matter: https://docs.espressif.com/projects/esp-matter/en/main/esp32/developing.html

Thank you very much for your reply. I'm making another attempt at the VM since I can now get past my work proxy, but if I still have trouble under native linux, I'll take your advice and try the linux subsystem.

@fkkjohn Any updates you would want to share? Please close this issue if it is working as expected

Thank you. No related problems at the moment. Closing.


VAD solutions?

Thanks for posting this.  It's been a real inspiration!  I've got something very similar set up, and took some inspiration from your "Using GPT3 for intent recognition" post.  I'm currently struggling with VAD.  Is there something you use for VAD, or are you just using a fixed time window?  I'm using Whisper-Live and have toyed with the idea of running the current query through a local "is this a complete sentence" model, but haven't found anything fast or reliable enough yet.  I'm currently testing out picovoice's Cobra VAD solution, but haven't been able to nail down a reliable threshold.

Thanks for responding!  I have been using picovoice's porcupine on a pi4b,
and it seems to pick up the wake word 90% of the time.  No false positives
yet.  For VAD I'm trying out picovoice's solution.  It turns out I had a
bug and wasn't always filling the buffer completely, which led to most of
my issues.  I did try running Mistral-7b locally to determine sentence
completeness but it was slow and not very accurate.  Maybe with a beefier
GPU and more VRAM, but I'll set that aside for now and experiment with it
more later.
Thanks again for taking the time to respond!
On Thu, Oct 31, 2024 at 5:26 AM Dale @.***> wrote:

Hey! Sorry for taking some time to respond. It's been a while since I used
Rhasspy, so I cannot remember all my VAD settings, but I wasn't doing
anything fancy, just using the built in models to detect when there was no
voice, and tweaked the settings so it didn't stop too early (like when
pausing while asking the assistant something) or take too long after I had
stopped speaking.
I think your idea of using Whisper-Live + "is this query complete?"
interesting. Maybe you could build your own model or fine-tune an existing
model? The good people on https://community.rhasspy.org/ might have some
ideas.
What I struggled most with was reliable wake word detection. I found that
the assistant had false positive tiggers – it would trigger even when the
room was silent, or something was on the TV or if music was playing in the
background. I used openwakeword, and dockerised that so I could run
detection on a beefier computer than the Pi. (See
https://github.com/dalehumby/openWakeWord-rhasspy)
If I was going to restart I'd probably investigate what was built into
Home Assistant, since that also has integrations into various LLMs, and the
voice part is built by the same developer who built Rhasspy.
—
Reply to this email directly, view it on GitHub
https://github.com/dalehumby/rhasspy-config/issues/1#issuecomment-2449725983,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACCTBGDEGCEJR6VFRYDEVDZ6IOYXAVCNFSM6AAAAABQOE54POVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBZG4ZDKOJYGM
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Updating Documentation for final product

Includes UG/DG/PPP

That is a really weird bug.. I'll fix it and push again


OpenAPI functions RenderOAuth2Redirect, RenderOpenApiDocument, RenderSwaggerDocument, and RenderSwaggerUI missing when published to folder

Published my function app to a folder so I can zip it and put it on a share and I noticed that the functions created by Microsoft.Azure.WebJobs.Extensions.OpenApi are not there.  I see the functions when debugging.
Did I miss a step somewhere?
Using V3 Functions SDK and VS2019.

Hi there, I am also rather confused by this.
When running via debug, the additional RenderOAuth2Redirect, RenderOpenApiDocument, RenderSwaggerDocument, and RenderSwaggerUI endpoints are present.
When publishing the Azure Function (via func tasks in VSCode) it does deploy successfully however as @gitmadness has noted our shiny OpenAPI pages got lost somewhere.
Assuming that it might be some hidden magic, I have tried visiting;

{functionUri}/swagger.json?code={code}
{functionUri}/swagger/swagger.json?code={code}
{functionUri}/swagger/ui?code={code}

..all to no avail.
Justin Yoo over at MSFT has created some extensions to allow publishing of these as functions but I am not sure whether these work with the current landscape we're using (dotnet6, inproc, v4 runtime - posting this primarily for @gitmadness's benefit as this may help them).

Blog post describing the implementation: https://devkimchi.com/2019/02/02/introducing-swagger-ui-on-azure-functions/
Repository containing the libs: https://github.com/aliencube/AzureFunctions.Extensions

I have run into some implementation issues with the above, like GetSpecVersion() and GetExtension() methods, these are used in the example without any prefixes; suggesting they should be present in the example somewhere, but aren't, nor can I find them in the assembly anywhere. Some more search-fu later and I landed on this github issue.

Ok so I made some headway on this, started a fresh test project using the AlienCube nuget mentioned previously and got it working quite happily.

Dotnet 6 IoC, v4 runtime
Builds ok
Debugs locally ok with all the right endpoints
Deploys to Azure FunctionApp ok
Has all the right endpoints published, and they all work

To be brutally honest with myself I just RTFM with a fresh test project and it worked 🤷 I think my test yesterday must have had some conflicting config from all the headscratching trying to get it working.
Given the nature of the lineage of these nugets, I tried porting the same working config to use the Microsoft.Azure.Functions.Worker.Extensions.OpenApi nuget, all works flawlessly... apart from no OpenApi Functions published to the Function App on deployment.
No offense to anybody, but I suspect the current release here is borked. With the above workaround it's no biggie (for me) now though - and porting the same config to this repo's way of doing it is actually quite trivial, technical debt can be paid off at some point with a single paycheque 😁
@gitmadness feel free to give me a yell if you can't get the AlienCube way of doing it working, happy to help.

@gitmadness @JohnGe0rge Thanks for the issue! I assume that you're using "in-proc" worker of Azure Functions app, by using the Microsoft.Azure.WebJobs.Extensions.OpenApi package. Because it's the characteristics of the "in-proc" worker, those endpoints are showing up on your local machine, but they're hidden (encapsulated) when deployed to Azure.
Just FYI – The Aliencube one is no longer maintained, and this official extension has more features than that.


Upgrade UI - Add Components for Transaction Screens

[x] Screen Tab (use in Stake screen) (Quang)
[x] Value Input (output ra số full) (Quang)
[x] Data Tables (LW )

[x] 		Confirmation
[x] 		Staking Screen
[x] 		Sign Screen

[x] Staking Network details (LW)
[ ] Select Pool Input (Quang)
[ ] Select Nominator Input (Quang)

Version test: https://file.cdmteck.com/s/4oimk9zDJ8R5Aai/download?path=%2F&files=./upgrade-ui-build-0.8.1-1678083121116.zip

[ ] Do not show background in case below
Steps:

Open the any dApp (ex: connect.subwallet.app)
When the connect wallet popup displayed, the user closes this popup
Re-open the SubWallet extension
Click on Cancel/Connect/Block button to close the connect request
Go to the any select modal then observe the background displayed

Actual: Do not show background

Expected: Show background


Initial Calibration: Cancel Calibration

As a user, I would like to be able to cancel calibration (and still have a valid config file)
Acceptance Criteria

Add a function to the calibration endpoint to cancel calibration
User should be left with a their initial calibration file

Implementation Notes

How might we leave the initial calibration in a valid state until user does a final save on calibration?
-- Note this is covered by #953

All functionality covered by other tickets
-- We will not implement an explicit 'cancel' on the BE
-- Starting a new calibration will overwrite any in-progress calibrations


Footnote numbering in table - incorrect

Can you please update the footnote numbers in the table on this page. No1 is referenced below, but it doesn't appear in the table.
No3 is listed in footnote but doesn't appear in table.
No4 says retention is supported for the shadow mailbox but the article that is linked to says that holds nor retention is supported for shadow mailbox.
Additionally, can you please list for each column if OAuth is needed or not? This would add welcome clarity. OAuth isn't needed for private chat journaling for example. Having OAuth and AADConnect sync listed on the row header makes it look like OAuth is needed for all columns which it isn't.
Thanks.

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

ID: 8456689d-6f04-fa5e-4f51-f413cb0b9d89
Version Independent ID: e9b85126-0db7-4378-ad44-514e52f9fde1
Content: How Exchange and Microsoft Teams interact - Microsoft Teams
Content Source: Teams/Exchange-Teams-interact.md
Service: msteams
GitHub Login: @dstrome
Microsoft Alias: dstrome

@causeway74  Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@causeway74 thank you for reaching out and helping improve MS Docs. It looks like the article has been updated in the interim. Please let me know if you need further assistance.

@lindspea Hi, I don't see this reflected in the current article. The footnote numbering is still incorrect.

@dstrome please could you assist regarding footnotes.

Thanks for your feedback. I've updated the footnote ordering and removed the link to the outdated blog article. I will need to investigate the OAuth issue so this will take longer.


pretrained S/4 Chair model's performance

May I ask why the result of using V100 16G to validate the pre trained model is 1-NN-CD: 64.50 1-NN-EMD: 63.75 COV-CD: 39.88 COV-EMD: 43.05 instead of the expected 1-NNA-CD: 56.31 1-NNA-EMD: 55.82 COV-CD: 47.21 COV-EMD: 50.75 in the ReadMe

Hi, thanks for your interest in our work! Please follow README step by step to achieve the expected results.

May I ask why the result of using V100 16G to validate the pre trained model is 1-NN-CD: 64.50 1-NN-EMD: 63.75 COV-CD: 39.88 COV-EMD: 43.05 instead of the expected 1-NNA-CD: 56.31 1-NNA-EMD: 55.82 COV-CD: 47.21 COV-EMD: 50.75 in the ReadMe

Hi  Did you achieved their results?


Code error when importing a new midi file into nbs

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Import any custom .mid file
play
See error

Expected behavior
Playing the song like normal
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context

Hi, and thanks for reporting the issue!
Could you please attach the affected MIDI file here? We need to inspect it in order to determine what's causing the problem. :)

I have the same issue, @Bentroen here is my midi file: https://onlinesequencer.net/app/midi.php?id=2660660

Error:

___________________________________________
############################################################################################
ERROR in
action number 1
of Draw Event
for object obj_controller:

Push :: Execution Error - Variable Index [255] out of range [128] - -1.midi_ins(100607,255)
at gml_Script_open_midi
############################################################################################
gml_Script_open_midi (line -1)
gml_Script_draw_window_greeting
gml_Script_draw_windows
gml_Script_control_draw
gml_Object_obj_controller_Draw_0 (line -1)

The Loose Hi-Hat of the instrument 2013 Drum kit in onlinesequenser when imported to midi has a instrument id of 255, which is illegal according to the midi specification. Try using another instrument when exporting.

Also duplicate of #285


updated docs functionality to account for dataclasses refactor

Following up on this PR: 1713
This code has been refactored to account for the new dbt dataclasses re-factor.

Hi @drewbanin. I believe CLA stuff should be in order as well? I changed my primary github email to match the info on the CLA, but let me know if you have any additional issues.
Thanks!

Thanks @talagluck! I'm kicking off the test suite now. I do expect that there will be many tiny changes required to the unit/integration test suite - lmk if you have any questions about how to resolve those once the test results roll in :)

Hey @talagluck - you should be able to see a list of failed unit and integration tests now. There are a good number of them, but the fixes are all really tiny - you'll just need to add an empty docs dict to each of the "expected" data structures in the tests.
Definitely let me know if you'd like a hand or some pointers! To run the unit / integration tests locally, check out our contributing guide

Hi @drewbanin. I fixed a few of the tests and committed without thinking, so I'm waiting for the CI to complete again.
I tried setting up an environment to run tests locally and got this error when trying to run PGHOST=localhost PGUSER=root PGPASSWORD=password PGDATABASE=postgres bash test/setup_db.sh:
+ grep '^PG'
PGUSER=root
PGPASSWORD=password
PGDATABASE=postgres
PGHOST=localhost
+ PGUSER=root
+ export PGUSER
+ PGPORT=5432
+ export PGPORT
+ PGHOST=localhost
+ [[ -n '' ]]
+ createdb dbt
createdb: could not connect to database template1: FATAL:  role "root" does not exist
+ psql -c 'CREATE ROLE root WITH PASSWORD '\''password'\'';'
psql: FATAL:  role "root" does not exist
+ psql -c 'ALTER ROLE root WITH LOGIN;'
psql: FATAL:  role "root" does not exist
+ psql -c 'GRANT CREATE, CONNECT ON DATABASE dbt TO root WITH GRANT OPTION;'
psql: FATAL:  role "root" does not exist
+ psql -c 'CREATE ROLE noaccess WITH PASSWORD '\''password'\'' NOSUPERUSER;'
psql: FATAL:  role "root" does not exist
+ psql -c 'ALTER ROLE noaccess WITH LOGIN;'
psql: FATAL:  role "root" does not exist
+ psql -c 'GRANT CONNECT ON DATABASE dbt TO noaccess;'
psql: FATAL:  role "root" does not exist
+ set +x

Hey @talagluck - just kicked off the tests again.
Re: that error: I think you may need to run:
docker-compose up -d database

via the contributing guide. Let me know if you've already tried that!

Thanks, @drewbanin! I did do that. I basically followed the instructions line by line until that point.

huh! And you're still not able to get it working?
I don't think you should need to do this, but maybe you can try running:
PGHOST=localhost PGUSER=root PGPASSWORD=password PGDATABASE=postgres createuser root

This should create the root user, which should allow the rest of the commands to succeed. Let me know if that fixes it - we might need to update the docs?

Hi @drewbanin. I ran createuser root right before PGHOST=localhost PGUSER=root PGPASSWORD=password PGDATABASE=postgres and that worked! I'll let you know if I have any more trouble.

@drewbanin, I've updated some more tests based off of the results of the CI. I've been having a bunch of trouble getting my environment set up for proper testing, so I did it sort of blindly. Would you have some time today or tomorrow to pair on figuring out that devloop?
Thanks!

Sure thing @talagluck - feel free to drop me a line on Slack - happy to help you get this sorted.

Hi @drewbanin, I've made some updates to the tests and a minor update the NodeConfig dataclass, and it seems as though all the tests have passed! Could we rerun CircleCI?

@talagluck should be running now!

hey @talagluck - looks like the unit tests are passing! I can see some failures on the Postgres integration tests, and I bet there are other minor test tweaks to make around Snowflake/BigQuery/Redshift too.
Let me know if there's anything I can help out with here!

Thanks, @drewbanin! I was out last week and now I'm catching up on things, but I'm hoping to get back to this tomorrow. I might have a slight issue running the Postgres tests, and I'll let you know tomorrow if I can't figure anything out.

Hi @drewbanin - I'm having some trouble getting a testing postgres environment to work on my machine, and also understanding the failing errors. Do you have a few minutes to pair today?

Hi @drewbanin, I'm back from parental leave, and I'm looking to finish this up. I'm going to go to the #development channel of dbt slack for broader questions, but wanted to check in here to find out what branch I should rebase from/to at this stage.
Thanks!

hey @talagluck - welcome back! We're still on dev/louisa-may-alcott, but we're wrapping up that release within the next couple of weeks. I'd love to slip this change in, but if not, we can definitely target it for the 0.15.1 release shortly thereafter. Either way, dev/louisa-may-alcott is going to be the move for the time being :)

Hi @drewbanin, apologies - this keeps getting away from us. Where do things stand now? What branch would it be best to work from at this point?
Thanks!

hey @talagluck - this can go out in our 0.16.0 release - you'll want to change the base branch to dev/barbara-gittings and pull in any new commits as required!

Thank you @drewbanin! I've been out on leave and just came back. Apologies for the delays on this. I'll let you know when we have the resources to jump back in!


Create our own executor if none is provided.

We don't want to run on the common pool as that might block
application threads.

👍


[PR] Adding Json controller tests

Description of change

Summary: Adding Json Controller tests.

Codecov Report

Merging #30 into develop will increase coverage by 1.22%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           develop     #30      +/-   ##
==========================================
+ Coverage     3.14%   4.37%   +1.22%
==========================================
  Files           73      73
  Lines         1144    1144
  Branches        79      79
==========================================
+ Hits            36      50      +14
+ Misses        1106    1091      -15
- Partials         2       3       +1

Impacted Files
Coverage Δ

source/EduCATS/Helpers/Json/JsonController.cs
77.77% <0.00%> (+77.77%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 311f693...da30d62. Read the comment docs.


ginzo, endways

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[e,g]

/trunk merge


Dependencies permanently lost when 0-byte file is read

Description
I was experiencing an issue when Brunch detected changes to my files. Sometimes, the dependencies of the file that I saved would permanently disappear. Randomly when I would save saved-file.js, which did require('foo'), I would see the error Uncaught Error: Cannot find module 'foo' from 'saved-file.js'. I also saw, in my terminal, that X had decreased from the previous message when compiled saved-file.js and X cached files into app.js in Y ms was printed.
I did a lot of digging into the brunch, deppack and finally fcache source code and found that sometimes fcache was reading 0-byte files. When it would read these empty files, it would still analyze them, and when it found that dependencies of this momentarily-empty file were no longer referenced by any other file, those files would be "unlinked" and not included in the build any more.
This led me back to the Brunch issues, and I found this: https://github.com/brunch/brunch/issues/971 and now awaitWriteFinish has solved my problem.
However, I think Brunch probably could have handled 0-byte files more gracefully - in case someone didn't read the docs and find out about awaitWriteFinish (which should maybe even be on by default), or if awaitWriteFinish ever doesn't work (I expect it will eventually fail, since it's timer-based).
Expected behavior
Even after a file is read with 0 bytes and its dependencies are dropped, those dependencies should be re-added if the same file is read again and found to have the same dependencies. If Brunch at least does that, it may be able to recover from reading these 0-byte files, or at least some less knowledgeable users (like I was) wouldn't need to restart their whole server (they could just re-save).
Also, I think that awaitWriteFinish should be in the docs and recommended for various editors, including Emacs.
Actual behavior
When Brunch reads a saving 0-byte file that has dependencies, it stops including those dependencies in the build forever.
Here's a demo: https://github.com/jacksonrayhamilton/brunch-lost-dependencies-demo
Environment

Brunch: 2.9.1
Node: 6.9.2
NPM: 3.10.9
Operating system: Debian GNU/Linux 8

package.json contents
{
  "private": true,
  "scripts": {
    "start": "brunch watch --server"
  },
  "devDependencies": {
    "auto-reload-brunch": "^2.7.1",
    "brunch": "^2.9.1",
    "lodash": "^4.17.2"
  }
}

brunch config contents
/* eslint-env node */

'use strict';

module.exports = {
  files: {
    javascripts: {joinTo: 'app.js'}
  },
  modules: {
    autoRequire: {
      'app.js': ['main']
    }
  }
};

Hi, @jacksonrayhamilton! Thanks for reaching out!
Your issue is related to problems with deppack module.
Here are other issues: deppack related issues.
We're working on it. I believe we will fix that as soon as it possible.


Automator: update istio/client-go@master dependency in istio/istio@master

Generated by Automator - 2021-06-01T15:15:15+00:00

@istio-testing: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Conflict due to change already being merged. Closing this PR.


(PUP-9093) Add test for facts sanitize regarding to_s returning 8bit str

This adds a test that prevents a string returned from to_s on
some arbitrary "alien" runtime value to be returned with ascii-8bit
encoding (which is bad because that is later taken as being a binary
value and not text).

CLA signed by all contributors.

@joshcooper fail due to unused variable fixed


Creates a read subcommand that allows for reading the story in terminal.

This is a pretty simple reader -- I use pandoc to convert to markdown, and pydoc to display the content in a nice pager interface.
I use this for previewing and reading over SSH when making a full EPUB is too heavy weight.
May be beyond scope of leech, but I find it handy when I'm just trying to read a single update, and not interested in sending an epub to my reader. Notable problems include that it's overkill to load the entire story just to view a chapter.

Also -- if this seems in scope, I plan to set it up so that pandoc and pydoc are not full dependencies -- if they are not installed, I'll just show a message encouraging the user to install them.


Added file and line to optional assert

As suggested in the forums this PR adds #file and #line default parameters to an optional() reducer method to add the information into the assertion message.
Questions I would like to ask:

Are you happy if the function replaces the property (breaking change) or do you want to keep both?
Feel free to suggest edits on the wording of the message.
What other places would you like to see this change? forEach maybe?

Once this is clarified I will update the examples and tests 😄

Thanks @alexito4!

Questions I would like to ask:

Are you happy if the function replaces the property (breaking change) or do you want to keep both?

Let's keep both but deprecate the property if possible. We keep all of our deprecations here for now:
https://github.com/pointfreeco/swift-composable-architecture/blob/a03611d45da994a706911ca024fa441b148e79cb/Sources/ComposableArchitecture/Internal/Deprecations.swift

What other places would you like to see this change? forEach maybe?

Definitely on forEach! Ideally any higher-order reducer that can assert.

Thanks for the feedback.

Made the optional property deprecated:

Updated examples to use the new optional function.
Added file/line to all forEach variations.
I haven't found other high order reducers that can fail, not on the lib or in any example.
Since Xcode doesn't seem to show up the file/line anywhere https://github.com/pointfreeco/swift-composable-architecture/pull/218#discussion_r451712447 should we add the info in the message itself?
Should I mark the deprecation as unavailable as per #219 or is gonna be to early for that?
swift-format run locally ^^

Should I mark the deprecation as unavailable as per #219 or is gonna be to early for that?

We can wait a few versions to change this "deprecated" to "unavailable" so that folks have a chance to more slowly migrate to the new APIs rather than force them to all at once.

All good then. Ping me if you need any further changes 👍


Error running googleapi on flutter

I run the example in flutter and i get an error:
import 'package:googleapis/storage/v1.dart';
import 'package:googleapis_auth/auth_io.dart';

final _credentials = new ServiceAccountCredentials.fromJson(r'''
{
  "private_key_id": ...,
  "private_key": ...,
  "client_email": ...,
  "client_id": ...,
  "type": "service_account"
}
''');

const _SCOPES = const [StorageApi.DevstorageReadOnlyScope];

void main() {
  clientViaServiceAccount(_credentials, _SCOPES).then((http_client) {
    var storage = new StorageApi(http_client);
    storage.buckets.list('dart-on-cloud').then((buckets) {
      print("Received ${buckets.items.length} bucket names:");
      for (var file in buckets.items) {
        print(file.name);
      }
    });
  });
}

E/flutter (  752): [ERROR:topaz/lib/tonic/logging/dart_error.cc(16)] Unhandled exception:
E/flutter (  752): type '_InternalLinkedHashMap' is not a subtype of type 'Map<String, List>' where
E/flutter (  752):   _InternalLinkedHashMap is from dart:collection
E/flutter (  752):   Map is from dart:core
E/flutter (  752):   String is from dart:core
E/flutter (  752):   List is from dart:core
E/flutter (  752):   String is from dart:core
This is my pubspec.yaml:
name: consumos_app
description: A new Flutter application.

dependencies:
  flutter:
    sdk: flutter
  cupertino_icons: ^0.1.0
  path_provider: ^0.3.1
  json_annotation: ^0.2.2
  shared_preferences: "^0.3.3"
  intl: "^0.15.2"
  googleapis: "^0.50.0"
  googleapis_auth: "^0.2.5"

dev_dependencies:
  flutter_test:
    sdk: flutter
  build_runner: ^0.7.6
  json_serializable: ^0.3.2

flutter:
  uses-material-design: true

@jakobr-google Would you mind taking a look?
There might be some cases where the JSON decoded data, inside a Map<String, dynamic> gets assigned to typed Map<String, List>. Would you mind taking a look?

@jakobr-google This is becoming a blocker to internal google3 teams as well.


CLI: Adding shebang to setup_env.sh script

Adding shebang to local-cli's setup_env script.
Depending how your *nix environment is setup, running react-native bundle will throw this error:
Error: spawnSync /opt/build/in/node_modules/react-native/local-cli/setup_env.sh Unknown system error -8
    at exports._errnoException (util.js:1022:11)
    at spawnSync (child_process.js:461:20)
    at Object.execFileSync (child_process.js:498:13)
    at repl:1:14
    at sigintHandlersWrap (vm.js:22:35)
    at sigintHandlersWrap (vm.js:96:12)
    at ContextifyScript.Script.runInThisContext (vm.js:21:12)
    at REPLServer.defaultEval (repl.js:346:29)
    at bound (domain.js:280:14)
    at REPLServer.runBound [as eval] (domain.js:293:12)

In my case I was running react-native on docker (using alpine-node:6 as base image).

In my case it wasn't the shebang.
It appears there were some stray /r or ^M characters.
I had to run:
dos2unix -k -o ./node_modules/react-native/local-cli/setup_env.sh
Runs without a shebang.

I get the same error with docker.
The same error for any base images of node:alpine-6, node:alpine-7.

Same error, using docker with alpine:3.5. This PR fixed the issue for me.


Contrast too low for headings on the accesskit.dev website

I'll keep it short: many pages on https://accesskit.dev/ use dark grey over black (#333333 over #000000) for their level 1 heading. I'm sure this slipped through because maintaining the website is very low priority, but having contrast issues on the home page for an accessibility toolkit gives a bad first impression.
I'm not sure where would have been the most practical place to report this. I'll ping someone on Fostodon if this doesn't get noticed.
PS — The site's logo is also crushed vertically when the site is viewed on a wide screen.

Thanks for the report. I want to completely redo the AccessKit.dev site, replacing the current WordPress site with something lighter, probably generated with a static site generator. I don't like the current site myself.

Thank you @DominoPivot for this bug report.
We have just launched a new version of the accesskit.dev website. The new design should be accessible to a wider audience.
If you encounter further issues, please report them on the website's repository.


[Bug]-[736]:Date picker Widget drop down for the year takes away the whole screen.

Is there an existing issue for this?

[X] I have searched the existing issues

Description
As you can in the below app, the year option takes the whole screen.

Steps To Reproduce

Create an App and add a date picker widget
Deploy the app and in view mode click on year option

Public Sample App
No response
Version
Cloud

Stats

Stat
Values

Reach
736

Effort (months)
0.5

Only show some years (Current year+5 and current year -5) and show a scroll for the rest of the years.

@techbhavin can you check this out?

@dilippitchika
this is iOS-specific issue DatePicker BluePrint Libary.
At the moment we are not able to fixed this issue.

@techbhavin how is this specific to ios this is happening because of there is no library being used in the dropdown for year right?

@dilippitchika  As per our discussion about a specific issue in macOS. Here I have attached a screenshot of the windows system.

Thanks @techbhavin, currently no other option exists other than replacing the library. @AnaghHegde FYI, we can't fix this right now. Keeping this in icebox and updating the score.


iox-#2041 Rename 'CTorAndAssignmentOperatorTestClass' to 'LifetimeAndAssignmentTracker'

Pre-Review Checklist for the PR Author

[x] Code follows the coding style of CONTRIBUTING.md
[x] Tests follow the best practice for testing
[x] Changelog updated in the unreleased section including API breaking changes
[x] Branch follows the naming format (iox-123-this-is-a-branch)
[x] Commits messages are according to this guideline

[x] Commit messages have the issue ID (iox-#123 commit text)
[x] Commit author matches Eclipse Contributor Agreement (and ECA is signed)

[x] Update the PR title

Follow the same conventions as for commit messages
Link to the relevant issue

[x] Relevant issues are linked
[x] Add sensible notes for the reviewer
[x] All checks have passed (except task-list-completed)
[x] All touched (C/C++) source code files from iceoryx_hoofs are added to ./clang-tidy-diff-scans.txt
[x] Assign PR to reviewer

Notes for Reviewer

On popular demand  'CTorAndAssignmentOperatorTestClass' is renamed to 'LifetimeAndAssignmentTracker' :)
Checklist for the PR Reviewer

[x] Commits are properly organized and messages are according to the guideline
[x] Code according to our coding style and naming conventions
[x] Unit tests have been written for new behavior
[x] Public API changes are documented via doxygen
[x] Copyright owner are updated in the changed files
[x] All touched (C/C++) source code files from iceoryx_hoofs have been added to ./clang-tidy-diff-scans.txt
[x] PR title describes the changes

Post-review Checklist for the PR Author

[x] All open points are addressed and tracked via issues

References

Closes #2041

I'll merge this to the base branch. It is not that big and the approvals would be dismissed once I merge the base branch sot master. You'll have to re-approve on the base branch


DD2 Crashes on launch with OpenXR or OpenVR

Describe the bug
Tried running several versions of the mod for DD2: 1.5.7, 1.5.6, 1.5.5, 1.5.4 and all crash the game if either OpenVR or OpenXR dlls are present. Running the 1.5.7 DD2 release on Kunitsu-Gami works fine.
Upload logs and any crash dumps
Here are logs from a recent build of Master.
re2_framework_log.txt
reframework_crash.dmp
Identify the REFramework version
d4777c5e6fdcc35066dccf192c1a37f79aa26428
To Reproduce
Steps to reproduce the behavior:

Install DD2
Install REFramework
Launch DD2
Game crashes

Expected behavior
DD2 boots and initializes VR
Desktop and VR (please complete the following information):

OS: Windows 11
GPU: RTX2080ti
CPU: i7-8086k

Virtual Reality (please complete the following information):

Headset model: VRto3D (SteamVR)
Runtime: OpenXR

Additional context
Don't know if there's some compatibility issue with SteamVR + REFramework + DD2, but REFramework works fine with all other games and SteamVR. VRto3D can be found here if testing with SteamVR is necessary to reproduce the crashing.

Seems to be fixed as of f2254d2
Thanks @praydog !


identify logic adapter using prefix

I think most of the chatbots are doing identifying their logic adapter using specific keywords
For exmaple
CHATTERBOT = {
    'name': 'test',
    'logic_adapters': [
        'chatterbot.adapters.logic.TimeLogicAdapter',
        'chatterbot.adapters.logic.MathematicalEvaluation',
        'chatterbot_jira.JiraLogicAdapter'
    ]
}

I can call logic adapter from chatterbot is something like below
/time what is current time
/time how many seconds in a minute
/math 2+2
/jira search xx-1234

The workaround i am doing
if re.search(r'^/jira', user_input):

I like what this idea is trying to accomplish, unfortunately I think it might be going slightly against what ChatterBot aims to accomplish. This seems more like a command line application than an interactive conversation.
The goal for ChatterBot is to understand the user's input and then respond appropriately. If there is any opportunities to improve it's ability to answer questions and respond to input I will be happy to execute the necessary changes. However, I don't think specifying the logic adapter can be supported in the project's core.

Agree. Why cant your logic adapter handle this? For example
In your chatterbot_jira.JiraLogicAdapter
def can_process (self, statement):
    return re.match(r'^/jira', statement)


Change image background to transparent

I think the images in the sprites are always ordered as a nx10 matrix right?
If the sprite cannot be filled with images, then black images will be added as placeholder right?
E.g.

I think this looks a bit misleading because the user may not know if the black placeholder images are "real" images at first glance. I would suggest we make these images transparent so they don't show up at all.

The best solution would be to make the background transparent and implement it here: https://github.com/Caleydo/malevo/blob/3d1186d0dae888ac1d0221b80dadf03ba0a9f0ae/malevo/api.py#L116

Done.


OBJ loader, use material group if provided?

I was attempting to make the OBJ loader use my custom material group when it loaded the model from a file, but it wouldn't.  I had to change the obj.py code (attached).  Is there a better way to have the OBJ loader use my shader?
define our custom coordinate frames we want to apply when loading models
bodyCF = groups._dmb_cf
antCF = groups._antenna_cf
use our custom OBJ decoder that will use our materialGroups
objDecode = obj.get_decoders()[0]
load the models
model_obj = pyglet.model.load("./models/Body.obj", decoder=objDecode, batch=batchBody, group=bodyCF)
radome_obj = pyglet.model.load("./models/Radome.obj", decoder=objDecode, batch=batchRadome, group=bodyCF)
gimbal_obj = pyglet.model.load("./models/GimbalMount.obj", decoder=objDecode, batch=batchBody, group=bodyCF)
antenna_obj = pyglet.model.load("./models/Antenna.obj", decoder=objDecode, batch=batchBody, group=antCF)
antcone_obj = pyglet.model.load("./models/cone.obj", decoder=objDecode, batch=batchAntCone, group=antCF)
groups are defined in attached
groups.zip
groups.zip file:
obj.zip

Hi, thanks for opening the ticket. The model module is one are that needs work for 2.0+.
Your custom solution is fine, as long as it's working for your current needs. (I see you're ignoring the texture that's defined in the Material, but I suppose that's intentional in your case?)
One thing we want to do is finish the GLTF decoder, and see what extra requirements or issues come up. It's probably not going to fit the way that the OBJ decoder is currently working. One thing we may want to do is pull the Batch and Group logic out of the decoders completely, so that they can be used standalone. They can instead just return the raw vertex, material, etc. data. Nobody is currently working on this, but it's planned for the future.


Docker hangs when converting csv

Problem description
When IEEE123 is run using docker, the process hangs when using the csv converter #input "ica_config.csv" -f config -t config. This problem only occurs when running the model using docker.
Steps to reproduce

Go to branch use_case/integration_capacity_test
Make sure that ieee123/ica_ieee123.glm has line #input "ica_config.csv" -f config -t config uncommented, and line #include "ica_config.glm" commented.
From repo gridlabd-models run: docker run -itv "$(pwd)":/test slacgismo/gridlabd:latest
cd into ICA_models
Run: python3 ica_main.py

Expected behavior
ica_config.csv will not be converted, and process will hang.
System information

GridLAB-D version: HiPAS GridLAB-D 4.2.12-201009 (master) Linux
Operating system: ( x86_64 GNU/Linux

Does this happen with beau harbors-13?

Not sure I understand. What is beauharnois-13?

The version info is patch 12 to version 4.2 (which is named beauharnois).  Have you pulled the latest docker image? Or is this the latest image you're getting when you pull the latest image?

Also, the branch use_case/integration_capacity_test is not available on Github.  Have you pushed it yet?

Yes, I've pulled the latest docker image. I am using tag 'latest'. The image ID is 0a08f9ff2f7c. And yes, I've pushed! It's on gridlabd-models

Interesting. That's the right id, but when I run
docker run -it slacgismo/gridlabd:latest gridlabd --version=all I get the following output for the version info:
HiPAS GridLAB-D 4.2.13-201027 (master) Linux

which is not what you reported above.

One thing I see is that your print statements are being buffered and are not being displayed right away.  You can cause them to be output right away using print(...,flush=True).

I think it's waiting on keyboard input. It starts when I hit Ctrl-D whenever it pauses.  Somewhere in your code is something that's reading /dev/stdin. A quick fix is to use the command python3 ica_main.py < /dev/stdin.  But ultimately, you're going to have to find what's reading the input and remove it.

BTW, did you intend to run a realtime simulation?

That quick fix worked for me! I intended to run the ICA analysis on IEEE123.

IEEE-123 runs a realtime simulation when it is not configured to use a start and stop date. Take a look at the config you commented out and consider adding the items you find listed there in your new config.

I tested it on both MAC and Docker and the issue only arises on Docker.

Here's the barebones test file. Note, only apparent on Docker.
Archive.zip

@aivanova5, I am unable to reproduce the after install<EMAIL_ADDRESS> Can you confirm that this problem is fixed when upgrading?  If so, we could simply change the required version in configure.ac to ensure that 3.9 is used.


Handle Android RN 0.47 breaking change

https://github.com/facebook/react-native/commit/ce6fb337a146e6f261f2afb564aa19363774a7a8
Alternate proposal to #181. The advantage of this PR is that it's not a breaking change. Users can continue to update react-native-linear-gradient without being forced to update RN to >= 0.47.
There is of course a down-side to this approach, createJSModules() is effectively dead code in RN >= 0.47 and at some stage in the future someone has to remember to remove it. However, this is the approach I've taken with my own RN NPM packages.

I agree, this is a better solution

This comment is just a nudge to try and get this merged in and published on NPM sooner than later.
This PR doesn't seem very controversial. Anything holding this up?

cc @GantMan
Could you have a look at this PR please? This issue is common through all the NPM packages now since RN 0.47 release, and is very important to avoid forking. Thanks!

Looks good to me!  Also looks like @dgladkov is on it!

Yes, it's already merged and version is bumped. Thanks!

Thanks! Released as 2.2.0


⚠ ClamAV has degraded performance

In b07593c, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 66 ms

Resolved: ClamAV performance has improved in 855581d.


Video Player

Video Player
New script that plays the video on a path with a custom video player.
Instructions

Create a new folder for your script and file/folder name should be appropriate.
Create a README.md in your folder for program Instructions
add requirements.txt if needed

Please add/delete options that are not relevant.

[x] Adding New Code
[ ] Improving Code
[ ] Improving Documentation
[ ] Bug Fix

Programming Language

[x] Python

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server  - https://discord.gg/FXyh2S3
Happy Coding,

I have the code ready for this. Please assign me.

I have created a pull request.


Ease Business Glossary Accessibility and Enhance Searchability

Is your feature request related to a problem? Please describe.
Feature Request: Promote Business Glossary Terms to Exploration search and Enable Search Filters.
Currently Business Glossary is not part of Explore Menu as it only focuses on Tables, Pipelines and so on.  Current search from Business Glossary is hidden away and limited. However, to promote the use and value of Business Glossary, I believe it should be a part of exploration tab. Also, the search needs to be enhanced to help navigation.
Describe the solution you'd like

Business Glossary searchable from Explore tab
Business Glossary item count visible next to tables count
Also supports search filters:  able to filer Business Glossary search by "where term's Glossary is XYZ; where term is with a tag XYZ; where term parent term is XYZ;"

Add Glossary & Glossary Term count here. @pmbrull has provided API support here


Run SonarCloud only on pull requests to master

Run SonarCloud less often to reduce build times for normal CI builds.

 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.:x: lferreira-sw sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


docs: remove unnecessary use of sync_to_thread=True.

Pull Request Checklist

[ ] New code has 100% test coverage
[ ] (If applicable) The prose documentation has been updated to reflect the changes introduced by this PR
[ ] (If applicable) The reference documentation has been updated to reflect the changes introduced by this PR
[ ] Pre-Commit Checks were ran and passed
[ ] Tests were ran and passed

Description

This PR removes any use of sync_to_thread=True from our sqlalchemy examples when there is no blocking IO in the dependency/handler.
Close Issue(s)

Closes #2508

Maybe we can just remove the param as False is the default value.

I think without it it prints the warning message? I am not up to date on this, though.. would have to look

Maybe we can just remove the param as False is the default value.

@JacobCoffee is correct - when a handler is sync, i.e., def ... not async def ... we warn if the parameter is not explicitly set.


Scrollbar Width not considered

The width of the scrollbar is not considered. If no margin on the left would be specified, the content of whatever has to be scrolled would be partly hidden behind the scrollbar. It would be nice if I wouldnt have to manually calculate the width of the scrollbar. This is also an issue when overflow is set to auto. Then the margin that was  calculated for the scrollbar to not hide the content behind it would be too large because the scrollbar is not there

If I'm understanding correctly, what you're saying is that you'd like it if the dimensions of a scrollbar were not taken into account when calculating the dimensions of a box. This is not how browsers natively handle scrollbars, and I think for good reason: one expects two elements with the same width, height, and padding to have the same dimensions, regardless of whether they have scrollbars. If the scrollbar were not taken into account, you could end up with it spilling outside of a container, and you'd be right back to the same problem, having to calculate how much width to take off of a box with a scrollbar. I think the sensible thing for the plugin to do is follow the lead of browsers, and let users deal with overlaps in specific cases on their end.
If I've misunderstood what you're saying, I'd be happy to look into it more, though it might be helpful to have a concrete example of what you're talking about (and, if possible, a way you'd suggest solving it through vanilla CSS).

I appreciate the fast response! A native scrollbar is not part of the margin of the scrolled container, this scrollbar however, takes that space of the margin and if the margin is too small, content will be covered by the scrollbar. I have a working example but its quite large. I could share my screen if youd wish to see it

The deprecated property overflow: overlay seems to be the problem in Chrome.
From https://developer.mozilla.org/en-US/docs/Web/CSS/overflow,

overlay
Behaves the same as auto, but with the scrollbars drawn on top of content instead of taking up space. Only supported in WebKit-based (e.g., Safari) and Blink-based (e.g., Chrome or Opera) browsers.


receivers: add support for the Forward logging receiver

This change does the following:

Adds support for the fluent_forward logging receiver allowing
Fluentd and Fluent Bit to be used as inputs to the Ops Agent
Adds configgenerator unit tests

Note: I'm OOO from Feb 17 - 22. If the pending comment thread are addressed and approved by other reviewer(s) before I get back, feel free to merge.

Added a commit that ensures fluent_forward behaves sanely when used with receiver_ids and pipeline_ids that have "." in them. In the above image, this was run with a pipleine_id and receiver_id with "."s and things seem to work fine. The commit message gives some more info:
The receiver_id and pipeline_id currently allows "." to be used. This
creates a problem as thats what the Lua filter must rely on as a
delimeter when adding the old tag to the logname and removing the
pipeline_id from it. Additionally, allowing "." in the receiver_id
creates issues for when a receiver_id is a prefix of another one.
(its a very specific edge case where another receiver_id would have
another receiver_id followed by a ".")
Consider the following example:
logging:
  receivers:
    fluent_logs:
      type: fluent_forward
      listen_host: <IP_ADDRESS>
      listen_port: 5678
    fluent_logs.collision:
      type: fluent_forward
      listen_host: <IP_ADDRESS>
      listen_port: 5679
  service:
    pipelines:
      fluent_pipeline:
        receivers: [fluent_logs, fluent_logs.collision]

Notice that the receiver_id fluent_logs.collision completely contains the other
receiver_id fluent_logs. And so when we try to match on records
ingested by fluent_logs, since it will be suffixed by the tag of the
forwarder, we will need to match on fluent_logs.*. However because of
this, we will match on records from fluent_logs.collision as well. Which is not
desired. To get around this issue, we prefix the tag with a unique hash
that will deterministically ID the pipeline + receiver combination.
This change does the following:

Add a hash string as the prefix to the tag to prevent collisions
Update lua script to only append existing tag to the LogName
Have the Lua filter update logname for fluent_forward by appending
the existing tag.
Add test cases for the collision scenario


Clean any pre-existing containers when rdt pushing

If a container exists before the push the process will fail with an error similar to this :
- Creating 'yolo' container
resin push failed. Error: (HTTP code 409) unexpected - Conflict. The name "/yolo" is already in use by container d5ad23b9edd57747a799fb790b82d322f984b60144f6d2027e20734f9980dd0d. You have to remove (or rename) that container to be able to reuse that name.

Closed by https://github.com/resin-io-modules/resin-sync/issues/60