chore(deps): Remove @expo/spawn-async

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.67%. Comparing base (8aa4b81) to head (3253d99).
Report is 1 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #170   +/-   ##
=======================================
  Coverage   98.67%   98.67%
=======================================
  Files           7        7
  Lines        1432     1432
  Branches       52       52
=======================================
  Hits         1413     1413
  Misses         19       19

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


make sub expansions only run for the classification that generated them

makes sure tjat sub expansions only run for the classification that generated them. It also subs out the newly created names for the phantoms that were in their place in the temp graph so we can reuse the temp graph
changed the weighting of the service api to fix the lambda being placed into the vpc issue (we may not see vpc endpoint get created because of this, but thats at least hidden for now and we dont support many)
efs iac template fix
policy adds permissions classification to help weighting in path selection
Standard checks

Unit tests: Any special considerations?
Docs: Do we need to update any docs, internal or public?
Backwards compatibility: Will this break existing apps? If so, what would be the extra work required to keep them working?

Only thing is I'm not sure whether we want to do sub-expansions as separate vertices - only because other vertices which depended on the upper expansion would be unblocked, but I'm not confident those shouldn't also wait until sub-expansions are resolved as well.

yeah im not sure either, but we could definitely make that change. This is previously the way it was operating so im thinking lets keep it and evaluate if we want to do the expansion holistically later on


add epsilon to RankCrossEntropyLoss to avoid NAN

@jibrilfrej It's ok to merge with to approved reviews, and I just invited you as a collaborator. Feel free to merge it once you accept the invitation.

I accepted the invitation but I still cannot merge.

Oh it's my fault. I forgot the master is protected. Would you mind making the change on branch 2.2-dev instead?

I created a new pull request : #779


Ref number of people

Expected behavior
Actual behavior
Steps to reproduce
Minimal yet complete reproducer code (or URL to code)
Netty version
JVM version (e.g. java -version)
OS version (e.g. uname -a)

RepositorypatternandWorkManager�21caaea31a8b4273b58215924c452728files/zh-cn/learn/javascript/first_steps/what_is_javascript/index.html

RepositorypatternandWorkManager�files/zh-cn/learn/javascript/first_steps/what_is_javascript/index.html21caaea31a8b4273b58215924c452728

files/zh-cn/learn/javascript/first_steps/what_is_javascript/index.htmlRepositorypatternandWorkManager�21caaea31a8b4273b58215924c452728files/zh-cn/learn/javascript/first_steps/what_is_javascript/index.html


Feature request: docker specific outbound firewall implementation

Description
It would be nice to be able to say on the docker run command line or from docker-compose what protocols/ports are allowed or denied outbound access from the container, rather than resorting to Host OS specific firewall solutions like IPTABLES

/cc @aboch @sanimej

Let me close this ticket for now, as it looks like it went stale.


Avoid panic when using partition_packages on Rust < 1.71

#59 added support for using the workspace default members when no parameter is specified. However, this only works on Rust 1.71+, so using clap-cargo on Rust < 1.71 results in a panic when calling partition_packages.
I suggest a simple change: avoid using workspace_default_members unless the flags actually do ask for default packages. This way, it would for example be possible to call a clap-cargo-powered tool with --workspace on Rust < 1.71 and it would work.
(A better change might be to check if workspace_default_members are available before using them, but alas this doesn't seem to be available in cargo_metadata currently.)
If this change is acceptable, I volunteer to work on a PR for this.

Could you clarify what panic you are seeing?
I can see us cleaning up the code to only calculate things we need but I don't see how that would be causing a panic.

I guess there's another choice: determine the running Cargo's version to see if it's safe to access the field. However, that doesn't seem to be an ideal solution. (That could be implemented in the tool using clap-cargo too.)
FWIW, I've also volunteered to add something to detect this in cargo_metadata: oli-obk/cargo_metadata#254

Do nothing. I generally recommend people run the latest version of rustc for development and worry about their deployment / MSRV version for production builds / CI.

I indeed detected the issue in a CI workflow that validates a crate's MSRV.


Breaking Change Request: Don't restore terminal state on exit

Proposal
We propose to make the Dart VM stop automatically restoring the terminal settings upon exit. A proposed implementation is at https://dart-review.googlesource.com/c/sdk/+/190484.
Programs that set Stdin.echoMode or Stdin.lineMode must now manually restore the terminal settings upon exit if they wish.
Rationale
The automatic terminal setting restoration happens unconditionally and can lead to undesirable behavior e.g. when the program does not care about terminal settings and is sharing a process group with a program that does care. E.g. if dart's output is piped into less(1), then there is a race condition where dart might see the raw terminal settings set by less(1), and if the dart VM exits after less(1) has exited, then it will restore the raw terminal settings, leaving the user with a seemingly defective shell with echo disabled. This race condition can be reproduced using:
cat > yes.dart << EOF
main() {
  for (int i = 0; i < 1000000; i++) {
    print("yes");
  }
}
EOF
stty; (sleep 1 && dart yes.dart) | less; stty; stty sane

The user will end up with a shell with echo behavior disabled. The stty command shows the current terminal settings, where the difference can be seen, and stty sane fixes the settings before returning to the shell prompt. The stty sane call can be omitted to see the defective shell prompt.
dart:io already provides all the functionality for programs to correctly implement the restoration behavior themselves. This class of terminal setting corruption bugs are eliminated by shifting the burden onto the programs that use interactive input and change these settings. This design matches that of other programming languages.
Impact
Affected programs won't break for the duration of their run, although the terminal state settings for line mode and echo mode may be unexpected afterwards, especially if the program is terminated by a signal or an unhandled exception. Users of POSIX systems can manually repair their terminal session by running stty echo echonl icanon or stty sane.
Mitigation
Programs that change the Stdin settings lineMode and echoMode are now responsible for restoring the settings upon program exit. E.g. a program disabling echoMode will now need to restore the setting itself and handle exiting by the appropriate signals if desired:
import 'dart:io';
import 'dart:async';

main() {
  bool echoWasEnabled = stdin.echoMode;
  try {
    late StreamSubscription subscription;
    subscription = ProcessSignal.sigint.watch().listen((ProcessSignal signal) {
      stdin.echoMode = echoWasEnabled;
      subscription.cancel();
      Process.killPid(pid, signal); /* Die by the signal. */
    });
    stdin.echoMode = false;
  } finally {
    stdin.echoMode = echoWasEnabled;
  }
}

fyi @jonahwilliams

Also fyi @zanderso
We already have the existing hooks to be able to restore terminal state on exit, so the required tool change should be fairly minor. My understanding is that if we get sigkilled we don't get a chance to run these, but then neither does the existing VM code? As long as we don't regress this behavior I don't imagine there would be any problems.

This was originally done because the way the Dart VM sets the terminal states makes it impossible for a Dart program to explicitly restore the original state after modifying it. So backing this out will re-introduce that issue: https://github.com/dart-lang/sdk/issues/30318. /cc @cbracken.

I can't remember if we discussed this, but is there a reason echo and echonl couldn't be modelled as separate properties? Also a breaking change, but actually gives the developer the ability to correctly restore the original state.

Thanks @zanderso and @cbracken for that valuable context about the #30318 bug. A solution is required in order to prevent this bug from returning if this breaking change is made.
I propose we amend this breaking change proposal with making Stdin.echoMode only control the echo local terminal mode and introducing a new Stdin.echoNewlineMode field to control the echonl local mode on POSIX systems. On Windows, this new field will always be false and throw if enabled since Windows does not have such a mode. This amendment is also a breaking change in its own right, since the addition of a new field can break implementations of the Stdin interface (if any).
Does that resolve your concerns?
I have implemented this proposed behavior in https://dart-review.googlesource.com/c/sdk/+/190484.
cc @lrhn @aam

@sortie that sounds great to me.
I'm curious if @zanderso remembers why that wasn't the proposed solution the first time; maybe just that it was a breaking change -- or we thought it was likely to be a bigger breaking change than resetting the values on exit?

@sortie Please add a post to https://groups.google.com/a/dartlang.org/g/announce

I don't think the reason is documented anywhere, and my memory is imperfect, but it was probably a combo of:

Avoiding a breaking change,
Avoiding thinking about how to handle it on Windows,
Following a general principle that the VM should clean up after itself.

On (3), I'm still a little skeptical of this change. It adds the requirement that any program that modifies the echo modes must record them before modifying them, and install signal handlers and on-exit handlers to restore them to their previous state. I suspect that there aren't many command line programs that currently have that. The flutter_tool has the signal and on-exit handlers, but I'm sure it doesn't record the previous state on startup.
But I guess this is a breaking change proposal, so the mitigation would be updating the documentation for Stdin.echoMode etc. to explain the right way to clean up after any modification of the modes.

Is this still something we want to investigate? @Hixie @vsmenon @grouma

This shouldn't have an impact on AngularDart nor do I see the need for it.

@sortie - is this something we're still pushing on?

I think this is still worth landing. @sortie do you have cycles to see this through?

I have updated https://dart-review.googlesource.com/c/sdk/+/190484 so it can land once this breaking change request is approved.

Any updates on this from the review meetings? Any objections to fixing currently broken behavior?

Before landing, please test using the flutter tool. In particular, I'm unsure what will happen when the tool is killed by a signal during flutter run.

I don't see any adverse effects on flutter tools being killed neither on Windows or on Linux with proposed patch applied: the console shell/window that used to launch flutter tools seems to continue normally after flutter tools is killed.

Nice. Another test case might be the usecase that caused me to file the original issue. Running with zsh, run flutter test ... and verify it looks good; then do the same but kill it partway through. Ideally, neither case will result in a terminal with a changed echonl setting (evidenced by emitting an extra blank line after every command you type).

terminating running flutter test [still] seems to result in well-behaving terminal session(on linux, windows and, thanks to you, on mac too :-) )

ping @mit-mit @vsm, @Hixie, and @grouma for approval

LGTM from an AngularDart perspective.

@Hixie @mit-mit @vsmenon pinging again for approval.

SGTM

SGTM.

@a-siva @brianquinlan - you ok with the dart:io change?
Otherwise, LGTM

