GH-40616: [Docs][GLib] Ensure overwriting placeholder front pages

Rationale for this change
Place holder font pages are still used because Sphinx build is executed before GLib docs build.
What changes are included in this PR?
Build Sphinx docs before GLib docs are built.
Are these changes tested?
Yes.
Are there any user-facing changes?
Yes.

GitHub Issue: #40616

@github-actions crossbow submit preview-docs

+1
http://crossbow.voltrondata.com/pr_docs/40618/c_glib/arrow-glib/index.html


#469: Cleaner counties modal behaviour on Android

Small code changes to fix two things:

First commit: a recent fix for something else caused the 'X' icon on the Counties dropdown modal to stop working (verfied on two Android devices, both in latest nys and build 46 from play store). Works with this change.
Second commit: for #469 - the juddering I saw seems to be cause by the keyboard coming into view while the dropdown items list is still rendering and there seem to be some random possible timing glitches depending on the random order things happen. This seems to stop that and makes the whole thing much smaller, ensuring the list has content and is sized correctly before bringing up the keyboard.

I've verified that it's still reachable by physical keyboard tab


[SNMP] Remove metric_prefix from snmp_tile integrations

What does this PR do?
Remove metric_prefix from snmp_tile integrations
Motivation
We recently added some tile-only integrations that do not generate metrics per se, and that are meant to show up in the integrations list so that customers can easily see which vendors we support with snmp.
This was causing issues on the integration monitoring side: having multiple integration definitions with the same metric prefix makes it impossible to define which integration generated a metric.
As snmp_tile integrations are all using metrics generated by snmp we decided to remove the metric_prefix from these integrations.
This PR updates also the template manifest used to generate new snmp_tile integrations in ddev
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Synced snmp_cisco and seems to work fine on staging https://dd.datad0g.com/account/settings#integrations/snmp-cisco


Features extracted from layer -1  represent sentence embedding for a sentence?

Hi Jacob,
Just to make sure, Do features extracted from layer -1  represent sentence embedding for a sentence ?
And if I want to extract the ssentence embedding for a sentence. Should I add sapce between Chinese character by myself in the input file?
Thanks a lot !

-1 means the last hidden layer. There are 12 or 24 hidden layers, so -1,-2,-3,-4 means 12,11,10,9 (for BERT-Base.) It's extracted for each token. There is not any "sentence embedding" in BERT (the hidden state of the first token is not a good sentence representation). If you want sentence representation that you don't want to train, your best bet would just to be to average all of the final hidden layers (or second-to-last hidden layers, i.e., -2, would be better).
If you're using the latest version of the repo then you don't need to tokenize it yourself, the Chinese character tokenization is handled by tokenization.py.

Hi,
I wonder if you extract feature vectors with Chinese successfully. In my practice,it seems that the procedure can't recognize Chinese characters, and display UNK for a Chinese sentence.I don't know if it's a character encoding problem.

Chinese wasn't supported until this weekend, please pull the latest version and then re-try.

If you want sentence representation that you don't want to train, your best bet would just to be to average all the final hidden layers of all of the tokens in the sentence (or second-to-last hidden layers, i.e., -2, would be better).
Thanks Jacob. Curious as to why second to last hidden layer may be better?

Hi @jacobdevlin-google, when you say "the hidden state of the first token is not a good sentence representation", is the "first token" here the CLS token? If so, I'm wondering why it's not a good sentence representation.

I got the answer from https://bert-as-service.readthedocs.io/en/latest/section/faq.html#why-not-use-the-hidden-state-of-the-first-token-as-default-strategy-i-e-the-cls

@ajbarber  https://bert-as-service.readthedocs.io/en/latest/section/faq.html#why-not-the-last-hidden-layer-why-second-to-last

thanks @hlums. My take on it is I believe the last layer is subject to the encoder-decoder attention mechanism, which produces this bias referred to in your link.


ci: add cargo release job

Add a CI job to manage Cargo releases only when a GitHub release is published.
Action for @estk, you will need to set the secrets.CARGO_REGISTRY_TOKEN value in the project settings.

https://docs.github.com/en/actions/security-guides/using-secrets-in-github-actions


Possible edge case bug in higher order function type inference

TypeScript Version:  3.4.1 and 3.5.0-dev.20190405

Search Terms: higher order function type inference
Code
type F<T> = (t: T) => T;

function id<T>(f: F<T>): F<T> {
  return f;
}

function foo(s: string): string {
  return s;
}

function obj<T>(ks: { [K in keyof T]: F<T[K]> }): F<T> {
  return void 0 as any; // implementation is irrelevant
}

const o1 = obj({
  a: foo
});

const o2 = obj({
  a: id(foo) // compiler doesn't complain, vscode 1.33 does: Type 'F<string>' is not assignable to type 'F<{}>'.
});

const out1 = o2({ a: 123 }); // compiler doesn't catch this

const fields = {
  a: id(foo) // vscode is fine with this
};

const o3 = obj(fields);
const out2 = o3({ a: 123 }); // compiler catches this, as expected: Type 'number' is not assignable to type 'string'

Expected behavior:
I expected inference to work the same whether the fields object is defined inline or not.
Actual behavior:
Assigning the object to variable fields before calling obj with it makes inference work as expected, while defining the object inline in the call to obj breaks subsequent type checks.
Playground Link: here
Related Issues: none

Moving type parameters to the return type only occurs when the return type of a function is syntactically a function type expression. A type alias like F<T> isn't one, so this doesn't occur.

But as far as I can tell, it does occur when I do the const fields = assign? The reason I raised an issue was the inconsistent behavior between the case where the object is assigned to a variable and then passed to the function, vs. when it's directly passed to the function.

This works as I would assume on 3.4.4, closing.


[ Linux ]Java Runtime Environment Runtime Fatal Error

on some systems (testing in this case on linux) this error might pop up when trying to run faf :
./downlords-faf-client
14:07:46.192 [main] DEBUG com.faforever.client.preferences.PreferencesService - Logger initialized

(java:11636): Gdk-CRITICAL **: 14:07:46.320: gdk_x11_display_set_window_scale: assertion 'GDK_IS_X11_DISPLAY (display)' failed
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x00007f2ff528bbc5, pid=11636, tid=11853
#
# JRE version: OpenJDK Runtime Environment (10.0.2+13) (build 10.0.2+13)
# Java VM: OpenJDK 64-Bit Server VM (10.0.2+13, mixed mode, tiered, compressed oops, g1 gc, linux-amd64)
# Problematic frame:
# C  [libX11.so.6+0x28bc5]
#
# No core dump will be written. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again
#
# An error report file with more information is saved as:
# /home/jabard/faf/hs_err_pid11636.log
#
# If you would like to submit a bug report, please visit:
#   http://bugreport.java.com/bugreport/crash.jsp
# The crash happened outside the Java Virtual Machine in native code.
# See problematic frame for where to report the bug.
#

FAF client is visible for a second but disappears almost instantly.
right now I' can't determine the cause.
has anyone had this?

figured it out, this is a linux-only issue : https://bugs.openjdk.java.net/browse/JDK-8210411
On ubuntu 18.04 systems the parial implementation of Waylan, makes javaFX crash (maybe not always because I never had this issue back when I was running ubuntu 18.04 on my old PC, but my work pc that is running ubuntu 18.04 does present this issue). ubuntu 19.04 on the other hand won't produce this issue.
also reverting to gtk 2 apprently fixes this on ubuntu 18.04 systems.


added roles endpoints

[x] Get Role
[x] Get Roles
[x] Delete Role
[x] Create or Update Role

@iammukeshm This PR is ready for review

Thanks for the PRs! @wtmaxim


0.3.8

fixes #126

Note: some checks are failing because Hatcher was published a few minutes ago.


Scale SceneMesh to match scale of CameraRig

In my project the concept revolves around the player being larger than the dynamically spawned toy cars in the scene. Due to physics these toy cars need to remain at their original scale of 1 (which is real world car scale). So in order to achieve the effect of the player being much larger than the cars I have scaled up my CameraRig to 10.
This generally works fine but now I wish to use SceneAPI so that the cars are contained within the walls of the room but due to the scaling factor of the CameraRig the SceneMesh does not render to scale to match the CameraRig causing issues with colliders etc.
Is there a way to scale the SceneMesh so that it works with a scaled CameraRig?

Hey, I assume you're using MRUK in your project because Spirit Sling also uses it, right?
If this is the case, MRUK currently doesn't support custom OVRCameraRig scale, unfortunately. A workaround is possible, but please bear in mind it can break other MRUK tools: after the room is loaded, you can scale MRUKRoom.transform together with OVRCameraRig. In my test, the following code works fine with the EnableWorldLock setting (it's recommended to prevent camera drift) but may interfere with other MRUK tools like EffectMesh or FindSpawnPositions.
using Meta.XR.MRUtilityKit;
using UnityEngine;

public class ScaleTest : MonoBehaviour
{
    [SerializeField] private OVRCameraRig _cameraRig;

    private void Update()
    {
        var currentRoom = MRUK.Instance.GetCurrentRoom();
        if (currentRoom != null)
        {
            float input = OVRInput.Get(OVRInput.RawAxis2D.LThumbstick | OVRInput.RawAxis2D.RThumbstick).y;
            if (Mathf.Abs(input) > 0.01f)
            {
                const float scaleSpeed = 5f;
                var newScale = _cameraRig.transform.localScale * (1f + input * scaleSpeed * Time.deltaTime);
                _cameraRig.transform.localScale = newScale;
                currentRoom.transform.localScale = newScale;
                print($"newScale: {newScale}");
            }
        }
    }
}

Please tell me if this helps.


Update to point to geopandas/pyogrio

Updates source repository to github.com/geopandas/pyogrio and updates documentation links.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Looks like we are failing on some architectures (aarch64 / ppc64le) due to missing Cython, but these builds were previously successful on master and Cython is listed in set of packages to be installed, so I'm not sure what is causing those builds to break.

@jorisvandenbossche this is passing now; all it does is change the code location and links in the README, which I think with #21 should now have fixed all references from brendan-ward/pyogrio => geopandas/pyogrio.

@conda-forge-admin, please rerender


IDriss CTA

Add a CTA to setup a free IDriss if you hold $100 Glo Dollar. This consists of

A modal with instructions
A backend that checks whether a user has $100 Glo Dollar and then sends that wallet address to IDriss for inclusion in their allowlist of 1 free IDriss
A link to an IDriss hosted flow for the user to claim their free IDriss

