[center] and [right] bbcode align tags in RichTextLabel causes text to overlap itself

Godot version: 3.2.1 (stable release)
OS/device including version: Windows 10 Pro Version 1809 (build 17763.1217)
Issue description:
When using either the [center] or [right] align tags in a RichTextLabel, it will sometimes cause the last line to overlap the previous one under certain conditions. Both the length of the bbcode text string (after tags are parsed) and the size of the Rect are variables in causing this to happen. Additionally, the two tags do not act like their alignment counterparts in a regular Label, as seen in the attached screenshot:

RichTextLabel on the left, plain Label on the right. Along with the overlapping issue, the RichTextLabel [center] is not a true center as displayed in the regular Label; it appears to be left aligned and simply shifted toward the middle.
Steps to reproduce:
-Create a RichTextLabel
-Enable BB Code
-In the BB Code Text field, type a string preceded by either the [center] or [right] tag
-Adjust the size of the Rect until the displayed text overlaps itself
Minimal reproduction project:
RichTextLabel Overlap Bug.zip

Can you test with 3.2.2 beta 2? https://godotengine.org/article/dev-snapshot-godot-3-2-2-beta-2

Can you test with 3.2.2 beta 2? https://godotengine.org/article/dev-snapshot-godot-3-2-2-beta-2

Tested with v3.2.2 beta 2, and it looks exactly the same; not seeing any improvements or changes from v3.2.1

Found the bug, will be making a PR soon.


ENSCORESW-2695: expand methods for REST support

Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion;
Review the contributing guidelines for this repository; remember in particular:

do not modify code without testing for regression
provide simple unit tests to test the changes
if you change the schema you must patch the test databases as well, see Updating the schema
the PR must not fail unit testing

Description
New fetch_all_by_name method, used for REST endpoints.
Use case
Some REST endpoints retrieve information on biotypes by their names. The original implementation uses straight SQL queries. Replacement methods have now been implemented on the BiotypeAdaptor.
Benefits
Some direct SQL removed, makes it easier to protect the REST endpoints from schema changes.
Possible Drawbacks
Some scenarios might not be tested for
Testing
Have you added/modified unit tests to test the changes?
Yes. Added some tests in the biotype.t
If so, do the tests pass/fail?
Yes
Have you run the entire test suite and no regression was detected?
Yes.

Coverage increased (+0.01%) to 82.457% when pulling e69b49d8e60ebe61d61978bb356a2d6d2507825f on feature/biotype_methods into 71dab88b9fc7b25da994668360ab29102e4a15b2 on master.

Seems alright, good to catch a parameter potentially being undef.


FEATURE - Add change-password command to broker/cli

Description
This PR adds a sparkswap wallet change-password <symbol> command to allow the user to change the passwords for a given wallet.
Related PR: https://github.com/sparkswap/lnd-engine/pull/209 (already merged and version is bumped in broker)
Related PRs
List related PRs if applicable
Todos

[ ] Tests
[ ] Documentation
[ ] Link to Trello

 Change Password RPC for broker


MIT License and copyright

Hi.
I'm glad you like Nodepub and I appreciate the very positive comment in the README file.
It's MIT licensed so you're free to do what you want with it, obviously.  That said, the MIT License is only almost a total giveaway of rights - the main right that is reserved is the copyright one.  I'm sure it's an oversight but the license does state "The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software."
The license file in your repo only has your copyright on it, whereas the code prior to your changes was under my copyright and according to the license that needs acknowledging.  The custom when building on existing MIT code is to include multiple copyrights where necessary and fortunately in this case there is a clear dated cut-off so an easy solution would be something along the lines of the following:

MIT License
Copyright (c) 2022 Mohanad Hesham
Copyright (c) 2019-2021 kcartlidge
...
As I say, I'm sure it's an oversight.  I'd greatly appreciate you updating it when you get a moment. Thanks!

No worries; I presumed it was just an oversight and didn't take offence :)
On Fri, 14 Jan 2022, at 13:21, Mohanad Hesham wrote:

Hi, I am sorry I don't have enough knowledge of the open-source community.
I will update the LICENCE file with the noted changes.
And thank you again for your awesome work! 😄
—
Reply to this email directly, view it on GitHub https://github.com/mohannadize/nodepub-lite/issues/1#issuecomment-1013111549, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC7OUEOSJ4IY6SG7G64GIEDUWAPPNANCNFSM5LP3STKQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID: @.***>


Zuul seems not to be peer aware

If zuul connects to a standalone eureka server like in the following configuration, then the forwarding to my microservice works fine.
server:
  port: 8900
eureka:
  client:
    service-url:
      defaultZone: http://eureka:admin@localhost:8761/eureka

But when zuul connects to a eureka server cluster like in the following configuration
server:
  port: 8900
eureka:
  client:
    service-url:
      defaultZone:
         http://eureka:admin@localhost:8761/eureka,http://eureka:admin@localhost:8762/eureka

then i get the following error

Am i missing something in the zuul configuration to work with a cluster?

Can you provide the stacktrace?

Please don't cross post http://stackoverflow.com/questions/40218630/enableswagger2-forces-microservice-to-register-as-unknown-to-registry

I think I have the same issue. See my question on stackoverflow
I think this is because the timeout, but I don't quite sure how to configure it. Any clue?

@timpamungkas your problem is not related to this issue.  Please don't cross post between SO and GitHub, use one or the other.


Update typescript from 3 to 4

TSDX uses typescript version 3.
Many new features are available in version 4. Please update to the latest version.

TSDX uses typescript v4

If you use for example use Template Literal Types, for instance;
type World = "world";

type Greeting = `hello ${World}`;

Compilation fails since it is a TS 4.1 feature.

@tim-phillips That's not correct. At least with yarn. Rollup gets to use ts 3.9. I have a script that removes that typescript from  local node_modules of tsdx. That makes it use ts 4 installed. Something like this:
"prebuild": "rm -rf node_modules/tsdx/node_modules/typescript"

There are a number of other issues in this repository about the super-old TypeScript 3.9.9 inside<EMAIL_ADDRESS>causing problems for people, where this is documented:

Yarn Resolutions or npm Overrides: https://github.com/jaredpalmer/tsdx/issues/1149#issuecomment-1272043961
pnpm Overrides: https://github.com/jaredpalmer/tsdx/issues/926#issuecomment-882102507

I think this repo is abandoned by @jaredpalmer

Using a fork weiran-zsd/dts-cli.


[release-2024a] sync the release-2024a branch with upstream opendatahub 2024a branch

[release-2024a] sync the release-2024a branch with upstream opendatahub 2024a branch
Automation failed: https://github.com/red-hat-data-services/notebooks/actions/runs/9211483733/job/25340954965

Merging to get the builds ready.


Overworld scene should be cached for instant loading

The overworld scene should be cached for instant loading.
There is a branch at https://github.com/Poobslag/turbofat/tree/instant-puzzle-launch-time where we attempted this for the puzzle scene, but it introduced some bugs and is incomplete.
This will also necessitate a way to populate the overworld with the default moving creatures. These creatures should follow arbitrary paths instead of having multiple scripts. For example, 'Run toward (150, 235) until you get there... then return to (0, 446)... then repeat'

This idea seems very error prone. After adding scene transitions the delay isn't noticeable anymore. Closing.


Declare both formats of crypto [pseudo]randomBytes

randomBytes and pseudoRandomBytes have synchronous and
asynchronous versions determined by arguments. If a single argument is
passed, a Buffer is returned synchronously. If a second function
argument is passed, the Buffer is asynchronously passed to the
function argument.

I haven't tested this, but I think the single-parameter declaration needs to come after the two-parameter declaration, otherwise any invocation will match the former and return a Buffer, even when the second parameter is a function matching the signature specified in the latter.

Aha, I didn't realize that. I will test it both ways.

@samwgoldman Actually neither way seems to get the desired response from Flow. It looks like Flow combines the signatures with union types rather than selecting a specific signature, so it's happy if you treat the return value as void | Buffer regardless of the arguments.
I tried the signatures in both orders.

It also seems to combine the arguments into a single, union'd signature in a way that I can't quite comprehend. pseudoRandomBytes(1) is considered legal, but so is pseudoRandomBytes(1, 1).

@ssorallen Hmm, I think you're right. There's an unfortunate interaction between intersection types (the natural representation of overloads), and Flow's treatment of functions (which is in line with JavaScript's runtime semantics).
It's a bit involved to explain, but I'll do my best here. Consider the following code:
function f(a, cb) {
  if (typeof cb === "undefined") {
    // just return the number
    return a;
  } else {
    // callback
    cb(a);
  }
}

We'd like to declare this function as returning a number given some arguments. If the a is a number, no second argument is required. Otherwise, we must pass b, which converts a to one.
declare function f(a: number): number;
declare function f(a: number, cb: (a: number) => void): void;

The declaration above is the same as the following intersection type:
declare var f: ((a: number) => number) & ((a: number, cb: (a: number) => void) => void);

When Flow sees a function call, like f(0, 0), it tries each branch of the intersection type in order. If there is an error, Flow will catch it and try the next branch, until we're out of branches. If every branch errors, Flow outputs an error. Flow stops once it finds a function that applies without errors.
In the first branch, we try to apply (a: number) => number with arguments 0 and 0. This is "too many" arguments, but JavaScript allows this and Flow models this behavior of JavaScript. However, this means we stop here and Flow thinks there is no error, but there absolutely is (bug).
If we reversed the branches, we'd try the form with a callback first, and flow would realize that 0 does not inhabit (a: number) => void, so we'd fall back to the second branch, which would succeed (same reason as above).
So, reordering doesn't actually buy us anything here. My bad.
I think this is a bug in Flow, and unfortunately a blocker for this PR. We've kicked around the idea of being strict about argument length when processing intersection/union, which might work. Another idea might be to have Flow look at all branches of an intersection/union instead of myopically, one at a time.
@jeffmo We've had this discussion before, but this is the first time I've realized our current behavior makes typing certain functions impossible. Is this on anyone's radar internally?

Thanks for the thorough explanation, @samwgoldman.
It does seem valuable to merge this still and be lenient on the signature rather than enforce an incorrect signature. Without this change, Flow shows an error for pseudoRandomBytes(10), for example, which is a supported call by the API. A false negative seems better than a false positive.

One way to force Flow to be strict on function arity at calls is to have the function take an extra parameter of type void. Then only undefined args can be passed to such a function.
In this particular case it actually matches the code, too! :) Let me know if that works.

@avikchaudhuri Smart! Now why didn't I think of that... :P

@avikchaudhuri Clever! I'll try that.

