Geolocation example issue

I have created a CodePen to illustrate the problem, I'm programming an equivalent geolocation application with Angular and there are some bugs : (not the same bugs with Chrome and Firefox, I'm using the last versions)

Chrome : when I'm changing the navigation tab and reopen it (I'm not closing the tab but only swap it) I see that (with a console.log("on")) the geolocation.on function is exectued every time I do this experience.

Firefox : same thing but also every five seconds the geolocation.on function is executed with no reason ...

That's very problematic because I'm working with GeoJSON files on servers ... But also the map refocuses incessantly.

(I have only pressed one time the track checkbox)

geolocation.on('change:position') is called every time a change is detected by the browser but OpenLayers doesn't check if the coordinates have really change. This is probably what happens in your example.

@fredj Is there a way to prevent incessant calls from the function or navigators are blocking the situation ?

You can add a test in the callback to check if the position really changed

To fix the Firefox problem of updating every 5 seconds you can call geolocation.once instead of geolocation.on as said here https://openlayers.org/en/latest/apidoc/ol.Geolocation.html#once


DRAFT: Experiment with reusable dedupe model running in the web process

Overview
Is your feature request related to a problem? Please describe.
We eventually need an API for MSI integration that allows for a single facility to be matched quickly.
Describe the solution you'd like
Questions to answer

How does having 100,000 facilities affect the model?

What is the training time?
How large is the saved model?

All the experimentation is available in the research/single-item-match branch.
I started by writing CSV processing scripts that would allow me to download the production CSV, convert it to a CSV suitable for uploading, and then make permutations of each row, expanding the record count from ≈20,000 to ≈100,000.
I was able to upload this file in development and run through all the processing steps in ≈15 minutes while still fitting in the memory of the 4GB vagrant VM. I mistakenly believed that this would allow us to easily hold a model in memory on each app server and match single items.
I experimented with saving the state of a model after training, but before calling index. This model was ≈9KB in size. I proceeded to build a Django Matcher model and a supporting management command that would allow for storing a trained model in the database so that newly launched app servers could query it, create a StaticGazetteer instance from it then call index and feed in all the records in the Facility table to build a fully working model. I created a simple management command that would train a new model and save the results to the Matcher table.
It is at this point that I discovered the true cost of calling index when the canonical list of items is ≈100,000. I was only able to avoid out-of-memory crashes by increasing the memory available to the Vagrant VM to 14GB.
The good news is that once the large model is loaded into memory, matching a single new item takes less than a second, and calling index to add a new facility to the model is even faster.
Our likely path forward, at this point, it to run our production containers with more memory. We have a relatively large amount of money set aside for hosting and we have only used a small portion of it so far. We will not need 16GB right away and could migrate to larger memory sizes as our Facility list grows closer to 100,000 items.
Next steps:

Determine the cost increase of running large containers in production.
Take ideas and code research/single-item-match and build an branch that
maintains an in-memory persistent model and provides an endpoint for
single-item submissions.


Messing up the ledger file since v0.4.6

Reproduction steps

Ledger file selected via usual Android Document Picker.
Nano Ledger settings - Default status = * cleared, Currency after amount
Whenever a new entry is added, the previous entries in addition to the new entry got appended at the end of file causing duplicate entries.

Before the new entry
2022-12-10 * Self | ATM Withdrawal
    Assets:SAVING  -290 USD
    Assets:CASH  290 USD

2022-12-10 * Self | Cash Deposit
    Assets:CASH  -100 USD
    Assets:SAVING2  100 USD

2022-12-10 * Self | Cash Deposit
    Assets:CASH  -100 USD
    Assets:SAVING3  100 USD

2022-12-10 * Restaurant | Oishi Udon, Beef Noodle, Matcha Milk Tea
    Liabilities:CREDIT1  -19.5 USD
    Expenses:EATINGOUT  19.5 USD

2022-12-10 * Pet Shop | Cat Litter
    Liabilities:CREDIT1  -10 USD
    Expenses:PETS:CONSUMABLE  10 USD

2022-12-10 * Bakery | Buns
    Assets:CASH  -3.9 USD
    Expenses:GROCERIES  3.9 USD

2022-12-10 * Supermarket | Chicken, Cat Treats
    Liabilities:CREDIT1  -23.45 USD
    Expenses:GROCERIES  12.15 USD
    Expenses:PETS:CONSUMABLE  11.3 USD

2022-12-10 * Cosmetics | Cosmetics
    Liabilities:CREDIT1  -25 USD
    Expenses:SHOPPING:BEAUTY  25 USD

2022-12-11 * Shop | 2x Face Masks
    Assets:CASH  -3 USD
    Expenses:SHOPPING:HOME-GARDEN  3 USD

2022-12-12 * Online | Cat Products
    Liabilities:CREDIT2  -67.3 USD
    Assets:AR:XXX  57.85 USD
    Expenses:PETS:CONSUMABLE  9.45 USD

2022-12-12 * Landlord | Rental 2022-12
    Assets:SAVING   -1039 USD
    Expenses:HOME:RENTAL     1000 USD
    Expenses:UTILITY:COMMUNICATION  39 USD

2022-12-15 * Employer | Salary 2022-12
    Income:SALARY  -7745 USD
    Assets:SAVING   7745 USD

2022-12-15 * Employer | Annual Bonus 2022
    Income:SALARY  -7076 USD
    Assets:SAVING   7076 USD

2022-12-15 * Bank | CREDIT 2022-12
    Assets:SAVING    -50.58 USD
    Liabilities:CREDIT1  50.58 USD

2022-12-26 * Self | Transfer
    Assets:SAVING        -200 USD
    Assets:VCARD          200 USD

2022-12-16 * Bank | CREDIT 2022-12
    Assets:SAVING    -581.61 USD
    Liabilities:CREDIT1  581.61 USD

2022-12-29 * Shop | 4x Box
    Assets:CASH  -6 USD
    Expenses:EATINGOUT  6 USD

2022-12-29 * Shop | Drinking Water
    Assets:CASH  -0.9 USD
    Expenses:GROCERIES  0.9 USD

2022-12-31 * Restaurant | Breakfast
    Assets:CASH  -4 USD
    Expenses:EATINGOUT  4 USD

2022-12-31 * Pizzeria | Pizza
    Liabilities:CREDIT1       -26.75 USD
    Expenses:EATINGOUT              26.75 USD

2022-12-31 * Shop | Pepsi
    Assets:CASH  -1.9 USD
    Expenses:EATINGOUT  1.9 USD

2022-12-31 * McDonald's | Double Cheeseburger
    Liabilities:CREDIT1  -5 USD
    Expenses:EATINGOUT  5 USD

2022-12-31 * Self | 2022
    Equity:OPEN-BAL  -7200 USD
    Liabilities:AP:XXX  7200 USD

Supposed new entry
2024-10-23 * testPayee | testNote
    Expenses:HEALTH:MEDICATIONS      546 USD
    Assets:CASH      -546 USD

Actual content appended to the end of file

2024-10-23 * testPayee | testNote
    Expenses:HEALTH:MEDICATIONS      546 USD
    Assets:CASH      -546 USD

f | Cash Deposit
    Assets:CASH  -100 USD
    Assets:SAVING3  100 USD

2022-12-10 * Restaurant | Oishi Udon, Beef Noodle, Matcha Milk Tea
    Liabilities:CREDIT1  -19.5 USD
    Expenses:EATINGOUT  19.5 USD

2022-12-10 * Pet Shop | Cat Litter
    Liabilities:CREDIT1  -10 USD
    Expenses:PETS:CONSUMABLE  10 USD

2022-12-10 * Bakery | Buns
    Assets:CASH  -3.9 USD
    Expenses:GROCERIES  3.9 USD

2022-12-10 * Supermarket | Chicken, Cat Treats
    Liabilities:CREDIT1  -23.45 USD
    Expenses:GROCERIES  12.15 USD
    Expenses:PETS:CONSUMABLE  11.3 USD

2022-12-10 * Cosmetics | Cosmetics
    Liabilities:CREDIT1  -25 USD
    Expenses:SHOPPING:BEAUTY  25 USD

2022-12-11 * Shop | 2x Face Masks
    Assets:CASH  -3 USD
    Expenses:SHOPPING:HOME-GARDEN  3 USD

2022-12-12 * Online | Cat Products
    Liabilities:CREDIT2  -67.3 USD
    Assets:AR:XXX  57.85 USD
    Expenses:PETS:CONSUMABLE  9.45 USD

2022-12-12 * Landlord | Rental 2022-12
    Assets:SAVING   -1039 USD
    Expenses:HOME:RENTAL     1000 USD
    Expenses:UTILITY:COMMUNICATION  39 USD

2022-12-15 * Employer | Salary 2022-12
    Income:SALARY  -7745 USD
    Assets:SAVING   7745 USD

2022-12-15 * Employer | Annual Bonus 2022
    Income:SALARY  -7076 USD
    Assets:SAVING   7076 USD

2022-12-15 * Bank | CREDIT 2022-12
    Assets:SAVING    -50.58 USD
    Liabilities:CREDIT1  50.58 USD

2022-12-26 * Self | Transfer
    Assets:SAVING        -200 USD
    Assets:VCARD          200 USD

2022-12-16 * Bank | CREDIT 2022-12
    Assets:SAVING    -581.61 USD
    Liabilities:CREDIT1  581.61 USD

2022-12-29 * Shop | 4x Box
    Assets:CASH  -6 USD
    Expenses:EATINGOUT  6 USD

2022-12-29 * Shop | Drinking Water
    Assets:CASH  -0.9 USD
    Expenses:GROCERIES  0.9 USD

2022-12-31 * Restaurant | Breakfast
    Assets:CASH  -4 USD
    Expenses:EATINGOUT  4 USD

2022-12-31 * Pizzeria | Pizza
    Liabilities:CREDIT1       -26.75 USD
    Expenses:EATINGOUT              26.75 USD

2022-12-31 * Shop | Pepsi
    Assets:CASH  -1.9 USD
    Expenses:EATINGOUT  1.9 USD

2022-12-31 * McDonald's | Double Cheeseburger
    Liabilities:CREDIT1  -5 USD
    Expenses:EATINGOUT  5 USD

2022-12-31 * Self | 2022
    Equity:OPEN-BAL  -7200 USD
    Liabilities:AP:XXX  7200 USD

Expected behaviour

Expected to append the new entry only. Likely due to change in 0.4.6
Don't use append writing mode for compatibility with more content providers (https://github.com/chvp/NanoLedger/pull/162) @chvp
System information

Tried on ledger files synced by Syncthing or separate file not synced at all, selected via Android Document Picker

Samsung Galaxy S22, OneUI 6.1, Android 14, NanoLedger version 0.4.6 up to latest version 0.6.2
Poco F2 Pro, Lineage OS 21, Android 14, NanoLedger version 0.4.6 up to latest version 0.6.2
Mi 8, Lineage OS 21, Android 14, NanoLedger version 0.4.6 up to latest version 0.6.2

I am not able to reproduce this. In my case ledgers are synced via Nextcloud.

