Add hover cards

See the story in the release criteria.

Is the geometry (property) in the correct format (numbers...numbers)?

Some form names are cut off in the Firefox browser

Thank you, those both look like issues. :+1: I've also noticed that in some cases, there is too much padding below the list of properties in the hover card for an entity list. For example, if you hover over the "basic" entity list here: https://test.getodk.cloud/#/projects/414/entity-lists

I didn't have a chance to make these changes today, but I'll do so soon.

Hey, shouldn't the Infocard be visible also on that page after hovering the mouse over any of the links?

shouldn't the Infocard be visible also on that page after hovering the mouse over any of the links?

This should now be fixed.

@sadiqkhoja mentioned to me yesterday that he was seeing an issue with hover cards:

Forms with lengthy names show the card outside of the screen

In other words, if there wasn't enough space for the hover card on either the right or the left of the link, it was being shown partly offscreen. Instead, we want to show the hover card above or below the link if there isn't space for it on the right or left.
getodk/central-frontend#1097 was merged to address this issue, and the issue should now be fixed. Let me know if you ever notice a hover card being shown offscreen.

We're also tracking another issue with hover cards. It definitely affects hover cards for forms and may affect other hover cards as well. The issue is: In a hover card for a form, if the text in the second column is shorter than the text in the first column (i.e., if the form version is short), then the hover card will widen to fit the full width of the form name. The problem is that hover cards are not supposed to become arbitrarily wide. The form name is supposed to truncate if it's too long. I'm planning to fix this issue tomorrow.

hover cards are not supposed to become arbitrarily wide. The form name is supposed to truncate if it's too long.

This should now be fixed.

Is the geometry (property) in the correct format (numbers...numbers)?

Some form names are cut off in the Firefox browser

I think these are the last two things that need to be fixed.

Is the long property name already fixed- should it be "bbbbb..."?

Is the long property name already fixed- should it be "bbbbb..."?

That one is actually by design. If the list of properties doesn't fit, it will be visible up until the very edge of the hover card. In the release criteria, there's a screenshot along those lines that you can see.

Is the geometry (property) in the correct format (numbers...numbers)?

This should now be fixed. I haven't updated the QA server yet, but you can see the fix on staging. I'll update the QA server soon as well.

Some form names are cut off in the Firefox browser

I actually wasn't able to reproduce this issue on my machine, perhaps due to differences in fonts. However, I made some changes that I'm hoping has addressed this. That change can be seen on staging.

I've also noticed that in some cases, there is too much padding below the list of properties in the hover card for an entity list.

This should now be fixed on staging. The issue was due to property names that contained a hyphen.

I'm not sure about this case- it feels ok but on the other hand there are 2 different values. When the user is on the page with entity lists and a new entity is created (while being on the page), the hover card shows the newest number of entities (10) while the page (which wasn't refreshed) shows the number of entities without the newly created entity (9). Should I file an issue or it's ok?

After editing an entity with a long name, the hover card in submission detail page is broken (wrong width of property name, entered values are show beyond the hover card.
https://staging.getodk.cloud/#/projects/97/forms/trees_registration/submissions/uuid%3A2d68092f-0d00-4123-b98a-b87f8c19f9f8

Steps to reproduce:

Go to the Entities "Data” tab.
https://staging.getodk.cloud/#/projects/97/entity-lists/trees/entities
Click "More”.
In the detail page click "Edit”.
In e.g. circumference_cm field enter a long name/number and update the entity.
In the "Activity” part click the hyperlink to the submission.
Hover over the "Created entity” hyperlink.

I'm not sure about this case- it feels ok but on the other hand there are 2 different values. When the user is on the page with entity lists and a new entity is created (while being on the page), the hover card shows the newest number of entities (10) while the page (which wasn't refreshed) shows the number of entities without the newly created entity (9). Should I file an issue or it's ok?

I personally think it's OK. Maybe it's even a good thing, as it's a signal that the page is out-of-date. This question also came up during code review, and I wrote some comments about it here.

After editing an entity with a long name, the hover card in submission detail page is broken (wrong width of property name, entered values are show beyond the hover card.

I was able to reproduce this, and we should have a fix ready soon. Since we put out the release yesterday, I filed a new issue to track this bug: #854.

Tested with success!


Using saliency with custom nets and grayscale images

Hi,
Currently the saliency method is not working for images with a single channel (ex (1,244,244)). At some point the code seems to require three channels. This results in shape errors, etc.
Could you please help me run the saliency code with grayscale images? Thanks.

Thanks for pointing this out! Are you using MNIST data? Also, is there a specific explainer you are interested in?

I am using a custom dataset. I'm interested in saliency approaches for a modified GoogLeNet architecture (with grayscale input and custom output classes).
Do you think this will be compatible anytime soon? What sort of changes will you have to make for this?
Thanks for the prompt reply.


Update openai autocomplete provider to use prompt similar to anthropic infill mode

Previously the unstable-openai provider was based on the original anthropic prompt, this updates it to be similar to the newer anthropic infill mode and accompanying context improvements.
Test plan
Ran completions tooling & benchmarks against Azure OpenAI (GPT 3.5-Turbo) and OpenAI (GPT-3.5-instruct)
Verified chat & autocomplete works with OpenAI and Azure OpenAI

OpenAI testing based on https://github.com/sourcegraph/sourcegraph/pull/57872

@chwarwick this is awesome! Going to message you to see how I can set these up to test the new prompt 🙇♀


Update understanding-permission-levels.md

#934
-Created an additional Note
"Public and Private groups in certain Tenants may experience differing behavior such as defining permission set for "Everyone Except External Users" caused by a background job ."
Known Issue:
https://techcommunity.microsoft.com/t5/SharePoint/Everyone-except-external-users-disappears-form-Group-Visitors/td-p/330066
Previous PR: https://github.com/MicrosoftDocs/OfficeDocs-SharePoint/pull/928

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@MikePlumleyMSFT Hi Mike  Changes look correct. Could you please review and merge? Thanks!

The note is ambiguous - it doesn't help the customer take an action. We wouldn't include a note like this in the article.


Cannot Create Tags in Note Editor

Bug Description
When trying to create a tag in the note, a popup saying "Add " shows up but hitting Enter or clicking on the popup does nothing.
To Reproduce
Steps to reproduce the behavior:

Open a new note
Press # to create a new tag and enter the name of the tag
Press Enter to add the tag

Expected Behavior
When adding a tag, it should add the tag.
Screenshots
beavernotes-tags.webm
Desktop (please complete the following information):

Device Specification: Intel i5-9600KF, NVIDIA GeForce RTX 2060
OS: Ubuntu 22.04.4
Application Version: 3.1.0 Flatpak

Thanks again for reporting this issue. I've already released version 3.1.1 to address this issue. Thanks again for your patience.


Support JW Library 14

Fixes https://github.com/AndreasSko/ios-jwlm/issues/57

coverage: 89.384% (+3.3%) from 86.118% when pulling 99acfa05bf189e3e1264c525cae70dfbdb08e690 on playlist_version into 39bf323161784ff5af340ae491cd3e21cb21f4b2 on master.


Not working in IE11 and safari

I have updated the flash player to the latest, even then the image not getting captured in IE11 and safari
function camfunctionality()
{
var error = true;
Webcam.set(
{
width: 320,
height: 240,
image_format: 'jpeg',
jpeg_quality: 90,
	});
Webcam.attach( '#cam' );
Webcam.on( 'error', function(err)
	{
		error = true;
		alert("We could not access the webcam. Please allow the Webcam Access to Proceed");
		Webcam.attach( '#cam' );
	});
Webcam.on( 'live', function()
	{
		error = false;
		$('#cam').hide();
		capturecam();
	});
function capturecam()
{
	setInterval(function()
		{
			console.log("test");
			Webcam.snap( function(data_uri)
			{
				Webcam.upload( data_uri, 'fun() to save image', function(code, text) {
					console.log(code);
					console.log(text);
				});
			});
		},6000);
}
	}

The error function is not getting triggered, whenever I press deny for camera to access, and the image is not getting captured in IE11 and safari.

I'm honestly not sure how to help you.  The code looks fine.  I guess there is a flaw in my library.  I am very sorry.  Please try jQuery Webcam -- it should work much better for you:
http://www.xarg.org/project/jquery-webcam-plugin/
https://github.com/infusion/jQuery-webcam
Sorry I could not help!

Joe

Stop hiding the camera when it goes live.  Remove this and try again:
Webcam.on( 'live', function()
    {
        error = false;
        $('#cam').hide(); // REMOVE THIS LINE
        capturecam();
    });

Still not working in safari, the error I am getting is
'[object BlobConstructor]' is not a constructor (evaluating 'new Blob( [ this.base64DecToArr(raw_image_data) ], {type: 'image/'+image_fmt} )')

Your Safari is very, very, very old.  Please upgrade it.  We only
support Safari 7 and 8.
--
Joseph Huckaby
<EMAIL_ADDRESS>On Sun, Apr 12, 2015, at 12:45 PM, mani541 wrote:

Still not working in safari, the error I am getting is
'[object BlobConstructor]' is not a constructor (evaluating 'new Blob( [
this.base64DecToArr(raw_image_data) ], {type: 'image/'+image_fmt} )')

Reply to this email directly or view it on GitHub:
https://github.com/jhuckaby/webcamjs/issues/62#issuecomment-92107153


Allow passing --deviceType to 'particle compile'

Right now it is not possible to compile binaries for the Photon using particle-cli, this PR allows the passing of an optional --deviceType to particle complile (which defaults to "core") which allows you to specify the compilation target.

Similar to https://github.com/spark/particle-cli/pull/40 ?

Yes but with a few subtle differences:

based off master
follows existing argument parsing techniques
fewer changes
On Thu, Jul 2, 2015 at 9:55 PM Kenneth Lim<EMAIL_ADDRESS>wrote:

Similar to #40 https://github.com/spark/particle-cli/pull/40 ?
—
Reply to this email directly or view it on GitHub
https://github.com/spark/particle-cli/pull/54#issuecomment-118232580.

Closing since #40 has been merged


An image (icon) from NexusMods doesn't shown in application (iPhone)

Hello, is there any method how to fix that? I love app but some icons (NexusMods, CTemplar) doesn't show in application.
Thank you for this super great program, Calleluks!

Have you updated to the latest version of the app? (Released yesterday)
If yes, would you please write what issuer you see in the app?
To see it, click edit, select the account in question. Then you will see the issuer in the second line.

Have you updated to the latest version of the app? (Released yesterday) If yes, would you please write what issuer you see in the app? To see it, click edit, select the account in question. Then you will see the issuer in the second line.

