4 GPUs, each with 12.2MiB. The utility loads more into rank 0, but it only gets up to about 4 plus GiB never close to 12.2GiB

cuBLAS error 15 at /go/src/github.com/jmorganca/ollama/llm/llama.cpp/gguf/ggml-cuda.cu:7586
current device: 0
⠸ 2023/12/02 22:53:21 llama.go:436: exit status 1
2023/12/02 22:53:21 llama.go:510: llama runner stopped successfully
[GIN] 2023/12/02 - 22:53:21 | 200 |  1.311500885s |       <IP_ADDRESS> | POST     "/api/generate"
Error: llama runner exited, you may not have enough available memory to run this model
The model in question is orca-2-13b.Q6_K:latest. A 6 bit quantized model, which I converted using ollama instructions.
EDIT: I have now also tried it with "mistral" doing the standard download via
ollama run mistral. When I enter something, it either produces lines of "####...." or fails altogether and dies.
The original model file is GGUF V3.
The converted size is about 10GiB
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 520.61.05    Driver Version: 520.61.05    CUDA Version: 11.8     |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|                               |                      |               MIG M. |
|===============================+======================+======================|
|   0  NVIDIA GeForce ...  On   | 00000000:04:00.0 Off |                  N/A |
| 22%   15C    P8    33W / 275W |   4774MiB / 12288MiB |      0%      Default |
|                               |                      |                  N/A |
+-------------------------------+----------------------+----------------------+
|   1  NVIDIA GeForce ...  On   | 00000000:05:00.0 Off |                  N/A |
| 22%   16C    P8    32W / 275W |   2979MiB / 12288MiB |      0%      Default |
|                               |                      |                  N/A |
+-------------------------------+----------------------+----------------------+
|   2  NVIDIA GeForce ...  On   | 00000000:08:00.0 Off |                  N/A |
| 22%   14C    P8    32W / 275W |   2979MiB / 12288MiB |      0%      Default |
|                               |                      |                  N/A |
+-------------------------------+----------------------+----------------------+
|   3  NVIDIA GeForce ...  On   | 00000000:09:00.0 Off |                  N/A |
| 22%   13C    P8    32W / 275W |   2979MiB / 12288MiB |      0%      Default |
|                               |                      |                  N/A |
+-------------------------------+----------------------+----------------------+
|   4  NVIDIA GeForce ...  On   | 00000000:85:00.0 Off |                  N/A |
|  0%   19C    P8     7W / 177W |      0MiB /  4096MiB |      0%      Default |
|                               |                      |                  N/A |
+-------------------------------+----------------------+----------------------+

+-----------------------------------------------------------------------------+
| Processes:                                                                  |
|  GPU   GI   CI        PID   Type   Process name                  GPU Memory |
|        ID   ID                                                   Usage      |
|=============================================================================|
+-----------------------------------------------------------------------------+

Looks like another instance of a multi-gpu bug that has been on-going. Linking for future reference: https://github.com/jmorganca/ollama/issues/969

I dropped the version to 0.1.11 and it started working.
On Mon, Dec 4, 2023, 6:02 PM Bruce MacDonald @.***>
wrote:

Looks like another instance of a multi-gpu bug that has been on-going.
Linking for future reference: #969
https://github.com/jmorganca/ollama/issues/969
—
Reply to this email directly, view it on GitHub
https://github.com/jmorganca/ollama/issues/1359#issuecomment-1839681899,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDD3ZPKPFX7IC2EYV7WTUDYHZJCHAVCNFSM6AAAAABAEPOVT2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZZGY4DCOBZHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

We've improved our memory prediction calculations over the past few weeks.  Please give 0.1.22 a try and see if you're still seeing the problem.

If you're still having problems with 0.1.22 or newer, please re-open.


Configurable Keep Alive Extension Added

Check the latest commit first one was just resharpe and codemate

Can you create a new pull request without the Refactor commit?

Sure....


webkitgtk: Use enchant2 instead of enchant

webkitgtk does now seem to support the new version.
old enchant 1.x version is gone from oe-core
Signed-off-by: Khem Raj<EMAIL_ADDRESS>

ah perhaps this is not ok for older webkitgtk ( < 2.26)

I think either you need enchant recipe or perhaps not house older release in master branch anymore since they are now broken.

Building with enchant2 its supported since https://trac.webkit.org/r228826, which means since 2.20.0
I'll merge this after we branch for zeus and look forward to remove old recipes and update webkitgtk ones after that

seems good.


Support for MSI DS200 mice?

Hi,
Do you plan support for MSI mices?

This mouse doesn't appear to have an sdk available, so there isn't much I can do to support it.


bootstrap: More robust handling of a zk.pid file while waiting for exhibitor

High-level description
DCOS_OSS-1919 has revealed that it is possible for Zookeeper to not to create zk.pid even though Exhibitor reports that all the ZK instances have joined the quorum.
This change does not solve the root cause of the problem, it just makes the bootstrap handle it more gracefully and it adds some extra logging so that debugging of the underlying issue will be easier next time.
See the Jira issue for more details. Tests will be made green after I get all the required thumbs up. Same goes for porting these changes to master and 1.9.
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

https://jira.mesosphere.com/browse/DCOS_OSS-1919 Terminated Master node in 3 HA cluster, Failed to start.

Related PRs
EE PR: https://github.com/mesosphere/dcos-enterprise/pull/1888

@mesosphere-mergebot bump-ee

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label ReadyForReview

@mhrabovcin @gpaul Please have a look
@jgehrcke FYI

Pawel, I would appreciate if you could add a comment here to the GitHub conversation describing why precisely this change would help in precisely which scenario. It's a little hard to understand which case specifically this handles more gracefully, and how it's doing it.

I would appreciate if you could add a comment here to the GitHub conversation describing why precisely this change would help in precisely which scenario.

There seems to be a way for the exhibitor to launch ZK in a way that does not result in ZK PID file being created. Despite numerous attempts, I was unable to recreate this situation but removing ZK PID without restarting exhibitor service yielded the very same error log entries as the ones reported by the issue submitter.
After a quick debug of the bootstrap.py code, it turns out that exhibitor.py module used by bootstrap script makes a shortcut while waiting for the local ZK by comparing the ZK PID file's mtime with the one stored internally. The problem is though that the shortcut always assumes that there is a PID file as long as there is a stashed mtime value (see [1] -> [2] -> [3]). This was resulting in FileNotFound exception being thrown as reported by the user.
This PR changes that:

even if the mtime was stashed by the module, it is ignored by it if the ZK PID is missing. In such case, the normal (i.e. long) wait for the ZK quorum is performed and the "shortcut" is terminated early.
after the normal wait for the ZK quorum has been performed, the module does not try to stash the ZK PID's mtime if the PID does not exist.

This, of course, does not solve the underlying issue - i.e. why the ZK did not create the PID or created it in the wrong place. But it will allow the bootstrap script to handle the error more gracefully and in cases when the ZK quorum, as seen by Exhibitor, is healthy, it will allow the bootstrap script to proceed, even though the ZK PID is missing. The extra logline that was added will maybe allow us to debug the root cause at some point in the future.
@jgehrcke Does this help?
[1] https://github.com/dcos/dcos/blob/98d0ac1f34737540a1f9b60fe9d4db2aeffc20db/packages/bootstrap/extra/dcos_internal_utils/exhibitor.py#L28-L33
[2] https://github.com/dcos/dcos/blob/98d0ac1f34737540a1f9b60fe9d4db2aeffc20db/packages/bootstrap/extra/dcos_internal_utils/exhibitor.py#L36
[3] https://github.com/dcos/dcos/blob/98d0ac1f34737540a1f9b60fe9d4db2aeffc20db/packages/bootstrap/extra/dcos_internal_utils/exhibitor.py#L20

Thanks, that describes it really well @vespian.
Can you please add the two bullet points to the commit message? That will make a lovely commit message explaining the gist of this patch in simple, precise terms.

@jgehrcke Done, ready for another pass.

Re-run the teamcity/dcos/test/aws/cloudformation/simple  - seems to be AWS issue - stack was rolled back.

@mesosphere-mergebot override-status  teamcity/dcos/test/aws/cloudformation/simple https://jira.mesosphere.com/browse/DCOS-18825

I commented in DCOS-18825.
Quoting myself:

As of today the CI run teamcity/dcos/test/aws/cloudformation/simple is comprised of more than 100 individual tests.

I do not feel comfortable overriding teamcity/dcos/test/aws/cloudformation/simple or other CI jobs when they failed entirely as of a cluster provisioning error.

I am re-running the job now.

@mesosphere-mergebot bump-ee


installation on premis

Hi,
Is it possible to run DagsHub on premis?

Hey @sephib, thanks for reaching out. It is possible to install DAGsHub on premise. Can you please reach out to me via my email at dean at dagshub dot com to continue the conversation?


Client FPS Drops

Basic Information
Mission version: 0.955
Map used: Malden
Mods used: None
Dedicated or local game? Dedicated
GitHub or Workshop mission file? Github
Individual Things
Did you edit anything within the mission files?
No
If yes, what did you edit exactly?
Short Issue Description
Every 2 hours Client FPS go to 10 server stay stable and high
Expected Behaviour
No Client FPS drops
Steps to Reproduce the Issue
Just playing
Screenshots, Serverlog or any other helpful Information
arma3-server-console.log

Arma3_x64_2017-10-16_01-38-46.txt

Concerning to the logs "no edits to the missionfile" is wrong.
Stats from each client in the serverlog is in this issue helpful, but wasn't part of 0.955.
As you can see in the log, the fps of the server are fine and all AI is also located on the server (clints have only 1 local unit all the time). If there would be a real issue connected to the mission, the server would have "issues", too.
I recommend to use a not edited missionfile with 0.955 or with 0.96, but you're not playing a "not edited" 0.955 version currently.
Personally I can't reproduce this kind of performance issues in Liberation locally or on our weekly sessions, where also nobody experienced this kind of behaviour. But maybe someone else around might have a similar thing to report.
After using a not edited version provide a new serverlog. Or state all edits you've made to your current missionfile.

To investigate FPS issues please provide profiling reports: https://community.bistudio.com/wiki/Performance_Profiling
Just click "copy-all" paste it into a textfile and upload it.
Atleast 2 or 3 files would be good.
Also atleast one diag_captureSlowFrame ['total',0.04];

proffps.txt

and here is a diag where we screwed around for a bit with low fps and all of the sudden i got a fps boost again
proffps2.txt

any news on the reason behind the fps drops?

Remind me again in about 9 hours from now ^^

like this? :P

proffps.txt: That's soundbug. https://feedback.bistudio.com/T124746
friend one is.. nothing I guess everything is a little slow in general.. but nothing special.
proffps2.txt That's an eventhandler in exiting RscDisplayMPInterrupt.
You closed the escape menu and some unscheduled script took 50ms. Looks like a BIS script.

doesnt sound like it at all as the fps drop over time not at one alt tab ?

@HandiworksInteractive what?

Interesting that the soundbug appears here. I only had that on Liberation so far.. Never thought about that.. Could very well be some thing in Liberation causing that.


Update Ingress Resource API version

What this PR does / why we need it:
Ingress resources in extensions/v1beta1 are deprecated and will be
removed in v1.18, it is updated to  networking.k8s.io/v1beta1.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

NONE