The snippet inside the report is cut off from the main file. As you did, when I tried to reproduce with the snippet as a new ledger file, it does not happen. But when I tried with the whole main file, it still happens.
I was thinking there may be illegal characters in the main file causing parsing and appending error. However, when I run my ledger file via hledger windows in strict mode, there is no error thrown.
May be you can create a new journal file with my old data  and can check

whether the bug happens or not
why it is happening

Thanks.

I won't have time until tomorrow evening, but I'll check it out then.

Sorry, my blood donation appointment ran out, it'll have to be for this weekend :confused:.

I wasn't able to reproduce this issue, but I had a look at the code and I'm pretty sure I caught the oversight that caused this behaviour. Would you be able to test this APK and check whether that fixes the issue?

It does solve the issue. Thanks for your kind help.

Re-opening since the fix isn't merged yet.


Kundera - Cassandra Transaction Rollback

In kundera-cassandra 3.2 the transaction management is not working. When using the EntityManager.merge(Entity) method kundera creates an INSERT statement instead of an UPDATE statement. If the transaction fails and I call the rollback method then kundera deletes the entity from the cassandra database instead of rollback the update.
Kundera-cassandra 3.2
Cassandra 2.2.4
Link to a more detail information: http://stackoverflow.com/questions/35013300/kundera-cassandra-transaction-rollback-and-entity-update

It was a mistake from me: I fetched the entity with another instance of EntityManager class and also in another transaction, therefore Kundera was not able to correctly rollback the transaction. Take a look at http://stackoverflow.com/questions/35013300/kundera-cassandra-transaction-rollback-and-entity-update
In my opinion this issue can be closed.

Closing the issue :)


Removed PR check in s3 deploy script

@dougmartin I think this can be closed now, right?

Yep - closing now.


Unable to parse binding expression

IsVisible="{Binding !$parent[cont|MetroWindow].DataContext.LockScreen.IsLocked}"
gives a runtime error.
but:
IsVisible="{Binding !$parent[Window].DataContext.LockScreen.IsLocked}"
works

gives a runtime error.

It's Avalonia issue then. Probably something related to the avalonia-specific runtime type resolver.


