Fix TODOs in outbound channel adapter

always use successChannel for metadata
unwrap ExecutionException

I wonder if failure on Travis says you anything:
org.springframework.integration.kafka.outbound.KafkaProducerMessageHandlerTests > testOutboundWithTimestampExpression FAILED
    org.opentest4j.AssertionFailedError at KafkaProducerMessageHandlerTests.java:231

Thanks

Seems unrelated - built clean for me - rebuilding on Travis...

...green


COMMON ISSUES: [dl4j-cv-labs or dl4j-labs module not imported, Run symbol can't be found]

To-do

Tag anyone if help is required
Copy this issue link to TrainingLabs Support Project.

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

Go to '...'
Click on '....'
Scroll down to '....'
See error

Screenshots
If applicable, add screenshots/video to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. Windows, Ubuntu]
Browser [e.g. chrome, firefox]
Version [e.g. 1.0.0]

Additional context
Add any other context about the problem here.

Solved in https://github.com/CertifaiAI/certifai-wiki/wiki/Import-Module


Release 0.7.2 - [merged]

In GitLab by @Asacolips on Jun 21, 2023, 20:48
Merges release/0.7.2 -> master

In GitLab by @Asacolips on Jun 21, 2023, 20:48
mentioned in commit 12720e48a2f72c40ae1eda71c1f77342886c4342


Make libsweep compilable with a C++ compiler

Currently the sources use GNU extensions and C99-only features, so are not compilable with a standard C++ compiler.  This PR fixes this.
Also, this PR renames time.h to sweep_time.h to avoid issues with include paths and name conflicts with the standard library time.h header.

