Document using custom and offline tiles

Merging all similar issues here.
MapView.UrlTile
MapView.LocalTile

@alvelig Updated documentation for LocalTile in main README file within PR #1876

@alvelig I guess we could close this one..

hi i am using below url of soil map tile
<MapView.UrlTile
urlTemplate=" ....../api/soil/wms/?x=5458&y=12859&z=15 " />
in x,y i pass converted lat long and in z i pass zoom level.
But my map overlay tile repeat, why i don't know?
Please help

@pawan8525
I am having the same issue with localTiles. Did you found a solution?

@DanielBoening don't compute x, y, z for each tile. just pass http://.../tile/{z}/{y}/{x} and UrlTile will do the compute and replaces the values.

my tile set is generated from an indoor floor plan, and the width and height of my floor plan map is in pixels. when i use the//.../tile/{z}/{y}/{x} format i also see the upper left most tile repeating, and the zoom levels do not seem to change in the same way they did before when using leaflet in a web view. i am unsure how to calculate the proper bounds and zoom levels within react native maps. i have been trying to find info on GIS stack exchange, understanding mercator projections and working on formulas to derive a function to translate from  a world map to a pixelated floor plan map, if i can figure this out i will share here.

@aminmahrami I did have a folder with that structure.
@hariDasu I kept having problems when working with my map that was not projected on the real world as this library tries to map it to real world coordinates.
I did end up building my own mechanics.

@DanielBoening i am still thinking there is no other way than using leaflet.js in a webview, such as this person did https://github.com/react-native-community/react-native-maps/issues/1460. were you able to successfully translate your non-world map to a  spherical coordinate system?

@hariDasu
I guess you are right. There even is a react-native library trying to implement leaflet.js but it did not seem good enough for our project. We show a map on browser with leaflet.js and on the phone with a self-made solution. It works quite similar and user experience is similar.
Translating your tiles to the x/y/z was quite easy. Handling touches and such was a bit complicated. I could create a library from that and publish it if you are interested. But I could do it only next week.

How can we create custom indoor tile to show on map ?


lint-staged integration

Do you want to request a feature or report a bug?
Feature request
What is the current behavior?
Jest have only --onlyChanged and --lastCommit.
What is the expected behavior?
Add some --changed FILES to mark changed files manually.
Just jest FILES will not works, because lint-staged could report not only test files changes, but also project JS files changes.

There is a awesome project lint-staged. It works as pre-commit hook and takes all changed files from stage (added to staged by git add) and run linter on it.
Seems like Jest have awesome changes detection. As I understand If I will change scr/main.js, Jest will execute test/main.test.js on --onlyChanged.
But --onlyChanged works on every changed file. So it could not be used in pre-commit hood, because user could change few files, but add to stage only one.
/cc @okonet

I totally agree we should have this! In fact, in our more advanced runner internally at FB we have this. Do you mind working on a PR for this?
Check out how changedFiles is used in SearchSource: https://github.com/facebook/jest/blob/master/packages/jest-cli/src/SearchSource.js#L20 – we could make it so if a list of files is passed and the --changed option is passed, we sidestep the git/hg check and just use that list of files (and do path.resolve(cwd, file) on all to turn them to absolute paths).

Thanks for link — it will help for PR.
I could make PR after few weeks (sorry, have few open source duties in queue).

@ai thanks for initiating it! I appreciate it.
@cpojer hey! We met at ReactVienna once :)
I'll create a PR ASAP

@cpojer what do you think of #1678? Also need some help with the output -- see comment.

Thanks @okonet for the proposal. To make sure we don't have duplicate issues, I'm gonna close this issue in favor of the PR.


KM to edit Research need for MSE

https://github.com/pacific-hake/hake-assessment/blob/9d11bc9d2fbf61fc97cad76b053d5a9e0d66bb0b/doc/executive-summary/research-needs.rnw#L24-L39
@kristinmarshall-NOAA can you make sure that this text is appropriate for what we need for the MSE? I just want to make sure that it aligns with the funding we got and are asking for. Thanks. Feel free to put edits here or push directly to the repo.

