Check if custom element is already defined before defining it

Added a conditional check using customElements.get() to verify if the custom element is already defined.
Prevents DOMException by ensuring elements are not redefined when using Pinecone router that fetch content from url
Ensured compatibility and stability by avoiding duplicate element definitions.

Thank you for accepting the pull request. I'm glad to contribute to the project. If there are any further improvements or additional tasks you'd like assistance with, please let me know.
Thank you :)


inkscape: fix on darwin (fixes #15768)

Motivation for this change
Currently, building inkscape on darwin results in an error:

error: invalid argument '-std=c++11' not allowed with 'C/ObjC'

This issue has been addressed upstream but the fix is not part of the current released version (0.91). Therefore this PR applies the relevant patch to the released sources.
Tested on OSX and on a virtualized NixOS: builds and runs fine in both cases.
Things done

[ ] Tested using sandboxing
(nix.useChroot on NixOS,
or option build-use-chroot in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[x] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

I'm building on linux and darwin now.

Reviewed 1 of 2 files at r1.
Review status: 1 of 2 files reviewed at latest revision, 1 unresolved discussion, some commit checks broke.

pkgs/applications/graphics/inkscape/default.nix, line 21 [r1] (raw file):

  sha256 = "1j45hc7s5n9m7pam638mwwrpdsfs87bxkaa2kkmki3jrhkgv05ca";
};

Can this patch be fetched with fetchpatch (https://github.com/NixOS/nixpkgs/blob/master/pkgs/build-support/fetchpatch/default.nix, https://github.com/NixOS/nixpkgs/blob/master/doc/coding-conventions.xml#L663-L679) and included in the standard patches list?

Comments from Reviewable

It builds on OSX,but told me it has nothing to do when I ran it. What about that patch question?

I will use fetchpatch instead of fetchurl.
However, I don’t know how to properly add this patch to the patches list: the default patchPhase calls patch with -p 1 as argument whereas we need -p 0.
I don’t understand your remark about “told me it has nothing to do”: who is “it”? Sorry.

Nothing to do!```

let me know when you've updated the fetchurl to fetchpatch and I'll go ahead and merge it.

---

Review status: 1 of 2 files reviewed at latest revision, 1 unresolved discussion, some commit checks broke.

---

*Comments from [Reviewable](https://reviewable.io:443/reviews/nixos/nixpkgs/17962#-:-KQ0uy_FATaAj8a3u1eo:b-uqv7uv)*
<!-- Sent from Reviewable.io -->

That’s weird: running inkscape should open a GTK window.
The fetchpatch issue has been addressed.

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

pkgs/applications/graphics/inkscape/default.nix, line 21 [r1] (raw file):
Previously, grahamc (Graham Christensen) wrote…
> Can this patch be fetched with `fetchpatch` (https://github.com/NixOS/nixpkgs/blob/master/pkgs/build-support/fetchpatch/default.nix, https://github.com/NixOS/nixpkgs/blob/master/doc/coding-conventions.xml#L663-L679) and included in the standard patches list?

OK

Comments from Reviewable


The missing core packages in rust(But have in golang)

Hello , I am using rust/golang for years.
I found the rust-lang hardly to producing more product.
Lack of stable/usable core packages(libraries)
Golang have better encoding/crypto/net packages.
https://golang.org/pkg/net/
https://golang.org/pkg/encoding/
https://golang.org/pkg/crypto/
I hope the rust lang provided us these packages by core team.
Do not let us find the solution(most not very stable, if core team can provided these libraries. it will gains better stable/performance)

Please be more specific/constructive about the actual areas you perceive as lacking, or people won't pay attention and you're just considered troll from the another camp.
P.S. I see you're overwhelmingly likely Chinese and may actually have conveyed your mood different than you intended. You could talk to me in Chinese and I'll try to paraphrase you.
（你可能没能用英语准确传达出你的想法，你可以用汉语表达你的意思，我来帮忙翻译。）

The modern language can do more . not the core language it self. but a powerful standard library.
Networking programming,  Code/Serialize/Unserialize, Crypto, was the very common code works.
Rust lang miss these parts.
We can not easily to using rust lang built our software.
Because it lacks these core libraries.
The third party libraries not very stable as you want . You may lead into buggly.

Hmm... I think everyone agrees Rust is not quite ready for primetime? See the roadmap and AWWY. Sure there is much work left to do, it's only 2017 and Rust is only over 2 years old.
As for the areas you reiterated, we have

tokio for (really) high-performance async networking and hyper for all your HTTP needs,
serde for similarly high-perf (de)serialization,
rust-crypto and ring for common crypto building blocks,

I've used all of them to build high-performance crawlers and backend components, and I encountered bugs very rarely, I haven't filed a single issue during my usage of them. Only they're currently kind of "incubating" thus not reaching the "stability" or "maturity" you hope for. This is actually recognized by core team members, and they recently launched the Libz Blitz (announcement, irlo thread) to address precisely your concerns. Several libraries are currently being worked on now, and it's gonna take time.
So, your concerns are being worked on, but it'll be a while before everyone could jump straight in. Just be considerate and patient, there's always This Week in Rust to track the contributions and roadmap status. (Best of all, start contributing yourself! But seeing you likely have most of your time devoted to work, this is only a suggestion.)

As for your latest edit:
Please kindly share your experience, no matter how frustrating. Is the syntax bothering you? Borrow checker misery? Unable to find a community crate for your need? We need to know more about people's acceptable learning curve. Also, your experience as a Chinese developer will help a lot towards Rust's diversity goals.
As the discussion is getting long here, you could write about it in your blog or any medium you prefer. Both Chinese and English are okay, in fact I have some difficulties understanding your English so you may well write Chinese instead and I'll understand.


Make namespaces isolation policy updates take effect on time

Motivation
The change of namespaces isolation policy takes effect only when load-manager re-assign the bundles to brokers again.
This change tries to make the isolation policy takes effect on time.
Modifications

change setNamespaceIsolationPolicy method into async.
add parameter to enable this feature: enableNamespaceIsolationUpdateOnTime.
add test to cover this feature.

Verifying this change
tests passed

/pulsarbot run-failure-checks


Polkit

Please, include instructions about how to intagrate Polkit. For example, launch a vm in Virt-Manager don't work.

Sorry, I'm a bit confused by this issue. I just looked up Polkit and it doesn't seem wayland-specific, or even wayland-related. And apart from that, even if it were wayland-specific, there's also the fact that this repo doesn't include instructions for anything - it's simply a place to link out to projects.
I would suggest you repost your question/issue in the Polkit repository itself!

Ok. I did a mess.


Generated paths with parameters do not keep in mind the 'style' and 'exploded' properties

Describe the bug
Generating a query path from an OAS3 file does not keep in mind the ability to explode an object into a different query format as is described in the Open Api Specification 3
To Reproduce

Given this OpenAPI document

openapi: "3.0.2"
info:
  title: Test
  version: "1.0.0"
servers:
  - url: https://api.server.test/v1
paths:
  /recipes:
    get:
      parameters:
        - $ref: "#/components/parameters/Pagination"
      responses:
        "200":
          description: OK
components:
  parameters:
    Pagination:
      name: pagination
      in: query
      required: true
      style: form
      explode: true
      schema:
        type: object
        properties:
          offset:
            type: integer
            example: 0
          limit:
            type: integer
            example: 25

Run this CLI command docker run -p 4010:4010  --rm stoplight/prism:4 mock -h <IP_ADDRESS> https://gist.githubusercontent.com/tommie822/a2f055edac4928743108a3d488fc7761/raw/2db47051ea21b6fbcc4dd89dfb8e74b24cb04791/oas3.yml
See error
The path created directly uses the parameter object instead of using the properties 'style and explode' to determine the path. See screenshot.

[11:30:58 AM] › [CLI] ℹ  info      GET        http://<IP_ADDRESS>:4010/recipes?pagination=[object Object]

Expected behavior
According to specification a parameter object such as
    Pagination:
      name: pagination
      in: query
      required: true
      style: form
      explode: true
      schema:
        type: object
        properties:
          offset:
            type: integer
            example: 0
          limit:
            type: integer
            example: 25

should result in http://<IP_ADDRESS>:4010/recipes?offset=0&limit=25 see screenshot
The code should probably be placed here
Screenshots

Hey team! Please add your planning poker estimate with Zenhub @chohmann @EdVinyard @daniel-white @brendarearden


Replace Synctrayzor with Syncthing Tray

The toolbox includes Synctrayzor, but that appears to be abandonware (last commit 3+ years ago). I've used this before and it works fine, but it will probably break with a syncthing update in the future.
Syncthing Tray is a alternative that, afaik, supports all the features Synctrayzor does and is also actively developed on. I'm currently using this and had no issues whatsoever. Might be a better alternative to future proof?

#1495 should be good to close this issue.

Please close this issue as the PR is merged.


Basic unit tests for chart re-rendering

This change introduces couple simple unit tests for ChartComponent#shouldComponentUpdate() function.
Tests written using using Jest unit testing framework. They mostly cover issue fixed in gor181/react-chartjs-2#17.

Hello @gor181, this is a simple set of unit tests that we've discussed in #17.

Hey @lutovich ,
Thanks for the effort, this runs and works really nice.
I really didn't had time to dig into tests for this lib myself, so I really appreciate your engagement.
Thanks,
Goran


If filename has not been created before we have to pass force=true

The rm(filename) may be a bit dangerous anyways for a generic default filename="code.jl"

Agreed, wrt the filename being dangerous, this was not intended to be public but still.
For safety, Could you improve and/or add a random hash to the default filename to ensure that this never becomes a problem?

I think a more specific filename will do. With the random hash, we may start to accumulate files

Alright looks good, good to merge


Further options for identifying your GP

BTW this is how it looks:

:+1: for including a screenshot


Feature/connect UI frontend

will fix android in follow up PR

This is missing a full android implementation and contains left over logs. Both will be fixed in a follow up PR.


About having home screen in app

We should have a home screen in application, currently Tracks list is the home screen but it should not be actually as it's not our prime screen of the app.
Point of having home/prime screen is to display important details like:

About Event
Event Date
Venue
And other important details.

@PareshMayani The home screen should be similar to the web app. Also the other parts of the Android app should have the same structure as the webapp, but we also have other screens like sponsors. Please open relevant issues.

@mariobehling yes I am referencing web app only but making application same as web would not be possible. And also we should not make it exactly same as web! Because all the platforms (android, iOS, web) are having their own standards and we should follow those.
When I say I am referencing web app, I did checked http://fossasia.github.io/open-event-webapp/speakers.html and then suggested speakers screen enhancement.
I really didn't get about opening relevant issues!

Closing due to inactivity and app design has progressed in meantime. Open new issue with details if needed.


Layout of window buttons on the custom title bar looks bad

Steps to reproduce

Open VSCode 1.71.0 on Windows 11
Maximize the window

Merging to https://github.com/microsoft/vscode/issues/157086


[chip dv] GPIO test using DIFs

This PR adds the chip level GPIO test using DIFs. See SW test description for more details. Since it needs some testbench side stimulus and checks, the test is placed in sim_dv directory.

Adding @alphan who wrote the original DIF.


Add DRS interface

The internal data structure should allow easy exposition of datasets using GA4GH compatible interface: https://www.ga4gh.org/news/drs-api-enabling-cloud-based-data-access-and-retrieval/
This should be implemented, probably as a separate service?

Wrong repo. This should if any be stated in the server repo not in the documentation section.

Because this came back up recently, i guess we have all the building blocks already together, we just need to implement a small wrapper that implements DRS <-> Aruna at least for public repos


Error loading extension on Godot 4.1

System information: Windows 10 , maxOS 13.4
Godot version: 4.1 beta 2 and up.
Plugin version:  170e_1
Error loading extension on Godot 4.1 (effekseer.gdextension) when open project.

This plugin does not yet support 4.1
Please wait a bit as it is currently being supported.
Thanks!

Effekseer plugin for Godot 4.1 has been released.
https://github.com/effekseer/EffekseerForGodot4/releases/tag/1.70e.2

Well done. I have tested and have no issue. Thanks!

Crashes every time on windows after shutdown of Godot Editor for version 4.1.1 and 4.2dev2.
I added and activated the addon to an empty project. I did not use the addon anywhere in my project.
You can see the effect if you have the debugger attached (or auto attached).


ne-en pretrained model cannot reproduce results

The Ne-En pretrained model in the model zoo is either mixed with another language pair or is uploaded without training. When I run the eval script with the gdrive file I get a correlation around 0.2

I will check it again. Is there any chance that you can run this file - https://github.com/TharinduDR/TransQuest/blob/master/examples/wmt_2020/ne_en/trans_quest.py so that you can train Ne-En model from scratch and let me know the results?

Hi I haven't tried this but trained a new XLM-R model and the results are much close to that of the paper. So I think it is probably a simple mixup

With the new release we have fixed this. You can find the new models here - https://tharindudr.github.io/TransQuest/models/sentence_level_pretrained/


Print image with React Native

How to print image base 64 width React Native

@hoanganhnh2009 hello! If you need to print using Epson printer you can try this lib - https://www.npmjs.com/package/react-native-esc-pos-printer

Thankyou but i used some printer: Xprinter, Gprinter, Antech Printer, etc...


[Bug]: Can't exit from a raid

Discord Username
Wagor#7007
What happened?
Whenever im dying i keep being in the raid i was - i still can hear ambient sounds, therefore the raid results message doesn't appears and i have to shut the game, whereas when i'm exiting alive everything is fine
SPT Version
3.6.1
Server Log
Made a winrar with all the logs i've found. I was a scav and died ~7mins after the beginning: https://drive.google.com/file/d/1urDdVJjtBU0QCs701anYtBOOl_LmUxgY/view?usp=drive_link

I requested access to the file.

@WagorWT Alternatively you can paste the user/logs/ logfile in a pastebin and then drop a link to it in here.

sorry for being so late, i thought it was available for everyone who have the link. Just have given you access

@WagorWT I received the file above but they appear to be the wrong log files. The log files I'm looking for are the server log files found in the /user/logs folder. The log files that I did skim through, I noticed that you're having a client error that has something to do with an airdrop, which this mod doesn't interact with.
Are you still experiencing this issue?
Have you tried to removed just this mod and see if the issue has been resolved?

Hello. i'm no longer use this mod(at least yet) as i found another that did what i wanted(spawning bots instantly after starting), nevermind. And yes, airdrops always were almost empty. I guess LootingBots might be the reason, anyway thanks for help


Markdown: show "open the external website" alert only when URL is actually clicked not on hover

Type: Feature Request
If the mouse pointer is (accidently) hovering over a URL in a Markdown document, pressing CTRL (Windows) to perform another command where the actual input cursor is positioned, always triggers the "Do you want Code to open the external website" depite neither the URL under the mouse pointer itself was clicked nor was the "Follow Link" popup.

In this scenario the text "Kirby CMS Panel Vue GUI" was selected to make it bold using the Ctrl+B shortcut. The mouse pointer was hovering over a URL a few lines further down.
It turned blue the second I press CTRL and the alert box prevents me from performing my intended task, to make bold text.
This happens constantly as I shove the mouse away and it ends up hovering a URL somewhere else in the document.
The "Do you want Code to open the external website" confirmation alert should only appear if a URL was actually "Ctrl-Clicked" similar to the "Follow Link" tooltip and NOT if only the CTRL key is pressed in order to perform a command unrelated to the URL under the mouse pointer.
If one has a document wil loads of URLs this becomes increasingly annoying because there's a very good chance the mouse pointer will be hovering over any URL at any give time.
VS Code version: Code 1.88.1 (e170252f762678dec6ca2cc69aba1570769a5d39, 2024-04-10T17:41:02.734Z)
OS version: Windows_NT x64 10.0.19045
Modes:

Does this reproduce in the latest VS Code insiders build with all extensions disabled?

Doesn't reproduce for me

Does this reproduce in the latest VS Code insiders build with all extensions disabled?

extensions... right... I totally forgot.
Sorry!
It seems like the culprit is indeed an extension: disabling all "fixes" the issue.
I'll do a bisect to find out which one's causing this and report in their repo if needed.
Thank you for your time!


Test Result Reporter

Repository URL
https://github.com/akhmadnurmuhammad/qa-matrix-jenkins-plugin
New Repository Name
qa-matrix-plugin
Description
this plugin allow to automated post test result to our service, this plugin help QA engineer job to make easier and more clean
GitHub users to have commit permission
basil
Jenkins project users to have release permission
basil
Issue tracker
Github Tracker

/hosting re-check

/hosting re-check

/hosting re-check

/hosting host

/hosting host

Nice try, it needs one of the hosting team to do that

Initial review without looking at the code much:

You're missing a .gitignore file, our template is at https://github.com/jenkinsci/archetypes/blob/master/common-files/gitignore
Delete the target and work directories, you probably want to remove your git history so the binaries are not stored in history
Use your artifact id for the package name: https://github.com/akhmadnurmuhammad/test-result-reporter-jenkins-plugin/tree/develop/src/main/java/io/jenkins/plugins/sample not sample
Name your class something meaningful not helloworld

What does this plugin actually do? the name is very generic?

not sample

There's also still https://github.com/akhmadnurmuhammad/test-result-reporter-jenkins-plugin/blob/a412ce1b124da64f0e746be1e9bdeec9cde5cd64/src/main/java/io/jenkins/plugins/sample/HelloWorldBuilder.java#L99-L117 from the sample plugin which will be reflected in serialized configuration and behavior, as well as https://github.com/akhmadnurmuhammad/test-result-reporter-jenkins-plugin/blob/a412ce1b124da64f0e746be1e9bdeec9cde5cd64/src/main/java/io/jenkins/plugins/sample/HelloWorldBuilder.java#L187-L197

the name is very generic

I suggest the name be changed to include the name of the service it integrates with.

The plugin requires Java 11 to build but still claims 8 at https://github.com/akhmadnurmuhammad/test-result-reporter-jenkins-plugin/blob/a412ce1b124da64f0e746be1e9bdeec9cde5cd64/pom.xml#L18

Hi all, thanks for correction but i cannot build at java, the build is successfully by my plugins missing in build step, after i saw the error message, the message return like this, can you help me?
Caused by: java.lang.AssertionError: class io.jenkins.plugins.QaMatrixBuilder is missing its descriptor
at jenkins.model.Jenkins.getDescriptorOrDie(Jenkins.java:1599)
at hudson.tasks.Builder.getDescriptor(Builder.java:67)

/hosting re-check

/hosting re-check

/hosting re-check

The bot also runs on edits which is why it has been double running.
Maybe this needs bolding

You can re-trigger a check by editing your hosting request or by commenting

sorry my bad

Plugin still has an incorrect version requirement in the pom. Did you ever actually fully build this?

Plugin pom should be the latest version, the minimum requirement of 4.0 just means it must be 4.x
Lots has changed since then.
Where did this come from?
https://github.com/akhmadnurmuhammad/qa-matrix-jenkins-plugin/blob/a284d2b3791283a3cccdd3a31badd4a5d08765e5/pom.xml#L35-L42
It seems like you've built it from an old version of the archetypes, you should have a URL set there and it should point to GitHub
Your code should be in a package named after your artifact id, i.e. not just in io.jenkins.plugins
https://github.com/akhmadnurmuhammad/qa-matrix-jenkins-plugin/tree/master/src/main/java/io/jenkins/plugins
Parameters and fields should follow Java naming, reportName rather than ReportName.
There should be a cleaner way of doing this than building it up yourself:
https://github.com/akhmadnurmuhammad/qa-matrix-jenkins-plugin/blob/master/src/main/java/io/jenkins/plugins/QaMatrixBuilder.java#L149-L161
There's no tests included, you might want to take a look at an example Builder test we provide:
https://github.com/jenkinsci/archetypes/blob/master/hello-world/src/main/resources/archetype-resources/src/test/java/HelloWorldBuilderTest.java

@basil is this supposed to have you as the committer and release permissions?

@basil is this supposed to have you as the committer and release permissions?

No. Perhaps a copy-paste error?

@akhmadnurmuhammad Is there still an interest from you to host this plugin? Asking as it's since Jan we had not heard from you.
I did a superficial security audit, and my gut feeling is that the QaMatrixBuilder.java#L42 should be a secret, but not sure as the code does not describe what that Token is supposed to be.
/audit-findings
If that token is not expected to related to any authentication mechanism, please tell us (and improve the comments/documentation around it to ease future audit)

Hey @akhmadnurmuhammad,
are you still interested in this hosting request? If so, please address the open concerns. If not, I'll go ahead and close this request, given there's been no activity in over 9 months.

I'll go ahead and close this request due to lack of response. If you are still interested in continuing with this request, please open a new issue.


[BUG] Theme conflicts between EquinoxApp and EqLayout

Hi,
when the EquinoxApps theme is EqThemes.defaultLightTheme but the theme of the EqLayout below in Widget tree is EqThemes.defaultDarkTheme the Screen is dark in first build, but after first reload / rebuild the screen gets light and is never going dark again.
The theme of EqLayout should override the ThemeData of EquinoxApp.

#12


Pool Name option missin in lxd 2.0.11

Hi everyone,
i would like to let you know if you don't already know that when i tried to make fresh install of lxd 2.0.11 in Ubuntu 16.04.3 i noticed that the step to create Pool Name in automatic mode with command "lxd init" its not present.
This missing step doesn't let to finish configuration ending with message below :
error: Failed to create the ZFS pool: cannot create ' ': name must begin with a letter pool name may have been omitted
I've managed to make it work with a previous version of lxd and than upgrading to 2.0.11.
Have a nice day and best of luck for the future projects.

What exact "lxd init" command did you run?

Just lxd init without any flag,  all the questions about storage type, storage device/size, bridge configuration came one after the other but the question for naming the pool didn't.

Thanks, reproduced this here, wondering how this was missed in testing... working on a fix now and will upload it to Ubuntu ASAP.

I've uploaded the fix to the Ubuntu archive and am now waiting for someone to review it.
This will be treated as an emergency update so as soon as it's reviewed, it'll get manually tested and released, waving the normal week of -proposed testing.

I've just tested the latest update and everything looks great, thank you.


Wrapped ${mailbox.name} in "s to allow for space in mailbox names.

Motivation for this change
Without this fix the following nix expression:
{
  name = "Trashy McTrashface";
  auto = "subscribe";
  specialUse = "Trash";
}

becomes
mailbox Trashy McTrashface {
  auto = subscribe
  special_use = \Trash
}

but it should be
mailbox "Trashy McTrashface" {
  auto = subscribe
  special_use = \Trash
}

or else dovecot2 won't start
Feb 05 16:00:28 nixos dovecot2-pre-start[20738]: doveconf: Fatal: Error in configuration file /etc/dovecot/dovecot.conf line 36: Expecting '{'
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Thanks. Also backported to stable as 5e35f55571f.

@dotlambda Absolutely. I blame me writing too many bash scripts lately.

Has a fix been submitted? Don't forget to also backport.

How about asserting mailbox.name doesn't contain "?

I think this should be the easiest way:
diff --git a/nixos/modules/services/mail/dovecot.nix b/nixos/modules/services/mail/dovecot.nix
index c5cb06cf54e..1408de4e162 100644
--- a/nixos/modules/services/mail/dovecot.nix
+++ b/nixos/modules/services/mail/dovecot.nix
@@ -113,7 +113,7 @@ let
   mailboxes = { lib, pkgs, ... }: {
     options = {
       name = mkOption {
-        type = types.str;
+        type = types.strMatching ''[^"]+'';
         example = "Spam";
         description = "The name of the mailbox.";
       };

What do you think?


Events: Detail view image sizing issues

ITS-Pro time category: https://its-pro.ucsd.edu/browse/MA-34
Issue Description

Event pictures are not always sized correctly (black bars) in detailed view.

Steps to Reproduce / Code Snippets

Load the Events Detail view

This is done when:

[ ] All event images are sized correctly on small and large form factor devices

Resolved in #914


Number of Epochs during Training

Hi, I wonder how many epochs do you train PSALM during the finetuning phase?
In your training script the num_train_epochs is set to 10, but in your paper you stated 56k with batch_size 64 are used during training. Setting num_train_epochs=10 results in the learning rate barely decreases and the loss is not going down for a long time. Should we decrease the num_train_epochs to a smaller number? For example, 3 or 4.
Thanks!

Hi @YunzeMan
we did 56K iterations totally during training. If you only training segmentation data, numbers for 10 epoch can be calculated as:
(~120,000 (COCO) + ~120,000 (RefCOCO) + ~120,000 (COCO-Interactive)) * 10 (epoch) / 64 (bs) = ~56K
And when adding QA dataset, we keep this number and choose an average of  these four tasks.
If you are confusing loss, you can send your loss curve for each loss and maybe I can help you find problem


Nothing to commit, error code 1

I'm trying out using flat to fetch a json file. This json file changes very infrequently so when testing the file is exactly the same. When I try to run the action the commit command gives an error.

This is my repo where I'm testing things out. If anyone has any pointers that'll be great

Hi @henryjameslau, sorry to hear you're having trouble! Your action looks fine to my eyes, and unfortunately I'm unable to reproduce the issue you're running into on my end 😢
Before digging any deeper, are you able to scaffold a new, blank GitHub repository and run the action again? I'd like to know if you're still seeing this error message! Thanks!

Also @henryjameslau would you mind enabling debug output on your repo so we can see more logs? You can do this by:

Going to the repo settings > Secrets (from the left side menu) > New repository secrets (Button on the top right)
Give it the name ACTIONS_RUNNER_DEBUG and value true
Save the secret
Then run the action again and let us know the output.

Ok I think it has to do with the .gitattributes file. I deleted it and it worked fine for me in a fork of the repo. Can you try deleting it and see if it works for you?

@irealva deleting the .gitattributes file worked. Thanks for looking into the issue.


Initializing a points layer with 3D empty data creates a 2D layer

🐛 Bug Report
See the title and the failing test.
💡 Steps to Reproduce
The following test fails.
def test_empty_3d_points():
    pts = Points(np.empty((0, 3)))
    assert pts.ndim == 3

💡 Expected Behavior
I would expect to get a 3D layer when passing a numpy array of shape (0, 3).
🌎 Environment
napari: 0.5.0a2.dev495+gd507dd64e
Platform: macOS-10.16-x86_64-i386-64bit
System: MacOS 14.2.1
Python: 3.11.5 (main, Sep 11 2023, 08:19:27) [Clang 14.0.6 ]
Qt: 6.6.1
PyQt6:
NumPy: 1.26.2
SciPy: 1.11.4
Dask: 2023.12.1
VisPy: 0.14.1
magicgui: 0.8.1
superqt: 0.6.1
in-n-out: 0.1.9
app-model: 0.2.4
npe2: 0.7.3
OpenGL:

GL version:  2.1 INTEL-22.1.29
MAX_TEXTURE_SIZE: 16384

Screens:

screen 1: resolution 3440x1440, scale 1.0
screen 2: resolution 1440x900, scale 2.0

Settings path:

/Users/asweet/Library/Application Support/napari/napari-dev_94a2c6ecd2bb4e54de7c17f279d1ba0ad46bb93d/settings.yaml
Plugins:
napari: 0.5.0a2.dev495+gd507dd64e (77 contributions)
napari-console: 0.0.9 (0 contributions)
napari-svg: 0.1.10 (2 contributions)

💡 Additional Context
Not dissimilar to #2273, but also not the same.

I think the bug is here:
https://github.com/napari/napari/blob/4a1b0b547f2dc6ae804f66af77a5d7154820463b/napari/layers/points/_points_utils.py#L236
It should be easy to solve. If someone do it fast, it may land in 0.4.19

I don't think this particularly critical and it's also easy to workaround (by specifying ndim=3), so need to hold up for v0.4.19. But if someone does fix it and it's easy to cherry-pick, that's also good.

Because in general, points data has shape (n, dim) where n is a number of points.

Hi, just in case, maybe adding a validation around https://github.com/napari/napari/blob/4a1b0b547f2dc6ae804f66af77a5d7154820463b/napari/layers/points/points.py#L381-L384 before fix_data_points is called to initialize ndim, so something like:
        if ndim is None and data is not None:
            if len(data.shape) == 2:
                ndim = data.shape[1]

Also, is there any other shape that the data could have when provided that would also need for ndim to be initialized over the layer constructor when the kwarg ndim is not passed?

btw thanks for raising this @andy-sweet, I ran into it the other day and was super frustrated, but I was doing something else and forgot to raise an issue. It's particularly funny because this was once the canonical way to get an empty 3D points later, until we added the ndim kwarg "to make it more ergonomic". 🤦♂


THREEjs post processing

Explore:

SSAO
Bloom
FSAA
Tone and Gamma

The facilities to do post processing in THREEjs are really nice. It exists lot of ready-to-use shaders or effects. I have now to look at the pipeline step by step.

Regular
Post processing (Bloom, SAO, FXAA)

Thanks to the gamma + FXAA passes, I can now achieve a quite good result for a "regular" rendering (i.e. without bloom and gtao) approaching the Webots rendering. The light model is still slightly different, but nothing slim in my point of view. The cost to improve this regular rendering will explode from now, because it would imply to rewrite the shaders and multipass system.
About GTAO and Bloom, the results are less good. I tried several existing shaders and passes, but no one is doing exactly what we would like. I started to migrate our implementation of the bloom.

test me

Build Passed :ok_hand:
You can safely merge 'feature-threejs-post-processing' into 'develop'.

test me

Build Passed :ok_hand:
You can safely merge 'feature-threejs-post-processing' into 'develop'.

test me

Build Failed :confused:
https://cyberbotics2.cyberbotics.com:8443/job/Webots/577/consoleText

test me

Build Failed :confused:
https://cyberbotics2.cyberbotics.com:8443/job/Webots/578/consoleText

test me

Build Passed :ok_hand:
You can safely merge 'feature-threejs-post-processing' into 'develop'.

Too wide objectives.


Fix tables and formattings in pathcch* APIs

Fix broken tables
Fix bold format for identifiers
Remove some blank lines & spaces

@phuclv90 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Absolute Import with baseUrl and path in tsconfig

Hello,
Does backstage support Absolute Imports?
I've tried setting it up in the root dir tsconfig file without success.
In my plugin I want to change this
import { Text } from './../../../../../typography';

To this
import { Text } from 'my-plugin/typography'

We do not. Too much hassle supporting it throughout all build systems and not enough benefit. Gotta support it in webpack, rollup, type declarations, storybook, jest, eslint, and prolly more ._.
See #729

We do not. Too much hassle supporting it throughout all build systems and not enough benefit. Gotta support it in webpack, rollup, type declarations, storybook, jest, eslint, and prolly more ._.
See #729

@Rugvip Thanks for the quick clarification, and explanation.
I will close this issue, hopefully one day we will have this feature.

@Rugvip Thanks for the quick clarification, and explanation.
I will close this issue, hopefully one day we will have this feature.


minimize with COBYLA constraints not respected

When I log the values passed to the objective function in the minimize method, I see that they do not respect my constraints...
I want them to be greater than 0 and the sum < 1, so I wrote these constraints:
constraints = ({'type': 'ineq', 'fun': lambda x: 1 - (x[0] + x[1])},
                {'type': 'ineq', 'fun': lambda x: x[0]},
                {'type': 'ineq', 'fun': lambda x: x[1]})

Did I do something wrong or is there a bug?
Here is my question on StackOverflow with full example: http://stackoverflow.com/questions/38356781/numpy-minimize-cobyla-constraints
Thank you!

You must always check that output.success is True.
If the solver does not converge, it is not guaranteed the constraints are satisfied.

I checked and It does converge apparently, since output.success == True for every iteration...

For the code you gave in the stackoverflow piece, it gives output.success == False.
If it's true for you, you are probably using an older version of scipy.

I am indeed working with an older version of Scipy, didn't know there could be such as difference... Thank you for your help!


fix(client-reports): Record lost sample_rate events only if tracing is enabled

Spurious client report outcomes were recorded before even if traces_sample_rate and traces_sampler were both None in the config.
Fixes https://getsentry.atlassian.net/browse/WEB-279

Also need to investigate the failed tests


pihole-FTL: no process found

veerendra@atom:~/projects/raspberrypi-homeserver/apps/pihole$ sudo docker-compose up
...
  [✓] Creating new gravity databases
pihole    |   [i] Storing downloaded domains in new gravity database...Stopping pihole-FTL
pihole    |
pihole    |   [✗] Unable to fill gravity table in database /etc/pihole/gravity.db_temp
pihole    |
  [✓] Cleaning up stray matter
pihole    |   [✗] DNS service is NOT running
pihole    | Stopping pihole-FTL
pihole    | pihole-FTL: no process found
pihole    | Stopping pihole-FTL
pihole    | pihole-FTL: no process found
...


Fixed issue #599

Fixed issue #599

If this does work, why do we keep the multi-os terminal files? Couldn't we now have a single one?

I've only tested this on windows. For windows this works, it looks like it may also work on the other platforms. I'm not able to test on linux for now.. I would like a single function as well.

Provided it works, we could actually even entirely retire that method.

Hello. Was this issue fixed? I am still getting:
terminal_windows.go:78: undefined: windows.Syscall

This isn't merged. Also, there's another PR resolving it on #604.
Feel free to try it and confirm it works on your OS.

I tried go get -u -v just now and find it still cannot be build.
terminal_windows.go:78: undefined: windows.Syscall
still this error.


Fix: Build process includes jest file and breaks type checking

The typescript directory is building with the jest file first and therefore isn't working in any type checking. Adding this ignore to that file seems to fix the jump in directory structuring.

Here's my case in point:

@1fabiopereira Any idea when you might get time to look at this project again?

@1fabiopereira Any idea when you might get time to look at this project again?

I will update some things this week


Code overhaul

This changes a number of things - namely, the original two arguments have been replaced with a larger params object, which includes more extensive usage of the API, like so:
import { loremIpsum } from 'https://deno.land/x/lorem_ipsum/mod.ts'

// Only using the required parameters
console.log(
  await loremIpsum({
    paragraphs: 3,
    length: 'medium'
  })
);

// More complex example using some formatting options
console.log(
  await loremIpsum({
    paragraphs: 3,
    length: 'verylong',
    decorate: true,
    ol: true,
    code: true,
    prude: true
  })
);

This has every single option in the API, although only paragraphs, and length are required.
If there's any problem you have with these changes, let me know.

Your changes have been applied as they are way better than what I wrote earlier, thank you for this amazing help!


Remove temp files and scratch space in favor of an Importer

Add BuildImporter class to handle canonicalization and reading of imports.
Add a check for .sass extension on inputId and use indented syntax.

cc @nex3


Docker Hub token in build secrets seems to have expired

@erikbra I took the liberty of pushing a new release today, however the docker step of the build pipeline has failed to connect to Docker Hub.
I rightfully don't have any access to the pipeline secrets, would you mind checking on this when you get a chance??  I don't use the docker image myself so I'm in no hurry.
Winget, dotnet global tool etc all worked just fine, and are available for immediate use 🥳

Hi, @wokket . I saw that you pushed a release, and I really appreciate it! This is what open source is all about, cooperation :) and I know I haven't been super-active here lately, as I have had a lot of other stuff to do in other parts of my life (you know, life, work, etc 😆 )
I'll look into the docker token:)

The docker hub token looks good, but the issue is probably due to using github.actor in the pipeline step, because that is probably not me when someone else has triggered the build:

https://github.com/erikbra/grate/blob/main/.github/workflows/build.yml#L174
I re-ran the build, and now it worked. It would make sense to replace github.actor here with a build environment variable instead, e.g:
username: ${{ env.DOCKER_HUB_USERNAME }}

I created a new issue for the actual fix: https://github.com/erikbra/grate/issues/200


Many, many compilation issues (especially with crates.io versions)

The only solution that works is cloning the repo and using it directly as set up. Attempting to make a standalone project causes a need for -Z build-std to be run with cargo; using crates.io versions causes the uefi crate to be extremely messed up and uefi-services to be unable to compile (it needs core::arch::asm, not just asm, apparently). Running nightly-x86_64-unknown-linux-gnu as my base toolchain.

I've found a configuration that makes it work: repository-based uefi-services, crates.io-based uefi.

Thanks for the heads up! I've released new versions of the uefi and uefi-services crates (https://github.com/rust-osdev/uefi-rs/issues/340), so your issue should be fixed.
I also filed https://github.com/rust-osdev/uefi-rs/issues/343 to add some periodic CI for the released versions of crates (currently we just test master) to hopefully prevent issues like this from lingering too long in the future.


Mac Smooth Scroll

What would be involved in implementing platform scrolling for text windows? IIRC Bjorn had already implemented a prototype of this.

MacVim already have it
https://github.com/macvim-dev/macvim/blob/53793980d40e6254a82f8f38277574f151cbce7d/src/feature.h#L1391-L1396

I think I did not explain very clearly what I meant. You'll notice that MacVim still only scrolls line by line. The scrolling on Mac should scroll individual pixels. (The Emacs Mac port does this fwiw)

you can create a new issue, pixel by pixel scroll or something like that.


fix(Button) fallback value for theme is global

What:
Provide a fallback value of global for a button theme name when an incorrect name is provided in a themeProvider.

Why:
At the moment when a user provides an incorrect theme name an application crashes.

How:
Provide additional themeProvider that checks if a theme name specified in a provider higher up the components tree exists. If it does nothing happens, if it doesn't global is provided as a theme name.

Checklist:

[x] Documentation
[x] Tests
[x] Ready to be merged

Looks good


Documentation/Example error: Middleware return

Thanks for the awesome project!
There is actually two things with the middleware example:

It returns 401, but according to code 401 spec: "The response MUST include a WWW-Authenticate header field (section 14.47) containing a challenge applicable to the requested resource." I think 403 would be the correct return code...
I had a hard time replacing the error_response function with something more succinct, like returning a quart.Response. I ultimately gave up, last error was from werkzeug missing app context. Maybe it is a bad pattern to include quart in a separate, quite standalone class's probably separate file.
But generally it could be better with some type annotations about app, scope, receive, send, about which the reader is quite lost.

The only constraint on the app is that is be a ASGI framework (it need not be Quart at all). Are you sure middleware is the best solution for your problem?

Are you sure middleware is the best solution for your problem?

Thanks for the reply!
I needed exactly the same X-Secret stuff, which is better handled there than in the app, as latter uses Quart-Schema, and this way I can make request don't reach validation wo. proper header.
BTW, in the example the strings shouldn't be compared to b'<value>' (bytestrings)? At least that's how it works for me...

Thanks, I've improved the docs in c60d3f7181a9632a96f6ffb6f1d4add09eaff20c
Btw I still think a decorator makes more sense here, as you can prevent the request reaching validation by placing the authentication decorator before the validation one.


Add clippy and target other architectures

Check: https://github.com/filecoin-project/neptune/blob/master/.circleci/config.yml

Clippy is run in CI by make clippy, with the same arguments as this example config.


Worldgen fixes

A bunch of small fixes for worldgen definitions, to fix errors reported when opening the vanilla datapack.

Thank you!


Refactor/resources post

Refactoring post resources by creating tests for context and controller.

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@2bff990). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop       #6   +/-   ##
==========================================
  Coverage           ?   78.12%
==========================================
  Files              ?        7
  Lines              ?       32
  Branches           ?        0
==========================================
  Hits               ?       25
  Misses             ?        7
  Partials           ?        0

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 2bff990...c302699. Read the comment docs.


[WIP] Unify LevelDB usage to single instance

Perfile LevelDB instance usage are changed to use single instance
per FLU server.
machi_csum_file reference is managed with machi_flu_filename_mgr
as an aim to manage filenames with leveldb
Not only chunk checksums, but the list of trimmed files are also
stored in LevelDB.
Remove 1024 bytes file header; instead put any metadata into
LevelDB if needed.
LevelDB db_ref() lifecycle is same as that of machi_metadata_mgr
machi_file_proxy just uses it as it's passed at process startup
There are several optimization space still left as it is

This is still WIP but it might be hard to continue the work for me. Currently this has 14 tests failing, mostly minor issues like mismatch of lists given to a function that only accepts binaries.

Also, I don't have permissions to push to machi original repo, so I'm opening a PR from my fork.

Heh, I see fewer than 14 test failuers ...  I see 6. Or 7, depending on the run.
*failed*
in function machi_proxy_flu1_client_test:api_smoke_test/0 (test/machi_proxy_flu1_client_test.erl, line 72)
**error:{badmatch,{error,partition}}

  machi_flu1_test: flu_smoke_test...*failed*
in function machi_flu1_test:flu_smoke_test/0 (test/machi_flu1_test.erl, line 124)
**error:{badmatch,{ok,{[{"prefix!^^00000000^e31aae63-9f81-46a0-9baa-60c66b72d7b0^0",0,
                 <<"yo!">>,
                 <<2,139,98,163,106,247,61,129,61,...>>}],
               []}}}

  machi_file_proxy_eqc:41: eqc_test_...*failed*
in function eqc:quickcheck/1 (../src/eqc.erl, line 1270)
in call from machi_file_proxy_eqc:'-eqc_test_/0-fun-1-'/1 (test/machi_file_proxy_eqc.erl, line 41)
**exit:{badarg,[{eleveldb,async_get,
                   [#Ref<<IP_ADDRESS>751>,<<>>,
                    <<16,0,0,0,2,12,179,90,109,150,...>>,
                    []],
                   []},
         {eleveldb,get,3,[{file,"src/eleveldb.erl"},{line,152}]},
         {machi_csum_table,is_file_trimmed,2,
                           [{file,"src/machi_csum_table.erl"},{line,158}]},
         {machi_file_proxy,init,1,
                           [{file,"src/machi_file_proxy.erl"},{line,224}]},
         {gen_server,init_it,6,[{file,"gen_server.erl"},{line,306}]},
         {proc_lib,init_p_do_apply,3,[{file,"proc_lib.erl"},{line,237}]}]}

.*failed*
in function eqc:quickcheck/1 (../src/eqc.erl, line 1270)
in call from machi_ap_repair_eqc:'-prop_repair_test_/0-fun-1-'/3 (test/machi_ap_repair_eqc.erl, line 93)
**exit:{timeout,
    {gen_server,call,
        [<0.10322.0>,
         {req,
             {append_chunk_extra,
                 {1130,<<225,248,169,11,186,142,...>>},
                 [],0,<<"pre">>,
                 <<2,47,38,147,...>>,
                 0}},
         2000]}}

  machi_ap_repair_eqc:101: prop_repair_par_test_...*failed*
in function eqc:quickcheck/1 (../src/eqc.erl, line 1270)
in call from machi_ap_repair_eqc:'-prop_repair_par_test_/0-fun-1-'/3 (test/machi_ap_repair_eqc.erl, line 103)
**exit:{timeout,
    {gen_server,call,
        [<0.11008.0>,
         {req,
             {append_chunk_extra,
                 {1115,<<168,193,7,104,172,160,...>>},
                 [],0,<<"pre">>,
                 <<169,152,13,55,...>>,
                 0}},
         2000]}}

  [done in 18.022 s]
module 'machi_admin_util_test'
  machi_admin_util_test: verify_file_checksums_test_...*failed*
in function machi_util:wait_for_death/2 (src/machi_util.erl, line 357)
in call from machi_test_util:start_flu_packages/1 (test/machi_test_util.erl, line 56)
in call from machi_admin_util_test:verify_file_checksums_test2/0 (test/machi_admin_util_test.erl, line 48)
**exit:{not_dead_yet,<0.10485.0>}

This PR is stalled, alas, until staff is available to resume the unfinished work.


Power usage during defrost operation

Hi,
I have build my ESPAltherma with an M5StickCPlus, everything is working as it should. But now it is getting colder and the unit starts more defrost cycles during heating.
When the defrost start the primary inv current drops to 0, but in my P1 meter i can see that is peaks from 4Kw up to 9kw. Defrost cycle takes about 5 min.  But it is not counted in my total for the day for the heatpump.
How can i achive this?

These are the data my unit logs during defrost:

I use Home Assistant to log everything. I used my P1 data to measure the kwh spent during the 4/5 minutes of defrost and i averages about 0.4kwh for a defrost cycle. But the primary inv (A) does not reflect this so my logging for the usage of the heatpump total kwh for the day is missing the defrost cycle

So something is wrong with your primary current data…

Yeah i dont know. the normal heating proces is registerd just fine. It looks like the defrost is is not using the primary inv. But somtething like the heating element in the DHW tank (that is 9kw) but i do not se a register for that value

I'll close this one. Open a discussion if still not clear


runners: nrfjprog: workaround for multi-core hex files

The nrfjprog utility is not capable of flashing a hex file which
affects the flash memories of both coprocessors of the nRF53 family of
SoCs.
However, the user is capable of creating such a hex file using the
HEX_FILES_TO_MERGE build system variable.
An example use case is to build a bluetooth controller application for
the network core, then use the zephyr.hex file in that build directory
as the HEX_FILES_TO_MERGE argument for a separate Bluetooth
application build targeting the app core.
Work around this by detecting the situation and doing the right thing
by splitting the hex file back up again, even if thats a bit awkward.
Splitting the hex into app and network core components allows them to
be flashed separately. This is the only way we can get the job done
with nrfjprog.
This is arguably nicer since there's just one 'west flash' invocation.
At least in the use case named above, you wouldn't need to rebuild the
controller application very often, so this is a simpler user workflow.

In which situation can this occur upstream ?

I explained this in detail in the commit log and in the source code comment.

Being able to create a list of hex files to use for the runner.

That is the logical effect of HEX_FILES_TO_MERGE.

In which situation can this occur upstream ?

I explained this in detail in the commit log and in the source code comment.

Except for the following:

No CMake code in Zephyr or Zephyr modules uses HEX_FILES_TO_MERGE (or in NCS for that matter)
The HEX_FILES_TO_MERGE uses a fixed name name merged.hex which is actually not so nice. Such output names should ideally be user configurable.
The runners will not flash the merged.hex per default, even when using HEX_FILES_TO_MERGE.

Therefore I think that doing this in Zephyr, as currently proposed in this PR is only a partial solution.
In NCS, the actual hex to flash is handled by modifying the runners yaml properties, like this:
https://github.com/nrfconnect/sdk-nrf/blob/20e438ad68ff982985ec7334dcdd098586c580d5/cmake/partition_manager.cmake#L541-L550
get_target_property(runners_content runners_yaml_props_target yaml_contents)

string(REGEX REPLACE "hex_file:[^\n]*"
  "hex_file: ${merged_hex_to_flash}" new  ${runners_content})

set_property(
  TARGET         runners_yaml_props_target
  PROPERTY       yaml_contents
  ${new}
  )

so therefore, this is not something that happens in a vanilla Zephyr today.
What i'm asking, is that if we want to base this on HEX_FILES_TO_MERGE as you state in the commit message, I would like to see a proper integration of HEX_FILES_TO_MERGE and the flashing of that hex-file by the runner.
This will also be beneficial for this use-case:
https://github.com/zephyrproject-rtos/zephyr/pull/32490#pullrequestreview-596230831
where it was discussed offline that a more generic solution should be used, but a nRF53 specific solution was acceptable short term.

Being able to create a list of hex files to use for the runner.

That is the logical effect of HEX_FILES_TO_MERGE.

No, this still doesn't provide a list of files to the runner itself.
It provides a single merged file to the runner, a file which then has to be split by the runner.
What I'm asking, as the possibility to provide a list of files to flash and / or list of runners to use (each domain to flash could require dedicated arguments, just like the nRF53, that requires different --coprocessor arguments.)
The principle of splitting merged hex files is not scalable among multiple socs / runners.
As example, this fix won't work if flashing using the JLink runner.

Except for the following:
* No CMake code in Zephyr or Zephyr modules uses HEX_FILES_TO_MERGE (or in NCS for that matter)

That doesn't matter because it is publicly documented user visible functionality:
https://docs.zephyrproject.org/latest/application/index.html#application-cmakelists-txt

The Cmake property HEX_FILES_TO_MERGE leverages the application configuration provided by Kconfig and CMake to let you merge externally built hex files with the hex file generated when building the Zephyr application.

The user really can make this happen in vanilla zephyr.


Remove babel

Use, instead of babel, the new node's features.

hey, can I fix this issue

hey, can I fix this issue

Sorry, Mate. I've made this fixing without checking the responses.

hey, can I fix this issue

Would you like to help on this one?
https://github.com/henrybarreto/linkly/issues/5


Update spikeglxdatainterface.py

fix for SpikeGLXLFPInterface.get_source_schema()
Won't change anything in the conversion, this is relevant only for the GUI.

@sbuergers the tests are failing at axonaunitrecordingextractor.py due to a neo import, can you check this out?

I'll get this through for now while I analyze the best way to reduce code duplication. Thanks!


Checks the type of torque-specific CSS rules

Fixes #150
Sending a malformed CSS style in the query will make Windshaft respond with a JSON error like the following:
{
    errors: [
    "Unexpected type for property '-torque-aggregation-function', expected string"
    ]
}

@rochoa what do you think?
cc @javisantana

Thanks :heart:


Larger margin on small and tiny screens

I added some margin to the main content on small screens, which should help make articles easier to read.
In deciding the margin, I looked at two other text-focused sites on mobile to see what they use for small devices. Wikipedia uses 16px, New York Times uses 20px. I ended up going with 14px.
Before:

After:

Sure! I'm free tomorrow any time after 5:00pm Central time
On Mon, Apr 23, 2018, 8:17 PM Raemon<EMAIL_ADDRESS>wrote:

Important note: different mobile devices automatically insert different
amounts of whitespace. We'd actually gone back and forth on
amount-of-mobile-whitespace and ended up with the current (smallish) amount
because it showed up at least reasonably on a wide variety of devices (some
of which automatically at 10px).
BTW, I think it might be good to spend an hour skyping if you're up for
doing a variety of LW PRs, to help orient. Would you be up for that (maybe
sometime tomorrow?)
ᐧ
On Mon, Apr 23, 2018 at 6:06 PM, James Lamine<EMAIL_ADDRESS>wrote:

I added some margin to the main content on small screens, which should
help make articles easier to read.
In deciding the margin, I looked at two other text-focused sites on
mobile
to see what they use for small devices. Wikipedia uses 16px, New York
Times
uses 20px. I ended up going with 14px.
Before:
[image: image]
<
https://user-images.githubusercontent.com/863251/39160537-6ff96600-4731-11e8-8c35-8f46a7d90406.png
[image: image]
<
https://user-images.githubusercontent.com/863251/39160551-85d2cd54-4731-11e8-8e9a-c4cfbe677f60.png
After:
[image: image]
<
https://user-images.githubusercontent.com/863251/39160463-1f81a0c0-4731-11e8-8dcb-243a5b2dcf7a.png
[image: image]
<
https://user-images.githubusercontent.com/863251/39160584-b309aba8-4731-11e8-9b18-fbcac2b3f2ae.png

You can view, comment on, or merge this pull request online at:
https://github.com/Discordius/Lesswrong2/pull/538
Commit Summary

Larger margin on small and tiny screens

File Changes

M packages/lesswrong/styles/_layout.scss
https://github.com/Discordius/Lesswrong2/pull/538/files#diff-0 (4)

Patch Links:

https://github.com/Discordius/Lesswrong2/pull/538.patch
https://github.com/Discordius/Lesswrong2/pull/538.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Discordius/Lesswrong2/pull/538, or mute the thread
<
https://github.com/notifications/unsubscribe-auth/ADGKduLsPb1yVVJwdxryyqCMq0jnzUa2ks5trnqPgaJpZM4Tg2J8
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Discordius/Lesswrong2/pull/538#issuecomment-383771707,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0sE29auv3JvxclSHU695Qth3v5dJ87ks5trn0WgaJpZM4Tg2J8
.

Email sent.
Adding padding to small (but not tiny) screens sounds good. I'll also do
some research into how different browsers add default padding - ideally the
solution here would be to reset that default padding to zero so that it's
consistent across browsers.
On Mon, Apr 23, 2018 at 10:21 PM, Raemon<EMAIL_ADDRESS>wrote:

Cool. That'd be 4pm PT? My email is<EMAIL_ADDRESS>ping me there to
coordinate.
For this particular branch: Because of the variable auto-added padding, I
wouldn't add more padding to the tiny-size screen. However, I do think it
makes sense to add a small amount of padding to the small-size screen
(since I just looked and realized the small-size has literally zero right
now)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Discordius/Lesswrong2/pull/538#issuecomment-383790977,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0sE_kGTPdnQy8nN29ptIllt_ivsZv4ks5trpohgaJpZM4Tg2J8
.

So the devices I remembered causing an issue with default-padding didn't seem to be doing that, not sure if my memory is bad. Fairly confident at least some devices did it.
We also used to have generally wider margins on mobile-sizes, and people complained about it because the screen real estate was so precious (i.e. people just double-tapped on everything to zoom in, which resulted in other slightly weird experiences).
So I wouldn't go wider than 10px. The 5px we currently have for the .main container might be on the small size but I think 14px would just cause the "zoom in all the time" effect, even before some browsers start tacking larger paddings on.

I do think it makes sense to wrap this PR up with a simpler "add a small padding to small-size devices" so that the padding is at least consistent between tiny and small screens, and then if you want to do the css-reset thing, do that in a separate issue.

5px seems fine for the 'tiny' size - I still think 14 is more appropriate
for the 'small' size although I can do 10 if that's what you prefer.
On Tue, Apr 24, 2018, 7:31 PM Raemon<EMAIL_ADDRESS>wrote:

So the devices I remembered causing an issue with default-padding didn't
seem to be doing that, not sure if my memory is bad. Fairly confident at
least some devices did it.
We also used to have generally wider margins on mobile-sizes, and people
complained about it because the screen real estate was so precious (i.e.
people just double-tapped on everything to zoom in, which resulted in other
slightly weird experiences).
So I wouldn't go wider than 10px. The 5px we currently have for the .main
container might be on the small size but I think 14px would just cause the
"zoom in all the time" effect, even before some browsers start tacking
larger paddings on.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Discordius/Lesswrong2/pull/538#issuecomment-384122668,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0sExwYIbrhk3eY1LvJbGp3BtFsnU2Sks5tr8PQgaJpZM4Tg2J8
.

Sounds reasonable.
So keep 5px for tiny screens and make it 10px for small screens?
On Tue, Apr 24, 2018, 7:32 PM Raemon<EMAIL_ADDRESS>wrote:

I do think it makes sense to wrap this PR up with a simpler "add a small
padding to small-size devices" so that the padding is at least consistent
between tiny and small screens, and then if you want to do the css-reset
thing, do that in a separate issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Discordius/Lesswrong2/pull/538#issuecomment-384122821,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0sE3K2ar4sceTZjxiFnhQdXyrdbAp5ks5tr8QOgaJpZM4Tg2J8
.

10px for small-size screens seems fine
ᐧ
On Tue, Apr 24, 2018 at 5:35 PM, James Lamine<EMAIL_ADDRESS>wrote:

Sounds reasonable.
So keep 5px for tiny screens and make it 10px for small screens?
On Tue, Apr 24, 2018, 7:32 PM Raemon<EMAIL_ADDRESS>wrote:

I do think it makes sense to wrap this PR up with a simpler "add a small
padding to small-size devices" so that the padding is at least consistent
between tiny and small screens, and then if you want to do the css-reset
thing, do that in a separate issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<https://github.com/Discordius/Lesswrong2/pull/
538#issuecomment-384122821>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AA0sE3K2ar4sceTZjxiFnhQdXyrdbAp5ks5tr8QOgaJpZM4Tg2J8>
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Discordius/Lesswrong2/pull/538#issuecomment-384123187,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADGKdvCPNXTQ8j3gr02ywzKwHyfZgLOAks5tr8S4gaJpZM4Tg2J8
.

I've updated it to have a 10px margin on both small and tiny screens. I think this is what you wanted? If not, let me know.


(#100) Update .transifexrc format

Description Of Changes
Motivation and Context
The format of this file has changed with the new CLI and the deprecation of the older CLI tool, so we need to update this in order to work with the newer tool.
Testing
Tested with the ChocolateyGUI build by copying the changed lines into the recipe file locally, and then commenting out line 53 so I could force the step to run locally. Previously there was an error that no hostname/token was provided; now the error was that I was using an API token with incorrect auth, so it accepted the new hostname and token, I just didn't have the auth personally to do anything with uploading files.
Operating Systems Testing
Win11
Change Types Made

[ ] Bug fix (non-breaking change).
[x] Feature / Enhancement (non-breaking change).
[ ] Breaking change (fix or feature that could cause existing functionality to change).
[ ] Documentation changes.
[ ] PowerShell code changes.

Change Checklist

[ ] Requires a change to the documentation.
[ ] Documentation has been updated.
[ ] Tests to cover my changes, have been added.
[x] All new and existing tests passed?
[ ] PowerShell code changes: PowerShell v2 compatibility checked?

Related Issue
Fixes #100

@vexx32 thanks for getting this fixed up!


Update cloud-sdk-go to v1.15.0

Description
Update cloud-sdk-go to v1.15.0.
Readiness Checklist

[ ] My code follows the code style of this project
[ ] My change requires a change to the documentation
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[x] All new and existing tests passed

Closed in favor of https://github.com/elastic/terraform-provider-ec/pull/725.


fade animation and .current class on content DIVs

Hello,
great tabs!  i'm currently trying to use them for a project i`m working on and i have 2 questions:
Is there any way to change the current animation (slide from top left to bottom right) to a simple "fade" animation?
Also, if i enable "currentClass", only the tab buttons receive a ".current" class on click. Is there any way to add the .current class to the tab content DIVs items as well?
thank you & best regards!
andi

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/34273171-fade-animation-and-current-class-on-content-divs?utm_campaign=plugin&utm_content=tracker%2F654828&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F654828&utm_medium=issues&utm_source=github).

For the first question, let me take a look to the code and I'll try to change it.
About the second question: you can use the onChange callback/event to apply that class wherever you want.


fixes issues: #2189

Now in the right order panel, displayed is the last payment status.

This issue was fixed in #2348.


Release of Optimization Results on KITTI 0018 and 0027 sequence

Dear authors,
Thank you for your great work and open-sourced code. Due to the limited configuration of my device, I cannot train the network with the default parameters, so I cannot obtain results as good as yours. Would you mind releasing the optimized trajectory (i.e. the final pose of each frame) on the KITTI 0018 and 0027 sequence (Corresponding to the DM2 results listed in Table 1. in the paper)? I just want to stack the point cloud together and visualize the map quality. Thank you!

@Gaaaavin @Joechencc please take a look

@llihku One Google Drive link is added to the results folder in the recent commit. The link contains the best mapping results (poses) reported in the paper.

Please follow this link https://github.com/Gaaaavin/DeepMapping2. We amended some version problems and tested under several environments. Now it should be ok. We will update this repo soon and provide more details of how to generate a custom group matrix and prior pose


Allow override of NuGetCacheFolder location through environment variable

The NuGet CLI (2.x) provides an environment variable override for specifying the location of the NuGet cache.  This is key when running multiple processes that pull NuGet packages to avoid file locks in the cache (i.e. in a multi- build agent per server environment).  This change enables support for the override in Paket, via an environment variable (e.g. Windows PS):
$env:NuGetCachePath = "C:\Custom\Cache\Location"

can you please add this to the docs as well?

Done - also added a bit of extra blurb about the machine cache, as it seems it's used regardless of any additional caches.  If you'd prefer it elsewhere, happy to do so?

Also, looks like there's problem with the CI checks.. unrelated?

awesome. thx


Can not minimize or change window size on Linux

VSCode Version: 1.19.3
OS Version: Ubuntu 17.10

Steps to Reproduce:

Install and Run Visual Studio Code
Try to change the window size

Does this issue occur when all extensions are disabled?: Yes
I do not know if this is intentional design (I hope not) but I am unable to un-maximize or move the Visual Studio Code window.  I have installed taskbar tweaker which puts maximize / minimize / resize options on all of my windows, but Code does not display them (see image).
How can I resize the window?

(Here you can see chrome in the foreground with the resize buttons and Code in the background with no such buttons available)

@computersarecool could you try with this newer build for Linux to reproduce: https://az764295.vo.msecnd.net/exploration/ed422f909ea470f79e3e3e39c948803d9b764919/code-exploration-1.22.0-1522303929_amd64.tar.gz


Universal area cutting tool

A universal tool to cut areas would be very useful in particular for landuse mapping.
The tool proposed here need two different set of operand. According to #2970 I call these flagged and unflagged. Both of theses sets can include lines and areas/MPs. Therefore we can differentiate between four types of operands: unflagged lines, flagged lines, unflagged area/MP, and flagged area/MP. The tool is modifiying the unflagged areas/MPs by using all other operands as tools to define the modification.
The tool shall do the following:

Temporarily generate areas  surrounding the flagged lines with a little distance. The generated areas are handeld like flagged areas in the next step. This step is intended to allow ungluing landuse from highways.
Start to modifiy the unflagged areas by cutting out the flagged areas where they do overlap.
Split the resulting areas of the previous step along the unflagged lines. Incomplete cuts of areas are ignored.

I will add application examples later.

Thanks for your suggestions @slhh, but I think you might misunderstand the intended audience and scope of iD. iD is a basic to mid-level editor. We do not intend to add features to it that introduce complicated UI for advanced editing tasks -- that domain is already well covered by JOSM. I'm going to close this and several other of your requests because I believe they fall into this category.


Runchecker failure with no-asm: tests hang

Output is:
05-test_cast.t ..................... ok
05-test_cmac.t ..................... ok
05-test_des.t ...................... ok
05-test_hmac.t ..................... ok
05-test_idea.t ..................... ok
05-test_rand.t ..................... ok
05-test_rc2.t ...................... ok
05-test_rc4.t ...................... ok
05-test_rc5.t ...................... skipped: rc5 is not supported by this OpenSSL build
06-test_algorithmid.t .............. ok
06-test_rdrand_sanity.t ............ ok
make[1]: *** [Makefile:3274: _tests] Terminated
make[1]: *** wait: No child processes.  Stop.
make[1]: *** Waiting for unfinished jobs....
make[1]: *** wait: No child processes.  Stop.
make: *** [Makefile:3271: tests] Terminated

The test following rdrand is the big number test which is slow.

The ending point isn't deterministic.  For example, these have been seen too:

25-test_x509.t ..................... ok
15-test_ecdsa.t .................... ok

So apparently it is just reallyreally slow. Still can be problem with seeding the rng if the runchecker OS is old and does not support getrandom.

I still can't login to the run-checker machine to figure this out.  @levitte.

Hmmm....
This test runs on the run-checker host in just over 25 minutes.
The two longest running tests are:

04-test_encoder_decoder_legacy.t ... ok  1191522 ms
20-test_dhparam_check.t ............ ok   700571 ms
which amount to 32 minutes (4 processor test, so it's possible)

This could depend heavily on system load at the time.

This is probably addressed by openssl/tools#86

Optimised, the build time doesn't change a lot.  The test time is circa 6 minutes which is much better.

Should be fixed, closing.


Any progress/plan about code open sourcing?

Hi, I've just read the paper "BoGraph: Structured Bayesian Optimization From Logs for Systems with High-dimensional Parameter Space" and I think the paper is very interesting and relavent to my research field
Is there any plan about releasing the code?

Hi, yea I am planning on open-sourcing it as soon as the paper goes through, we should find out in 2-3 months.

Awesome!

Hello, any updates on open-sourcing the code?


Consider a different documentation generation engine

Sphinx is okay and is fully featured, however, I am not fond of to the rst format nor the way sphinx generates documentation. I would like a solution that is closer to the tool-chain that we and many other data science minded individuals use (e.g. markdown).
The solution choice should (IMO) be driven by approachability. Meaning:

it should be easy for someone to read and contribute to our documentation without knowing a markup specification. I would argue that markdown is much more intuitive that rst and thus approachable.
documentation issues and errors should have a low barrier for a user to report them.
approachability should also be taken to mean searchable and scanable (scrolling should be limited or at least a choice).
Our common interfaces should be easy to find and digest.
A version change log should be accessible.
The developer documentation experience should be straight forward and require little thought.
It should be something we want to use when we need to reference our work.

I'd like to submit pdoc for consideration:
https://pypi.org/project/pdoc/
I generated the attached documentation for hydrotools.metrics by adding __docformat__ = "numpy" to the top of hydrotools.metrics.metrics.py and running pdoc hydrotools.metrics -o htmldocs from the command line.
ht_metrics_docs_example.tar.gz

Note pdoc supports embedding Markdown directly in docstrings (the default) and equation rendering.

Here's an updated example with rendered math (pdoc hydrotools.metrics -o htmldocs --math`). This required no changes to the module itself.
ht_metrics_math.tar.gz


assert_downcast!

There was a suggestion by pie-flavor from Rust Discord to add that kind of macro:
macro_rules! assert_downcast {
    ($e:expr as $t:ty) => {
        $e.downcast::<$t>().unwrap_or_else(|e| panic!(concat!("Expected ", stringify!($t), ", got {:?}"), e.into_object()))
    }
}

Should it be added here? Is it valueable enough?

What does downcast do?

@vladinator1000 as far as I remembed the discussion, the idea was to assert that some Box<dyn Trait> can be downcasted into a concrete type.
I'm still not sure if it worth to add it, so I would prefer to collect some feedback if it will be useful for anyone.


Startup failure if default event loop was already initialised (CON-559)

https://github.com/espressif/esp-matter/blob/915405c0ab9f339710b85027ff587b594e9eb686/components/esp_matter/esp_matter_core.cpp#L944
From my understanding at this line we can just check if creation was successful or underlying method returned ESP_ERR_INVALID_STATE which means that event loop is already created. Am I right or there's some specific reason we need a freshly created event loop to function?

@rfortunatov That can be done, thanks for pointing it out,ESP_ERR_INVALID_STATE is returned when default event loop is already created.

Exactly, so, should I open a PR?


WIP: fix checking for lockfiles

If I have checked correctly then _.some(_.pick(repository.files, projectLockFiles)) does not behave how we want it now that we changes the value of each files-file from true to array (empty or with paths)
--> hasProjectLockFile & hasModuleLockFile need to check with a different function
we often just check if newJob = falsy but we should check if branches get created for in-range dependency updates too! Often they don't result in a PR  most of the time they just get deleted again and that also doesn't return a new job!
--> we need better/more tests here.
Currently the tests pass (to my surprise) and I'm nearly sure that they should not... that's why I think we need to test if in some cases a branch is created instead

fixed in master


[ICD] How can we get the old key when verification/old key has not yet been stored with plain text in controller side

Currently we have not yet stored old/verification key via ICDClientInfo in  persistent storage, and we store this key in KeyStore. When we try to refresh key after validating check-in message, we need new key and old key, but it seems we don't have API to retrieve the key from key store. Clearly it is not recommended to store the old key with plain text in persistent storage. Any good idea on this? maybe i missed something?  @bzbarsky-apple  @mkardous-silabs  thanks

When we try to refresh key after validating check-in message, we need new key and old key

Why do you need the old key?

@bzbarsky-apple  verification key is the old key, we use it when it is not admin.
chip-tool is usually an admin and doesn't need to have the verificationKey.
https://github.com/project-chip/connectedhomeip/blob/6ea3c34c119ec56eb425d4486d0b6a3f62742ef6/src/app/clusters/icd-management-server/icd-management-server.cpp#L250

@bzbarsky-apple this issue may relate to the design for the callback, OnRefreshKeyGenerate, https://github.com/project-chip/connectedhomeip/pull/30731/files#r1439672537, thanks

Non-admin clients have to store the old key in plaintext somewhere, yes, given our spec setup.  So yes, we need some API to get the current key for cases when we are not an admin.
Clients that always expect to be admins can just return error from that API presumably.

@bzbarsky-apple @yunhanw-google
Since we refactored everything server side to enable PSA storage for the keys, i think we could modify the spec to avoid clients to have a plaintext version of the key.

@mkardous-silabs does that mean we can remove verification key from RegisterClient command in ICD Management cluster? thanks

@yunhanw-google It would get transformed into something else but that change is not a high priority. In the mean time, boris is correct, non-admin clients would need to store it in plaintext some where

thanks, close it now.

@bzbarsky-apple @yunhanw-google Since we refactored everything server side to enable PSA storage for the keys, i think we could modify the spec to avoid clients to have a plaintext version of the key.

@mkardous-silabs  have we resolved it in server side for old key storage?


[PE-D][Tester A] Inconsistent Selection of Team Member for Task Assignment

Description
When assigning a task to a team member in a project where there are multiple team members with the same name, the application consistently selects the first occurrence of the name for task assignment. However, there are instances where users may want to assign the task to a specific person among the duplicates, such as the third person of the same name in the list of team members.
Steps to Reproduce

Open the application.
Add a project by entering add project Duke chatbot.
Assign the team by entering assign team Joe, Joe, Joe, Joe, Cody /to Duke chatbot.
Assign a member to do the task by entering assign person Joe /to Rename Duke /in Duke chatbot.
Enter show project Duke chatbot to see the updated member assigned to the task and observe that only 1 'Joe' is assigned to the task.

Expected
When assigning a task to a team member named "Joe", users should be provided with the option to select which specific "Joe" among the duplicates to assign the task to, or the application should prompt for clarification if there are multiple team members with the same name.
Actual
The application automatically assigns the task to the first occurrence of "Joe" in the list of team members, without providing an option for users to specify which "Joe" to assign the task to.

Note
This issue can lead to misassignment of tasks and confusion among users, especially in projects with multiple team members sharing the same name. Providing users with the ability to specify which team member to assign the task to would improve the flexibility and usability of the task assignment feature.
How to improve product
If met with people having the same name, assign person function can include an index, e.g., assign person <NAME> <INDEX> /to <TASK_NAME> /in <PROJECT_NAME>. The purpose of the index is to identify specifically which person of that particular name to assign the task to.
For example,
Project: Duke chatbot
Team members: Joe, Joe, Joe, Joe, Cody
If the third person is to be assigned the task, the command would be assign person Joe 3 /to Rename Duke /in Duke chatbot

Labels: type.FunctionalityBug severity.Medium
original: ashleyy2444/ped#10

Fixed, can close after test

cannot be fixed in v1.4


Update AtkUnitBase and AddonContextMenu

RE #195

Double check Dalamud doesn't use this since it was from the context menu

It does
https://github.com/goatcorp/Dalamud/blob/master/Dalamud/Game/Gui/ContextMenus/ContextMenu.cs#L215

I dunno when goat plans to pull clientstructs again but a note will need to be made of this


fix: gitignore wasm binaries

Description
Ignore builtin wasm binaries
Motivation and Context
These often cause conflicts due to non-deterministic builds. Until we can build deterministically, having these committed is unnecessary.
How Has This Been Tested?
Code still compiles
What process can a PR reviewer use to test or verify this change?
Check that wasms are still built
Breaking Changes

[x] None
[ ] Requires data directory to be deleted
[ ] Other - Please specify

:roll_eyes: :laughing:


Update ruby_installer.rb for ruby 2.3.1

fixes #318
seems to work when I tested locally ¯\_(ツ)_/¯

how do i pr?????


[BUG] MongoParseError: Unsupported Option 'useFindAndModify' in Connection String

Describe the bug
When starting the Node.js server using server.js, an error occurs due to the unsupported option useFindAndModify in the MongoDB connection string, resulting in a MongoParseError. This error prevents the application from connecting to the MongoDB database and causes the server to crash. To resolve this issue, the connection options in db.js should be reviewed to remove or replace the useFindAndModify option with supported alternatives, and it may also be necessary to update the Mongoose version to ensure compatibility with the latest MongoDB driver.
To Reproduce
Steps to reproduce the behavior:

Go to 'talk-trove'
Click on 'talk-trove/backend'
Start Serve  ' yarn run dev'
See error -
Server running on PORT 5000...
C:\Users\codeb\Downloads\project\talk-trove\backend\node_modules\mongodb\lib\connection_string.js:272
throw new error_1.MongoParseError(${optionWord} ${Array.from(unsupportedOptions).join(', ')} ${isOrAre} not supported);
^

MongoParseError: option usefindandmodify is not supported
at parseOptions (C:\Users\codeb\Downloads\project\talk-trove\backend\node_modules\mongodb\lib\connection_string.js:272:15)    at new MongoClient (C:\Users\codeb\Downloads\project\talk-trove\backend\node_modules\mongodb\lib\mongo_client.js:52:63)
at NativeConnection.createClient (C:\Users\codeb\Downloads\project\talk-trove\backend\node_modules\mongoose\lib\drivers\node-mongodb-native\connection.js:312:14)
at NativeConnection.openUri (C:\Users\codeb\Downloads\project\talk-trove\backend\node_modules\mongoose\lib\connection.js:835:34)
at Mongoose.connect (C:\Users\codeb\Downloads\project\talk-trove\backend\node_modules\mongoose\lib\mongoose.js:447:15)
at connectDB (C:\Users\codeb\Downloads\project\talk-trove\backend\config\db.js:6:33)
at Object. (C:\Users\codeb\Downloads\project\talk-trove\backend\server.js:12:1)
at Module._compile (node:internal/modules/cjs/loader:1364:14)
at Module._extensions..js (node:internal/modules/cjs/loader:1422:10)
at Module.load (node:internal/modules/cjs/loader:1203:32) {
[Symbol(errorLabels)]: Set(0) {}
}
Node.js v18.20.4
[nodemon] app crashed - waiting for file changes before starting...
Expected behavior
The expected behavior is for the Node.js server to start successfully and establish a connection to the MongoDB database without encountering any errors. Upon successful connection, the server should log a message indicating that it is running on the specified port (e.g., "Server running on PORT 5000...") and be ready to handle incoming requests. Removing or replacing the unsupported useFindAndModify option in the MongoDB connection string should allow the application to function as intended, enabling seamless interaction with the database and ensuring that the server remains stable and responsive.
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome, Edge
Version: Latest

Smartphone (please complete the following information):

Device: Motorola G54 5G
OS: Android 14
Browser: Chrome
Version: Latest

Additional context
The useFindAndModify option was deprecated in Mongoose 5.x and later versions due to changes in the MongoDB driver. This deprecation may lead to compatibility issues when using recent versions of the MongoDB Node.js driver, as certain options are no longer supported. This bug was encountered during the development phase while attempting to establish a database connection, which is a critical step for the application to function correctly. The application aims to provide a robust backend service, and resolving this issue is essential for maintaining stability and ensuring that all database-related operations can be performed without interruption.
This fix is particularly important for developers working on the project, as it will prevent future issues related to outdated configuration options and facilitate a smoother development process. Additionally, addressing this bug will enhance the overall user experience by ensuring that the application runs reliably and efficiently.

Gssoc'24 - Extended Contributor
HacktoberFest'24 Contributor

@anothercoder-nik  test branch is active not the main

u have skipped the command "yarn" before yarn run dev

Raise question in discussion if u have further problem @anothercoder-nik

#115 go here


SyntaxError: Parse error - Wkhtmltopdf with react-pdf lib

Hi all,
I am using wkhtmltopdf version <IP_ADDRESS> (with patched qt) to render my reactJs app! It worked fine, until I added the react-pdf lib to render the pdf generated inside my app. I followed the code described on react-pdf documentation (see https://www.npmjs.com/package/react-pdf) to make it work.
The pdf is rendered inside my page, and it looks pretty cool. But when I try to run wkhtmltopdf again, to generate a pdf of any page of my app, the following error is returned:
desenv01@desenv01-PC:~$ wkhtmltopdf -O Landscape --print-media-type --debug-javascript  http://localhost:3000/report/1 report.pdfLoading pages (1/6)
Warning: undefined:0 ReferenceError: Can't find variable: Float64Array
Warning: http://localhost:3000/assets/js/app.js:46789 SyntaxError: Parse error
Counting pages (2/6)
Resolving links (4/6)
Loading headers and footers (5/6)
Printing pages (6/6)
Done

Then, I went to my app.js to see what's on line 46789:
set href(href) {
  clear.call(this);
  parse.call(this, href);
},

get protocol() {
  return this._scheme + ':';
},
set protocol(protocol) {
  if (this._isInvalid)
    return;
  parse.call(this, protocol + ':', 'scheme start');
},

The error happens on the line that says parse.call(this, href); which is part of pdf.combined.js script.
I could not find any solution online so I wondered if there is anyone who might know if I did something wrong or a way to work around it.
Thanks..

Hi @marlon407, would you kindly check is that's an issue on react-pdf 1.6.0?

Closed because of no interest. Happy to reopen if the discussion continues.


chore(deps): Update postgrest-js version to 1.15.0

Updates postgrest-js to version 1.15.0 to support latest features #994

Seems like this can be closed. The version used is 1.15.6

Closing - thanks for the PR!


Fix CSGShape debug_collision_shape crash

Fixes CSGShape debug_collision_shape crash.
Fixes https://github.com/godotengine/godot/issues/84304
While the immediate crash source was root_collision_shape being null I also do not trust a function that does not guard itself against crashes from e.g. get_world_3d() or get_global_transform(). In fact the only guard was a well-meaning comment.

What about print some warning with the origin comment as it's content ?

I am not sure. Do people benefit from a printed warning here  about this?
The original comment was a code comment, not directed or with useful information for engine users in particular.

Thanks!

Should be cherry picked, see:

https://github.com/godotengine/godot/issues/89640

Cherry-picked for 4.1.4.


Automatic test framework detection

Includes separation of the coverage runner logic into different strategies according to the test frameworks.
Includes new logic in test detection.
Includes new tests for JUnit3 and a mixed project with tests from JUnit3, JUnit4 and JUnit5.
Closes #34.
Related to #31.

Ready as soon as https://github.com/STAMP-project/test-runner/pull/99 is merged.

Follow-up to the discussion in #34, I ended up using the heuristics that are already implemented in test-runner.
Then, I run all the tests, even if there is a mix of test frameworks being used.

Hi André
That's a great feature.
So, If I well understand, now we have one test strategy associated to each test (here) instead of being associated to the test suite (all the test). Then, for each test, the corresponding strategy is executed (here).
This strategy is fine for me.  However, in the future, we may rethink the strategy for targeting the scalability issue: Ideally, it would be nice if we call to each strategy once, passing a list of test cases that correspond to that strategy. Unfortunately, TestRunner is not capable of returning us the coverage per test. (That's why we currently have to call test runner  once per each test.
Regards
Matias

Hi André

Hi Matias

So, If I well understand, now we have one test strategy associated to each test (here) instead of being associated to the test suite (all the test). Then, for each test, the corresponding strategy is executed (here).

Small correction: we have one test strategy associated to each TestInformation, which points to a test class and one or more methods. Then, we execute iterate over the list of those objects, and execute all the methods inside each of them at the same time.

This strategy is fine for me. However, in the future, we may rethink the strategy for targeting the scalability issue: Ideally, it would be nice if we call to each strategy once, passing a list of test cases that correspond to that strategy. Unfortunately, TestRunner is not capable of returning us the coverage per test. (That's why we currently have to call test runner  once per each test.

For sure. test-runner actually already supports this, and it would be quite easy to do the change but there is a problem with same named methods form different classes (see https://github.com/STAMP-project/test-runner/issues/86#issuecomment-867439084). Once that is fixed, the change to running everything at once is easy enough, as we only need to change TestInformation and adapt the test-runner calls inside of each strategy.

Small correction: we have one test strategy associated to each TestInformation, which points to a test class and one or more methods. Then, we execute iterate over the list of those objects, and execute all the methods inside each of them at the same time

Thanks for the clarification :)

Ready for review as STAMP-project/test-runner#99 has been merged.

Ok, this one can be merged now.


export quirrel-dev function

Coverage increased (+2.6%) to 80.229% when pulling 37248f1836fa838ce6fe4af0f28cbb85d18daf53 on export-quirrel-dev-function into c6cb5693ecd4be888c43dfa0bdc12ce683c9fd75 on main.


add deprecation warning for docker

add deprecation warning for docker in container runtime section

pls check CLA

Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please follow instructions at https://git.k8s.io/community/CLA.md#the-contributor-license-agreement to sign the CLA.
It may take a couple minutes for the CLA signature to be fully registered; after that, please reply here with a new comment and we'll verify. Thanks.

If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address.  Check your existing CLA data and verify that your email is set on your git commits.
If you signed the CLA as a corporation, please sign in with your organization's credentials at https://identity.linuxfoundation.org/projects/cncf to be authorized.
If you have done the above and are still having issues with the CLA being reported as unsigned, please log a ticket with the Linux Foundation Helpdesk: https://support.linuxfoundation.org/
Should you encounter any issues with the Linux Foundation Helpdesk, send a message to the backup e-mail support address at<EMAIL_ADDRESS>

pls check with CLA

/check-cla


👻 Adds Fantom Testnet support

Adding support for Fantom Testnet and fixing a test case from the previous commit adding Fantom support.
I'm not sure how to handle/where to find the multicall address missing in my commit.
Any help/tips are appreciated!

Continued in #414


services/{bridge,compliance}: delete

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
services/friendbot, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.
[ ] I've updated any docs (developer docs, .md
files, etc... affected by this change). Take a look in the docs folder for a given service,
like this one.

Release planning

[ ] I've updated the relevant CHANGELOG (here for Horizon) if
needed with deprecations, added features, breaking changes, and DB schema changes.
[ ] I've decided if this PR requires a new major/minor version according to
semver, or if it's mainly a patch change. The PR is targeted at the next
release branch if it's not a patch change.

What
Delete the bridge and compliance services, and the internal package that contained code shared by only those two services.
Why
The bridge and compliance services were built to facilitate SEP-3. The SEP was deprecated in December 2019. We don't seem to be making changes to bridge and compliance and from what I can tell they aren't being actively supported anymore.
We're making changes to the SDKs in the monorepo and bridge and compliance both need updating, but that effort seems unnecessary given the services aren't otherwise being maintained.
Known limitations
N/A

If this PR is approved for merging I will extract the two services at the commit just before this is merged into a single repository named bridge-compliance at https://github.com/stellar-deprecated and retain their Git history in that extraction as was done with other projects.

We still have these developer docs talking about SEP-3, so maybe SEP-3 is not really deprecated. @rice2000 Can I delete this docs page too?
https://www.stellar.org/developers/guides/compliance-protocol.html
Also, this website is still alive supporting compliance, who runs that?
https://gostellar.org

I think @brahman81 knows about https://gostellar.org ?

@tomerweller Gave me a 👍off Github.

@leighmcculloch that page already has a deprecation notice and the new docs are about to go live. So I wouldn't bother. However, @rice2000 if the new docs will take a while, maybe it's worth removing the link to the compliance page from the navigation pane.

The navigation page is generated by a fairly gnarly script, so figuring out how to delete this may take some digging, but 'll look into it.  If it's not a rabbit hole, I'll do it.  If it is, the deprecation warning may have to suffice.  I don't think this page is many views at the moment.

For the sake of these deprecated services being easily accessed and revived by anyone else who might be interested in them I have extracted the Git history for those services from this repository and made them compilable in isolation over at https://github.com/stellar-deprecated/bridge-compliance.

Ingenious.


Infinity values showing as null

Hello guys,
I just noticed properties of components with value of Infinity are showing as null when inspecting component. When I send $vm to console, I can see its actual value is Infinity.
See screenshot:

The good news is it's just in the inspector, the underlying object still contains the right value.

Thank you, @yyx990803!


Update deps due to missing license file

Hi,
would it be possible to update two dependencies of this project because of a missing license file in NPM packages?
Those packages are:
<EMAIL_ADDRESS>can be upgraded to v1.0.1
<EMAIL_ADDRESS>can be upgraded to v3.0.0
Both of the packages can be upgraded without any additional code changes.
Thanks

@mjackson hey, is there a reason we can't update value-equal to v1? Happy to do the work myself.
Just asking as we also depend directly on value-equal for something else and we're getting warnings about using two versions. We can downgrade to 0.4 if there's good reason to but would love to move forwards to v1 if possible. I'm just not sure of the context of this lib + why it's not on the newest version. Thanks!

Hey, sorry this issue was closed. I'm not sure why I did that. Happy to accept a PR that updates to the latest versions with license files.

I'm giving this a shot a sec...


Single file upload support

I am trying to
const uploader = new FineUploaderTraditional({
  options: {
    request: {
      endpoint: '<redacted>',
      clockDrift: 500
    },
    validations: {
      itemLimit: 1
    },
    autoUpload: false
  }
})

<FileInput name={name} uploader={uploader} />
{fileId ? <Filename id={fileId} uploader={uploader} /> : null

uploader.on('onSubmitted', id => {
      // This method sets this.state.fileId
      setFileId(id)
})

I tried submitting 3 files. However there are problems with it

The <Filename /> code does not update the name of the latest file to be submitted
When running uploader.methods.uploadStoredFiles() when submitting, it list all the 3 files as stored, even if I sent the validations.itemLimit option to `.

Do I have to manually clear stored files using uploader.methods.clearStoredFiles(). If yes, then where do I need to do this?

The filename component needs to be updated to re-render when the id prop changes.

The filename component was not designed with reuse in mind. You’ll need to update it such that the name is recompiled when the id changes.

Ok got it.
Is there a way to restricting uploads to only one file? I know there is multiple attribute but will multiple=false work?


fixed decode-and-evaluate-device-id

args.decode_and_evaluate_device_id is not a list
Pull Request Checklist

[X] Changes are complete (if posting work-in-progress code, prefix your pull request title with '[WIP]'
until you can check this box.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Thanks!