Stuck on "Registering Device"

Describe the bug
After updating to latest DP release today on my C3, my device got stuck on "Registering Device" screen. Could not reset to factory default on boot screen. Many others reported similar problem on Discord.
Provide a route where the issue occurs
serial: 2246c0fa
openpilot version
Latest
Additional info
Thanks

Wrong thread for a Dragon Pilot problem.

When I power on, my device got stuck on "Registering Device" screen.
I dis connect the power, and then connect power. It's still stuck on "Registering Device" screen.
How can I reset my device?
What is Dragon Pilot problem?


[Bug]: Linting is not reactive when path is incorrect

Is there an existing issue for this?

[X] I have searched the existing issues

Description
See loom recording here
Steps To Reproduce
1 Drag a button widget
2. Create an entity (Api, Query or JSObject)
3. In any field, add an incorrect path eg. entityname.tetete
4. Delete entity, and see that linting doesn't get updated
Public Sample App
No response
Version
Cloud

Cause of issue
When we extractReferencesFromBinding, we do not add subpath top-level entities, except they are explicitly referenced (to prevent unnecessary evaluations and performance dip).
so, for binding {{Api1.data.unknownProperty}}, extracted reference is Api.data. However, for {{Api1.unknownProperty}} no reference is extracted because Api1.unknownProperty isn't present in the data tree and top-level subpath entities are not added as references except explicitly referenced (eg. for binding {{Table1}}, extracted reference is Table1 ).
Different explored solutions

Add subpath top-level entities if present in the dataTree. For {{Api1.unknownProperty}}, add Api1 as a reference.

Cons
a. Leads to unnecessary evaluations.
b. Might increase the chances of a cyclic dependency in the application.

Create a data structure that stores a list of all extracted identifiers from bindings throughout the application.

Cons
a. As an application grows, the number of extracted identifiers grows much faster than the number of entities present in the App, manipulating the data structure might get very expensive.

a. As an application grows, the number of extracted identifiers grows much faster than the number of entities present in the App, manipulating the data structure might get very expensive.

@ohansFavour Do we need to store list of all extracted identifiers? or only those which are not active?
Currently, our dependencyMap already stores all active extracted identifiers, we could just create a store for non-active references. correct?
Also, can you help me understand how we maintained this value ( data structure of variable to store extracted references )?

@Rishabh-Rathod Apparently, the way we currently extract identifiers from code has some abnormalities hence invalid identifiers are getting added, resulting in a large number of extracted identifiers. I have already started working on that here . This should reduce the number of untracked identifiers which is then stored in a "dependencyMap" type object.


There should be a busybox-like volume plugin

To support scratch images (or just images that don't have a tool you want), the should be a volume plugin that provides a shell and busybox-like functionality.  Of course, it must have selinux compiled in, not like the mainline 'busybox' image.
@vmarmol @thockin

I am not understanding what busybox has to do with volume plugins.  What am
I missing?
On Fri, May 8, 2015 at 12:45 PM, Paul Morie<EMAIL_ADDRESS>wrote:

To support scratch images (or just images that don't have a tool you
want), the should be a volume plugin that provides a shell and busybox-like
functionality. Of course, it must have selinux compiled in, not like the
mainline 'busybox' image.
@vmarmol https://github.com/vmarmol @thockin
https://github.com/thockin
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/7986.

@thockin The idea is mount a shell into a container.  I can't think of a great way to do this as a sidecar.  Open to suggestions.

oh, you want a container as a volume?  #831 ish?
On Tue, May 12, 2015 at 4:05 PM, Paul Morie<EMAIL_ADDRESS>wrote:

@thockin https://github.com/thockin The idea is mount a shell into a
container. I can't think of a great way to do this as a sidecar. Open to
suggestions.
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/7986#issuecomment-101454195
.

@thockin yes, that is what I want.  I'll close this out and document the need there.


Revert last week of changes

Test for CI

/test all

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
cfac3fcb8b83a730f99a2fbaca573e002f560d06
link
/test release-notes_istio

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
cfac3fcb8b83a730f99a2fbaca573e002f560d06
link
/test release-notes_istio

gencheck_istio
cfac3fcb8b83a730f99a2fbaca573e002f560d06
link
/test gencheck_istio

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
cfac3fcb8b83a730f99a2fbaca573e002f560d06
link
/test release-notes_istio

gencheck_istio
cfac3fcb8b83a730f99a2fbaca573e002f560d06
link
/test gencheck_istio

integ-security-multicluster-tests_istio
cfac3fcb8b83a730f99a2fbaca573e002f560d06
link
/test integ-security-multicluster-tests_istio

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

/test all

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
cfac3fcb8b83a730f99a2fbaca573e002f560d06
link
/test gencheck_istio

integ-security-multicluster-tests_istio
cfac3fcb8b83a730f99a2fbaca573e002f560d06
link
/test integ-security-multicluster-tests_istio

release-notes_istio
cfac3fcb8b83a730f99a2fbaca573e002f560d06
link
/test release-notes_istio

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-multicluster-tests_istio
cfac3fcb8b83a730f99a2fbaca573e002f560d06
link
/test integ-security-multicluster-tests_istio

release-notes_istio
cfac3fcb8b83a730f99a2fbaca573e002f560d06
link
/test release-notes_istio

integ-helm-tests_istio
cfac3fcb8b83a730f99a2fbaca573e002f560d06
link
/test integ-helm-tests_istio

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


Fix link to the repo

Included this in the release of version 1.1.0

Included this in the release of version 1.1.0


DF Manual Rolls: allow flagging a roll as forced automatic.

Is your feature request related to a problem? Please describe.
There a number of places that midi-qol/dae generate numbers via a roll request. Currently all of those will cause the user to have to enter a dice value which is not ideal, since for dae (at least) this could be on every character update.
Describe the solution you'd like
A way to flag a roll as never manual. I would think that passing something through the evaluate options would be simple enough.
Describe alternatives you've considered (Optional)
A clear and concise description of any alternative solutions or features you've considered.
Additional context (Optional)
Add any other context or screenshots about the feature request here.

So unfortunately I have no way of knowing the source of a roll. It would require MIDI to add some kind of specialize flags to all of their rolls that I could try to detect. This is not really viable, as also there's no way to "flag" rolls in Foundry. You could use Flavour text, but that gets stripped away before the rolls get to my module. Modules like Dice So Nice get away by not actually generating any sort of value. Instead it just listens for the Chat Message to arrive and figures out which dice in that card were rolled, but the result cannot be changed at that point.
Some similar tickets and discussions:
#76
#107
#111


Make additional fields translatable on the campaign template

We've run into a few instances in which we wanted to create translations for campaigns (GL EN or otherwise) but couldn't because certain fields weren't translatable. It seems like this was mainly caused by:

campaign had a scholarship
campaign had a file in the Plan It Section in English
campaign was in high season (and that content is not ready for POEditor)

So because of this, it seems like we need to make the following fields translatable:

[ ] field_scholarship_amount
[ ] field_official_rules
[ ] field_downloads
[ ] field_high_season
[ ] field_display_end_date

cc: @namimody

cc: @julielorch @mnelson338
Also @mnelson338 I can't remember, but did we need to make any/many translations unpublished because they were MCC?

Not a pressing need right now for MCC @mikefantini


Transparency regarding payment when using Paymaster | Include price

🌟 Feature Request
Adding : actual number of tokens I need to pay and the corresponding amount converted into fiat currency.
📝 Description
When using the Paymaster feature for payment, many DeFi projects allow me to choose specific payment tokens and discount information. But during the transaction confirmation step, I cannot see the actual number of tokens I need to pay and the corresponding amount converted into fiat currency.
This lack of transparency makes it difficult to understand the true cost of the transaction, especially when considering potential gas fee fluctuations and discount calculations.
🤔 Rationale
Improved User Experience (UX): Displaying the token amount and fiat conversion would provide greater clarity to users regarding transaction costs. Users would be able to see exactly how many tokens they are paying and the corresponding fiat value, allowing them to make informed decisions before confirming the transaction.
Increased Adoption of Paymaster: Transparency around costs can attract more users to the Paymaster feature. Users are more likely to adopt a payment method if they can clearly understand the associated fees and potential savings from discounts.
Empowered Users: By displaying the token amount and fiat conversion, users can compare costs associated with different payment tokens and choose the most cost-effective option for their transaction.

try using Rainbow wallet instead


"Upstream Request Timeout" error on Firefox Nightly

Browser Version
Firefox Nightly
Exension Version
1.5.4
Actual behaviour
Going to any quizlet link redirects to the cloudflare human check and then never loads the actual website. Instead displays "upstream request timeout"
These console logs are shown.

Reproduction steps

Go to any quizlet textbook link such as: https://quizlet.com/explanations/textbook-solutions/probability-and-statistics-for-engineering-and-the-sciences-9th-edition-9781305251809

Expected behaviour
Normally the page should be loading the textbook solutions.

Quizlet servers are offline at the moment. The only thing we can do is wait.


schPinStyle should accept pin labels

-> schPinStyle should accept pin labels along with pin numbers
-> Example usage:
const params = {
  schPinStyle: {
    pin1: { leftMargin: 10 },
    2: { rightMargin: 5 },
    "3": { topMargin: 15 },
    VCC: { bottomMargin: 20 },
  },
};

closes #301
/claim #301
cc: @seveibar

you need a usage test that uses schPinStyle

@seveibar I've added a new test using schPinStyle along with its snapshot. I hope now we are good to go : )

from the visual snapshot, i can't confirm that schPinStyle is working. Looking at WRITING_TESTS.md for the basic format for creating a usage test

Got your point thanks!! adding a test for that

@seveibar I've updated the PR with required test 👍🏻


Attempt to tackle timeouts during startup

This PR enables the deadlock detection for our spinlocks and unlocks the lock before throwing an exception during loading of components.

This didn't fix the timeouts :/ (https://circleci.com/gh/STEllAR-GROUP/hpx/58464)

No, migrate_component suffers from a different problem than those at startup... no idea what yet...


[Epic] GPU Codegen: Achieve 75% parity with libraries on A100 on Matmul with non-square, aligned shapes with custom codegen

@mattwalsh closing as not planned per our discussion. We have Epics already for aligned and unaligned.


Partial Unique Index (Postgres) aren't introspected as normal unique indexes

Problem
I'm trying to work with a partial unique index and prisma.  Unfortunately, once I add a where clause to an index, Prisma stops seeing this index.  I've discovered now that this happens only because someone requested that it work that way.
Well, I'm here with the opposite problem.  I need Prisma to work the way it did before - where it treated the partial index as if it was a full index.  Without the ability to do this, I can't deploy any partial unique indexes AND also have those indexes declared in my schema (and therefore make use of them), because Prisma tries to create it when it's already there.
Is there any way to make this optional so that developers can elect to have this work either way?    I understand the original request's purpose but we are not using introspection for anything other than verifying that our schema is up to date with the database and vice-versa (ie, we never use db pull and would never try to recreate our schema from the DB, because we are also using RLS and a variety of other features that Prisma would eradicate otherwise).  I don't need Prisma to support the partial unique index - I just need Prisma to stop stumbling over it.  Were it not for the previous feature request, this would already be possible, so clearly it can be done.
Suggested solution
Add a feature flag to the schema generator to allow it to treat partial indexes as full indexes, as it used to do.  This doesn't have to be the default behavior - it just needs to be an option.
Alternatives
Alternatively, add an option to ignore a portion of the schema for the purposes of introspection.  In this case, the Prisma engine would simply ignore that index and not try to create it, nor bother to look for it in the existing database.   We would take on the responsibility for ensuring that it's there.    This could be a flag like "@ignore" placed on the line with the index:
@@unique([name, categoryType, tenantId]) @ignore
Or whatever implementation you think is appropriate.
Additional context
The goal here is to make soft deletes of models with unique indexes possible.  Right now, soft delete is not possible because deleted records would prevent new records with the same keys from being created.
We flag soft deletes with a field "deletedAt", which contains the timestamp when the field was deleted.  When this field is null, the record is considered to be not deleted - once the record has a value in this field, it's considered deleted.
We first considered adding the deletedAt column to the unique index, but Prisma doesn't support this - there's no way to pass NULL as part of a unique constraint, and even in Postgres you would have to enable extra flags in order to support this.
Simply using a boolean field like "deleted" is also problematic, because then multiple deleted records with the same set of keys couldn't exist - so basically you couldn't delete a record with the same keys twice, which users will definitely run into.
Partial unique indexes are a far better, more precise approach to the problem, allowing any number of deleted records with the same keys to exist as long as there is only one not-deleted record with those keys.
If the unique index is not listed in the prisma schema, then we can't use features like findUnique, connectOrCreate, or upsert, but once the index is included there, prisma migrate will keep trying to create the field over and over again, creating problems for CI and for the developer workflow.

Partial unique indexes are not the same as unique indexes, so representing them as such was wrong and broke many workflows.
The current state is less wrong, even if it makes some other workflows - like yours - not work anymore either.
We are not going to change this back, or invest additional design and implementation effort to add an option, but will implement support for partial unique indexes some time: https://github.com/prisma/prisma/issues/6974 Sorry.
(As a temporary workaround, and if I really, really needed to automate this, I personally would implement an additional step after introspection via a script that adds the missing indexes to the Prisma schema file. Alternatively you can also try to use the old Prisma version just to introspect your database - maybe there are not other bugs that will make this impossible.)

I don't really see how this works as a temporary workaround - introspection is something that the prisma migrate command is doing internally, I am not running introspection myself - in our CI environment, any unmigrated Prisma changes result in failure (since any change in the schema should always be accompanied by a migration and the migrate command should essentially always return clean). This is just standard practice I've followed with any ORM that has migrations.  It's a pretty important, basic CI check I wouldn't want to do without:
npx prisma migrate diff --to-schema-datamodel prisma/schema.prisma --from-schema-datasource prisma/schema.prisma --exit-code

On top of that, any time any dev runs migrate dev to introspect new changes, they will also be forced to reset their database and have to recreate it, and then modify the generated migration file to remove the extra lines removing and creating the index again.  So it's very inconvenient for the developer trying to maintain the project and prone to having the developer make big mistakes (like, deleting the index).
My temporary workaround right now is not to use a partial unique index, or any unique index, at all but to do all the checks for uniqueness at the application layer instead.  So, a core feature is basically just unavailable to me.
On the issue where you made the original change, the OP got a response and a fix within a day.  At that time, you told him "making sure developers are not blocked is our highest priority besides actually implementing that functionality".   I'm just not sure why it mattered so much for that person but not for me.

I have the same issue, are there any workarounds for this?

The workaround is pretty simple - don't use unique indexes.  Write all the code to ensure uniqueness in your application layer.  This is what we're doing (referencing the same model from my earlier:
// before create
    const existingCategory = await this.prisma.category.findFirst({
      where: {
        name: validatedParams.name,
        categoryType: validatedParams.categoryType,
        deletedAt: null,
      },
      select: defaultCategoryObject,
    });

    if (existingCategory) {
      throw new KiefaError({
        code: "CONFLICT",
        message: `A ${validatedParams.categoryType.toLowerCase()} category with the name ${
          validatedParams.name
        } already exists`,
      });
    }

// before update

    if (
      existingCategory.name !== params.name ||
      existingCategory.categoryType !== params.categoryType
    ) {
      const existingCategory = await this.prisma.category.findFirst({
        where: {
          name: params.name,
          categoryType: params.categoryType,
          deletedAt: null,
          id: {
            not: id,
          },
        },
      });
      if (existingCategory) {
        throw new KiefaError({
          code: "CONFLICT",
          message: `A ${params.categoryType.toLowerCase()} category with the name ${
            params.name
          } already exists`,
        });
      }
    }

Obviously this has drawbacks.  It's very easy for any bad actor to ignore this API,
With Prisma, RLS and unique indexes are essentially incompatible.  You can use one or the other with any one table, but not both.  If you want to associate records, you have to look up the primary-key-id and then connect that - there's no connectOrCreate.


Proper export of the libraries for external use of kml

The find_library failed on my windows machine (kml library not found). Using the target is better for exporting target and fixes the issue.

Thanks for the comments !
I pushed a newer version that should address them :)

@mwoehlke-kitware @aashish24
What is the status of this ? Can it be merged ?

Seems okay to me; if @aashish24 doesn't comment/object within a few days, I think we can merge this. (Ping me again if I forget :grin:.)

@mwoehlke-kitware Ping :)

