[Question] Getting nested XML.

Hello. I'm building an app and I'm using nasa imagery fetcher as a reference for it. The app formats things how I'd like to, so I figured I'd learn from it.
The RSS feed I'm trying to parse has this:
<description>
     <![CDATA[
            <img src="https://website.com/image.jpg" width="100%" style="margin: 20px 0;">
            Olivier, 29 years-old, has been drifting for far too long. Today, an unexpected announcement gives him a chance to reset his life.
]]>
</description>

Can you give me any tips on parsing this? I'm thinking I can nest them the way enclosure does, but ![CDATA[] concerns me. I'd like to separate things and getImage() and grab image source and getDesc to grab the text only.
The code isn't really commented so its difficult to know whats going on and how to disassociate the app from giving users notifications on new images. But I'm learning.
This is the RSS Feed I plan on parsing: https://www.shortoftheweek.com/feed

Hi there. RSS feeds are parsed using Simple XML. Here's some guidance on parsing CDATA blocks (basically, you just need to set the data attribute to true).
It would look something like this, then:
@Element(name = "description," data = true)
private String description;

You can see how my class is set up in this file.

I can see that. But I'm concerned about the nested img src. I'm thinking...
public static class IMG {
                @Element(name = "img", required = false)
                String img = "";
                @Attribute(name = "src")
                String src = "";
}

But likely a Description class with IMG class nested inside?

As CDATA means character data, you cannot parse it as an XML element. You'll need to parse the string itself to get the image URL. There are lots of different methods for doing that in Java. Here's a discussion: http://stackoverflow.com/questions/17908302/extract-url-from-java-string-containing-img-src-tag


Hue integration tests

These scripts can be used for testing Hue init action on single, standard and ha configurations.

@karth295 All changes are applied. Could you check this once again?


Wrong url of Duke.tar.gz

Hi!
I have successfully downloaded 'Market-1501-v15.09.15.zip' from google drive, but the url of 'Duke.tar.gz' in the code showed 'Error 404'. Any help?
By the way, I have downloaded a file named 'DukeMTMC_reID.zip' before. Is it same to 'Duke.tar.gz'?
Thanks!

@dongb5 Thank you very much for reporting the issue. It seems that the authors have changed the link and name of the dataset a little bit. I have fixed it in the recent commit. You can directly put DukeMTMC-reID.zip to the directory examples/data/dukemtmc/raw/


Have camera datas from Arnold in EXR

In GitLab by @kekefreedog on Mar 1, 2023, 09:38

From Olivier Osotimehin

Need
In Maya submitter, we have to bring camera datas into the EXR

Need to be pushed on our studio branch (The changes affects fixstudio only)

I would like to explore if it is possible to put it in a repository that will contain all our quad specific pyblish plugins. it will be easier to maintain.

