Heap out of memory after many hours

Running vite for many hours (within a workday) causes heap out of memory
<--- Last few GCs --->

[57364:0x138040000] 19512444 ms: Mark-sweep 4036.0 (4139.3) -> 4030.6 (4142.6) MB, 737.7 / 0.0 ms  (average mu = 0.461, current mu = 0.052) allocation failure; scavenge might not succeed
[57364:0x138040000] 19513801 ms: Mark-sweep 4040.6 (4144.0) -> 4035.3 (4147.0) MB, 1341.2 / 0.0 ms  (average mu = 0.226, current mu = 0.011) allocation failure; scavenge might not succeed

<--- JS stacktrace --->

FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory

 1: 0x1007b3c40 node::Abort() [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]
 2: 0x1007b3e24 node::ModifyCodeGenerationFromStrings(v8::Local<v8::Context>, v8::Local<v8::Value>, bool) [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

 3: 0x10090b608 v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, bool) [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

 4: 0x100ab5eec v8::internal::EmbedderStackStateScope::EmbedderStackStateScope(v8::internal::Heap*, v8::internal::EmbedderStackStateScope::Origin, cppgc::EmbedderStackState) [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

 5: 0x100ab4894 v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace, v8::internal::GarbageCollectionReason, v8::GCCallbackFlags) [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

 6: 0x100aa8928 v8::internal::HeapAllocator::AllocateRawWithLightRetrySlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

 7: 0x100aa916c v8::internal::HeapAllocator::AllocateRawWithRetryOrFailSlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

 8: 0x100a8f0ec v8::internal::Factory::NewFillerObject(int, v8::internal::AllocationAlignment, v8::internal::AllocationType, v8::internal::AllocationOrigin) [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

 9: 0x100e2f80c v8::internal::Runtime_AllocateInYoungGeneration(int, unsigned long*, v8::internal::Isolate*) [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

10: 0x10117d04c Builtins_CEntry_Return1_DontSaveFPRegs_ArgvOnStack_NoBuiltinExit [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

11: 0x10114ee30 Builtins_CloneObjectIC [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

12: 0x10698bb68

13: 0x101108198 Builtins_InterpreterEntryTrampoline [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

14: 0x101108198 Builtins_InterpreterEntryTrampoline [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

15: 0x101108198 Builtins_InterpreterEntryTrampoline [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

16: 0x101108198 Builtins_InterpreterEntryTrampoline [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

17: 0x101108198 Builtins_InterpreterEntryTrampoline [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]
18: 0x1068e0b38

19: 0x101139ef4 Builtins_AsyncFunctionAwaitResolveClosure [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

20: 0x1011c8738 Builtins_PromiseFulfillReactionJob [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

21: 0x10112bc4c Builtins_RunMicrotasks [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

22: 0x1011063a4 Builtins_JSRunMicrotasksEntry [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

23: 0x100a37d38 v8::internal::(anonymous namespace)::Invoke(v8::internal::Isolate*, v8::internal::(anonymous namespace)::InvokeParams const&) [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

24: 0x100a38228 v8::internal::(anonymous namespace)::InvokeWithTryCatch(v8::internal::Isolate*, v8::internal::(anonymous namespace)::InvokeParams const&) [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

25: 0x100a38404 v8::internal::Execution::TryRunMicrotasks(v8::internal::Isolate*, v8::internal::MicrotaskQueue*, v8::internal::MaybeHandle<v8::internal::Object>*) [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

26: 0x100a5eb6c v8::internal::MicrotaskQueue::RunMicrotasks(v8::internal::Isolate*) [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

27: 0x100a5f308 v8::internal::MicrotaskQueue::PerformCheckpoint(v8::Isolate*) [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

28: 0x1006f8c4c node::InternalCallbackScope::Close() [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

29: 0x1006f87c4 node::InternalCallbackScope::~InternalCallbackScope() [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

30: 0x1007b7464 node::fs::FileHandle::CloseReq::Resolve() [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

31: 0x1007cdef8 node::fs::FileHandle::ClosePromise()::$_0::__invoke(uv_fs_s*) [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

32: 0x1007afbb0 node::MakeLibuvRequestCallback<uv_fs_s, void (*)(uv_fs_s*)>::Wrapper(uv_fs_s*) [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

33: 0x1010e42e8 uv__work_done [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

34: 0x1010e7aa4 uv__async_io [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

35: 0x1010fa058 uv__io_poll [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

36: 0x1010e7f74 uv_run [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

37: 0x1006f96e0 node::SpinEventLoop(node::Environment*) [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

38: 0x1007f0850 node::NodeMainInstance::Run() [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

39: 0x1007806a4 node::LoadSnapshotDataAndRun(node::SnapshotData const**, node::InitializationResult const*) [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]

40: 0x10078095c node::Start(int, char**) [/Users/deathemperor/.nvm/versions/node/v18.18.2/bin/node]
41: 0x184a010e0 start [/usr/lib/dyld]

error: "vite" exited with code 6 (SIGABRT)

Do you feel like maybe this is related to this plugin? Looking at the output here I think it's hard to say what the source of the memory leak is. I would be inclined to think because the surface area of this plugin is relatively small compared to Vite and GraphQL Codegen (and this is just some glue between them) that it may be something to do with them.
That said, if you have any more insight that might help provide a clue or hint at it being this plugin I'm open to investigate!

I have no ideas how these memory issues are and they are normally not easy to investigate. I'm more on running vite with bun but as of today this https://github.com/oven-sh/bun/issues/887 prevents it. will have to wait.


In-memory Engine: do not evict range on applying snapshot

What is changed and how it works?

Issue Number: ref #16141

What's Changed:
Do not evict the cache when applying a snapshot because the peer
must be in the follower role during this process. The cache is
already evicted when the leader steps down.

Check List
Tests

Integration test

Release note

None

/approve


Data question

hi, tkipf:
I get a question about datasets.  As far as I know, all three datasets you use are directed graph data. But in your experiment, its seems like undirected.

Everything is undirected in our experiments. We are indeed using citation
network datasets, but we treat citation links as undirected edges.
On Sun, Aug 25, 2019 at 11:38 PM yujack333<EMAIL_ADDRESS>wrote:

hi, tkipf:
I get a question about datasets. As far as I know, all three datasets you
use are directed graph data. But in your experiment, its seems like
undirected.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tkipf/gae/issues/49?email_source=notifications&email_token=ABYBYYAAGEDF2FMK5NYELZLQGN27BA5CNFSM4IPMKB5KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HHJSWSQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABYBYYF2XABYLHTGT5RFJATQGN27BANCNFSM4IPMKB5A
.

ok,thank you.


Drag & Drop Card

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
Add Drag & Drop Card
Describe alternatives you've considered
Nothing
Additional context
Sortable Library used in this Card
https://user-images.githubusercontent.com/73753957/180598117-7379771c-56ad-4b2c-8129-d87b33d342b4.mp4

@siddhi-244 Assign

Assigning it to you


Add config option to enable/disable size indicator on resize

I'd like it to be false by default but just kept it to true for no particular reason. What you guys think.

I'd like it to be false by default but just kept it to true for no particular reason. What you guys think.

I think that false is also good, personally i dont see this indicator to much, but i guess if  it was bothering you we can set it to false


Fix next_background on PowerLineDecoration

I think you forgot this, it doesn't update the next_background color:

Thanks.
Actually, I didn't forget it because I tested it locally and it worked. What I did forget was to save that change before pushing the PR!!


Bootcamp dev

Ticket No

https://trello.com/c/lW3WXqYC
https://trello.com/c/94zi5wpb

Summary of changes

Changed the navigation menu title from angie to Project Zoe and also menu item Chat to Connect

How should this be tested? [Screenshots, Video or Type instructions]

screenshot

Notes for reviewers

Notes for reviewers here

Out of scope

Out of scope here

Alright. This is probably why my client side is currently not working as it
should
On Tue, Aug 23, 2022, 11:58 AM Jackson Onyango @.***>
wrote:

@.**** requested changes on this pull request.
@FREDVUNI https://github.com/FREDVUNI I have left a few comments that
need to be addressed.
In src/modules/login/Login.tsx
https://github.com/kanzucodefoundation/project-zoe-client/pull/118#discussion_r952339304
:

@@ -32,10 +36,12 @@ function Login() {

     history.push(localRoutes.profile);

   },

   () => {

   console.log("whyy")

This console log should be removed.
In src/modules/login/Login.tsx
https://github.com/kanzucodefoundation/project-zoe-client/pull/118#discussion_r952339705
:

// dispatch(handleLogin({}));

Toast.success('Authentication success');

history.push(localRoutes.profile);

if ((1 + 3) == 2) {

 post(

There should be no changes here
In src/index.tsx
https://github.com/kanzucodefoundation/project-zoe-client/pull/118#discussion_r952340494
:

+// const SENTRY_STATUS = process.env.NODE_ENV === 'production';

+// Sentry.init({
+//   dsn: process.env.REACT_APP_SENTRY_DSN,
+//   autoSessionTracking: SENTRY_STATUS,
+//   integrations: [
+//     new Integrations.BrowserTracing(),
+//   ],
+//   tracesSampleRate: 0.5,
+//   beforeSend: (event) => {
+//     if (
+//     // Exclude "localhost" envs from issue tracker
+//       window.location.hostname === 'localhost'
+//     ) {
+//       return null;
+//     }
+//     return event;
+//   },
+// });
This should not be commented out
In src/index.tsx
https://github.com/kanzucodefoundation/project-zoe-client/pull/118#discussion_r952340900
:

+// import * as Sentry from @.***/react';

+// import { Integrations } from @.***/tracing';
This should not be commented out
In src/components/layout/NavMenu.tsx
https://github.com/kanzucodefoundation/project-zoe-client/pull/118#discussion_r952341675
:

@@ -195,7 +197,8 @@ const NavMenu = (props: any) => {

     alignContent="center"

     justify="center"

   >

   {/*  */}

It should be safe to delete this line since we are replacing it with the
line below it.
In src/components/layout/NavMenu.tsx
https://github.com/kanzucodefoundation/project-zoe-client/pull/118#discussion_r952341895
:

@@ -56,7 +56,7 @@ const routes: IAppRoute[] = [

 icon: TodayIcon,

},
{

name: 'Chat',

name: 'Connect',

👍🏾
In package.json
https://github.com/kanzucodefoundation/project-zoe-client/pull/118#discussion_r952342717
:

@@ -18,6 +18,7 @@

<EMAIL_ADDRESS>"^4.9.1",

<EMAIL_ADDRESS>"^4.0.0-alpha.54",

<EMAIL_ADDRESS>"^3.2.10",

@.***/styles": "^4.11.5",

Why did you need to add this package?
In .eslintignore
https://github.com/kanzucodefoundation/project-zoe-client/pull/118#discussion_r952343125
:

@@ -4,3 +4,4 @@ my-app*

packages/react-scripts/template
packages/react-scripts/fixtures
fixtures/
+src/*
This folder should not be ignored
—
Reply to this email directly, view it on GitHub
https://github.com/kanzucodefoundation/project-zoe-client/pull/118#pullrequestreview-1081738271,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ6ME2CMXJVLLVI7PE22HQ3V2SHDRANCNFSM57ILLABA
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

Ticket No

https://trello.com/c/00zJRFve

Summary of changes

Edit person details through pop-up modal in place within the Persons Table
Closing modal dialog returns to the people page when the URL has parameter showEdit

How should this be tested? [Screenshots, Video or Type instructions]

screenshot

Notes for reviewers

Notes for reviewers here

Out of scope

Out of scope here

Ticket No

https://trello.com/c/DCDA6rAH

Summary of changes

Removed coming soon text and moved the send email button

How should this be tested? [Screenshots, Video or Type instructions]

screenshot

Notes for reviewers

Notes for reviewers here

Out of scope

Out of scope here

Ticket No

https://trello.com/c/tUAwrgWp

Summary of changes

Changed the landing page from profile to dashboard

How should this be tested?

Demo

Notes for reviewers

Notes for reviewers here

Out of scope

Out of scope here


Add snmp on vqfx, reconfigure compute resources on the servers, move …

…s-srv2 to vqfx1

Merge the changes required as per workshop.


chore(fuselage, fuselage-tokens): Update button-on-secondary-disabled color

Proposed changes (including videos or screenshots)

Issue(s)

Further comments

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Missing auth_client_demo in the github repository

I am pretty disappointed to again not be able to advance in developing an app on the ICP by followng the documentation.
I want to create a dapp that 1. Logs in.
The auth_client_demo has been removed from the  github repository.
Maybe a dependency could be executed to ensure this does not happen?
It must be disheartening for Jessie to put all the effort into making the developers journey/ladder and then things break.

Hey @testpuddle , this example got moved to an archive folder in the repo: https://github.com/dfinity/examples/tree/master/archive/motoko/auth_client_demo
There's probably a few links that need to be updated. I'll take a look, thanks!


Feature Request: allow for moving different task patterns to different pages

I would like to move cancelled tasks items [-] to % - Scrapped pages (what I would write under "separate archive file name"), and completed task items [x] to % - Archived pages.

Hello @YousufSSyed,
Thank you for your input! Check out 0.16.1. You can now configure different files for different task statuses:

Hi, @YousufSSyed,
I tried it out with the settings you provided, and it worked as expected. Could you please share the file in which you ran the command and your settings file (<your-vault>/.obsidian/plugins/obsidian-task-archiver/data.json). Also check what you see in the console when you run the command (you can see it by pressing F12).

Before I thought of doing that, I uninstalled and reinstalled the plugin, and then manually changed all the settings to match the screenshot. It works now.
Just so you know, a while ago I actually made a second copy of the plugin to handle [-] tasks, and created a macro with Better Command palette to run the archive tasks command from both copies. Maybe something happened when I copied it, idk.


combine_latest thread safe

when I try to run this code, it turns out that the member is_set of the class rxcpp::util::detail::maybe is false and std::terminal() was called, like this:

anyone can help me?

`#include
#include "rxcpp/rx.hpp"
#include
#include <unistd.h>
using test = std::shared_ptr;
void fun(rxcpp::subjects::behavior subject, int us, int init){
int init_ = init;
while(true){
init_ += 2;
subject.get_subscriber().on_next(std::make_shared(init_));
usleep(us * 100);
}
}
int main()
{
rxcpp::subjects::behavior a(std::make_shared(1));
rxcpp::subjects::behavior b(std::make_shared(2));
auto c = rxcpp::synchronize_new_thread();
a.get_observable().observe_on(c)
 .combine_latest(b.get_observable().observe_on(c))
 .subscribe([](std::tuple<test, test> tuple){
	 auto one = std::get<0>(tuple);
	 auto two = std::get<0>(tuple);
	 std::cout << "thread:" << std::this_thread::get_id() << " [" << *one << "," << *two << "]" << std::endl;
 });

std::thread a_(&fun, a, 1, 0);
std::thread b_(&fun, b, 2, 1);
a_.join();
b_.join();
return 0;

}`

how can i promise thread safety about using combine_latest with two different observable in two different thread

Try to pass "coordination" to combine_latest, e.g.
o1.combine_latest(rxcpp::observe_on_new_thread(), o2)
For operators receiving it's input from two or more observables injected by multiple threads, you should put coordination to handle the synchronization.

Try to pass "coordination" to combine_latest, e.g.
o1.combine_latest(rxcpp::observe_on_new_thread(), o2)
For operators receiving it's input from two or more observables injected by multiple threads, you should put coordination to handle the synchronization.

thank u


Type mismatches in exports and headers

The header file that describes the function uses a different type than the actual implementation - this may lead to miscompiles / optimization errors as there may be subtle differences between bool and limb_t.
https://github.com/supranational/blst/blob/a8398ed284b0d78858302ad1ceb25a80e7bbe535/bindings/blst.h#L142
https://github.com/supranational/blst/blob/a8398ed284b0d78858302ad1ceb25a80e7bbe535/src/exports.c#L244
blst/bindings/blst_aux.h:17:6: warning: type of ‘blst_p1_affine_is_equal’ does not match original declaration [-Wlto-type-mismatch]
   17 | bool blst_p1_affine_is_equal(const blst_p1_affine *a, const blst_p1_affine *b);
      |      ^
blst/src/exports.c:241:8: note: return value type mismatch
  241 | limb_t blst_p1_affine_is_equal(const POINTonE1_affine *a,
      |        ^
blst/src/exports.c:241:8: note: type ‘limb_t’ should match type ‘_Bool’
blst/src/exports.c:241:8: note: ‘blst_p1_affine_is_equal’ was previously declared here
blst/src/exports.c:241:8: note: code may be misoptimized unless ‘-fno-strict-aliasing’ is used

this may lead to miscompiles / optimization errors

How? Well, given that you compile blst C source separately, as you're expected to (see below). Either way, it should be noted that formally speaking C standard is incomplete. Most notably it doesn't prescribe anything about calling convention. In other words one can't consider it in total isolation, but complement the context with something else, most notably platform-specific ABI specifications. Now, with this in mind, the assertion is that as long as the limb_t type doesn't exceed "natural" register size, and as long as its value is 1 or 0, it can be safely reinterpreted as it's currently done. On supported platforms. That's the keyword, "on supported platforms," or more specifically for relevant ABIs at hand.
Moving on. But there is something else that C standard is silent about... About how booleans are to be ... "instantiated" if you wish. More specifically it doesn't say that code evaluating (bool)x, where x is non-boolean, should be constant-time. But we have to have constant-time code! This is a non-negotiable requirement! Nor can it be left to compiler's mercy... Two possible solutions: a) implement everything in assembly; b) require separate compilation and utilize guarantees implied by relevant ABIs. Life is a trade-off, b) it is. This is why blst has to internally return non-boolean type, but it does return boolean value.
Now, you can say "but why doesn't blst.h return limb_t instead of bool?" Well, because there is another level of abstraction. Note that in blst.h limb_t is unconditionally 64-bit type, while internally it can be 32 bits. In other words, limb_t is effectively an opaque type to application. You can ask why aren't the types incomplete then? Well, because we didn't want to manage memory in the library. Why? Because we didn't want to limit application language to C...
Now, you can say "but why doesn't blst.h return say size_t then?" Yeah, but it would make interface to foreign languages less elegant. Not to mention that it might trigger casts, which, again, are not guaranteed to be constant-time...
Bottom line. It's not an oversight, and there is rationale behind it. And compile separately.

Two possible solutions:

Well, formally speaking there is a 3rd option, implement the constant-time cast from non-boolean to boolean in assembly... It feels like an overdo, but if we come to the conclusion that separate compilation is not sufficient guarantee after all, it would be the way to go...

Oh! I notice that you ought to come from nim, which means that you rather not compile it separately... Well, bool is not really a native C type, and it's actually defined in blst.h. So one can actually discuss custom definition. I mean if there is a macro defined that says "I come (with peace;-) from nim," we can add an additional #if to harmonize it for compile-together scenario...

Actually, not so much nim, as LTO :) We're considering disabling it for libraries like BLST though that are already well-optimized as it might interfere with, like you mention, constant time.
I guess the difference that can happen is that some compiler optimize more aggressively around bool, assuming that it takes a value of 0 or 1 exclusively which might affect code paths down the road.
Aliasing should be fine I guess since it's a return value that doesn't have an address - though in theory a function pointer could get messed up.

assuming that it takes a value of 0 or 1 exclusively

And as already implied, it would be 100% correct assumption. Because all the subroutines that are declared bool in blst.h are explicitly coded to return 1 or 0 [in constant time]. They do not return a non-0 or 0.
As for LTO. I customarily argue against it for any cryptographic algorithm. Because there is direct contradiction between the goals. I mean security countermeasures are most likely to make code slower, while optimization's goal is to produce fastest code possible. And there is no guarantee that having the complete picture it won't find a way to bypass the countermeasure.

On a tangential note. More specifically on bool type in constant-time context. I've run into an article that pointed out that llvm has select IR opcode and that the corresponding machine code tends to be generated as a conditional branch. And that the select opcode tends to be emitted in response to (a & mask) | (b & ~mask) pattern. So I wondered... What if the loop body in vec_select looked as
rp[i] = (ap[i] & mask) | (bp[i] & ~mask);

where mask is calculated as 0 - sel_a. No, no select in sight... Well, not really surprising, as I might as well worked on individual bits. But then I've changed type of sel_a argument to _Bool and there it was, select:
  %14 = load i64, i64* %13, align 8, !tbaa !2
  %16 = load i64, i64* %15, align 8, !tbaa !2
  %17 = select i1 %4, i64 %14, i64 %16
  store i64 %17, i64* %18, align 8, !tbaa !2

I've omitted some instructions from the above transcript, but left the most important ones, two loads. This is what we want, right? I mean for constant-time-ness we want two loads, some calculations, and store. So it looks proper. But what about the final machine code? Well, it didn't have a conditional branch, not in this case, but the compiler transformed the IR to a selection between pointers instead of values and ended up using one load, thus making code non-constant-time:-(
So a non-bool type appears more suitable for the purpose, but does it actually guarantee the constant-time-ness? As mentioned earlier, even though blst returns a non-bool type, it's Boolean by value, and formally speaking, if given the opportunity, compiler could figure it out and apply same optimizations as to bool type. Or is there some provision in the standard that would prevent the compiler?
Just in case for reference, the loop in vec_select subroutine is effectively dead, because all possible input lengths are covered by assembly vec_select_N subroutines. Not that they, vec_select_N, were explicitly motivated by this bool thing, but they effectively counter all imaginable problems.

Or is there some provision in the standard that would prevent the compiler?

I'd think often ends up being more of a practical barrier than a standard issue due to how c files typically are compiled in separate translation units - as long as compilation, optimization and translation to machine instructions is done in isolation, optimization information such as possible value constraints are not passed between the TU's - with LTO however, that changes in two ways: value constraints that stem from flow analysis (and not type information) can now be applied in interprocedural analysis and ABI constraints can be relaxed as symbols no longer need to conform to an ABI description which incidentally also allows constant specialization and more aggressive inlining to take place.
Conversely, when returning bool, the compiler adds instructions to clamp the value, if along a code path it cannot deterministically say that the value will be 1 or 0, it will add instructions accordingly to clamp it.
FWIW, we've switched off LTO for now for our crypto dependencies, including blst, but it's a bit of a blunt way to address the problem - what I'd ideally like to do is to extend our compiler to annotate the llvm IR it generates with specific optimization constraints.

Couple of closing comments.
As it turns out the above mentioned (a & mask) | (b & ~mask) is not the only recognized candidate pattern for select opcode. I've tried to disincentivize clang (there are couple commits), and confirmed that it stopped generating select this time, but it's not an absolute guarantee for all the future. Again, only non-assembly platforms can suffer from this.
Secondly, as time progresses, the initial assertion about supported ABIs is no longer rock-solid. Because wasm doesn't have as well-defined ABI as more traditional platforms. As result, it's int that gets reinterpreted now. Yes, reinterpretation is still going on, for same reason, constant-time-ness [on C side], but it should be less controversial.


Support creating ajv bundle in Windows

Hi,
While install ajv throug npm I'm getting error. Attached is the error log "npm-debug.txt"
npm-debug.txt
I was also not able to create separate UMD bundle for browser using npm run bundle script. Attached in the error log "npm-run-bundle-debug.txt" Is it required that I install ajv before creating the bundle?
npm-run-bundle-debug.txt
Let me know what could be the problem here.
Thanks.

Can't reproduce, although I tried with npm 3.9.3 (I used nvm to install, that's the latest it has), yours is 3.9.5

Ah, just noticed, you have windows...

Also, did you just 'npm install'ed or 'git clone'd and then installed?

Yes. I'm trying it on windows.
I tried two ways. First downloaded the zip and copied in into file system and then tried npm install. Then also tried to git cloned and then install, however the issue was same.

It all works on mac...
Can you at least install ajv as a dependency?
The only existing circular dependency, if that's what it is, is that ajv depends on gh-pages-generator that depends on ajv. gh-pages-generator is a dev dependency (although it makes no difference on mac, it works ok if it's a normal dependency as well)... Can you maybe try removing it from package.json and run npm install without it?

@sushant160185 Can you provide us with a package.json of your project so we can try and reproduce this?

It's in repo: https://github.com/epoberezkin/ajv/blob/master/package.json

@ralphtheninja Ah, sorry, got it :)

As I understood, @sushant160185 did just the following:
git clone git@github.com:epoberezkin/ajv.git
cd ajv
npm install

So ajv is not a dependency, it was installed separately.
@sushant160185 is it correct?

Works well on linux as well. I suspect something is fishy here ;)

@epoberezkin Yes, all I did was create a clone of ajv (using git desktop), "cd ajv" and "npm install ajv" and I got this error. I haven't changed or added anything in the ajv repository. I'm trying it on windows 7 os.
@ralphtheninja package.json is part of the ajv repository. No changes done in that.
I'm new to this npm environment so if you can give me some steps on how should I do it on windows environment then it will be helpful for me.
Thanks.

Did you try removing gh-pages-generator from package.json before npm install?

Yes, I did. Seems it still gave same exception. Attached is the log.
npm-debug.txt

I'm new to this npm environment so if you can give me some steps on how should I do it on windows environment then it will be helpful for me.

You shouldn't need to do anything special. Unfortunately I don't have access to a windows machine at home :/ I can try tomorrow.

@sushant160185 Can I ask what are you trying to do in general? To make some change in ajv?

No, not to change anything in ajv. I wanted to try this in a simple browser based app. So at first I tried creating bundle from ajv package, however, "npm run build" script failed. I then tried to do installation first and then check the build process, however, that also failed.
In the meantime, I picked up the file from cdnjs site for my app so as such I'm not stuck but was just hoping that I should be able to install and/or create the bundle independently.

Bundle is included in npm package. If you make ajv a dependency in your app (npm install ajv --save), there will be a bundle in the folder node_modules/ajv/dist. Using CDN is also an option.
I still need to figure out what is wrong with npm install inside ajv in windows.

It's not a bug, it's caused by the command you've tried to run. You don't want to do npm install ajv inside ajv - you can not install the module inside itself. If you're trying to use it locally, you can use npm install (no ajv) to install dependency and npm link to link it to other places. Most likely, you just want to install ajv inside your project - so you'd cd into your project and run npm install ajv there.

Thanks @blakeembrey. This copied the ajv and other dependencies inside my project app with some warnings.
For the package creation for web ("npm run bundle" mentioned in the browser documentation section), what should I do since there is no bundle script in this copied folder. Although as @epoberezkin mentiond, it already has those files inside dist folder. Still how to create the ajv bundle for browser usage? Can you give me the steps?

The bundle shipped with ajv can be used as is - this is recommended.
If you really want to re-build you need:
git clone git@github.com:epoberezkin/ajv.git
cd ajv
npm install
npm run build
npm run bundle

@epoberezkin  Thanks. I ran these commands. Everything ran fine except npm run bundle, which gave error

'.' is not recognized as an internal or external command,operable program or batch file.

Most likely because https://github.com/epoberezkin/ajv/blob/master/package.json#L21 isn't a Windows executable. I think if you change ./scripts/ to node script/ it will work on Windows.

@blakeembrey I made the changes however it still failed with following error.

ajv\scripts\bundle:9
name=$(./scripts/info $package name)
^
SyntaxError: Unexpected token .
at Object.exports.runInThisContext (vm.js:53:16)
at Module._compile (module.js:513:28)
at Object.Module._extensions..js (module.js:550:10
at Module.load (module.js:458:32)
at tryModuleLoad (module.js:417:12)
at Function.Module._load (module.js:409:3)
at Function.Module.runMain (module.js:575:10)
at startup (node.js:160:18)
at node.js:456:3

@sushant160185 You have to fix lines 9-12 in bundle script in the same way.

I did that as well. The error this time was:

ajv\scripts\bundle:9
name=$(node scripts/info $package name)
^^^^
SyntaxError: missing ) after argument list

in 4.11.5


Error while opening PO?

Hi All,
So what i am getting , when trying to open a purchase order on SANDBOX environment in CRONUS USA Inc.

However, this is not causing any issue in my work.
Intent is just letting know about this error.
Thanks

Hi @SUD-D365, thanks for reporting this issue, we'll take a look.

Hello @SUD-D365 and thank you for reporting the issue.
It looks like one of the knows issues that we looked into recently, did you have "Microsoft Dynamics GP" extension installed in your sandbox?

Hello @IhorHandziuk , thanks for prompt revert.
Yes i have installed below extensions, If you are talking about one of these. But these extensions came with default sandbox.

Thank you for the quick response. This bug will be fixed in an upcoming update. As a mitigation we can suggest to uninstall these extensions.


Allow custom jwt_payload_handler

Like rest_framework_jwt, it would be nice to be able to use our own function for payload

https://jpadilla.github.io/django-rest-framework-jwt/

JWT_PAYLOAD_HANDLER setting

Function code: https://github.com/jpadilla/django-rest-framework-jwt/blob/master/rest_framework_jwt/utils.py

Thanks a lot

@SorianoMarmol Can you make a PR of any implementation perhaps?

@Andrew-Chen-Wang : Hey, I'd like to work on this.

@bhaveshpraveen Go ahead!

@bhaveshpraveen It might be easier if there is a callable, like a path to a function (I think). The functionality is technically already supported: https://github.com/SimpleJWT/django-rest-framework-simplejwt/issues/162
We can further review it later, but I think it's safe to close this as a duplicate.


Request from Red Cross SL : Change status ticks to check boxes with spacing

The Red Cross mentioned that the status line and ticks were not clear enough, and were confusing to them.

They would prefer the following:

[ ] Check boxes which clearly show what the current status is
[ ] Greyed out statuses that have been achieved

I would like to work on this

I would like to work on this

Go ahead @Tharindu37

@anjula-sack @nshamit How about this?

I think this looks good.


instapdf and cordova

Hello,
I want to implement your solution instapdf in my example with cordova.
Below are the steps that I made
cordova create NativeCamera fr.test.nativecam NativeCamera
cd NativeCamera
cordova platform add ios
cd platforms/ios/
pod init
then I add  pod "IPDFCameraViewController" to my pod file
pod install
Pod installation complete! There is 1 dependency from the Podfile and 1 total pod
installed.
Next, I add IPDFCameraViewController/IPDFCameraViewController.h andIPDFCameraViewController/IPDFCameraViewController.m
When I wnt build my application I have this error
diff: /../Podfile.lock: No such file or directory
diff: /Manifest.lock: No such file or directory
error: The sandbox is not in sync with the Podfile.lock. Run 'pod install' or update your CocoaPods installation.
Ticket: https://instapdf.freshdesk.com/helpdesk/tickets/20

Sorry this is not supported in this library


Update openvino to 2022.3 in requirements

New version of OpenVINO has improved support for Stable Diffusion on Intel CPU and GPU. Also, 4th Gen Xeon is now supported and stable_diffusion.openvino will be used in a webinar on Jan 24th. Please consider upgrading requirements.txt to use OpenVINO 2022.3 LTS.
Thank you,
Ryan

@ryanloney great news :muscle:


Enable Plugin Upload -> more specific where

Summary
As the setting is only available in the config.json, i think it is better to write it out. See also https://community-daily.mattermost.com/core/pl/qhcbmsg3njygjr396iujmr9yur

Reviewed with @JtheBAB and my feedback was incorrect in this context. Thanks for the update, @JtheBAB!


Introduce unified manifest file

Closes #1321
This writes out unified checksum files like sha256.sum or sha512.sum (which are compatible with GNU-style sha256sum -c sha256.sum invocations) depending on which checksum style is chosen from the dist config.
It's a new global artifact which is built after all the local artifacts, re-using the checksums computed earlier. The plan is to eventually stop publishing individual checksum files like some-binary.sha256 and only publish the unified checksum file, which will reduce noise / make publishing faster, etc.

I wonder if we might want to consider adding the hash file to the snapshots, so we can confirm the format stays stable?

Just did that, which taught me about the snapshotting system for tests.

Hm, I guess the NPM package hash isn't actually stable, is it?
If you're interested in learning about snapshot filtering, we could use that here to remove the unstable hash from the snapshots. Or we could punt on snapshotting the file contents at this point.

Just noting here that @ashleygwilliams asked me to document why the npm packages aren't stable — which I will do ~around merging this PR, because I'll probably just end up skipping/censoring the checksums from the snapshots for the time being.

Okay I guess none of our tars are stable right now so.. I'll just remove the unified manifest file from the snapshots for the time being? or just censor everything, but that's silly

Right, I forget it doesn't hash the text installers, which are the things that would be stable. Yes, fine to drop it at this point.


Support for creating sketches other than by drawing

Request for discussion cc @underbluewaters, @avmey, @wmcclin, @petermenzies
I know the legacy shapefile import hasn't found it's way to next, and there are reasons we don't like it.  But I believe import of sketches has real need, and could be tweaked to support some additional use cases we have:

importing a network (sketch collection) of existing MPA's into seasketch. (This feels essential to Azores, and eventually others as greenfield mpa planning becomes no longer the norm but I'm open to thoughts.).  The workaround of producing a visual map layer and trying to trace it for all the MPA's and attributing them by hand is not great and makes it harder and less accurate than it needs to be in my opinion.
importing handcrafted test sketches produced by a developer into seasketch for demonstrating bugs, report behavior, etc.
importing sketches produced externally such as via QGIS, or another planning tool, etc. into seasketch (Bermuda this is being done in legacy like it or not, and we would benefit from being able to add them as multipolygon (because it's over 100 polygons derived from data where we don't need them to be individual sketches)
importing sketch types into seasketch that aren't supported by the drawing tools but seasketch internals and the reports can/will be able to work with (e.g. multipolygons, polygons with holes not produced by preprocessor).

Requirements:

Be able to add polygons and multipolygon at least.
full sketch attributes can be provided in the json payload, that seasketch can't otherwise create.
support for individual sketches or a whole collection.

Simplest implementation, the UX could be:

after selecting to create an instance of a sketch class (or collection), if a superuser you have access to a relatively hidden input, a textbox or something for pasting geojson (sketch geojson specifically) as an alternative to drawing.
preprocessor on submit may be optional, need to discuss
sketch attributes could be provided up-front in the text payload, with validation.

This is a "thar be dragons" type feature that would be for powerusers only.  Limiting access will limit misuse.
I'm not suggesting this is a high priority now.  But at some point I think it will be.

I agree this will be essential to have, primarily for existing MPAs. This has lead to a lot of grief in the past and it's worth listing out why.

We end up with shapes that cannot be practically edited. This is obviously problematic for a collaborative design process if shapes can't be iterated on. The Simple Shape -> Complex Geometry Product design pattern needs to be preserved somehow and often that's just impossible for shapefile uploads. (For use-cases like drawing a circle, we could address that with digitizing tools instead of uploads).
People often create shapes that are clipped to the shoreline... a different shoreline. If we perform clipping again that creates even more needlessly complex shapes.
These more complex shapes often can't be analyzed as efficiently and reports may time out. This often hits us and our users unexpectedly, creating an urgent problem to sort out. I've seen 25MB+ sketches uploaded in the past and when that happens there really isn't any practical solution.

Power users and project admins are the ones who usually cause these issues, so limiting these features to just admins wouldn't likely help.
For existing MPAs I think we need to have some sort of workflow that addresses issue 1. In principal everything that is a "sketch" ought to be editable by end users. I'm not sure how to get there though.

I would be okay with it only being possible by a superuser, so that you have to work with someone that understand the nuances. of this, and it's only use for use cases that make sense.

Another option might be documenting an API we can hit to publish sketches and sketch collections.  Which might have looser requirements than the drawing (e.g. allow for multipolygons)


Slider (Potentiometer) Module Firmware Support

Support for a potentiometer based module in the module firmware, as well as data processing by master board

Completed in branch/gamepad-firmware


Hotfix: Catapult client aggregate v2

BREAKING - catapult client register aggregate transaction hash validator and require aggregate version 2 after fork.  Client version is <IP_ADDRESS>.
rest-gateway update to v2.4.2
symbol-sdk update to v2.0.3

Are you able to release? please update dev when finished

You may want to add https://github.com/fboucquez/symbol-bootstrap/pull/377


chore: More Badges

:tada: This PR is included in version 0.5.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Stop drag mid-drag

I have an interaction (long press) that needs to cancel the drag and fire another event after x milliseconds, figured  I just fireactions.setNodesDraggable(false) and set to true afterwards, but it doesen't seem to work mid drag. Any way to implement this?

Could you create a simplified codesandbox or repo so that we better understand the issue?

👆

Have you found a solution? I'm encountering the same issue

Hey @mikkmartin, have you already updated to React Flow version 10? The updates includes changes in the event handling as well, so I would be interested to know if this issue still persists. Thanks!

Please re-open if the issue still persists.


Cache NL search results

cache nl search results, showed cached result when going back to search page

Closing this for now. If deemed necessary rework with changes from https://github.com/BloopAI/bloop/pull/144


feat: Security classes

Closes #22

Codecov Report

Merging #68 (599b996) into main (3da65c9) will decrease coverage by 1.46%.
The diff coverage is 63.72%.

@@            Coverage Diff             @@
##             main      #68      +/-   ##
==========================================
- Coverage   97.48%   96.01%   -1.47%
==========================================
  Files         142      147       +5
  Lines        4565     4769     +204
  Branches      585      625      +40
==========================================
+ Hits         4450     4579     +129
- Misses         71      143      +72
- Partials       44       47       +3

Impacted Files
Coverage Δ

xpresso/binders/_security/utils.py
20.00% <20.00%> (ø)

xpresso/security.py
29.62% <29.62%> (ø)

xpresso/binders/_security/oauth.py
56.71% <56.71%> (ø)

xpresso/binders/dependants.py
84.90% <65.21%> (-15.10%)
:arrow_down:

xpresso/binders/_security/apikey.py
68.88% <68.88%> (ø)

tests/test_security/test_security_key_header.py
100.00% <100.00%> (ø)

xpresso/binders/api.py
100.00% <100.00%> (ø)

xpresso/binders/_body/extractors/file.py
97.82% <0.00%> (-2.18%)
:arrow_down:

tests/__init__.py
100.00% <0.00%> (ø)


Prefetch JUMPDESTs through RPC with progressive proving

This continues the work in #427 and addresses one problem: CREATE and CREATE2 required memory dumps to obtain the relevant contract code and code_hash for call stack simulation, and we opted instead to fail over to simulation, since memory dumps were slow.
This branch introduces the following changes:

[x] Reinstantiate the match case for CREATE and CREATE2, but ignore code and code_hash.
[x] Prove only the current context.  The old version generates all Jumpdest proofs in the very first context. This meant that the witness had to provide the relevant mapping between code_hash and context.  This change means two things: current_code and current_code_hash is always available, so we do away with the mapping in the witness, essentially simplifying it to a mapping ctx->[offset].
[ ] Simplify the witness including removal of the tracking of code and code_hash in the Jumpdest generation.

Status: The first 100 blocks on testchain2 succeed, but there are still many errors later.

QQ: shouldn't this be targeting the JD branch instead of develop?


completed polishing and testing timestamp ordering

performed code review for timestamp ordering implementation; add test cases for timestamp ordering.

Can you resolve merge conflicts ?

@jarulraj resolved


[ServiceBus] RuleManager small improvements

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

Sorry my bad. Fixed

Sorry my bad. Fixed

All good.   I'm just being a goon.


parquet-251 causes the data read to be incorrect.

Bug description
If the Parquet version is below 1.8.0, the statistics may be incorrect, leading to row group filtering issues and making the data unreadable.  (parquet-251)
System information
Velox System Info v0.0.2
Commit: 3fbe31d01e714e553d7f7d16623980f8a78ebe8b
CMake Version: 3.25.1
System: Linux-5.4.0-150-generic
Arch: x86_64
C++ Compiler: /opt/rh/devtoolset-9/root/usr/bin/c++
C++ Compiler Version: 9.3.1
C Compiler: /opt/rh/devtoolset-9/root/usr/bin/cc
C Compiler Version: 9.3.1
Relevant logs
No response

This is a parquet-java bug that was fixed ~10 years ago. Could you elaborate in what way this affects velox?

If the version of Parquet used for writing is lower than 1.8.0, the row group statistics may be inaccurate, potentially leading to the filtering of row groups that fall within the data range. Parquet-java, when retrieving statistics, will determine whether to use them based on the version. Especially when we read data that was written by Hive or versions of Spark below 2.0

Thanks. So it's basically about compatability with data that was written with these ancient (8-10 year old) versions? Depending on how much effort it is to maintain this fix I'd probably be against it. But I am also not to familar with how relevant this really is today etc. so I'll someone else deciede ^^

When this issue occurs, there are no exceptions to notify the user, making it difficult to detect that incorrect data has been read. Due to historical reasons, some Hive or older version jobs are still in use, so we have implemented an internal fix.

When this issue occurs, there are no exceptions to notify the user, making it difficult to detect that incorrect data has been read. Due to historical reasons, some Hive or older version jobs are still in use, so we have implemented an internal fix.

 Either a fix or a clear error message would be good I agree. @kgpai


Bug: Redirect is not allowed for a preflight request.

Description
Having issues with hosting the litestar application with a separate frontend.
Running pnpm run dev with litestar run -r works just fine.
Problem is getting this to production level infrastructure.
In my repository that I've linked, I place a caddy server infront of it and I end up receiving this error.
Access to fetch at 'http://localhost/accounts/login' from origin 'http://localhost:5173' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: Redirect is not allowed for a preflight request.

URL to code causing the issue
https://github.com/BrianLYS/sveltekit-litestar-cors-error
MCVE
Access to fetch at 'http://localhost/accounts/login' from origin 'http://localhost:5173' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: Redirect is not allowed for a preflight request.

Steps to reproduce
1. Start Caddy, Backend and Postgresql Server with `docker compose up` in root dir
2. Start frontend server, `cd frontend/app`, `cp .env.example .env`, `pnpm install`, `pnpm run dev`
3. Open localhost:5173 in browser
4. Click register, fill in form.
5. Login with credentials.
6. Inspect console to see error log.

Screenshots
"![SCREENSHOT_DESCRIPTION](SCREENSHOT_LINK.png)"

Logs
No response
Litestar Version
2.8.3
Platform

[X] Linux
[X] Mac
[ ] Windows
[ ] Other (Please specify in the description above)

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

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

Are you interested in submitting a PR @BrianLYS?


Ionic emulate problem

Hey hey, on ionic, when i try to emulate with -> ionic cordova emulate ios -l --external --consolelogs
When recompiling & launching, always a new browser page is opened with that url -> https://xxx.firebaseapp.com/__/auth/iframe?apiKey=xxxx

This plugin wraps the native iOS and Android SDKs.
Browser emulation is not supported.

Agree, but i speak about device emulation on my mac

Please use the example project as a working reference to diagnose issues with your project.
i.e. Build and run the example project app and compare to your own.

that was the problem


Ensure we can re-run the script multiple times

Until now, we had to trash and recreate the container if we wanted to
iterate on ansible-test runs.
This patch corrects this issue, and we can now run the following without
any issue:
podman run --rm -ti \
        --security-opt label=disable \
        -v .:/opt/sources --user root cifmw:latest bash
./scripts/run_ansible_test
./scripts/run_ansible_test

It makes the whole process faster and easier for developers.
This PR has:

[x] Appropriate testing (molecule, python unit tests)
[x] Appropriate documentation (README in the role, main README is up-to-date)

/retest

Bypassing the CI workflow.
Reason: it seems the whole signature system on centos 9 stream is broken due to the use of sha1 algorithm in gpg related content, while that very same sha1 has been marked as weak:
https://kojihub.stream.centos.org/koji/buildinfo?buildID=31391


Routing inbound calls to different agents for same retell phone number

I'm trying to customize what agent a caller get's routed to when making an inbound call.
For my use case, there are a few different flows to which a caller can get routed (and I'd be able to determine this with dynamic variables / based on user phone number). The tasks are complex enough and the variation in the llm behavior is large enough that different mulit-state graphs (which are very useful by the way!) seem necessary to drive the desired experience.
Is this possible?
Thanks for your help!

Any update on this? Thanks again!


Very hard to disable 'Show title' option

BaseElement includes a ShowTitle boolean field which is added to the Title field on all blocks. This is overly opinionated IMO and I can think of lots of block types where you would never want to show a Title. I realise I don't have to respect this setting in templates, but showing the 'Display' checkbox on blocks that don't let you display the title is a bad UX. Users will think something is broken.
I eventually worked out how to hide this option but it's a hassle:
public function getCMSFields()
{
    $fields = parent::getCMSFields();

    // Target the TextCheckboxGroupField combo field (it's name is Title but so is the text field within it)
    $wrapper = $fields->fieldByName('Title'); // No result - doesn't work
    $wrapper = $fields->dataFieldByName('Title'); // Wrong - returns the text field nested within the TextCheckboxGroupField
    $wrapper = $fields->findTab('Root.Main')->Fields()->first(); // This does the trick but it's fragile and could break very easily

    // Change the title to remove the '(displayed if checked)' part
    $wrapper->setTitle('Title');

    // Hide the checkbox field
    $fields->removeByName('ShowTitle');
    return $fields;
}

You could probably replace the TextCheckboxGroupField (composite) with a simple TextField for Title as well

This has been pointed out to me before. I think it's our use of a composite that has a weird name clash (or something - not currently diving into the code).
I'll put this as an impact/low bug because there are workarounds - you just have a play a bit to find them.

I've been looking into this today. I think this could be a valid bug on silverstripe/framework. Currently you can't just "replace" a composite field:
https://github.com/silverstripe/silverstripe-framework/blob/5276b6cbb1ebd49d704734d940b0467c51b0d064/src/Forms/FieldList.php#L389
There's a call to hasData when replacing - I assume this is so the replaceField API can dive into composites.
It's a little better for removeByName because it assumes that you don't just mean datafields - but we do have some code that attempts to ensure "Title" is namespaced (for multiple element forms on one page). I think we can just remove that though.

I've raised a PR that resolved #632 . Now you can also remove the composite ->removeByName('Title') and then add back just a text field for title ->addFieldToTab('Root.Main', TextField::create('Title', 'Title'))

Okay I've raised an issue and PR on framework for fixing the issue with replaceField: https://github.com/silverstripe/silverstripe-framework/issues/8875 .
Referring to the example code:
$wrapper = $fields->fieldByName('Title');

This API needs the tab names. $wrapper = $fields->fieldByName('Root.Main.Title'); works.
$wrapper = $fields->dataFieldByName('Title');

This "correctly" gets the title field from inside the composite. I guess the composite having the same name is confusing, but the composite isn't a "dataField" so it should return null if the inner title field was not named "Title".
$wrapper = $fields->findTab('Root.Main')->Fields()->first();

$wrapper = $fields->findTab('Root.Main')->Fields()->fieldByName('Title') also works but it's in the same vein as $wrapper = $fields->fieldByName('Root.Main.Title');.

Thanks @ScopeyNZ for the quick work! Sorry for the radio silence, had the week off

I found that using $fields->fieldByName('Root.Main.Title') in the context of getCMSFields() returns the text field nested within the TextCheckboxGroupField which @jonom mentioned initially.
It does works nicely in the context updateCMSFields().

Actually scratch that, I'm actually finding that $fields->fieldByName('Root.Main.Title') doesn't work when you aren't using inline editing 🤷♂.

Bringing this back from the dead but removing and replacing ShowTitle actually causes problems in regards to viewing archives.
I've settled on this approach:
$showTitle = $fields->fieldByName('Root.Main.Title.ShowTitle');
$showTitle->setDisabled(true);

And just setting $defaults for the elements as needed.


Running run.sh on Linux subsytem

I'm using Linux Ubuntu on Windows 11 and everything works fine until I want to run run.sh file, I get this error:
ConfigBase::load: Unable to open /etc/xdg/terminator/config ([Errno 2] No such file or directory: '/etc/xdg/terminator/config')
I installed terminator multiplke times and I never get the terminator folder or config inside xdg. I read online that it's probably a DBUS issue on Linux susbsystem. I tried installing other terminators such as Konsole and xterm but they don't work for me.
Is there a solution? Thank you!

Run python3 -m http.server 8000 --bind localhost instead of running run.sh.

Run ./standalone/local.sh

There is also a standalone/server.cmd, so in theory you should be able to run this on Windows, too.

I also have problem and don't want to open another bug report.
On linux computer/virtual machine.
mkdir -p ~/opt/brouter
cd ~/opt/brouter
wget https://github.com/nrenner/brouter-web/releases/download/0.18.1/brouter-web-standalone.0.18.1.zip
unzip brouter-web-standalone.0.18.1.zip
copy brouter-web/config.template.js to brouter-web/config.js
download segments4 for my region

Now i run two tmux sessions.
./brouter/standalone/local.sh
And another while in ./brouter directory
python3 -m http.server 8000 --bind localhost
I check both ports are open localhost on mylinuxvm 8000 and 17777.
I forward those two ports via ssh on a windows machine like:
ssh -N -L 8000:<IP_ADDRESS>:8000 sshfwd@**mylinuxvm** ``ssh -N -L 17777:<IP_ADDRESS>:17777 sshfwd@**mylinuxvm**
I can access the webserver and see the map, but if i want to plan a route it says no answer from server (Keine Antwort vom Server).

Now i run two tmux sessions.
./brouter/standalone/local.sh

Maybe try running ./brouter/standalone/server.sh instead of local.sh?
The BRouter server is written in Java and should also run on Windows, if you have a Java runtime installed. Try running with ./brouter/standalone/server.cmd on Windows.


Document usage on Windows

How can I use this on Windows? Add some documentation. There is no reason that shouldn't be possible -- Docker works just fine, and a lot of devs actually use Windows.
So I think documentation on how to get the setup up and running on Windows is quite critical to reach the full potential of this setup.
via @revett: Is there a make equivalent for Windows?

Closing as solved by documentation that is now in README: https://github.com/CityOfZion/neo-local#windows-1


Update compatibility to latest omnisharp-roslyn

Properly load responses with a utf-8 BOM.
Newest versions of omnisharp-roslyn don't understand the -config launch argument.
Switch some default keybindings away from official Sublime Text bindings.

Thanks for the pr but with the latest changes to how omnisharp-sublime will be handling roslyn your changes in this package are mostly unneeded. I did end up taking the utf-8 bit out and move it into the plugin so thanks!

Is this an old branch?  Does the master branch support roslyn?

This PR was good, but the code that this effects is getting removed (excluding the keybindings). As the new maintainer I decided to focus on Roslyn and bringing everything up to date. Roslyn support is being brought into the master branch once a little more testing happens.
I appreciate the effort that you put into this and If you have some free time would you mind checking out #208?


Video Quality - poor contrast and bright video images from Branch videos

Question
When playing videos on OnlyM downloaded from jw.org the contrast is poor and video appears visibly brighter than the original video.  Other videos from other sources also. I would like some ideas to try.  Thanks
Additional context
Add any other context or screenshots about the your question here. The more detail, the better.

Hi @Tennisbum1 - thanks for the question!
I'm wondering if you could provide some more detail about the problem that you see? How are you playing videos with the twilio-video-app-react code? Are you sharing your screen to other participants in the room, and playing these OnlyM videos through the screen share?
Some screenshots of the problem that you see could make it easier for us to help you. In the meantime, it may be possible to change the brightness or other qualities of the screen share track by changing this code here: https://github.com/twilio/twilio-video-app-react/blob/master/src/components/VideoProvider/useScreenShareToggle/useScreenShareToggle.tsx#L19-L23
In that object, you can specify other properties, like brightness or exposureCompenstation (support for these may be device-specific). To see all the properties that could be used, you can look at this documentation from the Mozilla Developer Network: https://developer.mozilla.org/en-US/docs/web/api/mediatrackconstraints#properties_of_image_tracks

Closing due to inactivity. I'm happy to reopen the issue if more information is provided!


Update "Friends' Statuses" to show all friends

Even if they haven't created a status their name should still appear in the list with an empty status so it's easy to find everyone you're following.

Set it up for them to be alphabetized ascending (a-z) by default. Added a method to change the sort order to alphabetized descending (z-a) but didn't expose it in the UI.


Nested Navigation with Navigattie

Hello,
I have a situation that I would like to discuss with you regarding the different 'userflows' in my application.
In my current situation, I have a TabView as the entry point of my application, which serves as the base for four distinct and independent 'userflows'. Each 'userflow' has several nested views. My concern is that, with the current implementation of Navigattie, the level 0 (Root) seems to be defined at the entry point of the application rather than for each 'userflow'. So, if I call popToRoot, I return to the first screen of the application rather than the beginning of the specific 'userflow'.
For instance, I have a code that looks like this for the FirstFlow:
import SwiftUI
import Combine
import Navigattie

struct FirstFlowScreen: NavigatableView {

    @Environment(SharedVariables.self) var shared

    @StateObject var navState = NavigationStateManager()

    var titleView: String
    var sharedFlowVM = BigFlowViewModel()

    var body: some View {

        NavigationStack(path: $navState.inventorySelectionPath) {
            ScrollView(showsIndicators: false){

                //Content
            }
            .navigationDestination(for: NavigationState.self) { state in
                switch state {
                    case .goToView1: View1()
                        .environment(sharedFlowVM)
                        .environment(shared)
                        .environmentObject(navState)
                    case .goToView2: View2()
                        .environment(sharedFlowVM)
                        .environment(shared)
                        .environmentObject(navState)
                    case .goToView3: View3()
                        .environment(sharedFlowVM)
                        .environment(shared)
                        .environmentObject(navState)
                    case .goToView4: View4()
                        .environment(sharedFlowVM)
                        .environment(shared)
                        .environmentObject(navState)
                    case .goToView5: View5()
                        .environment(sharedFlowVM)
                        .environment(shared)
                        .environmentObject(navState)
                }
            }
        }
    }
}

With Navigattie, if I do a popToRoot, I return to the first screen of my application, which could be the beginning of my FirstFlow. However, if I have a SecondFlow, the same action would always take me back to the beginning of my FirstFlow. In other words, currently popToRoot does not consider the beginning of each 'userflow' as a "level zero".
Do you have any suggestion on how to handle this situation with Navigattie? Thank you in advance for your help and looking forward to your response.

Hey, did you try using a NavigationStateManager() for each individual flow?
If not, how did you solve the issue?


Can't copy saved games over

I tried copying the game folder over to my laptop so I could play this game on the road, but none of my save data (single player agent files, combat simulator player files, etc...) copied over. Is it located in a different folder?

The save file is called eeprom.bin and the config file is called pd.ini.
On Windows they are normally going to be in the same folder as pd.exe. If you put your game in Program Files or some other folder with no write access, it will instead write them to %APPDATA%\perfectdark (either paste that directly into the File Explorer's address bar or go to C:\Users\YOUR_USERNAME\AppData\Roaming\perfectdark).
On Linux they're normally located in ~/.local/share/perfectdark.

The save file is called eeprom.bin and the config file is called pd.ini.
On Windows they are normally going to be in the same folder as pd.exe. If you put your game in Program Files or some other folder with no write access, it will instead write them to %APPDATA%\perfectdark (either paste that directly into the File Explorer's address bar or go to C:\Users\YOUR_USERNAME\AppData\Roaming\perfectdark).
On Linux they're normally located in ~/.local/share/perfectdark.

OK, I deleted the copy of the game folder and tried copying it again, and it works! Thank you!


Fix default params messing set cache time and rename Ttl suffixes into TTL

Intro
Hello there!
I wanted to say hello for your work by submitting this PR.
Summary

It tries to fix globTTL bug (TTL not being passed, but used default value instead)
It renames Ttl to TTL
It extracts "2" seconds cache time into const.

Reasoning
I think it makes sense to rename Ttl into TTL as it's abbreviation itself, so no need to lowercase it (on the plus side - IMO it reads easier).
When calling RootTypeMapperFactoryContext constructor from SchemaFactory, there was globTtl variable missing causing this class to use default value (2). Same for FactoryContext.
I also extracted magic "2" into const and removed it from optional params. Hopefully this will avoid such situations in future.

Hey @mailopl ,
From a quick view, it seems great!
I'll double check it later today, but wanted to say thanks a lot for this PR!

Man I hope everything's fine on your end, virus-wise!
Any chance to get this checked/merged?

Hey @mailopl ,
Thanks a lot for your patience. Indeed, I have less time to do proper work with 2 kids at home and no school :D
Anyway, let's merge this. Sorry for the delay and thanks a lot for your contribution!


Config Flow: Text Form Description missing

The problem
Currently the config flow is missing a description what to enter

Unfortunatly it has been configured in https://github.com/FaserF/ha-deutschebahn/blob/main/custom_components/deutschebahn/strings.json but seems not to work
What version of Home Assistant Core has the issue?
2022.9.7
What version of DeutscheBahn Integration has the issue?
2.0.0
What type of installation are you running?
Home Assistant OS
Example YAML snippet
No response
Anything in the logs that might be useful for us?
No response
Additional information
No response

Feedback from a friend:

Expand the instructions a bit on how to install the update, which brings me to the second point
Update didn't work for me, the application couldn't be found via HA integration after installation, it only worked after uninstalling and installing.
When it comes to the options, I'm in a fix, at least it doesn't show me on my cell phone what I have to fill out and in what format. A zero is still there and a selectable field, all without description.


Custom Lookups and Permissions

Hi,
More of an FYI than an issue, since I have already fixed it. :)
I am assuming this is intended behavior but it confused me for a bit. Might be worth dropping into the guide something about it.
When you make a custom lookup and forget to set the Permission="?" key. When you go to a grid that has that lookup script included on its row definition. The row displays fine in the grid but you obviously can't do much else.
What confused me more than anything else was that the New (whatever) button was not clickable so I didn't easily connect that it was permissions on a lookup causing that.

What is so suprising that a lookup uses permission of row you put it on. This is correct behavior security-wise. If something doesn't work, first thing you should do is to look at browser console.


made a Sampler type that wraps the underlying algorithms

The sampler should know the current simulation time. That causes problems if you have hierarchical samplers, so this PR creates a hierarchical sampler that contains multiple SSA algorithms. That's the idea, and it shows why there is a difference between the next() function to get the next time and the sample!() function that chooses the next time.
It does raise a question about enabling times. Maybe it's easier for user code to give a relative enabling time, so the simulation at time t=37.2 doesn't say "enable this function at time 36.4" but says "enable this function so it's offset by -0.8 seconds into the past.

Maybe not all the files made it into the git commit? The crucial sample/sampler.jl does not seem to be there.

You're right. I made the PR and walked away before tests fully ran. This PR came from that last conversation. It's true that the time should be part of the sampler, not passed in with every distribution. So this makes time part of the sampler. But hierarchical SSA's shouldn't have a different time for each one, so I put the time into a sampler wrapper class.


can not login to admin panel on hosting

Laravel Version: 5.4.
Voyager Version: 0.11.9
PHP Version: 7.1
Database Driver & Version: MySQL 5.4

Description:
I moved a website to hosting, (used a migration) and now cannot see voyager admin panel. when I trying to login - http://myhosting.in.ua/admin/login
I see 4 errors:
ErrorException in Connection.php line 647:
SQLSTATE[HY000] [2002] Connection timed out (SQL: select * from settings where key = title limit 1) (View: /home/nameDB/myhosting.in.ua/laravel/vendor/tcg/voyager/resources/views/login.blade.php)
QueryException in Connection.php line 647:
SQLSTATE[HY000] [2002] Connection timed out (SQL: select * from settings where key = title limit 1)
PDOException in PDOConnection.php line 47:
SQLSTATE[HY000] [2002] Connection timed out
PDOException in PDOConnection.php line 43:
SQLSTATE[HY000] [2002] Connection timed out
Steps To Reproduce:
in a local development was another name of DB, user, password - I changed it in .env file. import of DB was successful.

@Ivanenko-O it states connection timed out, which means perhaps your hosting database not good enough. Your local and your hosting have different settings i guess. You might want to take a look at your mysql version or its spec.

Thks. everything that I need - change  .env file at hosting.
instead
DB_HOST=<IP_ADDRESS>
to
DB_HOST= "real host of my provider."


Amazon linux no longer does it this way.

Amazon Linux no longer uses upstart for it's package management. It can support whatever is configured globally in puppet ( yum, rpm, etc.) This code caused an error on current amazon linux instances:
/Stage[main]/Newrelic_infra::Agent/Service[newrelic-infra]: Provider upstart is not functional on this host

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Turn on atomic requests

This turns on atomic requests, so that if some part of a response fails, all changes to the DB will be reverted.
This should help https://github.com/policykit/policykit/issues/642 kind of bug from corrupting the database.

I also added a missing migration from #678


Crash/Out Of Memory because sending queue size is not controlled

I'm sending 50..100 Kb images very quickly of the websocket connection, asynchronously.
If connection is slow, the send queue obviously grows and grows and at some point the app simply gets shut down from the system.
What are the possible way to control this behaviour / set the queue size ?

After new finding crash happens because of peak of memory consumption in our app.
Also there is a possibility to use closures for writing and use our own queue, so I'll close the issue.


Implement creating root hash in frontend

We changed the approach by creating a zip file from all uploaded files, calculate a SHA256 hash from the zip file and save this information in the token metadata file.
That being said, we currently store the following information. Under formats you can see the upload url as well as the calculated hash from the zip file.
{
  "artifact_uri": "http://localhost:33080/download/1",
  "creator_id": "tz1VSUr8wwNhLAzempoch5d6hLRiTh8Cjcjb",
  "description": "Test",
  "files": [
    {
      "fileName": "battery.json",
      "fileSize": 4530,
      "mimeType": "application/json",
      "ontologyUri": null
    },
    {
      "fileName": "camera.json",
      "fileSize": 3840,
      "mimeType": "application/json",
      "ontologyUri": null
    },
    {
      "fileName": "gnss.json",
      "fileSize": 4560,
      "mimeType": "application/json",
      "ontologyUri": null
    },
    {
      "fileName": "imu.json",
      "fileSize": 3810,
      "mimeType": "application/json",
      "ontologyUri": null
    },
    {
      "fileName": "lidar.json",
      "fileSize": 3840,
      "mimeType": "application/json",
      "ontologyUri": null
    },
    {
      "fileName": "radar.json",
      "fileSize": 4080,
      "mimeType": "application/json",
      "ontologyUri": null
    },
    {
      "fileName": "slidar.json",
      "fileSize": 3840,
      "mimeType": "application/json",
      "ontologyUri": null
    }
  ],
  "formats": [
    {
      "uri": "http://localhost:33080/download/1",
      "hash": "sha256://57ef051535698fcec1bc54c4173c43c73a766ee5fc201df94c61f9d367690a2d",
      "mimeType": "application/zip"
    }
  ],
  "id": 1,
  "metadata": "ipfs://QmcTjMSbHoUDC9x2pr8nusZ6zug5PPDAAJfqifnrLXVToY",
  "name": "Mercedes Benz AMG",
  "price": 1,
  "tags": [
    "luxury",
    "benz",
    "mercedes",
    "car"
  ],
  "timestamp": "2022-01-25T23:41:03+00:00"
}


Users can raise RescheduleTask but TaskSets cannot?

Prerequisites

[X] I am using the latest version of Locust
[X] I am suggesting a new feature, not asking a question

Description
So as far as I understand, Users can raise RescheduleTask error directly in the code, but TaskSets use this error internally so inside a TaskSet task one needs to use self.interrupt(), otherwise raising RescheduleTask causes unexpected behaviour?
I had an issue with my code where I first implemented some tasks directly inside a User instance and they raised RescheduleTask error, but then I refactored the code so that the tasks are inside a TaskSet, and there was unexpected behaviour with these tasks (locust seemed to be stuck inside one as soon as it was executed) until I replaced all instances of raise RescheduleTask() with self.interrupt().
Should the interface be somehow unified so that both Users and TaskSets have methods e.g. self.reschedule() and self.interrupt()and RescheduleTask error is only used internally and never raised in user-written code?

I guess? I dont really use TaskSets and have never seen the point in using them so I dont know :)


Feature Request: Producer and Consumer Interfaces for IOC and Unit testing

Description
Unit testing is critical for any production application and Dependency Injection is a very common design pattern used in C# applications.
With the absence of interfaces for the producer and consumer in Kafka-dotnet mocking/substituting the kafka dependency in unit tests is difficult and requires abstracting the library with an extra layer.
Additionally the producer and consumer does not support the IOC design pattern that is common in many modern applications.
Additionally adding producer and consumer interfaces should not be a breaking change to implement but would provide great value and flexibility to the project.
How to reproduce
Using Kafka Server version 2.11-<IP_ADDRESS>.
Currently developing Windows 7 / 10 / Server 2012 operating systems but moving towards .net core 2.0 and supporting Linux Centos/Ubuntu distros as well.
Create any new project utilizing Confluent Kafka Dotnet version 11.2 or earlier with default configuration.

Only way to create producer or consumer is to directly instantiate them inside a class.
Lacking interface makes kafka dotnet dependecies impossible to substitute directly in unit tests with tools such as N-Substitute.

Checklist
Please provide the following information:

[X] Confluent.Kafka nuget version:
[X] Apache Kafka version:
[X] Client configuration:
[X] Operating system:
[ ] Provide logs (with "debug" : "..." as necessary in configuration)
[ ] Provide broker log excerpts
[ ] Critical issue

Thanks @David-V-Spencer - this is high priority, yes. the second most requested feature behind avro/schema registry integration. i will try to get this done soon, but it'll happen after avro/sr as there is huge demand for that.

@mhowlett  Thanks for the response. Looking forward to this addition as well as schema registry.

Any idea when this will be completed? Currently I see there is ISerializingProducer<TKey, TValue>, but it doesn't cover whole class

These are in 'experimental' builds already, which hopefully will be fully reviewed and on a 1.0.x branch relatively soon.

closing - track the experimental branch for progress.


added QQP dataset as new paraphrase problem.

Added Quora Question Pairs training dataset containing ~400k training pairs as new paraphrase problem.

PR closed as this problem has been already implemented by the following commit dd5d1e567c537cfe468ba89f97303ad5f3db95cd


[improve][broker] Make Consumer#equals more effective

Motivation
It's from a production case in my org, one of our consumer application constantly create consumer instance after increase topic partition due to old pulsar-client bug. The application got OOM eventually and broker publish latency goes more than 30s, so many publish requests timeout, broker latency go back normal after application restart.
It could be reproduce below:
pulsar-admin namespaces create -b 128 public/perf
pa namespaces set-max-consumers-per-topic -c 0 public/perf
pa namespaces set-max-consumers-per-subscription -c 0 public/perf
pulsar-perf produce -r 1000 persistent://public/perf/perf
pulsar-perf consume -st Shared -n 100000 persistent://public/perf/perf

After stop consume program, pulsar-io thread will be block for a while, publish latency will be very high (35s+ in my case).
It's a cpu profile during consumer stops:

profile.100kconsume1k.stoping.122601.html.zip

The time complexity of CopyOnWriteList.remove is O(n), so Consumer#equals method is a hot method.
https://github.com/apache/pulsar/blob/master/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentDispatcherMultipleConsumers.java#L192
Modifications
Flip && expression
Verifying this change

[x] Make sure that the change passes the CI checks.

This change is a trivial rework / code cleanup without any test coverage.
Documentation

[x] doc-not-needed

Matching PR in forked repository
PR in forked repository: https://github.com/Shawyeok/pulsar/pull/5

Thank you! Merging...


Lay groundwork for documenting Metal (Apple) configuration.

We do already have a page on building for iOS: https://openxla.github.io/iree/building-from-source/ios/, but that page is less polished than the other building from source pages. This moves us closer to having a complete set of documentation pages for iOS/macOS development and deployment, now that the Metal HAL is moved out of experimental/.

Add placeholder gpu-metal documentation page

Drop "experimental" qualifier on Metal compiler target and runtime HAL driver

before

after

Refresh language and icon usage for the other pages in this section

Ah, the screenshots also show some new pages that I started sketching out 😛


Unable to upload file in Google Storagee

AssertionError [ERR_ASSERTION]: Cannot wrap non-Error object       at new AssertionError (internal/assert.js:269:11)
Followed the steps present in the readme

Did you resolve this? I'm also having the same issue

I had the same issue and resolved it with the following steps:
The storage bucket created in GCP needs to have Access control : Fine-grained
as mentioned in readme.
"The bucket should be created with fine grained access control, as the plugin will configure uploaded files with public read access."


Build a draft release and then publish once complete

Instead of publishing partial releases (even for a few minutes) as the matrix of builds adds their artefacts, create a draft release that we then publish in once the build has finished.
Background: the build process involves some set-up, then a matrix of building for different platforms, then some finalisation. This change is moving the "publish the release" from the matrix jobs to the finalisation job.

This has a few benefits:

Atomic publishing: a release is only published once it has artifacts for all platforms. Before this change, if a builder fails for one platform, we'll have published a "latest" release visible to get-pants.sh, and so running that on the broken platform will fail. (Example: #430.)
Reduced duplicate work: rather than have each builder in the matrix compute changelogs etc., just do it once as part of the publishing. This is more conceptually helpful, than an optimisation, since it only takes a few seconds to prep the changelog.

Waiting on #427

I tagged a v0.12.3-beta.0 to test. It needs
The build was https://github.com/pantsbuild/scie-pants/actions/runs/12226797242
While that was running, there was a draft release:

... but then the publish and announce step created a whole new release:

My guess is the idempotency might be relying on the name key too: https://github.com/pantsbuild/scie-pants/pull/439


Adding mxnet inference for squeezenet model

Description of changes:
Adding Squeezenet model for inference using MMS.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@vdantu  please make the appropriate changes according to mnist model in this file.

Will close this PR as MXNet inference is now working and documented clearly. However still need to have these steps running on EKS as mentioned in https://github.com/aws-samples/machine-learning-using-k8s/issues/88.


Chore: Minor CSS Update

Close #266
Details

Updates CSS to fix the rendering differences between LP and Reader while preserving the reason for margins in the first place.
Adds a SCSS file for Main.css
Updated .gitignore to ignore map files because.. SASS.
Math-container styling updated to .math-block to apply more evenly across different code bases.

@sigrunixia if we have the main sass file, do we still need the css file at all?

@sigrunixia if we have the main sass file, do we still need the css file at all?
Did you regenerate the css file with the scss file?

Yes, and it should be the "same" at the moment mostly as I did not do SCSS nesting yet. Some repos I have submitted to want both .scss and css file, but I can remove it. Let me merge in your new settings changes.


Bundle update  raises a RuntimeError for non-local platform gems

Given a Gemfile and Gemfile.lock for a JRuby project, like the below, running bundle update json will raise a RuntimeError saying Unable to find a spec....
# Gemfile
source 'https://rubygems.org'

gem "json"

# Gemfile.lock
GEM
  remote: https://rubygems.org/
  specs:
    json (1.4.3-java)

PLATFORMS
  java

DEPENDENCIES
  json

BUNDLED WITH
   1.16.1

The cause of the above is these lines which call expand_dependencies on the list passed gem names without allowing them to be expanded to non-local platforms, and then pass the returned (ruby-only) list SpecSet#for with a java platform dependency.
I think the fix is to change those lines to:
unless @unlock[:lock_shared_dependencies]
    eager_unlock = expand_dependencies(@unlock[:gems], true)
    @unlock[:gems] = @locked_specs.for(eager_unlock, [], false, false, false).map(&:name)
end

(I.e., pass remote=true to the dependency expansion, so we don't exclude any platforms, and pass raise_on_missing=false to for, so that the missing ruby platform isn't an issue.)
However, I know very little about different Ruby platforms, Bundler's handling of them, or where this should be specced. I'd love someone to take it over.

Tests:
diff --git a/spec/commands/update_spec.rb b/spec/commands/update_spec.rb
index 799aac1f5..b5938b909 100644
--- a/spec/commands/update_spec.rb
+++ b/spec/commands/update_spec.rb
@@ -437,6 +437,47 @@ RSpec.describe "bundle update in more complicated situations" do
     bundle "update"
     expect(the_bundle).to include_gems "thin 1.0"
   end
+
+  context "when the lockfile is for a different platform" do
+    before do
+      build_repo4 do
+        build_gem("a", "0.9")
+        build_gem("a", "0.9") {|s| s.platform = "java" }
+        build_gem("a", "1.1")
+        build_gem("a", "1.1") {|s| s.platform = "java" }
+      end
+
+      gemfile <<-G
+        source "file://#{gem_repo4}"
+        gem "a"
+      G
+
+      lockfile <<-L
+        GEM
+          remote: file://#{gem_repo4}
+          specs:
+            a (0.9-java)
+
+        PLATFORMS
+          java
+
+        DEPENDENCIES
+          a
+      L
+
+      simulate_platform linux
+    end
+
+    it "allows updating" do
+      bundle! :update, :all => true
+      expect(the_bundle).to include_gem "a 1.1"
+    end
+
+    it "allows updating a specific gem" do
+      bundle! "update a"
+      expect(the_bundle).to include_gem "a 1.1"
+    end
+  end
 end

 RSpec.describe "bundle update without a Gemfile.lock" do


nullable not recognized when checking default value that is null

Detailed description
There seems to be a bug with Schema validation where nullable is set and the default value is null.
I've got a schema, which I think is completely valid:
    budget:
      type: number
      nullable: true
      default: null
      example: 100

But I get the following error:
#/components/schemas/budget
expected 'number' to be 'object'

This probably happens because null in JS is of type object and for some reason it doesn't recognize that nullable is set to true.
Context
It rejects a valid schema and therefore should be fixed.
Your environment
Speccy version: 0.9.0
OS: Windows 10
Node version: 11.0.0
Link to the full OpenAPI document: https://open-eo.github.io/openeo-api/v/0.4.0/openapi.json

Full example to reproduce the error:
openapi: "3.0.0"
info:
  version: 1.0.0
  title: Example
  license:
    name: MIT
servers:
  - url: http://example.io
paths:
  /pets:
    get:
      summary: Example
      responses:
        '200':
          description: Example
          content:
            application/json:
              schema:
                type: number
                nullable: true
                default: null

Bug(s) in oas-kit. Will be fixed in next version.


deemph.py contains invalid python

The second import line in the deemph.py file, "from caps import caps" throws up an error, and I can't find any reference to a caps module.
I tried digging in the history to see what happened but couldn't get far. Alternatively, in Python 'foobar'.upper() is 'FOOBAR', so that would be a quick fix.

My bad, didn't recall it was another package in the examples folder!


(doc) How to update operators documentation

Signed-off-by: Vu Dinh<EMAIL_ADDRESS>

/retest

/retest

Can we move this forward soon and get it merged? I would like to contribute additional docs about z-stream management before I forget what we settled on; that'll be a lot easier with this in-place. Thanks!

I will get this PR updated to address some of the feedbacks and will get it moving forward and merged soon. Thanks.

Quick note - docs now live in /docs, so these should be moved as well.

/retest

/retest

doc-only, merging


x/build/env/fuchsia: add new Fuchsia builder

@dhobsd is planning on adding a Fuchsia port.
Fuchsia is not yet self-hosting, so the builder plan is to use a linux/amd64 host to do the build with nested virtualization enabled and run qemu+kvm to run the tests.
/cc @dhobsd @dmitshur

Will This Still Contain Jiri...

Freeze is tomorrow, so I guess Fuchsia is waiting until Go 1.14.

No activity on this, and it should be preceded by a port proposal anyway. Closing for now.


Bidirectional GBIF Registry Synchronisation

Up to now we sync checklist datasets with good licenses from the GBIF registry to ChecklistBank, but not the other way around. In the future we want CLB to be the only entry for new checklist registrations and only track these with the GBIF registry. For the GBIF portal to show links to ChecklistBank we also want the native CLB integer datasetKeys in the GBIF registry.
As a start extend the GBIF registry sync with the following:

update existing GBIF checklist entries

adding the CLB dataset key as an alternative identifier of new type CHECKLISTBANK

create new checklist datasets for missing entries

only use

public datasets of origin=EXTERNAL, ignoring projects and releases for now
require an CC0, CCBY or CCBYNC license

publisherOrg = ChecklistBank
hostOrg = ChecklistBank
installationOrg = ChecklistBank
endpoint: the original archive in CLB, not a generated one

For this to work CLB needs write access to the registry API. For adding identifiers CLB needs access to all checklists in the registry. For adding new datasets it might be good enough to use the existing permission system that gives access to a single publishing organisation, CLB in this case.
The GBIF API needs changed to cover:

EndpointType: ACEF & TEXT_TREE
IdentifierType: CHECKLISTBANK

We could start only with adding CLB dataset keys to existing gbif checklists