Awesome ! Thanks !


MathJax equations is not mobile responsive?

I have this equation of MathJax in my site, while viewing the size of the site in mobile view, all the other text are responsive but the equations of mathJax are not responsive.
How can I make these responsive, can you please help me? I've attached the screenshot below.
This is my equation:
$\begin{align}
\text{Power set of A, P(A)} & = \{ \emptyset, \{ x\}, \{ y\}, \{ z\}, \{ x, y\}, \{ y, z\}, \{ x, z\}, \{ x, y, z\} (again) \emptyset, \{ x\}, \{ y\}, \{ z\}, \{ x, y\}, \{ y, z\}, \{ x, z\}, \{ x, y, z\} (again) \emptyset, \{ x\}, \{ y\}, \{ z\}, \{ x, y\}, \{ y, z\}, \{ x, z\}, \{ x, y, z\}\} \\
\end{align}$

I tried to add this but still is not responsive:

Screenshot:

The align environment produces a table, and line breaking with tables is not as good as you might like.  The line breaking within a cell is done based on the full width of the expression, and doesn't take into account the material in other columns. In your case, the right-hand-side of the equal sign is in a column, and that is broken to the full width of the mobile device.  Because your initial column contains a long text string (that is not breakable), the right-hand-side of the expression (broken to the full width of the divide) may extend too far to the right by the width of the left-hand-side of the equal sign.
Of course, this example doesn't require an align environment since it is just one equality.  If you remove the alignment, the expression will wrap appropriately:
$$
\text{Power set of $A$, $P(A)$} = \{ \emptyset, \{ x\}, \{ y\}, \{ z\}, \{ x, y\}, \{ y, z\}, \{ x, z\}, \{ x, y, z\} (again) \emptyset, \{ x\}, \{ y\}, \{ z\}, \{ x, y\}, \{ y, z\}, \{ x, z\}, \{ x, y, z\} (again) \emptyset, \{ x\}, \{ y\}, \{ z\}, \{ x, y\}, \{ y, z\}, \{ x, z\}, \{ x, y, z\}\}
$$

produces

Wrapping within tables is on our list of things to improve, but is a complicated process.
As an aside, note that your initial expression
$\begin{align}
\text{Power set of A, P(A)} & = \{ \emptyset, \{ x\}, \{ y\}, \{ z\}, \{ x, y\}, \{ y, z\}, \{ x, z\}, \{ x, y, z\} (again) \emptyset, \{ x\}, \{ y\}, \{ z\}, \{ x, y\}, \{ y, z\}, \{ x, z\}, \{ x, y, z\} (again) \emptyset, \{ x\}, \{ y\}, \{ z\}, \{ x, y\}, \{ y, z\}, \{ x, z\}, \{ x, y, z\}\} \\
\end{align}$

does not need the outer dollar signs, as the \begin{align}...\end{align} will be picked up by MathJax as a math block automatically.  Also, the align environment is a display-style environment, so should not be within in-line delimiters in any case.

@dpvc I tested with same code in mine but its is still not responsive as of your screenshot. Can you please share me your JavaScript for the configuration of MathJax?
Mine is:

And, another thing, as you said $ sign should not be in \begin{align}...\end{align}??. Is my following code wrong? I don't want  those math to be in the center of the screen
BUT I want to align the text after "A ∪ B =" serially. So wraped \begin{align}...\end{align} with the dollar sign.