@Riglanto For the CTA copy let's do
Claim free IDriss handle
Hold $100+ of Glo Dollar to claim an IDriss registration for this wallet
With this icon: https://mirror-media.imgix.net/publication-images/D51AJ4naPXuSXvZokOw5W.png?h=64&w=64

@Riglanto for the modal with instructions let's do
IDriss 🤝 Glo Dollar
We've teamed up with IDriss to gift a free registration to community members that own at least $100 Glo Dollars. With an IDriss you can use an  📧, 📱 or @Twitter instead of your wallet address. You can then share your impact using this name: https://app.glodollar.org/impact/@geyr_garmr
Availability is limited, so act fast!

Buy $100 Glo Dollars (or more!)

Across all blockchains

Request a free IDriss registration

Just click here

Complete IDriss registration

Connect your wallet on idriss.xyz


iOS: Textoverlay container to small?

When applying a textoverlay to a video, on iOS the letter "g" is not visible entirely.

Thank you for a question.

#10 @Gorter-dev please see.


Updating existing alias does not appear to work

I am using version 1.0.0-canary.4 and running tf-next alias set <my alias> <deployment id> --force to update an existing alias to point at a new deployment. However when I access my application the change is not being reflected. It feels like a cache somewhere has not been invalidated but I'm not entirely sure where.

I've tried a couple of things which have not worked.

Disabling browser caching.
Invalidating the Proxy-Config CloudFront distribution.

Manually changing the cache policy on the "Default (*)" behaviour on the "Main" CloudFront distribution to "Managed-CachingDisabled" appears to work around this issue.

Thanks for reporting.
Can confirm that the creation of invalidations is not working correctly in the latest canary release.
Currently evaluating different solutions to this problem since we plan to move from SQS to Lambda Step functions to handle the invalidation queue.
Since we have to handle multiple deployments, we now need a mechanism to deduplicate invalidations when they are enqueued.
An update to this is planned in one of the next releases.

Is there a way to manually remove an alias?
tf-next deployment rm <deployment-id> --force
doesn't work.  Same issue, /* invalidation of both CF distributions doesn't work as well


Response file could be a rsp file

For the moment, respfile argument is for rmf file and ancrfile is for arf file.
It should be taken into account that respfile might be a RSP file (rsp = rmf * arf) and in this case, there is no ancrfile but the instrument response is entirely encoded in the respfile.

Lies in extension name, could either "MATRIX" or "SPECRESP MATRIX"


MainfluxLabs vs mainflux

Hello all,
I noticed there was a fork at around August 4th of 2022. I didn't find any details about this on either this repo, the website, the other repo, or anywhere on Google/reddit etc. Is there any detail about this? This fork has much less of a following based on Github metrics, but has more commits, and better SEO? What are the goals of this fork vs the other? Why not just contribute back to the other project?
Thanks.

Hi William,
Thank you for reaching out and raising these questions. We appreciate your interest in our project.
You're absolutely right regarding info about the fork on this repository, the website, or any other platforms. This is currently in the process of being addressed.  We will soon provide transparent and accurate information to the community.
Regarding the goals of the fork compared to the original project, the fork was created due to differences among the founders that led to a split in 2020.  The BDFL of the original repository, and other founders/contributors who are also founders of Mainflux Labs company, had fundamental divergences that couldn't be resolved through contributions alone.
These differences stemmed from different overall visions, development dynamics, and also specific needs of EU Funded projects and clients of Mainflux Labs company.
As a result, the forked project aims to achieve its own distinct vision, direction, and goals maintaining its own development pace and path.
We hope this clarifies the situation for you. If you have any further questions or concerns, feel free to ask.
BR

Got it, thanks for taking the time to respond! Looking forward to the update, as it'll be helpful to understand what the differences are.


more widget sizes and styles

Cool project!
Could you please add more sizes and styles?

I could... But can you elaborate on what you are missing at this time?

Widgets such as a week overview, also some with more of an overview a d more resizaable.

I see what you mean. I'll put it on my list :)

Maybe a 4x1 widget?  :)

Bump for the 4x1 widget. It's a must. It should show the current weather and a forecast for 4 days imho.

What about something like this? Would love to see a widget with a daily overview.


How to install

Hi
I'm beginner of HA, cannot found installer instruction in github, can some help and write how to install dashboard on HA?
BR, Krzysiek

Hi
Install through HACS


(bug) DirectML execution provider does not function

Describe the bug
When attempting to use the DirectML execution provider (on my machine), the provider errors out and does not apply acceleration. This does not appear to be the case on (some) other user's machines, we should get to the bottom of this.
To Reproduce
Steps to reproduce the behavior:
Run the app in its current state.
Expected behavior
DirectML execution is utilized on Windows 10 and higher.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 10 x86
Version 22H2

After further testing, I am coming under the impression this is a just an issue with my current machine. I will close this issue once I am able to test on another Windows 10/11 computer.


Cell export failed.

Then restart memcached_cache and memcached_lock, cell export succeeded.
Related: #311

Could you show how to reproduce this?

Sorry, I forgot the detail of this issue. I cannot reproduce it. So I'll close it and reopen if this issue is happend again.


Need to updated roles claims in access token of an authenticated user on change of organization In Implicit flow . I could see lot of people where asking this question can you please suggest the how can we achieve..

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@brockallen @leastprivilege  I have been waiting for answer of this question.Please provide us the solution.


Save/restore cache locally.

First of all, nice lib! Simple yet powerful.
I wonder if it provides any interface for saving/restoring cache locally? This is useful when an app using Kingfisher has accumulated some [remote url -> local path] mappings, and before the device restarts, the app can save such mappings so that after restarted, the app can read the mappings from drive locally, so that it doesn't have to fetch images from the server again.
Thanks!

Maybe you could use ImageCache for it? There is a storeImage(_:originalData:forKey:toDisk:completionHandler:) method for it. You could call it with your images/keys(which is the absolute string of your url by default) to add it to Kingfisher's cache system.


Public Tickets are not created

Describe the bug
So I had 2 public queues everything was working fine until I added, a non public queue. I add all the information it sends a POST request but it doesn't create the ticket. This problem doesn't occur when I make the tickets while logged in. I have absolutely no idea how this came to be any tips to fix?
Expected behavior
The thing I wanted to create was 2 public queues where all people can submit tickets, and one private(internal) queue for employees to submit.

Which version is this?

Which version is this?

0.2.22
I literally reinstalled my whole project, I thought I had messed something up but still the same problem continues, a new fresh install, 2 public queues and one that needs to be signed in. When I submit a ticket to the public queue it goes to localhost/helpdesk/#submit and nothing happens, it sends a POST request but doesn't actually do anything.

Can you share the post request/code are you doing/using?
So we can replicate it and investigating, maybe after all those years the issue is fixed...

Sorry I don't think I have the code


Restore yb attribution

@palmskog or @Zimmi48, Pierre is uncomfortable with merging so can you please check and act accordingly?
What I did was:
git rm fib.v make_fib_tests.txt stub.ml
copies the files back in place
git add fib.v make_fib_tests.txt stub.ml
git commit
This way the author appears normally in the git history.
I have more improvements to propose on the fib.v file, in future version, I will remove the comments "author: Yves Bertot" that do not belong in the file itself.

Done.

@ybertot @Casteran for the record, everything looked fine to me, and git blame returns the expected results.


[BUG] Knex - Already up to date