@a-siva @brianquinlan - you ok with the dart:io change?

The changelist has been reviewed and was waiting for this breaking change approval before it could land.

lgtm

Team, marking this breaking change as approved.


chore: update dependencies

Codecov Report
All modified lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files
Coverage Δ

src/app-data/loaders/cdfLoader.ts
86.34% <100.00%> (+0.41%)
:arrow_up:

:loudspeaker: Thoughts on this report? Let us know!.


Broken

Even the example is broken. Transformation bumps the element off screen.

Apologies, did not understand the behavior from the docs. It appears once you scroll up


[BUG] Edge kinds of schema returned by groot is wrong.

The src_vertex_id and dst_vertex_id is same even when the src label and dst label is different.
Reported by @lidongze0629

Closed as resolved.


SWITCHYARD-2920 Tests with SwitchYardRunner do not call CXF Intercept…

…ors of SOAP Service Binding

Can one of the admins verify this patch?

ok to test

Sorry @cunningt my bad - I've never thought the CXFJettyEndpoint is also used for karaf deployment. It means that the interceptors were not enabled on karaf as well, and this fix also addresses karaf deployment.


Installing via winget

It would be great if one could install ludusavi on Windows using winget.
It is blocked by lack of installing software from zip: https://github.com/microsoft/winget-cli/issues/140
Possible solutions are:

waiting for zip support
implementing installer

Then one have to add a manifest to this repo: https://github.com/microsoft/winget-pkgs

I'd be happy to submit a manifest to winget-pkgs once they support zips :+1: I'm not sure if I want to maintain an MSI, but I'll think about that option since some people may prefer it anyway.

https://github.com/microsoft/winget-pkgs/issues/35471

winget version 1.3 (currently unreleased) will support portable/standalone apps, so there won't be a need to maintain an MSI. See https://github.com/microsoft/winget-cli/issues/182

That's great news! I'll watch out for the release so that we can get Ludusavi listed :+1:

Support for installing via Zip files will be supported in winget 1.4 it seems: https://github.com/microsoft/winget-cli/releases/tag/v1.4.2011-preview

Nice work. Thanks!


find . -name *1.8.0*

Howdy,
Did you find something from v1.8.0 in the v1.8.1 release? And find/grep command didn't come up with anything 1.8.0 related in a fresh build/install of 1.8.1 or in the binary packages.

bad noise. one too many mouse click? I don't remember. Let's ignore this. Sorry.

no problem, just making sure there wasn't an issue. :)


Complete Rewrite

I'm taking this in a different direction that simplifies things while still offering flexibility.
Certainly a WIP, but wanted to open it up early as it's evolving. fairly stable api at this point, but would love feedback to see if there are things I'm missing!
Why change things up?

I think the previous implementation was too complicated. It fundamentally tried to do too much.
We can make it simpler, extend functionality, and make it easier to use at the same time.
Removes things that were complicating the situation -- the ability to refresh an individual page, for example.

Advantages of this path

One relatively small class to handle pagination.
Easy to extend and add custom pagination
Easy to type erase and operate over a custom model if you'd like
Lets folks use different initial queries vs next-page queries.
First class support for async/await!

Disadvantages of this path

The Hashed stuff is a little... I mean it's fine, I don't love it, but everything else requires a qualified type that adds a ton of complexity when you're working with the API.
Thread safety issues are still present fixed

As a heads up -- I'm likely going to pull the type erasure out of this prior to merge -- but developing it with a type erasure amount has been pretty crucial to making sure this API is flexible.

This definitely seems like a lot simpler API! It looks to me like we are sacrificing the included page merging functionality for the simplicity right? Is your thought process that users can write their own code to handle page merging if they want?

Exactly. If folks want to handle page merging they certainly have all the resources needed to do so, as the Output provides the raw results from each page.

Super excited for your work here to make pagination nice to work with. @Iron-Ham is the hero we need!
I'd love to add this to my app to help dogfood this in the wild, but I'm having trouble getting a basic Hello World setup. Any ideas what I'm doing wrong here?
My extractPageInfo and nextPageResolver never run.
    var pager: GraphQLQueryPager<TransactionsQuery, TransactionsQuery>? = nil
    struct CanLoadMore: PaginationInfo {
        let canLoadMore: Bool

        init(_ canLoadMore: Bool) {
            self.canLoadMore = canLoadMore
        }
    }

    func extractPageInfo(
        data: GraphQLQueryPager<TransactionsQuery, TransactionsQuery>.PageExtractionData
    ) -> CanLoadMore {
        print("extractPageInfo..") // This never runs
        print(data)
        return CanLoadMore(true)
    }

    func nextPageResolver(canLoadMore: CanLoadMore) -> TransactionsQuery {
        print("nextPageResolver..") // This never runs
        print(canLoadMore)
        return TransactionsQuery(updatedSince: .null)
    }

    func startPager() {
        print("Set up pager..")
        pager = GraphQLQueryPager(
            client: ApiManager.shared.client,
            initialQuery: TransactionsQuery(updatedSince: .null),
            extractPageInfo: extractPageInfo,
            nextPageResolver: nextPageResolver
        )

        pager?.subscribe { result in
            switch result {
            case .success(let dataGql, let data, let source):
                print("..subscribe success") // This runs
                print(dataGql) // Data(__data: ApolloAPI.DataDict(_storage: ApolloAPI.DataDict._Storage))
                print(data) // []
                print(source) // fetch
            case .failure(let err):
                print("..subscribe failure \(err)")
            }
        }

        print("refetch..")
        pager?.refetch()
    }

Here's the log output showing what actually runs. The query is definitely hitting my backend and returning data (I can see from backend logs)
Set up pager..
refetch..
..subscribe success
Data(__data: ApolloAPI.DataDict(_storage: ApolloAPI.DataDict._Storage))
[]
cache

My actual query looks like this
query Transactions($updatedSince: String) {
    transactionsNew(updatedSince: $updatedSince) {
        newUpdatedSince
        hasMore
        totalTransactions
        transactions {
            ...TransactionFragment
        }
    }
}

Hi @joshuakcockrell! Very kind of you 😄
I'm happy to help out where I can! A few caveats before we dive in:

This is all subject to change, as we're actively using & making changes to these APIs at GitHub.
My mental model of these APIs is a bit closer to this state (which I think for your purposes, is a minor change but an important one with regards to loadMore).
Do note the usage in the tests

Let's dive in! For the purposes of this example, I will be using your query as written above:
query Transactions($updatedSince: String) {
    transactionsNew(updatedSince: $updatedSince) {
        newUpdatedSince
        hasMore
        totalTransactions
        transactions {
            ...TransactionFragment
        }
    }
}

I assume that this should give me the following generated models:

TransactionsQuery
TransactionFragment

For the sake of simplicity, I'll use the provided CursorBasedStrategy.ForwardPagination, as it provides a few convenience methods in an extended static function. Let's start by declaring two necessary functions:

QueryProvider: This function's purpose is to go from an optional PageInfo? type to a Query type.
extractPageInfo: This function takes Query.Data and outputs a PageInfo of some sort.

Given my assumption of using ForwardPagination, the PageInfo in this case is a CursorBasedStrategy.ForwardPagination struct.
Declaring a QueryProvider:
func queryProvider(page: CursorBasedStrategy.ForwardPagination?) -> TransactionsQuery {
  return TransactionsQuery(updatedSince: page?.endCursor ?? .none)
}

Declaring a PageProvider (extractPageInfo):
func extractPageInfo(data: TransactionsQuery.Data) -> CursorBasedStrategy.ForwardPagination {
  .init(
    hasNext: data.transactionsNew.hasMore,
    endCursor: data.transactionsNew.newUpdatedSince
  )
}

Declaring a GraphQLQueryPager:
Now that we have both functions, we can easily declare our pager using the makeForwardCursorQueryPager static function on GraphQLQueryPager.
let pager = GraphQLQueryPager.makeForwardCursorQueryPager(
  client: ApiManager.shared.client,
  queryProvider: queryProvider(page:),
  extractPageInfo: extractPageInfo(data:)
)

We now have a valid GraphQLQueryPager object. However, we won't be notified of new data until we subscribe.
pager.subscribe { result in ... }

Finally, as you have, we would call pager.refetch().

A note to address this question:

My extractPageInfo and nextPageResolver never run.

They weren't called yet.
nextPageResolver is called when you call loadMore. extractPageInfo is called when you call either loadMore or canLoadNext.

I said earlier that my mental model is closer to that open pull request. There's a reason for that: The implementation in main, as it is now, wont' call its completion blocks after loadMore is called -- rather, it will call its blocks simultaneously with loadMore. If you don't pass a completion in, this isn't of any concern.
Additionally: I tend to work with these purely in the form of type erasures. That's because at GitHub we tend to work with multiple queries that produce the same outputs. The type erasure means that we can anonymously treat any of these Pagers the same, so long as they produce the same output model.

I hope this was helpful!
If it's a bit confusing, fret not -- I'll make sure that there's better documentation & examples prior to the first release of this package.