The trick unfortunately doesn't work because adding a second parameter makes the functions have the same arity.
declare function foo(size: number, UNUSED: void): Buffer;
declare function foo(size: number, callback: () => mixed): void;

foo(5, () => 5);
> ✗ Error arrow function. This type is incompatible with number
> ✗ Error number. This type is incompatible with undefined.

So you could add another bogus argument to ensure the declarations have different arity, but this seems rather goofy.
declare function foo(size: number, UNUSED: void): Buffer;
declare function foo(size: number, callback: () => mixed, UNUSED: void): void;

foo(5, () => 5);
> ✓

What do you all think?

@ssorallen I just tried your first example and I didn't get the same error, nor can I see why you would have gotten the error you did. Can you double check that?

@samwgoldman You're right. The test I was trying wasn't correct. The error from Flow is unfortunately difficult to understand with the intersection types, but it is correctly returning an error.
lib.js:
declare function foo(size: number, UNUSED: void): string;
declare function foo(size: number, callback: () => mixed): void;

test.js:
const fakeStr = foo(10, function() { return false; });
if (fakeStr.length > 5) {
  console.log('howdy');
}

flow check output:
test.js:1
  1: let fakeStr = foo(10, function() { return false; });
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ function call. Function cannot be called on
  1: let fakeStr = foo(10, function() { return false; });
                   ^^^ intersection type

@samwgoldman check out the diff again when you get a chance. I pushed an update.

@ssorallen Great! Looks good to me, although a FB employee will need to do the merge.
You're totally right that the error message is hard to understand. I got into a lot more detail in a comment for another PR, if you're curious about that behavior.

@facebook-github-bot shipit


Files is Unable to Drag & Drop Files Into Folder Within Same Directory

Description
FIles is unable to use drag and drop to move files into a folder that is in the same directory as the file being moved.
Steps To Reproduce

Create a File
Create a Folder (In the same directory as the previously created File)
Attempt to drag the File into the Folder

Expected behavior
Files should be able to move the file into the folder without issues.
Files Version
Version: <IP_ADDRESS> OS Version: 10.0.22000.469
Windows Version
Windows 11 22000.469
Relevant Assets/Logs

Can you add the log files?

debug.log
debug_fulltrust.log

It seems that the list item wasn't selected when they tried to drag & drop to the destination folder, I might try to look into this.

Update: I think something is wrong with the binding in AllowDrop, removed the binding and that fixed it.

@gave92 is this still an issue?

This was recently fixed.


Difficulty deploying template to gitlab pages due to project name in the url

I got GitLab to build the project using this config. .gitlab-ci.yml
image: node:latest

pages:
  script:
    - npm install
    - npm run build
  artifacts:
    paths:
    - public
  cache:
    paths:
      - node_modules
  only:
  - master

Gitlab Pages deploys projects as http://username.gitlab.io/project-name/.  When visiting https://cyprias.gitlab.io/elderjs-app/, the project's index.html shows up without any of the css and links point to http://cyprias.gitlab.io/xyz instead of http://cyprias.gitlab.io/elderjs-app/xyz.
When setting settings.prefix to elderjs-app, the http://cyprias.gitlab.io/elderjs-app/ link becomes a 404 error. Instead the webpage is accessible at https://cyprias.gitlab.io/elderjs-app/elderjs-app/. The index.html is shown but without the css again as the css points to https://cyprias.gitlab.io/elderjs-app/_elderjs/assets/svelte-9e1a324b.css when it's now at https://cyprias.gitlab.io/elderjs-app/elderjs-app/_elderjs/assets/svelte-9e1a324b.css.
Setting settings.origin to https://cyprias.gitlab.io or https://cyprias.gitlab.io/elderjs-app doesn't seem to have any effect, with or without prefix having the project's name in it or not.
Any assistance would be greatly appreciated.

@Cyprias It sounds like you're basically asking to prefix the assets but not the output content. Hard to follow.
I haven't used Gitlab's hosting process but it sounds like it needs some nonconventional deploy settings. I know things work well with pages.dev, netlify, s3, etc. Sorry I can't be of more help.

I think that's accurate. The html needs to prefix assets/links to deal with GL adding the project's name to the url.
So that /assets/blaw.js becomes /some_prefix/assets/blaw.js.
In GL's snapper example, it uses npm run export -- --basepath $CI_PROJECT_NAME to provide the project's name to snapper's export script. Is there a similar type of parameter that can be provided to elder's build script?


Patch 1

fix for icon ask

:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/resources/design/framework/submitting.md
:warning:Warning

Details

msteams-platform/resources/design/framework/submitting.md

Line 129: [Warning] Invalid file link:(~/resources/design/framework/bots).
Line 193: [Warning] Invalid file link:(~/resources/design/framework/compose-extensions).

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

:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/resources/design/framework/submitting.md
:warning:Warning

Details

msteams-platform/resources/design/framework/submitting.md

Line 130: [Warning] Invalid file link:(~/resources/design/framework/bots).
Line 194: [Warning] Invalid file link:(~/resources/design/framework/compose-extensions).

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

@NormSohl @charlesprakash Consider verifying your relative URLS in some of these topics. Many are missing the ~/msteams-platform/ and throwing build errors. Thanks


Question: How to position labels (in radar) without making chart wider?

Hello. I have a radar chart with three axis. The bottom two axises, the labels are drawn outside o the chart area making the chart wider than it needs to be. I want the labels positioned underneath the chart so that the chart isn't wider and can thus be made larger to accommodate my layout.
This is what it currently looks like:

This is what I'd like it to look like:

This may be related to #6549 but in my case, I want much simpler implementation as I do not require interactive labels.

This is not currently possible (afaik). Flagged as enhancement instead.


Make CSV output in schedule.createEvents() optional

To verify:

run scripts/make_calendar_pdf.py --> this should produce the PDF only, not the CSV.

run civilite/schedule.py --> this should produce the CSV file as before.

Nice work


Lints table doesn't seem to work with doctests

Problem
Lints defined in the lints table doesn't seem to work with doctests, whereas lints defined as an attribute do work. This seems inconsistent, as I would expect them to behave the same.
Steps
Example rust file:
/// Test
///
/// [`Foo`]
///
/// ```
/// let s = "rust";
/// let ｓ_ｓ = "rust2";
/// ```
pub fn f() {}
pub const Ě: i32 = 1;
pub const Ĕ: i32 = 2;

With manifest:
[package]
name = "foo"
version = "0.1.0"
edition = "2021"

[dependencies]

[lints.rust]
confusable-idents = 'allow'

Running cargo check -Zlints produces no warnings:
    Checking foo v0.1.0 (/Users/eric/Temp/foo)
    Finished dev [unoptimized + debuginfo] target(s) in 0.04s

Running cargo test --doc -Zlints produces warnings:
   Compiling foo v0.1.0 (/Users/eric/Temp/foo)
    Finished test [unoptimized + debuginfo] target(s) in 0.02s
   Doc-tests foo
warning: found both `Ě` and `Ĕ` as identifiers, which look alike
  --> src/lib.rs:11:11
   |
10 | pub const Ě: i32 = 1;
   |           - other identifier used here
11 | pub const Ĕ: i32 = 2;
   |           ^ this identifier can be confused with `Ě`
   |
   = note: `#[warn(confusable_idents)]` on by default

warning: 1 warning emitted

running 1 test
test src/lib.rs - f (line 5) ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.06s

If you put #![allow(confusable_idents)] at the top of the file, then no warning is generated.
Possible Solution(s)
I think lint flags should be passed to rustdoc here when building the rustdoc comand line. I'm not sure if there was a reason this wasn't added.
Notes
No response
Version
cargo 1.73.0-nightly (d78bbf4bd 2023-08-03)
release: 1.73.0-nightly
commit-hash: d78bbf4bde3c6b95caca7512f537c6f9721426ff
commit-date: 2023-08-03
host: aarch64-apple-darwin
libgit2: 1.6.4 (sys:0.17.2 vendored)
libcurl: 7.88.1 (sys:0.4.65+curl-8.2.1 system ssl:(SecureTransport) LibreSSL/3.3.6)
ssl: OpenSSL 1.1.1u  30 May 2023
os: Mac OS 13.4.1 [64-bit]

Huh, wasn't aware of this compilation pass since, for the most parts, lints don't apply.
In talking to ehuss, this is from cargo calling into rustdoc to extract doc comments.  The lint pass isn't being run but some lints are done in other passes anyways.  See also https://rustc-dev-guide.rust-lang.org/diagnostics.html#when-do-lints-run


Getting Invalid Day in Date when using moment.js in Mozilla Firefox

Hello! Found a issue when trying to use the .add(x, days) in Mozilla Firefox, it's really weird.
I Described the entire process in StackOverflow, take a look:
http://stackoverflow.com/questions/32573951/getting-invalid-day-in-date-when-using-moment-js-in-mozilla-firefox
When using a current time instance (var variable = moment()) it seems to work, but when using a ISO8601 coded date, it presents '2015-10-17' twice!
Any help apreciated :)

Answered on StackOverflow


Official, easy to deploy: docker/docker-compose setup

Greetings!
I'm seeking for an easy to reproduce setup for a docker/docker-compose environment. As we still can't find an objective and direct solution to have the main product docker setup with SSO/Keyckoak yet.
I came across many topics, discussions and references on that matter to no avail. And yes, I read this beautiful guide from @VonDerBeck , that did shine a light at the end of the tunnel. but ultimatley I wasn't able to reproduce the expected results.
There's a really simple and promissing attempt on getting such setup running easily by Gabepurnam. But it doesn't seem quite ready yet. Nor pressing like another BPMN product that actually has an official image/instructions for a quick docker deploy with keycloak support enabled and installed by default.
Yet I still find Camunda to be the best candidate solution for our team, and I really wish we had such solution for a quick deployment of the system with Keycloak as simple as the default camunda docker image.
Are there any plans to get it available for us anytime soon?

Hi
I put together this simple arrangement using the camunda-platform-run image here for my own purposes. It simply copies the required .jar and configuration files to the container.
You basically just need to edit the configuration files to point to your Keycloak server, and launch using docker-compose.
I hope this is useful to you.

Wow, thank you so much. This seems to be exactly the starting point I was looking for to demonstrate Camunda's possibilities. I'll give it a try later today.

@bowerta,
Thanks a lot. This is definitely useful to have a simple starting point with Camunda BPM Run. 👍


Cross platform support (i.e. Linux)

It would be great to be able to make this cross platform (especially since imagui is platform agnostic).
I believe this would involve two (fairly large) changes; moving from DirectX to OpenGL and moving from .NET to a cross-platform framework like QT.
It's always a shame to have open software restricted to windows