Hi RazziaDK, i did update now (I haven't seen it) but i don't see NexusMods icon.
Issuer in the app: NexusMods
I don't know why i don't see this icon. Amazon icon works, PayPal too.
I am wondering is it a possible to add CTemplar icon i was trying to add by myself by since my knowledge is below 0 i can't figure how to run command from root.
Kind regards

RazziaDK i did solve the problem by myself.
What i did:

I have opened this link https://github.com/calleluks/Tofu/blob/master/Tofu/AccountCell.swift#L15

Line 85: Nexus Mods (In my app on iPhone NexusMods was without space)

I've put space in my app in iPhone in issuer field. Wrote: Nexus Mods. And i saw an image! Perfect! Thank you very much for helping!

P.s. please add CTemplar icon.

Great to hear. I just created a Nexus Mods account to see if there is a space or not, if a new users creates 2FA on a Nexus Mods account today. My issuer field was "Nexus Mods", so I think the app has it right at the moment. :-)

P.s. please add CTemplar icon.

I added CTemplar in this pull request: https://github.com/calleluks/Tofu/pull/92
So I will most likely be in the next version of the app.


Regex replace fails on group operation.

VSCode Version: Code 1.18.1 (929bacba01ef658b873545e26034d1a8067445e9, 2017-11-16T18:23:26.125Z)
OS Version: Darwin x64 17.2.0
Extensions:

Extension
Author (truncated)
Version

vscode-dash
dee
1.8.0

code-runner
for
0.8.5

python
ms-
0.8.0

java
red
0.15.0

vscode-java-debug
vsc
0.4.0

vscode-java-pack
vsc
0.2.0

(1 theme extensions excluded)

Steps to Reproduce:

Use Command + Alt to call the replace window, enter your regex with group selection like '(\d+)' to find all numbers in the form of string.
In the Replace input box type as \1 to replace the selected matches with the 1st group. For example, here we type \1 to convert the matched strings to numbers.
Finally we can get strings like '1' replaced as 1, but it turns out \1.

Reproduces without extensions: Yes

I too stumbled into a similar issue - seems like using a \1 backreference to replace a string is not supported yet?

@LenzGr Yes. Seems regex is not supported perfectly in current version.
By the way, Sublime Text works well in my case. Maybe that's why I have to keep both on my machine~

@paperant I had to resort to using gvim for that bulk regex-based search and replace. Unfortunately there is little documentation about the regex capabilities of the search/replace tool.

Isn't the placeholder for the first capture group $1?
This is JavaScript RegExp, it does not support \1 to my best knowledge.

@Simran-B $ works! Thanks bro.
By the way, does it mean that RegExp in VS Code works under the JavaScript style due to Electron?

Good to know that VS Code expects JavaScript-Style Regular Expressions...
Ist that documented somewhere? If not, this would be a nice addition to the documentation...

Honestly, I can't find any documentation for the Find widget... That's really odd.
The only thing I found is about advanced search:
https://code.visualstudio.com/docs/editor/codebasics#_advanced-search-options
But yes, Electron is based on V8, which is a JavaScript engine and VS Code uses its regular expression implementation.

$1 doesn't work for me in VS Code 1.28.2 on macOS Mojave:

I see this issue is 11 months old and still open.
Is there anything planned to make groups work in search/replace?

