PARQUET-416: C++11 compilation, code reorg, libparquet and installation targets

Reorganize code into a top level src/parquet directly, add a libparquet shared library, and add install targets for libparquet and its header files. Add cpplint script and make lint target for code linting.
Replaces earlier PR #13

OS X is failing with a homebrew error on brew install thrift; trying work around for now, if not we can disable OS X for the time being.

I disabled OS X in the Travis CI setup for now; the .travis.yml will need some more twiddling which can go in a separate patch. For some reason brew install thrift was failing with a checksum error

I tried this patch on osx by cloning and following the build steps in the readme. It worked on master but not this patch.
{code}
cmake .
...
Make Warning (dev):
Policy CMP0042 is not set: MACOSX_RPATH is enabled by default.  Run "cmake
--help-policy CMP0042" for policy details.  Use the cmake_policy command to
set the policy and suppress this warning.
MACOSX_RPATH is not specified for the following targets:
parquet
This warning is for project developers.  Use -Wno-dev to suppress it.
{code}
{code}
make
[ 50%] Linking CXX shared library build/debug/libparquet.dylib
Undefined symbols for architecture x86_64:
"snappy::RawCompress(char const*, unsigned long, char*, unsigned long*)", referenced from:
parquet_cpp::SnappyCodec::Compress(int, unsigned char const*, int, unsigned char*) in libparquet_compression.a(snappy-codec.cc.o)
"snappy::RawUncompress(char const*, unsigned long, char*)", referenced from:
parquet_cpp::SnappyCodec::Decompress(int, unsigned char const*, int, unsigned char*) in libparquet_compression.a(snappy-codec.cc.o)
"snappy::MaxCompressedLength(unsigned long)", referenced from:
parquet_cpp::SnappyCodec::MaxCompressedLen(int, unsigned char const*) in libparquet_compression.a(snappy-codec.cc.o)
"typeinfo for apache::thrift::transport::TMemoryBuffer", referenced from:
boost::shared_ptrapache::thrift::transport::TMemoryBuffer boost::dynamic_pointer_cast<apache::thrift::transport::TMemoryBuffer, apache::thrift::transport::TTransport>(boost::shared_ptrapache::thrift::transport::TTransport const&) in parquet.cc.o
"typeinfo for apache::thrift::transport::TTransportException", referenced from:
apache::thrift::transport::TMemoryBuffer::TMemoryBuffer(unsigned char*, unsigned int, apache::thrift::transport::TMemoryBuffer::MemoryPolicy) in parquet.cc.o
apache::thrift::transport::TTransport::open() in parquet.cc.o
apache::thrift::transport::TTransport::close() in parquet.cc.o
apache::thrift::transport::TTransport::read_virt(unsigned char*, unsigned int) in parquet.cc.o
apache::thrift::transport::TTransport::write_virt(unsigned char const*, unsigned int) in parquet.cc.o
apache::thrift::transport::TTransport::consume_virt(unsigned int) in parquet.cc.o
unsigned int apache::thrift::transport::readAllapache::thrift::transport::TTransport(apache::thrift::transport::TTransport&, unsigned char*, unsigned int) in parquet.cc.o
...
"vtable for apache::thrift::transport::TMemoryBuffer", referenced from:
apache::thrift::transport::TMemoryBuffer::TMemoryBuffer(unsigned char*, unsigned int, apache::thrift::transport::TMemoryBuffer::MemoryPolicy) in parquet.cc.o
NOTE: a missing vtable usually means the first non-inline virtual member function has no definition.
"vtable for apache::thrift::transport::TTransportException", referenced from:
apache::thrift::transport::TTransportException::TTransportException(apache::thrift::transport::TTransportException::TTransportExceptionType, std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator > const&) in parquet.cc.o
NOTE: a missing vtable usually means the first non-inline virtual member function has no definition.
ld: symbol(s) not found for architecture x86_64
{code}

Dynamic linking on OS X is a nightmare. With the fix I put in ./debug/decode-benchmark works and dynamically links to libparquet.dylib properly.

As soon as we get the build environment question sorted out it would be good to use the same linkage as libparquet for the dependencies that have both static and shared libraries available.

Cool, I tried the latest version in your branch and it builds.

@wesm can you fix the build issue. I think you just need to add -fPIC

This looks good to me. Can you file a jira and prefix the commit so the title is
PARQUET-XXX: description?

Ready for cherry-picking. I updated the PARQUET-416 title and description. Thanks!

@wesm is disabling osx still necessary?

I can try the build again to see if the Homebrew thrift error cleared. I also could not get Thrift to build from source on Travis yesterday

Nvm. I got confused. It's not the osx issues that that been fixed.
Thanks. I'll merge this later today.

@grundprinzip sure; I'm seeing if OS X travis build will pass now and then will squash all these again

Let me know when this is no longer in flux and I'lll rebase on top of it again with my pull request.

@kalaxy I got gcc-4.9 working and got the OS X build passing again (the magic trick was to call brew update before using brew install >:|). @nongli this should be good to cherry pick if the build is passing

@wesm Thanks! Can you not squash in the future? It makes it hard to review the deltas and the merge tool we use will squash them.

Sure, no problem.


Move email/notification sending from callbacks to explicit methods

This PR moves callback-triggered email/notification sending to explicit methods.
Pull request checklist

[x] My code contains tests covering the code I modified
[x] I linted and tested the project with bin/check
[ ] I added significant changes and product updates to the changelog

@joemasilotti Sorry I got waylaid by other stuff and didn't get back with that extra feedback. But I just quickly skimmed through your latest changes in this PR, looks really great! So clean & readable :+1:


Added python3 to Monte Carlo Integration

I believe the code is incorrect. If you pick your random numbers between 0 and radius, you don't need to renormalize by the radius later on, you only need to do so if you pick between 0 and 1.
Overall, I don't like having this radius parameter laying around, everyone is getting confused about it and it doesn't add anything except for making the in_circle function more general.

If the only goal is to approximate pi, multiplying with radius squared is indeed unnecessary. My thought was  that the example for monte carlo integration should contain a function to determine the area of the circle, rather than just pi. Also the use of the radius parameter is to stay consistent with the Julia code already present.