Environment
Knex version: 0.16.3
Database + version: postgresql
OS: heroku
Bug
I am struggling with a problem connected with the migrations in knex on the production environment (heroku). I have made some changes and created 8 new migrations to update the model. After creating new migrations I have reached 47 migration files. After uploading the changes to the Heroku I executed as usual knex migrate:latest and there was no effect. I have received the message Already up to date.
I have decided to take a look at the knex_migrations table to see if maybe something is wrong there. The result that I can see that the new migrations were not written into that database (please see the picture for reference.

When we take a look at the migrations it looks like this:

Clearly we can see that knex was able to run "20200216191040_receiver_company.js" as lat one.
When I take a look at the knex_migrations_lock table the only entry that is existing over there looks like this:

Maybe your code has not updated until heroku or you have forgot to git add those migration files from local machine to heroku. Anyways sounds like an user error and not a knex bug. Can be reopened if reproduction is provided.

You could check if knex starts failing when there is around 40-50 migration files though...

@elhigu I have checked if the migration files are present on heroku and they are. Additionally, I have run those migrations locally and the were executed.

Right... it works locally. Then the only reason I can think of is that files are not found from heroku. Or knex configuration is wrong and files are not found because of that. Though I suppose that in that case knex would give an error because of mussing migration files (unless db configuration is also so wrong that db looks empty.


feat: psp22 standard for decrease_allowance and burn pallet methods

Discussion: https://github.com/r0gue-io/pop-node/pull/300#discussion_r1777416957

For decrease_allowance: Fungibles::decrease_allowance() saturating_sub the value, it should checked_sub and throw an error instead. Hence, we don't have to handle InsufficientAllowance on the contract side.
For burn: InsufficientBalance case is not returned in the burn pallet api if the value exceeds the minted value. In decrease_balance method of pallet-assets, it is also saturating_sub.

NOTE: Original PR https://github.com/r0gue-io/pop-node/pull/311 includes file permission changes which is hard to revert and makes the commit history unclean. Hence, I created this PR to substitute the original on and move all comments / resolve comments left by reviewers.

Codecov Report
Attention: Patch coverage is 76.78571% with 13 lines in your changes missing coverage. Please review.

Project coverage is 51.57%. Comparing base (5083a46) to head (d8a1bc4).

Files with missing lines
Patch %
Lines

pallets/api/src/fungibles/weights.rs
50.00%
13 Missing :warning:

@@            Coverage Diff             @@
##             main     #322      +/-   ##
==========================================
+ Coverage   51.47%   51.57%   +0.10%
==========================================
  Files          48       48
  Lines        4894     4907      +13
  Branches     4894     4907      +13
==========================================
+ Hits         2519     2531      +12
- Misses       2326     2327       +1
  Partials       49       49

Files with missing lines
Coverage Δ

pallets/api/src/fungibles/mod.rs
92.51% <100.00%> (-0.39%)
:arrow_down:

pallets/api/src/fungibles/tests.rs
99.69% <100.00%> (+<0.01%)
:arrow_up:

pallets/api/src/fungibles/weights.rs
50.00% <50.00%> (ø)


Request: Provide strong typing of numeric Builtin Types

At the moment all numeric types are Type Aliases of number. This means that there is no type-checking, so accidentally using the wrong type doesn't raise an compilation error.
function test(
    player: LuaPlayer
) {
  const position: MapEntityPosition = player.position
  const x1: double = position.x // correct
  const x2: uint = position.x // incorrect, x is not a uint

  const playerIndex1: uint = player.index // correct
  const playerIndex2: double = player.index // incorrect, index is not a double
}

Is it possible to make all numeric types type-safe, so const x2: uint = position.x creates a compilation error?
I've done a bit of research and I think a couple of the approaches described here Nominal Typing in Typescript might be the most intuitive and least intrusive, but I don't know if they are technically compatible with TypeScriptToLua.
type BuiltInNumeric<T extends "uint" | "double"> = number & { __precision: T }

type uint2 = BuiltInNumeric<"uint">
type double2 = BuiltInNumeric<"double">

function double2(value: number) { // util function
  return value as double2;
}

function test(
    playerIndex: uint2,
    playerXPos: double2
) {
  const x1: double2 = playerXPos // compiles
  const x2: uint2 = playerXPos // error, Type 'double2' is not assignable to type 'uint2'.

  const playerIndex1: uint2 = playerIndex // compiles
  const playerIndex2: double2 = playerIndex // error, Type 'uint2' is not assignable to type 'double2'.

  const fooX: double2 = 123 as double2
  const barX: double2 = double2(123)

  const bazX: uint2 = double2(123) // error
}

Typescript still just use number to represent all numeric types, regardless of precision, as its all the same representation in the lua runtime. The typescript nominal typing you described would technically work, however it will likely cause more frustration than benefit:
The result of any math operation (like a + b) with "branded" numbers is a plain number, so simple code would be full of casts and hard to use. Another example, uint should be assignable to double, but it doesn't in the example you provided.
Another idea, perhaps, is to "brand" numbers which weren't intended to have arithmetic done to them, like player ids, entity ids, etc. This would run into less of the above problems. However, this would be stricter than many people would like, and could be confusing to newer users of typescript, so I'm hesitant to implement this.

It would be a very impactful change, so I think it's good to think thoroughly about if it's worth
implementing.
Casting
Can you give an example of some code that would be full of casts? I tried it out and from what I can
see, casting is only required when assigning a value with a specific numeric type.
This basically makes it 'opt-in', so I don't think it's too intrusive.
function example(myData: MyData) {
  // no type specified, so no casting needed
  const someCount = myData.count
  const result = ((someCount * 100) + 123) / 654
  // now a cast is needed, because `printCount` needs a specific type
  printCount(result as uint2)

  // a cast is needed because 'result2' is explicitly typed
  const result2: uint2 = ((someCount * 100) + 123) / 654 as uint2
  // no casting needed, because `result2` is the correct type
  printCount(result2)
}

interface MyData {
  count: uint2
}

function printCount(count: uint2) {
  print(`count is ${count}`)
}

The biggest problem I see is that as uint implies, but doesn't ensure, the value is an unsigned
integer. This post
creates a roundToInt(...) function to do this, as well as casting.
Explicit conversions
Personally I'd really, really prefer having explicit casting. Automatic conversion causes problems
in Java, and this is fixed in Kotlin, which requires explicitness.
It's explained here:
https://kotlinlang.org/docs/basic-types.html#explicit-conversions
But I don't know how applicable my preference is to the world of TypeScript/Lua/C :)

Thanks for your comments. However I still see several major problems with explicit int types in Typescript:
Explicit/implicit conversions aside: something as simple as (a + 1) would require a cast; literally any math operation will require a cast. Additionally, there is no integer division, numeric conversion, specifying the type of a numeric literal, integer overflow, etc.. So, using this would require tons of casts that aren't checked by the compiler, easily leading to wrong code that is type annotated otherwise. This is unlike other statically typed languages that have actual int types represented in the runtime environment.
I don't want separately typed integers unless typescript itself supports them.
Some examples:
const pos = ...
const north: uint = (pos.y + 1) as uint // more boilerplate than arithmetic

let counter = 0 as int
let id = (++counter) as int // this is nessecary

const x = 3 as int
const y = (x / 2) as int // no compiler complaints, evaluates to 1.5

Numbers that represent "ids" which are usually not meant to have arithmetic done to them, such as player_number, surface_number, etc., still may benefit from being strongly typed in this way. This probably addresses a good portion of the use cases this issue wished to address.
The above can be considered in a separate issue.
Unless there are additional comments I'll close this issue for now.

I've put your examples into the tstl playground.
It doesn't look like casting is required for ++counter?
I also added example util functions. This would help "more boilerplate than arithmetic" and produces a correct result.
const x = 3
const y: int = to_int(x / 2)
print("y = " + y) // y = 2

wrong code that is type annotated otherwise

That describes the current situation too. At worst 'brand typing' would make the problems visible.
To demonstrate, I monkey-patched my local project, it highlights a LOT of mistakes I've made, that weren't previously visible. Here are some of them
schema definitions
I made a mistake in my schema definition for MapTilePosition
  export function mapTilePosition(mapPos: MapPositionTable): MapTilePosition {
    return {
      x: mapPos.x, // TS2322: Type 'double' is not assignable to type 'int'.
      y: mapPos.y, TS2322: Type 'double' is not assignable to type 'int'.
    }
  }

I made a mistake in my schema definition for EntityData
export function entityToTable(entity: LuaEntity): EntityData {

    let player: LuaPlayer | null = entity.is_player() ? entity.player!! : null

    return {
      objectName: entity.object_name,

      name: entity.name,
      type: entity.type,
      active: entity.active,
      health: entity.health ?? null, // TS2322: Type 'float | null' is not assignable to type 'double | null'.
      healthRatio: entity.get_health_ratio(), // TS2322: Type 'float' is not assignable to type 'double'.
      surface: entity.surface.index,
      unitNumber: entity.unit_number ?? null,
      position: mapEntityPosition(entity.position),
      force: entity.force.index,
    }
  }

athematic mistakes
A mistake when converting a map position to a chunk position.
let chunkPosition: ChunkPosition = {
  x: (event.position.x / 32), // TS2322: Type 'number' is not assignable to type 'int'.
  y: (event.position.y / 32), // TS2322: Type 'number' is not assignable to type 'int'.
}

// this should be
let chunkPosition: ChunkPosition = {
  x: to_uint(floor(player.position.x / 32)),
  y: to_uint(floor(player.position.y / 32)),
}

And a bug which would have tried to have create an event in the past. I wanted to create an event in the future, and when the trigger is nearer the origin the event should be triggered sooner), but because x + y is an int, this creates events in the past, when the trigger is in some parts of the map.
  SpecialEventHandler.createEvent(key, key.position.y + key.position.y) // TS2322: Type 'int' is not assignable to type 'uint'.

Numbers that represent "ids" which are usually not meant to have arithmetic done to them, such as player_number, surface_number, etc., still may benefit from being strongly typed in this way. This probably addresses a good portion of the use cases this issue wished to address.

Yeah, that would help for sure.

What I meant was that (a + 1), if a is an int, gives number instead of int. even (a + b), which are both ints, yields number.
Support for integral numbers simply doesn't exist in Typescript, and it's not worth trying to patch it in like this.

UPDATE:
Since version 0.20.0, Strict index types have been added, which addresses some cases this issue covers. This is an opt-in feature, see the readme doc here for more details.
Additionally, custom-index-template.d.ts has been added (at the root of this repo). You can use this to create "branded" numeric types in your own project as you see fit without as much monkey-patching.
However, I still don't want to add this officially to this project (yet).

After further consideration, this will likely not be implemented. Typescript (and lua) has no separate integer type and typescript did not decide to implement integers; it's not worth attempting to work around this.


Add some new components

List of components :
-MVerticcal DONE
-MHorizontal DONE
-MCard DONE

MFIeldSet DONE
MTitle DONE


PaneItemEpander Widget crashes the app when the Pane is collapsed🐛

Describe the bug
PaneItemEpander Widget crashes the app when the Pane is collapsed.
To Reproduce
Steps to reproduce the behavior:

Create PaneItemExpander
Click on the Icon while Pane is collapsed
'This widget has been unmounted, so the State no longer has a context (and should be considered defunct). \n'
'Consider canceling any active work during "dispose" or using the "mounted" getter to determine if the State is still active.', thrown

Expected behavior
I expected the Navigation to not crash the entire application and throw an error.
Screenshots

It's be very helpful if you could run the app with flutter run, and post the full error here

══╡ EXCEPTION CAUGHT BY SCHEDULER LIBRARY ╞═════════════════════════════════════════════════════════
The following assertion was thrown during a scheduler callback:
This widget has been unmounted, so the State no longer has a context (and should be considered
defunct).
Consider canceling any active work during "dispose" or using the "mounted" getter to determine if
the State is still active.
When the exception was thrown, this was the stack:
#0      State.context. (package:flutter/src/widgets/framework.dart:947:9)
#1      State.context (package:flutter/src/widgets/framework.dart:953:6)
#2      NavigationIndicatorState.pane (package:fluent_ui/src/controls/navigation/navigation_view/indicators.dart:70:39)
#3      NavigationIndicatorState.fetch. (package:fluent_ui/src/controls/navigation/navigation_view/indicators.dart:60:28)
#4      SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:1175:15)
#5      SchedulerBinding.handleDrawFrame (package:flutter/src/scheduler/binding.dart:1113:9)
#6      SchedulerBinding._handleDrawFrame (package:flutter/src/scheduler/binding.dart:1015:5)
#7      _invoke (dart:ui/hooks.dart:148:13)
#8      PlatformDispatcher._drawFrame (dart:ui/platform_dispatcher.dart:318:5)
#9      _drawFrame (dart:ui/hooks.dart:115:31)
════════════════════════════════════════════════════════════════════════════════════════════════════
Another exception was thrown: setState() called after dispose(): _StickyNavigationIndicatorState#0405b(lifecycle state: defunct, not mounted, tickers: tracking 0 tickers)
Another exception was thrown: This widget has been unmounted, so the State no longer has a context (and should be considered defunct).
Another exception was thrown: setState() called after dispose(): _StickyNavigationIndicatorState#80327(lifecycle state: defunct, not mounted, tickers: tracking 0 tickers)
Another exception was thrown: This widget has been unmounted, so the State no longer has a context (and should be considered defunct).
Another exception was thrown: setState() called after dispose(): _StickyNavigationIndicatorState#85c9b(lifecycle state: defunct, not mounted, tickers: tracking 0 tickers)

I have same problem but different error message. Please reopen this issue. Also I see same problem on example website(https://bdlukaa.github.io/fluent_ui/).
My code:
Widget getNavBar() {
    return fluent.NavigationView(
      //appBar: const fluent.NavigationAppBar(title: Text('NavigationView'), backgroundColor: Colors.blue),
      pane: fluent.NavigationPane(
        selected: topIndex,
        onChanged: (index) => setState(() => topIndex = index),
        displayMode: fluent.PaneDisplayMode.auto,
        items: [
          fluent.PaneItem(
            icon: const Icon(Icons.baby_changing_station),
            title: const Text('Home'),
            body: dashboard(),
          ),
          fluent.PaneItem(
            icon: const Icon(Icons.baby_changing_station),
            title: const Text('Track an order'),
            body: WorkerProfilePage(worker!),
          ),
          fluent.PaneItemExpander(
            icon: const Icon(Icons.baby_changing_station),
            title: const Text('Account'),
            body: Container(),
            items: [
              fluent.PaneItem(
                icon: const Icon(Icons.baby_changing_station),
                title: const Text('Mail'),
                body: WorkerProfilePage(worker!),
              ),
              fluent.PaneItem(
                icon: const Icon(Icons.baby_changing_station),
                title: const Text('Calendar'),
                body: WorkerTimesheetPage(worker!),
              ),
              fluent.PaneItem(
                icon: const Icon(Icons.baby_changing_station),
                title: const Text('Calendar'),
                body: const JobTrackingCalendarPage(),
              ),
            ],
          ),
        ],
      ),
    );
  }

My Error:

[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: 'package:fluent_ui/src/controls/flyouts/flyout.dart': Failed assertion: line 470 pos 12: 'isAttached': This controller must be attached to a FlyoutTarget
package:fluent_ui/…/flyouts/flyout.dart:470
#0      _AssertionError._doThrowNew (dart:core-patch/errors_patch.dart:51:61)
#1      _AssertionError._throwNew (dart:core-patch/errors_patch.dart:40:5)
#2      FlyoutController._ensureAttached
package:fluent_ui/…/flyouts/flyout.dart:470
#3      FlyoutController.showFlyout
package:fluent_ui/…/flyouts/flyout.dart:544
#4      __PaneItemExpanderState.toggleOpen
package:fluent_ui/…/navigation_view/pane_items.dart:702
#5      __PaneItemExpanderState.build.
package:fluent_ui/…/navigation_view/pane_items.dart:806
#6      PaneItem.build.
package:fluent_ui/…/navigation_view/pane_items.dart:140
#7      GestureRecognizer.invokeCallback
package:flutter/…/gestures/recognizer.dart:253
#8      TapGestureRecognizer.handleTapUp
package:flutter/…/gestures/tap.dart:627
#9      BaseTapGestureRecognizer._checkUp
package:flutter/…/gestures/tap.dart:306
#10     BaseTapGestureRecognizer.acceptGesture
package:flutter/…/gestures/tap.dart:276
#11     GestureArenaManager.sweep
package:flutter/…/gestures/arena.dart:163
#12     GestureBinding.handleEvent
package:flutter/…/gestures/binding.dart:464
#13     GestureBinding.dispatchEvent
package:flutter/…/gestures/binding.dart:440
#14     RendererBinding.dispatchEvent
package:flutter/…/rendering/binding.dart:336
#15     GestureBinding._handlePointerEventImmediately
package:flutter/…/gestures/binding.dart:395
#16     GestureBinding.handlePointerEvent
package:flutter/…/gestures/binding.dart:357
#17     GestureBinding._flushPointerEventQueue
package:flutter/…/gestures/binding.dart:314
#18     GestureBinding._handlePointerDataPacket
package:flutter/…/gestures/binding.dart:295
#19     _invoke1 (dart:ui/hooks.dart:164:13)
#20     PlatformDispatcher._dispatchPointerDataPacket (dart:ui/platform_dispatcher.dart:361:7)
#21     _dispatchPointerDataPacket (dart:ui/hooks.dart:91:31)


fix(readme): fix add-driver-docker url in Readme

SUMMARY
fix broken link in readme regarding doc for driver for docker

Thanks for the catch!


Problems with reproducible (deterministic) random streams across Godot versions

Godot version:
3.1+
OS/device including version:
Windows 10 and others
Issue description:
Solved: see reason below.
According to RandomNumberGenerator documentation (Godot 3.2) (#27591):

The underlying algorithm is an implementation detail. As a result, it should not be depended upon for reproducible random streams across Godot versions.

That's quite unfortunate... I've figured that my unit tests fail each time I rebuild Godot that use procedural generation given the same seed. I wonder whether it's a bad practice to rely on reproducible random streams anyway. I see no way to test procedural generation stuff otherwise, the only way would be to stick to the same Godot version. Having custom modules dependencies in a project leads to rebuilding Godot anyway.
Should probably just drop unit tests that rely on randomness, or make the RandomNumberGenerator implementation deterministic not only within a build, but across builds (and potentially platforms? too big to ask).
If it also depends on the compiler itself, that would be fine as long as the same compiler is used, I don't see much issue here, just need to pass those tests across builds so I can still be adventurous with Godot master.
The issue is rather inconvenience rather than a bug, so this can be interpreted as enhancement.
Implementations
To note, I've been using some thirdparty library to generate noise, and always got the same reproducible noise across Godot builds. It seems like KISS algorithm is used there:
https://github.com/Xrayez/godot-anl/blob/0e6d442cc87e2438ec0cc3de903600def89cd5a0/thirdparty/anl/VM/random_gen.h#L208-L243
Steps to reproduce:

Build Godot from source.
Create random stream using RandomNumberGenerator (generate a random array of values)
Test that the steam is reproducible given the same seed.
Rebuild Godot at different version.
The random stream from previous build doesn't match with current build given the same seed.

@toasteater

If you rely on random stream determinism you better implement your own simple random number generator and use it instead of one embedded into engine. This way you will be independent of implementations. Godot tries to do that but noody really focused on the subject.

I don't necessarily need to rely on random stream determinism. The output of "procedural stuff" is intended to be saved externally or transmitted over the network directly (i.e. sending actual collision shapes vs seed alone). It's just that I want to be sure I don't break anything during development, and reproducible randomness across builds would suffice for that purpose.

Found a plugin that aims to solve this:
SeededRandomSequence
The issue is still relevant though.

Godot's RandomNumberGenerator has gone through multiple changes already since its inception. It's never guaranteed to have identical output across versions, it's just never properly documented.
If you want reproducibility, the best way to do it is with code that you control yourself: link to a third-party RNG implementation with a module or GDNative. That way you can rest sure that changes in Godot won't affect your procedural generation.
If you want a frozen PRNG in Godot, you will want to talk to the maintainers, instead of some mere mortal who can't and didn't decide it :)

@toasteater maintainers are always busy, so mere mortals have to breakthrough the challenges on their own :)
This just defeats the whole purpose of RandomNumberGenerator for me, but I always tend to have specific use cases anyway, so no wonder! I can live with that for now.

Well, your use case is not specific, but software changes,,,
I still suggest you write your own  small generator which will work
across versions. I have a few written in GDScript even.
Also you can revert changes in tree to keep the same bahavior. In
general at current state of affairs everyone is on their own and all
use cases are specific.
On Wed, Apr 10, 2019 at 3:13 PM Andrii Doroshenko
<EMAIL_ADDRESS>wrote:

@toasteater maintainers are always busy, so mere mortals have to breakthrough the challenges on their own :)
This just defeats the whole purpose of RandomNumberGenerator for me, but I always tend to have specific use cases anyway, so no wonder! I can live with that for now.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

OK, just rebuilt Godot at recent version, the stream is reproducible and tests passes...
Coincidentally, every time I've rebuilt Godot it always introduced some changes to PCG implementation. 😄
Because of this, I just thought that any change to the engine leads to the issue. I just can replace my test samples if I see if any changes are made to PCG implementation and I see some tests fail because of this, then continue on with other stuff.
Closing, could be reopened for discussion I guess.

I would vote for this to be reopened. Even if @Xrayez got their tests to pass, the documentation still contains this:

Note: The underlying algorithm is an implementation detail. As a result, it should not be depended upon for reproducible random streams across Godot versions.

There should be a way to get a stream that is reproducible across Godot versions. It's not always feasible to save the entire output of procedural generation algorithm, because the output might be very large or infinite (think Minecraft).
For example, allow the user to specify explicitly which PRNG algorithm to use, with a suitable default. The default can then be changed freely in a new Godot version, without breaking anyone who has been explicit about which PRNG they want.

I would vote for this to be reopened.

Feature proposals should be made on the https://github.com/godotengine/godot-proposals repository now.


Hardware support URL in README is broken

In the README.md, "Hardware and Software Support" section (https://github.com/RadeonOpenCompute/ROCm#Hardware-and-Software-Support), the URL at the end for the detailed list of supported hardware is broken.  It just redirects to the main ROCm docs page.

Thank you for your feedback. The URL is now corrected. You can access more information about AMD Graphics Processing Units at:
https://en.wikipedia.org/wiki/List_of_AMD_graphics_processing_units
AMD ROCm Documentation Team

Thanks @doctorcolinsmith
As the issue is fixed now, we request you to close this ticket.


feat: replace create with prepare

Description

This PR replaces create command functionality with prepare command functionality.
Motivation and Context

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have updated the documentation accordingly. For updating Oclif commands documentation use oclif-dev.
[ ] I have read the CONTRIBUTION_GUIDE document.
[ ] I haven't repeated the code. (DRY)
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

I'm just thinking: — create:mock-map could be simply create:mock?
create:mock-map-test could be simply create:mock-test?

Yep, I just wanted command name sto be self explanatory 🤷🏻♂

Or would it be too much to merge both into create:mock?
It depends on expected user flow. If we are expecting user who creates mock map to also need test for the we can do it :)

Yep, I just wanted command name sto be self explanatory 🤷🏻♂

Then maybe can we do alises?

I think it's good to go but first please go through docs and all places we used create command at and make sure the instructions are updated :)

Yes, already on it :)