@nhoizey Something is strange about your search phrase.
To make it work in the Find widget (Ctrl+F), I had to change it to (tags:\s*\[[^]*)Paris Web.
My test text:
tags:    [Paris Web Something, RWD]
tags:    [Paris Web, company]
tags:    [qualité, em, Paris Web]

Replace all works fine using $1parisweb:
tags:    [parisweb Something, RWD]
tags:    [parisweb, company]
tags:    [qualité, em, parisweb]

If I try the same search and replace phrases in the search panel however, I get an "Error: unclosed character class". In your screenshot, I can see that you placed an unescaped closing square bracket after the character class to match all characters ([^]), which makes the error go away for me, but then it won't find matches for me (Windows 10, VS Code 1.28.2). The error shows if I escape the second closing bracket as well, which made me think that escaped opening square brackets might be broken.
However, I then replace the [^] by [\s\S] which is equivalent, and it works: (tags:\s*\[[\s\S]*)Paris Web

Can you try this search expression? If it still does not work, then it appears to be related to your OS, which would be rather strange though.

@Simran-B thanks for taking time to search for an answer!
Your suggestion (tags:\s*\[[\s\S]*)Paris Web seems to work indeed.
But after reading my search string once again, I saw that I should have escaped the first closing square bracket: (tags:\s*\[[^\]]*)Paris Web, because I want any character but the closing square bracket. This works.
But your version would also find tags:  [twitter] Paris Web (Paris Web is after the closing square bracket), which I don't want.
It looks however there is an issue: my first search regex without closing square bracket escaping found relevant strings, as the screenshot showed, but $1 was not replaced with the group value it matched.

@nhoizey Ah, now I understand your regular expression. [^] wasn't meant as match-all, but match everything but a literal ]. It works here with your fixed phrase:

If your initial phrase is the only one which leads to $1 not being substituted with the first matched group, then it's a side-effect of an unsupported search expression perhaps? But yeah, still worth to report as separate bug, because if it's actually not supported (using [^] as short hand for [\s\S]) then I would expect to see an error. Something is definitely off with the search panel, whereas the find widget behaves as expected.

@nhoizey I was able to reproduce a problem and reported it here: #63297. It tries to substitute $1 literally although there is a capture group with value array, but it changes to the expected value as soon as you open the Replace Preview. Maybe you saw some similar behavior?
Another related issue, but said to be fixed: #58374

Hi @Simran-B, sorry for the very late answer. It looks like I don't have issues anymore.

The Find Widget input box honors JavaScript regex engine, closing it as the issue here is no longer valid.

@rebornix I disagree.

The documentation does not mention the Find Widget in GET STARTED > User Interface
Instead, it is mentioned under USER GUIDE > Basic Editing # Find and Replace, but it only lists Regular Expression as option under Advanced find and replace options, not saying anything about JavaScript regex engine being used nor providing an example
If you are lucky, you might find the well hidden GET STARTED > Tips and Tricks # Search and Modify paragraph under the IntelliSense headline, which briefly mentions that you can use parentheses and $1 etc. to reuse the content matched. But it also fails to mention that the JS regex engine is used.

IMO, the Search and Modify paragraph should be removed and the content added to USER GUIDE > Basic Editing # Find and Replace instead, plus a sentence added about the regex engine being used.


Check instrumentation store enhancer's reducer and not to be included twice

Related to https://github.com/zalmoxisus/redux-devtools-extension/issues/36

Added another error related to https://github.com/zalmoxisus/redux-devtools-extension/issues/47.

Don't suppose there's a way to have more than one live in harmony. Trying to merge two apps into one both with custom monitors that only make sense to a particular context.
Using two docks seemed like the easy approach, but perhaps it's easier to combine them instead?


ci(git-actions): ci.yml

Coverage remained the same at 84.516% when pulling 1299f250cfc3626e3eab53965d8c7837395f0b4d on ci.yml into 9d5a4fde78c076123ed1a1d733b4d06c44cc591a on master.


User Guide Missing

There are no instructions explaining input file formats, choice of parameters, code execution, or interpretation of results.

Hello Dario,
Thanks for your interest in TRANSACT and for raising an interesting and fair point.
I will refactor documentation and make it available as soon as possible. In the meantime, you can refer to the reproduction code of our manuscript, for example (https://github.com/NKI-CCB/TRANSACT_manuscript), e.g. going from cell lines to PDXs : https://github.com/NKI-CCB/TRANSACT_manuscript/blob/main/figure_2/2C-I_PDX_predictions.ipynb
In case you run into any issue, please do not hesitate to contact me, I will be happy to help out.
Best regards,
Soufiane


Proper strategy for binding a class to

I couldn't see a way to do this even through my layout file; I want to add a class to the <html> tag that does overflow: hidden !important when a modal is present so that I can prevent scrolling in the background.
I figure since I have modals in different components I would use vuex to globally communicate:
my store/index.js
import Vuex from 'vuex'

const store = () => new Vuex.Store({
  state: {
    modal: false,
  },
  mutations: {
    modalOn (state) {
      state.modal = true
      document.getElementsByTagName('html')[0].classList.add('is-clipped')
    },
    modalOff (state) {
      state.modal = false
      document.getElementsByTagName('html')[0].classList.remove('is-clipped')
    },
  },
})
export default store

Then in my component I add a watcher that hits these mutations when the modal is on/off:
  watch: {
    'modal' (value) {
      if (value) {
        this.$store.commit('modalOn')
      } else {
        this.$store.commit('modalOff')
      }
    },
  },

Am I doing it wrong? is there a better approach?

I think you're over complicating it, I'm not sure you really need to use the store for this.
A better approach would probably be a child component that adds the class in its created lifecycle event and removes it on its destroyed lifecycle event. You could probably even house the modal's logic/template within the child component and bind it to a route - that way user could even refresh the page and get back the same modal state (open/closed).

@homerjam well the main point is i want to simplify and globalize this functionality so that all my modals can use this, and i don't really see any other way besides a store.
I don't see a reason to need it w/in routes (yet) but nice suggestion

To add a custom class to your HTML tag, you can use Document.

@alexchopin thanks but I actually not only need to add a custom class but I need to be able to add/remove this class based on wether or not the modal is on

@acidjazz I needed exactly that too (to fix some wonky iOS scrolling) and this did the trick for me
export default {
  head() {
    return {
      bodyAttrs: {
          class: this.modalOpen ? 'm-open' : 'm-close'
      },
      htmlAttrs: {
          class: this.modalOpen ? 'm-open' : 'm-close'
      }
    }
  }
}

On one hand, that's reactive.
On the other hand, it doesn't have all the affordances :class in template has, such as passing {class1: true} or ['class2'], or camelCase.
You can do this in a jiffy though:
export default {
  head() {
    return {
      htmlAttrs: {
        class: (Array.isArray(this.classes)
          ? this.classes
          : Object.entries(this.classes).reduce((acc, entry) => {
              if (entry[1]) acc.push(entry[0])
              return acc
            }, [])
        )
          .join(' ')
          .replace(/([a-z])([A-Z])/g, '$1-$2')
          .toLowerCase()
      }
    }
  }
}


Allow alias list to contain default/derived/provided name.

It can be useful to perform an action like:
class Oxygen(Sensor, aliases=["oxygen", "o2"]):
    pass

for more explicitness in some code bases.

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

Comparison is base (4c061ee) 100.00% compared to head (ea40527) 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #43   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         5
  Lines          290       291    +1
  Branches        61        61
=========================================
+ Hits           290       291    +1

Flag
Coverage Δ

unittests
100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

autoregistry/_registry.py
100.00% <100.00%> (ø)

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


chore(ci): add a job for cargo build --no-default-features

From https://github.com/vityafx/serde-aux/pull/29#issuecomment-1321215039

Perhaps, I need to add more checks in CI.

If you want to go this route, this can be the minimal CI  job that would detect the issue.

Here's how it would look in the older master:
cargo build --no-default-features                                                                                master ✱ ◼
   Compiling serde-aux v4.1.1 (/tmp/serde-aux)
error[E0433]: failed to resolve: use of undeclared crate or module `chrono`
  --> src/field_attributes.rs:60:9
   |
60 |     use chrono::prelude::*;
   |         ^^^^^^ use of undeclared crate or module `chrono`

error[E0433]: failed to resolve: use of undeclared crate or module `chrono`
  --> src/field_attributes.rs:56:13
   |
56 | ) -> Result<chrono::DateTime<chrono::Utc>, D::Error>
   |             ^^^^^^ use of undeclared crate or module `chrono`

error[E0433]: failed to resolve: use of undeclared crate or module `chrono`
  --> src/field_attributes.rs:56:30
   |
56 | ) -> Result<chrono::DateTime<chrono::Utc>, D::Error>
   |                              ^^^^^^ use of undeclared crate or module `chrono`

error[E0433]: failed to resolve: use of undeclared type `DateTime`
  --> src/field_attributes.rs:67:8
   |
67 |     Ok(DateTime::<Utc>::from_utc(
   |        ^^^^^^^^ use of undeclared type `DateTime`

error[E0433]: failed to resolve: use of undeclared type `NaiveDateTime`
  --> src/field_attributes.rs:68:9
   |
68 |         NaiveDateTime::from_timestamp_opt(seconds, nanos)
   |         ^^^^^^^^^^^^^ use of undeclared type `NaiveDateTime`

error[E0412]: cannot find type `Utc` in this scope
  --> src/field_attributes.rs:67:19
   |
54 | pub fn deserialize_datetime_utc_from_milliseconds<'de, D>(
   |                                                         - help: you might be missing a type parameter: `, Utc`
...
67 |     Ok(DateTime::<Utc>::from_utc(
   |                   ^^^ not found in this scope

error[E0425]: cannot find value `Utc` in this scope
  --> src/field_attributes.rs:70:9
   |
70 |         Utc,
   |         ^^^ not found in this scope

Some errors have detailed explanations: E0412, E0425, E0433.
For more information about an error, try `rustc --explain E0412`.
error: could not compile `serde-aux` due to 7 previous errors

Thanks! I thought I had it but it seems in another project :-)


enhancement: clean pdf elements (bump unstructured-inference)

Comment on the .json output for (so far):
test_unstructured_ingest/expected-structured-output/s3-small-batch/small-pdf-set/recalibrating-risk-report.pdf.json

This PR is introducing a bunch of Image tags that are all noise on Page 1. Page 3 looks a little better in that there is now:
    "type": "UncategorizedText",
    "text": "Executive Summary"

which was not there before (though it'd probably be better characterized as a Title).

LGTM


Wrapping useFetch in a function causes hydration mismatch

Environment

Operating System: Windows_NT
Node Version:     v20.11.1
Nuxt Version:     3.12.2
CLI Version:      3.12.0
Nitro Version:    2.9.6
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      components, css, devtools, vite
Runtime Modules:  -
Build Modules:    -

Reproduction

Describe the bug
Since recent update, wrapping useFetch in a custom function causes hydration mismatch.
This doesn't happen if useFetch is in the root of the setup script.
I didn't receive this error before upgrading Nuxt today. Previous version I had was 3.11.2. However I cannot replicate NOT having this mismatch error even after downgrading the packages again. Did something change recently?
Benefit of wrapping useFetch in a custom function was that in isolates the logic, and allows using destructured variables without renaming them.
Additional context
No response
Logs
No response

useFetch is generally used for for async data pre-mounting. Depending on what kind of function you wrapped it in, there is no point in using it if it's after user interaction, as that's already on the client and will have no benefit over the regular $fetch (which is used under the hood)

Something like this was an intention, when you need to use useFetch multiple times in a single file, it makes its usage easier, when the logic is separated in a function..
const state = reactive({
  data1: [],
  data2: [],
});

(() => {
  const { data } = await useFetch("/api/route-1");
  state.data1 = data.value || [];
})()

(() => {
  const { data } = await useFetch("/api/route-2");
  state.data2 = data.value || [];
})()

Anyway, regarding my issue, is there any reason why useFetch cannot be used elsewhere other than top level of the sfc setup? Why is this throwing hydration mismatch warning?

I think i have the same issue with https://github.com/johannschopplich/nuxt-api-party


Installing ext-oci8 fails on PHP 8.3

Describe the bug
When enabling the OCI8 extension on PHP 8.3, Setup PHP fails with exit code 1.
Version

[ ] I have checked releases, and the bug exists in the latest patch version of v1 or v2.
[x] v2
[ ] v1

Runners

[x] GitHub Hosted
[ ] Self Hosted

Operating systems
Ubuntu 22.04
PHP versions
8.3
To Reproduce
https://github.com/doctrine/dbal/blob/195aad31e9596e4093d417583f98342b6f0ec986/.github/workflows/continuous-integration.yml#L137-L143
Expected behavior
PHP is installed with ext-oci8 enabled
Screenshots/Logs
Run shivammathur/setup-php@v2
  with:
    php-version: 8.3
    extensions: oci8
    coverage: pcov
    ini-values: zend.assertions=1
    ini-file: production
  env:
    fail-fast: true
/usr/bin/bash /home/runner/work/_actions/shivammathur/setup-php/v2/src/scripts/run.sh

==> Setup PHP
✓ PHP Installed PHP 8.3.0-dev (null)

==> Setup Extensions
Error: The process '/usr/bin/bash' failed with exit code 1

https://github.com/doctrine/dbal/actions/runs/5760171338/job/15616096049?pr=6118#step:4:33
Additional context
Are you willing to submit a PR?
I wouldn't know how to debug this. 😓

@derrabus
It should work now, please restart your workflow.
I added a check in https://github.com/shivammathur/php-builder/commit/d048151777913d3f5f8b96de691b021937d27993 to prevent this in future.

It does! Thank you very much for your fast response!


Add maintainer to eccodes - maintainer not reachable PEP 541

Hi,
The eccodes PyPi repo (https://pypi.org/project/eccodes/) is limited, out-of-date and the documentation is missing. We tried to contact user "hcsoftcn" - but there are no contact details and we failed to make contact.
ECMWF develops the ecCodes software and we maintain successfully other PyPi repos. We already receive bug/feature reports for this repo, but are not able to fix these. If we could be added as maintainer we will take an active role to support the ecCodes repo and add the missing parts.
Could I please be added as maintainer?
For PEP 541, perhaps we could suggest that every account has to be associated with a valid email address?
Thanks
Stephan

Comment for #1506 - PEP 541 : perhaps we should require valid email addresses for maintainers?


Can't automatically upgrade due to lack of assets/icons

Required Prerequisites for filing a bug
You must check both boxes

[y ] Have you searched opened / closed issues on this repo for a similar problem?
[y] Have you searched the fastai forums for a similar problem?

Required information
Failure to provide this information or going through these steps will result in automatic closing of the issue
Steps to reproduce:

Attempt to update a fastpages repo by making the fastpages template issue.

This results in failure by github actions. With the following errors in "copying new files":
Run # remove files you don't want to copy from current version of fastpages
copying Gemfile
copying Gemfile.lock
copying Makefile
copying _action_files
copying _fastpages_docs
copying _includes
copying _layouts
copying _pages
copying docker-compose.yml
cp: cannot stat 'assets/icons': No such file or directory
##[error]Process completed with exit code 1.

The culprit is the recent removal of the assets/icons directory in favor of the images directory.
Link to the github actions failure: https://github.com/KeAWang/lower-the-entropy/runs/482918372

Thank you for reporting this!  Fixed this with https://github.com/fastai/fastpages/commit/f60c582e371e8db6b54a9c20bd44a9ed621431dc


libuv CMake failure missing Threads

We are getting the failure below when trying to build our project with libuv. It is relaying that we have  links to target "Threads::Threads" but the target was not found.   We have found that if we also add "find_package (Threads)" resolves it. However we noticed at https://github.com/microsoft/vcpkg/blob/master/ports/libuv/CMakeLists.txt#L65 is already there. Is this expected that we need to add in the find package for Threads? We would expect that the PRIVATE keyword would fix this for us. Should we be required to bring in Threads or should this be provided already?
This is how we are building it:
cmake %~dp0 ^
-G "Visual Studio 15 2017" ^
-B %~dp0\builds ^
-D CMAKE_TOOLCHAIN_FILE=%HOMEPATH%\vcpkg\scripts\buildsystems\vcpkg.cmake ^
-D VCPKG_TARGET_TRIPLET=x86-windows-static

This is our CMakelists.txt file:
project (Tutorial)
set(VCPKG_TARGET_ARCHITECTURE x86)
set(VCPKG_CRT_LINKAGE static)
set(VCPKG_LIBRARY_LINKAGE static)
find_package(unofficial-libuv CONFIG REQUIRED)
add_executable(Tutorial Tutorial.cpp)
target_link_libraries(Tutorial PRIVATE unofficial::libuv::libuv)

Our main project "Tutorial.cpp" is just a empty file.
The errors we get when generating with CMake:
-- The CXX compiler identification is MSVC 19.16.27032.1
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Professional/VC/Tools/MSVC/14.16.27023/bin/Hostx86/x86/cl.exe
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Professional/VC/Tools/MSVC/14.16.27023/bin/Hostx86/x86/cl.exe -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Professional/VC/Tools/MSVC/14.16.27023/bin/Hostx86/x86/cl.exe
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Professional/VC/Tools/MSVC/14.16.27023/bin/Hostx86/x86/cl.exe -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Configuring done
CMake Warning (dev) at %HOMEPATH%/vcpkg/scripts/buildsystems/vcpkg.cmake:169 (_add_executable):
  Policy CMP0028 is not set: Double colon in target name means ALIAS or
  IMPORTED target.  Run "cmake --help-policy CMP0028" for policy details.
  Use the cmake_policy command to set the policy and suppress this warning.

  Target "Tutorial" links to target "Threads::Threads" but the target was not
  found.  Perhaps a find_package() call is missing for an IMPORTED target, or
  an ALIAS target is missing?
Call Stack (most recent call first):
  CMakeLists.txt:7 (add_executable)
This warning is for project developers.  Use -Wno-dev to suppress it.

CMake Warning (dev) at %HOMEPATH%/vcpkg/scripts/buildsystems/vcpkg.cmake:169 (_add_executable):
  Policy CMP0028 is not set: Double colon in target name means ALIAS or
  IMPORTED target.  Run "cmake --help-policy CMP0028" for policy details.
  Use the cmake_policy command to set the policy and suppress this warning.

  Target "Tutorial" links to target "Threads::Threads" but the target was not
  found.  Perhaps a find_package() call is missing for an IMPORTED target, or
  an ALIAS target is missing?
Call Stack (most recent call first):
  CMakeLists.txt:7 (add_executable)
This warning is for project developers.  Use -Wno-dev to suppress it.

CMake Warning (dev) at %HOMEPATH%/vcpkg/scripts/buildsystems/vcpkg.cmake:169 (_add_executable):
  Policy CMP0028 is not set: Double colon in target name means ALIAS or
  IMPORTED target.  Run "cmake --help-policy CMP0028" for policy details.
  Use the cmake_policy command to set the policy and suppress this warning.

  Target "Tutorial" links to target "Threads::Threads" but the target was not
  found.  Perhaps a find_package() call is missing for an IMPORTED target, or
  an ALIAS target is missing?
Call Stack (most recent call first):
  CMakeLists.txt:7 (add_executable)
This warning is for project developers.  Use -Wno-dev to suppress it.

CMake Warning (dev) at %HOMEPATH%/vcpkg/scripts/buildsystems/vcpkg.cmake:169 (_add_executable):
  Policy CMP0028 is not set: Double colon in target name means ALIAS or
  IMPORTED target.  Run "cmake --help-policy CMP0028" for policy details.
  Use the cmake_policy command to set the policy and suppress this warning.

  Target "Tutorial" links to target "Threads::Threads" but the target was not
  found.  Perhaps a find_package() call is missing for an IMPORTED target, or
  an ALIAS target is missing?
Call Stack (most recent call first):
  CMakeLists.txt:7 (add_executable)
This warning is for project developers.  Use -Wno-dev to suppress it.

CMake Warning (dev) at %HOMEPATH%/vcpkg/scripts/buildsystems/vcpkg.cmake:169 (_add_executable):
  Policy CMP0028 is not set: Double colon in target name means ALIAS or
  IMPORTED target.  Run "cmake --help-policy CMP0028" for policy details.
  Use the cmake_policy command to set the policy and suppress this warning.

  Target "Tutorial" links to target "Threads::Threads" but the target was not
  found.  Perhaps a find_package() call is missing for an IMPORTED target, or
  an ALIAS target is missing?
Call Stack (most recent call first):
  CMakeLists.txt:7 (add_executable)
This warning is for project developers.  Use -Wno-dev to suppress it.

-- Generating done
-- Build files have been written to: %HOMEPATH%/test/builds

@erikchris2011, thanks for posting this issue.
The target "Threads::Threads" not found in your project looks due to condition  'if("@VCPKG_LIBRARY_LINKAGE@" STREQUAL "static" AND NOT WIN32)' in xxxcmake.config file, see https://github.com/microsoft/vcpkg/blob/master/ports/libuv/unofficial-libuv-config.in.cmake#L2.

@erikchris2011, currently this issue has been fixed, please get latest source and try again.


Persist stack traces and display in UI

This PR makes sure that when runs fail, the stack trace is captured and persisted in the DB. It is then displayed in the Logs tab in the UI.

Codecov Report

Merging #71 (141a345) into main (a41641d) will increase coverage by 0.57%.
The diff coverage is 67.90%.

:exclamation: Current head 141a345 differs from pull request most recent head 22c5589. Consider uploading reports for the commit 22c5589 to get more accurate results

@@            Coverage Diff             @@
##             main      #71      +/-   ##
==========================================
+ Coverage   52.14%   52.71%   +0.57%
==========================================
  Files          86       86
  Lines        6914     6943      +29
==========================================
+ Hits         3605     3660      +55
+ Misses       3309     3283      -26

Impacted Files
Coverage Δ

sematic/db/models/run.py
11.11% <0.00%> (+0.49%)
:arrow_up:

sematic/resolvers/cloud_resolver.py
21.68% <16.66%> (-0.04%)
:arrow_down:

sematic/resolvers/state_machine_resolver.py
42.10% <50.00%> (-0.15%)
:arrow_down:

sematic/resolvers/local_resolver.py
82.03% <75.00%> (-0.53%)
:arrow_down:

sematic/resolvers/tests/test_local_resolver.py
75.00% <83.33%> (+4.12%)
:arrow_up:

sematic/resolvers/worker.py
46.72% <87.50%> (+18.14%)
:arrow_up:

sematic/resolvers/tests/test_worker.py
51.72% <90.90%> (+23.94%)
:arrow_up:

Continue to review full report at Codecov.

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


minor touch-ups

the "usual" node.js .gitignore
add jshints
report only sensors (let the thermostat report itself)

enjoy!

Good morning @mrose17,
Thanks for the touch-ups. I like your changes a lot! It's great to have another pair of :eyes: go over the code, I'm quite ignorant developer :see_no_evil: :smiley:
I have only one question about excluding the thermostat itself. What do you think about making it a config.json option? There's more details in the code review comments.

hi. i couldn't find the comments, but i agree that's a better option. the default can remain as you initially coded it, and if the option is set, then only sensors need be considered. how does that sound?

Sounds good!
PS: Sorry, I forgot to submit the review comment... :roll_eyes:

Can you reveal the Thermostat to HomeKit please! I unfortunately have an Ecobee 3 before they rereleased it with HomeKit Support...

Hello @AppleTechy,
Thanks for your comment. Good news :slightly_smiling_face: is that's it's certainly possible to get HomeKit unofficially working with earlier Ecobees, but the bad news :frowning_face: is that it's not currently implemented...
I opened a new issue #7 for you. If you know of any other people or even an online discussion about this problem, please try to get as much support for this as possible.
I don't know how experienced are you with software development, so maybe you can tackle it yourself. It's a challenge, but it's doable since a lot of the boring authorization boilerplate code is done. I wrote some implementation notes in a comment there, while I still remember it... :smiley:

@mrose17 Merged! Thanks a lot for your very nice improvements. I like the new "exclude_thermostat" option a lot.

enjoy!


Support for Jedis cluster

Codecov Report

Merging #194 into master will increase coverage by 0.33%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #194      +/-   ##
============================================
+ Coverage     85.03%   85.37%   +0.33%
- Complexity      251      252       +1
============================================
  Files            54       54
  Lines           989     1005      +16
  Branches         59       58       -1
============================================
+ Hits            841      858      +17
  Misses          126      126
+ Partials         22       21       -1

Impacted Files
Coverage Δ
Complexity Δ

...edlock/provider/redis/jedis/JedisLockProvider.java
91.83% <100%> (+6.98%)
7 <1> (+1)
:arrow_up:

Continue to review full report at Codecov.

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


Remove irrelevant test log of temporary feature branch from branch report

descrition: remove file pdm_test_2024-03-03_feature-update-transitive-deps.txt from /Fundsoft/test_logs since we now have a test output log of the latest version of develop branch.
definition of done: new branch made from report, file removed, branch merged back to report.

closed by PR #56


2.0 Tag helper components, head and body tag helpers

@pkellner would you be interested in this one?

Rough outline:

What are Tag Helper Components?
Benefits of Tag Helper Components
Built-in Tag Helper Components [head and body]
Tag Helper Components compared to Tag Helpers
Authoring Tag Helper Components

@Rick-Anderson Yes, not in the next month or two. I'm pretty over committed right now. I'd love to read what @rachelappel does though.  Even better. :)

@rachelappel Is this is a duplicate of https://github.com/aspnet/Docs/issues/4033?

@scottaddie It looks like some of it is. We can merge these.

duplicate of #4033


Structures for CHEBI:132123 and CHEBI:132124

Hi ChEBI
would you please consider adding the attached structures for
CHEBI:132123 1,2-benzoquinones
CHEBI:132124 1,4-benzoquinones
These are mol files saved with a txt extension to allow attachment here.
These structures would allow us to create balanced Rhea reactions representing these classes of molecules, as well as "child" reactions representing actual structures
CHEBI_132124.txt
CHEBI_132123.txt
.
Thanks and Best Regards, Alan

Now done.
Cheers,
Gareth


Set identityProvider as URL type

Motivation
I was having issues sometimes with the VC flow getting stuck.
The problem was that the identity provider I was setting in the RP had a trailing slash, while the origin of the event didn't. Therefore, the comparison of whether the message came from the expected source failed and the library never sent the credential request.
Changes

Set the type of identityProvider as URL as well as converting the event.origin to URL before comparing them both.

Tests

Remove a test with an invalid identityProvider URL and add a test with a URL with a trailing slash.

Todos

[x] Add entry to changelog (if necessary).

@frederikrothenberger please review


Inject credentials via environment variables

Credentials can't be injected via project properties into settings scripts.
In order to access the credentials here, we're now injecting them directly
as environment variables.

@bot-gradle test and merge

OK, I've already triggered a build for you.


fix(lightspeed): update lightspeed logo

Update lightspeed logo to match the brand guideline.
Before:

After:
In Light mode:

In Dark mode:

Full color logo in chat messages container:

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or Updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)

@karthikjeeyar @yangcao77 I think you'd also need a changeset for this change.


Add copilot-workspace extension

Description
Quick access to Copilot Workspace, GitHub's first AI-native developer tool for completing everyday tasks (technical preview launching soon).
Screencast

https://share.cleanshot.com/TR5BtbYw
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

@peduarte can you confirm that we're all set to publish this on Monday (April 29)?


Particle system broken on Samsung Galaxy S10

We are currently experiencing that the PlayCanvas particle system behaves incorrectly on specific Samsung devices, namely Galaxy S10, Galaxy S10+ and Galaxy S10e with Android 11 and Chrome 93.0.4577.62
The corresponding Forum post can be found here (including videos and a link to our test project):
https://forum.playcanvas.com/t/particle-system-broken-on-samsung-galaxy-s10/21787

Possibly related to https://github.com/playcanvas/engine/issues/3415

FYI: Cannot reproduce on a Galaxy Fold 3

I couldn't reproduce this on the Browserstack S10+
Wondering if it's specific to the Exynos CPU and not the Snapdragon variant.
@CMRDX Are you devices the Snapdragon or the Exynos variant?

The device seen in the Video (S10e), as well as the S10+ is featuring a Exynos 9820 2.73GHz.

That may explain why my Fold 3 doesn't have the issue, it's using a Snapdragon 🤔

I've tried it on a Samsung G10 with the same CPU and I am getting the same issue on the project provided on the forum.

@CMRDX or @leonidaspir Can you try running this with WebGL 1 forced, ie: https://launch.playcanvas.com/1225034?debug=true&webgl1=true

Works as expected when running with WebGL1.

@yaustar We got the same result as @leonidaspir. Seems to work as expected when enforcing WebGL 1.

Thanks, that helps with extra information/debugging.
We are working on getting a device to replicate the issue.

Tried on multiple Exynos CPUs https://github.com/playcanvas/engine/issues/3415#issuecomment-921687853
No joy in reproducing the issue :(

I'm taking the opportunity to report how the particles show up on a Samsung Galaxy S10 I got my hands on.
https://playcanvas.github.io/#/graphics/particles-snow

https://playcanvas.github.io/#/graphics/particles-spark

Reported not rendering correctly on RedMi 9 Pro device: https://forum.playcanvas.com/t/dat-cube-my-first-game-with-playcanvas-underdev/23757/7?u=yaustar

Reported not rendering correctly on Xiaomi Note 8 Pro https://forum.playcanvas.com/t/i-have-particlesystem-mobile-problem/23531/3?u=yaustar

As we think this is a hardware driver issue, we've created a workaround that forces using the CPU on ARM based Samsung devices: https://github.com/playcanvas/engine/pull/4147
I expect this to go out with engine 1.53

Looks like this isn't limited to Exynos CPUs but it's related to the GPU used which is Mali based.
See thread: https://forum.playcanvas.com/t/particles-not-rendered-on-amazon-fire-tablet-hd-8/30660

Closing this as this is no longer reproducible. See PR https://github.com/playcanvas/engine/pull/5255/


Custom Filter with Backref

class FooAdminView(sqla.ModelView):
    column_filters = [
        FilterEqual(Foo.bar, 'Bar')
    ]

OneToMany(Bar, Foo). So Bar can have many Foos. Foo has a .bar (via backref), but I'm seeing the following error:
AttributeError: Neither 'InstrumentedAttribute' object nor 'Comparator' object associated with Foo.bar has an attribute 'table'
What do I do?

You can't filter on "model" - you have to filter on a property of a model. Try using Bar.some_field instead. If it is related model, Flask-Admin will figure out JOIN.


Qwik Server makes endless API-Requests

Qwik Version
0.0.34
Operating System (or Browser)
Mac Chrome
Node Version (if applicable)
16
Which component is affected?
Qwik Runtime
Expected Behaviour

When a wrong API-Endpoint is mistakenly used, SSR fails because the Endpoint cannot be reached.

Actual Behaviour

A previous request that was successful is executed endless times until the server process gets terminated.

https://user-images.githubusercontent.com/444278/178786890-7148733c-5e3e-4d9d-a994-06b5540b914b.mp4
Additional Information
Reproduction

You can refere to this video

git clone https://github.com/GregOnNet/kanban-board
cd kanban-board
npm install

# Console 1
npm run app:start

# Console 2
npm run api:start

# Open kanban-board in your editor
# Open list.service.ts
# Change the port from `3030` to `3000`
# Open browser at http://localhost:3000
# Refer to Console 2 GET /board 200 is executed over and over again

Can you update to 0.0.36 and check if this is still a problem?

Solved in 0.0.36.
There are also comprehensive Error-Messages: Thanks a lot


Implement PUSHDATA

Minimal PUSHDATA ops are generated automatically when we push data with @

nm don't think I did this yet.


Multipart response parsing

We support multipart response content; however, we don't provide a parser for it. See ParsersSpec.groovy for multipart response content handling. The main reason is that this is not supported by browsers and would only be for APIs - there is little existing library support documented about this. We should be able to bring in JavaMail and use its multipart API to create a parser for multipart response content.

Since this is an edge case, and the implementation is so simple, I am just going to leave it as an example and test case, literally, this is what it takes to use JavaMail:
new MimeMultipart(new ByteArrayDataSource(fs.inputStream.bytes, fs.contentType))

If there is an expressed need for this, we can add a pre-built decoder from it later.


Prepare a scratch/distroless release image for the VictoriaMetrics operator

It would be helpful to have those types of releases for the VictoriaMetrics operator

Scratch images are added in https://github.com/VictoriaMetrics/operator/pull/974, will be available since v0.46.0.

Operator uses distroless image since v0.46.0


nixos: nix snowflake logo for the nixos manual launcher

The existing nixos-artwork derivation is pretty specific to that gnome wallpaper, and the Makefile in the artwork repo only builds the icons, so I figured having this one be separate would be simpler than combining them.

Note: this depends on https://github.com/NixOS/nixos-artwork/pull/19

Hmmm how can I test this?

ah! The nixos-artwork PR this depends on has been merged, so you shouldn't have to do anything special.  Just do a nixos-rebuild with this patch and you should get the new icon.


New version - first loading of Rhino widget option I got an error only for 1 sec and then it's gone

As design, messages disappear after 5 seconds.

As design, messages disappear after 5 seconds.


Add runtime parameter tensor_diffusion.use_gauss_seidel

The default is true. One can use Jacobi smoothers for tensor, mac and nodal solves with
tensor.use_gauss_seidel = false
mac_proj.use_gauss_seidel = false
nodal_proj.use_gauss_seidel = false

@siddanib With https://github.com/AMReX-Fluids/AMReX-Hydro/pull/143 and this PR, you should be able to Jacobi smoothers for tensor, mac and nodal solves.

Thanks @WeiqunZhang !!

Thank you very much, @WeiqunZhang !!


Bump version to mongoid 5.0.0.beta

Bump version to the last mongoid release 5.0.0.beta

Really appreciate this pull request! I'm planning to hold off updating the gem for Mongoid 5 until after the final release is out, however. Thanks though!

+1
Mongoid v5 has been released: https://github.com/mongodb/mongoid/tree/v5.0.1 @thetron

Thanks again for this PR, I merged an update a while ago that brought the library up to Mongood 5.0.0. I'll close this now, but feel free to comment or reopen if there's any concerns 😀


Set concurrency to 1 for publish artifacts

Temporary set concurrency to one while pushing artifacts to avoid
hitting a concurrency issue on luet-cosign
Signed-off-by: Itxaka<EMAIL_ADDRESS>

should not affect PR workflows


vollkorn: init at 4.105

Motivation for this change
adding a popular Serif font. This addition should have a low maintenance burden due to low complexity and slow release schedule.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 117096 at 430676da run on aarch64-linux 1

1 package built successfully:

vollkorn

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/data/fonts/vollkorn/default.nix:15:3:
   |
15 |   installPhase = ''
   |   ^


CSS for Compare tool orientation

We need guidance (well specifically @shoeswat ) to be able to fix the legend on the holos instance of the Compare tool. Let us know the best way to interact on this-- may be a short call or email, whatever is easiest. See below:

Holos version of the Compare's legend does not size properly. The tool is unusable as is so needs to be fixed prior to Tuesday 5/5.
Here's what it should look like (taken from Stamen github site):

So enlarge the search elements so we can see their color swatches, name and drag/drop handles.

@mkoo Id be happy to help troubleshoot this. How can I get to a place where I can see it in this state myself.

You can see the problem if you launchholos.berkeley.edu/compareand the
githib pages stamen page
Try adding a layer. The entire dragdrop item doesnt show in ours.
@mkoo https://github.com/mkoo Id be happy to help troubleshoot this. How
can I get to a place where I can see it in this state myself.
—
Reply to this email directly or view it on GitHub
https://github.com/stamen/ecoengine/issues/156#issuecomment-98005465.

Ex. Taricha torosa
You can see the problem if you launchholos.berkeley.edu/compareand the
githib pages stamen page
Try adding a layer. The entire dragdrop item doesnt show in ours.
@mkoo https://github.com/mkoo Id be happy to help troubleshoot this. How
can I get to a place where I can see it in this state myself.
—
Reply to this email directly or view it on GitHub
https://github.com/stamen/ecoengine/issues/156#issuecomment-98005465.

I just committed a min-height to the scss file in the master branch. It seems like there are cases where no label will be set. This will make sure there is still a placeholder if there is no label

@shoeswat for quick reference, you can see that change here: https://github.com/stamen/ecoengine-compare/commit/6712c40276d1838fb909a4d21cc0f9f7871d57dc#diff-b1ccd74920d712d955af577292ddcca5R168

@standardpixel still doesn't fix the problem completely. Search Taricha torosa on dev-holos. The draggable box waits until everything is loaded. I think replacing the line to
height: auto;
Should work but I'm working on getting my local copy up for testing.

i overrode the compiled css with another style sheet (not ideal) and resolved some class conflicts. thanks @postfalk for the help


Install into dependencies or devDepenendies?

npm i lit

as currently in docs for installing this into dependencies of package.json.
or
npm i -D lit

which will install it into devDependencies?
I'm new in lit and just wonder, if this is a runtime library or compiling library. If this is only a library required during development, it should be installed with -D into devDependencies. If this library is included in the build and run in browser then keep it as it is.

If you're making components with Lit, it is meant to be used in runtime so it is a regular dependency, not a dev dependency. However, the size is very small as it takes advantage of existing platform features.
Lit is meant to work without any compile or build step. It's published as JavaScript modules to work directly in modern browsers. Of course, users can choose to use TypeScript with a compiler or other build tools for the benefits they bring.
There is a @lit-labs/compiler package but it's optional and only meant to offload initial template preparation step. It'll still require Lit client code for updates.


Performing an in-depth data analysis and visualisation as well as using more models to it

Prerequisites

[X] I am an Open Source Contributor
[X] I checked to make sure that this issue has not already been filed

Description
Good Evening,
I am a GSSoC'23 contributor
I am willing to perform an in-depth pre-processing and EDA on the dataset, as well as adding more visualisations in order to make the analysis more easy to understand and more effective.
Furthermore, I will use more models like XGBoost, SVM, RandomForest in order to improve the accuracy.
Please assign it to me.
Screenshots
No response
Code of Conduct

[X] I follow Contributing Guidelines of this project.

Hello Sir, please assign this issue to me.

@devanshidevanshidevanshi Assigning you


Update ngx_http_static_module.c

NGX_HTTP_POST is a invalid method, and should return NGX_HTTP_NOT_ALLOWED at line 63 not at line 208

Please note that this repository is unofficial and pull requests have no chances to be merged. Proper way to submit changes to nginx is via the nginx development mailing list, see http://nginx.org/en/docs/contributing_changes.html.


selectAll in terminal accessible buffer

fix #174078

this will be fixed via the monaco editor approach

this is actually still needed for terminals without shell integration, but seems like a lot of changes just to make this work. let's see if it's reported as an issue


matching and pipelining

Added positive matching (-M & -m)
Fixed negative matching (-X & -x)
Made -n an option rather than just on
When no matching required by passed the hash table, so very large filesets can be processed without running out of RAM.

If you could take a look and pull this back into your master that would be splendid.
kind regards,
Chris


JavaScript formatter doesn't respect html.format.wrapLineLength

I understand Code is using js-beautify for both HTML and JS/TS formatting.
By setting "html.format.wrapLineLength": 96 I can get HTML tags to wrap at ~96 chars. This seems to be equivalent to html-beautify --wrap-line-length 96 on the command line.
But for JavaScript files, the formatter doesn't wrap lines.
On the command-line if I run js-beautify --wrap-line-length 96 I get the behaviour I expect - the lines wrap. In Code, the lines don't wrap.
Sample input:
function hasTooManyParams(blah, blam, oneTwoThreeFour, fiveSixSevenEight, nineTenElevenTwelve, thirteenFourteenFifteen, eighteenTwelve) {
    var a = 1; var b = 'shouldbeonextline';
    if (a) {
     misalignedOne();
       properlyAlignedTwo();
    }
}

Output of js-beautify --wrap-line-length 96:
function hasTooManyParams(blah, blam, oneTwoThreeFour, fiveSixSevenEight, nineTenElevenTwelve,
    thirteenFourteenFifteen, eighteenTwelve) {
    var a = 1;
    var b = 'shouldbeonextline';
    if (a) {
        misalignedOne();
        properlyAlignedTwo();
    }
}

Actual output from Code formatter:
// First line is not wrapped
function hasTooManyParams(blah, blam, oneTwoThreeFour, fiveSixSevenEight, nineTenElevenTwelve, thirteenFourteenFifteen, eighteenTwelve) {
    // Variable assignments aren't split onto separate lines
    var a = 1; var b = 'shouldbeonextline';
    if (a) {
        // alignment is fixed, as expected
        misalignedOne();
        properlyAlignedTwo();
    }
}

VSCode Version: 1.16.0
OS Version: Mac OS X 10.12.6

Reproduces without extensions: Yes

We don't use js-beautify for our built in formatting. Our built-in formatter also does not have a line length setting. For that, an extension such as eslint or js-beautfy


Add new discordant read calls version

Updating Ag3 to use the new analysis version (20230911) of the discordant read calls.
Similar update for Af1  for consistency with an empty version (although we currently do not have any released discordant read calls for Af).
Note I'm suggesting to specify the version directly on the class (rather than using the config file) as, this version supersedes the previous one in Ag3 (and includes the previous sites) and to avoid including a default parameter on the Af1 config (as there are no discordant read calls yet).

Thanks @alimanfoo! Will put this one on hold until we catch-up tomorrow during the data meeting.

Thanks @alimanfoo! We spotted this issue with test_allele_frequencies_with_region before in different regions in X.
tests/anoph/test_snp_frq.py::test_allele_frequencies_with_region[af1_sim] - ValueError: The genomic region 'X:117,109-117,609' is invalid for contig 'X' with length 117228.

I am trying to debug why this is happening, but it isn't consistent so far (i.e. doesn't seem to be related to specific regions or contigs/not failing consistently).
We can re-run this test to pick another region but trying to see if I can pick the mistake , @leehart, do you happen to have any ideas?

@ahernank The error message seems to be saying that contig X is only 117,228 positions long, so it can't give a region that goes up to position 117,609.
I don't know which is right: Does X have 117,228 positions or 117,609 positions? Somewhere there seems to be a disagreement, and I suspect the tests need to be modified to align with reality. Maybe the number of positions available has changed since the test was written, or maybe it's based on some other variable, I'm not sure.

Thanks @leehart. This was my same thought but the (full) X contig is much longer (24,393,108). This test doesn't seem to use that value, but rather a simulated one.
Which I don't quite understand is why this simulated value changes every time (not in line with the region per se), so sometimes the same test on the same region fails or passes.

Hi @ahernank . I just had a chat with Alistair and it looks like we just need to fix the random_region_str function so that it takes the contig_size into account. Basically, it's coming up with a random 500 position region and not being smart enough to make sure that doesn't go over the end of the synthetic contig.
https://github.com/malariagen/malariagen-data-python/blob/18bf2f354fdc22982ec234927d363d7fe66fba48/tests/anoph/conftest.py#L1014
Do you want me to try to fix this on this branch, or another branch, or do you want to try?

Plan to fix random_region_str on a different PR.
Checks now pass due to random nature of tests, i.e. random test region falls within synthetic contig.


The "Something is already running on .." error is incorrect

To Reproduce
Steps to reproduce the behavior:

First, try running OpenRefine on an invalid interface: ./refine -i gjksjd -p 333sd4

Current Results
The "Something is already running on ..." will be displayed even although noting is running there
Expected Behavior
The check running script is only checking if OpenRefine is running on the given interface, not if "something" is running, the error message should therefore, not say that something is already running there but rather give a more generic error. "OpenRefine isn't running on $URL."?
https://github.com/OpenRefine/OpenRefine/blob/ea017a7ff614aea7d0220e6230c6db18b6322622/refine#L143

Can I work on this issue ?

Hi, @wetneb created a PR. Please have a look.


[WIP] Configure and start the failover monitor from the core app

We need the PglogicalSubscription model to properly remove, add, and query for subscriptions. Pushing this into the application makes this much easier.
This PR also adds the manageiq-postgres_ha_admin gem to the core Gemfile.
Depends on https://github.com/ManageIQ/manageiq-postgres_ha_admin/pull/10
https://bugzilla.redhat.com/show_bug.cgi?id=1391095

@jrafanie @gtanzillo this is ready for review. I'm going to release the new version of the ha admin gem later today and then I'll un-wip.
My only open question is if we should also start/restart this service when the MiqServer process starts or if we should continue as we are now and have users start it directly from the console.
I think this wouldn't cause a problem if we start it at server start, but I don't know how much more churn we want to introduce with this change. If we go this way I think we should also bring down the service when the server stops so we don't start up evmserverd if the database is not available when the user thinks they shut down the server.
What do you guys think?

Ah, sorry, I missed that it was updated with the tagged release for ha admin.  LGTM.


problem: error in onData callback is not handled

one test failed with timeout, it has been already fixed in other pr


chore(Container)!: Remove Container

Overview

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: ABC-123
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change
[ ] This PR includes testing instructions tests for the code change
[ ] The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

Testing Instructions

Don't make me tap the sign.
PR Links and Envs

Repository
PR Link
PR Env

Monolith
Monolith PR
Monolith Env

Portal
Portal Link
Portal Env

Another Repo
Another Link
Another Env

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.179-alpha.80882b.0@codecademy/gamut-labs@33.0.15-alpha.80882b.0@codecademy/gamut@50.2.1-alpha.80882b.0@codecademy/monaco@0.6.72-alpha.80882b.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://62faa1277f883f0086e3e55e--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.194-alpha.845915.0@codecademy/gamut-labs@33.2.2-alpha.845915.0@codecademy/gamut@50.6.1-alpha.845915.0@codecademy/monaco@0.6.83-alpha.845915.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://630e464544eb2910463e478a--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://632214df0c360b3b0b56d0dd--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.205-alpha.426b34.0@codecademy/gamut-labs@33.5.1-alpha.426b34.0@codecademy/gamut@50.6.6-alpha.426b34.0@codecademy/monaco@0.6.88-alpha.426b34.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://632a091395aa6600663b3425--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.206-alpha.4be962.0@codecademy/gamut-labs@33.5.2-alpha.4be962.0@codecademy/gamut@50.6.7-alpha.4be962.0@codecademy/monaco@0.6.89-alpha.4be962.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://632dde88746bfd2f93083bee--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.208-alpha.6d2c3e.0@codecademy/gamut-labs@33.5.4-alpha.6d2c3e.0@codecademy/gamut@50.6.7-alpha.6d2c3e.0@codecademy/monaco@0.6.89-alpha.6d2c3e.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://63373f898add1e093eba9e97--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.209-alpha.149a67.0@codecademy/gamut-labs@33.5.5-alpha.149a67.0@codecademy/gamut@50.7.1-alpha.149a67.0@codecademy/monaco@0.6.90-alpha.149a67.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://633b3f121912192e223f8cf6--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.214-alpha.b5543f.0@codecademy/gamut-labs@33.6.5-alpha.b5543f.0@codecademy/gamut@50.9.1-alpha.b5543f.0@codecademy/monaco@0.6.94-alpha.b5543f.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6349bab7f98643049d6dc47e--gamut-preview.netlify.appDeploy Logs


QR code not scannable by Nintendo 3DS

The generated QR code is not identified by the Nintendo 3DS (tested on a O2DS) but if I generate the code for the same URL on this site, the QR code works.
The phone scans both codes, but not sure why the one generated by the program does not work on the handheld.

Hi, thanks for reporting.
Can you please try scanning this?

Hey, unfortunately it didn't work either.
Nintendo shows this QR code as an example for testing, if it helps (and it works):

Got it from here.
Thanks for sharing the software. Awesome idea!

Ok thanks, I will research this soon!
Il gio 22 mar 2018, 22:18 José Júnior<EMAIL_ADDRESS>ha scritto:

Hey, unfortunately it didn't work either.
Nintendo shows this QR code as an example for testing, if it helps (and it
works):
[image: image]
https://user-images.githubusercontent.com/228200/37798879-ca253a80-2dfc-11e8-98b3-4fdac0b582e8.png
Got it from here
http://en-americas-support.nintendo.com/app/answers/detail/a_id/371/~/error-message%3A-this-qr-code-is-not-supported
.
Thanks for sharing the software. Awesome idea!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/claudiodangelis/qr-filetransfer/issues/6#issuecomment-375460602,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA5fizIJDqIjKsTC5eqehThLeCyyqpZPks5thBUKgaJpZM4S3qf8
.

--
Claudio d'Angelis
<EMAIL_ADDRESS>Skype: claudiodangelis
Phone: (+39)<PHONE_NUMBER>

@jjnilton can you please try to scan these two codes and tell me which one works (level H, M or neither one):
Level H

Level M

Hey, @claudiodangelis, neither one worked. Same thing as the one before.

😔

@claudiodangelis would the color interfere? I tried one QR code generated on https://www.nayuki.io/page/qr-code-generator-library and it worked. I used the default settings and since it's open source, maybe it helps to find the solution.

FWIW: I have a 3DS with 11.2.0-35E. The images in this issue did not work but generating a new one did.

Hi @BakeRolls, do you mean: you have successfully generated an image with qr-filetransfer and it worked?
If so, this is a really great news :-)

@BakeRolls your code worked with my 2DS on 11.6.0-0-39U. And I tried again with the newest version of the qr-filetransfer and it's working. :smile:
Thanks @claudiodangelis!


Side menu with back views fix. New attribute enable-menu-swipe-with-b…

Short description of what this resolves:
Fixes bug when user swipes from edge with "enable-menu-with-back-views" enabled.
Now if  "enable-menu-with-back-views" is enabled and has some back views, side menu will be opened only if it starts dragging from edge with 45px offset (in the middle of content).
Added new attribute "enable-menu-swipe-with-back-views" to "ion-side-menus" directive which enables just swipe to open menu and button is still hidden with back views.
Ionic Version: 1.x

Please hold on with merging.
I found a new bug with slide to close the menu.


Expand copy-tabs functionality

Rebasing from github set the author dates to Aug 19 when they were supposed to be Jun 13. I've had to forcibly remove those 3 commits and manually do a git rebase --committer-date-is-author-date 1-update-copy-tabs from my terminal.


Try to reproduce issue2142 on all clusters

Description

Motivation and Context
The problem https://github.com/networkservicemesh/networkservicemesh/issues/2142 is not reproduced on kind clusters, we need to check can the problem be reproduced on all our clusters.
How Has This Been Tested?

[ ] Covered by existing integration testing
[ ] Added integration testing to cover
[ ] Tested locally
[ ] Have not tested

Types of changes

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

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Looks like problem is solved, fixed has been moved to https://github.com/networkservicemesh/networkservicemesh/pull/2168


[🐛 BUG] Lambda function not working in delta property of metric

What went wrong? 🤔
Having a lambda function inside the delta property does not work. It would be really useful for it to work. Maybe, other properties are impacted.

WARNING:root:
--- 1 warning(s) were found for page '/'  ---
 - Warning 1: Metric: delta cannot be transformed into a number:
could not convert string to float: '<lambda>'
----------------------------------------------

Expected Behavior
It should work; the delta property should be able to use the lambda function to get the proper delta displayed on the metric. Maybe, other properties are impacted.
Steps to Reproduce Issue
# Import necessary libraries
import pandas as pd
import taipy as tp

import datetime as dt
import taipy.gui.builder as tgb

value = 10
previous_value = 5

with tgb.Page() as page:
    # Don't work
    tgb.metric(lambda value: value if value else 0,
        delta=lambda value, previous_value: (value - previous_value)
                                                if value and previous_value else 0,
        max=20,
        title="Comparison",
        height="250px")

    # Works
    tgb.metric(lambda value: value if value else 0,
            delta="{(value - previous_value) if value and previous_value else 0}",
            max=20,
            title="Comparison",
            height="250px")
# Main execution block with GUI setup
if __name__ == "__main__":
    tp.Gui(page).run(title="Bug replication", port=3248)

Browsers
Chrome
OS
Windows
Version of Taipy
Develop - 7/15/24
Acceptance Criteria

[ ] Ensure new code is unit tested, and check code coverage is at least 90%.
[ ] Create related issue in taipy-doc for documentation and Release Notes.

Code of Conduct

[X] I have checked the existing issues.
[ ] I am willing to work on this issue (optional)

This is the same issue as this one: https://github.com/Avaiga/taipy/issues/1518
I will close this issue.


DeleteDirectory: Directory not empty

FTP OS: Unknown, third party.
FTP Server: Unknown, third party.
I upgraded from version 19.1.3 to 29.0.4 and I am having a problem deleting a directory that has folders and files in it. There is nothing in the documentation about this. Is there something I am doing wrong?
Code:

Directory:

Logs :
2020-01-31 07:12:36,656 | ERROR | /27-01-2020/db: Directory not empty | UploadFolderBackupFtp method in file BackupService.cs at line 335
FluentFTP.FtpCommandException: /27-01-2020/db: Directory not empty
   at FluentFTP.FtpClient.DeleteDirInternal(String path, Boolean deleteContents, FtpListOption options)
   at FluentFTP.FtpClient.DeleteDirInternal(String path, Boolean deleteContents, FtpListOption options)
   at FluentFTP.FtpClient.DeleteDirectory(String path)
   at WindowsService.Backup2.BackupService.UploadFolderBackupFtp

Paste the full FTP logs here so we can debug. See https://github.com/robinrodricks/FluentFTP/wiki/Logging#how-do-i-trace-ftp-commands-for-debugging

Paste the full FTP logs here so we can debug. See https://github.com/robinrodricks/FluentFTP/wiki/Logging#how-do-i-trace-ftp-commands-for-debugging

Unless you reply we cannot help you.

Sorry noted. I reverted back to 19.1.3 in the end.

If you can paste the FluentFTP logs of  19.1.3 and of the latest version, then we can compare and fix this behavior.


Event loop ev / event installation

Hello,
I am looking for some information regarding ev/event installation. I have posted this question in Stackoverflow but I did not get any answer so posting here.
I have laravel-websockets installation on Ubuntu OS. Inspired from the information under https://docs.beyondco.de/laravel-websockets/1.0/faq/deploying.html#changing-the-event-loop, I have installed ev and PHP is loading the extension successfully.
AFAIK the default event loop is stream_select for the package. My questions are,

Do I need to perform any other actions in order to get ev activated?
Is there a way to check which event loop the package uses?
Is there a hard limit for ev?

Thanks in advance!

Maybe this will help you.
Since it uses Ratchet this document explains how it works and what you need to do.
http://socketo.me/docs/deploy


add support for open("about:blank")

Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Checklist

[x] Checkstyle and unit tests pass locally with my changes by running gradle check chrome htmlunit command
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)

tagging https://github.com/selenide/selenide/issues/914


Upload Bug: Fore (EP) by Hungry Jack

Album uploaded but track list does not show ...
Click on Play Song and get stuck in a prompt loop that reads:
"No tracks available to play."
Here's a screen shot with the console logs from devtools:

I have a suspicion that it is that I uploaded a file format that is not supported onto the ipfs. Could taht be a thing?

yes! 🐞
I might just have to add it to the acceptable files whitelist. let us know what it was 🙏

Its .wma 💚 thank you

⚸ ✅
added the '.wma' file type - to 3 different sections - in three different ways 😅
#pilotPlAiyngWithFire
for V1.3 another coder and I will be meticulously shaving away the redundancies and long-form solutions used to get the base case scenarios hashed out for initial takeoff ...
relevant lines:
setupUi.js
UpdateRightLCD()
const validAudioExtensions = [".mp3", ".wav", ".m4a", ".mp4", ".aac", ".mov", ".wma", ".mkv"];
playback.js
playSong()
if (fileExtension === "mp4" || fileExtension === "mov" || fileExtension === "wma" || fileExtension === "mkv") { ...
playAlbum()
if (["mp4", "mkv", "mov", "wma"].includes(fileExtension)) { ...
obviously, we will only need one global whitelist. just caught neo jumping rooms again ;)


Code for Nth from end in Linked List

I can add codes for that.

@Vidhanjain143  yeah, Make a folder name with the language name and insert here your code.
Happy Hactoberfest 🎉🎊


Add catch error function for all effects.

Codecov Report

Merging #2485 into master will decrease coverage by 0.06%.
The diff coverage is 1.53%.

@@            Coverage Diff            @@
##           master   #2485      +/-   ##
=========================================
- Coverage    9.91%   9.85%   -0.07%
=========================================
  Files         195     195
  Lines       12601   12677      +76
  Branches     1776    1776
=========================================
- Hits         1250    1249       -1
- Misses      11282   11359      +77
  Partials       69      69

Impacted Files
Coverage Δ

src/app/reducers/iteration-reducer.ts
1.53% <ø> (ø)
:arrow_up:

src/app/reducers/work-item.reducer.ts
2.77% <ø> (-0.08%)
:arrow_down:

src/app/reducers/space.reducer.ts
10% <ø> (ø)
:arrow_up:

src/app/reducers/label.reducer.ts
8.33% <ø> (ø)
:arrow_up:

src/app/actions/work-item.actions.ts
15.38% <ø> (-0.25%)
:arrow_down:

src/app/reducers/filter.reducer.ts
10% <ø> (ø)
:arrow_up:

src/app/reducers/area.reducer.ts
10% <ø> (+0.9%)
:arrow_up:

src/app/reducers/group-type.reducer.ts
6.25% <ø> (ø)
:arrow_up:

src/app/reducers/collaborator.reducer.ts
10% <ø> (ø)
:arrow_up:

src/app/effects/filter.effects.ts
6.25% <0%> (-1.45%)
:arrow_down:

... and 30 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 c93e159...7bbcdde. Read the comment docs.

@vikram-raj fabric8/fabric8-ui:SNAPSHOT-PR-2485-13 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2485-fabric8-planner.badger.fabric8.io

This partially fixed the following issue
https://github.com/openshiftio/openshift.io/issues/2469


feat(initializer): do not specify backends to autoload

We can simply try to autoload the backends extracted in the asset dir, instead of hardcoding those in the binary.  Besides simplifying maintenance as we don't need to keep the index up-to-date, it also simplifies building as we have to just drop the backends in the folder which is then embedded in the binary during build time.
This will allow ALSO to build variants of the same backend (for e.g. with different instructions sets)  without having to specify an hardcoded name, easing out to have a single binary for all the variants.
Partly related to #1888

note to myself: we might want to add ccache to the images to speedup compilation times for different llama.cpp variants

This finally makes possible to have multiple versions along, so we can fix issues like #2220 #973 #288 #1447 #1968

This also fixed https://github.com/mudler/LocalAI/issues/1916


✨ Uplift controller runtime, controller-tools and cluster-api

In this PR we,

uplift controller-runtime to v0.10.1
uplift controller-tools to v0.7.0
uplift CAPI to v0.4.3

The changes are influenced by: https://github.com/kubernetes-sigs/cluster-api/pull/5249

/test-integration

/assign @fmuyassarov
/cc @jan-est

/test-centos-integration
/test-v1a4-integration
/approve

/test-centos-integration

/test-v1a4-integration

v1a4 would fail on this no @fmuyassarov ?

v1a4 would fail on this @fmuyassarov

Ops, sorry Kashif. I completely forgot about the API type here.

/test-centos-integration
/test-integration

Ops, sorry Kashif. I completely forgot about the API type here.

No worries. Re-ran v1a5 tests again. Lets see.

LGTM

/test-integration

/lgtm


hello  , regarding executeBatch is it better ?

hello , can i ask about stmt.executeBatch is it better than single insert in our case ? as i believe that stmt.executeBatch will reduce database load alot

What are you talking about and how is it related to Traccar server?

i am asking about
https://github.com/tananaev/traccar/blob/master/src/org/traccar/database/DataManager.java > addPosition()
if we use JDBC - Batch Processing
https://www.tutorialspoint.com/jdbc/jdbc-batch-processing.htm
insert  multi row at 1 time , it will reduce database load if we have a large amount of  data to insert
https://stackoverflow.com/questions/1793169/which-is-faster-multiple-single-inserts-or-one-multiple-row-insert

We receive locations one-by-one in most cases, so there is no way to batch them.

hmmm what about add them to array and batch it every configured tick for example 2 second ?
so
new location > add to array , and each 2 second we release  data on our memory as batch ,

I don't see much point. Performance improvement will probably be negligible.


[enhancement][MultiCatalog]Add strong checker for hms table

Proposed changes
Issue Number: close #10723
Problem Summary:
Add a strong checker for the hms table. Now we only support a small subset of the hive table, so we need intercept the execution of this nonsupport table in the analyse phase.
Also this modify will fix issue #10723
Checklist(Required)

Does it affect the original behavior: (Yes/No/I Don't know)
Has unit tests been added: (Yes/No/No Need)
Has document been added or modified: (Yes/No/No Need)
Does it need to update dependencies: (Yes/No)
Are there any changes that cannot be rolled back: (Yes/No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

Hudi and Iceberg also have some nonsupport case, I will add it tomorrow


Roll engine 5646e86a6f44..2c389678f08f (14 commits)

https://github.com/flutter/engine/compare/5646e86a6f44...2c389678f08f
git log 5646e86a6f44..2c389678f08f --no-merges --oneline
2c389678f Fix a minikin crash (nullptr& EXC_BAD_ACCESS) on iOS8. (flutter/engine#6795)
faabf0ced Fix MakeRenderContextCurrent to handle software rendering (flutter/engine#6827)
14db8b397 SkPaint::FontMetrics -> SkFontMetrics (flutter/engine#6824)
453fc8f51 Roll src/third_party/skia<PHONE_NUMBER>eb..eb4d641d9437 (10 commits) (flutter/engine#6825)
1f7399313 Roll src/third_party/skia c7a34cae21d2..0243949545eb (11 commits) (flutter/engine#6823)
06377b107 Document not passing empty-strings into Locale (flutter/engine#6818)
bdd5640f3 Roll src/third_party/skia 1c3f17fffa79..c7a34cae21d2 (2 commits) (flutter/engine#6821)
a5bc33546 Revert "Roll Dart version to f9ebf2129732fd2b606286fdf58e500384b8a0bc (#6812)" (flutter/engine#6820)
961930688 Roll src/third_party/skia 95af4726bf91..1c3f17fffa79 (1 commits) (flutter/engine#6819)
4dbdf94aa Ensure that the EGL rendering context is bound to the GPU thread in Rasterizer::MakeRasterSnapshot (flutter/engine#6816)
799ce8742 fix log (flutter/engine#6817)
3e4c8cca7 Roll src/third_party/skia ef4709b7850d..95af4726bf91 (22 commits) (flutter/engine#6815)
2584816d7 Roll Dart version to f9ebf2129732fd2b606286fdf58e500384b8a0bc (flutter/engine#6812)
9669aeb95 Roll src/third_party/skia 06a477c3300d..ef4709b7850d (31 commits) (flutter/engine#6811)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.

@liyuqian : New roll has been created by https://autoroll.skia.org/r/flutter-engine-flutter-autoroll


add option to select nwp channels

Pull Request
Description
Give optional nwp channels to dataloader
need for https://github.com/openclimatefix/nowcasting_forecast/issues/94
How Has This Been Tested?

added to unittests

[ ] Yes

If your changes affect data processing, have you plotted any changes? i.e. have you done a quick sanity check?

[ ] Yes

Checklist:

[ ] My code follows OCF's coding style guidelines
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked my code and corrected any misspellings

Codecov Report

Merging #122 (1e5d25d) into main (f04c47a) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #122      +/-   ##
==========================================
+ Coverage   93.90%   93.94%   +0.04%
==========================================
  Files          17       17
  Lines         903      909       +6
==========================================
+ Hits          848      854       +6
  Misses         55       55

Impacted Files
Coverage Δ

nowcasting_dataloader/datamodules.py
80.82% <100.00%> (+0.26%)
:arrow_up:

nowcasting_dataloader/datasets.py
96.87% <100.00%> (+0.08%)
:arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


fix: notifications should be fire&forget rather than having a timeout

Summary
Fixes #8036

Test plan
Verified locally. Due to #9454 I didn't wanna add a test. Writing that justification now though, makes me wanna just skip the test on windows, though...

Codecov Report

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

@@            Coverage Diff             @@
##           master    #9567      +/-   ##
==========================================
+ Coverage   65.05%   65.07%   +0.01%
==========================================
  Files         286      286
  Lines       12128    12128
  Branches     3005     3005
==========================================
+ Hits         7890     7892       +2
+ Misses       3600     3599       -1
+ Partials      638      637       -1

Impacted Files
Coverage Δ

packages/expect/src/utils.ts
96.22% <0.00%> (+1.25%)
:arrow_up:

Continue to review full report at Codecov.

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

Hmm, adding a test will actually fire a notification. Is that annoying? I pushed it for now

Codecov Report

Merging #9567 into master will decrease coverage by <.01%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master    #9567      +/-   ##
==========================================
- Coverage   65.05%   65.05%   -0.01%
==========================================
  Files         286      286
  Lines       12128    12127       -1
  Branches     3005     3005
==========================================
- Hits         7890     7889       -1
  Misses       3600     3600
  Partials      638      638

Impacted Files
Coverage Δ

packages/jest-reporters/src/notify_reporter.ts
74.5% <66.66%> (-0.5%)
:arrow_down:

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 2793c67...85aa69f. Read the comment docs.

We can remove the test later if it turns out to be annoying

Thanks man this issue had wasted my whole day lol. And I was like if it was redis/ typeorm or something else and i was sure there should not be any stuff. And is there anyway to know how to debug such issue if we face it in future?


Add วิกฤติ

@mattleff Can we add this to the current forbidden word PR on the other repo as I am still going through and making changes there and haven't merged?

@bountonw Will do.

@mattleff Thank you!


PeasyCam on Processing 2.2.1

Hi jdf,
I just installed Processing 2.2.1 and imported PeasyCam in a new Windows 10 computer.
Apparently Processing 2.2.1 runs java 1.7.0_40, and I get exactly the same message as in issue #24 when running scripts that work fine in my old computer:
java.lang.RuntimeException: java.lang.UnsupportedClassVersionError: peasy/PeasyCam : Unsupported major.minor version 52.0
Would it be possible to have access to old PeasyCam versions compatible with Processing 2.2.1 runnig java 7?
Thanks,
F

Hello.
I have the same problem.
I examined the version of  .class files in the .jar file, finally I found that they were compiled by jdk 1.8.

same here, in Processing 2.2.1 on OSX 10.11.6

Ditto, Processing 2.2.1 OSX 10.12.2

nothing yet. up


MAINT: add type ignores to sympy and matplotlib imports

Reference issue
https://github.com/scipy/scipy/issues/11749
What does this implement/fix?
Similar to what we did with mpmath; sympy and matplotlib don't have stubs and aren't actual runtime dependencies, so we should just ignore them.
Additional information
None

Just curious: why is this an improvement? It seems like the ignores just get moved around and nothing in the type checking behavior changes?

When we distribute SciPy as a package with types our mypy.ini Isn’t going to be there, so users will get type errors about stuff inside SciPy. (I should probably go check on that.) I don’t think (?) there’s a global way for us to say “hey treat sympy as any inside SciPy” that we can distribute with the package. That would be very handy though...

Ah yes that makes sense, thanks! Merged.


Move last operation to Instance.Status (Issue 248 followup)

Move the instance.Spec.OSBLastOperation to instance.Status.LastOperation
Fixes #732

@pmorie addressed your comment.

@vaikas-google rebase needed, unfortunately.

Thanks to @pmorie ninja-git-fu skillz, this has actually now been correctly rebased. Wheeee, that was fun. Looks like Jenkins failed to pull the image. I'll try to rekick it.

@vaikas-google thanks a ton for doing this, this is awesome.


Jenkins looses all its state on pod restart.

At the moment Jenkins master stores all data inside container. As result, all state is lost on a pod migration or restart.

Jenkins pod must bootstrap itself from information available inside Kubernetes, at least in regard of security credentials and basic configuration.

Workspaces should be stored on a separate volume which will survive simple issues.

Has been merged into develop


How to config hmac-auth plugin?

Kong Ingress controller version
kong:1.2
Kong or Kong Enterprise version
kong:1.2
Kubernetes version
1.11.5
Steps To Reproduce
1.install plugin
echo "apiVersion: configuration.konghq.com/v1
kind: KongPlugin
metadata:
  name: hmacbin-auth
plugin: hmac-auth
config:
  enforce_headers: date,request-line
  algorithms: hmac-sha1,hmac-sha256
" | kubectl apply -n ai-test -f -

2.provision user
echo "
apiVersion: configuration.konghq.com/v1
kind: KongConsumer
metadata:
  name: alice
username: alice
" | kubectl apply -n ai-test -f -

3.create credential
echo "apiVersion: configuration.konghq.com/v1
kind: KongCredential
metadata:
  name: alice-hamc
consumerRef: alice
type: hmacauth_credentials
config:
  username: alice123
  secret: secret
" | kubectl apply -n ai-test -f -

4.config ingress
kubectl -n ai-test patch ingress test-ingress \
-p '{"metadata":{"annotations":{"plugins.konghq.com":"hmacbin-auth\n"}}}'

But the plugin didn't work,Why?

thx

Please share the logs of the controller to inspect what is going wrong.
I cannot say for certain but it is likely that the following is the root cause:
In the following YAML: enforce_headers is a string, but Kong expects it to be an array of string:
apiVersion: configuration.konghq.com/v1
kind: KongPlugin
metadata:
  name: hmacbin-auth
plugin: hmac-auth
config:
  enforce_headers: date,request-line
  algorithms: hmac-sha1,hmac-sha256

If you change it to:
config:
  enforce_headers:
  - date
  - request-lien

It should work.

Please share the logs of the controller to inspect what is going wrong.
I cannot say for certain but it is likely that the following is the root cause:
In the following YAML: enforce_headers is a string, but Kong expects it to be an array of string:
apiVersion: configuration.konghq.com/v1
kind: KongPlugin
metadata:
  name: hmacbin-auth
plugin: hmac-auth
config:
  enforce_headers: date,request-line
  algorithms: hmac-sha1,hmac-sha256

If you change it to:
config:
  enforce_headers:
  - date
  - request-lien

It should work.

It worked. Thank you very much


pre-commit: update black to Python 3.9

Signed-off-by: Filipe Laíns<EMAIL_ADDRESS>

https://github.com/pradyunsg/installer/blob/main/noxfile.py#L23
This is intentionally 3.8 — since that’s the same as all the sessions in the noxfile, in case some contributor only has single version of Python in their environment. Bump everything to 3.9 if you’d prefer. :)

Oh, and you’ll also need to update GA.

Eh, we can close.


uv lock requires python dev versions

I encountered an issue with the  elasticsearch 7.17.1 while trying to use python>=3.9. The relevant requirement for elasticsearch is:
Requires: Python >=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*, <4.
When attempting to lock dependencies, the following error is displayed:
  ╰─▶ Because the requested Python version (>=3.9) does not satisfy any of:
          Python>=2.7,<3.0.dev0
          Python>=3.4.dev0,<4
      and elasticsearch==7.17.1 depends on one of:
          Python>=2.7,<3.0.dev0
          Python>=3.4.dev0,<4
      we can conclude that elasticsearch==7.17.1 cannot be used.

Relatedly, this also results in the following error message, but uv lock doesn't accept prerelease as an argument.
      hint: Python was requested with a pre-release marker (e.g., any of:
          Python>=2.7,<3.0.dev0
          Python>=3.4.dev0,<4
      ), but pre-releases weren't enabled (try: `--prerelease=allow`)

You need to do Requires-Python = ">=3.9,<4" to satisfy that constraint.

We changed this in #4086. But as-is, there are Python versions that you claim to support (>4) that your dependencies do not.

As of #4086, we're going to treat Requires-Python as a lower bound, which will resolve this for you.

Cool thanks! I'll just close this then.

The hint is wrong though :) I created https://github.com/astral-sh/uv/issues/4096.

I can create a new issue if this is a bug.
Is there a reason why uv lock traverses down to urllib3==1.2.1 when I have   'urllib3>=1.22.0 listed in my dependencies?
dependencies = [
  'urllib3>=1.22.0',
  'elasticsearch==7.17.1',
  'dockerspawner==13.0.0',
]

urllib3==1.2.1 fails to build on my machine but there are many versions of urrlib3 that'd build fine and would satisfy the requirements.

Feel free to create a new issue.


test with my own images

How do I run inference on my own images?
Thanks,

Hi, there is no inference script for testing a single image now. But I guess it will not be hard to do that.

@tfzhou
It will be great if you can create one or provide a guidance of how to do it.
Thanks,

Hi @fjremnav , sorry for that I have no plan to write this script. It will be perfect if you would write it. The file may have a simple pipeline:
# 1. define the model and load the pretrained weight
# 2. load the image using opencv
# 3. convert the image array to tensor with shape 1 x 3 x h x w
# 4. call the forward function of the model

you can also refer to this line.