@ClementHector this repository has been created for this kind of case (https://github.com/quadproduction/quad-plugins), the changes for this ticket will be implemented into it soon.
Once it will be done, we'll need a CI/CD solution as the quad-modules repository to be set

The repository has been setted and it works for the Publish Arnold Camera Metadata, I had some trouble to find the best way to set it up, I followed the old logic (https://openpype.io/docs/pype2/admin_presets_plugins/) but it still working, I'm not sure about this solution for the futur. I've added load.json, load.json, publish.json in the repo in order to initialize the logic.
The github runner are currently not running because it does not exists at the moment (Its/Systems are aware of and currently working on it). In order to be tested asap, I've deployed a first version 0.0.1 on the network.

I've deployed a new logic which use IPluginPaths() Class provided by Openpype, this class allow use to manage different plugin path for different hosts as the path is referenced in the get_plugin_paths() function (cf. https://github.com/quadproduction/OpenPype/tree/develop/openpype/modules#ipluginpaths).
In order to use IPluginPaths(), I had to create an OpenPypeModule() and update the QUAD_PLUGIN_PATH environnment variable.

hello, how is going this feature ? is it fully deplyoed ? can we test it in nuke ?

Hello, the feature has been deployed since a while now, it should works in Nuke. Please, be aware, the Arnold Metadata appears only on sequence which has been rendered through Maya

oh thanks, the status has never been put on done, so we didn't know ...

will have to wait till we have the nuke script to read the data to test

i managed to test with the nuke py script it is working thanks


c# example doesn't work for list of deleted objects

As per title. I think the API changed along the way and not it does not support a generic query against the deleted Objects. You have to specify additional URL part, i.e. group, user.

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

ID: ddf76efc-5ca2-e829-1604-fa858179858a
Version Independent ID: bbf2d629-8795-3d87-76de-f147489e4942
Content: List deletedItems (directory objects) - Microsoft Graph v1.0
Content Source: api-reference/v1.0/api/directory-deleteditems-list.md
Product: directory-management
Technology: microsoft-graph
GitHub Login: @keylimesoda
Microsoft Alias: MSGraphDocsVteam

odata type casts are not supported in the current version of the SDK. This should be fixed in snippets generated for kiota based SDK.

Cast support was added in SDK v5 and snippets are updated to reflect that.


Use tf.contrib.image.rotate in place of lambda:tf.py_func(random_rotate_image, [image], tf.uint8)

In the graph we are using
lambda:tf.py_func(random_rotate_image, [image], tf.uint8)
inside
create_input_pipeline
This causes problem while using tensor flow serving as python layers are not supported on TensorFlow serving.
Can we convert this to a TensorFlow operations ? Something like tf.contrib.image.rotate ? What could be the downside ? @davidsandberg @xolott, @Cecilwang, @scotthong Please comment.
This issue would also get resolved.

Having the same issue, waiting for a solution.

I think you can remove this line and re-train the model. Because based on the training parameters that the author @davidsandberg provided in this train example, '--random_rotate' is not set and thus this function will not be used in training.

what I did is: download the pre-trained model, export it to TF serverable using the code provided in this thread, and got the same error as described here by @Dr-Champ

I used the SavedModelBuilder to generate the servable model. Load the model, get the 3 tensor (input, embeddings y phase_train), create the prediction_signature. Then add this signature and the meta_graph (tag_constants.SERVING) to the builder... The just save the servable model.
I don't have the code anymore, but this script took me no more than 30 lines of code.

Hi, @xolott
Thanks for your reply.
I found your script which you used for generate the servable, but still got the same issue. However, I also found an older version '20170512-110547' instead of the latest one '20180402-114759' on the page . The old one can be run on TF-Serving.
Anyway, I will still remove the 'py_func' and re-train the model. will let you know once i have done.

Hi @FortiLeiZhang,
I was able to create the servable. Then when I was trying to invoke the tensorfow_model_server to specify the port, it threw me the 'PyFunc' error. I then went back and commented the 'tf.py_func()' line in facenet.py, created the servable again and ran the model server. It still threw me the 'PyFunc' error which was quite weird.
You mentioned that you tried an older version, viz. '20170512-110547' and there was no problem in serving the model. But, using '20180402-114759' throws this 'PyFunc' error. Now, it got me thinking, is it because the older version doesn't have py_func() used anywhere (while creating the pre-trained .pb file) but the latest version does? That's why even if we remove the py_func() line from facenet.py it still throws this error?
Can you kindly tell me where you found the '20170512-110547' file? I don't seem to be able to find it on the page.

Hi, @tamaghnadutta
In the facenet package, you can find in "download_and_extract.py". Read the code, then you will get the download link.

a quick update.  I removed all the lines
            image = tf.cond(get_control_flag(control[0], RANDOM_ROTATE),
                            lambda:tf.py_func(random_rotate_image, [image], tf.uint8),
                            lambda:tf.identity(image))

and re-train the model. The first checkpoint is serverable without the py_func issue.

@FortiLeiZhang can you share your retrained weights without py_func preprocessing layer ?

快速更新。我删除了所有行
            图片 =  tf。COND（get_control_flag（控制[ 0 ]，RANDOM_ROTATE），
                            拉姆达：TF。py_func（random_rotate_image，[图像]，TF。UINT8），
                            拉姆达：TF。身份（图像））

并重新训练模型。第一个检查点可用于服务器，而不会出现py_func问题。

Hello, where is this update? I can't find it

a quick update. I removed all the lines
            image = tf.cond(get_control_flag(control[0], RANDOM_ROTATE),
                            lambda:tf.py_func(random_rotate_image, [image], tf.uint8),
                            lambda:tf.identity(image))

and re-train the model. The first checkpoint is serverable without the py_func issue.

Hello, where is this update? I can't find it

a quick update. I removed all the lines
            image = tf.cond(get_control_flag(control[0], RANDOM_ROTATE),
                            lambda:tf.py_func(random_rotate_image, [image], tf.uint8),
                            lambda:tf.identity(image))

and re-train the model. The first checkpoint is serverable without the py_func issue.

I delete py_ func, and use training to copy the weight, the problem has been solved, thank you


Location not showing in "Charge Details" dashboard

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Within the Charge Details dashboard the location is shown as N/A.
In the Charges dashboard the location is shown correctly.
Expected Behavior
Location information should be shown correctly on Charge Details dashboard.
Steps To Reproduce

Go to Charges Dashboard
Select a charge and click on the date and time
Now you are in Charge Details dashboard

Relevant log output
No logs needed. It is a settings issue for the dashboard. See second screenshot.

Screenshots

Issue can be fixed by changing these two values, but dashboard can not be saved, because it is provisioned.

Additional data
No response
Type of installation
Docker
Version
v1.27.3

Closed /w https://github.com/teslamate-org/teslamate/pull/3294


feat: send email only when node is down

Description
This PR is for reducing the amount of emails Node Monitor sends. More concretely, NM will only send emails when a node is down.
Changes made
In run_once() I added a check to see if self.t2 == DOWN. Only if this is true and email is sent.
Testing
Passes all tests in test_node_monitor.py.
Additional notes
This is the "quick fix" solution that I was able to make. However, I will spend more time cutting in cutting out the code that is not needed based on the new functionality we would like NM to have.

All good my mistake!
self.t2 = t2 is set by default in the constructor of ChangeEvent


Evaluate and improve data-warehouse file structure

Currently we only divide Parquet files by resources, i.e., all files with the same schema end up in the same directory. This might be good enough for now but we probably need to add some other types of partitioning, e.g., time based. In particular, this might be necessary if the performance evaluation #111 turns out to be not satisfactory.
This might also be needed for better batch performance. For example, if we divide Observations by day, then we can limit regeneration of the DWH to recent events only (using the batch pipeline).
Evaluation of the current structure in a big database setting (e.g., AMPATH) is P2; the rest might be P3.

This is superseded by #363.


Support restarting containerd in tests, add restart test case

This PR adds support in the cri-containerd test suite to allow tests to restart containerd.
A test case is also added that validates containerd properly terminates running containers
when it is restarted.
Note this test case depends on functionality in the kevpar/cri windows_port branch, and
that the terminate_containers_on_restart option is set on the plugins.cri section of the
containerd config.
See commits for more details.

Needs a rebase I believe. The pullRequiredLcowImages function was renamed to have Lcow be capitalized here https://github.com/microsoft/hcsshim/commit/8debf44d62391eaebbc5d2a80999ab1221d31873#diff-a8c710332541b245cc336ff7f50e609f30213b47a1f67abe39908fe212862322

Needs a rebase I believe. The pullRequiredLcowImages function was renamed to have Lcow be capitalized here 8debf44#diff-a8c710332541b245cc336ff7f50e609f30213b47a1f67abe39908fe212862322
Edit: Err not a rebase, this has the changes. Just need to update your uses of the old function name

Aha, I hadn't checked back on CI yet.


[ko] Translate concepts/workloads/pods/user-namespaces.md into Korean

This is a Feature Request

What would you like to be added

Translate docs/concepts/workloads/pods/user-namespaces.md into Korean.

Page: https://kubernetes.io/docs/concepts/workloads/pods/user-namespaces/
File: https://github.com/kubernetes/website/blob/main/content/en/docs/concepts/workloads/pods/user-namespaces.md

Why is this needed
Has not been translated into Korean yet.
Comments

/assign
/language ko

@kyungjin99 감사합니다!
/triage accepted

Resolved by

#36998
#38192

Thanks!
/close


Ollama.md Documentation

A guide on setting up a fine-tuned Unsloth FastLanguageModel from a Google Colab notebook to:

HF hub
GGUF
local Ollama

Preview link: https://github.com/ollama/ollama/blob/66f7b5bf9e63e1e98c98e8f487427e19195791e0/docs/ollama.md

wrong repo, was intended for unsloth repo sorry


[5.0] Boilerplate controllers confusing names

Currently, Laravel 5 has two boilerplate controllers: HomeController and WelcomeController.
Each time I see them I think "oww which one it the frontend one ?". Then I open both files and spend a tiny bit of time to figure out... I can't stop mixing their names :disappointed_relieved:
Case in point, for quick debugging I just throw code in the homepage controller (it's WelcomeController).
How about naming the backend controller something like DashboardController? It would be so much clearer :grinning:

(BTW, the "home" view is currently missing, but I guess you're already aware)

We're open to pull requests.


Suggestions for addition to 'safe' function list / XSS sniff

Might these be considered safe ?
get_the_time()
get_the_date()
wp_dropdown_pages()
paginate_links()

For get_the_time() and get_the_date(), shouldn't you just be using the_time() and the_date()? However, these really don't qualify for the auto-escaped functions list, because there is no explicit escaping, and the value is passed through a filter (which means absolutely anything could get incorporated into the value).
wp_dropdown_pages() is inconsistent in escaping. It escapes some of the passed arguments but not others. All arguments passed to it should therefore be escaped. Maybe it should be added to the list of printing functions (though direct output can be disabled by setting echo to false, as you are probably doing).
paginate_links() also doesn't escape all of the values passed to it, but it doesn't output the generated HTML directly. This makes things difficult. It might seem like overkill, but I'd suggest you run the function through wp_kses().

For get_the_time() and get_the_date(), .... the value is passed through a filter (which means absolutely anything could get incorporated into the value).

Fair point, I retract those.
Regarding wp_dropdown_pages() and paginate_links() - maybe the error message should be made cleared for those kind of functions - saying that variables passed to them should be escaped - would be helpful for people trying to figure out what to do.

or we could send in a PR to WP Core to make sure the escaping within those last two functions becomes consistent ;-)

or we could send in a PR to WP Core to make sure the escaping within those last two functions becomes consistent ;-)

That would be nice, but it would probably not be accepted, due to concerns with back-compat. See ticket #17182.

I've opened #455 to add wp_dropdown_pages() to the printing functions list. This will make escaping errors for it more consistent.
Unless the VIP team speaks to the contrary, I don't think that we should add either wp_dropdown_pages() or paginate_links() to the list of auto-escaped functions. In case that does happen, #455 can be reverted.
Closing as wontfix for now. Feel free to reopen if the VIP team says that these are OK.

For what it's worth, if paginate_links() is not allowed in the list of auto-escaped functions, paginate_comments_links() should probably also be removed on the same grounds, since it merely passes all arguments blindly to paginate_links().

Hey folks, I just encountered this issue with paginate_links() in my theme. What is the solution to this? Is there a different function I can use?
Thanks!

@paulcanning You could either make sure all of the args you pass to it are properly escaped, and then add a whitelisting comment: // WPCS: XSS OK. Or, you could pass the result through wp_kses(), but you'd probably have to manually construct the list of allowed tags.

Hmm, i'm not passing any args to it...

In that case, you are probably safe to just add the whitelist comment.

I added the whitelist comment you posted above, but it doesn't seem to clear the warning.
<?php
// WPCS: XSS OK.
echo paginate_links();
?>

The comment goes at the end of the line. This is also covered in the wiki: https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/wiki/Whitelisting-code-which-flags-errors 😃

I have now!

I'm looking at paginate_links() function, and I'm not 100% sure why this is not in the $autoEscapedFunctions variable?
It returns the HTML, but all the apply_filters calls are escaped, aria-current attribute is escaped. Isn't this safe by design?

As @JDGrimes says above:

paginate_links() also doesn't escape all of the values passed to it, but it doesn't output the generated HTML directly. This makes things difficult. It might seem like overkill, but I'd suggest you run the function through wp_kses().

I've just had a look at the function to see if the status has changed, but it hasn't.
While most arguments are escaped within the function, the following are not:

$args['prev_text']
$args['before_page_number'] . number_format_i18n( $n ) . $args['after_page_number']
$args['next_text']

Oh, ok. Then the paginate_comments_links should be removed from the list as it either echoes or returns the paginate_links function.

@dingo-d Yes please ;-)


Column, line and char number are off by one

Lapce Version
0.4.0
System information
OS: ArchLinux (up-to-date)
Kernel Version : 6.8.9.zen1-2
Plasma Version : <IP_ADDRESS>-1
KDE Version : 24.02.2-1
Frameworks Version : 6.2.0-1
Qt5 Version : 5.15.13+kde+r7-1
Qt6 Version : 6.7.0-6
Wayland Version : 1.22.0-1
XOrg/Wayland Version : 23.2.6-2
Mesa Version : 1:24.0.7-3
LibVA Version : 2.21.0-1
VDPAU Version : 1.5-2
Describe the bug
At least line numbering is 1-based when displayed in the editor panel (left column to the text) while it is 0-based when displayed in the status bar.
Consistent column and character numbering is highly desired.
I think that 1-based numbering is the "de facto" standard across all editors I have seen so far, also because most compilers, interpreters and other tools all provide diagnostics with 1-based line numbering.
I personally don't mind as long as it is consistent within the same program.
Additional information

Please, rename the report as "Inconsistent line numbering"


Feat(eos_designs): Expose rack, pod, dc, fabric information in the structured_config metadata

Having this information will be used in future integrations.
Proposal:
metadata:
  # Physical location metadata
  rack: <str>
  pod: <str>
  dc: <str>
  fabric: <str>

Let's use the key names directly so rack, pod_name, dc_name and fabric_name.


performance improvement minimizing flushing to disk

This change drastically improves save times by limiting a flush to the end of appending all the records
Before:

After:

Whoa!! That's a nice improvement!

I've tested that thought, if I understand correctly, where essentially the LevelDbWritableFile::Flush() shouldn't call the file system flush() and only relies on the LevelDbWritableFile::Sync() to handle the actual flushing to disc. There is a similar magnitude of improvement, though my meagre testing shows this method here thrashes less and performs faster. I'll post a PR for comparison.

I've tested that thought, if I understand correctly, where essentially the LevelDbWritableFile::Flush() shouldn't call the file system flush() and only relies on the LevelDbWritableFile::Sync() to handle the actual flushing to disc. There is a similar magnitude of improvement, though my meagre testing shows this method here thrashes less and performs faster. I'll post a PR for comparison.

👍
I suspect that ideally we'd do some work on Flush - some of our file implementations do have exactly the sort of buffer that we could pass along to the OS to get more things moving in parallel.  The key change would be to shift the wait for the commit to fully complete over to the Sync call.
Makes sense that any change that preserves Flush calls would be slightly less performant on this - there are a handful of callback layers that needs to bounce through even if it's going to eventually reach a no-op.  So if we were calling it enough to see the kind of perf delta this change brings then imagine we're calling it enough for all that function call overhead to be measurable though hopefully small.


Added possibility to specify own status id for pass/fail

On current project we have own custom statuses for pass/fail when running autotests.
This PR adds possibility to specify custom status_id in config

Awesome! Thanks @Xotabu4


Converting a callback to an observable?

Hey guys, I'm working with an old HTTP framework and I'm trying to convert the requests into observables.
The framework works like this:
// Define callback object
class MyCallback : CkCallback
{
	void OnUpdate(HttpUpdate update)
	{
		// handle progress, or success, or failure, or canceled
	}
}

// Set callback
CkHttp http;
auto callback = new MyCallbackObject;
http.set_callback_object(callback);

// Start asynchronous GET
CkTask task = http.GetAsync("http://www.google.ca/");

// (Optional) Cancel asynchronous GET
task.Cancel();

I want to turn this into an observable, but I'm not really sure how to do it. Any tips on it? I've looked it up online but haven't found anything similar in RxCpp or RxJS.

You should use a Subject.
Here is a good example of adapting a push interface to RX, with support of cancelation.


LTXV img2vid

Feature Idea
I want to know if img2vid of LTXV will be supported by SwarmUI ? thanks
Other
No response

it's in-progress, see comments in the Discord news-talk channel


Skip rendering when dynamic range is zero

When range_min==range_max, the rendering equation divides by zero and the
resulting scaled channel image is all NaN. This ends up forcing the entire
image to black as the NaNs propagate through the compositing and get turned
into zeros when cast to uint8. This change just skips rendering and
compositing when the dynamic range is zero.

Looks good to me!  If nothing is rendered, Minerva Story will display a black tiled image. So, this will not change anything visually for the end user.


Added NotoSansOldHungUI UFO files

Added NotoSansOldHungUI UFO files into src/NotoSansOldHungUI directory. Main UFO file:
src/NotoSansOldHungUI/NotoSansOldHungUI-Regular.designspace

@marekjez86 Merge it, please!

I'm looking at it and evaluating it. I will take me a week to do it.

@marekjez86 Don't merge this pull request, please! It got several form problem!


Hello I found one issue from iPhone X I have uploaded the video using the Swifty Cam but the server guy received the video but the problem is the video is not playing only in iPhone X what is the issue please help me

The video is probably saved in hevc format and your server does not handle it.
If the file is not corrupted anyone who can read the file extension should be able to use a software to read it


chore: Use python-version 3.10 and bump python-semantic-release

Signed-off-by: Tibor Dudlák<EMAIL_ADDRESS>

I encountered problems releasing here: https://github.com/neoave/mrack/actions/runs/2739462757 hopefully this MR fixes that

We need to updated this version in other files too e.g  pre-commit.yml

I have decided to go back to 3.9 as python semantic version is using 3.9 container


AssertMeasurementProbability() Doc is missing some info.

This fragment is missing the info about the last param - tol (possible text is here).
This likely affects the same missing info here.
To be fixed together with the QIR Runtime implementation of

operation AssertMeasurement(bases : Pauli[], qubits : Qubit[], result : Result, msg : String) : Unit is Adj + Ctl
operation AssertMeasurementProbability(bases : Pauli[], qubits : Qubit[], result : Result, prob : Double, msg : String, tol : Double) : Unit is Adj + Ctl

Also make the fragment Measure(bases, qubits) a link to Measure(), otherwise it is not that fast to find Measure() page.

Agreed, it looks like that input is not documented.
One of the recent improvements made in the documentation generation tool is to raise warnings about undocumented inputs such as this; we should check that this warning is correctly emitted when compiling the runtime repo.

Related (if only by which part of the generated docs are impacted): https://github.com/microsoft/qsharp-compiler/issues/949.

The root cause for this one may be different, in that the original comment being interpreted by the docgen tool is indeed missing a /// ## tol section:
https://github.com/microsoft/qsharp-runtime/blob/974a385cc57c2b663e8134c1f3170f9cb8ae5fb1/src/Simulation/QSharpFoundation/Diagnostics/Assert.qs#L47
Ideally, this should raise a warning, but I don't see it amongst the 36 documentation generation warnings raised in the latest e2e build...


Add support for product flavors for Perf testing

Current dependencies on/for this PR:

main

PR #85   👈

This comment was auto-generated by Graphite.


Fix typo on automated expressions validation for MiddlewareServer

This is a continuation of #9441.
I could not send in time this PR as result of the last review done by @dclarizio.
This PR fixes a typo found in the validation of the expression.
It also adds additional validation on the parameters as suggested.
@dclarizio , I have failed to find where are defined the specific tests for form expressions related to alerts.rb.
The alerts_spec.rb shows very basic coverage and I thought that perhaps MiddlewareServer specific tests are suited better in a different location.
If you are agree, I can add additional tests for this UI specific point in a following PR, just let me know what is the preferred format for that and if I can follow a similar test for the same topic.
@miq-bot add_label fix, providers/hawkular

@miq-bot add_label bug, ui

LGTM.
@miq-bot assign @chessbyte


within a DeepExplain context,  de.explain run multiple times, it becomes slower and slower .

within a DeepExplain context,  de.explain run multiple times, it becomes slower and slower .

Why?

Please see related issue #31


the latest tag should be removed when the CI is done

Just tried to commit to my backlogged-updates branch:
> git pull --tags origin backlogged-updates
From https://github.com/cpp-linter/cpp-linter-action
 * branch            backlogged-updates -> FETCH_HEAD
 ! [rejected]        latest     -> latest  (would clobber existing tag)

After #83 was opened, the latest tag was moved to a different branch which seemed to cause conflicts when my machine tried to do git pull.
I would suggest
--- a/.github/workflows/run-test.yml
+++ b/.github/workflows/run-test.yml
@@ -23,6 +23,7 @@ jobs:
           git push origin latest

   test-action:
+    needs: [add-tag]
     runs-on: ubuntu-latest
     steps:
     - uses<EMAIL_ADDRESS>@@ -38,6 +39,7 @@ jobs:
         trigger_workflow: true
         wait_workflow: true
   test-package:
+    needs: [add-tag]
     runs-on: ubuntu-latest
     steps:
     - uses<EMAIL_ADDRESS>@@ -52,3 +54,14 @@ jobs:
         propagate_failure: true
         trigger_workflow: true
         wait_workflow: true
+  remove-tag:
+    needs: [test-action, test-package]
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+      - name: remove latest tag (cleanup)
+        run: |
+          git config user.name 'github-actions'
+          git config user.email<EMAIL_ADDRESS>
+          git push --delete origin latest || true

Removing the tag latest let me push/pull again.

CI failed with:
To https://github.com/cpp-linter/cpp-linter-action
 - [deleted]         latest
To https://github.com/cpp-linter/cpp-linter-action
 ! [remote rejected] latest -> latest (shallow update not allowed)
error: failed to push some refs to 'https://github.com/cpp-linter/cpp-linter-action'

This is getting annoying when there's 2 PRs open. related to #71

From the CI failed log, tt seems need to checkout all branches and tags.
- uses: actions/checkout@v3
   with:
     fetch-depth: 0

This issue should be fixed in #81, can it be closed?

The only thing I did to the workflow in that PR was add the needs: [add-tag] to the subsequent steps. We can close this for now, but we may encounter this issue again...
I think the main culprit was having more than 1 PR open at the same time, which caused problems moving the latest tag around difficult because the repo was checked out with a shallow history
https://github.com/cpp-linter/cpp-linter-action/blob/600b1aabe9753af4fe2c6ad0f4220166505b4252/.github/workflows/run-test.yml#L15-L16
This could use an added
      with:
        fetch-depth: 0


Card not working

Which version of Room Card? : 1.7.11
Describe the bug
I cannot use the card. It cannot be found in the card-selector
To Reproduce
Steps to reproduce the behavior:
No idea. Just did install via hacs, restart windows PC, started VM: no card to see.
Expected behavior
Working card
Screenshots
room-card is added via hacs

room-card cannot be found by the system

tried also via yaml:

Desktop (please complete the following information):

OS: windows10/VM
Browser firefox
Additional context
Add any other context about the problem here.

Try clearing the cache of your browser. If that doesn't help check Developer tools if the javascript file can be found.

I cleared cache, restarted, tried 2 other devices (iphone and ipad): no result.
Files are where I think they have to be:
www.comunity.room-card.room-card.js
and
www.comunity.room-card.room-card.js.gz
Don't know how to search fir files with developer tools.

Started all over.
Removed hacs, restart HA, cleared cache, load room-cards again via hacs, cleared cache, restart HA.
Now it works.
Thanks for your help!


Honnor PollAdminChooser signature

Hi,
I propose to honnor the PollAdminChooser signature: #14
wagtailpolls/widgets.py:17
    def __init__(self, content_type=None, **kwargs):

in the wagtailpolls/edit_handlers.py:16:
    @classmethod
    def widget_overrides(cls):
        model_meta = cls.target_model()._meta
        content_type = ContentType.objects.get(app_label=model_meta.app_label, model=model_meta.model_name)
        # Changes the call parameters of ``AdminPollChooser`` from ``model`` to ``content_type``
        return {cls.field_name: AdminPollChooser(content_type=content_type)}

This avoid loading errors...

Hi Francis,
Thank you for your pull request! I have added the missing migrations in e8b13a1. For the other changes, could you please make separate pull requests for each change if you believe they are still required?
Cheers! 😄


models/usage, mockapi issue

I found there are two issues in the mockapi.ts of models/usage case.

The HTTP method of Scenarios.Models_Usage_output case shoud be ``getinstead ofpost`. See here for the corresponding cadl definition.
We defined the status code in the response as 202, see here, but the cadl definition does not indicate it will return 202, so in our code, we are expecting status code 200 (the default one) instead of 202 like below.
May I know the intention of making this call return 202? Shall we still return 202 and modify the cadl definition or return 200 in mockapi.ts?

@Get("/models/usage/output")
@ExpectedResponses({200})
@UnexpectedResponseExceptionType(
        value = ClientAuthenticationException.class,
        code = {401})
@UnexpectedResponseExceptionType(
        value = ResourceNotFoundException.class,
        code = {404})
@UnexpectedResponseExceptionType(
        value = ResourceModifiedException.class,
        code = {409})
@UnexpectedResponseExceptionType(HttpResponseException.class)
Mono<Response<BinaryData>> output(
        @HeaderParam("accept") String accept, RequestOptions requestOptions, Context context);

Thanks for bringing this up! We also met these issues,

For first one we should change from post to get
https://github.com/Azure/cadl-ranch/blob/d1ffa96dc92bed37e96d05ca25cf204f2aa01b2d/packages/cadl-ranch-specs/http/models/usage/mockapi.ts#L16
For second one the 200 should be more reasonable compared with 202
https://github.com/Azure/cadl-ranch/blob/d1ffa96dc92bed37e96d05ca25cf204f2aa01b2d/packages/cadl-ranch-specs/http/models/usage/mockapi.ts#L17

Bug fix has been merged, closing


NSTextField in AppKit encounters a bug with 'backgroundColor': Ambiguous use of 'backgroundColor'.

Xcode 14.3 or Xcode 15.0 beta
macOS 13.4

I have the same issue. NSTextField already has its own backgroundColor implementation, leading to problems using it.

Hello @Vithanco @xtaykhksghy
For now you can fix this using (t as NSView).backgroundColor = .purple.


reactive support

any plan to support spring-boot-starter-data-redis-reactive?

When Jedis gets an async and a reactive API, we will. But there's no ETA or plans it on that yet.

@sazzad16 👀


Zero Value takes you back to home page on error (Bug or intentional)?

Im targeting the event for Zero Value checkout
Handle(OrderTransactionUpdatingNotification evt)

Inside this event i write some business logic to test the order doesnt exceed available stock. Following this i add the line
evt.Order.Finalize(....,Payment.Error);

This sets the order as an Error in the CMS however upon the error occurring it redirects me to the homepage. I looked into all the available menus and cant see where this would be set. I compared a payment provider with the Zero Value provider and the payment provider has a cancel URL but the Zero Value only has Continue Url for me to set.
This only occurs when setting Finalize and evt.Order.SetOrderStatus (that is it redirects back to the home page). If i remove these lines then it redirects to a successful page but for a failed order.
Is this intentional, a bug? I just need to to go to an error page when the transaction fails
@mattbrailsford Sorry to ping you in but dont suppose you could nudge me in the right direction?

Setting the payment status to error is a finalized status, meaning the order can't be processed any further and it is considered closed. If you wish to return the user back to the checkout you would need to leave the order unchanged to allow further updates, but then redirect to the checkout page yourself.
As I mentioned in other issues raised, you are probably better of implementing a validation handler, potentially in response to the final review step. There you can validate the acceptance and return the user to the cart if stock has changed.
I'm not 100% sure if this will all be possible using the Checkout add on as you may need some special error handling, but you can give the validation handler a try first and if that doesn't give the best result, then you may need to implement your own checkout flow with better handling in it.

Thank you @mattbrailsford
Im proceeding with the Validation and i think im almost there.
Changed the event from Notification to Valdate
public override void Validate(ValidateOrderTransactionUpdate evt)

I check the PaymentStatus to target the final step
if (evt.PaymentStatus.To == PaymentStatus.Captured)

Following this i am using the supplied code to check for stock (the one you linkd in my other post)
The stock comes back as 0 and i then add
evt.Fail("Out of sock");
This returns me to
https://localhost:12345/umbraco/commerce/payment/continue/zeroValue.....
Anything im missing?

Yea, I'd say that is too late as if the payment status is changing to captured, it means payment has already been taken. Your only real option is to check it before you send the customer to the payment gateway. There is a potential that someone could checkout at the same time, but it's how much of a risk that is for you.
If you have to absolutley make sure, then the only other option I could think is for you to implement your own payment provider that does the stock check just before capturing the payment.

Thanks again. I did the same but one step earlier which shows the error on the checkout page. I appreciate it cant be done at the last step but this would be enough for now. Thanks again for your input

You might be able to get away with just copying the review page view from here https://github.com/umbraco/Umbraco.Commerce.Checkout/blob/main/src/Umbraco.Commerce.Checkout/Views/UmbracoCommerceCheckout/UmbracoCommerceCheckoutReviewPage.cshtml and dropping into your project at Views/UmbracoCommerceCheckout/UmbracoCommerceCheckoutReviewPage.cshtml. This should then override the review page of the Checkout plugin and then you could tweak that to do an AJAX call to check stock before the submit button is triggered.
Either way, this is considered custom integration work and so isn't something Checkout will handle OOTB. Checkout is really meant for simple drop in solutions so as soon as you need custom behavior, it's intended you would go with a custom solution.

Sure, i understand where youre coming from that its starting to become more custom, if i start to add templates and start to tweak them. Im going to keep this in mind and see if we need to go to that extra step.
Just in case we do, the only thing to do here would be once the chtml file is copied across add an AJAX call to check stock (I assume it might have to call an API endpoint) and if its 0 stock disable the Prev/Next partial accordingly.
@Html.Partial("~/Views/UmbracoCommerceCheckout/Partials/UmbracoCommerceCheckoutPrevNext.cshtml")

Doing it this way would mean whatever functionality comes OOTB would remain should we change a payment provider or something in future and any future updates should be ok to apply as this folder/cshtml wont be overridden?

That's correct 👍


Fixed entity_types arr to sweep_scope

There was usually empty list of entity types sent to sweep_scope, this PR fixes it

@Ladas pls :eyes:

@miq-bot assign @Ladas
(testing miq-bot :)

@miq-bot add-label bug


Improve try-runtime::on_runtime_upgrade and fix some storage version issues

We now run all try_on_runtime_upgrade and collect the errors, instead of bailing on the first error. This makes it easier to see directly all the things that are failing instead of fixing one, recompiling and then checking the next one.
Then this pr also sets the correct storage version for pallet-bounties that was already correctly set in the storage with the appropriate migration.

bot merge


Fix fromModel to fromModal in modal docs

fromModel also exists but here the page talks about modals, so it should be fromModal.

Thanks, good catch.


Updated data loader in oxford_pets_image_segmentation.py

Updated custom data loader for image segmentation task.
X was in float32 datatype and Y was in unit8 datatype. It caused error in training.
I have now changed Y datatype to match X. Both are now float32.

@googlebot I signed it!

Hi, thanks for the PR. The example works fine without casting -- what error were you seeing?

Sir I was getting following error
TypeError                                 Traceback (most recent call last)
 in ()
17    validation_data=prepro_val
18 )'''
---> 19 model.fit_generator(prepro_tr,epochs= 5, validation_data= prepro_val )
2 frames
/usr/local/lib/python3.7/dist-packages/tensorflow/python/framework/func_graph.py in autograph_handler(*args, **kwargs)
1127           except Exception as e:  # pylint:disable=broad-except
1128             if hasattr(e, "ag_error_metadata"):
-> 1129               raise e.ag_error_metadata.to_exception(e)
1130             else:
1131               raise
TypeError: in user code:
File "/usr/local/lib/python3.7/dist-packages/keras/engine/training.py", line 878, in train_function  *
    return step_function(self, iterator)
File "/usr/local/lib/python3.7/dist-packages/segmentation_models/base/objects.py", line 114, in __call__  *
    return self.l1(gt, pr) + self.l2(gt, pr)
File "/usr/local/lib/python3.7/dist-packages/segmentation_models/losses.py", line 130, in __call__  *
    return F.binary_crossentropy(gt, pr, **self.submodules)
File "/usr/local/lib/python3.7/dist-packages/segmentation_models/base/functional.py", line 256, in binary_crossentropy  *
    return backend.mean(backend.binary_crossentropy(gt, pr))
File "/usr/local/lib/python3.7/dist-packages/keras/backend.py", line 5158, in binary_crossentropy
    return tf.nn.sigmoid_cross_entropy_with_logits(labels=target, logits=output)

TypeError: Input 'y' of 'Mul' Op has type int8 that does not match type float32 of argument 'x'.


switch back to sarama

this is replacing confluent with sarama again. i verified that it is functional, although i did not do much load testing yet. a lot of the code is copied from the version before the switch to confluent

any more notes on why the switch back? We observed performance improvements in our Kafka cluster with the confluent lib.

Found it https://github.com/raintank/tsdb-gw/issues/99

other then @jtlisi's comments, this looks good to me. :+1:

@tehlers320 i replied to you on slack

git rm -r vendor/github.com/edenhill

do we still need all these libs?
$ git diff 0c201c80c3ce82d9807da3baba8495c4ac75bda8 HEAD
diff --git a/.circleci/config.yml b/.circleci/config.yml
index 39f4a73..2cb79d9 100644
--- a/.circleci/config.yml
+++ b/.circleci/config.yml
@@ -5,6 +5,7 @@ jobs:
     docker:
       - image: circleci/golang:1.9.3
     steps:
+      - run: sudo apt-get install libssl-dev openssl libsasl2-2 libsasl2-dev zlib1g-dev
       - checkout
       - run: scripts/build.sh
       - persist_to_workspace:
@@ -17,9 +18,9 @@ jobs:
     docker:
       - image: circleci/golang:1.9.3
     steps:
+      - run: sudo apt-get install libssl-dev openssl libsasl2-2 libsasl2-dev zlib1g-dev

for completeness, the last sarama used pre-confluent was 1.12.0 (2017-05-08)

looking good now, i'm just doing a bit of load and functionality testing

experiments done using the docker-dev-custom-cfg-kafka stack in MT repo
and fakemetrics feed --gnet-addr http://localhost:9000/metrics --gnet-key not_very_secret_key --mpo 80000 --orgs 1
I also confirmed with fake-metrics-data dashboard the data looks good.
confluent
metrictank master, but updated to use gw 0.8.0-149-gcb39554 (latest gw with confluent, just before switch to sarama) and metrics-buffer-max-ms = 50
https://snapshot.raintank.io/dashboard/snapshot/y7PUwSGUZ3ObRLo21MjWFS66JFEaCWZ8?orgId=2
https://snapshot.raintank.io/dashboard/snapshot/GYcDDJGCZ7D8Pq4hvMezFvUHz0oiQtpb?orgId=2
latest gw with sarama (code in this PR)
metrictank branch new-tsdbgw-sarama , gw config basically default (just with stats and publishing enabled basically)
https://snapshot.raintank.io/dashboard/snapshot/ghXNdeeVFkYaaOGStHX53zCfW5yqM2C7?orgId=2
https://snapshot.raintank.io/dashboard/snapshot/RRv39ulD1r6ciWDR1XR1jtw26PlEvAXo?orgId=2


better error logging

will hit

[x] #956
[x] #1127
[x] #1133
[x] #1117

Adds shell.log for logging errors and messages from Atom Shell. shell.log and app.log now have timestamped errors, stack traces, and log exit codes when the application quits or crashes.

Looks great @camilleanne \o/


[#1530] handle exception when request entity don't support getContent

handle exception when request entity don't support getContent

please review @emeroad


add --verbose flag to always report status

When using large batchsizes (eg., 1000), it is possible to go long
periods of times (over a minute) without a status message being
disabled.

Ping?


Android Performance Issues

[ ] Large images in the message list kill scrolling speed
[ ] Drawers are not animating smoothly (will be fixed with #521)
[ ] Drawers buttons sometimes don't work or appear laggy (cause unknown)

The release Android build exhibits performance issues.
The causes might be one or several.
This is a public release blocking issue.

@zulipbot claim

Drawers buttons sometimes don't work or appear laggy (cause unknown)

I probably found a reliable way to reproduce this bug. It happens when new data is fetched in the message list (on both platforms). Also during the load, all touchables didn't seem to respond.

Maybe it is during the actual render phase since I don't think the fetching should be a blocking action.
But your hunch sounds reliable, maybe test a bit more?
A suggestion on how to test: edit the config.js file and change messagesPerRequest: 50, to something much larger. Add console.log on start fetch and end fetch and see if fetching is the issue or rendering.

@borisyankov I think everyone should do that on other Android Versions.

👍  its probably the render phase, I profiled this before and found expensive component update calls.

A suggestion on how to test: edit the config.js file and change messagesPerRequest: 50, to something much larger. Add console.log on start fetch and end fetch and see if fetching is the issue or rendering.

great, will also try this next

Closing as fixed.


Remove old dependence_* variables

Remove the custom dependence_* variable from module release

[ ] concourse
[x] cert_manager
[x] external_dns
[x] external_secrets_operator
[x] ingress_controllers_v1
[x] modsec_ingress_controllers_v1
[x] monitoring
[x] velero
[ ] kuberhealthy
[x] logging

We have 8 out of 10 modules moved away from the old way of doing depends_on.
Concourse - we've not touched this since it's not part of a normal test cluster build. It will be updated at some point in the future though.
Kuberhealthy - we had issues with this module:
Error: Invalid count argument

  on .terraform/modules/kuberhealthy/main.tf line 72, in resource "kubectl_manifest" "namespacecheck_rule_alert":

  72:   count      = length(data.kubectl_path_documents.namespace_check_manifests.documents)

The "count" value depends on resource attributes that cannot be determined

until apply, so Terraform cannot predict how many instances will be created.

To work around this, use the -target argument to first apply only the

resources that the count depends on.

FATA[1561] failed to apply terraform: exit status 1

Error: Invalid count argument

  on .terraform/modules/kuberhealthy/main.tf line 72, in resource "kubectl_manifest" "namespacecheck_rule_alert":

  72:   count      = length(data.kubectl_path_documents.namespace_check_manifests.documents)

The "count" value depends on resource attributes that cannot be determined

until apply, so Terraform cannot predict how many instances will be created.

To work around this, use the -target argument to first apply only the

resources that the count depends on.  subcommand=create-cluster

Where possible we've removed dependencies as well for example gatekeeper no longer requires any deps to install. Further work should be done to remove more dependencies with the ultimate goal being able to create a parameterised cluster build process.

infrastructure testing branch:
https://github.com/ministryofjustice/cloud-platform-infrastructure/compare/main...test-branch

Further testing:
We need to spin up a test cluster and apply our test-branch changes to see what happens in a semi-live environment.
Next steps for final release:

Decide on day to pause infrastructure work.
Notify cp channel of pause and plan of work
Cut releases for all new modules, rebasing on top of new changes that have come in
Update test-branch with release tags
Build custom-cluster as a check to see that it builds correctly
Destroy custom-cluster using concourse job
???

On hold till post 1.26


Realm crash: terminating with uncaught exception of type realm::KeyNotFound: No such object

Goals
I want my app to work without crashes
Expected Results
I expected my app to not crash
Actual Results
Thread 10#0	0x00007fff6112f766 in kevent ()
#1	0x000000010423bcb9 in realm::util::network::Service::IoReactor::wait_and_activate(std::__1::chrono::time_point<std::__1::chrono::steady_clock, std::__1::chrono::duration<long long, std::__1::ratio<1l,<PHONE_NUMBER>l> > >, std::__1::chrono::time_point<std::__1::chrono::steady_clock, std::__1::chrono::duration<long long, std::__1::ratio<1l,<PHONE_NUMBER>l> > >) ()
#2	0x000000010423c342 in realm::util::network::Service::IoReactor::wait_and_advance(std::__1::chrono::time_point<std::__1::chrono::steady_clock, std::__1::chrono::duration<long long, std::__1::ratio<1l,<PHONE_NUMBER>l> > >, std::__1::chrono::time_point<std::__1::chrono::steady_clock, std::__1::chrono::duration<long long, std::__1::ratio<1l,<PHONE_NUMBER>l> > >, bool&, realm::util::network::Service::OperQueue<realm::util::network::Service::AsyncOper>&) ()
#3	0x000000010423cc95 in realm::util::network::Service::Impl::run() ()
#4	0x00000001041b331d in realm::sync::Client::run() ()
#5	0x000000010434ba4d in void* std::__1::__thread_proxy<std::__1::tuple<std::__1::unique_ptr<std::__1::__thread_struct, std::__1::default_delete<std::__1::__thread_struct> >, realm::_impl::SyncClient::SyncClient(std::__1::unique_ptr<realm::util::Logger, std::__1::default_delete<realm::util::Logger> >, realm::SyncClientConfig const&, std::__1::shared_ptr<realm::SyncManager const>)::'lambda0'()> >(void*) ()
#6	0x00007fff61167109 in _pthread_start ()
#7	0x00007fff61162b8b in thread_start ()

Thread 11#0	0x00007fff6112d882 in __psynch_cvwait ()
#1	0x00007fff61167425 in _pthread_cond_wait ()
#2	0x000000010423fc52 in realm::util::network::Service::Impl::resolver_thread() ()
#3	0x000000010423fb2d in void* std::__1::__thread_proxy<std::__1::tuple<std::__1::unique_ptr<std::__1::__thread_struct, std::__1::default_delete<std::__1::__thread_struct> >, realm::util::network::Service::Impl::add_resolve_oper(std::__1::unique_ptr<realm::util::network::Service::ResolveOperBase, realm::util::network::Service::LendersOperDeleter>)::'lambda'()> >(void*) ()
#4	0x00007fff61167109 in _pthread_start ()
#5	0x00007fff61162b8b in thread_start ()

Realm notification listener (12)#0	0x00007fff6112f766 in kevent ()
#1	0x0000000104246908 in realm::_impl::ExternalCommitHelper::listen() ()
#2	0x0000000104246a8e in void* std::__1::__thread_proxy<std::__1::tuple<std::__1::unique_ptr<std::__1::__thread_struct, std::__1::default_delete<std::__1::__thread_struct> >, realm::_impl::ExternalCommitHelper::ExternalCommitHelper(realm::_impl::RealmCoordinator&)::$_0> >(void*) ()
#3	0x00007fff61167109 in _pthread_start ()
#4	0x00007fff61162b8b in thread_start ()

CRASH HERE ===> Realm notification listener (13)#0	0x00007fff6113133a in __pthread_kill ()
#1	0x00007fff61166e60 in pthread_kill ()
#2	0x00007fff200fab94 in abort ()
#3	0x00007fff20253818 in abort_message ()
#4	0x00007fff20244e65 in demangling_terminate_handler() ()
#5	0x00007fff201780d9 in _objc_terminate() ()
#6	0x00007fff20252c47 in std::__terminate(void (*)()) ()
#7	0x00007fff202555b6 in __cxa_rethrow ()
#8	0x0000000104246b38 in void* std::__1::__thread_proxy<std::__1::tuple<std::__1::unique_ptr<std::__1::__thread_struct, std::__1::default_delete<std::__1::__thread_struct> >, realm::_impl::ExternalCommitHelper::ExternalCommitHelper(realm::_impl::RealmCoordinator&)::$_0> >(void*) ()
#9	0x00007fff61167109 in _pthread_start ()
#10	0x00007fff61162b8b in thread_start ()

Console:
uncaught exception in notifier thread: N5realm11KeyNotFoundE: No such object
libc++abi.dylib: terminating with uncaught exception of type realm::KeyNotFound: No such object
terminating with uncaught exception of type realm::KeyNotFound: No such object

Steps for others to Reproduce
I have a pretty complex setup but I can provide all details if needed. The crash reproduces with 100% frequency. A brief explanation below.
I have 2 Mongo Atlas databases. The first for public data to which all users have access. The second for user data. Public data is lazily loaded from the network using an insert trigger for one of the user collections. The app listens for a user object in the user database and for a container object in the public database using separate Realms.
The container object is always present and has relationships to all other public data and just aggregates this data. The app observes that object. Realm function adds leaf objects and so the container data became available when relationships are resolved.
So what I got from logs is that when I add an object to the user database I receive an update about that, then Realm trigger calls function to update 3 different tables with data when it received from the network and app crashes after that, probably on trying to fetch an update. It might be a race condition of some sort because the Realm function updates those objects concurrently.
The app works fine after restart. I tried to terminate and start sync again without success. I tried to erase and fetch all data without success. I tried to deleted and install the app without success.
Update 1: I tried to put 5 seconds delay between fetches but it didn't help so it looks like the problem is more general.
Update 2: When I removed container object observation the app stopped to crash and works fine even after the restart when new data is populated. So the root cause seems to be the observation during relationship resolve from nil to some new object.
App logs:
04.03 05:03:34.282 | R | Performing write
04.03 05:03:34.286 | R | User Realm update
04.03 05:03:34.287 | R | Deletions: [] | Insertions: [] | Modifications: [0]
04.03 05:03:34.287 | R | User objects count: 1
uncaught exception in notifier thread: N5realm11KeyNotFoundE: No such object
2021-04-03 05:03:35.514950+0300 Divtracker[74750:22811057] uncaught exception in notifier thread: N5realm11KeyNotFoundE: No such object
libc++abi.dylib: terminating with uncaught exception of type realm::KeyNotFound: No such object
terminating with uncaught exception of type realm::KeyNotFound: No such object

Trigger logs:
Logs:
[
  "04.03 02:03:34.626 | Symbol: EVV",
  "04.03 02:03:34.645 | Company for symbol 'EVV' not found. Fetching and fixing data...",
  "04.03 02:03:34.646 | Previous day price for symbol 'EVV' not found. Fetching and fixing data...",
  "04.03 02:03:34.731 | Previous day price for symbol 'EVV' not found. Fetching and fixing data...",
  "04.03 02:03:34.791 | Inserting company for symbol 'EVV'...",
  "04.03 02:03:34.791 | Company for symbol 'EVV' successfully inserted",
  "04.03 02:03:34.850 | Inserting previous day price for symbol 'EVV'...",
  "04.03 02:03:34.850 | Previous day price for symbol 'EVV' successfully inserted",
  "04.03 02:03:34.946 | Inserting dividends container for symbol 'EVV'...",
  "04.03 02:03:34.947 | Dividends for symbol 'EVV' successfully inserted"
]

Sync logs:
OK - Apr 03 5:03:35+03:00 - 61ms - SyncWrite

Source:

Write originated from MongoDB
Logs:
[
  "Upload message contained 1 changeset(s)",
  "Integrating upload required conflict resolution to be performed on 0 of the changesets",
  "Latest server version is now 6"
]
Partition:

P
Write Summary:
{
  "CompanyRealmModel": {
    "inserted": [
      "EVV"
    ]
  },
  "DividendsContainerRealmModel": {
    "inserted": [
      "EVV"
    ]
  },
  "PreviousDayPriceRealmModel": {
    "inserted": [
      "EVV"
    ]
  }
}

Code Sample
I can provide Realm functions, schemes, objects, and code I use for objects observing later if needed.
Version of Realm and Tooling
ProductName:	Mac OS X
ProductVersion:	10.15.6
BuildVersion:	19G2021

/Applications/Xcode.app/Contents/Developer
Xcode 12.4
Build version 12D4e

/usr/local/bin/pod
1.10.1
Realm (10.7.2)
RealmSwift (10.7.2)

/bin/bash
GNU bash, version 3.2.57(1)-release (x86_64-apple-darwin19)

/usr/local/bin/carthage
0.37.0
(not in use here)

/usr/bin/git
git version 2.24.3 (Apple Git-128)

The other thing I noted is that when I delete leaf objects they are no longer available when I use realm.objects(Leaf.self) but they still exist in the container object. Do I need to delete them manually? It doesn't look right though 😕

I reworked db not use any relationships and collecting everything manually as a workaround

Hey @anton-plebanovich !
Thanks for providing all the information. Since you already noted you can reproduce this issue 100% I think it would be a good idea to just use that as a starting point.
In case this information (code, repo, etc.) is not supposed to be posted publicly, can you send it to<EMAIL_ADDRESS>so we can have a look at it?
Thank you!

Hey @DominicFrei , thanks for the response. I already reworked the database and the source code for the app but I'll try to set up a demo project you can use tomorrow.

@anton-plebanovich  Alright. :)
Glad to hear your rework works for you. No rush with the repo then. Whenever you'll be able to put something together we'll sure appreciate it! 👍

@DominicFrei sadly, I can't reproduce it anymore I tried to recreate the same setup but everything just works right now. I think we can close it as not reproducible or I can try it during the week since it might be some weird race conditions related to the network or server load I can't capture right now.

@anton-plebanovich Thank you for the feedback. I'll close it then and in case you stumble upon it again I'm happy to re-open as soon as we got more information. 👍

I'm hitting this same crash now, after moving my previously-fine app to the new @Persisted property wrapper. I don't have anything more to report currently, but just adding a data point.

@bdkjones I have seen a similar issue where this exception occurs when a primary key value is nil but the property is declared as non nil. I had solved it by declaring my property as optional.
e.g.
@Persisted(primaryKey: true) var _id: ObjectId
// becomes
@Persisted(primaryKey: true) var _id: ObjectId?

Maybe you are hitting something similar?


No tests

I wonder how would one be confident using this package without some concrete tests. You guys should think about it!!

See #42

I've been very busy in recent months with my work projects, which eventually pulling away of my personal projects, so it could not provide the necessary support to you, I am more relaxed now and I'm checking all the issues in order to work on fixes and new implementations, apologize to everyone.


Confirmation on exit if "Disable Automatic Suspending" is active

If "Disable Automatic Suspending" is active and user will clicks on close button, there could be confirmation dialog to prevent the app from accidentally closing:
https://gnome.pages.gitlab.gnome.org/libadwaita/doc/main/class.MessageDialog.html

fixed in https://github.com/sigmaSd/Stimulator/commit/a54db66c4e6b43ca72319e7ee4965f43ad8fdaad


January Update 1

[ ] Implements schemas update defined in https://github.com/nlpsandbox/nlpsandbox-schemas/pull/135
[ ] Configure openapi-generator to not save _all_of.py files (see ROCC server)

Flask-Testing doesn't seem to have an assert201

Flask-Testing doesn't seem to have an assert201


Refactored project build, bumped dependencies

Refactored Build.scala
Bumped SBT, Scala versions
Bumped plugins & library dependencies
GitHub is https by default
Removed legacy Eclipse project settings
Also some minor tweaks:
Removed unused import
Removed legacy Scala version switcher for scalatest
Guarded on resolution of credentials file so that its non-existence doesn't spam in SBT console
Added some SBT best practices (CrossVersion.partialVersion, Def.setting)

add to whitelist

This project configuration would very easily be converted into a multi-project build.sbt, as that seems to be the best-practice nowadays (with auto-plugins such as assembly, buildinfo)...
If you wish, I could update the PR with a build.sbt definition?

Beside of one change, LGTM. If you want you can convert the .scala into a .sbt one.

Done. I took the liberty of splitting it into build.sbt and version.sbt, my rationale being the assumption that the latter will probably be more frequently updated than the actual build definition.

LGTM. Thanks!


How do I set the plug-in to ignore the prompts in the YML file

The experience I had using the plug-in in a Docker-composer. Yml file was far less useful than other plug-ins provided by Vscode, so a hint that I would like to disable the plug-in when using a YML file is there any way

@AirBashX  - you can disable Tabnine in vscode by setting  tabnine.disable_file_regex:  ["[.]yml$"]


Update documentation for v1.2.0 and wiki

The documentation needs to be updated to reflect v.1.2.0. We should also address the issues raised in #92 and #87 and possibly #47.
I've started editing the readme to address these. However, the readme is starting to feel bloated and clunky. @apclary @n1ru4l, how do you feel about moving the bulk of the installation and usage sections to a wiki?
I'll put in a draft PR soon with everything in the readme so you can see what it would look like.

That sounds good, I think target first “what is it” and “how do I run it as
a DM” in that order and development notes can go elsewhere with a link. The
screenshots could probably use updating.
On Tue, Jun 25, 2019 at 11:44 AM maxb2<EMAIL_ADDRESS>wrote:

The documentation needs to be updated to reflect v.1.2.0
https://github.com/apclary/dungeon-revealer/releases/tag/v1.2.0. We
should also address the issues raised in #92
https://github.com/apclary/dungeon-revealer/issues/92 and #87
https://github.com/apclary/dungeon-revealer/issues/87 and possibly #47
https://github.com/apclary/dungeon-revealer/issues/47.
I've started editing the readme to address these. However, the readme is
starting to feel bloated and clunky. @apclary https://github.com/apclary
@n1ru4l https://github.com/n1ru4l, how do you feel about moving the
bulk of the installation and usage sections to a wiki?
I'll put in a draft PR soon with everything in the readme so you can see
what it would look like.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apclary/dungeon-revealer/issues/94?email_source=notifications&email_token=AAGW6T36QO7OODSP2GLI3STP4JDPLA5CNFSM4H3KH7J2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G3TEN7A,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGW6T44Z57Q5H6CNZYUOBDP4JDPLANCNFSM4H3KH7JQ
.

--
Alex Clary

Yeah development specific stuff could be moved into a Contributing.md

Also I am still pretty sure that there must be a way to make those binaries executable without relying on the user to do so 🤔

Making binaries executable after download is a pretty standard practice for Linux. Maybe it works on other platforms. You can take a look at my fork with travis doing the chmod. Here is my release that is supposed to be executable.

@maxb2 On OSX (for most executables) I usually only have to do the following: https://www.howtogeek.com/205393/gatekeeper-101-why-your-mac-only-allows-apple-approved-software-by-default/
Now I have to make it executable first 😕
I will try to investigate further, dunno about linux.
My intention is to make using dungeon-revealer as easy as possible without shying away users because they have to open the terminal 😕

I agree that we should make it as easy to use as possible. If we want to move away from the terminal, then we should find another way to display the url to the user. Right now, doubling clicking the binary just starts the process silently. If I were a normal user, I wouldn't know where to go to use the app.

It might not be too difficult to just sign our app. https://developer.apple.com/developer-id/

Closed via #95


Add additional Azure usage types

I've added the additional usage types based on the additional subscriptions we now have access to. Moving this to review.


easypy.collections - bug in predicate ordering with FilteredCollection

>> ListCollection([Bunch(a='b', b=5), Bunch(a='c', c='c')]).filtered(a='b').select(lambda o: o.b > 4)
AttributeError: <class 'easypy.bunch.Bunch'> has no attribute 'b'

I think, rather than trying to fix the order, we should make the order irrelevant. I suggest that if we get an exception, we keep applying the rest of the predicates and only rethrow if no predicate returned False:
    for obj in objects:
        first_exception = None
        for pred in preds:
            try:
                if not pred(obj):
                    break
            except Exception as e:
                if first_exception is None:
                    first_exception = e
        else:
            if first_exception is not None:
                raise first_exception
            yield obj

This will make the order irrelevant - and we could just do this:
>> ListCollection([Bunch(a='b', b=5), Bunch(a='c', c='c')]).select(lambda o: o.b > 4, a='b')
Out[5]: [Bunch(a='b', b=5)]

@koreno @DoronCohen @YuvalEvron


feat: CLI Custom Prompts [HACKATHON]

🔧 Changes

📚 References

🔬 Testing

📝 Checklist

[ ] All new/changed/fixed functionality is covered by tests (or N/A)
[ ] I have added documentation for all new/changed functionality (or N/A)

Closing this PR in favor of #1031 because we needed to make additional changes and do not have access to update the original branch. Please refer to the new PR for continued work on this feature.


'Webmail Login ID' on the login page

It cannot make sense.

Checked the source that it was for in-house use strings.


Reject transaction

Frontend PR https://github.com/opencollective/opencollective-frontend/pull/5070
Relates https://github.com/opencollective/opencollective/issues/3484

Adds capability for Collective admins to refund transactions
Adds 'isRejected' field to Transaction v2 object
Adds rejectTransaction mutation which:

refunds the transaction
changes the related Order status to REJECTED
destroys the membership with the role BACKER

We can iterate in another PR, but it would be great to have some tests to make sure permissions are properly covered. Things that would be interesting to test:

canReject: Collective admins can only refund during 30 days
rejectTransaction

Can't reject if not root, host admin or collective admin
Refunds the transaction, removes all memberships and mark the order as rejected

I have added the permissions check for canReject then in the condition that it doesn't go on to legacyRefundTransaction, do you think that's sufficient?

@sbinlondon I'd do the check just after fetchTransactionWithReference, it's not a big deal if we check two times (in case we're refunding) and that way we can provide an unified error message (without relying on the one from legacyRefundTransaction). But that's details, no requested change.


SARAALERT-1509: When a symptom report is edited, include the change in "Reporter" in the resulting history entry

Description
Jira Ticket: SARAALERT-1509
When a symptom report is edited, there's currently no way to track changes to the reporter, which is always updated to the last person to change the record. This appends "Reporter updated("old_id" to "new_id")" to the end of the original comment of the update history so these changes are always tracked.
(Feature) Demo/Screenshots

Important Changes
Please list important files (meaning substantial or integral to the PR) along with a list of the general changes that should be highlighted for reviewers.
app/controllers/assessments_controller.rb

refactors update to, if changes were made to the assessment, adds the old who_reported value and the new who_reported to the end of the history comment iff they do not equal each other.

Testing Recommendations
Edge cases: If a user updates the same record twice, if two different users update the same record in sequence.
Checklists
Submitter:

[x] This PR describes why these changes were made.
[x] This PR is into the correct branch.
[x] This PR includes the correct JIRA ticket reference.
[x] Comment added to the relevant JIRA ticket(s) with a link to this PR
[x] Code diff has been reviewed (it does not contain: additional white space, not applicable code changes, debug statements, etc.)
[x] If UI changes have been made, Chrome Dev Tools Lighthouse accessibility test has been executed to ensure no 508 issues were introduced.
[x] Tests are included and test edge cases
[x] Tests have been run locally and pass (remember to update Gemfile when applicable)
[x] Test fixtures updated and documented as necessary

@jkufro :

[x] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[x] The tests appropriately test the new code, including edge cases
[x] You have tried to break the code
[x] If applicable, you have tested changes against a large database, and considered possible performance regressions
[x] Tested all recommendations listed in the "Testing Recommendations" section. The application behaves as expected with this PR.

@timwongj :

[x] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[x] The tests appropriately test the new code, including edge cases
[x] You have tried to break the code
[x] If applicable, you have tested changes against a large database, and considered possible performance regressions
[x] Tested all recommendations listed in the "Testing Recommendations" section. The application behaves as expected with this PR.

@tstrass :

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code
[ ] If applicable, you have tested changes against a large database, and considered possible performance regressions
[ ] Tested all recommendations listed in the "Testing Recommendations" section. The application behaves as expected with this PR.

@jkufro @tstrass this should be ready for a re-review now!


langchain-google-genai handling of "system messages/prompt/context" broken

Hi Team,
In the current implementation of google-genai whenever the library encounters a system messages which is carrying additional context or a system wide prompt, it is concatenated into the user part.
https://github.com/langchain-ai/langchainjs/blob/9adb2731bff5f741301f3a721588c474f9d73027/libs/langchain-google-genai/src/utils.ts#L134
This is resulting in a request which has multiple user parts and is not interpreted by the Gemeni pro model
e.g
{
  "generationConfig": {
    "candidateCount": 1,
    "stopSequences": [],
    "temperature": 0.1,
    "topP": 1,
    "topK": 1
  },
  "safetySettings": [],
  "contents": [
    {
      "role": "user",
      "parts": [
        {
          "text": "<System Prompt>"
        },{
          "text": "User Prompt"
        }

      ]
    }
  ]
}

This is specifically happening when we are using the libary in a chain e.g RetrievalQAChain with a retriever like an implementation of a MemoryVectorStore which is using the GoogleGenerativeAIEmbeddings
ideally for this to work we need to send the system prompt as a separate part and also add a model response to it ( as the model expects alternating user and model responses ) like shown below
{
  "generationConfig": {
    "candidateCount": 1,
    "stopSequences": [],
    "temperature": 0.1,
    "topP": 1,
    "topK": 1
  },
  "safetySettings": [],
  "contents": [
    {
      "role": "user",
      "parts": [
        {
          "text": "<System Prompt>"
        }
      ]
    },
    {
      "role": "model",
      "parts": [
        {
          "text": "Understood, I will use these rules to answer the subsequent questions""
        }
      ]
    },
    {
      "role": "user",
      "parts": [
        {
          "text": "<User Prompt>"
        }
      ]
    }
  ]
}

For this to work
We will have to change the method convertBaseMessagesToContent https://github.com/langchain-ai/langchainjs/blob/9adb2731bff5f741301f3a721588c474f9d73027/libs/langchain-google-genai/src/utils.ts#L93
We might have to update the case
if (acc.mergeWithPreviousContent) {
            const prevContent = acc.content[acc.content.length - 1];
            if (!prevContent) {
                throw new Error("There was a problem parsing your system message. Please try a prompt without one.");
            }
            // prevContent.parts.push(...parts);
           // Since this will be called after the system prompt is added add a model prompt and dont concatenate the parts
            acc.content.push({role:"model", parts:[{"text": "Understood, I will use these rules to answer the subsequent questions"}]});
            acc.content.push({role:"user", parts:parts});
            return {
                mergeWithPreviousContent: false,
                content: acc.content,
            };
        }

or when a system prompt is detected we can insert a new user param and not concatenate it with the other user one and also add a model response in the end which Signals that them model should follow these rules.

Hey Suchit,
Concatenating messages is just a workaround because Gemini is not supporting system messages at the moment.
I don't think that inserting artificial model responses into prompt as base functionality is a good idea.
If you want to do it this way you can implement it in your code, i.e. not push any system messages.


Fix for /devicesview endpoint not returing device group info.

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
What is it?

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[ ] Tests update
[ ] Refactor

Checklist:

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[X] I run go fmt ./... to check that my code is properly formatted
[X] I run go vet ./... to check that my code is free of common Go style mistakes

Test results are available in Ibutsu. The test run IDs are:
8efcc384-6db5-4745-91b0-748ed1e13ed4
e448cfe2-ec3f-44ea-8f41-ac1feb406179

Codecov Report

Merging #977 (4544404) into main (0fb170a) will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #977      +/-   ##
==========================================
+ Coverage   30.75%   30.86%   +0.10%
==========================================
  Files          45       45
  Lines        5466     5466
==========================================
+ Hits         1681     1687       +6
+ Misses       3547     3543       -4
+ Partials      238      236       -2

Impacted Files
Coverage Δ

pkg/services/devices.go
61.34% <100.00%> (+1.26%)
: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 0fb170a...4544404. Read the comment docs.

Test results are available in Ibutsu. The test run IDs are:
26cd4833-4c43-4e7e-aeeb-054a6810f836
cbc36d6c-a4ee-47f5-b23c-931d51c7f4ff


Dogecoin Improvement Proposal (DOIP-XXXX) Block Size Increase This proposal suggests increasing Dogecoin's block size to enhance scalability.

Dogecoin Improvement Proposal (DOIP-XXXX)
Block Size Increase This proposal suggests increasing Dogecoin's block size to enhance scalability. The proposed changes are encapsulated in a C++
Implementation Details
The DOIPXXXX class contains a method named implementBlockSizeIncrease, which accepts a Network object as a parameter. This method sets the block size to 8 megabytes, aiming to improve scalability.
Usage
To apply this proposal, create an instance of the DOIPXXXX class and call the implementBlockSizeIncrease method with the initialized Network object. This should be executed in the main function.

Very technology.


Better error report for max traces exceeded

In my experience, when chex reports max traces exceeded, it's usually because of me passing parameters to the function with different shapes or data types. Is it possible for chex to report such inconsistency?
e.g.,
AssertionError: [Chex] Function '_wrapper' is traced > 1 times!
Difference in input shapes. Last time variable `x` traced with shape "(10, 1)", this time traced with shape "(9, 1)".

Hey, thanks for the question - yes, it can be added! Would you like to send a PR?

@hbq1 I actually don't know how to do it in chex.


initial my.core.query implementation

This is already quite complicated, so I'll probably add a subcommand to hpi like my hpi_query script in another PR (perhaps while migrating it to click?; or maybe thats another ordeal on its own...)
Opening this PR so that this may be reviewed, and for possible feedback/changes
Notes:

parse_timedelta_string isn't currently used, it'll eventually be used in the CLI interface to be able to parse durations like this, so you could do something: hpi query module function --within 6h to query for items from that function within the last 6 hours.
I'm a bit unsure on some of the typing, generics isn't my strong-suit, so some things may be wrong.
Tests still need to be added for this, but I'm unsure how to do that? Since this requires some 'data source' to use query. Should I just create a 'dummy' namedtuple/dataclass in tests/query.py and use that? or do you already have something here I could use
Should docs be created for this? I wrote a substantial docstring for the select function, so hopefully thats understandable. With some examples as tests perhaps thats good enough?
Totally open to changes, this was a rough implementation with the first ideas that come to mind
Though the CLI interface doesn't current exist, this works when calling from in python, e.g.:

import my.browsing
import my.mpv
import my.zsh
import my.ttt
import my.reddit

from itertools import chain

# wrap items which dont have datetime as a NT/DT attribute

def wrapped_reddit():
    for r in my.reddit.comments():
        yield {
            attr: getattr(r, attr)
            for attr in
            ("created", "url", "text")
        }

# any mixed events could be chained here, if were using `order_value`
events = chain(
    wrapped_reddit(),
    my.zsh.history(),
    my.ttt.history(),
    my.mpv.history()
    my.browsing.history()

)

from my.core.query import select

timeline = select(events, order_value=lambda o: isinstance(o, datetime))

count = 0
for item in timeline:
    count += 1
    print(item)

print(count)

As some more basic examples
In [8]: list(select(("my.zsh", "history",), limit=5))
Out[8]:
[Entry(dt=datetime.datetime(2020, 5, 18, 8, 23, 22, tzinfo=datetime.timezone.utc), duration=0, command='z'),
 Entry(dt=datetime.datetime(2020, 5, 18, 8, 23, 22, tzinfo=datetime.timezone.utc), duration=0, command='en env_config.zsh'),
 Entry(dt=datetime.datetime(2020, 5, 18, 8, 23, 22, tzinfo=datetime.timezone.utc), duration=0, command='ls'),
 Entry(dt=datetime.datetime(2020, 5, 18, 8, 23, 22, tzinfo=datetime.timezone.utc), duration=0, command='ec'),
 Entry(dt=datetime.datetime(2020, 5, 18, 8, 23, 12, tzinfo=datetime.timezone.utc), duration=0, command='ls')]

In [9]: next(select(("my.zsh", "history",), where=lambda c: "ffmpeg" in c))
Out[9]: Entry(dt=datetime.datetime(2020, 9, 7, 7, 31, 49, tzinfo=datetime.timezone.utc), duration=0, command='ffmpeg')

In [1]: import my.zsh; from datetime import date; td = date.today()

In [3]: # more analysis like, how many zsh commands today

In [4]: # can also pass function without calling

In [7]: ilen(select(my.zsh.history, where=lambda o: o.dt.date() == td))
Out[7]: 2107

In [16]: from my.body import food

In [17]: # thing with most calories?

In [18]: next(select(food, order_key="calories", limit=1, reverse=True))
Out[18]: Food(when=datetime.datetime(2020, 11, 30, 2, 47, 7, tzinfo=datetime.timezone.utc), food='barbeque corn chips', calories=1125)

Related issue: https://github.com/karlicoss/HPI/issues/138

parse_timedelta_string isn't currently used, it'll eventually be used in the CLI interface to be able to parse durations like this, so you could do something: hpi query module function --within 6h to query for items from that function within the last 6 hours.

Oh nice -- definitely useful. I'm usually using pandas.to_timedelta in ad-hoc applications. Guess we want to keep it decoupled from pandas, but guess good to keep it compatible with it? (it is now, just for the future). But perhaps this won't need to be more complicated than it's now.

Tests still need to be added for this, but I'm unsure how to do that? Since this requires some 'data source' to use query. Should I just create a 'dummy' namedtuple/dataclass in tests/query.py and use that? or do you already have something here I could use

Yeah -- I've added some test data repos as submodules: https://github.com/karlicoss/HPI/blob/master/.gitmodules#L1-L6
For example, these run on CI, against 'real' data
https://github.com/karlicoss/HPI/blob/1cdef6f40a6e4c0dcb545a1f9e0e592f909301f3/tests/bluemaestro.py#L41-L46
https://github.com/karlicoss/HPI/blob/1cdef6f40a6e4c0dcb545a1f9e0e592f909301f3/tests/location.py#L35-L38
But perhaps for this would make sense to test on less specific things (since it doesn't need modules even, just takes iterables) Yeah, like dummy dataclasses.

Should docs be created for this? I wrote a substantial docstring for the select function, so hopefully thats understandable. With some examples as tests perhaps thats good enough?

Yeah, that's awesome thanks. Hmm maybe it's about time to start doc/FEATURES.org or something like that? Could at least mention it and point at the docstring/tests, probably no need for too much duplication. Maybe later it would make sense to write docstrings for the most useful core functions and generate some automatic html docs for it, or somathing like that.

But perhaps this won't need to be more complicated than it's now.

Yeah, I dont see any usecases to query ms/us or anything smaller. Perhaps week and month should be added, though I haven't found myself needing to use them often. Most of the times I find myself using --days 1 or --days 7, so the user can just do --within 30d to do a month; isn't that difficult. Also means we dont have to handle leap years or dealing with dates more than necessary

But perhaps for this would make sense to test on less specific things

Yeah, I agree -- this just needs to have some function/iterable to query against, so best to remove the additional point of failure

Added all the changes we discussed, along with the tests (though, due to the volume/dummy data iterators, unsure if they should be here or in tests/query.py)

Github is doing something weird with me not being able to mark of one of the comments resolved since I somehow replied to the parent? Anyways, regarding making the sort function for order_by_chosen lazy:
I had tried to wrap my head around how to do that - the limiting factor made me write it this way was that sorted needed to be passed a function, and I couldn't figure out a nice way to make it lazy, while having to pass it an flags/other info on how to sort items...
It just became too much a mess of dynamically creating additional named-functions (i.e. def some_function) to handle state
In principle it could be, I believe, I wasn't able to figure out a nice way to do so

added the other changes described -- feel free to squash and merge (by the way, In general you don't have to ask to squash, for any future PRs)

Awesome, thanks! :tada:


Is it possible to use variables from settings.json in c_cpp_properties.json?

Is it possible to use variables from settings.json in c_cpp_properties.json?
I'm saw notice on this theme, but this feature doesn't work for me :(
settings.json:
{
    "arduino.path": "/home/r3d9u11/Software/arduino-1.8.5"
}

c_cpp_properties.json:
{
    "configurations": [
        {
            "name": "Linux",
            "includePath": [
                "${config:arduino.path}/hardware/arduino/avr/cores/arduino",
                "${config:arduino.path}/hardware/arduino/avr/variants/mega",
                "${config:arduino.path}/hardware/tools/avr/avr/include",
                "${config:arduino.path}/hardware/tools/avr/avr/include/util",
                "${config:arduino.path}/hardware/tools/avr/lib/gcc/avr/4.9.2/include"
            ],
            "defines": [
                "UBRRH",
                "UNICODE"
            ],
            "browse": {
                "limitSymbolsToIncludedHeaders": false,
                "databaseFilename": "",
                "path": [
                    "${config:arduino.path}/hardware/arduino/avr/cores/arduino",
                    "${config:arduino.path}/hardware/arduino/avr/variants/mega",
                    "${config:arduino.path}/hardware/tools/avr/avr/include",
                    "${config:arduino.path}/hardware/tools/avr/avr/include/util",
                    "${config:arduino.path}/hardware/tools/avr/lib/gcc/avr/4.9.2/include"
                ]
            },
            "cStandard": "c11",
            "cppStandard": "c++17",
            "compilerPath": "/usr/bin/gcc",
            "intelliSenseMode": "clang-x64"
        }
    ],
    "version": 4
}

main.c:

(searching process is freezed)
Test project: TestAvrC.zip
VSCode 1.23.1
Thanks.

@r3d9u11, take a look at this document.  The C++ extension exposed new VS Code settings that you can reference in your c_cpp_properties.json file (or you can use them instead of having a c_cpp_properties.json at all)

You can also add your own custom "env" to c_cpp_properties.json that will be recognized using the standard ${env:var} syntax.  See this document.

But I noticed that the initialized variables don't work inside env when it is referenced right next to the definition. For example,
{
    "env": {
        "envRoot": "apps/tool/buildenv",
        "arm6.include": "${envRoot}/arm6/include",
    },
    "configurations": [
        {
            "name": "Custom",
            "includePath": [
                "${arm6.include}"
            ],
            "browse": {
                "limitSymbolsToIncludedHeaders": false,
                "path": [
                     "${arm6.include}"
                ]
            },
            …
        }
    ],
    "version": 4
}

This doesn't seem to work. That would be really useful


Can't find LEGO READER in VirtualHere

I've followed the steps so far and they've worked fine but now I have the virtual machine running and in VirtualHere it only shows: USB Hubs - Desktop Hub - USB Tablet. I turned off my antivirus so it's not that and I went through the troubleshooting and i'm stuck on this.

Turns out Oracle VM VirtualBox isn't very reliable so I switched to VMware and it fixed the problem.


Updated readme to v1.1

fixes #581

Preview - https://github.com/okfn/opendataeditor/tree/581/update-readme-1.1

Thanks!


Create a Web UI for administering BX-bot

Create a Web UI (written in Angular) for administering the bot.
This work will be done in a new GitHub project:
https://github.com/gazbert/bxbot-ui

Still ToDo:

New Settings screen to configure the bot's server url, credentials, etc.
New Runtime screen to: start|stop|status the bot, display/tail/download bot log, display bot resource usage, e.g. memory, cpu, disk.

On-hold.
Decided to learn React and build new UI - #101

Closing.
A partial Angular implementation can be found here but is no longer maintained.
React is the way forward...  #101


[Issue]: triton fails to compile due to non-existent llvm tarball

Problem Description
llvm-49af6502-ubuntu-x64.tar.gz is no longer available as a download.
[I'm not sure if this belongs in aotriton or if it belongs here in this triton fork.  Ultimately the issue is with the triton compile requested by aotriton so I decided to file it here.  If that's a mistake, please let me know and I'll open an issue with aotriton.]
Operating System
RHEL9, Fedora (likely all OSes fwiw)
CPU
Any CPU
GPU
AMD Instinct MI300X
ROCm Version
ROCm 6.1.0
ROCm Component
ROCm/triton
Steps to Reproduce
An aotriton compile points this triton repo which, when compiled, does
2024-08-18T13:02:58,456   downloading and extracting https://github.com/pybind/pybind11/archive/refs/tags/v2.11.1.tar.gz ...
2024-08-18T13:02:58,456   downloading and extracting https://tritonlang.blob.core.windows.net/llvm-builds/llvm-49af6502-ubuntu-x64.tar.gz ...
2024-08-18T13:02:58,457   error: HTTP Error 409: Public access is not permitted on this storage account.

This download reference comes from the upstream triton repo https://github.com/triton-lang/triton/tree/release/2.3.x which now has moved to hash 5e5a22caf88ac1ccfa8dc5720295fdeba0ad9372 which is currently available at the download above (see https://github.com/triton-lang/triton/issues/4527).
However, the ROCm/triton repository (this repository) still references the old 49af6502 package, which no longer exists at that download location -- and the above error is seen when attempting to compile torch, aotriton, or triton from this repository.
To reproduce, build one of torch-2.3.1, aotriton (https://github.com/ROCm/aotriton.git) , or triton (from this repo, not triton-lang).
(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support
ROCk module is NOT loaded, possibly no GPU devices
(I will note that I am compiling and not executing on this system)
Additional Information
No response

This change needs to be cherry-picked on this fork of triton: https://github.com/triton-lang/triton/commit/06e6799f4eba6035ec35c528e8fefd3d4d724b6f
Even so, the new blob store may not contain all of the older LLVM drops, so it may only resolve the build issues for newer versions of triton.

Even so, the new blob store may not contain all of the older LLVM drops, so it may only resolve the build issues for newer versions of triton.

https://oaitriton.blob.core.windows.net/public/llvm-builds/llvm-ce80c80d-ubuntu-x64.tar.gz << stolen from triton-lang/triton/python/setup.py on ubuntu 22.04 and it doesn't work.. i believe you are correct

This is what I did:
git clone https://github.com/llvm/llvm-project
(cd llvm-project; git checkout 49af6502c6dcb4a7f7520178bd14df396f78240c;)
(cd llvm-project; cmake -G Ninja -DCMAKE_BUILD_TYPE=Release -DLLVM_ENABLE_ASSERTIONS=ON ../llvm -DLLVM_ENABLE_PROJECTS="mlir;llvm" -DLLVM_TARGETS_TO_BUILD="AMDGPU" -DCMAKE_C_COMPILER=/usr/bin/gcc -DCMAKE_CXX_COMPILER=/usr/bin/g++ -DCMAKE_ASM_COMPILER=/usr/bin/gcc; ninja)

followed by setting the following environment variables for torch
LLVM_BUILD_DIR=/path/to/your/llvm-project/build
LLVM_INCLUDE_DIRS=$LLVM_BUILD_DIR/include
LLVM_LIBRARY_DIR=$LLVM_BUILD_DIR/lib
LLVM_SYSPATH=$LLVM_BUILD_DIR

and then building torch.
I based this on
https://github.com/triton-lang/triton?tab=readme-ov-file#building-with-a-custom-llvm
The above works for me on RHEL9.4.


Separate profiles

Description
[Include a brief description of the changes you've made]
Related Issues nice
[Include any related issues or pull requests that this PR addresses or is related to. Use GitHub's shorthand syntax to link to them, like #1234.]
Changes Proposed
[Describe the changes you've made in detail. Be specific and include any relevant code snippets.]
Checklist

[x] I have read and followed the Contribution Guidelines.
[x] All new and existing tests passed.
[ ] I have updated the documentation to reflect the changes I've made.
[ ] My code follows the code style of this project.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots
Note to reviewers

yfg


f32 Mul operator

It would be convenient to implement Mul with f32 as an alias for .scale(). This would make it possible to do
b += m.transformation(gravity) * dt;

instead of
b += m.transformation(gravity).scale(dt);

Done, see: 642d8bdbdb28ce746349b534bc460d4197df4578


Facebook Connect - Extended Profile Infomation

When I enabled it, I was getting an error, turns out they do not use 'bio' anymore, they use 'about'
so I edited the code, seemed to work, but it's not populating any of the fields except for the first name being in the Username slot, and your e-mail address.
After I set a password, and click Next, no information was grabbed from Facebook, it's all blank.

As I see "bio" field is equivalent to "about" field:
https://developers.facebook.com/docs/graph-api/reference/user
Also to be able to collect these additional info your Facebook app must be manually approved for the same set of fields.

Yes, that is what I explained in my OP. I replaced the instances where "bio" appeared in the code, and replaced it with "about", and now I don't have an Error, but it still not grabbing the info.
Also, I'm the "developer" for the App, so I have the permissions required to obtain from the fields.
It is not grabbing the information however. It only grabs the first name, and e-mail address. No birthday, about, or anything else.
I'm sorry my first post wasn't clear: something is wrong in the code, and it needs to be fixed, but I'm not too good with code, replace some key words, sure, but I can't totally dissect code. I am however familiar with that fact that if I see one thing out of place ie. bio instead of about, there may be something else in the Facebook API that was changed that is preventing Dolphin from grabbing the information correctly, and therefore have made a guess that this code need to be updated to support the current API.


Create version 0.10 by hand

As you can see at https://github.com/OpenElements/hedera-enterprise/releases/tag/v0.11.0 the version v0.10.0 has never been created at GitHub. That is related to https://github.com/OpenElements/hedera-enterprise/issues/68
We should create v0.10.0 by hand and move all changes up to the creation of v0.10.0 from the v0.11.0 change log to v0.10.0

@hendrikebbers  fixed here


(maint) Remove testResults.xml

This removes a file that was erroneously added to the repo.
!no-release-note

Woops! Good catch Tom.


Bumped Bigtable client library to 0.9.4

Bigtable client library 0.9.4 downgraded protobuf as well, which is why getColumnFamiliesMap() change to getColumnFamilies()

Current coverage is 45.21% (diff: 0.00%)

Merging #130 into master will increase coverage by 0.02%

@@             master       #130   diff @@
==========================================
  Files           598        598
  Lines         15427      15427
  Methods           0          0
  Messages          0          0
  Branches       1585       1585
==========================================
+ Hits           6971       6975     +4
+ Misses         7994       7992     -2
+ Partials        462        460     -2

Powered by Codecov. Last update cf81df2...37b67ab

:+1:

:+1:


add CircleCI build status logo

@bacongobbler,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


[Bug]: reuse und checks cannot be configured via classpath:testcontainers.properties

Module
Core
Testcontainers version
1.19.3
Using the latest Testcontainers version?
Yes
Host OS
Windows
Host Arch
x64
Docker version
n/a

What happened?
Placing a testcontainers.properties file at the root of the classpath should allow configuration of testcontainers.
For testcontainers.reuse.enabled and checks.disable this is not possible as these properties are checked using getEnvVarOrUserProperty (which ignores classpathProperties) instead of getEnvVarOrProperty.
Source code references:

https://github.com/testcontainers/testcontainers-java/blob/main/core/src/main/java/org/testcontainers/utility/TestcontainersConfiguration.java#L179
https://github.com/testcontainers/testcontainers-java/blob/main/core/src/main/java/org/testcontainers/utility/TestcontainersConfiguration.java#L185

Relevant log output
No response
Additional Information
No response

Thanks for raising this issue, but this works as intended for these properties, since we believe this should be a user decision, and not a project decision.


Setup hermes as a docker image

Hey! I would like to help with this.
Please let me know when can we have quick about it.
I have some ideas.


Update to support Omnipay version 3

Can you please update the library to support Omnipay version 3
https://github.com/thephpleague/omnipay
Please see https://github.com/thephpleague/omnipay-common/blob/master/UPGRADE.md
Thank you!

Any news on this?

Any news on this?

We maintain an updated version here: https://github.com/lxrco/omnipay-elavon you can find it on composer under lxrco/omnipay-elavon