Open PR to trigger workflow

Signed-off-by: Melvin Hillsman<EMAIL_ADDRESS>

Until further notice do not close this PR as it is used to trigger dependabot runs and close notifications of a vulnerability.


Add env var to enable detailed consensus metric logging

Add an env var for opting-in to detail mem metrics logging.

bors r+


Font rendering in PopupPanel

Godot version
4.0.3
System information
Fedora and macOS
Issue description
I have a problem with font rendering in PopupPanel.
Everything with fonts is ok in whole game apart of Popups. Settings per label is the same like in other places in game.
The problem occurs with any font where I would like the rendering to be as "nearest".
Popup (Bad):

Other Place (Good):

Minimal reproduction project
font-rendering-problem.zip

This sounds similar to https://github.com/godotengine/godot/issues/54030#issuecomment-1094143542
Do you use the canvas_items stretch mode?

Yes

Then this is as a known issue a duplicate of the other bugreport.

Ok, I see. I didn't find this earlier because I focused my search on fonts.

Duplicate of #54030

It seems that it's probably not the same issue after all.
BTW, there's a minimal reproducible project attached in the post:


Requesting New Release (v1.0.2 seems outdated)

Using this package in my code gives this error:
masterKey.Derive undefined (type *hdkeychain.ExtendedKey has no field or method Derive)
Seems like go modules refer to https://pkg.go.dev/github.com/btcsuite/btcutil -> github.com/btcsuite/btcutil v1.0.2  which don't include the commits after Apr 14, 2020. Looks like I need to refer to the commit:  - hdkeychain: correct BIP-32 derivation issue on Oct 21, 2020 for my code to work properly.
Any help on this matter would be appreciated.