/test cloud-provider-openstack-acceptance-test-k8s-cinder

/assign @lingxiankong

/lgtm
@adisky we have the CI issue again?

Build failed.

cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.12  : POST_FAILURE in 15m 52s

@lingxiankong yes this test should no run

cc @mrhillsman @kiwik

@adisky this test is not running on other patches, so may be retest should fix .. eg: https://github.com/kubernetes/cloud-provider-openstack/pull/657 , but we might have diff issue as we are seeing POST_FAILURE again on 1.15 e2e conformance jobs

@ramineni Thanks for the info
/retest

/retest

/approve

/test cloud-provider-openstack-acceptance-test-e2e-conformance

/test cloud-provider-openstack-acceptance-test-e2e-conformance

/test cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.15

/test cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.15

/test cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.15

/lgtm


"editor.minimap.mode": "contain" not behaving as expected with wrapping on

Issue Type: Bug
Testing #90808 by @alexdima
I set "editor.minimap.mode": "contain" and opened a document for which, prior to this new feature landing, the minimap would extend off the bottom of the window. My actual doc was docs\editor\command-line.md from the microsoft/vscode-docs repo.
With "contain" the minimap behaved as though I had set "actual" (or simply not defined the setting at all), in that it scrolled off the bottom. To get the expected behaviour I had to set "cover".
VS Code version: Code - Insiders 1.43.0-insider (c94724193451009962e2c116808f71a3328a9836, 2020-02-19T09:02:21.024Z)
OS version: Windows_NT x64 10.0.18363

@gjsjohnmurray I am really looking forward to ideas on how to name these settings and perhaps how to improve their documentation, but what you have observed is correct:

cover: the minimap is stretched or shrinked (sampling) such that it always has its contents take the height of the editor.
contain: the minimap is never stretched, just shrinked (sampling) such that it never has its contents take more theight than the editor.
actual: the minimap is never stretched, nor shrinked, each line in the minimap gets a height attributed to it, like 2px, and then the line count determines the height of the minimap content.

Please let me know if you have other ideas for the names or for the descriptions, the current settings I took with a lot of inspiration to the CSS background-image properties.

@alexdima I like the names you have chosen, and I was able to understand the descriptions you gave them. But here is a GIF showing the same document, first in  "actual" mode, then in "cover" and finally in "contain". I show the Settings page before each demonstration.

I don't understand why "contain" behaves like "actual" (notice how the minimap scroller isn't the same size as the adjacent scrollbar scroller). I expected "contain" to be the same as "cover".

Thank you, I can reproduce! It reproduces when using wrapping

FYI after #91405, the new setting is minimap.size and the values proportional, fit and fill.


Issue with CSV ingestion with non-default delimiter

The code assumes that the delimiter for CSV in the reader config should also be used when specifying the header string in the ingestion spec. While this is (arguably) a valid assumption, it might be non intuitive for the user since these are two different configs in two different places.
Filing this issue for a better user experience, perhaps:

If the CSV delimiter does not work for header String in job spec, try comma?
At the very least, error out notifying the user that the two are inconsistent.

I can pick this up @mayankshriv

Thanks @suddendust, assigned to you. Let's first figure out the right approach, should we auto try a few separators, or gracefully error out with good information? First approach has the danger of picking the wrong delimiter. And the second one requires user to make a string with delimiters (that could be control characters) to specify in a text file.
Let's solicit some inputs here, and then go for an implementation.

@mayankshriv thanks for assigning the story. So just to clarify, the record reader config takes in the delimiter for CSV as defined here. In the ingestion spec, what header are you referring to exactly?

@suddendust: In the ingestion job spec yaml file, we provide delimiter, header as two configs. It is implicitly assumed that header strings will also be separated by delimiter.

First approach has the danger of picking the wrong delimiter.

Yes, and the bad thing about this is that this can silently keep working (incorrectly) in the background. The max we can do in this approach is warn the user.

@mayankshriv regarding

And the second one requires user to make a string with delimiters (that could be control characters) to specify in a text file

Is this not the case even now? Say the header has a a control character as the delimiter, he will face the same problem even now right?
Also, how about accepting a different config for the header delimiter? This can be optional, in which case we can fallback to using the currently supplied delimiter, and throw an exception if it doesn't work. I think this is better than the application assuming the delimiters by itself and giving the user a surprise at some later point.

@mayankshriv let me know what you think of this.

I think the code and user point-of-view are contradicting here. It is very natural for users to provide a comma separated header string. May be we should gracefully error out right now if we don't find the delimiter in the header?

That will be a good starting step for sure. However, can we also have an optional config that the user can set explicitly if the header delimiter is different than the CSV delimiter? If this is absent, we'll just keep using the existing config, and will error out if they're incompatible. It will be absent most of the time, because as you said, it's for users to provide a comma separated header string. But in cases where it isn't, this config will come in handy @mayankshriv.

@mayankshriv I think gracefully erroring out if they don't match is the least we can do here. Shall we start with that?

@mayankshriv Have raised a preliminary PR here


What is deep.transcript_engraved_frontispiece?

I'm migrating the fields with formatting from the old database. There is a field called transcript_engraved_frontispiece in the data, but it does not seem to appear on the site. What is it?
['1182   This edition is not in Greg.',
'898   Engraved portrait inscribed "Vera effigies. Abrahami Cowley. regii alumni scholæ West:" and "Ætatis Suæ 13" [A1v]',
'953   [A1v] Engraved portrait inscribed "Vera, ac viva Effigies Nathanaelis Richards Gen:" with motto and coat of arms, signed signed "TR sculp:" [A1v]',
'1005   Engraved portrait inscribed "Bapt: Guarini the author of Pastor fido æt: suæ LXIII" with arms below and couplet in English, signed "T: Cross sculpt:" [A1v]',
'1006   Engraved portrait inscribed "Bapt: Guarini the author of Pastor fido æt: suæ LXIII" with arms below and couplet in English, signed "T: Cross sculpt:" [A1v]',
'1035   Engraved portrait of Prince Charles (later King Charles II) with  view inscribed "Iersey" (where Charles was in 1649) and below “Ætatis suæ.19." [π1v]. Engraved portrait of  Princess Elizabeth (second daughter of King Charles I) with four lines of verse below [π2r] ',
'1044   Engraved frontispiece inscribed with verses [π1v] (a folding leaf)',
'1052   Engraved portrait (with six lines of verse below subscribed "Leo: Willan ipse Scripsit") signed "T. Cross sculpsit" [A1v] ',
'1064   Engraved portrait inscribed "Effigies Hugon. Grotij"  and signed "Cross fecit" [π1v]. Engraved portrait of translator subscribed with four lines of verse and signed "T. Cross fecit" [B3+1v].',
'5167   Engraved frontispiece signed "p stent exc:" [A1v]',
'5168   Engraved frontispiece signed "p stent exc:" [A1v]',
'1176   Engraved portrait (unsigned) surmounted by a coat of arms with the motto "Amico rosa inimico spina" [A1v] ',
'1193   Engraved portrait with "Ignoramus" on scroll held by figure [A1v].',
'1195   Engraved portrait with "Ignoramus" on scroll held by figure [A1v].',
'1196   Engraved portrait (rougher copy) with "Ignoramus" on scroll held by figure [*1v].',
'1197   Engraved portrait (rougher copy) with "Ignoramus" on scroll held by figure [*1v].',
'1198   Engravings of "Dromodotus" [A1v] and of "Pedantius" [A2r].',
'5062   Engraved portrait inscribed "S W.A. Aut. Spero Aut. Sperno. 1616." [A1+1v]',
'5106   Engraved portrait inscribed "Vera effigies Gulielmi Comitis de Sterlin. Ætats suæ LVII." and signed "Wilhel: Marshall. sculpsit" [A1+1v]',
'5076   Engraved portrait inscribed "Vera effigies doctissimi poetarum Anglorum Ben: Iohnsonii" and signed "Ro: Vaughan fecit" with verses in Latin and English signed "Ab: Holl." [A1v]',
'5133   Engraved portrait inscribed "Ioannis Miltoni Angli Effigies Anno Ætatis ViGes: Pri:" with verses below and signed "W. M. sculp:" [a1+1v]',
'5134   Engraved portrait with verses below and signed "W. Marshall sculpsit. 1646" [A1+1r]',
'5135   Engraved portrait inscribed "Obijt anno Ætatis suæ 28." with verses (unsigned) on "Sucklin" below and signed "W. Marshall fecit" [πA1+1v]',
'5136   Engraved portrait inscribed "Obijt anno Ætatis suæ 28." with verses (unsigned) on "Sucklin" below and signed "W. Marshall fecit" [πA1+1v]',
'5137   Engraved portrait inscribed "Obijt anno Ætatis suæ 28." with verses (unsigned) on "Sucklin" below and signed "W. Marshall fecit" [πA1+1v]',
'5141   Engraved portrait inscribed "Ætat: Suæ 17" and signed "W. Marshall sculpsit" with below a Latin couplet signed "John Hobart Gent:" [π1v]',
'5143   Engraved portrait, with coat of arms above and Latin verses signed "J. Berkenhead" below, inscribed "Poetarum ingeniosissimus Ioannes Fletcherus Anglus, Episcopi Lond: Fili Obijt 1625 Ætat: 49." and signed "Guliel Marshall Fecit" [π1v].',
'5159   Engraved portrait inscribed "Thomas Stanley Arm:" and signed "P: Lilly pinxit. Guil: Faithorne scul" [1+π1v]',
'5160   Engraved portrait inscribed "Thomas Stanley Arm:" and signed "P: Lilly pinxit. Guil: Faithorne scul" [1+π1v]',
'5138   Engraved portrait inscribed "Obijt anno Ætatis suæ 28." with verses (unsigned) on "Sucklin" below and signed "W. Marshall fecit" [πA1v]',
'5142   Engraved portrait inscribed "Ætat: Suæ 17" and signed "W. Marshall sculpsit" with below a Latin couplet signed "John Hobart Gent:" [π1v]',
'5146   Engraved portrait signed "P Lombart, sculp. A. london," inscribed "Obijt Anno 1643" and with verses below [π1v].',
'5153   Engraved portrait, with verses signed "A. B." below, signed "T. Cross Sculpsit" [π1v].',
'5155   Engraved portrait with verses below and signed "W. Marshall sculpsit. 1646" [π1+1v]',
'5165   Engraved portrait inscribed "Vera ac Viva Effigies Philippi Massinger. Geñ" and signed "T. Cross fecit" [π1v]',
'5154   Engraved portrait, with verses signed "A. B." below, signed "T. Cross Sculpsit" [π1v].',
'5171   Engraved portrait inscribed "Vera Effigies Tho: Midletoni Gent:" (unsigned) [π1v]',
'5173   Engraved portrait (unsigned) with verses on "Cockains Head" (unsigned) below [A1+1v].',
'5174   Engraved portrait (unsigned) with verses on "Cockains Head" (unsigned) below [A1+1v].',
'5180   Engraved portrait inscribed "Iacobus Shirlæus:" subscribed "G Phenik pinx:" "R. Gaywood fecit 1658" [A1+1v]',
'5139   Engraved portrait inscribed "Obijt anno Ætatis suæ 28." with verses (unsigned) on "Sucklin" below and signed "W. Marshall fecit" [π1v]',
'5175   Engraved portrait (unsigned) with verses on "Cockains Head" (unsigned) below [A1+1v].',
'5181   Engraved portrait inscribed "Iacobus Shirlæus:" subscribed "G Phenik pinx:" "R. Gaywood fecit 1658" [A1+1v]',
'1194   Engraved portrait with "Ignoramus" on scroll held by figure [A1v].',
'5156   Engraved portrait with verses below and signed "W. Marshall sculpsit. 1646" [π1+1v]',
'5140   Engraved portrait inscribed "Obijt anno Ætatis suæ 28." with verses (unsigned) on "Sucklin" below and signed "W. Marshall fecit" [π1v]']

