Eliminate cv2 dependency from GRAD CAM example

The extra dependency for cv2 library have been eliminated by use of plt.imshow with alpha set according to required intensity instead of cv2.applyColorMap without affecting any functionality in the Grad CAM example as present.
This PR is intended at reducing build times for future versions by eliminating an extra package where an automatic CI/CD pipeline would be used.

I have updated the source code to use IPython display instead of plt (plus, applying the jet cmap using numpy and keras only eliminating cv2) so as to facilitate better visuals. Request to your review the changes on this PR.


Store.Chat is not available

It seems WhatsApp has updated their api's again and it is observed today morning that many api's from Store. is not available and hence the failures from library. Please look into this issue.

@mukulhase  @Arlington1985

Can you be more specific maybe show with the error? Because I have not a problem with my applications which is running currently.

Same error here

One guy solved it: https://github.com/mukulhase/WebWhatsapp-Wrapper/issues/716#issuecomment-535933731

The delay in loading is working as the modules are taking time. But fun fact is, the second call to getStore() (One without waiting time during load and another after the complete load) is failing.


Fix 'Possible SQL injection vector through string-based query construction.' issue in fundspy\fundspy.py

CodeFactor found an issue: Possible SQL injection vector through string-based query construction.
It's currently on:
fundspy\fundspy.py:255-257
Commit d3e1dfc602db369256652081f37b76dfbf3bc6a3

fixed


trying a fix

hi @Marigold ! sorry for so many pull requests but my build just failed. I had an error where the variable names didn't correspond correctly and now it's throwing an error - I am adding population to all of them to see if this will fix it?

Give it a try. If that doesn't work I'll fix it tomorrow morning.

thank you! it worked :) something to remember for the future, think it's the second time I've done this


Unable to log anything in UWP C# project

Attempting to integrate NLog into my UWP project to run on a Windows 10 system and I'm unable to get any sort of output from NLog. Neither my custom file target nor the internal logging seem to be writing out anything.
nlogTestApp_UWP.zip
Type : bug
NLog version: 4.3.0-alpha3
Platform: UWP
Current NLog config (xml)
<?xml version="1.0" encoding="utf-8" ?>
<nlog xmlns="http://www.nlog-project.org/schemas/NLog.xsd"
      xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
  <targets>
    <target name="file" xsi:type="File" fileName="C:/Users/Dev/Documents/Project/myTestFile.txt" archiveEvery="Day"/>
  </targets>
  <rules>
    <logger name="*" minLevel="Trace" writeTo="file"/>
  </rules>
</nlog>

In case of a BUG:

What is the current result?
-> Currently no logs are written by either main logger or internalLogger
What is the expected result?
-> Log file is written to file myTestFile.txt
Did you checked the Internal log?
->Yes, although its not turned on in submitted code it was previously and that wasn't being written to either.
Please post full exception details (message, stacktrace, inner exceptions)
-> No exceptions are thrown.
Are there any work arrounds? yes/no
-> None found.
Is there a version in which it did worked?
-> None found.
Can you help us by writing an unit test?
-> Probably not, depends on issue.

Thanks! I will have a look to it

FYI:
there are some small mistakes in the config, but even after fixing that it won't work. I will check more in-depth why not.

Got this exception :(
System.InvalidOperationException: Synchronous operations should not be performed on the UI thread.  Consider wrapping this method in Task.Run.
   at System.IO.WinRTFileSystem.EnsureBackgroundThread()
   at System.IO.WinRTFileSystem.Open(String fullPath, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options, FileStream parent)
   at System.IO.MultiplexingWin32WinRTFileSystem.Open(String fullPath, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options, FileStream parent)
   at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options)
   at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options)
   at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, Boolean useAsync)
   at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize)
   at NLog.Internal.FileAppenders.BaseFileAppender.TryCreateFileStream(Boolean allowFileSharedWriting)
   at NLog.Internal.FileAppenders.BaseFileAppender.CreateFileStream(Boolean allowFileSharedWriting)
   at NLog.Internal.FileAppenders.RetryingMultiProcessFileAppender.Write(Byte[] bytes)
   at NLog.Targets.FileTarget.WriteToFile(String fileName, LogEventInfo logEvent, Byte[] bytes, Boolean justData)
   at NLog.Targets.FileTarget.Write(LogEventInfo logEvent)
   at NLog.Targets.Target.Write(AsyncLogEventInfo logEvent)

I looks like we have to do a lot before file writing will work in UWP. Sorry about that. Will try to fix it in 4.4

Thank you so much for taking a look. That's definitely strange but I know things are still pretty messy with UWP sdk so hopefully things get easier for you guys as MSFT irons it out.
Looking forward to your fixes!

Have been able to use a NetStandard1_3 build of NLog 5.0 on a UWP-app both in debug and release-mode. See  #2055

:+1:

@304NotModified Think this can be resolved with the release of NLog 5.0 BETA7

yes indeed, thanks!


Accordion Header title is not affected by headerLevel property

Describe the bug
Header level is never applied to Accordion header text

headerLevel is set to 'h1'

headerLevel is set to 'h6'
Steps to reproduce the bug
Create a simple Accordion with a Header and set the headerLevel property
Expected behavior
<Accordion headerLevel="h1">
  <Accordion.Item>
    <Accordion.Header>Header</Accordion.Header>
    <Accordion.Panel>Panel content</Accordion.Panel>
  </Accordion.Item>
</Accordion>

Given this html a header of h1 is expected, but header title remains unaffected.
Looking into the HTML it seem like the h1 have been applied to the incorrect part of the DOM.
Specifications

Version: "@equinor/eds-core-react": "0.19.0"
Browser: Google Chrome v101
OS: Windows 10 (21H2)

Additional content
As a sidenote the header title also changes color between being open and closed, which seems like an unintentional side effect of some styling.
If this is intentional it would be nice to be able to disable it or set a constant color for the title.

Thanks for reporting this!
I was not able replicate the bug with headerLevel, tested using latest with codesandbox here: https://codesandbox.io/s/ecstatic-swartz-jwd5qr?file=/src/App.tsx (Make sure you inspect the parent node when clicking inspect element)
Thus header changing colors when open, might however be a bug, we'll have someone take a a look at that atleast :)

Are you sure? The header sizing remains the same no matter the headerLevel, even in your codesanbox example to me
I do see that the headerLevel is applied, but it has no effect on the actual header than being a container for the contents within.
In other words, the headerLevel only seems like a glorified version of the as="h1" property for Typography and does not affect the styling of the Header title

Here's some test code to display every headerLevel available in "@equinor/eds-core-react": "0.19.0"
import { Accordion } from "@equinor/eds-core-react";

export default function App() {
  return (
    <div className="App">
      {["h1", "h2", "h3", "h4", "h5", "h6"].map((x, i) => (
        <Accordion headerLevel={x as "h1" | "h2" | "h3" | "h4" | "h5" | "h6" } key={i}>
          <Accordion.Item>
            <Accordion.Header>{`Header is ${x}`}</Accordion.Header>
            <Accordion.Panel>Panel content</Accordion.Panel>
          </Accordion.Item>
        </Accordion>
      ))}
    </div>
  );
}

Its not meant to change the styling but important for accessibility, SEO and semantical HTML to have correct header levels in your application as a whole, thats what this property is for :)
You make a good point on that its almost the same as as, albeit this is just restricted to headings. This is something we could clarify/streamline.

The problem we get is that we would like to style the title as an h4, but that will result in the h4 being a child of an h2.
The somewhat hacky workaround will be to use the as="span" to get our title styled as an h4
<Accordion.Header>
  <Typography variant="h4" as="span">
    Header actual 'h4' title
  </Typography>
</Accordion.Header>

Understandable :)
We have had some discussions on this in general for Typography in regards to styling vs element type. Something we would like to make it easier in the future.
We do also have an issue in the backlog for changing how the Accordion.Header is used. This use-case will be good to take into consideration here #2091

@oddvernes to check

I think the "workaround" above using Typography is good enough, the headerLevel is only meant for accessibility, Part of the reason Typography have separated style from element-type is use cases such as this.


Support for arbitrary sidecar pod

Is your feature request related to a problem? Please describe.
I need to connect the vault sts to cloudsql using cloudsql-proxy for dynamic mysql credential generation
example externalConfig
---
# Source: myapp/templates/mysql-vault-config-configmap.yaml
apiVersion: v1
kind: ConfigMap
metadata:
  labels:
    app.kubernetes.io/name: vault-configurator
    vault_cr: vault
  name: myapp-mysql-vault-config
data:
  vault-config.yml: |-
    secrets:
      - type: database
        description: MySQL Database secret engine.
        configuration:
          config:
            - name: myapp-mysql
              plugin_name: "mysql-legacy-database-plugin"
              connection_url: "root:{{password}}@unix(/cloudsql/gcpproject123:europe-west1:myapp-staging)/"
              allowed_roles: [myapp-mysql]
              username: "${env `MYSQL_ROOT_USER`}"  #  read environment variable
              password: "${env `MYSQL_ROOT_PASSWORD`}"
              rotate: true  # rotate the root credentials of MySQL
          roles:
            - name: myapp-mysql
              db_name: myapp-mysql
              creation_statements:  "CREATE USER '{{name}}'@'%' IDENTIFIED BY '{{password}}'; GRANT ALL ON *.* TO '{{name}}'@'%';"
              default_ttl: "24h"
              max_ttl: "2190h"  # renewable for 3 months max

Describe the solution you'd like
ability to define an arbitrary sidecar container via the vault CR. i think we could basically copy the logic from #796
  vaultContainers:
    - name: cloudsql-proxy
      image: "gcr.io/cloudsql-docker/gce-proxy:1.22.0"
      imagePullPolicy: IfNotPresent
      command:
        - /cloud_sql_proxy
        - '-projects=gcpproject123'
        - '-dir=/cloudsql'
      volumeMounts:
        - name: cloudsql
          mountPath: /cloudsql
      resources:
        requests:
          cpu: 100m
          memory: 128Mi

Describe alternatives you've considered
injecting vault pods with a sidecar at runtime. i'd imagine that could get messy

Did you ever solve this? We're thinking about putting together a PR to implement this feature inspired by https://github.com/banzaicloud/bank-vaults/pull/796, per your suggestion.