@kristinmarshall-NOAA  - are you able to confirm that this is good? [We're submitting today, hopefully!]

This looks good!  I added growth to the list of links and just pushed a change.


[5.10] Options: Make -experimental-lazy-typecheck a driver option

Cherry pick of https://github.com/apple/swift/pull/69259.
It should not just be a frontend option.
Partially resolves rdar://117168788.

@swift-ci please test

@swift-ci please test

@swift-ci please test

It turns out this change is unwanted, closing.


suggestions for fastlane scan

Any suggestions on how to use this with fastlane scan?

Hmm, good question. I've not used it with scan before, I don't think it's likely to work out of the box. The key would be getting the xcodebuild output into second_curtain somehow. It's possible scan writes the raw output to a log file? Maybe we could read that in and pipe it to second_curtain?

Haven't used scan either, but the readme states that the raw output xcodebuild generates is written to ~/Library/Logs/scan.
So maybe something along the lines of cat ~/Library/Logs/scan/ | second_curtain will do the trick?
I'm not sure about the format of the log files that scan puts there, but you'll want to use the file with the test output, as this contains the information that second_curtain needs.

Thanks for all the input, in the end I took the a simpler solution. I used Travis's artifacts feature to upload all uncommited changes to S3. It's not as nice as the web preview with second_curtain, but it works good enough.


Node module won't resolve

After updating from 1.5.1 to 1.5.3 via npm update I can no longer import Gifffer from my node_modules folder using the normal syntax, e.g.:
import Gifffer from "gifffer"
Webpack will report:
Module not found: Error: Can't resolve 'gifffer' in [file]
I am working around this temporarily by downgrading to 1.5.1.

same here. But downgrading to 1.5.1 did not work for me... did you do anything else besides that?

Check the package.json file of the v1.5.3
The "main" value is "build/gifffer.js", but it cannot find this file under build folder.
So you can figure it out by either option:

Update the "main" value in package.json;
Rename the "build/gifffer.min.js" to "build/gifffer.js";

Thanks. Fixed. I deployed a new minor release containing your fix.


Made dummy implementation to prevent infinite loop

I fixed csr_matrix to raise an error on methods which are not implemented. Because spmatrix uses to_csr method, it causes infinite loop without this patch.

LGTM!


🐞 - Popover not visible on android with custom template

Which ngx-ui-tour-* package is the source of the bug?
ionic
Please provide a link to a minimal reproduction of the bug
https://ionic6-angular13-fybot1.stackblitz.io
Description
I use a custom-template and it's workign on ios but not on Android..
It seems that popover is behind everything...
Angular version
15.2.2
Ngx UI Tour version
1.4.0
Which browsers have you used?

[X] Chrome
[ ] Firefox
[ ] Safari
[ ] Edge

Which operating systems have you used?

[ ] macOS
[X] Windows
[ ] Linux
[X] iOS
[X] Android

I would like to center my popover on my screen , that's why I set position: fixed.
However with position: relative I see my popover but impossible to center it.
By the way is there any way to change width of popover with css ?
Thanks

Have you read FAQ: https://hakimio.github.io/ngx-ui-tour/ion-popover/FAQ#center-tour-step ?

Yes and that changed style of my div... and dont want that

How can fixed 0px width and height div change anything in your app?
Anyway, it seems to work fine in this demo: https://stackblitz.com/edit/ionic6-angular13-rypta7?file=src%2Fapp%2Fapp.component.css

I tested on my Android phone with custom template and can not reproduce the issue. The step is shown correctly for me.
I'll make it possible to customize tour step width with CSS variable in the next minor release.

@ludonoel1 I have released a new version where you can customize tour step width. Check out the release announcement for more info.


Display string topic

I have a need to display Strings to visualize the system state. There is a display for Image messages, but not for String messages. It would be convenient if there was a display type like Image for Strings, which simply displayed the string content in its window and had formatting options in the dropdown menu.

Am I overlooking a function that would make displaying strings easy, except publishing a Marker? I don't think that one is an alternative in this case.

Would a PR for this be welcome if I implement it myself?

I'm not aware of such a display, but you might want to ask on ROS anwers first.
PRs are always welcome :wink:

I forgot to search on ROS Answers after googling. It looks like this package does what I need, according to this answer. My bad for the noise!


Deprecate or improve typings in ScenePlayr

Cutscene and routine arguments are untyped. This is bad! How do we fix it?

A good part 1 of this would be to stop using routines altogether and instead pass local variables around. I've started recommending this to contributors.


Unable to download beyond 900 chapters - OutOfMemoryError

Tried to download some novels on novelfull, when I get to chapter 900-1000ish, it just stops.
Ran it with the console and got the following error code:
Exception in thread "pool-2-thread-1" java.lang.OutOfMemoryError: Java heap space
at org.jsoup.parser.CharacterReader.(CharacterReader.java:36)
at org.jsoup.parser.CharacterReader.(CharacterReader.java:41)
at org.jsoup.parser.TreeBuilder.initialiseParse(TreeBuilder.java:38)
at org.jsoup.parser.HtmlTreeBuilder.initialiseParse(HtmlTreeBuilder.java:65)
at org.jsoup.parser.TreeBuilder.parse(TreeBuilder.java:46)
at org.jsoup.parser.Parser.parseInput(Parser.java:35)
at org.jsoup.helper.DataUtil.parseInputStream(DataUtil.java:175)
at org.jsoup.helper.HttpConnection$Response.parse(HttpConnection.java:835)
at org.jsoup.helper.HttpConnection.get(HttpConnection.java:287)
at grabber.scripts.ChapterContentScripts.defaults(ChapterContentScripts.java:222)
at grabber.scripts.ChapterContentScripts.fetchContent(ChapterContentScripts.java:45)
at grabber.Chapter.saveChapter(Chapter.java:48)
at grabber.Novel.downloadChapters(Novel.java:137)
at gui.GUI.lambda$null$2(GUI.java:332)
at gui.GUI$$Lambda$67/20323159.run(Unknown Source)
at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
at java.lang.Thread.run(Unknown Source)
It seems that Java ran out of memory. Is there a way to allocate more RAM to the program? Thanks for your work.

I seem to have fixed it by downloading and using the Java 64-bit. Downloaded from here: https://www.java.com/en/download/manual.jsp
Maybe you can let Java 64-bit to be one of the prerequisite of using the program? Anyways, thanks again for your work. It makes my military life so much better.

Thanks for the info! I'm not sure what the default heap size on 32-bit is but it seems to be pretty low. You can manually increase it with the parameters -xms / -xmx (java -Xms1g -Xmx2g -jar NovelGrabber...) so I don't think 64-bit is a requirement just yet.
If a novel take more than 4gb of memory (32-bit limit) I'll seriously need to reevaluate my memory management.

For reference, Library of Heaven's Path and Gate of God are the two novels
I tried that failed to go over the 930 chapters mark.
On Sun, 6 Sep 2020, 02:57 Flameish<EMAIL_ADDRESS>wrote:

Thanks for the info! I'm not sure what the default heap size on 32-bit is
but it seems to be pretty low. You can manually increase it with the
parameters -xms / -xmx (java -Xms1g -Xmx2g -jar NovelGrabber...) so I
don't think 64-bit is a requirement just yet.
If a novel take more than 4gb of memory (32-bit limit) I'll seriously need
to reevaluate my memory management.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Flameish/Novel-Grabber/issues/58#issuecomment-687648556,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQ4JJULCIJYSKMU3MDIYHDDSEKCYTANCNFSM4Q3FTR4Q
.

Tested it with 50 MB. Ran out of memory after 160 chapters / 800 pages w/o images. I'm going to take a look where I can optimize it.

On my system, it seems to be using about half a GB at first while grabbing
and slowly increasing to just over a GB after it is complete.
On Sun, 6 Sep 2020, 03:09 Flameish<EMAIL_ADDRESS>wrote:

Tested it with 50 MB. Ran out of memory after 160 chapters / 800 pages w/o
images. I'm going to take a look where I can optimize it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Flameish/Novel-Grabber/issues/58#issuecomment-687649904,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQ4JJUKHHN4LGRNW665ZQ4TSEKEFDANCNFSM4Q3FTR4Q
.

 Managed ~750 chapters on 50 MB
This is after 1300 chapters on default java settings:

What changed from before where i could download 4000+ chapters without issue

Sometimes it could look like this for some novels: (~200 chapters)


Breadcrumb bugfix

See: https://www.pivotaltracker.com/story/show/108242580
And: https://www.pivotaltracker.com/story/show/109717458
(They are the same bug)
As well as fixing breadcrumbs after a validation failure - as described in the bugs - this also adds a breadcrumb that was missing for multi-lot services.
Before:

After:

Breadcrumb disappears if you try to change the service name to an empty string.
Not sure if we care about that. Seems unlikely anyone gets into that state by themselves.

Yep I know about the empty name thing - but at the point you've submitted an empty string for the name then the service doesn't have a name so there's nothing to put in the breadcrumb... I did have a go using the previous name (i.e. the one that you deleted), but that was weird.
I guess we could "New service" or "New lab" as a default?

I think the blank breadcrumb if you submit a blank lab name is OK.  Have switched out that horrible logic to use the helper method everywhere it was needed.

Just bringing it up so that you knew about it.
Submitting an empty name is never going to work, so I'm not sure how much value there is in keeping the breadcrumb.


Fixing V20.html bugs after live test

Noticed some errors on the new i18n formatted sheet and i'm fixing them.

Hey @JackNexus, these changes look good, great job getting translation support started on this sheet.  We'll do a proper review and should be merging mid-week as usual, but I thought I'd point you to this thread in case you wanted to see if there are any other issues you can fix, or let folks know that you have fixes coming down the pipe.  That's up to you.

@JackNexus I apologize, this is on me. I went to double check that everything loaded correctly, but it seems I checked the original v: tm and not the 20th anniversary edition. Very sorry!

Oh ok lol no problem, always better than bugging a sheet for everyone like
i did >.<
Il 2 lug 2017 21:28, "wing-it"<EMAIL_ADDRESS>ha scritto:
@JackNexus https://github.com/jacknexus I apologize, this is on me. I
went to double check that everything loaded correctly, but it seems I
checked the original v: tm and not the 20th anniversary edition. Very sorry!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Roll20/roll20-character-sheets/pull/2744#issuecomment-312511907,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ab64_tiPER9ryUpVYLZMhHMtAXWWT4LFks5sJ-9ugaJpZM4OITHD
.

I'll pull this down and check it Monday. That way, if changes are still needed, you'll have more time to fix them before we push it to the live server. We'll make sure this gets fixed this week and doesn't end up waiting any longer.

One thing is that all the files in the folder translations need to be
redone (cause of the change in the main translation.json) except for the
it.json one that i fixed myself.
I found today another little text visualization issue in the italian
translated sheet, but nothing more than that.
I also asked for other potential issues on the thread you linked, so if
there are any others i'll fix them asap.


GPC Shell Script - Websocket Connection Failure

I recently attempted a deployment on Google Cloud Platform (GCP) using the provided Google Cloud Shell script by @genome21. While the instance was successfully created and is accessible via a browser, I'm experiencing issues with the functionality of the tool.
Specifically, the chatbox feature is non-operational to test anything. When attempting to open the chatbox after creating or importing any kind of flow, I'm met with a persistent websocket error in the console.
WebSocket connection to 'ws://<IP_ADDRESS>:8080/chat/9d2d13fa-5fe6-4d1d-ae96-bb3922ee7c33' failed

This issue is consistently reproducible by simply creating or importing a flow and then trying to access the chatbox feature.

Pull Request #301 addresses this

It seems you are using a Vector Store. They require embeddings to function which use the OpenAIEmbeddings endpoint.

See #298
This thread might close


Custom target support

I tried setting a custom target while also using build-std:
set(Rust_CARGO_TARGET "${CMAKE_CURRENT_SOURCE_DIR}/rust_lib/custom-x86_64-unknown-linux-gnu.json")

FetchContent_MakeAvailable(Corrosion)
corrosion_import_crate(MANIFEST_PATH rust_lib/Cargo.toml
    FLAGS -Z build-std=std
)

target_link_libraries(the_bin PRIVATE rust_lib)

The build seems to have succeeded, but this warning appeared:
CMake Warning at build/_deps/corrosion-src/cmake/FindRust.cmake:113 (message):
  Failed to parse target-triple `custom-x86_64-unknown-linux-gnu`.Corrosion
  attempts to link required C libraries depending on the OS specified in the
  Rust target-triple for Linux, MacOS and windows.

  Note: If you are targeting a different OS you can surpress this warning by
  setting the CMake cache variable
  `CORROSION_NO_WARN_PARSE_TARGET_TRIPLE_FAILED`.Please consider opening an
  issue on github if you encounter this warning.
Call Stack (most recent call first):
  build/_deps/corrosion-src/cmake/FindRust.cmake:750 (_corrosion_parse_target_triple)
  build/_deps/corrosion-src/cmake/Corrosion.cmake:63 (find_package)
  build/_deps/corrosion-src/CMakeLists.txt:73 (include)

Removing the "custom-" also removes the warning, but I feel a custom target should have a slightly different name than the original, so it's clear what you're compiling to when you're checking the logs.
Is setting CORROSION_NO_WARN_PARSE_TARGET_TRIPLE_FAILED the official™ solution here? At the very least, I feel the documentation should mention this, or maybe I just missed it.

Corrosion attempts to link required C libraries depending on the OS specified in the
Rust target-triple for Linux, MacOS and windows.

This part of the warning is actually outdated and should/will be removed.
Corrosion does still attempt to parse the target triple and sets the following variables for users convenience:
set(Rust_CARGO_TARGET_ARCH "${rust_arch}" CACHE INTERNAL "Target architecture")
set(Rust_CARGO_TARGET_VENDOR "${rust_vendor}" CACHE INTERNAL "Target vendor")
set(Rust_CARGO_TARGET_OS "${rust_os}" CACHE INTERNAL "Target Operating System")
set(Rust_CARGO_TARGET_ENV "${rust_env}" CACHE INTERNAL "Target environment")

I guess we don't need a warning message here anymore, the variables could just remain unset.
You could also rename your target to something like x86_64-unknown-custom_linux-gnu. Corrosion splits the string at - dashes, so you can use underscores in the OS or env part of the target triple.

Not strictly related, but i'm also getting this error with xtensa-esp32s3-none-elf
So maybe we need to add esp32[a-z0-9]*  as known vendors.
Or just deprecate this target parsing if not needed since it's error prone.

I see I already added esp into the list of known vendors for the target riscv32imc-esp-espidf. Is this vendor related to  your esp32s3? If yes we could just replace esp with esp[a-z0-9]*.

Or just deprecate this target parsing if not needed

In some places we need to check the ABI, and CMake seems to treat that as an implementation detail / internal variable. This mostly affects the msvc abi, but since regular clang can also output msvc abi, I don't know a good way how to reliably detect the abi, aside from parsing the Rust target triple. Strictly speaking we don't need to parse it, but I think it might be useful for users. At least for the big targets, this is tested and works - I just don't actively follow which new targets / vendors are added.

Update from my side:

I relaxed the vendor parsing for espressif, so all current esp toolchains should be parsed correctly.

I guess we don't need a warning message here anymore

I retract this statement. I want to keep the warning, since corrosion still does some target specific things based on the result of the parsed target triple, mainly determining file extensions or prefixes (e.g. .exe / .dll on windows OS, .dylib for darwin). Currently besides windows and mac all other operating systems fall into the other category, and basically are assumed to be the same as linux, but this could be different for new targets.
CMake also (at least partially) provides such information for the target OS, but we also support the hostbuild option, so we end up needing to parse the information from the target triple anyway.

Is setting CORROSION_NO_WARN_PARSE_TARGET_TRIPLE_FAILED the official™ solution here

It ends up working out despite the warning because you are targetting linux which falls into others category. But if you e.g. defined a custom target for windows or mac or some other OS that uses different file extensions, then things would start failing, so giving a warning here seems like the right direction.
I wonder if in the future we may be able to query file extensions for a given target triple directly from cargo / rustc.


Adding support for cert-manager certificates to manage webhook TLS.

This PR can resolve: https://github.com/GoogleCloudPlatform/spark-on-k8s-operator/issues/1178
This adds the option for the mutation webhook TLS Certificate management to be done by cert-manager.

cert-manager will be disabled by default
when cert-manager is enabled the webhook-init-job and webhook-cleanup-job are disabled, since it will be unnecessary to generate certificates adhoc.

Open to suggestions on adjustment of the values interface and configuration.

@dskatz This change would be helpful to the stability of the operator, the operator breaks from time to time after the cleanup job. any reason for not merging?

We need this, any update on merging this?

@dskatz Could you resolve the merge conflicts?

I opened my own fork and cloned this PR trying to resolve the merge conflicts: https://github.com/kubeflow/spark-operator/pull/2079
@yuchaoran2011 WDYT?

Was this effectively handled as part of the big rewrite?


⚠ Bofrost.lu has degraded performance

In d1ea896, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1036 ms

Resolved: Bofrost.lu performance has improved in 8f208c0 after 6 minutes.


Skip/Ignore duplicate primary key errors in CSV import

System information:

OSX 10.15.4
DBeaver Version <IP_ADDRESS>005311732

Connection specification:

MariaDB

Describe your question:
I have to do periodic imports from CSV files. There isn't a convenient way for me to get CSV files that only have new records since the last time I did an import. If I import CSV files with some existing records (overlap) and then choose to Ignore the duplicate primary key error when it pops up during the import, will that ignore all errors or only duplicate primary key errors?
I'd still like to see other errors if they occur, such as bad data. I just want to ignore duplicate primary key errors.

Dbeever uses batches when importing to speed up the process. Therefore, one error is shown for the entire batch. You can choose line-by-line import in import settings by putting 1 in the item: "Do commit after row insert". Then errors will be shown for each line. Use the skip after the first error to see all possible errors.

Now, if you want to control the import and all possible import errors line by line, then you can use the function of disabling batches in the import settings

verified


fix: cascade delete organization user roles

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 5 (40.0%)  changed or added relevant lines in 3 files are covered.
14 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.1%) to 50.601%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/api/v1beta1/user.go
1
2
50.0%