Thank you so much @Iron-Ham . I was able to mostly get it working this weekend with your help. I've used this for enough hours over the past several days to become quite opinionated about it, hopefully my opinions on the API are helpful for you to get some real world feedback, either to adjust things during development or just to know what to document once the first release happens (what tripped me up may trip up other people).
.subscribe
90% of the issues I encountered were from the .subscribe callback.
var results: [Result<GraphQLQueryPager<Query, Query>.Output, Error>] = []
  pager.subscribe { result in
    results.append(result)
    ...

I found it quite unintuitive to need to manage my own list of results like this. Combine's .sink api is great when you want to watch for changes to a list.
@Published var cars = ["Kia", "BMW", "Ford"]

self.$cars.sink(
    receiveValue: { cars in
        // Do something with the updated cars array
        print(cars) // ["Kia", "BMW", "Ford"]
    })

And I instinctively assumed the .subscribe api would handle the page aggregation for me and behave the same way. Instead, I was surprised to find the pagination API behaves like this:
(For this example, assume my page size is only 1 result, so we would need 3 total queries)
pager?.subscribe { result in
    switch result {
    case .success((let initialData, let paginatedData, let updateSource)):

        print(paginatedData)
        // 1st time subscribe emits: ["Kia"]
        // 2nd time subscribe emits: ["BMW", "BMW"]
        // 3rd time subscribe emits: ["Ford", "Ford", "Ford"]
    ..

Seems strange that the api returns the same results n times for the n number of total results.. Was I was doing something super wrong to get this kind of output and it's a bug? Or maybe this is intentional? Maybe it's common practice for pagination libraries in React to do this? I'm not sure, just giving feedback!
loadMore
Because the api works this way, I also got a little tripped up at the pager.loadMore() usage because it throws an error if it's missingInitialPage. Instead of calling pager.loadMore() at the start, I'm supposed to call pager.fetch(). Basically, it puts the burden on me of managing the has-the-pager-queried-the-first-page state. Managing this isn't that bad, buuuuut I'm already managing the state for my cursor, isn't that a great place to store this info? If my cursor is nil, then I want to start from scratch, if my cursor has a value, then I want to start where the cursor is. Maybe that's too simplified of a use case and not all api consumers have the luxury of getting to decide their queries start at nil, but maybe the api could just start at whatever the first String cursor you submit is?
I believe the thing that would make this work better is if pager.subscribe just immediately emitted whatever data it has. The same way Combine's publisher.sink immediately emits values. Then when I call pager.subscribe I could immediately respond to the data that's in the cache and decide how to set up my initial cursor.
func extractPageInfo
I feel like we could just get rid of this. Maybe there are use cases I'm not accounting for, but the job of extractPageInfo is to decide what the next cursor is and to set canLoadMore. But what if we just pass those two arguments into pager.loadMore(nextCursor) directly? Actually, what if we just drop the idea of setting canLoadMore and just assume that calling pager.loadMore() means we canLoadMore. We're usually calling pager.loadMore() from inside .subscribe, so we have the context to decide our next cursor and if we canLoadMore, and if we're forcefully calling pager.loadMore() from an external location (like the user scrolled to the bottom of the page) then we know we need to force canLoadMore = true and we already know what the cursor is.
After thinking through this, here's my ideal API changes:
pager.subscribe(.combined) { results in .. } // ["Kia", "BMW", "Ford"] Emits all pages as one array any time the query updates (emits right when you subscribe)
pager.subscribe(.lastPage) { results in .. } // ["Ford"] Emits only the lastPage any time it updates (emits right when you subscribe)
pager.loadMore(cursor: String?) // Loads the next page of data (or the first page if no data exists yet)
pager.fetch() // Remove this method to keep one way of doing things
extractPageInfo // Remove this and just put more responsibility on the pager.loadMore() method.
And here's the full flow I would see myself using:
// On app startup
let pager = GraphQLQueryPager(..)

// Set up my default cursor if I don't have any data
let cursor = nil

// Subscribe to entire array
// This would also immediately emit a value which would force me to set up my cursor
// If I have pages in the cache, users would instantly have offline data
// If I don't have pages in the cache, my cursor stays `nil`
pager.subscribe(.combined) {

  // When I want to load more pages, I run
  pager.loadMore(cursor)
}

// Externally, when I want to load more pages (like a user scrolled to the bottom), I run
pager.loadMore(cursor)

Hope this is helpful to explain what's going on inside the brain of someone using the code you wrote! Feel free to ignore everything as I'm sure you've spent more time thinking through this than me. Just appreciate the work you're doing here and want to help and use it 🙂

@Iron-Ham Thanks for the response. Sounds like ~all of my feedback is just preference stuff that doesn't account for the many ways people paginate data. Really cool you're building such a flexible approach that works for so many use cases!

I'd love to hear more about this though. In my usage, results look something like this:

Sounds like I may be encountering a bug that's responding with repeated results? (or I'm doing something really wrong)

I had never considered that someone would want to call loadMore from within subscribe

Oops. Not sure how I stumbled upon this pattern? Sounds like I'm doing something wrong.
I suppose I'm doing this because my use case involves (kinda) large amounts of data (dozens of pages with hundreds of entries). I'm using pagination to split up the network requests into smaller chunks. So for me, I often want to call .loadMore inside subscribe if there are more pages I need to load.

I suppose I'm doing this because my use case involves (kinda) large amounts of data (dozens of pages with hundreds of entries). I'm using pagination to split up the network requests into smaller chunks. So for me, I often want to call .loadMore inside subscribe if there are more pages I need to load. It's often unrelated to user scroll behavior.

To clarify -- you're doing a loadAll kind of setup from the get-go, as opposed to a trigger-based setup? So I assume your user sees a spinner while you:
Load Page 1
  |> Load Page 2
    |> Load Page 3
      |>...
Complete!

I haven't tried using this class for that just yet -- but I'm going to soon as I continue migrating things over.
Off the bat (and without trying this), I would think something like this might work?

let pager = GraphQLQueryPager(...)
pager.refetch()

let subscription = pager.subscribe { _ in
  guard pager.canLoadMore else { return }
  pager.loadMore()
}

while pager.canLoadMore {  } // probably don't do exactly this 😬

subscription.cancel()

pager.subscribe { result in
  // You now have all the data being returned as part of `result`.
}

Understood – hopefully with time, when this library is ready for release, it can help solve those problems for you.
With https://github.com/apollographql/apollo-ios-dev/pull/114, I've implemented a loadAll functionality (inspired by your use-case). The loadAll functionality doesn't have to worry about your initial fetch, tracking your cursor, and so on.
Merging page results is still up to the caller unless you're using the type-erased AnyGraphQLQueryPager. I'd love some additional feedback from your usage or experimentation when it's more fully baked & when there are some public examples & docs 😄


Row value cannot be accessed at expected key despite row object having expected keys and values

Operating System: MacOS BigSur 11.4
Node Version: 20.9.0 & 16.17.0
NPM Version: 10.1.0
csv-parser Version: 3.0.0

Expected Behavior
Logs the content of the object at the specified key. EG:
console.log(row['siteName'] )
would log "Your site" or "My site"
Actual Behavior
Doesn't extract the value of the object at the given key. The value at the column "siteName" can only be extracted using Object.keys(row)[0] 0 being the index of the first column.
How Do We Reproduce?
Here is the data

siteName
July
August

Your site
323
444

My site
111
222

const fs = require('fs');
const csv = require('csv-parser');

const file1 = './data.csv';
let totalGenerationData = {};

// Function to read CSV and extract the column data based on siteName
function readCSV(filePath, columnName, dataObject, callback) {
    fs.createReadStream(filePath)
        .pipe(csv())
        .on('data', (row) => {

            // As expected logs :
            // { 'siteName': 'My site', July: '28926', August: '37746' }
            // { 'siteName': 'Your site', July: '7107', August: '9352' }
            console.log(row)

            //As expected logs  "siteName" "siteName"
            console.log(Object.keys(row)[0])

            // Logs undefined undefined
            console.log(row['siteName'])

            // Logs "My site"
            // Logs "Your site"
            console.log(row[Object.keys(row)[0]])
        })
}

// Read both files and compare the data
readCSV(file1, '', totalGenerationData, () => { });

Interestingly the bug cannot be reproduced if siteName is the last column of the csv file.

It strangely seems like it could be a Node bug and not csv-parser , except if there is some sort of  unlikely delayed side effect.

Did you check that your file does not contain invisible characters in the header line? Try hexdump -C data.csv. Typical candidate migth be a BOM which adds 3 invisible chars at the beginning of the file.


Update cluster ssh

#1597  👈 (View in Graphite)
main

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


gapi.client.webmasters.searchanalytics.query Works only with parameter 'key' and not 'access_token'

I'm using the Google Search Console API, I was able to create a service account, connect it to my Search Console account and retrieve a valid access token through C#.  Then through Javascript I have to query the Search Console. The function I'm trying to use is the following:
gapi.client.webmasters.searchanalytics.query(
        {
            'access_token': token,
            'siteUrl': 'http://www.WEBSITE.com',
            'fields': 'responseAggregationType,rows',
            'startDate': formatDate(cSDate),
            'endDate': formatDate(cEDate),
            'dimensions': [
                'date'
                ]
        })
        .then(function(response) {
            console.log(response);
        })
        .then(null, function(err) {
            console.log(err);
        });

The problem is that the value for 'access_token' is not appended to the generated url so I get a 401 unauthorized response from the query.
If I replace 'access_token' with 'key' it gets appended but since this is an access token for a service account it still return an error. Is there a way to append the access token correctly? Or does the library not provide it yet?

hi I am having the same issue anyone have an idea on it?

hi I am having the same issue anyone have an idea on it?

Hey, this is what worked for me: https://stackoverflow.com/questions/38653502/unable-to-query-google-search-console-api-using-a-service-account


Maintainer

Would it help to have an additional maintainer for this project? I would be willing to lend a hand in getting the library updated and managing incoming PRs.

Yes, I don't have time / reason to maintain this library at the moment so I'd be happy to:

Add you to the project
Not sure if I can grant you access to Hackage, but that'd be fine by me

However, it might be a better suggestion that somebody else maintains a fork.

Hi @yamadapc, thank you for the reply.
What would be better in the forking scenario?
That would change the package name on hackage, or retain the same?
I'm not too strongly opposed or attached to a particular route - it's not really my original work, I'm just looking to carry it forward in some form :)

I’m not sure. If you have time I’ll just grant you access to the repository.
Fork would change the package name yes; but I’d be surprised if there’re a lot of consumers at this point as this library hasn’t been updated in years.
On 23 Apr 2021, 09:56 +1000, Ketzacoatl @.***>, wrote:

Hi @yamadapc, thank you for the reply.
What would be better in the forking scenario?
That would change the package name on hackage, or retain the same?
I'm not too strongly opposed or attached to a particular route - it's not really my original work, I'm just looking to carry it forward in some form :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

I don't have a lot of time, but I intend to chip away at things.
I'm fine creating haskell-hcl or haskell-language-hcl2 as a fork, if you'd prefer to archive this or otherwise leave it be.


Fix package references and add SBT file

Huge thank you! Been meaning to fix this for a while

My pleasure!


Allow pkexec to run X11 applications

From man pkexec:

pkexec will not allow you to run X11 applications as another user since
the $DISPLAY and $XAUTHORITY environment variables are not set.

One solution is to create a custom policy for an application setting the
org.freedesktop.policykit.exec.allow_gui option, which causes these
environment variables to be retained, however an easier workaround is to
manually preserve them by using env.
Fixes: https://github.com/jorangreef/sudo-prompt/issues/15
Signed-off-by: Juan Cruz Viotti<EMAIL_ADDRESS>

Uhm, actually, this works when I run it directly from my terminal, but not when including it in this module.


bug: repo index uses a different date format than all other dates in Helm

pkg/helm/repo.go uses a different time string for index.Created than the rest of Helm. This should be changed to use time. ANSIC or a UNIX timestamp.

#1257 includes an incidental fix for this, since it changes the repo format to use the standard time.Time type instead of a string type for date fields.


Buttonify

This implements part of https://github.com/datproject/dat-desktop/issues/280,
namely buttons and icons. It creates a slightly more intuitive API to create
elements, and decouples the icons from the buttons itself making them more
flexible. All elements should be accessible and reusable between projects.
The app runs and all event seem to work as expected which means I caught all
instances I think. Thanks!
API
var button = require('./elements/button')
var icon = require('./elements/icon')

icon('exit')                    // render an exit icon
icon('exit', { class: 'pt3' })  // render an exit icon with extra classes

button('some-button')                                      // render a plain button
button.green('my-cool-button', { onclick: handleclick })   // render a green button with an onclick handler
button.icon('my-cooler-button', { icon: icon('exit') })    // render an accessible icon-only button
button('some-button', { class: 'pr2 mt3' })                // render a plain button with extra classes

hah, found some beautiful glitches - hold on with the merge for a lil bit ok?

@Kriesse ran into an SVG styling issue; not sure what changed but could you perhaps take a look - styling SVGs has never been my forte. Once that's fixed this should be ready to merge.

@yoshuawuyts the intro screen and a few other small misalignments are fixed, ready to merge from my side! Preview:

Awesome, just ran through it and it's looking :100: - merging!


Fix Image#composite docs

The Vips::Image#composite docs weren't accurate, because the other can only be Image or Array<Image>. I think it might be a leftover from a copy-paste of the Vips::Image#bandjoin docs above.
Additionally, I also added Vips::BlendMode to document the possible values for mode (copy-pasted from VipsBlendMode libvips docs). I added that change as a separate commit, so you can take just the first one if you would prefer :)

I also renamed other to overlay, as that argument can only be a Vips::Image. This matches the naming in the Vips::Image#composite2 docs.


AsyncDirectives prevent mobx observables retriggering update

Expected Behaviour
Using async directives should 'just work' with the rendering component updating whenever an observable inside of the async directive is updated. The async directive of course should do what it always does.
Actual Behaviour
Mobx observables are not tracked within the callbacks of renders invoked by async directives like when and until.
Couple of possible approaches to fixing/workaround this:

Wrap all async directive render functions with helper functions to track the invocations and re-render the hosting component
Never use mobx observables directly in async directive render functions and instead always pass down the observable to a MobxLitElement that itself handles the re-render.
Extend Lits ReactiveElement with new hooks that are driven by AsyncDirective calls to setValue within their render tree. This is a much bigger change at the core of the library and would likely need updates to the AsyncDirective API.

Use runInAction() in async.
    @action
    async deleteComment(id: number): Promise<void> {
        try {
            await api.delete(`comments/${id}`);
        } catch (error) {
            console.error(error);

            return;
        }

        const changedCommentThreads = { ...this.commentThreads };

        changedCommentThreads.data[id] = null;
        runInAction(() => {
            this.commentThreads = { ...changedCommentThreads };
        });
    }

Not quite the same thing, thats an async action. In an async directive like asyncAppend that will invoke the template and update the DOM in an asynchronous callback that is outside of the reaction that is being used to capture template usage of mobx observables. Therefore observables inside of the asyncAppend callback will not cause further updates to the template. This can be quite surprising given the magic that happens elsewhere.

Thanks a lot Benjamin, I was about to lose my mind.


setup key mappings for other operating systems/keyboard layouts?

right now i've only mapped the keys for my macbook with a uk layout... i.e. 2 is @, etc.

though the original issue I'm closing due to #26


Update LSP services

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

GitHub release
Atom package

Your semantic-release bot :package::rocket:


fix(ci): nightly build

Fix the errors happening on the release action flow

Release Note
fix(ci): nightly build

No need to wait for the checks as this affects the github actions.


dsc order form

develop a form for dsc customers to enter shipping and billing information.

Please don't use submodules to bring dicty-react-components, instead add as dependencies in package.json.

Following are the suggestions made by Petra regarding the form

[ ] Country dropdown - country list must be in alphabetical order
[ ] Email field should be right below Last Name field
[ ] Have shipping services radio buttons horizontally
[ ] Only when one of the three shipping services (FedEx, UPS, DHL) is selected, Shipping Account Number field appears
[ ] Add checkbox when shipping info is the same as billing info. Checkbox text: "If payer is same as customer click here"
[ ] change Required error message into lower case --> required
[ ] Add Payment Method field - Three radio buttons: Credit Card, Wire Transfer, Purchase Order (PO). When PO is selected, Text box must be available to enter PO number (optional field)

The following information must be included in the form
Credit Card: Secure payment when billed
Wire Transfer: Northwestern bank info will be emailed
PO: Add PO # if available or send ASAP to<EMAIL_ADDRESS>For full payment info please click here: http://dictybase.org/StockCenter/OrderInfo.html

#31


Update version gonzales-pe 4.0.3

Gonzales-pe released some big changes, that fixes a lot of bugs due to this package.
Can de version be Updated in sc5 too?
Gr

@sarabarrerariano Hi. Updated :-) https://github.com/SC5/sc5-styleguide/releases/tag/1.4.0