Did you ever solve this? We're thinking about putting together a PR to implement this feature inspired by #796, per your suggestion.

not yet. i'd surely appreciate it!

Did you ever solve this? We're thinking about putting together a PR to implement this feature inspired by #796, per your suggestion.

not yet. i'd surely appreciate it!

It's been merged!


feat: support get porperty to get nginx variables

Support get Nginx variables by GetProperty. #53

@membphis Could we invite @bzp2010 to this repo?

Update
I have some questions about the specific implementation.
Do we need to use Get Property to get other variables later (we can specify an implementation prefix, such as "ngx_var" or "var", for the call to get the Nginx Variable).

Update
I have some questions about the specific implementation. Do we need to use Get Property to get other variables later (we can specify an implementation prefix, such as "ngx_var" or "var", for the call to get the Nginx Variable).

IMHO, we don't need it, as Envoy doesn't add a envoy_ prefix to its property. And ngx_remote_addr will be strange.

@membphis Could we invite @bzp2010 to this repo?

current way is enough. this repo is an open source project

@membphis Could we invite @bzp2010 to this repo?

current way is enough. this repo is an open source project

What's the relationship with Open Source project and Invitation?

@membphis Could we invite @bzp2010 to this repo?

current way is enough. this repo is an open source project

What's the relationship with Open Source project and Invitation?


Duplicate uuid values on Transactions

It seems that, up to 2020-10-06 13:22:34.388000 +00:00, transactions were fixed by duplicating the original one and deleting the previous one without changing the uuid. A few issues around this:

In dev, there are un-deleted duplicates crashing the transactions page (React multiple entries with same key)
In prod, enforcing uuid uniqueness is critical from a business point of view

WITH duplicates AS (
  SELECT uuid, COUNT(*), MAX("createdAt") AS "createdAt"
  FROM "Transactions"
  GROUP BY uuid
  HAVING COUNT(*) > 1
) SELECT COUNT(*), MAX("createdAt")
FROM duplicates;

Suggested fix:

Change uuid for all deleted duplicates
In dev, change uuid for other duplicates
Make sure there's a unique index in all envs

This is likely related to the latest upgrade to Postgres 16, where we disable constraints during the DB restore. The original DB likely has these duplicates which were previously erroring.


Content and bug fixes

Resolves #26 & #12

@brandonmichaelhunter I completed the PR but it didn't resolve the issue in the live version.  Can you take another look?

@roberth05 Will do.

I went to the live site and after I logged in I see my adopted drains displayed as links (see below):

Let me know if I'm missing something.

What browser are you using?
On Mon, Nov 11, 2019 at 7:23 AM Brandon Michael Hunter <
<EMAIL_ADDRESS>wrote:

I went to the live site and after I logged in I see my adopted drains
displayed as links (see below):
[image: image]
https://user-images.githubusercontent.com/1362610/68587027-096b1c80-0454-11ea-9b37-f46ca8cbb371.png
Let me know if I'm missing something.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/opensavannah/app.drainsavannah.org/pull/33?email_source=notifications&email_token=AL5MSROJRZ4WRFHHMP5VOW3QTFFCJA5CNFSM4JIIOXFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDWVDAQ#issuecomment-552423810,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL5MSRLNKTTUM4APKUB7SQ3QTFFCJANCNFSM4JIIOXFA
.

@roberth05  I was using Chrome version 78.0.3904.97... I will try in safri now.

Links appears in Safri for me as well.

Very strange.   Might be my account. I'll try with a different account.
On Tue, Nov 12, 2019, 7:12 PM Brandon Michael Hunter <
<EMAIL_ADDRESS>wrote:

Links appears in Safri for me as well.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/opensavannah/app.drainsavannah.org/pull/33?email_source=notifications&email_token=AL5MSRNNKJQCAUBOHJNXHU3QTNBAPA5CNFSM4JIIOXFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOED4M74A#issuecomment-553177072,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL5MSRLK5ZO2D5HNDKTF76DQTNBAPANCNFSM4JIIOXFA
.

@brandonmichaelhunter Still no good for me, even with a new account.
On my side, it looks like a zero length link is being created to the left hand side of the drain address instead of being the entire text of the address.

@roberth05 - Hey Robert I have some time tonight to take a look at this. Please send me a screenshot of what you're seeing.

See below.  It shows the render and the code I'm seeing.  Notice there is a link, but it has no text.

@roberth05 can you check the database to see if there is actual text associated with the user?
Try running this query against your database:
SELECT name, lat, lng
FROM public.things a INNER JOIN public.users b ON a.user_id = b.id
WHERE email = ''
And let me know if you see values in the name field?

Nice catch!  That was the issue.


fix: interaction handle not covering the whole tab transition animation

Motivation
Currently the interaction handle covers only swipe gesture. It should cover the whole tab transition animation, so work can be delayed until it is finished and thus prevent stutters. Might be a breaking change for someone depending on the current behaviour.
Test plan
Tested on iOS and Android, the handle is created/cleared as expected both on swipe gesture and tab press.

Thanks for the PR and sorry for the delay. I need to close the PR since we no longer use Reanimated. Thanks again.


added method to install rpm package in OpenMandriva from default syst…

…em repository.

I would prefer to link to a Github repository.
https://github.com/OpenMandrivaAssociation/amdgpu_top

I have not used OpenMandriva.
Is downloading directly from the build list the standard method in OpenMandriva?

Applied, thanks.


[Bug] required image not appear in web

Describe the bug
required static image source not appear
To Reproduce

run yarn web
go to Card

Expected behavior
appear properly
Screenshots

Desktop (please complete the following information):

OS: macOS
Browser chrome

Please check the file path and try again.


feat: Add getDatabaseName() method to ClickHouseContainer

Add getDatabaseName() method to ClickHouseContainer.java.
avoid such exceptions:

Thanks for your contribution, @colin-lee !


portable-simd: add test for non-power-of-2 bitmask

@calebzulawski is that the intended behavior? Specifically for arrays, the bitmask [1, 0, 0, 1, 0, 0, 1, 0, 1, 0] becomes

[0b01001001, 0b01] on little endian
[0b10010010, 0b10] on big endian

Yes, I think that's expected (we reverse the bitmasks in std::simd)
FYI, we actually just removed use of that variation of the intrinsic in https://github.com/rust-lang/portable-simd/pull/423

FYI, we actually just removed use of that variation of the intrinsic in https://github.com/rust-lang/portable-simd/pull/423

The intrinsic is still called in other places though. In which sense is it "removed"?

Yes, I think that's expected (we reverse the bitmasks in std::simd)

Great, thanks. :)
@bors r+

:pushpin: Commit 2ecd6ed21996ba4ce5f000988ddf7170907fc423 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 2ecd6ed21996ba4ce5f000988ddf7170907fc423 with merge 813c0bfa0dcf13e22a9b7af1e0496fa0dfc16638...

FYI, we actually just removed use of that variation of the intrinsic in rust-lang/portable-simd#423

The intrinsic is still called in other places though. In which sense is it "removed"?

We still use the intrinsic, but currently (not yet synced to rust-lang/rust) we only use integer return types

Ah, this seems to fail on big-endian...

We still use the intrinsic, but currently (not yet synced to rust-lang/rust) we only use integer return types

I see. If the array support ever gets removed entirely, please ping me.

The docs say

No matter whether the output is an array or an unsigned integer, it is treated as a single contiguous list of bits.

On big-endian, the bitlist 0b1001001010u16 corresponds to the array [0b10, 0b01001010].
So either the docs are wrong or my examples listed above are wrong.
Maybe that explains why you had trouble with this in the non-power-of-2 PR?

@bors r+

:pushpin: Commit 7a30c8e392346b1d9d4ef5b3bb57a6a5d12d6d79 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 7a30c8e392346b1d9d4ef5b3bb57a6a5d12d6d79 with merge d46c2e7a73c34a2f03132f67afeb1d9406601aa3...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing d46c2e7a73c34a2f03132f67afeb1d9406601aa3 to master...


Review lnurl-auth compatibility with Zeus, Blixxt,...

Describe the bug
LNURL-auth with LND does not generate the same key as Zeus.
To Reproduce
Steps to reproduce the behavior:

Connect a LND node

Visit stacker.news
login with lightning

Use Zeus and connect the same LND

Visit stacker.news
login with lightning

Those should log you into the same account.
Expected behavior
The lnurl-auth should create the same keys.
Information About Alby

Alby Version: 1.10.0
Alby installed through: Chrome store
Wallet connected with Alby: LND

Further info:

Code in Zeus:
https://github.com/ZeusLN/zeus/blob/c0003371b125c378115219970428ee93365ac31c/views/LnurlAuth.tsx#L95-L129
Code in Blixxt:
https://github.com/hsjoberg/blixt-wallet/blob/f27afc779f2b0429743cf993f31155a1da88dd36/src/state/LNURL.ts#L314-L373
Code in Alby:
https://github.com/getAlby/lightning-browser-extension/blob/master/src/extension/background-script/actions/lnurl/auth.ts#L116
Spec:
https://github.com/fiatjaf/lnurl-rfc/blob/luds/13.md

balance of satoshi: https://github.com/alexbosworth/balanceofsatoshis/blob/6d83d2c8ef133a0fad7fb89b067bb9f0dc8400be/lnurl/sign_auth_challenge.js#L34

There seems to be in issue with the hmac generation:
we use crypto.js hmacSHA256
zeus uses HMAC from fast-sha256
const hashingKey = '...'
//Alby
const AlinkingKeyPriv = hmacSHA256("getalby.com", hashingKey).toString(Hex);
console.log({ AlinkingKeyPriv });

//Zeus
const ZlinkingKeyPriv = new sha256HMAC(Buffer.from(hashingKey, "hex"))
  .update(stringToUint8Array("getalby.com"))
  .digest();
console.log({ ZlinkingKeyPriv });


Enable local cache only when the process type is client

Our master/workers also use clients, and they should not use the client cache. This should be fixed in a different way when we can better scope our property keys and separate the client-side client from the server-side client.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8833/
Test FAILed.

jenkins, test this please.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8834/
Test FAILed.

Failed tests are flakey and unrelated to this PR.

alluxio-bot, merge this please.

alluxio-bot, cherry-pick this to branch-2.2 please.


Wait update before sending spec-loaded