cmd/serve.go
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

internal/store/postgres/policy_repository.go
1
71.43%

internal/api/v1beta1/user.go
6
45.88%

internal/api/v1beta1/permission_check.go
7
44.94%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
6065

Relevant Lines:
11986

💛  - Coveralls


Set php executable path to docker or lando

The extension doesn't recognize php binaries residing in containers. Tools like Docker (and implementations like Devilbox or Lando) are becoming more and more popular, and are considered normal development environments by many.
Issue #150 addresses this but no good answer was proposed and it was closed without any commentary.
We need a way to use commands like (lando php or docker-compose exec php php) in place of the usual php binary

Was fiddling around a bit, bit couldn't manage to get it working.

And modifying the following lines: https://github.com/felixfbecker/vscode-php-intellisense/blob/4898e75f4ec7aa030290ad71e3cfde347e6891df/src/extension.ts#L82-L109
To look like:
            // Don't spawn a server and connect as a client, run the php-language-server.php as a server instead
            server.listen(() => {
                // The server is implemented in PHP
                const childProcess = child_process_1.spawn(executablePath, [
                    context.asAbsolutePath(path.join('vendor', 'felixfbecker', 'language-server', 'bin', 'php-language-server.php')),
                    '--tcp-server=<IP_ADDRESS>:1337',
                    '--memory-limit=' + memoryLimit,
                ]);
                childProcess.stderr.on('data', (chunk) => {
                    const str = chunk.toString();
                    console.log('PHP Language Server:', str);
                    client.outputChannel.appendLine(str);
                });
                // childProcess.stdout.on('data', (chunk: Buffer) => {
                //     console.log('PHP Language Server:', chunk + '');
                // });
                childProcess.on('exit', (code, signal) => {
                    console.log(childProcess.stderr);
                    console.log(childProcess.stdout);
                    client.outputChannel.appendLine(`Language server exited ` + (signal ? `from signal ${signal}` : `with exit code ${code}`));
                    if (code !== 0) {
                        client.outputChannel.show();
                    }
                });
                return childProcess;
            });

I can see the server spawning properly, but it doesn't seem to do anything:
DEBUG     Checking PHPLS_ALLOW_XDEBUG
DEBUG     The xdebug extension is not loaded
DEBUG     Server listening on <IP_ADDRESS>:1337
NOTICE    PCNTL is not available. Only a single connection will be accepted

@alcohol that seems like a lot of work that should be handled by the extension..

I know, I was just exploring the possibility of what it would take to submit a PR that could enable this functionality. So far, the amount of changes is rather minimal. One could put those behind configuration toggles and it would still work as-is and support execution within a Docker container. But as stated above, I did not yet manage to get it completely working.

Unfortunately my experience with Javascript is rather limited in comparison to my experience with PHP and Docker and such. So I'm not sure how to proceed.

Windows users couldd do following:
php.bat:
@echo off
powershell -noexit "docker exec php_container_name php %*"

settings.json:
...
"php.executablePath": "C:\\Projects\\php.bat"

The original code created a local node server that starts the language server in "tcp" mode that connects to this server over TCP. The local server is then passed to vscode / extension and when they talk to this server they actually talk to the language server.
The new code starts the language server in "tcp-server" mode, listening on port 1337 which is mapped between the container and the host. But it is still the local node server that is passed to vscode / extension, only now it isn't connected to the language server.
What I think should happen is: the language server should start just like now, then a socket should be created from "here" to the language server and it is this socket that has to be passed to resolve({ reader: socket, writer: socket });
The problem to overcome is that socket must be created asyncronously, after the language server has started and is listening.

The only way for now to make the extension work is installing this Remote - Containers
After installation, you can attach your running containers (devilbox PHP for me)
to your workspace and then you can install PHP-intellisense

@kukocrunch good shit! I hopefully will not find myself working with php much in the future; but if I do, I'll give this a shot


Token address in asset list

Overview
As a user I would like to easily retrieve the public address of a token contract within my asset list.
Goals

adding a copy address button to each asset in the asset list
(optional) adding a link to the respective blockchain explorer

Requirements
tbd
Screens
first draft

Links

We already had a similar discussion in the mobile team. We need to be super careful that users do not confuse the token address with an address to receive assets on, because they would be lost forever. We should definitely coordinate this together with mobile to come to a solution that works with both platforms and that is not a security risk.

@tschubotz already created a ticket addressing this issue https://github.com/gnosis/safe/issues/442


Replace oscar dashboard navigation with menu registry

This PR aims to provide an alternative approach to the dashboard navigation menu registration. It replaces the OSCAR_DASHBOARD_NAVIGATION setting with a hopefully more flexible registry pattern. See the updated doc on how the approach works.

Please let me know if this is ready for review again

Please let me know if this is ready for review again

@specialunderwear it is - please review.


docs: Add support for including markdown files in others.

This uses a third-party dependency, python-markdown-include.
The example use for settings is intended primarily as a demonstration.

Automated message from Dropbox CLA bot
@timabbott, it looks like you've already signed the Dropbox CLA. Thanks!


API for overriding Sampler default parameter values at construction time

It would nice to be able to set the default values for Sampler.sample keyword arguments on sampler construction.
Something like
class ASampler(dimod.Sampler):
    ...

    def sample(self, bqm, a=1, b=1, c=1):
        ...

sampler = ASampler(parameter_default=dict(a=5, b=3))
sampleset = sampler.sample(bqm, a=3)  # 3 overrides the 5, but b=3 is used

Generalizes #501
See the use of embedding_parameters in EmbeddingComposite