@Shachindra Did you update your locally installed package? Try running go get -u github.com/btcsuite/btcutil again

Yes @ashfame . I did it using go version go1.15.7 windows/amd64 . So far this is what I found:

Error:

Running go get -u github.com/btcsuite/btcutil

My go.mod file

Clicking on the link points to<EMAIL_ADDRESS>where the directory hdkeychain doesn't have function named 'Derive'.

Nevertheless, Maybe there is some issue in my go modules which is fetching the files from the older version of the package. I wanted to verify whether the error gets fixed if there is a new release.
I (temporarily) fixed the error by copying the entire hdkeychain directory from here locally into my repo. This seems to have fixed the error but I'll have to manually check for updates into the btcutil/hdkeychain/ repo.
I can always fork the repo and refer to that as a last resort.

Agreed, a version bump would be helpful. The latest version isn't compatible with the latest PSBT format from Electrum and Specter. The parsing only worked when I fetched the latest commit.

One more vote for a release.

Please release a new version :pray:

A new v2 module exists in the new home for this repo (btcd/btcutil).


Add difference-of-squares exercise

Adds the 'difference-of-squares' exercise

Everything looks good to me :+1:

Just one nitpick, otherwise LGTM.

Ready for another look


Sensors with wrong state class total_increasing

Hi, mentioned this error in my log with the latest version. Is it possible to change the class by myself? Or has it to be changed in a fix:
Entity sensor.sma_energy_meter_i3 from integration mqtt has state class total_increasing, but its state is not strictly increasing. Triggered by state 1.5 (1.6) with last_updated set to 2024-08-23T11:20:15.325234+00:00.
Entity sensor.sma_energy_meter_cosphi from integration mqtt has state class total_increasing, but its state is not strictly increasing. Triggered by state 0.9 (1.0) with last_updated set to 2024-08-23T17:54:41.317826+00:00.
BR

Which addon? The dev is probably has the best behavior

This Add-On which I am posting on… I thought ?
Home Assistant SMA Energy Meter Add-On

Try using the developer version, it is more up to date


OCPBUGS-14696: Merge https://github.com/kubernetes/cloud-provider-aws:master (cab0100) into master

Reverts openshift/cloud-provider-aws#43
The rebase was reverted, this PR un-reverts the rebase so that we can re-inroduce the changes from the rebase. Will be building images off of this to explore the disruption data that was found.

/payload 4.14 nightly blocking
/payload 4.14 ci blocking

/payload-job periodic-ci-openshift-release-master-ci-4.14-upgrade-from-stable-4.13-e2e-aws-ovn-upgrade

/payload-job periodic-ci-openshift-release-master-nightly-4.14-e2e-aws-sdn-upgrade periodic-ci-openshift-release-master-ci-4.14-upgrade-from-stable-4.13-e2e-aws-ovn-upgrade

/payload-aggregate periodic-ci-openshift-release-master-ci-4.14-upgrade-from-stable-4.13-e2e-aws-ovn-upgrade
Have added a commit to switch the health checks over, which I believe should resolve the issue we have been seeing. Need multiple runs to validate that it does solve the issue

/payload-aggregate periodic-ci-openshift-release-master-ci-4.14-upgrade-from-stable-4.13-e2e-aws-ovn-upgrade 10

/label qe-approved

/approve
Based on test results we've seen here, the additional patch that I've added has resolved the disruption issues we've seen that cause this PR to be reverted originally. Have started the process of getting the patch in upstream, will bring any updates from that PR back in post merge if required.

the carry patch makes sense to me, we just need to be careful about the 1.28 rebase in case upstream decides to change something, but i don't think that's a big risk.
/lgtm


test opensession softhsmv2

Hello I'm trying to write a test for softhsmV2 but i'm not able to load libsofhsm2.so
$ echo $PKCS11_MODULE
/usr/lib/softhsm/libsofthsm2.so

import os
import pkcs11

lib   = pkcs11.lib(os.environ['PKCS11_MODULE'])
token = lib.get_token(token_label='My SoftHSMv2 token')

with token.open(rw=True,user_pin='2023') as session:
    print(session)

Traceback (most recent call last):
  File "pkcs11/_errors.pyx", line 88, in pkcs11._pkcs11.assertRV
RuntimeError: Initialisation error (not initialized). Should never see this.
Exception ignored in: 'pkcs11._pkcs11.lib.__dealloc__'
Traceback (most recent call last):
  File "pkcs11/_errors.pyx", line 88, in pkcs11._pkcs11.assertRV
RuntimeError: Initialisation error (not initialized). Should never see this.
Traceback (most recent call last):
  File "/home/jk/pqapp-build/test_pkcs11.py", line 4, in <module>
    lib   = pkcs11.lib(os.environ['PKCS11_MODULE'])
  File "/home/jk/.local/lib/python3.10/site-packages/pkcs11/__init__.py", line 32, in lib
    _lib = _pkcs11.lib(so)
  File "pkcs11/_pkcs11.pyx", line 1432, in pkcs11._pkcs11.lib.__cinit__
  File "pkcs11/_errors.pyx", line 88, in pkcs11._pkcs11.assertRV
pkcs11.exceptions.GeneralError

What i did wrong ?

it was a permission problem with softhsm's config file


menu bar item selected while window open

it would be nice if the menu bar icon stayed selected while the window is open (right now it selects and unselects). it's non-obvious to users where the screen popped up from.
in particular, at least make it so on the landing page image, so that users know what instantly icon they should expect to appear after installing.

Seems like something menubar should handle, but I'm not sure how. The Tray docs don't seem to provide a way to do it
If a Tray instance has a context menu (via tray. setContextMenu), then the button looks pressed whenever the menu is visible. I tried adding a small dummy menu to the menubar that screencat is using, but nothing at all would appear when I clicked the tray icon, so that seems to break other things


Crash on launch in production due to PackMCMetaVersionsGetter failing

https://mclo.gs/sobKI41
https://github.com/Matyrobbrt/GroovyModLoader/blob/main/src/mod/groovy/com/matyrobbrt/gml/mod/PackMCMetaVersionsGetter.groovy#L19
Using a simple hard-coded values fixes the issue.

This issue is caused by https://github.com/Matyrobbrt/GroovyModLoader/blob/main/src/mod/groovy/com/matyrobbrt/gml/mod/PackMCMetaVersionsGetter.groovy not getting remapped.

Neither options are going to fix it, the property syntax uses dynamic calls, and the metaclass remapping is not yet set up that early. The fix is to make that class use Java.

https://github.com/Matyrobbrt/GroovyModLoader/commit/98c127c9f514d13a2ae0ddaa46b1f01421a47abd has fixed it in production, but now crashes in dev.


Archive repository

I has become clear that ipfs-npm is unmaintained since the last commit is from June 2019, the latest release is still impossible to install and attempting to install it throws exactly 49 deprecation warnings. I suggest archiving the repository to not give false hope to anyone trying to use it.

:-(
That's a pity this project is a really nice idea.

I wonder if I should just try updating the dependencies and see what happens. curiosity intensifies

Just do it ✔

@achingbrain is lead maintainer, so is the one who can speak to whether or not to archive, as I believe it's more experiment than product.
But yes, definitely helping update deps and fixing is much appreciated either way :)

This repo lives in ipfs-shipyard which is a place for experiments and demos, and projects here sometimes don't see as much attention as they perhaps deserve.
That said, this is still a pretty interesting idea but it's tough to find time to work on all the interesting ideas so some help would be appreciated.

I wonder if I should just try updating the dependencies and see what happens.

Please do and submit a PR with the updates!

I thought IPFS had a stated goal to focus on increasing the usefulness of IPFS for things such as code repos/cdns?

No, I honestly don't believe archiving this project will solve anything.
In fact, it may harm the project (we really need this project 😛).
I would be open to helping, but I'm definitely going to need to learn more about IPFS before I jump in...

No, I honestly don't believe archiving this project will solve anything.
In fact, it may harm the project (we really need this project stuck_out_tongue).
I would be open to helping, but I'm definitely going to need to learn more about IPFS before I jump in...

Would be amazing if you did this :+1:

Apologies, I've been super busy with my blog and college 😅
So, after a quick thonk, would it be better to create a stub registry that interfaces with IPFS, instead of maintaining a wrapper for the CLI?

@resynth1943 What would be best is to make a mirror of the registry, pin and sync it using IPNS and use it with a localhost:8080/ipns address. This way, you wouldn't need to create a new package, all you'd need to do is change your local registry address for the IPNS hash and it would automatically work with your local node.

@NatoBoram what about seeking fund from the IPFS grant program, then ? 😇