This data should be preserved -- it was a field that we never implemented, I think, but that ultimately probably belongs as one of the Paratexts in filter searching and in display.


Refactor image download logic to group images by artist name. fixes #55

Files are being renamed while downloading.

I didn't think about completely rebuilding the download logic at first. That is why there are three commits, the first two of which are actually unnecessary.


NormalizeEntityEvent::setResultField signature

The method accepts only an array as second argument. I tried to use this event to change a string field after normalization , but with the current method signature this is not possible.
Is there a reason , why the method accepts only an array as an object´s field value ?

Thank you for the solution !


Fix stack_rasters now that is_loaded is a property

https://github.com/GlacioHack/geoutils/blob/1016312c07268e49ba3cf0d45fc21286b81b9e93/geoutils/raster/multiraster.py#L174
With is_loaded that cannot be set, this code fails and needs to be fixed

Also, it looks like the multiraster.py modules is raising a lot of warnings that should be silenced.


Voice synthesis by model is not the same as the voice with speaker ID

I trained model on my Vietnamese dataset ( 46 speakers ). But when inference my output voice is not the same as the voice with speaker ID. Can you explain more detail to solve this problem. Thank you!


[Feature Request] Use median GPA instead of Mean

Overview
The median GPA is closer to the bulk of the data, and is less affected by skew. This also might mean most of the letter grades will not be a B -> easier to tell apart. Median also (from preliminary research) seems to represent the shape of the graph better.
Proposal
Consider using the median GPA instead of the mean/average in all places (Letter grade, distribution graphs, compare table, new line graph)
Supplementary Information
Open to debate, but do some mockups first

I want to first just use it on LHS so we can compare how the avg letter grade maps to the graph vs median letter grade


feat: disable user db creation by default

Still able to enable it with an option.

Ah I forgot the why.
In my dev env, the app is expected to create the database. If it already exists, it skips initialization and thus fails. Also if there's a separate database, it fails some tests because they expect a single database to be found.
This should help these cases.

thanks!


A bug in tooltip generation

In your code, you have
var indexRight = idx == stack.length ? idx - 1 : idx;
var valueRight = x(values(stack)[indexRight]);

but stack cannot be guaranteed to be an array and you should use values to find the array. Like this:
var indexRight = idx == values(stack).length ? idx - 1 : idx;
var valueRight = x(values(stack)[indexRight]);

Otherwise, when user moves mouse outside the xdomain to the right, it will generate an error due to index out of bound.

Oh yeah, nice catch! Stack isn't even supposed to be an array so I don't know what I was thinking there!

Wasn't thinking clear when I changed from the old tooltip code to the new one :)
Fixed now and will push soon.


Issue265 fixed

This PR fix the issue #265
To test it please use the code
[X,Y] = meshgrid(-3:6/17:3);
XX = 2*X.*Y;
YY = X.^2 - Y.^2;
colorscale = [1:18; 18:-1:1];
C = repmat(colorscale,9,1);
pcolor(XX,YY,C);

fig2plotly(gcf, 'offline', 1, 'TreatAs', 'pcolor');

As you can see it is necessary to set 'TreatAs' Plotly Option to 'pcolor'
results looks as follow

Looks good, regardless that default axis titles "x" and "y", but this is the subject of other issue


one improvement and one enhancement

using IterBuffered instead of Iter can get better performence
add Keys() function with test cases and remove TODO

@sillydong Thanks Chen, I am testing it right now, we will review it and update.

LGTM


MeiliSearch removes indexes when data.ms is not compatible

Steps to reproduce

Start MeiliSearch v0.22.0
Add documents
Stop MeiliSearch v0.22.0 and run MeiliSearch v0.23.0 using the same data.ms

-> MeiliSearch starts without any error, but should not, and remove the indexes in data.ms
MeiliSearch should display the following error
Cannot open database, expected MeiliSearch engine version: X.X.X, current engine version Y.Y.Y

at it was before, and should not start if data.ms is not compatible without removing the indexes

Wanted to share this experience (which might be related):
Our environment

Running Meilisearch on Ubuntu 20.04 with Docker using the "latest" image tag (getmeili/meilisearch)
Our data.ms database is mounted via Docker Volumes

What happened

We originally built indexes on v0.23.1
Everything worked great
Our Meilisearch instance automatically updated to v0.24.0
All of our indexes disappeared

How we were able to get our indexes back

We forced our image to go back to v0.23.1

I am also going to cc @danpastori on this issue since I am the sysadmin and he is the developer who helped me troubleshoot this 😁
Any additional information on this, please let me know!


Multi-tenancy

I'm doing a free service to a couple of churches that use the same phone number. I think it would be nice to have a message in the form of join <group>, which might require approval.
A hack would be to auto treat groups as join keywords.
Another way would be to support multiple phone numbers are assign them to organizations. The thing is that it is possible (in the case of Orthodox churches, at least) that one person goes to more than one church.

Maybe to avoid redundant moderation, groups can be open or closed, and then everybody can join a group unless it's not public, which could then be added to only via the admin UI.

Not quite sure I understand. You seem to be talking about 2 different options?
Option 1
You want to let people join a group by SMS? And then you want to have each group be responsible for a different church?
Though each church would still be sharing the keywords with this setup.
Feel free to send a PR if you want to implement this feature.
Option 2
You want to receive and send messages from multiple phone numbers? But share contacts across these phone numbers? I think this is probably beyond the scope of the project and would make apostello significantly more complicated...
Am I understanding you correctly?
If it were me, I would probably just setup multiple instances of apostello...

In option 1, if there's filtering by groups, then sharing keywords is not really a problem. For example, there can be a keyword for helping with Easter lunch, then it will be the same across churches anyway, but you bring a valid point as then it won't be clear to which lunch they are responding to.
Separate instances is definitely fine, but increases the costs - hosting, dedicated numbers, etc.
Anyway, I will think more about refining the proposal.

You could run two instances on one server to keep hosting costs down. You would need to run both of them and then reverse proxy them. But you would still have the increased cost from multiple numbers...

I'm going to close this for now. Feel free to re-open it or another issue in the future.


No PRs or Isuues are shown after update to 1.2.0

Quests 1.2.0 not working on OSx big sur 11.0.1. It does no get the issues and PRs can you help me?
there is only spinner but the list never appears/
2021-01-05 12:02:14.052 [Quests] > Quests Version: 1.2.0 Build:<PHONE_NUMBER>00 PID: 34633
2021-01-05 12:02:14.052 [Quests] > XCGLogger Version: 6.1.0 - Level: Debug
2021-01-05 12:02:14.052 [Quests] > XCGLogger writing log to: file:///Users/kristian/Library/Containers/com.steamclock.quests/Data/Documents/quests_debug.txt

it just stays like this forever:

Anybody with a similar problem?

it just stays like this forever:

Anybody with a similar problem?

fixed in 1.2.1

fixed in 1.2.1


Add @material/cdk to plunker examples

Bug:
Hey,
It would be good to add the '@angular/cdk' dependency to the plunkers.
Thanks! :)

Already fixed. Be sure to clear your browser cache.


Are cryptocurrencies supported by this library?

Hello,
I was wondering if criptocurrencies (DASH, ETH, VTC,  XBC, XBT,BTC,XLM,XMR,XRP,ZEC as defined by standard ISO 4217) was supported, i.e:
$example = ( new Builder() )->add('fixer')->build();
$rate = $example->latest( 'BTC/EUR');
Actually it seems not working (btw they are actually showed in Yahoo Finance). Where could I find list of currencies OK with Swap?

@salvatorecapolupo You can use Cryptonator
$example = ( new Builder() )->add('cryptonator')->build();

Btw, using:
	$swap = (new Builder())
    ->add('fixer')
    ->add('cryptonator')
    ->build();

you can support traditional and crypto together. Thanks very much!

Closing this ;)

I contacted original API developers fixer.io, and they say you maybe using legacy API of fixer, which does not support BTC.  Hope this could help improving next versions.


Research Fix for Issues with the Mocking Framework

User story

As a Team
We want a reliably working test pipeline
So that that we do not get false positives from the pipeline

Acceptance criteria

Research what the issue is and how to handle it

Definition of done (DoD)

Feature is done when the acceptance criteria are fullfiled
Feature is done when there are test where it is necessary and they pass
Feature is done when the linter passes
Feature is done when it is merged onto our main branch (after the code has been reviewed by at least one other person)
Feature is done when the code has been documentend or commented properly

DoD general criteria

Feature has been fully implemented
Feature has been merged into the mainline
All acceptance criteria were met
Product owner approved features
All tests are passing
Developers agreed to release

Comment the code where the test may be breaking


Infinite range gives InexactError: trunc(Int64, Inf)

Hi,
Trying out one of the examples to create a Hamiltonian on an infinite chain:
using MPSKit, MPSKitModels, TensorKit
H = @mpoham sum(σᶻᶻ{i, i+1} + h * σˣ{i} for i in -Inf:Inf)

yields ERROR: InexactError: trunc(Int64, Inf). Tried replacing Inf with ∞ but it is not recognized.
Am I missing a certain package or something?
I have Julia 1.10

Hi,
My apologies for this inconvenience. The macro is supposed to handle these kinds of syntactic sugar features (in this case just replacing the range with InfiniteChain(L) where L is the unit cell) but at some point it became quite inconvenient to keep track of what are intuitive things that should be supported, and what is not. My own opinion on the matter is that it has taken too much development time for very little benefit, so I have been neglecting the macro a bit (again my apologies for that).
In this case, you can just simply replace the iterator with some type of lattice, which should then work.
I will make sure to update the examples to reflect this.
As a side note, do you feel like this kind of syntax adds a lot of value? If so, I will definitely try and fix it and support this again.
(Next week, as I am currently on holiday)

Thanks for the quick response. I think I managed to construct the Hamiltonian by copying parts of the source code for some of the predefined models. The syntax there is however somewhat cumbersome.
I am not sure what you mean by

simply replace the iterator with some type of lattice
An example would help:)
I tried
H = @mpoham sum(σᶻᶻ{i, i+1} + h * σˣ{i} for i in InfiniteChain(1)) but that didnt work.

I think you need to add an explicit iteration over the vertices for it to know what to do. This should work I think:
H = @mpoham sum(σᶻᶻ(){i, i+1} + h * σˣ(){i} for i in vertices(InfiniteChain(1)))


Add click to sync option into demo app

User story: As a logged in user I can choose to sync the mobile client with a FHIR server and visually see the status changes
Steps:

User is logged in
Clicks on the settings which opens a drawer menu
From the drawer menu, user can see sync option and last sync'd time/date
User clicks the sync option which starts the sync process. The status changes visually by showing a spinner (or other) and continues in the background

End state:

On completion, shows a status change (complete) and summary of # records syncd
On error, will attempt to retry until it either completes or fails. Status should reflect this

