Limit and Range chart price draggers should pin to the tick grid when being dragged

Consider when the user is dragging the price selector bars in the chart, in either Limit or Range mode. Currently as they move their cursor up and down, the selector line moves smoothly and exactly with their cursor. This means as they drag, the dragger can be at a Y position that does not conform to a valid tick grid point:

Change this behavior so that the dragger line is always pinned to the closest tick grid point. As the user moves the cursor, the line should stay pinned until the Y-position of the cursor is closer to the next grid value, then the line should immediately snap to that grid point.

relates to #2215

We currently have the desired behaviour on the range module and would like the same for the limit module.
Range:
https://github.com/CrocSwap/ambient-ts-app/assets/45405267/816894ca-6034-4ad6-b052-a30f95560392
Limit:
https://github.com/CrocSwap/ambient-ts-app/assets/45405267/8db636d6-c212-47d5-ac5f-02a2c2d0af08


Oracle Connector does not support RAW or BINARY types

When selecting from a table that features a RAW or BINARY column conversion is not performed.
BINARY/RAW in Oracle should probably be converted to Presto VARBINARY type.
AnalysisException: Unsupported type 'BINARY' in 'default.fnd_user.user_guid'.
The column type of user_guid is actually RAW.

@findepi, @ebyhr do you know if that was conscious decision to not support those? I see that RAW is not recommended by Oracle and only kept around for backward compatibility. Was that driving fact to not add mapping for it?

Hey just to update, one of our users pointed this out to me.
But this is happening with a CTAS against Hive 2.1.1.
A raw "SELECT" statement seems to work fine.
The user told me they have tried with Parquet, and ORC.  Parquet supports BYTE_ARRAY and Hive supports "BINARY" types.
So not sure where the mapping is breaking down?

@losipiuk If I remember correctly, there was no related conversation about RAW type.


Combination carriers

It would be useful to have functionality that allows a specific combination of query outputs to only occur once; for example, first/last name pairs. This is very close to how the unique carriers work, except it works over a set of terms rather than just one, and a certain word may repeat; just not a specific combination of words.
The carrier symbol for combinations would be !+, similar to the unique carrier symbol !. It would be complimented by a function ([combine: carrier-name]) that marks the end of a combination, so that it can be submitted to the QueryState of the current sandbox.
The following example demonstrates a possible usage of the proposed carrier by printing a list of ten unique first/last name pairs.
[rs:10;\N]
{
    <name:: !+ cmb_name> <surname:: !+ cmb_name>
    [combine: cmb_name]
}

Due to the complex nature of the problem presented by this feature proposal, I am removing this from the 2.0 milestone so that it does not get in the way of the development of more integral features.
If anyone else wants to try implementing this later on, feel free to give it a shot.


Developer Guild presentation

There is an opportunity to take part in presenting about GraphQl at the Developer Guild presentation on Jul 27th. Oamar will be the main presenter and will create the presentation and allocate a bit of time for you to present something about GraphQl. This would be something small and manageable like showing an example query from the new Registry API.
ToDo

[ ] Meet with Oamar to go over the presentation and decide on what you will be presenting

https://docs.google.com/presentation/d/1jp8bYYIDdAgMl9Us3E7CbLHW1oZZrwo6IxkljYwjcY0/edit?usp=sharing


Repeated responses after 5 chats

Hi,
i appreciate the great repository and details added for fine tuning the dialouge GPT. I tested for fine-tuned covid data but responses are repeated after 5 chats. I have tested this scenario around 10 times.
I have attached a screenshot for reference.

Thankyou in advane

Hi,
i appreciate the great repository and details added for fine tuning the dialouge GPT. I tested for fine-tuned covid data but responses are repeated after 5 chats. I have tested this scenario around 10 times.
I have attached a screenshot for reference.

Thankyou in advane

@saleemsum did you have any idea why it is happening?


Accepting tokens from "access_token"-field

For compatibility with OAuth 2.0, token under the name access_token should also be accepted. More info https://docs.docker.com/registry/spec/auth/token/#token-response-fields

This is frustrating, the project seems to be stale, the pr could not be merged


Password error / can't open XML / beginner issue

I can't seem to type the password. I keep getting this message:
Password: error: unable to open output file: open backup.xml: Access is denied.

Operating system: Windows 10 Pro
Build version (signal-back --version): v0.1.7-alpha.2

Detailed description
Hi,
I would very much appreciate getting this to work. I am brand new to Github... hope you can help me out. It might be very basic, what I'm doing wrong.
I have my backup file, signal-2020-02-17-18-24-50.backup and signal-back_windows_amd64.exe in the same folder.
I hold shift and right-click in the folder and choose Open PowerShell window here
In the PowerShell window I type signal-back_windows_amd64.exe format -f XML -o backup.xml signal-2020-02-17-18-24-50.backup
I then get a line with this text:
Suggestion [3,General]: (...) If you trust this command, instead type: ".\signal-back_windows_amd64.exe". See "get-help about_Command_Precedence" for more details.
I follow the suggestion and instead type:
.\signal-back_windows_amd64.exe format -f XML -o backup.xml signal-2020-02-17-18-24-50.backup
I then get a Password: line but it doesn't respond to any inputs (*** stars or numbers don't show up when I type numbers). I try typing in my encryption key anyway but I always get the same response:
Password: error: unable to open output file: open backup.xml: Access is denied.
I tried typing in the 6x5 digit encryption key without spaces and with a single space in between each digit group.
Full relevant output

Password characters will NOT appear in a secure command line input. Type the password in the prompt(after Password: ) and press enter. If you got it right, it'll work
You can also decrypt the backup by passing the password in the invocation line, with --password '00000 00000 00000 0000'. Spaces are optional, single quotes are a good habit to make sure the program treats your password as a single argument
However, your error seems like a file access problem - you might not have the right to write files in the location you're in. Or possibly the backup.xml file in that directory already exists and PowerShell is trying to protect you from overwriting it, so you should try calling the output file backup2.xml instead


Understanding IOPMP Models

I am currently reviewing the various IOPMP models outlined in the specification and wanted to confirm my understanding. I have created a table that categorizes each model according to the values of SRCMD_FMT and MDCFG_FMT. Kindly take a quick look and let me know if my interpretation is correct. @tyshyu @paul-andes

Model Name
SRCMD_FMT
MDCFG_FMT
Description

Full Model
0
0
Max Supported RRIDs: 65536  Max Supported MDs: 63  Uses RRID to obtain SRCMD_EN(H), indicating the associated MDs. Traverse MDs, find associated IOPMP entries, perform address matching, and check permissions.

Rapid-k Model
0
1
Max Supported RRIDs: 65536  Max Supported MDs: 63  Uses RRID to obtain SRCMD_EN(H). No physical MDCFG table; each MD has k IOPMP entries:  MD0 → 0 to (k - 1)  MD1 → k to (2k - 1)  MD2 → 2k to (3k - 1), and so on.  Traverse IOPMP entries associated with MDs linked to RRID, perform address matching and permission check.

Dynamic-k Model
0
2
Same as Rapid-k Model, but k is programmable.

1
0
Max Supported RRIDs: 63  Max Supported MDs: 63  No physical SRCMD table; RRID i directly maps to MD i.  Associated IOPMP entries: MD(i-1).t to entries < MD(i).t.  Perform address matching and permission check.

Compact-k Model
1
1
Max Supported RRIDs: 63  Max Supported MDs: 63  No SRCMD table; RRID i directly maps to MD i. No physical MDCFG table; each MD has k associated IOPMP entries  Associated Entries: (i × k) to ((i+1) × k - 1).  Perform address matching and permission check.

1
2
Same as Compact-k Model, but k is programmable.

2
0
Max Supported RRIDs: 32  Max Supported MDs: 63  SRCMD_EN(H) and SPS Extension registers replaced with SRCMD_PERM(H). Traverse all MDs, locate associated IOPMP entries, perform address matching, and check permissions.

2
1
Max Supported RRIDs: 32  Max Supported MDs: 63  SRCMD_EN(H) and SPS Extension registers replaced with SRCMD_PERM(H). No MDCFG Table; each MD has k associated IOPMP entries. SRCMD_PERM(H) defines permissions but not associated MDs therefore, all IOPMP entries must be traversed.  Perform address matching and apply permissions from SRCMD_PERM(H).

2
2
Same as above, but k is programmable.

Hi @YazanHussnain-10x, your interpretation correct.
Note that SRCMD_FMT=1 and MDCFG_FMT=0 has model name called "Isolation".
It's unintentionally removed in v0.9.2. It will be added back in next verion.

Thank you @gagachang

Hi @YazanHussnain-10x, your interpretation is correct. Note that SRCMD_FMT=1 and MDCFG_FMT=0 has model name called "Isolation". It's unintentionally removed in v0.9.2. It will be added back in next version.

BTW, Isolation model had been added back in commit 148660ebc9313671bc571bbaa6b3263b6c92a7e8

You're welcome @YazanHussnain-10x
Do you have more questions about IOPMP models ?

Not at this time. Thank you for the clarification, @gagachang and @tyshyu.


⚠ Bofrost.be has degraded performance

In 0011b5a, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1534 ms

Resolved: Bofrost.be performance has improved in 28959c9 after 305 days, 10 hours, 52 minutes.


npm install failing to build for node-webkit

I am trying to build sqlite3 for node-webkit v0.8.6 with node 0.10.26 on Windows 10. The install fails at a very early stage. I have attached the console output and the npm-debug log file.
npm-debug.txt
console.txt

I concluded that I needed to update npm, so I have updated it to v3.5.2.
I am now trying to build from inside the git repository on my computer. The first indication of trouble in the npm-debug.log file is an unsafe-perm message so I am wondering whether I still have some incompatibilities in my system.
npm-debug.txt

I am closing this issue because it has become rather irrelevant and open another for more recent findings.


Register Option added on Home page

@mariobehling @championswimmer It solved #575

[x] Improved description
[x] Added Ticket link

Current coverage is 47.26% (diff: 0.00%)

Merging #596 into development will decrease coverage by 0.35%

@@           development       #596   diff @@
=============================================
  Files                5          5
  Lines              546        548     +2
  Methods             52         52
  Messages             0          0
  Branches            88         88
=============================================
- Hits               260        259     -1
- Misses             286        289     +3
  Partials             0          0

Powered by Codecov. Last update 0725b06...6ca0d6a

Please solve houndci errors. Lots of them are quite easy to solve.

@aayusharora
i'll merge this after hound errors are solved :)


fix: Improved Footer layout and design! #41 (Ademola111)

I just made an adjustment to the footer kindly make a review  on this for possible merge.
I look forward to your congratulatory feedback.

Hello @Ademola111 , Appreciate the adjustment! We're not incorporating social icons in this iteration. We'll consider this for future updates. Thanks for your contribution!


Update for Xcode 9

This pull request adds an include statement to fix a bug introduced with Xcode 9. It also includes some test updates for changed behavior.

Codecov Report