Here,
To Prove: A - B = A ∩ $\overline{B}$, where A and B are subset of ∪

        Taking L.H.S,

        $\begin{align}
        \overline{A \cup  B}  & = \{x : x \in A - B\} \\
 & = \{x : x \in \text{A and x} \notin B\} \\
 & = \{x : x \in \text{A and x} \in \overline{B}\} \\
 & = \{x : x \in A \cap \overline{B}\} \\
\end{align}$

This is the file I tested with
<!DOCTYPE html>

<title>Test of mobile linebreaks</title>

$$
\text{Power set of $A$, $P(A)$} = \{ \emptyset, \{ x\}, \{ y\}, \{ z\}, \{ x, y\}, \{ y, z\}, \{ x, z\}, \{ x, y, z\} (again) \emptyset, \{ x\}, \{ y\}, \{ z\}, \{ x, y\}, \{ y, z\}, \{ x, z\}, \{ x, y, z\} (again) \emptyset, \{ x\}, \{ y\}, \{ z\}, \{ x, y\}, \{ y, z\}, \{ x, z\}, \{ x, y, z\}\}
$$

It uses the latest version of MathJax (2.7.8) rather than version 2.7.4 that you are suing, but that shouldn't be an issue.  I also used a smaller configuration file (TeX-AMS_HTML) than you are.
Note that your configuration does not turn on automatic line breaks, so that may be why your test didn't work.  Your original message indicated that you used

but that doesn't appear in your configuration above.  Also, note that you turned on line breaks for HTML-CSS and SVG output formats, but not for CommonHTML, but you are loading TeX-MML-AM_CHTML, which uses CommonHTML output.  Since line breaks are not enabled for CommonHTML, that could also be the source of your problem.
You don't need the

in your configuration, since it is already included in the previous configuration just above it.

Is my following code wrong?

Yes, you should not use dollar signs around \begin{align}...\end{align}.  The align environment is a display-style environment, and it acts as math delimiters all by itself, so there is no need for any other delimiters around it.  If you want to use an alignment in an in-line setting, use $\beign{aligned}...\end{aligned}$, as the aligned environment is the corresponding in-line version.
I would recommend

Here,

To Prove: $A - B = A \cap \overline{B}$, where $A$ and $B$ are subset of $\cup$

aking L.H.S,

$\begin{aligned}
        \overline{A \cup  B}  & = \{x : x \in A - B\} \\
                              & = \{x : x \in \text{A and x} \notin B\} \\
                              & = \{x : x \in \text{A and x} \in \overline{B}\} \\
                              & = \{x : x \in A \cap \overline{B}\} \\
\end{aligned}$

BUT I want to align the text after "A ∪ B =" serially. So wraped \begin{align}...\end{align} with the dollar sign.

As I said, line breaking within tables (which is what the align and aligned environments produce) can lead to equations that exceed the width of the container, as in your case.  If you don't want that to happen, you can't use an environment that produces tables.  That means you can't reserve space on the left and will have to accept centering.  It may be that this will be fixed in the future, but for now that is the situation.
Finally, I would recommend using math mode for all mathematical expressions rather than mixing HTML, entities, and math as you do with To Prove: A - B = A ∩ $\overline{B}$, which is better as To Prove: $A - B = A\cap \overline{B}$.  This will produce more consistent results, and will allow proper support for screen readers and those with other assistive needs.

@dpvc I replaced my config code of MathJax with yours and it is now working. 👍

Note that your configuration does not turn on automatic line breaks, so that may be why your test didn't work. Your original message indicated that you used

Yes, exactly.
I replaced my config code with yours it it worked. 👍 Okay, my problem is solved and I'm going to close the issue.
Before, I want to take a advice from you that, people say, putting MathJax config's js above the  makes it load faster. Is it true? What can I do to make the equation load faster in slow internet connection?

putting MathJax config's js above the  makes it load faster. Is it true?

No, not really.  It will make the page appear sooner (with the math not yet processed), but will probably make the math show up slower.  MathJax won't do any of its typesetting until the DOMContentLoaded event occurs, and that won't happen until the  in any case.  But MathJax can do things like process the configuration, load the needed files (like the TeX-AMS_CHTML file), set up CSS style and pre-load some fonts before the .  The earlier MathJax is loaded, the sooner it can start doing those tasks.
The file I list above has a


Dispose get called twice.

Hi, I encoutered another problem with closing.
This also can be reproduced with Stylet.Samples.RedditBrowser sample, but need a bit of change:

Add IDisposable to the SubredditViewModel class.
Implement the Dispose method with any content.
Put a breakpoint inside the Dispose method.
Launch the sample application.
Open a tab, then click the Close button.
The breakpoint will be triggered twice.

Thanks for spotting this.
Technically this isn't an issue - the Microsoft guidelines are that Dispose methods may be called multiple times, and Dispose implementations should be resilient to this. That said, it's a bit ugly, so I'll try and fix.

I'll give it another day or two in case you spot something else, then push a new release.


Can torch.linalg.eig() be replaced with torch.linalg.eigh() in _get_steering_vector_evd() in MVDR

As psd_n and psd_s are Hermitian and symmetric matrices, can torch.linalg.eig() be replaced with torch.linalg.eigh() in _get_steering_vector_evd() for  faster computes the eigenvalue decomposition?

Sorry, my torchaudio's version is 0.11.0, and I found it has been replaced in 0.12.0


Update trie diff tool diff points to go down to the leaf/hash/empty node

Discussed here https://github.com/0xPolygonZero/zk_evm/pull/655#issuecomment-2393946925

Duplicate of #703


WFLY-19888 Update documentation stating that JDK 17 is now required

No Jira required.

<--- THIS SECTION IS AUTOMATICALLY GENERATED BY WILDFLY GITHUB BOT. ANY MANUAL CHANGES WILL BE LOST. --->

Wildfly issue links:

WFLY-19888

<--- END OF WILDFLY GITHUB BOT REPORT --->
More information about the wildfly-bot[bot]

Couple of these were missed in @bstansberry's original https://github.com/wildfly/wildfly/pull/18323.

Thanks @jamezp !


fix(console): console.warn should never be called

Fixes #61.
nb: I could probably improve the commit message to better address the issue, but the main point is that this plugin should never call the console.warn function [which was the only way it would log before].

Looks OK, but I'm on a mobile - I'll have a proper look when I get to the office.


Update to response data structure for kudos in RB items.

What's this PR do?
This PR uses some of the work completed in #6642 to update the response when Kudos reaction data is included in the /reportback-items response. It makes the response much more useful (and waaaay more compact) for how we actually use the Kudos data (particularly in the campaign galleries), and also takes into account the current user and indicates in the response data whether they reacted on each Reportback Item, removing a lot of the logic we currently have to do on the client-side.
Example of the new response:

How should this be reviewed?
Hit the /reporback-items endpoint and make sure the nested kudos objects in the data for each RB item looks correct.
Any background context you want to provide?
Since the response data for the reportback items is changing I will be double-checking to make sure nothing gets broken where we use Kudos now.
NOTE: I think there maybe be a better approach in general for how the above is accomplished, but by modifying the query for when RB items are retrieved. Instead of getting all the Kudos IDs for each RB item, gathering totals for each item on the fly. Not sure if this is more efficient or not... someone with better SQL chops would know better 😣
Relevant tickets
Fixes #6594

@deadlybutter @aaronschachter pinging you early on this PR for a quick heads up on the response structure changes in case it affects stuff you might be aware of...
@deadlybutter probs affects the Profile kudos stuff
@aaronschachter mobile app?

@weerd sorry I didn't touch much of the JS work around this, mostly just 🎨 so not sure what will/could break. I'm assuming you've tested giving a kudos on the user profile + gallery and it works ok?

@deadlybutter testing that out now. Updating the JS cause it definitely affects it. Just wanted to give a heads up and ask to make sure I'm aware of everything to test, etc :) I'll keep ya posted!

We're not using Kudos in iOS at all... I know Android doesn't display them, but there is for sure non-user-facing code the repo that references of them.
@lkpttn -- It'd be quite nice to run a test build of the Android app on staging or thor before we to prod to sanity check whether this could break anything in the production Android app..

👍

@weerd Can you provide example of what the current response to get a better idea of the upcoming changes?

Oh shiza. Forgot to update the docs. The example is in the screenshot in the description, but I will update the docs right now!

BUT IT'S A CHECKBOX IN THE TEMPLATE @weerd!

I know. I accidentally deleted it...

I meant what the response looks like before this was merged. Screenshot from prod:

Checklist

[ ] Documentation added for new features/changed endpoints. 😬


crypto-bigint

Hello!
I thought you might be interested in the crypto-bigint crate we've been working on as part of RustCrypto, which we are slowly beginning to use in our elliptic curve crates:

https://crates.io/crates/crypto-bigint
https://docs.rs/crypto-bigint

We're targeting Rust 1.51+ with it and aggressively trying to leverage both const generics and const fn as much as is possible on stable Rust.
We're targeting constant-time operation by default, with some secondary variable-time implementations explicitly called out as *_vartime.
I'm not sure we yet provide the functionality you need, or if the approach we're using (fixed-width, stack-allocated, no_std friendly big integers) actually maps well to this particular problem, but I thought I'd give it a mention.
If there's missing functionality you'd like to see implemented first, that's definitely something we'd like to know about:
https://github.com/RustCrypto/utils/issues/453

Cool, thanks for the heads up. I'll check it out. The first I notice is I need the ability to represent negative numbers which often affects how computations happen in groups of unknown order.

Crypto-bigint will be a nice addition for going to no-std environments and constant-time operations. Let me know how I can help.

What is the current status on this?
Can you guesstimate the effort and/or steps needed to include crypto-bigint in
unknown_order or comment on the obstacles?

There’s some methods missing from crypto-bigint I’m working to add then it will be quick

Cool! Is there anything specific I could help with?

You stub it in for now and implement what can be done

hi, wondering if this will help with some of the no_std. trying to use this unknown_order presently and getting a std error from num-traits