FWIW I tested with some Macbook .brd files I found and it seems to work well under WINE (1:1.6.2-0ubuntu1, on Kubuntu 15.10).

@pbhj Thanks for the tip. Full disclosure: I'm not actually using this software, but I saw it featured on youtube and was disappointed to find windows only software based on a lib that has put so much effort into supporting multiple platforms!
From further inspection it looks like this is based on imgui's directx9 demo, so it may not be too late to transplant it into the SDL/openGL boilerplate, the bigger challenge would be abstracting or removing all the .NET stuff and using cross platform filesystem (etc) APIs.

Somewhat done: https://github.com/piernov/OpenBoardView/commit/a5c53970b723e432ea1b9e52d830277434ad21db
(based on https://github.com/pafmaf/OpenBoardView/tree/slight-refac )
Now we have to make it build again on Windows, and add support for loading textures with the OpenGL backend. But i've no idea about the latter, no experience in OpenGL.

The reason I say to use Wine is because I don't have a proper build environment for OSX, and would have to spend time putting together Makefiles for OSX and for Linux. I'd also have to figure out how to replace GetOpenFileName, and I would have to either do ELF/Mach-O resource embedding or loose files for the two embedded assets the program depends on.  Transplanting the example ImGui code for other platforms is the easy part.
As an aside, there's no ".NET stuff" here, just Win32 stuff.

@piernov Very impressive.
Once I'd remembered how to git submodule update --init --recursive it worked like a dream, with the exception of textures on the pins as you pointed out (and a small bug I've fixed over in PR #1)
It might make sense for both the dx9 and opengl engines to make use of SDL, in theory this could allow OBV to use the SDL input events for (hopefully better) keyboard shortcuts - though I don't have any experience in either imgui or SDL, I get the feeling there may be some nice-to-haves tucked away in SDL.
@chloridite Thanks for the response.

I don't have a proper build environment for OSX, and would have to spend time putting together Makefiles for OSX and for Linux.

Yes, the build environment is a bit of a stumbling block, but @piernov has done a good job of setting up the CMake makefiles for linux, so all you really need now is VirtualBox and the OBV build dependencies.

and I would have to either do ELF/Mach-O resource embedding or loose files for the two embedded assets the program depends on

I assume these are bitmap assets? If so I'm not sure I see a problem with exporting bitmap files into the assets directory. If they are libs, why not the libs directory?

As an aside, there's no ".NET stuff" here, just Win32 stuff.

My mistake! You may be able to tell I'm unfamiliar with all this Windows stuff :)

If on the other hand you simply don't want to support platforms you don't use (perfectly understandable) many other projects will use a teamwork approach: one person handles windows and another linux.

Works on OSX too(with a few fixes on my osx-support branch) but I'm still trying to figure out how CMake/CPack works to make an app bundle and a DMG. Also it uses GTK3 for the open file dialog instead of the native OSX framework (I might try to implement that later). For now you have to build it the same way as for Linux, and install with homebrew all the dependencies needed (pkgconfig, gtk+3, sdl2 at least).
Btw, I used pafmaf's branch just because the code organization felt better.
Lastly, I know my branch is not suitable for a PR. If you have any advice for making commits suitable for a PR, please notice me.

Btw, I used @pafmaf's branch just because the code organization felt better.

From the looks of pafmaf@9a66e98f4dcf9637be4f7fefa395af6248dc3d1d it looks like he's planning to send a PR soon.

I'm still trying to figure out how CMake/CPack works to make an app bundle and a DMG

These wiki docs are the best info I can find on CPack package generators. Looks like it should be fairly trivial to support Windows NSIS installers, OSX archives and .deb and .rpm.
Apparently most of the docs are built in to CPack

From the looks of pafmaf@9a66e98 it looks like he's planning to send a PR soon.

Soon-ish, since I don't find much time. That branch is WIP. Originally I just wanted to have an abstract Board class and a listing of components but things escalated a bit. (I'll clean up the history before PRing.)
 Then it's good enough and usable for me personally.

Updated my linux-support/osx-support branches. Now it displays the texture for the pads, and OS X build doesn't depend on GTK3 since it now uses Cocoa for the open file dialog. Also fixed VS project files, but I couldn't get CMake to build the VS files, and I still have no idea on how to include the assets with the app bundle on OS X.

@piernov Good progress!
I've been trying to look into CMake and CPack. I've made some, since I don't have a Mac I've not tried building a mac bundle yet, but it looks like the Cmake files need a bit of a refacter, and the build.sh script is becoming more and more redundant.
In this branch I've been running
mkdir build && cd build
cmake -D CMAKE_BUILD_TYPE=DEBUG ..
make
cpack --config ./CPackConfig.cmake -G DEB
cpack --config ./CPackConfig.cmake -G ZIP

It's very WIP as I'm new to CMake, but though I may as well share in case anything helps.
Current known issues:

Does not link correctly (error while loading shared libraries: libimgui.so: cannot open shared object file: No such file or directory)
Only tested on .deb and .zip so far
Last commit was a "going to dinner" type so may be broken

Jumping in because I want to see this on linux (ubuntu 16.04 as of current).   Downloaded the slight-refac git branch but as soon as I try to run cmake I just end up with  "...does not appear to contain CMakeLists.txt"

@inflex this is probably a good place to start.
clone that branch and remember to git submodule update --init --recursive then just install build dependencies and run ./openboardview.sh
The reason is the slight-refac branch doesn't have @piernov's unix port, @pafmaf merged it into a different branch.

@MattSturgeon thanks for that, running nicely now in linux.  Couldn't get json11 to build out of the box but I cloned it off the dropbox git repos and it all seems fine now with the appending of the LD_LIBRARY_PATH to make it load up the libimgui.
Thanks, now I can get back to fixing broken electronics :)

@MattSturgeon  is there anything you need to have worked on for the linux release?  I do a lot of linux development, though I admit, it's all CLI tools/filters, rarely do I work within the GUI realm.

Since I couldn't figure out how to properly make a DMG with CMake, I ended up creating an XCode project. Now you have a (sort of) redistribuable OS X DMG here: https://github.com/piernov/OpenBoardView/releases/tag/v0.3-osx1

It would be nice to get to the point where we can use cake to generate any build system we need (msvs, makefiles, xcode...) automatically - after all that's what it's designed for. That way we only maintain one set of build system files.
Using CPack for DMG, RPM, DEB and exe installers would be a great plus too.
I've been struggling to work on this since I've not really worked with build systems properly before and I don't have access to a Mac or Windows machine currently

@MattSturgeon  not sure if I'm doing this completely wrong (oh wait, that would be exactly what I'm doing) but I noticed that in the linux build it doesn't support the letter-based keyboard shortcuts.  For some reason the shift-key requirement for things like R N L C K shortcuts just doesn't work, it's almost like SDL is remapping them back to lower-case letters?
BoardView.cpp.zip
I created a lower-case check with the uppercase in BoardView.cpp as well and now it works

@inflex Yeah I've been noticing problems the shortcuts system (though I'm not sure its issues are linux specific).
Interestingly most of the shortcuts still don't work on my system, I suspect this may be because I have a UK keyboard layout.
By the way, I stuck your changes here for safe keeping. If your interested I've also been doing some work and playing around with the build system and started work on embedding the assets.

@MattSturgeon can't imagine it'd be a UK layout issue, at least I wouldn't have thought it would be as they're not generating different keycodes scratches head
Right now I'm being horribly kludgy and working on a recently-used history list, because I'm lazy and don't like trying to filter through my folders reloading boards I used 3 sessions ago :)
Good luck with the build system, I think once that's pinned down and running smooth it should help a lot.  I still can't seem to get json11 to build as part of the script, have to do it manually, it's odd.

@MattSturgeon  I've finished the file-history change and I've also made the window title show the currently loaded board file.  Small things I know but I find it helps me when I'm switching around a lot.
The bad thing right now is that, well, I'm not sure how to properly offer these changes in to git, guess I better get learning on this too (sorry, ancient old school from the days of UNIX RCS )

@MattSturgeon have you noticed that OBV will chew up a lot of CPU when it's minimised/shaded?
I had to add an explicit sleep/do-nothing in main_opengl.cpp
uint32_t w;
while(SDL_getevent(&event)){
....
}
w = SDL_GetWindowFlags(window);
if ((w & SDL_WINDOW_INPUT_FOCUS)==0) { usleep(50000); continue; }
After that I went from my system dragging down with OBV hitting 60~70% CPU back to insignificant.

Sorry to keep bumping this thread, please let me know if I should make notifications elsewhere.
Anyhow, I've had a bit of a run of work here and have added numerous things I think might be useful for other users of OBV, including;

variably sized parts/pins display based on part geometry
Arbitrary angled bounding boxes around parts based on the minimal-bounding-box algorithm
Resistors/caps/diodes/inductors that aren't at 90/0 degrees are now displayed with more realistic angled box.

I still have to clean up the code, as it's been a real hack & slashfest while I sorted this out.
Attached is a screen shot demonstrating this.

@inflex would you make an executable file? thanks!! there´s a post were they added features of zoom in and out and dragging and dropping files, check them out and maybe put them together with your man! you rule!

@kikef333 I'll have to see about the zoom in/out, since that's already there (using mouse and num-pad, oh that's right yes, I added pan/zoom with numpad a few days ago).
The drag-drop I'm not sure about yet as I think a lot of that will depend on the actual desktop-management system OBV is run in; however, I have already added the ability to load via the command line (along with the history of course).
While I can make a binary on my machine I'm not sure if it'll run elsewhere properly (linux).   Are you on Linux or windows?

@kikef333 I just tested/checked, adding drag-drop support is a 4~5 line addition so that's a definite easy add on the SDL/Linux side.  Probably would want a confirmation dialogue though if you've already got a file open.
Anyhow, working out some issues with my minimum-bounding-box algorithm and I'll get back to adding the drag-drop.

@inflex https://github.com/charliebruce/OpenBoardView/releases/tag/v0.4

@kikef333 finished it all up about 10 hours ago :)   About the only thing I need to change on my fork to match will be the filtering of *brd files in the file picker dialogue, other than that, it's good to go.

@MattSturgeon how far apart is the linux fork from the Windows?  Is it practical/possible to keep them fairly parallel or do we think that it's going to degenerate in to two moderately separate builds?
Seems like things had to be reworked a fair bit to get the build to happen, sharing mostly just the loading/board-interpreter core?

Have registered #openboardview on Freenode IRC is anyone is wanting to have casual discussions without my posts clogging up the system

I personally don't have a problem with IRC but maybe some of the potential devs aren't too familiar with it / wouldn't bother to set things up.
So what about some alternative service like Gitter?
To sum up:

more beginner friendly UI in Web browser

no external client needed
unread messages etc.

login using your GitHub account
history is easily accessible
didn't try but it seems there's https://irc.gitter.im/ for interop
free up to 25 users (but I doubt there will be more active users than that)

I guess in this instance it was a case of greybeard habits, since I already have IRC on/open for other projects on Freenode.

@pafmaf Gutter sounds good, the irc integration sounds nice too.
@inflex as it stands in the Unix port, things are split up according to @pafmaf's restructure. @piernov split things further (into unix.h, osx.h, opengl-stuff and dx9-stuff) and added CMake to get things working cross platform.
I have been thinking about re doing the port in a less fragmented way, but I don't have as much time (or code familiarity) as I'd like.
It would be nice to get @pafmaf's PR #14 into @chloridite's main branch as (if I'm not mistaken) this is the bulk of the code refractor the current port is based on.(?)

@MattSturgeon any word back from @chloridite ?

@inflex I still need to send the email.
I've been distracted, focusing on different ways to do the port and trying to get CMake to build a working VisualStudio project. I've got a draft that uses GLFW instead of SDL, for example (though it looks like some of your changes are SDL specific?)

Well, if the port is buildable across platforms then it shouldn't be a lot of trouble to rewrite stuff, since the hard part was mostly the math/algorithms; the final rendering really only called on fundamental poly/circle/rect generation calls.   There was the drag-drop stuff which I'm not sure if OpenGL handles.
It'd only take me a day or so to re-port all my mods, though yes, it'd have been nice if we worked out how to magically make them in the current form :)

R7.3 has support for Linux (64bit RPM, DEB and compiling from sources), OS X (DMG for 10.9 and later I think) and Windows (32bit for Windows 7 and later).


[Upgrade] Remove staking, balances, treasury and demoncracy

cherry pick to 0.11.1

cherry pick to 0.11.1


Significance of the reported results

Hey, I was wondering how did you report the scores in table 1 of the paper, was it the accuracy over the test set for one trial or an average over several trials? Thanks in advance.

@ilemhadri should be the best person to answer this question.

Do you still need help with this?


add possibility to pass connection; make RedisDict generic

This PR does two things:

Add the possibility to pass a prepared connection object to RedisDict. This is meant to allow usage of Redis.from_url and other methods that could be used to create a redis object
Make RedisDict generic. This allows users to apply a contracts on their api with help of linters like mypy. One could do var: dict[str, ...] = RedisDict(...) as well, but then methods like the extend_type methods would be "lost".

I'm open to feedback. Let me hear what you think :)

I fixed the linting errors ✅

Great work.
The MR currently contains 2 changes: passing redis instance during init and typing improvements.
The feature is a great addition, as shown in these examples:
your example
dic = RedisDict(redis=redis.StrictRedis.from_url("redis://<IP_ADDRESS>/0"))

And this one
redis_already_part_of_project = redis.StrictRedis()
dic = RedisDict(redis=redis_already_part_of_project)

For typing there is still an ongoing discussion about what will be the best route for this project. A Discussion on the topic here.
So let's park the typing for now and focus on passing the redis instance.
Passing the redis instance can be advantageous in some scenarios,  such as: your example, testing, and using RedisDict within a project with an existing redis instance.
We just have to be sure the "redis" is not part of redis config, as it is currently possible to pass redis config:
dic = RedisDict(host='<IP_ADDRESS>', port=6380)

So if we could rescope this MR on passing redis instance during init. We can merge and release it:)


various GNOME packages: fix build with meson 0.61

Description of changes
https://hydra.nixos.org/eval/1751498?filter=aarch64-linux&compare=1751214&full=1#tabs-now-fail
Things done

[ ] Check if there are any stupid mistakes
[ ] Check if this is a duplicate of something
[ ] Check if there are some new upstream release

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@ofborg build gnome.devhelp gnome.gnome-books gnome.gnome-screenshot gnome-2048


Error converting columns in join

I have a sql where the field in select clause is null and I get following error
An exception of type 'System.InvalidCastException' occurred in Server.dll but was not handled in user code: 'Unable to cast object of type 'System.DBNull' to type 'System.Guid'.'

The sql has a join clause.
I think the columns on left joined tables should be wrapped as option.

It looks like all columns except Primary key are wrapped as option.

What is the best way to debug type provider?
I have checked out the Npgsql source code from here and in the solution created a new console app but when I reference the Runtime project in the console app the compiler can't find the referenced project and the console app fails to build.

Unfortunately this is postgres+ Npgsql limitation. There is nothing the type provider can do about it.

The issue I have is the query executes correctly but when I try to access the column value I get the above error. It looks like DataReader return it as System.DBNull but in the mapping in type provider it is not mapping it to None.

This breaks at runtime.

The type provider uses
https://github.com/demetrixbio/FSharp.Data.Npgsql/blob/fba9eafb14dc0e933f3da6f8fc6a9f12c4dc929e/src/DesignTime/InformationSchema.fs#L230
to get information about output columns including null-ability.
PostgreSQL engine doesn't know null-ability of derived columns.
See https://github.com/npgsql/npgsql/issues/1693 for details.
In case of left join PostgreSQL doesn't know that right part (person_address table in your case) becomes optional and reports columns coming from pa.* as non-null-able.
Run-time conversion from DbNull to non null-able type value fails.

Since PostgreSQL lacks the correct API to automatically report this nullability, and it causes a catastrophic runtime exception, then wouldn't it make sense to let the user manually force the option wrapping?
Eg. adding a AssumeNullable optional parameter to NpgSqlCommand / NpgsqlConnection.Create, which if set to true will make the type provider assign option types to all columns regardless of what GetColumnSchema returns.
Users will need to remember to set that flag whenever they using left join, and there will be some superfluous unwrapping for the main table, but at least it's a workaround.
The alternative workaround, I think would be to save the query in a view, for which PostgreSQL's always returns attnotnull = false, but that might not always be possible (say, if you don't control the database) and it could be a little burdensome to do every time you use a non-inner join.

Thanks for detailed analysis. Great job.
Let's hold off on adding AssumeNullable flag because to me it will turn F# codebase into Java/C#-pre v8 codebase where is anything can be null. At this point I would question value of the type provider.

Currently getting stung by this. The only user-side workaround appears to be modifying the query itself?

If there are no plans from the upstream, think we should somehow provide a workaround, as it crashes some of the queries during the runtime. I can't think of anything else other than all Optional flag unfortunately. :/

I think you could use COALESCE:
SELECT
    p.firstname,
    COALESCE(pa.id),
    COALESCE(pa.addr1)
FROM person p
LEFT JOIN person_address pa ON p.id = pa.person_id

Should infer to:
{ firstname: string
  id: Guid option
  addr1: string option }

No idea what affect this could have on performance for more complicated queries though.


partly addresses C4946 warnings

https://msdn.microsoft.com/en-us/library/y775w13y.aspx?f=255&MSPPError=-2147217396 violations

Thanks for this! Really appreciate the effort.


Database errors

`[08:37:02] [Thread-1337/WARN]: java.sql.SQLException: The prepared statement has been finalized
[08:37:02] [Thread-1337/WARN]: 	at org.sqlite.core.NativeDB.throwex(NativeDB.java:487)
[08:37:02] [Thread-1337/WARN]: 	at org.sqlite.core.NativeDB.column_count(Native Method)
[08:37:02] [Thread-1337/WARN]: 	at org.sqlite.core.DB.column_names(DB.java:731)
[08:37:02] [Thread-1337/WARN]: 	at org.sqlite.jdbc3.JDBC3Statement.getResultSet(JDBC3Statement.java:131)
[08:37:02] [Thread-1337/WARN]: 	at org.sqlite.jdbc3.JDBC3PreparedStatement.executeQuery(JDBC3PreparedStatement.java:82)
[08:37:02] [Thread-1337/WARN]: 	at com.oheers.fish.database.Database.hasFish(Database.java:76)
[08:37:02] [Thread-1337/WARN]: 	at com.oheers.fish.fishing.FishEvent.run(FishEvent.java:211)
[08:37:02] [Thread-1337/WARN]: 	at java.lang.Thread.run(Unknown Source)
[08:37:57] [Thread-1359/WARN]: java.sql.SQLException: The database has been closed
[08:37:57] [Thread-1359/WARN]: 	at org.sqlite.core.NativeDB.throwex(NativeDB.java:487)
[08:37:57] [Thread-1359/WARN]: 	at org.sqlite.core.NativeDB.column_text_utf8(Native Method)
[08:37:57] [Thread-1359/WARN]: 	at org.sqlite.core.NativeDB.column_text(NativeDB.java:249)
[08:37:57] [Thread-1359/WARN]: 	at org.sqlite.jdbc3.JDBC3ResultSet.getString(JDBC3ResultSet.java:437)
[08:37:57] [Thread-1359/WARN]: 	at com.oheers.fish.database.Database.hasFish(Database.java:81)
[08:37:57] [Thread-1359/WARN]: 	at com.oheers.fish.fishing.FishEvent.run(FishEvent.java:211)
[08:37:57] [Thread-1359/WARN]: 	at java.lang.Thread.run(Unknown Source)`

Fixed in a previous version


Wierdness updating text field when text is longer than field (minor issue)

Here is a text field from my simulation

After "Step Trial"

During Step trial you see a piece of the missing text appear and then disappear
Here is the field after mouse click in field

Here it is after a second mouse click

After 4 mouse clicks the full text is shown

this case should not happen with new rendering framework.  if see again, file a new ticket with reproducible example.


Drop-down menus no longer opening

Drop-down menus within cells/column headers no longer appear upon selecting the drop-down icon.

Hm.. it looks like there's been a core change to include the Bootstrap framework by default (see 181cc52a9f0c695d5e17d862ce1ffdc92e091a79 and fdb93118007d631885790039b344624cfd40c750).  I imagine that's conflicting with our own.  I think we'll will need to remove our own inclusion of Bootstrap and hope that doesn't break anything too much!

Hi @payten ,
The drop-downs are now appearing again, which is great. Quick formatting bit that I noticed - with the new native bootstrap, drop-down options are now appearing as underlined. Can we remove this formatting to clean up the drop-downs?

Looks good!


What is the meaning of the progress bars?

I can not understand what the meaning of the bars

@so77id  t indicated the importance of the references.


test: add tests for activation capability

:tada: This PR is included in version 0.22.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


How to debug push being sent but not received by app?

This official codelab says:

Run the updated application.
Hit the device's home button (or otherwise send the app to the background).
In Firebase console select Cloud Messaging from the Grow section of the left navigation bar. Select Send Your First Message. Set Message Text to "Friendly Chat?". Select the app we connected earlier as the App target. Click Send Message.
Confirm that the message is received and notification is displayed on the device. The user should receive a notification that takes them back to the application when tapped.

But I am not seeing the push, nor am I seeing the debug logs. I have put the google services json in the app directory and I'm even using the reference app (as opposed to the starter app). Further my console shows that the message was completed AND sent.
Any ideas?

I am having the the same issue.


Translate May 8 covid report into Italian

Happy weekend @nextstrain/italian,
The situation report from 2020-05-08 is ready to be translated to Italian. This week's report is short and sweet and focuses on Asia. We tried to tie the trees in with other epidemiological info; I'm curious to hear what you think of it. As with other regional reports (and really all reports) if you don't have time or are not able to translate this report, totally fine. Just let me know.
We've been publishing weekly reports on covid for two months now! As such, we're taking time this next week to re-evaluate narratives and figure out how we can make these reports as useful as possible going forward. I'd welcome your feedback on the situation reports -- what you find useful, what you don't, how often you think reports should be published, how these reports help (or don't help) Italian speakers.
Branch to translate on: sit-rep_it
File to translate: https://github.com/nextstrain/translations/blob/sit-rep_it/narratives/translations_ncov_sit-rep_it_2020-05-08.md You can translate directly in this file, but please remember to push often as other people may be working on the translation!
Link to preview: https://nextstrain.org/community/narratives/nextstrain/translations@sit-rep_it/ncov/sit-rep/it/2020-05-08
Twitter post to translate:
“Want to learn more about outbreaks in Asia? This week's situation report uses #SARSCoV2 genomes to track the recent spread of #COVID19 in Asia. The report is now available in Italian. Thank you @, @, and @ for the rapid translation."
Thanks so much for your help -- it's always a pleasure to work with you. See below for standard translation stuff.
Best,
Cassia
Logistics
Please update the translations board as you work on the translation: https://github.com/orgs/nextstrain/projects/7
Please assign yourself to the translation if you’re working on it this week.
When a translation is ready for publication -- at minimum meaning two people have worked on it and both agree that it’s ready -- please create a pull request to merge to master and tag me and the issue in it!
Discussion board for your team is here: https://github.com/orgs/nextstrain/teams/italian
Standard rules for translation:
Note that the file to translate is in Markdown. In general, it is important not to change any formatting or links. (Although it is okay to update Wikipedia or CDC links to the appropriate language version.) Comments to help with translation are noted in comment symbols like <! -- Comment -->.
Please add your name under translators: with new line, tab, and dash, formatted just like authors are. Please also add a link to your professional page (can use Twitter) under translatorLinks:.
Note: You may know of other great resources available in Italian; please don’t add them to the situation report. We need to keep content the same across all translations, so we can be confident in the accuracy of what we publish. Thanks for understanding.
At minimum, I require two sets of eyes on every translation; one of those people should have a background in phylogenetics. Although if you have more, that’s fine too.
Feel free to use a live, online editor like hackmd (sign in with GitHub) to translate together.

Hi all,
this week I can work on the translation a lot, in order to compensate last week absence!
@al-epo let me know if you want a "free weekend"!
Best wishes!
Sara

That's really nice of you @sarameiqi !
I could do the Scientific  credits  and the what you can do  paragraph.
I will use the extra time to prepare some experiments for kids.  I am part of this https://www.scientist-next-door.org/  ( in case you want to give a look) :-)

Hi all,
I finished the translation. I have a few doubts on line 176, always "clade", translated as "ramo", but I'm not sure.
From What you can do (included) on, it still needs translation.
If I have time I'll continue later. If @al-epo wants to do it, she's welcome!
Best

Hi! Wow!
Sure, I will start translating ting in around 30 mins. I will write when I finish

Hi all,
I finished.
@sarameiqi  I saw you didn’t  translate  patterns , it is completely fine for me. I also did not for line 234.
I am going  to read everything  now. So, I can also update myself on the situation in Asia. ;-)
Best,
Alessia

@al-epo, the Scientist Next Door project is really cool! I was actually homeschooled all the way through secondary school and something like this would have been so neat.

Hi,
I corrected  small typos. It is very clear.  Well done @sarameiqi !
@sarameiqi: sono curiosa (ed ignorante).  Mi potresti spiegare la regola dietro mettere Paesi e paesi ( senza la maiuscola)? Ho messo maiuscolo in una frase nei takeaways ma non sono sicura sia corretto. Grazie!

Volete che controlli?
On Sat, May 9, 2020, 16:17 al-epo<EMAIL_ADDRESS>wrote:

Hi,
I corrected small typos. It is very clear. Well done @sarameiqi
https://github.com/sarameiqi !
@sarameiqi https://github.com/sarameiqi: sono curiosa (ed ignorante).
Mi potresti spiegare la regola dietro mettere Paesi e paesi ( senza la
maiuscola)? Ho messo maiuscolo in una frase nei takeaways ma non sono
sicura sia corretto. Grazie!
—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nextstrain/translations/issues/145#issuecomment-626191226,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACW4WUMS4OLR7EWJOULDIZLRQVXZZANCNFSM4M4SMEUA
.

@al-epo Paesi indica Stati, entità nazionali, mentre paesi indica i paeselli, piccoli aggregati urbani. Quindi la maiuscola serve per marcare lo "status" che attribuisci alla parola. Anche se non è più una regola normativa forte, perché di solito le due entità si possono distinguere abbastanza bene dal contesto, sul piano stilistico preferisco sempre evidenziare la differenza, perché a volte ti trovi in contesti dubbi, in cui un uso sistematico della maiuscola-minuscola può garantire massima comprensibilità.

@sarameiqi : Grazie!
@aliceLedda : se vuoi/puoi certo :-)

Hello gals! What's the status of this translation?

Hi @cassiawag!
We completed the traslation. We were waiting to see if @aliceLedda could check it.
best,
Alessia

Hi @aliceLedda did you find any time to review this report?
Let us know!

@aliceLedda, any chance you'll be able to review this one? Since previous situation reports are now more accessible, I think it'd be useful to have it translated/

Sorry I just missed these emails...
I check in half an hour!
Alice
On Mon, May 18, 2020, 13:44 cassiawag<EMAIL_ADDRESS>wrote:

@aliceLedda https://github.com/aliceLedda, any chance you'll be able to
review this one? Since previous situation reports are now more accessible,
I think it'd be useful to have it translated/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nextstrain/translations/issues/145#issuecomment-630158032,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACW4WULEAFPIW4FS262BLPTRSEUUNANCNFSM4M4SMEUA
.

Ok checked!
It's fine with me, you can publish!
And sorry again for the delay!
Alice
On Mon, May 18, 2020 at 9:30 PM alice ledda<EMAIL_ADDRESS>wrote:

Sorry I just missed these emails...
I check in half an hour!
Alice
On Mon, May 18, 2020, 13:44 cassiawag<EMAIL_ADDRESS>wrote:

@aliceLedda https://github.com/aliceLedda, any chance you'll be able
to review this one? Since previous situation reports are now more
accessible, I think it'd be useful to have it translated/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nextstrain/translations/issues/145#issuecomment-630158032,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACW4WULEAFPIW4FS262BLPTRSEUUNANCNFSM4M4SMEUA
.

Thanks Alice! I'll get it published.

Published: https://nextstrain.org/narratives/ncov/sit-rep/it/2020-05-08!


Roadmap

@eduardoleao052 Just curious as to where you see the project going in the short term, medium term and long term. Realise this has mostly been a one person endeavour and you won't have lots of things figured out but thought I would pose the question and whether its worth thinking of a quick roadmap. You've mentioned that you want it to be accessible and low barrier for JS learners but would good to see your understanding of it as it stands now and what the future direction might look like.
See https://github.com/KnowledgeCanvas/knowledge/wiki/Feature-Roadmap - for examples.  Somethings relatively quick and easier to do.
You can also consider doing a github projects to help gather your thoughts.

@medic-code
That's super cool! Many of these tools you're pointing out I didn't even know lol
I'll take a look.
But to summarize, at least mid-term, I see this project as being able to fill a (not so small) niche of JS developers that are interested in AI (maybe have been following the GEN-AI hype, Devin and such) but do not have Python knowledge. So if these guys want to work with decent ML tools and run say demos or small apps on the web they can do it with this project.
I don't see it as a way to "replace" PyTorch and the bigger players, but it still could have a good role to play.
On another topic, did you see the new issue templates? I tried to add some more fields to the bug report, and kept the feature suggestion mostly the same. Is the result close to what you were imagining?

langchain-js's community is larger than you think for example! I would maybe start to break that down knowing your objective and what you think we should and should not implement going forward.
Few comments,

On the bug report it seems pedantic but instead of a small snippet of code, I would say minimal reproducible code that exemplifies the bug.

1 Sometimes worth providing a link for those who don't have a clue what this means - https://support.gurobi.com/hc/en-us/articles/13941323633681-Tutorial-Preparing-a-Minimal-Reproducible-Example - for example

Consider adding a category for actual behaviour next to expected behaviour - you could somewhat argue you provide this information in the description but it can be quite nice to see actual and expected behaviour next to each other concisely.

Add in optional screenshot section (or screengrab)- I try always to provide screen grabs in cases where the description in words is too verbose.

Add in a section for environment - Sometimes this can be important in teasing out bugs that are to do with the system
4.1 Operating system
4.2 Version of the repo or branch
4.3 Version of node.js
4.4 If applicable browser name and version

A check list for a bug report something like https://github.com/langchain-ai/langchainjs/issues/new?assignees=&labels=02+Bug+Report&projects=&template=bug-report.yml

For the feature request, maybe provide a check list like https://github.com/langchain-ai/langchainjs/discussions/4862

@medic-code
I think I expressed myself poorly, sorry! I've heard of langchain.js, the tools I wasn't really familiar with that I mentioned were building Issue Templates and Project Roadmaps.
By the way, I updated the Issue templates, if you want to take a look. Now they have input fields, and seem to be a lot better organized. Plus, I implemented the changes you suggested. If you think it could still be improved, let me know!

Issue templates are looking good no comments.

Thanks again for the tip!


[release/2.2] Update ProdCon to 20181110-02-2202235

Update to just-published https://github.com/dotnet/versions/tree/cae6079213809461b63e60ecd2b7b40e053d04fc/build-info/dotnet/product/cli/release/2.2.0.
Includes updates to CoreCLR, CoreFX, Core-Setup submodules.

My local build to get the baselines updated is halfway done.