Merging #664 into master will increase coverage by 7.58%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #664      +/-   ##
==========================================
+ Coverage    64.7%   72.28%   +7.58%
==========================================
  Files         268      258      -10
  Lines       21233    19090    -2143
==========================================
+ Hits        13738    13799      +61
+ Misses       7495     5291    -2204

Impacted Files
Coverage Δ

Tests/DiscoveryV1Tests/DiscoveryTests.swift
88.2% <100%> (ø)
:arrow_up:

Tests/ConversationV1Tests/ConversationTests.swift
93.74% <100%> (ø)
:arrow_up:

Source/VisualRecognitionV3/VisualRecognition.swift

...ualRecognitionV3Tests/VisualRecognitionTests.swift

...e/VisualRecognitionV3/Models/PositiveExample.swift

.../VisualRecognitionV3/Models/ClassifiedImages.swift

...ource/VisualRecognitionV3/Models/WarningInfo.swift

Source/VisualRecognitionV3/Models/Collection.swift

...e/VisualRecognitionV3/Models/ImagesWithFaces.swift

.../VisualRecognitionV3/Models/CollectionImages.swift

... and 5 more

Continue to review full report at Codecov.

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


How to use  roomUrl?

I want to use my roomUrl like this ,but can't work. how to use roomUrl?

Hi @Xiamingmin5, thanks for the question, unfortunately that image link doesn't work for me. The URL should be set to an RTVI-compatible endpoint such as https://api.daily.co/v1/bots/start.


mathtools' mathic option incompatible with math grabbing code

The mathtools option mathic errors with testphase={phase-III,math}. This makes sense as it redefines \(...\).
\DocumentMetadata
  {
    lang=en-US,
    pdfversion=2.0,
    pdfstandard=ua-2,
    testphase={phase-III,math}
  }
\documentclass{article}
\usepackage{mathtools}

\begin{document}

\itshape
Compare these lines: \par
\mathtoolsset{mathic} % or \mathtoolsset{mathic=true}
Subset of \(V\) and subset of \(A\). \par
\mathtoolsset{mathic=false}
Subset of \(V\) and subset of \(A\).

\end{document}

errors with
Runaway argument?
\MH_fi: V\) and subset of \(A\). \par \mathtoolsset {mathic=false} Su\ETC.
! File ended while scanning use of \__math_grab_dollar:w.
<inserted text>
                \par

My expectation would be that mathtools would largely work with a number of such things failing, so that we end up with partially-supported and a list of items (envs or options) that do not work. What is therefore needed is determining which parts are failing (and which work), i.e. more tests.


Shell Command Problem

We have written the needed data into your clipboard because it was too large to send. Please paste.

shell command not listing path
can't turn on sync
can't sign into Microsoft or GitHub on vs.code


Remove Tatiana from mailchimp

This PR removes Tatiana entries from the mail chimp file

@cbaines Thank you for clarifying this. I am working on a ticket which requested this change be carried. I believe it may already have been actioned.


Bug 1932182: Support jittering relatively small resync intervals.

The previous jitter computation truncated intervals shorter than
around 1 minute to 0. Intervals that short are atypical for resyncs,
but still need to jitter to a reasonable value to avoid hot resync
loops.

/lgtm

/retest

/cherry-pick relerase-4.7

relerase-4.7

D'oh!
/cherry-pick release-4.7


Added Back to Top feature

#3
Desktop:

Mobile:

@2pai I don't get why the Summary check is failing?


Updated error message.

Can you please rebase the feature/aquav1.0.1 ? The feature/aquav1.0.1 just sync with main.


Enhancement: email service provider integration

Summary
In the Django world https://github.com/anymail/django-anymail is the kind of de-facto standard for integrating email service providers via a unified API. It would be nice to have an equivalent functionality in Litestar for the fullstack use case.
Basic Example
No response
Drawbacks and Impact
No response
Unresolved questions
No response

[!NOTE]
While we are open for sponsoring on GitHub Sponsors and
OpenCollective, we also utilize Polar.sh to engage in pledge-based sponsorship.
Check out all issues funded or available for funding on our Polar.sh dashboard

If you would like to see an issue prioritized, make a pledge towards it!
We receive the pledge once the issue is completed & verified
This, along with engagement in the community, helps us know which features are a priority to our users.

I don't think that this is something that makes sense to have, as there's virtually zero framework integration necessary. Could you elaborate what features that are specific to Litestar would be useful here? The way I see it, there's nothing from preventing a 3rd party ESP library to work inside a Litestar app just like it would anywhere else.

I agree that this would be useful, but I still don't see why this unified API has to be Litestar specific?


problem with creating roles even after setting up tiller to have admin clusterrolebinding

Closing in favour of #3444.


When adding the webpack alias the first import gets error, 'can not read property development of undefined'

my .eslintrc.js
module.exports = {   extends: 'airbnb',   parser: 'babel-eslint',   rules: {     'react/jsx-filename-extension': 0,     'react/forbid-prop-types': 0,     'class-methods-use-this': 0,     'no-case-declarations': 0,     'jsx-a11y/label-has-for': 0,   },   settings: {     "import/resolver": {       webpack: {         config: "webpack.config.js"       },     },   }, };
my webpack alias config
 module.exports = (env) => { .......configs resolve: {       alias: {         Utils: path.resolve(__dirname, './src/utils/'),         Store: path.resolve(__dirname, './src/store/'),         Components: path.resolve(__dirname, './src/components/'),       },     }, .....otherConfigs }
I searched a lot about the error and read in problems that the webpack returns a function, but in webpack 1 in my case it is version 3.8, I believe that the error is because of the env parameter, which is not defined in some cases. And the error continues.
Example:
import { Accordion } from 'Components';
(import/no-extraneous-dependencies)
Any idea how to proceed?
Ps: I think this is a fantastic project, congratulations to all involved

My problem is solved in #1050


Improve Report panel

Overview
It's hard to consume when there are many files and errors. Also, we need to get a read of some not-needed styling like zigzag picture usage etc. Also, it will be good to migrate it to MUI

Hi @roll I have created issue #342.  think this is related to what you highlight here. Is that correct?

HI @romicolman, this issue is only about the Report component that renders the validation report that comes from frictionless-py

@roll can we close this? I think we addressed it when improving the report feature. Let me know.


Add support for Slack workflows format in alerts

The current format does not support workflows, as it only supports key-value pairs.
{
  "description": <ALERT_DESCRIPTION>,
  "table": <table_name>,
  "detected_at" <detected_at>
}

Possible solution:
Config property where you can set the type of slack integration you are using (either 'workflow' or 'webhook'). If you set the config to 'workflow' the response body would be formatted in the proper way.

Hi,
Suggested design
Configuration
In the config.yml the slack webhook config should look as follows -
slack:
    notification_webhook: <webhook_url>
    workflows: true / false

This requires small changes to the config.py file. The default value for slack worfklows should be set to false.
Alerts

Alert base class should expose a new function in its interface called 'to_slack_workflows_message' and each alert type should implement this new function.
The template that should be implemented in 'to_slack_workflows_message' should follow the restrictions slack workflows messages must adhere to.
The 'send_to_slack' function in Alert base class should receive a new parameter that indicates if the regular slack template or the workflow template should be used and call 'to_slack_message' or 'to_slack_workflows_message' accordingly.
Then in data_monitoring.py, pass the new slack workflows flag using the config object to the 'send_to_slack' function of each alert.

Please let me know your thoughts or questions about it :)

Added by @aaron-westlake in version 0.2.1 :tada:


[🐛 BUG] TGB.table rendering blank table

What went wrong? 🤔
I have written a somewhat simplified bit of code that matches my current usage pattern so apologies if it is a bit long.
There are no console errors and the dataframe is accessible within the method (printed to console to check)
The output I get on the web app is as follows:

Expected Behavior
I expected to see both the response text as well as a rendered tgb.table populated with data from the dataframe
Steps to Reproduce Issue
import os, sys, ast

import pandas as pd

from taipy.gui import Gui, State, notify, Markdown, invoke_long_callback
import taipy.gui.builder as tgb

def on_init(state):
    state.conv.update_content(state, "")
    state.messages_dict = {}
    state.response_dict = {}
    state.messages = [
        {
            "role": "assistant",
            "style": "assistant_message",
            "content": "Hi, I'm GPT-4o! How can I help you today?",
        },
    ]
    state.gpt_messages = []
    new_conv = create_conv(state)
    state.conv.update_content(state, new_conv)

def create_conv(state):
    messages_dict = {}
    df_results = pd.DataFrame()
    with tgb.Page() as conversation:
        with tgb.layout(columns="1 3"):
            with tgb.part():
                for i, message in enumerate(state.messages):
                    text = message["content"].replace("<br>", "\n").replace('"', "'")
                    messages_dict[f"message_{i}"] = text
                    tgb.text(
                        "{messages_dict['"
                        + f"message_{i}"
                        + "'] if messages_dict else ''}",
                        class_name=f"message_base {message['style']}",
                        mode="md",
                    )

            try:
                if state.response_dict["result_value"] is not None:
                    with tgb.part():
                        print(
                            f"DF within New Conv: \n{state.response_dict['result_value'].to_dict('records')}"
                        )
                        df_dict = state.response_dict["result_value"].to_dict("records")
                        df_results = state.response_dict["result_value"].copy()
                        tgb.table(
                            data="{df_results}",  # Always need to use f-strings with these guys
                            filter=True,
                            editable=False,
                            allow_all_rows=True,
                            rebuild=True,
                        )
            except KeyError:
                with tgb.part(render=False):
                    pass

    state.messages_dict = messages_dict
    return conversation

def make_request(state: State):
    """
    Return the response.

    Args:
        - state: The current state of the app.

    Returns:
        The response from the API.
    """
    response_dict = {}
    """
    if response_dict["result_type"] == "df":
        response_value = response_dict["result_value"].to_markdown()
    elif response_dict["result_type"] == "txt":
        response_value = response_dict["result_value"]
    else:
        response_value = None
        return response_dict
    """
    data = {"row_1": [3, 2, 1, 0], "row_2": ["a", "b", "c", "d"]}

    response_dict["result_value"] = pd.DataFrame.from_dict(
        data, orient="index", columns=["A", "B", "C", "D"]
    )
    response_dict["query_value"] = "placeholder"

    return_string1: str = (
        """Query Code: {query_value}\n Result: {return_value}""".format(
            query_value=response_dict["query_value"],
            return_value=response_dict["result_value"].to_markdown(),
        )
    )
    response_dict["parse_result"] = return_string1
    print(f"DF:\n{response_dict['result_value']}")
    state.response_dict = {}
    state.response_dict = response_dict

def send_message(state):

    state.messages.append(
        {
            "role": "user",
            "style": "user_message",
            "content": state.query_message,
        }
    )

    state.conv.update_content(state, create_conv(state))
    notify(state, "info", "Sending message...")
    make_request(state=state)
    state.messages.append(
        {
            "role": "assistant",
            "style": "assistant_message",
            "content": state.response_dict["parse_result"],
        }
    )
    notify(state, "success", "Received API response, updating shortly . . .")
    state.conv.update_content(state, create_conv(state))
    # create_conv_results_partials(state, results_dict["result_value"])
    state.query_message = ""