That's exactly what **runopts in Hybrid are for -- but right now the interface is a mess and I'm working on generalization (under https://github.com/dwavesystems/dwave-hybrid/issues/141). It proved to be non-trivial.

The standard terminology for this functionality seems to be feed_dict, which I personally dislike, but has permeated the tech world via TF. What about parameter_dict instead of ..._default? It's more descriptive of what is expected.


Leg Eater doesn't repair broken charms

Player.log
ModLog.txt
What was expected to occur: Leg Eater repairs fragile charms that are broken
What actually occurred: Leg Eater has no way to purchase broken charm repair
Steps to reproduce:

Die with any fragile charm equipped
Visit Leg Eater

Thanks, will look into it.

I have been informed on HK Modding Discord that this requires the LegEaterRepair item to be placed in Leg Eater's placement. This can be done by altering ShopLocation.defaultShopItems when placing that particular shop.


NRE on any API method

A customer is using some of our sample code from our docs.
AssemblyAI client = AssemblyAI.builder()
        .apiKey("")
        .build();

String audioUrl = "https://storage.googleapis.com/aai-web-samples/5_common_sports_injuries.mp3";

var params = TranscriptOptionalParams.builder()
        .speakerLabels(true)
        .build();

try {
    Transcript transcript = client.transcripts().transcribe(audioUrl, params);

    System.out.println(transcript.getText());

    transcript.getUtterances().ifPresent(utterances ->
            utterances.forEach(utterance ->
                    System.out.println("Speaker " + utterance.getSpeaker() + ": " + utterance.getText())
            )
    );
} catch (Exception exception) {
    log.error(exception.getMessage() + " " + exception.getCause());
}

They're getting the following error:
Cannot invoke "com.assemblyai.api.core.RequestOptions.getTimeout()" because "requestOptions" is null null

When I take a peek at the code, every API method accepts request options, but they're never checked for null, so every API method is broken.

This change has been released in 1.1.0.


Always pass CoreCLR msbuildonunsupportedplatform

This makes CoreCLR try to produce packages where it would normally skip doing so, e.g. Fedora 26. This allows the CoreCLR dependency to flow.

Thanks for verifying!
I saw in CI that this arg is only supported in CoreCLR's build.sh, not build.cmd, so I added a fix for that.


Blocked navigation - Should transitions be local?

We ran into an issue which blocked navigation with Routify: https://github.com/sveltech/routify
After a lot of research it seems that the animation (fade) of the Alert component blocks the navigation. It's hard to debug because it's a timing issue so hard to reproduce.
Advice received is that the transition should be local:
https://svelte.dev/tutorial/local-transitions
https://github.com/bestguy/sveltestrap/blob/master/src/Alert.svelte#L28
Is that something which has been considered for Sveltstrap?

I haven't seen the repo in question, but it sounds like this bug.
https://github.com/sveltejs/svelte/issues/3685
If that's what we're looking at, the easiest fix would be adding a local directive to the transition.
out:fade => out:fade|local

Thanks @jakobrosenberg
I did some further research on this issue and came to the following conclusions:
The fade = true / false setting and the UncontrolledAlert do not make any different on this issue.
What does work is just using the static example HTML from bootstrap:

Well done!

Aww yeah, you successfully read this important alert message. This example text is going to run a bit longer so that you can see how spacing within an alert works with this kind of content.

Whenever you need to, be sure to use margin utilities to keep things nice and tidy.

Then the issue is gone. So that seems to show we can really pin it down to how the alert is created in svelte strap.

+1 for this issue, still blocking the Alert component with routify.
Any update ? Thanks !


Bottom temperature plots dont display < 1999 data

@seanhardison1 @slarge
The bot_temp_fall_spdf.rdata file in data-raw only contains data from 1999 data.
I don't see any R files (in data raw) that creates this data. The data files on the google drive do indeed have earlier years.
Where is the code that processes the google drive data? Does it reside outside of this project?

The files are too big to have on github uncompressed, but the code to process them is in data-raw/collapse_into_stack.R. Let me run it again locally and see what happens

Not sure why older years weren't stacked but it's fixed now :man_shrugging:


Graph Mark with example nbs

Thanks @ChakriCherukuri
I have a few suggestions:

Change the default color of the line in the tooltip
Add a bit of extra padding to the graph

Thanks @ChakriCherukuri This looks great!


should u extension restrict or expand the return of  defaults ?

We need to decide the semantic of u extension toward default
Let's let me use zh locale to explain
zh Locale has ["gregory", "chinese"] as commonly used calendars
zh-TW has  ["gregory", "roc", "chinese"] as commonly used calendars
now, what is the commonly used calendar for "zh-TW-u-ca-japanese" locale?
There are two possible answers
A.  ["japanese", "gregory", "roc", "chinese"]
B. ["japanese']
Answer B mean since zh-TW-u-ca-japanese mean Chinese in Taiwan using Japanese calendar system, the commonly used calendar is only Japanese, but not including [ "gregory", "roc", "chinese"] since the ca-japanese already restrict the calendar to japanese in locale
Another example to consider
ar has  ["gregory", "coptic", "islamic", "islamic-civil", "islamic-tbla"] as commonly used calendars
ar-SA has  ["islamic-umalqura", "gregory", "islamic", "islamic-rgsa"] as commonly used calendars
ar-EG has ["gregory", "coptic", "islamic", "islamic-civil", "islamic-tbla"] as commonly used calendars
now, what should be the defaults.calendars for ar-u-ca-persian, ar-SA-u-ca-persian, ar-EG-u-ca-persian ?
A
(new Intl.Locale("ar-u-ca-persian")).defaults.calendars  return
["persian", "gregory", "coptic", "islamic", "islamic-civil", "islamic-tbla"]
(new Intl.Locale("ar-SA-u-ca-persian")).defaults.calendars  return
["persian", "islamic-umalqura", "gregory", "islamic", "islamic-rgsa"]
(new Intl.Locale("ar-EG-u-ca-persian")).defaults.calendars  return
["persian", "gregory", "coptic", "islamic", "islamic-civil", "islamic-tbla"]
B
(new Intl.Locale("ar-u-ca-persian")).defaults.calendars  return
["persian"]
(new Intl.Locale("ar-SA-u-ca-persian")).defaults.calendars  return
["persian"]
(new Intl.Locale("ar-EG-u-ca-persian")).defaults.calendars  return
["persian"]
@sffc

My intuitive response is that if we are returning ordered list of preferences, then we should not restrict that list, but extend and order based on the signals. The only signal we get from U extension is the "most preferred" and in result it should be in the front, as it's the strongest most specific signal we have. The second is the list of ordered calendars from the lang/reg pair and it should follow.

The only signal we get from U extension is the "most preferred"

that is the question- is it represent "most prefer" or "the only ONE preferred"?
is that really true it represent"the most preferred" instead of "THE only preferred"? That is the question.
if you look at the ar-SA vs ar case (I am not talking about ar-u-ca-persian vs ar-SA-u-ca-persian, I am talking about ar vs ar-SA) you will see where I came from
notice ar has ["gregory", "coptic", "islamic", "islamic-civil", "islamic-tbla"] but
ar-SA has ["islamic-umalqura", "gregory", "islamic", "islamic-rgsa"]  but
NOT ["islamic-umalqura", "gregory", "coptic",  "islamic", "islamic-rgsa"]
and "coptic" is not in that list
so ar-SA mean the user "ONLY PREFER" what used in Arabic in Saudi Arabic not "most prefer" what used in Arabic in Saudi Arabic therefore "coptic" is now not in that list, right? If it carry a "most prefer' semantic, then how could we justify the removal of "coptic"?

I don't believe that there's ever "the only one preferred" for two reasons:

No instrument in Unicode Locale is intended for listing exclusion lists
No fallback can ever really be a single-item. It's an illusion. If you may need a fallback, and you carry a single-item, then you implicitly build a two-item fallback list of ["the-item", "last-fallback"]. Once you accept it, then all we can do is improve from that two-item list so that there's something to fallback on before the last-fallback.
In this case I believe that the list of items that are not "the-item" and not "last-fallback" in the middle improves the user experience.

so... now... look at the hc case
en is ["h12"]
en-GB is ["h23"]
what should en-GB-u-hc-h11 return?
if I follow what you said, it should be
["h11", "h23"]
right?
But why it shouldn't be
["h11", "h23", "h12"]
if I follow your logic?

No instrument in Unicode Locale is intended for listing exclusion lists
Agree, but they all carry "more specific" information, right? when it express more specific information, it should "narrow down" the possible choices, right?

No fallback can ever really be a single-item
Sorry, I do not understand the relationship between "fallback" and the issue we are discussing here. Could you express your link explicitly?

But why it shouldn't be
["h11", "h23", "h12"]

I see no reason why it shouldn't be.

Agree, but they all carry "more specific" information, right? when it express more specific information, it should "narrow down" the possible choices, right?

I think it should set the preference.

Sorry, I do not understand the relationship between "fallback" and the issue we are discussing here. Could you express your link explicitly?

Since Intl API design is inherently "best effort", it means that we roughly collect all the possible signals, prioritize them, and come back with the best possible answer. That always involves fallbacks because there's always a "what if the best option is no available".
In most cases the "what if" is hypothetical (what if we don't have western arabic numerals? if we don't have them, we have a much bigger problem that this single format output ;)), but in most of the long tail, this is a reasonable problem - user tells us that they like "h23", and they use a locale that prefers "h12", cool, let's try "h23", and fallback on "h12" if the former "doesn't work" for some reason. And if those two don't work, then we can either randomly select one that works, or try to prioritize it - we can prioritize it by knowing the parent locale of the locale the user gave us, or the second best locale that the user gave us, if they gave us a fallback chain, and maybe this one has some value that we can support, before we fallback on "last resort fallback",
The "hourCycle" in my narrative is just an example and applies to every setting and data key in Intl as far as I can see.
What I'm saying is that any single value is not guaranteed. You ask for "en-US"? Maybe we have it, maybe not You ask for "u-hc-h12"? Maybe we have it, maybe not. You ask for "gregorian"? Maybe we have it maybe not.
The only real question is what we'll do if we don't.
And my take is that we always end up encoding "last resort" fallback and the whole API design game is trying if we can interpret a signal that will allow us to fallback on something better than "last resort".
So, to loop it back to your first post: ar-SA-u-ca-persian - user gave us an explicit signal that they want persian calendar. So that's the strongest signal we have and i result it should be top of the ordered list.
What then? If we don't have persian, and you returned ["persinal"] then what happens next? Well, the only thing that can happen is that the system will jump directly to "last resort" which in case of ICU is likely und and gregorian.
But is it possible that we can do better? Well, yes we can! The user also told us that they want ar-SA so maybe we can use that signal to give them a fallback that is better than last resort!
So, here it is: ["persian", "islamic-umalqura", "gregory", "islamic", "islamic-rgsa"].
You just provided a better fallback scenario than you would allow for if the return list was ["persian"].

but then you can argue user also tell use they want "ar", and "coptic" is also used in "ar", then should we add "coptic" to that list too? even "coptic" is not part of ar-SA?

Soo, I would :) But I believe that it's a part of the game and I believe others may have other, valid opinions because eventually, if you drag my position ad extremum, you get to "take all possible values and order them". So the sweet spot we're looking for is between getting as many ordered meaningful values as possible but not including meaningless.
What is meaningful and meaningless is subjective and hard to evaluate (the "weights" in language negotiation model is an attempt that I don't think works as well as it was intended to).
For coptic, you get diminishing returns the further down the chain you go - first value is critical, first fallback after it is very important, second much less, third very little, and so on, until "last fallback" because it is a catch all for the long tail.
So my take would be that added complexity of ordering from parent locale chain is likely not valuable, but I can see an argument that my position is subjective and arbitrary and I agree with it :)
The bottom line is that I'd love to underspec it and allow for "best effort" to produce such list and allow us to iteratively improve it over time as the fallback logic improves.

At first, I was in the camp that we should append the preference to the front of the list, but now, I feel that we should return an array of length 1.  Here's why:

I see this API as a way to fill in "likely extensions", in the same vain as "likely subtags".  If the calendar is not specified, we return a list of likely calendars.  But if the calendar is specified, we only need to return one.
Easy to explain.
We aren't losing any functionality, because you can just unset the Unicode extension keyword, and you get the full list back again.
(minor point) Surprising/unpredictable behavior of the calendar is already in the list.  For example, what do we return for ar-u-ca-coptic, given that "coptic" is already in the list of likely calendars?  Do we reorder the list and pull "coptic" to the front?  Do we have two entries for it?

Do we reorder the list and pull "coptic" to the front?

Reorder. It should really be a Set.

Do we reorder the list and pull "coptic" to the front?

Reorder. It should really be a Set.

I am very against to treat them as a Set, right now is an order list and that is what it really is.

I am very against to treat them as a Set, right now is an order list and that is what it really is.

I apologize. I meant a set as "deduplicated", not "unordered". I believe it should be ordered.

Another reason of the "adding" approach is that assume the user try to communicate "I also know X and I prefer X" instead of "I only know X but nothing else" . For example,
ar-SA has ["islamic-umalqura", "gregory", "islamic", "islamic-rgsa"]
How can an users express "I only know about 'gregory' calendar and I know nothing about "islamic-umalqura",  "islamic", "islamic-rgsa" calendars and please don't bother to show them to me?