The documentation says that spec-loaded is sent after the rendering is complete, but the current implementation only requests an update, then fires the event without waiting its completion.
With this change, all the needed updates are waited until the event is dispatched.
Thank you @bhdurak for discovering the bug and helping me test the code!

Thanks @bhdurak and @onlined for digging and fixing this one for us !!!


nro controller: generate the machine config label if it impossible to get labels from the machine config pool

Provide a backup plan when the targeted machine config pool does not have a machine config
selector with matchLabels field. In the above case, we will generate the label and drop a
warning message under the log.
It should be important for machine config pools that target more than one machine config
label. For example, our worker-cnf machine config pool definition looks, like this:
...
  machineConfigSelector:
    matchExpressions:
      - key: machineconfiguration.openshift.io/role
        operator: In
        values: [worker,worker-cnf]
...

It is important to say that the code also validates if the generated labels matched by
MachineConfigPool machine config selector, if it not matches the NRO will have
degraded state until the user will update the machine config pool or NRO object with
correct data.
Signed-off-by: Artyom Lukianov<EMAIL_ADDRESS>

We can improve it by adding events under the NRO object.

We can improve it by adding events under the NRO object.

yes please. Events AND logs seems the best solution.

We can improve it by adding events under the NRO object.

yes please. Events AND logs seems the best solution.

...From reporting perspective in general. For this specific issue, we should probably just stop and put the NRO conditions to Degraded (see inline comments)

@fromanirh I refactored a little bit of the code to drop the NRO resource into the degraded state if the generated label does not satisfy the MCP machine config selector.

@fromanirh I refactored a little bit of the code to drop the NRO resource into the degraded state if the generated label does not satisfy the MCP machine config selector.

thanks! this seems the best approach


I am not getting the issues on my discord channel

Test works. It sends an example sentry message to my discord channel. But issues are not sent.

make sure you create an alert in the Alerts tab, like so:

Screenshot

These instructions should be added to https://sentrydiscord.dev/create

If you want to PR them, I can merge it in!


Column highlighting

Is your feature request related to a problem? Please describe.
While it is possible to highlight certain lines in a code block, it does not seem to be possible to highlight certain columns. This would be a nice feature to have, allowing you to highlight different sections of a single line.
Describe the solution you'd like
The solution could be implemented as an optional extension to the current format for specifying lines, i.e. for each line specification, an optional column specification could follow. The format for specifying columns could be the same as the format for specifying lines.
A couple examples of what that might look like:

```ts {2(1-20),3}
function add(
  a: Ref | number,
  b: Ref | number
) {
  return computed(() => unref(a) + unref(b))
}
```

This would highlight line 2, columns 1-20 and line 3. The column specification is in parentheses behind the line specification here. This shows that something like this would preserve backward compatibility: omitting the column specification simply highlights all columns.
An example with a single line, but multiple steps:

```js {1(1-7|9-11|13-19)}
console.log("Hello")
```

This would highlight console, then log, then "Hello".
How about combining "line highlighting steps" and "column highlighting steps"?
Since columns are the "secondary" dimension, it would walk through the column steps for a given line step first, and when the column steps are exhausted, move on to the next line step. Example to illustrate:

```js {1-2(1-3|1)|2(5-7)}
foo quz
bar baz
```

This would induce the following order:

foo and bar are highlighted
f and b (first column) are highlighted
baz is highlighted

Describe alternatives you've considered
This syntax suggestion is just that: a suggestion. I came up with this on the spot without giving it a lot more thought, there may be better solutions. Another alternative could be to separate line and column highlighting - this would make the syntax simpler, but would make it impossible to highlight specific line and column combinations like above.

see also: https://gohugo.io/content-management/syntax-highlighting/#highlighting-in-code-fences


Prescriptive Display is not opening

Prescriptive Display is not opening

Bug has been generated due to enhancement of FCA Tree Structure in FMEA, Bug has been resolved, Now working Fine.
Done

Looks Good Working fine


Make solc wrapper solidity v4.0 compatible

Handle the 'fallback' type
Better handle absence of inputs
Take the newly introduced payable attribute into account

@konradkonrad Done! :)

can you rebase on latest develop?

Done. Merge when ready.


Fail the tests if t.end() is missing

It just silently goes through right now

Switching to Jest makes this dumb thing unnecessary.


Jruby1722

:+1:  - maybe sometime today or tomorrow you could just walk me through the details of the new API

:+1:


Sync eng/common directory with azure-sdk-tools for PR 8119

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/8119  See eng/common workflow

/azp run net - core - ci


Reducing docker image size

Hello,
there are not many solutions for a docker based mailserver out there and from what I've looked at this is a very promising project. Starting at feature-completness to active maintenance this features a lot of the things I'm looking for to use in production.
One thing, that would totally win me over is to reduce the docker image size to a minimum. Currently the image size is around 500mb, which is okay, but could be improved a lot:
https://imagelayers.io/?images=tvial%2Fdocker-mailserver:latest
Here is a checklist of things that worked well:

[ ] move to a minimal base image like docker alpine (100-200mb reduction)
[ ] do installs and cleanup in a single RUN-command
[ ] reduce number of installed packages (do you really need vim in a container?)

This might include some compatibility breaking changes and requires some fine tuning, like looking for equivalent packages in the docker alpine package index. On the plus side, this would:

Decrease download size and increase download/build speed
Increase start/general performance
Decrease resource usage and footprint
Improve security by minimising the attack surface

The docker fluentd image, is a excellent showcase, where this worked well:
https://github.com/fluent/fluentd-docker-image/pull/9

Hi @Brandl and thanks for your support.
These tasks can be done very quickly:

do installs and cleanup in a single RUN-command
reduce number of installed packages (do you really need vim in a container?)

If you wanna PR this, it can be merged very fast.
Regarding the base image, why not! I'll have to have a look to these images. I mostly work with Ubuntu and Debian so I don't know the other.

I just replied to a closed request going in the same direction.
It would be superb to see this project with an Alpine base and it's smaller attack surface.

I have removed vim, and merged lots of commands in Dockerfil in #102

The move to Alpine will break a lot of changed.
We can plan that later in a specific branch to test things before merge.
Anyone can work on that?
The rest is being discussed in #102 with @PhilippeChepy.

An image can be tested at tvial/docker-mailserver:lightweight

tvial/docker-mailserver           lightweight                fe2efbb8c8e8        47 hours ago        432.4 MB
tvial/docker-mailserver           latest                     294dcb629180        2 weeks ago         540 MB

Anyone tried?

i have tested with debian, but no improvements in image size
REPOSITORY                TAG                 IMAGE ID            CREATED             SIZE
tvial/docker-mailserver   debian              b2e55093fdc4        2 minutes ago       432.5 MB
tvial/docker-mailserver   lightweight         fe2efbb8c8e8        2 days ago          432.4 MB
tvial/docker-mailserver   latest              f8595bb70bb2        3 days ago          540.9 MB

As proposed above, we can work on a POC with alpine if we think that it will really reduce the image size (based on docker images, it's 180MB)
This could be the base of a docker-mailserver:v2 with some improvements that I have in my head but I will need contributors.

Let me know if few of you are ready to help. Thanks.

I would like to help, just that I'm not a developer. I could point a domain which I don't use for testing to an install. Are there specifics you'd like to be tested? Or how could one assist in moving it to an alpine based image? I've set up some docker images on alpine - I just started with docker some weeks ago - and it feels a bit gentoo'ish - which is fine by me, but can be anal compared to debian based distros.

@Gymnae we also need testers :-)

@tomav I'd be willing to help with the POC / testing of an alpine based container.

Check this: https://github.com/tomav/docker-mailserver/issues/109

So I've started to work on the alpine container. A thing that I noticed is that the opendkim and courier-authlib packages are both in the testing repository of alpine linux.
I'm not sure if we using packages from a testing repo is a great idea.

The time we build this new image, perhaps the packages will be stable. We need some time to refactor some things and test it before out stable release.
But good point and open question.

I have a good amount of knowledge with shippable.com. I see the site tests the build using Travis-CI but using Shippable the entire build process could be tested. Thoughts?

@gegere what do you mean by "the entire build process could be tested"?

@KyleOndy it seems that some other packages are not yet available (even in testing). You confirm?

@tomav Confirmed. I am having a really tough time getting anywhere with the alpine container.

Regarding my https://github.com/tomav/docker-mailserver/issues/98#issuecomment-203830743 you are using Travis-CI to build and check various merges on the build. Correct? As more and more variables are added various builds would need a matrix of environmental variables to test all the various options. This could be part of the .travis.yml file.
Here is another Docker container and I am using CircleCi and Shippable to manage testing of the builds and verification that everything is working as it should.
Here is a sample of the .yml file I use for this build process.
https://github.com/htmlgraphic/Apache/blob/master/shippable.yml
I understand the need for testing and would very much would like to lend my skills and knowledge.

You're right, the way we do it today is not pretty clean.
If you have an idea to test env vars configurations using travis (before trying to move to another CI service), feel free to PR something. I'm gonna add this to v2 requirements.

#102 if a first step in size reduction but has ti be finished.
@KyleOndy is there another base image we can use instead of alpine?

I am not super familiar with Travis-CI and regarding Docker builds they were late to the game when I was investigating a year ago, this looks to have changed. Recently Shippable released a big update the beginning of March. Many improvements to their backend service and processes which required a large amount of time to refactor my tests on a similar repo and in doing so I realized new methods and functions that really make it the best CI system available.
I have been continuing to compare Circle-CI and Shippable directly as any pull request would allow a build to spin up within both services. I have had one CI service fail because of a connection issue to a package download while the other build service continued successfully.
Basically my point is this. Using docker-compose and other methods should give you the flexibility to test the build with any CI service. As each CI service becomes more robust you can test us minimal effort because commonalty between each service will be very similar.
$ docker-compose -f docker-compose.whatever.yml up -d

Closing this issue because:

master has been reduced
same for v2 on which efforts are concentrated
an attempt on moving to alpine is available here: https://github.com/tomav/docker-mailserver/pull/151


Answers chapter 14 exercises

Here is the final chapter, ready for review 🎉
Just note I had to use pm.compare instead of az.compare in problem 14M2. When ArviZ implements multiple-likelihood models I'll update this NB, as discussed with @aloctavodia
I'm here for any edit 😉
PyMCheers!

Did someone get some time to review this PR?