def reset_chat(state):
    state.messages = []
    state.response_dict = {}
    state.gpt_messages = []
    state.query_message = ""
    state.conv.update_content(state, create_conv(state))
    on_init(state)

if __name__ == "__main__":
    index = 0
    query_message = ""
    messages = []
    gpt_messages = []
    messages_dict = {}
    response_dict = {}
    df_results = pd.DataFrame()

    with tgb.Page() as page:
        with tgb.layout(columns="300px 1"):
            with tgb.part(class_name="sidebar"):
                tgb.text("## Taipy x GPT-4o", mode="md")
                tgb.button(
                    "New Conversation",
                    class_name="fullwidth plain",
                    id="reset_app_button",
                    on_action=reset_chat,
                )
                tgb.html("br")

            with tgb.part(class_name="p1"):
                tgb.part(partial="{conv}", height="600px", class_name="card card_chat")
                with tgb.part("card mt1"):
                    tgb.input(
                        "{query_message}",
                        on_action=send_message,
                        change_delay=-1,
                        label="Write your message:",
                        class_name="fullwidth",
                    )

    gui = Gui(page)
    show_pane = False
    conv = gui.add_partial("")
    gui.run(title="🤖Taipy", dark_mode=False, margin="0px", debug=True)

Solution Proposed
No response
Screenshots

Runtime Environment
Windows 11 Home, Chrome Browser, Taipy 3.1.1, tabulate 0.9.0
Browsers
Chrome
OS
Windows
Version of Taipy
3.1.1
Additional Context
[2024-07-30 11:10:56][Taipy][INFO] 'allow_unsafe_werkzeug' has been set to True
[2024-07-30 11:10:56][Taipy][INFO] 'async_mode' parameter has been overridden to 'threading'. Using Flask built-in development server with debug mode
[2024-07-30 11:10:58][Taipy][INFO]  * Server starting on http://<IP_ADDRESS>:5000
 * Serving Flask app 'Taipy'
 * Debug mode: on
DF:
       A  B  C  D
row_1  3  2  1  0
row_2  a  b  c  d
DF within New Conv:
[{'A': 3, 'B': 2, 'C': 1, 'D': 0}, {'A': 'a', 'B': 'b', 'C': 'c', 'D': 'd'}]

Acceptance Criteria

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

Code of Conduct

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

You forgot to assign your value to the state.
df_results = state.response_dict["result_value"].copy()

should be:
state.df_results = state.response_dict["result_value"].copy()

Please read the Getting Started or the GUI tutorial here.

A new visual element will be released next September that will allow you to not use partials.


adding formatCurrency utility function to simplify legacy i18n migration

this is a utility passthru function to simplify migration from legacy i18n solutions that simply calls back into formatNumber with {as: 'currency'} automatically added to the format options.
relates to #396

I forgot to ask, but can you update the README?


[VL] Spark mask function support

spark document https://spark.apache.org/docs/latest/api/sql/#mask
spark code - https://github.com/apache/spark/blob/master/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/maskExpressions.scala#L103
related issue to track https://github.com/facebookincubator/velox/issues/10263
the UT part in this PR cover all uts of spark already have - https://github.com/apache/spark/blob/0db5bdecfa6cbfff1be7690bb783a858989776b9/sql/core/src/test/scala/org/apache/spark/sql/DataFrameFunctionsSuite.scala#L5677

@rui-mo / @PHILO-HE please help review.

hey @rui-mo i saw the spark fuzz test fail with below user error - i remember you mentioned that the user error would be catched ?

@gaoyangxiaozhu This failure is not about the user error itself, but about different behaviors being detected for the simplified and common eval engines. Probably the bug is on the different handling of constant encoding and dictionary encoding, and you could try and debug it locally.
https://github.com/facebookincubator/velox/blob/main/velox/expression/tests/EvalSimplifiedTest.cpp#L43

@gaoyangxiaozhu This failure is not about the user error itself, but about different behaviors being detected for the simplified and common eval engines. Probably the bug is on the different handling of constant encoding and dictionary encoding, and you could try and debug it locally. https://github.com/facebookincubator/velox/blob/main/velox/expression/fuzzer/FuzzerToolkit.cpp#L95-L96

I see, will check later

@rui-mo for help new round review. I have run locally fuzz test 5 minutes and it work run.

@gaoyangxiaozhu, is there any reason to implement it as vector function instead of simple function?

no any specific reason @PHILO-HE ,  Both simple and vector implementations are acceptable for this mask function. I just personally prefer the vector implementation

@gaoyangxiaozhu, is there any reason to implement it as vector function instead of simple function?

no any specific reason @PHILO-HE , Both simple and vector implementations are acceptable for this mask function I think. I just personally prefer the vector implementation

@PHILO-HE Thank you for bringing this up. As previously discussed with Masha, without a specific reason (e.g. vector function provides performance improvement, while sometimes it is not the case), it would be preferred to use simple function, which helps simplify both the implementation and testing. @gaoyangxiaozhu How do you think?

oh, i see, i think at least here using vector given us the chance to do some optimization , for example here - fase path for flat, constant, * https://github.com/gayangya/velox/blob/gayangya/spark_mask/velox/functions/sparksql/Mask.cpp#L80, we decode only first row to apply for all the rows, for future possbile inplace replacement https://github.com/facebookincubator/velox/issues/10454
So how do you think if it is better to keep vector implement ?
If it not the case , we can do move to simple function.

@gaoyangxiaozhu, is there any reason to implement it as vector function instead of simple function?

no any specific reason @PHILO-HE , Both simple and vector implementations are acceptable for this mask function I think. I just personally prefer the vector implementation

@PHILO-HE Thank you for bringing this up. As previously discussed with Masha, without a specific reason (e.g. vector function provides performance improvement, while sometimes it is not the case), it would be preferred to use simple function, which helps simplify both the implementation and testing. @gaoyangxiaozhu How do you think?

oh, i see, i think at least here using vector implement given us the chance to do some optimization , for example here - fase path for flat, constant, * https://github.com/gayangya/velox/blob/gayangya/spark_mask/velox/functions/sparksql/Mask.cpp#L80, we decode only first row to apply for all the rows, any future possbile optimization - inplace replacement #10454
So how do you think if it is better to keep vector implement ? If it not the case , we can do move to simple function.

btw, looks once do move, the test part change is big, when i do the PR , i do not have context As previously discussed with Masha, without a specific reason (e.g. vector function provides performance improvement, while sometimes it is not the case), it would be preferred to use simple function, which helps simplify both the implementation and testing, and this context more likely let developer can do code implement easily but not hard mandarary.
I think if we share this context in the very begining (even before) of reviewing this PR, we can do simple function. But since this PR
already ready here with multiple round reviews, i perfer keep it current vector version, due to above context I think is more like help reduce the burden on developers for code implement part not the opposite

Some background for the choice of simple function v.s. vector function: https://github.com/facebookincubator/velox/pull/5842#discussion_r1391072258.

@gaoyangxiaozhu I apologize for not bringing this up right away. I did not recognize this point until @PHILO-HE's reminder. To determine which implementation would be preferable, I drafted below simple implementation for 'mask' and included a benchmark in rui-mo@5e7c3fb, and below are my findings:

simple function is easier to implement and test, which helps improve the code maintainability.
the performances of simple and vector functions for 'mask' are close. BTW, in the simple implementation, a fast path for constant inputs could be added with initialize method after this issue is resolved.

Vector function:
============================================================================
[...]hmarks/ExpressionBenchmarkBuilder.cpp     relative  time/iter   iters/s
============================================================================
mask_function##constant                                    19.02ms     52.57
mask_function##flat                                        21.42ms     46.70
----------------------------------------------------------------------------

Simple function:
============================================================================
[...]hmarks/ExpressionBenchmarkBuilder.cpp     relative  time/iter   iters/s
============================================================================
mask_function##constant                                    20.06ms     49.85
mask_function##flat                                        20.01ms     49.98
----------------------------------------------------------------------------

Give above points, I would prefer to implement it as simple function. Hope it would not bring too much additional efforts using my draft.

got and thanks @rui-mo , then please go ahead with your simple vesion mask support ,  and I can re-use this PR https://github.com/apache/incubator-gluten/pull/6271 to enable it in gluten side

@gaoyangxiaozhu Would you like to continue this work in your PR?

@gaoyangxiaozhu Would you like to continue this work in your PR?

oh, sure if you'd like, let me re-design with a simple version.

also run locally 5 minutes fuzz test for mask function and works fine

@PHILO-HE / @mbasmanova for any other comments

@mbasmanova for any new comments ?

@mbasmanova for any new comments ?

ping @mbasmanova again, let help speed up the review process


English grammar fixes

Not a huge code change but it contains English grammar fixes in output.

Thanks for all @eren 😊


Upgrading to v1.7.1 requires RPC port update for LND testnet node

Upon upgrading from ~v1.6 to v1.7.1 I had to ensure I updated the lnd.conf to use the bitcoind testnet RPC port. This should probably be handled in the migration stage if possible.
Let me know if you'd like any more info. Cheers.

Thank you for reporting. I am aware that the upgrade from a testnet node is not handled well due to the change to parallel miannet - testnet - signet usage now (since 1.7.1).
Will think how can we improve this.
Can you share which config settings you needed to change manually?
@frennkie had a similar issue.
Let's aim to handle them automatically.

bitcoind.rpchost in lnd.conf
Seems like it gets reset often too

yes, it will be reset after every lnd restart by the precheck script.
Set the default lnd.conf for mainnet
and there will be a tlnd.conf for testnet when you activate the TESTNETS in SETTINGS as in https://github.com/rootzoll/raspiblitz/blob/v1.7/home.admin/config.scripts/lnd.install.sh
The default password for tetsnet LND node is: raspiblitz.

@kaloudis  is this still an issue with 1.9.0?
Closing this for now, please re-open if it still persists.


feat: build on infra.ci.jenkins.io too

Agent label changed to one existing on both ci.jenkins.io and infra.ci.jenkins.io
Complement of https://github.com/jenkins-infra/kubernetes-management/pull/2889

