navigation.dispatch not working for web

Dear Friends,
I have a react-native web and recently i`m start migrate to web application (with react-navigation for web) but i have a problem navigation.dispatch not working just in web in react-native working as well
in Luanch page in some case i will be reset AuthNavigator for navigat to Join page
NavigationService.reset(['AuthNavigator'])
here is data of resetAction:

Current Behavior
index.js
// @flow

import React, { Component } from 'react'
import { View } from 'react-native'
import { createSwitchNavigator } from '@react-navigation/core'
import { createBrowserApp } from '@react-navigation/web'
// import Launch from 'src/components/init/launch/Launch'
import AuthNavigator from 'src/navigators/AuthNavigator'
// import MainTabNavigator from 'src/navigators/MainTabNavigator'

const MainNavigator = createSwitchNavigator(
  {
    Launch: { screen: Launch },
    AuthNavigator: { screen: AuthNavigator },
    // MainTabNavigator: { screen: MainTabNavigator },

  },
  {
    initialRouteName: 'Launch',
  }
)

type Props = {
  lastSupportedVersion: string,
  lastStableVersion: string,
  needForceUpdate: string,
  needOptionalUpdate: boolean
}

const AppContainer = createBrowserApp(MainNavigator)

class MainNavigatorWithMeta extends Component<Props, void> {
  _onBackPress: Function
  _getActiveRouteName: Function

  constructor(props) {
    super(props)
    this._getActiveRouteName = this._getActiveRouteName.bind(this)
    this._onBackPress = this._onBackPress.bind(this)
  }
  componentDidMount() {
    // BackHandler.addEventListener('hardwareBackPress', this._onBackPress)
  }

  componentWillUnmount() {
    // BackHandler.removeEventListener('hardwareBackPress', this._onBackPress)
  }

  _onBackPress() {
    // analytics().logEvent('back_hardware')
  }
  // gets the current screen from navigation state
  _getActiveRouteName(navigationState) {
    if (!navigationState) {
      return null
    }
    const route = navigationState.routes[navigationState.index]
    // dive into nested navigators
    if (route.routes) {
      return this._getActiveRouteName(route)
    }
    return route.routeName
  }

  render() {
    return (
        <AppContainer
          screenProps={{ ...this.props }}
          ref={(navigatorRef) => {
            NavigationService.setTopLevelNavigator(navigatorRef)
          }}
        />
    )
  }
}

export default Metadata.providers.metadata(MainNavigatorWithMeta)

NavigationService.js
// @flow

import { NavigationActions, StackActions } from '@react-navigation/core'

let _navigator

function setTopLevelNavigator(navigatorRef: any): any {
  _navigator = navigatorRef
}

function navigate(routeName: string, params: any): void {
  const navigateAction = NavigationActions.navigate({
    routeName,
    params
  })
  _navigator._navigation.dispatch(navigateAction)
}

function reset(routeNames: Array<string>): void {
  let actions = null
  for (let i = routeNames.length - 1; i >= 0; i--) {
    let params: any = { routeName: routeNames[i] }
    if (i !== routeNames.length - 1) {
      params.action = actions
    }
    actions = NavigationActions.navigate(params)
  }
  const resetAction = StackActions.reset({
    index: 1,
    key: null,
    actions: [actions]
  })
  console.log('resetAction', resetAction)
  _navigator._navigation.dispatch(resetAction)
}

export default {
  navigate,
  reset,
  setTopLevelNavigator
}

AuthNavigator
// @flow

import { createSwitchNavigator } from '@react-navigation/core'
import Join from 'src/components/init/auth/Join'
import Login from 'src/components/init/auth/Login'
import { COLOR_WHITE } from 'src/shared/styles/colors'

const AuthNavigator = createSwitchNavigator(
  {
    Join: { screen: Join },
    Login: { screen: Login }
  },
  {
    initialRouteName: 'Join',
    headerMode: 'none',
    navigationOptions: {
      gesturesEnabled: false
    },
    cardStyle: { backgroundColor: COLOR_WHITE }
  }
)

export default AuthNavigator

Expected Behavior
navigate to Join page same as react-native
Your Environment

software
version

react-navigation core
"3.2.0"

react-navigation web
"1.0.0-alpha.7"

react-native-web
"0.11.2"

node
8

npm or yarn
yarn

hi, what does it mean "not working"?
There has been a recent PR, which fixed an issue with the dispatch method being prefixed with a _. As you are using a ref on the navigation container like in the issue, it's probably the same error if you got "dispatch is not a function" or some error like that
https://github.com/react-navigation/react-navigation-web/pull/19

It seems it has not yet been released however (cc @brentvatne @ericvicenti) so you can try calling _dispatch

@slorber
not working mins not happen when i dispatch StackActions.reset noting happen
I`m already try _dispatch but not different in fact and after check test speck released the correct one is "ref._navigation.dispatch"
and its working as well on when i dispatch navigateAction

Hey guys,
@brentvatne @ericvicenti
i Have few issue on this repo when try for working on web can you take a look
actualy navigation goBack and this issue and some more little thinks not working if you need more information or a sample i can make it take hands on this issue
appreciate that

@soroushm - please create a new issue and provide a runnable example that we can use on https://snack.expo.io, thanks!

tips on how to create a good reproducible example: https://stackoverflow.com/help/mcve


Add Static Web Hosting functionality

I have added the ability to use s3rver to simulate the Static Web Hosting functionality of S3.
If you are happy with it, please pull and create a new release.
thanks,
David

Merged. I didn't know about this feature of S3. Thanks for contributing! I'll create a release.

Created new release 0.0.3. Thanks!

Thats great!
Thanks


buildah: don't modify Dockerfile in place

STONEBLD-2795
Instead, make a copy outside the repo and modify the copy. This is to
avoid dirtying the repo.
For example, when combined with the push-dockerfile task, the pipeline
will now push the original dockerfile instead of pushing the modified
one.

/retest

/test

/retest

/test

/test

/test

/retest

/test

Without the fix, the e2e-tests fail as expected (though with a message not exactly readable to humans)

Now let's re-apply the fix and see :crossed_fingers:

/test

It's working :tada:

/test

/retest


Publish checks for job and use task listener

Publish checks fro job directly instead of run and item
Add task listener for console log

it'll need a PR to warnings-ng as well I think?

maybe it will be better to remove the abstract from job one since in most cases it will not be needed, and no need to change warnings as well.


Button Text Color Adaption (DEV)

This PR adapts the text color of a button in light mode when it is disabled from white to gray for better visibility.

Tested it on S8 , Android 9 lgtm


Mean shift movers query

Coverage remained the same at 94.034% when pulling bf201fbf0d660ad7ca5331ffa2c997db6d6e4c85 on MeanShiftMoversQuery into f8706b1c83d291d3fdfe94c2b5b961620e383ba2 on master.


Add RegExp support to ChangeTagValue recipe

What's changed?
Allows ChangeTagValue to take in regex in oldValue parameter.
What's your motivation?

https://github.com/openrewrite/rewrite/issues/4731

Anything in particular you'd like reviewers to focus on?
Please leave your thoughts here or on the issue https://github.com/openrewrite/rewrite/issues/4731.
Anyone you would like to review specifically?
@timtebeek? - Because I see you actively replying ;)
@gadams00 - Because, history.
Have you considered any alternatives or workarounds?
Not really. I found this to be more appealing.
Checklist

[x] I've added unit tests to cover both positive and negative cases
[x] I've read and applied the recipe conventions and best practices
[x] I've used the IntelliJ IDEA auto-formatter on affected files

@timtebeek / @gadams00 Can you kindly help with this change? Thanks!

@timtebeek / @gadams00 Any thoughts on this PR?

Thanks for the work here so far @anthochristen ! Didn't get to a review earlier beyond the automated suggestions, thanks for applying those already! From https://github.com/openrewrite/rewrite/issues/4731 I understand you want to use this from a declarative recipe to update a tag at a specific XPath ? Seems fair enough.
I see you've taken care to add backwards compatible constructors; that's appreciated but perhaps not necessary. Typically we roll forward by running this recipe against the OpenRewrite org

https://app.moderne.io/recipes/org.openrewrite.java.AddNullMethodArgument
https://app.moderne.io/devcenter/OpenRewrite

The method pattern in this case would be org.openrewrite.xml.ChangeTagValue <constructor>(..) once this PR is merged.
Feel free to remove those backwards compatible constructors and then we can merge this in.

Thanks for the work here so far @anthochristen ! Didn't get to a review earlier beyond the automated suggestions, thanks for applying those already! From #4731 I understand you want to use this from a declarative recipe to update a tag at a specific XPath ? Seems fair enough.
I see you've taken care to add backwards compatible constructors; that's appreciated but perhaps not necessary. Typically we roll forward by running this recipe against the OpenRewrite org

Thanks @timtebeek for reviewing the change. Yes, thats the purpose of issuing this PR. To do find and replace for a specific X-Path, using the existing declarative ChangeTagValue recipe.
About removing the old constructors, was un-sure if that'd amount to semver issues. So refactored my regex changes as a separate visitor without affecting the old one.
Hope that works? If not tell me know I can revert this and remove the constructors, as suggested, & update the usages.


Add warning for code generation

This resolves #673. The first commit adds it to the template and the second commit are the generated changes. Please let me know if the comment wasn't final and should be changed to something else. :)

Thanks for adding that! There are a couple other files that get auto-generated that would be useful to add the header to as well: lib.rs, Cargo.toml and README.md. lib.rs should be easy enough (same as generated.rs). From some brief research, comments can be added to toml files with a # prefix and can be put in Markdown inside a <!--- ... --->.

Sorry for the delay, I've moved the comment, rebased and pushed the new commits :)

Hey @kpcyrd, thanks for following up!
It looks like the header got removed from generated.rs when it got added to lib.rs. Since both files are generated, it should be on both files. 😄

What can I do to help finish this up? 😄

Hey @kpcyrd - you still interested in wrapping this up?  If not, I think someone else will likely grab it and finish it.  It's definitely something we want to get merged.

@adimarco sorry, I was running low on time. I figured out how to add the comment to generated.rs (I missed that file before the rebase as well).
I squashed and rebased on the latest master to resolve the merge conflict. :)

Looks good, and the generated docs look right too.  Thanks!


Improved default key for page with children

I think this as a default key would be better if a page being loaded from the layout has children. Ideally I'd like to find the page's parent fullpath but I haven't found a good way to do that just yet. This would satisfy my needs on a basic website.
P.s. with these huge releases I know this is not top priority and isn't important enough to take attention away from all the great work going on!

Codecov Report

Merging #1209 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #1209   +/-   ##
=======================================
  Coverage   98.39%   98.39%
=======================================
  Files          12       12
  Lines         685      685
=======================================
  Hits          674      674
  Misses         11       11

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 8563c0f...23ebb41. Read the comment docs.