I will, sorry for the delay.

Thanks @AlexAndorra! And sorry about the delaaaaaaaayyyyyyyyyy

🎉 🎉 🎉
Thanks for the review Oslvado!


Reconsider some testing bash sctipts in golang

Overview
@nickolaev suggested to move some existing testing bash scripts in golang, such as helm-test-charts.sh and cloud creating tools
Lets use this "Issue" for duscussion

@edwarnicke @haiodo @lobkovilya @Denis @alex-yust @a-bastrykina

Personally I don't like the idea to eliminate shell tests.
If we had some UI, we would have automatic ui tests. Deploying of helm charts in some way our UI, because that what users contact with. Users contact with helm charts usually using bash. That's why I believe it's really important to us to have tests that reproduce user's steps as accurately as possible.

Before #1247 our testing was a mess of shell scripts and unmaintainable CircleCI yaml:
https://github.com/networkservicemesh/networkservicemesh/pull/1247/files#diff-1d37e48f9ceff6d8030570cd36286a61
Fortunately this was removed with the excellent cloud-test framework that @haiodo introduced.
Now we are slowly returning to these testing practices. Yes, the people that use NSM will default to the helmcharts, that is where we have a nice guide
https://networkservicemesh.io/docs/setup/run/
helm repo add nsm https://helm.nsm.dev/ # Add the latest release nsm helm repo
helm install nsm/nsm # Install the nsm infrastructure in your Kubernetes Cluster

But I really fail to see how a shell script like this one helps any user to understand that:
https://github.com/networkservicemesh/networkservicemesh/blob/master/scripts/helm-test-charts.sh
#!/usr/bin/env bash

error(){ echo "error: $*" >&2; exit 1; }

[ -n "$*" ] && CHARTS="$*"
[ -n "$CHARTS" ] || error "chart name(s) expected as positional arguments or CHARTS variable"
[ -n "$NSM_NAMESPACE" ] || error "NSM_NAMESPACE must be set"

: "${CHECK_CMD:=make k8s-check}"

reverse() {
  { for e in "$@"; do echo "$e"; done } | tac | tr '\n' ' '
}

echo "Testing HELM chart deployment, chart(s): ${CHARTS}, namespace: ${NSM_NAMESPACE}"

for ch in $CHARTS; do
  dp="deployments/helm/$ch"
  [ -d "$dp" ] || error "Deployment $dp not found"
done

DEPLOYED_CHARTS=''

cleanup() {
  [ "$SKIP_CLEANUP" == '1' ] && echo "Skipping cleanup by flag" && return
  charts=$(reverse $DEPLOYED_CHARTS)
  echo "Deleting chart(s): $charts"
  for ch in $charts; do
    make "helm-delete-${ch}"
  done
  [ "$DELETE_NAMESPACES" == '1' ] && make k8s-delete-nsm-namespaces
  make k8s-deconfig k8s-config
  kubectl delete pods --force --grace-period 0 -n "${NSM_NAMESPACE}" --all
}

onfail() {
  errmsg="Helm chart deployment '$CHARTS' failed"
  echo "$errmsg"
  echo "Cleanup failed chart testing..."

  trap - ERR
  set +o errexit

  kubectl get pods -n "${NSM_NAMESPACE}"
  make k8s-logs-snapshot
  DELETE_NAMESPACES=1 cleanup

  echo "$errmsg"
  exit 1
}

trap onfail ERR
set -o errexit

make k8s-deconfig
# deploy charts
for ch in $CHARTS; do
  make "helm-install-$ch"
  DEPLOYED_CHARTS="$DEPLOYED_CHARTS $ch"
done
# run checks
if [ -n "$CHECK_CMD" ]; then
  echo "Executing checks: $CHECK_CMD"
  $CHECK_CMD
  echo "All checks are OK"
fi

trap - ERR
set +o errexit

make k8s-logs-snapshot-only-master
cleanup

echo "Helm chart(s) deployment '$CHARTS' succeeded"

Now compare the two-line guide and that huge pile of shell trickery. I continue to think that a simple ggolang implementation will be much better than any shell script. I absolutely support the "use the right tool for the job" principle, but here we are digging a rabbit hole. We already have 75 shell script files and 488 go files. That is 15%! It is a lot and we should not continue in this direction.

Yes, you are right, script helm-test-charts.sh were introduced in #1923 and it had to eliminate redundancy that we had before.
But probably we have to keep shell tests as simple as possible, so they could be treated by user as concrete steps. Following that steps user could easily reproduce the whole use-case.
So what do you think about moving back to tests like:
make helm-install-nsm
make helm-install-vpn
make k8s-check

Yes they will be excessive, but much more readable than go-test or what we have now. Also they could be easily correspond with documentation that you mentioned.

I like the idea to simplify examples. It is a super good point. We should keep our examples as easy as we can for new users.
I also think that we can use bash scripts but it should be extremely simple. For example, as @lobkovilya  proposes.
Something like
make helm-install-nsm
make helm-install-icmp-responder
make helm-install-nsc
make nsc-ping-all

Yes, I am supportive of having simple scripts. Still, we need to be cautious into building a whole "framework" around them again. And it is not only the helm tests, but we also have scripts for a lot of "automation", and that may be fine. Yet let's keep a general concept that the shell scripts are only good if they (a) are simple and (b) automate some real steps to follow.


Start passing enabled feature flags to server

Motivation

Implementation

Automated Tests

Manual Tests

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#2826  👈
#2825
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @vinistock and the rest of your teammates on  Graphite


Too much stringification

In main.rs
Some(path) => configuration.load(&path.to_string()),
the arg ends up passed to File::open, which takes a reference as an argument, so I think this could be simplified to just passing "path" to configuration.load. Similarly
configuration.load(&"weld.json".to_string());
why not just
configuration.load("weld.json") ?

path is std::string::String, but hardcoded parts are &str. How can ı unify these ?

If you look at the definition of open: https://doc.rust-lang.org/std/fs/struct.File.html#method.open
"AsRef" means that the "open" function can take anything (string-like), that can be converted into a reference. This includes a String and an &str too.
In configuration.load(&path.to_string()):

in path.to_string the to_string converts path into a String, so I guess path itself is not a String but a &str anyways. But if it is indeed a String, then why call to_string at all?
.. then due to the & it is passed as a reference...

Thanks for the detailed description.


Fixed typo in README.md

Description
Please explain the changes you've made
Fixed typo (Documenation)
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #[issue number]
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[x] Unit tests passing
[x] End-to-end tests passing
[x] Extended the documentation
[x] Specification has been updated
[x] Provided sample for the feature

Thanks for your contribution


Connection string is missing the option to create dsn as a SRV dsn

Laravel-mongodb Version: 4.0
PHP Version: 8.2
Database Driver & Version: Mongodb atlas

Description:
In the documentation it says you can either use dsn env or pass host, username, password etc as their own env variables and it builds the string automatically. This works great for local development, but not if its required to use a SRV string
Steps to reproduce

Add connection details to mongodb atlas as host, username, password etc
Try to connect

Expected behaviour
Should build the dsn string correctly
Actual behaviour
It does not, because it builds the string as a standard connection string "mongodb://hostname.....". To connect to mongodb atlas you need to use a SRV string which looks like this<EMAIL_ADDRESS>Logs:
Insert log.txt here (if necessary)

Hello @ingLomeland,
Indeed we could add a scheme option and use it in the MongoDB\Laravel\Connection class.
https://github.com/mongodb/laravel-mongodb/blob/cff277384ca0d8da45663c00a494ea84cafd0459/src/Connection.php#L256
To make sure I understand your needs, why don't you want to use DB_URI? It allows you to use just a single environment variable for your configuration.

The username and password are not required in the DSN:
return [
    'connections' => [
        'mongodb' => [
            'driver' => 'mongodb',
            'dsn' => 'mongodb+srv://hostname',
            'username' => env('MONGODB_USERNAME'),
            'password' => env('MONGODB_PASSWORD'),
            'database' => env('MONGODB_DATABASE', 'mydb'),
        ],
    ],
];


span.select2-container width:1px

Hi,
WHAT am I trying to do:
my form have some fields dinamically added.
the js code below is part of a click handler of a button "add field".
This field just created in DOM with  $(wrapper).append() needs clearly a select2 init.
I'm facing the issue in select appended, after the select2 initalization is rendered with span.select2-container with width=1px.
but it's a bit tricky to reproduce:
this is my code (simple as possible):

var wrapper is a DOM reference
targetnode is a jquery object that refers to a DOM element that contains a select2 element.
obiouvsly JCore.randomString() returns an alphanumeric random string of 5 char.

var d = JCore.randomString(5)
$(wrapper).append('
'+targetnode.html()+'Remove
')
$('[data-identifier='+d+'] select[data-select]').select2({...})

in the code below you can see a  already initialized with select2.
I removed the select2 spans to avoid a spans duplication too.
targetnode.html( ) content:
https://www.diigo.com/item/image/51ynl/gkaw
(sorry about the image but im not sure there is a way to post html code here :) )
This is the result html code
https://www.diigo.com/file/image/rercsrdzdadqbbpaazcbooocrd/res.jpg
(ps: not sure how I messed up the github editor)

+1
I have the same problem : dynamicly generated . When I reinit select2 on all select, the old ones become 1px wide.

I'm facing the same issue - any known workaround?

Same issue. I'm also using bootstrap theme.
Workaround for me has been to overwrite and make the width important in my own css file:
.select2-container--bootstrap {     width: 100% !important; }

Alternate Solution:
$(document).ready(function() {
$("#myselect").select2({ width: '100%' });
});


[SPARK-21455][CORE]RpcFailure should be call on RpcResponseCallback.onFailure

What changes were proposed in this pull request?
Currently, when there is a RpcFailure need be sent back to client, we call RpcCallContext.sendFailure, then it will call NettyRpcCallContext.send. However, we can see the follow code snippets in the implementation class.
private[netty] class RemoteNettyRpcCallContext(
    nettyEnv: NettyRpcEnv,
    callback: RpcResponseCallback,
    senderAddress: RpcAddress)
  extends NettyRpcCallContext(senderAddress) {

  override protected def send(message: Any): Unit = {
    val reply = nettyEnv.serialize(message)
    callback.onSuccess(reply)
  }
}

This is unreasonable, there are two reasons:

Send back failure message by RpcResponseCallback.onSuccess, we can get the details exception messages(such as StackTrace) in currently implements.

RpcResponseCallback.onSuccess and RpcResponseCallback.onFailure could have different behavior. Such as:
NettyBlockTransferService#uploadBlock

new RpcResponseCallback {
        override def onSuccess(response: ByteBuffer): Unit = {
          logTrace(s"Successfully uploaded block $blockId")
          result.success((): Unit)
        }
        override def onFailure(e: Throwable): Unit = {
          logError(s"Error while uploading block $blockId", e)
          result.failure(e)
        }
}

How was this patch tested?
Existing tests, and modified tests.
Please review http://spark.apache.org/contributing.html before opening a pull request.

Can one of the admins verify this patch?

Hi, @cloud-fan, @zsxwing, would you mind take a look? Thanks a lot.

ok to test

Test build #79706 has started for PR 18670 at commit 962b605.

The transport library's failure callback does not do the same thing as the current code:
respond(new RpcFailure(req.requestId, Throwables.getStackTraceAsString(e)));

Meaning you don't get a deserialized version of the original exception, just its stack trace. So it is not the same behavior. Things like checking the original exception type or its causes will stop working if you do this.

Test build #79706 has finished for PR 18670 at commit 962b605.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79706/
Test PASSed.

As @vanzin pointed it out, this is done on purpose. RpcResponseCallback and RpcCallContext are in different modules. RpcResponseCallback is a low level api, and RpcCallContext is on top of it and interrupts a failure as a message to send via RpcResponseCallback.

Thanks for your reviewing. @vanzin @zsxwing I can understand what you mean. But if we really need a Throwable message in client, such as NettyRpcEnv.scala#L205, why we don't let org.apache.spark.network.protocol.RpcFailure hold one Throwable body message instead of a String error message. Because we reply back error message by RpcResponseCallback.onSuccess, the onSuccess and onFailure can have different behavior.

why we don't let org.apache.spark.network.protocol.RpcFailure hold one Throwable

Because that breaks wire compatibility, and you need to be able to talk to older versions of the library (e.g. an old shuffle service).

+1 to keep wire compatibility. Just for curious, if we can re-design it, shall we use RpcFailure to send exception?

Ok, thanks for the answer. I think we can use ByteBuffer to solve the wire compatibility problem, maybe I'm wrong.
I think we should change this, because we also using onFaliure to send failure message, as follow:
respond(new RpcFailure(req.requestId, Throwables.getStackTraceAsString(e)));

or
(e) => callback.onFailure(e)

Then the client will receive a error messages wraped in RuntimeException.
If all these do not need to change, I can close it.

@ConeyLiu the network layer doesn't know how to serialize Throwable, or in other words, it cannot use JavaSerializer in Spark core.

Ok, thanks a lot for all. close it.


Added workaround for SRAM trim issue

ME10 has SRAM trim issue and requires setting gcr.scon[17:16]=2'b10 - for 1.1V +/- 10% on every power up which overwrites the wrong trims to the correct SRAM trims.
The ME14 also has the same issue and already has this implementation.

Please update your fork with the changes in main. The checks should all pass now once you do that.

/clang-format-run

/clang-format-run


What is the data of the mht file, where can I find the parsing of its data?

We try to be helpful and answer community questions as we're able, but your questions are much more likely to be answered if they follow these guidelines:
https://stackoverflow.com/help/how-to-ask
That looks like a DICOM file? We use cornerstone-wado-image-loader under the hood to query. It uses a collection of codecs to decode, and dicom-parser to parse.

I think you are looking at the response to the RetrieveFrames WADO-RS Request (http://dicom.nema.org/dicom/2013/output/chtml/part18/sect_6.5.html). As @dannyrb said, we use Cornerstone WADO Image Loader for that. If you are looking for a generic DICOMWeb client library, you can use dicomweb-client.

@swederik Thank you, your answer is very helpful to me.


Add component for 404

Addresses #110, specifically:

adds PageNotFound component
adds wildcard router for 404

Additionally, routes have been rearranged to follow the order as suggested by Angular: static paths first, empty path after, and wildcard last.

FIXME:
Routing does not behave as intended quite yet, as an unknown URL will go to the profile page, since its path is /:username.  For now, the error page can be accessed by going to /404 or by using an unknown nested URL like /explore/foo.

Routing does not behave as intended quite yet, as an unknown URL will go to the profile page, since its path is /:username. For now, the error page can be accessed by going to /404 or by using an unknown nested URL like /explore/foo:


CSV export uses single quotes instead of double quotes for dates

Description
Try this query in a MySQL database:
SELECT CURRENT_TIMESTAMP() AS 'date', CAST(CURRENT_TIMESTAMP() AS CHAR) AS 'text';
then export the table as CSV.
The date is surrounded by single quotes while the text is using double quotes. I believe it should be double quotes for both.

The result when loaded into Excel is that the single quotes are interpreted literally, instead of as string delimiters:

DBeaver Version
Community Edition Version <IP_ADDRESS>410061621
Operating System
Windows 10
Database and driver
Driver for MySQL 8 and later
Steps to reproduce
No response
Additional context
No response

Hello @pauljura , please show me what settings do you have in the settings format - General - data formats?
https://github.com/user-attachments/assets/5abf94b6-8ef3-4767-82ab-6780c4f5cc44

Hi @MashaKorax here are my settings:

Hi team, any progress on this? Is there something wrong with my settings, or do you require further info from my end?


fix: finding git root should not change workdir

What kind of change does this PR introduce?
Bug fix https://github.com/supabase/cli/issues/1143
What is the current behavior?
GetGitRoot changes workdir, causing subsequent write to vscode setting to fail if the search recurses to /.
What is the new behavior?
Use go-git library for finding git root.
Additional context
Add any other context or screenshots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

13 of 15 (86.67%)  changed or added relevant lines in 4 files are covered.
7 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.03%) to 62.886%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/testing/fstest/create.go
4
5
80.0%

internal/testing/fstest/stat.go
4
5
80.0%

Files with Coverage Reduction
New Missed Lines
%

internal/gen/keys/keys.go
7
7.94%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
4497

Relevant Lines:
7151

💛  - Coveralls


update temporary test output location

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command
No updates to command. This is a test update to create a subdirectory .gitignore file to ignore temporary test output files.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Thank you for your contribution! We will review the pull request and get back to you soon.


chore(deps): update dependency @bull-board/express to v5.2.1

This PR contains the following updates:

Package
Type
Update
Change

@bull-board/express
dependencies
patch
5.2.0 -> 5.2.1

Release Notes

felixmosh/bull-board
v5.2.1
Compare Source

feat: make queue card clickable, closes #577 #577

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

:tada: This PR is included in version 1.1.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Using JSON format for cache

Looking into issue #414, I noticed that using JSON could be a solution. I made the following changes in FileCache.php:
$datas = json_decode(file_get_contents($this->getCacheFile()), 1);
$writeStream = file_put_contents($this->getCacheFile(), json_encode($datas, JSON_PRETTY_PRINT));
However, are they any problems using JSON instead of serialized for cache ? My limited tests found no problem, aside the transition from older serialized cache to new JSON cache (easily resolved by changing the hash calculation of getCacheName())

Looking into issue #414, I noticed that using JSON could be a solution.

This issue should be fixed now via ee7ddcf992fce4d4665e95145922bcf235ac7a1e

However, are they any problems using JSON instead of serialized for cache ?

According to this the serialize function allows to serialize object instances, which makes it somewhat superior to JSON. However in our case this is not a factor to consider, because the cache should store the object data, not the object instance.
I think going for JSON is a very good idea, because it prevents us from doing things like storing object instances into the cache (like we did with FeedExpander before ee7ddcf992fce4d4665e95145922bcf235ac7a1e). On the other hand we get a format that is very well known and easy to read. Also many editors support JSON trees, which is another plus.

Oh yeah in my example above I made it decode the JSON into an array, but decoding into an object is another option. Overall JSON takes a bit less disk space, but is slower.. by a few microseconds !
So yeah, good for JSON :+1:

Done via https://github.com/RSS-Bridge/rss-bridge/commit/ffb584f7de14e330ff1a8410ef6aaa18ed7693ca
Output format is JSON and hash algorithm changed to md5 in order to prevent collisions.


Fix bug that deletes model checkpoints

Changes:

Fix bug that deleted saved models when multiple processes are started concurrently
Fix logo in README.md

LGTM!


change error msg

change error msg on

sign_nhs with wrong date, msg ("Start date must be greater than create date")
sign_nhs with closed LE, msg ("Legal entity must be Active")
sign_nhs with inactive OWNER ("Contractor owner must be an active OWNER or ADMIN and within current legal entity in contract request")

tested

Start date must be greater than current date
Legal entity is not active
Contractor owner must be active within current legal entity in contract request


does it can run on intel cpu or gpu?

hi , i see you can run on mac m1 cpu, but it can run on intel cpu or not?

All CPU, onnx model can run , TensorRT model need GPU @ziyanxzy

intel arc770 etc can we use intel gpu?

@ziyanxzy you can test and if have any bug , please let me know

how can i set to run on cpu?

because i find it always run on tensorrt instead of cpu

@ziyanxzy you have to remove flag --run_time, it'll run by onnx model

i do that but still can not run on cpu

This is run with onnxruntime-gpu, if you want to cpu only, you have to install requirements-cpu.txt

yes i install requirement-cpu,but onnxruntime-gpu in the requirements

@ziyanxzy so sorry, just remove it and replace onnxruntime only , I'll update later

if i uninstall onnxruntime, so i guess it can not run on cpu without any code change?

@ziyanxzy onnxruntime is for CPU and onnxruntime-gpu is for GPU
You need to pip uninstall onnxruntime-gpu and install again by pip install onnxruntime (CPU only)

oh thank you ~, but where is the output video? i can not found the output..

@ziyanxzy it's in animations folder , it's auto create it

yes, i  use this cmd, but can not create a animations folder. python run_live_portrait.py --driving_video "/home/sas/Desktop/valencia/Efficient-Live-Portrait/experiment_examples/examples/driving/d0.mp4" --source_image "/home/sas/Desktop/valencia/Efficient-Live-Portrait/experiment_examples/examples/examples/source/s9.jpg"  --task ['video']