How can an users express "I only know about 'gregory' calendar and I know nothing about "islamic-umalqura", "islamic", "islamic-rgsa" calendars and please don't bother to show them to me?

I'll repeat my position from https://github.com/tc39/proposal-intl-locale-info/issues/12#issuecomment-799648371

Unicode does not have exclude-lists. There is no way "I can't read h12" or "I don't understand german". You only say "I prefer X" or "I prefer Y and if not Y, then Z".
All signals in Unicode Locale are positive signals that are meant to bring more value to the ordering of the fallback.

But, ultimately, and much more importantly, such exclusion would give us nothing.
In a scenario of a fallback of [A, B, C] the only case in which B matters is if we don't have A.
And if we don't have A, and the user told us that they only understand A then we still have to fallback, and in the absence of any better signal we'll fallback on last resort.
Which brings me back to the notion that all we're doing is trying to improve over the last resort. And in general deriving the calendar system from the communicated to us locale is imho much better than falling back on last resort.
To rephrase again my position, the idea that you can return a single element and somehow that single element is meaningful on its own is an illusion because in practice I have never seen any other behavior of the consumer of such input than remodelling [A] into [A, last_resort] and falling back through the ordered list. So every scenario in which you can improve upon it by doing [A, B] to allow for [A, B, last_resort], you are improving the UX in error scenario.

By my recollection, it is in scope for UTS 35 locale identifiers to eventually add support for multiple preferences if it doesn't support it already.  For example, in the future, something like en-US-u-ca-hebrew-ca-gregory could mean that your preference list is ["hebrew", "gregory"].  But as of now, UTS 35 supports only one preference at a time, and I think we should reflect that.  We should work with Unicode to add multi-preferences (as well as improvements to things like the measurement system preference) to UTS 35.

ok, I put both approach in the updated spec now. see https://tc39.es/proposal-intl-locale-info/#sec-create-array-from-list-and-preferred. the algorithm between [[[ BEGIN OF DISCUSSED CHANGES]]] and [[[END OF DISCUSSED CHANGES ]]]
in CreateArrayFromListAndPreferred specify what zibi think we should do. Removing that part just add the extension as a single item in the list.
I also split each part into it's own abstraction operation to return an array, so if we decide to change the structure we can easily do so. I added both approaches under the two different part below the section
https://tc39.es/proposal-intl-locale-info/#sec-Intl.Locale.prototype.region
and mark with red
[[[ BEGIN OF APPROACH 1]]]

[[[ END OF APPROACH 1]]]
[[[ BEGIN OF APPROACH 2]]]
[[[ END OF APPROACH 2]]]
So in the next ECMA402 we can just decide which version we like to take and maybe do a simple name changes.

@zbraniecki - are you willing to serve as one of the two Stage 3 reviewers? #9 so I can close down that issue?

@zbraniecki - are you willing to serve as one of the two Stage 3 reviewers? #9 so I can close down that issue?

I'm concerned about my capacity to pick it up at the moment. I may end up slowing you down :(
Maybe if you can't find anyone else in a couple weeks I can serve as a fallback reviewer?


material.admin doesn't work on version 1.2

Steps to reproduce

Download and unpack test_material.zip
Run pip install -r requirements-latest.txt
Run ./manage.py runserver
Navigate to http://<IP_ADDRESS>:8000/admin (username: demo, password: demodemo)

Expected result: admin site looks differently from default.
Actual result: admin site looks exactly the same as if there were no material.admin listed in INSTALLED_APPS.
The issue is reproducible on Django 1.11 as well (pip install -r requirements-latest-django1.txt).
The issue is not reproducible on django-material 1.1 (pip install -r requirements-working-admin.txt).

Yep, check the changelog.

Oh, I see. I did not expect major features to be removed in a minor release, so I assumed this could be a regression.


NSwag Studio typescript client generate setting request

Here is situation, if an operation has many optional parameters, like:
[HttpGet]
public void Operation(int? a, string b, bool? c) {}

the Studio will generate typescript client operation like this:
operation(a?: number | undefined, b?: string | undefined, c?: boolean | undefined): Observable;

if I only want pass 'c' parameter when I invoke this operation, I have to also pass 'a' and 'b':
operation(undefined, undefined, false);

if there are many many optional parameters, it will become:
operation(undefined, undefined, undefined, undefined, undefined, undefined, false);

so, my point is, can Studio provide a setting that wrap all parameters into a query object, it will become more friendly when invoke an operation:
// operation declaration
operation(query: QueryPamaeter): Observable;

// query object declaration
interface QueryPamaeter {
  a?: number | undefined;
  b?: string | undefined;
  c?: boolean | undefined;
}

//invoke
operation({
  c: false
});

https://github.com/RicoSuter/NSwag/issues/950

closed, see: #950


maint: update cloud sdk action

closing this out for now, can do update at a later point when we refresh/update credentials


Argument/option values that are "sentences"

Is there a way not to split the values on whitespace? I would like to take in some argument/option values that have whitespace, and I was wondering if there was a built-in feature for this, other than using an environmental variable or a prompt.
ex. taking in a filename that has whitespaces as an argument.

@matejcik That doesn't seems to work for arguments.

@romabysen it works for me, can you be more specific?

Same here Python 3.6.5, click v6.7
@click.command()
@click.option('--sentence')
def sayit(sentence):
    print(sentence)

$ python cli.py --sentence "The quick brown fox jumps under the lazy dog."
"The quick brown fox jumps under the lazy dog."

@click.command()
@click.argument('sentence')
def sayit(sentence):
    print(sentence)

$ python cli.py "The quick brown fox jumps under the lazy dog."
"The quick brown fox jumps under the lazy dog."


Can thermald and zcfan be used together?

I currently have thermald installed, but I wonder if it will have any conflicts with this package, many thanks in advance

They should work, yes.


Skill list

To make the content more readable, place the skill lists side by side

Fixed this in: 67cc2387617b1b18c7da611701bb9d9116946b0c


Rebased #617

What does this PR do?

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

Could you help fixing this failing test ? https://github.com/huggingface/text-generation-inference/actions/runs/5892788243/job/15982858619?pr=868#step:10:411

Could you help fixing this failing test ? https://github.com/huggingface/text-generation-inference/actions/runs/5892788243/job/15982858619?pr=868#step:10:411

On it

Can't easily commit to this branch, but this should fix it:
diff --git a/server/tests/utils/test_tokens.py b/server/tests/utils/test_tokens.py
index 46b1220..0b014ca 100644
--- a/server/tests/utils/test_tokens.py
+++ b/server/tests/utils/test_tokens.py
@@ -47,9 +47,10 @@ def test_stopping_criteria_max():

 def test_batch_top_tokens():
     top_n_tokens = [0, 2, 3, 4, 5]
+    top_n_tokens = torch.tensor(top_n_tokens)
     inp_logprobs = torch.tensor([[-1., -3., -4., -2., -3.]] * 5)

-    topn_tok_ids, topn_tok_logprobs = batch_top_tokens(top_n_tokens, inp_logprobs)
+    topn_tok_ids, topn_tok_logprobs = batch_top_tokens(top_n_tokens, top_n_tokens, inp_logprobs)

     assert topn_tok_ids[0] == []
     assert topn_tok_ids[1] == [0, 3]

I basically just forgot to update the test after placing the requested n on the gpu 010c7e7

top_n_tokens = [0, 2, 3, 4, 5]

top_n_tokens = torch.tensor(top_n_tokens)

Are they always supposed to be the same ?
If yes, we should remove the arg, no ?
If no, then can we use a different value to showcase the test a bit better/ catch more errors ?

They are (should be) always the same. During the calculation. We need the standard top_n_tokens for some CPU calculations, but also need them on the GPU to filter the logprobs without either having to move the top_n_tokens to the GPU or the logprobs to the CPU. The tensor is normally allocated during batch creation to reduce overhead.

They are (should be) always the same. During the calculation. We need the standard top_n_tokens for some CPU calculations, but also need them on the GPU to filter the logprobs without either having to move the top_n_tokens to the GPU or the logprobs to the CPU. The tensor is normally allocated during batch creation to reduce overhead.

Makes a lot of sense ! Then perfect !.

Seems the issue is nothing linked here; https://forums.developer.nvidia.com/t/unable-to-add-apt-repo-mirror-sync-in-progress/263504/24
I'm merging this now...

Hey @Vinno97 !
Finally getting to merge this !
Thanks for bearing with me while I coped with everything while @OlivierDehaene was away :).

Thanks for merging! And no worries, understandably a busy time ;)

@Narsil Top tokens were not implemented for the Galactica models and is causing issues. https://github.com/huggingface/text-generation-inference/issues/1004

Hello, thanks for this PR, I'm struggling to get the logprobs in the response even with top_n_tokens > 1 when serving Falcon 7B with TGI ? Do you think this is due to the model being incompatible or a configuration error on my part ?

Can you open an issue with the full template filled in ?
This is not enough information to help you.

The issue that others and I have is that top_n_tokens doesn't work, even in the newest huggingface-hub package (0.19.4):
TypeError: InferenceClient.text_generation() got an unexpected keyword argument 'top_n_tokens'

Seems like the huggingface-hub package misses this argument in their implementation. Though this should be reported to their repo, then: https://github.com/huggingface/huggingface_hub/tree/main


Update Firefox ESR to newer point release

It's currently Firefox ESR 91.6.0. Phyrox ESR is lag behind at 91.5.0.

@3oheicrw As explained on this page, "We are aware of and notified of new application updates. So there’s no need to create an issue to update an application.". But feel free to open a PR.

Please update to 102.4.0esr, you are still at 102.3.0esr. Thanks.


Navigation drawer MDNavigationDrawerItem cursor pointing highlight at wrong position

Referencing sample code from https://kivymd.readthedocs.io/en/latest/components/navigationdrawer/#standard-content-for-the-navigation-bar , when using MDNavigationDrawerItem with MDNavigationDrawerMenu, the item highlight is dislocated slightly above consistently.
from kivy.lang import Builder

from kivymd.app import MDApp