I would honestly love to maintain this project if I received a grant for it. Otherwise, I cannot afford the time. I LOVE the idea of this project.

I thought IPFS had a stated goal to focus on increasing the usefulness of IPFS for things such as code repos/cdns?

This was, one year a stated a goal of IPFS. I know they have been working with Nixos integration, which is very very cool. But I would love to see NPM decentralized.

@resynth1943 What would be best is to make a mirror of the registry, pin and sync it using IPNS and use it with a localhost:8080/ipns address. This way, you wouldn't need to create a new package, all you'd need to do is change your local registry address for the IPNS hash and it would automatically work with your local node.
That's something I wanted to do with Ubuntu packages (offline proof of concept here), but it's quite expensive in terms of storage. The NPM registry is about 8 TB or something and you'd need at least double that to store it on IPFS and then you need room to support its continuous expansion. Just to be safe, one would need to start with 32 TB of SSDs to store it on IPFS. About 4 of these, totalling 3'863.96$.
I would totally do it if I could afford that for my side projects :D

I don't think that storage space on one physical system is the right approach, it is also very much does not take advantage of the power of the IPFS ecosystem and its distributed nature. Ideally, we should have very powerful sponsored nodes, but more preferred, I think the algorithm should be more like this:

A developer needs an NPM package, they query the metadata of that from NPM, or a third party(maybe ipfs hosted) source. This is only for metadata. For the hash
The developer takes this hash, and queries ipfs for this content
3a) If the content exists on IPFS, it is downloaded.
3b) If the content does not exist on IPFS, it is downloaded from NPM.
After the content is downloaded, it is by default served on IPFS so that other users can now download this package from you.

This way, packages can be managed incrementally. There isn't a requirement for the entire repo on one machine. Possibly it would help to widely mirror all the metadata on NPM. Maybe even to each machine. I wonder how heavy that metadata is? It would be great to have a repo of metadata on each machine that is auto updated.


[NG] Clean up datagrid footer's flexbox

The only child of the footer that now expands is the
.datagrid-foot-description which contains any custom text or buttons
the consumer decides to add to the footer. Since this element is always
here, it will naturally push its siblings to the left and to the right
of the footer, preserving the current styles.

Wait I forgot to ask if this was tested on all browsers? IE11 usually behaves different than other browsers when it comes to flexbox. Actually when it comes to everything CSS.

It does work on IE, this isn't even a gray area, I'm just swapping which element is flexible, so we have the exact same CSS as before except the second child stretches instead of the first. I did check. 👍
But I just realized I'm assuming our users use the new-ish API for pagination, which isn't the case. Even our CSS tests don't use it yet, so it messes up the position of the pagination text. I'm pushing a quick fix for that right now.


Data provided question

Hello,
I am a student trying to use your repository for a project.
Using your data from the readme on Zendono.
I am trying to reproduce your results from this paper and trying to run your jupyter notebook files.
However, I am having a lot of problems.
Is this data given, ready for use to run the jupyter notebook files, or do i need to process them using the other code you provided in different repositories?
All I have is this repo and the data file from Zendono.
Thank you for your help.

Hi!
It's very nice that you are interested in our project :)
The data uploaded in zenodo should be working with the code. What errors are you having? Could you provide us a screenshot with them?
Thanks!

When trying to load the last part of the first notebook, "Finally, we generate the optimal parcellation (oPA) image, and save it in bha2/data/processed/CONN_SIZE". This is my error.

Hi!
The repo was thought for starting building the multiscalar trees, so in Usage section in the Readme.md was described how to build them as initial step. But indeed, It would not be mandatory if a user like you use directly the data from Zenodo.
I have added the code fix_ipa_rois.py to be used as an initial step if a user doesn't want to build the trees. I have added also how to use it in Usage section.
Thanks for the concern.


problem with progressive rendering when use search/filter

Great gem 👍
progressive_render works on all my views beside one. In my posts#index view, I use filter to filter in my posts.
Some part of my posts#index:
#results
      = render partial: 'posts/list', locals: { posts: @posts }

My posts/list:
= progressive_render do
  - posts.each do |post|
     = post.title

My index.js.erb:
<% js = escape_javascript(render(partial: 'posts/list', locals: { posts: @posts }) ) %>
$("#results").html("<%= js %>");

My problem now is that when I search for a word, results are not showing and the loading part is just loading.
How can I fix this?
Thanks in advanced!

I'm guessing that the event to load the partial content is never getting kicked off. Can you try raising the event progressive_render:end after you set the HTML?
index.js.erb
...
$("#results").html("<%= js %>");
$(document).trigger('progressive_render:end')

Thanks for your help :)
Adding $(document).trigger('progressive_render:end') works great 👍


fix(server): add upgrading process to poll_without_shutdown()

Fixes #1529.

Thanks for catching this!


[FLINK-28639][Runtime/Checkpointing] Preserve consistency of events from subtask to OC

test

@flinkbot run azure


Development updates

MongoDB uses native promises; do not fail if files are missing; some housekeeping;

Got at least some of the tests to work, though I suspect the missing ones are probably crucial... will investigate later.


katib db error

Hello, after deploying kubeflow bundle with k8s using this script I've got issues with katib-db:
Version: '10.3.17-MariaDB-1:10.3.17+maria~bionic'  socket: '/var/run/mysqld/mysqld.sock'  port: 3306  mariadb.org binary distribution
2020-06-17 19:29:36 9 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:29:41 11 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:29:46 13 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:29:51 14 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:29:56 16 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:30:01 17 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:30:06 19 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:30:11 20 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:30:16 22 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:30:21 23 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
Any idea about that ?

same for pipelines db:
2020-06-17 19:29:17 0 [Note] Reading of all Master_info entries succeeded
2020-06-17 19:29:17 0 [Note] Added new Master_info '' to hash table
2020-06-17 19:29:17 0 [Note] mysqld: ready for connections.
Version: '10.3.17-MariaDB-1:10.3.17+maria~bionic'  socket: '/var/run/mysqld/mysqld.sock'  port: 3306  mariadb.org binary distribution
2020-06-17 19:29:17 0 [Note] InnoDB: Buffer pool(s) load completed at 200617 19:29:17
2020-06-17 19:35:44 73 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:35:44 74 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:35:45 75 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:35:47 77 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:35:48 78 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:35:49 79 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:35:50 81 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:35:51 82 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:35:52 83 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:35:53 84 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:35:55 85 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:35:57 87 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:36:12 91 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:36:13 92 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:36:14 93 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:36:15 94 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:36:16 96 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:36:18 97 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:36:43 103 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:36:44 104 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:36:44 105 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:36:46 107 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:36:47 108 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:36:50 109 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:37:40 121 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:37:40 122 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:37:41 123 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:37:42 124 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:37:43 125 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:37:45 126 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:39:13 145 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:39:14 146 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:39:14 147 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:39:15 148 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:39:17 150 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:39:17 151 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:39:17 152 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)
2020-06-17 19:39:18 153 [Warning] Access denied for user<EMAIL_ADDRESS>(using password: YES)

This should be fixed with #225, or by downgrading Juju to 2.7


Fixed #27849: Add support for filtering postgres aggregates