Implemented: spinner inside the logout button(#60)

Closes #60
Short Description and Why It's Useful

Screenshots of Visual Changes before/after (If There Are Any)

IMPORTANT NOTICE - Remember to add changelog entry
Contribution and Currently Important Rules Acceptance

[ ] I read and followed contribution rules

Closing as we need a loader instead of a spinner in the button.


pull request for e2e test at 01/06/2017 00:26:12

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


Paper reference

Hello, I was wondering if I could refer to this project on my paper.
I would like to know if there is any publicized research papers this project is based on, especially the text line detection part?
Thanks!

A paper about Eynollah has now been accepted at HIP'23 and will appear in August 2023. The citation information has already been added to the readme: https://github.com/qurator-spk/eynollah#how-to-cite.


Added the applicationID to the context object

This allows the server to check the context for the application ID to be verified
during AudioPlayback requests instead of automatically returning a 400 response
when the session property is not available.
Also added a new OnAudioPlayerState handler to handle requests caused by playback state changes.
Fixes #30

Looks good!


Merging SIMD parts

It merges SIMD parts together.
It waits for

[ ] #262

I think it's too early to merge things now.
I was developing my stuff inside a "simd" subdirectory to avoid conflict with the current SIMD implementation.
Since it's still a work in progress, I think we should keep them separate for now.


Create base16 schema

Hello!
I love this theme, its perfect!!
Thank you!!
Can you create a schema in base16 framework?
https://github.com/chriskempson/base16
I created for myself but is better if you can design your base16-theme for more consistence with your vim theme.
My theme kyotonight.yaml:
# scheme: "kyotonight-"
# author: "https://github.com/shrikecode/kyotonight.vim/"
scheme: "kyotonight"
author: "shrikecode"
base00: "1a1b26"
base01: "292e42"
base02: "32344a"
base03: "33467c"
base04: "545c7e"
base05: "a9b1d6"
base06: "c0caf5"
base07: "e4e4e5"
base08: "f7768e"
base09: "9ece6a"
base0A: "e0af68"
base0B: "7aa2f7"
base0C: "ad8ee6"
base0D: "73daca"
base0E: "ff9e64"
base0F: "db4b4b"

Sorry for the delay, I'm looking into this to make sure it's all nice, but your code looks good to me. But will let you know. I'm not very familiar with base16, so I need a moment, but it also seems pretty cool. :)

Ok, I think I've got something right here, but I tested it only with base16-vim template. Can you take a look and maybe suggest changes? @guifuentes8
scheme: "kyotonight"
author: "shrikecode"
base00: "1a1b26"
base01: "292e42"
base02: "32344a"
base03: "565f89"
base04: "545c7e"
base05: "a9b1d6"
base06: "c0caf5"
base07: "b4f9f8"
base08: "f7768e"
base09: "bb9af7"
base0A: "e0af68"
base0B: "7aa2f7"
base0C: "ad8ee6"
base0D: "73daca"
base0E: "7dcfff"
base0F: "db4b4b"

I also think that to follow what base16 is doing, I will create a new repo for this scheme (so you don't get all of the vim stuff and extras if you just want the scheme to run with builder).

It's a good idea to leave it in a new repo, it makes more sense. As for the theme you created, I think it's perfect! I had taken the colors from your theme and created them randomly (what came to mind =) ), but yours was more consistent with Base16's proposal. Everything is fine on my part, thank you very much for your help and attention!! @wmwnuk

base16 scheme repo was created: https://github.com/shrikecode/base16-kyotonight-scheme
Thanks for the suggestion!


[Question] How to transfer backup from one longhorn to another?

How to transfer backup from one longhorn to another?

I am not sure what is transfer backup from one Longhorn to another. A backup target can be shared by multiple Longhorn clusters.
And Longhorn can rely on a shared backup target to achieve some disaster recovery mechanisms, See the doc for details:
https://longhorn.io/docs/1.0.0/concepts/#disaster-recovery-volumes

@shuo-wu
For example, I have 5 longhorn separated instances in different cloud providers.
I need to just share some of backups, how to do?

Set the same backup target (and secret if necessary) on the Setting page.
https://longhorn.io/docs/1.0.0/snapshots-and-backups/backup-and-restore/set-backup-target/

Partially unrelated, but right now i've already imagined a way to quickly export/import longhorn volumes:

Detatch volume from pods
Attach to node
dd the raw device (under /dev/longhorn) into an iso file
Carry iso file to the target cluster
Create PVC with equal size, or create longhorn-static volume
Attach volume to node
dd the iso file into the raw device
Detach and start pods

I know the correct way to do it is via backups, but this can be a simple-n-quick way to transfer big volumes (>1TB) over large distances, or between air-gapped solutions, without needing to bisect the blocks out of the backup target that're required for this volume.
(note: I've only learned of dd just a while back about how useful it is, please read up on it before attempting to do this, and also always please make a snapshot/backup before attempting to do this)

@ShadowJonathan Your solution sounds practical.
But I don't follow without needing to bisect the blocks out of the backup target that're required for this volume. The backup means uploading/copying the volume data to somewhere. The restore means downloading/fetching data from somewhere to a new volume. I think your solution is pretty similar to backup & restore.

Iirc (and please correct me if im wrong), if multiple volumes have the same backup target, their blocks arent seperated by sub-directory/path, and instead all mingle while metadata files can annotate the blocks needed for that specific backup target, and refer to them by their hashes.
So either all of those hashed blocks are non-seperated by volume, or seperated by volume, I thought it was the former one, but if it's the latter one, a backup target can be viable.
My solution is more a "folder of (iso) disks" for "classic" cold storage, and/or for a more simple-and-quick migration.

Each backup volume holds its own directory and the data blocks. The data blocks of all backup (all history backup and the latest backup) of this volume are mingled. But the metadata file of each backup can indicate the data blocks it consists of.

Yeah, my note specifically was that putting it on an iso would make it more straightforward than resolving all relevant blocks from a latest backup on that service (if you dont/cant connect to that service from your destination cluster) and copy-pasting them in the same structure on the end side.
Maybe something interesting like "download ISO" of a snapshot/backup could be done from the UI, which'd handle all of this for you, but from/for a specific snapshot so forensics/diagnostics can be applied on it.

Do you mean this? It's not via UI. Since Longhorn is all about volume, it can't suddenly create a file somewhere and it doesn't know where it stores the file.
https://longhorn.io/docs/1.0.0/references/examples/#restore-to-file

No, I'm not referring to that, I'm referring to ssh-ing into the host node, and then attaching the disk to it so it's visible from /dev/longhorn/, then you get the raw device, not a mounted file system, which could be a critical difference.
With a raw device you can get a byte-by-byte perfect copy, but with mounting on a filesystem you'd get a subdirectory copy.

Yeah. If an iso(right now we are using raw or qcow format) file can be download from UI based on a backup or a snapshot, that's more straightforward. We need to check if it's possible first.


grpcproxy: support WithRequireLeader on watchers

Adds leaderNotifier to notify all require leader streams when
the leader is lost. Adds better error support to the grpc
stream adapters.
Fixes #6303

superseded by #6822


clearing the already loaded collections and keywords

Hi,
I tried to understand the rfhub2 project regarding the smart documentation of keywords. I installed rfhub2 via pip and setup the localhost:8000 . Now I have 100+ collections in my localhost and I want to remove them all and start from fresh but not sure how to do this. I request you to please help me in this regard.
Thanks

Hi @divakar5,
You can do it in three ways:

You can go to SWagger -> localhost:8000/docs and triggerdelete on collections: ![image](https://user-images.githubusercontent.com/35737166/164202384-ef608ef8-f865-41e6-90ed-ed62700afbba.png) You will be asked for credentials, default is rfhub:rfhub`
You can run cli using insert mode, and providing no collection to the path ie. rfhub2-cli --no-installed-keywords --load-mode=insert 'path_to_no_library'
You can delete .db file which will be recreated in a place where You started rfhub2 server

Thank you! it worked


Peak memory usage may be suboptimal

I’ve done some measurements of how much memory StrobeAlign uses depending on the reference size (by truncating a single reference to varying lengths). This is the resulting pattern:

There’s a step at around 50, 100, 200, 400. I had to stop at a reference size of 570 Mbp because I did this on my machine at home which doesn’t have enough RAM for longer references.
The pattern could be explained by an std::vector that is being appended to. (Since an std::vector doubles its size when it needs to grow.) There are lot of calls to std::vector.reserve() already, perhaps one of them needs to be adjusted or we need one more.

Ok, it’s the mers_index (hash map of type robin_hood). The mers_index.load_factor() is 0.77 at 90 Mbp and then goes down to 0.42 for 100 Mbp. I’m not sure, but does the robin_hood implementation only allow hash table sizes that are powers of two? Then there’s little we can do.

Agreed with your last comment.
The initial allocation of the flat_vector is based on a prediction of the number of seeds that will be created from the reference. The prediction is roughly given by seed_thinning_factor * ref_length. Therefore, it should be fairly good memory allocation.
I don't know about the intrinsic of the robin hood hash table. It is a bit memory consuming and I read somewhere that 2/3 of the memory is consumed with storing 'internal nodes'. The hash table is indeed the most consuming one in strobealign.  This hash table benchmark from 2022 (found by @johan-gson) may suggest viable alternatives, if any.

Short summary of what I’ve learned since opening this issue:

Yes, robin_hood hash tables can only have sizes that are a power of two.
The overhead is one byte per entry. Our entries have 16 bytes, so memory usage is 17 bytes times size of the table.
I haven’t really found a viable alternative hash table implementation that does not resize to powers of two.

As I said above, I don’t think there’s much we can do. Also, we’ve reduced memory usage already quite a bit in other ways, so maybe this is not as pressing.

The changes in memory layout have also resolved this issue.


0012 - Implement code of conduct provision from bylaws

Steps to formally implement the code of conduct provision of our bylaws.

Updated this PR to

Include example CODE_OF_CONDUCT.md in the MPSF repo
Add language to the above file about who reviews reports of misconduct and that MP Staff are subject to LBNL's required reporting policies.

This has been majority approved, pending MP leadership review. Action item for @tschaume and @yang-ruoxi to follow up and merge.

As to the question of enforcement, I recently found this section from another scientific project. I'll leave here for reference:
https://qsdsan.readthedocs.io/en/latest/CODE_OF_CONDUCT.html#enforcement

As discussed in the last foundation meeting, we can merge this and start a separate PR for the implementation of policies and enforcement (if needed).


Compiler produces invalid JavaScript for multiline strings if input file uses CRLF line endings

SSCCE:
module Main exposing (..)

{-| Make sure to save with Windows line endings!
-}

import Browser
import Html exposing (Html)

multilineString : String
multilineString =
    """Multiple
line
string
"""

main : Program () () ()
main =
    Browser.staticPage (Html.text multilineString)

Save this using CRLF line endings, and the resulting HTML will have the string literal split across multiple lines, leading to a JavaScript syntax error; it should look something like
var author$project$Main$multilineString = 'Multiple
\nline
\nstring
\n';

If instead the file is saved with Unix line endings, the output looks like
var author$project$Main$multilineString = 'Multiple\nline\nstring\n';

I am (as you might guess!) on Windows 10 using Elm 0.19 (second alpha).
Note: make sure your editor doesn't auto-convert to LF on save, or is configured to run elm-format on save (which looks like it also normalizes to LF). This bug was frustrating to track down because I had that setup, so the issue would only trigger on freshly checked out files (Git was configured to check out as CRLF), but then would disappear as soon as I made any change to the files in question (since they would at that point silently get switched to LF).

Thanks for the great report! The fix will produce the following JS:
var author$project$Main$multilineString = 'Multiple\r\nline\r\nstring\r\n';

This is what it compiles to in Elm 0.18 for me (tested on Ubuntu 18.04):
var _user$project$Main$multilineString = 'Multiple\r\nline\r\nstring\r\n';

Similar Elm 0.18 program with CRLF
module Main exposing (..)

{-| Make sure to save with Windows line endings!
-}

import Html exposing (Html)

multilineString : String
multilineString =
    """Multiple
line
string
"""

main : Html msg
main =
    Html.text multilineString

elm-format converts the CRLFs to LF, which technically changes the meaning of the program (the string contents has changed), so perhaps we should open an issue for elm-format as well ... or should Elm compile \r\n to \n in multiline strings? Perhaps that could help prevent confusing bugs. CoffeeScript works like that, btw.

Related elm-format discussion was here: https://github.com/avh4/elm-format/issues/74#issuecomment-267775197, but apart from the bug that was fixed there, no one else has logged any issues related to eol handling on Windows.

Cool, thanks! Looks like we're all good then.

I tried looking for (relatively) authoritative references that specifically discussed handling of \r and \n in multiline string literals, but couldn't find a lot...so far it looks like in C# "Even carriage returns and line feeds can appear in the literal" while the Swift book is pretty explicit: "Line endings in a multiline string literal are normalized to use the line feed character. Even if your source file has a mix of carriage returns and line feeds, all of the line endings in the string will be the same." (I was able to read that text using Google Books, but you may hit reading limits.)
Would it be helpful for me to just try things out in various languages that support multiline string literals and see what happens? (I'd be a bit worried that if the behavior isn't specified somewhere, then it could just be an accident of the implementation instead of a conscious choice.) The Swift example is the only place I could find where it looked like someone had actually put some deliberate thought into the issue.

should Elm compile \r\n to \n in multiline strings?

Creating multiline JS strings using backticks results in \n (and only \n) inserted at the newlines. Running the following on windows with node v9:
multiline.js
module.exports = `first line
second line`;

$ node
> require('./multiline.js')
'first line\nsecond line'
> fs.readFileSync('./multiline.js', 'utf8')
'module.exports = `first line\r\nsecond line`;'

I would argue that elm should do the same.


Version check

Include a version check/outdated notice in logs and settings screen, for binaries and docker version

Complete. in v1.11.0


fix: Performance improvement of /new API by updating the api used for fetching apps. Eliminating public applications from the result

Fixes #16498

/ok-to-test sha=f07bd9e

/ok-to-test sha=b59f16b


Use more appropriate package for generated go.

In general in go, package names should match the directory name they're in, this lines them up.

Hey @tmc.
We had taken note of your PR. With the v1.11.1 release, this is resolved. Thank you.


adding pipenv files to .gitignore

Description

Adding Pipfile and Pipfile.lock to .gitignore, so those of us who like to use pipenv won't accidentally push pipfiles to repo on our pull requests.
Types of Changes

[ ] Core
[ ] Bugfix
[ ] New feature
[ ] Enhancement/optimization
[ ] Documentation

Issues Fixed or Closed by This PR

Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] I have tested the changes and verified that they work and don't break anything (as well as I can manage).

Thanks for contributing to Strawberry! 🎉 You've been invited to join
the Strawberry GraphQL organisation 😊
You can also request a free sticker by filling this form: https://forms.gle/dmnfQUPoY5gZbVT67
And don't forget to join our discord server: https://strawberry.rocks/discord 🔥


Accessing UI running in container throws a panic

The second i try to hit my ui on any page. it throws below stack. I disabled the panicfilter as suggested in another thread similar to this.
Below lines are the culprit of the panic it seems my module returns nil.
This works fine if i run the server manually outside of a container.
        println("Returned ", module)         action := module.Namespace() + recvType.Name() + "." + method.Name
Stack:
Listening on.. localhost:80

Returned  0x0
INFO  22:49:29    app server.go:2753: http: panic serving <IP_ADDRESS>:47290: runtime error: invalid memory address or nil pointer dereference
goroutine 40 [running]:
net/http.(*conn).serve.func1(0xc42019e000)
/usr/local/go/src/net/http/server.go:1721 +0xd0
panic(0xc223c0, 0x13b7390)
/usr/local/go/src/runtime/panic.go:489 +0x2cf
github.com/revel/revel.getRedirectURL(0xbe7ec0, 0xd53e98, 0x0, 0x0, 0x0, 0x708699,
0xcab3e0, 0xc4205163b0, 0x6)
/go/src/github.com/revel/revel/results.go:472 +0x271
github.com/revel/revel.(*RedirectToActionResult).Apply(0xc42058aff0, 0xc4203b5d40,
0xc4203fd900)
/go/src/github.com/revel/revel/results.go:439 +0x5c
github.com/revel/revel.handleInternal(0x13d09a0, 0xc420411de0)
/go/src/github.com/revel/revel/server-engine.go:147 +0x9cc
github.com/revel/revel.(*GoHttpServer).Handle(0xc42040c570, 0x13d4da0, 0xc420122380,
0xc4201fc200)
/go/src/github.com/revel/revel/server_adapter_go.go:101 +0x239
github.com/revel/revel.(*GoHttpServer).Init.func3(0x13d4da0, 0xc420122380, 0xc4201fc200)
/go/src/github.com/revel/revel/server_adapter_go.go:43 +0x48
net/http.HandlerFunc.ServeHTTP(0xc4204038d0, 0x13d4da0, 0xc420122380, 0xc4201fc200)
/usr/local/go/src/net/http/server.go:1942 +0x44
net/http.serverHandler.ServeHTTP(0xc420241d90, 0x13d4da0, 0xc420122380,
0xc4201fc200)
/usr/local/go/src/net/http/server.go:2568 +0x92
net/http.(*conn).serve(0xc42019e000, 0x13d5c60, 0xc42048a000)
/usr/local/go/src/net/http/server.go:1825 +0x612
created by net/http.(*Server).Serve
/usr/local/go/src/net/http/server.go:2668 +0x2ce  section=system

Any idea what i'm doing wrong? this was all working on revel 0.17.0

Looks like this might actually be a dupe of Issue #1263
going to try to change my redirects and see

Ok so i'm out of the woods but i still don't understand why i had to change all my redirects.

how you defined your redirect route? If only use "c.Redirect(App.Index)" will introduce the error (but works fine before 0.18. The same way to use "c.Redirect(routes.App.Index())", works before and after 0.18. Just need import app routes.

I tried both ways. routes.App.Index and /app/index  and (*App).Index
routes.App.Index() throws an error.

routes.App.Index() is format, you should using your own type and func names. If routes.App.Index() function throws error, check your defined controller and functions. it should return a url string.
I noticed you run app in container, set http.address= <IP_ADDRESS> in app.conf to listen all ip address (coming).
And another issue (0.18) is do not set "result.compressed = true" keep it false (default is false).

@lujiacn The results.compressed config option should be fixed now.
I will check into seeing why the routes.App.Index() is breaking things

@notzippy Great! Thanks :)


Add animal module

Add a new module with following functionalities

bear
bird
cat
cetacean
cow
crocodilia
dog
fish
horse
insect
lion
rabbit
rodent
snake
type

Hello :)
I'll take this issue, can you assign me pls ?
I'll mostly take data from faker-js

Hello, sure, assigned :)


Add Document Deletion Functionality to Workspace

What would you like to be added:
Currently, there is no functionality to delete documents within the Workspace. It is essential to introduce a feature that allows users to delete documents. Additionally, it would be beneficial to implement a feature that moves deleted documents to a "Recycle Bin" or similar location, enabling users to restore them if needed.
Why is this needed:
The ability to delete documents is a fundamental feature that enhances user control and organization within the Workspace. Providing a mechanism to move deleted documents to a separate area for potential restoration adds flexibility and usability to the platform.
Additional Information:

This issue is duplicated with #141.

Hi 👋 May I try this?

@dener8
Sure!
Refer to this Figma file: https://www.figma.com/design/OYc1Cr0nvFuBnWZxhscfDk/Code-Pair?node-id=131-16396&t=teU9epAOrf5Cz8Gh-1

@dener8 Could you share your progress? If there are any issues, I can help.


Allow using the current database during for tests

... by setting TEST_TOKEN=none
During development it can be helpful to use the current database to run tests on.
Just a suggestion, if you agree, I would add the changelog.

1. Why is this change necessary?
It's a dev-experience pull request.
During development it can be helpful to test on the current database, instead of adding a TEST_TOKEN
2. What does this change do, exactly?
Allowing setting the TEST_TOKEN to "none", to explicitly not add it
3. Describe each step to reproduce the issue or behaviour.
in the .env test