KV = '''
<DrawerClickableItem@MDNavigationDrawerItem>
    focus_color: "#e7e4c0"
    text_color: "#4a4939"
    icon_color: "#4a4939"
    ripple_color: "#c5bdd2"
    selected_color: "#0c6c4d"

<DrawerLabelItem@MDNavigationDrawerItem>
    text_color: "#4a4939"
    icon_color: "#4a4939"
    focus_behavior: False
    selected_color: "#4a4939"
    _no_ripple_effect: True

MDScreen:

    MDNavigationLayout:

        MDScreenManager:

            MDScreen:

                MDTopAppBar:
                    title: "Navigation Drawer"
                    elevation: 4
                    pos_hint: {"top": 1}
                    md_bg_color: "#e7e4c0"
                    specific_text_color: "#4a4939"
                    left_action_items: [["menu", lambda x: nav_drawer.set_state("open")]]

        MDNavigationDrawer:
            id: nav_drawer
            radius: (0, 16, 16, 0)

            MDNavigationDrawerMenu:

                MDNavigationDrawerHeader:
                    title: "Header title"
                    title_color: "#4a4939"
                    text: "Header text"
                    spacing: "4dp"
                    padding: "12dp", 0, 0, "56dp"

                MDNavigationDrawerLabel:
                    text: "Mail"

                DrawerClickableItem:
                    icon: "gmail"
                    right_text: "+99"
                    text_right_color: "#4a4939"
                    text: "Inbox"

                DrawerClickableItem:
                    icon: "send"
                    text: "Outbox"

                MDNavigationDrawerDivider:

                MDNavigationDrawerLabel:
                    text: "Labels"

                DrawerLabelItem:
                    icon: "information-outline"
                    text: "Label"

                DrawerLabelItem:
                    icon: "information-outline"
                    text: "Label"
'''

class Example(MDApp):
    def build(self):
        self.theme_cls.theme_style = "Dark"
        return Builder.load_string(KV)

Example().run()

https://github.com/kivymd/KivyMD/assets/33471049/2fdfd671-6381-4d38-a8dd-1385bf725102
Environment:
Windows 10 22H2 Build 19045
Python 3.10.5
Kivy 2.2.1
KivyMD 1.2.0.dev0 from master branch

I have the same problem yet couldn't find any solutions. Can you share if you find a solution please?

This is a Windows-only problem. Check it out only with Kivy widgets

So are there any solutions for this or not?

Check it out only with Kivy widgets...

Do you planning to fix this in the newly coming kivy versions?

The KivyMD library is not Kivy

I meant kivymd

Check it out only with Kivy widgets


Using the following I get a strange quirk...

interface HomePageState {
    translate: any;
}

type HomePageProps = HomePageState

class Home extends React.Component<HomePageProps, HomePageState> {
    /// the below will only render "I"
    public render(): any {
        return <label>{ this.props.translate("home.slogan") }</label>;
   }
}

const addTranslateToProps = state => {
    console.log('state.locale', state.locale);
    // this will log out "translations: { home.slogan: "I'm the correct translation" }
    return {
        translate: getTranslate(state.locale)
    }
};

export default connect(
    addTranslateToProps
)(Home) as typeof Home;

To be clear:
The translations state seems to contain the correct translation / localization, but the translate function exposed via props, does not work as expected.
I do believe it might be a dependent lib (reselect??) thats causing the problem, but not too sure... if anyone has encountered this before, any advice would be appreciated. I'm happy to supply any other information thats needed.
Other info:
Stack is on top of dotnetcore, React + Redux + SSR (with webpack for bundling front end stuff)

Would you be able to provide the following:

what the unexpected result is you are encountering? What is being rendered for translate('home.slogan')
a sample of your original translation data that you pass to addTranslate
a look at the languages array you are passing to initialize
what version of library are you using?

Thanks