Web examples

@oleggrib - as you mentioned. The logic inside the Negotiator must keep the types e.g. big int. I've updated the logic to send the data in it's intended format, and then inside the react view component, use to string() to cast the types to strings for the web view only.
Please review, comment or merge if you are happy with the changes.

I'm going to close this PR and raise a new with just the changes we talked about @oleggrib .

I'm going to close this PR and raise a new with just the changes we talked about @oleggrib .


Added error check in ats

Codecov Report

Merging #719 (1a626d8) into develop (72d5450) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #719   +/-   ##
========================================
  Coverage    95.48%   95.48%
========================================
  Files          138      138
  Lines         9046     9046
========================================
  Hits          8638     8638
  Misses         408      408

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 72d5450...1a626d8. Read the comment docs.


The presence of a  tag makes the all entire text bold, not just the text inside this tag.

I have text like that:
Title/nSome text
In output i get all text in bold, except only title.

richtext not supported


13 perfoming updates on models with custom managers

Resolves #13

Codecov Report
All modified lines are covered by tests :white_check_mark:

Comparison is base (ab8a41a) 100.00% compared to head (a34d563) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #21   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines           12        12
=========================================
  Hits            12        12

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


its not working in iOS 9.0 with Swift 2.0 in Xcode 7

Error -> "OBJC_CLASS$_NMSSHSession", referenced from:

ld: warning: directory not found for option '-F/Volumes/Important/Dropbox/bitbucket/Dew Drive/Dew'
ld: warning: directory not found for option '-FDrive'
ld: warning: ignoring file /Volumes/Important/Dropbox/bitbucket/Dew Drive/Dew Drive/NMSSH.framework/NMSSH, missing required architecture x86_64 in file /Volumes/Important/Dropbox/bitbucket/Dew Drive/Dew Drive/NMSSH.framework/NMSSH (2 slices)
Undefined symbols for architecture x86_64:
"OBJC_CLASS$_NMSSHSession", referenced from:
type metadata accessor for ObjectiveC.NMSSHSession in ViewController.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
I have these "arm64 armv7 armv7s" in my "valid architectures" in build settings. Please help me out to solve this.

How are you installing NMSSH and which version are you trying to compile?

I just built the NMSSH iOS project in iOS 9.0, and dragged the framework into my project. After that i tried to run my project, then it shown this error.

Successfully integrated NMSSH, by using CocoaPods.


样式被墙了

https://fonts.googleapis.com/css?family=Roboto:300,400,500,700|Material+Icons
这个字体样式不翻墙访问不到哟

the ip seems in Beijing, do you have a problem with accessing the fonts？

#136

I've fixed it, please update to the latest version and try again


Simplify Install Process

Right now the install process is quite lengthy (see the README).
We should be able to work with just: npm install -g .

fixed with commit: efe9daf14b60108a9a1a3eabc82316ae5ea97ed4


Fix accidental removal of moisture sensor pins from config.

This change is

Changes Unknown when pulling 7d9bdc6ad6bbee5f49c29fd2bdfa4f4bde41fd1b on fix_terrible_jeet_mistake into ** on master**.


added solution for LC problem 706

Related Issue
None
Description
Added solution for LeetCode problem 706 (Design HashMap). This includes a detailed description of the problem, constraints, and multiple solution implementations in Python, JavaScript, TypeScript, Java, and C++.
Type of PR

[x] Feature enhancement

Screenshots / Videos (if applicable)
N/A
Checklist

[x] I have performed a self-review of my code.
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have followed the code style guidelines of this project.
[x] I have checked for any existing open issues that my pull request may address.
[x] I have ensured that my changes do not break any existing functionality.
[x] Each contributor is allowed to create a maximum of 4 issues per day. This helps us manage and address issues efficiently.
[x] I have read the resources for guidance listed below.
[x] I have followed security best practices in my code changes.

Additional Context
This solution follows the format provided in the project documentation. It includes an array-based implementation of a hash map, with methods for adding, retrieving, and removing key-value pairs.
Resources for Guidance
Please read the following resources before submitting your contribution:

[x] Code Harbor Hub Community Features
[x] Docusaurus Documentation
[x] React.js Documentation
[x] Markdown Guide
[x] MDX Documentation
[x] Mermaid Documentation