Hey @silverbackdan
Could you make a reproduction about it with Glitch (by forking https://glitch.com/edit/#!/nuxt-hello-world).
I'm not sure this is the best way by splitting with /, we should instead check the matched components and remove the children path (something like this for example: https://github.com/nuxt/nuxt.js/blob/dev/lib/app/client.js#L255-L256)

Hiya @Atinux Sure I'll be happy to make something up to show what I mean. I'll try and have the replication done later today. I think you're right that it won't be the best way to go the more I think about it. That's a really good example too, thanks. I'll see about replicating that behaviour.

I've created a reproduction here:
https://glitch.com/edit/#!/nuxt-child-key
https://nuxt-child-key.glitch.me
Switching between 'definition 1' and 'definition 2' pages
And a quick demonstration for why one may want that structure.

Hey @silverbackdan
I believe this should be the right way: https://glitch.com/edit/#!/nuxt-pull-1209
Demo: https://nuxt-pull-1209.glitch.me/

That's really good and would work for me. As this is from the 'nuxt' tag too there should never be more than 1 child (otherwise should be using nuxt-child) Updating the PR now.

Awesome :)


Update README.rdoc

Added info for non-English localization use.

@cinic Thanks!

@amatsuda u'r welcome :wink:


Pre commit

Adds pre-commit hooks without the flake8 part. See #55 for the discussion.
Feel free to close this if pre-commits are not wanted. They can cause some annoyance in some workflows.

:rofl: me everytime you spell check my comments

me everytime you spell check my comments

You know I cannot really do that without codespell, right? :stuck_out_tongue_winking_eye:


Run action recognition model on mobile

According to your paper, you run the action recognition model with tflite, if I want to perform action recognition on mobile(android, like mate30) too, which model you would recommend(Smallest TSN model from the model zoo)?

Thanks

Hi, this is not our paper.
This framework is based on Pytorch, not tensorflow as in the paper. We haven't tried running models on mobile devices. You may look at TVM for quantization and model deployment. Or you can consult the authors of the paper.


CSV Download of Software Development Lifecycle (SDL) events.

In this document, there is a table.
This issue focuses on this row (note that the copy/paste below may be out of date, consult live document for updates):

--
--
--
--

GitHub (Events)
Software Development Lifecycle (SDL) Events
L
Total number of GitHub events involving the creation and editing of milestones, issues, pull requests and the project (Kanban) board.

In this issue, we want to explore which of these events we can get, and ultimately implement a CSV download for "Software Life Cycle Events".
This is an exploratory issue: we'll need to define specifically which events we can get and what fields we want for each event, and add detail to the acceptance criteria as we do.
We can document the GraphQL queries needed in comments on this issue until we are ready to code them.
TODO

[ ] We have GraphQL queries for each SDLC, and a list of the fields we want to store in the database for that event.
[ ] We have a job to perform the queries and update db
[ ] We have logic to download the CSV of these queries

GraphQL Objects

AddedToProjectEvent This is an event when an issue or PR is placed on a Project (Kanban board).

A GraphQL Query to get "moved on kanban board" and "created on Kanban board" events:
Note that the details of these moves are only in the API preview, and can't be gotten in the explorer.
{
  repository(owner: "ucsb-cs48-s20", name: "project-s0-t1-budget") {
    issues(last: 100) {
      nodes {
        number
        timelineItems(itemTypes: [ADDED_TO_PROJECT_EVENT, MOVED_COLUMNS_IN_PROJECT_EVENT], first: 100) {
          nodes {
            __typename
            ... on AddedToProjectEvent {
              id
              actor {
                login
              }
            }
            ... on MovedColumnsInProjectEvent {
              id
              actor {
                login
              }
              createdAt
            }
          }
        }
      }
    }
  }
}


Traincarts tickets.yml playerBound property

BkCommonLib version: 1.12.2-v5
TrainCarts version: 1.12.2-v3
Spigot version: 1.12.2-R0.1-SNAPSHOT

Problem or bug: I have created a ticket in tickets.yml for TrainCarts and I set playerBound property to "true"
Expected behaviour: I expected that only players who have been gifted the ticket to are able to use the ticket. Regardless of what is the value (true or false), everyone is able to use the ticket of someone else.
Steps to reproduce: Type /train ticket give
and drop the ticket to another player who will be able to enter a train with it.

You were right, and I've fixed it. Thanks!
https://github.com/bergerhealer/TrainCarts/commit/9cc56f5ab8bd53e2ea8bc2192ad2ef976e583805

Thanks a lot ! :-)

The version linked above works on 1.12.2

Hello, I tried Traincarts latest version on 1.12.2 but it needs latest
version of BkCommonLib which doesn't load on my 1.12.2 Spigot.
Cordially.
Le mer. 31 juil. 2019 à 01:41, bbayu123<EMAIL_ADDRESS>a écrit :

The version linked above works on 1.12.2
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bergerhealer/TrainCarts/issues/279?email_source=notifications&email_token=AJQUKZIP4QHR5POH7XHL653QCDGRDA5CNFSM4IH5B4TKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3FT7NQ#issuecomment-516636598,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJQUKZNFE2GHLJU6T7CAN2TQCDGRDANCNFSM4IH5B4TA
.

It loads on my 1.12.2 spigot server just fine, what error are you having?

Also use the BKCommonLib version I linked in my answer above ( https://ci.mg-dev.eu/job/BKCommonLib/593/ )

I have this error: https://paste.md-5.net/wilideyami.css
I use Traincarts #487 and BkCommonLib latest version 1.13.2-v3
Cordially,
Mr_Kittle.
Le mer. 31 juil. 2019 à 08:04, bbayu123<EMAIL_ADDRESS>a écrit :

It loads on my 1.12.2 spigot server just fine, what error are you having?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bergerhealer/TrainCarts/issues/279?email_source=notifications&email_token=AJQUKZOF2QBJN6Y5HZEKJ53QCETO3A5CNFSM4IH5B4TKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3GFYPQ#issuecomment-516709438,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJQUKZNGFGIB7QS7EPOU6KLQCETO3ANCNFSM4IH5B4TA
.

you need to update BOTH BKCommonLib AND TrainCarts

Thank you very much, it works now :-)
Have a good day.
Mr_Kittle.


Update MN URIs that are broken

Closes #4715
Closes #4714
Closes #4713
Closes #4712
Closes #4711
Closes #4710
Closes #4709
Closes #4708
Closes #4707
Closes #4706
City instead, CO didn't have address info #4720

#4706 & #4707

Whew, this is great @Garrett-W! Thanks for finding all these. Are you ready for this to be merged?

@Garrett-W Just wanted to chime in and also express how much we appreciate PRs like this!!

Whew, this is great @Garrett-W! Thanks for finding all these. Are you ready for this to be merged?

It's as ready as I'm going to get it. Their's a couple of format cleanups, but otherwise the data looks pretty valid.

Great, thanks!


scroll issue with two modals

when open a new modal above the first one and close the second modal the first modal loses its ability to scroll

ran into the same issue
@reactor123 found a solution at https://stackoverflow.com/questions/28077066/bootstrap-modal-issue-scrolling-gets-disabled
this could be integrated somehow in this library i think

seems like this lib already takes care of the body class
I found if I comment this line: https://github.com/Innologica/vue-stackable-modal/blob/master/src/components/StackModal.vue#L144
everything works as intended. but I am not sure why it was there in the first place

it is already integrated in the extension. I guess it's some bug that it is not working. I will test it with an empty project because in m projects this doesnt happen.


Make SSL impl (i.e. boringssl vs openssl) pluggable

Title: Make SSL impl (i.e. boringssl vs openssl) pluggable
Description:
Overall goal is to support the existing boringssl implementation as well as other ssl implmentations such as openssl. In order to support alternate ssl libraries, boringssl must be able to be compiled out with only bazel configuration changes. In order to support compiling out boringssl the following steps need to be accomplished:

decouple common/ssl from common/network (completed)
move boringssl dependencies to extensions.
decouple extensions/transport_sockets/ssl from server

Parallel implementation of extensions/transport_sockets/openssl is underway.
Relevant Links

https://github.com/envoyproxy/envoy/pull/5403
https://github.com/envoyproxy/envoy/pull/5543
WIP