TEST_TOKEN=none
Run tests
The should use the current database and not try to setup a new one

4. Please link to the relevant issues (if any).
5. Checklist

[not applicable] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[not yet] I have written or adjusted the documentation according to my changes
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hey @amenk thanks for your contribution! 🎉 💙
Can you provide a little changelog entry?

@raknison done, Linter problem also fixed, where would the TEST_TOKEN parameter be documents (seems in general undocumented so far)

@raknison Changelog check fails, because there is no JIRA ticket

@raknison How do I get a Jira entry? Shall I just create a issue?

@amenk its fine we set it internally

@shyim great - is anything from my side left to-do on this MR or will the merge be processed?

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-21608
Please use this issue to track the state of your pull request.

Hi @amenk ,
thank you for your distribution, your change is now merged (96ffb46) 🎉 💙


RPC timeouts are not handled

RPC timeout causes crash..
How to reproduce - run main.py, wait 10 seconds..
`
bob@coin:~/bitcoind-ncurses2$ python3 main.py
cookiefile not found, falling back to password authentication
Traceback (most recent call last):
File "/home/bob/bitcoind-ncurses2/rpc.py", line 128, in _fetch
async with session.post(self._url, headers=self._headers, data=req) as response:
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/client.py", line 690, in aenter
self._resp = yield from self._coro
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/client.py", line 277, in _request
yield from resp.start(conn, read_until_eof)
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/client_reqrep.py", line 624, in start
(message, payload) = yield from self._protocol.read()
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/streams.py", line 554, in read
yield from self._waiter
File "/usr/lib/python3.5/asyncio/futures.py", line 380, in iter
yield self  # This tells Task to wait for completion.
File "/usr/lib/python3.5/asyncio/tasks.py", line 304, in _wakeup
future.result()
File "/usr/lib/python3.5/asyncio/futures.py", line 285, in result
raise CancelledError
concurrent.futures._base.CancelledError
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/bob/bitcoind-ncurses2/rpc.py", line 129, in _fetch
return await response.text()
File "/home/bob/.local/lib/python3.5/site-packages/async_timeout/init.py", line 35, in exit
self._do_exit(exc_type)
File "/home/bob/.local/lib/python3.5/site-packages/async_timeout/init.py", line 80, in _do_exit
raise asyncio.TimeoutError
concurrent.futures._base.TimeoutError
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "main.py", line 261, in
mainfn()
File "main.py", line 250, in mainfn
loop.run_until_complete(t)
File "/usr/lib/python3.5/asyncio/base_events.py", line 454, in run_until_complete
self.run_forever()
File "/usr/lib/python3.5/asyncio/base_events.py", line 421, in run_forever
self._run_once()
File "/usr/lib/python3.5/asyncio/base_events.py", line 1424, in _run_once
handle._run()
File "/usr/lib/python3.5/asyncio/events.py", line 126, in _run
self._callback(*self._args)
File "/usr/lib/python3.5/asyncio/tasks.py", line 307, in _wakeup
self._step(exc)
File "/usr/lib/python3.5/asyncio/tasks.py", line 241, in _step
result = coro.throw(exc)
File "main.py", line 64, in poll_client
await callback(method, d)
File "main.py", line 176, in on_bestblockhash
await blockview.on_bestblockhash(key, obj)
File "/home/bob/bitcoind-ncurses2/block.py", line 416, in on_bestblockhash
await self._blockstore.on_bestblockhash(bestblockhash)
File "/home/bob/bitcoind-ncurses2/block.py", line 106, in on_bestblockhash
block = await self.get_block(blockhash)
File "/home/bob/bitcoind-ncurses2/block.py", line 30, in get_block
j = await self._client.request("getblock", [blockhash])
File "/home/bob/bitcoind-ncurses2/rpc.py", line 142, in request
j = await self._fetch(session, req)
File "/home/bob/bitcoind-ncurses2/rpc.py", line 131, in _fetch
raise RPCTimeoutError
rpc.RPCTimeoutError
Task was destroyed but it is pending!
task: <Task pending coro=<ThreadedResolver.resolve() done, defined at /home/bob/.local/lib/python3.5/site-packages/aiohttp/resolver.py:28> wait_for=<Future pending cb=[_chain_future.._call_check_cancel() at /usr/lib/python3.5/asyncio/futures.py:452, Task._wakeup()]> cb=[shield.._done_callback() at /usr/lib/python3.5/asyncio/tasks.py:719]>
Task was destroyed but it is pending!
task: <Task pending coro=<ThreadedResolver.resolve() done, defined at /home/bob/.local/lib/python3.5/site-packages/aiohttp/resolver.py:28> wait_for=<Future pending cb=[_chain_future.._call_check_cancel() at /usr/lib/python3.5/asyncio/futures.py:452, Task._wakeup()]> cb=[shield.._done_callback() at /usr/lib/python3.5/asyncio/tasks.py:719]>
Exception ignored in: <coroutine object poll_client at 0x7fb26f065d00>
Traceback (most recent call last):
File "main.py", line 58, in poll_client
File "/home/bob/bitcoind-ncurses2/rpc.py", line 142, in request
File "/home/bob/bitcoind-ncurses2/rpc.py", line 128, in _fetch
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/client.py", line 690, in aenter
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/client.py", line 280, in _request
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/connector.py", line 111, in close
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/connector.py", line 495, in _release
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/client_proto.py", line 52, in close
File "/usr/lib/python3.5/asyncio/selector_events.py", line 622, in close
File "/usr/lib/python3.5/asyncio/base_events.py", line 572, in call_soon
File "/usr/lib/python3.5/asyncio/base_events.py", line 357, in _check_closed
RuntimeError: Event loop is closed
Exception ignored in: <coroutine object poll_client at 0x7fb26f0659e8>
Traceback (most recent call last):
File "main.py", line 58, in poll_client
File "/home/bob/bitcoind-ncurses2/rpc.py", line 142, in request
File "/home/bob/bitcoind-ncurses2/rpc.py", line 128, in _fetch
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/client.py", line 690, in aenter
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/client.py", line 280, in _request
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/connector.py", line 111, in close
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/connector.py", line 495, in _release
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/client_proto.py", line 52, in close
File "/usr/lib/python3.5/asyncio/selector_events.py", line 622, in close
File "/usr/lib/python3.5/asyncio/base_events.py", line 572, in call_soon
File "/usr/lib/python3.5/asyncio/base_events.py", line 357, in _check_closed
RuntimeError: Event loop is closed
Exception ignored in: <coroutine object poll_client at 0x7fb26f065938>
Traceback (most recent call last):
File "main.py", line 58, in poll_client
File "/home/bob/bitcoind-ncurses2/rpc.py", line 142, in request
File "/home/bob/bitcoind-ncurses2/rpc.py", line 128, in _fetch
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/client.py", line 690, in aenter
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/client.py", line 280, in _request
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/connector.py", line 111, in close
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/connector.py", line 495, in _release
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/client_proto.py", line 52, in close
File "/usr/lib/python3.5/asyncio/selector_events.py", line 622, in close
File "/usr/lib/python3.5/asyncio/base_events.py", line 572, in call_soon
File "/usr/lib/python3.5/asyncio/base_events.py", line 357, in _check_closed
RuntimeError: Event loop is closed
Exception ignored in: <coroutine object poll_client at 0x7fb26f065780>
Traceback (most recent call last):
File "main.py", line 58, in poll_client
File "/home/bob/bitcoind-ncurses2/rpc.py", line 142, in request
File "/home/bob/bitcoind-ncurses2/rpc.py", line 128, in _fetch
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/client.py", line 690, in aenter
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/client.py", line 280, in _request
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/connector.py", line 111, in close
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/connector.py", line 495, in _release
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/client_proto.py", line 52, in close
File "/usr/lib/python3.5/asyncio/selector_events.py", line 622, in close
File "/usr/lib/python3.5/asyncio/base_events.py", line 572, in call_soon
File "/usr/lib/python3.5/asyncio/base_events.py", line 357, in _check_closed
RuntimeError: Event loop is closed
Exception ignored in: <coroutine object poll_client at 0x7fb26f0657d8>
Traceback (most recent call last):
File "main.py", line 58, in poll_client
File "/home/bob/bitcoind-ncurses2/rpc.py", line 142, in request
File "/home/bob/bitcoind-ncurses2/rpc.py", line 128, in _fetch
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/client.py", line 690, in aenter
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/client.py", line 280, in _request
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/connector.py", line 111, in close
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/connector.py", line 495, in _release
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/client_proto.py", line 52, in close
File "/usr/lib/python3.5/asyncio/selector_events.py", line 622, in close
File "/usr/lib/python3.5/asyncio/base_events.py", line 572, in call_soon
File "/usr/lib/python3.5/asyncio/base_events.py", line 357, in _check_closed
RuntimeError: Event loop is closed
Task exception was never retrieved
future: <Task finished coro=<poll_client() done, defined at main.py:52> exception=RPCTimeoutError()>
Traceback (most recent call last):
File "/home/bob/bitcoind-ncurses2/rpc.py", line 128, in _fetch
async with session.post(self._url, headers=self._headers, data=req) as response:
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/client.py", line 690, in aenter
self._resp = yield from self._coro
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/client.py", line 277, in _request
yield from resp.start(conn, read_until_eof)
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/client_reqrep.py", line 624, in start
(message, payload) = yield from self._protocol.read()
File "/home/bob/.local/lib/python3.5/site-packages/aiohttp/streams.py", line 554, in read
yield from self._waiter
File "/usr/lib/python3.5/asyncio/futures.py", line 380, in iter
yield self  # This tells Task to wait for completion.
File "/usr/lib/python3.5/asyncio/tasks.py", line 304, in _wakeup
future.result()
File "/usr/lib/python3.5/asyncio/futures.py", line 285, in result
raise CancelledError
concurrent.futures._base.CancelledError
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/bob/bitcoind-ncurses2/rpc.py", line 129, in _fetch
return await response.text()
File "/home/bob/.local/lib/python3.5/site-packages/async_timeout/init.py", line 35, in exit
self._do_exit(exc_type)
File "/home/bob/.local/lib/python3.5/site-packages/async_timeout/init.py", line 80, in _do_exit
raise asyncio.TimeoutError
concurrent.futures._base.TimeoutError
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "main.py", line 261, in
mainfn()
File "main.py", line 250, in mainfn
loop.run_until_complete(t)
File "/usr/lib/python3.5/asyncio/base_events.py", line 454, in run_until_complete
self.run_forever()
File "/usr/lib/python3.5/asyncio/base_events.py", line 421, in run_forever
self._run_once()
File "/usr/lib/python3.5/asyncio/base_events.py", line 1424, in _run_once
handle._run()
File "/usr/lib/python3.5/asyncio/events.py", line 126, in _run
self._callback(*self._args)
File "/usr/lib/python3.5/asyncio/tasks.py", line 307, in _wakeup
self._step(exc)
File "/usr/lib/python3.5/asyncio/tasks.py", line 241, in _step
result = coro.throw(exc)
File "main.py", line 64, in poll_client
await callback(method, d)
File "main.py", line 176, in on_bestblockhash
await blockview.on_bestblockhash(key, obj)
File "/home/bob/bitcoind-ncurses2/block.py", line 416, in on_bestblockhash
await self._blockstore.on_bestblockhash(bestblockhash)
File "/home/bob/bitcoind-ncurses2/block.py", line 106, in on_bestblockhash
block = await self.get_block(blockhash)
File "/home/bob/bitcoind-ncurses2/block.py", line 30, in get_block
j = await self._client.request("getblock", [blockhash])
File "/home/bob/bitcoind-ncurses2/rpc.py", line 142, in request
j = await self._fetch(session, req)
File "/home/bob/bitcoind-ncurses2/rpc.py", line 131, in _fetch
raise RPCTimeoutError
rpc.RPCTimeoutError
`