This adds support for filtering on all Postgres aggregates using the FILTER (WHERE ...) syntax.
I thought it would be best to make it a bit queryset-like, as in you can do .filter() and .exclude() on all Postgres aggregates, like so: SomeModel.objects.annotate(x=ArrayAgg('y').filter(y__z=1).
Under the hood this just converts the keywords to Q() objects, but it feels a lot nicer than passing them in manually. They also support chaining.
I'm at the DjangoCon sprints if anyone wishes to comment in person about this 👍
Edit: I've added some tests specifically for the ArrayAgg aggregate, but I'd like some feedback about the implementation before I commit to writing more tests + documentation etc.
I'm also not entirely sure how I would go about writing the tests, should I write the same tests for every postgres aggregate?

Hello @orf, thanks for your contribution!
Adding FILTER support for all PostgreSQL aggregates was proposed in the past in #5956 and one comment I had is that it would be much better to implement this for all existing Aggregates by using CASE for databases that don't have FILTER support. This also has a bit of overlap with #7611 if I'm not mistaken.
While I like your filter() syntax I find it a bit unintuitive that it's not chainable like QuerySet.filter() is. I think we should stick to using a kwarg for that instead as we don't want to get into the semantic of filtering chaining e.g. filter(A, B) can produce different queries than filter(A).filter(B) under certain conditions.

Hey @charettes, thanks for commenting! I think emulating FILTER using CASE is a good idea in theory but it might get a bit tricky and I'm not sure how you would do it in a generic way. Maybe I'm over thinking it though. If you have any hints I could try and take this PR in that direction, because then it could be added to the base Aggregate class and would work with standard aggregates like Count and Max etc.
The filter() syntax in this PR is chainable? The FilterMixin handles this by returning new FilterWhere instances. FilterWhere uses the FilterMixin.

Ok, I've managed to implement this using CASE statements (with FILTER upgrades to Postgres). I'll close this MR and push it as a separate MR.

@orf did you push this to another PR?

Hey @charettes, I spoke to Marc about this ticket and he re-opened because it was a new feature, independent of the CASE WHEN fallback, it but in the course of the discussion we realized it was not strictly necessary as there is apparently no performance difference between FILTER (WHERE x) and CASE WHEN x other than the syntax.
I had an implementation that worked (ish), but with this new knowledge I think this ticket should stay closed. I really like the idea of a .filter(x) on aggregates, but the implementation would more simple than my current one (just produce the necessary Case and When instances). It would just be syntactic sugar rather than a whole new aggregate.
So yeah, IMO this ticket should be closed in replacement for a 'adding .filter()/.exclude() to aggregates' ticket, and perhaps some discussion should be had about the specifics on the mailing list. I can push a proof of concept branch for you to review about this if you like, and send the initial email asking for comments.
Let me know!

Hey @orf!
I agree that adding a generic filter kwarg would only be syntactic sugar but I think it would still be worth adding.
Just like PostgreSQL added support for the FILTER clause as syntactic sugar to make COUNT(CASE(WHEN(...))) more readable I can see value in a filter kwarg to aggregate expressions to make them more readable as well.
Taking the conditional aggregation documentation as example, I find the following way more easier to parse with all the Case(When(...)) boilerplate stripped.
Client.objects.aggregate(
    regular=Count('pk', filter=Q(account_type=Client.REGULAR)),
    gold=Count('pk', filter=Q(account_type=Client.GOLD)),
    platinum=Count('pk', filter=Q(account_type=Client.PLATINUM)),
)

From my non-regular contributions on the django-user mailing list confusion about conditional aggregations came out a couple of times. It could also be considered a reduction of functionality over what the django-aggregate-if library that supported conditional aggregation before built-in support shipped through Case(When(...)) in Django 1.8 offered.

In all cases moving the discussions about adding a filter kwarg as syntactic sugar and filter()/exclude() methods to the ML is good idea @orf. It will get much more exposure and will gather more valuable feedback.

Ok @charettes, I'll try and compose an email tomorrow with my thoughts, as well as a draft merge request. I agree with what you are saying about the CASE statements being fairly unreadable. I'll leave any ticket creating up to you if that's OK, feel free to close this MR if you want to create a fresh ticket or leave it open if you want to morph the existing ticket into this new scope.

Hello, @orf @charettes !
I'd like to add my 2 cents to the discussion.
There is a performance difference between CASE and FILTER:
From the doc - "If FILTER is specified, then only the input rows for which the filter_clause evaluates to true are fed to the aggregate function; other rows are discarded."
In case with CASE all rows will be passed to the aggregate function.
Thus it depends on complexity of the aggregate function, but anyway it is measurable - at my work we gained about ~15% performance improvement in our statistical reports after switching from CASE to FILTER with multiple MAX and COUNT aggregates on ~2M of input rows.

I've made an initial merge request with this functionality here: #8352. I'm going to close this now, thanks @Stranger6667 for your feedback. I will send a message to the mailing list now.


Dark mode and light mode

I can make a great button for switching between dark and light mode

Great! Assigning it to you!

I want to try this issue ? @technophyle66

@Vishrut19 okay, assigning it to you as well.

Hey @technophyle66 I am getting this error ! Please help me fix this

@Vishrut19 I believe the ESlint configs were causing an issue, I remove those, Try the new branch, also you're on 3001 port, it wont work unless you change the code because the app is coded to work on the 3000 port

Now the code is working for me @technophyle66

Hey @technophyle66 is it fine that I change the color from orange to white in the app ?

@Vishrut19 sure! Design changes are also welcome


fix(docs): rename vault migrate command

dotenvx ext vault migrate

thank you!

@gander also thank you for the sponsorship!


[RTPS_WRITER Error] Cannot add RTPS submesage to the CDRMessage. Buffer too small -> Function insert_submessage

Thanks for being the pioneer make ros2 example available on internet.
some issue I tested on cuda10.0  nvidia-410
here is the report
https://hackmd.io/s/rkRj5iG5N

missing dependency image-transport
the point cloud freeze after start 2 seconds, recieved frames are not stable
roslaunch cannot do nested launch
[RTPS_WRITER Error] Cannot add RTPS submesage to the CDRMessage. Buffer too small -> Function insert_submessage

wish someone get what I mean.

Hi @YuehChuan
thank you for the full report, the ROS2 wrapper is still a beta version so the more we get feedback, the more it will be improved.

You can install image_transport with the following command:
$ sudo apt install ros-crystal-image-transport
as you correctly reported.
If you speak about "RVIZ2" it's a known problem and it is reported in the "Known Issues". RVIZ2 is still missing message_filters implementation, so topics synchronization is not working very well. We hope that Rviz2 will get message_filters support soon.
[For more info see this issue and this pull request]
I'm going to investigate and fix the ros2 launch problem.
We never got that error message. Are you using a not default configuration for RTPS middleware?

@Myzhar
thanks for your quick response
4. using the default RTPS middleware


Azure DevOps Power Platform Build Tasks - Import Solution

With task version .2 the Import solution does not provide the Error logfiles form Dataverse anymore when the import fails.
We only get the message

The only way the get the details is importing the solution manually via Maker portal.
Please provide the error log from Dataverse again within the pipeline run details to avoid the manual steps.

Topic is already covered in this repo:
https://github.com/microsoft/powerplatform-build-tools/issues/239

Topic is already covered in this repo:
https://github.com/microsoft/powerplatform-build-tools/issues/239


Timeout hardcoded

If I understand correctly, the connection timeout should be taken from the config and should be in defaultCredentials.
Reference
Possible Solution.
=<< timeout chConnetionTImeout

It's not a socket TTL or DBMS response waiting timeout. It's a TCP socket opening timeout which is OS level syscall
As far as I can understand this operation should be handled much more faster than 3 seconds and It's a common practice to hardcode this value in TCP protocol. After connection established there is no more timeouts
You can find an example in Haskell library network-simple connectSock function, which I was using as a reference
If there are any considerations about it in RFCs to provide control, increase or delete this timeout than leave me a link please

@KovalevDima I understand you and will use your idea! Thanks for the answer!


devdocs don't mention which fields are optional or required

I am using the devdocs to test out the /query/getPlace API, however this occurs for other dataSets too.
For example, a query for 'Photo' with no other fields set results in an error, the documentation doesn't mention the offset field is required.

Good catch!

The offset field should be optional
The dev docs should at least list, if not enforce required (vs optional) fields


Multiword script

Here is the script I have made for handling multi-word expressions, as mentioned in https://github.com/emugel/VIM-thesaurus-files/pull/1#issuecomment-140843348.
It uses thesaurus files with underscores and adds and removes them from the text when entering/exiting thesaurus-mode. It only does so if a string of words is found as a main entry in the thesaurus file.

Closing.


TP-634 Architectural documentation.

An architecture primer. Best viewed in the rendered documentation, so would recommend checking out this branch and building it.

It looks good, definitely a fan of the way this partially comes from the docstrings too.

Codecov Report

Merging #225 (7c56ae8) into master (371127c) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #225      +/-   ##
==========================================
- Coverage   61.13%   61.12%   -0.02%
==========================================
  Files         184      184
  Lines       11121    11123       +2
  Branches      838      838
==========================================
  Hits         6799     6799
- Misses       4133     4135       +2
  Partials      189      189

Impacted Files
Coverage Δ

common/models/records.py
47.64% <0.00%> (-0.51%)
:arrow_down:

measures/business_rules.py
26.27% <ø> (ø)

Continue to review full report at Codecov.

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


grafana: Update to version 10

Unfortunately ports only has www/grafana9 and www/grafana.
So at some point this might pull in grafana 11
(for better or worse).

Best to squash the merge (too many weird commits).


Worker-side Checks

Before receiving any tasks (which are darn hard to debug with all the Celery fluff and bubble), let's run a bunch of quick permissions / access tests. E.g:

Can we access the proxy_media path? Do we have R/W permissions?
Are there any configuration conflicts?

Then we can pop up some user-friendly warnings before attempting any encoding.

This might require some thought actually...
We'd have to make a decision about putting these worker side checks either in the worker launcher or per worker initialization. In the latter case, that means every single worker is going to complain if they're all running on the same host. The worker launcher is still very windows-centric atm. Linux and Mac users can properly daemonize and prefork and do whatever they want, so they don't really need the launcher. That's probably a project / issue in itself. If we did the checks in the worker launcher, Linux and Mac users would likely miss out.

Before receiving any tasks (which are darn hard to debug with all the Celery fluff and bubble), let's run a bunch of quick permissions / access tests. E.g:

Can we access the proxy_media path? Do we have R/W permissions?
Are there any configuration conflicts?

Then we can pop up some user-friendly warnings before attempting any encoding.

Proxy media path is actually set by the queuer and defined in the job task I believe, so there's no easy way to do this except at runtime.


[FR] insert schedule/deadline sentence in the second line of the block

Hi, the plugin is extremely helpful. But the schedule/deadline: <date-time> is not rendered properly in Logseq when not put in the second line of the block.
It would be perfect if the plugin allows to insert schedule/deadline in the correct line to make it render.
Thanks!

Oh sorry I don't understand. Do you have an example please?

Oh sorry I don't understand. Do you have an example please?

Yes, here is a minimal example.

DEADLINE/SCHEDULED has to be in the second line to be visible.
Hope this helps to clarify my request. Thanks!

Thanks for the clarification. The plugin simply replaces %today with the SCHEDULED reference. What is stopping you from putting in %today in line 1 to begin with since it is going to apply to the whole block?

Thanks for the clarification. The plugin simply replaces %today with the SCHEDULED reference. What is stopping you from putting in %today in line 1 to begin with since it is going to apply to the whole block?

Sorry I thought the plugin by default could parse dates no matter where they are. If that's not a designed behavior then I will close this FR. Thanks for the plugin again!


Inventory: what is 'JS_codes'?

In the tree inventory data there's a column JS_codes that's not documented in the README file; entries are blank, or "SP", or "SP, IR". What is this? The README file needs updating to explain this.

These are Jess Shue's codes/ notes I believe. We should integrate these into the appropriate existing column and remove it ideally.


sdkman is blocked from liveperson side

the installation script (https://get.sdkman.io/)
point to "https://api.sdkman.io/1" which is blocked from liveperson (destination side not liveperson firewall.
can you check this please?

Sorry, but don't really understand what you are trying to say. Where are you calling from? From CI or developer machines? Who is liveperson? What do you mean by from destination side not liveperson firewall.

hi,
we have problem to install sdkman, we investigate it and here are the
results.
in the installation doc there is the following instruction "curl -s "
https://get.sdkman.io" | bash"
when looking into the script (under https://get.sdkman.io) there is a
global variable defined "SDKMAN_SERVICE="https://api.sdkman.io/1""
now...
https://api.sdkman.io/1 gives 403 error with nginx forbidden.
our IT said that it comes from the destination side and not from our
firewalls (liveperson company).
is there something that you can help us with?
On Mon, Apr 3, 2017 at 11:25 PM, Marco Vermeulen<EMAIL_ADDRESS>wrote:

Sorry, but don't really understand what you are trying to say. Where are
you calling from? From CI or developer machines? Who is liveperson? What do
you mean by from destination side not liveperson firewall.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sdkman/sdkman-cli/issues/557#issuecomment-291262966,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APRdjjp3GD8_hoXOAJrZfXq_8KFNbqnUks5rsVW2gaJpZM4MxHKL
.

--
Oren Amir
System RnD Team Leader
T: +972-74-700-4334
http://www.linkedin.com/company/164748 http://twitter.com/liveperson
http://www.facebook.com/LivePersonInc We Create Meaningful Connections
https://liveperson.docsend.com/view/8iiswfp
--
This message may contain confidential and/or privileged information.
If you are not the addressee or authorized to receive this on behalf of the
addressee you must not use, copy, disclose or take action based on this
message or any information herein.
If you have received this message in error, please advise the sender
immediately by reply email and delete this message. Thank you.

Okay, I think I know what's going on here. This is due to IP blacklisting. Can we take this conversation offline to Gitter so I can help you resolve this?
You will need to sign up with your github account, and once signed up you can find me here.

done?
is this enough info?
On Tue, Apr 4, 2017 at 10:29 AM, Marco Vermeulen<EMAIL_ADDRESS>wrote:

Okay, I think I know what's going on here. This is due to IP blacklisting.
Can we take this conversation offline to Gitter so I can help you resolve
this?
You will need to sign up with your github account, and once signed up you
can find me here https://gitter.im/marc0der.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sdkman/sdkman-cli/issues/557#issuecomment-291417881,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APRdjh4ADP5u5-YZLjbxljsg4itqd_KJks5rsfF2gaJpZM4MxHKL
.

--
Oren Amir
System RnD Team Leader
T: +972-74-700-4334
http://www.linkedin.com/company/164748 http://twitter.com/liveperson
http://www.facebook.com/LivePersonInc We Create Meaningful Connections
https://liveperson.docsend.com/view/8iiswfp
--
This message may contain confidential and/or privileged information.
If you are not the addressee or authorized to receive this on behalf of the
addressee you must not use, copy, disclose or take action based on this
message or any information herein.
If you have received this message in error, please advise the sender
immediately by reply email and delete this message. Thank you.

I have the same problem, actually.  I'm directly connected via my home ISP - no blacklisting that I know of, and the curl statement fetching the script archive fails. :-(   I'll play with it and post any results, but if you know any obvious reasons (outage, auth requirements, etc.) please let me know.


Publish v0.3.1 to npm

npm still installs v0.3.0
$ npm install modaal --save-dev
<EMAIL_ADDRESS>/vagrant
└──<EMAIL_ADDRESS>

npm has ben updated - thanks for letting us @renestalder!


Rename entities package to examples.

:tada: This PR is included in version 3.0.0 :tada:
The release is available on:

3.0.0
GitHub release

Your semantic-release bot :package::rocket:


Fix a segfault in kv

Fixes https://github.com/apache/skywalking/issues/9956
Signed-off-by: Gao Hongtao<EMAIL_ADDRESS>

Codecov Report

Merging #231 (91045d4) into main (e7b08be) will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #231      +/-   ##
==========================================
+ Coverage   45.17%   45.19%   +0.02%
==========================================
  Files          87       87
  Lines        8872     8872
==========================================
+ Hits         4008     4010       +2
+ Misses       4485     4483       -2
  Partials      379      379

Impacted Files
Coverage Δ

banyand/measure/measure.go
84.61% <0.00%> (+3.84%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Preserve line breaks in multiline subtitles

This is an interesting idea, but it tends to produce weird-looking output in two important modes:

When generating bilingual *.srt files, which need to conserve vertical real estate on the TV screen.
When there isn't a simple 1-to-1 mapping between the two languages' subs, and multiple subtitles need to be merged. This works much better if I join lines across the merged subs.

How would you feel if I submitted a (tested) patch for adding a manual control, something like --multiline?

Please consider this in conjunction with #59. I've been using your tool as part of a script that produces EPUBs for offline study of subtitled content and would love to preserve line breaks to improve the formatting.
Would also love to see support for overlapping subtitles - this has been on my list to figure out but would be great if substudy just supported it directly. Sounds like you're at least thinking about it per #37.
Thanks for your great tool!

I've been using your tool as part of a script that produces EPUBs for offline study of subtitled content and would love to preserve line breaks to improve the formatting.

Oh, cool! I've started a discussion of the new file format at https://github.com/emk/subtitles-rs/issues/59#issuecomment-2094363029, and I'd love your feedback on supporting your ebook exporter.

I've also started a discussion about how to handle overlapping subs at #60.


build: fix relative paths resolutions in tslint rules

When working on a case-insensitive filesystem, source files paths resolved by tslint are in lowercase format. Relative paths computed from the current working directory are not valid and do not match globs defined in tslint configuration.

Do you have an example of your local CWD and a source file path? I haven't seen any issues with that and it looks like my working directory also has an upper-case segment.

@devversion i deleted line 166 from src/cdk/table/text-column.ts // tslint:disable-next-line: lightweight-tokens
trik@resiliency components % pwd
/Volumes/Data/trik/Projects/3rdparty/components
trik@resiliency components % yarn tslint
yarn run v1.22.0
$ tslint -c tslint.json --project ./tsconfig.json
no-unused-variable is deprecated. Since TypeScript 2.9. Please use the built-in compiler checks instead.
../../../../../../volumes/data/trik/projects/3rdparty/components/src/cdk/table/text-column.ts

the last output row is coming from these lines added for debug purpose in lightweightTokensRule
  if (ctx.sourceFile.fileName.indexOf('src/cdk/table/text-column.ts') > 1) {
    console.log(relativeFilePath);
  }

working on macOS 10.15.6 / node 12.18.3 / APFS case insensitive

@devversion just a few notes. i already tried to downgrade tslint to v5 just to check if it's a bug introduced in new versions.
since tslint is deprecated, could it be a good change to migrate to typescript-eslint?


Added classes to reduce motion

Updates website to respect user OS settings to reduce motion
For a lot of users, this will not change anything
tl;dr of reading below: animations like this can cause some users to experience nausea, and it's an incredibly quick fix to improve their experience.

Loom demo
Further reading:

https://web.dev/prefers-reduced-motion/
https://www.w3.org/WAI/WCAG21/Understanding/animation-from-interactions.html

@delete-44 Thanks for helping make the Orbit Model site more accessible! 🙏


Provide an option to activate/desactive key-value overriden

Hi,
I would like to have an option to activate/desactivate the key-value.
The reason is that I will have some mail_trigger jobs that will receive emails which contains key-values, which i do not want to override the mail_trigger parameters.
In another mail_trigger_2 job, i will receive emails which contains key-values which i do want to override the mail_trigger_2 parameters.
It could be a simple option, or a setting in the "Advanced Email Properties". Both solution are fine for me.
Something like :
"parameterOverride=false"

This feature would be really nice as for example, I'm currently working with emails that sometimes contains "PATH=", then as you can guess the PATH environment variable get screwed up for the remaining of the job execution...


ART-7243 merge custom into ocp4

Ref. ART-7243
Give ocp4 extra custom superpowers:

IGNORE_LOCKS -> https://github.com/openshift-eng/aos-cd-jobs/commit/04acc15daf908c4c5ab039515b136ebfc7a884bc
Doozer data gitref -> https://github.com/openshift-eng/aos-cd-jobs/commit/19cd8f7c055b2e595716cdd34ef92589a7d3ae71
possibility to skip plashets creation -> https://github.com/openshift-eng/aos-cd-jobs/commit/cfaa829018043d90a0afe0a88c61eeb439c12239

Previously, custom had some logic to auto-decide whether or not plashets should be built. UPDATE_REPOS allowed us to override this behavior. New proposal: let's have ocp4 decide in its on way, and let's keep for us the possibility to skip plashets creation when testing.
Test build: https://saml.buildvm.hosts.prod.psi.bos.redhat.com:8888/job/hack/job/dpaolell/job/ocp4/173/parameters/

I see ocp4 job changes, but don't see custom job changes

Once this merges, custom will simply go.

/lgtm
Let's have one more +1 :)


Add multi-remove feature to owners list #901

There is a codacy issue in the GroupingsRestControllerTest, it says that there needs to be a assert() or fail() in the test. We use a specific testing framework so I don't believe that codacy picks this up.
Kind of a hack since the asserThat doesn't really test do anything but I remember getting this issue before and doing something like this:
MockMvc Result = MockMvc.perform(...) assertThat(result, notNullValue());

I am not too sure about how codacy checks this. Maybe @zakgilbert could help with this?

This code looks pretty odd to my eyes...
if (person.result === "SUCCESS" || person.resultCode === "SUCCESS")

Also... we've discussed the "JUnit tests should include assert" issue quite a number of times now... Michael keeps hinting that someone should add a comment so that we will remember what is going on here... but alas, it's not getting traction. I guess my comment regarding this codacy issue is to step back and think about what it is happening.

Also... we've discussed the "JUnit tests should include assert" issue quite a number of times now... Michael keeps hinting that someone should add a comment so that we will remember what is going on here... but alas, it's not getting traction. I guess my comment regarding this codacy issue is to step back and think about what it is happening.

are you talking about this line?
        assertThat(result, equalTo(result)); or just the fact that there is no usage of assert in a test?

This code looks pretty odd to my eyes... if (person.result === "SUCCESS" || person.resultCode === "SUCCESS")

This is because the object that is returned from removing a person from exclude/include is different than the removing an owner.
Here is the object the values that are returned from removing a person from include/exlcude:

and here is the object that is returned from removing a person from owners:


[Question]: I am unable to determine the number of API calls and the cost associated with llama_index and GPT-3.5 Turbo calls. I'm trying to optimize the cost for one of my models.

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
Steps to Reproduce:
llm = OpenAI(temperature=0.1, model="gpt-3.5-turbo")
service_context = ServiceContext.from_defaults(
llm=llm,callback_manager=callback_manager
)
from llama_index.indices.struct_store.sql_query import NLSQLTableQueryEngine
query_engine = NLSQLTableQueryEngine(
sql_database=sql_database,
service_context=service_context
)
custom_prompt = """
follow the instructions if needed:
When given an inquiry of id, select the 'status' column"
"""
token_counter.reset_counts()
query_str = "What is the inqiry of id 1234?"
response = query_engine.query(custom_prompt + " " + query_str)
print(response.response)
print('token_counter:',token_counter.total_llm_token_count)
this is printing the token counts but i need to print api calls and cost for api call

The token counts are actually tracked per API call, and total_llm_token_count is just adding them up
You can access token_counter.llm_token_counts and token_counter.embedding_token_counts for a list


Delete and move Python binding

This PR deletes the Python binding codes from this repository. They are now populated at https://github.com/primitiv/primitiv-python.

@vbkaisetsu, please fix travis scripts to adjust this separation.