[Please correct me if I'm wrong]:
The intention behind this PR is to allow the CI process of this project to run on both ci.jenkins.io and infra.ci.jenkins.io.
Currently, the CI is successfully running on ci.jenkins.io, and to make it run on infra.ci.jenkins.io we need to add the acceptable agent label for both the services.
And we're extending our CI to infra.jenkins.io because we want to publish the project's container image to the jenkinsciinfra/plugin-health-scoring repository which needs the CI job to execute on infra.ci.jenkins.io.

The intention behind this PR is to allow the CI process of this project to run on both ci.jenkins.io and infra.ci.jenkins.io.

Exactly.

Currently, the CI is successfully running on ci.jenkins.io, and to make it run on infra.ci.jenkins.io we need to add the acceptable agent label for both the services.

The previous label docker && linux applied for ci.jenkins.io didn't had any match in the available agents on infra.ci.jenkins.io, I replaced it with the one used in buildDockerAndPublishImage (as this shared pipeline function spawn its own agent), available on both ci.jenkins.io and infra.ci.jenkins.io

And we're extending our CI to infra.jenkins.io because we want to publish the project's container image to the jenkinsciinfra/plugin-health-scoring repository which needs the CI job to execute on infra.ci.jenkins.io.

You've understood it correctly 🙂

Note: I've simplified the build process as there was an error retrieving the new folders from the builder Docker layer on infra.ci.jenkins.io
Feel free to reimplement a multi-layer Dockerfile later.


Editor - Cut subtitle tracks

A new feature is currently in development for the new editor -  a subtitle editor. It will allow editing or creating subtitle tracks in the new editor UI and then saving them in Opencast. This modifies the videoeditor service to be able to cut subtitle tracks based on the webvtt format.
As ffmpeg is severly limited in how it allows cutting subtitles (does not work with filters), this parses the webvtt subtitles into a list container to allow for fine grained editing by the videoeditor. The WebVTT parser is still rather rudimentary and likely needs some improvements before merging. EDIT: The WebVTT parser is still rather rudimentary, but should not fail as long as the given WebVTT file is valid.
The videoeditor now discriminates between the usual audio/video elements and reference elements (aka everything that is not explicitly specified to be an audio or video element). Within the reference elements, it looks for .vtt files and cuts them according to the SMIL.
 Unfortunately, subtitle file language is encoded in the flavor subtype, making this rather awkward.
    <operation
        id="editor"
        exception-handler-workflow="partial-error"
        description="Cutting the recording according to the edit decision list">
      <configurations>
        <configuration key="source-flavors">captions/vtt+en</configuration>
        <configuration key="smil-flavors">smil/tmp</configuration>
        <configuration key="target-smil-flavor">smil/tmp</configuration>
        <configuration key="target-flavor-subtype">vtt+en+trimmed</configuration>
        <configuration key="interactive">false</configuration>
      </configurations>
    </operation>

Furthermore, this PR modifies the engage-paella-player module to allow the Paella Player to display subtitles from tracks. It also removes the "vtt+language" requirement from the flavor subtype, so you can now have "source+en" or "prepared+en".
Depends on #3601
Your pull request should…

[ ] have a concise title
[ ] close an accompanying issue if one exists
[ ] be against the correct branch
[ ] include migration scripts and documentation, if appropriate
[ ] pass automated tests
[ ] have a clean commit history
[ ] have proper commit messages (title and body) for all commits

Hi Arnei, have you tried with Paella 7? Because you modified Paella 6 to make it work

Hi Arnei, have you tried with Paella 7? Because you modified Paella 6 to make it work

I have not, thanks for the notice.

Rebased this on r/12.x

Looks hood and we can definitely add it like this although overall captions handling in Opencast is still pretty messy due to various different flavors being used. Bringing this in productions, the previous habit of always using captions/vtt+{lng} doesn't really work any longer since we need a way to distinguish between cut and uncut subtitles. Maybe we can use captions/source and captions/delivery. But that's a problem for another day.

As mentioned elsewhere, we intend to use captions/source+{lng} and captions/delivery+{lng}. The issue, of course, is that replacing only source with delivery and leaving the +{lng} untouched, is tricky with Opencast. Currently, we simply enumerate supported languages.


MiXCR import no longer works

🐛 Bug
Hi, I keep getting an error trying to load any MiXCR output to Immunarch. The message "Samples found in the metadata, but not in the folder" is not helpful, but the issue I think is that Immunarch requires the clones.txt files from MiXCR, but MiXCR outputs clones.tsv files now, which differ in content and are not recognized. Is exporting-importing airr fles the recommended solution now, or is there another way to make these compatible? Thanks

Okay, I figured Immunarch can import the newer clones.tsv files from MiXCR output too, but - just as with AIRR files - will recognize them or not depending on the file names (rules unclear). I will enquire with Immunarch, and MiXCR is not really involved in the problem, hence solved.

The issue is on the Immunarch side, not MiXCR.

Hi, could you try to export MiXCR files in the AIRR format please? We are working on the next version of Immunarch, and consider dropping the support for non-AIRR standard formats as they are too cumbersome to support.

(responded to you on Linkedin)

Hi, thanks for the reply and that's great news - looking forward to the next release!
I apologize I forgot to close this issue - I finally figured out I can import new MiXCR files just as well as airr - but both only when the files have some specific names and not others... Opened another issue about this, co closing here.


fix(react-query): fix useSuspendedQuery to receive various argument formats like original react-query v3

Fix useSuspendedQuery to receive various argument formats like original react-query v3
We can see below sentence in slash.page

The basic API is the same as react-query.

But, actually @toss/react-query's basic API is not same with react-query v3
@toss/react-query was still only available in the format below.
arg1: queryKey, arg2: queryFn, arg3: Omit<SuspendedUseQueryOptions<TQueryFnData, TError, TData, TQueryKey>, 'queryKey' | 'queryFn'>
But, anyway I thought we don't needed to change @toss/react-query's basic api to fit really well following react-query v3. Because it was not too inconvenient.
Necessity to make change @toss/react-query
But recently, from @tanstack/react-query v5, I saw a BREAKING CHANGE guide that the form of parameters received in the future will not be received in various forms, but only in object form.

So, we need to make the queryKey of arg1 or the queryFn of arg2 possibly included in SuspendedUseQueryOptions.
AS-IS
const query = useSuspendedQuery(queryKey, queryFn, options)

TO-BE
// Now we can use the same function in these different ways.
const query = useSuspendedQuery(queryKey, queryFn, options)
const query = useSuspendedQuery(queryKey, { queryFn, ...options })
const query = useSuspendedQuery({ queryKey, queryFn, ...options })

So I created this Pull Request.
Expected effect when merging this Pull Request
With this Pull Request, we can hope that @toss/react-query users will gradually migrate from react-query v3 to @tanstack/react-query v5 without big change
Add tsd test case more to test d.ts files built
tsd test case was added more in this Pull Request to test that this method does not return the wrong type, so we can easily type-check the built d.ts while working.
References
This Pull Request was made by referring to this Pull Request(https://github.com/suspensive/react/pull/38)

PR Checklist

[x] I read and included theses actions below

I have read the Contributing Guide
I have written documents and tests, if needed.

As much as there is a @tanstack/eslint-plugin-query's @tanstack/query/prefer-query-object-syntax rule, @tanstack/react-query tries to avoid receiving 2 or 3 arguments. Instead, it is induced by receiving only an object for one argument. Therefore, the current @toss/react-query must support receiving 2 or 1 arguments, the same as react-query v3.


(MAINTENANCE) /premium -> 'soldout' backgroundImage update

new image https://res.cloudinary.com/ecommerceberlin/image/upload/v1684149426/Website/PREMIUM SERVICES/sold-out.png

as @ EBE ... settings -> premium -> soldout -> URL (i have already pushed that)


fix bug in https://github.com/kean/Pulse/pull/73

fix bug in https://github.com/kean/Pulse/pull/73

In the last pull request https://github.com/kean/Pulse/pull/73 , I set the tabItem  insideNavigationView , which caused thetabItem to not be displayed. Because my project is a UIKit  project, theMainView class was not tested. very sorry

.


Create manish

@Manish-Gatla Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


Effects when a controller is connected

When a controller is connected to ds4drv we could rumble the it to notify the user. This should be enabled by default, but there should be an option to disable the feature.
This was coined by @poconbhui in #84.

I've implemented something in #92.


Update HeapSort.cpp

I have added a new feature to allow user input and sorting of elements. This modification enhances the functionality of the code by enabling users to input elements. The user-input elements are then sorted to provide a more organized output.

The sorting algorithm used is Heap sort. Users can now run the program and input a list of elements, which will be sorted and displayed.


Add TwoEmptyLinesFixer

question to community - would anyone use this rule ?

👎  here

sorry no interest, @denplatov do you know of a community which uses this style?

This codestyle would be usefull for OWOX company

what is OWOX ?
if it's only company-related and nobody else would use it - it should be your company fixer. We call it custom fixer. just register it in your config file and that's all ;)
As there is no interest of community into this fixer for one month - closing.
If you would have any problems using custom fixers feel free to ask on our gitter chat !


Tracking backport candidates

The Github system doesn't natively offer very simple ways of tracking what is needed to backport.
Suggestions on this regard:

Use backport-candidate tag, to track both open and closed pull requests/issues, cf. https://github.com/scipy/scipy/pulls?q=is%3Apr+label%3Abackport-candidate+is%3Aclosed  The tag should be removed after all backports are done.
Open separate tickets for each backport required to be done.

Now, (1) is quite simple to use in practice, but is not as visible. (2) requires a bit more work, and is more noisy. Probably up to the release manager, but doing (1) also if (2) is done would sound OK to me.

I'm fine with (1). Or otherwise (1) plus open a single issue per bugfix release that says "check all items labeled backport-candidate". (2) is a bit noisy for my taste

Ideally we're rotating the release manager role while keeping the way of working for backports unchanged. So maybe this can go in the to-be-written devguide...

We went with (1). This will be documented in gh-5424, which then closes this issue.

@rgommers, please close this issue. Thanks!

Thanks @rusergeev, closing.


Fix background reactions in the arc-green theme

As title
Before:

After:

Can you add ?v={{MD5 AppVer}} in templates/base/head.tmpl https://github.com/go-gitea/gitea/blob/master/templates/base/head.tmpl#L164-L169 and also https://github.com/go-gitea/gitea/blob/master/templates/pwa/serviceworker_js.tmpl#L34-L39 so that we can cache-bust the arc-green theme? (I noticed this after gitea.com CDN was caching old version of that file).


Start interactive mode with closed tree

A cli option to start the tree with all leaves closed would be really usefull. If you want to get a feeling for where the most storage is spent and what must be optimized it's easier to search with a breadth-first (leaves closed) instead of deep-first (all leaves open) setting.

There is already a key combination within the terminal application


4x4 OLL Parity

When the 4x4 is finished solving both yellow-green wings appear to be flipped. This is known as a OLL Parity. The algorithm to fix this issue can be found in this video. I will have to add another solve phase after parity to see if either of these cells are flipped. If either are flipped, then do the OLL algorithm, and if not, then the cube is solved.

While testing, it was found that when the cornerSwapCount was 8, the OLL parity was needed and not the normal parity. This could possibly mean that only 1 parity can occur, but further testing will be needed to confirm. If this is the case, then I could only check if it is OLL parity after the condition for normal parity is not met.

isCubeFixed will have to be revised to allow for OLL parity as well as the normal parity. The only difference would be that the counter will increase when cornerSwapCount is even rather than when it is odd.


Process Confirmation of Enrolment file for institutions

Additional Context

File layout:
-https://app.zenhub.com/files/319429847/123a8930-13e2-47b1-80f0-b720cff7b7cb/download
Sample file:
https://bcgov.sharepoint.com/:t:/r/teams/03177/Shared Documents/General/Analysis/Institution File Specs/CONR_008.TXT?csf=1&web=1&e=alE9WO
File is based off all applications that have an eligible COE request waiting
When SIMS creates ECE file and passes it to SFAS/HARRY, the SFAS COEs will be appended before sending to institutions.  So the response file will contain approvals/denials for both systems meaning we need to skip over any apps that don't match.
If one award is approved, then assume all awards are approved for that disbursement.
Sum all tuition remittance from awards for a single disbursement.
Notifications WILL NOT be included in this ticket.

Acceptance Criteria

[x] System user to read file and update database.
[x] For each disbursement, sum all tuition remittance amounts for all awards.
[x] When processing the file, treat it the same as the manual COE confirmation process and perform the same validations and notifications.
[x] Move API COE approval process to shared services to be used by schedule
[x] When COE is declined, there needs to be a reason.  If response file does not include a reason for declines, then we need to have a default reason to populate the field.
[ ] Search institutions by the institution code (it's a field in the file and also in the file name)
[x] Search disbursement by the Award_disbursement_id we sent them
[x] Assume a record can be received twice, update the first time only.
[x] If parsing the file fails, stop processing the file.
[x] When processing the records after the file has been passed, if a record does not pass validation, skip over the WHOLE DISBURSEMENT and continue.
[x] In the log, identify which records did not pass validation and report how many applications were not found and report how many applications were duplicates.
[ ] If a COE is denied, there is no "reason" provided in the CONR_008.txt file, so default to "Please speak to your institution."
[x] Create E2E test.

Questions

[ ]  What do we do when any records in the response file does not pass validations? How is this handled in SFAS currently?

SFAS stops processing the file if it encounters an error and automatically emails the Institution user to let them know there was an issue with the file. Any records that were not updated, get included again in the next file.
Notifications WILL NOT be included in this ticket

[ ] What is the folder and file name of the response file?

Reponse file name is CONR_008.TXT.  Will be on Linda. E:\OpenSSH_Root\Institution-Response[XXXX] <-- folder named by institution code.

[ ] Is the file we receive based on disbursement or awards?

Harry said it's by disbursement, but to him, it's one disbursement per funding type.  So if one ecert has CSL, BCSL, and a BC grant, then it contains 3 separate disbursements.

[ ] Is it one record per disbursement or one record per award (multiple awards to a disbursement)? Is one yes, considered yes for all?

They SHOULD NOT be approving only some of the awards.
If One is approved, then assume all approved.

[ ] Get an example file.

Submitted request for Harry to provide example file.

[ ] What is maximum number of records we can expect to process in one file?

Unlimited, but reallistically, it would be in the thousands.

Updates and assumptions to the story which were identified during implementation (Updated by @dheepak-aot )

As per discussion with @JasonCTang 24 institutions are expected to be part of the ECE response file integration with an expected transaction volume of 1000 records per day.
To identify the disbursements which belong to SIMS in the file records, we check the record with disbursement schedule Id and the application number.
Only following values in the ECE detail records are read ignoring all other values. Institution code, Award_Disbursement_Idx, Application number, Confirmed Enrollment  for Disbursement  Flag, Enrollment Confirmation Date and Pay to School $ Amount.
It is assumed that pay to school $ amount must be 0 if tuition remittance is not requested by institution.
It is assumed that Pay to school $ amount will be 0 for BGPD and SBSD as it cannot be requested for these awards.
When an enrolment is confirmed, the enrolment period validations are done based on enrolment confirmation date in the file.
The enrolment decline reason content has been updated for declining enrolment through ece file integration  as per discussion with @hellolynn-tbtb as A reason was not provided by your institution. Please speak to your Financial Aid Officer at your institution for details.

@JasonCTang During the implementation, at a discussion with @andrewsignori-aot we found that, the institution location code is unique only within the institution as per our database constraint. e.g. InstitutionA can have a location with code KHID and Institution B can have a location with same institution code KHID.

So is it possible that 2 different institution's locations can have same institution code?
All the institution integration efforts were done based on institution code being unique across SIMS.

Test scenarios covered by E2E tests.


Improvement of two little bugs

Improvement of two little bugs like:
PDI-13657 - Loging from unknown step - propouse solution
PDI-13655 - Repository explorer - opening long time - proposed solutions
It's my first commit to pentaho Kettle.
I'm java developer who use Kettle in job. I think this changes is necessary.
I don't want recompile Kettle code for each new release and update libraries.
Please commit it to trunk.
Regards,
Pawel M

Hey Pawel M,
Can you please create a test case for your fix of PDI-13655 so we can merge in your code fix. Thanks.

Yes, ofcourse. I need litle time cuz i have small actident on playground.

Test case:
Download test_repo.zip from https://github.com/Dentka15/pentaho-kettle/blob/master/test_reop.zip and put folder "test_repo" in your any repository.
There is 2000 *.ktr files in 1000 folders.
Now start spoon. In menu choose File->Open... (Ctrl + O)
I have to wait 3m 15s until window pop up.
After my fix i'm waiting 2s
My cpu: Intel® Xeon(R) CPU E31225 @ 3.10GHz × 4
Is it enough to test?

@Dentka15
Thank you for the attempt at providing a pull request for this.  I can see the issue and understand how you are attempting to resolve it here.  Here is some feedback, both general and specific to this pull request:

Please split this up into two separate pull requests since the issues resolved here are unrelated.  That makes it easier for us to pull them in as the opportunity arises.

Not a requirement, but please consider making a separate branch in github for each issue you are working on and issuing the pull request from that branch instead of from your pentaho-kettle/master branch.  That can prevent difficulties when trying to rebase or update the branch.

When you have a test case or example support files, make sure to attach them to the JIRA case rather than putting them in your source tree.  If we were to merge this, that sample would end up in the root of the PDI source folder.

I think the idea of caching the repository tree is a reasonable plan, but it needs some more work in addition to unit tests and QA.

There are many places in the class where loadRepositoryDirectory() is called which would repeatedly blow away the cached object.

There are also several places where the "tree" variable is overshadowed in methods which can lead to nasty bugs if a developer doesn't notice.

The "noop" comment in the clearSharedObjectsCache() method should be removed if it is no longer a no op.

Instead of the refreshRepo() method, setting the tree object to null should just happen in clearSharedObjectsCache().  I'd expect a refresh to actually rebuild the tree which is what happens in loadRepositoryDirectory() already.

We'll keep the JIRA case for the repo work on the backlog, and one of our developers can look at it after it is triaged.  If you want to take another crack at the repo pull request, I'd be happy to review it again.


Update omg-lol extension

Description

Adds support for create, list, delete of statuses in your omg.lol Statuslog
Updated screenshots to be consistent with new ones

Screencast

Checklist

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

Hi @rknightuk 👋
Thank you for your contribution 🔥
@danpalmer, could you please review this? I don't have an omg.lol account 🙂

I have reached out to Dan on Mastodon too, he was aware I was doing the work so he should be along at some point to confirm he's happy with it.

Awaiting last push from @rknightuk

Updated to use unicode-emoji (https://www.npmjs.com/package/unicode-emoji) for emoji listing instead of the hard-coded list.

Thanks for the push @rknightuk
@danpalmer do you have anything else?

Nope all good from me! :shipit:


README.md: Add model compatibility list

The models listed at https://github.com/odya/esphome-powmr-hybrid-inverter use the same dongle, but for some reason they are not mentioned in its page:

I'm going to add them.

The POW-HVM2.0H-12V and POW-HVM2.4H-24V seem to be discontinued, I only found an archived page for the latter.

Please note that, as I realized in odya/esphome-powmr-hybrid-inverter#9, support for the POW-HVM5.5K-48V is provided by a different dongle.

David or Leo - I need some project help connecting my PowMr through some controller (Raspberry PI, ESP, Arduino) to an EV charger.  I can pay a nominal hourly fee.  Are either of you available and interested?

Could you provide us some info, such as brand and model, of the EV charger?

Hello - I will get back to you.  Thank you
On Sun, Feb 18, 2024 at 1:50 PM Davide Beatrici @.***>
wrote:

Could you provide us some info, such as brand and model, of the EV charger?
—
Reply to this email directly, view it on GitHub
https://github.com/leodesigner/powmr_comm/pull/5#issuecomment-1951412746,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGH4UTFCGI6PNILJZDBN2H3YUJEQRAVCNFSM6AAAAAAZYMC6BGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJRGQYTENZUGY
.
You are receiving this because you commented.Message ID:
@.***>

I am using the go-e homefix charger 11kWhttps://go-e.com/en/products/go-echarger-homefixIt sends many data via MQTT and is also controllable via MQTT.Thx   Manfred-- Manfred ReisnerCEO, Smart Mobility, System Safetye-mail: @.: +4368181113224ACTARON GmbHHauptstrasse 297441 PilgersdorfAustria
-------- Ursprüngliche Nachricht --------Von: Davide Beatrici @.> Datum: 18.02.24  19:50  (GMT+01:00) An: leodesigner/powmr_comm @.> Cc: Subscribed @.> Betreff: Re: [leodesigner/powmr_comm] README.md: Add model compatibility list (PR #5)
Could you provide us some info, such as brand and model, of the EV charger?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

Hello,well the Solarinverter does not need to communicate with the wallbox - just data exchange via MQTT would be nice.A commonly used MQTT- Broker in the smart home domain is Mosquitto.Like all smart home devices the wallbox fully supports MQTT and is also controllable.Manfred-- Manfred ReisnerCEO, Smart Mobility, System Safetye-mail: @.: +4368181113224ACTARON GmbHHauptstrasse 297441 PilgersdorfAustria
-------- Ursprüngliche Nachricht --------Von: Sledje @.> Datum: 20.02.24  11:49  (GMT+01:00) An: leodesigner/powmr_comm @.> Cc: Subscribed @.> Betreff: Re: [leodesigner/powmr_comm] README.md: Add model compatibility list (PR #5)
Hello - I will get back to you.  Thank you
On Sun, Feb 18, 2024 at 1:50 PM Davide Beatrici @.***>
wrote:

Could you provide us some info, such as brand and model, of the EV charger?

—

Reply to this email directly, view it on GitHub

https://github.com/leodesigner/powmr_comm/pull/5#issuecomment-1951412746,

or unsubscribe

https://github.com/notifications/unsubscribe-auth/BGH4UTFCGI6PNILJZDBN2H3YUJEQRAVCNFSM6AAAAAAZYMC6BGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJRGQYTENZUGY

.

You are receiving this because you commented.Message ID:

@.***>

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>


Recompiled for 1.8 hotfix

Hi there,
The mod was crashing on game start with the last 1.8 hotfix. It just required a recompile so I did it to resume my game :D I'm sharing it here in case it can save you some time.
Anyway thanks for your great mod !

Thanks mate, just noticed this, was fixed with my own commit at
https://github.com/hamzah-hayat/AdditionalQuests/releases/tag/v1.1.3


do not handle errors when registering

https://github.com/charlesread/fastify-jwt-webapp/blob/master/plugin.js#L26-L29 the 2nd argument function is not needed.

fixed in 931b8d6


[bootstrap] Don't copy unnecessarily.

This fixes non-null builds when using the SwiftPM build as part of another
build script.

rdar://problem/30450144 SwiftPM is always rebuilding

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please test

Whee


Conn res fix

Fix for random connection resiliency test failures in Linux with Linux SQL Server.

Coverage increased (+0.03%) to 73.693% when pulling 3770d3c42b6ce9e6385aa16dac99a5334ee9d197 on david-puglielli:conn-res-fix into e90fd2c4a402ceeb7211d517735e11fb0096795f on microsoft:dev.


Patient panels does not work

It looks like the check-in that updated the project to use fhir_dbt_utils broke patient panels. This command will not run as specified in the docs:
dbt run --selector patient_panel

Agree.  Tried running this and receive this error:
dbt run --selector patient_panel
14:16:04  Encountered an error:
Compilation Error
Model 'model.fhir_dbt_analytics.latest_demographics' (models/summaries/patient_panel/latest_demographics.sql) depends on a node named 'Patient_view' which was not found

Thank you both for flagging this. Indeed, this model was referencing a legacy model name. This should now be resolved with commit 80414ca


Optimizations table now reflect that OPT-4 is enabled by default

According to the documentation (See https://pyjion.readthedocs.io/en/latest/opt/opt-4.html#configuration), OPT-4 is enabled when the optimization level is at least 1.
This contradicts what is specified in the table.
This commit fixes just that.

Good spot! thanks


java.nio.channels.ClosedChannelException

Getting this error spammed in console, also attempting to login to the server results in a json error in client.
[17:58:52] [Server thread/INFO]: cloakfox[/xxxxxxxx:45070] logged in with entity id 1550 at ([world]90.00878933169102, 12.686565755031928,<PHONE_NUMBER>542614)
[17:58:52] [Netty Epoll Server IO #0/WARN]: An exception was thrown by a user handler's exceptionCaught() method while handling the following exception:
java.nio.channels.ClosedChannelException
[17:58:52] [Server thread/INFO]: cloakfox lost connection: Disconnected
[17:58:52] [Netty Epoll Server IO #0/WARN]: An exception was thrown by a user handler's exceptionCaught() method while handling the following exception:
java.nio.channels.ClosedChannelException
[17:58:52] [Netty Epoll Server IO #0/WARN]: An exception was thrown by a user handler's exceptionCaught() method while handling the following exception:
java.nio.channels.ClosedChannelException
[17:58:52] [Netty Epoll Server IO #0/WARN]: An exception was thrown by a user handler's exceptionCaught() method while handling the following exception:
java.nio.channels.ClosedChannelException
[17:58:52] [Netty Epoll Server IO #0/WARN]: An exception was thrown by a user handler's exceptionCaught() method while handling the following exception:
java.nio.channels.ClosedChannelException
[17:58:52] [Netty Epoll Server IO #0/WARN]: An exception was thrown by a user handler's exceptionCaught() method while handling the following exception:
java.nio.channels.ClosedChannelException
[17:58:52] [Netty Epoll Server IO #0/WARN]: An exception was thrown by a user handler's exceptionCaught() method while handling the following exception:
java.nio.channels.ClosedChannelException

what version is this on?

Oh sorry this is on version 0.4.4

Should be fixed in the latest commit.


fix: #70 and enforce use pnpm

When we enforce pnpm, we still need add legacy-peer-deps=true?

When we enforce pnpm, we still need add legacy-peer-deps=true?

Not necessary, but  can skip preinstall to use npm  T,T

It seems vercel build not work. Please update new pnpm-lock.yaml with pnpm@8.


slab: Add Slab::create_and_insert.

Hi,
I find current insert limiting, so I come up with this. Comments?

Looks good. I agree, I have hit cases where this would be useful.
Minor nitpicks. What about naming this insert_with and refactoring insert() to use this function?

OK. If this is mergeable idea than I will work on re-factoring.

I think it is. Just to make sure that I understood it, the function reserves a token, then calls a builder function w/ that token allowing creating the value w/ the token in one go (vs. having to store Option).
However, there is one thing to consider: How to handle panics. Aka, what happens if the builder function panics? I believe, in this case, the token will leak. Is that correct?

The token is passed to builder before anything is done with the slab, so in case of panic in the builder the token should not leak. Maybe I should write a test for this.
I've refactored insert, it's a bit tricker than ideal, because I can't see a way to both pass val to the builder function and potentially pass in Err. I think optimizer should have no problem with removing the additional check.

BTW. The slab allocator seems like something that could be usable as a small separate library.

Looking good. The main reason that this isn't a separate lib is that it is bound to Token (and so is the rest of mio).
It may still be a good idea to split it out though.

I don't know how to test leaks on panic. I need to use threads for it, and then if I need to wrap shared slab in Mutex to make it Sync, which will leads to problem if builder panics when lock is held anyway. In mio the handler is working in a single thread anyway, right?

Thanks! I merged this here: https://github.com/carllerche/slab/commit/ac96f1a8ab99af4e1a2b3eea0ce86371806da78e


fix-1506-optional-in-params

Closed #1506

Pull request type

Please check the type of change your PR introduces:

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

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

Project coverage is 41.03%. Comparing base (90106e7) to head (4264123).

:exclamation:  There is a different number of reports uploaded between BASE (90106e7) and HEAD (4264123). Click for more details.
HEAD has 520 uploads less than BASE

Flag
BASE (90106e7)
HEAD (4264123)

macOS
24
2

unit
72
4

go-1.21.x
48
2

ubuntu
103
2

go-1.22.x
48
2

go-1.23.x
55
0

ydb-23.3
24
0

integration
71
0

windows
24
0

ydb-24.2
24
0

ydb-24.1
23
0

ydb-nightly
8
0

experiment
8
0

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #1517       +/-   ##
===========================================
- Coverage   70.93%   41.03%   -29.90%
===========================================
  Files         356      353        -3
  Lines       37180    36942      -238
===========================================
- Hits        26372    15158    -11214
- Misses       9707    21276    +11569
+ Partials     1101      508      -593

Flag
Coverage Δ

experiment
?

go-1.21.x
42.37% <100.00%> (-28.01%)
:arrow_down:

go-1.22.x
40.99% <100.00%> (-27.45%)
:arrow_down:

go-1.23.x
?

integration
?

macOS
41.02% <100.00%> (-0.03%)
:arrow_down:

ubuntu
41.02% <100.00%> (-29.91%)
:arrow_down:

unit
41.03% <100.00%> (-0.03%)
:arrow_down:

windows
?

ydb-23.3
?

ydb-24.1
?

ydb-24.2
?

ydb-nightly
?

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

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


scala: {2.9.3, 2.10.6, 2.11.11, 2.12.4} -> {2.10.7, 2.11.12, 2.12.4}

Motivation for this change
Update minor versions, remove unsupported 2.9.x

@GrahamcOfBorg build scala_2_10 scala_2_11 scala_2_12

Failure on aarch64-linux (full log)
Partial log (click to expand)
error: while evaluating the attribute 'installPhase' of the derivation 'scala-2.10.7' at /var/lib/gc-of-borg/nix-test-rs-1/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-1/pkgs/stdenv/generic/make-derivation.nix:148:11:
while evaluating the attribute 'buildInputs' of the derivation 'openjdk-8u172b02' at /var/lib/gc-of-borg/nix-test-rs-1/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-1/pkgs/stdenv/generic/make-derivation.nix:148:11:
while evaluating the attribute 'buildCommand' of the derivation 'openjdk-bootstrap' at /var/lib/gc-of-borg/nix-test-rs-1/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-1/pkgs/stdenv/generic/make-derivation.nix:148:11:
No bootstrap for system

Success on x86_64-linux (full log)
Partial log (click to expand)
patching script interpreter paths in /nix/store/4xw2ppkmqcc8z5a9xhz6qn3c8wkzh15n-scala-2.11.12
/nix/store/4xw2ppkmqcc8z5a9xhz6qn3c8wkzh15n-scala-2.11.12/bin/.scalap-wrapped: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/zqh3l3lyw32q1ayb15bnvg9f24j5v2p0-bash-4.4-p12/bin/bash"
/nix/store/4xw2ppkmqcc8z5a9xhz6qn3c8wkzh15n-scala-2.11.12/bin/.scaladoc-wrapped: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/zqh3l3lyw32q1ayb15bnvg9f24j5v2p0-bash-4.4-p12/bin/bash"
/nix/store/4xw2ppkmqcc8z5a9xhz6qn3c8wkzh15n-scala-2.11.12/bin/.scalac-wrapped: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/zqh3l3lyw32q1ayb15bnvg9f24j5v2p0-bash-4.4-p12/bin/bash"
/nix/store/4xw2ppkmqcc8z5a9xhz6qn3c8wkzh15n-scala-2.11.12/bin/.scala-wrapped: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/zqh3l3lyw32q1ayb15bnvg9f24j5v2p0-bash-4.4-p12/bin/bash"
/nix/store/4xw2ppkmqcc8z5a9xhz6qn3c8wkzh15n-scala-2.11.12/bin/.fsc-wrapped: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/zqh3l3lyw32q1ayb15bnvg9f24j5v2p0-bash-4.4-p12/bin/bash"
checking for references to /tmp/nix-build-scala-2.11.12.drv-0 in /nix/store/4xw2ppkmqcc8z5a9xhz6qn3c8wkzh15n-scala-2.11.12...
/nix/store/p960682b0mydd7my5x8i2chfl1bhvlga-scala-2.10.7
/nix/store/4xw2ppkmqcc8z5a9xhz6qn3c8wkzh15n-scala-2.11.12
/nix/store/59njnk3z72cshj64pjwg4mifccl80i3a-scala-2.12.4

Success on x86_64-darwin (full log)
Partial log (click to expand)
/nix/store/kixlzgbsnpngis27sinpykc9qv99b6iv-scala-2.11.12/bin/.scala-wrapped: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/pkjmwq7sqrvjg7cjiph6hq0khsmfl6p8-bash-4.4-p12/bin/bash"
/nix/store/wvkbcmywmwgg2kgcirka0rnwpqjq2hdr-scala-2.10.7/bin/.scalac-wrapped: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/pkjmwq7sqrvjg7cjiph6hq0khsmfl6p8-bash-4.4-p12/bin/bash"
/nix/store/kixlzgbsnpngis27sinpykc9qv99b6iv-scala-2.11.12/bin/.scalac-wrapped: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/pkjmwq7sqrvjg7cjiph6hq0khsmfl6p8-bash-4.4-p12/bin/bash"
/nix/store/wvkbcmywmwgg2kgcirka0rnwpqjq2hdr-scala-2.10.7/bin/.scaladoc-wrapped: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/pkjmwq7sqrvjg7cjiph6hq0khsmfl6p8-bash-4.4-p12/bin/bash"
/nix/store/kixlzgbsnpngis27sinpykc9qv99b6iv-scala-2.11.12/bin/.scaladoc-wrapped: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/pkjmwq7sqrvjg7cjiph6hq0khsmfl6p8-bash-4.4-p12/bin/bash"
/nix/store/wvkbcmywmwgg2kgcirka0rnwpqjq2hdr-scala-2.10.7/bin/.scalap-wrapped: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/pkjmwq7sqrvjg7cjiph6hq0khsmfl6p8-bash-4.4-p12/bin/bash"
/nix/store/kixlzgbsnpngis27sinpykc9qv99b6iv-scala-2.11.12/bin/.scalap-wrapped: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/pkjmwq7sqrvjg7cjiph6hq0khsmfl6p8-bash-4.4-p12/bin/bash"
/nix/store/wvkbcmywmwgg2kgcirka0rnwpqjq2hdr-scala-2.10.7
/nix/store/kixlzgbsnpngis27sinpykc9qv99b6iv-scala-2.11.12
/nix/store/hy7ph31bcw0h9lhg4r5chnm4v3jxpq3f-scala-2.12.4


Viewer does not react when call the methods autoLayout and resetZoom in an specific complex

In the Complex Portal when we are showing the complex EBI-847160 the methods autoLayout and resetZoom does not work. I tried even calling them directly from the JavaScript console of Firebug but it does not react. Could you please double-check or trace what is the behaviour within the viewer?

OK, I've checked it in FireFox and Chrome and it is working for me. I think this is related to #48 and I do understand why its confusing...
Try this:
a) Click and drag on the complex (pale blue blob) - this moves the complex (and the molecules inside it). Calling autoLayout will move the molecules back to the middle. Calling resetZoom will do nothing because the pan/zoom hasn't changed.
b) Click and drag on the white background - this pans the map. Now, calling autoLayout will do nothing because the positions of the molecules haven't changed but calling resetZoom will have an effect.
Can you confirm this is the behaviour you're getting?
I think this will be fixed by just having a reset() function that calls both resetZoom() and autoLayout(), see #48.

OK, a single reset() function has been added by changes in pull #51 - try using it instead. Closing for moment.


privatize spin label

Ensures no symbol name clashes.

Nice! I didn't know about  ${:private} and ${:uid}. Thanks!

Published as 0.3.11


B-18110 Update Rank and Address Labels

B-18110
Summary
Updates rank and address labels
Verification Steps for Reviewers
These are to be checked by a reviewer.

[ ] Has the branch been pulled in and checked out?
[ ] Have the BL acceptance criteria been met for this change?
[ ] Was the CircleCI build successful?
[ ] Has the code been reviewed from a standards and best practices point of view?

Setup to Run the Code

Instructions for starting storybook
Instructions for starting the MilMove application
Instructions for running tests

How to test

Create a new customer

Navigate through profile setup form to see new values

After creating a new profile, navigate to profile screen to see updated cards

Log in as a service counselor

Open existing move

Under move details, navigate to allowances and see new values

Also click "Edit Allowances" to view new values

Screenshots

Warnings

:warning:

Files located in legacy directories (src/shared or src/scenes) have
been edited. Are you sure you don’t want to also relocate them to the new file structure?
View the frontend file org ADR for more information

  Generated by :no_entry_sign: dangerJS against cc9a7e0fc24e5fafc3d3819049f234bcc176cfa1


[WIP]test/test_connectionConfig.py:Adds test to for connectionConfig

Added test for test_connectionConfig.py and bumped coverage upto 75%

Closes #123

@Cheukting will it be possible to merge the PR, the coverage bumped from 62% -> 75%.
Since issue #123  would eventually need a handful of PR for other modules to bump the coverage. So leaving it 75% isn't that bad I guess that way I can pick up other modules other than connectionConfig.py and bring them up to 75 - 80%
Name                                                     Stmts   Miss  Cover
----------------------------------------------------------------------------
terminusdb_client/__init__.py                                4      0   100%
terminusdb_client/__version__.py                             9      0   100%
terminusdb_client/woql_utils.py                             66     43    35%
terminusdb_client/woqlclient/__init__.py                     1      0   100%
terminusdb_client/woqlclient/api_endpoint_const.py          16      0   100%
terminusdb_client/woqlclient/connectionCapabilities.py     250    149    40%
terminusdb_client/woqlclient/connectionConfig.py           186     46    75%
terminusdb_client/woqlclient/dispatchRequest.py             53     11    79%
terminusdb_client/woqlclient/documentTemplate.py            27     27     0%
terminusdb_client/woqlclient/errorMessage.py                36     36     0%
terminusdb_client/woqlclient/errors.py                      21     10    52%
terminusdb_client/woqlclient/id_parser.py                   37     11    70%
terminusdb_client/woqlclient/woqlClient.py                 187     90    52%
terminusdb_client/woqldataframe/__init__.py                  1      0   100%
terminusdb_client/woqldataframe/woqlDataframe.py            61      3    95%
terminusdb_client/woqlquery/__init__.py                      2      0   100%
terminusdb_client/woqlquery/woql_core.py                    96     63    34%
terminusdb_client/woqlquery/woql_library.py                279    239    14%
terminusdb_client/woqlquery/woql_query.py                 1409    560    60%
----------------------------------------------------------------------------
TOTAL                                                     2741   1288    53%
Coverage XML written to file coverage.xml```


a.empty, a.bool(), a.item(), a.any() or a.all() when inserting DF into InfluxDB

Hi! I've updated both the influxdb client and pandas to the latest versions, using pip3. These are my versions.
Name: pandas
Version: 0.22.0
Summary: Powerful data structures for data analysis, time series,and statistics
Home-page: http://pandas.pydata.org
Author: The PyData Development Team
Author-email<EMAIL_ADDRESS>License: BSD
Location: /usr/local/lib/python3.5/dist-packages
Requires: python-dateutil, numpy, pytz

Name: influxdb
Version: 5.0.0
Summary: InfluxDB client
Home-page: https://github.com/influxdb/influxdb-python
Author: UNKNOWN
Author-email: UNKNOWN
License: MIT License
Location: /usr/local/lib/python3.5/dist-packages
Requires: python-dateutil, requests, pytz, six

Connected to http://localhost:8086 version 1.4.2
InfluxDB shell version: 1.4.2

After the upgrade, I can no longer insert my DF into influxdb. This is the class that implements the insertion:
class inflxDbClass:

    def connectDF(self,dbAddress,dbPort,dbUser,dbPassword,dbName):
        self.dbConnDF = DataFrameClient(dbAddress, dbPort, dbUser, dbPassword, dbName)

    def insertDF(self, dbName, tbName, df, tags):

        try:
            self.connectDF(DB_INFLX_ADDRESS, DB_INFLX_PORT, DB_INFLX_USER, DB_INFLX_PASS, dbName)
            self.dbConnDF.write_points(df, tbName, tags = tags)
            return GENERAL_SUCCESS

        except Exception as e:
            logPrint("Something happened ...",DEBUG_ID)
            logPrint(e,DEBUG_ID)
            logPrint("DF: " + str(df), DEBUG_ID)
            return GENERAL_ERROR

As said, this was working before the upgrade happened. Now I'm getting the following error when trying to do the insert (I'm catching the exception):
Something happened ...
The truth value of a DataFrame is ambiguous. Use a.empty, a.bool(), a.item(), a.any() or a.all().

The code that constructs the DF, I insist, has not been changed: only the versions of both influx+pandas have changed.
In order to insert the DF into influxDB I would do:
inflxObjectD    = inflxDbClass()
result          = inflxObjectD.insertDF(DB_INFLX_DBNAME, tableName, df, tags)

I really do not understand the error message. My code is rather big so I do not know what to share about it.
The DF is fine. Indeed: I can export it to a csv file by isuing df.to_csv() and open it with Excel. And this works. So I guess that there is, a priori, no problem with the DF itself. My guess is with the influxdb client.
Has anyone have a hint on what to search for?
Thanks!

I'm getting the same.  If I pip install 4.1.1 everything is fine, but 5.0 causes the "Truth value of DataFrame is ambiguous" issue to show up.  My df creation code has not changed.  Pip Upgraded influxdb to 5.0 and the issue showed up.  I have pip uninstalled it, and re-installed but it didn't help.  Went back to running 4.1.1.

Can confirm this issue still exists on v5.2.2


Display Dashboard

Display the data from backend to frontend and show all the fields.
For any doubts ask on discord.
Go to localhost:3000/dashboard to see your changes.

claim

Assigned to @GammaMicrowave


fixed_model_name behaviour

Rasa NLU version: 0.10.1
Used backend / pipeline: spacy_sklearn
Operating system: osx
Issue:
When using "fixed_model_name" in configuration file, it would be nice if the server would treat the fixed_model_name as the default model for requests. Currently one needs to specify the model in the query, which seems unnecessary when providing a fixed model name in the config.
The behaviour is also misleading when specifying a fixed_model_name that has not the required date format, as requests will not work at all then.
E.g. config
{
  "pipeline": "spacy_sklearn",
  "language": "en",
  "path": "./projects",
  "fixed_model_name": "model_en_deploy",
  "data": "data/test.json",
  "port": 5000
}

with query
curl -X POST \
  http://localhost:5000/parse \
  -H 'content-type: application/json' \
  -d '{
	"q":"I am looking for blue jeans"
}'

results in
{
    "error": "time data 'en_deploy' does not match format '%Y%m%d-%H%M%S'"
}

@tmbo: it looks like an enhancement to me, not really a bug. To begin with:

The behaviour is also misleading when specifying a fixed_model_name that has not the required date format, as requests will not work at all then.

It's not that misleading: you're not supposed to have custom model names as models only represent a snapshot of a project: you have for instance an English speaking customer support chatbot project you named "en_customer_support", the "model_XXXXXX" and "model_YYYYYY" just represent two versions of this same chatbot project. Models are a way to version your project so you're not supposed to ask for  "en_deploy" as a model.
As for setting a "fixed_model_name", more generally it would be interesting to have some per project metadata specifying the default model to use, the language, the pipeline and if the project is production ready or in development.

@PHLF if you are not supposed to have custom model names, a fixed_model_name configuration seems misleading to me..
In terms of ensuring that you deploy a certain model, which might not be the most current, it would be helpful to specify its name in the configuration, and throw an error if this model is not available, without a fallback to the latest model

Just to be sure, do we both agree that "fixed_model_name" is currently not a genuine parameter name ?
It wouldn't be misleading even if you don't have a custom model name as it would allow you to specify the model you want to use as a default for a given project: "fixed_model_name": "model_MMDDYYYY".
As for throwing an error, a fallback means what it means: it allows you to deliver your service in a worst case scenario, because maybe at some points the models may be stored in a separate DB that gets corrupted and the server is not able to find the right model anymore. Moreover there is currently a server side warning when the server uses a fallback model.

so are we saying the fixed_model_name is completely broken at the moment? Does it ever work as expected?

Exactly.
The error message "error": "time data 'en_deploy' does not match format '%Y%m%d-%H%M%S'" says it all: the model name need to match a specific format thus nullifying this fixed_model_name parameter as long as it differs from the model_YYYYMMDD-HHMMSS format.
You can set a fixed_model_name but you cannot set a custom model name.

So we either need to remove this parameter or fix it. One proposal for fixing it is to make it model_prefix rather than fixed_model_name and just have it override the model_ prefix?
@clennan can you help me with your use case, why are you trying to use a fixed model name?

Seems fixed_model_name parameter has been used only while training and it creates the model name with specified value. However the status and parse endpoints expect model name in specific format. Hence good to remove fixed_model_name (at least from Configuration documentation  ) until status and parse endpoint does not recognize it.

this should be fixed.


Documentation for DISABLE_NEWLINE_AUTO_RETURN is wrong

See: https://github.com/microsoft/terminal/issues/4126#issuecomment-571418661

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

ID: 34ea4fba-5919-fff6-9941-f213a557c651
Version Independent ID: 5c22080b-6969-4721-5e5c-40c348680701
Content: SetConsoleMode function - Windows Console
Content Source: docs/setconsolemode.md
Product: console
GitHub Login: @bitcrazed
Microsoft Alias: richturn

https://github.com/MicrosoftDocs/Console-Docs/blob/main/docs/includes/console-mode-flags.md is the source file, drafting some potential content here to update the documentation to more accurately reflect the flag behavior.
DISABLE_NEWLINE_AUTO_RETURN: When writing with WriteFile or WriteConsole, this prevents the cursor from automatically moving to the next line when text reaches the end of the line. The written character will be printed in the final position on the line and the cursor will remain above this character as if ENABLE_WRAP_AT_EOL_OUTPUT was off, but the next printable character will be printed as if ENABLE_WRAP_AT_EOL_OUTPUT is on. No overwrite will occur. Specifically, the cursor quickly advances down to the following line, a scroll is performed if necessary, the character is printed, and the cursor advances one more position.
The typical usage of this flag is intended in conjunction with setting ENABLE_VIRTUAL_TERMINAL_PROCESSING to better emulate a terminal emulator where writing the final character on the screen (../in the bottom right corner) without triggering an immediate scroll is the desired behavior.
ENABLE_VIRTUAL_TERMINAL_PROCESSING
When writing with WriteFile or WriteConsole, characters are parsed for VT100 and similar control character sequences that control cursor movement, color/font mode, and other operations that can also be performed via the existing Console APIs. For more information, see Console Virtual Terminal Sequences.
Additionally this adds an additional state to end-of-line wrapping that can delay the cursor move and buffer scroll operations. Normally when ENABLE_WRAP_AT_EOL_OUTPUT is set and text reaches the end of the line, the cursor will immediately move to the next line and the contents of the buffer will scroll up by one line. In contrast with this flag set, the scroll operation and cursor move is delayed until the next character arrives.
@j4james would this content be more correct/align with the flag functions?

I think the ENABLE_VIRTUAL_TERMINAL_PROCESSING description sounds good, but the DISABLE_NEWLINE_AUTO_RETURN description is still wrong.
Usually when you output a linefeed character (U+000A), the cursor is moved down a line, and also moved across to the leftmost column (essentially executing a carriage return). However, when the DISABLE_NEWLINE_AUTO_RETURN mode is set, that carriage return step is skipped. The cursor moves down a line, but remains in the same column in started from.

Thank you @j4james, I worked on incorporating that clarification into the document's pull request.

@j4james does the associated PR look correct?
cc: @DHowett

@pizzaz93 The changes to ENABLE_VIRTUAL_TERMINAL_PROCESSING look good to me, but the description of the DISABLE_NEWLINE_AUTO_RETURN flag is still not quite right.
It doesn't have anything to do with ENABLE_WRAP_AT_EOL_OUTPUT or what happens when text reaches the end of the line. It's also got nothing to do with writing a character in the bottom right corner without trigger a scroll. All it effects is the behavior of the linefeed control character (U+000A) the way I explained in the comment above.

Yeah. The second paragraph should be removed entirely.

@DHowett or @zadjii-msft, I've implemented @j4james's feedback would you be able to review the referenced PR?

I'd completely forgotten about this and it just bit us over at https://github.com/vezel-dev/cathode/issues/156. Would be good if we could get this very wrong documentation corrected. It's a bit discouraging to see that #262 went ignored...

The highly misleading docs issue about DISABLE_NEWLINE_AUTO_RETURN is known since 2020, and yet the docs are still not fixed - https://github.com/MicrosoftDocs/Console-Docs/blob/main/docs/includes/console-mode-flags.md and also not at the "real" docs site - https://learn.microsoft.com/en-us/windows/console/setconsolemode .
Please fix the docs. The console is complex enough on its own. Highly misleading docs don't help...
Also, as far as I can tell from my own testing, it seems that ENABLE_VIRTUAL_TERMINAL_PROCESSING also does what the wrong DISABLE_NEWLINE_AUTO_RETURN docs still say, and also make the console ignore the ENABLE_WRAP_AT_EOL_OUTPUT bit.
I.e. when VT is enabled it also enables the "additional state at EOL where the cursor stays once at the last column, and another char will then make it wrap" (to prevent scroll etc), and it's impossible to disable wrap-at-eol, and it's also impossible to disable auto-return.
I don't think it's necessarily a bad thing, but it would be nice to have it documented.


Improve documentation of address space exports

The documentation of address space exports should be improved:

uses wrong syntax export instead of exports
needs explanation of RBAC that must be applied by applications + examples (also consider a clusterrole example for testing purposes)

This has been added in ref-address-space-example-exporting-endpoints.adoc


fix: dummy smtp server

Fixes #401
context for workaround: https://github.com/go-gomail/gomail/issues/113

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 29.436%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/start/start.go
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1450

Relevant Lines:
4926

💛  - Coveralls


Rate⭐ From Search 🔍

Is your feature request related to a problem? Please describe.
Currently there's no way to rate a book after searching it
Describe the solution you'd like
I'm able to rate through the modal after I search for a book

Additional context
The stars that are currently displayed is the average rating. Changes are made and all tests pass. If user is not logged in, and the book is rated he/she should be redirected to login page.

I would love on this issue @Praful932

Can I please contribute to this issue? (GSSoC 21)

Hey! This issue seems interesting and it is a part of my work as well so I would like to contribute to it. (GSSoC 21) @Praful932

Heyy!! Can I work on this issue as a GSSOC participant?

Go ahead @sk1122

Thanks @maneprajakta

Hey , I am participant in GSSOC'21. Can you please assign this to me?


Remove swizzle node for 1.39

This changelist removes the swizzle node from the MaterialX data libraries for v1.39, adding an upgrade pathway for instances of this node in legacy documents.
Additionally, it addresses a minor inconsistency in the specification for the extract and extractrowvector nodes, which take zero-based indices rather than positive-valued selections.

Attaching updated comparison renders in GLSL and OSL:
MaterialXRenderTests_04_28_2024.pdf

@ld-kerley Absolutely, and there's a dedicated file in the MDL shader generation system that can now be removed as well, but I'm taking this change in stages!


🌗  Remote CDN and theme button for the article theme

Fixes #342.

Regarding #466, simply adds the ThemeButton to the article theme. Tested locally:

Enables setting CONTENT_CDN  through env to replace localhost:{port} with https://remote.example.com.

[!NOTE]
Right before opening this PR, I realized that a similar PR #346 exists, enabling {hostname}:{port}. This is slightly different as the purpose is to connect to a remote CDN host (tested with https://cdn.neurolibre.org) without specifying a port. Happy to account for the changes from #346 to avoid conflict.

Thanks for the changes @agahkarakuzu -- really awesome to have you contributing here. Excited to see your launch in a few weeks. 🚀

Thank you so much @rowanc1! The event will be much cooler thanks to all the amazing effort you've put in this project!


fix: Properly close resources and handle errors in agent and storage

This PR adds proper closure of resources and error handling for code related to agent and storage to avoid memory leaks and surface potential errors.

/retest

@dtrifiro Great idea! Updated.

/lgtm
/approve


TypeError: label_list() got an unexpected keyword argument 'doc'

I'm trying to add skydoc on my rules, but it fails with this "got an unexpected keyword argument 'doc'". In fact, I can reproduce with any other existing repository

bazel release 0.18.0
Python 2.7.13 (default, Nov 24 2017, 17:33:09)
[GCC 6.3.0 20170516] on linux2

Steps:
git clone https://github.com/georgeliaw/rules_wheel.git
cd rules_wheel
bazel build //:wheel-docs

WORKSPACE
git_repository(
    name = "io_bazel_rules_sass",
    remote = "https://github.com/bazelbuild/rules_sass.git",
    tag = "0.0.3",
)
load("@io_bazel_rules_sass//sass:sass.bzl", "sass_repositories")
sass_repositories()

git_repository(
    name = "io_bazel_skydoc",
    remote = "https://github.com/bazelbuild/skydoc.git",
    tag = "0.1.4",
)
load("@io_bazel_skydoc//skylark:skylark.bzl", "skydoc_repositories")
skydoc_repositories()

Also, it seems the version of the "getting started" is quite outdated https://skydoc.bazel.build/docs/getting_started.html

It sounds like you've discovered that the doc param on various attr.() calls isn't supported.  the current expectation of skydoc is that docs come in a comment right after the rule declaration, e.g.
foo = rule( ...
)
""" docs go here""

but I would love it if we had support for doc parameters.  A bunch of my rules share some of their attrs, and we end up documenting these multiple times at the moment since we need the docs after every rule; even though we've deduped the actual attr declaration by declaring some dicts that we reuse like attrs = dict(_some_attrs.items() + _other_attrs.items())


Consolidate CloudAdapter classes

@msomanathan, can you please finish up the spec for this? Let me known when is done so we can move it to ready.

Merge CloudAdapterBase with CloudAdapter
Merge CloudChannelServiceHandler with CloudSkillHandler
Remove TurnStateConstants

We will not address this at the moment.


Introduce Gradle Version Catalog

The CI seems to fail due to a bug in the exoplayer library.
So maybe we should stay on the previous version for now and emphasize again that the video is just a workaround for the buggy animation that we want to replace at some point anyway with a proper one.

If I understand the conversation in the GitHub issue correctly, only apps using DownloadService need to add this permission to their manifest and request the permission at runtime. The issue is classified as a bug in their demo app, not in the library, so the issue will not be fixed for us by future updates. Reverting the update of ExoPlayer dependency does not solve the CI failure either - it was triggered by updating AGP plugin which introduced a new lint check. Since we don't use DownloadService, there is no need to include the permission in the manifest nor request the permission at runtime. I would propose to exclude the lint check specifically for the notification in ExoPlayer's DownloadService via lint.xml.