Resources for DSA Features on Our Site:

[x] Tabs: Switch Languages
[x] Line Numbering
[x] Math Equations

Contribution Guidelines
Thank you for considering contributing to our project! To ensure smooth collaboration and effective contribution management, please adhere to the following guidelines:
Issue Creation

Limit on Issues:

Each contributor is allowed to create a maximum of 4 issues per day. This helps us manage and address issues efficiently.

Contribution Levels

Basic Contributions:

This project is primarily focused on documentation. Most of the setup has been completed, so contributors will generally need to work on basic code tasks, such as writing tests.
For these tasks, issues will be assigned the level1 label.

Acknowledging Hard Work:

If a contributor puts in significant effort on a task, the issue will be upgraded to level2. This is our way of recognizing and appreciating extra effort.

Feature Additions and Component Work:

Contributors working on new features or components using JSX/TSX will be assigned a level based on the complexity and quality of their work.
The more complex and valuable the contribution, the higher the level assigned.

Level Definitions

Level1:

Tasks are straightforward, such as fixing minor bugs, writing tests, or making simple documentation updates.

Level2:

Tasks require more effort, such as addressing complex bugs, improving existing features, or making substantial documentation improvements.

Level3:

Tasks are highly complex and involve significant new feature development, major refactoring, or extensive contributions to the project’s core components.

We look forward to your contributions and appreciate your effort in helping us improve the project!

@Ajay-Dhangar Sir i have added the solution for LC problem number 706. Please review it and let me know if there is a need to change.

@Ajay-Dhangar Sir if there is a way to see best sample solution format, pls let me know that so i can more nicely prepare solutions. Thank you!


Can't create computation graph that train on datasets on Azure storage

I am trying to create a Unet computation graph that should train on datasets stored on Azure but it fails with NullPointerException.
Please check attached screeshot:

Deeplearning4j version:1.0.0-beta4
Platform information (OS, etc): ubuntu

I created a docker image to reproduce the error. All you have to do is clone this repo, checkout branch "1.0.0-beta4-unet-comp_graph-azure" and from the main folder run "docker-compose up"
https://github.com/localmaxima/dl4j-azure-minimal-example.git

Given the amount of time spent to create such a ticket with all required tools setup(Docker/Spark...etc),  and given that it's been six months without even a comment or any reaction on my ticket, I will just close it and thank you guys for your past efforts.

@localmaxima have you tried the latest release of dl4j?


[Compile]: add support for web:1.0.0

Operating System
any
Plugin
package_info_plus
Version
8.0.1
Flutter SDK
3.24.0
Compilation command
flutter pub add package_info_plus
Dart Dependencies
web:^1.0.0

Native dependencies
No response
Compilation Error
Resolving dependencies...
Because <APP> depends on package_info_plus ^8.0.1 which depends on web >=0.5.0 <=0.6.0, web >=0.5.0 <=0.6.0 is required.
So, because <APP> depends on web 1.0.0, version solving failed.

You can try the following suggestion to make the pubspec resolve:
* Consider downgrading your constraint on web: flutter pub add web:^0.5.1

Flutter Doctor
not relevant

Checklist before submitting a bug

[X] I searched issues in this repository and couldn't find such bug/problem
[X] I Google'd a solution and I couldn't find it
[X] I searched on StackOverflow for a solution and I couldn't find it
[X] I read the README.md file of the plugin
[X] I'm using the latest version of the plugin
[X] All dependencies are up to date with flutter pub upgrade
[X] I did a flutter clean
[X] I tried running the example project

There is a pinned ticket regarding this: https://github.com/fluttercommunity/plus_plugins/issues/3093
package_info_plus has already been migrated to web 1.0.0: https://github.com/fluttercommunity/plus_plugins/pull/3102
The package is pending for a release.
Meanwhile, you can simply set dependency overrides for web 1.0.0.
Please don't create new tickets when there is already a pinned ticket covering this topic.


Update Diga Client Api to use jakarta

Description
The upgrade to Quarkus 3 or the latest version of Spring introduces a number of improvements, including using the latest stable releases of a plethora of Java libraries, including the migration from Java EE to Jakarta EE. Java EE platform classes were bundled in the package javax.*.
More applications have started the move from JavaEE or J2EE to Jakarta EE, and Quarkus 3 marks this transition.
Overall, The classes that previously resided in javax.* now are placed in jakarta.*.
Changes:

We have added extension to cxf-xjc-plugin Plugin to generate classes using jakarta also we added dependencies for Jakarta XML bind and activation, as a result we upgraded the JAXB runtime to 4.0.9.

Thanks

Thank you for your contribution @khaledmohamed-oviva ! I will look into this shortly

thanks for your PR! I'm sure more people will have run into this issue
I just had some questions about some POM decisions when it comes to versions, extensions, etc

Thank you so much for reviewing the PR and for the nice tool you developed.
I have resolved the comments you raised, and I could be able to build the project without any issue. Thanks for suggestions.
Could you please let me know what is the new version that I can use after merging the PR ? Thanks a lot

This is live now in 2.0.2. Thanks for your contribution!


fix(MD-2978): fix missed subtitle prop

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat: add unit argument and apply conversion for kms

This is a:

[x] bug fix PR with no breaking changes — please ensure the base branch is master

[x] new functionality — please ensure the base branch is the latest dev/ branch
To be honest I'm not too sure what this counts as. #338  is labelled as a bug, but it's also kind of a feature request --let me know what you want to consider this one as and I'll adjust the diff target

[ ] a breaking change — please ensure the base branch is the latest dev/ branch

Description & motivation
the haversine_distance() calculation outputs results in miles only at the minute. Interest in being able to output haversine in kilometers was registered in #338 .
Changes
calling with unit='km' (distance between Amsterdam and Paris)
select {{haversine_distance(48.864716, 2.349014, 52.379189, 4.899431, unit='km')}} as hav_dist
from table_a

returns ~ 430

calling witout a unit param or (unit='mi')
select {{haversine_distance(48.864716, 2.349014, 52.379189, 4.899431)}} as hav_dist
from table_a

returns ~267

Closes #338
Checklist