The main reason was to make it easier to vendorize for FRC use, as a C++ compiler is always used in the vendor template (https://github.com/PeterJohnson/sweep-sdk).  The vendorized build makes it very easy to integrate into eclipse as teams can just unzip it rather than needing to build it themselves, plus it provides Java wrappers.
There also doesn't seem to be a lot of benefit to doing the C99 things like piecewise initialization in this case.

Sweet, love it!
If you maintain the integration feel free to drop a link into this repository's main readme, e.g. under a "who's using this" section.
Also do you have any plans to upstream the Java wrapper? How big of a lift would that be? The main reason we don't have one is my lack of experience and use-cases in that regard.

Ok, I definitely plan to keep it up to date with changes here.  Upstreaming a variant of the Java wrapper is possible, although the version in my repo is FRC specific so it wouldn't be identical.


webgl renderer causes oni to become unusable

Oni Version:
0.3.7
Operating System:
Windows 10
Issue:
After changing the editor.renderer to webgl and reloading oni, nothing works. The editor throws the error: Unhandled Exception:Error: Not a fixed size
Expected behavior:
Sweet, sweet webgl rendering as in oni 0.3.6
Actual behavior:
A black screen and no keys working
Steps to reproduce:
Install oni 0.3.7 on Windows 10, set editor.renderer to webgl

Hmm...that is odd.
I wonder what could have caused the regression between 0.3.7 and 0.3.6? I don't know of anything that would have caused it outside of a few updates to add ligatures to the webgl renderer. Is there any other debug information when it fails to load?
Otherwise, we will have to try and repro, though I know the webgl renderer is working well for me on Windows.


HWY-100: Use and check instance IDs

This adds the logic for inserting instance IDs into votes transmitted over the network and validating them upon reception.

Since InstanceID is fixed for the whole era, it would be nice if it didn't have to be thread through everywhere but checked only at the entry/exit to/from era.

@goral09 I'm not sure what you mean. We can't only check the instance ID at the era entry/exit, because we need to make sure that the votes we receive actually belong to the current era. So we have to be checking the instance IDs contained in votes all the time.

@fizyk20 I meant when the messages enters the era.

@goral09 Isn't that what this PR does, though?


SignalXY does not respect OffsetX or OffsetY

var sig = plt.AddSignalXY(/* random walk data starting at (0, 0) */);
sig.OffsetX = 10_000;
sig.OffsetY = 100;

I would expect the data to start at (10000, 100) not (0, 0). Automatic axis limit detection works as expected though.

Hi @swharden,
I made PR #894 to fix that.
But I would like to discuss the very necessity of using offsets.
As far as I understand, it all started with a Signal, where OffsetX is the only way to set an offset from 0 and its use is fully justified. OffsetY was added rather for symmetry, since you can always add an offset manually to the ys array on the user side.
SignalXY has both arrays xs and ys, and personally I don't see any particular problem adding an offset on my own to this arrays.
I fully admit that my vision is very limited, and for many, the ability to set the offset may be a very convenient feature. I just wanted to bring my opinion up for discussion.
From my point of view, these parameters do not significantly degrade performance, significantly complicate the SignalXY code, and will never be used (at least by me).

Thanks for #894 @StendProg!
X/Y Offset Use Case

I would like to discuss the very necessity of using offsets

Hi @StendProg, you raise a fair point! Many people may not have a need for offsets, and even so it is possible to achieve an offset effect by manipulating original data.
The use case I typically have for offsets is when displaying electrical signals over time. These signals are long and continuous recordings, but they are often separated into "sweeps" where a repetitive action occurs every several seconds. I often write GUIs which switch between different display styles, so it is not convenient to modify the data of the array every time I want to change the offset strategy. Also some of these recordings are very long (millions of data points), so manipulating the data points can be costly. Finally, I often want the user to adjust a slider to change the offset settings, and applying these offsets at the pixel level (rather than the raw data level) is much faster. Here are some screenshots to illustrate the type of plots I typically create:
It is common to display this data as overlapping sweeps:

However it is also very common to display data as "stacked sweeps", so the Y offset becomes useful here:

This data is 12 one-second sweeps. Using X offset it can be displayed to appear as one continuous signal:

Using both X and Y offsets, 3D effects are achievable too:


GH-731: Replying template improvements

Resolves https://github.com/spring-projects/spring-kafka/issues/731

Auto detect reply topic/partition
Improve docs to explain how to use a shared reply topic
Add boolean to reduce log level for unexpected responses

Looks like there is some conflict with the master.


Object.values wrongly excludes undefined with optional properties

Bug Report

🔎 Search Terms
object values entries undefined optional properties
🕗 Version & Regression Information

This changed between versions 4.2.3 and 4.3.2

⏯ Playground Link

Playground link with relevant code
💻 Code

const a: { b?: 1 | undefined } = { b: undefined };
const c = Object.values(a);
console.log(c); // Returns [undefined] but c has type 1[]

🙁 Actual behavior

c has type 1[] but can be equal to [undefined]
🙂 Expected behavior

c should have type (undefined | 1)[]

It's legal to initialize a with that value even if b doesn't include undefined, but Object.values should not exclude undefined from the possible values. How can it be 'intentional' since it leads to wrong dangerous types?
Same issue with Object.entries.

How can it be 'intentional' since it leads to wrong dangerous types?

Well, that's why we added the strictOptionalProperties flag.

What is exactly this flag ? Is it yet available ? I can't find it in the doc...

It's new (as in, unreleased); see #43947

It will still be possible to allow undefined as a value for an optional property

?. The entire purpose of the flag is to make that not be allowed.

In my understanding, its purpose is to allow (not to force!) an optional property to never be undefined if its present.

This is an error under strictOptionalProperties:
let x: { foo?: string } = { foo: undefined };

You either have to give foo a value, or else omit it, in which case it would be omitted from Object.values() too.  Omitting the property was allowed before, so the flag would serve no purpose if it didn't enforce it.

Yes but that is not an error:
let x: { foo?: string | undefined } = { foo: undefined };

And this is the repro for my issue...

The cause of this change is #43086. It's debatable what the correct behavior is, as undefined in the type of an optional property sometimes should be interpreted as "the property isn't there" and other times as "the property could be there with the value undefined". In --strictOptionalProperties mode it is possible to distinguish (and we should, but don't currently), but without that option it isn't, and we simply have to commit to one or the other behavior. Our current behavior (stripping undefined from optional properties with inferring to an index signature) is likely the desired behavior most of the time, but strictly speaking it isn't sound--as illustrated by this issue.

But there’s no index signature in the repro…

There's actually an index signature in the type of Object.values: https://github.com/microsoft/TypeScript/blob/5540364c1233ddcbaef0fa72a932a0a69a0d7ed7/lib/lib.es2017.object.d.ts#L26
I think it is understandable to strip undefined from optional properties when --strictOptionalProperties=false, because { b?: 1 | undefined } is equivalent  to { b?: 1 } in this case.
But is it OK to strip undefined when --strictOptionalProperties=true?
https://www.typescriptlang.org/play?strictOptionalProperties=true&ts=4.4.0-dev.20210613#code/MYewdgzgLgBAhgLhgbxgIwPxIIwwD4wCuYAJgKYBmAlmGSTAL4wC8K6Sx51t9DA3AChQkWMBYwA8mgBWZYFAB0ANzgAbQmQgAKOAEpBwiCFVkFqkAHMtwfQKA
In my opinion, { b?: 1 | undefined } under --strictOptionalProperties=true has a strict meaning that "b is 1 or undefined, or it does not exist" without ambiguity, so I expect c: (1 | undefined)[] here.

Don't forget that any external module can choose to export something like { b?: 1 | undefined }. And in that case the undefined possibility should really not be stripped since it could lead to the most common JS bugs that TypeScript wants to address.

So, there are a few things we should definitely fix:

In --strictOptionalProperties mode, inference from an optional property to an index signature should simply infer the property type as written. For example, inferring from { a?: number } to { [key: string]: T } should infer number for T, and inferring from { a?: number | undefined } to { [key: string]: T } should infer number | undefined for T.
In --strictOptionalProperties mode, a mapped type that strips optionality should not strip explicitly included undefined types. For example, Required<{ a?: number }> should be { a: number } and Required<{ a?: number | undefined }> should be { a: number | undefined }.

This will make the current definitions of Object.values and Object.entries behave as expected.
That said, I'm a bit surprised with have strongly typed definitions for Object.values and Object.entries in the first place. We've previously rejected having a similar definition for Object.keys (because an object might have more properties than we know about). I would have expected all of them to follow the same pattern.
In addition, if we stick with strongly typed Object.values and Object.entries we ought to improve their definitions as the current ones have issues. For example, applying Object.values to an object of a class or interface type with no index signatures yields any[], not a strongly typed array. I'm thinking the following would be better declarations:
values<T>(obj: T): T extends readonly any[] ? T[number][] : Required<T>[keyof T][];
entries<T>(obj: T): T extends readonly any[] ? [string, T[number]][] : [string, Required<T>[keyof T]][];

@DanielRosenwasser Opinions?

That said, I'm a bit surprised we have strongly typed definitions for Object.values and Object.entries in the first place

The "improvements" to those APIs were incorrectly merged at some point and now it's too much of a break to take them out. Maybe we should just give up on Object.keys, since people complain about its incompleteness relentlessly but never notice the unsound definitions for values / entries
The inference rule changes seem both correct and necessary for SOP.

With #44595, the original example behaves as expected in --strictOptionalProperties mode.
I'm leaving this issue open to track possible changes to Object.values and Object.entries.

The key/value/entries issue is tracked at #38520, so closing this

Mentioning --exactOptionalPropertyTypes since that's the actual name of the flag (to help with searches)


Add def_ir and iir to stencil object

We are building a tool for data flow analysis, and naturally want to include the stencils as a sub-tree of nodes in the data flow diagram. When the parser comes across a stencil call, e.g. stencil like this:
def compute(...):
    stencil(...)

as far as I can tell it actually has no way of getting back to the implementation IR from both the live object and Python AST node. There is no information stored in the StencilObject about the method it came from, nor the build info. Since it exists anyway, could we just always store this info in the StencilObject instead of passing it out as a key in the build_info dictionary argument?

I should note that we can work around this for now with hacks in our application, but it would be nice to have this soon in gt4py as a whole, since the data exists, but it's not quite in the right place.

It looks like a reasonable request, although I'm not quite sure if it's very useful to access ImplementationIR right now. Note that StencilOobject stores a reference to the actual definition function in StencilOobject.definition_func

Thanks @egparedes.

Note that StencilOobject stores a reference to the actual definition function in StencilObject.definition_func

That's good to know. Actually definition_func._gtscript_ does store the full annotation including externals and nonlocals captured at the time, so it is possible to re-lower to IIR from there. That said, we still opted to create out own wrapper class for now, that exposes the build_info.

I would therefore consider this low priority then for now, but if @DropD or you are refactoring that portion of the frontend, we would support removing build_info in favor of adding it to the StencilObject.

Is this still relevant? The issue could be updated with GTC IR's. But is the original motivation still applicable? Otherwise, I propose to close.

I wanted this for a custom dataflow library at one point, but now that we've centered around DaCe, I think we can close this.


Question on Learning a Space of Implicit Functions

In section 4.4 of your paper you go into an interesting hypernetwork idea that can generate the siren parameters for a space of "functions" (images in that case).
In section 9 of the appendix, you go more into details, and I specifically care about the part where you predict the siren parameters from the input:

As far as I understand

the weights of a 5-layer SIREN with hidden features of size 256

Are:
W1 ∈ R(2, 256) # maps x,y
b1 ∈ R(256)

W2,W3,W4 ∈ R(256,256)
b2,b3,b4 ∈ R(256)

W5 ∈ R(256,3) # maps to rgb
b5 ∈ R(3)

Total params: 2*256 + 256 + 3*(256*256 + 256) + 256*3 + 3 =198,915

So, do I understand correctly that your hypernetwork takes the input from the convnet, input, and does the following:
h1 = relu(U1*input + c1)
h2 = U2*h1 + c2

Where
U1 ∈ R(|input|, 256)
c1 ∈  R(256)

U2 ∈ R(256, 198915)
c2 ∈ R(198915)

This doesn't feel right to me.

Yes, your description of the hypernetwork architecture is correct.

Yes, your description of the hypernetwork architecture is correct.


Bug: Handling multiple facets

There is a bug when handling multiple facets in a post.  I think the byte location shifts.
{
  "record": {
    "$type": "app.bsky.feed.post",
    "text": "I have an open discussion topic if you want to add your thoughts on supporting 2FA in the PSBluesky #PowerShell module.",
    "createdAt": "2024-11-18T18:02:15.8878051-05:00",
    "facets": [
      {
        "index": {
          "byteStart": 162,
          "byteEnd": 173
        },
        "features": [
          {
            "$type": "app.bsky.richtext.facet#tag",
            "tag": "PowerShell"
          }
        ]
      },
      {
        "index": {
          "byteStart": 15,
          "byteEnd": 31
        },
        "features": [
          {
            "$type": "app.bsky.richtext.facet#link",
            "uri": "https://github.com/jdhitsolutions/PSBluesky/discussions/21"
          }
        ]
      }
    ]
  },

The original string was: "I have an open [discussion topic](https://github.com/jdhitsolutions/PSBluesky/discussions/21) if you want to add your thoughts on supporting 2FA in the PSBluesky #PowerShell module."

The byte offsets for tags and mentions are being calculated first. After that the record.text is being modified here which makes the offsets calculated for tags/mentions against the original record.text version, not the revised one.
You can quickly fix that by handling the markdown style links first because that's the only one mutating the original text.
The approach works until the message contains unicode characters that take more than one byte btw. It's unlikely to happen when someone is sending a message from the terminal but it can.
I only noticed because I was doing this in reverse. I was taking posts from my timeline and trying to use the facets to render them as links in the terminal and they were all off-by-one when emoji were present in the record.text.
https://github.com/jdhitsolutions/PSBluesky/pull/24/files#diff-1994ad15cb20c326911f9a70a3308644bc32ba2986d778de258a9c630bf3f46dR74

I haven't had a chance to work on this so I appreciate your suggestion. That makes the most sense.

I think I have this fixed in v2.1.0.

This change has been released. Keeping this open until I'm sure this has been resolved.

I think this has been resolved.,


Why the command openssl displays its output through stderr?

When I use the openssl help command, I found that the help information is output to stderr. This is not the same as other software, I am confused. Is there any reason for this way?

My guess would be that this is how it was originally done and that it has stayed that way since.
A lot of modern command line tool conventions didn't exist when the project began.


Scptoolkit driver installed over mouse driver. Please help

Im new at computers and watched another source install the program and drivers wrong. I found the fourm after it was to late. I have uninstalled the mouse drivers and factory reset but for some reson it still say scptoolkit compatible device for manufacturer in the mouse properties. I have looked up everything i know to do, please help.

That sounds weird.  Glad you solved it.  Going to close this for now.


Jackett outdated

Hello,
Jackett has not been updated for 2 months, the current version is v0.20.2660 while the version here is v0.20.2297
https://hub.docker.com/r/linuxserver/jackett
Can you update ?
Thanks

Hi @boced66 I have stopped maintaining a jackett container in favor of using Prowlarr. Unlike binhex, linuxserverio or hotio I don't strive to support every application out there, only the ones I find useful.
Maybe consider using Prowlarr as an alternative?
I created an issue and had no one raise up and question it since. See https://github.com/onedr0p/containers/issues/106


Off-canvas menu defaults not set properly

When using the WP menu editor to set a menu to the "Mobile" theme location, the off-canvas menu will not display until you go through the following process:

Open Customizer > "Mobile Menu Settings"
Open "Mobile navigaton layout"
Default is set to off-canvas, select topbar then off-canvas in order to re-save the menu

Note: if you save and leave the customizer at this point, the front end will display the off-canvas menu but it will be broken.

Go back, open "Mobile navigation position"
Default is set to left, select right then left in order to re-save the menu
Off-canvas menu now displays as expected

Tested in WP 4.3 using FoundationPress 1.2.0.
PS - Love Zurb, love FoundationPress, thanks for this awesome resource. :+1:

Thanks for your bug report and quickfix :+1:

Fixed https://github.com/olefredrik/FoundationPress/pull/449

I still had this issue still after the merge. I had to do exactly as @ethanmay described to get the offcanvas to work correctly. I verified I had the changes in #449.

Issue still exists indeed. I pulled in the code just a few hours ago so i'm sure i have the newest version.

Still seeing this issue today. Pulled code this past monday.

Hi guys. @skoldin has made a bugfix: https://github.com/olefredrik/FoundationPress/pull/471
Please pull the latest version and verify that everything is working as expected.


[openmpi] find_package(MPI) doesn't work correctly for debug configuration

Describe the bug
When trying to enable netcdf-c utilities with parallel hdf5 (work in progress), the following warning is issued to stderr:
CMake Warning at /home/dg0yt/Projekte/vcpkg/vcpkg/scripts/buildsystems/vcpkg.cmake:664 (_add_executable):
  Cannot generate a safe runtime search path for target ncgen because there
  is a cycle in the constraint graph:

    dir 0 is [/home/dg0yt/Projekte/vcpkg/vcpkg/installed/x64-linux/debug/lib]
      dir 1 must precede it due to runtime library [libmpi.so.40]
    dir 1 is [/home/dg0yt/Projekte/vcpkg/vcpkg/installed/x64-linux/lib]
      dir 0 must precede it due to runtime library [libmpi.so.40]

  Some of these libraries may not be found correctly.
Call Stack (most recent call first):
  ncgen/CMakeLists.txt:23 (ADD_EXECUTABLE)

I assume this is caused by the way how MPI is used with CMake. CMake's FindMPI.cmake relies on finding an MPIEXEC_EXECUTABLE via  find_program. But even for debug builds, CMAKE_PROGRAM_PATH doesn't guide cmake to look for the tool in tools/openmpi/debug. So the release tools are found and used, leading to the mix release and debug configuration.
Environment

OS: Linux

To Reproduce
$ ./vcpkg install hdf5[parallel]
$ cat buildtrees/hdf5/*-dbg/CMakeCache.txt  | grep MPIEXEC_EXECUTABLE:
MPIEXEC_EXECUTABLE:FILEPATH=/home/dg0yt/Projekte/vcpkg/vcpkg/installed/x64-linux/tools/openmpi/bin/mpiexec

Expected behavior
$ ./vcpkg install hdf5[parallel]
$ cat buildtrees/hdf5/*-dbg/CMakeCache.txt  | grep MPIEXEC_EXECUTABLE:
MPIEXEC_EXECUTABLE:FILEPATH=/home/dg0yt/Projekte/vcpkg/vcpkg/installed/x64-linux/tools/openmpi/debug/bin/mpiexec

Additional context
Package installation layout for tools #17607

For tools, I don’t think we need to use the debug version, because its performance is much lower, and it can’t help us to debug.

For tools, I don’t think we need to use the debug version, because its performance is much lower, and it can’t help us to debug.

I can agree for gettext tools, moc and similar. But there are tools which carry lib path and linkage information:

curl-config
geos-config
qmake

and probably this is true also for some or all MPI tools? FindMPI.cmake is non-trivial IIRC.

Hit me today with:
1> [CMake] CMake Warning at /home/me/development/vcpkg/scripts/buildsystems/vcpkg.cmake:596 (_add_library):
1> [CMake]   Cannot generate a safe runtime search path for target my_target because there
1> [CMake]   is a cycle in the constraint graph:
1> [CMake]
1> [CMake]     dir 0 is [/mnt/d/my_project/out/build/WSL-GCC-Debug/vcpkg_installed/x64-linux/debug/lib]
1> [CMake]       dir 1 must precede it due to runtime library [lib7zip.so]
1> [CMake]     dir 1 is [/mnt/d/my_project/out/build/WSL-GCC-Debug/vcpkg_installed/x64-linux/lib]
1> [CMake]       dir 0 must precede it due to runtime library [libQt5Core.so.5]
1> [CMake]
1> [CMake]   Some of these libraries may not be found correctly.
1> [CMake] Call Stack (most recent call first):
1> [CMake]   CMakeLists.txt:15 (add_library)


Suggestion: show currency name when it's not USD

If a transaction involves a non-USD account, the frontend currently uses $ in various views (dashboard, balance sheet, income statement).
I suggest to replace $ with the currency's code (e.g. EUR, CHF, etc.) when the currency is not USD.
At a later point, the backend and frontend code could be updated to allow the user to provide various metadata for a given currency (preferred thousand and decimal separator, optional currency code, etc.)

Thanks for the contribution Alok! This is been something I've been meaning to do for awhile but never got around to it. How do you feel about using currency symbols for USD, EUR and GBP ($, €, £) and then defaulting to your behavior otherwise?

I can update my PR to show the symbol for common ones. I'll pick the one-character symbols from https://en.wikipedia.org/wiki/Currency_symbol.

Thanks, that would be perfect.

Expect an updated PR by the end of next week.

Great idea using Intl.NumberFormat. I'll go ahead and merge your PR.


Timeline Visual Failing - Adjust Year Assignment Logic in the Code

Evidence:

Years in the start and end columns are being assigned as 4041 for December dates
The report date for December dates is always 2020, even if it was reported this year.

The logic in our code is no longer working. It relied on December dates being from 2020, but now that it's December 2021, we have a problem. This will also be a problem in January when we switch to 2022, so we need to fix the code.
Possible solution: keep the columns with parsed dates from previous days rather than re-processing the dates every day. Only process new dates.

I added some (hopefully) better year assignment logic for the report date that usually comes in with no year. We keep the data we processed in previous days and only process new data that comes in, allowing us to not have to deal with more than two years at a time.


Fix overflow edge case in overscrolled RenderShrinkWrappingViewport

This fixes a regression caused by https://github.com/flutter/flutter/pull/87143
The original change fixed #10949, by fixing the layoutOffset when the viewport is overscrolled. This change should have also updated the remainingPaintExtent though to account for edge cases like the one reported in https://github.com/flutter/flutter/issues/89717.
Fixes #89717
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Gold has detected about 1 new digest(s) on patchset 1.
View them at https://flutter-gold.skia.org/cl/github/90419

Gold has detected about 1 new digest(s) on patchset 2.
View them at https://flutter-gold.skia.org/cl/github/90419

Will this PR be merged into the stable branch？This morning I used the stable branch to update to version 2.5.2, but the #89717 still exists.


size() unsupported argument

Describe the bug
expected where size(v) to work.
How are you accessing AGE (Command line, driver, etc.)?

terminal

What data setup do we need to do?
SELECT * FROM cypher('graph',$$
    CREATE (n)-[:KNOWS {n:'hello'}]->({n:'hello'})
$$) as (a agtype);

What is the command that caused the error?
SELECT * FROM cypher('graph', $$
    MATCH (m)-[v *]->(b)
    WHERE size(v) >1
    RETURN b
$$) AS (a agtype);

ERROR:  size() unsupported argument

Expected behavior
Should have returned
 {"id":<PHONE_NUMBER>10658, "label": "", "properties": {"n": "hello"}}::vertex

Environment (please complete the following information):

latest (PG13 master)

Additional context
works when used in return
SELECT * FROM cypher('graph', $$
    MATCH (m)-[v *]->(b)
    RETURN size(v)
$$) AS (a agtype);

 a
---
 1
(1 row)

The condition beside the WHERE clause is > 1, this will return an error as the size is not greater than 1.


Timer passes reservoir only for Histogram and not to Meter, causes discrepancies

Issue
When constructing a Timer, it accepts a Reservoir. The docs say:

reservoir: the {@link Reservoir} implementation the timer should use.

But the reservoir is passed only to the Histogram. Meter is still using default reservoir. So if we use a SlidingTimeWindowReservoir, we get discrepancies with the histogram reporting sliding time window, but Meter reporting the default exponential decay.
Expectation
There is a way to achieve consistency between the histogram and meter – either both are using exponential decay, or both are using sliding time window.

Nice feature to have
I created a P.R for that https://github.com/dropwizard/metrics/pull/1517


Speed up page loading via caching name canonicalization

Page loading on our Gollum wiki is very slow. In part that's due to the large number of page and having a fair number of links per page.
Investigation with various profiling utilities  (e.g. ruby-prof) indicated that the majority of the time was spent in text processing, particularly in page and file name canonicalization. The same text normalization steps were being done for every page/file in the repository, for each link being rendered.
To fix this, I've taken the caching scheme from the GitAccess object and added it to Wiki object. In the Wiki object the appropriately pre-processed page and file names are now cached, on a per-SHA1 basis. This pre-processed data gets stored in a Hash, so the rendering of each link after the very first should be sped up significantly.
On my local machine, for my particular wiki of interest, launching Gollum, rendering 4 example pages and exiting out takes a total of ~22 CPU seconds ("user" time) with current master. Using the changes in this pull request, this is reduced to ~1.9 CPU seconds (a 10-fold speedup).

One issue I'm still a bit hazy on is the memory impact on long-running processes. I think I have set things up such that the cached values in the Wiki object get cleared out occasionally, but I'm not confident enough in how long-running Gollum systems work to say that the new scheme won't slowly accumulate memory as additional SHA1s get added.

Thanks for the PR! This sounds great. I'll have a closer look ASAP.
Meanwhile, I've also been working on improving performance in gollum itself, see my last three comments in https://github.com/gollum/gollum/issues/1078. Cumulatively, these changes should make significant dents in the load times. (Although only this PR addresses some of the fundamental issues in gollum-lib)

Very promising speedups! Ideally we also look into improving the underlying code, but caching does seem desirable here. Let's test this for inclusion in the 5.x release.

Just wanted to double check that there isn't anything I still needed to do in order to move this pull request forward.

Actually, it would be great if you could open this PR against the gollum-lib-5.x branch instead of master. Do you agree, @dometto ?
If you happen to have used this improvement in your own environment, do you have any more insights into how it might affect long-running Gollum instances?

Yes, agreed! Sorry for the much too slow response, and thanks for the great work. Let's discuss further when you reopen the PR.

@roccomoretti Could you please base this PR against the gollum-lib-5.x branch?

Ping @roccomoretti

Actually, in 5.x we've completely removed page name canonicalization as it's no longer necessary. That sort of obviates the need for this kind of caching. So the choice is whether to merge this into master or not at all.

Not at all in that case; only bugfixes to master at this point as far as I'm concerned.

@roccomoretti thanks for the excellent PR, but as you can see I'm afraid we should have come up with something like this a couple of years ago. :) At least when 5.x comes out you can have the benefit of much faster page loads.

@dometto That's perfectly fine. I unfortunately haven't been able to find the free time to give this the attention it needed. -- But as long as page loading got faster, I'm happy.


Update usage.md

Fixed incorrect command name / typo Get-XrmClient changes to New-XrmClient

Thanks @nrpieper! 👍


Precompiled shaders with opengl es 2.0

Hi.
I've found one possible issue with precompiled shaders and opengl es 2.0. Currently I'm using bgfx with define BGFX_CONFIG_RENDERER_OPENGLES set to 32. Problem occurs when code is run on some open gl es 2.0 device that supports OES_get_program_binary extension and I try to use glGetProgramBinary. If define BGFX_CONFIG_RENDERER_OPENGLES is set above 30, bgfx will try to setup glGetProgramBinary, but since this is opengl es 2.0 it will set the function pointer zero. On the other hand glGetProgramBinaryOES is present, but this function is only taken into account if BGFX_CONFIG_RENDERER_OPENGLES  is less than 30. Whats the best course of action. Thanks in advance.

When you explicitly set version like that, GLES should only run in 3.2 mode. So all GLES 3.2 functionality should be available.

Ok. I would still like to point out, that if this define is used on opengl es 2.0 device that supports precompiled shaders through extension, it will cause a crash.


Improve Estimand relationship to IntercurrentEvent and AnalysisPopulation

Based on the PHUSE CSS estimand meeting:

There can be multiple estimands defined for a study
an analysisPopulation can be reused for multiple estimands
an IntercurrentEvent can be reused for multiple estimands
However, the way they are nested in the API means that they have to be repeated.

Merged into #355 and closed


Where can I find what's changed when NSwagStudio updates?

Is there any document that shows what's been fixed / changed /added, etc. when NSwagStudio updates?

Please have a look at the commits before the vx.y.z commits:
https://github.com/RicoSuter/NSwag/commits/master


API version change

Looks like Terraform is gearing up for an API version change in the master branch. Following your installation instructions and then running terraform, I get an error:
Error configuring: 1 error(s) occurred:

* Incompatible API version with plugin. Plugin version: 3, Ours: 2

I was able to fix the problem by doing:
cd ~/go/src/github.com/hashicorp/terraform
git checkout v0.8.7
go build -o ~/go/bin/terraform-provider-localfile github.com/mcoffin/terraform-provider-localfile

Hope this helps.

@tradel I wasn't able to reproduce this locally, even with terraform from the master branch. How exactly are you compiling or otherwise getting the terraform binaries that are giving you problems?
If you upgraded terraform without re-compiling terraform-provider-localfile, then that would almost certainly cause the problem you were seeing, but I don't see why you would have to check out v0.8.7 of terraform in order to make this plugin still work.
Here's some output locally.
% ~/go/bin/terraform -version
Terraform v0.9.0-dev

% ~/go/bin/terraform apply
localfile_file.helloworld: Refreshing state... (ID: helloworld.txt)
localfile_file.helloworld: Creating...
  content: "" => "Hello, world!"
  path:    "" => "helloworld.txt"
localfile_file.helloworld: Creation complete

Apply complete! Resources: 1 added, 0 changed, 0 destroyed.

The state of your infrastructure has been saved to the path
below. This state is required to modify and destroy your
infrastructure, so keep it safe. To inspect the complete state
use the `terraform show` command.

State path: terraform.tfstate

Ahh, I see what happened here.
I made this project a while back, and since it was so simple, I didn't use godep or anything similar to version my dependencies (just terraform really).
As long as you build this plugin against terraform binaries that use the same version of the plugin API as what you plan to use when running terraform, you'll be fine.
So, in fact, what you did above is the correct solution to the problem!

@tradel I added some documentation to the README to hopefully help others with a similar problem in the future, but I'm going to leave the build system simple so that it's easy for others to perform these kinds of version upgrades on their own in a canonical manner.
Thanks for the help!

@mcoffin Not everyone wants to run the bleeding edge of terraform, which is what go get would give you.
Most package managers currently install Terraform 0.8.7, which is API v2, but by importing github.com/hashicorp/terraform, you're pulling in Terraform 0.9.0-dev (as your output above shows), which is API v3.


[10718] Add Fast DDS Statistics module documentation and API reference

In order for some aliases to work, #243 has to be applied.

Build status:

CI is failing because it is using the Fast DDS master branch.
Here the correct Fast DDS branch has been launched. It also fails because this branch does not include eProsima/Fast-DDS#1763. I have checked locally and the tests are passing correctly. Retest after merging rebasing branch feature/statistics/api to master.

Manual CI running against the correct branch of Fast DDS passes the tests. However, at the last stage fails because it does not know where to post the results.

Build status:

Build status:

Build status:

Build status:

Build status:

Build status:

Build status:


Add PLC Ultima (plcu) support

Context

Impacted projects: [live-desktop, @ledgerhq/live-common]
Linked resource(s): N/A

Checklist

[ ]  Test coverage: Did you write any tests to cover the changes introduced by this pull request?
[x]  Atomic delivery: Is this pull request standalone? In order words, does it depend on nothing else?
[x]  No breaking changes: Does this pull request contain breaking changes of any kind? If so, please explain why.

Demo
N/A
Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

@plcudeveloper  Can you please fill in the PR template? Otherwise this PR will get closed. Thanks 🙇.

@plcudeveloper Can you please fill in the PR template? Otherwise this PR will get closed. Thanks bow.

Done.
Thank you, Julien !

@elbywan
hi Julien,
the PR is assigned on me, should it be be re-assigned to someone from Ledger to proceed?
Thanks!

Hey @plcudeveloper,

the PR is assigned on me, should it be be re-assigned to someone from Ledger to proceed?

No need, our blockchain team will review the PR when possible 😉.

hi @haammar-ledger , could you please review the PR?
thank you!

Hello @haammar-ledger ,
Could you please review the PR or tell whom I can ask for that?
Thanks you!

Hi @plcudeveloper thank you for your contribution, but it appears the Nano application is still under review.
Closing this for now, we will contact you to reopen it once the application is reviewed.


Can't get w2ui up, strange issues

Short description
Cannot get w2ui started, downloaded from the website. Maybe related to jQuery?
What is current behavior

Steps to reproduce or sample
I'm using Flask for local hosting & this simplified setup:
app.js
import * as w2ui from './w2ui-1.5.js'
// [...]

index.html
<!DOCTYPE html>

        <link rel="stylesheet" href="css/w2ui-1.5.min.css">

if i import /w2ui-1.5.min.js instead, more weird behavior:

I hadn't had this happen with other libraries like highlight.js, why is this the case for this?

was mixing up old version, got new from dist/


Is it possible to reuse the same browser?

OpenTest version number: 1.1.0

Is it possible when creating a session that it can use the same browser that is already opened? Or reuse a browser that was recently used, or closed?
For example:
I want to automate my browser navigating to a web game and log me in. But whenever I get it to the page it has the "Flash Player is needed" screen so I have enable it everytime I create a new session.

The problem you're facing is due to the fact that the browser starts each new test run with a fresh, newly-created user profile. Basically, it resets all the user settings to the default ones every time. To avoid that, you can specify the directory where you have your user profile information. Not sure how to do this in other browsers, but here's how to do this in Chrome: https://github.com/mcdcorp/opentest/issues/70#issuecomment-493253703.

So after checking out the other post I have it added, but now when I run my script it may get stuck on the main page instead of "Navigating" to the page I have in the YAML code, or it will be stuck on a blank screen. Could that be due to the saved profile?

Yes, I think it's possible, since you are now storing the "state" of the browser across all tests. Maybe something got corrupted in the profile data. You could try deleting the contents of the profile folder and run the test again to see if it solves the issue. The browser will create a fresh profile again.


fix(deps): update dependency rxjs to v6.5.4

This PR contains the following updates:

Package
Type
Update
Change

rxjs
dependencies
patch
6.5.3 -> 6.5.4

Release Notes

reactivex/rxjs
v6.5.4
Compare Source

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Enabled.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: 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

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

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


Add alvie pubkeys

I previously used detached signatures, however I have pushed a change to use the same form of signed message as vertiond in their commit here faf3b0a3daa80a4ac159f2142064c5087bf8f34f


chore: ignore xamlmerge output from git

GitHub Issue: re: unoplatform/private#314
PR Type
What kind of change does this PR introduce?

Other... Please describe:

Description
Remove XamlMerge ouput files from git.
PR Checklist
Please check if your PR fulfills the following requirements:

Commits must be following the Conventional Commits specification
[ ] Tested UWP
[ ] Tested iOS
[ ] Tested Android
[ ] Tested WASM
[ ] Tested MacOS
[x] Contains No breaking changes

If the pull request contains breaking changes, commit message must contain a detailed description of the action to take for the consumer of this library. As explained by the Conventional Commits specification

Other information
These files are regenerated on each build and are often the source of git-merge conflict when style changes are involved.

the irony:


Add appId to extension endpoint

WHY are these changes introduced?
Fixes https://github.com/Shopify/checkout-web/issues/9350
We are working on enabling direct Storefront API access from within extensions.
In order to authenticate against SFAPI, we must supply a real api_clients id for further identification by SFAPI code.
This is id is obtained by the CLI after partners authorisation call.
WHAT is this pull request doing?
This PR is propagating the mentioned id to via the development server, to be accessible on /extension endpoint for checkout-web.
How to test your changes?
Generate an extension, run yarn dev and query /extension endpoint. You should see app.id entry on the returned JSON response.
Post-release steps
This PR requires a release, as our work [for extension development] depends on a published CLI version.
Measuring impact
How do we know this change was effective? Please choose one:

[x] n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix

Checklist

[x] I've considered possible cross-platform impacts (Mac, Linux, Windows)
[x] I've considered possible documentation changes
[x] I've made sure that any changes to dev or deploy have been reflected in the internal flowchart.

I'd get a review from @isaacroldan

I think I'm missing something here...
You defined the ID in the interface, but is not being set anywhere right? I 🎩 and can't see the ID value on my extensions.
The ID is a different thing from the API_key of the app right?

/shrimpit


frontpage: add TU Wien logo

Add it the frontpage of InvenioRDM and to https://inveniosoftware.org/products/rdm/
Ping Lars to get the logo.

I cannot link a pull request from other repository, so I will post here the pull request of the inveniosoftware.org part.
https://github.com/inveniosoftware/inveniosoftware.org/pull/156.


in Q._generateCollisionPoints, obj.p.points is undefined, which throws an error

Uncaught TypeError: Cannot read property 'length' of undefined
Q._generateCollisionPoints	@	quintus_sprites.js:312
update	@	quintus_sprites.js:676
(anonymous function)	@	my game
Q.gameLoopCallbackWrapper	@	quintus.js:591
requestAnimationFrame (async)
Quintus.Q.gameLoop	@	quintus.js:595
startGameLoop	@	my game
(anonymous function)	@	my game
loadedCallback	@	quintus.js:1892
img.onload	@	quintus.js:1623

I worked around this by calling _generatePoints manually after this._super in my Sprite's init method.
Q._generatePoints( this, true );

It's hacky though, so if there is a real solution, please let me know. Other than that one issue, I am enjoying this framework so far.


Fixed a bug in joining urls. urllib is the way to go to join urls.

I found this while I was contributing to https://github.com/akademikbilisim/ab-kurs-kayit
The bug was causing an url like this: /admin/userprofile/userprofile//foreignkey_autocomplete/ (both app and model name are the same, problem is the double slashes.)

Coverage increased (+0.01%) to 36.185% when pulling bd8d26a3c87a4ac101d7383b1ba3b7291034a6e8 on Cediddi:master into 2f5d7f2bbd78fc7b86f6685f8f610aeeb3e4213b on django-extensions:master.

Thanks !

Thank you for being extremely fast (13 minutes!) :heart:

new release should be out shortly :)


Release/0.26.0 ONLY CI

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: lasote:white_check_mark: uilianries:x: YourYour seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #374 into master will increase coverage by 0.05%.
The diff coverage is 96.15%.

@@            Coverage Diff             @@
##           master     #374      +/-   ##
==========================================
+ Coverage   92.39%   92.45%   +0.05%
==========================================
  Files          32       32
  Lines        3012     3034      +22
==========================================
+ Hits         2783     2805      +22
  Misses        229      229

Impacted Files
Coverage Δ

cpt/run_in_docker.py
0% <ø> (ø)
:arrow_up:

cpt/test/integration/docker_test.py
88.23% <100%> (+1.93%)
:arrow_up:

cpt/test/integration/base.py
97.29% <100%> (+0.23%)
:arrow_up:

cpt/packager.py
86.36% <100%> (+0.03%)
:arrow_up:

cpt/__init__.py
100% <100%> (ø)
:arrow_up:

cpt/runner.py
90.73% <88.88%> (-0.14%)
:arrow_down:

cpt/ci_manager.py
93.45% <0%> (-0.47%)
:arrow_down:

Continue to review full report at Codecov.

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


Add min.c

Why:
Closes #66
Check off the following:

[x] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Hey @luiscff
Thanks for contributing to this open source project 😃
This PR will be reviewed and merged shortly

Preview : https://codinasion--pr36-dev-igku8tg2.web.app/programme/minimum-of-numbers/
(expires Tue, 29 Mar 2022 08:11:27 GMT)

LIVE : https://codinasion.web.app/programme/minimum-of-numbers/


Make TypeScript Compilable to Native Machine Code!

🔍 Search Terms
native, machine, binary, aot, ahead-of-time
✅ Viability Checklist

[X] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[X] This wouldn't change the runtime behavior of existing JavaScript code
[X] This could be implemented without emitting different JS based on the types of the expressions
[X] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[X] This isn't a request to add a new utility type: https://github.com/microsoft/TypeScript/wiki/No-New-Utility-Types
[X] This feature would agree with the rest of our Design Goals: https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals

⭐ Suggestion
ReasonML, a language compiled to javascript can also be compiled to native binary. I hope TypeScript could also be compiled to native binary like ReasonML.
📃 Motivating Example
Adding the capability to compile to native binary, in addition to replacing and complementing JavaScript as it currently does, could encourage developers to use TypeScript for a wider range of purposes.
💻 Use Cases
Adding the capability to compile to native binary, in addition to replacing and complementing JavaScript as it currently does, could encourage developers to use TypeScript for a wider range of purposes.

This is out of scope. You checked the point with the "Design Goals", I suggest you to read those design goals again.
Same issue as #27362 and others.

Ok I will .,
Thx a lot
في سبت، 8 يونيو، 2024 في 7:47 م، كتب Martin Johns @.***

:

This is out of scope. You checked the point with the "Design Goals", I
suggest you to read those design goals again.
Same issue as #27362
https://github.com/microsoft/TypeScript/issues/27362 and others.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/TypeScript/issues/58808#issuecomment-2156081943,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A55MQ3UQH5MPMDWZQTWGV73ZGMRRRAVCNFSM6AAAAABJAB3FI2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJWGA4DCOJUGM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@AlQubaiisi maybe this will help? https://www.assemblyscript.org/

See also #375, etc..


Submit app to TestFlight on behalf of our users. Any way to avoid password prompts?

Hello!
We are trying to give our users the ability to be able to push their apps to both TestFlight and Google Play. (In other words, using Fastlane to push apps that are created by other users on our platform.) We are doing this by asking them to give us access from Apple Store Connect API. This is all working, except when we try to send the build to TestFlight, it asks for Appstore username and password.
Is there a way to avoid that prompt?  We are able to solve all other issues like certificate, profile with the p12 file we create.
Thanks for the help!

We are doing this by asking them to give us access from Apple Store Connect API.

What does this mean exactly? How are you supplying the authentication information?


Linking variables w/different varkeys

@bqpd I was wondering what your thoughts on the below are. Hopefully you can understand what I'm trying to describe...if not I can try to clarify any or all of it. It's possible I'm just not understanding what is happening in linked.py, if this is the case I apologize in advance.
I'm trying to plan out linking together my engine and the discretized flight profile models, as well as eventually some of philippe's aircraft models. I'm looking over the linked.py file and I'm a little concerned with how the linking is going to be performed. I need to create an engine model for each flight segment. One model will be an on design engine model and the rest will be off design. The vast majority of the variables in each on and off design model share the same name, since they're constructed from the same submodels. This in and of itself doesn't seem like an issue, just put all these variables into the exclude only list. However, I also need to link the ambient pressure in each engine model with the pressure from the flight profile model, which is a vector variable.
Of course, I only want to link each engine ambient pressure with the piece (not sure what the proper word for this is) of the vector variable corresponding to the atmosphere's pressure for that flight segment.
It seems to me, that since the variables will be linked based on the varkeys with .str_without(["models"]) applied there is not a way to perform the linking above. I think the best I can do is link every instance of engine ambient pressure and flight profile ambient pressure. This is because all the engine ambient pressures will link to each other, since their varkey with .str_without(["models"])  will be identical. This is in turn will link all of the atmosphere pressures to each other.
Also, I don't see a way to link variables with different varkeys. Is there a way to do this?

You're in luck! First, you don't need exclude_only if you use Named Models; the variables with the same name will have different models attributes and therefore not collide.
Second, you can link between things with different names manually; LinkedConstraintSets are just a convenient automation. Do this by making a constraintset with the various models you want to link in it, and then doing constraints.subinplace({x1: y, x2: y, x3: z}) or similar.

ah okay linked.py won't drop the _model off the varkey of a variable in a named model? Also, is named model a class (I couldn't find it in the code base just now)?
Sub in place makes a lot of sense! sorry for all the questions, thanks for your help!

Nope! and the class is NamedConstraintSet, but Model inherits from it, and if you inherit from Model then you get this by default

No worries about the questions! You're finding the parts of the linking interface that are still just (udocumented) drafts.

ah okay that makes sense thanks!

@bqpd when using the subinplace method of linking the x1, y, etc. from your example are varkeys, correct?

Or variables, doesn't matter. :)

or a mix of both...just figured out a minute ago that works which is really quite nice!

I talked this over with @whoburg this morning. However, I still can't get the linking to work and I think I know why, although I'm not sure what to do to fix the issue. The piece of code where I attempt to perform the substitution is below. TSFC is  a vector variable of length 6.
self.submodels = [cmc, climb1, climb2, cruise2, eonD]
constraints = ConstraintSet([self.submodels])
constraints.subinplace({TSFC[4]: TSFCe})
lc = LinkedConstraintSet(constraints)

I believe the problem is that I'm calling sub in place on something that isn't a list. For example if I run this code
constraints = ConstraintSet([self.submodels])
 print TSFC[4]
print TSFCe
print constraints
constraints.subinplace({TSFC[4]: TSFCe})
print constraints
lc = LinkedConstraintSet(constraints)

I get the following output
TSFC_(4,) [1/hr]
TSFC_E [1/hr]

        CommericalMissionConstraints_0
        Climb1_0
        Climb2_0
        Cruise2_0
        EngineOnDesign_0

        CommericalMissionConstraints_0
        Climb1_0
        Climb2_0
        Cruise2_0
        EngineOnDesign_0

so the variables print as expected but constraints isn't a list. How should I be creating the submodels so that constraints becomes a list? @bqpd thoughts?
Also, the code that I'm running can be found at https://github.com/hoburg/gpkit-models/pull/73/commits/9a944cadcb549f6b5b5a25eef1f2e9d29b969d43

That really should work; the constraints are always nested lists, they just print sometimes in a "collapsed" form.

Hmm I don't think it is because I'm getting a warning that TSFC[4] is unbounded. I am assigning values to all the other TSFC variables and linking TSFC[4] to the engine model. If I link all the TSFC variables to the engine then the model solves, so this method should work.

When I try to run that file I get ImportError: No module named engine_components

Is engine_components not added to the repository yet?

Sorry, it wasn't. just pushed it with this commit https://github.com/hoburg/gpkit-models/pull/73/commits/17751baa290c932b68986658b56db9c86dccefc7

Also, the model eonD is itself composed of a few submodels that live in engine.py. Not sure if that has anything to do with what's happening.

Was there any ideas on what was wrong here? Getting these variables linked is my biggest hold up for this Aurora model right now

I think part or all of the problem has to do with the named submodels Climb1, Cruise2, etc. When these models get created gpkit.Model appends their name to the TSFC varkeys.
In [15]: m["TSFC"]
Out[15]:
[gpkit.Variable(TSFC_Climb1, CommercialAircraft_(0,) [1/hr]),
 gpkit.Variable(TSFC_Climb1, CommercialAircraft_(1,) [1/hr]),
 gpkit.Variable(TSFC_Climb2, CommercialAircraft_(2,) [1/hr]),
 gpkit.Variable(TSFC_Climb2, CommercialAircraft_(3,) [1/hr]),
 gpkit.Variable(TSFC_Cruise2, CommercialAircraft_(4,) [1/hr]),
 gpkit.Variable(TSFC_Cruise2, CommercialAircraft_(5,) [1/hr])]

Note above you can access the string "TSFC", but you can't similarly use the varkey TSFC[4] -- because that doesn't collide with the varkeys in the named models:
In [29]: m[TSFC[4]]
Out[29]: []

@bqpd will probably weigh in on the design decisions and global vector variables. For now some options to consider are making the submodels ConstraintSets instead of Models (note this will cause all same-named variables to collide), or using full string access, e.g. constraints.subinplace({"TSFC_Cruise2_(4,)": TSFCe}).
@mayork, note you currently have two Variables named TSFC_E.

Ah yes that makes sense!
@whoburg just to make sure I haven't missed something you're referring to the TSFC_E declared in flight profile and the second instance in the engine_components.py, right? I created the instance in engine_components.py as an experiment to try and better understand what was happening

yep, the one in FanAndLPC

okay that's what I thought, thanks!

@mayork, let's meet up sometime after wednesday next week and go over vecvars :D

@bqpd sounds good I'll be gone  next week but when I'm back we should meet up

After making everything into constraint sets and TSFC into a series of local vector variables I get the same error...

Hmm, that does seem odd.
It seems the code has moved along since it no longer contains the line constraints.subinplace({'TSFC_Cruise2_(4,)': 'TSFC_E'}). Is there a commit hash where this can be debugged? Instructions on what file to run or what object to create would also be helpful.

Yeah sorry...ended up moving on without linking which is suboptimal, but it lets me work on engine integration. I believe this commit should demonstrate what I was seeing https://github.com/hoburg/gpkit-models/pull/73/commits/9eea9e55f6bd2623f7815dad427ad4de9a4d519f....All the code needed to run should be on git, just run the file Commerical_Flight_Profile.py

the problem linking vector variables is a duplicate of #822 so I'm going to close this isssue


cannot run

please update this app, thank you very much!

sorry, this project is deprecated
On May 28, 2017, 1:40 PM +0800, linleo10<EMAIL_ADDRESS>wrote:

please update this app, thank you very much!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub (https://github.com/stormluke/Mili-iOS/issues/4), or mute the thread (https://github.com/notifications/unsubscribe-auth/AAsFniWSlnH6xshjloorh2tCwU-P3loiks5r-Qi6gaJpZM4NojkW).


feat: cuda acceleration for PQ builds/assignments

Currently if an accelerator is used, it's only used for IVF training and assignments. This PR extends it to also run on PQ training & assignments.
I benchmarked on a gcloud n1-standard-16 instance with an attached nvidia T4, using the wikipedia dataset with 50 in-sample queries (so qps will be a bit noisy).
Before:

After:

There's some noise due to randomness, but these plots can be considered to be essentially the same, except for the faster build time.
Update: I've verified that there are no regressions from the latest changes.

some style nits, mostly looking good. Let's create a ticket to create recall regression test in CI.

https://github.com/lancedb/lance/issues/2982


Safely delete ClusterDeployments

We're deleting ClusterDeployments from two different controllers: clusterpool and clusterdeployment. This means there's the possibility of timing problems whereby e.g.

controller 1 loads a CD and decides it needs to be deleted
controller 2 loads the same CD and updates it such that it should no longer be deleted
controller 1, having already decided to delete, deletes

This commit adds a SafeDelete utility that asks the server to check the ResourceVersion of the object being deleted, and fail with a Conflict error if it's out of date. It then changes all deletions of ClusterDeployments to use that new method.

/hold
for testing

Looks good but could you elaborate on the details of the two controllers that might do this? Would would controller 1 (I'm assuming clusterdeployment_controller) delete a CD that controller 2 (presumably clusterclaim_controller) be grabbing?

could you elaborate

Yeah, this is more or less hypothetical at this point. We don't need to merge it necessarily, but I wanted to get it working.
It's not even just about two controllers -- theoretically an outside agent (like ACM :) ) could race with one controller.
IMO Delete should check ResourceVersion by default like Update does; and the consumer should have to "force" explicitly in the cases where we're sure we don't care about the state of the object we're deleting. With SafeDelete we could implement this policy throughout hive if we wanted to.

Actually I think what led me to this was the race between { clusterclaim_controller Updating the cd to assign a claim } and { clusterpool_controller Deleting excess CDs }. I originally thought that was the cause of the behavior @jnpacker was seeing.

Ahh I see. Sounds good.

Rebased this to pick up the controller-runtime update, which allowed me to fix the unit test.

/lgtm

/hold cancel


remove reflect deep equal in endpoint comparison

Removes reflect.DeepEqual in frequently used endpoint comparison
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[X ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

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

Test name
Commit
Details
Rerun command

lint_istio
182464db2cd150f5e623ff32a31c4b36376d39bd
link
/test lint_istio

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

There are already tests that validate compare endpoints behaviour.


Const cleanup

[X] finish moving all defined constant values into separate module
[X] remove deprecation warnings for constant values
[X] bundle values into enum's for ease of use
[X] refactor existing code to use enum's
[X] update documentation for the use of enums

This should make the code much more readable and make it more maintainable. In some cases this will have the effect of breaking compatibility because of the use of enums instead of plain values but the necessary changes should be acceptable.


Fix encode_content_disposition - replace all "+" with " ", not just the first one

encode_content_disposition was using sub instead of gsub, so only the first + was being replaced.
I had a test PDF called "Demo PDF - Google Docs". When I opened the S3 URL in Chrome and pressed CMD+s to save the PDF, the filename was set to: "Demo PDF+-+Google+Docs". This PR fixes the issue, by replacing all of the + characters with spaces.

@ndbroadbent Good catch, thank you for fixing this!


Allow civiform admins see applications. flag guarded

Description
Added new config value: allow_civiform_admin_access_programs. When set to true Civifrom admin will be able to see applications for all programs.
Applications are accessible through 3 dot menu on the Programs page.

If the setting is turned off later Civiform Admins will lose access to Programs.
Release notes:
Describe the change as it should be communicated to partners in the release notes.
Checklist

[ ] Added the correct label: < feature | bug | dependencies | infrastructure | ignore-for-release >
[ ] Created tests which fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

Issue(s)
Fixes #1593

Codecov Report

Merging #3475 (5461fd6) into main (415bac8) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main    #3475      +/-   ##
============================================
+ Coverage     80.57%   80.60%   +0.02%
- Complexity     2388     2391       +3
============================================
  Files           291      291
  Lines          9463     9466       +3
  Branches        643      643
============================================
+ Hits           7625     7630       +5
+ Misses         1590     1588       -2
  Partials        248      248

Impacted Files
Coverage Δ

server/app/auth/CiviFormProfile.java
100.00% <100.00%> (ø)

server/app/auth/ProfileFactory.java
72.22% <100.00%> (+2.41%)
:arrow_up:

server/app/controllers/CiviFormController.java
100.00% <100.00%> (ø)

server/app/featureflags/FeatureFlags.java
95.00% <100.00%> (+0.26%)
:arrow_up:

server/app/models/Account.java
87.17% <0.00%> (+2.56%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add CI jobs that run tests using LDC AddressSanitizer

Closes https://github.com/symmetryinvestments/concurrency/issues/29.

I don't understand why the
Run asan tests linux
job is skipped at
https://github.com/symmetryinvestments/concurrency/pull/33/checks?check_run_id=3869610457


Spoiler message can be read if they are listed in the notifications

If a user A submits a comment, and user B tracks the topic (e.g. a kata where user B has commented already), user B will get a notification with an excerpt, even if user A marked the comment as spoiler.

Yeah this was intentional. We figured its not so common that it was worth trying to add additional security around that flow (we already have so many complicated flows caused by security reasons). Generally it should be within context. If you post a root level comment, you will automatically track comments for that kata but only for a few days, then you become auto-unsubcribed to any additional notifications.

Ok, this seems reasonable. Just checking whether that behaviour was intended.


Mismatch in the number of properties that are part of the relationship between 'PerformanceTest.Models.Apm' (0 properties) and 'PerformanceTest.Models.Bpm' (1 properties)

I have two sample entities:
[Dapper.Table("Apms")]
public class Apm
{
    [Key]
    [DatabaseGenerated(DatabaseGeneratedOption.Identity)]
    public int Id { get; set; }
    public string Name { get; set; }
    public string BigText { get; set; }
    public DateTime? DateOfBirth { get; set; }
    public decimal Height { get; set; }
    public float Weight { get; set; }

    public IEnumerable<Bpm> Bpms { get; set; }
}

[Dapper.Table("Bpms")]
public class Bpm
{
    [Key]
    [DatabaseGenerated(DatabaseGeneratedOption.Identity)]
    public int Id { get; set; }
    public string Name { get; set; }
    public string BigText { get; set; }
    public DateTime? DateOfBirth { get; set; }
    public decimal Height { get; set; }
    public float Weight { get; set; }
    [ForeignKey("Apm")]
    public int? ApmId { get; set; }
    public Apm Apm { get; set; }
}

But when using the code below:
           var b = connection.Find<Apm>(statement => statement
                 .Include<Bpm>(join => join.LeftOuterJoin()))
                 .ToList();

I get the error:
Mismatch in the number of properties that are part of the relationship between 'PerformanceTest.Models.Apm' (0 properties) and 'PerformanceTest.Models.Bpm' (1 properties)
Can anyone help me to identify the problem?

Yeah same issue here...

Did you solve the problem? I have the same issue here!

Solution : I had a different situation, (2 properties) to (1 property) just make sure SetPrimary(false) for the ID column is set on that entity having 2 properties. We had a Guid as PK and an auto-incremental ID column too but I forgot to exclude ID from PKs. You may have a different situation but check your db table for PK cols and set the config accurately.


Incorrect inputTooShort, inputTooLong, maximumSelected for "fr"

In "fr" localization a few items have the same bug (e.g. for inputTooShort):
return "Saisissez au moins " + t + " caractère" + (t > 1) ? "s" : ""
will return "s"
the fix:
return "Saisissez au moins " + t + " caractère" + ((t > 1) ? "s" : "")
See http://jsbin.com/bohoteqesa/1/edit?html,js,output

click on select to see inputTooShort issue
type 5 characters or more to see inputTooLong issue

This has already been fixed; see #5122.
For future reference, you MUST confirm your problem in the latest version of Select2 before opening an issue.


Removed constraints that are unnecessary due to the usage of GADTs.

Since we are using GADTs, we can use Data.Vector.Generic instances after matching the Matrix constructor. This makes it possible to remove the Vector constraints on all the functions that take a Matrix as an argument.
I am submitting an alternative pull request (#2) that keeps the constraints and removes the GADT entirely.

@andersjel You are right! I think it's better to keep GADTs, and remove constraints, so that the Matrix type is well defined.

I made a small bencmark. GADTs seem to hurt performance here quite a lot.
Maybe it gets in the way of inlining and fusion.
>>> git checkout withGADTs
>>> cabal clean
>>> cabal configure -O2 --enable-benchmarks
>>> cabal build
>>> dist/build/benchmarks/benchmarks
benchmarking takeDiag largeMatrix
time                 5.820 μs   (5.735 μs .. 5.907 μs)
                     0.998 R²   (0.998 R² .. 0.999 R²)
mean                 5.901 μs   (5.832 μs .. 5.964 μs)
std dev              221.2 ns   (201.2 ns .. 253.7 ns)
variance introduced by outliers: 48% (moderately inflated)

>>> git checkout withoutGADTs
>>> cabal clean
>>> cabal configure -O2 --enable-benchmarks
>>> cabal build
>>> dist/build/benchmarks/benchmarks
benchmarking takeDiag largeMatrix
time                 1.065 μs   (1.041 μs .. 1.099 μs)
                     0.995 R²   (0.985 R² .. 0.999 R²)
mean                 1.065 μs   (1.052 μs .. 1.102 μs)
std dev              62.00 ns   (31.61 ns .. 117.8 ns)
variance introduced by outliers: 73% (severely inflated)

OK, I will try to reproduce this on my side later. I did notice that the performance of some functions is not as good as I expected before. But I didn't think GADT is the problem. If that's the case, we need to fix it.

I've confirmed that the GADT indeed hurts performance. So I merged #2 . Thank you!


add titles and favicon to all pages

Added Titles & Favicon for browser tabs on all the pages
fixes #5

@ee33 Updated this PR and removed formatting changes, please check.


requirements: Update psycopg2 to latest version

Fixes #4580

Automated message from Dropbox CLA bot
@sinwar, it looks like you've already signed the Dropbox CLA. Thanks!

Please remember to bump PROVISION_VERSION as part of this change.

@sinwar remember to post a comment when you push an updated version!  Otherwise reviewers may not know to look again.

I'll take care from now. I thought mentors use needs review label to maintain PRs.

@sinwar the "needs review" label helps aesthetically, but mentors don't know when you have updated your PR. (Also you are missing a period in your 2nd commit.)

@timabbott the PR is up for review.

I moved the PROVISION_VERSION update to the second commit, as it should be, and merged this.  Thanks @sinwar!!


Add support for Binary and Decimal encodings

Decimal serialization (outside of write_decimal_bytes) is currently only supported when the alloc flag is enabled, as the encoded length is potentially unbounded for Uint<L> and digits are produced in reverse. In practice anything up to U8192 can be encoded without allocating. Note that unlike the other encodings, decimal encoding is not constant-time.
Unresolved questions:

Add a trait for encoding numbers as decimal, to match FromDecimal? This is a bit tricky as it would require the alloc feature (or gain additional methods when the feature is enabled).
Implement Error for ParseDecimalError? This would require a std flag.
Add an error type for encoding errors?

Sorry for taking awhile to review this. The implementation is somewhat complex which makes it difficult, but I also have some initial thoughts.
The existing support for hex encoding is implemented in a constant-timeish manner, avoiding branches on the inputs in favor of arithmetic operations.
This implementation appears to branch on the input in several places. That's okay, but the corresponding functions should be marked as *_vartime to reflect that, per this crate's constant-time-by-default policy.

Closing as stale


Issue with ubuntu apt repo on Azure Pipelines hosted ubuntu-latest (xenial)

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Describe the bug
As of today I am receiving:
Get:37 https://packages.microsoft.com/repos/azure-cli xenial/main amd64 Packages [39.5 kB]
Err:37 https://packages.microsoft.com/repos/azure-cli xenial/main amd64 Packages
  Writing more data than expected (40278 > 39457)

When trying to install a random package on an "ubuntu-latest" Microsoft hosted Azure pipeline host (e.g. "sudo apt update && sudo apt install qemu"). Not sure if this is an issue with the repo or with the Azure pipeline host itself.
To Reproduce
sudo apt update && sudo apt install qemu (also after apt-get clean)
Expected behavior
Installing a package
Environment summary
apt using the official microsoft ubuntu repo (see above)

Get it for xenial (16.04) and bionic (18.04) as well. There was an update to the packages just some hours ago, before it worked for me.

@fengzhou-msft Please take a look.

We're also having the same error over here on xenial (16.04)
2020-01-07T08:25:44.9161012Z Err:61 https://packages.microsoft.com/repos/azure-cli xenial/main amd64 Packages
2020-01-07T08:25:44.9161683Z   Writing more data than expected (40278 > 39457)
2020-01-07T08:25:51.4358553Z E: Failed to fetch https://packages.microsoft.com/repos/azure-cli/dists/xenial/main/binary-amd64/Packages  Writing more data than expected (40278 > 39457)
2020-01-07T08:25:51.4360631Z E: Some index files failed to download. They have been ignored, or old ones used instead.

Same in my Github Action:
https://github.com/baruchiro/israeli-bank-scrapers-desktop/runs/377085755

Seeing the same in my project.

We'are contacting internal teams to resolve this issue. Please download the package and install it locally for temp fix:
$curl -O https://packages.microsoft.com/repos/azure-cli/pool/main/a/azure-cli/azure-cli_2.0.79-1~xenial_all.deb
$apt install ./azure-cli_2.0.79-1~xenial_all.deb

The repo gets updated correctly and it should work now.

The repo gets updated correctly and it should work now.

Still failed:
https://github.com/baruchiro/israeli-bank-scrapers-desktop/runs/377185914#step:4:43

The repo gets updated correctly and it should work now.

Still failed:
https://github.com/baruchiro/israeli-bank-scrapers-desktop/runs/377185914#step:4:43

bionic is also fixed. Please try again.

Confirmed for xenial at least. I assume you will also fix the root cause to prevent further occurrences, but thats not for this issue of course 👍
Thanks for the help!


Remove Discount codes from Dashboard and WebApp

Description

This PR solves # .
Steps completed:

According to our definition of done, I have completed the following steps:

[ ] Acceptance criteria met
[ ] Unit tests added
[ ] Docs updated (including config and env variables)
[ ] Translations added
[ ] UX tested
[ ] Browsers / platforms tested
[ ] Rebased & ready to merge without conflicts
[ ] Reviewed own code

Why is this feature removed? Customers depend on this and it is still supported by both integrations. I suggest that we remove this when we have an alternative.

@darkoatanasovski @borkopetrovicc
Let's keep this feature in:

We are still rolling out InPlayer 'standalone' implementations
We don't have an alternative yet

We should still hide coupons in the dashboard.

Why is this feature removed? Customers depend on this and it is still supported by both integrations. I suggest that we remove this when we have an alternative.

This is part of the initiative to remove features that we will not be supporting at the moment, and this applies only to the JWP integration.


Add hello-world exercise

Hello! I wanted to learn purescript, but alas, there is no exercise course. I figure I may as well try and build one while learning.

The PureScript code looks good to me 👍

Thanks :D

@kytrinyx CI is passing but I'm not sure if all of the config files here are correct. The code looks good though.

I think there should be a script that makes sure the examples pass the tests, but I don't think it's been written yet.

@lpil Yeah, it would be great to get something that tests the tests.
@paf31 The config looks good to me. I'll merge this and deploy the tracks and you should be able to test it with
exercism fetch purescript hello-world

Great, thank you. :)
I guess I'll keep going!


Range and NumericRange

This PR is a rebased version of #11.
I also tried to make Range extend NumericRange[Int] but the compiler was crashing with a StackOverflow when I wanted to override the by(newStep: T): NumericRange[T] method for range (by(newStep: Int): Range).
I @specialized(Unit) the foreach method in both Range and NumericRange but this didn’t seem to have any measurable effect on NumericRange.

Superseded by #102


ingest issues for kafka messaging

Affected module
Ingestion Framework
Describe the bug
we have kafka ingest in airflow and data sync every 1h but on any config changes in the topic data is not getting updated into openmetadata automatically.
To Reproduce

Expected behavior
on any changes data has to be patched automatically from the Ingestion
Version:

OS: openmetadata docker image for k8s
Python version: openmetadata docker image for k8s
OpenMetadata version: 13.2 latest
OpenMetadata Ingestion package version:

Additional context
API works fine to use update or patch and i am asking from the airflow ingest scheduler its not updated on changes in the server config of it. ex: when  Partitions change to 5 from server data is not updating in OM until we delete topic name or service itself.

Also with reference to the document https://docs.open-metadata.org/connectors/messaging/kafka/cli  markDeletedTopics not working as well.
i am using python to run directly from the airflow pod which is deployed from the OM image itself to run ingest ..
airflow@openmetadata-dependencies-web-64749b789f-v6r99:/opt/airflow$ which python3
/usr/local/bin/python3
airflow@openmetadata-dependencies-web-64749b789f-v6r99:/opt/airflow$ metadata ingest -c pp.yaml
[2023-04-04 02:29:33] INFO     {metadata.Utils:logger:165} - Error loading INGEST configuration: We encountered an error parsing the configuration of your MessagingServiceMetadataPipeline.
You might need to review your config based on the original cause of this failure:
- Extra parameter 'markDeletedTopics'
[2023-04-04 02:29:33] INFO     {metadata.Utils:logger:165} -
Make sure you are following the following format e.g. 'kafka':
[2023-04-04 02:29:33] INFO     {metadata.Utils:logger:165} - ------------
[2023-04-04 02:29:33] INFO     {metadata.Utils:logger:165} - source:
type: kafka
serviceName: local_kafka
serviceConnection:
config:
type: Kafka
bootstrapServers: localhost:9092
schemaRegistryURL: http://localhost:8081
consumerConfig: {}
schemaRegistryConfig: {}
sourceConfig:
config:
type: MessagingMetadata
topicFilterPattern:
excludes:
- _confluent.*
generateSampleData: true
sink:
type: metadata-rest
config: {}
workflowConfig:
openMetadataServerConfig:
hostPort: http://localhost:8585/api
authProvider: openmetadata
securityConfig:
"jwtToken": "eyJraWQiOiJHYjM4OWEtOWY3Ni1nZGpzLWE5MmotMDI0MmJrOTQzNTYiLCJ0eXAiOiJKV1QiLCJhbGciOiJSUzI1NiJ9.eyJzdWIiOiJhZG1pbiIsImlzQm90IjpmYWxzZSwiaXNzIjoib3Blbi1tZXRhZGF0YS5vcmciLCJpYXQiOjE2NjM5Mzg0NjIsImVtYWlsIjoiYWRtaW5Ab3Blbm1ldGFkYXRhLm9yZyJ9.tS8um_5DKu7HgzGBzS1VTA5uUjKWOCU0B_j08WXBiEC0mr0zNREkqVfwFDD-d24HlNEbrqioLsBuFRiwIWKc1m_ZlVQbG7P36RUxhuv2vbSp80FKyNM-Tj93FDzq91jsyNmsQhyNv_fNr3TXfzzSPjHt8Go0FMMP66weoKMgW2PbXlhVKwEuXUHyakLLzewm9UMeQaEiRzhiTMU3UkLXcKbYEJJvfNFcLwSl9W8JCO_l0Yj3ud-qt_nQYEZwqW6u5nfdQllN133iikV4fM5QZsMCnm8Rq1mvLR0y9bmJiD7fwM1tmJ791TUWqmKaTnP49U493VanKpUAfzIiOiIbhg"
[2023-04-04 02:29:33] INFO     {metadata.Utils:logger:165} - ------------
[2023-04-04 02:29:33] INFO     {metadata.Utils:logger:165} -
For more information, please visit: https://docs.open-metadata.org/connectors/ingestion/workflows/metadata
Or join us in Slack: https://slack.open-metadata.org/
airflow@openmetadata-dependencies-web-64749b789f-v6r99:/opt/airflow$

any updates ?

markDeletedTopics works as expected

topicConfig is now supported in UI. All of your topicConfigs are visible including updated ones in 1.0
markDeletedTopics works as expected in my testing
cc @chethan-bs


Could DBeaver please stay focused???

Is your feature request related to a problem? Please describe.
When using the session manager, selecting a PID and activating auto-refresh, DBeaver loses the focus on the selected line.
This is frustrating, as in the SQL pane below it would nicely show which statement is currently run, and would allow for observing at a distance what a current session is currently at. Since it loses focus on refresh, that SQL pane gets emptied on refresh, and you've to click again on the pid.
Describe the solution you'd like
A clear and concise description of what you want to happen.
With a PID selected, on refresh, keep focus on that PID as long as that PID is there.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
None.
Additional context
Add any other context or screenshots about the feature request here.

Automatically reselect the current row after a refresh would indeed be nice. It probably requires a per-database support. On Oracle, for example, you would need SID+serial# (+ instanceid when in RAC mode) to identify the row.

Verified

This fix breaks in the situation when querying sessions takes a somewhat significant time and we change the selection after the query has started.

Fixed once again.

Verified


Fix typo, add tests for Hurley::Test

Fixes a little :bug: I ran into when converting Sawyer over to Hurley.
/cc @technoweenie

I must've made that typo very recently. I know I've seen those "no handler found" results in some of my tests.


Correct punctuation on Excel.Application.OnUndo.md

Replace question marks with apostrophes.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/Excel.Application.OnUndo.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add verifyContract address normalization

Explanation

This PR aims to add address normalization for EIP712Domain.verifyContract
References

Fixes: https://github.com/MetaMask/MetaMask-planning/issues/2229
Changelog

@metamask/message-manager

FIXED: Support adding octal and decimal values for EIP712Domain.verifyContract by normalizing the signature value

Checklist

[ ] I've updated the test suite for new or updated code as appropriate
[ ] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
[ ] I've highlighted breaking changes using the "BREAKING" category above as appropriate

@metamaskbot publish-preview

@metamaskbot publish-preview

https://github.com/MetaMask/eth-json-rpc-middleware/pull/309


Database: delete old statuses from cache

This doesn’t need to be done in 1.0.

Duplicate of #70.


Multimedia?

Excuse if I'm asking a too elementary question, I don't know much of Markdown. Can be audio and video files be embedded using Markdown-here? AFAIK images can, but what about voice notes, GIF or MNG animations, or videos? With Evernote you can do all this, but I think they use HTML; well, could the same results be achieved with Markdown to compose mails or notes with multimedia content?
Regards.

The short answer is: links and images are fine, but maybe not other types of media. MDH is mainly geared towards writing Markdown in mail clients, so if you want anything other than images (including GIFs) you may just be stuck using links or attachments.
Markdown is basically a convenient notation that relate to commonly-used HTML (lists, blockquotes, etc.). If you write HTML and run it through a Markdown parser, the general expectation is that it returns the HTML. In the case of Markdown Here, you need to write the HTML text in your editor first and THEN use MDH to convert it so that it renders properly. If you have a method of embedding video (Youtube embeds) or audio (links?), you write them as raw text and then use MDH to put them in as HTML. For instance, if you wrote the following:
![some gif](http://i.imgur.com/7wjjB5a.gif)

...you'd get this:

GIFs should work for MDH. For video on Youtube, using HTML on a web page you could do this:
<iframe src="//www.youtube-nocookie.com/embed/uzjFEMmM0Xs?rel=0" frameborder="0" allowfullscreen></iframe>

But iframe usage is dependent on the platform you're using. I used that embed on a post I wrote for my personal website, but Github and Gmail don't allow iframes for security reasons below. When I paste that snippet in this post, nothing gets rendered.

Likewise, when I try to paste an iframe into Gmail and use MDH to render, it seemingly disappears because Gmail refuses to render them for the same reason. For those cases, I would usually just use a link like:
http://www.youtube-nocookie.com/embed/uzjFEMmM0Xs?rel=0

You mentioned Evernote -- Evernote has its own storage and method of presenting rich media like audio and video. Unfortunately, that is outside of the scope of Markdown. If you can't write it in HTML on your mail clients, then you probably can't do it with MDH.


fix: [Approach 2] Watchdog does not shut down executor on client closing if the executor is provided by ExecutorProvider.

This approach has fewer code changes, but it makes shouldAutoClose not the only source of truth for us to decide should we close the Watchdog(and the executor) or not, it is now implicitly decided by shouldAutoClose and needExecutor in ClientContext. It may cause some confusion if a customer decides to implement their own WatchdogProvider.

Closing in favor of https://github.com/googleapis/gax-java/pull/1890 and https://github.com/googleapis/gax-java/pull/1884


Decision on broad geographic regions

Confirm with Fedor the decision on broad geographic regions
Compile list of broad geographic regions (depends on availability of Justin)

The details are in ticket #71


Implements hd keys for xpub/xprv

Implements BIP32 and adds some basic wallet helpers.

Codecov Report

Merging #210 (0a044af) into main (6bcfa10) will decrease coverage by 0.03%.
The diff coverage is 91.49%.

@@            Coverage Diff             @@
##             main     #210      +/-   ##
==========================================
- Coverage   93.62%   93.59%   -0.04%
==========================================
  Files         183      192       +9
  Lines        4815     5198     +383
  Branches      568      613      +45
==========================================
+ Hits         4508     4865     +357
- Misses        171      188      +17
- Partials      136      145       +9

Impacted Files
Coverage Δ

packages/crypto/lib/key.ts
94.44% <50.00%> (-5.56%)
:arrow_down:

packages/bitcoin/lib/BitcoinError.ts
79.31% <73.68%> (+5.97%)
:arrow_up:

packages/bitcoin/lib/HdPublicKey.ts
78.94% <78.94%> (ø)

packages/bitcoin/lib/HdPrivateKey.ts
88.31% <88.31%> (ø)

packages/bitcoin/lib/HdKeyCodec.ts
90.32% <90.32%> (ø)

packages/bitcoin/lib/Network.ts
93.33% <93.33%> (ø)

packages/bitcoin/lib/Address.ts
94.44% <94.44%> (ø)

packages/bitcoin/lib/PublicKey.ts
95.83% <95.83%> (ø)

packages/bitcoin/lib/Bech32.ts
100.00% <100.00%> (ø)

packages/bitcoin/lib/BitcoinErrorCode.ts
100.00% <100.00%> (ø)

... and 5 more

Continue to review full report at Codecov.

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


Port 80 is not available on Windows

Any suggestion on getting pass the following port issue?  I'm running on Windows 10.
KinD version is kind v0.10.0 go1.15.7 windows/amd64
Creating cluster "knative" ...
• Ensuring node image (kindest/node:v1.20.2) �  ...
✓ Ensuring node image (kindest/node:v1.20.2) �
• Preparing nodes �   ...
✗ Preparing nodes �
ERROR: failed to create cluster: docker run error: command "docker run --hostname knative-control-plane --name knative-control-plane --label io.x-k8s.kind.role=control-plane --privileged --security-opt seccomp=unconfined --security-opt apparmor=unconfined --tmpfs /tmp --tmpfs /run --volume /var --volume /lib/modules:/lib/modules:ro --detach --tty --label io.x-k8s.kind.cluster=knative --net kind --restart=on-failure:1 --publish=<IP_ADDRESS>:80:31080/TCP --publish=<IP_ADDRESS>:50865:6443/TCP kindest/node:v1.20.2@sha256:8f7ea6e7642c0da54f04a7ee10431549c0257315b3a634f6ef2fecaaedb19bab" failed with error: exit status 125
Command Output: 58bba70d289885b015187855f6b3bfbb107b746946a9dbcf5825d85250eb8e54
docker: Error response from daemon: Ports are not available: listen tcp <IP_ADDRESS>:80: bind: An attempt was made to access a socket in a way forbidden by its access permissions.

I don't have windows to test this on.
But I'm guessing that port 80 is not allowed since is a privilege port.
You could try asking in channel kind in kubernetes Slack to see if there are any windows users that could help answer how to get port 80 to work on windows host. so you can have a nice URL http://hello.default.<IP_ADDRESS>.nip.io
This is the kind config I use you can paste in slack
cat <<EOF | kind create cluster --name knative --config=-
kind: Cluster
apiVersion: kind.x-k8s.io/v1alpha4
nodes:
- role: control-plane
  image: kindest/node:1.20.2@sha256:8f7ea6e7642c0da54f04a7ee10431549c0257315b3a634f6ef2fecaaedb19bab
  extraPortMappings:
  - containerPort: 31080 # expose port 31380 of the node to port 80 on the host, later to be use by kourier ingress
    hostPort: 80
EOF

you can also put the same info in a git issue in kind git repository as a question

@zachariahyoung I got some tips from the developers of kind to bind only to address <IP_ADDRESS> should solve the problem, give it another try.


Fix example in 'Component callbacks' section

About this pull request
Fix example in 'Component callbacks' section
Changes
The function in the example had a parameter called button_context, but ctx was used instead inside the defined function, so I replaced button_context with ctx.
Checklist

[ ] I've run the pre_push.py script to format and lint code.
[ ] I've checked this pull request runs on Python 3.6.X.
[ ] This fixes something in Issues.

Issue:

[ ] This adds something new.
[ ] There is/are breaking change(s).
[ ] (If required) Relevant documentation has been updated/added.
[x] This is not a code change. (README, docs, etc.)

Merging, thanks for contributing


Fix split function results with whitespace separator

closes: #4639
Fix the split function results when the separator is whitespace.
split('token1 token2 token3', ' ')

returns ["token1", "token2", "token3"]
split('token1 token2 token3', '  ')

returns ["token1 token2 token3"]
split('token one', '')

returns ["t", "o", "k", "e", "n", " ", "o", "n", "e"]

Can you also investigate in which PR this regression is introduced?


Update for Go 1.11.1 release

@karalabe I think the Docker image needs an update for the new Go 1.11.1 release. I've cross-compiled for some plugins I've built and they cannot be loaded because the container image uses the Go 1.11 release while my plugins use Go 1.11.1 toolchain.
Please can you upload a new image. 👍

Fixed, merged and images rebuilt :)

Thanks @karalabe 🌟


Ability to configure/perform sparse checkout

It's a pain to have to setup & maintain additional specific repositories for bower releases while they only often are just the contents of the build/dist folder.
Would be great if somehow we could configure our component in bower.json to specify which folder should be checked-out, for instance:
"sparse": ["dist"]

On install, bower would perform a git sparse checkout of the selected paths.

2 years later and this is still relevant. You as consumer wan't to decide what you want, and you as Author want to give them that freedom. I have for example an asset library that has a dist, and my consumers either want to use the LESS/SASS format or they just want CSS format that is in dist, they usally never want both. And now for me give this freedom of choice to users I have to have one repo with originals and one with dist ... So, this is still a very relevant issue as a propsal for better user experience.


Missing manage.py

Test snippet in README suggests to run
python manage.py test --settings "tests.settings"

But there is no file named manage.py in the repo. I think either manage.py should be added, or README should be edited
Originally posted by @belkka in https://github.com/fabiocaccamo/django-extra-settings/issues/4#issuecomment-642990944

This is a third-party library, you must use it in a django project that always has a manage.py file.

Is it supposed that one need to copy-paste files from the root of this repo to existing django project in order to run tests? Actually, django project does not necessary has a manage.py file or it can be edited to conform projects needs.
Looks like
python -m django --settings "tests.settings"

does the job. This is equivalent to running manage.py generated by default and it does not require creating a django project.

@belkka it should be python -m django test --settings "tests.settings" ... anyway thank you for the suggestion, I will update the README.


Add support for opal.dev terraform import

I added support for https://registry.terraform.io/providers/opalsecurity/opal/latest.
I also added support for specifying a source in the required_providers configuration to avoid errors in terraform 1.2.9. See that commit message for details.

I've signed the CLA. @sergeylanzman could you unblock the github action run and ptal?
(I have already verified both actions in our fork: https://github.com/opalsecurity/terraformer/actions)

@sergeylanzman any updates here? I also signed the CLA. Would appreciate if you could take a look here.

@sergeylanzman would you be able to take a look here pls?


Updating Sculpin Skeleton instructions to use Composer installation

Given the reasons outlined in the Deprecating Phar blog post, it seems like we need to update our installation procedures and examples to use Sculpin. This is the first step in that process.

Been running from composer for years, will compared with my set up later. But LGTM :+1:

I should have looked at this first.  I've just provided a pull request that brings the update to composer more in-line with the original skeleton app. (IE> component installation.)
See #44


Implement encryption / decryption for all libp2p crypto keys

Currently, we're only supporting ed25519 keys.

Related to #474

Related to #474


why handle data-* and aria-* as properties

data-* and aria-* should be handled as attributes rather than have a special case for them as properties.
https://github.com/nthtran/vdom-to-html/blob/fe04172917f596dc4b24a4b0dc20c747c9454144/test/test.js#L58-L64 is currently
  it('should render `data-` and `aria-` properties', function () {
    var node = new VNode('web-component', {
      'data-click': 'clicked!',
      'aria-labelledby': 'label'
    });
    assert.equal(toHTML(node), '<web-component data-click="clicked!" aria-labelledby="label"></web-component>');
  });

these shouldn't need to be handled as a special case in properties because it is inconsistent with what virtual-dom does and so any rendering code which uses these as attributes will not produce the same results in the browser.  as attributes these don't need any special handling.
  it('should render `data-` and `aria-` attributes', function () {
    var node = new VNode('web-component', {
      attributes: {
        'data-click': 'clicked!',
        'aria-labelledby': 'label'
      }
    });
    assert.equal(toHTML(node), '<web-component data-click="clicked!" aria-labelledby="label"></web-component>');
  });

is this something you want to continue to support or are you open to dropping them?

I am ok with dropping it, seems like we should use attributes anyway and it works.
ref: https://github.com/Matt-Esch/virtual-dom/blob/master/docs/vnode.md#custom-attributes-data-

Agreed. Support for these is dropped in v2.0.0.


Add new user preferences, remove transitional preference migration stuff.

What's New?

Remove transitional user preference migration function (transferred local storage options to the API while we made the transition to server-side user options)
Add a user preference to disable the "Unsaved changes" dialogs when exiting the app.

Coverage increased (+0.02%) to 88.156% when pulling b32eaebe24b38b529183988b3d61d130d2c719c4 on RickCarlino:no_unsaved_changes into 2a4da7ee535678d9b824275b6b2dd6e82415f982 on FarmBot:staging.