fix a bug when token is in query string

when we use query string to pass the jwt token, if "next" is in the url, the token will be regarded as a substring of the value of the param "next":
/hub/login?next=%2Fhub%2F%3Ftoken%3DeyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJhdWQiOiIyMDE3MjExNTExIiwiZXhwIjoxNjE0MDk4OTI2LCJ1c2VySWQiOiIyMDE3MjExNTExIn0.AkchQQRtQ5v_a2t5smbT1scWU7TU48kf1VhhTWToqs8

In this case, the authenticator will not be able to get the token properly.
Also, we should not set a default value for header_name, cookie_name, and param_name. They should be set by the user, otherwise if we both have a header and a query, the authenticator will not work properly.
This PR fixes the above problems.

Wow, missed it. LGTM, let's give it a try

@JunweiSUN Release as 2.0.3 to PyPI


adding a !SUBSCRIPT! one as an example if pull...

the repository or update is done, then it presses any key and exits the cmd itself

Frick u

its only 1.3.3 can we close this pull?


⚠ Turnos 24x7 has degraded performance

In 828a4c6, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 813 ms

Resolved: Turnos 24x7 performance has improved in df29eca.


Fix false-positive in RUF027

Summary
It is a convention to use the _() alias for gettext(). We want to avoid
statement expressions and assignments related to aliases of the gettext API.
See https://docs.python.org/3/library/gettext.html for details. When one
uses `_() to mark a string for translation, the tools look for these markers
and replace the original string with its translated counterpart. If the
string contains variable placeholders or formatting, it can complicate the
translation process, lead to errors or incorrect translations.
Test Plan

Test file RUF027_1.py was extended such that the test reproduces the false-positive

Closes https://github.com/astral-sh/ruff/issues/10023.

Thank you! Will review.

(The ecosystem checks look good.)


ADDING: bindgen/gen_nim.py and related changes

Ok, merged! I'll play around with this a bit now.
Btw, I just remembered that there's also a separate sokol-nim-examples repository: https://github.com/floooh/sokol-nim-samples
...but I think we should add the Nim examples to the sokol-nim project, like I did in the sokol-zig project.

Oh and btw, I notice you're still on the master branch ;) It's really useful to put PRs in dedicated branches. That way old changes like this:
https://github.com/floooh/sokol/pull/487/commits/e4177472c8209d553135b7e61e2ccc87ec629449
...don't show up in the PR's changelist and also can't accidentally slip into the main branch when merging.

Do you have some example code to get started with? It seems the sokol_app.h integration works a bit differently than I expected :)
E.g. I was basically expecting this:
proc myInitFunc(): void =
    sg.setup(...)

proc myFrameFunc(): void =
    ....

proc myCleanupFunc(): void =
    ....

sapp.run(sapp.Desc(
    initCb = myInitFunc,
    frameCb = myFrameFunc,
    cleanupCb = myCleanupFunc,
    width = 800,
    height = 600
))

Also, wouldn't it make sense to automatically convert those "value-to-pointer" function wrappers here?
https://github.com/floooh/sokol-nim/blob/2bdc2d2be0d798527c832c96c1b882f3c69a1421/src/sokol/ext/gfx.nim#L110
...that's what I do in Zig too, the Zig functions take value arguments, and there are wrapper functions which call the exported C function with a pointer:
https://github.com/floooh/sokol-zig/blob/c0b820ede7978f3ec248c4ca301e3f6f2d84ec43/src/sokol/gfx.zig#L654-L656
...this is taken care of here where function arguments are treated differently depending on their type (struct pointers, char pointer, or "other"):
https://github.com/floooh/sokol/blob/4c56a2ee15d81cac91b738bb7e740c964dc6bd56/bindgen/gen_zig.py#L456-L466
Cheers :)

I have 3 samples converted to work with this: clear, cube, and blend.  I can put them in the sokol-nim repo following the pattern in sokol-zig.  I have removed some sapp freedom in favor of convenience and correct Nim behavior, but I think you have persuaded me to back off that a bit.  Also warming to the idea of generating the value->ptr functions.


Parse docstring to get arguments for builtin functions

As described by Alex Martelli here

can access the docstrings of these functions at fn.__doc__


Playback stop after a few seconds in background

Playback stop after a few seconds in background

Are you able to test with previous releases and determine if it was always an issue?

It seems fixed with latest update 022.
Still some buffering issues stoping pkayback when network connection is unreliable.

Fixed with 0.2.2


Upgrade generator to .NET 8

Motivation
Improved performance and new features.
Modifications

Upgrade all generator and unit test projects to target .NET 8
Bump to C# 12 for all projects
Upgrade various unit test dependencies
Remove NullableAttributes from Yardarm, they're all available OOB now
Fix some nullability warnings
Remove reference to Microsoft.SourceLink.GitHub, it's OOB now
Update SDK references
Update Dockerfile

/merge


JBPM-5391 & JBPM-5392: Commands API improvements and bug fixing & Redo/Refresh features added.

Hi,
The idea of JBPM-5391 is making stunner more robust by fixing and adding test for core commands. So made those ( which are portable on the server bus as well ) smaller, fix some already known bugs and basically add tests almost each one. See https://issues.jboss.org/browse/JBPM-5391
As working with the command API stuff, it was a quick win to add redo and refresh features -> See https://issues.jboss.org/browse/JBPM-5392
Please this older PR should be merged before this one, as it's the fix a the kie-wb-common build. This branch is not rebased with that commit but should not appear any conflicts, lemme know if any issues.
Thanks!

Tested and looks good.
+1

thanks for the review @csadilek ! Added new commit with the updates.


Update how-to-avoid-getting-throttled-or-blocked-in-sharepoint-online.md

Category

[ ] Content fix
[ ] New article
[x] Example checked item (delete this line)

DELETE THIS LINE BEFORE SUBMITTING | For the above list, an empty checkbox is [ ] as in [SPACE]. A checked checkbox is [x] with no space between the brackets. Use the PREVIEW tab at the top right to preview the rendering before submitting your issue.

Related issues

fixes #issuenumber
partially #issuenumber
mentioned in #issuenumber

DELETE THIS LINE BEFORE SUBMITTING | If this fixes (aka: closes) or references an issue, please reference it here. This helps maintaining the issue list as it will (1) link the PR to the issue & (2) automatically close the issue when this PR is merged in.

What's in this Pull Request?

DELETE THIS LINE BEFORE SUBMITTING | Please describe the changes in this PR. Sample description or details around bugs which are being fixed.

Submission guidelines

!!IMPORTANT!! - All submissions must complete the baseline sections included in this template. Ignoring or deleting this template may result in closing the issue with the label type:incomplete-submission.
Follow our guidance on How To Create Good Pull Requests.
Target the master branch of this repo. Released documents are in live branch.

DELETE THIS SECTION BEFORE SUBMITTING

Docs Build status updates of commit 44775d7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/general-development/how-to-avoid-getting-throttled-or-blocked-in-sharepoint-online.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit cc4785f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/general-development/how-to-avoid-getting-throttled-or-blocked-in-sharepoint-online.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Please use the PR form provided to explain what this change address and why it's being added.


Add sudo to curl bash one-liner

Signed-off-by: Ricardo Carrillo Cruz<EMAIL_ADDRESS>

https://github.com/open-cluster-management-io/clusteradm/blob/cc826fe8723ed3c3ea0ac9b52250c33252cb264d/install.sh#L7-L8
a better way of sudo installation is exporting the USE_SUDO env var, but the script seems incomplete. anyway i think we should fix the issue by providing sudo support from the script.

@rcarrillocruz do you think if https://github.com/open-cluster-management-io/clusteradm/pull/125 will help solve the issue?

@rcarrillocruz do you think if open-cluster-management-io/clusteradm#125 will help solve the issue?

It would, I lgtm'd that PR but I'm not a collaborator.
Will close this PR so it gets fixed on yours.
Thanks


Fixing links of the gallery

Introduction
As big image files were moved to AWS the links have to be updated to point to AWS.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 77.27%. Comparing base (80f9691) to head (a4fddc2).
Report is 237 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #189      +/-   ##
==========================================
+ Coverage   77.14%   77.27%   +0.13%
==========================================
  Files          35       35
  Lines        2459     2460       +1
==========================================
+ Hits         1897     1901       +4
+ Misses        562      559       -3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Understanding Graphics.MeasureFPS

Graphics has a property MeasureFPS that is set to true in the examples. I don't really understand what this does however.
In my application scenario, I don't care about knowing the actual FPS of the Overlay, but in order to reduce the required graphics power, I would like to limit the FPS (to maybe 30).
Do I need to set it to true or is it enough to set FPS on the GraphicsWindow?

just set the FPS in the GraphicsWindow.
They both run independent and MeasureFPS is there to assist you in displaying them.

Thanks for the explanation. I think I can get away with leaving the default values for Graphics then.


Add multi search api for ES 7.10

What
Add multi search api for ES 7.10. For the other version of elastic search, legacy (2.2) this returns a not supported response.
Trello Ticket: https://trello.com/c/KFRIgs4f/950-add-multi-search-method-to-elasticsearch-client-in-dp-elasticsearch
How to review
These changes are done according to the doc in: https://pkg.go.dev/github.com/elastic/go-elasticsearch/esapi#MsearchRequest
Who can review
Anyone except me

Hi Rahul, some general comments in addition to Jess comment.

Are we planning to add a Search() api in addition to MultiSearch() api in new library?  In a general library like dp-elasticsearch search which may be used by many services (not just dp-search-api), it may be good to have it?(also just fyi I ended up using the Search() api in the ES client in dp-search-api just to avoid the additional/confusing overhead for multi searches when only one was needed).
Are we planning to add a MultiSearch() implementation to the v2/client (it could be implemented relatively easily with code that exists dp-search-api)? I’m not sure how useful a client is without search functionality, and if we are not planing to add it, such as removing support for ES 2 altogether, we should probably remove it and the ability to create  a V2client from the main client.go file [NewClient(cfg client.Config)]
It would be good to consider adding component tests, there are very few tests. Could consider traditional component tests such as in dp-frontend-search-controller or the dp-mongodb library, or more ‘lightweight’ api tests such as in dp-dataset-api/dp-frontend-release-calendar

So to explain on these points

Currently the ticket only covers multisearch. If there is a requirement to add the search api in addition to multisearch, we can always create a ticket and work on it.
currently as the ticket suggests this will only be a 7.10 functionality and we will be switching to 7.10 very soon. So even if we add a functionality to 2.2 , not sure if it is going to add any value. But again going by the trello ticket only on this one.
Regarding the component tests, i am not sure how much value can the component test add here, as this is a wrapper/facade around the go-elasticsearch library and all it does is calling to the library. Yes definitely we can/should add component tests to the consumers, but not sure if it is needed to a library.

Hi Rahul, some general comments in addition to Jess comment.

Are we planning to add a Search() api in addition to MultiSearch() api in new library?  In a general library like dp-elasticsearch search which may be used by many services (not just dp-search-api), it may be good to have it?(also just fyi I ended up using the Search() api in the ES client in dp-search-api just to avoid the additional/confusing overhead for multi searches when only one was needed).
Are we planning to add a MultiSearch() implementation to the v2/client (it could be implemented relatively easily with code that exists dp-search-api)? I’m not sure how useful a client is without search functionality, and if we are not planing to add it, such as removing support for ES 2 altogether, we should probably remove it and the ability to create  a V2client from the main client.go file [NewClient(cfg client.Config)]
It would be good to consider adding component tests, there are very few tests. Could consider traditional component tests such as in dp-frontend-search-controller or the dp-mongodb library, or more ‘lightweight’ api tests such as in dp-dataset-api/dp-frontend-release-calendar

So to explain on these points

Currently the ticket only covers multisearch. If there is a requirement to add the search api in addition to multisearch, we can always create a ticket and work on it.
Okay thanks
currently as the ticket suggests this will only be a 7.10 functionality and we will be switching to 7.10 very soon. So even if we add a functionality to 2.2 , not sure if it is going to add any value. But again going by the trello ticket only on this one.
If there is no support planned for 2.2. I would probably recommend to remove it altogether from the library at some point. It will just be confusing to have it in the library as a separate package when it's not really supported.
Regarding the component tests, i am not sure how much value can the component test add here, as this is a wrapper/facade around the go-elasticsearch library and all it does is calling to the library. Yes definitely we can/should add component tests to the consumers, but not sure if it is needed to a library.
It's a reasonable point and the component tests for the lib could be more regarded as smoke tests, but without them there is actually no automated way of telling the library is actually callable. The only way is to do a manual smoke test, and this is not a good regression mechanism. Having said that am okay to let this pass, but will let Jess do it when you have addressed her points.


Add intro about MSTest

Internal previews

📄 File
🔗 Preview link

docs/core/testing/unit-testing-mstest-intro.md
MSTest overview

Thanks, I have applied the change @BillWagner


ImportError: No module named builtins

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock
example script provided in TensorFlow):
OS Platform and Distribution (e.g.,
Linux Ubuntu 16.04): Linux Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if
the issue happens on mobile device:
TensorFlow installed from (source or
binary): Pip3
TensorFlow version (use command below): 2.1.0
Python version: - Bazel
version (if compiling from source):
GCC/Compiler version (if compiling from
source):
CUDA/cuDNN version: - GPU model and memory:

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with: 1. TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)" 2. TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
Describe the current behavior

I am unable to complete the util test.
Describe the expected behavior
Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
bazel test ${flags} //tensorflow/python/keras
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.

@ayushmankumar7
As there is an existing PR monitoring the issue, proceeding to move this issue to closed status as it would be a duplicate for #37425


xilinx: use FDPE instances to implement get_async_ff_sync()

This closes #721 by implementing get_async_ff_sync() using FDPE primitives to obtain exactly the netlist that we want. This consits of a chain of N FPDEs (by default N = 2) with all their PRE pins connected to the reset for a positive edge reset or to the ~reset for a negative edge reset. The D pin of the first FDPE in the chain is connected to GND.
To make timing analysis work correctly, two new attributes are introduced: amaranth.vivado.false_path_pre and
amaranth.vivado.max_delay_pre. These work similarly to amaranth.vivado.false_path and amaranth.vivado.max_delay, but affect only the PRE pin, which is what is needed for this synchronizer. The TCL has been modified to generate constraints using these attributes, and there are comments explaining how to use the attributes directly in an XDC file in case the user wants to manage their XDC file manually instead of using the TCL.

I have tested this with the following example.
#!/usr/bin/env python3

from amaranth import *
import amaranth.cli
from amaranth.lib.cdc import ResetSynchronizer
from amaranth.vendor import XilinxPlatform

class Test(Elaboratable):
    def __init__(self):
        self.rst_in = Signal()
        self.toggle = Signal()
        self.toggle2 = Signal()
        self.rstdomain = ClockDomain()
        self.sync2 = ClockDomain()

    def elaborate(self, platform):
        m = Module()
        m.domains += [self.rstdomain, self.sync2]
        rst_in_q = Signal(reset=1)
        m.submodules.sync_rst = ResetSynchronizer(rst_in_q)
        m.submodules.sync_rst_maxdel = ResetSynchronizer(
                rst_in_q, domain="sync2", max_input_delay=3e-9)
        m.d.rstdomain += rst_in_q.eq(self.rst_in)
        m.d.sync += self.toggle.eq(~self.toggle)
        m.d.sync2 += self.toggle2.eq(~self.toggle2)
        return m

class MyPlatform(XilinxPlatform):
    device = "xc7z010"
    package = "clg400"
    speed = "1"
    resources = []
    connectors = []

if __name__ == '__main__':
    top = Test()
    platform = MyPlatform()
    amaranth.cli.main(
        top, platform=platform,
        ports=[top.rst_in, top.toggle, top.toggle2,
               top.rstdomain.clk, top.rstdomain.rst,
               top.sync2.clk, top.sync2.rst])

These are the constraints I'm using
create_clock -period 5 -name clk [get_ports clk]
create_clock -period 5 -name sync2_clk [get_ports sync2_clk]
create_clock -period 5.01 -name rstdomain_clk [get_ports rstdomain_clk]
set_false_path -to [get_cells -hier -filter {amaranth.vivado.false_path == "TRUE"}]

set_false_path -to [ \
  get_pins -of [get_cells -hier -filter {amaranth.vivado.false_path_pre == "TRUE"}] \
    -filter { REF_PIN_NAME == PRE } ]

set_max_delay -to [ \
  get_pins -of [get_cells -hier -filter {amaranth.vivado.max_delay_pre == "3.0"}] \
    -filter { REF_PIN_NAME == PRE } ] \
  3.0

The implementation schematic is as follows. Paths that are timed normally are highlighted in blue. Paths timed of a 3 ns max delay are highlighted in red. The paths that are not highlighted are not timed.

This looks correct to me. I haven't found detailed information about the timing characteristics of the FDPE, but my thinking is that if PRE is asserted then the output of stage1 will go high in a short time regardless of what is happening with the clock. If PRE is deasserted, the worst that can happen is that this happens close to a clock edge. In this case I think that the output of stage0 might go metastable, but the output of stage1 should be a stable 1, because when this happens, stage1 has a stable 1 at its input. At the next clock edge, hopefully the metastability of stage0 should have resolved to a stable 0 or a stable 1 (and if not, use a longer chain). If it has resolved to a 0, then the output of stage1 will pick the 0 in this clock cycle. If it has resolved to a 1, then stage0 will pick the 0 in this clock cycle, and stage1 will pick it in the next clock cycle.
Besides testing this example with the constraints shown above, I have deleted the set_false_path and set_max_delay constraints and checked that running the TCL commands that I have added generates the correct constraints.
A question to make sure I understand what async_edge="neg" means: my interpretation is that when the input is 0, then the output should be 1 immediately. When the input is 1, then the output should be 0 at the next clock edge. Is this right?
By the way, now that I have given some more thought to this reset synchronizer, I wonder how useful it is in a typical Xilinx design. Flip-flops are normally synchronous reset, so if the clock is off, they won't reset even if the reset is asserted (I guess that the main use case of AsyncFFSynchronizer instead of FFSynchronizer is to be able to assert the output even when there is no clock).

(I guess that the main use case of AsyncFFSynchronizer instead of FFSynchronizer is to be able to assert the output even when there is no clock).

Yes. It's only useful in a situation where a clock domain has async reset.


Mathjax review request: CRPS for Ensemble

The current formula could be clearer about the term (1 / 2 * K) whether it is 0.5K or 1/2K.

I think it should be 1 / (2 * K)
(1 / 2 * K) is ambiguous, but in most context is equivalent to (1 / 2) * K = 0.5 K. This is not the same as what's in the implementation.
The code uses 1 / (2 * K), which is consistent with the formula from one of the references:

with K = M ** 2  # ecdf variant.

Thanks @nikeethr I've put up a PR which I think results in a much clearer formula, but if you could check it that would be much appreciated.


[Bug] HTMX cannot work in history

https://github.com/laike9m/Python-Type-Challenges/assets/52167622/a9b6d16a-8b18-4093-8ceb-bb37fcf2a236

Yep, definitely a bug


Chromecast for Youtube Music

Checklist:

[X] I have checked that there are no issues with similar or the same content
YOU SHOULD CHECK CLOSED ISSUES AS WELL

[x] I have checked the FAQ (https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/wiki/FAQ) and the answer I am looking for is not there

[x] I have double checked and can reproduce the issue

OS:
Windows
GPMDP Version:
4.7.1 probably(how to check,really?)
Issue Descriptions:
Can not stream music from the Youtube music mode to another Google device, as it already works in Google music mode. Just can not find see such button.
P.S. Thx for the great app by the way!

Also missing the little "cast to device" button that was present with GPMDP, there is currently no way to enable continuous play mode, shuffle mode, or repeat mode through the mobile yt music app so I'm hoping this casting feature is made available eventually.

just for clarities sake, google play music had an in app cast button. Youtube music has no such button and relies on the browsers built in cast button. The requested enhancement would be to add cast functionality.


"Go to Definition" inside an *.dal file opens an new unnecessary duplicate

1. Describe the bug
If you navigate to code inside a dal file and try to navigate even further a new unnecessary duplicate tab is opened.
2. To Reproduce

Search for a record variable inside your app, e.g. Record "Prod. Order Line".
Execute the "Go to Definition" on the "Prod. Order Line"
A new tab opens with the "Prod. Order Line"
Select a line with a local reference, e.g. TestField("Finished Quantity", 0);
Place your cursor inside "Finished Quantity" and execute "Go to Definition"
A new (unnecessary) tab opens and shows the field definition of "Finished Quantity".

Path of the first tab:

Path of the second unnecessary tab:

3. Expected behavior
The vsx should notice that the dal file is already open and not open another dal file with the same name (but other path).
4. Actual behavior
It just opens a new tab with the same content.
5. Versions:

AL Language: v12.3.900902
Visual Studio Code: 1.84.2
List of Visual Studio Code extensions that you have installed: Deactivated all except AL

The path looks wrong on the second tab, so maybe this is why this is happening

My tests were all done in a multi project workspace.
The default "Go to definition" is using the parent folder name (as in the windows explorer) as the first path segment: app in my case
The "AL Explorer" is opening yet another path. It is using the app-id (guid) without dashes:

And AZ AL Dev Tools is using the workspace folder name (which can be changed in the workspace file via name)

(Also in AZ AL Dev Tools: Name is URL encoded, space to +, slash to %2F, etc.).
After some time my breakpoints are getting cluttered, looks something like this but more mixed up:

I like it that way, seeing the calling and the called procedure parallel is a boon.

I like it that way, seeing the calling and the called procedure parallel is a boon.

To switch between the caller and called Position I use Alt-LeftArrow or Alt-RightArrow
If you want to open it in a split view you can press Alt+F12 then Ctrl+Enter


Pagination - set boundary-links dynamically

I try to set pagination "boundary-links" to true, only if i have more then 10 pages.
How can i do that, way is this not working:
http://plnkr.co/edit/sybAKPVyDZe5PUazlTvx?p=preview
Maybe it's not an issue, just I'm doing it wrong... (?)

Closing due to inactivity. Also, this seems like a question that should be asked on StackOverflow instead of in an issue tracker. We keep track of bugs here, not questions.


Orchest install stucks at setting up the Orchest Cluster

Describe the bug
I am trying to deploy Orchest.io on AKS. I am running:
sh orchest install
and it gets stuck at:
Installing the Orchest Controller to manage the Orchest Cluster...
Setting up the Orchest Cluster...
🚶 Initializing

Logs from orchest-controller:
kubectl -n orchest logs -l app.kubernetes.io/name=orchest-controller
E1108 11:55:44.149894       1 controller.go:172] failed to get container runtime version
E1108 11:55:45.150468       1 cluster_controller.go:331] failed to get container runtime version
W1108 11:55:45.150498       1 controller.go:170] dropping Object "orchest/cluster-1" out of the queue: failed to get container runtime version
E1108 11:55:45.150512       1 controller.go:172] failed to get container runtime version
E1108 11:55:46.152360       1 cluster_controller.go:331] failed to get container runtime version
W1108 11:55:46.152391       1 controller.go:170] dropping Object "orchest/cluster-1" out of the queue: failed to get container runtime version
E1108 11:55:46.152405       1 controller.go:172] failed to get container runtime version
E1108 11:55:47.154170       1 cluster_controller.go:331] failed to get container runtime version
W1108 11:55:47.154200       1 controller.go:170] dropping Object "orchest/cluster-1" out of the queue: failed to get container runtime version
E1108 11:55:47.154215       1 controller.go:172] failed to get container runtime version

Expected behavior
Orchest gets installed.
To Reproduce
Steps to reproduce the behavior:
Run sh orchest install
Environment

OS (e.g. macOS): WSL2
Browser (e.g. Chrome): Edge
Orchest's version (in the settings page): v2022.11.1

Hi @everestas, thank you for posting the logs of the orchest-controller. At this time we do not support installing Orchest on AKS.  Is using AKS a requirement for you?
In case you are looking to create a PR, the problem seems to be caused by: https://github.com/orchest/orchest/blob/adc0e45239f0bf5f7030313aae989312af6d56b4/services/orchest-controller/pkg/controller/orchestcluster/cluster_utils.go#L411-L426
Not sure though exactly why that fails, since containerd (which we support) is used by default on AKS (although not sure you are using it).
(cc @nhaghighat)

Hi, @yannickperrenet, we have everything on set up on Microsoft products, including our k8s cluster. At the moment AKS is a requirement for us.
Anyway, thanks for your answer!

Totally understandable! Thank you for your response, helps us to prioritize.

@everestas if you're keen we can plan a session to help you get Orchest up and running on AKS. Although we don't officially support it yet, we should be able to get it to work without much issue. Send me an email at<EMAIL_ADDRESS>and we'll get something scheduled.
It would help you try out Orchest in your desired cloud environment and we'd get some more hands on experience with the AKS peculiarities.


Ability to run in the browser?

I am trying to use sanitizeHtml in the browser but it doesn't appear to work. Webpack gives errors that it can't find the package. Sometimes I receive html2parser errors.
Can sanitize-html run in the browser?

I can't think of any reason why it wouldn't work, but it would probably be
very large; someone tried this before and complained that htmlparser2 had a
lot of dependencies that made it huge in the browser or something.
My response was that sanitizing things in the browser is a fundamentally
bad idea because while it might have some legitimate use, it presents a big
moral hazard that you'll actually trust that data when it gets to your
server!
On Mon, Aug 31, 2015 at 4:50 PM, Chris McKnight<EMAIL_ADDRESS>wrote:

I am trying to use sanitizeHtml in the browser but it doesn't appear to
work. Webpack gives errors that it can't find the package. Sometimes I
receive html2parser errors.
Can sanitize-html run in the browser?
—
Reply to this email directly or view it on GitHub
https://github.com/punkave/sanitize-html/issues/74.

--
*THOMAS BOUTELL, *DEV & OPS
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

Open to chatting about this, but since running in the browser is currently not considered in scope or a good idea, I'm going to close the issue for housekeeping.

Yeah, I agree that it's a bad idea to sanitize on the client only. My use case for it right now is giving a preview of the sanitized version of the text.

That does sound useful. I suggest using an AJAX call to get it though.
On Mon, Aug 31, 2015 at 7:10 PM, Chris McKnight<EMAIL_ADDRESS>wrote:

Yeah, I agree that it's a bad idea to sanitize on the client only. My use
case for it right now is giving a preview of the sanitized version of the
text.
—
Reply to this email directly or view it on GitHub
https://github.com/punkave/sanitize-html/issues/74#issuecomment-136524429
.

--
*THOMAS BOUTELL, *DEV & OPS
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com


[FP]: dnsjava CVE-2024-25638

Package URl
<EMAIL_ADDRESS>CPE
null
CVE
CVE-2024-25638
ODC Integration
{"label"=>"Maven Plugin"}
ODC Version
10.0.3
Description
https://github.com/dnsjava/dnsjava/security/advisories/GHSA-cfxw-4h78-h7fw
https://nvd.nist.gov/vuln/detail/CVE-2024-25638#VulnChangeHistorySection
This vulnerability is still under analysis on the nvd website. In github, this vulnerability only affects version 3.5.0 of dnsjava. 2.1.7 shall be unaffected.

GHSA listed is not a source for DependencyCheck. NVD is, but as you indicate still needs to attribute the versions. OSSINDEX is also as source and in there your exact version of the library is flagged as affected by the CVE.
Note that typically OSSINDEX does not take the CVE report at face value, but has their own team that decides on the applicability and may even decide to not accept a software change as fixing the reported vulnerability.
ODC correctly reports that one of the consulted resources (in this case OSSINDEX) is flagging the evaluated library as affected by the CVE.


Doesn't work with embeds

Embeds are broken, seems like #73 was overwritten, is this something that could be added back in?
** (RuntimeError) changing embeds with change/2 or put_change/3 is not supported, please use put_embed/4 instead
      (ecto) lib/ecto/changeset.ex:1011: Ecto.Changeset.put_change/7
    (stdlib) lists.erl:1263: :lists.foldl/3
      (ecto) lib/ecto/changeset.ex:325: Ecto.Changeset.change/2
      (ecto) lib/ecto/changeset/relation.ex:152: Ecto.Changeset.Relation.do_change/4
      (ecto) lib/ecto/changeset/relation.ex:237: Ecto.Changeset.Relation.single_change/5
      (ecto) lib/ecto/repo/schema.ex:516: anonymous fn/4 in Ecto.Repo.Schema.surface_changes/4
    (elixir) lib/enum.ex:1755: Enum."-reduce/3-lists^foldl/2-0-"/3
      (ecto) lib/ecto/repo/schema.ex:503: Ecto.Repo.Schema.surface_changes/4

@jeroenvisser101 I don't have a problem with adding it back in, but I don't use embeds much so this isn't a high priority right now. If you (or someone else) wants to take a look at adding it back in/fixing it, that would be great.
Also it may already be working on master, but since I don't use the feature myself I am not sure. Can you give master a try and see if it fixes your issue?

@jeroenvisser101 I added some tests in #200 so embeds should work if you use the latest from master. We're working on 2.0 which will include any fixes on master, plus a few other things

Thanks! I'll try to test these this weekend!

Sorry for the delay, we tested this and seems to work fine, thanks so much!


In Plain Sight audio guides: hotjar tracking

We want to know if the following change has improved the user experience
User experience of exhibition is enhanced by audio guides.
Definition of success is
Users are engaged.
We'll know if we're succeeding if we measure the following metric and this happens
Hotjar tracking shows large numbers of users clicking on the play buttons, mousing over text and scrolling down through the guides.
Definition of failure is
Users are distracted, not engaged.
We'll know if we're failing if we measure the following metric and this happens
Hotjar tracking shows large numbers of users clicking on the "Change guide type" or "Change exhibition" buttons, non-existent mousing over text, few play button clicks and shallow scroll depth.

Just want to query the use of Hotjar for all of this. I would prefer we use Hotjar only if we've deemed alternative methods to be ineffective, just because it is more invasive, privacy-wise.
Breaking this into the multiple metrics mentioned above:

Buttons in header: We can see the data for "Change guide type" and "Change exhibition" in GA using Navigation Summary/Next Page Path

Hover data not going to be very useful on mobile

Play buttons: For Play buttons, it seems like a play is something we'd want to track for all audio content, not just temporarily in the exhibitions. Could we add an event to the play button instead?

That leaves scroll depth as the one metric that Hotjar is pretty good at.
What do you think @davidpmccormick and @gestchild

That seems fair enough. Navigating from the IPS audio guides page to another type of guide means specifying the next page path as https://wellcomecollection.org/guides/exhibitions/YzwsAREAAHylrxau because the user then has to choose a guide type. This could also be the result of someone clicking Back but I think we cal live with this.

Play button tracking is in ticket #9007. I'll cross out everything except scrolling from the first part of this ticket.


[Translation] Setting up the Serverless Backend chapter to portuguese

Translated the "Setting up the Serverless Backend" to portuguese.

Thank you for this! @bernardobugmann can you take a quick look and we'll merge it?
Here is the deployed PR: https://deploy-preview-361--serverless-stack.netlify.com

You're welcome. @jayair @bernardobugmann please take a look, give some feedback about the quality and if everything is good to go. After your feedback i'm planning to translate all the chapters.

@IgorGRibeiro Sorry to take so long to look this. Until tonight I will check if everything is ok and merge the PR.
Sorry again. This year was super busy to me :(

Everything looks great! 👍