@ziyanxzy Have you check in the source code, it auto generate in this, also I'm update new feature and fixing the requirements.txt file, you can git clone again

@ziyanxzy Our not have issues with it , please capture your tree project for me

still can not generate i use example driving video and image

@ziyanxzy Please capture your inside Efficient-Live-Portrait folder, if not you need to check here for make sure the path is where

If you run it succesfully, it'll print you the path to save output

I'm running on colab or local not have issues

show me the log when finished running

@ziyanxzy --task not is a list, it's a option image, video or webcam , --task image not --task ['image']

ok i see. i see the code and fint out the task is not a list but a str. now it succeed!!!thank you very much. but i think it will be a little  easy to confuse, because your example is a list. ^^

@ziyanxzy If you run with tensorrt, please make sure your latest ubuntu and package, and it cannot run on window with TensorRT . Noted this my friend

it doesnt matter~would be run on windows in the future? , and i also want to try run on igpu (intel), and deployment

@ziyanxzy If you have window os, Please help me build plugin for TensorRT again on window and pull request . Thank you

@ziyanxzy BTW, I don't like window more haha

if i use tensorrt it will run on cuda, but i want to run on windowscpu or igpu in my pc. i use cpu to run i think it takes not a long time. haha.
i also didnt like windows!

let me try on windows cpu, i think if we want to run on igpu on windows, it may need to convert model to openvino. i will try windows cpu first and give you speed runtime. ^^ thank you . btw , why the onnx run on cpu much faster than python run on cpu. i try to run pytorch model on cpu, but it takes a long time!!!

@ziyanxzy because onnx is built to run on cpu with back-end api is C++ , and pytorch is not , it’ll take all core of your CPU to run, onnx just take half core

i think  the inference time on cpu depend on the cpu, because i run on a better cpu ,it takes about 4min49s, but the worse one , it takes about 12min25s


diff-so-fancy: 1.3.0 -> 1.4.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/so-fancy/diff-so-fancy/releases.
meta.description for diff-so-fancy is: "Good-looking diffs filter for git"
meta.homepage for diff-so-fancy is: "https://github.com/so-fancy/diff-so-fancy"
meta.changelog for diff-so-fancy is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.4.0 with grep in /nix/store/lflwqvh67ph8pj2s00mmdirvijq3x8dd-diff-so-fancy-1.4.0

directory tree listing: https://gist.github.com/9308a9cb5597a5d3f06702f5cdefe090

du listing: https://gist.github.com/90d7265abf65a73bbce8b662e8ffb18a

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
diff-so-fancy

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/lflwqvh67ph8pj2s00mmdirvijq3x8dd-diff-so-fancy-1.4.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A diff-so-fancy https://github.com/r-ryantm/nixpkgs/archive/67ff7413ebeea814e21ae74dc6b322a71e4b83eb.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/lflwqvh67ph8pj2s00mmdirvijq3x8dd-diff-so-fancy-1.4.0
ls -la /nix/store/lflwqvh67ph8pj2s00mmdirvijq3x8dd-diff-so-fancy-1.4.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

diff-so-fancy

Maintainer pings
cc @fpletz @globin for testing.

Ported to stable as bee1e205c5cb268e579df8b12c5ff9c5f6fb7f9b.


How to set the date in hijri date picker

how to set the date in hijri date in hijri date picker.

you can use jquery to fill the input. and make sure the format is matching hijriFormat option
default hijriFormat is iDD-iMM-iYYYY
see docs for more informaiton

for your kind to clarify how to fill the input using jquery, I can add value attribute to the input but it's still empty.

go around :
set value as placeholder :)

same problem
$('#mydateinput').val('29-08-1441');
this not make calender select the date and when open it the input go empty

maybe you can only set the Gregorian date with the following format DD-MM-YYYY if you have not changed the default format
I will investigate more for hijri date

hi
my problem Solved
the problem was hijri  format
this is work with me fine
$('#mydateinput').val('30-11-2020');
$('#mydateinput_hijri').val('1441-08-29');


[Performance] Improve performance, reduce renders on opening RHP and pages within

Problem
When opening the RHP and pages within, there is many more re-renders happening than necessary. https://github.com/Expensify/App/pull/21049#issuecomment-1637864117
This leaves us with sometimes janky animation in the RHP as there is big load for the app https://github.com/Expensify/App/pull/21049#issuecomment-1635971473
Solution
Investigate why there is so many re-renders and find ways to reduce them to as few as possible.
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~019347f7a628ea6599
Upwork Job ID:<PHONE_NUMBER>746444800
Last Price Increase: 2023-07-17

Making this internal and exporting to some agency

Me and @adamgrzybowski are looking at it as a part of adding smooth RHP animations :rocket:

I would like to check the re-render count when applying the changes proposed here:

https://github.com/Expensify/App/issues/22644

won't interfere with you, its your task, just curious if it gets any better with the mentioned changed.

Yeah, we can do that for sure 🚀

Thank Hanno! Assigned you Wojti, this will be in good hands 🙇

From a fast check, when opening the personal details, this line executes: https://github.com/Expensify/App/blob/7770b866bff96efa42c337fb5db8473d34102e71/src/libs/actions/PersonalDetails.js#L371 and it updates the ONYXKEYS.PERSONAL_DETAILS_LIST, which causes a huge amount of rerenders, slowing down the app. @hannojg could you check if applying both your PR to onyx and this one: https://github.com/Expensify/App/pull/21406 makes it perform much better? Hopefully your PR introduces better caching for this part also 🚀

@WoLewicki Thank, I think there is lots of things happening at once now to improve the performance, but it would still be great to profile this flow to see whats the bottleneck. Let us know how that goes

@WoLewicki I commented it in the wrong place, so here is it posted again:

Okay, the PR doesn't directly help with the BaseModal, but here is an interesting number.

Web, render whole app with a user's profile open: 243 commits, 2.5 seconds till last commit
Web, but with PR + changes in place: 176 commits, 1.5 seconds till last commit

That's nice. Did you check if reopening the profile again makes that numbers smaller? I believe there shouldn't be any/much more info coming with each update of ONYXKEYS.PERSONAL_DETAILS_LIST, but we'd have to research what is changed there that causes rerenders.

Will check that by tomorrow!

Not overdue

@WoLewicki Let us know if you will have any updates.

@WoLewicki any update here?

Hmm @hannojg did you get a chance to check it?

We are consistently working on the performance now, making this weekly in the meantime as the improvements are coming in

Nope sorry, the new PR is this one. Let's retest once it gets merged?

https://github.com/Expensify/App/pull/24041

@mountiny how does this sound ^^

It sounds good with me, we'll see what @mountiny says when back from OOO!

@mountiny bump 🙏

@WoLewicki The Onyx PR has been merged, are you able to investigate the re-renders now?

Yeah, will do now 🚀

I just checked the newest main and only opening Personal Details (same as https://github.com/Expensify/App/issues/22996#issuecomment-1638397965) triggers rerenders right now. Unfortunately there are many of them then. The good thing is all of the other RHP flows do not trigger any rerenders of SidebarLinks or ReportScreen.

On the other hand it seems weird that this update is triggered at all, since looking at the props: https://github.com/Expensify/App/blob/867d61e237e6a4e0affaf71e6735d0cdf6969a70/src/pages/personalDetailsPropType.js, none of these should change each time we ask for them, or am I missing something?

Is there some API call returning data which could cause the re-renders?

Yeah, there is the API call mentioned in the comment: https://github.com/Expensify/App/issues/22996#issuecomment-1638397965

@mountiny is there a reason we have the whole props.personalDetails in the dependency array (https://github.com/Expensify/App/blob/eae12248d39ea5266bc18a9d6ac201c99b3fbb88/src/components/withCurrentUserPersonalDetails.js#L38) if we only use props.personalDetails[accountID]?
Each of personal details object has a prop called isLoading. Each time I open personal details, there are at least two updates of the component, and the isLoading prop always comes as true first and then changes to false for the account of the person that I'm in report with.
Combining these two problems, since withLocalize uses withCurrentUserPersonalDetails: https://github.com/Expensify/App/blob/329cac3b49fc47da8eac1b7a74a511171fe22345/src/components/withLocalize.js#L152, and withLocalize is used in Expensify.js(https://github.com/Expensify/App/blob/329cac3b49fc47da8eac1b7a74a511171fe22345/src/Expensify.js#L236) effectively opening the personal details makes the whole app rerender at least twice from what I saw.

I don't think it will unfortunately.

Well we can address that later, one step after another :D

@WoLewicki could you create a PR with the improvement proposed?

I created a PR with two probable improvements. I am not sure about the second one (https://github.com/Expensify/App/pull/26217/files#diff-fbf18141daa663ca1e0398aa28431fc6268fec4405b4d231bbdb76a29d595aaaR66), but since only those 2 fields seem to be used in the binded methods, there should be no need to provide new context values if they did not change. It fixes the whole app rerendering each time we open the personal details.
@mountiny could I get more information about the isLoading prop in withCurrentUserPersonalDetails which brings many many rerenders as I mentioned in https://github.com/Expensify/App/issues/22996#issuecomment-1691628470? I couldn't get to the code that uses it.

I added memoization of WithCurrentUserPersonalDetails: https://github.com/Expensify/App/pull/26217/files#diff-7a254ee6fc92c6fd404d8e3d60431e33697e69d186745f4c8d9b2848b73a1a48R50, but I am not sure if it is the right thing to do (but seems so). Fortunately isLoading prop only changes for the person you are in a report with, and the withCurrentUserPersonalDetails is used for your account, so it does not affect the memo right now.

@mountiny Does this require C+ review?

@mananjadhav yeah, you can also revie the pr then
What do you think of @WoLewicki proposal?

I think we can use @WoLewicki's proposal to reduce the number of renders. This would work. Looking at the PR now.

@NicMendonca I believe we just need to pay $1000 to Manan as this was before the payment change
No checklist needed as this was performance improvement

@mananjadhav remind me, do you request payment via Expensify, or Upwork? Thanks!

@NicMendonca I'll raise it via Expensify.

$1,000 approved for payment to @mananjadhav based on BZ summary.


Migrate logging to golang.org/x/exp/slog

Pull requests:

https://github.com/Senzing/go-logging/pull/55


Add new domain to takeover database

How do i add a new domain like *.trafficmanager.net
Or could you release an update... i think its missing allot of new possible takeoverable CNAME hosts.

good idea


Bump EndToEndTest timeouts to 5 seconds

We still see way too many CI failures due to timeouts, presumably because the
CI infrastructure is slooooow. So let's set a ridiculously high timeout to work
around it.

:star: ⌚


test: Improve tests of position_utils library.

Description
Implement and improve tests of position_utils library.
File: test/position/position_utils.cairo.
Acceptance criteria

Unit tests (nominal and failure) are implemented and passing.

Sure !

Any updates ? @MavericksFive

I am still working on it!

I'll like to take care of this issue


Update zio-schema, zio-schema-derivation, ... to 0.4.14

About this PR
📦 Updates

dev.zio:zio-schema
dev.zio:zio-schema-derivation
dev.zio:zio-schema-json
dev.zio:zio-schema-optics
dev.zio:zio-schema-protobuf

from 0.4.12 to 0.4.14
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "dev.zio" }
}]