I'm actually close to having crypto-bigint almost integrated. Expect it within a week or two

rad. super stoked on your timing. thanks

rad. super stoked on your timing. thanks

checking in on this to see if there is anything that we can test? thanks

Just finishing some testing. I'll push what I have. Also doing some code cleanup

awesome stoked to try it out.
thanks.

Sorry I'm slow on this. Been traveling a ton and had zero time. Just a minor issue with serialization then I think I've got it

Published an RC candidate `unknown_order = "0.7.0-rc0".  The one issue I'm having is when you try to serialize to json or toml or anything human readable I get
serializer is human readable, which requires the alloc crate feature.
Have you ever seen this @tarcieri with crypto-bigint?

hi. any updates. i noticed  the repo  is now with hyperledger. are you still working on it? thanks much.

Yes I've just been really busy. Take a look and see if you can figure out why the serialize bug happens. That's the only hinderance right now.

Addressed in #9


ZFS not compatible with PR #172

This PR (https://github.com/nix-community/impermanence/pull/172) is not compatible with ZFS. It relies on the device defined in fileSystems.<name>.device, but for instance, if fileSystems."/persist".device = "zpool/persist", the corresponding zpool-persist.device won't exist in ZFS.
To make it work with ZFS, the code needs to be modified.
boot.initrd.systemd.services.create-needed-for-boot-dirs = {
  after = lib.mkForce ["zfs-import-<pool>.service"];
  wants = lib.mkForce ["zfs-import-<pool>.service"];
  unitConfig.DefaultDependencies = "no";
};

I'm not sure why unitConfig.DefaultDependencies = "no" wasn't set in the first place though.

I see. I don't use ZFS, so I'll for sure need some help with this. Would it perhaps suffice to make run after zfs-import.target? Regarding DefaultDependencies, it's not set to false because that wasn't necessary in testing.

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/impermanence-and-zfs-causes-the-system-to-hang-waiting-for-persist-device-at-boot-time/52783/2

I see. I don't use ZFS, so I'll for sure need some help with this. Would it perhaps suffice to make run after zfs-import.target? Regarding DefaultDependencies, it's not set to false because that wasn't necessary in testing.

Yes, zfs-import.target works. I tested it on my machine, and it booted successfully.

Please try #220.


TLS-PSK

Is your feature request related to a problem? Please describe.
While using gRPC in resource constrained, isolated environments TLS with pre-shared keys (TLS-PSK) is preferable over TLS with certs. Using a proxy just to solve TLS-PSK can degrade performance.
Describe the solution you'd like
Extension of the Auth API to support TLS-PSK
Describe alternatives you've considered
TLS-PSK proxy over insecure gRPC services

Hi, any updates on this feature request?

We do not have any plan to support TLS-PSK now. We are busy re-designing TLS API to support credential reloading, custom authorization callback, SPIFFE support, and so on.

Thanks for the update. Does the custom authorization callback effort enable us to plug-in a TLS-PSK backend ?

@7hacker could you describe more about TLS-PSK backend? It is hard for me to evaluate.

@jiangtaoli2016 sure, just spitballing here since I'm not clear on the custom auth callback on gRPC either.
Essentially can the custom callback defer to our implementation of TLS and we choose to use PSK's for authorization?

Please take a look at new API at https://github.com/grpc/proposal/pull/205 and see if it helps your use case.

I think the proposed grpc/proposal#205 couldn't support TLS-PSK mode.
@jiangtaoli2016 do we support TLS-PSK in current gRPC stack? If not, supporting it in gRPC would be a non-trivial amount of effort: we need to design the API, pass it down and finally to the appropriate OpenSSL lib.
For now I would suggest to use TLS with certs, or take the alternative to use the proxy.

We do not support TLS-PSK in grpc core now. We do support session resumption in gRPC core and wrapped languages.

I will close this issue, but feel free to reopen if anything else was brought up.

I will close this issue, but feel free to reopen if anything else was brought up.

So not to be too much of a necromancer, but I was wondering what kind of progress was made in the TLS redesign, and whether support for TLS-PSK is back on the table.


Bug: Search Opportunities in navbar not functioning

When I search for an opportunity in the navbar, I should be redirected to the /opportunities page. Instead, I'm directed to the home page.

I'll work on this issue.

Removed Search Opportunities in the navbar for initial build. We can add this back later. Needs to be pulled from my react fork.


feat: create initial docusaurus setup (A1ODT-897)

This PR contains the initial structure for a Docusaurus (https://docusaurus.io/) based website.
The initial navigation structure will be created for documentation about products and release guidelines.
There is also a separate navigation item for KITs.

The updated (initial) DEPENDENCIES file will follow shortly


Relicense under MIT

Hi there 👋!
Just wondering what's the status of relicensing this module under MIT. As of today, GraphQL and Relay has already been relicensed under MIT, but this module ties things together and still includes the PATENT license.
Is this on the scope to be relicensed?

cc @leebyron who has been taking point on this stuff with our legal team.

Ping!

1 month later, we can close this with 4fdadd3bbf3d5aaf66f1799be3e4eb010c115a4a!


chore(container): Enable tests

==> Building the environment
1970-01-01 00:00:00.000 [stderr   ] error: listing workers for Build: failed to list workers: Unavailable: connection error: desc = "transport: authentication handshake failed: tls: failed to verify certificate: x509: certificate has expired or is not yet valid: current time 2024-08-04T23:22:17Z is after 2024-07-25T08:55:23Z"

Non GPU builds are failing, need to look further at the backend.


u-boot: fix cross-compilation

Fix cross-compiling for u-boot when the build process requires using a tool under u-boot's ./tools directory (e.g. mkeficapsule). Since these tools can be run by the build platform during the build, duplicate the dependencies for building these packages to nativeBuildInputs.
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2424

@ofborg build pkgsCross.aarch64-multiplatform.ubootPine64
(I actually wonder if this won't always cross-compile, even without pkgsCross...)

(I actually wonder if this won't always cross-compile, even without pkgsCross...)

It does not; the device-specific builds require native or cross-compilation to be used explicitly, by design. It's just another package, so it uses the usual Nixpkgs semantics. Since it only exposes here the aarch64 platform, it'll only work when builds are for aarch64.

Closing this as I don't think this is necessarily the right approach to just try and support all things u-boot can build in nixpkgs, as there are many tools and dependencies required by board-specific builds. There are surely many other tools in u-boot that don't work yet due to missing dependencies that probably don't need to be added until there is a corresponding uboot<BoardName> derivation added.


Is newline possible in PFCanvasFillText?

I need to wrap string by calling PFCanvasFillText. Newline(/n) character is not honored in passed character string here. Is there a way to do this?

@neerajClaysol why not call it for each line?


graphml writer

I just found your great package and your graphml loader https://github.com/sparkling-graph/sparkling-graph/blob/master/loaders/src/main/scala/ml/sparkling/graph/loaders/graphml/GraphMLLoader.scala
and wonder if a similar writer exists?
for gexf I have found the following code online but that will not play nice with gephi.
def toGexf[VD, ED](g: Graph[VD, ED]): String =
"<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n" +
      "<gexf xmlns=\"http://www.gexf.net/1.2draft\" version=\"1.2\">\n" +
      "  <graph mode=\"static\" defaultedgetype=\"directed\">\n" +
      "    <nodes>\n" +
      g.vertices.map(v => "      <node id=\"" + v._1 + "\" label=\"" +
        v._2 + "\" />\n").collect.mkString +
      "    </nodes>\n" +
      "    <edges>\n" +
      g.edges.map(e => "      <edge source=\"" + e.srcId +
        "\" target=\"" + e.dstId + "\" label=\"" + e.attr +
        "\" />\n").collect.mkString +
      "    </edges>\n" +
      "  </graph>\n" +
      "</gexf>"

Hello, currently only data reading is supported. Providing that feature will be quite easy if we only want to export nodes and edges without data. I can try to support that in near future .

It would be cool if export supports (selected columns) as data as well.

Ok, i will try to do it.

I created the following function which is "nearly there", but the XML produced will not really work for gephi. Maybe you spot a problem.
def toGraphML(g: GraphFrame): String =
    s"""
       |<?xml version="1.0" encoding="UTF-8"?>
       |<graphml xmlns="http://graphml.graphdrawing.org/xmlns"
       |         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
       |         xsi:schemaLocation="http://graphml.graphdrawing.org/xmlns
       |         http://graphml.graphdrawing.org/xmlns/1.0/graphml.xsd">
       |<!-- Created by igraph -->
       |  <key id="v_name" for="node" attr.name="name" attr.type="string"/>
       |  <key id="v_fraud" for="node" attr.name="label" attr.type="int"/>
       |  <key id="e_edgeType" for="edge" attr.name="edgeType" attr.type="string"/>
       |  <graph id="G" edgedefault="directed">
       |${
      g.vertices.map {
        case Row(id, name, fraud) =>
          s"""
             |      <node id="${id}">
             |         <data key = "v_name">${name}</data>
             |         <data key = "v_fraud">${fraud}</data>
             |      </node>
           """.stripMargin
      }.collect.mkString.stripLineEnd
    }
       |${
      g.edges.map {
        case Row(src, dst, relationship) =>
          s"""
             |      <edge source="${src}" target="${dst}">
             |      <data key="e_edgeType">${relationship}</data>
             |      </edge>
           """.stripMargin
      }.collect.mkString.stripLineEnd
    }
       |  </graph>
       |</graphml>
  """.stripMargin

  val v = spark.createDataFrame(List(
    ("a", "Alice", 1),
    ("b", "Bob", 0),
    ("c", "Charlie", 0),
    ("d", "David", 0),
    ("e", "Esther", 0),
    ("f", "Fanny", 0),
    ("g", "Gabby", 0)
  )).toDF("id", "name", "fraud")
  val e = spark.createDataFrame(List(
    ("a", "b", "A"),
    ("b", "c", "B"),
    ("c", "b", "B"),
    ("f", "c", "B"),
    ("e", "f", "B"),
    ("e", "d", "A"),
    ("d", "a", "A"),
    ("a", "e", "A")
  )).toDF("src", "dst", "relationship")
  val g = GraphFrame(v, e)

Unfortunately, we can not do that this way. For large graphs, it will cause memory problems.

That is an assumption, but I think if a feature like this were to go in, it would be nice to analyze/visualize the whole graph on gephi rather than just a node.

We should not assume that export of data to graph ml is only for Gephi visualization. What is more, implementing it using data bricks XML writer, or Hadoop XML support should not be so complex.


CLion works with headers but not sources

Description of the bug:
Context
Here's my environment:

CLion 2024.1.3
Bazel Plugin 2<IP_ADDRESS>.1-api-version-241

I'm importing Ray source code: https://github.com/ray-project/ray
Here's my .bazelproject:
directories:
  # Add the directories you want added as source here
  # By default, we've added your entire workspace ('.')
  .
  -external
  -java
  -cpp
  -rllib
  -rllib_contrib
  -dashboard
  -python

derive_targets_from_directories: true

targets:
  -//:chaos_redis_store_client_test
  -//:mutable_object_provider_test
  -//:mutable_object_test
  -//src/ray/common/test:memory_monitor_test

test_sources:
  src/ray/*/test/*

Issue
After importing, all header files work:
src/ray/raylet/raylet.h:

But source files don't work:
src/ray/raylet/raylet.cc:

Can you help it? Let me know if you need additional information from me.
Which category does this issue belong to?
CLion
What's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.
Git clone Ray: https://github.com/ray-project/ray
Import bazel project and use the above view file.
Which Intellij IDE are you using? Please provide the specific version.
No response
What programming languages and tools are you using? Please provide specific versions.
No response
What Bazel plugin version are you using?
No response
Have you found anything relevant by searching the web?
No response
Any other information, logs, or outputs that you want to share?
No response

I find this duplicate to https://github.com/bazelbuild/intellij/issues/6384
Closing


Add details about nonbacktracking

Fixes #34174

Internal previews

📄 File
🔗 Preview link

docs/core/whats-new/dotnet-7.md
What's new in .NET 7

docs/standard/base-types/regular-expression-options.md
Regular expression options

Thanks for reviewing @stephentoub @BillWagner


feat: add lockactivegroup dispatcher

Describe your PR, what does it fix/add?
The lockactivewindow dispatcher takes lock, toggle or unlock as arguments. When a group is locked, no window or group can be added to it, nor can it be added to another group, but the moveintogroup and moveoutofgroup dispatches are not affected. Has no effect on non-group.
Implementation details:
the lock is implement via SGroupData.locked flag (defaults to false).
The flag is only relevant to the group head, and upon the group head's succession, the flag will be passed down to the new head. Meanwhile, the old head's flag will be set to false.
The flag is set to false when a group is dismissed.
New condition checks have been added to the dwindle and master layout to check if target group is unlocked (and if the source is also a group and unlocked) before adding windows to the target group.
Is it ready for merging, or does it need work?
tested locally.

Submitted


torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 3.50 GiB. GPU 0 has a total capacty of 23.64 GiB of which 1.42 GiB is free. Process 2722 has 22.22 GiB memory in use. Of the allocated memory 20.13 GiB is allocated by PyTorch, and 1.75 GiB is reserved by PyTorch but unallocated. If reserved but unallocated memory is large try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

Cannot allocate memory using AWQ and Mistral based model.
WARNING 12-06 19:09:20 config.py:140] awq quantization is not fully optimized yet. The speed can be slower than non-quantized models.
INFO 12-06 19:09:20 llm_engine.py:73] Initializing an LLM engine with config: model='TheBloke/dolphin-2.1-mistral-7B-AWQ', tokenizer='TheBloke/dolphin-2.1-mistral-7B-AWQ', tokenizer_mode=auto, revision=None, tokenizer_revision=None, trust_remote_code=False, dtype=torch.float16, max_seq_len=32768, download_dir=None, load_format=auto, tensor_parallel_size=1, quantization=awq, seed=0)
Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.
Traceback (most recent call last):
  File "app.py", line 140, in <module>
    engine = AsyncLLMEngine.from_engine_args(engine_args)
  File "/usr/local/lib/python3.8/dist-packages/vllm/engine/async_llm_engine.py", line 495, in from_engine_args
    engine = cls(parallel_config.worker_use_ray,
  File "/usr/local/lib/python3.8/dist-packages/vllm/engine/async_llm_engine.py", line 269, in __init__
    self.engine = self._init_engine(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/vllm/engine/async_llm_engine.py", line 314, in _init_engine
    return engine_class(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/vllm/engine/llm_engine.py", line 112, in __init__
    self._init_cache()
  File "/usr/local/lib/python3.8/dist-packages/vllm/engine/llm_engine.py", line 208, in _init_cache
    num_blocks = self._run_workers(
  File "/usr/local/lib/python3.8/dist-packages/vllm/engine/llm_engine.py", line 750, in _run_workers
    self._run_workers_in_batch(workers, method, *args, **kwargs))
  File "/usr/local/lib/python3.8/dist-packages/vllm/engine/llm_engine.py", line 724, in _run_workers_in_batch
    output = executor(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/torch/utils/_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/vllm/worker/worker.py", line 88, in profile_num_available_blocks
    self.model_runner.profile_run()
  File "/usr/local/lib/python3.8/dist-packages/torch/utils/_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/vllm/worker/model_runner.py", line 321, in profile_run
    self.execute_model(seqs, kv_caches)
  File "/usr/local/lib/python3.8/dist-packages/torch/utils/_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/vllm/worker/model_runner.py", line 279, in execute_model
    hidden_states = self.model(
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/vllm/model_executor/models/mistral.py", line 290, in forward
    hidden_states = self.model(input_ids, positions, kv_caches,
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/vllm/model_executor/models/mistral.py", line 256, in forward
    hidden_states, residual = layer(
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/vllm/model_executor/models/mistral.py", line 217, in forward
    hidden_states = self.mlp(hidden_states)
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/vllm/model_executor/models/mistral.py", line 76, in forward
    gate_up, _ = self.gate_up_proj(x)
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/vllm/model_executor/layers/linear.py", line 203, in forward
    output_parallel = self.linear_method.apply_weights(
  File "/usr/local/lib/python3.8/dist-packages/vllm/model_executor/layers/quantization/awq.py", line 154, in apply_weights
    out = ops.awq_gemm(reshaped_x, qweight, scales, qzeros, pack_factor)
torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 3.50 GiB. GPU 0 has a total capacty of 23.64 GiB of which 1.42 GiB is free. Process 2722 has 22.22 GiB memory in use. Of the allocated memory 20.13 GiB is allocated by PyTorch, and 1.75 GiB is reserved by PyTorch but unallocated. If reserved but unallocated memory is large try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

Full logs
GPU is working on host.
❯ docker ps
CONTAINER ID   IMAGE     COMMAND   CREATED   STATUS    PORTS     NAMES
❯ nvidia-smi --query-gpu=memory.total,memory.used --format=csv,nounits,noheader
24576, 6
24576, 555
❯ nvidia-container-cli --load-kmods info
NVRM version:   545.29.06
CUDA version:   12.3

Device Index:   0
Device Minor:   0
Model:          NVIDIA TITAN RTX
Brand:          TITAN
GPU UUID:       GPU-8ec451e5-d67e-768b-0481-5e97728caa95
Bus Location:   00000000:08:00.0
Architecture:   7.5

Device Index:   1
Device Minor:   1
Model:          NVIDIA TITAN RTX
Brand:          TITAN
GPU UUID:       GPU-48931c47-c25e-39d0-468b-08e20b0b7c7e
Bus Location:   00000000:43:00.0
Architecture:   7.5

Also works in container if loading the model w/ full precision.
WARNING 12-06 23:24:55 config.py:406] Casting torch.bfloat16 to torch.float16.
INFO 12-06 23:24:55 llm_engine.py:73] Initializing an LLM engine with config: model='ehartford/dolphin-2.2.1-mistral-7b', tokenizer='ehartford/dolphin-2.2.1-mistral-7b', tokenizer_mode=auto, revision=None, tokenizer_revision=None, trust_remote_code=False, dtype=torch.float16, max_seq_len=32768, download_dir=None, load_format=auto, tensor_parallel_size=1, quantization=None, seed=0)
Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.
Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.
INFO 12-06 23:25:16 llm_engine.py:222] # GPU blocks: 308, # CPU blocks: 2048
Loading embedding model...
INFO:     Started server process [362]
INFO:     Waiting for application startup.
INFO:     Application startup complete.
INFO:     Uvicorn running on http://<IP_ADDRESS>:5085 (Press CTRL+C to quit)

Fixed by setting max_seq_len=8192, thanks to: https://github.com/vllm-project/vllm/issues/188#issuecomment-1785991729

After setting --max-model-len 8192 the OOM went away. I also use an AWQ quant with --quantization awq parameter. Works amazing!


[v1.6.0.rc-8] embedders key is present even when the vectorSearch is not enabled.

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Use the last prototype v1.6.0.rc-8
Do not enable the vectorSearch experimental feature
Run a simple GET /indexes/my_index/settings
See the "embedders": {} key

Expected behavior
To not see the embedders key because this is part of a experimental feature.
Screenshots
If applicable, add screenshots to help explain your problem.
Meilisearch version:
v1.6.0.rc-8
Additional context
@curquiza is aware of the issue I let her know last Thursday.

Fixed by #4318


Day114

Resolves #237 .

Pull Request Test Coverage Report for Build 457

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 454:
0.0%

Covered Lines:
5

Relevant Lines:
5

💛  - Coveralls


US <IP_ADDRESS>

Code for viewing habit and details:
As a doer, I want to view a given habit and its details.
There will be a section of the app (a button that takes you to a new activity, or a sidebar)that contains a list of all the habits when the user clicks on a habit they can view and edit the details of the habit in a dialog fragment
Be able to see each piece of the habit the user has entered

Code Done


Upgrade to Elasticsearch 8

Chatting to Paul yesterday, this is the list we came up with of things that talk to Elasticsearch and might need upgrading:

Amusingly, the above diagram is missing a fairly crucial service: the API! 😬

oops


test:  metro-compat resolution tests

Summary
This PR introduces metro-compat package which houses tests suites designed for checking compatibility with parts of metro.
Right now, tests include resolver tests directly imported from metro-resolver from here
Test environment is modified to accommodate for how enhanced-resolve works. Basic principle is this: use unchanged tests and provide compatible resolve function for tests. While in metro-resolver everything happens via mocked methods for checking whether file exists, our tests create a fresh in-memory filesystem for each test to test resolution against, which yields more ensuring results.
Some tests suite are skipped:

assets-test.js - because enhanced-resolve, like node resolution, returns only a single entry, it's impossible to support these test cases. We handle this differently using AssetResolver which provides you with first matching entry for development, but all scales are exposed to the runtime
index-test.js - this suite contains some basic tests & failure tests which we could include but most of it contains tests for unsupported features like:

resolveRequest - this is essentially a custom resolver or a plugin, it should be documented how to achieve similar functionality in Re.Pack
redirectModulePath - this is equal to alias field in enhanced-resolve, if end user wants to do this dynamically, then custom resolver/plugin should be used for that - also needs to documented
disableHierarchicalLookup - can be achieved in enhanced-resolve by using modules - also needs to be documented

Some test cases are skipped (all come from package-exports-test.js):

[nonstrict] tests are skipped because we don't support fallback in form of legacy file resolution
asset tests with package exports enabled are skipped for the same reason as above (regarding the assets)
rest of the test skipped are due to inline snapshot not matching up with what enhanced-resolve returns - this could be potentially fixed by trapping toThrowErrorMatchingInlineSnapshot calls and checking for existence of errors only

Tests will be excluded from CI for now until everything is aligned properly and some issues are brought up with metro team. Alignment of getResolveOptions will be done in a separate PR. Documentation will follow in a separate PR as well.
Side-note: this PR also unveiled few issues with package exports implementation in metro-resolver which will be investigated more throughly and perhaps become more aligned in the future.
Side-note 2: the way we skip tests without modyfing the test files is rather hacky, we trap jest.describe, jest.it & jest.test and we check if the test name matches, TBD whether there is a better way to do this.
Checklist

[x] metro-resolver test suite
[x] setup mocks
[x] setup skip list
[x] exclude tests from turborepo so they are not run with the rest of the tests
[x] cleanup
[ ] README.md in metro-compat

Issue raised with metro: https://github.com/facebook/metro/issues/1236

@RafikiTiki don't mind the enabled tests failing for now, it's expected without reworking implementation


Unable to close Toast component

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
Unable to close Toast Notification by clicking close button
onClose method is not triggered when  close button clicked

Reduced test cases
Create a Toast with onClose method using modus react bootstrap
Click close button ,  onClose is not triggered
What operating system(s) are you seeing the problem on?
Windows, Linux
What browser(s) are you seeing the problem on?
Chrome, Microsoft Edge
What version of Modus React Bootstrap are you using?
1.1.1
Priority
High

hi @VinoSakthi checking the issue we found that the onClose is supposed to be triggered when using the default close button inside ToastHeader tag so we are updating only the API description and doesn't require any other fix. If you are using a custom close button it is recommended to handle the onclose event on the button directly.


OpenTherm device_info property missing

Home Assistant release with the issue: 0.101.0
Last working Home Assistant release (if known): n/a (integration since 0.101)
Operating environment (Hass.io/Docker/Windows/etc.): Linux venv
Integration: https://www.home-assistant.io/integrations/opentherm_gw/
Description of problem: since this version OTGW moved to integrations with config flow. All entities made by the integration (except climate domain) are missing device_info property, thus are missing from entities register and cannot be turned off in example.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant): n/a
Traceback (if applicable): no traceback
Additional information: n/a

Thanks for the report, I will have time to take a look later this weekend.

Basic device info support is completed, after I sort out support for re-enabling disabled entities I will file a PR.

The config-flow  introduced in 101.3 for this component is also buggy. The climate entity is not available anymore after updating to this version. Also removing it from config-file or removing en re-adding doesn't work.  I already saw 2 person with the same problem.

@jodur if you mean #28363 then a fix is in the pipeline. If not, then please file a separate report with as much information as possible.


Remove unused modules and variables

Added Tyson to review - not sure if some of these commented out lines are desirable to retain for posterity / future revival


App stuck on connecting to car after phone reboot

Hi,
I had this several times now.
The battery of my phone died during the day. The car wasnt connected at that point.
But after charging my phone and eventually connecting it to the car, the app does not connect anymore.
See screenshot. The App tells me it cant find the Mini App anymore, but Mini App itself is still installed, connected and working just fine.
The only solution i found (apart from reinstalling) is to go into the android settings for this app and wipe cache and data.
Then tap through the initial app screen again.
This does not happen, when I reboot my phone (Samsung S22 Ultra Android 13) regulary. Only when the battery dies.
Maybe the phone "shuts down" differently when the battery is flat ?
App Version 1.4.2-39-4b386f3 04.05.2024 04:38:23

Same problem for me.
I have a S20 and when it restarts the app does not connect to car.
Reinstall app makes it work again.

I believe that red color means that it can't locate the security service within MyBMW, there should be some text explaining why it can't connect.

There wasnt.
It just said the mybmw / mini app couldnt be found

That is the text that indicates the problem it encountered.
Unfortunately I don't know why it wouldn't be able to find the MyBMW/Mini app after restarting from a dead battery. My phone often experiences sudden reboots, which should be similar to your phone's sudden power-off, and I haven't experienced an issue when opening the app after reboots.
Reinstalling the app shouldn't matter, AAIdrive doesn't cache what BMW apps it sees. Does Force Closing the app resolve the issue too?

Last working day was 15.8.2024, after that the app 4.2-83 wanted not to work.
Reinstalled and reinstalled but nothing changed.
App always say "due to closing the app, now it has a problem with background working rights". I changed the rights from "optimized" to "not restricted" but nothing changed.
I installed the Version 4.2-39 and the app worked!
And after updating the app over Google Playstore now also 4.2-83 is working!
It seems, that something went wrong during the first app update and the error could not fixed through a reinstall. The installing of the older version cleared the error.

The "background permissions" error will clear the next time it connects to the car, and it has no influence on the connection process.
I'm glad it works again, but unfortunately there's nothing for me to help with. AAIdrive tries to connect to MyBMW the same way in every attempt, and I'm not sure why your phone rebooting suddenly would cause this to fail. My own phone's abrupt resets do not trigger it.


Terminal: prompt can scroll below window pane

User loses site of the prompt. It is still active, so 'clear' will make the cursor visible, for now it's an acceptable workaround.

Closed by #31.


Feature Branch CI/CD Workflow - Proposal

For one person, per ticket:

Create a local feature branch
Do development
Push local feature branch -> Create a remote feature branch

This creates a feature branch build on Vercel

Create a PR

Create a well written description of changes, a link to the issue, and things to test (based on the PR template)
Ask for review using the feature branch build (isolated to this particular feature)
Devs review the code changes, and functionality if no QA
Design reviews (if necessary)
QA can review if this is a big change

Once approved, merge PR directly into a main pipeline - GitHub Environments (Require Approvers)

Merges into the dev environment
Then merges into the prod environment, after manual approval

Feedback is welcome from anyone, especially @sashamaryl and @mvictor55

This describes our current process (and it's good this should be what we put in the process docs discussed tuesday 9/12).
The conversation we had a few weeks ago if I remember it right was about having a shared (rather than local) feature branch, which would include multiple related tickets, maybe even a whole epic, with multiple people contributing.
This addresses features that will take a long time or will span multiple parts of the project, e.g. front end and back end, where it's not always appropriate to merge one without the other, but we need to share that code to develop & test that feature.
The shared feature branch would be merged into dev (main) as a whole when the feature was ready, keeping unfinished untested code off of dev (main). The goal is to uncouple (feature) development tracks from each other, protect prod, keep the deployment pipeline clean & clear.

I would definitely be pro shared feature branches in this context! That should go in hand with encouraging commiting early and often, which is easy not to do when you're the czar of your own secret branch.

Honestly given our limited resources and time constraints I think simpler is better.
It is okay to have components that are not complete as long as they are not being displayed. This is something already being done (see the pr I just pushed #1307 - the dropdown button I revamped was already existing but was unfinished and not displayed).
@timblais Adding Tim here because he was the original proposal for a feature branch.

For larger features with multiple contributors, I do still think it is best to separate that feature out from the main (dev) branch. In my opinion, this will help to keep main clean, prevent confusion for others looking at the code, and make joint development of the feature easier since you don't have to jump through hoops of turning things on and off/keeping track of the changes and how to piece them together. I also think it will make it much easier to peer review when the feature is ultimately moved into main.

So generally speaking, I love the idea of feature branches. They work well in large companies that manage large teams with tight deadlines, well-coordinated timelines, and robust methods to ensure equitable management of collaborators. They also have the resources to handle the extra costs that come with adding in another layer to the workflow (with every layer comes an additional cost- time & labor- needed to deal with bugs and management).
In this space, our contributors have varying commitments and timelines which means we need to stress flexibility above all else. If you look in the MAPLE codebase timeline, you'll see sketches of components stemming all the way from a year ago. What will end up happening is that a feature branch will be left open for potentially 9+ months which is not advisable for both workflow and contributor wellbeing.
Main is kept clean and high-quality through reviews and linting. Also, looking at the exceptionally high quality code that is being contributed to MAPLE, fears of "quality" are not applicable at this moment in time.
What I'm thinking is the priority for right now is EASE and FLEXIBILITY for contributors who are the primary producers of the MAPLE product. This means merging contributions within reasonable timelines and having reasonable expectations for quality in ways that are consistent with OPEN-SOURCE and not necessarily well-oiled corporate giants. We should see the current MAPLE website as version 1 with a focus on "just getting it done" (while meeting reasonable standards of quality) over perfection.
It also helps to think concretely. What features are you referring to? The only feature I'm aware of that is being worked on right now is the global search bar. I don't think this workflow is necessary to manage the 2 people involved in that project.
Moving forward, if tickets are decomposed in the manner I expressed above (functionality and styling) this will eliminate confusion. After a component is merged, styling should be done afterwards and merged.
In theory, feature branches are great. Practically, given MAPLE's unique constraints, not so much.

If the feature branch was experimented with first with willing participants that would be interesting.
My resistance is against making this a required workflow before it has been tried and tested atm.


Create a UID Module

Create a UID Module
Background
We need a module which exposes a single function which can generate a string which is extremely unlikely to be generated again - a unique identifier.
Purpose
This unique identifier will be used to generate ids used in the interaction ADT.
Expected Outcomes

[ ] A new module @onflow/uid is created.
[ ] The new module exposes a single function to generate a UID.
[ ] A UID which is generated is extremely unlikely to be generated again - unique.

handled in #238


[NoQA] Bump date-fns-tz to 3.2.0 and remove patch.

Details
Updates date-fns-tz to 3.2.0 and removes related patch.
Fixed Issues
$ https://github.com/Expensify/App/issues/48448
PROPOSAL: https://github.com/Expensify/App/issues/48448
Tests

[x] Verify that no errors appear in the JS console

Offline tests
n/a
QA Steps
n/a

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[ x I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

Hi! So everything here looks good except for the 14k line change in .github/actions/javascript/createOrUpdateStagingDeploy/index.js - is that a compiled file? Basically how should I look at that one, or do I need to? Thanks!

@dangrous yea, it's a compiled file and I just executed gh-actions-build script. I think there's an action that validates it, so if it pass it should be good to go.

There was also one conflict in DebugUtils.ts file, should be resolved now! 👍

Thanks for the explanation!

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.


Implement attribute/property/event part SSR & hydration

Description
Currently only NodeParts are serialized/hydrated. This task is to add a serialization scheme for attribute, property, and event parts and the respective client-side hydration.
Acceptance Criteria

Tests demonstrating client-side hydration of serialized templates that include attributes, properties, and events are checked in and passing.

Closing in favor of smaller issues in lit-ssr


Support for loose search

First attempt at implementing looser search as proposed in https://github.com/pangeo-data/pangeo/issues/522, #15.
prototype in action: https://gist.github.com/andersy005/9d6f8b861a6f72b9a666a9e92c0f45cb

Just in case it's useful, the MongoDB-backed catalog mentioned in my comment above is now documented. Our documentation on search might be slightly helpful.

Thank you, @danielballan. I've been making some progress on dynamic search and the documentation will definitely be useful.


Unknown types under non-executed macro

#define MYMACRO 1

#ifndef MYMACRO
void use_some_type(UnknownType hello);
#endif

@kernel void kern() {

    for (int i = 0; i < 10; i++; @outer) {
        for (int j = 0; j < 10; j++; @inner) {

        }
    }
}

Should transpile fine and just skip UnknownType, but fails

fixed in https://github.com/libocca/occa-transpiler/pull/131


Datadog: customizable additional tags for metrics

Is your feature request related to a problem? Please describe.
We have deployments in the cloud (autoscaling or failover models, CSTv1)
We need some additional data (in form of tags) to be added to the metrics. That will allow us to build relations between metrics and/or add environment information.
Describe the solution you'd like
We need to have the possibility to add some custom Tags which would be added to all metrics.
Example of declaration:

      "My_Datadog_Metrics_Consumer": {
        "class": "Telemetry_Consumer",
        "type": "DataDog",
        "apiKey": "<redacted>",
        "dataType": "metrics",
        "customTags": [
          {
            "deploymentName": "f5aws01",
            "environment": "production",
            "instanceId": "i-0cd111111b224831"
          }
        ],
        "enable": true,
        "trace": false,
        "allowSelfSignedCert": false
      }

Have you tried using the Data Modification feature to add tags at the system poller level? https://clouddocs.f5.com/products/extensions/f5-telemetry-streaming/latest/data-modification.html#tag-property

@dstokesf5
I've tried this declaration, but without success => I cannot see the tags added in the consumer trace output:
    "My_Metrics_Namespace": {
        "class": "Telemetry_Namespace",
        "My_Poller": {
            "class": "Telemetry_System_Poller",
            "interval": 60,
            "actions": [
             {
                 "enable": true,
                 "setTag": {
                     "deploymentInfo": {
                         "tag1": "tag1value",
                         "tag2": "Another tag"
                     }
                 }
             }
             ]
        },
[...]

What am I missing? As far as I understand, excludeData, IncludeData and IfAllMatched are optional (I would not want them anyway, as it should be applied to all keys in the SystemPoller)

After further discussion with the developers I have learned that this method is not appropriate for adding global metric tags. I have added  AUTOTOOL-2795 to our internal product backlog to support specifying global tags on the DataDog consumer.

I guess it had been fixed in v1.25.0?

This has been resolved in the 1.25.0 release.


orchestra/remote: throw exception on reconnect failed

In run method if a reconnect required there is a remote.reconnect
method called which is supposed to return True or False.
On failure it is the best to raise an exception early,
to make it clear what was the original error instead of
waiting until somewhere later teuthology fails with
some undeclared variables.
Signed-off-by: Kyr Shatskyy<EMAIL_ADDRESS>

@susebot run deploy

one fail and one pass, the fail is none for missing route:
2022-04-25T21:03:42.837 DEBUG:teuthology.orchestra.run.target-pr-1698-003:> route
2022-04-25T21:03:42.865 DEBUG:teuthology.orchestra.run:got remote process result: 127
2022-04-25T21:03:42.867 INFO:teuthology.orchestra.run.target-pr-1698-003.stderr:bash: route: command not found

which is unrelated issue.


ModuleNotFoundError: No module named 'jupyter_tabnine'

Hello,
Thank you for your effort.
`(py37) E:\coding2>pip3 install jupyter-tabnine --user
Requirement already satisfied: jupyter-tabnine in c:\users\neuez\appdata\roaming\python\python37\site-packages (1.1.0)
Requirement already satisfied: notebook>=4.2 in c:\users\neuez\appdata\roaming\python\python37\site-packages (from jupyter-tabnine) (6.2.0)
Requirement already satisfied: ipython in c:\coding\anaconda3\envs\py37\lib\site-packages (from jupyter-tabnine) (7.19.0)
Requirement already satisfied: jupyter-core in c:\coding\anaconda3\envs\py37\lib\site-packages (from jupyter-tabnine) (4.7.0)
Requirement already satisfied: nbconvert in c:\users\neuez\appdata\roaming\python\python37\site-packages (from jupyter-tabnine) (6.0.7)
Requirement already satisfied: traitlets>=4.2.1 in c:\coding\anaconda3\envs\py37\lib\site-packages (from notebook>=4.2->jupyter-tabnine) (5.0.5)
Requirement already satisfied: ipykernel in c:\coding\anaconda3\envs\py37\lib\site-packages (from notebook>=4.2->jupyter-tabnine) (5.4.3)
Requirement already satisfied: argon2-cffi in c:\users\neuez\appdata\roaming\python\python37\site-packages (from notebook>=4.2->jupyter-tabnine) (20.1.0)
Requirement already satisfied: nbformat in c:\users\neuez\appdata\roaming\python\python37\site-packages (from notebook>=4.2->jupyter-tabnine) (5.1.2)
Requirement already satisfied: jupyter-client>=5.3.4 in c:\coding\anaconda3\envs\py37\lib\site-packages (from notebook>=4.2->jupyter-tabnine) (6.1.11)
Requirement already satisfied: prometheus-client in c:\users\neuez\appdata\roaming\python\python37\site-packages (from notebook>=4.2->jupyter-tabnine) (0.9.0)
Requirement already satisfied: jinja2 in c:\users\neuez\appdata\roaming\python\python37\site-packages (from notebook>=4.2->jupyter-tabnine) (2.11.2)
Requirement already satisfied: terminado>=0.8.3 in c:\users\neuez\appdata\roaming\python\python37\site-packages (from notebook>=4.2->jupyter-tabnine) (0.9.2)
Requirement already satisfied: pyzmq>=17 in c:\coding\anaconda3\envs\py37\lib\site-packages (from notebook>=4.2->jupyter-tabnine) (22.0.2)
Requirement already satisfied: tornado>=6.1 in c:\coding\anaconda3\envs\py37\lib\site-packages (from notebook>=4.2->jupyter-tabnine) (6.1)
Requirement already satisfied: Send2Trash>=1.5.0 in c:\users\neuez\appdata\roaming\python\python37\site-packages (from notebook>=4.2->jupyter-tabnine) (1.5.0)
Requirement already satisfied: ipython-genutils in c:\coding\anaconda3\envs\py37\lib\site-packages (from notebook>=4.2->jupyter-tabnine) (0.2.0)
Requirement already satisfied: python-dateutil>=2.1 in c:\coding\anaconda3\envs\py37\lib\site-packages (from jupyter-client>=5.3.4->notebook>=4.2->jupyter-tabnine) (2.8.1)
Requirement already satisfied: pywin32>=1.0 in c:\coding\anaconda3\envs\py37\lib\site-packages (from jupyter-core->jupyter-tabnine) (300)
Requirement already satisfied: six>=1.5 in c:\coding\anaconda3\envs\py37\lib\site-packages (from python-dateutil>=2.1->jupyter-client>=5.3.4->notebook>=4.2->jupyter-tabnine) (1.15.0)
Requirement already satisfied: pywinpty>=0.5 in c:\users\neuez\appdata\roaming\python\python37\site-packages (from terminado>=0.8.3->notebook>=4.2->jupyter-tabnine) (0.5.7)
Requirement already satisfied: cffi>=1.0.0 in c:\users\neuez\appdata\roaming\python\python37\site-packages (from argon2-cffi->notebook>=4.2->jupyter-tabnine) (1.14.4)
Requirement already satisfied: pycparser in c:\users\neuez\appdata\roaming\python\python37\site-packages (from cffi>=1.0.0->argon2-cffi->notebook>=4.2->jupyter-tabnine) (2.20)
Requirement already satisfied: decorator in c:\coding\anaconda3\envs\py37\lib\site-packages (from ipython->jupyter-tabnine) (4.4.2)
Requirement already satisfied: pygments in c:\coding\anaconda3\envs\py37\lib\site-packages (from ipython->jupyter-tabnine) (2.7.4)
Requirement already satisfied: colorama in c:\coding\anaconda3\envs\py37\lib\site-packages (from ipython->jupyter-tabnine) (0.4.4)
Requirement already satisfied: setuptools>=18.5 in c:\coding\anaconda3\envs\py37\lib\site-packages (from ipython->jupyter-tabnine) (52.0.0.post20210125)
Requirement already satisfied: prompt-toolkit!=3.0.0,!=3.0.1,<3.1.0,>=2.0.0 in c:\coding\anaconda3\envs\py37\lib\site-packages (from ipython->jupyter-tabnine) (3.0.14)
Requirement already satisfied: jedi>=0.10 in c:\coding\anaconda3\envs\py37\lib\site-packages (from ipython->jupyter-tabnine) (0.18.0)
Requirement already satisfied: backcall in c:\coding\anaconda3\envs\py37\lib\site-packages (from ipython->jupyter-tabnine) (0.2.0)
Requirement already satisfied: pickleshare in c:\coding\anaconda3\envs\py37\lib\site-packages (from ipython->jupyter-tabnine) (0.7.5)
Requirement already satisfied: parso<0.9.0,>=0.8.0 in c:\coding\anaconda3\envs\py37\lib\site-packages (from jedi>=0.10->ipython->jupyter-tabnine) (0.8.1)
Requirement already satisfied: wcwidth in c:\coding\anaconda3\envs\py37\lib\site-packages (from prompt-toolkit!=3.0.0,!=3.0.1,<3.1.0,>=2.0.0->ipython->jupyter-tabnine) (0.2.5)
Requirement already satisfied: MarkupSafe>=0.23 in c:\users\neuez\appdata\roaming\python\python37\site-packages (from jinja2->notebook>=4.2->jupyter-tabnine) (1.1.1)
Requirement already satisfied: bleach in c:\users\neuez\appdata\roaming\python\python37\site-packages (from nbconvert->jupyter-tabnine) (3.2.3)
Requirement already satisfied: mistune<2,>=0.8.1 in c:\users\neuez\appdata\roaming\python\python37\site-packages (from nbconvert->jupyter-tabnine) (0.8.4)
Requirement already satisfied: entrypoints>=0.2.2 in c:\users\neuez\appdata\roaming\python\python37\site-packages (from nbconvert->jupyter-tabnine) (0.3)
Requirement already satisfied: defusedxml in c:\users\neuez\appdata\roaming\python\python37\site-packages (from nbconvert->jupyter-tabnine) (0.6.0)
Requirement already satisfied: testpath in c:\users\neuez\appdata\roaming\python\python37\site-packages (from nbconvert->jupyter-tabnine) (0.4.4)
Requirement already satisfied: jupyterlab-pygments in c:\users\neuez\appdata\roaming\python\python37\site-packages (from nbconvert->jupyter-tabnine) (0.1.2)
Requirement already satisfied: nbclient<0.6.0,>=0.5.0 in c:\users\neuez\appdata\roaming\python\python37\site-packages (from nbconvert->jupyter-tabnine) (0.5.1)
Requirement already satisfied: pandocfilters>=1.4.1 in c:\users\neuez\appdata\roaming\python\python37\site-packages (from nbconvert->jupyter-tabnine) (1.4.3)
Requirement already satisfied: async-generator in c:\users\neuez\appdata\roaming\python\python37\site-packages (from nbclient<0.6.0,>=0.5.0->nbconvert->jupyter-tabnine) (1.10)
Requirement already satisfied: nest-asyncio in c:\users\neuez\appdata\roaming\python\python37\site-packages (from nbclient<0.6.0,>=0.5.0->nbconvert->jupyter-tabnine) (1.5.1)
Requirement already satisfied: jsonschema!=2.5.0,>=2.4 in c:\users\neuez\appdata\roaming\python\python37\site-packages (from nbformat->notebook>=4.2->jupyter-tabnine) (3.2.0)
Requirement already satisfied: importlib-metadata in c:\coding\anaconda3\envs\py37\lib\site-packages (from jsonschema!=2.5.0,>=2.4->nbformat->notebook>=4.2->jupyter-tabnine) (3.4.0)
Requirement already satisfied: pyrsistent>=0.14.0 in c:\users\neuez\appdata\roaming\python\python37\site-packages (from jsonschema!=2.5.0,>=2.4->nbformat->notebook>=4.2->jupyter-tabnine) (0.17.3)
Requirement already satisfied: attrs>=17.4.0 in c:\users\neuez\appdata\roaming\python\python37\site-packages (from jsonschema!=2.5.0,>=2.4->nbformat->notebook>=4.2->jupyter-tabnine) (20.3.0)
Requirement already satisfied: packaging in c:\users\neuez\appdata\roaming\python\python37\site-packages (from bleach->nbconvert->jupyter-tabnine) (20.9)
Requirement already satisfied: webencodings in c:\users\neuez\appdata\roaming\python\python37\site-packages (from bleach->nbconvert->jupyter-tabnine) (0.5.1)
Requirement already satisfied: zipp>=0.5 in c:\coding\anaconda3\envs\py37\lib\site-packages (from importlib-metadata->jsonschema!=2.5.0,>=2.4->nbformat->notebook>=4.2->jupyter-tabnine) (3.4.0)
Requirement already satisfied: typing-extensions>=3.6.4 in c:\coding\anaconda3\envs\py37\lib\site-packages (from importlib-metadata->jsonschema!=2.5.0,>=2.4->nbformat->notebook>=4.2->jupyter-tabnine) (<IP_ADDRESS>)
Requirement already satisfied: pyparsing>=2.0.2 in c:\users\neuez\appdata\roaming\python\python37\site-packages (from packaging->bleach->nbconvert->jupyter-tabnine) (2.4.7)
(py37) E:\coding2>jupyter nbextension install --py jupyter_tabnine --user
Traceback (most recent call last):
File "c:\users\neuez\appdata\local\programs\python\python39\lib\runpy.py", line 197, in _run_module_as_main
return _run_code(code, main_globals, None,
File "c:\users\neuez\appdata\local\programs\python\python39\lib\runpy.py", line 87, in run_code
exec(code, run_globals)
File "C:\Users\neuez\AppData\Local\Programs\Python\Python39\Scripts\jupyter-nbextension.EXE_main.py", line 7, in
File "c:\users\neuez\appdata\local\programs\python\python39\lib\site-packages\jupyter_core\application.py", line 254, in launch_instance
return super(JupyterApp, cls).launch_instance(argv=argv, **kwargs)
File "c:\users\neuez\appdata\local\programs\python\python39\lib\site-packages\traitlets\config\application.py", line 845, in launch_instance
app.start()
File "c:\users\neuez\appdata\local\programs\python\python39\lib\site-packages\notebook\nbextensions.py", line 980, in start
super().start()
File "c:\users\neuez\appdata\local\programs\python\python39\lib\site-packages\jupyter_core\application.py", line 243, in start
self.subapp.start()
File "c:\users\neuez\appdata\local\programs\python\python39\lib\site-packages\notebook\nbextensions.py", line 708, in start
self.install_extensions()
File "c:\users\neuez\appdata\local\programs\python\python39\lib\site-packages\notebook\nbextensions.py", line 679, in install_extensions
full_dests = install(self.extra_args[0],
File "c:\users\neuez\appdata\local\programs\python\python39\lib\site-packages\notebook\nbextensions.py", line 203, in install_nbextension_python
m, nbexts = _get_nbextension_metadata(module)
File "c:\users\neuez\appdata\local\programs\python\python39\lib\site-packages\notebook\nbextensions.py", line 1114, in _get_nbextension_metadata
m = import_item(module)
File "c:\users\neuez\appdata\local\programs\python\python39\lib\site-packages\traitlets\utils\importstring.py", line 38, in import_item
return import(parts[0])
ModuleNotFoundError: No module named 'jupyter_tabnine'
(py37) E:\coding2>`
Can you tell me how to resolve this issue?
Thank you.

Duplicate of https://github.com/codota/jupyter-tabnine/issues/21

If you use a virtual python env like conda, you should activate your python env that you often used first.


Segfault if not run in a term

While in an emacs eshell, I did a git clone, make, then make install, then:
`$ pms
Practical Music Search 1ef37a8
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x10 pc=0x95f849]
goroutine 1 [running]:
github.com/ambientsound/pms/widgets.(*UI).Start(...)
/home/james/src/practical_music_search/widgets/ui.go:101
github.com/ambientsound/pms/pms.(*PMS).setupUI(0xc0001402d0)
/home/james/src/practical_music_search/pms/setup.go:74 +0xa9
github.com/ambientsound/pms/pms.New(0xacc29f)
/home/james/src/practical_music_search/pms/setup.go:41 +0x60d
main.main()
/home/james/src/practical_music_search/main.go:94 +0x1ef`
I didn't try to debug beyond a very simple fix:  I ran it in a proper terminal (xterm and also emacs's term).  It was silly not to do that from the beginning, but I didn't realize this needed a tty since I didn't actually know anything about it.
Suggested remediation:  return a proper, informative error message instead of seg faulting.  Add a line to documentation.
Other than that, this is a minor issue.

Hi @jvasile,
Latest master should fix this error. Unfortunately I could not reproduce this behavior, would you mind taking it for a spin and see how it behaves?