No problem:
This is my actual translation. (I actually scrubbed out my real translations in the previous sample, so the apostrophe shouldn't be an issue... but yolo, please see below.)
localizations.json
{
    "home": {
        "slogan": "Search among thousands of jobs in one place!",
        "search-label": "What are you looking for?",
        "search-button-text": "Search",
        "input-placeholder": "Search for Job Title, Location or Company"
    }
}

Gets loaded in my boot server like this.
       // get translation file
        const json = require('./localizations.json');
        store.dispatch(initialize(['localizations'], { defaultLanguage: 'localizations'}));
        store.dispatch(addTranslation(json));

I'm using 2.10 (as of last night, prior i was on 2.8... but same issue)

Ok this helped thanks for sending over.
The main issue appears that you are using the wrong action to add your translations. You are using "addTranslation", but you should be using "addTranslationForLanguage". This is because you're adding translations in the "single language format". If you haven't already, make sure to review the Formatting Translation Data section in the docs
https://ryandrewjohnson.github.io/react-localize-redux/formatting-translation-data/
In addition to this there are some oddities in the way you initialized things:

It appears that you have only passed a single language to initialize. I'm going to assume that you'll be adding more languages later?

The single language you passed is called "localizations", which will work but the intention is this should be an identifier for  the language. (E.g. en, en-US, english) etc..

You passed in a defaultLanguage setting to initialize, but by default the first language you pass to initialize will already be the default. The only time you need to pass defaultLanguage is if you want it to be something other than the first language in the array of languages.

Hope that helps.
Sent from my iPhone

On Sep 18, 2017, at 10:15 PM, Rohan Büchner<EMAIL_ADDRESS>wrote:
No problem:
This is my actual translation. (I actually scrubbed out my real translations in the previous sample, so the apostrophe shouldn't be an issue... but yolo, please see below.)
localizations.json
{
"home": {
"slogan": "Search among thousands of jobs in one place!",
"search-label": "What are you looking for?",
"search-button-text": "Search",
"input-placeholder": "Search for Job Title, Location or Company"
}
}
Gets loaded in my boot server like this.
   // get translation file
    const json = require('./localizations.json');
    store.dispatch(initialize(['localizations'], { defaultLanguage: 'localizations'}));
    store.dispatch(addTranslation(json));

I'm using 2.10 (as of last night, prior i was on 2.8... but same issue)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Thank you... feel so dumb for missing that O_o
It's working. Thank you for the guidance (writing it off to 3, 12 hour days)

Not sure yet, It might be sufficient for me to swap out the same file as part of my build process (eg the users wont have the option to change localization (or in my case app branding))

See above, i'm in effect building a white label portal, and I'm using the localize/translation feature to swap out new app lables

My TypeScript / webpack build was b*tching about it... not sure why :/

Thanks for this awesome lib. (It actually wasn't too tough to get up an running... albeit my ~scrappy~ doc reading abilities :P

Glad it worked out. I think I'm going to try and tighten up the typings for the translation data, as that might help catch this issue for anyone using TypeScript or Flow.


Remove use of StandardAction

Fixes #78
Related to https://github.com/ReSwift/ReSwift/pull/270

Codecov Report

Merging #82 into master will increase coverage by 3.63%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #82      +/-   ##
==========================================
+ Coverage   80.28%   83.91%   +3.63%
==========================================
  Files           5        5
  Lines         208      199       -9
==========================================
  Hits          167      167
+ Misses         41       32       -9

Impacted Files
Coverage Δ

ReSwiftRouter/NavigationActions.swift
100% <ø> (+60%)
: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 02f8333...5853249. Read the comment docs.

We should see about moving this conformance into an extension in a separate file, and have a cocoapod subspec for recording conformance IE pod 'ReSwift-Router/Recording' or something

@mjarvis that sounds reasonable. How will we do it through Carthage? Might it be reasonable to move standard action back to ReSwift and add conformance in ReSwift-Recorder, ReSwift-Router and other future extensions and development tools built upon ReSwift?

@hlineholm I'm not certain how it would be done with carthage. Perhaps a separate framework would be needed, ReSwift-RecordableRouter or such, that provides the extensions, and has ReSwift & ReSwift-Router as dependencies?
I'm not inclined to add StandardAction back to ReSwift, as its not a core framework feature, but it does seem to make things confusing in regards to other frameworks providing Recorder capabilities...

@mjarvis FYI: I'm playing around with an action setup where I'm creating a "standard action" which u then specialise by setting the payload type. And I thought maybe this could be something we could use as a StandardAction in ReSwift. Not sure though how to make the dynamic payload type conform to CustomStringConvertible so that it can be serialized/deserialized.  This might be totally unuseful but I thought I'd put it out there for anyone to discuss with me.
class Action<Payload>: ReSwift.Action, CustomStringConvertible {
    let payload: Payload
    init(_ payload: Payload) {
        self.payload = payload
    }
    var description: String {
        return "\(type)(\(payload))"
    }
    var type: String {
        return typeString(of: self)
    }
}

class CreateNode: Action<(path: ImagePath, node: Node)> {}
class CreateNodeSuccess: Action<Node> {}
class CreateNodeError: Action<Error> {}
class CreateSubnode: Action<(path: ImagePath, key: ParentKey)> {}
class CreateSubnodeSuccess: Action<Node> {}
class CreateSubnodeError: Action<Error> {}

Hi @hlineholm do you know when this pull request will be merged?
Thanks,
Ran.

Hi @mjarvis Are you planning to merge it to master?
Thanks.

A propos serializing/deserializing the payload: what about Swift Codable conformance? Wouldn't that be feasible?

@mjarvis @DivineDominion I suggest we approve this PR and add an Issue for further discussions related to StandardAction. This is currently causing a build error.

I'm currently in the mindset that ReSwift-Recorder is going to be marked deprecated / unfinished, and as such, I'll merge this PR as-is, and we can shelf the re-addition of the StandardAction work to these actions in subspecs. If a user chooses to use the two of these they can easily extend the actions to conform or fork the repo themselves.


All properties required option

Hello,
I would need a new option to make all properties in the openapi schema required:
--all-properties-required
This option will not be use to create valid example but to test the backward compatibility of minor versions.
My plan is to create for each minor version a test that requires to provide a maximum JSON.
Then, this json will be tested with the following minor version to make sure the backward compatibility is not violated.
Example

V1.0
JSON schema

type: object
properties:
  username:
    type: string

Maximal example
{"username":"admin"}

V1.1
JSON schema backward compatible

type: object
properties:
  username:
    type: string
  password:
    type: string

Maximal example
{"username":"admin", "password":"12345"}

V1.2
JSON schema NON backward compatible

type: object
properties:
  username:
    type: string
  password:
    type: array
    items:
      type: string

Maximal example
{"username":"admin", "password":["12345", "34567"]}

For the version 1.2 I will run the commands:
openapi-examples-validator --no-additional-properties --all-properties-required -s api1.2.yaml -e maximalExampleV1.2.json
openapi-examples-validator --no-additional-properties -s api1.2.yaml -e maximalExampleV1.1.json
openapi-examples-validator --no-additional-properties -s api1.2.yaml -e maximalExampleV1.0.json

And the test of maximalExampleV1.1.json will fail because V1.2 is not backward compatible
Let me know what you think about this new feature.
And if you don't have time to implement it, I can try to do it.

I like the idea 👍
Unfortunately, I don't have the time to attend this issue right now. So, feel free to create a pull request.


Fixing CI Testing function

This PR is due to Github actions changing the way they deal with conda environments (change occurred Nov 2020).
To accomadate, we are using a different github action (https://github.com/conda-incubator/setup-miniconda) to set up our virtual environment and to test.
Additionally, we are now using pytest instead of tox to run the tests.
Pull Request Checklist

[ ] Will your PR merge into the dev branch?
Exceptions will be made for urgent bugfixes.
[ ] Have you forked from dev?
If not, please rebase your PR onto the most recent dev tip.
[ ] Does your PR title succinctly describe the changes?
Explain to a new user by completing the sentence: 'This PR will: ...'
[ ] Fill out the template below.

Description of the Change

Release Notes

Looks excellent to see those green ticks come back!
This PR drops:

testing against openblas vs. mkl (but I can see a pretty easy way of bringing it back, just a separate job in workflows/main.yaml)
pytest-cov/coverage (should be reasonably easy to bring back)
but otherwise it’s a lot simpler.

Testing is urgent, so those two feature regressions are now in #138.

Looks excellent to see those green ticks come back!
This PR drops:

testing against openblas vs. mkl (but I can see a pretty easy way of bringing it back, just a separate job in workflows/main.yaml)
pytest-cov/coverage (should be reasonably easy to bring back)
but otherwise it’s a lot simpler.

Testing is urgent, so those two feature regressions are now in #138.

One last thing:
You’ve changed shell in the runner steps to just bash -l {0}.
I would be much more comfortable if you had the “Run pytest in Conda environment” step use bash -l -eo pipefail {0}.
If you don’t enable pipefail, then it will only report whether the last command succeeded or failed.
If you enable pipefail (as it was), then the errors for all commands in a step get checked—if any one of them failed, the whole step reports failure.
This is important, since it's not guaranteed that pip install . will always succeed (and we want to be alerted when this happens).

One last thing:
You’ve changed shell in the runner steps to just bash -l {0}.
I would be much more comfortable if you had the “Run pytest in Conda environment” step use bash -l -eo pipefail {0}.
If you don’t enable pipefail, then it will only report whether the last command succeeded or failed.
If you enable pipefail (as it was), then the errors for all commands in a step get checked—if any one of them failed, the whole step reports failure.
This is important, since it's not guaranteed that pip install . will always succeed (and we want to be alerted when this happens).

Build logs check out!
(I note we're missing build artifacts. That's OK, I'll just hand that off to #138 .)

Build logs check out!
(I note we're missing build artifacts. That's OK, I'll just hand that off to #138 .)


Error 500 upon attempting to add new host

Attempted to add a new host, Aristida longiseta to the db today yet was met with this error.

Was it this host? https://www.gallformers.org/host/2619

I am not sure what happened but I need to delete this: https://www.gallformers.org/host/2619
It has no data associated with it so it should not be a big deal. I will try and re-create it. It is somehow missing both its genus and family which is causing the host admin to not load..

I deleted and re-added and all seems ok: https://www.gallformers.org/host/2619 Not sure what happened. Something went wrong when adding that host originally. Like I said, somehow it had not Genus or Family assigned to it.

Alright, I originally added that host and genus because I was reading the Gagne catalog and thought that there was a species that formed galls on there. So I added the family and species, but I read more closely later and found out it was an inquiline. And so I attempted to delete the host family that was added.

Looks like it works now, just successfully added host

I will test (and fix) the Family deletion code that seems to be what went wrong.


Update publication-add-bot-to-microsoft-teams.md

Added information regarding existing conversation update on Teams and permission required to sideload app.

Closing as this was resolved in a separate internal PR.


Copy and move operations always report success even on error

Description
On Windows Explorer, when you drag a file to another directory but the file is still open, it will prompt an error message that it can't move the file because it is currently open with an another program. In Files (Community) however, it doesn't show an error, but instead shows that the task has been completed.
Steps To Reproduce

Take one file and one folder
Important: Open the file first
Move the file to a folder
The file is still on the original place pre-"moving", but Files (Community) shows that the move has been complete

Expected behavior
I expected that one should see an error message saying that the file has not yet been transferred because the file in question is currently open and one should save all work before closing and trying again the task they tried to do.
Files Version
Version: <IP_ADDRESS> OS Version: 10.0.19043.1466
Windows Version
Windows 10 21H1 OS Build 19043.1466
Relevant Assets/Logs
https://user-images.githubusercontent.com/88814097/151035531-a4ea4fe8-3545-4505-b06b-76bb92332de8.mp4

I am new to the Open source community. For one of my courses for college, we have to contribute to a project - this seems like an issue that I may be able to help with. Can you assign this to me please? I'll update along the way as I troubleshoot

@lightningHoneypot is this still an issue in the preview version? https://files.community/download/preview

@lightningHoneypot is this still an issue in the preview version? https://files.community/download/preview

Nope! It seems that everything is good to go!
There is one issue that Files still shows a checkmark on the ongoing tasks icon that it makes it look like a file transfer is successful, but then the error box appears stating that a process is still using the file in question. However, it might as well be an inconvenience (see video on what am I saying, if there's any questions tell me).
(excuse the crappy editing in DaVinci Resolve, I am still in the beginner stages and that is the reason why it took so long for my response)
https://user-images.githubusercontent.com/88814097/167963318-842da106-8d91-400f-9773-a05b8838813c.mov

@lightningHoneypot thanks for the response, can you open a separate issue for the status icon? It's deffinitly something that should be fixed.


fix(Confirm): All "show" buttons on the 'Confirm' docs page are not working

The docs show the issue: http://react.semantic-ui.com/addons/confirm
Just trying to use the 'Confirm' component and checking the docs, however, all Show button on the doc page doesn't work. And there isn't any debug information on the console.
I tried the page on Chrome, Firefox and Safari on OSX, all the same.

This is being worked on #634

Released in<EMAIL_ADDRESS>docs are also shipped.


Use vt to split and normalize vt - replacing vcfparser

http://gemini.readthedocs.org/en/latest/content/preprocessing.html

Added and passed v.0.5

Nice!!


Support for mobiles and small screens

Hi,
I would like to report small visual bug. When visiting on my iPhone, description is only partially visible. Probably overflow hidden.

The same problem is with landscape. In this case, there is no description at all.

Hello,
I was about to write the same issue. Been testing and I can see that you need to scroll in a specific place on the screen for the content to scroll. I think it has something to do with the position: absolute as well.
When I try to scroll on the content, the whole page scrolls.
Tested on iPhone X with latest stable release of iOS and iPhone 6 with iOS 12.4.5.

Hey! Thanks both of your for reaching out. I know that the current mobile experience if far from perfect. I'm working to make it better.
As a matter of fact, I've discovered and fixed an issue that rendered the GitHub icon in the footer to posses way too much height on mobile. I believe the current live version should look slightly better now.

Hi, @kettanaito thank you for fast reply.
It is much better now. But still, on the “port” page, last line is not visible and I could not find the right spot to scroll down.
Next, when I change to landscape, there is a lot of white space after headline and the text is distorted.

Finally, the part with the URL gets hidden when returning from landscape to vertical view. And once I could not even find the right spot to scroll it down and I had to reload the page to see it again. Sadly, this is not visible in the video.
Recorded on iPhone 6s, iOS 13.5.
Hope this helps.


Anyways thanks for helping me out with the Readme so much.


"I'm all done" message can be confusing

Yeoman tells us he is "all done" but then goes on to run bower install and npm install for us. It's a nit-picky point, but some members of our team have gotten confused by this message.
Something to the effect of "I am running bower install & npm install for you, and then I'll be done" could be an improvement.

Are you using the last version?

Now I am. Sorry!


GetWindowLong/Ptr functions can fail, but don't return a Result

Suggestion
If we look at the docs for GetWindowLongPtrA (or any of the A/W Long/LongPtr variants), they all can fail by returning 0, however the generated bindings just return an isize: GetWindowLongPtrA.
The error handling might not be as simple as checking for 0 and returning a Result since a proper return type would be Result<Option<isize>>, where it returns Ok(None) when the function returns 0 without having an error (for example when SetWindowLong has not been called).
I've never actually seen it fail so I'm not entirely sure what the fail conditions vs succeed conditions on a 0 return look like (and the docs aren't very clear), but it would be nice to have the error helpers applied to these functions.
The returned value on the Ptr variants is also a pointer, but it's probably best to keep it as an isize since GetWindowLongPtrA and GetWindowLongPtrW return pseudo-values when the wrong A/W version is called.

A developer may also store/retrieve a 0 via DWL_USER, making this difficult to handle generically.

I think this is a case where the caller needs to figure things out since the way that errors are reported is very API specific and cannot be readily inferred through metadata.

These APIs are already somewhat specialized and problematic (#1932) so I'd rather just leave this as is.


Why does Dapr pub/sub use Redis Streams and not Redis PubSub?

Ask your question here
Why does Dapr pub/sub use Redis Streams and not Redis PubSub?
https://redis.io/topics/pubsub

Because Redis Pub/Sub does not offer an at-least-once guarantee.


WIP: Dummy PR to test the checks

@sdake: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e_cni
0d5aef1c6e980721d0cf1acadd1f48e64a1ca007
link
/test e2e_cni

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

@sdake: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e_cni
dfa1386e552b243a610d5f9a44d4364400d5a27d
link
/test e2e_cni

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

/test e2e_cni

@sdake: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e_cni
dfa1386e552b243a610d5f9a44d4364400d5a27d
link
/test e2e_cni

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

/test e2e_cni

@sdake: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e_cni
dfa1386e552b243a610d5f9a44d4364400d5a27d
link
/test e2e_cni

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

@sdake: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e_cni
23704032446ab00fc5d9faf22a909edf71f1a16e
link
/test e2e_cni

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

@sdake: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e_cni
c14c63d8124ea27faf82e116b438e29079eb2c8f
link
/test e2e_cni

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

/test e2e_cni

@sdake: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e_cni
c14c63d8124ea27faf82e116b438e29079eb2c8f
link
/test e2e_cni

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

@sdake: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e_cni
279565f932d0340744438dee6043a841e68e208c
link
/test e2e_cni

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


Show description only visible in thumbnail

Description
Sometimes when looking for a new show to watch, Jellyseer only shows me the first 5 lines of the show's description in the thumbnail.
That's alright, no issue so far.
But when I then click the show to get more info, Jellyseerr states that there is no description available. Hence, the show page gives me LESS information than the thumbnail, which is a bit weird.
I could understand if there is no description available in general, but the description I can see in the thumbnail had to come from somewhere.
I want to point out that this does only happen with some shows, not all of them, but always the same ones. So there is probably some difference in the sourcing for the thumbnail's and the actual page's description.
If that's the case, a fallback logic which picks the thumbnail's source in case the page's source has no content would be very much appreciated!
Version
1.9.2
Steps to Reproduce

Search for a show, e.g. "A good day to be a dog"
Check the thumbnail and see the first 5 lines of the show's description
Click the thumbnail, head to the show page and find "No description available"

Screenshots
Thumbnail:

Description page:

Logs
No response
Platform
smartphone
Device
Samsung S23 Ultra, Samsung Flip 4
Operating System
Android 14
Browser
Chrome
Additional Context
Shows I've seen this behavior on:

A good day to be a dog
As beautiful as you
The vanished elephant
Piers Morgan Uncensored
Detective Maniam

I mostly clicking through the recommendations and picked those out showing this behavior.
Code of Conduct

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

This is a TMDB issue occuring when there is no description in the specified language.
When using the search endpoint, TMDB fallback to english if there is no description available in the specified language.
When using the item endpoint, TMDB doesn't fallback to english, hence the absence of a description.
I don't think there is much we can do on our side.

I assume falling back to the search endpoint in case no description is available and caching this somewhere is no option?
Not really, it would be an overload just for this specific use case.
But maybe we could do the request in the specified language, and then do a second request in English if the description is missing? @Fallenbagel wdyt?

You can also go on TMDB and contribute to add the translated description.

I assume falling back to the search endpoint in case no description is available and caching this somewhere is no option?

Not really, it would be an overload just for this specific use case.
But maybe we could do the request in the specified language, and then do a second request in English if the description is missing? @Fallenbagel wdyt?

We could do that. Tmdb is supposed to fallback to en but it seems it only works on specific endpoints (a bug apparently afaik) but for the time being we could manually fallback i guess? 🤔

Tmdb is supposed to fallback to en but it seems it only works on specific endpoints (a bug apparently afaik) but for the time being we could manually fallback i guess? 🤔

Yep I think we should do it. Not having any description on a media is kinda critical.
@MrMcDaddy could you please check that all fields other than the description are filled in correctly?

:tada: This issue has been resolved in version 2.0.0 :tada:
The release is available on:

v2.0.0
GitHub release

Your semantic-release bot :package::rocket:


standardize focus-areas READMEs

This PR addresses the issue of Standardizing READMEs for:

focus-areas/README.md
focus-areas/<focus-area>/README.md

The issue can be found here, and the doc for template can be found here
Updates done:

Added specific relative path format for pointing to the focus-areas as well as the metrics
Added subheading in for each focus-area
Reordered the focus-areas and metrics listed in the tables in lexicographical order to match the way github displays those files and folders

Signed-off-by: ritik-malik<EMAIL_ADDRESS>

this looks good
Thanks @ritik-malik


test(cases): multiple entry points ordering

Issues

Closes https://github.com/webpack-contrib/extract-text-webpack-plugin/issues/573

@evilebottnawi There seems to be an issue with one of the snapshots could you update please ?
https://circleci.com/gh/webpack-contrib/extract-text-webpack-plugin/70?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link

Either the snapshots need to be update or the PR should be closed

DEPRECATED
Please use: https://github.com/webpack-contrib/mini-css-extract-plugin


support for additional_info in track_metadata

https://listenbrainz.readthedocs.io/en/production/dev/json.html#payload-json-details

Thanks, I never got round to doing this bit. Do you need this functionality, or is it a nice-to-have?

I guess, it's nice to have for now.

Released as 1.1.0.


UHP Tyler Mapping Support

Benefit
Around 650 UHP troopers will benefit a refreshed mapping system, and the IT people in DTS will have less to maintain after moving the local physical GIS server to combine with an existing cloud server.
Acceptance Criteria

[ ] To create an map package that can be fed into Tyler's mapping platform.
[ ] Change mapping sources to be dynamic, either point to REST services, or automate the update process on the serverX.
[ ] Retire the physical ArcGIS server and move it to the 911 Geovalidation server.

Notes
Tyler is the record management system for the Utah Highway Patrol. Fatpot is the component for issuing citations and doing the statistical work where there has to be a lat/long.
This project will bring in new income for the group.
Risks
No response
Issue Reference
refs #325

Hank attended training with Tyler Mapping and learned more about the project. Hank and Greg will draft up a plan and billing agreement. Hank feels this project will take about 2 weeks.

Greg is currently negotiating the project scope and the contract cost with the agency. This project will start in FY25.
2 side project supports were provided: CAD-to-CAD statewide maps with CentralSquare and the Fatpot system.

@gregbunce - reach out to Trotta to see if they are still interested.

They are yet to switch over to Tyler Technologies. They are saying "soon".


ng-change and ng-show not working with Angular Chosen

Trying to create a simple experiment with ng-change and ng-show.
Using a select box to control what other divs are shown or hidden. So long as I include the "chosen" directive the below example does not work. (Once I remove the chosen directive, it works as expected).

                        Was an error experienced?
                        <select id="errorstatus" ng-change="checkError()" data-placeholder="Yes / No..." chosen class="chosen-select" ng-model="formData.errorstatus" ng-options="o.value as o.alias for o in options.yesno" >
                        </select>

                        Describe the error in detail:
                        <textarea  rows="3" class="form-control" id="errorDetails" placeholder=" " ng-model="formData.errordetails"></textarea>

Controller Function
$scope.checkError = function(){
    if($scope.formData.errorstatus)
    {
        console.log("TRUE");
        $scope.errorTrue = true;
    }
    else
    {
        console.log("FALSE");
        $scope.errorTrue = false;
    }
};

I should point out this issue only takes place if a user attempts to switch back and forth between options more than 2 times. (First two times, it works correctly. Third time onward it fails. The only way around this is to enable single_deselect mode and then to have the user manually hit the x").

Can you use $scope.$watch instead of ng-change?
Similar to #119 ?
http://plnkr.co/edit/DuFbSh?p=preview

Instead of ng-show and ng-hide you must use ng-if

Instead of ng-change, you must use ng-model and $scope.$watch

Read Docs for more info: http://leocaseiro.github.io/angular-chosen/

Need to update docs about that, actually, Also create a plunkr example.

Thanks @lpsBetty
I also found this
By the way, actually ng-change works well: http://plnkr.co/edit/UYyOVw?p=preview

So you can close this issue?


Instances now get assigned a default security group - "service-maker"

if the group doesn't exist on the AWS console, it is created and a rule for SSH is added.
if the group exists, nothing is done with it.

Looks pretty good, I have one suggestion for the test block structure.

Looks better, thanks!


How can I put the training data into the code?

I saw that the data used in this project is ffhq_256. I am wondering how I can input the data into the project. As I observed in the ffhq drive, there are many folders, for example, 0-1000, 1000-2000, etc. When training, do I need to merge them into one folder, or should I just leave them in their original folders on the drive?

Do you find the method that how to use the dataset? @Johnx69 Johnx69

Not yet, I have no idea about how to use the dataset


converting between strings and keywords

I've been working with a bunch of semi-structured data recently and frequently need to convert between clojure keywords, humanized tsv/csv headers, and camelized json keywords, depending on the data's source and heading.
What do you think about making the <style>izeing functions (humanize, captialize, camelize, etc) keyword-safe by adding clojure.core/name to their some-> threads?
cuerdas.core/keyword would be the counterpart, dasherizeing any strings before passing them to clojure.core/keyword.
I'll be writing this code anyway, so if you think it would fit in cuerdas, i'd happily write it here!

+1 please, the current versions are only work with strings but if you make them keyword safe, :+1:

Covered by #33


121641 audio player ii

react-native-linear-gradient wurde deinstalliert und wird über die news_app gezogen.

What is this? I've looked through it and it looks like you've implemented an audio player and all kinds of cool thing. Why was this never merged?
When I play audio in RNAF Video currently, I get all kinds of weird errors. I wish this had been merged :(


updated search recommendation

Added 10 more buttons in search bar
Related Issuse

Info about Issue or bug

fix #1863
fix #1864
Describe the changes you've made
A clear and concise description of what you have done to successfully close your assigned issue. Any new files? or anything you feel to let us know!
Checklist:

[ ] My code follows the style guidelines of this project.
[ ] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[ ] My changes generate no new warnings.

Screenshots

Original
Updated

original screenshot
updated screenshot

@sairish2001  i hv added 10 btns in search bar recommendation


[Feature] Creating new MAUI project from meteor commands

Hello,
so my idea is, maybe it'd be nice and not as hard to include a 'create new project' command in the meteor command palette.
As MAUI is using templates from CLI I think that this, should? be possible to implement.
Maybe in future.
BTW. love the project!

Hello! Thank you for your comment. It is a great idea but I need extra time to research a UI part of this feature (it may require creating a wizard that will configure a new project).

I request too 😁

As I described above, I do not have resources to implement this functionality. Therefore, I do not plan to add this feature in the nearest future. Still, I keep this inquiry open so that I can return to it when I have enough resources.
If this functionality is essential in your usage scenario, I recommend that search for alternative extensions as I cannot promise any updates here right now.
Thank you for your understanding.
Regards,
Nikita

The new C# Dev Kit seems to make this option no longer needed.


[Hotfix] Change to streaming reader for CSV schema inference.

This PR leverages pyarrow's streaming CSV reader for schema inference; instead of reading and parsing the entire CSV file to fetch the schema, this will fetch the schema from the first "block" read by the streaming reader. The block size is configurable at the pyarrow level as part of the CSV ReadOptions (although we don't currently expose this to the user), with a default of 1 MB.

Could we also do some quick sanity-check benchmarking to make sure this is working before merging?