Update deps of @fluid-internal/eslint-plugin-fluid

Description
Update deps of @fluid-internal/eslint-plugin-fluid.
Some were not fully updated as they require newer NodeJS
Reviewer Guidance
The review process is outlined on this wiki page.

Looks good!
How urgently do we need to update ingestion of this? If we need to publish a new version, might be worth bumping the version here. Looks like the most recently published was 0.1.1. https://www.npmjs.com/package/@fluid-internal/eslint-plugin-fluid?activeTab=versions

I don't have any urgency. Updates just seemed like a good idea, and I had these changes so I figured I might as well make a PR.
I should note I didn't test integrating this into other release groups, which we might want to do before merging this.


Add Warning when writes are ignored for idempotence

What went wrong?
There's no any warning when a write is ignored because the combination of txnAppId and txnVersion already exists in the log.
We should let the user know when this happends.

Merged on https://github.com/Qbeast-io/qbeast-spark/pull/284


Could not find com.voxeet.sdk:uxkit:3.2.0.

Sorry to bother you but I really do not know what went wrong when I try to migrate the V2.4.0 to the V3.2.0.
First, I add  maven { url "https://android-sdk.voxeet.com/release" }  in build.gradle file at the project layer.
Second, I add the api ('com.voxeet.sdk:uxkit:3.2.0') { transitive = true } in build.gradle file at the app layer.
But I run the project when something wrong happened as below:

Please do me a favor if you have something to tell me.
Thanks!

Inside the logs, you created the conference however you don't try to join it. You either don't use the join method or don't resolve the promise returned by the call

Inside the logs, you created the conference however you don't try to join it. You either don't use the join method or don't resolve the promise returned by the call

Yes.
But I tried the code：
VoxeetSDK.conference().create(getModel().getConference()).then(new JoinPromise()).error(new ConnectFailurePromise());, which was not receiving anything within the JoinPomise class as below:
private class JoinPromise extends DefaultExecPromise<CreateConferenceResult> {

        @Override
        public void onCall(CreateConferenceResult result, Solver<Object> solver) {
            LogUtil.getInstance().w("JoinPromise");
            try {
                if (result != null && result.conferenceId != null) {
                    VoxeetSDK.conference().join(VoxeetSDK.conference().getConference(result.conferenceId)).then(new ConnectPromise()).error(new ConnectFailurePromise());
                } else {
                    onConnectErrorCallback("Invalid Conference.");
                }
            } catch (Exception e) {
                LogUtil.getInstance().w("JoinPromise error: " + e.getMessage());
                onConnectErrorCallback(e);
            }
        }

    }

The issue here is that you made the class expects a CreateConferenceResult when it's actually a Conference
On a sidenote, you are using a promise which is expecting the resolve its solver, the new version can let you use .then((ThenVoid) object -> { ... });
And when you have intermediates promises you can use the ThenPromise<InputClass, OutputClass>

(ThenVoid) object -> { ... }

Thank you very much！
I tried like below：
VoxeetSDK.conference().create(getModel().getConference()).then((ThenVoid<Conference>) object -> {
                    try {
                        if (object != null && !TextUtils.isEmpty(object.getId())) {
                            VoxeetSDK.conference().join(VoxeetSDK.conference().getConference(object.getId())).then(new ConnectPromise()).error(new ConnectFailurePromise());
                        } else {
                            onConnectErrorCallback("Invalid Conference.");
                        }
                    } catch (Exception e) {
                        onConnectErrorCallback(e);
                    }
                }).error(new ConnectFailurePromise());

then succeed.


go build error

$ cd eris-db
$ git checkout v0.12.0
$ cd cmd & go build
github.com/eris-ltd/eris-db/event
../event/events.go:74: this.eventSwitch.AddListenerForEvent undefined (type *events.EventSwitch is pointer to interface, not interface)
../event/events.go:80: this.eventSwitch.RemoveListener undefined (type *events.EventSwitch is pointer to interface, not interface)
$ go version
go version go1.6.2 linux/amd64

We use glide for dependency management, please run glide install after checking out a version, as mentioned in the instructions https://github.com/eris-ltd/eris-db#for-developers


Fixed markdown headings

The headings in the README.md were missing a space between the hash and the heading.

Duplicate of https://github.com/Coopdis/easyMesh/pull/29


finish the lab

Objectives
Clone the project repo.
Modify the html file as per requirements.
Modify the css file as per requirements.
Modify the javascript file as per requirements.
Verify if the app is working properly.
Upload to GitHub and submit the github pages URL for peer review.

final project finish lab


Message box does not collapse in Production

Hi, I followed https://demo.creativebulma.net/components/bulma-collapsible/1.0/usage/#accordion . To create an accordion with multiple message boxes. In local, it works fine and it is great. However it does not collapse in production.
I deployed to Heroku as right now the app is in FREE tier.
Ruby: 2.7.0
Rails: 6.0
Bulma-collapsible: 1.0.4
Bulma: https://github.com/joshuajansen/bulma-rails
Do I need to install bulma npm packages? Or Bulma-rails is enough?

I found the problem that bulma-rails somehow does not work. So I add a cdn script and it works well.


Add dist env detection via env vars

Before submitting

[X] Was this discussed/agreed via a Github issue? (no need for typos and docs improvements)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[X] Did you write any new necessary tests?

What does this PR do?
Fixes #91
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in GitHub issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

@tchaton here is the PR based on our conversations in #91. I realized that there are several other objects that used _DistributedEnv, so decided not to go down the route of passing _DistributedEnv as an optional argument. Instead, I went with adding environment variables that can override the detected vars.
Two things regarding this PR's precheck.

I couldn't find a Contributor guideline file, is this signifcantly different than the lightning contributer guideline file?
I didn't find a lot of documentation worth updating, but if you know of any documentation relevant here, I would be happy to update.


Handling an error  from c.BindQuery(&params) properly

Hi! I think it will be great if it allows to handle an error after calling c.BindQuery(&params). For example, I need to return 400 Bad Request if any query parameter is invalid.
My proposal is to add another func like FilterByQueryParams(params QueryParams, config int) func(db *gorm.DB) *gorm.DB and make queryParams public. At first glance these changes fit with current API.
I can prepare PR, but decided to discuss before writting any code

Yes please

Hi @ildargafarov! Thanks for your issue! It's a good point that there should be proper error handling. But instead of a new function, I suggest using the db.AddError() function. It looks quite convenient for users to handle it that way and should be quite simple from code perspective. Do you mind creating a PR?

Hi @ActiveChooN Thanks for your answer. I disagree with you suggestion, to my regret. I think there is nothing in common between validating API parameters and ORM errors. From my point of view, these errors don't relate to DB level. I understand that you strive for retaining the current library's API, but it had incorrect design from the beginning. I have chosen to implement my own solution, so I won't create a PR. Sorry.
By the way, this is not only one thing that have influenced my decision. I have found that the lib doesn't work well with integer, bool and nil values. That is significant issue for me

@ildargafarov, thank you for your reply. After looking at your proposal a second time, I can actually see more benefits in it. It can make it easy to use lib with different params source (pagination with X-Pagination headers) or names (limit and offset) or even not rely on gorm lib. And I don't mind breaking API specs before a major release. So feel free to create PR with your proposed solution. Also feel free to create any issues related to non-string values.

Ok, I'll push a PR soon


AWS details progress bars

This replaces the pie chart with PF4 progress bars in the AWS details page. There is no mock, but worked with Natalie to create progress bars similar to the overview page. The difference here is that we only display the top 3 and use a scrollable modal to show more.

Codecov Report

Merging #476 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #476   +/-   ##
=======================================
  Coverage   85.68%   85.68%
=======================================
  Files         148      148
  Lines        2158     2158
  Branches      353      353
=======================================
  Hits         1849     1849
  Misses        271      271
  Partials       38       38

Impacted Files
Coverage Δ

src/testIds.ts
100% <ø> (ø)
:arrow_up:

src/components/commonChart/chartUtils.ts
81.57% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Improve three-valued-logic processing for NOT-Predicate

Lets add 3 categories of functions:

Null input = null output. We can optimize these with an additional fields exist check
Never null output. Like concat: null || null returns an empty string. I think we could just use a not-query in that case, without   applying 3vl and without applying the fields exist check. I think currently that broke the concat (?)
Conditional/unclear. NULL might result in NULL, but it might not. case is the prime example here. We need to use strict 3vl for them.

I think this kind/category/behaviour description should become part of the function somehow, instead of maintaining a list in the NotPredicate. Using the feature property we already have might work.
Follow up https://github.com/crate/crate/pull/15104#pullrequestreview-1755171524

I think this kind/category/behaviour description should become part of the function somehow

FWIW, PostgreSQL has the proisstrict flag in pg_proc


Commit author and message are not working properly

I've just enabled wslgit.exe and it works great inside of Visual Studio Code with a couple issues:

As you can see there is no message and an incorrect author. Am I doing something incorrectly here?

Regarding the author, you probably haven't setup Git inside WSL. As wslgit forwards all calls to the WSL version of Git, you need to configure it there, i.e. you need to run something like
$ git config --global user.name "John Doe"
$ git config --global user.email<EMAIL_ADDRESS>
inside WSL to set the author name and email address.
Commit messages should work just fine, either using -m "message" on the command line, or entering a commit message in VS Code and committing with Ctrl+Enter . Otherwise, can you provide more information how you committed this revision (e.g. versions of VS Code, wslgit, WSL distro, VS Code extensions)?

@andy-5. Regarding the user name/email you were correct. Making that change fixed the issue. However the commit message still shows up as blank. I have been doing Ctrl+Enter as you mentioned.
If I use the command line in my terminal (not inside VS Code) it works great.
I am using wslgit release 0.6.0
I'm on VS Code 1.25.1 (latest).
Extensions:

Can you try disabling the GitLens extension? I remember someone reporting problems with GitLens in the past. Maybe it somehow affects the commit process.
You can also start VS Code with code --disable-extensions to temporary disable all extensions.

@andy-5 Okay I tried disabling just GitLens and that still didn't fix it, then I opened with no extensions like you mentioned and still no commit message... Any other ideas?

Can you post the log from the Git output panel within VS Code? Maybe we can find something of interest there.

hey @andy-5 sorry for the delay in writing back.  This is what I see when I commit:
> git commit --quiet --allow-empty-message --file -