labels: library-update, early-semver-minor, semver-spec-patch, commit-count:1

Superseded by #645.


packer-0.12.0_packer-builder-vagrant_linux_amd64 - where is it?

That plugin is missing. As I understand it requires building. Any doc on how to get it would be nice

@devopsman do you mean you want a plugin for 0.12.0 instead of 0.12.2?

I'm using packer 0.12.2, but my question was where to get that plugin? Or should I compile it myself? If so could you please provide some steps to do that as I didn't work with go code..
I'm not able to locate  "packer-0.12.0_packer-builder-vagrant_linux_amd64" as said in that step:
$ cp packer-0.12.0_packer-builder-vagrant_linux_amd64 ~/.packer.d/plugins/packer-builder-vagrant

@devopsman Sorry, must be forgot to mention that. You can download compiled binaries from here https://github.com/themalkolm/packer-builder-vagrant/releases

Thanks, that what I was looking for!

@devopsman Happy to help! I update README.md with the link. Thanks!


Fix typo

Just fix several words.

Coverage decreased (-0.004%) to 24.586% when pulling d2afaf37b81ac1bb2b63fafa658409da3322a728 on fix-typo into a5d261039278019b6d288d72a760ba3beda7444a on master.


badge proposal

On most of the repos I use on GitHub they have badges.
I can make a PR for the following services:

https://travis-ci.org/
https://landscape.io/
https://coveralls.io/
https://requires.io/
https://www.openhub.net/p/python-stencil-template

let me know if you want this, all of them or just some.
cheers!

Wow, cool... thanks!
Given there are no dependencies, is requires.io worth it?  Or do we need to subscribe for when we are a dep?
Landscape seems interesting, but (a) requires an email even when you say don't email me, and (b) somehow can't get it from github info like everyone else.
Does not impress me.
Otherwise, bring it on!
I have to ask, are you using stencil for anything?

true, requires.io wouldn't be necessary but it would be harmless if it was added (it would always be up2date).
I don't recall how I signed up for it (several years ago) but I typically login using my GitHub account.
not yet but I will be using.
I'm just getting more acquainted with what it does and how it does and making these minor contributions at the same time.
the fact you now have tests and builds on travis-ci means I'll probably start using it (I already have a personal project in mind where I could possible replace Jinja2 or add this as an alternative).

Cool.  Well, some simple benchmarks already show it's quite performant, so... :)

Hello from the future! Those years go by quick!
Your README is neatly wrapped at 80-ish columns, readable in a text terminal and Notepad alike.
I know there's a whole GitHub subculture that treats READMEs like GeoCities homepages—and that's fine! There's a tipping point at which the READMEs become unreadable, though. ;)


Miami dade

@chunder I'm wondering if the Python file is intentional?

No, sorry. Let me send a clean PR.


SOAP MTOM attachment support

Bug fixes:

Support loading of WSDL files and their imports from jar files

Enhancements:

Support of mtom for attachements (XOP package), new attribute "mtom-enabled" (defaults to false)
Support mtom xop references and mtom inline data with auto-detection of data type, new attribute "mtom-inline" (defaults to false)
Binary attachment support (no dynamic data replacement, no string conversion) based on content type
Improved handling of dynamic content in attachments
Added "schema-repository" to SendSoapMessageAction to support defining a custom schema repository to resolve mtom attachments
Added SOAP mtom attachment integration test

Known issue:

Schema validation for SOAP messages with MTOM attachments (not inline) is not working, because the XOP reference is not valid data. To support schema validation, XOP references need to be resolved and attachments must be inlined before validation. Inline MTOM attachments are validated successfully. An example of this issue is added to the integration tests.

Thanks a lot, this is what makes open source software so nice! I struggled earlier to get MTOM working. Just a view things that I have changed:

put attachment encoding type (base64, hex) to attachment itself instead of reading the type from xsd schema. This way Citrus is possible to send invalid encoding in error scenarios.
refactored soap attachment content handling (static String content, content resource and binary data handler)
some minor adjustments

Known issue with schema validation is not fixed yet.

Hello Christoph,
As you might remember I have contributed the mtom attachment support for SOAP. In this commit I have indirectly also fixed an issue with resource attachment handling, but unfortunately during merging you reverted this fix and named it optimization. Maybe the fix was not clear in that context.
Problem:
Assume you have a template to upload a file to a SOAP server. The file is given as template parameter variable, and the attachment is a resource using this variable. When you call this twice the second call takes the variable of the first call because it is resolved too early.
For example:

                Here we call the template to upload file 1

                Here we call the template to upload file 2

The problem is, that on the first template call in SoapAttachment.java:305 the variable ${param.filename} is resolved by “classpath:file1”. The problem is that the second template call uses the same instance as the first template call and nothing is resolved anymore, meaning that two times the same file is uploaded and file2 is totally ignored.
I fixed this in commit d1316344db2437b83d89a35229673a14e9f8fea3 by creating always a new “resolved” data handler (in my commit SoapAttachment.java:319) whenever resolve is called. I keep the unresolved values to resolve them again on the next call. This ensures that changed variables are always taken into account.
You optimized that out in commit ca8166f6cedde62a736d706fef5e5a7d5cf06f6d and now the bug is here again. Could you please review your changes and fix this? Otherwise I need to fix this again in a custom branch as we really need that feature.
Kind regards,
Reinhard
From: Christoph Deppisch<EMAIL_ADDRESS>Sent: Dienstag, 10. Februar 2015 19:34
To: christophd/citrus
Cc: Stonator
Subject: Re: [citrus] SOAP MTOM attachment support (#36)
Thanks a lot, this is what makes open source software so nice! I struggled earlier to get MTOM working. Just a view things that I have changed:

put attachment encoding type (base64, hex) to attachment itself instead of reading the type from xsd schema. This way Citrus is possible to send invalid encoding in error scenarios.
refactored soap attachment content handling (static String content, content resource and binary data handler)
some minor adjustments

Known issue with schema validation is not fixed yet.
—
Reply to this email directly or view it on GitHubhttps://github.com/christophd/citrus/pull/36#issuecomment-73755269.


ESPHome: Authentication support

The ESPHome dashboard supports a simple authentication by defining username and password.
The credentials should be stored in variables:
esphome_ui_user: user
esphome_ui_password: password

This must be happen in the systemd service configuration.
ExecStart=/opt/esphome/bin/esphome dashboard --username 'USER' --password 'PASSWORD' /home/esphome/esphome/config/

@JanLeshy Thank you for the fast review and merging.
May it be possible to relese a new version, so I can avoid using main from GitHub :)


Revert high dpi changes

As mentioned in #804, there are some GUI things that don't look good after the changes for high DPI. Therefore this change reverts those changes.
Sorry @jmoralesv. I hope you will find the time to get this working for those scenarios.

@ErikMogensen, could you please squash those two commits? A revert is better to be a single commit. Thanks.

@ErikMogensen, could you please squash those two commits? A revert is better to be a single commit. Thanks.

Don't you think it is enough to do it when merging it into the main branch? The end result will be same, as far as I can tell.

When looking at the PRs, merged or not, they retain all the commits. When a PR is "revert", it's better to have a single commit.

When looking at the PRs, merged or not, they retain all the commits. When a PR is "revert", it's better to have a single commit.

That's true. Looks like I have to abort this PR and create a new one.


Upgrade CNRM version to 1.88.0

The latest version is 1.88.0: https://storage.googleapis.com/cnrm/

I'm trying to switch to the GKE addon version of CNRM in #58


Deploy to Heroku

Hello, I have tried deploy to Heroku. But there is a issue.
No app.json located in the repo URL provided.
Make sure that an app.json file exists in the project root directory.

Yep, Deploy to Heroku is temporarily disabled as we've changed the app structure and this needs to be reorganised a bit. A new way will land in this week.
#448 just for reference.


Add license information

Hello!
I found your project from https://hackaday.com/2023/12/12/can-googles-new-ai-read-your-datasheets-for-you/
Could you please add information about the license of this repository — is it allowed to copy, change and use the code, documentation and design files in any purpose?
It would be great to see any popular open source license.

Sure thing.  Added MIT LICENSE.txt.  Go nuts!


Add thread control to exrmetrics

Add ability to test threading modes with exrmetrics

actually, -t 0 will be equivalent to -s, which is the same as if you don't setGlobalThreadCount at all - it defaults to 0 which is how I noticed this missing bit in the first place...


Adjust the FindContent JSON-RPC call to also retrieve data over uTP

Currently it was implied, due to the ConnectionId result, that the returned values are these of the CONTENT protocol call and that the uTP transfer was not done all or not awaited for. It is however more interesting to have a call that also does the data transfer over uTP.
Here the call gets adjusted to do the transfer over uTP, but the result still allows for returning ENRs when the data is not available, and to indicate whether a uTP transfer was used or not.
Looks like this in openrpc playground:
edit: new picture of how it looks after the additions of https://github.com/ethereum/portal-network-specs/pull/206/commits/1a462d57479feab46b4362dab04ea8d7a85e2186 and https://github.com/ethereum/portal-network-specs/pull/206/commits/e120d3a285c281cd4ee10e5a2d4774590fe6cfa2

The response that is accepted according to this schema is for example:
{
  "jsonrpc": "2.0",
  "id": "1",
  "result": {
    "content": "0x1234...",
    "utpTransfer": false
  }
}