[x] I have verified that these changes work locally on the following warehouses (Note: it's okay if you do not have access to all warehouses, this helps us understand what has been covered)

[ ] BigQuery
[x] Postgres
[ ] Redshift
[ ] Snowflake

[x] I have updated the README.md (if applicable)

[x] I have added tests & descriptions to my models (and macros if applicable)

[ ] I have added an entry to CHANGELOG.md

Not sure if this change would cause a minor or patch increment. In semver world since this add a backwards compatible features it should trigger a bump to 0.7.0 but not sure if you follow that here, so let me know and then I can update the changelog accordingly.

🙌 Thanks @bastienboutonnet!
This repo contains another project which imports dbt-utils and runs integration tests against seed files. Check out https://github.com/fishtown-analytics/dbt-utils/pull/334/files for an example of configuring tests

Thanks for the pointers @joellabes I'll see about writing a test for this function at some point.

@joellabes I added some tests and they seem to pass locally and on circle CI except for snowflake and BQ. I'm assuming because I'm a 3rd party these tests are bound to fail? Either way, let me know if that's the issue of if there is anything I should do (or not do) in the code.
Regarding the changelog todo on the PR template my question still stands let me know what I should do when we get closer to merge I guess?

I'm assuming because I'm a 3rd party these tests are bound to fail?

I don't think that's true - I've had all the adapters succeed on past PRs. My best guess is that BQ and SF do something differently - perhaps their column type inference isn't playing nicely with the seed, or the conversion_rate multiplication doesn't work for some reason. I only have access to a Redshift warehouse so can't muck around to find out why, and I'm external to Fishtown too so I can't see the CircleCI errors - @clrcrl will have to let you know the specifics.
In integration_tests/models/geo/test_haversine_distance_mi.sql, it might be nice to union that model and run it again without passing in the mi argument to test the default/backwards-compatible case as well (unless you have that covered somewhere else that I missed).
0.7.0 is the next significant (intentionally avoiding semver words) version of dbt-utils, so I expect that this sort of net-new functionality would go there as opposed to a patch version. Again a good q for Claire

@joellabes with the union part of your message I'm not sure I understand. I think I understand you'd like a test where we call the macro without the uni argument. But I don't know why union. Did you have something in mind?

Did you have something in mind?

Broadly, I was thinking something like
select convert(lat_1, lon_1, lat_2, lon_2, ‘mi’)
from table

union all

select convert(lat_1, lon_1, lat_2, lon_2)
from table

(written on my phone so only very broadly accurate)
By doing that, you’d get 0, 1 or 2 failing rows depending on which part broke. It’s probably bad practice to overload the test, but I was trying to avoid having to make a third separate query 😬
not sure if there’s a SF-friendly version of the macro sorry! Weird that it got this far without breaking. Maybe it’s never worked but no one uses this on BQ or SF

@joellabes Ok I see what you meant. I can get that done in that way I think it's not that bad actually coming to think about it.
So yeah I checked into my team's macro for haversine and we have used the pow() functions on our Snowflake warehouse. I think there is a way to distribute warehouse specific macros and let the macro adaptor choose it at runtime, I'll see if there is an example of this already on the repo or wait till you or @clrcrl let me know how best to do it.
But YES! very strange people didn't complain so far, or maybe they have been like us and have used their own macros for haversine. Anyway, would be a great idea to fix that for everyone right?

@joellabes I just implemented the unioned test and it seems to work fine on pg and redshift.
What do you think should happen qua the other dbs?

I think I may just be good with creating a new macro and name it snowflake__haversine_distance with the db specific function in it and dbt would take care of selecting it based on the adapter type?
For BQ I was looking and they don't seem to have radians function to convert something that's in degrees + I cannot test it as I can't play with a BQ warehouse. Looks like if we implemented it for BQ we'd also have to make some maths happen to convert radians to degrees. It's probably doable though.
Let me know what makes sense.

Sorry for the delay getting back to you!

I think I may just be good with creating a new macro and name it snowflake__haversine_distance with the db specific function in it and dbt would take care of selecting it based on the adapter type?

Yes I think that's right!
Re BigQuery, if it was already broken and you don't use BQ, I don't think you need to feel obliged to bend over backwards to make it work. You could either add an implementation that no-ops and returns null, or throw the equivalent of a NotImplementedException, or do nothing at all and it will throw a more confusing error. When a BQ user comes along who wants this, they can always open another PR. I think there are other macros that do this but I don't remember which offhand.

no worries at all!
Ok I'll have a go at a snowflake implementation for sure because I can actually test it out. For BQ I think what you propose makes sense, however not sure why you're pointing me to the Microsoft website here:

or throw the equivalent of a NotImplementedException,

I'd actually like to be able to throw a proper error if BQ users were to use this macro rather than a null. How best do you think we can do that?

not sure why you're pointing me to the Microsoft website here

In a past life I was a C# developer, so that was the concept I knew it as - I'm not sure what the dbtonic way of doing this is.
The closest equivalent I could find was https://github.com/fishtown-analytics/dbt-utils/blob/c64b52068070023bc3524525e06a14ad72d485a4/macros/cross_db_utils/datediff.sql#L55. I think you'd basically have to make a stub of a bigquery implementation that threw the exception.
Then there would have to be some way of excluding that one from CI, because an exception that's intentionally thrown is still an exception... Hopefully @clrcrl has some way of excluding them somehow 🤔

@joellabes so I looked into the postgres and redshift documentations and they also have the pow() function that is the only way to make snowflake calculate power. So in the interest of not duplicating too much code I went ahead and made the default__ implementation use pow(). Judging by the tests this seems to work fine for PG, redshift and snowflake which is really nice.
I'll work on making the BQ exception raising happen soon.

@joellabes @clrcrl in the end I went ahead and also worked an adaptor macro for big query, it was a bit of a pain but I managed. I created so many failing CI runs I hope you folks don't get notified on each failures.
Anyway, let me know if you have any additional feedback

Incredible, thanks so much @bastienboutonnet! Sorry that this turned out to be so much more work than first thought 😭

Ah no worries, if I didn't want to do it I would have said it. I was stuck on my other open source projects today so it was something fun and useful to do.
Is there a changelog entry I should be writing btw? And or some documentation?
Also, not sure if this should be merged into the main branch or develop.
I guess by this point this PR is more of a feature than a bugfix so I rebase on dev/ and change the target of the PR.

Not sure on those logistics - Claire's got "clear the packages backlog" on her sprint for this fortnight so I imagine she'll be checking this out pretty soon though!

Hey @clrcrl thanks a lot for the feedback all done. Hope this works 🥳

@clrcrl Thanks for merging the PR. Btw, I had not added any changelog entries yet because I wasn't sure whether this was bug or feature. Would you like me to add something to the changelog or you will do it when you release?


Upload Progress details not working in heroku app

I'm trying to make a rails app with refile that has user images.
For these images I want a progress bar that displays how much and image has uploaded, and then when the upload is complete
Here is my code
/** users/show.html.erb **/
        <% if @user == current_user %>
            <%= link_to "Edit Account", edit_user_registration_path %>
            <%= render 'image' %>
        <% end %>

/** image partial **/

    <% @user.images.each do |image| %>

            <div class="picture-card" style="background-image: url('<%= attachment_url(image, :picture) %>')">

            <%= link_to "Delete Image", url_for([@user, image]), :method  => :delete %>

    <% end %>

        <%= render "image_fields" %>

/** image_fields partial **/

    <%= form_for [@user, Image.new] do |f| %>
        <%= f.label :picture, class: "image-label" do %>

                Add an image

        <% end %>
        <%= f.attachment_field :picture, direct: true, class: "image-form hidden" %>

    <% end %>

The issue is that detail.loaded and detail.total are working locally, but are displayed as NaN in production on heroku, rendering the progress bar useless

What do you mean by "are not numbers"? It's string? Would a parseInt() work?

@wkurosawa I should have specified that better. Detail is part of the refile customEvent for ProgressEvent. It works fine locally, but displays as NaN instead of a number that jquery can use to style a progress bar

I see, but I have been using a parseInt and it's working like a charm.
 var progress = parseInt(data.loaded / data.total * 100, 10);

We have created a gem with Refile to encapsulate our file upload front end and been using Heroku, as you can see on this demo:
https://filemagic-demo.herokuapp.com/
Also, code is at: https://github.com/flama/filemagic

@wkurosawa Thanks, but it didn't work. Its fine locally but in heroku it still shows that percentComplete is "NaN"

Have you tried using presigned uploads? Im assuming you must have some sort of storage solution since you are using heroku.
https://github.com/refile/refile#presigned-uploads

Closing due to the lack of activity.


Fix ios first access to addressBook after request permisions

Platforms affected
iOS
Motivation and Context
iOS crashes on first request to AddressBookContacts after accepting permissions
Description
Added dispatch_sync
Testing
In iOS, open the app for first time and accept the permision of access on contacts and open the contactbook. It will not crash
Checklist

[ ] I've run the tests to see all new and existing tests pass
[ ] I added automated test coverage as appropriate for this change
[ ] Commit is prefixed with (platform) if this change only applies to one platform (e.g. (android))
[ ] If this Pull Request resolves an issue, I linked to the issue in the text above (and used the correct keyword to close issues using keywords)
[ ] I've updated the documentation if necessary

We are archiving this repository following Apache Cordova's Deprecation Policy. We will not continue to work on this repository. Therefore all issues and pull requests are being closed. Thanks for your contribution.


TransferCredentials returns session key instead of bool

This commit adds support for returning a session key from pseudonymys's TransferCredential function, for both modular and EC implementation (up until now we were only returning bools indicating whether authentication was successful or not). Server struct now embeds a new sessionManager struct tasked with generation of session keys.
When session key cannot be returned (either because of failed authentication or some other error), ProtocolError field of the Message is set, while SessionKey (to be inserted within Message.Content) remains unset. Newly generated Go-code for protocol buffers is included.
You will notice that tests for the pseudonym system schemes are not ideal, and have grown quite a bit (not only a a result of this PR). I would prefer some refactoring to keep tests really unitary and therefore more manageable, but I'll handle this in another PR (remember, @miha-plesko? :wink:)

Oh, I remember pretty well @mancabizjak, my hand started shaking that day when a PR with 43 (that's fourty-three, yes) modified files stroke me out of blue ⚡ 🌩 😄


FEATURE REQUEST

REALLY in need of a 1.19.3 version. :D Would love you forever. Hope to hear from you soon. Thanks!

Insee this was closed but I see no 1.19.3 forge version in the files. Is it uploading and waiting approval or this is closed/completed as in its not happening? Sorry just unclear.


Download speed of database pdb_mmcif is very slow, about 80 KB/s

Is there any solution

You could try using a different mirror, see https://www.wwpdb.org/ftp/pdb-ftp-sites for a list of mirrors available.
Change the mirror in scripts/download_pdb_mmcif.sh. Here is an example that uses EBI's mirror:
rsync --recursive --links --perms --times --compress --info=progress2 --delete --port=33444 \
  rsync.ebi.ac.uk::pub/databases/pdb/data/structures/divided/mmCIF/ \
  "${RAW_DIR}"

Try to use a mirror that is close to your geographic location (PDBj for Asia, EBI for Europe, RCSB for America).


Introduce file-storage support for Windows

Introduce a new source file, which implements the full set of file
chunk API using Win32 functions.

The main blockage for storage support was the lack of mmap(2) on
Windows. This patch circumvents the limitation by using "files"
as shared buffers.

Even though the base technology is different, the functionality
is all the same from users' standpoint. It can read and write
files with chunkio's file layout.

Metadata and checksums are fully supported too.

A major motivation behind this patch is to enable Windows users
to persist input data on disks on Fluent Bit.
Signed-off-by: Fujimoto Seiji<EMAIL_ADDRESS>

awesome! thank you!


Lack of lines or background color changes on panel/viewlet make it difficult to resize panes

VSCode Version: code-oss-dev 1.9.0 (Commit unknown, Date unknown)
OS Version: Linux x64 4.8.0-34-generic
Extensions:

Extension
Author
Version

I noticed that the borders and background colors are gone in the light theme. This makes it very difficult to tell what is part of the panel and what is part of the editor. The viewlet and panel are also hard to grab since I needed to search for an invisible line to grab.
Master:

Stable:

The dark theme seems fine:

Maybe we should not remove the border we used to have given some themes might be broken now?

Yes, that doesn't look good in the "Quiet Light" theme.
We need to either 1) add back the top border or 2) use an equivalent rgba color that will always darken or lighten whatever it sits upon (depending on dark or light).
1 is simpler, judging that we are so close to code freeze.  We can work on exploring 2 or another option later.

@bgashler1 to be fair, this does not look good either:

Maybe we should not have a separate background color at all then:

@bpasero perhaps until we have enabled full workbench UI theme awareness this might be the right choice.  Any word on when that will be in the product?  I think the red theme has always looked awful with our gray UI next to it.  Would be nice to get that in.
As a workaround, we could use an equivalent-looking alpha color that looks exactly as it does in light and dark theme today but would blend better in the color themes.


defaults: Add Rollback defaults

Right now, the updater uses Update defaults for a rollback if no Rollback settings are provided in the service. Make the Rollback defaults explicit instead.

Just double-checking - by the time the service gets to the updater, it will be interpolated already with the defaults?

No, interpolation is never used internally. It's an option for clients retrieving the service. See #1744.

@aaronlehmann Ah ok, thanks.

LGTM

Thanks


Refactor planner interface

This refactors the planner interface, making improvements for readability, improving debugging and results of the planner.
Change-type: minor

I self-certify!


Sometimes when you add an item, the new item empty entry is set to yesterday

I don't really know the specifics, but sometimes when I add an item, the new "empty" time to start tracking time on again gets set to yesterday as a start date.

Looks like it has to do with having the Thyme app opened from yesterday. Maybe an interval which updates the user's current date will fix it.


Unsupported top level event type "onDragViro" dispatched

Hi, I tried to implement dragging into a Viro3DObject, and it drags although an error pos-up as soon as I start dragging it saying "Unsupported top level event type "onDragViro" dispatched". Any ideas how could I solve this?
Thanks Andras