I'm interested in this issue and would like to help with it.
So far I have prepared a couple of patches to Envoy bringing in OpenSSL support (https://github.com/rojkov/envoy/tree/openssl).
Basically

I moved the code of the TLS transport socket and tls_inspector to the boringssl subfolders of the respective extensions;
copied OpenSSL implementations of the extensions from https://github.com/bdecoste/envoy-openssl to the openssl subfolders with minor changes;
copied the boringssl compatibility layer from https://github.com/bdecoste/openssl-cbs and https://github.com/bdecoste/bssl_wrapper to source/extensions/common/boringssl_compat (with minor changes);
added a bazel option to build Envoy either with boringssl (default) or openssl.

So, in this experimental branch the extensions //source/extensions/transport_sockets/tls and //source/extensions/filters/listener/tls_inspector are just very short headers conditionally including either boringssl or openssl "plugins". It causes huge code duplicity though.
Currently all the unit tests pass when Envoy is built against boringssl. I started to fix the tests for the openssl builds (by now fixed the tls_inspector tests).
@bdecoste @lizan @PiotrSikora  Could you please add some comments on what is the current plan? Does my work make any sense? How can I help if it doesn't?
cc @ipuustin @mythi

The plan of record is for Envoy to officially support only BoringSSL, and OpenSSL being available as a third-party extension (which is what @bdecoste is working on). This allows supporting features that are implementation specific, e.g. BoringSSL's CRYPTO_BUFFERS, SSL_PRIVATE_KEY_METHOD, OpenSSL's engines, and not tying TLS transport to the common interfaces.

Ok, somehow I missed the point about availability of OpenSSL support as a pluggable third-party extension. I presume the easiest way to do that is to follow the custom envoy filter example and to have a similar repository importing Envoy as a git submodule. WDYT?


AjaxGrid Paging

Hi, thank you for the great work.
I'm trying to get AjaxGrid with paging to work without having to return the entire data set on each request.
Here is the partial called _ListGrid
@model IEnumerable<DoIT.Profiler.Core.Entities.Client>

@(Html.Grid(Model)
    .Named("ClientsGrid")
    .Empty("No clients found")
    .Build(columns =>
    {
        columns.Add(p => p.Code).Titled("Code");
        columns.Add(p => p.Name).Titled("Name");
        columns.Add(p => p.DateCreated).Titled("Date Created").Formatted("{0:dd/MM/yyyy}");
        columns.Add(model => $"Edit").Encoded(false);
    })
    .Filterable()
    .Sortable()
    .Pageable(pager =>
    {
        pager.PagesToDisplay = 10;
        pager.RowsPerPage = 20;
    })
)

Here is the method that is called by AjaxGrid
        public async Task<ActionResult> ListGrid(string search = "")
        {
            var page = Request.Query["ClientsGrid-Page"].ToString().ToNullable<int>();
            var rows = Request.Query["ClientsGrid-Rows"].ToString().ToNullable<int>();

            IEnumerable<Client> clients = null;
            if (page == null || rows == null)
                clients = await _clientService.SearchClients(search);
            else
                clients = await _clientService.SearchClients(search, page ?? 0, rows ?? 0);
            return PartialView("_ListGrid", clients.OrderBy(x => x.DateCreated));
        }

If I call clients = await _clientService.SearchClients(search); every time it will work but that takes ~20 seconds due to the number of records. As it's coded above the grid loads first time correctly with the paging set-up correctly, showing many pages. Then once I click to go to the next page (and the smaller call is made for just the 20 records of a single page), it reloads the grid and the paging just shows one page.
Is it possible for the grid to remember the total number of records from the first call, or even better for me to pass a simple count of the total number of records on each request to keep the paging working correctly? I've tried the latter manually by passing counts back but the 'TotalRows' and 'TotalPages' properties on the pager config are both read only.
Thanks again!
Gareth

Grid already takes care of paging for you, unless you are doing manual paging on raw sql queries?
Otherwise you will have to implement or extend the GridPager. The easiest way would be to create your own extension, similar to this, which would set your own pager implementation.
I was thinking about making TotalRows and TotalPages mutable, but I couldn't find a proper use case.
I guess I should try implementing user controlled paging and see if their mutability can be used in some way.

From what I can see it makes a call each time the grid is loaded and requires the entire collection of models to be returned to the partial each time, is that right?
        public virtual IQueryable<T> Process(IQueryable<T> items)
        {
            TotalRows = items.Count();

            return items.Skip((CurrentPage - 1) * RowsPerPage).Take(RowsPerPage);
        }

As I say, if I return the entire collection of models every time it's ok but when it's a huge collection that's very inefficient. I'll take a look at making a GridPager of my own.
It looks as though it would be the Process method I'd have to rewrite to take the smaller collection of models but to set total rows for the entire collection count. I'll have a think..

Yes you have to write your own processing, which in this case should just be return items as they are already paged by your controller. And total rows are actually mutable from grid pager implementation.
So the view code could look like this
@(Html
    .Grid(Model)
    .Build(columns =>
    {
        columns.Add(p => p.Code);
    })
    .YourPageableExtension(pager =>
    {
        pager.TotalRows = <your precalculated total rows>;
        pager.PagesToDisplay = 10;
        pager.RowsPerPage = 20;
    })
)

Gotcha, thanks!

In HtmlGrid you have
public virtual IHtmlGrid<T> Pageable(Action<IGridPager<T>> builder)
{
    Grid.Pager = Grid.Pager ?? new GridPager<T>(Grid);
    ....

How would you set Grid.Pager, as far as I can see it would always be null at this point?

You would change new GridPager<T>(Grid) with your own implementation and use Action<GridPager<T>> or Action<YourGridPager<T>> so that total rows are available.

I was hoping there was a way for me to extend it without changing the core package. Also, wouldn't I also have to change the interface IGridPager to make TotalRows mutable?

No grid pager interface is immutable, but it's implementation adds a setter for internal use.


inc-nl no longer violates approval

I am not seeing approval violations with inc-nl since the QoS value was fixed. What's odd is that the fix should not have impacted inc-nl. This could be run-to-run variance. Need to investigate.

OK. So after playing with this a bit, it seems that WA is currently being marked HIPRI (AF21) but AA is not being marked anything (when we disable rate limiting). Why?

Saw this error on a AA host but not WA
T=27364 [2021-10-01 20:56:25.599] [main] [error] [host-agent.cc:149] failed to init simulated WAN: UNKNOWN: failed to create htb classes or qdiscs: tc -batch: wait status: 256 exit status: 1; stderr:
distribution specified but no latency and jitter values
Usage: ... netem	[ limit PACKETS ]
			[ delay TIME [ JITTER [CORRELATION]]]
			[ distribution {uniform|normal|pareto|paretonormal} ]
			[ corrupt PERCENT [CORRELATION]]
			[ duplicate PERCENT [CORRELATION]]
			[ loss random PERCENT [CORRELATION]]
			[ loss state P13 [P31 [P32 [P23 P14]]]
			[ loss gemodel PERCENT [R [1-H [1-K]]]
			[ ecn ]
			[ reorder PERCENT [CORRELATION] [ gap DISTANCE ]]
			[ rate RATE [PACKETOVERHEAD] [CELLSIZE] [CELLOVERHEAD]]
			[ slot MIN_DELAY [MAX_DELAY] [packets MAX_PACKETS] [bytes MAX_BYTES]]
		[ slot distribution {uniform|normal|pareto|paretonormal|custom} DELAY JITTER [packets MAX_PACKETS] [bytes MAX_BYTES]]
Command failed -:3
distribution specified but no latency and jitter values
Usage: ... netem	[ limit PACKETS ]
			[ delay TIME [ JITTER [CORRELATION]]]
			[ distribution {uniform|normal|pareto|paretonormal} ]
			[ corrupt PERCENT [CORRELATION]]
			[ duplicate PERCENT [CORRELATION]]
			[ loss random PERCENT [CORRELATION]]
			[ loss state P13 [P31 [P32 [P23 P14]]]
			[ loss gemodel PERCENT [R [1-H [1-K]]]
			[ ecn ]
			[ reorder PERCENT [CORRELATION] [ gap DISTANCE ]]
			[ rate RATE [PACKETOVERHEAD] [CELLSIZE] [CELLOVERHEAD]]
			[ slot MIN_DELAY [MAX_DELAY] [packets MAX_PACKETS] [bytes MAX_BYTES]]
		[ slot distribution {uniform|normal|pareto|paretonormal|custom} DELAY JITTER [packets MAX_PACKETS] [bytes MAX_BYTES]]
Command failed -:8


fix(cloud_firestore, auth, core, messaging, remote_config, storage): Add the missing include header that defines TARGET_OS_OSX

Description
Add the missing include header that defines TARGET_OS_OSX.
Related Issues
None.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (melos run analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change.
[x] No, this is not a breaking change.

@ChristianEdwardPadilla  I don't think this is necessary as I believe the target conditionals are already pulled in from the firebase-ios-sdk.


refactor: un-export MessageT's attributes

In MapT, it's important to assign new event times to ensure that default time values are not mistakenly passed to the next vertex. While our MapT SDKs offer methods like MessageTToAll() and MessageTTo() to create MessageT objects with explicit event times, users can also construct MessageT objects from scratch. However, if they forget to assign a new event time while modifying other attributes, default time values may be used unintentionally.
To reduce this risk, we propose always requiring an explicit event assignment for MapT implementation. This code change achieves the proposal by un-exporting MessageT's attributes.
After this change, user can only use the following constructors to create MessageT objects:
functionsdk.MessageTToDrop()

functionsdk.MessageTTo(eventTime, key, value)

functionsdk.MessageTToAll(eventTime, value)

Following approaches to create MessageT objects are no longer allowed.
msgt := MessageT{EventTime: event_time, Key: key, Value: value}

msgt := MessageT{}
msgt.EventTime = event_time
...

Also applied similar changes to Message

@whynowy @vigith I am thinking about making similar changes to Message but I am not sure if we have any existing Numaflow users that are already using the Message{} approaches to instantiate Message object. Could you guys help confirm?
If we have, we might ask users to make corresponding changes to use MessageTo* before un-exporting.

@whynowy @vigith I am thinking about making similar changes to Message but I am not sure if we have any existing Numaflow users that are already using the Message{} approaches to instantiate Message object. Could you guys help confirm?
If we have, we might ask users to make corresponding changes to use MessageTo* before un-exporting.

Go with it. Ppl will not able to compile with the new change if they do, then they are gonna realize it.

how will the user test the output from handles if it's private? we should think from the unit testing perspective as well

how will the user test the output from handles if it's private? we should think from the unit testing perspective as well

For unit test, user will need to use our public constructors to prepare expected output. An example: https://github.com/numaproj/numaflow-go/blob/main/pkg/function/examples/event_time_filter/impl/filter_test.go#L51


Question (and maybe feature request) ability to use the .mylogin.conf file for authentication

I asked this question on stackoverflow and after submitting it I realized that this would probably be the best place to ask the question:
I'm using the mysql package to interact with mysql in a content deployment script.
At the moment I'm creating the connection via the createConnection method like so:
    createMysqlConnection () {
        this._db = mysql.createConnection({
            host: config.production.main.mysql.host,
            user: config.production.main.mysql.automator.user,
            password: config.production.main.mysql.automator.password
        })
        this._db.connect()
    }

This is working fine.
I know that normally from the command line I can set my login credentials into an encrypted .mylogin.cnf file by using the mysql_config_editor tool that was added in 5.6. I'd like to use this same config file when logging in with the node package so I don't have to store the credentials in an unencrypted config file.
I read through the docs for the package and I'm not seeing any information on how I could tell the package to use the .mylogin.cnf file or to tell it what login-path to use within the file.
Is this possible and, if so, how would I configure the package to do so?

That is not a feature of this module today, but if you wanted to write an implementation and make a pull request adding it, I'd definitely be open to that.

Sadly the StackOverflow question has been removed.  I was about to ask it there as well, too.


Label test structs with "wanted"

So that the test failure messages are a little more obvious.
Based on discussion from #3506.

I also renamed most of the "[cpfk]data" variables to test.  You now see failures like this:
    # --- wanted
    # +++ tmp + out_misalign
    # 0000:-3cf55d656e9c0664 513359
    # 0000:+3cf55d656e9c0664 513358
    #                            ^^

   # ERROR: (memory) 'test->tag == received_tag' failed @ test/evp_test.c:701
    # --- test->tag
    # +++ received_tag
    # 0000:-f0723bce23528ce6 ccb10ff9627038bf
    # 0000:+e0723bce23528ce6 ccb10ff9627038bf
    #       ^^

And perhaps push???

    # ERROR: (memory) 'test->wanted == md' failed @ test/evp_test.c:395
    # --- test->wanted
    # +++ md
    # 0000:-29fa61d75522a466 9b44e39c1d2e1726 c530232130d407f8 9afee0964997f7a7
    # 0000:+19fa61d75522a466 9b44e39c1d2e1726 c530232130d407f8 9afee0964997f7a7
    #       ^^
    # 0020: 3e83be698b288feb cf88e3e03c4f0757 ea8964e59b63d937 08b138cc42a66eb3

    # INFO:  @ test/evp_test.c:2146
    # Above error from the test at test/evpdigest.txt:266 unexpected error DIGEST_MISMATCH
    # ERROR: (memory) 'test->input == decode_out' failed @ test/evp_test.c:1587
    # --- test->input
    # +++ decode_out
    # 0000:-4f70656e53534c4f 70656e53534c0a
    # 0000:+6b70656e53534c4f 70656e53534c0a
    #       ^^

    # INFO:  @ test/evp_test.c:2146
    # Above error from the test at test/evpencod.txt:190 unexpected error BAD_DECODING

Additional commit (to be squashed if I remember) pushed.

ERROR: (memory) 'test->tag == received_tag'

I don't see that test->something is better. But wanted->something would be... Which means that instead of renaming fields one can rename variables...

Sorry about closing/re-opening. Wrong button.

I used output_wanted

updated commit pushed that renames things.

Instead of pointing out imperfections in my view I've chosen to simply create alternative #3550 and let you point out imperfections. [I'm not fond of 'wanted' as it sounds too "wild-west".]

i assume you also didn't like the "evp-most-wanted" branch name :)


MINIMOSD throttle still 0%, flight mode doens't change

Minimosd was connected to TELEM1 or TELEM4/5 because TELEM2 is used by RASPBERRY. Once starts PX4, MINIMOSD flight mode and throttle doenst change. Sometime nobody is displayed on this items.
extras.txt
mavlink start -d /dev/ttyS5 -b 57600 -f

Changed to https://github.com/night-ghost/minimosd-extra 926-MAVLINK-PX4 firmware and solved.


Traefik router should become default router in v1.22

Expected Behavior
In DDEV v1.22, traefik should become the default router.

[ ] Make it the default
[ ] Move it out of the experimental section of ddev docs
[ ] Explain how to use the traditional router
[ ] Perhaps explain about casual hosting not supported directly by traefik. It's already explained, but will likely have to be reworded.
[ ] We probably have to change (and migrate) the configuration of use_traefik: true and instead use something like router_type: traefik or router_type: traditional

This one is a lot easier than the other one. It's probably a one-liner.

why did a breaking change get a minor release number? :|

Well @allanlaal that stings.

If the change to Traefik broke you, please open an issue explaining how so it can be addressed.
Traefik was available as an option for 8 months and was widely tested.
If you'd like to use the legacy router, ddev config global --router=nginx-proxy
v1.22.0 is a major release, with the changes widely announced.

sorry for expecting everyone is using Semantic Versioning :)
..you should though :D


Does not work with App Sandbox

The crash files generated by macOS are stored outside the App Sandbox, and therefore not accessible when the app operates within an App Sandbox.

Yeah :( I think the next best thing we have is to intercept the crash and collect and save our own reports: https://github.com/ChimeHQ/Impact


ffmpeg not reconnect when youtube URL is expired

Hi, so i have been using your package for build my discord bot. Its great thank you.
I found this problem when i playing around with my bot. When playing music (from youtube using ytdl package) its stop in the middle of the song. The error said it's EOF. But the song is not finished yet.
When i print the encodingSession.FFMPEGMessages() its shows something like this:
ffmpeg message ffmpeg version 4.2.2 Copyright (c) 2000-2019 the FFmpeg developers
  built with Apple clang version 11.0.0 (clang-1<IP_ADDRESS>)
  configuration: --prefix=/usr/local/Cellar/ffmpeg/4.2.2_2 --enable-shared --enable-pthreads --enable-version3 --enable-avresample --cc=clang --host-cflags= --host-ldflags= --enable-ffplay --enable-gnutls --enable-gpl --enable-libaom --enable-libbluray --enable-libmp3lame --enable-libopus --enable-librubberband --enable-libsnappy --enable-libtesseract --enable-libtheora --enable-libvidstab --enable-libvorbis --enable-libvpx --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxvid --enable-lzma --enable-libfontconfig --enable-libfreetype --enable-frei0r --enable-libass --enable-libopencore-amrnb --enable-libopencore-amrwb --enable-libopenjpeg --enable-librtmp --enable-libspeex --enable-libsoxr --enable-videotoolbox --disable-libjack --disable-indev=jack
  libavutil      56. 31.100 / 56. 31.100
  libavcodec     58. 54.100 / 58. 54.100
  libavformat    58. 29.100 / 58. 29.100
  libavdevice    58.  8.100 / 58.  8.100
  libavfilter     7. 57.100 /  7. 57.100
  libavresample   4.  0.  0 /  4.  0.  0
  libswscale      5.  5.100 /  5.  5.100
  libswresample   3.  5.100 /  3.  5.100
  libpostproc    55.  5.100 / 55.  5.100
Input #0, matroska,webm, from 'https://r1---sn-h8oxupq-jb3e.googlevideo.com/videoplayback?c=WEB&clen=3527654&dur=217.701&ei=jG1zXvK2FsGDssUP07Gk-Aw&expire=1584644588&fvip=3&gir=yes&id=o-AJXrW4xcFCgVXlnPGCHr51o9dRz-dgyP9UvexvPh6ec3&initcwndbps=323750&ip=<IP_ADDRESS>&itag=251&keepalive=yes&lmt=1575023145707784&lsig=ABSNjpQwRQIgIjFTB6fN1SfHZ30_XtaFRYFq1NIpcqt0Vu4tCAJr8kACIQDixfhd0Rtib3QfmfKakfaTNMdR_Z2qXc8_gTyDmKQ0oA%3D%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpcm2cms%2Cpl%2Cinitcwndbps&mh=mM&mime=audio%2Fwebm&mm=31%2C29&mn=sn-h8oxupq-jb3e%2Csn-npoeenee&ms=au%2Crdu&mt=1584622906&mv=m&mvi=0&pcm2=no&pcm2cms=yes&pl=24&ratebypass=yes&requiressl=yes&sig=ADKhkGMwRQIhAP2gpJasp4VRgXqdPefpkyszHb2l90mnICnZr7nytENNAiAqagp5zDRdhtaxNyFzjjfjcbWiNuhdyi3ClDuKJed8eQ%3D%3D&source=youtube&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cpcm2%2Cvprv%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&txp=5531432&vprv=1':
  Metadata:
    encoder         : google/video-file
  Duration: 00:03:37.70, start: -0.007000, bitrate: 129 kb/s
    Stream #0:0(eng): Audio: opus, 48000 Hz, stereo, fltp (default)
Stream mapping:
  Stream #0:0 -> #0:0 (opus (native) -> opus (libopus))
Press [q] to stop, [?] for help
Output #0, ogg, to 'pipe:1':
  Metadata:
    encoder         : Lavf58.29.100
    Stream #0:0(eng): Audio: opus (libopus), 48000 Hz, stereo, flt, 96 kb/s (default)
    Metadata:
      encoder         : Lavc58.54.100 libopus
[tls @ 0x7fa970e07580] Error in the pull function.
[matroska,webm @ 0x7fa971000c00] Read error
[tls @ 0x7fa970e07580] The specified session has been invalidated for some reason.
    Last message repeated 1 times
https://r1---sn-h8oxupq-jb3e.googlevideo.com/videoplayback?c=WEB&clen=3527654&dur=217.701&ei=jG1zXvK2FsGDssUP07Gk-Aw&expire=1584644588&fvip=3&gir=yes&id=o-AJXrW4xcFCgVXlnPGCHr51o9dRz-dgyP9UvexvPh6ec3&initcwndbps=323750&ip=<IP_ADDRESS>&itag=251&keepalive=yes&lmt=1575023145707784&lsig=ABSNjpQwRQIgIjFTB6fN1SfHZ30_XtaFRYFq1NIpcqt0Vu4tCAJr8kACIQDixfhd0Rtib3QfmfKakfaTNMdR_Z2qXc8_gTyDmKQ0oA%3D%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpcm2cms%2Cpl%2Cinitcwndbps&mh=mM&mime=audio%2Fwebm&mm=31%2C29&mn=sn-h8oxupq-jb3e%2Csn-npoeenee&ms=au%2Crdu&mt=1584622906&mv=m&mvi=0&pcm2=no&pcm2cms=yes&pl=24&ratebypass=yes&requiressl=yes&sig=ADKhkGMwRQIhAP2gpJasp4VRgXqdPefpkyszHb2l90mnICnZr7nytENNAiAqagp5zDRdhtaxNyFzjjfjcbWiNuhdyi3ClDuKJed8eQ%3D%3D&source=youtube&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cpcm2%2Cvprv%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&txp=5531432&vprv=1: Input/output error
size=    2369kB time=00:03:24.30 bitrate=  95.0kbits/s speed=1.04x
video:0kB audio:2352kB subtitle:0kB other streams:0kB global headers:0kB muxing overhead: 0.702825%

When i try to google this error, it's turn out that the youtube URL was expired. So the ffmpeg immediately return EOF.
The specified session has been invalidated for some reason.

I am able to solve this issue (maybe) to add reconnect flag in ffmpeg command. I will try to link my PR to this issue.
Thank you.

Hi,
I just wanted to say that the pull request by fiwippi is exactly the change that resolved this problem for me.
So merging it should fix the whole EOF timeout issue.
Thanks for the great package! :)


Gather all the metrics taking into account possible provider changes

Right now we're using these methods to get the data services API usage metrics in order to either compute the over quota for that user or to gather those metrics for other purposes.
In this PR I've fixed the problem we were having to gather correctly the metrics using the defined provider for the user, but @javitonino has found another corner case in which if the user changes the provider we only gather his/her metrics partially becuase we only get the currently defined provider metrics.
That said we have to take that into account and try to get the metrics for all the possible providers in order to return a Hash like this:
{ mapbox: 200, heremaps: 100 }

BONUS: Also, we should change all the methods from things like remaining_mapzen_routing_quota to something more generic even if we leave the quota column names with that kind of names

Stale issue. Closing 👋


Suggestions for 1.0 milestone

Discuss. Feel free to write the problem inline as a comment or just to link to an issue or PR.
Ideally, full parity with all of Parse's REST features would be nice. Wondering how others feel?
In some ways, I'm really happy with the stability of the codebase. Everything is pretty modular and magic-free. That said, some issues like delta-saving are still bugging me.

My wishlist for the 1.0 is:

remove Parse.init and @@client class variables
add #169 and #170
fix as much bugs as possible

Also, documentation (probably with YARD?) somewhere that isn't just the readme. README would be for instructions for development, some quick examples, and a link to common issues in the wiki (which is another thing to start developing).

Another thing for 1.0:

wiki page explaining how to migrate from 0.3.0 to 1.0
wiki page explaining what's new

@rhymes So I was thinking- I think the client change was the only big breaking change we're going to do- a lot of the other changes are features and stuff.
Why don't we go ahead and cut the 1.0.0 release and then have all of the extra, currently missing parse features be minors after that?
Alternatively, we could burn simple tasks like adding features TODO in the readme (such as deleting objets, etc) for a real v1 feeling, then cut the release?

@Xavdidtheshadow Yep, we could do that, I feel that before though we should increase the test coverage a bit, review the doc (see also #190), write a changelog (we can use git-extras for that https://github.com/tj/git-extras/blob/master/Commands.md#git-changelog) and some sort of migration guide?

Is there an easy place to see where we're missing test coverage on existing code?
Totally. I'll go through tomorrow and make a set of tasks we should actually get ready for 1.0 and when those are done, we can release!

@Xavdidtheshadow only locally, I'll enable coveralls later on the repository so we can all see it. Thanks!

@Xavdidtheshadow https://coveralls.io/github/adelevie/parse-ruby-client?branch=master here you are :D I also added the badge in the README

@rhymes nice! that's way more than I thought it would be- it seems like we're in pretty good shape.

yes indeed :)


Add Cache-Control header only to index.html

How can I add Cache-Control header only to index.html and / only
I have tried many permutations but the automatic canonical rewrite is making it impossible.

Thanks for the issue.
This is better asked over at https://caddy.community


[flutter firebase messaging] 14.1.0 runApp call twice

Bug report
main first function and runApp call twice
Steps to reproduce
Steps to reproduce the behavior:

void main call
initilize firebase
runapp
after couple of seconds main + runapp call again

Expected behavior
call once
Flutter doctor
Run flutter doctor and paste the output below:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.3.8, on macOS 12.6 21G115 darwin-arm, locale en-IL)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.3)
[✓] IntelliJ IDEA Community Edition (version 2021.3.2)
[✓] Connected device (3 available)
[✓] HTTP Host Availability
Flutter dependencies
flutter_core 2.2.0
firebase_messaging 14.1.0

i faced the same issue after upgrading to 14.1.0


Do redisson store classes by creating extra keys or serialize and store in one key only?

### Expected behavior
Actual behavior
Steps to reproduce or test case
Redis version
Redisson version
Redisson configuration

Redisson stores classes in state of object.


QTY Available 0

There are 2 reasons to use KiCost,
1 is for manufacturing NOW.  So i can buy the cheapest parts NOW to build a board.
Another is to get budgetary prices for your board.
In the first case, KiCost is great, as it removes any parts from a supplier with 0 Qty.  But in the second case, thats not very useful because we dont care if we can get the parts right now, we just care "if" we can get the parts whats the BOM cost.  Maybe the parts are on 2 week lead time and thats OK for you.
I propose we need a option that tells Kicad no to ignore parts that have 0 Qty Available, as long as the price is valid and the part was able to be scraped from a supplier.  Then KiCost can fulfil both roles.

Actually this looks like some kind of scraping error.  I am getting the Link but no other information for the parts in the worksheet, and the link shows those parts have ZERO qty available.

I hacked in the following test code.  The Parts are found whcih is why they have a Link, but there was no explanation given why they were being dropped, as there are 3 possibilities this code checks if the link exists, but the part is going to be dropped, which indicates a scraping error, and then expalins why the part was dropped.
 # If the part number doesn't exist or the part quantity is None
        # (not the same as 0), then the distributor doesn't stock this part
        # so leave this row blank.
        # Also leave this row blank if the pricing info doesn't exist so a $0
        # price isn't injected that messes up the search for the best price.
        if len(dist_part_num) == 0 or part.qty_avail[dist] is None or len(list(price_tiers.keys())) == 0:
            if part.url[dist]:
                logger.warning("Part %s scraping error from %s", part.refs, dist)
                if len(dist_part_num) == 0:
                    logger.warning("  -- No Distributor Part Number!")
                if part.qty_avail[dist] is None:
                    logger.warning("  -- No Quantity (Not Even Zero)!")
                if len(list(price_tiers.keys())) == 0:
                    logger.warning("  -- No Pricing Tiers!")

            row += 1  # Skip this row and go to the next.
            continue

This is what it generates from my BOM:
Part ['R2'] scraping error from digikey
  -- No Quantity (Not Even Zero)!
Part ['R4'] scraping error from digikey
  -- No Quantity (Not Even Zero)!
Part ['R5'] scraping error from digikey
  -- No Quantity (Not Even Zero)!
Part ['D2'] scraping error from digikey
  -- No Quantity (Not Even Zero)!
Part ['U3'] scraping error from digikey
  -- No Quantity (Not Even Zero)!
Part ['D4'] scraping error from digikey
  -- No Quantity (Not Even Zero)!
Part ['JP1'] scraping error from digikey
  -- No Quantity (Not Even Zero)!
Part ['R1'] scraping error from digikey
  -- No Quantity (Not Even Zero)!
Part ['C44', 'C45', 'C47', 'C43', 'C46'] scraping error from digikey
  -- No Quantity (Not Even Zero)!
Part ['L1'] scraping error from digikey
  -- No Quantity (Not Even Zero)!
Part ['R2'] scraping error from mouser
  -- No Quantity (Not Even Zero)!
Part ['R5'] scraping error from mouser
  -- No Quantity (Not Even Zero)!
Part ['R1'] scraping error from mouser
  -- No Quantity (Not Even Zero)!
Part ['C44', 'C45', 'C47', 'C43', 'C46'] scraping error from mouser
  -- No Quantity (Not Even Zero)!
Part ['D2'] scraping error from newark
  -- No Quantity (Not Even Zero)!
Part ['P7', 'P10', 'P11', 'P8', 'P9'] scraping error from newark
  -- No Quantity (Not Even Zero)!
Part ['U4'] scraping error from newark
  -- No Quantity (Not Even Zero)!
Part ['U2'] scraping error from newark
  -- No Quantity (Not Even Zero)!
Part ['U7'] scraping error from newark
  -- No Quantity (Not Even Zero)!
Part ['L1'] scraping error from newark
  -- No Quantity (Not Even Zero)!

I added this test hack above the diagnostic i hacked in above.  Logging shows its triggered, but interestingly the spreadsheet now contains all valid data for those previously missing parts.  The Unit Price and catalog number show correctly.
I am not sure if this is the correct way to fix this otherwise I would submit it as a patch.
I inserted this at about line 1007:
# Test Hack...
        if len(dist_part_num) != 0 and len(list(price_tiers.keys())) != 0 and part.qty_avail[dist] is None:
            logger.warning("Part %s correcting QTY to 0 for Distributor %s", part.refs, dist)
            part.qty_avail[dist] = 0

Hi, Steven. While working on issue #45, I think I fixed the problem with only links appearing and no price/quantity for Digikey parts. It was related to changes Digikey made in their web page structure.
As for your original issue, KiCost currently chooses the lowest cost part without regard to availability, so it fits with your long-term, budgetary scenario. It doesn't fulfill the need for the short-term, buy-what-is-available-now scenario.


Imprecise log line: LocalQueue for workload didn't exist

What would you like to be cleaned:
When AddOrUpdateWorkloadWithoutLock returns false, there are several lines which may log that a LocalQueue does not exist
e.g.
https://github.com/kubernetes-sigs/kueue/blob/2c8e7da1a23f2bc7218a2f08a2c9858a316fb053/pkg/controller/core/workload_controller.go#L732-L734
Unfortunately, this function can also return false when the ClusterQueue is not located
https://github.com/kubernetes-sigs/kueue/blob/2c8e7da1a23f2bc7218a2f08a2c9858a316fb053/pkg/queue/manager.go#L336-L339
We should differentiate whether the ClusterQueue or LocalQueue is missing in the log lines which depend on (directly or indirectly) the return value of this function.
Why is this needed:
To assist with future debugging, and to make our logging more precise.

it seems like
/good-first-issue

I'll pick it up :)

/assign


Error upon DOM update when prerendered hydrated non-index page loaded

Problem
When a website that uses static prerendering with client-side hydration is loaded from a non-index (/) page, an update to the DOM results in the following console error:
"Uncaught (in promise) TypeError: root is undefined"
Steps To Reproduce
Steps to reproduce the behavior:

Go to https://www.persentia.com/
Open the browser dev console
Click on the Slideshow navigation link
The URL changes to https://www.persentia.com/slideshow
Wait five seconds without moving the mouse
The control panel disappears and no error shows in the console
Go to https://www.persentia.com/slideshow by entering the URL directly in the browser bar
Wait fives seconds without moving the mouse
The errors shows in the console when a DOM update is attempted to remove the control panel component
Source code: https://github.com/david-wallace-croft/dioxus-prototype/blob/main/src/components/slideshow/mod.rs
Setup instructions: https://www.croftsoft.com/library/tutorials/rust-dioxus-project-setup/

Expected behavior

No errors regardless of what their entry point URL into the application was

Environment:

Dioxus version: 0.4.0
Rust version: 1.73.0
OS info: Windows
App platform: web

Questionnaire

[X] I'm interested in fixing this myself but don't know where to start
[ ] I would like to fix and I have a solution
[ ] I don't have time to fix this right now, but maybe later

Context

I have been digging around for a day trying to figure this out
My guess is that there is an issue with rehydration and templates
I am currently looking into rehydrate.rs
I will keep looking into it but I am filing this bug report in case someone knows the answer off-hand

You are correct, generally that error is caused by hydration. You can debug hydration by turning on tracing to the trace level. It will print every node that it tries to hydrate. If you notice a message about failing to hydrate, you can start debugging the element that it did print or should have printed

Sometimes the browser will correct some html automatically that breaks hydration. For example, if you have a table without a table body, the browser will fill it in, but it will break hydration

When you are in hot reload mode on a subpage (/about) instead of the home page (/) and you click on the reload button in the browser, the hot reload will show the content twice on the page, with updated content shown in the second instance toward the bottom.  I assuming this is a relate issue.  Still looking into it.

Hydration should be a lot more resilient in 0.5 which will make hydration issues more rare. Hydration should succeed now even if the browser or an extension modifies the HTML


zsh Autocompletion not working properly

After adding
$(pip3 show dg | sed -En 's/Location: (.*)/\1/p')/dg/deps/autocomplete/dg-autocomplete-zsh.sh

to .zshrc, as described in the docs, the following errors come up when opening a new shell:
/usr/local/lib/python3.9/site-packages/dg/deps/autocomplete/dg-autocomplete-zsh.sh: line 14: syntax error near unexpected token `descr'
/usr/local/lib/python3.9/site-packages/dg/deps/autocomplete/dg-autocomplete-zsh.sh: line 14: `    for key descr in ${(kv)response}; do'

So at the moment, something seems to be broken with respect to autocompletion.

Closed due to user error, the correct line to add is
. $(pip3 show dg | sed -En 's/Location: (.*)/\1/p')/dg/deps/autocomplete/dg-autocomplete-zsh.sh

The one above was missing the . in front


FR: Support for NX workspace plugin

Is your feature request related to a problem? Please describe.
We use NX to manage our Workspace. The node-workspace plugin isn't resulting in the correct dep tree based on our set up. We have packages that depend on others, but its not resulting in that when using the node-workspce plugin.
Describe the solution you'd like
We'd like to be able to write our own custom plugin to handle this. If you'd like to add support for NX, that would be nice too
Describe alternatives you've considered
Only other alternative was switching to another package, but we like what release-please offers over others like semantic-release
Additional context
NONE

Was able to get it working by adding the workspace dependencies to the package.json dependency list. Supporting NX would still be nice!

I'm not familiar with NX, but if it's well known, we would accept a workspace manager for it. We have an abstract workspace plugin that does the heavy lifting for determining which modules need version bumps, so all that would be needed to add is a NX specific parser to build the initial dependency tree.

I'm also using Nx and I've experienced the same with the node-workspace plugin, I don't get the expected behaviour where for example: changes on lib A which depends on B are not detected when I make changes on B.
@sir-captainmorgan21 can you please describe how you got it working?
The node-workspace plugin is using the Lerna style dependency graph, while Nx resolves packages dependencies on tsconfig level, that's why release please currently doesn't understand the dependencies between packages.

@daton89 yea, so we just explicitly list the internal packages that we are dependent on in the package's package.json... So if packageA depends on packageB, then in packageA's package.json, we list packageB as a dependency like so...
"dependencies": [
   packageB: '^1.0.0'
]

@chingor13 yea, NX is one of the most widely used project management tools, especially for any mono-repo setup with Angular. I could take a stab at implementing an official plugin for this. Is there contributing docs somewhere I can get started?

I started a doc with a braindump about the design of the codebase: https://github.com/googleapis/release-please/pull/1551

@sir-captainmorgan21 I just saw you started to implement an nx-workspace on that branch.
Do you need any help from the community ?

@sir-captainmorgan21 I just saw you started to implement an nx-workspace on that branch.
Do you need any help from the community ?

Happy to test :+1:

@nhuray id love some help. I haven't been able to work on it as much as I'd like with everything going on at home. Last I remember I was in a decent spot where I was just cleaning up the PR output. However the way I was testing wasn't very good. I was building the package locally and manually updating the node modules in the repo where I use release please. Would love some assistance in setting up a test bed in the release please repo itself for NX

is this feature planned to be implemented?

@daton89 yea, so we just explicitly list the internal packages that we are dependent on in the package's package.json... So if packageA depends on packageB, then in packageA's package.json, we list packageB as a dependency like so...
"dependencies": [
   packageB: '^1.0.0'
]

Does this work for anyone? I tried but that does not work for me. I have "node-workspace" in plugins list

@daton89 yea, so we just explicitly list the internal packages that we are dependent on in the package's package.json... So if packageA depends on packageB, then in packageA's package.json, we list packageB as a dependency like so...
"dependencies": [
   packageB: '^1.0.0'
]

Does this work for anyone? I tried but that does not work for me. I have "node-workspace" in plugins list

I wouldn't consider this a valid approach, as previous commenters stated, that NX has a separate mechanism for detecting the dep graph.
This is for good reasons too:

single package.json is faster to install in a massive monorepo
use the tsconfig project reference pattern that NX creates for you results in faster DX in your editor
NX has tools to create package.jsons for "publishable" packages.
NX isn't just for nodejs, it can be used for any language: java, c#, golang, etc

this is all known to those familiar with NX (probably not to those new it it)
So the only viable solution is one that ends up using the @nx/devkit api tooling to discover the dep graph.

Hey 👋
Is this going to be implemented? Our team would love to be able to continue using release-please but failing to properly detect the dependency graph is a deal breaker 😢


Alert user not to leave while payment is processing

When a user makes a payment we don't want them to leave the screen while the mutation hasn't concluded because they may try to make the payment again and end up sending two payments.

Closing as PR is stale.


openjpeg: move jpylizer to nativeCheckInputs

Description of changes
This is useful when cross-compiling to platforms that we can't build Python for (and also just more semantically correct).
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Bisect says 3404d6b0f764b640d0c5a3d59a496039e914f8c3 openjpeg: move jpylizer to nativeCheckInputs broke openjpeg tests in staging as:
$ nix build -f. openjpeg -L
...
openjpeg> running testsable to find executable: /build/source/build/build/jpylyzer
...
openjpeg>     ..................................................  Size: 799K
openjpeg>     ............................................. Size of output: 845K
openjpeg> Unable to find executable: /build/source/build/build/jpylyzer
openjpeg> Unable to find executable: /build/source/build/build/jpylyzer
...

Ugh, what? I'm sure I tested that, although I rebased so maybe something changed.

Fix: https://github.com/NixOS/nixpkgs/pull/214824


Post: integrate Twitter preview card (text only)

Card with image preview is a future work item (on the level of "it's nice to have it"), I anticipate it will be a bit tricky unless the post metadata gets extended.
Tested in live PROD environment.

Transferred work in this PR into https://github.com/SumiMakito/hexo-theme-typography/pull/38, thus close this one.


Added sexyness

Sexyness added. Interface vastly improved, awaiting user feedback.

this is exactly as sensual as i wanted, i'm taut with excitement


🐛 BUG: Better Error Message for when Wrangler Login Fails when using Cloudflare Warp

What version of Wrangler are you using?
0.0.19
What operating system are you using?
Mac
Describe the Bug
I tried running the  wrangler login command but it timed out with this error message.

I was able to fix it by turning off WARP and retying the login process and it worked. We should include a better error message instead that tell me what the issue is and catch the error returned.

Fixed properly in https://github.com/cloudflare/wrangler2/pull/963, we don't need to have an error anymore


Update ssr-and-ssg.md

I noticed you start off explaining what SSR is without clarifying what the acronym stands for, so I've added a few hints here and there. Adding some extra keywords for SEO can't hurt either. I've also added a clarification around SSG to make it clear that it produces actual static html pages that can be hosted on a CDN, as the original reference to "running" on any server suggests a runtime, which can be misleading. Perhaps consider also writing out the acronyms in the menu.

i wish the title should be single line


Add imagePath attribute to cards in mods

I talked to @jay1148 and he will provide us generic logos for Cardshifter, Cyborg Chronicles and Mythos that can be used as placeholders until a card has an image of its own. Eventually, most if not all cards will have an image of their own. This will not require any coding, just adding this attribute to each card.

see https://github.com/Cardshifter/Cardshifter/commit/acb3b4482ba1f47ca657c826d0c2ac4cd9947772

All that's missing is the actual images

I'm making reasonable progress on card images and should be done within a day or two for everything that will go under the web client.

This is done as of today


Update msg_sender() to return sender when input type is Input::Message

Description
This change reflects an issue experienced by an internal contributor where msg_sender().unwrap() would revert when sending a message. Currently, the sender is only returned when the input is of type Input::Coin.
Checklist

[ ] I have linked to any relevant issues.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have updated the documentation where relevant (API docs, the reference, and the Sway book).
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have added (or requested a maintainer to add) the necessary Breaking* or New Feature labels where relevant.
[ ] I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
[ ] I have requested a review from the relevant team or maintainers.

I believe you should use input_message_recipient instead of input_message_sender because they will be the owner upon receiving the resource

Perhaps I'm misunderstanding, but why would we want the recipient when calling msg_sender()?

I believe you should use input_message_recipient instead of input_message_sender because they will be the owner upon receiving the resource

Perhaps I'm misunderstanding, but why would we want the recipient when calling msg_sender()?

Well, the high level method is called msg_sender() but the method being adjusted regards the owner of the inputs. When sent the owner of the message may be the sender field but the recipient will become the owner.
There was more talk of this, which yet again I forget the details, from @segfault-magnet for when and what the sender / recipient represent depending on when the message is constructed iirc.

Resolved by https://github.com/FuelLabs/sway/pull/6231


Client as server

I am curious why did you chose to make a client act as a server and not use a separate server. Is this for testing reasons only? Thanks.
Great work!

Sorry for the late response. Well, it is indeed a testing project, more like a proof of concept than a finished "product", so that's partly the reason.
However, the basic idea was to create a simple "hybrid" network architecture, trying to keep the good parts of the classic client-server and p2p models. In this case every client can act as host or connect to other clients hosting a game, without the need for a dedicated server and a p2p architecture (that would complicate things a bit, because of the decentralized communications).
On the other hand, the architecture still uses an authoritative model, allowing for a higher degree of control over the data sent between the players (thinking about cheating or unwanted packets, it would surely help having a server filtering), and the server code is an almost independent module, that could be easily detached from the whole project and converted into a dedicated executable.
This work is a bit of a "hobby project", the whole thing is still unfinished, and unfortunately I won't be able to actively work on it for a while, but I will almost certainly come back to it in the future, since it still has lots of flaws and stuff I'd like to change. For instance, the whole simulation module is currently organized to work on its own thread, but after some testing it seemed a bit overkill and almost unnecessary, so I'll probably switch back to the main unity thread, if I don't find any good reason to keep it separate.
Anyway, sorry for the wall of text, I hope I answered your question, more or less.
If you have any doubts or suggestions feel free to ask/comment! I'll try to answer a bit faster next time :)
Cheers!


Create Codegen to transform dataset jsons into a dataset.ts file

[ ] create codegen to take json formatted datasets and compile then into the datasets.ts file
[ ] if serviceAreaIds is empty, it should assume full state and replace it by serviceAreaIds: SERVICE_AREAS_BY_STATE.tx (SERVICE_AREAS_BY_STATE to be imported at the top of the file).

Alternatively, consider creating a dataset folder and a function to load these datasets in datasets.ts instead. I think I like this better actually and avoids lots of changes in PRs. @akegan

not sure what that would look like @ericboucher


POMP weights for other CLIP variants

Hi @RenShuhuai-Andy,
I understand that the POMP weights are dependent on the model used for pre-training.
So do you have weights for other variants of the CLIP model i.e. ViT-B/32 and ViT-L/14 backbone?

Hi, thanks for your interest.
Yes, you are correct that the POMP weights are dependent on the model used for pre-training.
Currently, we don't have weights specifically for the ViT-B/32 and ViT-L/14 backbone variants of the CLIP model. However, I think it's a great suggestion and I would be happy to pre-train prompts for these variants. This process may take around one or two weeks to complete.

@RenShuhuai-Andy Thanks for the reply. I'll be waiting for the pre-trained prompts for these variants.

@RenShuhuai-Andy
Thanks for yout great work! Are there pomp weights for the ViT-B/32 backbone variants of the CLIP mode now?

Hi, sorry for the late reply. Since I have finished my internship in AWS, it takes me much time to re-prepare the datasets and re-train the prompt.
The POMP weights for the ViT-B/32 backbone are available in https://huggingface.co/ShuhuaiRen/POMP-ViT-Base-32/tree/main. The average cross-dataset accuracy for vit_b32_ep5_randaug2_unc1000_16shots_nctx4_cscFalse_ctpend_seed42.pth.tar and vit_b32_ep20_randaug2_unc1000_16shots_nctx16_cscFalse_ctpend_seed42.pth.tar is 62.0% and 61.8%, respectively.
The POMP weights for the ViT-L/14 backbone are available in https://huggingface.co/ShuhuaiRen/POMP-ViT-Large-14/tree/main. The average cross-dataset accuracy for vit_l14_ep5_randaug2_unc1000_16shots_nctx4_cscFalse_ctpend_seed42.pth.tar and vit_l14_ep20_randaug2_unc1000_16shots_nctx16_cscFalse_ctpend_seed42.pth.tar is 72.4% and 71.9%, respectively.

Thanks @RenShuhuai-Andy!


Updated deps, fixes type error

Hi,
I wanted to use the plugin on the lume.land site and it was throwing a type error.
I updated the deps to fix the error. Can you please release the new version?

Thanks so much. I reworded the changelog entry, slightly. As long as you're okay with the changes, I'll go ahead and merge this.


Comment from a user  -- regarding presentation of info

For the Disabled category, it would make more sense to show % of people disabled in the district, and use that as the basis of comparison.
E.g. saying a district has 10,000 disabled, which is less than 10% of Nepal's 500,000 disabled, isn't that meaningful.
But instead, we could say 1.6% of people in this district are disabled, compared to 2.0% nationally.
And when the percentage is low, it may be more impactful to say "1 in 60 people is disabled"
The ability to compare 2 districts is nice, good layout.
But the site doesn't actually compare them numerically for you. You have to eyeball it.
You could have a feature where for each category, 1 of the 2 districts "wins" by having a higher value for the statistic, and is highlighted. Instead of "about 90 percent of the rate in Nepal," why not "about 10 percent lower..."?

@ravinepal
Here are two screenshots that display a percentage for disability rather than a population count. The first is just one district. The second is a comparison across two districts. This is a straightforward change. A change for "1 in 60" would require a little more work.
As far as the "about 90 percent of the rate in Nepal" language goes, those sorts of descriptive comparison texts are built into Wazimap. If we wanted something different, we would need to explore if and how to make that change.
Lalitpur

Humla compared to Jhapa

@cliftonmcintosh thanks for making the changes. I think it's good for now! We can def explore more in the future!


BrokenPipeError/OSError

one out of 3-4 times on xonsh startup with ptk i'm getting
Traceback (most recent call last):
  File "/home/laerus/projects/xonsh/xonsh/ptk/shortcuts.py", line 108, in prompt
    document = cli.run(reset_current_buffer=False)
  File "/home/laerus/.local/lib/python3.5/site-packages/prompt_toolkit/interface.py", line 389, in run
    self.eventloop.run(self.input, self.create_eventloop_callbacks())
  File "/home/laerus/.local/lib/python3.5/site-packages/prompt_toolkit/eventloop/posix.py", line 153, in run
    t()
  File "/home/laerus/.local/lib/python3.5/site-packages/prompt_toolkit/eventloop/posix.py", line 81, in read_from_stdin
    inputstream.feed(data)
  File "/home/laerus/.local/lib/python3.5/site-packages/prompt_toolkit/terminal/vt100_input.py", line 390, in feed
    self._input_parser.send(c)
  File "/home/laerus/.local/lib/python3.5/site-packages/prompt_toolkit/terminal/vt100_input.py", line 299, in _input_parser_generator
    self._call_handler(match, prefix)
  File "/home/laerus/.local/lib/python3.5/site-packages/prompt_toolkit/terminal/vt100_input.py", line 332, in _call_handler
    self.feed_key_callback(KeyPress(key, insert_text))
  File "/home/laerus/.local/lib/python3.5/site-packages/prompt_toolkit/interface.py", line 1019, in feed_key
    cli.input_processor.process_keys()
  File "/home/laerus/.local/lib/python3.5/site-packages/prompt_toolkit/key_binding/input_processor.py", line 209, in process_keys
    cli.invalidate()
  File "/home/laerus/.local/lib/python3.5/site-packages/prompt_toolkit/interface.py", line 333, in invalidate
    self.eventloop.call_from_executor(redraw, _max_postpone_until=_max_postpone_until)
  File "/home/laerus/.local/lib/python3.5/site-packages/prompt_toolkit/eventloop/posix.py", line 236, in call_from_executor
    os.write(self._schedule_pipe[1], b'x')
BrokenPipeError: [Errno 32] Broken pipe

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/laerus/.local/bin/xonsh", line 6, in <module>
    exec(compile(open(__file__).read(), __file__, 'exec'))
  File "/home/laerus/projects/xonsh/scripts/xonsh", line 3, in <module>
    main()
  File "/home/laerus/projects/xonsh/xonsh/main.py", line 221, in main
    shell.shell.cmdloop()
  File "/home/laerus/projects/xonsh/xonsh/ptk/shell.py", line 114, in cmdloop
    line = self.singleline(auto_suggest=auto_suggest)
  File "/home/laerus/projects/xonsh/xonsh/ptk/shell.py", line 83, in singleline
    line = self.prompter.prompt(**prompt_args)
  File "/home/laerus/projects/xonsh/xonsh/ptk/shortcuts.py", line 113, in prompt
    eventloop.close()
  File "/home/laerus/.local/lib/python3.5/site-packages/prompt_toolkit/eventloop/posix.py", line 252, in close
    os.close(schedule_pipe[0])
OSError: [Errno 9] Bad file descriptor

xonsh issue  #1710
any idea why this is happening?
on latest PyPI version prompt-toolkit==1.0.7

Hi @laerus,
Thank you for reporting this issue! That sounds like a bug.
Which OS, Python version and terminal application are you using?
I'd like to be able to reproduce this.

hey @jonathanslenders, i'm on debian-testing, python3.5.2 and terminator.

Hi @laerus,
So far, I have not been able to reproduce this.
However, it could be that the following commit will fix it: https://github.com/jonathanslenders/python-prompt-toolkit/commit/144f9c28cf54eaee437197c5dde5b2c6dc7ae385 There was definitely a race condition.
Can you have a try?

i cannot reproduce this anymore, thanks @jonathanslenders !


Replace Material UI Notification components with Notification components from the @olympusdao/component-library

Replace Material UI Notification components with Notification components from the @olympusdao/component-library

Can you guide me solving this ?

Absolutely @RishiKumarRay

So please help me

How can I help?

@alexleventer - cleaning up this stale issue. We don't have any currently active notifications that would use this component on the frontend.


LLVM 12

LLVM 12 (not yet released, currently at RC3) will be required for building Chromium M90+ (currently in the beta channel; stable release planned for 2021-04-13; s. 5e42fea124a2882cd6ac5e27c9827bdf97c29282).
cc @ggreif: I couldn't find a draft PR for LLVM 12 yet. Are you already working on it or currently busy?
LLVM Release Schedule:

12.0.0:

Jan 26: release/12.x branch, 12.0.0-rc1
Feb 23: 12.0.0-rc2
Mar 2: 12.0.0-final

I have't started yet. But I think I have a few minutes now!

Just a heads-up, we have

pkgs/top-level/all-packages.nix:    llvm = llvm_10;
Should that transition to llvm_11?

I have to hand this over to you, Borg says the unit tests fail (on both platforms, consistently). Maybe the testing framework needs a bump?

Actually, LLVM 12 rcX is not enough
I tested with 12-rc.2 and the key commit https://github.com/llvm/llvm-project/commit/81a9707723845a5b880245da24633c519493205c seems not included into 12-rc.3
You have to wait for 12.0.0 or compile LLVM master
related https://github.com/NixOS/nixpkgs/issues/114828

The pkgs/top-level/all-packages.nix: llvm = llvm_10; match is a bit out of context:
  faust2 = callPackage ../applications/audio/faust/faust2.nix {
    llvm = llvm_10;
  };

We have llvmPackages_latest = llvmPackages_11; so everything should be fine.
@kvtb ok, thanks for that information! If I understand that correctly you've tried compiling chromiumBeta or chromiumDev with #116646 and it requires cherry-picking https://github.com/llvm/llvm-project/commit/81a9707723845a5b880245da24633c519493205c as well (which would be unfortunate because that commit might not be part of the final release; they don't seem to have a dedicated branch for the LLVM 12 release and I don't know where/how/if they track and provide that information).

I built clang_13 (from llvm master, simple replacing fetchwith fetchFromGitHub and removing some lines in postUnpack) and it builds Chromium 91

RC3 added in https://github.com/NixOS/nixpkgs/commit/7c27d4981556c0a0a5ca9cf0e8eeaf579538883e.


DH-10111 Fix missing horizontal scroll bar for single column tables

Fixes #446 and related to DH-10111 test failure
This adds a horizontal scroll bar for any single column table which does not fit in the panel fully
Example Python to test with
from deephaven import new_table
from deephaven.column import string_col
import random
import string

letters = string.printable
strings = [''.join(random.choice(letters) for i in range(random.randint(0, 1000))) for j in range(0, 100)]
table1 = new_table([string_col("Random", strings)])

Also tested with changing IrisGridTheme.rowHeaderWidth to be non-zero

Updated so vertical scroll uses the same logic as horizontal
Tested by setting a big rowHeight in IrisGridTheme.module.scss. Also fixed/tested horizontal scroll with scrollSnapToColumn: true in IrisGridTheme.ts


[WIP] : ODC-7404 Integrate Results API in PipelineRuns card

@vikram-raj  @lokanandaprabhu - I still have to add the interval part in the hook.
Also is there some snippet on how to support error handling in these responses?

Closing in favor of - https://github.com/openshift-pipelines/console-plugin/pull/18


Cannot find minimap2

Hello,
I am having trouble to run Ra on a cluster. I tried on two different clusters, and get this:
[Ra::run] preconstruction overlap stage [Ra::run] error: unable to run minimap2!
But minimap2 is in my PATH. How can I make sure it finds minimap2?

Hi Nadege,
it is probably due to Ra being a python script connecting Minimap2, Rala and Racon. Try manually editing the ra.py script you are running by setting the right paths for all three tools. Add full paths at lines 13-15.
If I may ask, why are you using Ra instead of Raven?:)
Best regards,
Robert

Seems to work better. I have to wait for the job to run though.
I am using Ra and Raven because I still get sometimes on some genomes better results with Ra, so I test both.


replace validate_* calls with puppet4 datatypes

@bastelfreak I'm going to get #14 ahead of this one to help out on the testing side.

I created a conflict here, my apologies.  Would you mind a rebase here please?

@xaque208 all fixed

Merged out of band.  Thanks @bastelfreak!


add render attached file macro

Add new macro to render attached file.
Could you please check and merge?
Example1
{{plantuml_attach(diagram.pu)}}

Example2
with image format
{{plantuml_attach(diagram.pu, format=svg)}}

Would this macro find diagram.pu attach in any project?

@paulvi Macro arguments are files attached to the corresponding wiki or ticket


Ruby Sass is deprecated and will be unmaintained as of 26 March 2019.

See:
https://github.com/sass/ruby-sass/compare/3.5.6...3.5.7
+Ruby Sass is deprecated and will be unmaintained as of 26 March 2019.
+
+* If you use Sass as a plug-in for a Ruby web framework, we recommend using the
+  sassc gem: https://github.com/sass/sassc-ruby#readme

So, will be good to add support of https://github.com/sass/sassc-ruby for stylesheets

👍

:+1:

Any news about sassc support? Thanks.

This maybe a good start point: https://github.com/hkrutzer/sasscrack


Extensibility points

Journey part of this page is mentioning new extensibility points to be enabled, but it seems that it is not yet done. I already reported simillar issue here: https://github.com/MicrosoftDocs/azure-devops-docs/issues/6833.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 3c520dee-1218-777e-7405-551623817c03
Version Independent ID: 82b5d172-ae1a-2b4c-82ac-595cd7609d3c
Content: New test plans page - Azure Test Plans
Content Source: docs/test/new-test-plans-page.md
Product: devops
Technology: devops-test
GitHub Login: @ravishan-msft
Microsoft Alias: ravishan-msft

@JanSera  - this functionality has been completed and is part of the ongoing deployment, which should complete in 1-2 weeks. Please reactivate if you don't see this functionality post 2 weeks.


Make compatible with Good 6.4

Closes #5
Misc

Add support for creating instance without invoking constructor
Remove sinon as dependency
Fix tests

@PaulMougel I've mixed your changes with fixes on the tests. Would you mind testing if it's working for you?. I'll merge this with master if so.

Awesome, thank you for looking into this!
I think there's an issue with regards to how the library handles the channel option.
The configuration I used to send was the following:
{
    "url": "<my slack hook URL>",
    "slack": {
      "channel": "@paul",
      "icon_url": "<some URL>",
      "username": "Hapi Node.js"
    }
  }

which now doesn't work: Error: channel must be a string.
But if I use
{
    "url": "<my slack hook URL>",
    "channel": "@paul",
    "icon_url": "<some URL>",
    "username": "Hapi Node.js"
  }

the object instantiation works, but none of the settings (except from url) is taken into account.
If I drop the Hoek check on channel and use the first configuration object, then everything works as expected.
Should we drop the Hoek check? IIRC, Slack webhook integrations set up a default channel.

@PaulMougel Good call. I changed it to make config.channel optional. It should be working now with the first config snippet. I still need to update the README to reflect the correct values.

Good work on this. I've tested all the events and seems to work as expected. Would you like me to submit a PR for the fallback fields?

@cxam Thanks for the feedback!. Sure thing, contributions are always welcome. I'll merge this right away and publish the new version.


Not available in Ruby 2.6, any further plan?

It seems that win32scrrenshot is not available now because ffi-1.9.25 can not installed in Ruby 2.6. So do we have any plan on this?
Thanks very much!

Sorry for I forget upload the log:
ERROR:  Error installing win32screenshot:
        The last version of ffi (~> 1.9.0) to support your Ruby & RubyGems was 1.9.25. Try installing it with `gem install ffi -v 1.9.25` and then running the current command again
        ffi requires Ruby version >= 2.0, < 2.6. The current ruby version is <IP_ADDRESS>.

I don't have a Windows machine nearby to test it out, but would win32screenshot work if its ffi dependency would be upgraded to support Ruby 2.6+? Can you test it out by cloning this repository, upgrading ffi and running its specs (and looking at the images done by tests just in case)?

No problem, I'll try it.

Unfortunately, I think my Windows machine can not install win32screenshot now. I find that RAutomation should be installed first, and it also depends on ffi 1.9.25. It's not the problem until I see the sentence:
Currently only 32bit Ruby is supported. 64bit versions do not work at all!

Currently, I use Ruby 2.6.3-x64-Windows. 😭

You're welcome to upgrade RAutomation to support x64 too ;)

LOL, I will try it in x86 till I install a x86 version Ruby in another machine. I'd better to use x86 version one because some native codes should run with my Ruby codes, they also only support x86 in fact. :-P

Released 3.0.0, which should work with Ruby 2.6 too (only 32bit due to its dependency RAutomation!).


Installation Error: Missing 'xrt_phxcore.dll' File

Hi!
I followed steps documented in https://ryzenai.docs.amd.com/en/latest/inst.html#installation-steps
(base) C:\Users\user\Downloads\ryzen-ai-sw-0.9\ryzen-ai-sw-0.9>install.bat -env iput1
Windows 11: OK
Visual Studio OK
Python: OK
CONDA Available: OK
CMake: OK
IPU driver Available: OK
All deps are available. Proceeding to Conda env creation...
Do you accept EULA for RyzenAI? [Y/n]: Y
Proceeding further ...
Creating conda env: ryzenai-0.9-4 週五/1/2-042048 ...
Ran pip subprocess with arguments:
['C:\\Users\\user\\miniconda3\\envs\\iput1\\python.exe', '-m', 'pip', 'install', '-U', '-r', 'C:\\Users\\user\\Download]
Pip subprocess output:
Processing c:\users\user\downloads\ryzen-ai-sw-0.9\ryzen-ai-sw-0.9\vai_q_onnx-1.15.0+a27a6e0-py2.py3-none-any.whl (from)
Collecting onnxruntime (from -r C:\Users\user\Downloads\ryzen-ai-sw-0.9\ryzen-ai-sw-0.9\condaenv.wi3umat_.requirements.)
  Obtaining dependency information for onnxruntime from https://files.pythonhosted.org/packages/67/73/4e9a5166f4c709207a
  Downloading onnxruntime-1.16.3-cp39-cp39-win_amd64.whl.metadata (4.5 kB)
Collecting coloredlogs (from onnxruntime->-r C:\Users\user\Downloads\ryzen-ai-sw-0.9\ryzen-ai-sw-0.9\condaenv.wi3umat_.)
  Using cached coloredlogs-15.0.1-py2.py3-none-any.whl (46 kB)
Collecting flatbuffers (from onnxruntime->-r C:\Users\user\Downloads\ryzen-ai-sw-0.9\ryzen-ai-sw-0.9\condaenv.wi3umat_.)
  Obtaining dependency information for flatbuffers from https://files.pythonhosted.org/packages/6f/12/d5c79ee252793ffe8a
  Using cached flatbuffers-23.5.26-py2.py3-none-any.whl.metadata (850 bytes)
Collecting numpy>=1.21.6 (from onnxruntime->-r C:\Users\user\Downloads\ryzen-ai-sw-0.9\ryzen-ai-sw-0.9\condaenv.wi3umat)
  Obtaining dependency information for numpy>=1.21.6 from https://files.pythonhosted.org/packages/07/34/748ec8c81235277a
  Using cached numpy-1.26.2-cp39-cp39-win_amd64.whl.metadata (61 kB)
Collecting packaging (from onnxruntime->-r C:\Users\user\Downloads\ryzen-ai-sw-0.9\ryzen-ai-sw-0.9\condaenv.wi3umat_.re)
  Obtaining dependency information for packaging from https://files.pythonhosted.org/packages/ec/1a/610693ac4ee14fcdf2da
  Using cached packaging-23.2-py3-none-any.whl.metadata (3.2 kB)
Collecting protobuf (from onnxruntime->-r C:\Users\user\Downloads\ryzen-ai-sw-0.9\ryzen-ai-sw-0.9\condaenv.wi3umat_.req)
  Obtaining dependency information for protobuf from https://files.pythonhosted.org/packages/b6/4b/f4f3334784576822d781a
  Using cached protobuf-4.25.1-cp39-cp39-win_amd64.whl.metadata (541 bytes)
Collecting sympy (from onnxruntime->-r C:\Users\user\Downloads\ryzen-ai-sw-0.9\ryzen-ai-sw-0.9\condaenv.wi3umat_.requir)
  Using cached sympy-1.12-py3-none-any.whl (5.7 MB)
Collecting onnx>1.12.0 (from vai-q-onnx==1.15.0+a27a6e0->-r C:\Users\user\Downloads\ryzen-ai-sw-0.9\ryzen-ai-sw-0.9\con)
  Obtaining dependency information for onnx>1.12.0 from https://files.pythonhosted.org/packages/b8/d9/39382887c8fb4330aa
  Using cached onnx-1.15.0-cp39-cp39-win_amd64.whl.metadata (15 kB)
Collecting onnxruntime (from -r C:\Users\user\Downloads\ryzen-ai-sw-0.9\ryzen-ai-sw-0.9\condaenv.wi3umat_.requirements.)
  Obtaining dependency information for onnxruntime from https://files.pythonhosted.org/packages/6a/fb/99bc0e75f3d23eab0a
  Using cached onnxruntime-1.15.1-cp39-cp39-win_amd64.whl.metadata (4.1 kB)
Collecting onnxruntime-extensions (from vai-q-onnx==1.15.0+a27a6e0->-r C:\Users\user\Downloads\ryzen-ai-sw-0.9\ryzen-ai)
Processing c:\users\user\downloads\ryzen-ai-sw-0.9\ryzen-ai-sw-0.9\voe-4.0-win_amd64\voe-0.1.0-cp39-cp39-win_amd64.whl (from -r requirements_ryzenai_voe.txt (line 1))
Processing c:\users\user\downloads\ryzen-ai-sw-0.9\ryzen-ai-sw-0.9\voe-4.0-win_amd64\onnxruntime_vitisai-1.15.1-cp39-cp32023-11-24 04:24:37,672 - INFO - copying C:\Windows\System32\AMD\xrt_core.dll to C:\Users\user\miniconda3\envs\iput1\lib\site-packages\onnxruntime\capi
2023-11-24 04:24:37,688 - INFO - copying C:\Windows\System32\AMD\xrt_coreutil.dll to C:\Users\user\miniconda3\envs\iput1\lib\site-packages\onnxruntime\capi
2023-11-24 04:24:37,688 - CRITICAL - C:\Windows\System32\AMD\xrt_phxcore.dll does not exist.
2023-11-24 04:24:37,688 - INFO - copying C:\Users\user\Downloads\ryzen-ai-sw-0.9\ryzen-ai-sw-0.9\voe-4.0-win_amd64\voe-0.1.0-cp39-cp39-win_amd64\onnxruntime.dll to C:\Users\user\miniconda3\envs\iput1\lib\site-packages\onnxruntime\capi
Setting XLNX_VART_FIRMWARE env variable ...
Created conda env: iput1

The alert "CRITICAL - C:\Windows\System32\AMD\xrt_phxcore.dll does not exist" clearly signifies a serious issue. Despite this, if I choose to overlook the message and attempt to verify the availability of the vitisai EP, I encounter an additional error message.
(base) C:\Users\user\Downloads\ryzen-ai-sw-0.9\ryzen-ai-sw-0.9>conda activate iput1
(iput1) C:\Users\user\Downloads\ryzen-ai-sw-0.9\ryzen-ai-sw-0.9>python
Python 3.9.18 | packaged by conda-forge | (main, Aug 30 2023, 03:40:31) [MSC v.1929 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import onnxruntime as ort
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\user\miniconda3\envs\iput1\lib\site-packages\onnxruntime\__init__.py", line 55, in <module>
    raise import_capi_exception
  File "C:\Users\user\miniconda3\envs\iput1\lib\site-packages\onnxruntime\__init__.py", line 23, in <module>
    from onnxruntime.capi._pybind_state import ExecutionMode  # noqa: F401
  File "C:\Users\user\miniconda3\envs\iput1\lib\site-packages\onnxruntime\capi\_pybind_state.py", line 33, in <module>
    from .onnxruntime_pybind11_state import *  # noqa
ImportError: DLL load failed while importing onnxruntime_pybind11_state: 找不到指定的程序。

I searched the found a similar error has been reported here.  Below are related info.

Processor Model: Ryzen 7840U
OS: Windows 11 Home Edition
IPU driver version: <IP_ADDRESS>
Ryzen SW version: 0.9 preview

@changtimwu , your IPU driver version seems old, can you try installing IPU driver version <IP_ADDRESS> following installation page?

Thanks! After upgrading the IPU driver, the VitisAIExecutionProvider shows up.
>>> import onnxruntime as ort
>>> ort.get_available_providers()
['VitisAIExecutionProvider', 'CPUExecutionProvider']

I would argue that the official documentation is quite confusing...


⚠ Cioos National CKAN Sitemap has degraded performance

In df043d5, Cioos National CKAN Sitemap (https://catalogue.cioos.ca/sitemap/sitemap-1.xml) experienced degraded performance:

HTTP code: 200
Response time: 6202 ms

Resolved: Cioos National CKAN Sitemap performance has improved in 6b7d6cf after 7 minutes.


Improve logging of enabled features

This also adds a logger we can use for other things too.

Console.group is meant to be used for sync code: open, log, log, log, close.
In this case unrelated logging might be trapped in our group, especially if we open it up for use to any part of the extension.

It’s only a problem if we use console.log instead of logger.log, which we won’t. Otherwise it will be correctly split up.

I’m talking about logging done by other extensions

@bfred-it I could monkey-patch console.log and handle it. Then it will just be split up. Still better than no grouping:

I agree that monkey-patching console.log is better than no grouping, but what if some other extension did the same (monkey-patched console.log for themselves)?

We can do something else like store and flush logs, but we don't know when to flush the logs.
class Logger {
	constructor() {
		this.logs = [];
	}

	log(...messages) {
		this.logs.push(messages);
	}

	flush() {
		console.group(this.group);
		this.logs.map(message => console.log(...message));
		console.groupEnd();

		this.logs = [];
	}
}

Can’t monkey patch, extensions are run in a safe environment, same reason why we can’t access a jQuery loaded on a site.


text control ruins new form designer

When i drop this control into my form, the new form designer errors out. This started happening around last weekend, have you experienced anything similar?

I have the same issue. Clicking 'Technical Details' or 'OK' does nothing, only option is the 'Back' button.
Works fine (thankfully) in the old form designer.

I have a case going on with Microsoft support. They told me it's their fault and a hot fix will be deployed next weekend if all goes well.

Great, thanks v much for letting us know...

Seems to be working ok now for me in UK (crm11)

started working here today to :) so i guess i should close this issue


Update safe_list.json

Adding a number of podcasts and music channels

Adding a number of podcasts and music channels

Hi thanks for the pull request. I noticed some of the channel already exist in the elesticsearch database, can you remove them ? This list is only used for very special cases, when they can't be added automatically.

I added a few of the artists id as some of the channels don't have original content. I'll try another resync to see if they can be added.


fix: routes with conflicting services

What this PR does / why we need it:
Gateway routes with conflicting services in their backend don't break config build.

Which issue this PR fixes:
Fixes #2565

Special notes for your reviewer:

PR Readiness Checklist:
Complete these before marking the PR as ready to review:

[ ] the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

@pmalek @rainest this PR is ready to be reviewed again, PTAL

I'd like to have a review by @rainest as well, as he is the original author of the issue and questioned the first implementation.