@fredhersch @jingtang10 Since there is no way as of now to know the state of Sync from reference app, I am adding just a sync button on the Top bar. Clicking the button starts a Sync job and shows a Snackbar with sync message at the bottom of the screen and disappears after sometime . We can implement the Drawer with the UI explained in the ticket once we have an API to know the sync state.


flutter: Cannot copy hive dile to storage by file.copy

I am trying to copy hive box to my mobile storage. After acception sufficient permissions, I run my method:
  Future<void> exportVocabsDb() async {
    final dir = await getExternalStorageDirectory();
    final evbDirectory =
        await Directory('${dir!.path}/evb').create(recursive: true);
    hiveService.backupHiveBox<VocabularyModel>(
        VOCABULARY_BOX, evbDirectory.path);
  }

and :
  Future<void> backupHiveBox<T>(String boxName, String backupPath) async {
    if (!Hive.isBoxOpen(boxName)) {}
    final box = await Hive.openBox<T>(boxName);
    final boxPath = box.path;
    await box.close();

        try {
  File(boxPath!).copy(backupPath);
  } finally {
  await Hive.openBox<T>(boxName);
  }
  }

But I got this error:
    E/flutter (14955): [ERROR:flutter/lib/ui/ui_dart_state.cc(199)] Unhandled Exception: FileSystemException: Cannot copy file to '/storage/emulated/0/Android/data/com.example.english_vocabulary_builder/files/evb', path = '/data/user/0/com.example.english_vocabulary_builder/app_flutter/vocabularybox.hive' (OS Error: Is a directory, errno = 21)
There is hive file inside app_flutter :
https://pasteboard.co/KdI1Juxp.png
this is doctor:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.2.3, on Linux, locale en_US.UTF-8)
[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/linux#android-setup for more details.
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.2)
[✓] Android Studio
[✓] VS Code (version 1.58.2)
[✓] Connected device (2 available)

! Doctor found issues in 1 category.

Hey @tazik561, looks like you are trying to copy a file object and copying it into a folder object.
Trying correcting
    final evbDirectory = await Directory('${dir!.path}/evb').create(recursive: true);

to
    final evbDirectory = await File('${dir!.path}/evb').create(recursive: true);


Update theme_barbie so that facet titles follow the theme

I raised this as an issue here https://github.com/MatthewBJane/theme_park/issues/2
And these 2 lines fix the defaults to follow the theme. If you like it, I suggest adding to the "template" for new theme also

Excellent!


client keeps on opening on startup even after removing shortcut from startup

This is undesired behavior.  I am happy for the keybase client to run in the system tray however I don't like having a new "facebook here's your friends list" window open every time I start up my computer.
Keybase GUI Version: 1.0.47-20180403200126+d3b2e30

There is a GUI control, under settings -> advanced. It toggles an entry under the Run registry key.
The startup shortcut is for the background processes - it can be removed, in which case the GUI will start them as needed, it just takes it longer the first time and filesystem access won't work.

Afraid that is not the case for me:

It's in the settings "tab", not the native menus. But you raise a good point - why shouldn't it be in the native menus too?

because I had to manually pull the window down, huzzah!

How about either:  a)  force resizing the main application window just once upon client upgrade, or b) make the side menu scroll-able.  I never knew this new portion existed.  I feel like Fry when he discovered that Bender had a closet at the Robot Arms Apartments.  https://cs5.pikabu.ru/images/big_size_comm/2015-12_3/144978302417769378.jpg

that's a bug I never noticed, thanks, will ticket it, thanks.

Should work on all platforms now in Keybase 5.0. Make a new ticket if it doesn't.


Simplify layer creation with a "ctrl+click"

It would greatly simplify new layer design if it would be possible to add a new layer by simply clicking (e.g. with a modifier), or having a special button on the popup in the inspection mode. Also, it might make sense to show a certain icon/color indicator if the given layer is already added to the layer stack

What if there are multiple features under the cursor?
Maybe it should work more like the chrome developer tools in inspection mode

Click to enable inspection
Hover to list features in an inspection pane
Click to freeze the current selection in the inspection pane

Then from the inspection pane we can add an option to create a layer from a feature.
NOTE: This is from some new designs I've started to think about for the UI, in short I'd like to remove as many of the modals and tooltips as possible and put them in sidebar panels instead.

Closing due to inactivity.


Add Repository Mirror does not accept the gitlab repo

I would want to use gitlab as a mirror repo...however when i add the path to repo...example ssh://git@gitlab.com:myuser/test-mirror.git...Redmine report invalid url. Why we can not use gitlab as mirror?

Hi! Try with ssh://git@gitlab.com:22/myuser/test-mirror.git

Thanks, it works correctly :)


I refactored the late MapViewModel:

- Renamed the  `MapViewModel` to `ItineraryViewModel`
- Created a specific View Model for itinerary creation called `CreateItineraryViewModel`

@YofiY have you done some tests for the new view model you added ? If not, it would be great to add them before merging to main


How to omit column in preload?

Your Question

I want to full load this invoice but I need to omit some column in preload that I want to hide to a paticular user.

The document you expected this should be explained

type TableInvoiceDetail struct {
	ID            string          `gorm:"primary_key;type:uuid;index"`
	Parent        string          `gorm:"type:varchar(36);index"`
	TProduct      TableProduct    `gorm:"foreignKey:Product"`
	Product       string          `gorm:"type:varchar(36);index"`
	Quantity      int64           `gorm:"type:INT"`
	Price         decimal.Decimal `gorm:"type:numeric(19,4)"`
	Total         decimal.Decimal `gorm:"type:numeric(19,4)"`
}

type TableInvoice struct {
	ID           string          `gorm:"primary_key;type:varchar(36);index"`
	Date         time.Time       `gorm:"type:date;index"`
	TCustomer    TableCustomer   `gorm:"foreignKey:Customer"`
	Customer     string          `gorm:"type:varchar(36);index"`

	Details     []TableInvoiceDetail  `gorm:"foreignkey:Parent;constraint:OnUpdate:CASCADE,OnDelete:CASCADE;"`
}

func View(table any) error {
        return db.DB.Preload(clause.Associations).Omit(`Customer`, `Details.Product`).Find(table, `table_invoices.id=?`, `I2200001`).Error
}

Actual Result
{
    "ID": "IN2200001",
    "Date": "2022-07-28T00:00:00Z",
    "Customer": "",
    "Details": [
        {
            "ID": "c8f65f15-017e-4fa4-9c44-282893da1a7e",
            "Parent": "IN2200001",
            "Product": "P00001",
            "Quantity": 1,
            "Price": "20",
            "Total": "20"
        }
    ]
}

Expected answer

{
    "ID": "IN2200001",
    "Date": "2022-07-28T00:00:00Z",
    "Customer": "",   // omit Customer here is correct.
    "Details": [
        {
            "ID": "c8f65f15-017e-4fa4-9c44-282893da1a7e",
            "Parent": "IN2200001",
            "Product": "",  // <<--- This one here I want to omit it too.
            "Quantity": 1,
            "Price": "20",
            "Total": "20"
        }
    ]
}

	DB.Preload(..., func(tx *gorm.DB) *gorm.DB {
		return tx.Omit("Product")
	}).First(...)

@a631807682
Thank you for your help.
I've just tried with your solution but it is not working. my gorm version is gorm v1.23.8

This is a complete example, if it still doesn't work, please create a PR to the playground to describe your problem
func TestGORM(t *testing.T) {
	user := User{Name: "jinzhu", Age: 100, Company: Company{Name: "comp"}}

	DB.Create(&user)

	var result User
	if err := DB.Omit("Age").Preload("Company", func(tx *gorm.DB) *gorm.DB {
		return tx.Omit("Name")
	}).First(&result, user.ID).Error; err != nil {
		t.Errorf("Failed, got error: %v", err)
	}

	if result.Age != 0 ||
		result.Company.ID == 0 ||
		result.Company.Name != "" {
		t.Errorf("preload omit not work")
	}
}

@a631807682
Thank you so much I know my mistake now.
the problem is I wrote DB.Find() two times. The fisrt DB.Find with Omit is correct. the problem is I wrote a DB.Find again in AfterFind(tx *gorm.DB). that why my json data get the second result.

@a631807682 Hi, possible to omit ID column in preload?

add to your model
type Tag struct {
	ID		int		`json:"-"`
}

The json:"-" on field ID struct on main model will make the ID wouldn't returned when we do query to model Foo directly.
My case is to omit the ID column when we call the Foo model on Preload, but if use this approach https://github.com/go-gorm/gorm/issues/5551#issuecomment-1200663283, the query will throw error because Preload need to load foreign key column.
I wonder if there's a way to load Preload with foreign key field but not showing it's value on the returned result.


Fix replacement in hide-errors

Thanks! Very helpful.
Looks like this comes from a change where Desmos won't automatically add sliders anymore if you press "enter" with more than four slider variable suggestions.

Thanks! Very helpful.
Looks like this comes from a change where Desmos won't automatically add sliders anymore if you press "enter" with more than four slider variable suggestions.

Presumably to avoid spamming variables when you mistype something like "folder"


Lets-Plot: notes for pyramid

In section Common Plots / Pyramid there is a few issues with the plot:

Clipped labels: unfortunately, the 20 character limit is hardcoded, so y labels are cut off. But the full text can be seen in the axial tooltip.

Weird-looking tooltips on top of the pyramid: to improve tooltips displaying I suggest not to use identity statistic; you can calculate and add weight for users as shown below:
g = (
    ggplot(df, aes(x="Stage", y="Users", fill="Gender", weight='Users'))
    + geom_bar(width=0.8)  # baseplot
    + coord_flip()  # flip coordinates
    + theme_minimal()
    + ylab("Users (millions)")
)
g

It's already been replaced in PR #43, if you prefer that way of updating code.

closed in #43


Server crashes when channel id's and bot token is filled in

2021-02-14-4.log.gz
this is the log files with the crash. Before filling in the token and channel id's in fdlink.json server was running fine.

Fixed in a discussion on discord


10730 or 1024

https://github.com/adinhodovic/opencost-mixin/blob/1834a0578613f699170c6df4e0e833431eff40aa/dashboards_out/opencost-mixin-overview.json#L183
Hi,
I think, it should be 1024 / 1024 / 1024.
Best

Hi.
10730 does not have sense here.
Other charts are using 1024.
Best


0.9.28 and 0.9.29

0.9.28: switch from rlnorm to rnorm for K~Kpred in linear, binned models
0.9.29: functions in the specs of sim models can now refer to their own metabolism model (and therefore  also its info or data_daily slots)

my computer: tests & checks without error.
appveyor:

* checking tests ...
  Running 'testthat.R'
Warning message:
running command '"c:/R/bin/i386/R" CMD BATCH --vanilla  "testthat.R" "testthat.Rout"' had status 1
 ERROR