{
  "jsonrpc": "2.0",
  "id": "1",
  "result": {
    "enrs": [
      "enr:-abc...",
      "enr:-def..."
    ]
  }
}

Or also:
{
  "jsonrpc": "2.0",
  "id": "1",
  "result": {
    "content": "0x1234...",
    "utpTransfer": false,
    "enrs": null
  }
}

{
  "jsonrpc": "2.0",
  "id": "1",
  "result": {
    "content": null,
    "utpTransfer": null,
    "enrs": [
      "enr:-abc...",
      "enr:-def..."
    ]
  }
}


Expectation for llvm toolchain with -rtlib=compiler-rt

Compiling this:
$ clang -fuse-ld=lld -rtlib=compiler-rt -stdlib=libc++ foo.cpp -o foo

generates a binary which is linked against libgcc_s
$ ldd foo|grep libgcc_s

	libgcc_s.so.1 => /lib/x86_64-linux-gnu/libgcc_s.so.1 (0x00007f104bc66000)

Not sure it is expected?

When working on:
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=958783
to provide:
https://clang.llvm.org/docs/Toolchain.html
to make the following tests work:
https://github.com/opencollab/llvm-toolchain-integration-test-suite/blob/master/tests/whole-toolchain.c
https://github.com/opencollab/llvm-toolchain-integration-test-suite/blob/master/tests/whole-toolchain.cpp


Added zhimi.airpurifier.ma2 as Mi Air Purifier 2S

Hi,
I included as I understood only in China available Mi Air Purifier 2S.
Used for this, the data structure from zhimi.airpurifier.mc1.
https://miot-spec.org/miot-spec-v2/instance?type=urn:miot-spec-v2:device:air-purifier:0000A007:zhimi-ma2:1
The only noticeable difference had been the setting:
7
iid : 8
type : "urn:miot-spec-v2:service:air-purifier-favorite:000078B2:zhimi-ma2:1"
description : "Air Purifier Favorite"
properties
0
As I didn't find this setting in any other purifier, I just left it out, because all other settings worked really well.

For some reason the .ma2 is not supported any more.
I only get the following message in the js adapter:

13:17:06.541
error
javascript.0 (28953) script.js.common.AllMyMi: Error: Model zhimi.airpurifier.ma2 is not supported

13:17:06.542
error
javascript.0 (28953) at CreateDevices (script.js.common.AllMyMi:910:36)

13:17:06.542
error
javascript.0 (28953) at main (script.js.common.AllMyMi:842:11)

For some reason the .ma2 is not supported any more. I only get the following message in the js adapter:
Have you updated JS Controller or such? Try to copy the Def File again in the node.mihome devices folder as described in readme.

For some reason the .ma2 is not supported any more. I only get the following message in the js adapter:
Have you updated JS Controller or such? Try to copy the Def File again in the node.mihome devices folder as described in readme.

Jop that's it didn't read the part of the readme


vue-component-meta, exposed contains all slots, props and events as well

Vue - Official extension or vue-tsc version
latest
VSCode version
Not used here
Vue version
latest
TypeScript version
3.5.12
System Info
No response
package.json dependencies
No response
Steps to reproduce
Run the tests of this repository after adding the following in line 702:
expect(meta.exposed.length).toBe(1);
What is expected?
It should pass
What is actually happening?
It fails because exposed contains way more than just the exposed symbol.
Link to minimal reproduction
https://github.com/vuejs/language-tools/blob/master/packages/component-meta/tests/index.spec.ts#L691-L704
Any additional comments?
No response

It passed on my end.

Apologies! I forgot to mention that the following change is required as well (otherwise there is nothing besides the one expose that could end up in the exposed array :-)):
test-workspace/component-meta/reference-type-exposed/component.vue
Add defineProps(["testProp"]);
Result:

I encountered this behavior using<EMAIL_ADDRESS>The exposed contains:

All defined props.
$slots
events prefixed with 'on'(i.e. an emit defined as 'update:filtered' will be 'onUpdate:filtered'

There is no way currently to filter them and just get the exposed.


Can't update to 2.3.0 via NuGet

Error is "Unable to find a version of 'Mono.Cecil' that is compatible with 'Machine.Specifications.Runner.VisualStudio 2.3.0 constraint: Mono.Cecil (>= 0.10.0-beta6)'."
2.4.0 has the same issue.
My target is .NET Framework 4.7 x64

By allowing pre-releases I was able to get that dependency. Had to manually add the reference though.

Thanks for reporting. I am Re-opening in order to see if the experience could be improved. I think for new style projects this doesn't happen but if it does for old style projects then maybe we can target .Net Framework with the previous stable version of Mono Cecil

@UniSnake Please use 2.5.1 and let me know if this resolves this issue?

Yes, I was able to upgrade to 2.5.1 without issue. Thank you!


qa_openstack: Remove duplicate zypper -n switch

$zypper already has "--non-interactive". There is no need to add
another "-n".
This fixes the following warning:
Warning: The flag --non-interactive can only be used once.

My zypper man page says this is for something different
install (in) [options] name|capability|rpm_file_uri...
...
    -n, --name
               Select packages by their name, don’t try to select by capabilities.

@cmurphy -n before the subcommand is a global zypper option. from the man page:
       -n, --non-interactive
           Switches to non-interactive mode. In this mode zypper doesn’t ask user to type answers to various prompts, but uses default answers
           automatically. Those default answers also depend on other options like --no-gpg-checks or --ignore-unknown.


Towards JDE ... jwm  + desktop environment

Desktop Icons
Currently jwm can do pseudo desktop icons using trays and tray buttons, but they have formatting issues due to text widths vs icon widths since the icon cannot just be centered in the tray button and the varying text sizes would make the "icons" appear to have different offsets... not to mention lack of a "none" background color for tray/buttons.
I propose a new desk_icon as member of the Desktop tag (or global to be on all desktops) that has attributes for x and y offsets, icon widths/heights and various text attributes.  These would be non-movable to prevent confusion since jwm currently has no way to modify it's config files or communicate with config generators.
Tray Applets
The current tray buttons can have all of the aspects of an applet with the exception of changing the icon.
I propose a new "autorefresh" attribute for icons to update the icon if it changes.  I have used inotify for this on linux, bsd would use kqueue; however I think there are a few wrapper libs that can handle multiple platforms.
To account for changing tooltips I propose the option to read the tooltip text from a file on demand.
Notifications
With the autorefresh icons in place, notifications can be accomplished using tooltips
extra possibilities

expose jwm's built-in capabilities to external programs via pipes (dialogs, image handling, tooltips,...)

expand built-in capabilities to become a full GUI.

handle email (j/k... couldn't resist)

@technosaurus I completely agree with this!
The issue  I like most is the ability to make tray buttons into indicators.
The problem I see with that is that we would need a way to run external code to choose the icon... for example, if I make a volume indicator, how would I notify the icon to update when volume is changed?
Likewise with a battery, or network indicator.
I also really like the idea of having some sort of way to use JWM's existing dialog functions (perhaps the menu functions/trays can be extended into the windows to be able to make XML based JWM dialogs... which would be really cool!)
Maybe @technosaurus will make a JWM filemanager which also handles e-mail :smile:

@Israel- I have already created a gtk based program called Simple Icon Tray that does this by running a tiny daemon process in the background to update the icon and tooltip.  SIT uses gtkstatusicon which the idiot gtk3 devs removed in a random minor release, so its currently gtk2, but I have been meaning to port it to x11 or xcb if i could figure out what x11 hints are needed to get them to go in the dock (i could use jwm's swallow feature, but prefer something more portable)   I already have the rest of the code written for my xcb image viewer and an early inotify based version of SIT, ... maybe @joewing  can clue me in to what hints are needed.... I may even be able to extend it to handle desktop icons using transparent windows without title bars.  Recently goingnuts ported my xcb image viewer to Xlib, so that could help.

@technosaurus I have used sit/sdesk, I actually packaged it for Ubuntu in a PPA, though I have not updated it in a while, mainly because the Icon had to be a file, rather than using the gtk_icon_theme_load_icon()
I have since made a bash script using yad, as it provides dialogs as well as a status Icon.
I even wrote a small program to indicate battery status back when libupower-glib/upower.h worked (pre systemd engulfing everything.... I feel your nice library changing everything to break pain)
I suppose a JWM specific way to create indicators would be very helpful.  perhaps some inclusion in a traybutton.
<Traybutton ..... >
<Indicator>script-to-update-icon-and-menu</Indicator>
<Button><!-- whatever--></Button>
</Traybutton

for the icons, I was thinking autorefresh="true" with the default being false.  the true value adds an inotify (or kqueue) watch to that icon and a change would trigger a reload+redraw of the icon ... except now jwm caches images, and I'm not sure how that would affect this.  A similar thing could be done for tooltips if the string started with "file:/"

I have been adding some features to my settings manager recently thought about your idea for Desktop Icons.
I think @technosaurus the best route would be to use the existing structure of the TrayButton but make it something like DesktopButton but add in something like you were saying x="int" y="int"
So the configuration would be something like
<Desktops width="4" height="1" >
<!--  THIS desktop -->
    <DesktopButton icon="accessories-calculator" x="140" y="140" border="false">
      <Button mask="1">exec:xcalc</Button>
      <Button mask="23">root:p</Button>
    </DesktopButton>
    <Background type="image">/path/to/img-home.png</Background>
  </Desktop>
</Desktops>

I know this is pretty much what you were saying in the first place, my input is to visualize it and to bring in the possibility of reusing TrayButton code where possible


Error: failed to get contents

I am getting following error when trying to get data in this file:

hi, this PowerBI workbook is for Azure Datafactory V1 only and does not support V2 - I guess this is your issue here.
There are currently no plans to support V2 as it fundamentally different


Application don't open after exit and start Windows. (3.2.2)

Sorry for my bad English.
My Setup

Operating System: WIndows 8.1 Prof x64
App Version: 3.2.2
Installation type: exe

I have tested with the latest version
I can simulate the issue easily.

Description
Launch Rocket.Chat --> Exit from File - Exit or Ctrl+Q --> Launch again...
Application does not opening. In the Task Manager I see 3 process of Rocket.Chat.
Current Behavior
Application does not opening.
Expected Behavior
I see Rocket.Chat application's main window.

Please test if the latest release did solve this. If not please reopen the issue or create a new one and tell us. I will be closing this one for now.
Thank you!