Hi,
I'm also getting this error message. Any updates?
I'm using React Native v0.55.1
thanks

Hi @AntoniaBe,
This looks like a bug on our platform, a workaround you can do is to simply import a ViroNode and then add an empty <ViroNode /> component to your scene.
That should load most, if not all, the events and get around this issue.
We'll fix this again in our next release!
Thanks,


Seeing Draft projects in logged out state

When I view someone's profile and logged out, I'm able to click on the Projects tab and see their draft projects. I think I should only see their projects if they're approved. Maybe we can add a "No Projects Yet!" to the tab if there are no approved projets.

implemented! Verify!


[Feature Request] Please allow add edge properties in Loaders

Memgraph version Which version did you use? v2.13.0
Environment Some information about the environment you are using Memgraph
on: operating system, how do you connect, with or without docker, which driver
etc.
Describe the bug
Currently, there are no way to add properties to relationship created with "gqlalchemy.transformations.importing.loaders".
Please allow to add properties into created relationships in "many_to_many" cases.
Expected behavior
Allow to to add properties as well as column mappers.
many_to_many_relations:       # intended to be used in case of associative tables
  example:
    foreign_key_from:        # describes the source of the relationship
      column_name:
      reference_table:
      reference_key:
    foreign_key_to:          # describes the destination of the relationship
      column_name:
      reference_table:
      reference_key:
    label:
    column_names_mapping: {"current_column_name": "mapped_name"}

Logs
No log
Additional context
All properties in associate table should be added to relationship properties.

Hi @SuperBo thank you for opening this feature request and bringing more awareness to the issue :)


Gradient Descent in qGAN optimisation

Environment

Qiskit Machine Learning version: 0.3.1
Python version: 3.8.10
Operating system: macOS-12.2.1-x86_64-i386-64bit

What is happening?
I'm trying to run an optimisation of the qGAN model. However, when I try to change the generator optimizer from ADAM to GradientDescent, I get an error during the qgan.run(quantum_instance) : "[Expression cannot bind non-numeric values ({ParameterVectorElement(θ[0]): array([-5.15174274e-05, -4.29241330e-05, -4.65541899e-05, -5.62154682e-05,\n       -5.74424778e-05, -4.81389433e-05, -5.85206010e-05, -5.40145357e-05,\n       -4.95023381e-05, -5.36247832e-05, -6.18140874e-05, -4.41827414e-05])})]()"
How can we reproduce the issue?

seed=23
np.random.seed(seed)

from qiskit import QuantumRegister, QuantumCircuit, BasicAer
from qiskit.circuit.library import TwoLocal

from qiskit.utils import QuantumInstance, algorithm_globals
from qiskit_machine_learning.algorithms import NumPyDiscriminator, QGAN
from qiskit.algorithms.optimizers import ADAM, Optimizer, GradientDescent
algorithm_globals.random_seed = seed

# Number training data samples
N = 20000

# Load data samples from log-normal distribution with mean=1 and standard deviation=1
mu = 1
sigma = 1
real_data = np.random.lognormal(mean=mu, sigma=sigma, size=N)

# Set the data resolution
# Set upper and lower data values as list of k min/max data values [[min_0,max_0],...,[min_k-1,max_k-1]]
bounds = np.array([0, 7.0])
# Set number of qubits per data dimension as list of k qubit values[#q_0,...,#q_k-1]
num_qubits = [3]
k = len(num_qubits)

# Set number of training epochs
# Note: The algorithm's runtime can be shortened by reducing the number of training epochs.
num_epochs = 2000
# Batch size
batch_size = 2000

# Initialize qGAN
qgan = QGAN(real_data, bounds, num_qubits, batch_size, num_epochs, snapshot_dir=None)
qgan.seed = 12
# Set quantum instance to run the quantum generator
quantum_instance = QuantumInstance(
    backend=BasicAer.get_backend("statevector_simulator"), seed_transpiler=seed, seed_simulator=seed
)

# Set an initial state for the generator circuit as a uniform distribution
# This corresponds to applying Hadamard gates on all qubits
init_dist = QuantumCircuit(sum(num_qubits))
init_dist.h(init_dist.qubits)

# Set the ansatz circuit
ansatz = TwoLocal(int(np.sum(num_qubits)), "ry", "cz", entanglement='circular', reps=3)

# Set generator's initial parameters - in order to reduce the training time and hence the
# total running time for this notebook
init_params = np.random.uniform(low=-1e-1, high=1e-1, size=ansatz.num_parameters_settable)
# Set generator circuit by adding the initial distribution infront of the ansatz
g_circuit = ansatz.compose(init_dist, front=True)

# Set quantum generator

generator_optimizer=GradientDescent(maxiter=1, learning_rate=1e-3)
qgan.set_generator(generator_circuit=g_circuit, generator_init_params=init_params, generator_optimizer=generator_optimizer)
# The parameters have an order issue that following is a temp. workaround
qgan._generator._free_parameters = sorted(g_circuit.parameters, key=lambda p: p.name)
# Set classical discriminator neural network
discriminator = NumPyDiscriminator(len(num_qubits))
qgan.set_discriminator(discriminator)

result = qgan.run(quantum_instance)

What should happen?
It should optimize, instead of outputting the error.
Any suggestions?
No response

The QGAN implementation is deprecated in https://github.com/Qiskit/qiskit-machine-learning/pull/495. Unlikely this issue will be fixed.

Closing this issue as the original QGAN implementation has been removed.


Fix reanimated onSize

Reanimated onSize was not changed properly, but it works if make shallow copy of new sizes

I have signed the CLA!

Nice! Can you provide me with a small example where I can reproduce the issue and test the fix?

I will try to provide today

https://github.com/XantreGodlike/skia-on-size-not-working

Testing on android

I think this can be related with using of useEffect, because it working after layout, so  we will never receive first canvas size

@XantreGodlike this looks interesting but I couldn't figure out any change of behaviour from this PR.
Also the code contains a couple of changes from the original PR could go over the list of changes made and give an explanation of there indent. I like a lot the idea of factorizing the onSize logic into it's own private hook.

@XantreGodlike I'm able to reproduce the bug and see that your PR fixes. I made some small cosmetic changes (see below).
However I'm baffled that addEventListener appears to return the same object identity on each event. I need to investigate this before merging this PR.
const useOnSize = (resultValue: SkiaBaseViewProps["onSize"]) => {
  const onSize = useValue({
    width: 0,
    height: 0,
  });

  useLayoutEffect(() => {
    if (!resultValue) {
      return;
    }

    return onSize.addListener((newValue) => {
      const currentValue = isValue(resultValue)
        ? resultValue.current
        : resultValue.value;

      if (
        currentValue.height === newValue.height &&
        currentValue.width === newValue.width
      ) {
        return;
      }

      const valueCopy = Object.assign({}, newValue);
      if (isValue(resultValue)) {
        resultValue.current = valueCopy;
      } else {
        resultValue.value = valueCopy;
      }
    });
  }, [resultValue, onSize]);

  return onSize;
};

@XantreGodlike thanks a lot for bringing this to our attention. We we will get it sorted out in a timely manner. I opened #1682  and #1681 as a prerequisite to this PR.

In the meantime, I trust that you should be blocked by this bug?

I'am not block it for now, i've just wrapping my onSize shared values with useOnSize hook

Merging it in the integration branch so we can run the CI on it. I will finalize it and merge it.
Thanks a lot of catching this and taking the initiative on the fix. We hope to see contribute other wonderful things to the project :)


How do I generate a watershed boundry from a digital elevation model?

A graduate student in Agricultural & Biological Engineering is interested in delineating a watershed of her study area from a digital elevation model. User is interested in using elevation models and other geospatial environmental data to contextualize study area.
User has some experience in working with elevation models in different contexts; however, is looking for hydrologic examples and relevant information related to study area.

@olendorf no idea what the problem is.


Corrected capitalization.

This does not necessitate explanation.

Learn Build status updates of commit 5ad8c9e:
:bulb: Validation status: suggestions

File
Status
Preview URL
Details

windows/security/threat-protection/microsoft-defender-application-guard/install-md-app-guard.md
:bulb:Suggestion

Details

windows/security/threat-protection/microsoft-defender-application-guard/install-md-app-guard.md

Line 2, Column 1: [Suggestion: ms-topic-missing - See documentation] Missing required attribute: 'ms.topic'. Add a valid value to indicate the article type.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


set custom multicast hop limit

Codecov Report

Merging #102 into master will decrease coverage by 0.12%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #102      +/-   ##
==========================================
- Coverage   62.03%   61.91%   -0.13%
==========================================
  Files          34       34
  Lines        3324     3324
==========================================
- Hits         2062     2058       -4
- Misses       1048     1051       +3
- Partials      214      215       +1

Impacted Files
Coverage Δ

client.go
61.01% <100%> (ø)
:arrow_up:

net/conn.go
67.85% <0%> (-7.15%)
:arrow_down:

Continue to review full report at Codecov.

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


Fix typo, add missing space between two words.

Fix typo, add missing space between two words.

Hi!
Could you please open an issue on our project tracker describing the issue and attace a patch:
https://issues.apache.org/jira/browse/HBASE
for more information on submitting contributions to the project, please read through our new contributor section:
http://hbase.apache.org/book.html#getting.involved
http://hbase.apache.org/book.html#submitting.patches.create

still interested in working on this? without a response it'll get closed as stale.

Hi @busbey , sorry for the extremely late reply.
I have created it=> https://issues.apache.org/jira/browse/HBASE-15574
Thanks for your help.


CLOUD-51308 workaround for sporadic http url connection concurrency i…

@schfeca75
I will check whether this can be fixed with ApacheHTTP client instead of HTTPUrlConnection

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1976/


import/export project

requested by @szeke in #180, moving to its own issue
The desired capability:
Project export button: saves a zip file with a project, user can save it anywhere they want
Project import: import a zip file that was saved using export:
If the project is new, create it
If the project exists, give the user the option to overwrite or cancel.