Running the tests in 'tests/testthat.R' failed.
Last 13 lines of output:
  2: get_params.metab_bayes(mm, uncertainty = "none", messages = FALSE)
  3: NextMethod()
  4: get_params.metab_model(mm, uncertainty = "none", messages = FALSE)
  5: stop(paste0("can't find metabolism parameter", if (length(missing.metabs) > 1) "s",
         " ", paste0(missing.metabs, collapse = ", ")))

  testthat results ================================================================
  OK: 384 SKIPPED: 2 FAILED: 2
  1. Error: French Creek predictions are similar for streamMetabolizer & Bob Hall's code (@test-french.R#94)
  2. Error: get_params works for each model type, basic GPP & ER equations (@test-get_params.R#78)

  Error: testthat unit tests failed
  Execution halted
* DONE

appveyor gives
found:  date, GPP.daily.lower, GPP.daily_50pct, GPP.daily.upper, ER.daily.lower, ER.daily_50pct, ER.daily.upper, K600.daily.lower, K600.daily_50pct, K600.daily.upper, valid_day, warnings, errors

whereas my computer gives
1. Error: French Creek predictions are similar for streamMetabolizer & Bob Hall's code (@test-french.R#90)
found:  date, GPP, GPP_se, GPP.sd, GPP.lower, GPP_25pct, GPP_50pct, GPP_75pct, GPP.upper, GPP_n_eff, GPP_Rhat, ER, ER_se, ER.sd, ER.lower, ER_25pct, ER_50pct, ER_75pct, ER.upper, ER_n_eff, ER_Rhat, GPP.daily, GPP.daily_se, GPP.daily.sd, GPP.daily.lower, GPP.daily_25pct, GPP.daily_50pct, GPP.daily_75pct, GPP.daily.upper, GPP.daily_n_eff, GPP.daily_Rhat, ER.daily, ER.daily_se, ER.daily.sd, ER.daily.lower, ER.daily_25pct, ER.daily_50pct, ER.daily_75pct, ER.daily.upper, ER.daily_n_eff, ER.daily_Rhat, K600.daily, K600.daily_se, K600.daily.sd, K600.daily.lower, K600.daily_25pct, K600.daily_50pct, K600.daily_75pct, K600.daily.upper, K600.daily_n_eff, K600.daily_Rhat, err.obs.iid.sigma, err.obs.iid.sigma_se, err.obs.iid.sigma.sd, err.obs.iid.sigma.lower, err.obs.iid.sigma_25pct, err.obs.iid.sigma_50pct, err.obs.iid.sigma_75pct, err.obs.iid.sigma.upper, err.obs.iid.sigma_n_eff, err.obs.iid.sigma_Rhat, lp__, lp___se, lp__.sd, lp__.lower, lp___25pct, lp___50pct, lp___75pct, lp__.upper, lp___n_eff... <truncated>

ah hah! so the problem is with the Stan models, not with get_params:
1. French Creek predictions are similar for streamMetabolizer & Bob Hall's code (@test-french.R#90) - Modeling failed: cannot open the connection

5. get_params works for each model type, basic GPP & ER equations (@test-get_params.R#77) - Modeling failed: cannot open the connection

see https://github.com/USGS-CIDA/metab_tests/issues/21#issuecomment-269563362. i'm hopeful that upcoming rstan updates will fix this.

local computer:
R CMD check results
0 errors | 0 warnings | 2 notes

R CMD check succeeded

calling it good for next round of sims


Laptop slows down after 5 minutes of searches

Hi all, can somebody help me please? OK so after a while searching my laptop goes slow and the web app kinda freezes up. Once I click clear go off and back on chrome it works fine again.
My question is how can I make it so this doesn't happen? So I can just leave it for a lot longer?I have a razer blade pro 17 early 2020 so it's not problems on the hardware side lol. Is there any settings I can change in chrome or whatever to sort this? Already changed my chrome cache size to 2gb that should be more than enough.
Please help! Its bugging the life out of me.
Thanks in advance

Download the latest version and toggle auto log clear option, it will periodically clear the logs and you can prevent from web app getting froze


Spotify no rounded borders

As for the title, the spotify app (AUR package spotify) does not have the rounder borders.

Weird, works for me. Could you show a video demonstrating this?

The "outside" rounded corners are set by the compositor, but as you can see while xborders is adding the inside rouded corners to the terminal, it is not doing the same to spotify (nor for the terminal when it is not focused, but I can live with that)

Any update?

@deter0 any update?

With the merge of #42 does this possibly fix your issue?

Yes, it is fixed now 😄 thank you 😄


call and apply

At this point I don't see the necessity of having call/apply available.
The common case (my common case) can be solved with:
(_1, _2, ) => asyncZip(map(something, _1), filter(somethingElse, _2))
In case they are really needed they can be imported from ramda:
https://ramdajs.com/docs/#apply
I checked, apply can be curried:
https://ramdajs.com/repl/?v=0.26.1#?const nums %3D [1%2C 2%2C 3%2C -99%2C 42%2C 6%2C 7]%3B
R.apply(Math.max)(nums)%3B %2F%2F%3D> 42

@sithmel I was pretty sure that we just demonstrated powerful use cases for both call and apply as part of data pipelining. Also I don't think Ramda's apply accepts an iterable of arguments, and many people will want a dependency on either iter-tools or ramda, but likely not both. There's way too much overlap.

Integers aren't allocated unless I'm much mistaken.
On Sat, Mar 2, 2019, 10:52 Khải<EMAIL_ADDRESS>wrote:

@KSXGitHub commented on this pull request.
In src/apply.mjs
https://github.com/sithmel/iter-tools/pull/205#discussion_r261837591:

@@ -0,0 +1,9 @@
+import { curry } from './internal/curry'

+const emptyArray = []
+
+function apply (fn, args = emptyArray) {
I don't think creating an empty array is any more expensive than mere
integer allocation.
@sithmel https://github.com/sithmel Your opinion?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sithmel/iter-tools/pull/205#discussion_r261837591,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAhAaRWoJEE8yGj2vysmHkuVxU86uONZks5vSshdgaJpZM4bahBX
.

@KSXGitHub your observations are correct, and thanks for adding links and explanations backing your claims.
The thing is that different styles can be both correct, depending on what kind of trade offs we are doing.

Performance against readability
terseness against easy to debug
etc.

These discussions are very instructive but I am sure @conartist6 would like more useful feedbacks.
@conartist6 please keep the discussion constructive as usual

I was pretty sure that we just demonstrated powerful use cases for both call and apply as part of data pipelining. Also I don't think Ramda's apply accepts an iterable of arguments

Yes, that's true. I was wrong about suggesting Ramda.
There are 2 points that makes me lean to not include this in iter-tools.
1.
If I have to chose between:
apply(zip)

and
([_1, _2]) => zip(_1, _2)

I would go for the arrow function. Apply used in this way would surprise the reader. But I admit, this is purely an opinion and I won't judge harshly the dev using apply instead.
With this in mind, given that you are already using an arrow function, I would use it only once and do:
([_1, _2]) => asyncZip(map(something, _1), filter(somethingElse, _2))

2
Using an arrow function (instead of apply) is shorter and more clear
This is the biggest objection I have.

@sithmel You can still do that in your code and suggest it in the pipelining docs, but apply still seems broadly useful. It's still far easier to read and write apply(concat, iterables) than
iterables => concat(...iterables)(iterables)

I am ok fot the inclusion then. Let me review the code


Re-sync-up.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

Leave a comment


Cordova: Support for Handled errors

Reporting of Handled errors is currently supported for Xamarin only.
Would be nice to have this for Cordova (and other platforms also).
Nothing special is needed, just same functionality as exists for Xamarin

Any updates on this?
It would be great to have at least a statement about the status of this issue or what makes it so difficult to add support for other platforms than Xamarin.
I'm especially interested in tracking handled (JavaScript) exceptions for Cordova apps and as far as understood I can only achieve this by using custom events with properties which sometimes makes it a little bit difficult, when attaching a stack trace for example.

Hi there! This isn't on our immediate roadmap so unfortunately, I don't have an ETA. As Patrick mentioned, we definitely hope to bring handled errors to other platforms but due to other priorities, such as supporting HockeyApp scenarios, our team hasn't invested much time into this.
We will continue to monitor this feature for community support to help our team prioritize. In the meantime, the workaround would be to use custom events as you mentioned. I understand this isn't an ideal experience and I'll make sure to post here with any updates. Thanks!


Spamming HA with entities

What happened?
Each time the integration is reloaded, a new entitiy for each zone with suffix _n is created.
How to reproduce
Reload integration
config_entry-smart_irrigation-c34d6b3034b69cc97400485090710263.json
Relevant log output
No response
Which version are you running?
v2024.7.5
Diagnostics file

[X] I have attached a diagnostics file

Additional information
No response

duplicates https://github.com/jeroenterheerdt/HAsmartirrigation/issues/372

Duplicate of #372

uninstall, reinstall should fix this.

Unfortunately not. Completely removed the device, removed the integration, removed the HACS integration, rebooted, started from scratch with v2024.7.6, needed to configure everything (zones, modules, sensor group, weather API) new, but it still produces the *_n entities upon each reload.

OK, thanks. I will need to do some more digging then on why this happens. Thanks.

I tried, I tried, and I tried again but cannot reproduce this on any environment I have :(

I do have the same issue, entities just duplicate every integration refresh.

Me2.
3 zones, 57 Entities. More to come 😁

I see two potential issues that might be causing it. The add this sensor activity gets called twice per zone and somehow the name generator adds the extra N all the time. Are you seeing the original zone and zones with _N added? Or just zone_Ns?

Come to think of it, was it working correctly for you before? If so, what version worked normally?

I saw the regular one without the extension _n and a lot of same with the extension.
I endet up by removing the smart irrigation device ( not the integration) and added it again. The settings were kept and now everything seems to be fine. For now.
Not sure if this helps, but I played around with the experimental "Continuous updates for sensors". After my testing I realized that the enties were created. Maybe this is an side effect of the experimental function?

OK, maybe thanks for letting me know you found a solution. The others in this thread, can you let me know if the above helps for you as well?

Sorry, for me it is still not working and each reload of the integration doubles up my entities. I followed the steps to remove the smart irrigation device (which removes the integration as well) and re-adding it, with a HA restart in between. Unfortunately, all my settings were not kept and I had to reconfigure the integration, starting with the OWM Api Key and re-configuring all zones and modules.

Same here


[Refactor] 2: Switch marketplace to coordinator

Closes #243

This PR:
Ports over marketplace builder to the new builder state coordinator (see #237 )
This PR does not:
Change the legacy builder
Key places to review:
Everything.
crates/marketplace/src/service.rs is probably best read as a wholly new file rather than trying to read through the diff.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

301 of 364 (82.69%)  changed or added relevant lines in 6 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage increased (+1.8%) to 89.696%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

crates/marketplace/src/testing/mod.rs
119
121
98.35%

crates/marketplace/src/hooks.rs
4
10
40.0%

crates/marketplace/src/testing/integration.rs
0
19
0.0%

crates/marketplace/src/service.rs
95
131
72.52%

Files with Coverage Reduction
New Missed Lines
%

crates/marketplace/src/testing/integration.rs
1
0.0%

crates/marketplace/src/service.rs
2
67.8%

Totals

Change from base Build<PHONE_NUMBER>7:
1.8%

Covered Lines:
6616

Relevant Lines:
7376

💛  - Coveralls


feat: adjust colors

Description
fixes #393
Demo

Checklist:

[x] I have read and followed the Contributing Guide
[x] adjust styles by visual material

Hope I didn't miss anything .) please help me check

State channels dashboard description looks kind of "broken" to me, I did like the previous version as I already commented on Figma

No strong preference here. I reverted it

For some reason the bold font in the navigation bar looks better in the design compared to the live version. I would revert it for now and retry again once we get the new fonts.

It's a problem of rendering in Figma. I am also surprised that such a tool provides different results from the browser. Some other font will not resolve it

Thanks for the fixes, I noticed a few more things.
* I still think the weight font in the header does not fit well in this case

* Width of mobile header and width of the footer are not aligned. I think there was a misunderstanding on Figma. You asked about the width but the reply was about the text in the footer:

In short, easier readability. If i made it full-width gets too close to the links. I can make it all 50% of the full container if you want to have them identical or if its more practical to you :)

I don't think a different width is intentional and it's probably a leftover from the previous width, I would keep a single width. In my eyes things looks misaligned with different widths but that's my personal opinion.

Ok, agree. I reverted it back

I will merge this as it is hard for Liubov to test. It is vaguely defined and the visual material contains a lot of outdated things.


Support additional interface addresses

Fixes #4.

Ah.
https://github.blog/changelog/2022-11-09-github-actions-ubuntu-latest-workflows-will-use-ubuntu-22-04/

Fixed by #10.


Refactor core module

Codecov Report

Merging #457 into develop will increase coverage by 0.31%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #457      +/-   ##
===========================================
+ Coverage    91.62%   91.94%   +0.31%
===========================================
  Files           37       37
  Lines         1289     1278      -11
  Branches       277      273       -4
===========================================
- Hits          1181     1175       -6
+ Misses         107      102       -5
  Partials         1        1

Impacted Files
Coverage Δ

packages/ckb-sdk-core/src/index.ts
63.29% <100.00%> (+0.56%)
:arrow_up:


crash with no network

When my provider is reconnecting the internet connection, my server has no network for some time (seconds?). This apparently leads mail2most to crash. Systemd restarts it but stops to do so after some failed tries. I've added RestartSec=30 to the unit file and hope that this will be enough to fix this problem.
Aug 09 02:52:17 pi systemd[1]: Started mail2most.
Aug 09 02:52:17 pi mail2most[28310]: time="2019-08-09T02:52:17+02:00" level=debug msg="using logging method" format=text output=stdout
Aug 09 02:52:17 pi mail2most[28310]: 2019/08/09 02:52:17 dial tcp <IP_ADDRESS>:993: connect: network is unreachable
Aug 09 02:52:17 pi systemd[1]: mail2most.service: Main process exited, code=exited, status=1/FAILURE
Aug 09 02:52:17 pi systemd[1]: mail2most.service: Failed with result 'exit-code'.
Aug 09 02:52:17 pi systemd[1]: mail2most.service: Service RestartSec=100ms expired, scheduling restart.
Aug 09 02:52:17 pi systemd[1]: mail2most.service: Scheduled restart job, restart counter is at 9.
Aug 09 02:52:17 pi systemd[1]: Stopped mail2most.
Aug 09 02:52:17 pi systemd[1]: mail2most.service: Start request repeated too quickly.
Aug 09 02:52:17 pi systemd[1]: mail2most.service: Failed with result 'exit-code'.
Aug 09 02:52:17 pi systemd[1]: Failed to start mail2most.

i changed the behavior not to crash and just log the error instead and added your RestartSec to the documentation using 60 seconds. Can you confirm that it is working ? (also #10 is included in this binary)
linux-arm.tar.gz
linux-arm64.tar.gz

Great, works, thanks!


Determine if a defaultValue is feasible across platforms.

Not feasible across platforms in a consistent manner. Its unclear if setting on specific device to defaultValues should filter out to other platforms to their respective defaults. Closing.


remove uid in authentication optional config

Ⅰ. Describe what this PR does
II. Does this pull request fix one issue?

III. Special notes for reviewers if any.

LGTM


Using .join() on an empty Vec stalls

Is this the intended behaviour?
futures-concurrency = "7.4.1"

    pub use futures_concurrency::prelude::*;

    #[tokio::test]
    // #[ignore]
    async fn test_join_empty_vec() -> Result<()> {
        Vec::<AbortOnDrop<()>>::new().join().await;

        Ok(())
    }

Result:
running 1 test
        SLOW [>  5.000s] tokio_fixes::tests::test_join_empty_vec

You're right that this is not the intended behavior. I've posted a patch for it in https://github.com/yoshuawuyts/futures-concurrency/pull/154. Thank you for reporting this!

Fixed in v7.4.2


Split sqlValue into sqlName and sqlValue to facilitate property name …

…adjustments during SQL WHERE clause construction.

This control flow is getting more complicated.
I still think we should just refine toSQL() onto Property and then we just do
return this.arg1.toSQL() + ' IS NOT NULL'
and similar throughout all the expressions.  If all expressions implement toSQL(), then we don't need so many checks.

Abandoned.


resolves #92 tighten spacing between list items

tighten spacing between simple list items by default
allow simple list item spacing to be controlled by theme (outline_list_item_spacing)
align spacing between items in colist with simple items in outline list

https://github.com/asciidoctor/asciidoctor/issues/1087
https://github.com/asciidoctor/asciidoctor/issues/1221
https://github.com/asciidoctor/asciidoctor/issues/1346
https://github.com/asciidoctor/asciidoctor-pdf/issues/92
https://github.com/asciidoctor/asciidoctor-pdf/pull/164


Many-to-Many Population Suddenly Broken

Sails version: 0.12.3
Node version: 6.10.3
NPM version: 3.10.10
Operating system: Linux (Heroku Node.js Buildpack)

I'm using Sails 0.12.3 with PostgreSQL together with the sails-postgresql (0.11.4) module. I have a model, Mentor with an association:
properties: {
            collection: 'property',
            via: 'mentors',
            through: 'mentor_properties'
}

This association has custom column names using a join table definition:
identity: 'mentor_properties',

    attributes: {

        property: {
            model: 'property'
        },

        mentor: {
            model: 'mentor'
        }

    }

When using the above models in a controller, I have simply called Mentor.findOne(id).populate('properties') which has worked flawlessly until within the latest week. I have not updated the code, I have not updated the database - everything is exactly the same, except for a rebuild (which triggers npm install). All dependencies are for exact versions, so no new minor versions with bugs there. The data in the database is untouched.
Populate seemingly works as usual, it makes mentor.properties an array with add() and remove(), but the array is empty. I have one-to-many associations on the same model which do work. This only seems to affect many-to-many associations with custom column names.
This behavior is the same on Windows (Node 7.7.4, 4.1.2) as it is on Node 6.10.3 (LTS) with NPM 3 on Heroku. The exact same code, database settings, dependency versions in package.json and data in the database that worked a week ago, does now not populate properly. The only logical conclusion I can come up with is that somewhere down in the dependency tree, something broke and it makes many-to-many populations with custom column names silently fail. I do not know what more information to provide, but am happy to try anything upon request. The population works again after I made a Heroku rollback to the old build.

Ok, fixed!

Ok, fixed!

@macfeaster If you want to follow up on your hunch about dependencies, you could try logging into one of the Heroku boxes and running npm ls to get the whole dependency tree, and then doing the same with a new build that reproduces the problem.  If you can try that and see if there are any discrepancies, that might help to pinpoint what changed.

Thank you for the help. I did npm ls on the working build and a fresh build. There are some sub-dependencies of the pg dependency of connect-pg-simple which have changed, but shouldn't they relate to sessions and have no effect on Waterline? Otherwise, everything is the same. I'm so incredibly confused by this bug since the through association just stopped working for seemingly no reason.
As a quick fix, I tried to use the through model directly, but it is undefined even though the sailsjs documentation clearly states that it can be queried directly. How can I query the join table directly?

I really f'd up here... :) in the new build I had added a field to the Property model, which wasn't properly migrated to SQL. Instead of erroring, Waterline gave me an empty response, hence my confusion. Fixed by running migrations!


react-query v4: useSyncExternalStore error with Create React App

Describe the bug
Using<EMAIL_ADDRESS>(a.k.a. Create React App) does not work with<EMAIL_ADDRESS>The following is the error that occurs when running the development server:
./node_modules/react-query/lib/reactjs/useSyncExternalStore.mjs
Can't reexport the named export 'useSyncExternalStore' from non EcmaScript module (only default export is available)

Your minimal, reproducible example
https://github.com/tmm/react-query-debug
Steps to reproduce

Clone repo
Install dependencies npm i
Run development server npm start

Expected behavior
I expected to not get an useSyncExternalStore error
How often does this bug happen?
Every time
Screenshots or Videos

Platform

OS: macOS
Browser: Firefox, Chrome

react-query version
4.0.0-beta.X
TypeScript version
No response
Additional context
Reporting based on this comment by TkDodo for https://github.com/tmm/wagmi/issues/398

Same issue here

Same issue here and I'm using windows

@hirbod @sachinraja can you maybe also have a look here?
I'm not sure if we're doing some wrong in our imports / exports, or if this is CRA related? We have some examples that run on CRA 3.x and they are fine 🤔

FYI, If I change this to use the default import, like the error suggests, it doesn't complain anymore, however, it also doesn't work at runtime:

This suggests to me that the named re-export is actually correct ?

I am afraid that I am not the best person to answer this. All my knowledge goes down to RN

The error is correct. We need to change useSyncExternalStore.ts to this:
import pkg from 'use-sync-external-store/shim/index.js'
export const useSyncExternalStore = pkg.useSyncExternalStore

Ah yeah, I remember seeing this fix in another repo, I think it was reanimated or react-native-gesture-handler.

thx, I'll make a PR. One more question: is the code for useSyncExternalStore.native.ts correct?
export { useSyncExternalStore } from 'use-sync-external-store/shim/index.native.js'

The metro bundler only picks the cjs path for this package but yeah it should probably also be changed.


python312Packages.renault-api: 0.2.6 -> 0.2.7

Description of changes
Diff: https://github.com/hacf-fr/renault-api/compare/refs/tags/v0.2.6...v0.2.7
Changelog: https://github.com/hacf-fr/renault-api/releases/tag/v0.2.7

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 339366 run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.renault
python311Packages.renault-api
python311Packages.renault-api.dist
python312Packages.renault-api
python312Packages.renault-api.dist


Remove the deprecated Helm Chart option SMI

Proposal
Due to #495, we're planning to bump a new major version of the helm chart (v2.0.0). It's maybe the right timing to remove the newly deprecated SMI Mode.

It certainly is :)


Regression of Eloquent Query Builder where method

Larastan Version: 2.9.9
Laravel Version: 11.28.1

Description
I'm getting the following error with my static analysis today:
  :51    Parameter #1 $column of method Illuminate\Database\Eloquent\Builder<*>::where() expects array<int|model property of never, mixed>|(Closure(Illuminate\Database\Eloquent\Builder<*>):
         mixed)|Illuminate\Contracts\Database\Query\Expression|model property of never, 'record_status' given.

Indicating that I'm sending an incorrect value to the where method. However I'm sending a simple string for the column name. See code below.
Laravel code where the issue was found
        $query = $this->getBasePinsQuery()
            ->where(static function (Builder $query) {
                $query->where('record_status', '!=', Pin::RECORD_STATUS_ACTIVE)
                    ->orWhereNull('record_status');
            })
            ->whereNull('deleted_at')
            ->whereRaw("to_tsvector('simple', public.f_concat_ws(' ', address, city, state, zip))
                @@ plainto_tsquery('simple', ?)", $search);

        return $query->limit(20)->get();

The error refers to line 51, which is $query->where('record_status', '!=', Pin::RECORD_STATUS_ACTIVE)
However, if I replace the where that passes in the closure with a simple where as such:
        $query = $this->getBasePinsQuery()
            ->where('record_status', '')
            ->whereNull('deleted_at')
            ->whereRaw("to_tsvector('simple', public.f_concat_ws(' ', address, city, state, zip))
                @@ plainto_tsquery('simple', ?)", $search);

        return $query->limit(20)->get();

I get the same error:
  :50    Parameter #1 $column of method Illuminate\Database\Eloquent\Builder<*>::where() expects array<int|model property of never, mixed>|(Closure(Illuminate\Database\Eloquent\Builder<*>):
         mixed)|Illuminate\Contracts\Database\Query\Expression|model property of never, 'record_status' given.

Line 50, which would be ->where('record_status', '')
I downgraded larastan to 2.9.8 and it started working again, so the regression must be with this morning's release of 2.9.9

The $this->getBasePinsQuery() method just returns a model's query builder:
    public function getBasePinsQuery(): Builder
    {
        return Pin::select([
            'columns',
        ])->with(['relations']);
    }

You don't have the proper generics:
    /** @return Builder<Pin> */
    public function getBasePinsQuery(): Builder
    {
        return Pin::select([
            'columns',
        ])->with(['relations']);
    }

Also, I've add issues with Closure types overriding the generics so you might need to use:
            ->where(static function ($query) {
                $query->where('record_status', '!=', Pin::RECORD_STATUS_ACTIVE)
                    ->orWhereNull('record_status');
            })

@calebdw If I do that, with this code:
    public function getByAddress(string $search): Collection
    {
        $query = $this->getBasePinsQuery()
            ->where(static function ($query) {
                $query->where('record_status', '!=', Pin::RECORD_STATUS_ACTIVE)
                    ->orWhereNull('record_status');
            })
            ->whereNull('deleted_at')
            ->whereRaw("to_tsvector('simple', public.f_concat_ws(' ', address, city, state, zip))
                @@ plainto_tsquery('simple', ?)", $search);

        return $query->limit(20)->get();
    }

    /**
     * @return Builder<Pin>
     */
    public function getBasePinsQuery(): Builder
    {
        return Pin::select([
            'columns',
        ])->with(['relations']);
    }

I get the following errors:
  :51    Parameter #1 $column of method Illuminate\Database\Eloquent\Builder<*>::where() expects array<int|model property of never, mixed>|(Closure(Illuminate\Database\Eloquent\Builder<*>):
         mixed)|Illuminate\Contracts\Database\Query\Expression|model property of never, 'record_status' given.
  :64    PHPDoc tag @return contains generic type Illuminate\Contracts\Database\Eloquent\Builder<App\Models\Pin> but interface Illuminate\Contracts\Database\Eloquent\Builder is not generic.

Illuminate\Contracts\Database\Eloquent\Builder

There's your problem, don't use the contract

Gotcha, it's weird that it only broke on that specific line only though. Looking over this class, there's another place with a similar where using the contract that phpstan isn't complaining about... Oh well, thanks phpstan is passing now after replacing the contract import with Illuminate\Database\Eloquent\Builder

there's another place with a similar where using the contract that phpstan isn't complaining about...

It's not going to complain until you go to use the inner type, your Collection also needs to have its generics defined so that whatever calls the method knows what's inside of it:
    /** @return Collection<int, Pin> */
    public function getByAddress(string $search): Collection

Oh well, thanks phpstan is passing now after replacing the contract import with Illuminate\Database\Eloquent\Builder

Great!, sounds like you can close this then :smile:

I think the actual issue was that it detected somehow the builder as Illuminate\Database\Eloquent\Builder<*> (attention on *) Then of course it can't find a property on it because there is no model to check. A solution was to type hint it correctly, like you did.
But maybe we can make it so it doesn't produce an error when the generic type is *

But maybe we can make it so it doesn't produce an error when the generic type is *

The solution should be to tell PHPstan about the inner type, not to silently ignore potential errors. Ignoring * would mean that the user is not aware that there's an issue that needs to be fixed in the first place

PHPStan already produces error if you don't typehint the generic types. (like in the OP's example) But for checkModelProperties if the generic type is * there is nothing we can infer so we can just skip the if it is a model property check and treat it as a normal string.


Partially addresses Issue 2147

Update some of the ![image] issues #2147, add alt tags to images in the conversion where they are not specified.

@tdykstra and @Rick-Anderson - search source for ![image] to find, most are easy fixes.


Fix cookie encoding when maxAge is greater than 24 days

Unfortunately, this was missed when making maxAge an int again. I've added a test to protect against this in future.

AUTOMATED MESSAGE for ADMINS:Please verify and accept the pull request.To accept, say @netkins acceptTo build again, say @netkins buildTo whitelist the author, say @netkins whitelist

@netkins accept

@jroper thanks!
Cherry-picked into 3.9 (6e2d2ce9983b0e4926b3e0bd0419cffe19e36349) and 3.10 (46b2eff75701f6ffe3a12b926ccef6764b77d2cc)


fix: Add CGO_ENABLED=0 flag to Dataflow example

Description
Fixes #4405 by adding CGO_ENABLED=0 to build instruction of Dataflow example.
Checklist

[x] I have followed Contributing Guidelines from CONTRIBUTING.MD
[x] Tests pass via https://cloud.google.com/dataflow/docs/guides/templates/using-flex-templates
~Tests pass:   go test -v ./.. (see Testing)~
~- [x] Code formatted:   gofmt (see Formatting)~
~- [ ] Vetting pass:   go vet (see Formatting)~
~These samples need a new API enabled in testing projects to pass (let us know which ones)~
~These samples need a new/updated env vars in testing projects set to pass (let us know which ones)~
~This sample adds a new sample directory, and I updated the CODEOWNERS file with the codeowners for this sample~
~This sample adds a new Product API, and I updated the Blunderbuss issue/PR auto-assigner with the codeowners for this sample~
[x] Please merge this PR for me once it is approved

@damondouglas can you provide a bit more context on this change?
This disables the mechanism for working with c code which might be the right path if no c libraries are needed, and the reason it's failing in some places might be due to the different environments (containers or on a mac where glibc libraries aren't available). I'm a little worried this might cause a potential issue based on assumptions about where users might run the code. I haven't dug into the existing code to verify what is needed so if you can give some feedback on the context that would be reassuring to merge this.

I'm not too familiar with the Go compilation process and how Cgo affects it. From a quick search, it looks like Cgo allows access to C's stdlib in the target environment, which is useful in some targets like Android. This, however, introduces stdlib as a dynamically linked library, so the resulting Go binary is no longer statically isolated (one of the best features of Go). From some reading, it looks like if Cgo is enabled, it'll use stdlib functions, but if it's disabled then the Go compiler introduces its own versions of those functions and keeps the binary completely independent from the host system.
Here's a great blog post that goes into some more detail.
From what I can gather, this is not only safe, but will guarantee consistent binaries that don't depend on the system stdlib (which can be different on different operating systems and Docker images like alpine). I'm actually a bit surprised that Cgo is enabled by default, it would seem like a feature that should be opt-in only if you need it. It is (inconsistently) disabled by default when cross-compiling.


Throwable to string

Codecov Report

Merging #4 into master will increase coverage by 10.37%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           master       #4       +/-   ##
===========================================
+ Coverage   17.77%   28.15%   +10.37%
===========================================
  Files          17       20        +3
  Lines          90      103       +13
===========================================
+ Hits           16       29       +13
  Misses         74       74

Impacted Files
Coverage Δ

...la/http4s/extend/ByNameNaturalTransformation.scala
0% <ø> (ø)
:arrow_up:

src/main/scala/http4s/extend/Effectful.scala
0% <ø> (ø)
:arrow_up:

...end/syntax/ByNameNaturalTransformationSyntax.scala
0% <ø> (ø)
:arrow_up:

.../http4s/extend/syntax/MonadErrorModuleSyntax.scala
100% <ø> (ø)
:arrow_up:

...in/scala/http4s/extend/util/MonadErrorModule.scala
100% <ø> (ø)
:arrow_up:

...ala/http4s/extend/syntax/Http4sServiceSyntax.scala
0% <ø> (ø)
:arrow_up:

.../extend/instances/ErrorInvariantMapInstances.scala
100% <100%> (ø)
:arrow_up:

...a/http4s/extend/instances/InvariantInstances.scala
100% <100%> (ø)

...n/scala/http4s/extend/syntax/InvariantSyntax.scala
100% <100%> (ø)

...ain/scala/http4s/extend/util/ThrowableModule.scala
100% <100%> (ø)

... and 3 more

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 0674e84...1450a99. Read the comment docs.


gcloud kubectl vs. vanilla kubectl

Apparently kubectl installed via gcloud causes issues for at least some of our deploys (AWS in particular).  We should document this at the very least, or get that install path working:

thanks for checking, really helped me narrow it down. On a clean environment it worked. After investigating I see that I had installed kubectl by running gcloud components install kubectl. After removing this from my PATH and using vanilla kubectl, it worked :sweat_smile:

This was when hitting the 'waiting for nodes to come online' issue that we've seen w k8s 1.7.0 (but was resolved w an update to the aws.sh script afaict).
FWIW, I have kubectl installed via gcloud and validated that kops 1.7.0 and kubectl 1.7.2 work ok for me and the aws.sh works fine.

Seems like this is resolved. closing.


add timeout to http reqs in sink

bors r+

@ozgrakkurt it seems that we need to update the tests:
https://github.com/infinyon/http-sink-connector/blob/2723769f98b8cd3ae35af9b74b96e929104a03a5/crates/http-sink/src/sink.rs#L69

bors r+


Devel

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

31 of 31 (100.0%)  changed or added relevant lines in 3 files are covered.
38 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-2.2%) to 92.069%

Files with Coverage Reduction
New Missed Lines
%

node.go
38
75.82%

Totals

Change from base Build<PHONE_NUMBER>1:
-2.2%

Covered Lines:
1451

Relevant Lines:
1576

💛  - Coveralls


Design QA - Claim Stake & fee Modal

Please, update styles to match design

I've also included the processing & confirm status behavior:

And an example of error message: Do you have a text for that? I'm not able to reproduce the behaviour.

Figma reference:
https://www.figma.com/file/wogMuUDylgOHgTEYhZ3jgp/Augur-Bet-UI?node-id=8379%3A982

looks good


serviceName returns "Sonos Radio" instead of "TuneIn"

This is most likely an issue with Sonos itself and not with your extension. However, I came across an issue with one of my scripts, which was working just fine a few weeks ago. (It's meant to toggle between spotify and tunin). I fixed my script for my needs, but thought I might leave a note here just in case, since the sonos app shows the tunein icon as intended.
So here two examples of group.get.trackplus:
Sonos Radio
{
  "artist": "Tocotronic",
  "album": "",
  "title": "Hoffnung",
  "artUri": "https://sonosradio.imgix.net/station-images/1a928ce0-e562-4c79-9d84-c2171fec9b7c.png?w=200&auto=format,compress?w=200&mark-scale=93&mark-align=center,middle&mark=https%3A%2F%2Fapi.napster.com%2Fimageserver%2Fv2%2Falbums%2Falb.462900682%2Fimages%2F600x600.jpg&auto=format,compress",
  "mediaData": {
    "NrTracks": 11,
    "MediaDuration": "NOT_IMPLEMENTED",
    "CurrentURI": "x-sonosapi-radio:sonos:158299?sid=303&flags=40&sn=6",
    "CurrentURIMetaData": {
      "Artist": "Hansestadt-Indie",
      "AlbumArtUri": "https://sonosradio.imgix.net/station-images/33e70b54-8458-4722-9032-b0329c539d19.png?w=200&auto=format,compress",
      "Title": "Hamburger Schule",
      "UpnpClass": "object.item.audioItem.audioBroadcast#swimlane-genre",
      "ItemId": "000c0028sonos%3a158299",
      "ParentId": "%2fstations%2fde-DE%2fDE%2fc2Q6REU6aW5kaWUtYWx0ZXJuYXRpdmU"
    },
    "PlayMedium": "NETWORK",
    "RecordMedium": "NOT_IMPLEMENTED",
    "WriteStatus": "NOT_IMPLEMENTED"
  },
  "queueActivated": false,
  "radioId": "",
  "serviceId": "303",
  "serviceName": "Sonos Radio",
  "stationArtUri": "",
  "positionData": {
    "Track": 2,
    "TrackDuration": "0:00:00",
    "TrackMetaData": {
      "Artist": "Tocotronic",
      "AlbumArtUri": "https://sonosradio.imgix.net/station-images/1a928ce0-e562-4c79-9d84-c2171fec9b7c.png?w=200&auto=format,compress?w=200&mark-scale=93&mark-align=center,middle&mark=https%3A%2F%2Fapi.napster.com%2Fimageserver%2Fv2%2Falbums%2Falb.462900682%2Fimages%2F600x600.jpg&auto=format,compress",
      "Title": "Hoffnung",
      "UpnpClass": "object.item.audioItem.musicTrack",
      "ItemId": "-1",
      "ParentId": "-1",
      "TrackUri": "x-sonos-http:sonos:13d4086d1b2007d79f020b81aff80f07:83:1649944185563:middle:158299::tra.462900683:default:SD.mp4?sid=303&flags=0&sn=6",
      "ProtocolInfo": "http-get:*:audio/mp4:*"
    },
    "TrackURI": "x-sonos-http:sonos:13d4086d1b2007d79f020b81aff80f07:83:1649944185563:middle:158299::tra.462900683:default:SD.mp4?sid=303&flags=0&sn=6",
    "RelTime": "0:00:10",
    "AbsTime": "NOT_IMPLEMENTED",
    "RelCount":<PHONE_NUMBER>,
    "AbsCount":<PHONE_NUMBER>
  }
}

TuneIn
{
  "artist": "ZPSTR_CONNECTING",
  "album": "",
  "title": "stream.aac?aggregator=tunein&sid=27n62ycDWcWtabbjG4HQiaUSKrt&token=l2aBRvzVamln7wBl3P6-3wJUntITVHTnc64FPzlIMaI&tvf=89gOkVXa5RZkMTExLnJuZGZuay5jb20",
  "artUri": "",
  "mediaData": {
    "NrTracks": 3,
    "MediaDuration": "NOT_IMPLEMENTED",
    "CurrentURI": "x-sonosapi-stream:tunein:151906?sid=303&flags=8224&sn=6",
    "CurrentURIMetaData": {
      "Title": "Deutschlandfunk Nova",
      "UpnpClass": "object.item.audioItem.audioBroadcast",
      "ItemId": "-1",
      "ParentId": "-1"
    },
    "PlayMedium": "NETWORK",
    "RecordMedium": "NOT_IMPLEMENTED",
    "WriteStatus": "NOT_IMPLEMENTED"
  },
  "queueActivated": false,
  "radioId": "",
  "serviceId": "303",
  "serviceName": "Sonos Radio",
  "stationArtUri": "http://<IP_ADDRESS>:1400/getaa?s=1&u=x-sonosapi-stream:tunein:151906?sid=303&flags=8224&sn=6",
  "positionData": {
    "Track": 1,
    "TrackDuration": "0:00:00",
    "TrackMetaData": {
      "Artist": "ZPSTR_CONNECTING",
      "Title": "stream.aac?aggregator=tunein&sid=27n62ycDWcWtabbjG4HQiaUSKrt&token=l2aBRvzVamln7wBl3P6-3wJUntITVHTnc64FPzlIMaI&tvf=89gOkVXa5RZkMTExLnJuZGZuay5jb20",
      "UpnpClass": "object.item",
      "ItemId": "-1",
      "ParentId": "-1",
      "TrackUri": "aac://http://d111.rndfnk.com/ard/dlf/03/aac/192/stream.aac?aggregator=tunein&sid=27n62ycDWcWtabbjG4HQiaUSKrt&token=l2aBRvzVamln7wBl3P6-3wJUntITVHTnc64FPzlIMaI&tvf=89gOkVXa5RZkMTExLnJuZGZuay5jb20",
      "ProtocolInfo": "aac:*:application/octet-stream:*"
    },
    "TrackURI": "aac://http://d111.rndfnk.com/ard/dlf/03/aac/192/stream.aac?aggregator=tunein&sid=27n62ycDWcWtabbjG4HQiaUSKrt&token=l2aBRvzVamln7wBl3P6-3wJUntITVHTnc64FPzlIMaI&tvf=89gOkVXa5RZkMTExLnJuZGZuay5jb20",
    "RelTime": "0:00:00",
    "AbsTime": "NOT_IMPLEMENTED",
    "RelCount":<PHONE_NUMBER>,
    "AbsCount":<PHONE_NUMBER>
  }
}

Anyway, thanks a lot for your work.
cheers
Michael

Hi.
Yes, the service id is extracted from CurrentURI and 303 stands for Sonos Radio, 254 for TuneIn. The CurrentURI is provided by SONOS player.
Anyway it is a bit strange:
Using the SONOS Android App, browsing for "Deutschlandfunk Nova" with criteria "Station" shows that station for SONOS Radio, Radio player and Tune in. I selected the TuneIn version. group.get.trackplus shows the right service (Tune in) and the right radio id: s120806
Could you please check it?
The "ZPSTR_CONNECTING" indicates, that the player is still in the process to execute a change. It might make sense to put a delay node before the group.get.trackplus.

Lets keep that open - I will have a closer look after vacation. TuneIn is causing some trouble. Often when restarting the station they play a long information about their capabilities: podcasts, station etc - annoying.

You are right.

radio id is not anymore included in the uri of the tune in radio stations - therefore I can not extract it.
if you start any tunein station SONOS will always show it as SONOS Radio service

I will remove radioId from the properties and make a note that sonos radio will be shown instead of tune in as service.
You can still get the Tune In Radio Id on the Tune In web side (part of the url) and use that in group.play.tunein
Thank you for letting me know that change!


Does this support deserialization for APIs?

I'd like to use these Option types in an API, IE currently an interface might be:
interface Person {
  name: string;
  display_name?: string // I'd like this to be display_name: Option<string>;
  ...
}

Hi!
We don't yet provide this kind of feature, but that's something we'll be exploring in the future 🙂


Proxy Refactor

In order to reduce number of virtual machines, support adding proxy support to primary nimbus system, instead of creating a separate virtual machine.

Fixed with https://github.com/darkn3rd/storm-vagrant/commit/d7fcb3d19b1abd892aa2b1c278e82f58bc42ba1e
proxy supports both 0.10.x and 1.0.x scenarios

PR https://github.com/darkn3rd/storm-vagrant/pull/17


Bevy 0.7 migration, bumps minor versions of deps

Ultra simple, just needed it myself, so here it is for everyone else too!
Tested working in my tiny project, but ONLY that tiny project

Thank you for the contribution!


Cannot be used with burpsuite2024.1 version

It used to be available on the version released by burp in January, but it suddenly became unavailable in the past few days.

Thanks so much for bringing this to our attention @wt2266232 , we'll look into it!


[C++][Parquet] Add version to WriterProperties

PR: https://github.com/apache/parquet-cpp/pull/119
Reporter: Uwe Korn / @xhochy
Assignee: Uwe Korn](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=uwe) / @xhochy
Note: This issue was originally created as PARQUET-633. Please see the migration documentation for further details.

Wes McKinney / @wesm:
Issue resolved by pull request 119
https://github.com/apache/parquet-cpp/pull/119


godot game crash in "Bullet shower" demo

hey i have found a costant bug, godot game close, I think, to this point that not is error of my game, anyway try "Bullet shower" demo, the bug not is present in 1.0 , but if i test the demo with 1.1 the bug apppear when I collide with 3-2 bullet.

the terminal errors:
2d/shower_of_bullets -rdebug localhost:6007 res://shower.scn
ERROR: set_stream_peer: Condition ' p_peer.is_null() ' is true.
At: core/io/packet_peer.cpp:228.
table/2d/shower_of_bullets/data.pck, -1, -1
has connection!
has connection!
CLOSING CONNECTION
ERROR: set_stream_peer: Condition ' p_peer.is_null() ' is true.
At: core/io/packet_peer.cpp:228.
freed code id 5
freed code id 4
ERROR: free: Method/Function Failed.
At: servers/visual/visual_server_raster.cpp:4659.
freed code id 6
ERROR: free: Method/Function Failed.
At: servers/visual/visual_server_raster.cpp:4659.
ERROR: free: Method/Function Failed.
At: servers/visual/visual_server_raster.cpp:4659.
ERROR: set_stream_peer: Condition ' p_peer.is_null() ' is true.
At: core/io/packet_peer.cpp:228.
base resources missing 2
ERROR: finish: Condition ' instance_dependency_map.size() ' is true.
At: servers/visual/visual_server_raster.cpp:7483.
ERROR: cleanup: ObjectDB Instances still exist!
At: core/object.cpp:1723.

I guess it's duplicated issue with #1592

I too think it is a duplicate of #1592

this seems fixed

(can't reproduce)

I recall that there was a commit that fixed this in the days after the 1.1 beta release.
Grab any new build done either on your own or by Marynate and you shouldn't have any crashes.


[Compatibility] Trade Cycling

When refreshing trades with Trade Cycling, only the first two transactions are shown.

Only if you close the trading interface and right click villager again will all trades show up.

It's working fine, but can it get better, refresh all trades with Trade Cycling?
Sorry for issues again and again.

Has this issue been resolved like Easy villagers ?

Hello @DoloSaber. The locked trades are generated over several ticks to prevent large lag spikes due to how inefficient the vanilla trade generation code is. I have set a limit on the locked trade generation so only 1 level's worth of trades is generated at a time. Because of that, all trade cycle mods send the updated trades too quickly for Visible Traders to be ready. It would not be efficient performance wise to have them all generate as well as feasible for me to add compats for each trade cycle mod since they all do it slightly differently


[OpenAI] Implement ChatClient

- [ ] Protocol methods
- [ ] Convenience methods
- [ ] Models and model factory methods
- [ ] Architecture review

Complete the tasks above for each of the following features:

Create chat completion

Curious why we call this ChatClient and not ChatCompletionClient? We do have ChatCompletetion* names in the APIView?

Curious why we call this ChatClient and not ChatCompletionClient? We do have ChatCompletetion* names in the APIView?

This is to align with OpenAI's documentation, where they refer to it as the Chat APIs.