I prefer the radius to be in the code (but it ultimately doesn't matter). Having the radius variable makes it clearer what the formula is supposed to be and doesn't look like a bunch of magic numbers.
the in_circle(...) function is also there so people see that the code is easily extended to other shapes and objects.

@jonasvandervennet No, you misunderstand, beyond the question of using radius, your code is incorrect. I just ran it with a radius of 0.1 and I got
percent error: 98.999539%
Estimation for pi: 0.031430

It's a terrible approximation ^^
You can fix it by either replacing random.uniform(0,radius) by random.uniform(0,1)  (but in this case, you can only pick radiuses smaller than 1, and if you do you will lose precision, BTW @leios your Julia code has the same pitfall) or changing return 4*(count/n)*radius**2 by return 4*(count/n) (but then why even bother with a free parameter radius?).
My suggestion: keep the parameter radius in in_circle, but don't include it in monte_carlo as a free parameter (and you always call in_circle with radius 1). @leios, you should do the same.

hey @jonasvandervennet I apologize for this. The initial julia code was confusing. I had a discussion with @jiegillet and he convinced me to re-write the code without the radius parameter in #159.
If it is easier to use that code as a base, please feel free to do that. Sorry again for the confusion!

I've submitted a pull request to @jonasvandervennet fork, mainly fixing readability issues.

Actually, I just checked the PR from @GuyPozner . It seems to make all the necessary updates. If you merge that PR into your own fork @jonasvandervennet , we should be mostly good to go here.

Dear @jonasvandervennet, you've submitted your PR about 8 days ago, but haven't responded to our reviews for 7 days. In the mean time 3 new PRs were submitted for Monte Carlo in Python, we've turned them all down, even though they were closer to a finished product than your PR. I even believe on PR was submitted to your branch.
We'd like to add the algorithm on the AAA, so please show some sign of life within 24h of this comment. If you don't reply, we will move forward with someone else's PR. Cheers.

Time's up.


fix heater becoming unresponsive after turning off (#222)

Mode 'off' seems to be illegal (at least on the latest firmware for DR-HSH004S). Judging by incoming state, official app is changing 'poweron' without touching 'mode'.
fixes #222

@AlexLemminG  - Not sure why I can't kick off the checks to approve.  That something you need to do?

Looks like actions got into "awaiting approval" state now, after I into my branch


fix: use installed solc version

Motivation
use the installed solc bin

Solution

PR Checklist

[ ] Added Tests
[ ] Added Documentation
[ ] Breaking changes

the path is wrong. that test is getting removed anyway in #2279


chore: Added github issue templates

This will add some issue templates to make it easier when submitting issues

Thank you for this @kencochrane !


Operand fusion in MobileNet causing segfaults on ARM CPU

Describe the bug
Compiling MobileNetV2 with operand fusion generates modules that causes segfault when running after dfc2f7dcd810fc6e84b3a33552004845231bee98.
To Reproduce

Download input mhlo model from https://storage.googleapis.com/iree-model-artifacts/MobileNetV2-b0c5c584.tar.gz.
Compile with iree/tools/iree-translate -iree-input-type=mhlo -iree-mlir-to-vm-bytecode-module -iree-hal-target-backends=dylib-llvm-aot --iree-llvm-target-triple=aarch64-none-linux-android29 -iree-flow-inline-constants-max-byte-length=2048 -iree-flow-dispatch-formation-enable-operand-fusion --iree-llvm-loop-unrolling=true -o compiled.vmfb /path/to/MobileNetV2.
Run iree-translate --flagfile=/path/to/flagfile with the following flagfile content

--driver=dylib
--module_file=compiled.vmfb
--entry_function=call
--function_input=1x224x224x3xf32
--task_topology_group_count=1

Additional context
(Not sure operand fusion is the root cause though; but at least it reveals the problem. Maybe related to #5857 or not.)

I am not familiar with operand fusion. @antiagainst Does it create extra memory allocations when fusing ? if Yes, we should disable it for CPU especially dispatching CPU workgroups is very cheap and aggressive fusion while creating extra stack allocations won't have any performance benefits It might be worse.

+1 for disabling operand fusion on CPU. It is not intended to create extra stack allocation, but it needs to be plumbed through properly to cover all cases.

It was already disabled via #6089. I haven't looked into details why; I suspect it's triggering segfault due to other reasons. Also failing on GPU (#6086); hopefully after fixing that it will be more clear.

This is superceded by #6973. Closing.


Camera first tab

as enabled by default camera tab to display first?

i don't think so , but I successfully change the first mode  to camera   by adding this line
in FusumaViewController at the end off viewDidLoad
changeMode(Mode.Camera)

@resand Yes, as @trollblocks said, you can change the first tab to display by changeMode(Mode.Camera). However, you cannot change the order yet.


Issue when installing package

This get this error
sger/go-apns2/client.go:31: undefined: http2.ConfigureTransport

when I was installing your packge.
I'm using version 1.6.

@bachvtuan make sure you have this package golang.org/x/net/http2 and of course Go 1.6

I have already installed the package golang.org/x/net/http2.
I found the reason now. It's out of date so I removed it then install again.


Entering new search should show loading indicator

On the search screen, when you first enter a search, we show a loading indicator in the form of a spinning circle centered in the area where the results will go.
We do the same when you enter a new search and you're currently seeing a "no results" screen because your previous search had no results.
But when you're looking at a nonempty list of search results, and then you enter some other query, we keep showing you the old results until the new results come in; and we don't show any kind of indication that we're working on the search. We should show such an indication.

The simplest way to do this is probably to blank out the old results and show the circular loading indicator, just as we do when the old results were empty.
(An ideal design would probably keep the old results visible until the new ones were ready. One way to do this could be the Material linear progress indicator, which can go at the top of the results (at the bottom of the search bar) without displacing the results in the layout. But that is probably more work, since we don't currently use that design. Letting the results go blank for a moment is fine.)
Originally reported by @AkashDhiman at https://github.com/zulip/zulip-mobile/pull/5052#discussion_r1095171167 .

I'm sorry, I'm not sure about the design guidelines but a better way to handle this would be to have a translucent overlay above the existing results with a centre aligned loading indicator. The same overlay can be shown initially as well.
Something like this:

Let me know if you like this idea or your initial idea of clearing the previous results and then showing the new results and I can start working on this.

Thanks for the suggestion @manancodes ! That sounds like a good direction, but because of ongoing work on prototyping a Flutter mobile client to potentially replace the React Native apps (see here for details), it unfortunately won't make sense for us to work on this issue at this tie.


DDL CREATE FUNCTION: Casting type as empty set causes InternalServerError

Error Message:

InternalServerError: there is no range var for (__derived__::expr~6) source in <pg.SelectStmt at 0x7efbfbfa67c0>
Hint: This is most likely a bug in EdgeDB. Please consider opening an issue ticket...

EdgeDB Version: 1.0a2-2020012220~buster
OS Version: Debian 10

Steps to Reproduce:
0. Create the database and module.
CREATE DATABASE name;

\c name;
...
CREATE MODULE api;

Create the schema (shown at the bottom).
Add the following DDL functions. The error occurs after attempting to run CREATE FUNCTION api::LoginCredentialSession.

WITH MODULE api
CREATE FUNCTION validate_credentials(EMAIL: str, PASS: str) -> SET OF api::User
USING EdgeQL $$
    WITH MODULE api
    SELECT
        User
    FILTER
        .email = EMAIL
        AND
        .password = PASS
    LIMIT 1
$$;

WITH MODULE api
CREATE FUNCTION random_big_str_id() -> str
USING EdgeQL $$
    SELECT
        std::re_replace(
            r'^[ \t]+',
            r'',
            (
                SELECT std::to_str(
                    random() * 10 ^ 256,
                    '999999999999999999999\
<PHONE_NUMBER>99999999999\
<PHONE_NUMBER>99999999999\
<PHONE_NUMBER>99999999999\
<PHONE_NUMBER>99999999999\
<PHONE_NUMBER>99999999999\
<PHONE_NUMBER>99999999999\
<PHONE_NUMBER>99999999999\
<PHONE_NUMBER>99999999999\
<PHONE_NUMBER>99999999999\
<PHONE_NUMBER>99999999999\
<PHONE_NUMBER>99999999999\
                     99999'
                )
            )
        )
$$;

WITH MODULE api
CREATE FUNCTION create_new_session(EMAIL: str, PASS: str) -> api::Session
USING EdgeQL $$
    WITH MODULE api
    INSERT Session {
        token := <str>(SELECT api::random_big_str_id()),
        sessionID := <str>(SELECT api::random_big_str_id()),
        user := (SELECT api::validate_credentials(EMAIL, PASS) LIMIT 1)
    }
$$;
WITH MODULE api
CREATE FUNCTION LoginCredentialSession(EMAIL: str, PASS: str) -> SET OF api::Session
USING EdgeQL $$
    # First case to split on is whether the user
    # authenticates correctly:
    WITH MODULE api
    SELECT (
        SELECT Session
        FILTER .sessionID = (
            SELECT (SELECT api::create_new_session(EMAIL, PASS))
            ).sessionID
        )
    )
    IF
    (
        SELECT api::validate_credentials(EMAIL, PASS)) ?!= <Session>{}
    )
    ELSE <Session>{}
$$;

Schema:
CREATE MIGRATION %s TO {
    module api {
        type Session {
            property token -> str;
            required link user -> User;
            required property sessionID -> str;
            required property createdAt -> datetime {
                default := datetime_current();
            }
            required property allottedDuration -> duration {
                default := <duration>'24 hours';
            }
        }
        type User {
            required property name -> str;
            property isDevice -> bool {
                default := False
            }
            link device -> Device;
            multi link devices -> Device;
            property phone_1 -> str {
                default := '';
            }
            property phone_2 -> str {
                default := '';
            }
            property email -> str;
            property storageDirectory := ('/' ++ __source__.email ++ '/');
            property _storageDirectory := ('/ware/blob' ++ __source__.storageDirectory);
            property password -> str;
            property isAdmin -> bool {
                default := False;
            }
            required property isGuest -> bool {
                default := False;
            }
            multi link files -> File;
            multi link widgets -> Widget;
        }
        type Device {
            property device_name -> str;
            property ipv4_address -> str;
            property ipv6_address -> str;
            property mac_address -> str;
            property online -> bool;
        }
        type Widget {
            required property name -> str;
            property path -> str;
            property executable_name -> str;
            property indexHtmlPath -> str;
            property runCommand -> str;
        }
        type File {
            required property filename -> str;
            required property path -> str;
            required property readable -> bool {
                default := True;
            }
            required property writeable -> bool {
                default := True;
            }
            required property executable -> bool {
                default := False;
            }
            required property isDirectory -> bool {
                default := False;
            }
            required property isPublic -> bool {
                default := False;
            }
            required property isDownloadable -> bool {
                default := True;
            }
            property uploadedBy -> str;
            property uploadedAt -> datetime {
                default := datetime_current();
            }
            required property temporary -> bool {
                default := False;
            }
        }
    }
};
COMMIT MIGRATION %s;

This actually reproduces on a much simpler query (not related to an empty set cast).
WITH MODULE api
SELECT (SELECT api::create_new_session("foo", "foo")).sessionID;

Also, I your approach with INSERT in a function would not work, because in Postgres all operations are considered to be executing at the same time, so your SELECT in LoginCredentialSession would not be able to see the effect of the INSERT statement at all.
Normally, when you spell INSERT directly in a query (and not in a called function), we compile the INSERT with a RETURNING clause and then UNION all relations in subsequent results, so the effects are visible.  Since function bodies are compiled separately and are considered a black box, we can't do that anymore.
Unfortunately, in this case, using INSERT directly in an IF would not work either, because conditional inserts are prohibited (this is another Postgres quirk), so if you really want to do auth checks and session creation in a single statement, I can only suggest the following solution:
Add a sessions link from User to Session and remove the user link from Session (essentially spelling the same relationship in reverse), and then do a conditional UPDATE:
UPDATE User
FILTER User = api::validate_credentials(EMAIL, PASS)
SET {
    sessions := User.sessions UNION (INSERT Session ...)
}

I think we may need to consider adding some syntax to allow conditional DML.

Actually, there's a way to do this using a FOR statement. Here's the solution:
WITH MODULE api
FOR user IN {api::validate_credentials(EMAIL, PASSWORD)}
 UNION (
     INSERT Session { user := user, sessionID := ... }
);

Thank you for all of the smart pointers tips, @elprans. I was wondering, does the statement FOR user IN {api::validate_credentials(EMAIL, PASSWORD)} create a new array with elements of type 'A', create a set where the only element is an array of type 'A', or something else entirely for the {...} in the statement FOR user IN {...}?

Actually, there's a way to do this using a FOR statement. Here's the solution:
WITH MODULE api
FOR user IN {api::validate_credentials(EMAIL, PASSWORD)}
 UNION (
     INSERT Session { user := user, sessionID := ... }
);

By the way, I appreciate the elegance and simplicity of this block because it trims away about half of the original SQL expressions, and leaves behind a more idiomatic and highly-performing operation. Impressive!

The syntax is FOR variable IN { <set> }, where <set> is an arbitrary expression of type T, and variable would hold a singleton value of the type T on every "iteration". The curly brackets in this case are needed for grammar disambiguation.


ollama: 0.0.14 -> 0.0.15

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/jmorganca/ollama/releases.
meta.description for ollama is: Get up and running with large language models locally
meta.homepage for ollama is: https://github.com/jmorganca/ollama
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.0.15 with grep in /nix/store/dlhfaj0b5vsh6vfnwwfl1ny414svd7yw-ollama-0.0.15
found 0.0.15 in filename of file in /nix/store/dlhfaj0b5vsh6vfnwwfl1ny414svd7yw-ollama-0.0.15

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
ollama

Instructions to test this update (click to expand)

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

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/dlhfaj0b5vsh6vfnwwfl1ny414svd7yw-ollama-0.0.15
ls -la /nix/store/dlhfaj0b5vsh6vfnwwfl1ny414svd7yw-ollama-0.0.15/bin

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

1 package built:

ollama

Maintainer pings
cc @dit7ya for testing.

Result of nixpkgs-review pr 250480 run on x86_64-linux 1

1 package built:

ollama


Scenario selection is lost when changing the language

New function
Set Scenario to Data Processing
Set language to C#

Result: scenario resets to Core, even though C# has Data Processing templates.

dd435c940ed8e2e73f6b3d66d70bdd245932ca48


[BUG] Retrieving non-existing entity results in internal server error

Describe the bug
Retrieving an entity (e.g. GET /results/{uid}) when the entity doesn't exists in the database returns a 500 internal error response.
To Reproduce
Steps to reproduce the behavior:
call GET /results/{uid} with a non-existing result uid
Expected behavior
The response should carry a 404 not found status
Additional context

Should also check endpoints other than /results
The problem seems to be due to this:
https://github.com/mlcommons/medperf/blob/06d42d0ab6c03200942fd1076108a70904692248/server/result/views.py#L52-L55
causing this error:
TypeError: exceptions must derive from BaseException

raise None was the actual cause, not raise Http404. Being tackled in #286
`


update: president's message in page.tsx

Can you add different paragraphs inside  tags? I see that you have added various paragraphs in the message, but they are displayed in a single one.
Thanks.

Thanks, @erwinschrodinger1! I'll merge the PR now.


I cannot playtest my godot projects

Godot version:
3.1.1
OS/device including version:
Manjaro, also tested on Windows
both have the same problem
Issue description:
i am working on a game and want to playtest it
My laptop can load the project, export, and everything else
except for playtesting
It gives me an error that my video card drivers are outdated and do not support gles 2, but they do
Steps to reproduce:
It seems to be just on my laptop
Minimal reproduction project:
any project is the same

This may be caused by the renderer selection dropdown not persisting the change to the project.godot file correctly.
Try changing Rendering > Quality > Driver > Driver Name to GLES2 in the Project Settings.

i'll try that

That worked
OMG
Thank you


Show output in pager

Hi, thanks for this project. Great idea. :)
I've changed the print statements in the event functions to return the string instead of printing it, so the output then is displayed in a pager where one can scroll up and down while reading the feed.

Thank you so much @albalitz :+1:


sh: 1: aprs: not found

Hey! Thought I'd give this a go, but I'm encountering the following error when I run watch-aprs /dev/ttyUSB0 9600.
sh: 1: aprs: not found
I have a TNC attached via USB.
Running on the below versions.
$ npm -v && nodejs -v
6.1.0
v10.3.0

I didn't encounter any errors when installing. Am I missing anything?
Thanks!

That’s kind of funny shell error.  It seems to be saying it can’t find a command or file called ‘aprs’. It’s also funny that your ‘node’ compiler seems to be called ‘nodejs’ rather than ‘node’. What’s the operating system and is it a standard Node installation (i.e. the download files from ‘nodejs.org’) or is it an operating system package?
I’ll confess I happen to be running Node 8.9.4 on the MacBook that I do most of my development on, so I’ve never tested with the 10.x series, but there really shouldn’t be any problems - The thing that usually changes between versions is the V8 JavaScript engine’s version, and the error your seeing doesn’t look like it’s coming out of the V8 engine.
Could you copy/paste the full terminal output of the command, including the command line that you entered?  I’m sure we can figure it out.
Cheers,
Greg Trasuk, VA3TSK

On Jun 3, 2018, at 2:19 PM, Nodeswitch<EMAIL_ADDRESS>wrote:
Hey! Thought I'd give this a go, but I'm encountering the following error when I run watch-aprs /dev/ttyUSB0 9600.
sh: 1: aprs: not found
I have a TNC attached via USB.
Running on the below versions.
$ npm -v && nodejs -v
6.1.0
v10.3.0
I didn't encounter any errors when installing. Am I missing anything?
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Customize recurrence intervals

This adds a new interval parameter to create recurring reminders:

Property
Value
Description

recurrenceInterval
String
The interval between instances of this recurrence. For example, a weekly recurrence rule with an interval of 2 occurs every other week. Must be greater than 0.

This fixes https://github.com/wmcmahan/React-Native-Calendar-Reminders/issues/23
Still TODO:

[ ] Default to recurrenceInterval = 1, if it's not passed in.

Could also:

[ ] Take reccurenceInterval as a Number, instead of as a String.

Great, thanks for help!


Added support for Mica Light & Dark based on Win11

What does the pull request do?
Add support for Mica Light and Dark based on how it looks in Windows 11
What is the current behavior?
The Mica blur is very simple and does not allow for same feel as other Windows 11 apps.
Note
I had this implementation for a while in Fluent Search, and did not push it here as I thought Avalonia composition will eventually replace WinUI one (maybe I misunderstood).
Currently this Light/Dark is based on WindowsTransparencyLevel as it was the only way I found for end-user to be able to interact with WinUI stuff outside of program startup.
I am open for suggestions about changing it.
Light

Dark

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Breaking changes
Breaks the current Mica value for WindowsTranspencyLevel.
Obsoletions / Deprecations

Fixed issues
#10719

Looks promising, lack of proper Mica is a huge gap in Avalonia, especially on Windows 11!
Do you think it'd be possible to leave WindowsTranspencyLevel as Mica and choose the light or dark variant based on system settings?

did not push it here as I thought Avalonia composition will eventually replace WinUI one (maybe I misunderstood).

Avalonia compositor is not going to replace WinUI one. They work similarly, but one works on a framework level, and another works on OS level, where we can't have full control.

did not push it here as I thought Avalonia composition will eventually replace WinUI one (maybe I misunderstood).

Avalonia compositor is not going to replace WinUI one. They work similarly, but one works on a framework level, and another works on OS level, where we can't have full control.

Sure, don't have full control but also it will be a fully cross-platform implementation by migrating to the Avalonia one. Removing WinUI composition behind the scenes as many places as possible does actually seem like the right move in a cross-platform framework.

WinUI compositor is used onpy and only on windows. And only to prepare a surface to render into + a refresh timer.
Soon we will add Direct Composition support (one that existed before WinUI Composition) for the same goal, but support wider range or windows devices. So we could drop some old non-composition hacks. But again, it's all only about implementation details on Windows.
Avalonia compositor is well abstracted from that, and only is compositiing avalonia visuals.

Looks promising, lack of proper Mica is a huge gap in Avalonia, especially on Windows 11! Do you think it'd be possible to leave WindowsTranspencyLevel as Mica and choose the light or dark variant based on system settings?

Changed it to be based on theme variant :)

You can test this PR using the following package version. 11.0.999-cibuild0037772-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]


WIP: Responders

Hello, sorry for the delay.
This is the first WIP implementation of responders as mentioned in #4.
I`m going to improve the current implementation but thought it might be good if you can see what I'm up to.
I've implemented the responders similar to the existing extractors to provide a consistent usage of this lib.
The HX-Location header needs more work, the others I would consider relatively finished.
There are still a few other things I'm not completely happy with, but I currently don't see a better way (Error handling, usage of serde + serde_json).
And of course the documentation in Readme.md is still missing.
Greetings,
Paul

Awesome, thanks!
Taking on a serde dependency for the feature is unfortunate, but looks like it will be necessary with the arbitrary data htmx accepts in some of the headers. I'm almost thinking we allow HX-Location and HX-Trigger (plus its variants) to work without serde. Then, if the user wants to use data in their events / location, they can include the serde feature.
This would allow responders to be included in the core library, which seems preferable to me. Thoughts?

Something else to note is that  HxTrigger is both a request extractor and a response now, which means we have an ambiguous glob export from the top-level.
My initial thought is to maybe change it so feature-gated items have to be explicitly called (eg. axum_htmx::responders::HxTrigger, while leaving the extractors as glob exports. This becomes moot if we ditch responders as a feature for serde, though.
An alternative would be to remove the top-level exports and require explicit imports by default (eg. axum_htmx::extractors::HxTrigger). Though I do find the latter less dev-friendly, it is the more consistent option.
Renaming is also an option, but I think maintaining 1:1 naming with the htmx header names is ideal. Special casing HX-Trigger might be confusing. I'm not completely opposed to it, though.

I have to think about the error handling some more, but this seems to be the best way at present.
Looks good so far, though!

I like the approach of implementing basic versions of Hx-Trigger* and Hx-Location that use the non-json method described in the spec with an optional serde-feature.
As for the ambiguous naming, I think that a HxTrigger and Hx-Triggerare more confusing than helping. I would think a rename to something likeHxResTrigger` would be better.
Not using the glob exports would certainly  fix the problem on the library side, but not on the implementation-side as extractors and responders most likely will be used in the same context and would require a import renaming each time they are used.

As for the ambiguous naming, I think that a HxTrigger and Hx-Triggerare more confusing than helping. I would think a rename to something likeHxResTrigger` would be better. Not using the glob exports would certainly fix the problem on the library side, but not on the implementation-side as extractors and responders most likely will be used in the same context and would require a import renaming each time they are used.

I agree here. HxResponseTrigger* would be suitable then.

This all looks pretty good to me, everything seems to work as expected from the testing I've done. Thanks for your work on this!


fix: agal bits should be in little endian ordering

agal bits should be in little endian because of AGALMiniAssembler sources
agalcode.endian = Endian.LITTLE_ENDIAN; agalcode.writeByte( 0xa0 );// tag version agalcode.writeUnsignedInt( version );// AGAL version, big endian, bit pattern will be 0x01000000

Thank you :smile:


feat: migrated to AWS_DEFAULT_REGION env var

Fixes https://github.com/epam/ai-dial-adapter-bedrock/issues/45

/deploy-review

/deploy-review


AttributeError: 'ConfigLoader' object has no attribute 'USE_DJANGO'

Hi!
When using the same celery app for both normal celery tasks and with message consumer, I'm getting the below error on the backend (fastapi, sending tasks to workers). I tried adding the message consumer env variables on the backend as well but that didn't help. Any ideas how to fix this? Or will it be easier to just split the two flows into separate celery workers?
File "/app/app/worker/payment.py", line 3, in
from event_consumer import message_handler
File "/app/.venv/lib/python3.10/site-packages/event_consumer/init.py", line 1, in
From event_consumer.handlers import message_handler  # noqa
File "/app/.venv/lib/python3.10/site-packages/event_consumer/handlers.py", line 25, in
if settings.USE_DJANGO:
AttributeError: 'ConfigLoader' object has no attribute 'USE_DJANGO'

it's a long time since I wrote this and I'm no longer at Depop...
The readme suggests using separate Celery apps so you can have different config for the message consumer vs regular Celery workers, which was how we always used it. I'm not sure if there's stuff that breaks if you don't do that:
https://github.com/depoplabs/celery-message-consumer/tree/1.2.1#celery
But I think your error is just around env vars somehow
USE_DJANGO is supposed to get defaulted https://github.com/depoplabs/celery-message-consumer/blob/1.2.1/event_consumer/conf/defaults.py#L45
but if you have set like EVENT_CONSUMER_APP_CONFIG=django.conf.settings and your django settings doesn't have EVENT_CONSUMER_USE_DJANGO=True|False in it then perhaps the default gets lost 🤷♂

I'm currently experimenting with this package on a Django project and I get the same error.
Getting any attribute from the settings object imported in file event_consumer/handlers.py generates the error because it is a proxy to a ConfigLoader object, which is basically a dict, so the only way to access any setting is using dict operators (settings['USE_DJANGO'], settings.get('USE_DJANGO')).
I came up with a workaround by replacing the event_consumer.conf submodule with a custom version which converts the settings ConfigLoader into a plain old Python object.
# myproject/myproject/celery.py

import sys

from myproject import conf

sys.modules['event_consumer.conf'] = conf

# myproject/myproject/conf/__init__.py
from flexisettings import Settings

class SettingsObject:
    def __init__(self, d: dict):
        for k, v in d.items():
            setattr(self, k, v)

settings = SettingsObject(Settings('EVENT_CONSUMER', 'myproject.conf.defaults'))

# myproject/myproject/conf/defaults.py is identical

Hey! I eventually figured out that this was a compatibility issue from Python 3.9 to 3.10 if I remember correctly. I basically solved it by downgrading the project's docker base image from 3.10 to 3.9. I think something got removed or changed in 3.10 and another library used by celery-messeage-consumer depended on that. Can't remember what it was exactly, but maybe this Wil be helpful.

Getting any attribute from the settings object imported in file event_consumer/handlers.py generates the error because it is a proxy to a ConfigLoader object, which is basically a dict, so the only way to access any setting is using dict operators (settings['USE_DJANGO'], settings.get('USE_DJANGO')).

I don't think that's true
Code in here treats settings as an object with attrs https://github.com/depoplabs/celery-message-consumer/blob/1.2.1/event_consumer/handlers.py#L99
and flexisettings provided attribute access on the Settings object https://github.com/depop-archive/archived-python-flexisettings/blob/master/test_project/tests/test_conf.p

@anentropic , I have both os.environ.setdefault('EVENT_CONSUMER_APP_CONFIG', 'Owner.settings') in celery file and EVENT_CONSUMER_USE_DJANGO = True in settings file, I got the error while upgrading from python v 3.9.7 debian buster to python 3.10.13 debian bookworm


DL Migration fails because of duplicate recipients

I have been working on doing my first migration and setup a test group that mirrors one of my production groups. When I ran the migration command, it built the group in Exchange Online, but it threw an error on two entries and logged the migration as failed. The log entries I got for one of the users is:
[4/25/2024 5:04:41 PM] - =====
[4/25/2024 5:04:41 PM] - Post Create Errors:
[4/25/2024 5:04:41 PM] - Primary Email Address or UPN<EMAIL_ADDRESS>[4/25/2024 5:04:41 PM] - External Directory Object ID: d9f2b055-9f61-4120-8dc1-6257687e3fc4
[4/25/2024 5:04:41 PM] - Name: AHC_TEA_TEST
[4/25/2024 5:04:41 PM] - Alias: AHC_TEA_TEST
[4/25/2024 5:04:41 PM] - Attribute in Error: Cloud Distribution Group Member
[4/25/2024 5:04:41 PM] - Error Message: Member<EMAIL_ADDRESS>unable to add to cloud distribution group.  Manual addition required.
[4/25/2024 5:04:41 PM] - Error Message Details: Ex8155FB|Microsoft.Exchange.Configuration.Tasks.ManagementObjectAmbiguousException|There are multiple recipients matching the identity<EMAIL_ADDRESS>Please specify a unique value.
[4/25/2024 5:04:41 PM] - =====

The two errors were for cross-forest contacts that had been added to the group. I thought this might be the issue, but there is a third cross-forest contact in the group that migrated correctly. These contacts have been synced to the tenant and are available there as a contact. When I ran the Get-Recipient cmdlet for the email address it returns two results. One is the contact, but the other is a guest account where the user has been invited into a team in the tenant.
Name                                 RecipientType
----                                 -------------
username_domain.com#EXT#             MailUser
d80e7381-4895-4d0e-9803-b099fa26c792 MailContact

I can manually add the user to the migrated group afterwards, but would like to figure out if there is any way to resolve this during the migration process.

If you want to send the full log to dlconversionv2 at service dot Microsoft dot com or setup a time to review with me I can probably shed some more light.
What's happened though is that the add command users ANR I believe, and when looking for SMTP addresses or querying users to build the membership table it can get an ambiguous response of more than one user.  So you have to go back and heal it by hand.
Tim
From: Eric Carr @.>
Sent: Monday, April 29, 2024 7:50 PM
To: timmcmic/DLConversionV2 @.>
Cc: Subscribed @.***>
Subject: [timmcmic/DLConversionV2] DL Migration fails because of duplicate recipients (Issue #170)
I have been working on doing my first migration and setup a test group that mirrors one of my production groups. When I ran the migration command, it built the group in Exchange Online, but it threw an error on two entries and logged the migration as failed. The log entries I got for one of the users is:
[4/25/2024 5:04:41 PM] - =====
[4/25/2024 5:04:41 PM] - Post Create Errors:
[4/25/2024 5:04:41 PM] - Primary Email Address or UPN<EMAIL_ADDRESS>[4/25/2024 5:04:41 PM] - External Directory Object ID: d9f2b055-9f61-4120-8dc1-6257687e3fc4
[4/25/2024 5:04:41 PM] - Name: AHC_TEA_TEST
[4/25/2024 5:04:41 PM] - Alias: AHC_TEA_TEST
[4/25/2024 5:04:41 PM] - Attribute in Error: Cloud Distribution Group Member
[4/25/2024 5:04:41 PM] - Error Message: Member<EMAIL_ADDRESS>unable to add to cloud distribution group.  Manual addition required.
[4/25/2024 5:04:41 PM] - Error Message Details: Ex8155FB|Microsoft.Exchange.Configuration.Tasks.ManagementObjectAmbiguousException|There are multiple recipients matching the identity<EMAIL_ADDRESS>Please specify a unique value.
[4/25/2024 5:04:41 PM] - =====
The two errors were for cross-forest contacts that had been added to the group. I thought this might be the issue, but there is a third cross-forest contact in the group that migrated correctly. These contacts have been synced to the tenant and are available there as a contact. When I ran the Get-Recipient cmdlet for the email address it returns two results. One is the contact, but the other is a guest account where the user has been invited into a team in the tenant.
Name                                 RecipientType

username_domain.com#EXT#             MailUser
d80e7381-4895-4d0e-9803-b099fa26c792 MailContact
I can manually add the user to the migrated group afterwards, but would like to figure out if there is any way to resolve this during the migration process.

Reply to this email directly, view it on GitHubhttps://github.com/timmcmic/DLConversionV2/issues/170, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKGTN6JVVNVZ3TRWHRMGRITY73MEJAVCNFSM6AAAAABG7HKUJGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI3TAMRRGEZTKMQ.
You are receiving this because you are subscribed to this thread.Message ID<EMAIL_ADDRESS>

The script users the get-recipient - if the same address is returned for both that's probably going to be part of the problem.  Generally you'd remove the contact once the guest account is in place and mail enabled / unhidden from the GAL.
I'm surprised you do not have proxy address conflicts in your azure sync center.
Tim
From: Eric Carr @.>
Sent: Monday, April 29, 2024 7:50 PM
To: timmcmic/DLConversionV2 @.>
Cc: Subscribed @.***>
Subject: [timmcmic/DLConversionV2] DL Migration fails because of duplicate recipients (Issue #170)
I have been working on doing my first migration and setup a test group that mirrors one of my production groups. When I ran the migration command, it built the group in Exchange Online, but it threw an error on two entries and logged the migration as failed. The log entries I got for one of the users is:
[4/25/2024 5:04:41 PM] - =====
[4/25/2024 5:04:41 PM] - Post Create Errors:
[4/25/2024 5:04:41 PM] - Primary Email Address or UPN<EMAIL_ADDRESS>[4/25/2024 5:04:41 PM] - External Directory Object ID: d9f2b055-9f61-4120-8dc1-6257687e3fc4
[4/25/2024 5:04:41 PM] - Name: AHC_TEA_TEST
[4/25/2024 5:04:41 PM] - Alias: AHC_TEA_TEST
[4/25/2024 5:04:41 PM] - Attribute in Error: Cloud Distribution Group Member
[4/25/2024 5:04:41 PM] - Error Message: Member<EMAIL_ADDRESS>unable to add to cloud distribution group.  Manual addition required.
[4/25/2024 5:04:41 PM] - Error Message Details: Ex8155FB|Microsoft.Exchange.Configuration.Tasks.ManagementObjectAmbiguousException|There are multiple recipients matching the identity<EMAIL_ADDRESS>Please specify a unique value.
[4/25/2024 5:04:41 PM] - =====
The two errors were for cross-forest contacts that had been added to the group. I thought this might be the issue, but there is a third cross-forest contact in the group that migrated correctly. These contacts have been synced to the tenant and are available there as a contact. When I ran the Get-Recipient cmdlet for the email address it returns two results. One is the contact, but the other is a guest account where the user has been invited into a team in the tenant.
Name                                 RecipientType

username_domain.com#EXT#             MailUser
d80e7381-4895-4d0e-9803-b099fa26c792 MailContact
I can manually add the user to the migrated group afterwards, but would like to figure out if there is any way to resolve this during the migration process.

Reply to this email directly, view it on GitHubhttps://github.com/timmcmic/DLConversionV2/issues/170, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKGTN6JVVNVZ3TRWHRMGRITY73MEJAVCNFSM6AAAAABG7HKUJGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI3TAMRRGEZTKMQ.
You are receiving this because you are subscribed to this thread.Message ID<EMAIL_ADDRESS>


chore(apps/prod/tekton/setup): increase pruner keep size to 1000

Signed-off-by: wuhuizuo<EMAIL_ADDRESS>

/approve


[Bug]:  core dump on Ubuntu 24.04

Version

[X] I'm using version 1.17.1

Known issues

[X] I've checked Known issues

Existing issues

[X] I've checked Existing issues

Advanced Preferences

[X] I've checked Advanced Preferences

What operating system are you using?
Linux
Operating System Version
Ubuntu 24.04
Reproduction steps

Upgrade to 1.17.1 via package manager
Try to start Stretchly

...
Reproduces how often: [What percentage of the time does it reproduce?]
100%
Expected Behavior
Stretchly should start
Actual Behavior
[6197:1226/085742.884248:FATAL:setuid_sandbox_host.cc(163)] The SUID sandbox helper binary was found, but is not configured correctly. Rather than run without sandboxing I'm aborting now. You need to make sure that /opt/Stretchly/chrome-sandbox is owned by root and has mode 4755.
Trace/breakpoint trap (core dumped)

I've verified that chrome-sandbox has the permissions setup as indicated in the error message:
-rwxr-xr-x 1 root root     38448 Dec 25 06:57 chrome-sandbox*

After downgrading to 1.16.0, I was able to launch Stretchly successfully again.
Relevant log output
No response
Preferences
No response
Additional information
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi, seems like the error described here: https://github.com/hovancik/stretchly?tab=readme-ov-file#linux-note, part If Stretchly is not starting, you might need to run:

Thanks, I ran that command and re-upgraded to 1.17.1 but I'm still getting the same error and Stretchly doesn't start.

How did you install it again? I guess snap?
Ubuntu is troublemaker :) Seems like this starting again lately and previous fix is not working: https://github.com/electron/electron/issues/17972#issuecomment-2551237520

I'm using the apt repository https://apt.fury.io/hovancik/
Upgrade command
sudo apt upgrade

Downgrade command
sudo apt install stretchly=1.16.0

Ok, thanks. I see some people also do sudo sysctl -w kernel.apparmor_restrict_unprivileged_userns=0 https://github.com/toeverything/AFFiNE/blob/canary/.github/workflows/build-test.yml#L600

Oh thank you!  That works for me.

Great, I'll ad that to readme


Add 'dot' option to 'watch'

When listening to files, I can use the dot option for my src globs. However, if I want to watch those same globs, there's no option to do that and I have to list out all dot entries explicitly.
Edit: I'm sorry, I did this in haste. I should have at least gone to SO first. Unfortunately I can't delete this so I hope that there isn't an easy solution I missed. I apologize in advance if there is.

@Ursi you don't need to delete, actually issues are never deleted — you just need to close it, click the button to close this issue underneath

@revelt well if I close it then I don't think I can reopen it, and I don't want to close it if Gulp actually is missing this functionality.


Cache profiles.yml file when using Cosmos ProfileMapping

Context
In Cosmos 1.3, when users opt to use ProfileMapping Cosmos creates a profiles.yml file in potentially two places:

When using dbt ls to render the dbt project into an Airflow DAG
When running dbt commands

Once we implement #924, which makes sure the profile file is created in the same way, we can extend the caching mechanism introduced in #904 also to cache the profiles.yml file.
Depends on: #924
Acceptance criteria

[ ] Create a cache for the (if used, mocked and) non-mocked profiles.yml per DAG/TaskGroup when using ProfileMapping
[ ] Decide on a criterion to decide when to use or not the cached profiles.yml (perhaps we could create a hash based on the keys in the connection and the profile args passed by the user?)
[ ] Reuse the existing profiles.yml whenever possible without recreating it.
[ ] The logs should contain less of the following:

Creating temporary profiles.yml at /var/folders/td/522y78v91d1f5wgh67mj3p0m0000gn/T/tmprpp66fdh/profiles.yml with the following contents:
default:
    outputs:
        dev:
            dbname: postgres
            host: <IP_ADDRESS>
            password: '{{ env_var(''COSMOS_CONN_POSTGRES_PASSWORD'') }}'
            port: 5432
            schema: public
            type: postgres
            user: postgres
    target: dev

A key part to remember in this ticket is that we'll likely need two caches per ProfileMapping instance:

one when it's mocked
another when it's not mocked

Hi @pankajastro , following what we discussed, this PRs may help out:


Small file structure update

Updated the file structure and some other things. Everything has been pushed this time.

Resolved my first merge conflict with this pull request.


decktape: init at 3.14.0

Adds decktape package. Fixes #343052
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

Note: ofborg currently fails to build this package

Fixed suggestions. I'll try to investigate why ofborg fails after the next run finishes

Unless it's this last queued aarch64 ofborg job that still hasnt run yet that failed last time, it seems like its working now so maybe the failed build last time was something unrelated?

Checked again and looks like ungoogled-chromium is causing it to time out. Not sure what the best practice is in nixpkgs but looks like https://github.com/NixOS/nixpkgs/pull/347015 is the only package that depends on ungoogled-chromium and a similar package haylxon requires manually specifying the browser.
error: building of '/nix/store/c0c5nxq2jjh09h59507sljksaqy3qsqy-ungoogled-chromium-unwrapped-131.0.6778.85.drv^out,sandbox' from .drv file timed out after 3600 seconds
error: 1 dependencies of derivation '/nix/store/rkl8sqdpq4qbk3c2xwwyn6x3a1pggwix-ungoogled-chromium-131.0.6778.85.drv' failed to build
error: 3 dependencies of derivation '/nix/store/s6m0rvf7ig2rflxlwxrhn46cl77zinlb-decktape.drv' failed to build

Cc: @Pandapip1

I've switched to regular chromium instead of ungoogled-chromium so we can see. I noticed puppeteer-cli uses regular chromium.

Diff LGTM just need to add meta.mainProgram

Added

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/2150


improve scheduling performance on batch jobs

What this PR does / why we need it:
This PR aims to make the big job release allocated resource for other submitted job when all idle resource of cluster can not satisfy the big job.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes https://github.com/volcano-sh/volcano/issues/268

/cc @killwing , @zionwu , @hex108

Last question:  would big jobs starve?

Last question: would big jobs starve?
Yes. The starvation will happen if small job is submitted continuosly.  The starvation issue is being resolved in another project planed by Klaus. This ticket is only focusing on the pending issue.

/lgtm
Could you please squash these commits to one commit?

/lgtm
Could you please squash these commits to one commit?

Sure. I just squash these commits. Thanks.

The starvation issue is being resolved in another project planed by Klaus.

The starvation issue will be handled in https://github.com/volcano-sh/volcano/issues/13

/approve

Will this PR cherry-pick to kube-batch repo?


[FEATURE REQUEST] unreal version

could you please transition the unreal engine version to this and get it back in the epic launcher working with the latest engine versions?

Unfortunately, we are not looking to support Unreal at this time

Understood, please reconsider when you have the resources, being able to use mrtk, even if the existing one out of the box in unreal would be a great thing.


Add linspaced_int_array()

Summary
Adds support for ìnt[] linspaced_int_array(int, int, int) as a complement to real[] linspaced_int_array(int, real, real). This was proposed by @avehtari in a discussion and I agree it would be useful.
linspaced_int_array(5,0,8) returns 0,2,4,6,8
linspaced_int_array(5,0,9) returns 0,2,4,6,8
linspaced_int_array(5,0,11) returns 0,2,4,6,8
linspaced_int_array(5,0,12) returns 0,3,6,9,12
So the behaviour is the same as for Eigen::VectorXi v = Eigen::VectorXi::LinSpaced(K, low, high)
Tests
/
Side Effects
/
Release notes
Added linspaced_int_array.
Checklist

[x] Copyright holder: Rok Češnovar
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

I'm still confused on the docs on this.
This is how Eigen says it (from here):

For integer scalar types, an even spacing is possible if and only if the length of the range, i.e., high-low is a scalar multiple of size-1, or if size is a scalar multiple of the number of values high-low+1 (meaning each value can be repeated the same number of time). If one of these two considions is not satisfied, then high is lowered to the largest value satisfying one of this constraint.

That last sentence is hard to parse.

I'm still confused on the docs on this.
This is how Eigen says it (from here):

For integer scalar types, an even spacing is possible if and only if the length of the range, i.e., high-low is a scalar multiple of size-1, or if size is a scalar multiple of the number of values high-low+1 (meaning each value can be repeated the same number of time). If one of these two considions is not satisfied, then high is lowered to the largest value satisfying one of this constraint.

That last sentence is hard to parse.

I think it just mean when we specify
Eigen::VectorXi::LinSpaced(5, 0, 11); it lowers 11 until its a valid linspaced vector (under the two conditions).

I think it just mean when we specify
Eigen::VectorXi::LinSpaced(5, 0, 11); it lowers 11 until its a valid linspaced vector (under the two conditions).

All good! Thanks.

All good! Thanks.

@bbbales2 approve?

@bbbales2 approve?


update go deps

Fixes #4297

Still could not build success.
https://github.com/ollama/ollama/issues/4297#issuecomment-2105958377


Supply arguments after a subcommand.

dua -f binary i succeeds
dua i -f binary fails
Should work in both positions.

I disagree. There are global options which work before a sub-command is specified, and there are sub-command specific options which work after sub-command is specified.


[Enhancement] Always show active mail account in UI

Hi,
there's one thing, I'm really missing in rainloop in everyday use:
I use many different mail accounts and in the interface I'm not able to see, which account is the actual one,
I know, that there is a mouse over info on the account selector, but it would be a great thing, if I ALWAYS would be able see which account is the actual one - like an "always active mouse over" at this position.

What do you think about this?

Ok :)

Ok :)

thx :+1:

thx :+1:

I subscribe to deMattin's idea!

Done!

Thanks for adding that... There is an issue with that option though. See issue #412 (I just added it)


Nullable DateTime Value throwing error

Source/destination types
Nullable DateTime datatype of C#
Source/destination JSON
"enddate":-1
Expected behavior
null value is not allowed.
Actual behavior
Getting error as
"An unhandled exception occurred while processing the request.
InvalidCastException: Unable to cast object of type 'System.Int64' to type 'System.Nullable`1[System.DateTime]'.
Newtonsoft.Json.JsonTextReader.ReadAsDateTime()"
Steps to reproduce
I am having C# class with one data member as below
public DateTime? EndDate { get; set; }
Whenever i am trying to invoke with a below value from postman i am getting the error message
{
"enddate":-1
}

A number cannot be converter to a DateTime. I've improved the error message.


Automated cherry pick of #9629: fix(region): optimized disk allocate sched failed log

Cherry pick of #9629 on release/3.4.
#9629: fix(region): optimized disk allocate sched failed log

Codecov Report

Merging #9632 (cd9b632) into release/3.4 (fc16c37) will decrease coverage by 2.62%.
The diff coverage is 0.44%.

@@              Coverage Diff               @@
##           release/3.4   #9632      +/-   ##
==============================================
- Coverage         6.19%   3.56%   -2.63%
==============================================
  Files              761     306     -455
  Lines           118405   62969   -55436
==============================================
- Hits              7330    2244    -5086
+ Misses          110412   60570   -49842
+ Partials           663     155     -508

Flag
Coverage Δ

aFlag
3.56% <0.44%> (-2.63%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/cloudcommon/notifyclient/notify.go
1.78% <0.00%> (-0.03%)
:arrow_down:

pkg/compute/models/capabilities.go
0.00% <0.00%> (ø)

pkg/compute/models/cloudaccounts.go
0.47% <0.00%> (-0.02%)
:arrow_down:

pkg/compute/models/cloudregions.go
1.52% <0.00%> (-0.02%)
:arrow_down:

pkg/compute/models/cloudsync.go
0.00% <0.00%> (ø)

pkg/compute/models/dbinstance_skus.go
2.79% <0.00%> (-0.04%)
:arrow_down:

pkg/compute/models/domainquota.go
23.80% <0.00%> (ø)

pkg/compute/models/elasticcache_skus.go
3.13% <0.00%> (-0.05%)
:arrow_down:

pkg/compute/models/elasticipresource.go
0.00% <0.00%> (ø)

pkg/compute/models/elasticips.go
1.07% <0.00%> (-0.02%)
:arrow_down:

... and 451 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 ac8ba1e...cd9b632. Read the comment docs.


Fixci

I will squash the commit when I merge. There was a race condition in tests. I wasn't waiting for a closure to be set before calling it.
Commit message:
Fixing the race condition in tests that was causing travis-ci to fail. Fixes #11.

[Current coverage][cc-pull] is 89.97% (diff: 85.71%)

No coverage report found for master at 4bab72b.

Powered by Codecov. Last update [4bab72b...b8663fb][cc-compare]
[cc-base-branch]: https://codecov.io/gh/linkedin/RocketData/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/linkedin/RocketData/compare/4bab72b95b5c98bdb5bcb37c76383f6619248b26...b8663fbcf8a3d7b13c1efaacaac9d9c40942af1f?src=pr
[cc-pull]: https://codecov.io/gh/linkedin/RocketData/pull/12?src=pr

Huh. Looks like travis failed the first time...looks like it was because the simulator failed though :/
Probably something we just have to live with.

lgtm
the closure dance is tricky but I can't think of a better way to do it


fix: order of wasm and storage overrides

In #352, the order of WASM and storage override order introduced in #199 was reverted. I suppose this happened by accident as the inline docs still state WASM override to have order priority. Moreover, it destroys the purpose of WASM override as new pallets cannot override storage.

yes it's been reordered by accident


Integrate with Ingress

Description
This PR adds integration to ingress. This includes:

Adding the ingress relation
Improving documentation
Adding integration to ingress in integration tests

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[x] I have added tests that validate the behaviour of the software
[x] I validated that new and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Please fix the typo in the PR title

The PR is pending now. I will discuss it with @PietroPasotti

The PR is pending now. I will discuss it with @PietroPasotti

Done.

We need to deploy Traefik from edge channel.


req.body is empty in key function

When sending a payload with a file and additional text fields I would expect req.body to contain these text field but it just returns an empty object. Is this intentional?
My use-case is I want to construct the file path based on these additional fields.

req.body doesn't get populated by multer - you're probably looking for the body parser https://github.com/expressjs/body-parser


Fix [[ not found error when loading shell script from vim

When using let g:base16_shell_path="/pathto/base16-builder/output/shell/" in vim with vim-base16 themes. You'll see the following error every time you close vim:
/pathto/base16-builder/output/shell//base16-default.dark.sh: 50: /pathto/base16-builder/output/shell//base16-default.dark.sh: [[: not found

This PR fixes this issue by replacing the check at line 50 with a safer one that works with primitive interpreters.
I've submitted this a while ago to the original base16-builder project as well: chriskempson/base16-builder#325

After testing, the right behavior is achieved with
[ -n "${-##*i*}" ]

the asterisks are required to match "i" inside the string .

@sirfz
Yes, they got lost in the markup (without backticks before).

Is this ready to merge? Thank for your involvement btw 😄

@aloisdg yep it's good to go. Thank you for reviving this project!

@sirfz Thank you too!

Thanks a lot for the pull request, @sirfz and thank you, @blueyed for your input.
I have not personally experienced this problem nor do I know how to reproduce it.
I would like to verify the problem this solves before merging it. Could you please explain to me how I can reproduce the problem? Once I can do that I can confirm that this patch solves the problem and merge it.
Thanks :smile:

Hi @alexbooker,
This problem was introduced after recent commits to the original base16-builder (related to the shell template specifically) that in turn affected vim. To reproduce:
In your .vimrc file, add the following lines:
set background=dark
let base16colorspace=256
let g:base16_shell_path="/path-to-base16-shell-scripts/"
colorscheme base16-default

Make sure you have base16-default.vim in your vim's colors directory.
Now, every time you launch vim, it's going to execute the shell script (base16-default.dark.sh) located in the path set in g:base16_shell_path. Since it uses a primitive sh interpreter, you will see the following error when you exit vim (printed on your console where you launched vim from):
/path-to-base16-shell-scripts//base16-default.dark.sh: 50: /path-to-base16-shell-scripts//base16-default.dark.sh: [[: not found

Note that I didn't notice any problems with the vim colorscheme caused by this issue but the error message is annoying.

Thanks for the quick response.
I'm actually streaming right now, if you want to join me in the live chat?

I'm actually streaming right now, if you want to join me in the live chat?

Thank you, but unfortunately I can't join right now.

@sirfz No worries. I'm still having a bit of trouble reproducing this but I think I'll just merge anyway. The fact that @sirfz and yourself support it is good enough for me :+1:.

Thanks @alexbooker, I promise it's legit lol :+1:


Support Python 3

Hi Findepi,
Do we have any updates for this issues?  Will this support Python 3

If someone wants to work on it, the starting point would be to enable testing matrix in travis.yml to include python 3.


login: display message when account already logged in

Partially touches on #108 , though doesn't address other kinds of SQL errors, I set up some scaffold code to start handling errors that come from the store.
(This is my first PR (in Dart/Flutter/Zulip), so bear with me.)
My initial instinct was to use the Snackbar to signal this issue. (Screenshot in below comment)
However, I saw after that there's a showErrorDialog, so I'm updating it to use it in a separate commit.
The Snackbar is likely a better UX for this scenario since there's little the user can do in a dialog, and there's no input needed from the user. We can also add a action in the snackbar to navigate to the logged in account.
Happy to squash changes once I hear back on some feedback.
Screenshot (using macOS build to debug, blurred out credentials):

Thanks @chrisirhc for the contribution! This code generally looks good.
Can you post a screenshot for the snackbar approach? That'd help in thinking about which would be best.
Yep, here:

Before we can review this change fully, it will need tests. See the section in our README about writing tests.
Yep done ✅
Meanwhile, a couple of comments below from a first look.
Yep addressed ✅

Thanks for the detailed review and comments.
Apologies, I missed the commit guide earlier and some of the conventions. Rebased the commits, edited the commit messages, as well as the nit comments.

Oh, one other thought: please go ahead and mark this (both in the PR description, and the commit message of the main commit) as fixing #108. (See style guide here for details.) The case where the user is already logged into the account is the primary thing #108 is about; we can file a separate followup issue for handling other SQL errors like a full disk.
(We'll eventually have other SQL queries that modify things too, not just this one, and errors like a full disk will be cross-cutting across all of those. So the best time to write that handling might in any case be after we have a couple more such queries and a couple more call sites for them, the better to think about what the API should look like at those call sites.)

Yep done. Rebased and verified.

Thanks for your work on this! 🙂 Looks good; merging. (Greg is on vacation through Monday and asked me to re-review.)

Looks great. Thanks all!


How can i setup it

It is really what i am looking for, can you help add a document about how to use it. Thanks you in advance

@hzxuzhonghu Thank you for your interest, i have done this one for my testing purposes and never have/had a plan to maintain or to publish it.

Thanks for your quick response


Global styles linking in angular.json does not work is they come from library assets

Which @angular/* package(s) are the source of the bug?
compiler
Is this a regression?
No
Description
I have an library project that includes some assets like icons and some global styles. Those are packaged in assets folder of library. Consuming application includes library assets by extending the assets section in its angular.json:
"assets": [
    "projects/my-app/src/favicon.ico",
    "projects/my-app/src/assets",
    {
      "input": "dist/my-lib/assets", // with regular install this would go from node_modules instead of dist
      "glob": "**/*",
      "output": "assets/my-lib"
    }
  ],

when build, the library assets are copied into assest/my-lib folder of consuming application as expected.
The issue occurs when I want to use the library style from assest/my-lib/styles.css. If I add it to styles section of angular.json like this:
"styles": [
    "projects/my-app/src/styles.css",
    "projects/my-app/src/assets/my-lib/styles.css" // added
  ],

it ends with error:  Error: Can't resolve 'projects/my-app/src/assets/my-lib/styles.css'
However when I add it in index.html. like this ` it works as expected.
It looks like styles processing happens before assetscopying, which results in the error. Expectation is that assets are copied before the styles/scripts linking because those can reference assets.
Please provide a link to a minimal reproduction of the bug
https://github.com/yohny/assets-demo
Please provide the exception or error you saw
Error: Can't resolve 'projects/my-app/src/assets/my-lib/styles.css'

Please provide the environment you discovered this bug in (run ng version)
Angular CLI: 15.1.4
Node: 16.19.0
Package Manager: npm 8.19.3
OS: linux x64

Angular: 15.1.3
... animations, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1501.4
@angular-devkit/build-angular   15.1.4
@angular-devkit/core            15.1.4
@angular-devkit/schematics      15.1.4
@angular/cli                    15.1.4
@schematics/angular             15.1.4
ng-packagr                      15.1.1
rxjs                            7.8.0
typescript                      4.9.5

Anything else?
No response

When using the styles option you should not place the same stylesheet in the assets option. These option serve for different purposes. In your case you should  add"dist/my-lib/assets/styles.css" to the styles option and remove the assets entry. See: https://angular.io/guide/workspace-config for more information.
The error in this case is correct as indeed there is no such file as projects/my-app/src/assets/my-lib/styles.css.
Be sure to build you library prior of building your application.
You need to export the CSS/assets file https://angular.io/guide/creating-libraries#managing-assets-in-a-library

Closing as this issue tracker is not suitable for support requests, please repost your issue on StackOverflow using tag angular-cli. If you are wondering why we don't resolve support issues via the issue tracker, please check out this explanation.


Make Hystrix Thread Pool available while using Zuul

This PR adds a new property in ZuulProperties: ribbonIsolationStrategy to change default isolation (Semaphore) to thread.
fixes gh-339

New patch with some test for content length

Can we restart the failed Travis build ? it was failed without even starting:
No output has been received in the last 10 minutes, this potentially indicates a stalled build or something wrong with the build itself.

@stephlag restarted, Travis has been very flakey lately, sorry about that

@stephlag the latest failure is due to some test failures
postOnSelfViaRibbonRoutingFilter(org.springframework.cloud.netflix.zuul.filters.route.okhttp.OkHttpRibbonCommandIntegrationTests): expected:<200> but was:<500>
  patchOnSelfViaRibbonRoutingFilter(org.springframework.cloud.netflix.zuul.filters.route.okhttp.OkHttpRibbonCommandIntegrationTests): expected:<200> but was:<500>

Were you not seeing these locally?

OK Thanks I'm working on these failures
  De : Ryan Baxter<EMAIL_ADDRESS>
À : spring-cloud/spring-cloud-netflix<EMAIL_ADDRESS>Cc : Stephan L<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Envoyé le : Lundi 8 août 2016 16h56
Objet : Re: [spring-cloud/spring-cloud-netflix] Make Hystrix Thread Pool available while using Zuul (#1221)
@stephlag the latest failure is due to some test failurespostOnSelfViaRibbonRoutingFilter(org.springframework.cloud.netflix.zuul.filters.route.okhttp.OkHttpRibbonCommandIntegrationTests): expected:<200> but was:<500>
patchOnSelfViaRibbonRoutingFilter(org.springframework.cloud.netflix.zuul.filters.route.okhttp.OkHttpRibbonCommandIntegrationTests): expected:<200> but was:<500>
Were you not seeing these locally?—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

The failing tests have been fixed by commit https://github.com/spring-cloud/spring-cloud-netflix/pull/1221/commits/072f7760a4d817371e38ebd4efbf09e375ce0152

@ryanjbaxter  I think that there is a conflict with a recent PR you merged

@stephlag probably ;) mind resolving them in your PR?

Done

I changed the code based on Spencer's feedback
circleci and travisci both failed :worried:
not related to my change though

@stephlag Travis is failing for unknown reasons.  CircleCI is failing bc you are missing my circleci.yml changes.  If you merge them in to this PR than it SHOULD pass.

OK everything is good now

LGTM

Probably a note in the docs would be good.

Do you want me to add a note in the docs?

yes

I added a note near the beginning of zuul doc

@jebeaudet I had to merge a conflict with your latest PR, your new tests are OK with my code but can you check that my change is compatible with yours?

@stephlag It looks OK since you use request.getContentLengthLong() in RibbonRoutingFilter. However, I'm not sure about the constructor of RibbonCommandContext with no content length, shouldn't this constructor pass null instead of parsing the headers in the hope of a content-length? I don't have an IDE in front of me but is this constructor used anywhere? We could drop it I guess if it's not.

I made the changes you suggested (removed this additional constructor with content length header parsing)

Current coverage is 73.17% (diff: 76.47%)

Merging #1221 into master will increase coverage by 0.01%

@@             master      #1221   diff @@
==========================================
  Files           184        184
  Lines          5514       5513     -1
  Methods           0          0
  Messages          0          0
  Branches        841        842     +1
==========================================
  Hits           4034       4034
+ Misses         1170       1168     -2
- Partials        310        311     +1

Powered by Codecov. Last update 674e45f...62344dc

@ryanjbaxter @spencergibb It should be good now, what do you think?

LGTM, @spencergibb

Squashed and merged via 34befdf1d2c1c823c67f2dc53a105528629785a4

This broke some extensions in downstream spring-cloud-sleuth code, reverting and reopening.

I can do the PR for spring-cloud-sleuth code also

@stephlag we need to make things backwards compatible.  Removing the constructors in AbstractRibbonCommand and RestClientRibbonCommandFactory is causing issues in Sleuth

OK I will change put back these constructors to maintain this backwards compatibility

I added a new test to cover the old constructor but codecov still complains...

@marcingrzejszczak you might want to take a quick look at this

I had a quick look and it seems ok. What we could do is to check out the branch, build and install it locally and try to build Sleuth with the locally installed version. If it passes then it's ok :)

Ah I've built your version locally and then tried to build Sleuth master. It failed. Can you try to do the same locally? Sleuth master is using 1.2.0.BUILD-SNAPSHOT of SC-Netflix.

I is working for me locally, no compilation problem for spring cloud sleuth on master branch.
Can you please tell me where is the problem?

Let's maybe merge it and see if Jenkins breaks. Most likely I had sth broken locally then

I was able to checkout @stephlag's branch and built it and Sleuth fine.  I will merge the pull request and watch Jenkins!

Hi, how to get the zuul request when using hystrix thread pool strategy in  ribbon iRule.I want to do some sticky session work via get cookie from zuul request. Thinks…… @stephlag

@boycezeng please don't comment on old issues. Ask questions on stack overflow or gitter


feat: implement interactive background

Goal
Some interactive backgrounds could be fun.
Acceptance criteria
None.
Proposal

Vanta.js

Appendix
No response

See. https://gohugo.io/hugo-pipes/js/


Breaking up set of 5 .combine() tests

I'm not familiar with cats and I had trouble with the first question here.  When the answer I submitted failed, I could not tell whether it was because I didn't understand how .combine() worked at all, or because one of my answers was wrong.  Since the first 4 are roughy the same, I bumped the 5th into it's own test.

Will open new PR for branch called "uncombine"


Custom / self-signed certificates don't appear to be working

Custom certificates don't appear to be working
Custom certificates, for example self-signed certificates, don't appear to be working properly.
As discussed on this thread:
https://mastodon.social/@kdrag0n/110534899510779775
To Reproduce
The issue comes when pulling a Docker image from a repository which has TLS intercepted and decrypted using a custom root certificate.
Interestingly, there is a difference in behaviour when running Docker Compose without a hyphen and with a hyphen, for example this form hangs with the timer running endlessly, until you kill the process:
$ docker compose up
[...redacted...]
=> ERROR [app internal] load metadata for docker.io/library/ruby:3.1.2-bullseye

But with the hyphen you get a more detailed error message and task stops:
=> failed to solve: ruby:3.1.2-bullseye: failed to do request: Head "https://registry-1.docker.io/v2/library/ruby/manifests/3.1.2-bullseye": x509: certificate signed by unknown authority

I ran docker run -it --rm -v /:/host alpine sh so I could poke about inside, and I found there were extra certs present in /host/etc/ssl/certs and I found the custom root certificate in a file orb-extra-<sha>.crt, but Docker Engine still complains.
I found that /host/etc/docker/certs.d is a symlink to /mnt/mac/Users/<username>/.docker/certs.d which doesn't seem to exist (in fact /mnt is empty).
Outside of the Alpine image, I can confirm this does exist:
$ ls -al ~/.docker/certs.d
total 48
drwxr-xr-x   5 <username>  staff    160 Jun 13 10:33 .
drwxr-xr-x  11 <username>  staff    352 Jun 12 16:03 ..
-rw-r--r--@  1 <username>  staff   6148 Jun 12 16:44 .DS_Store
-rw-r--r--@  1 <username>  staff   1732 Jun 12 16:34 custom_cert.pem

Expected behavior
The image should work normally, as it's signed by one of the orb-extra certs.
Logs
I have emailed.
Info report
OrbStack info:
  Version: 0.11.3 (110300)
  Commit: f953c63b1cbfe3c2988c97e0b42680600cf0e060 (v0.11.3)

System info:
  macOS: 13.4 (22F66)
  CPU: arm64, 8 cores
  CPU model: Apple M1

Additional context
I'm not 100% sure how to debug further, but thanks so much for your help thus far :)

One possible unrelated difference between the orb-extra-<sha>.crt in /host/etc/ssl/certs/ and the local one - the presence of additional ^M control characters at the end of each line.

$ ls -al ~/.docker/certs.d
total 48
drwxr-xr-x   5 <username>  staff    160 Jun 13 10:33 .
drwxr-xr-x  11 <username>  staff    352 Jun 12 16:03 ..
-rw-r--r--@  1 <username>  staff   6148 Jun 12 16:44 .DS_Store
-rw-r--r--@  1 <username>  staff   1732 Jun 12 16:34 custom_cert.pem

In order for certs.d to work, you have to follow a specific directory layout: https://docs.docker.com/engine/security/certificates/

Ah, that's trickier then, I just have a custom root certificate, which the system is using to sign requests transparently to Docker, effectively it's MITM.

What about docker run -it --rm ruby:3.1.2-bullseye?
If that also works, then it looks a bit like Compose might be using a different Docker context.

Hi, yeah normal docker run does work indeed, but not via Compose. Interesting theory on the contexts, here's my current setup:
$ docker context ls
NAME         DESCRIPTION                               DOCKER ENDPOINT                                   KUBERNETES ENDPOINT   ORCHESTRATOR
default      Current DOCKER_HOST based configuration   unix:///var/run/docker.sock                                             swarm
orbstack *   OrbStack                                  unix:///Users/<username>/.orbstack/run/docker.sock

Is this still an issue? Haven't been able to reproduce it with Compose/buildx + custom CA:
[+] Building 0.0s (5/5) FINISHED                                            docker:orbstack
 => [test internal] load build definition from Dockerfile                              0.0s
 => => transferring dockerfile: 73B                                                    0.0s
 => [test internal] load .dockerignore                                                 0.0s
 => => transferring context: 2B                                                        0.0s
 => [test internal] load metadata for registry.registry.orb.local/a:latest             0.0s
 => [test 1/1] FROM registry.registry.orb.local/a                                      0.0s
 => [test] exporting to image                                                          0.0s
 => => exporting layers                                                                0.0s
 => => writing image sha256:474becdb3018bdef235179b4c78f849d132185a0d6c793b93454eb898  0.0s
 => => naming to docker.io/library/compose2-test                                       0.0s
[+] Running 2/0
 ✔ Network compose2_default   Created                                                  0.0s
 ✔ Container compose2-test-1  Created                                                  0.0s

I’ve actually left the job that had this requirement now, so if you’re unable to reproduce let’s close it and re-open it if it reoccurs? Thanks!

Sure, haven't seen any other cases of this so I think it can be closed for now. Sorry that we didn't get a chance to dive deeper.

No problem! 🙂 Thanks again.


Extracting option value in optgroup

Hi, i notice that when extracting a form, it doesnt detect the value that is in optgroup. The id is there in the rule but the value is empty.
Is there any way to extract the value and also fill it to form?

ok, i got it working for setting value which i did like below. But there's still the extracting the value issue if there are any work around. Thanks
"selector": "#visa_collection_center",
"value": function($e, data) {
$('#visa_collection_center option[value="'+ data.visa_collection_center +'"').attr('selected', 'selected').text();
}

Hi, thanks for the report!
I simply forgot optgroups in the extractor :) Well I'll see what I can do after my holiday.
Cheers,
Frank

Fixed in master, will be in 3.7.0

Thanks, glad to be of little help.

Finally had time to push 3.7.0 to the store. Try updating the extension in ~60 minutes and  should work as expected.


move SegmentedControl to Group/Button and add deprecation warning

Overview
This marks SegmentedControl as deprecated (to be removed in next major version) and switches it to use Group + Button internally (for visual consistency).
Screenshots (if applicable)

Documentation

[x] Updated Typescript types and/or component PropTypes
[x] Added / modified component docs
[x] Added / modified Storybook stories

Can you add some documentation to the segmented-control docs?


The "plugins" parameter went down

Plugins sent in the options parameter are no longed being initialized.
If you want an old plugin to work you have to initialize it manually, either this should be reflected in the documentation or the plugins parameter should be reinstated.
As an example i have to call now my "list.style.js" plugin like:
  var hackerList = new List('list', options);
  var listStyleActivated = ListStyle(styleOptions);
  listStyleActivated.init(hackerList);

Please can the documentation be updated to reflect this change?


How do we compile it

Compile instruction for gnu/linux?

I used Qt4's qmake to convert the .pro file to a Makefile on the FATX directory first.  Looks like it compiled correctly.  Did the same on the Up directory without any errors.  The Up program crashes on me when I select the load devices under file.  I don't have anything connected to my PC so that might be the cause but I am unsure.  Still trying to figure this program out.

I did change somethings to get it to compile under Qt5 but still crashes on 'Load Devices'.

@kermitdafrog8 this project was one of my first ever C++ projects and likely has many memory safety issues. Compile with -fsanitize=address and you'll likely shake things out. Otherwise I haven't touched this project in about 10 years and can't really help.

Is there any way to contact you directly?


refactor(routes): clean accept header conditionals

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
500

Relevant Lines:
500

💛  - Coveralls


Right click paste

Right click menu doesn't work on Ubuntu 16.04 with telegram and whatsapp.

Expected Behavior

Right click in chat window opens context menu.
Current Behavior

Context menu doesn't open.
Screenshots (if appropriate):
Possible Solution

Steps to Reproduce (for bugs)

right click in chat screen
nothing happens

Context

Your Environment

Franz Version used: v4.0.4 build 40, release 1.3.1 / linux / x64
Operating System and version: Ubuntu 16.04

@Longshot might be a Telegram web issue, did you try in the browser?

closing this for inactivity, feel free to reopen

Right-click Paste Not Working


fix(appeal): additional appeal policy override

bug:
policy example:
...
requirements:
  on:
    resource_id: 12345
  appeals:
    - resource:
         id: 6789
       role: viewer
       policy:
         id: policy-x # overriding policy
         version: 99

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

11 of 12 (91.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.07%) to 38.828%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core/appeal/service.go
11
12
91.67%

Totals

Change from base Build<PHONE_NUMBER>:
0.07%

Covered Lines:
2174

Relevant Lines:
5599

💛  - Coveralls


Condition is not updated after effect cancelation

Thanks for this very usefull module, I am using it with the 'Torch' one. As you can see in the video, the torch module is creating a 'ligth animation' for simulating a torch (or other light source). When the animation ends, the light condition is not updated. When the animation is created 'by hand' on the token itself, it works. Do you have any clue about this ?

Regards.

hmm not sure, i'll have to look at how it's terminating the 'light animation' as you say ( i.e. is it actually removing it, or just making it transparent/hidden ).
The module looks for certain hooks to be triggered to do a loop of the tokens... and if that removal isn't triggering one of those hooks...


How to convert llama family model to powerinfer.gguf?

If I want to use the llama-based Chinese-alpaca-2-13b model, how should I convert it to powerinfer.gguf format?

Thanks for your interest, but we only support ReLU-based models for now.
Please kindly refer to our FAQs and discussions from previous issues, including:

#56
#32
#17


Unnecessary warning when calling Admin.deleteTopicRecords with offset -1

Describe the bug
Admin Client log unnecessary warning when calling deleteTopicRecords with offset -1. Additional conditions may be missing.
https://github.com/tulios/kafkajs/blob/ba34eef11abdbeb69598947358fe220efb38d514/src/admin/index.js#L1045-L1054
To Reproduce
Simple code to reproduce issue:
const TOPIC = "some_topic";

const {Kafka} = require("kafkajs");

const kafka = new Kafka({
  brokers: ["localhost:9092"],
});

const admin = kafka.admin();

(async () => {
  await admin.connect();

  const topicOffsets = await admin.fetchTopicOffsets(TOPIC);

  await admin.deleteTopicRecords({
    topic: TOPIC,
    partitions: topicOffsets.map((x) => ({
      partition: x.partition,
      offset: "-1",
    })),
  });

  await admin.disconnect();
})().catch((error) => {
  console.error(error);
  process.exit(1);
});

Stdout:
{
  "level": "WARN",
  "timestamp": "1970-01-01T00:00:00.000Z",
  "logger": "kafkajs",
  "message": "[Admin] The requested offset is before the earliest offset maintained on the partition - no records will be deleted from this partition",
  "topic": "some_topic",
  "partition": 0,
  "offset": "-1"
}

Expected behavior
Admin Client must not log unnecessary warning.

Submitted pull request #1265 to address this issue

Closed by #1265


versionCode was not increased

Looks like you forgot to increase versionCode before building the last release; the APK shows the same VC as the release from October (33). Please remember to increase so Android accepts the update (and maybe replace the APK with a fixed one?). Thanks!

Looks like you still forget – there meanwhile were 4 releases with versionCode: 54. So existing installations didn't receive any update since (checks notes) around last August. @LagradOst can you please check and fix, and remember to always increase before releasing? :pray:


Make LayoutMaker s more generic

In addition of current XxxDidSetLayout protocols, add a new CanAddXxxLayout protocol, so it's able to write less code

Additional tasks:
[ ] Rename makeFrame(), which is used to generate a specific CGRect value while under layout, to evaluateFrame() to tell users this method should not be called in product (Since it's called as a protocol method it has to be exposed to user; Wishing someday I don't have to expose the protocol to users


Return non scannable assets in DisjoinAssetsInGroups

Problem
The endpoint (/team/{team_id}/assets) that retrieves the list of assets from a team does not return the correct list of groups when an asset is marked as non-scannable.
If the asset is marked as non-scannable, the global groups do not appear in the list of groups of the asset.
In addiction, the endpoint that retrieves a group's assets (/team/{team_id}/groups/{group_id}/assets) does not return non-scannables assets when the group is a global group.
Solution
The endpoints now return scannable and non-scannable assets.
Caution
Please, before merging this PR it's necessary to check with @julianvilas  in order to avoid any undesired effect on the services that use the affected endpoints.

The PR looks super good to me! But I think we need to be sure that this check being removed from here is performed in another part of the vulcan-api or the scan-engine.

Checking the scan creation workflow I think it should be safe to do it, but I'd suggest @manelmontilla to confirm this.
Workflow links:

https://github.com/adevinta/vulcan-api/blob/a14c3dd2101a22d3b8e07aefd87b0d0d7840e638/pkg/api/service/middleware/global/scans.go#L27
https://github.com/adevinta/vulcan-api/blob/a14c3dd2101a22d3b8e07aefd87b0d0d7840e638/pkg/api/service/scan.go#L81
https://github.com/adevinta/vulcan-api/blob/23f635dabb4e94cdb9e74e37f1398b4d080f3e7e/pkg/scanengine/scanegine.go#L87
https://github.com/adevinta/vulcan-crontinuous/blob/ca2ba0cb2adf94933de8517ecca8f89cc40ded9e/vulcanclient.go#L39
https://github.com/adevinta/vulcan-crontinuous/blob/c07ca7e88a18a28bba7d345d7e416c65babab8e2/scan_cron.go#L43


feat: add modal to import datasets

SUMMARY

Similar to https://github.com/apache/incubator-superset/pull/11884, this adds a modal to import datasets.
If the database doesn't exist and needs a password the user will be prompted for it; see the screenshot below.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

Tested importing virtual and physical datasets, both with the database already there as well as importing the database together with the dataset.
Added tests as well.
ADDITIONAL INFORMATION

[X] Has associated issue: https://github.com/apache/incubator-superset/issues/11167
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #11903 (daf9a14) into SO-1157 (4e7a189) will decrease coverage by 13.64%.
The diff coverage is 1.92%.

@@             Coverage Diff              @@
##           SO-1157   #11903       +/-   ##
============================================
- Coverage    67.89%   54.24%   -13.65%
============================================
  Files          919      413      -506
  Lines        45546    14708    -30838
  Branches      4245     3777      -468
============================================
- Hits         30923     7979    -22944
+ Misses       14520     6729     -7791
+ Partials       103        0      -103

Flag
Coverage Δ

cypress
54.24% <1.92%> (-0.75%)
:arrow_down:

javascript
?

python
?

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

Impacted Files
Coverage Δ

...nd/src/views/CRUD/data/dataset/AddDatasetModal.tsx
12.00% <ø> (-45.58%)
:arrow_down:

...ontend/src/views/CRUD/data/dataset/DatasetList.tsx
2.22% <0.00%> (-67.01%)
:arrow_down:

...nd/src/datasource/components/ImportModal/index.tsx
2.43% <2.43%> (ø)

...uperset-frontend/src/dashboard/util/dnd-reorder.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...rset-frontend/src/dashboard/util/getEmptyLayout.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...dashboard/components/resizable/ResizableHandle.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/dashboard/util/getFilterScopeFromNodesTree.js
0.00% <0.00%> (-93.48%)
:arrow_down:

...src/dashboard/components/gridComponents/Header.jsx
10.52% <0.00%> (-86.85%)
:arrow_down:

...rc/dashboard/components/gridComponents/Divider.jsx
13.33% <0.00%> (-86.67%)
:arrow_down:

...ontend/src/dashboard/util/getDashboardFilterKey.ts
14.28% <0.00%> (-85.72%)
:arrow_down:

... and 764 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 4e7a189...01afd38. Read the comment docs.

The Docker build didn't like the branch name generated by ClubHouse, closing this since I've renamed the branch and pushed https://github.com/apache/incubator-superset/pull/11910.


Replace removed replaceAllLayersWithLayers API (Sketch 45) with addLayers

Fixes https://github.com/airbnb/react-sketchapp/issues/133.
Alternative solution to https://github.com/airbnb/react-sketchapp/pull/134 as this also removes all the existing layers before adding the new layer (effectively simulating replaceAllLayersWithLayers).

Formerly, with the replaceAllLayersWithLayers method, one could theoretically render into any layer. Now, with the API change it is only possible to render into LayerGroups.
I believe detecting this and throwing a useful error would be improve the user experience. Can see my earlier PR for an example.

wooooooo thanks @donysukardi & @gdowens. this works with 43-44 too?

Tested it against 43-44 and it's working great!

Thanks for the suggestion @gdowens! I've added the check to throws meaningful message if the given container does not support children.