Interesting, and now I can see you are using zsh. There have been some issues before with alternative shells. How did you setup zsh as your default shell? Do you start it from .bashrc?
In that case, you should only execute it if there is no command (using -c) passed to bash, as shown in https://github.com/andy-5/wslgit/issues/1#issuecomment-319322075 for the fish shell. Another possible solution is in https://github.com/andy-5/wslgit/issues/10#issuecomment-355671820 .
You could also try to set the Windows environment variable WSLGIT_USE_INTERACTIVE_SHELL to false, as explained in the README. This should prevent .bashrc from executing and therefore not invoke zsh (and it will also be faster).

@andy-5 Yep, that was it. I use .bashrc to instantiate zsh. I just added the line from https://github.com/andy-5/wslgit/issues/10#issuecomment-355671820 and it worked perfectly.  Thanks so much for taking the time to help me troubleshoot!


Adding fetch_upload method to client

https://github.com/zulip/python-zulip-api/issues/628

@Ponni-M can you restructure the commits to first refactor the /api/ prefix stuff, with a commit message explaining why that's correct, and then one adding the new endpoint?
Check out the Zulip commit message guidelines for more details: https://zulip.readthedocs.io/en/latest/contributing/version-control.html#commit-messages

@Ponni-M thanks for working on this added a comment. Everything else LGTM.


fixing the prefix and suffix for multiple databite display options

Hey @jeffrainey - is this ready for review or are you still working on it?

Hey @adamdoe yes it is ready


Feature: Tooltip for Avatars

Type of feature
Feature
Current behavior
Each card has a list of contributors with no context of who they are. I would love to add a tooltip on wh

similar to https://github.com/open-sauced/hot/issues/21
Desired solution
This may be accomplished through a react tooltip library or similar.

Add a tooltip on hover
Ensure the cursor-pointer is added with Tailwind

Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

.take

.take

@kelvinsekx let me know if you have any questions on this.

Sure I will.
I'm particularly new to tailwind...so I will reach out if there be need.
I would work on all issues assigned to me today.

I think you disabled the eslint in the last merge.
Really struggling with eslint troubles.

I think you disabled the eslint in the last merge. Really struggling with eslint troubles.

Disable any editor integrations and run it using npm run lint and npm run format please

Hoo okay. I will do that and send back the feedback


Problem with the first run

Hi,
firstly I want to thank you all your work, I really appreciate your development.
Well, my problem is that I am having problems with the installation and first run. I am getting this error log:
0 info it worked if it ends with ok 1 verbose cli [ '/usr/local/bin/node', '/usr/local/bin/npm', 'start' ] 2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'prestart', 'start', 'poststart' ] 5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>7 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true 8 verbose lifecycle<EMAIL_ADDRESS>PATH: /usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/node-gyp-bin:/root/web_revenge/whatsapp-web-reveng/node_modules/.bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/opt/cpanel/composer/bin:/root/bin 9 verbose lifecycle<EMAIL_ADDRESS>CWD: /root/web_revenge/whatsapp-web-reveng 10 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c', 'npm run dev' ] 11 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null 12 info lifecycle<EMAIL_ADDRESS>Failed to exec start script 13 verbose stack Error<EMAIL_ADDRESS>start: npm run dev13 verbose stack Exit status 1 13 verbose stack     at EventEmitter.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/index.js:301:16) 13 verbose stack     at EventEmitter.emit (events.js:182:13) 13 verbose stack     at ChildProcess.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/lib/spawn.js:55:14) 13 verbose stack     at ChildProcess.emit (events.js:182:13) 13 verbose stack     at maybeClose (internal/child_process.js:962:16) 13 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:251:5) 14 verbose pkgid<EMAIL_ADDRESS>15 verbose cwd /root/web_revenge/whatsapp-web-reveng 16 verbose Linux 2.6.32-042stab129.1 17 verbose argv "/usr/local/bin/node" "/usr/local/bin/npm" "start" 18 verbose node v10.10.0 19 verbose npm  v6.4.1 20 error code ELIFECYCLE 21 error errno 1 22 error<EMAIL_ADDRESS>start:npm run dev 22 error Exit status 1 23 error Failed at the<EMAIL_ADDRESS>start script. 23 error This is probably not a problem with npm. There is likely additional logging output above. 24 verbose exit [ 1, true ]
Can you help me to depurate the problem?
Thanks a lot.

I have managed to make it work but now I am getting some problems. So I am closing this issue.


new changing square shape animation

Describe the project you want to add with tech stack

Expected behavior

Screenshots (optional)

https://github.com/pranjay-poddar/Dev-Geeks/assets/120590200/fb0414c1-f378-4167-ad5e-971194c9c115
Additional context (optional )

@MohitGupta121  @TusharKesarwani  please assign this issue to me

@MohitGupta121 @TusharKesarwani please review it i have raised the pr


[FLINK-9554] flink scala shell doesn't work in yarn mode

What is the purpose of the change
This PR is trying to fix the issue of scala-shell unable to run in yarn mode.
Brief change log
The root cause is the options of CustomCommandLine is missed which cause the "-m yarn-cluster" can not be parsed correctly.
Verifying this change
I verify it manually on a single node hadoop cluster. I think it is better to add integration for that so that we can avoid the regression issue in future. If necessary, I can create a ticket for it and do it in another PR.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): ( no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no )
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? ( not documented)

Seen the same issue, my workaround is to have JobManagerAddressOption in runOptions which is in Flink-1.4.2 but removed in Flink-1.5.0.
diff --git a/flink-clients/src/main/java/org/apache/flink/client/cli/CliFrontendParser.java b/flink-clients/src/main/java/org/apache/flink/client/cli/CliFrontendParser.java
index 1588aac..4872928 100644
--- a/flink-clients/src/main/java/org/apache/flink/client/cli/CliFrontendParser.java
+++ b/flink-clients/src/main/java/org/apache/flink/client/cli/CliFrontendParser.java
@@ -183,6 +183,7 @@ public class CliFrontendParser {
                Options options = buildGeneralOptions(new Options());
                options = getProgramSpecificOptions(options);
                options.addOption(SAVEPOINT_PATH_OPTION);
+               options.addOption(ADDRESS_OPTION);
                return options.addOption(SAVEPOINT_ALLOW_NON_RESTORED_OPTION);
        }

ping @tillrohrmann

@zentol  I have addressed the comments. Could you help take a look at it ? Thanks

merging. @zjffdu Please open a follow-up issue for adding a test so we don't break this again.


#5858 Ray normalise origin vector after set methods

issue #5858

Since direction field is public, wouldn't be better to let user code ensure providing unit vector and avoid normalization instead?
Javadoc could be improved in that way.

My java developer part said "if constructor normalise direction vector why set methods dont".
But you are right, if you use set method for re-use same object. its dump to add calcul with nor.
i change commit , i remove nor() on public Ray set (Vector3 origin, Vector3 direction) and precise in javadocs.

I keep on methods public Ray set (float x, float y, float z, float dx, float dy, float dz) cause i think its necessary with 3 float param.
Your opinion ?

i push force on my fork and i close this pr.
i reopen new at https://github.com/libgdx/libgdx/pull/5860
sry for miss

there was no need to create a new PR, force push on your branch automatically update this PR.


Entry Point perk effect "Mod Max Placeable Mines" is broken when using FSMP

Describe the bug
Some mods like Adamant or Ordinator use a perk Entry Point "Mod Max Placeable Mines" to change how many Destruction Runes one can have active at the same time. In Adamant, the perk name is Rune Mage under the Destruction Tree.
When using FSMP - I tested with Vanilla Hair Remake SMP and also with KS Hairdo SMP - the "Mod Max Placeable Mines" entry point doesn't work. FSMP somehow is causing the perk entry point to be skipped in code.
To Reproduce
Steps to reproduce the behavior:

Install only SKSE, USSEP, FSMP, Mysticism, Adamant and Vanilla Hair Remake SMP OR KS Hairdo SMP.
Start a new game.
Cheat your way (or play organically) until you can unlock the first level of perk Rune Mage under the Destruction Tree.
Add the Lightning Rune spell book to your inventory (player.additem 000A2702 1) and equip it.
See that you can place as many runes as you want - even though the perk sets it to maximum 2.
Close the game, disable FSMP, start the game, do the same steps and realize that the perk works correctly (only 2 runes can be active at the same time).

Expected behavior
With that perk unlocked in the repro steps, you should have maximum 2 runes at the same time. If you check [PERK:00105F32] in xEdit, you'll see that all the perk does is apply a multiplier to "Mod Max Placeable Mines" so 2 can be active at the same time.
Desktop (please complete the following information):

OS: Windows 11

** FSMP**

1.49 AE: Skyrim 1.6.640 - SKSE64 2.2.2
CUDA: No_CUDA
AVX: AVX

Required files
*Configs.xml renamed to .txt because github doesn't support xml upload.
configs.txt
hdtSMP64.log
Additional context
Add any other context about the problem here.