We should test that importing directories is working as expected, and then everything else in this issue we've decided won't be directly t2wml functionality-- zipping/unzipping files-- the user will just do that on their own.

version 2.3.4
please close when tested ok

Cannot be tested properly until #247 is fixed

should be checked now that #247 is fixed


Fix default values

What does this PR do?
Fix default values
Motivation
Accurate doc
Review checklist

[x] PR has a meaningful title or PR has the no-changelog label attached
[ ] Feature or bugfix has tests
[x] Git history is clean
[x] If PR impacts documentation, docs team has been notified or an issue has been opened on the documentation repo
[ ] If PR adds a configuration option, it has been added to the configuration file.

@zippolyte @l0k0ms 👌 ?


Lots of broken Markdown hides key documentation

There is a lot of broken Markdown on this page and its bad enough that it hides key documentation context.

I've taken a look and the page is rendering fine: https://docs.microsoft.com/en-us/power-virtual-agents/publication-connect-bot-to-custom-application
Is there a specific section or area where you are seeing problems?

Closing due to inactivity.


fix: run npm audit to resolve some security vulnerabilities

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Missing Type Annotations

There are missing type annotations for some functions in this library.
The typing module does not exist on CircuitPython devices so the import needs to be wrapped in try/except to catch the error for missing import. There is an example of how that is done here:
try:
    from typing import List, Tuple
except ImportError:
    pass

Once imported the typing annotations for the argument type(s), and return type(s) can be added to the function signature. Here is an example of a function that has had this done already:
def wrap_text_to_pixels(
    string: str, max_width: int, font=None, indent0: str = "", indent1: str = ""
) -> List[str]:

If you are new to Git or Github we have a guide about contributing to our projects here: https://learn.adafruit.com/contribute-to-circuitpython-with-git-and-github
There is also a guide that covers our CI utilities and how to run them locally to ensure they will pass in Github Actions here: https://learn.adafruit.com/creating-and-sharing-a-circuitpython-library/check-your-code In particular the pages: Sharing docs on ReadTheDocs and Check your code with pre-commit contain the tools to install and commands to run locally to run the checks.
If you are attempting to resolve this issue and need help, you can post a comment on this issue and tag both @foamyguy and @kattni or reach out to us on Discord: https://adafru.it/discord in the #circuitpython-dev channel.
The following locations are reported by mypy to be missing type annotations:

[ ] adafruit_bitmapsaver.py:39
[ ] adafruit_bitmapsaver.py:47
[ ] adafruit_bitmapsaver.py:57
[ ] adafruit_bitmapsaver.py:63
[ ] adafruit_bitmapsaver.py:70
[ ] adafruit_bitmapsaver.py:78
[ ] adafruit_bitmapsaver.py:109

PR opened and ready


feat: Update data science page RHCLOUD-18476

What this PR does / why we need it:
Implements requests updates to data science page requested in RHCLOUD-18476 jira ticket
Verification steps
TBD
Is there any work left / what's next :
Once released update app-services-ui
Special notes for your reviewer:
TODO:

[x] implement API interaction
[x] Tests

https://github.com/data-driven-forms/react-forms/discussions/1304

Yeah, it links the user to a page on the HCC UI. Not user if this something we want to configure or what.
I don't think we need approval from someone else ATM.


Use default arguments in addition to kwargs in local wf execution

TL;DR
Previously _local_execute would be called without checking for default arguments. See https://github.com/flyteorg/flyte/issues/951 for more details.
Type

[ X] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[ ] Code completed
[x] Smoke tested
[x] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Tracking Issue
https://github.com/flyteorg/flyte/issues/951

linting is still failing, did you make fmt?

i don't trust codecov in this case.


Does this tool provide the

Does this tool provide the same function as DVR-Scan?

Closing as duplicate of #43 (most recent discussion here).  TL,DR:  No, currently PySceneDetect does not provide the same functionality, although the plan of record is to eventually combine it's functionality and depreciate DVR-Scan.


load checkpoint when fine-tune QM9

Hi,
I met a problem when I tried to load 'L12-old.pt' in finetuneqm9.sh , the program told me that the checkpoint's  structure mismatched, how can I solve this problem?

Have you solved the problem?

Same Question


[usecase-cordova] Remove shared mode in DefaultCrosswalkVersion

For default crosswalk version for webview plugin, no need shared
mode testing, it will use local runtime lib verison. default version
should be testing in embedded mode only.
Failure Analysis:
XWALK 6879 Webview plugin default crosswalk version is not 19
even if crosswalk is 19 in maven2/org/xwalk/xwalk_core_library_beta.
Impacted tests(approved): new 0, update 1, delete 0
Unit test platform: WebView Plugin 1.7.0
Unit test result summary: pass 0, fail 1, block 0

@haoyunfeix,  PTAL

lgtm


Reuse the same sink-file in bdd_sink

Handles the sink case of #85

I find it kinda odd, that more pass after updating for the #117 was merged into master. I have confirmed that it truly checks out this feature branch and used it for the dependency for the benchmark. But, of course, this should not be merged before the unit tests pass too.


Change the severity if length of INSERT exceeds query_max_size

Overview
We can change the severity, if the length of INSERT statements generated by get_insert_rows_queries exceeds query_max_size. We can also refactor the macro to handle the limitation of the length an INSERT statement.
First, the current implementation might have a pitfall if the length of new_insert_query after adding row_sql exceeds query_max_size due to the large row_sql. We may be able to avoid the issue by increasing the value of query_max_size. However, there are some warehouses which have the limitations on the length of an INSERT statement as BigQuery.  Those can be the length as string or the number of elements in the VALUES clause.
Second, if the length of an INSERT statement is large, a warehouse might raise an issue. So, it might be ok for elementary to show warning messages which can be clues to inspect what happens. I think elementary just focus on how to chunk values to an INSERT statement.
https://github.com/elementary-data/dbt-data-reliability/blob/ea690c42601f340ea8d927afff0e2d82d688e6b1/macros/utils/table_operations/insert_rows.sql#L55-L87

@elongl I'm sorry for bothering you, but I would like to get your thought on it. The issue frequently blocks us to insert dbt results to the elementary's table.

Hi @yu-iskw, thanks a lot for raising this issue.
May I ask, why don't you raise the query_max_size if that's the case?
Also, do you know which table is being inserted into when this error raises?
I'm curious as to where there's so much data on a single row.
Regardless, this does make sense to me so I'll discuss this with the team and update shortly.

@elongl Good question. As we set the smaller value to query_max_size due to the quota of BigQuery, it can be failed to insert a large number of rows with a larger query_max_size.
https://github.com/elementary-data/dbt-data-reliability/blob/ddb5b41332ec151d56e9f554e9b00fdf3b583198/macros/edr/system/system_utils/get_config_var.sql#L75-L79


Better support for Hadoop native libs and compression codecs in Java Dataflow batch worker

Current(as of Beam SDK 2.24.0) Java Dataflow workers does not ship with Hadoop native library and popular native compression codecs(e.g libsnappy) .
If one try to read files using IO classes that assume native library support(e.g, org.apache.hadoop.hbase.mapreduce.TableSnapshotInputFormat) on GCP Dataflow, the following error will be throw if the file is compressed by snappy:

java.io.IOException: Failed to start reading from source: org.apache.beam.sdk.io.hadoop.format.HadoopFormatIO$HadoopInputFor

matBoundedSource@311fa603

...

Caused by: org.apache.hadoop.hbase.DoNotRetryIOException: java.lang.RuntimeException:
native snappy library not available: this version of libhadoop was built without snappy support.

There is very little one can do from within pipeline job code unless rewriting the IO class, so it might make sense to include these native libraries for Hadoop IOs
Imported from Jira BEAM-11231. Original Jira may contain additional context.
Reported by: lobatt.

Portability now allows custom containers for this.


Number of deletions and insertions required to convert one sstring into the other

💥 Proposal
I am a participant of GSSOC'21. I would like to add the code of number of deletions and insertions required to convert one string to another in C++
Have you read the Contributing Guidelines on Pull Requests?
Yes, I have read the guidelines
Assignees

@SAY-droid427 : C++
@simarjeet30 : C

I am a participant of GSSOC'21. I would like to add the code of number of deletions and insertions required to convert one string to another in C++ . I know C , C++ , python very well. Also have knowledge of DSA.

I would like to work on this issue as a part of GSSOC'21.

@SAY-droid427 Go ahead with C++

@simarjeet30 Go ahead with C

I would like to work on this issue as a part of GSSOC'21.

Please comment your desired language.

I would like to implement the above question in Python, I will write neat and clean code
please assign me this issue under GSSoC 2021

I created a Pull Request but it said that it has been closed. So should I merge it again?

@SavneetKaur01 The PR that had been closed so far have some validate reasons., Better to create a new legitimate PR.

sir i have done the coding part ,and its my first time so can u just review it once before i make a pull request.
…
On Tue, Mar 9, 2021 at 6:12 PM SavneetKaur01 @.***> wrote: I created a Pull Request but it said that it has been closed. So should I merge it again? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#2287 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASZVY2CVVSMHPEWBAMDBUDDTCYCQXANCNFSM4YZV3BZQ .

@simarjeet30  You need to create a PR, afterwards we can review your PR

@siddharth25pandey ,sir can i work on this issue as it is assigned to me .

Hello, I am a participant of GSSoC'21 and would like to work on this issue using Java.


Create ebextension for satisfying OpenEMR dependencies

We should use the linux tarball here: http://open-emr.org/wiki/index.php/OpenEMR_5.0.0_Linux_Installation
It is important to ensure the following values are in place: http://www.open-emr.org/wiki/index.php/FAQ#What_are_the_correct_PHP_settings_.28can_be_found_in_the_php.ini_file.29.3F
Use the following as inspiration: http://open-emr.org/wiki/index.php/OpenEMR_System_Architecture#OpenEMR_Dependencies

need to write this script: https://github.com/GoTeamEpsilon/OpenEMR-AWS-Guide#configure-openemr-servers-to-satify-system-dependencies-on-startup
Who wants to help me?

Ths work is covered in our new issue/task tracker at https://app.asana.com/0/346554951567668/346554951567703