Someone also posted on Reddit about it: https://www.reddit.com/r/skyrimmods/comments/vbxj3g/requesting_help_from_other_adamant_users/
Someone created a workaround mod for it: https://www.nexusmods.com/skyrimspecialedition/mods/83381
But it would be good if FSMP actually fixed this. It's very weird that it's hooking into a entry point perk, intended or not (I'm assuming the latter).

But it would be good if FSMP actually fixed this. It's very weird that it's hooking into a entry point perk, intended or not (I'm assuming the latter).

this is an Adamant bug that has nothing whatsoever to do with FSMP

But it would be good if FSMP actually fixed this. It's very weird that it's hooking into a entry point perk, intended or not (I'm assuming the latter).

this is an Adamant bug that has nothing whatsoever to do with FSMP

And you're saying this based on...?
It is not an Adamant bug, I tested, both with Adamant, Ordinator and I also created an empty mod that simply uses the aforementioned Entry Point perk.
Test it yourself. Start a new game with nothing but FSMP (and any requirements it might have) + a hair mod; create a new mod in xEdit with a perk that uses the "Mod Max Placeable Mines" entry point; add the perk to your character and see the bug. Disable FSMP, start the game again with your new mod and see the bug is gone.

Hey phlosber, FSMP 1.49?
So it is'nt related to my introduction of parallelism... I would have thought it was the major suspect.
Second suspect: a hook badly implemented in FSMP. I haven't much time lately, but I could build some test dlls to pinpoint the origin of the bug, and then solve it. Would you help me with the tests? Are you on discord? If so, could you contact me? I'm DaydreamingDay on the discord server which link I have put on the FSMP Nexus description page.


Site description

Fix https://github.com/KleeGroup/focus/issues/117

Code needs a refactor, but shoud be working for now (migrated from previous version).
Docs is needed , for now see https://github.com/KleeGroup/focus/wiki/fmk#site-description


Fix broken link in fr

The documentation contains lot of dead links in the FR language.
I have noticed that the {{< param "githubbranch" >}} was in other portions of the doc, and seems outdated.
I have replaced every occurrence with "master".

I just have compared with the english doc, and enforce the same.

Okay, thanks for the fix! ✨
/unhold


Update hosts

Hi @kang49 thank you for this.
We're mostly an aggregator of hosts files from active and reputable curators of hosts files, and we package these amalgamated hosts files in various ways.
The file changed in this PR is generated by a consolidation process, as explained in the readme.
Please submit this request to one of our curated gambling sources.
Closing.


Include parameter descriptions

Parameter descriptions are part of the schema object, not the root
parameter object. Correct this.
Signed-off-by: Stephen Finucane<EMAIL_ADDRESS>

I'm confused by this one. There is a description field on the schema object [1]. However, there is also a description field on the parameter object itself [2]. I'm not sure what takes priority.
I've opened a bug against the spec for this. In the interim, I'll simply rework the spec and close this.


`Package 'DotNetInit 1.0.3' has a package type 'DotnetCliTool' that is not supported by project

Hi
When I wanted to install your package in VS2017 Install-Package DotNetInit  I got an error ?
Package 'DotNetInit 1.0.3' has a package type 'DotnetCliTool' that is not supported by project 'WebApplication3'
How to fix it ?

Looks like as per https://github.com/NuGet/Home/issues/4190, tools packages need to be hand-added by editingg the .csproj as described in the README :-(

@maartenba  Thanks


Fixed invalid local variable reuse.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Jan Sikorski seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Fix podspec

It referred to Swiftcheck rather than Swiftx.

Oh dear...
Welp, the only thing to say is that we need the source search paths intact.  The only thing that needs changing is the repo endpoint

Closing due to lack of action.  See #41.


Release candidate 4.2.0

Coverage remained the same at 100.0% when pulling 265bf00970a574cef08679b949954c1a31c7524c on ReleaseCandidate-4.2.0 into 836cdb0d495718cbbb6adde0076b86b4c8c42c45 on master.

Coverage remained the same at 100.0% when pulling 440361522ed1b16007f30451967546a5b5d18510 on ReleaseCandidate-4.2.0 into 836cdb0d495718cbbb6adde0076b86b4c8c42c45 on master.


Rename USAGES as PRATICES in trek detail page

Can you provide us all the translations also please ?

FR : Pratique
EN : Practice
IT : Pratiche
As nav_trek_map.use in https://github.com/makinacorpus/Geotrek-mobile/pull/134
In this (/app/scripts/globalization/locales.js) file I also just noticed that in EN 'nav_trek_map.use': 'Uses' should be modified to 'nav_trek_map.use': 'Practices',

Ok thx


Add Another Batch of Domains

Adds Austria, Australia, Hong Kong, Italy, New Zealand, Spain

@vikram-narayan Build is passing now, does the rest look good to you?


Refactoring the ROCm Fusion pass code.

Removing code duplication by moving the following items to the base class (ROCmFusionOpBase)

"FusionOpData" data structure
"DoFusion" routine
"CreateFusionOp" routine

Previously all 3 of the above were duplicated in the each derived class.
After this change the dervied class (which represents the implementation of a specific fusion),
only needs to implement the "IsFusionEligible" routine.
The "IsFusionEligible" routine needs to continue doing the following

check whether the sequence of nodes ending at the given node is eligible for fusion.

if no, then return false
else, populate the "FusionOpData" structure, and return true

@whchung @parallelo
The CUDA CI log has this in it
Executed 868 out of 868 tests: 867 tests pass and 1 fails locally.
INFO: Build completed, 1 test FAILED, 17853 total actions
Build step 'Execute shell' marked build as failure
Setting status of ae200baf5dc070e16ac237640d1f6d93431d0443 to FAILURE with url http://<IP_ADDRESS>:21096/job/tensorflow-upstream-unit-tests-nvidia/248/ and message: 'Build finished. '
Using context: cuda
Finished: FAILURE

I could not a test failure in the log...."grep FAILED" returns only the summary line above.
I do see one test timing out...is that the 1 failure reported?
If yes, then it has nothing to do with this change.
//tensorflow/python/data/experimental/kernel_tests/serialization:numa_map_and_batch_dataset_serialization_test TIMEOUT in 465.0s


Feature request: renderList

I think it is nice to be able to render a custom list for the suggestions like the renderTextInput and renderItem props.

I agree, specially when using the component as a search bar over something else, such as a map (which is my case). I would like to use NativeBase Card, but haven't found a way to do it.


Cannot handle spaces in paths

If the included file paths have spaces, formatting fails because it interprets the path incorrectly.

Thanks for submitting this! I see you've opened a draft PR - you can mark it as ready for review when your fix is ready. I'm guessing you'll have to modify the find invocations?

Resolved in #131 🚀


Can't use the SDK in production environment

Some methods don't take a handler as an argument, but it's via this handler that you define the sandbox parameter. As a result, I can't NaturalUser.get(1) on a production environment.
Here is the patch I use that fixes the issue:
---
 mangopay/api.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/mangopay/api.py b/mangopay/api.py
index 77e8198..d4d0f33 100644
--- a/mangopay/api.py
+++ b/mangopay/api.py
@@ -33,9 +33,9 @@ requests_session = requests.Session()

 class APIRequest(object):
-    def __init__(self, client_id=None, passphrase=None, api_url=None, api_sandbox_url=None, sandbox=True,
+    def __init__(self, client_id=None, passphrase=None, api_url=None, api_sandbox_url=None, sandbox=None,
                  timeout=30.0, storage_strategy=None, proxies=None):
-        if sandbox:
+        if sandbox or mangopay.sandbox:
             self.api_url = api_sandbox_url or mangopay.api_sandbox_url
         else:
             self.api_url = api_url or mangopay.api_url
--

Hey @chocopoche, do you think you could do a pull request with your suggest please? It would make it a bit clearer :-) Thanks!

I second this issue. The way this library manages its default handler is convoluted and broken. I had to use a monkeypatch like this:
handler = mangopay.APIRequest(client_id=…, passphrase=…, sandbox=…)
mangopay.get_default_handler = mangopay.base.get_default_handler = \
    mangopay.query.get_default_handler = lambda: handler

This has now been implemented with #97 and has just been released with v3.0.5 :-)

Somehow, I can't connect to the production environment with this. When I call mangopay.APIRequest(client_id=..., passphrase=..., sandbox=False)
it still connects to the sandbox. How do you use this implementation?

I changed
if sandbox or mangopay.sandbox:
back into
if sandbox==True:
and it worked for me - but that also means one has to specify Sandbox with the handler initalisation.
I'm still curious for the 'real' way how to do it. Thanks in advance for your help.

@Iggwettmann did you configured the SDK to not be in sandbox mode? I'm using:
import mangopay
mangopay.client_id = 'xxx'
mangopay.passphrase = 'xxx'
mangopay.sandbox = True

Ah, that makes sense, of course.
it doesn't work if you inject sandbox into the handler initialization. You might want to update that in the docs. Thanks, @chocopoche


[FEATURE]: Refactor Filesystem Operations to Use pathlib Instead of os.path

Description
I would like to suggest refactoring the codebase to utilize the pathlib library for all filesystem operations instead of os.path. This change will provide improved readability, code consistency, and platform independence.
If this proposal is approved, I could work on it.
Additional Info
No response

@serengil Yes, of course.

In the codebase, both pathlib and os.path are used. It would be better to use just one library consistently, especially in the files deepface/commons/folder_utils.py and deepface/commons/image_utils.py;
Constructing paths like this output = home + "/.deepface/weights/" + file_name (there are a lot of cases of such path constructing style in the codebase) is not safe due to platform-dependent differences in path separators (/ and \);
The code in deepface/commons/constant.py could be rewritten in a more readable way;
There is inconsistency in how existence checks are performed. Sometimes it is written as if os.path.exists(exact_file_path) is False: and other times as if not os.path.exists(weights_path):. The same inconsistency exists with os.path.isfile;
There is inconsistent use of os.path.join and path concatenation, as mentioned in point 2.

Of course, these points are not crucial, and the code is effective as it stands. However, bringing file operations to a consistent style would improve the code even further.

1- we are doing all file related things with os.path. Pathlib is just being used to find the home folder, and given image is Path object (this is added with a PR from community, not really required feature). So, it would be better to give up Pathlib instead of os.path.
2- constructing paths like output = home + "/.deepface/weights/" + file_name makes java programs platform dependent. In python, this is not true. Even in windows, you have to use slash not backslash. If you say this usage is platform dependent, then it wouldn't work in windows but it is working. if you say that using join is more readable, that is fine but i disagree with platform independence thing.
3- don't know how you will do it better
4- I prefer to use 1st usage when I am coding, 2nd usage is most probably coming from a community PR. But that is still totally fine.
5- I am not a fan of join, prefer using slashes when I am coding. These are coming from community PRs.
So, PRs are more than welcome!

I agree with you; replacing Pathlib with os.path will also result in consistent code;
Yes, you are right if we use these paths only in the Python script. However, if I want to log these paths or use them in shell scripts later, it might be encountered potential problems while moving code from Windows to Linux and vice versa. Meanwhile, Pathlib returns platform-adapted paths. I have provided an example in Snippet1;
An example is provided in Snippet2;
It's totally fine; it's just about code style;
One of the biggest advantages of os.path.join instead of concatenation is handling trailing slashes in directories paths. Consider the following example: output = home + deepface_dir + "/weights/" + file_name. Here, if the home variable doesn't include a trailing slash, path concatenation will fail, but os.path.join(home, deepface_dir, "weights", file_name) will work correctly.

Snippet1:
import os
from pathlib import Path

root = "c:\\f1\\"
path = "f2/data.txt"

print(os.path.join(root, path)) # output: c:\f1\f2/data.txt can't be used in logs, shell scripts, etc.
print(Path(root) / Path(path)) #  output: c:\f1\f2\data.txt correct

Snippet2:
# Now
import os

SRC_DIR = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
ROOT_DIR = os.path.dirname(SRC_DIR)

# Suggested
from pathlib import Path

SRC_DIR = Path(__file__).resolve().parent.parent
ROOT_DIR = SRC_DIR.parent

@kremnik as i mentioned, PRs are more than welcome


fix: path -> absPath

We were reporting abs path as the file. Update that so we properly send it

It'd be nice to keep the filename in there. That's what we'd use to link to GitHub for example.

Actually @phacops @Zylphrex I dont think there is a good way of getting a filename relative to project root. I'm going to omit abs_path and just keep filename until I find a solution because I dont want to send duplicate data... Meanwhile I'm checking with sentry/node if they already do something we can reuse here.

I think it's better to send abs_path anyway, even if we don't have filename.

Ok, let me address filename in a separate PR then


Areas vs Global Areas

If you have a theme with an area like:
$a = new Area('Header Image');
$a->display($c);
and you change that area to a global area like:
$a = new GlobalArea('Header Image');
$a->display();
That area will not save blocks or show up in the list of areas in the Stacks and Global Areas Dashboard page.  You have use a Global Area name that has never been used before.

Pull request is up at #2759. Pretty simple change if you want to try it out.

Hi, I applied your patch, if you change an area to a global area it works perfectly
if you change a global area to a regular area (same name) blocks continue to be added into the global area and do not display on the page.  If you delete the global area then the new area works correctly.
Maybe not an issue though, you probably should have different names on your areas and global areas?

Good catch... It works properly if it originally started as a normal Area, but didn't if you started as a Global Area. Another small tweak (which I believe is safe) and it appears to be working for me, see the updated commit

Working for me too.

Ya I could imagine it to be especially problematic for those who might change themes and one theme having a global header and another just having it as a normal area. While it could probably be improved to copy blocks around, this seems like a good first step to me to at least allow them to function independently.


Bump to containerd v1.0.0-rc.0

Since that bumps to gogo protobuf v0.5 too do the same.
Note that there are no actual containerd changes here, although there are some
gogo proto ones.
Signed-off-by: Ian Campbell<EMAIL_ADDRESS>

Are there any changes for final 1.0?

Even if not, maybe change the PR name and commit message?


Web Design and Development Company in Kirkland

Future work is web design and development company in Kirkland. An attractive design, modern and in line with current trends will create a visual impact that will distract you from your competition. If your website is agile, navigable and fast and incorporates the necessary tools, you will have everything you need so that customers who visit it do not hesitate and decide for their services. Future work is a best web design and development agency. in Kirkland. https://futureworktechnologies.com/web-design-and-development-company-in-Kirkland/

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

ID: 956ee5f8-f7d2-bfd7-afa8-a9f9e3082840
Version Independent ID: 840a976a-43da-cae8-734c-90c9df49dc8e
Content: How to contribute to docs.microsoft.com - Contributor Guide
Content Source: Contribute/how-to-contribute.md
Service: n.a
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

This appears to be spam. Closing.


Make a list of super-cookies

We should make a list of all super-cookies so we know what first-party isolation tests to run. Let's put this list somewhere in the repository.

I added a file, spec.md


When will you fully support dotnet 3.1?

I see your using a version of the RabbitMQ client 5.1.2, that if I'm not mistaken is not compatible with dotnet core 2.2? This is assuming this table is accurate. https://docs.microsoft.com/en-us/dotnet/standard/net-standard
Quick check of the dependencies for this nuget version indicates net standard 1.5.

RabbitMQ client version was upgrade to 6.2.2 in #174. The extension (before and now) should be able to support most version of .NET applications including 3.1 and 6.0.


PhantomJS 1.9.2 crash running via Protractor/Selenium on Debian

I'm testing an angular app using Protractor with PhantomJS.
This happens about every 4th time or so I run the test.
I'm running on Debian 6.0.8 with PhantomJS 1.9.2.

12:06:09.319 INFO - Executing: [find element: By.id: importServerFileBtn] at URL: /session/ad8b0db2-89d0-45c1-993d-ce139a23a5ec/element)
12:06:09.337 INFO - Done: /session/ad8b0db2-89d0-45c1-993d-ce139a23a5ec/element
12:06:09.348 INFO - Executing: [execute async script: return (function () {
  var el = document.querySelector(arguments[0]);
  var callback = arguments[1];
  try {
    angular.element(el).injector().get('$browser').
        notifyWhenNoOutstandingRequests(callback);
  } catch (e) {
    callback(e);
  }
}).apply(null, arguments);, [body]] at URL: /session/ad8b0db2-89d0-45c1-993d-ce139a23a5ec/execute_async)
12:06:09.355 INFO - Done: /session/ad8b0db2-89d0-45c1-993d-ce139a23a5ec/execute_async
12:06:09.365 INFO - Executing: [click: 6 [[PhantomJSDriver: phantomjs on LINUX (daa28790-81c2-11e3-b990-51de5eea2349)] -> id: importServerFileBtn]] at URL: /session/ad8b0db2-89d0-45c
1-993d-ce139a23a5ec/element/6/click)
12:06:09.404 INFO - Done: /session/ad8b0db2-89d0-45c1-993d-ce139a23a5ec/element/6/click
12:06:09.509 INFO - Executing: [get: about:blank] at URL: /session/ad8b0db2-89d0-45c1-993d-ce139a23a5ec/url)
PhantomJS has crashed. Please read the crash reporting guide at https://github.com/ariya/phantomjs/wiki/Crash-Reporting and file a bug report at https://github.com/ariya/phantomjs/is
sues/new with the crash dump file attached: /tmp/6791aacc-b6a8-68fc-1979d8a9-30e0bbb6.dmp
12:06:11.619 INFO - Executing: [delete session: ad8b0db2-89d0-45c1-993d-ce139a23a5ec] at URL: /session/ad8b0db2-89d0-45c1-993d-ce139a23a5ec)

Attempting to shut down selenium nicely
12:08:43.024 INFO - Checking Resource aliases
12:08:43.026 INFO - Command request: shutDownSeleniumServer[, ] on session null
12:08:43.026 INFO - Shutdown command received
12:08:43.027 INFO - initiating shutdown
12:08:43.027 INFO - Got result: OK on session null
Selenium Standalone has exited with code 0

Dump file is here:
https://gist.github.com/tfnico/ded96ddf04e3edec4bc3
Here's the protractor/npm dependencies:

├─┬<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ ├──<EMAIL_ADDRESS>│ │ └─┬<EMAIL_ADDRESS>│ │   ├──<EMAIL_ADDRESS>│ │   └──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ ├──<EMAIL_ADDRESS>│ │ └──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>
Unfortunately, I cannot provide a simple code sample that recreates the problem. So, I won't be offended if you close this issue, as there seems to be plenty of other crash reports to look at. I just got the crash dump, and figured I'd post it somewhere.

Please upgrade to PhantomJS 2. 1.9.x is no longer supported, and known to have many crasher bugs that are fixed in 2.0.


Upgrade boto3(1.28.*) to allow Service-specific endpoints

Is your feature request related to a problem? Please describe.
Using the importorganisationfroms3 feature to replicate an instance across different environments. This will use an aws bucket to import the config json as defined by the documentation. In this instance, we're using localstack to configure a mocked aws client in local and test environments.
Essentially the change would be bump boto3 to 1.28.0 or later, however, this could impact other features using boto3, eg. dynamodb within the edge proxy. If so, please see the alternative methods.
Describe the solution you'd like
The ability to define which aws client we can use would be greatly appreciated, this would want to be with minimal config change and just using environment variables.
Using these Service-Specific endpoints would be great for the s3 importer.
Setting AWS_ENDPOINT_URL on the docker image and boto3 should pick up the localstack instance via the endpoint when this variable is set.
Describe alternatives you've considered
An alternative and simplier approach could be just to define a AWS_S3_ENDPOINT_URL in the flagsmith config, much like the SES variable found here and using this within the importer to set the endpoint_url in the s3_client here.
An alternative could be, allowing custom boto3 configurations, either created in the docker image or just extending flagsmith as a base image.
Additional context
version 1.28.0 CHANGELOG where this feature is introduced: https://github.com/boto/boto3/blob/develop/CHANGELOG.rst#L601
conversation on this feature for boto3: https://github.com/boto/boto3/issues/2099#issuecomment-1626117882

Nice idea - happy to receive a PR for this if you have time!

Resolved in https://github.com/Flagsmith/flagsmith/pull/2926


DefaultAppConfig simplification

Modify DefaultAppConfig to be a DelegatingConfig instead of a CascadingCompositeConfig to avoid exposing all of the CascadingCompositeConfig functionality.
Remove withLoadSystem from ConfigLoader
Expose getting any layer from AppConfig

archaius-pull-requests #222 FAILURE
Looks like there's a problem with this pull request

NetflixOSS » archaius » archaius-pull-requests #53 FAILURE
Looks like there's a problem with this pull request

archaius-pull-requests #223 FAILURE
Looks like there's a problem with this pull request

NetflixOSS » archaius » archaius-pull-requests #54 FAILURE
Looks like there's a problem with this pull request

archaius-pull-requests #224 FAILURE
Looks like there's a problem with this pull request

NetflixOSS » archaius » archaius-pull-requests #55 FAILURE
Looks like there's a problem with this pull request

archaius-pull-requests #225 SUCCESS
This pull request looks good

NetflixOSS » archaius » archaius-pull-requests #56 SUCCESS
This pull request looks good

NetflixOSS » archaius » archaius-pull-requests #57 SUCCESS
This pull request looks good

archaius-pull-requests #226 SUCCESS
This pull request looks good

@brharrington, thanks for the thorough review.  I believe all your comments have been addressed but there is one thing I'm not too happy about.  Should we allow setting an external application layer Config AND try to load the configuration properties files as well.  Currently, when an external application config is provided DefaultAppConfig will silently not load the application properties.  Does it make sense to load both into a composite layer?

Can it just be a special case of calling withApplicationConfig? In which case if you do both it would be the same as calling withApplicationConfig twice on the same builder. It seems like this is mostly a question of what the builder makes convenient, if a user needs to have a config and load the configuration properties then they could just set the application layer to a composite themselves.


Load configuration from a specific file - is it possible?

I didn't find a way to crate a Configuration using a specific file via constructor or a specific method.
Is there a way to load an yaml file containing configuration that is stored somewhere on non default location? I saw that I can pass file name using pros.file system property or env variable, but I'd like something like Config.from(configFileName) ?
My use case is I have a single properties file that is application configuration but also ebean configuration. I'd like to use yaml and achieve the same.

Full control what is loaded for example. Btw, I tried with load.properties. Not sure why it is not working.
I use such code:
        System.setProperty("load.properties", fileName);
        final Configuration configuration = Config.asConfiguration();
        final Properties properties = Config.asProperties();

Properties is almost empty - 2 keys only. Also add it via jvm arguments - -D style also did not worked.

Full control what is loaded for example.

you still have that control doing it via configuration.

it is not working.

is the property file on the classpath? if it's somewhere else on your system you'll need to use the file path instead of the name.

Ah I see, the issue is a bug on our side

not the file is not in the classpath. :) ok :) so... I got a 🐛 :)

you can also try loading with the -P command line argument

Also note that the default loading (of application.properties|yaml) etc can be bypassed by using Configuration.builder() like:
https://github.com/avaje/avaje-config/blob/master/avaje-config/src/test/java/io/avaje/config/CoreConfigurationTest.java#L150-L154
BUT wait ...

My use case is I have a single properties file that is application configuration but also ebean configuration

So what is the problem exactly? Ebean uses avaje-config by default so it isn't clear to me what the problem is?  You could have application and ebean configuration in the same place like application.properties - it isn't clear to me what the issue is here?

I'd like to avoid loading application.yaml file at all (and all other places that are possible) and load a specific file that is somewhere on the filesystem.

Right. So System.setProperty("load.properties", fileName); would almost work, in that currently (with that PR/fix) it would load that file but it currently would also by default load application.yaml etc and to get full control we don't want that.

but does not allow me to load a different yaml file.

We can add that capability, so that we can do
    var fileSource = new File("./src/test/resources/yaml/minimal.yaml");

    var configuration = Configuration.builder()
      .load(fileSource) // load from a file
      .load("hi.properties") // load from a resource
      .build();

... and this is the way to get full control over it. This is closer to what you were looking for?

Yes, thanks :) it seems quite good to me :)

Yes, thanks :) it seems quite good to me :)

Ok cool. I'll release that then as version 3.13 and it will also have the fix from @SentryMan, I'll do that now.

:) Thank you Rob :) will update my deps :)

Ok, I'll close this as fixed in 3.13. Comment back here if it doesn't work as you expect etc. Cheers !!