Error content not displayed

Hi,
I set this middleware at the top of the stack, The errors and exception are well intercepted (I saw them into my Logger) but my response content is not displayed.
I use the following code:
( new Middlewares\ErrorHandler( 'App::errorHandler' ) )->catchExceptions( true ),
And my handler is like
static public function errorHandler( ServerRequestInterface $request ) {
    //Get the error info using the "error" attribute
    $error = $request->getAttribute( 'error' );
    $response = ( new Response() )->withStatus( $error[ 'status_code' ] );

    switch ( $error['status_code'] ) {
      case 404:
        App::$view->setView( 'errorNotFound' );
        $output = App::$view->__invoke();
        $response->getBody()->write( $output );

      case 503:
        App::$view->setView( 'maintenance' );
        $output = App::$view->__invoke();
        $response->getBody()->write( $output );

      default:
        App::$logger->debug( $error[ 'exception' ]->getMessage() );
        App::$view->setView( 'errorServer' );
        $output = App::$view->__invoke();
        $response->getBody()->write( $output );
    }

    return $response;
  }

What am I doing wrong ?

Just for information I'm using middleman as dispatcher.

Middleman is using an old version of the PSR-15 interface (0.2 instead the current 0.4). I recommend using the dispatcher from Middleware\Utils.
Anyway, I suspect this is a bug in your code:

You should add a break after each case in the switch
If you don't see the response, are you sure the body has content?
Is being the response emitted?

Hi,
I used the dispatcher from Middleware\Utils but same result. I've already added a break after each case (not in the version I copied / pasted here sorry)
If I put a var_dump( $response->getBody() ); exit( 1); the page is well displayed.
Into my app I just do:
$dispatcher = new Middlewares\Utils\Dispatcher( $middlewares );
$response = App::$dispatcher->dispatch( ServerRequestFactory::fromGlobals() );
Do I have to do anything else to emit the response ? I thought the dispatcher handled that.

Nop, the dispatcher returns the response but does not output the content to the browser. If you're using diactoros, it provides some emitters, depending of your needs. For example:
$emitter = new Zend\Diactoros\Response\SapiEmitter();
$emitter->emit($response);

Thank you for all  your job and for your answer.


Bug: IServiceProvider::QueryService produces STATUS_ACCESS_VIOLATION

Which crate is this about?
windows
Crate version
0.48.0
Summary
When calling QueryService on IServiceProvider , using a pointer to a pointer as the documentation specifies, results in a STATUS_ACCESS_VIOLATION error.
I believe it is a similar problem to the one mentioned in this issue: Bug: Strange status access violation when using SHGetPropertyStoreFromParsingName?
The solution presented in that issue also appears to solve this one.
Toolchain version/configuration
Default host: x86_64-pc-windows-msvc
rustup home:  C:\Users\Nystik.rustup
installed toolchains
stable-i686-pc-windows-msvc
stable-x86_64-pc-windows-msvc (default)
nightly-2023-06-15-x86_64-pc-windows-msvc
nightly-i686-pc-windows-msvc
nightly-x86_64-pc-windows-msvc
installed targets for active toolchain
wasm32-unknown-unknown
x86_64-pc-windows-msvc
active toolchain
stable-x86_64-pc-windows-msvc (default)
rustc 1.71.0 (8ede3aae2 2023-07-12)
Reproducible example
use std::{error::Error, ffi::c_void, mem::ManuallyDrop, ptr::NonNull};

use windows::{
    core::{ComInterface, HSTRING},
    Win32::{
        Foundation::S_FALSE,
        System::Com::{
            CoCreateInstance, CoInitializeEx, IDispatch, IServiceProvider, CLSCTX_ALL,
            COINIT_MULTITHREADED, VARIANT, VARIANT_0, VARIANT_0_0, VARIANT_0_0_0, VT_EMPTY, VT_I4,
        },
        UI::Shell::{
            IFolderView, IShellBrowser, IShellWindows, SID_STopLevelBrowser, ShellWindows,
            CSIDL_DESKTOP, SVGIO_ALLVIEW, SWC_DESKTOP, SWFO_NEEDDISPATCH,
        },
    },
};

fn get_desktop_variant_loc() -> VARIANT {
    let vt = ManuallyDrop::new(VARIANT_0_0 {
        vt: VT_I4,
        wReserved1: 0,
        wReserved2: 0,
        wReserved3: 0,
        Anonymous: VARIANT_0_0_0 {
            lVal: CSIDL_DESKTOP as i32,
        },
    });
    let vtloc = VARIANT {
        Anonymous: VARIANT_0 { Anonymous: vt },
    };

    vtloc
}

fn get_empty_variant() -> VARIANT {
    let vt = ManuallyDrop::new(VARIANT_0_0 {
        vt: VT_EMPTY,
        wReserved1: 0,
        wReserved2: 0,
        wReserved3: 0,
        Anonymous: VARIANT_0_0_0 { llVal: 0 },
    });
    let vtloc = VARIANT {
        Anonymous: VARIANT_0 { Anonymous: vt },
    };

    vtloc
}

pub fn find_desktop_folder_view<T: ComInterface>() -> Result<T, Box<dyn Error>> {
    unsafe { CoInitializeEx(Some(std::ptr::null_mut()), COINIT_MULTITHREADED)? };

    let shell_windows: IShellWindows =
        unsafe { CoCreateInstance(&ShellWindows, None, CLSCTX_ALL)? };

    let mut dispatch: Option<IDispatch> = None;
    let mut vtloc = get_desktop_variant_loc();
    let mut vtempty = get_empty_variant();
    let mut lhwnd: i32 = 0;
    unsafe {
        shell_windows.FindWindowSW(
            &vtloc,
            &vtempty,
            SWC_DESKTOP,
            &mut lhwnd,
            SWFO_NEEDDISPATCH,
            &mut dispatch,
        )?;
        ManuallyDrop::drop(&mut vtloc.Anonymous.Anonymous);
        ManuallyDrop::drop(&mut vtempty.Anonymous.Anonymous);
    }

    let provider = dispatch
        .ok_or(windows::core::Error::new(
            S_FALSE,
            HSTRING::from("dispatcher not found"),
        ))?
        .cast::<IServiceProvider>()?;

    let shell_browser: &IShellBrowser = unsafe {
        let mut shell_browser: *mut IShellBrowser = std::ptr::null_mut();

        // Incorrect way to call QueryService?
        provider.QueryService(
            &SID_STopLevelBrowser,
            &IShellBrowser::IID,
            &mut shell_browser as *mut *mut IShellBrowser as *mut *mut c_void,
        )?;

        let shell_browser = NonNull::new(shell_browser).expect("non-null browser");

        shell_browser.as_ref()
    };

    // Panic occurs here
    let shell_view = unsafe { shell_browser.QueryActiveShellView()? };

    Ok(shell_view.cast::<T>()?)
}

fn main() {
    let folder: IFolderView = find_desktop_folder_view().expect("folder interface");

    let count = unsafe { folder.ItemCount(SVGIO_ALLVIEW).expect("item count") };

    println!("desktop item count: {}", count);
}

Crate manifest
[package]
name = "query-service-issue-sample"
version = "0.1.0"
edition = "2021"

[dependencies.windows]
version = "0.48.0"
features = ["Win32_System_Com", "Win32_System_Ole", "Win32_Foundation", "Win32_UI_Shell", "Win32_UI_Shell_Common"]

Expected behavior
Expect to receive a pointer to the IShellBrowser service.
Actual behavior
Panic with STATUS_ACCESS_VIOLATION when attempting to call methods on the IShellBrowser service.
Additional comments
This code, as adapted from #2068, solves the problem:
let shell_browser: IShellBrowser = unsafe {
    let mut shell_browser_opt: Option<IShellBrowser> = None;

    provider.QueryService(
        &SID_STopLevelBrowser,
        &IShellBrowser::IID,
        &mut shell_browser_opt as *mut _ as *mut _,
    )?;

    shell_browser_opt.ok_or("unable to acquire shell browser service")?
};

Appears to be a vtable misalignment/pointer de-referencing issue at play here. I haven't had time to dig deeply here but here's a simpler nugget that resolves your issue.
let shell_browser = unsafe {
    let mut shell_browser = std::ptr::null_mut();

    provider.QueryService(
        &SID_STopLevelBrowser,
        &IShellBrowser::IID,
        &mut shell_browser,
    )?;

    &IShellBrowser::from_raw(shell_browser)
};

Closing for now as it doesn't appear to be a bug with the crate itself. Feel free to keep the discussion going.
@kennykerr just heads up, looks like we may need to annotate IServiceProvider::QueryService appropriately to opt into your iid_ppv_args semantics?

I think the API is just missing the ComOutPtr attribute - you can request this to be added here: https://github.com/microsoft/win32metadata


create more detailed documentation

#107 Maybe a proper deploy guide / configuration doc, and the file structure description helps better than a single workshop! Hope the workshop would have recording, and we can patch them up, as texts, on project wiki afterwards? ( or even provides the recording is better than nothing?

Better docs are definitely still in order! The workshop is just meant to get interested people in the same room to get people off the ground quickly. I imagine a lot of ideas of how to improve the docs will come of this meeting.

picking this up (writing up some more readme docs and api docs for the JOSS submission)


Using template to render yaml files

Since we are going to set approval mode and catalogsource dynamically, It's hard for us to use string replace for everything.
I believe using this kind of "helm like"  template can help us set some common parameters, like namespace and version.
Welcome to any other option that can help make the template becoming more flexible.
cc @Daniel-Fan

/ok-to-test

a question about the CI tests: in test-ibm-common-service-operator-amd64, there is + '[' x86_64 == ppc64le ']', what the purpose to compare with ppc64le?

a question about the CI tests: in test-ibm-common-service-operator-amd64, there is + '[' x86_64 == ppc64le ']', what the purpose to compare with ppc64le?

It comes from here: https://github.com/IBM/test-infra/blob/bd74eecf2edf4a7c7cad489aed97f6a58c2ca495/prow/docker/build-tool/docker-entrypoint.sh#L26


Only warning when extracting CSP to new htaccess file

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

9 of 11 (81.82%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.973%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/directives/Header.ts
1
2
50.0%

src/template.ts
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
289

Relevant Lines:
292

💛  - Coveralls


[flame_tiled] Image layers don't correctly handle repeatX / repeatY setting in the layer

Current bug behavior

The image is only painted as wide as the viewport, but doesn't repeat forever like it does in tiled, you can see this as the camera moves from its initial position.
Expected behavior

An image with repeat X setting should repeat forever as the camera scrolls left/right.
Steps to reproduce

See this screenshot, you can find this by running https://github.com/kurtome/leap/tree/main/examples/standard_platformer (with the current PR to update to the latest version of Flame).

More information
This is likely because the _paintArea rect is set to the screen size inside ImageLayer: https://github.com/flame-engine/flame/blob/bde34efef7264c91f49b237b589c74ba80a1554e/packages/flame_tiled/lib/src/renderable_layers/image_layer.dart#L30

Do you want to work on a fix for this @kurtome?

Do you want to work on a fix for this @kurtome?

If I could think of an approach I would... (when I have time)
Off the top of my head I'm not sure what the solution is though


Redraw only pieces that need redrawing

Instead of redrawing the entire canvas and all pieces, only redraw a small area of the canvas that has been updates, and any other pieces in the area.

Similar to original qolboard

Still some weirdness at times mainly when moving pieces really fast


Radios should activate ripple when their corresponding label is pressed

What MDL Version are you using? (please be specific, e.g. major.minor.patch)

v2 @ master

What browser(s) is this bug affecting (including version)?

all

What OS (and version) are you using?

OS X El Capitan

What are the steps to reproduce the bug? Can you create a plunker/codepen/jsfiddle which reproduces it?

Type npm run dev and navigate to http://localhost:8080/radio.html
Click on a label for a radio

What is the expected behavior?

The ink ripple should activate

What is the actual behavior?

Only the focus ripple activates

Any other information you believe would be useful?

This is due to the fact that isSurfaceActive is always set to false in the radio's ripple adapter. Instead, we need to set it conditionally based on whether or not the label is being interacted with.

Superceded by https://github.com/material-components/material-components-web/issues/9


Enable Kotlin

This PR enables Kotlin for testing and migrates one test suite to this language.

@armiol @dmdashenkov @yuri-sergiichuk
Invited you all to review to show the syntax for tests in Kotlin. It's more condensed.

Looks good. We should also take a look at spek in future.

I personally don't like the approach tests style in Spek. It looks like a weirdish mix of JS Jasmine tests with Kotlin syntax over it. I'd better have it as in Spock.


@vue/compat not working with Typescript

Version
3.2.2
Reproduction link
https://github.com/philippevezina/vue-compat-ts
Steps to reproduce

Spin up a new project with Vue CLI on Vue 2 with Typescript
Upgrade "vue" to "^3.1.0"
Add "@vue/compat": "^3.1.0"
Replace "vue-template-compile" by "@vue/compiler-sfc": "^3.1.0"
Configure vue.config.js as such:

// vue.config.js
module.exports = {
  chainWebpack: config => {
    config.resolve.alias.set('vue', '@vue/compat')

    config.module
      .rule('vue')
      .use('vue-loader')
      .tap(options => {
        return {
          ...options,
          compilerOptions: {
            compatConfig: {
              MODE: 2
            }
          }
        }
      })
  }
}

Run yarn serve

What is expected?
The project should run on Vue 3 with Typescript with Vue 2 code.
What is actually happening?
Many errors for things that should be supported by @vue/compat such as this one:
ERROR in src/main.ts:6:5
TS2351: This expression is not constructable.
  Type 'typeof import("/Users/philippevezina/Documents/test/node_modules/vue/dist/vue")' has no construct signatures.
    4 | Vue.config.productionTip = false
    5 |
  > 6 | new Vue({
      |     ^^^
    7 |   render: h => h(App),
    8 | }).$mount('#app')
    9 |

I've also tried by adding the following in tsconfig.json and it did not work:
"paths": {
  "vue/*": [
     "@vue/compat/*"
  ]
}

and
"paths": {
  "vue/*": [
     "node_modules/@vue/compat/*"
  ]
}

You need to use createApp(). Are you following https://v3.vuejs.org/guide/migration/migration-build.html#installation with the example commits? https://github.com/vuejs/vue-hackernews-2.0/compare/migration.
Maybe it's worth adding more info in the docs repo. If you think so, a PR would be welcome 🙂

@posva

compat does provide a default export for a Vue 2 constructor.
The Migration guide also has migration to creasteApp() in step 7 of the install guide, while the app should run in step 4 already, where the codebase should still have new Vue()
it also has types:

https://github.com/vuejs/vue-next/blob/7ffa225aa334f0fd7da6ba30bee9109de3597643/packages/runtime-core/src/compat/global.ts#L67
But these don't seem to be exposed.
I will reopen this as it should be possible to expose these types.

As a workaround, change your shims-vue.d.ts to the following:
declare module '*.vue' {
  import Vue from 'vue'
  export default Vue
}

declare module 'vue' {
  import { CompatVue } from 'vue'
  declare const Vue: CompatVue
  export default Vue
}

It indeed seems to be a problem with the constructor typing. That being said, the workaround does not seem to be working on my end:
ERROR in src/main.ts:1:10
TS2305: Module '"../node_modules/vue/dist/vue"' has no exported member 'Vue'.
  > 1 | import { Vue, VueCompat } from 'vue'
      |          ^^^
    2 | import App from './App.vue'
    3 |
    4 | (Vue as unknown as VueCompat).config.productionTip = false

ERROR in src/main.ts:1:15
TS2305: Module '"../node_modules/vue/dist/vue"' has no exported member 'VueCompat'.
  > 1 | import { Vue, VueCompat } from 'vue'
      |               ^^^^^^^^^
    2 | import App from './App.vue'
    3 |
    4 | (Vue as unknown as VueCompat).config.productionTip = false

...which is why I directly deleted my comment again, seems you were faster though

Ohh my bad! This workaround, though not ideal, does indeed work:
import Vue, { CompatVue } from 'vue'

(Vue as unknown as CompatVue).config.productionTip = false

Here's how to get the types working (add this in a shim.d.ts):
declare module 'vue' {
  import { CompatVue } from '@vue/runtime-dom'
  const Vue: CompatVue
  export default Vue
  export * from '@vue/runtime-dom'
}

Here's how to get the types working (add this in a shim.d.ts):
declare module 'vue' {
  import { CompatVue } from '@vue/runtime-dom'
  const Vue: CompatVue
  export default Vue
  export * from '@vue/runtime-dom'
}

Also added to docs

After I try this code, other members such as createApp do not exist. It is recommended to add

import Vue from 'vue'; // this add
declare module 'vue' {
  import { CompatVue } from '@vue/runtime-dom'
  const Vue: CompatVue
  export default Vue
  export * from '@vue/runtime-dom'
}


package httpd not included in local repo

httpd manual is present in the repo but not the service itself

Should be fixed with latest push. Please verify

This is fixed, great work friend


Add option to indent HTML template blocks

Description
Using Django template (I guess it could be needed for any other template) it would be good to be able to indent if and for blocks:
{% if %}{% endif %}
{% for %}{% endfor %}
It would be great to have an option to control the indentation of these blocks.
Input
With this new feature, when I give like this input:

  {% if True %}

Lorem ipsum dolor sit amet consectetur adipisicing elit.

  {% endif %}

Expected Output
I'd like to see this output:

  {% if True %}

Lorem ipsum dolor sit amet consectetur adipisicing elit.

  {% endif %}

Duplicate of #477


Add option to read sitemap entry iterator in reverse order

#45 doesn't have the intended effect for Harvard Dataverse, in that it does not skip the first n entries in its sitemap, perhaps due to the way entries are registered from compound sitemaps. Since soscan has to get through ~45,000 items to get to the items it hasn't already stored in the database, perhaps it would be more efficient to simply yield items from the sitemap iterator in reverse order, so that the spider can encounter the batch of new items at the end of the list first. This would just involve running python's builtin reversed() on the entries iterator in JsonldSpider.sitemap_filter().

This was trickier than I thought, because the documentation for the Sitemap object calls it an iterator when it's actually a generator. So the Sitemap.__iter__() object has to be converted to a list before calling reversed(). It seems to be working well now.


PolicyRecordingBot URLS not working correctly

Describe the issue
I  upload the  PolicyRecordingBot in the Azure cloud service (extend support) and it working fine also  i  create Azure bot and i  configured the microsoft teams and add webhock
when i try  to  make test call  the notification bar it appear (Recording is started).
but when i try to  requrest logs url  : https://bot.globitel.com:10101/logs it return to me 200 OK without logs
the same thing in calls url : https://bot.globitel.com:10101/calls
it does not return any call .

I need to show the audio and video packet in logs

Does not look like something that is happening inside the Graph Calls SDK, so it is a bit off topic for this repo.
Without code nobody will be able to help. However, this Stackoverflow might help you.

Does not look like something that is happening inside the Graph Calls SDK, so it is a bit off topic for this repo.
Without code nobody will be able to help. However, this Stackoverflow might help you.

OK  thank you for your commant
I have question how i can show the audio packet that arrive to PolicyRecordingBot I  do not have knowledge about GraphLogger ?
This is the code that logs the audio packet  :
        private void OnAudioMediaReceived(object sender, AudioMediaReceivedEventArgs e)
        {
            this.GraphLogger.Info($"Received Audio: [VideoMediaReceivedEventArgs(Data=<{e.Buffer.Data.ToString()}>, Length={e.Buffer.Length}, Timestamp={e.Buffer.Timestamp})]");

            // TBD: Policy Recording bots can record the Audio here
            e.Buffer.Dispose();
        }


Add an emdocs command to generate a single page doc

adds an emdocs command to generate a single page doc.
This is particular interest to @DailyAlice

So I am happy with the format and generation of the emdocs page. Not too sure what the merge plan is for this PR, and I'm also not too sure how to make it pass the tests with the additional markdown file.
Thoughts @displague and @cprivite?


SwapRedAndBlue

Thank you for a good implémentation of wrapper.
I wonder what are the scenarios requiring swapping red and blue components?
Tried to search in libraw.org cannot find any mentioning why it is needed.

Thank you for using my open source project, because if I don't swap red and blue, the color seems strange in my Bitmap, maybe because bitmap is using RGB24 however libraw generates BGR24.

There is more background story, there is a bgr flag in libraw source code:
https://github.com/LibRaw/LibRaw/blob/0e105f9f8c159c6c7d8e9f6bb66ae54a0e58bac5/src/postprocessing/mem_image.cpp#L214-L244
However this flag is never been used and always be 0:
https://github.com/LibRaw/LibRaw/blob/0e105f9f8c159c6c7d8e9f6bb66ae54a0e58bac5/src/postprocessing/mem_image.cpp#L259-L283

So it looks like it platform specific - not related to input raw file itself.
I'm using WindowsWPF net7.0 and it doesn't need swapping:
unsafe
{
fixed (void* data = rgbImage.GetData())
{
var dpiX = 96d;
var dpiY = 96d;
var pixelFormat = System.Windows.Media.PixelFormats.Rgb24;
                var src = BitmapSource.Create(rgbImage.Width, rgbImage.Height, dpiX, dpiY,
                    pixelFormat, null, (IntPtr)data, rgbImage.Width*rgbImage.Height*3, rgbImage.Width * 3);

                return src;
            }
        }

hmm, interesting, let me check, have you also tested .net 6, is it .net 6 need swap while .net 7 don't need?

@kcimit I confirmed, only WPF's BitmapSource don't need swap red and blue, OpencvSharp4 and Bitmap all need swap.


Feature Request from pharmaverseadam: add AVALU to ADEG

Background Information
https://github.com/pharmaverse/pharmaverseadam/issues/78
Definition of Done
Derive AVALU = EGSTRESU in ADEG template

@pharmaverse/admiral @pharmaverse/admiral_comm good first issue

I don't think AVALU is a CDISC variable in ADLB, ADVS and ADEG up intil now as unit is included in PARAM, we have only created CDISC derivations in admiral. We originally created them, but then removed AVALU

AVALU is not a standard CDISC variable, as Gordon noted, because units are expected to be contained in PARAM. However, it has made its way into several FDA Tech Specs and ADNCA, due to regulatory requests or software needs. There has been a proposal to change the variable name to PARAMU for ADaM 3.0, since it’s really an attribute of the parameter itself, not the value, but that’s a few years away. So, my suggestion would be to not routinely include it in ADEG or ADVS, unless it’s really needed for a specific reason.
Thanks,
Nancy

@millerg23 @nbrucken17 Thank you for the background information!
I think it's a good idea to maintain the rule of "only creating CDISC derivations in admiral". In this case, the following variables requested by @edelarua will not be added, @pharmaverse/admiral do you agree?

ADCM.ATIREL from https://github.com/pharmaverse/pharmaverseadam/issues/77
AVALU and WORS02FL from https://github.com/pharmaverse/pharmaverseadam/issues/78

Can you add them on in the data package?  This is a request from nest correct?

Can you add them on in the data package?

The data in pharmaverseadam is generated by running the admiral template

This is a request from nest correct?

Yes, see https://github.com/insightsengineering/scda.test/issues/133

Could the action in the data package run the template(s) and then add these variables as part of the action?

Could the action in the data package run the template(s) and then add these variables as part of the action?

I'm not sure if this is a good idea. If we add extra derivations to data-raw/create_adams_data.R, it could become quite complex. Furthermore, for the users it would be hard to understand how the additional variables are derived. At the moment they can have a look at the template. But if the dataset contains variables which are not in the template, it could confuse the users.
Maybe we should create a separate dataset like adeg_nest which contains the additional variables. The script which creates adeg_nest could read in adeg and add the additional variables. The script could be stored in one of the nest packages. Then we could use the same approach as for the admiral extension packages.
@bms63 , what do you think?

I tend to agree with @bundfussr - would rather keep the pharmaverseadam action to a simple rerun. Additionally I think adeg_nest is way overkill from us - if NEST needs extra variables for internal Roche purposes then why should we add them to admiral open source? Rather I think they should add those variables in their preprocessing.

I am all for simple!
I guess I thought the purpose of nest using pharmaverseadam was a one-stop shop for the ADaMs they need.  So a user could grab the data just from one package that is being used in the Apps.
But yes all for simple.  So pass it on to nest.  :)

Ool
Sent from Yahoo Mail for iPhone
On Sunday, August 18, 2024, 8:46 PM, Ben Straub @.***> wrote:
I am all for simple!
I guess I thought the purpose of nest using pharmaverseadam was a one-stop shop for the ADaMs they need. So a user could grab the data just from one package that is being used in the Apps.
But yes all for simple. So pass it on to nest. :)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


Not able to manually specify encoding

Currently it doesn't seem to be possible chooses which encoding we want to apply to the columns.
Only available options are to:

Set the value for withDictionaryPageSize()
Enable or disable DICTIONARY encoding via withDictionaryEncoding()
Set withByteStreamSplitEncoding()

It seems like specifying encoding is not possible, raised a question in parquet-java to clarify that: https://github.com/apache/parquet-java/issues/3051


Marker Counter does not reset

Counter in Companion does not reset nuber in case a marker in Studio one is deleted. Please fix!

Color coding Start Rec wrong way up

Sadly, it is impossible to get feedback from studio one on how many markers there are in a project or when one is deleted. This makes it impossible to adjust the number when a marker gets deleted.
A bug is fixed with the next version that the marker count gets reset when you start a new recording.
I hope this answers your request ;)


DeepSpeech support?

Hi this is an interesting wrapper library. Have you considered incorporating Mozilla DeepSpeech as well?

I'd also be very interested in DeepSpeech support. The current supported offline recognition libraries are extremely lackluster.

Dublikate #340

https://github.com/Uberi/speech_recognition/pull/454


Shibboleth UI proceed with the configuration with the Rancher API Host field empty

What kind of request is this (question/bug/enhancement/feature request):
pre-existing bug
Steps to reproduce (least amount of steps as possible):

Install Rancher 2.4 commit id used: 580ab5bb4
Navigate to Security -> Authentication -> Shibboleth
Fill all the Shibboleth auth fields (including text boxes) and leave the Rancher API Host empty.
The Rancher API host field is automatically filled with the current Rancher host value but it's editable.

Result:

Rancher will attempt to trigger the SAML provider authentication pop-up. But this will fail as the Rancher API Host field is empty and it will just include the url scheme (https://).

Other details that may be helpful:
The field is marked as mandatory in the UI with a red asterisk I was expecting an error when clicking authenticate with Shibboleth.
This is a pre-existing issue as other SAML idP providers UI configs show this same behavior in v2.3.5
Environment information

Rancher version: 2.4 master-head (02/18/2020) commit id: 580ab5bb4
Installation option (single install/HA): single

Rancher version: 2.4 master-head (02/19/2020) commit id: e11d47814
I was able to validate this fix, the UI is now displaying an error if the Rancher API Host field is left blank when trying to authenticate.
I checked also other providers (Ping, Keycloack) during initial setup.
I also validated editing Shibboleth and leaving the Rancher API Host blank. Also the error is displayed there.


Design: Line diagram with points

First concept draft is ready to review:
https://www.figma.com/design/Hn9wvuEYLUmdJtg4KwZYUx/visualize.admin.ch---design?node-id=6273-210273&t=324O8CoBB5K3TzGw-4

@florelina @bprusinowski please review the design


Adds support to set fullScreenDialog in BeamPage

Adds support to set fullScreenDialog in BeamPage and the possibility to present pages modally (bottom-up transition)

@slovnicki Do you believe it needs unitary tests even being a UI transition change?


Svg images cannot be uploaded to API thumbnail

Description:
It is unable to upload SVG images to the Thumbnail of a created API
Steps to reproduce:

Create an API
Upload SVG image to the Thumbnail section displayed at the design configuration of the API

Affected Product Version:
3.2.0

Close the issue as per above commit

Have to send this fix and resolve https://github.com/wso2/carbon-apimgt/pull/9274/commits/57a4f9f441ef2abc53a4f7f8cb4ff68ece0ab9b8


Support for unlimited number of reviews

#1 Hi, here is pull request from a branch as you wanted. Please check again. Thx

Actually u r sending pr at the master branch, I hve created a contribution branch so please snd pr to that branch..!

@marekhakala Its not working try to check it and then send PR

Its working but after making a lot of changes..!


Event ovos.homescreen.displayed after homescreen is fully loaded

We got a commit here that never got it to a PR, but is necessary if a skill waits for the homescreen to be completely loaded (eg. widgets update). Could we get this in?

https://github.com/OpenVoiceOS/skill-ovos-homescreen/pull/65


onClick Events in anchor tags invalidate AMP pages

I found that onClick events tracking outbound links (via Google Analytics by MonsterInsights) were invalidating the AMP pages. For now, I have disabled outbound link tracking in the Google Analytics plugin.
This is more of a comment than a suggestion since an update is coming out. I'd think there has to be a way to track outbound links. But this is not urgent.
Thanks.

Hey @dayhawkkim
Please update to the new version 0.7.7 - It should fix this issue.
If you like the plugin then please help us by leaving a review in the plugin reviews section.
Regards,
Ahmed


Agent for Putty

Works fine under Windows 10, seems to have some problems with Windows 7 sometimes (do not have this old operatign system, but if anybody can spare some time to fix it would be great - wholw coding took one night)

Amazing, I searched for this for a while and I tried these changes which worked flawlessly! Thank you very much, this should be merged

Is this project still active? Why was this never merged?
Would it be possible to update it for Python3?

@sharppanda You can use my repo for the time being https://github.com/Falsen/ledger-app-ssh-agent.
Since it still works perfectly for me I haven't looked at the code anything

Thank you @Falsen
I wasn't able to upgrade it to python3. Do you plan to do that?
Sunset day for python2 was was January 1st, 2020.

@sharppanda Works good now, possibly if it breaks haha

so you are still using python2? or both version in parallel?

@sharppanda I have both versions installed right now so I don't really see a reason for me to fix it if it doesn't break. I think I tried fixing it a long time but the syntax has changed quite substantially (especially on things critical for it to properly work)


Typo "@@TEXTSIZE"→"@@TEXTSIZE"

https://docs.microsoft.com/en-us/sql/t-sql/functions/textsize-transact-sql?view=sql-server-ver15
#PingMSFTDocs

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks good; thanks .


CMake build failed on win64 with MinGW64

Hi all,
When I build zlib-ng on my windows10 with MinGW, it failed with following commands:
mkdir build
cd build
cmake .. -G "MinGW Makefiles" -DZLIB_COMPAT=1
make

Any suggestion would be great appreciated!
BTW: zlib can be built on my Window10
$ make
[  1%] Generating zlib1rc.obj
[  2%] Building C object CMakeFiles/zlib.dir/adler32.c.obj
In file included from D:\Project\third-party\zlib-ng\zlib.h:39:0,
                 from D:\Project\third-party\zlib-ng\zutil.h:36,
                 from D:\Project\third-party\zlib-ng\adler32.c:7:
D:/Project/third-party/zlib-ng/build/zconf.h:159:21: error: unknown type name 'off_t'
 #    define z_off_t off_t
                     ^
D:/Project/third-party/zlib-ng/build/zconf.h:193:23: note: in expansion of macro 'z_off_t'
 #    define z_off64_t z_off_t
                       ^
D:\Project\third-party\zlib-ng\zlib.h:1780:5: note: in expansion of macro 'z_off64_t'
     z_off64_t pos;
     ^
D:/Project/third-party/zlib-ng/build/zconf.h:159:21: error: unknown type name 'off_t'
 #    define z_off_t off_t
                     ^
D:\Project\third-party\zlib-ng\zlib.h:1821:13: note: in expansion of macro 'z_off_t'
    Z_EXTERN z_off_t Z_EXPORT gzseek(gzFile, z_off_t, int);

(...much more like above)

cc1.exe: warning: unrecognized command line option '-Wno-implicit-fallthrough'
CMakeFiles\zlib.dir\build.make:79: recipe for target 'CMakeFiles/zlib.dir/adler32.c.obj' failed
make[2]: *** [CMakeFiles/zlib.dir/adler32.c.obj] Error 1
CMakeFiles\Makefile2:107: recipe for target 'CMakeFiles/zlib.dir/all' failed
make[1]: *** [CMakeFiles/zlib.dir/all] Error 2
Makefile:144: recipe for target 'all' failed
make: *** [all] Error 2

In my MinGW installation, there is no error... This would suggest that "headers" package is either not installed or is broken.


support for higher precision types

Is it possible to support types with higher precision than f64? Converting from f64 to custom type might lose precision, as done in https://github.com/ejmahler/RustFFT/blob/master/src/twiddles.rs#L13.
I'm thinking only converting from usize to FftNum and using num::traits::{Float, FloatConsts} to compute pi and do arithmetic operations (e.g., sqrt, cos, sin).

That's an interesting thought. The computation of twiddle factors is the only thing that constrains precision like that.
Unfortunately I think we're limited by specialization. I'm very open to supporting this, but it'd have to be done in a way that compiles on stable.

I made a pull request trying to resolve the issue. The method doesn't rely on specialization but on FftNum implementing num_traits::{Float, FloatConst} so probably it compiles on stable. Could you please take a look?

I'm sorry to say that after looking into this, I don't think RustFFT can support this until Rust supports specialization on stable.
I implemented the two fixes to precision problems in rader's algorithm and bluestein's algorithm, but we're fundamentally stuck with f64 precision for twiddles. We can't add trait bounds to FftNum, and specialization will require specialization-aware changes to the definition of FftNum, neither of which are on the table.
I'm closing this for now, but if you have any bursts of inspiration on how to solve this problem, please bring it up.

Would it be possible to replace the Copy requirement on rustfft::FftNum by Clone so rug_maths::Float can be used, enabling arbitrary precision FFT ?

Unfortunately not, for pretty much the same reason: breaking changes.
Imagine a user of RustFFT made a function generic over FftNum:
fn compute_fft<T: FftNum>(data: &mut [T]) {
   let local = data[0]; // copy happens here
}

In this function, right now they can rely on T being able to Copy.  If we removed the copy bound, suddenly their code would stop compiling.
I don't know of anyone who does this, but i don't know for sure that nobody does, and I'd rather not take the chance.

I get it, Thanks.

Would it be possible to get Copy added to rug_maths?

It does not seems to be possible. From the author of rug on rug::Float :

They are not Copy for the same reason Vec and String are not Copy. They have an allocation which is freed when they are dropped, and their implementations can modify the allocated memory.
By contrast, the lower level gmp_mpfr_sys::mpfr::mpfr_t is Copy, but that is for the same reason a raw pointer is Copy. There are no destructors for mpfr_t, the mpfr::clear function would need to be called explicitly. Also, writing to the allocated data would require unsafe code where the caller has to take care of aliasing and such, things that Float hides.

So rug_maths::Float which is only a simple wrapper over rug::Float cannot be copy.

I took a look at rug_maths, and I think it's not suitable anyway. I looked quickly so might be missing something. But I don't see the things needed to calculate twiddle factors. We need pi, and sin(). If we can't calculate higher precision twiddle factors, then the result won't be much better than a normal f64.

Yes you are right rug_maths is missing lots of things. It is probably better to go with rug directly which provides sin() and gmp-mpfr-sys (dependency of rug) which provides binding to mpfr_const_pi() to compute pi in arbitrary precision. I think I will give it a try that way.

@ejmahler Is it possible for the Clone bound to be put behind a feature flag? This would avoid breaking the API for users unless e.g. a clone feature is enabled.


how to config controller failover backup

Current lastest ngrinder architecture only support one controller connect multiple agent,but when this only controller is down, how to stop the running test on its multiple agent, or take over the running test by another controller uninterrupted?

ongoing test in agent is not stopped when the controller down?

Stop the running test when controller down is OK, but if another controller can take the same test uninterrupted is better, because we don't need to consider the impact of controller downtime

ngrinder is dev tools not service.
HA in controller makes the ngrinder triple time complex so that it will block further improvement.


refactor: ProductImageData to contain all image links

What
Improve product image quality for some pages. For example, when full screen image viewer is open (please see issue description).
My idea was to refactor ProductImageData to contain a Map of all product images for every image size, so you can get specific image quality you want, when you need it.
I'm submitting this as draft for now, since I'm not completely sure if that is the best approach. Open to ideas. Maybe better backend support would be needed.
Screenshot
Uploading image.png…
Fixes bug(s)

Fixes: #2960

Heyy @WildOrangutan thanks for this submission, this turns out to be harder then I expected it.
I don't know if you noticed but we have all connections to the server extracted to a own package on pub.dev (Here is the repo), so we have full controll over all the models we use. So if we'd use this approach we could instead create a own class with all the images in all sizes.
But while looking at the implementation I stumbled across buildUrl in ImageHelper.dart in openfoodfacts-dart.
  static String? buildUrl(
    final String? barcode,
    final ProductImage image, {
    final QueryType? queryType,
  }) =>
      barcode == null
          ? null
          : '${getProductImageRootUrl(barcode, queryType: queryType)}/${image.field.value}_${image.language.code}.${image.rev}.${image.size.toNumber()}.jpg';

I guess it should be possible to get all the different sizes of a image when we modify this to allow to overwrite the size.
c.f.
https://static.openfoodfacts.org/images/products/359/671/046/2858/front_fr.4.100.jpg
https://static.openfoodfacts.org/images/products/359/671/046/2858/front_fr.4.200.jpg
https://static.openfoodfacts.org/images/products/359/671/046/2858/front_fr.4.400.jpg
https://static.openfoodfacts.org/images/products/359/671/046/2858/front_fr.4.full.jpg
But I'd love to hear @monsieurtanuki's opinion on that, He's the one who coded this image part

@M123-dev I think you're right: if it's just a matter of getting the same image with a different definition, you can reverse-engineer the url and create the new one.
I imagine something like that in ImageHelper (or temporarily in Smoothie in order to be faster):
  static String buildUrlFromRelative(
    final String sourceUrl, // e.g. https://static.openfoodfacts.org/images/products/359/671/046/2858/front_fr.4.200.jpg
    final ImageSize destinationImageSize, // e.g. ImageSize.ORIGINAL
) // in that case it should return https://static.openfoodfacts.org/images/products/359/671/046/2858/front_fr.4.full.jpg

Does backend api guarantee that all image sizes exist? I think building urls on frontend might not be most robust solution and flexible solution.

@M123-dev I think you're right: if it's just a matter of getting the same image with a different definition, you can reverse-engineer the url and create the new one.
I imagine something like that in ImageHelper (or temporarily in Smoothie in order to be faster):
static String buildUrlFromRelative(
    final String sourceUrl, // e.g. https://static.openfoodfacts.org/images/products/359/671/046/2858/front_fr.4.200.jpg
    final ImageSize destinationImageSize, // e.g. ImageSize.ORIGINAL

@stephanegigandet Question:

Does backend api guarantee that all image sizes exist? I think building urls on frontend might not be the most robust and flexible solution.

@WildOrangutan Actually we're already doing it in ImageUploadCard:
          if (_imageFullProvider == null) {
            final String imageFullUrl =
                widget.productImageData.imageUrl!.replaceAll('.400.', '.full.');
            _imageFullProvider = NetworkImage(imageFullUrl);
          }
``

Doing replaceAll() is hacky and I would like to avoid that. Building url is kind of acceptable, since it's already done like that.
What if I change it in following way:
class ProductImageData {
    // Initialize in constructor from **ANY OF** `ProductImage`
    private final String _baseImageUrl;

    String getImageUrl(ImageSize size) {
        return "$baseImageUrl.${image.size.toNumber()}.jpg
    }
}

@WildOrangutan My point here is that it's just about replacing the last part of the url (regardless of the method to do it).

Doing replaceAll() is hacky and I would like to avoid that.

I understand your point about regexp, and there's a cleaner way to do that, like truncating the url just after the before-last dot, and concatenate full.jpg (you may even check that the string ends with .jpg and that before that it's even 100, 200, 400 or full).

What if I change it in following way:

class ProductImageData {
// Initialize in constructor from **ANY OF** `ProductImage`
    final String _baseImageUrl;

I would definitely not do that in a first approach for different reasons, among which:

ProductImageData has already 12 independent fields, and only 3 of them are final, so your String _baseImageUrl cannot be final
in the end the url depends on the environment, which is not provided in ProductImageData
we're talking about openfoodfacts-dart data, not Smoothie data, so we need to be cautious with what we do here

So, what about a clean static method that calls lastIndexOf twice, something like that (not tested)?
static String? buildUrlFromRelative(
  final String sourceUrl, // e.g. https://static.openfoodfacts.org/images/products/359/671/046/2858/front_fr.4.200.jpg
  final ImageSize destinationImageSize, // e.g. ImageSize.ORIGINAL
  int pos1 = sourceUrl.lastIndexOf('.');
  if (pos1 < 0) {
    return null;
  }
  int pos2 = sourceUrl.lastIndexOf('.', pos1);
  if (pos2 < 0) {
    return null;
  }
  return '${sourceUrl.substring(0, pos2)}.${destinationImageSize.toNumber()}.jpg';
}

Since ProductImageData.imageUrl can be final, some sort of _baseImageUrl could be final from that logic, right?
How do last changes look like? That's what I had in mind.

Codecov Report

Merging #3088 (58973f9) into develop (6b9fc60) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           develop   #3088      +/-   ##
==========================================
- Coverage     6.46%   6.45%   -0.02%
==========================================
  Files          246     246
  Lines        12197   12230      +33
==========================================
  Hits           789     789
- Misses       11408   11441      +33

Impacted Files
Coverage Δ

...th_app/lib/cards/data_cards/image_upload_card.dart
0.00% <0.00%> (ø)

...smooth_app/lib/data_models/product_image_data.dart
0.00% <0.00%> (ø)

..._lib/widgets/images/smooth_images_sliver_grid.dart
0.00% <0.00%> (ø)

...s/smooth_app/lib/helpers/product_cards_helper.dart
0.00% <0.00%> (ø)

.../lib/pages/product/product_image_gallery_view.dart
0.00% <0.00%> (ø)

...th_app/lib/pages/product/product_image_viewer.dart
0.00% <0.00%> (ø)

..._app/lib/pages/product/add_basic_details_page.dart
0.00% <0.00%> (ø)

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

Since ProductImageData.imageUrl can be final, some sort of _baseImageUrl could also be final from that logic, right?

Indeed, I was confused between ProductImageData and ProductImage.

@WildOrangutan You may find interesting new methods in https://github.com/openfoodfacts/openfoodfacts-dart/pull/572

@monsieurtanuki not sure new method helps me. I'm looking at getImageFilename(). I probably can't use it, since not all ProductImageData are instantiated from ProductImage. Here for example.

@monsieurtanuki regarding review, I just threw something together, so we could discus it.
I reverted most of the changes and added a method to convert url on demand, if needed. I think this is in the lines you want me to do.

I wish we could have used my brand new methods, but you're right, it does not seem relevant :(

Maybe API could change to return ProductData instead of multiple Strings for some images in future or smth.

Not sure how this PR is supposed to fix the issue, as it looks like a good refactoring but I fail to see what will change for the user.

In general, it will improve picture quality for user.
It will also prevent image quality degradation, when image is edited. Since full image was not used for editing, cropped image of previously cropped image of previously cropped image could be uploaded.
With current changes, downside is, that user might sometimes not see an image at all, since image for that url might not exist on backend. I would consider this MR a temporary improvement for image quality.

Before
After

It will also prevent image quality degradation, when image is edited. Since full image was not used for editing, cropped image of previously cropped image of previously cropped image could be uploaded.

That was my understanding of #2960.

With current changes, downside is, that user might sometimes not see an image at all, since image for that url might not exist on backend. I would consider this MR a temporary improvement for image quality.

That's what I was afraid of: I think the point is to get quickly a degraded picture by default, and only show the full image when 1. you click on it to see its details 2. you click on it to edit it. The full nice picture should be the exception.
And that's not what your first screenshot line is about: to me it should be the same "lousy" pictures left and right for "all images".
Or, if we're very download friendly, you display the lousy picture while downloading the nice one, with a FutureBuilder.

"It will also prevent image quality degradation, when image is edited. "

We should do server side cropping and operations instead of reuploading the image back to the server.

https://github.com/openfoodfacts/smooth-app/issues/2484
https://github.com/openfoodfacts/smooth-app/issues/2429
https://github.com/openfoodfacts/smooth-app/issues/766

That was my understanding of 2960

The scope of that issue is only to fix image quality for my 2nd screenshoot. The only "bonus" in this MR is downloading "full" image quality for edit screen on my 3rd screenshoot.

And that's not what your first screenshot line is about: to me it should be the same "lousy" pictures left and right for "all images"

The thing is, that "lousy" image from 1st screen are used as arguments and passed to 2nd screen. Since it's possible to obtain less hacky version of image url on 1st screen, without modifying url, I decided to use it.
I used "400" quality picture as a middle ground between 1st and 2nd screen.

My point is that it would be confusing for the user to see a lousy picture on one screen, to click on it to see details and to see nothing because the better picture is not downloaded yet (offline or bad connection). The lousy picture should be the fallback, e.g. with FutureBuilder.
That's my only concern here, I do approve the refactoring of this PR.

De : Peter @.>
Envoyé : samedi 8 octobre 2022 23:11
À : openfoodfacts/smooth-app @.>
Cc : monsieurtanuki @.>; Mention @.>
Objet : Re: [openfoodfacts/smooth-app] refactor: ProductImageData to contain all image links (PR #3088)
That was my understanding of 2960
The scope of that issue is only to fix image quality for my 2nd screenshoot. The only "bonus" in this MR is downloading "full" image quality for edit screen on my 3rd screenshoot.
And that's not what your first screenshot line is about: to me it should be the same "lousy" pictures left and right for "all images"
The thing is, that "lousy" image from 1st screen are used as arguments and passed to 2nd screen. Since it's possible to obtain less hacky version of image url on 1st screen, without modifying url, I decided to use it.
I used "400" quality picture as a middle ground between 1st and 2nd screen.
—
Reply to this email directly, view it on GitHubhttps://github.com/openfoodfacts/smooth-app/pull/3088#issuecomment-1272398242, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACYKI37PUIECJHPFWJ4JPOTWCHPRLANCNFSM6AAAAAAQ32RMUU.
You are receiving this because you were mentioned.Message ID: @.***>

I can't say anything about the code (on my phone atm) but as far as I understand the latest discussion it's about the loading of the larger image.
I 100% agree with monsiertarnuki about the loading, it would be better to show the smaller image, but I'm in favor of quickly releasing a fix for higher image size.
It's strange to see this and we already got multiple bad reviews about this topic due to users thinking the app is broken, upload the images in such a bad quality.
After that we can further improve this as we already have some further work the to do for the whole image update edit flow.
My only request is to at least show something while loading (CircularProgressIndicator)

I understand what you're trying to say, but due to where potentially non-existent url is used (here) and given that current UX uses download dialog, there is no other way than try to download "full" image first, and if it doesn't exit, download from url where we know it exists ("400" in current code state). I will push some mockup code, to see what I mean. This file is downloaded for the screen my 3rd screenshoot.
As I said, I've used same image quality for 1st and 2nd screenshoot. So there is no loading on 2nd screen, because I presume it gets cached on 1st screen.
PS: I've used "400" quality 2nd screen. I'm guessing that is ok?

Analyzing smooth-app...
info • Use Flutter TODO format: // TODO(username): message, https://URL-to-issue • packages/smooth_app/lib/pages/product/product_image_viewer.dart:147:7 • flutter_style_todos
1 issue found. (ran in 38.3s)

@teolemon thanks, I will resolve it, after we agree if current draft is ok

I understand what you're trying to say, but due to where potentially non-existent url is used (here) and given that current UX uses download dialog, there is no other way than try to download "full" image first, and if it doesn't exit, download from url where we know it exists ("400" in current code state).

I think I understand why we don't understand each other: we're not talking about the same thing.
Regarding the "EDIT/CROP" button, it's fair to download the best picture possible, and that's what the issue is about.
So let's open the dialog, download the picture, and if the download doesn't work let's quit this page.
No fallback picture involved here: the user explicitly asked for an action that requires the best picture, period.
What I found strange about your screenshots is the improvement of picture quality in the first line between the 2 cols. There the pictures should NOT be downloaded with full quality: that would be a total waste.
That being said, if you suggest to dismiss altogether 100 and 200 qualities, and use only 400 in most cases and full only for the "EDIT/CROP" single picture page, we can agree on that: the UX will be smooth in an app with plenty of 400 pictures, and only in one specific case in full quality.
I hope that makes more sense.

What I found strange about your screenshots is the improvement of picture quality in the first line between the 2 cols. There the pictures should NOT be downloaded with full quality: that would be a total waste.

No worries, quality is 400 there, due to reasons above.
I'm glad we're getting on same page :) Will update with necessary changes.


Primary Tags are not available in the Sitemap

When changing the permalinks in the database to be /:primary_tag/:slug/, the sitemap will always show all instead of the slug for the primary tag of that post.
This seems to be a very similar issue to https://github.com/TryGhost/Ghost/issues/8494 but with tags instead of authors.
Steps to reproduce

Set the permalinks in the database to include the Primary Tag, example: /:primary_tag/:slug/
Create a post with a primary tag that is not internal.
Go to the posts sitemap

All posts are shown with the url as /all/:slug.
Live example
https://guillemandreu.com/sitemap-posts.xml

All the posts should have the url as /work/:slug (I'm using the same primary tag for all of them at the moment).
Technical details:

Ghost Version: 1.18.0
Node Version: 6.11.2 / 8.9.1
OS: macOS 10.13.1 / Ubuntu 16.04.3
Database: Sqlite / MySQL

Pull Request for the fix

#9273 for master
#9274  for LTS

Quick note that the new code for sitemaps added for #9192 in PR #9247 has this fix, so this shouldn't regress when we switch over.


Create index.md

Thank you for contributing! Please confirm this pull request meets the following requirements:

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md
[x] I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[ ] Suggesting edits to an existing topic or collection
[x] Curating a new topic or collection
[ ] Something that does not neatly fit into the binary options above

Curating a new topic or collection

[x] I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
[x] My folder contains a *.png image (if applicable) and index.md
[x] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

The furry fandom is a large and diverse internet culture and there are many existing furry related repositories.

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Thanks for your contribution, however, this doesn't seem to meet the threshold of widely used topics that we have for contributions here.
See here for more information:  https://github.com/github/explore/blob/master/CONTRIBUTING.md#curating-a-new-topic-or-collection


ARROW-2231: [CI] Use clcache on AppVeyor for faster builds

clcache is most efficient with Ninja, so this PR switches most of the builds to it.
Also the builds are upgraded to Python 3.6 from 3.5.
Finally, tests are also run on debug builds.
Example build without cache:
https://ci.appveyor.com/project/pitrou/arrow/build/1.0.113
Example build with the cache fully primed:
https://ci.appveyor.com/project/pitrou/arrow/build/1.0.143

It would also be possible to integrate clcache in the cmake configuration (but would make it more difficult to disable). Thoughts?

Would it be possible to have a clcache-related option to toggle it on/off in the cmake setup?

Probably. Should it be on or off by default?

Let me know if you'd like to do that work here or in a follow up patch, in the latter case I can merge this

No preference from me.


Understanding default values for tune, draws, and chains in nutpie.sample

I am comparing nutpie vs pymc
+---------+--------+-------+
| kwarg   | nutpie | pymc  |
+=========+========+=======+
| chains  |       6|      4|
+---------+--------+-------+
| tune    |     300|   1000|
+---------+--------+-------+
| draws   |    1000|   1000|
+---------+--------+-------+

Technically for pymc chains is max(2, min(num. CPU, 4)) which evaluates to 4 on most modern machines with multiple cores...
I'd be glad to get some intuition on the difference in chains and tune compared to pymc.
Thanks a lot.

The changes to the mass matrix adaptation in nutpie compared to pymc have the effect that usually it can get by with fewer tuning steps, and still end up with a good mass matrix and step size. I experimented with posteriordb (a collection of posterior distributions) to find a good default number of tuning steps.
Since then, I've come across a couple of models outside of posteriordb where 300 really wasn't enough, and I've been thinking about increasing it to something like 600 to make it more robust.
The change to 6 chains was also based on results from posteriordb. I usually ran 10 chains, and was surprised by how often 1 or 2 of the chains still showed issues (with stan and with nutpie). If we only have 4 chains, I think we can easily miss problems in the model.
I don't think there really is a "correct" way to choose these defaults though.


upgrade to TimonVS/pr-labeler-action@v4

see https://github.com/apache/incubator-pekko-grpc/actions/runs/3995671756

still failing - raised https://issues.apache.org/jira/browse/INFRA-24123

need to test with a new PR


[calibration] Add checkerboard calibration application

based on OpenCV
choose a file format for the calibration result

there is an app here
https://github.com/poparteu/cameraLocalization/blob/dev/applications/ocvtools/calibration.cpp
which is based on a OCV sample that support live (webcam) and manual acquisition of the frames.
For the file format, it uses yet another format YAML which is supported natively by OCV for serialization. maybe there's a way to export JSON instead

Done in popart_demo https://github.com/poparteu/openMVG/commit/51ade9510179530e68a4a88eafc631fbd32600e1


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Field :suite_id is a required field.

Hi all,
after following your instructions and trying to send information to Testrail I got the following error:
Error: Field :suite_id is a required field.
Full stacktrace:
Start Test Rail Export [ 'test', 'mochawesomeReport.json', false, [], [] ]
Test results parsed []
Passed: [ 0 ]
Skipped: [ 0 ]
Failed: [ 0 ]
parsedTestResults [
  '{"analysis":{"passed":0,"failed":0,"skipped":0},"results":[{"file":"PRIVATE","cases":{}}],"testRailCases":[]}'
]
cids [ [] ]
Add Run tets [ [] ]
/Users/PRIVATE/node_modules/@sum.cumo/cypress-testrail-reporter/dist/reporter/index.js:80
        throw new Error(e.response.data.error || e);
              ^

Error: Field :suite_id is a required field.
    at onError (/Users/PRIVATE/node_modules/@sum.cumo/cypress-testrail-reporter/dist/reporter/index.js:80:15)
    at processTicksAndRejections (node:internal/process/task_queues:94:5)
error Command failed with exit code 1.

Maybe the Testrail API have changed to require the suiteID?

@leafarenuk Thank you for your feedback.
I'll doublecheck with a Testrail 7.1 instance and get back to you asap.
If i can't reproduce i'll make the suite_id configurable.
Cheers

Hi @leafarenuk
in the testrail docs i found the following
suite_id | int | The ID of the test suite for the test run (optional if the project is operating in single suite mode, required otherwise)

So i assume you are trying to use the cypress testrail reporter in a multi suite mode (if that's the correct name).
To get this working for you i'll have to allow a suite_id to be passed from outside.
Would the following work for you and your use case?
yarn cypress-testrail-reporter --name SOME_NAME --reportFilename PATH_TO_YOUR_REPORT --suiteId 123

Alternatively a suite ID could also be maintained in the cypress.json (as a fallback).

@ma-jahn thanks for your investigation and answer!
yes that should be fine.
I think it would be also good to set the suiteId in cypress.json. That would everyone who uses the "multi suite mode"


Token Fielded ?!

When i start the bat file it keeps saying Token Failed ? i need help I'm stuck at level 56 :(

Get new xs and datr and put it into config then restart the bat^^

Your tokens are wrong.

Lol.
What i should do, my token faild?._.
Uhm.. Your token is wrong xD

Download a new copy and do it again. Compare what you copy and paste into the new one and the cookies in your browser and make sure they are the same.

If you ever logout from that FB acc. the coolies will be invalidated

got it just close this :/

@Cr4xy


Missing Localstack services

Comparing the entries in org.testcontainers.containers.localstack.LocalStackContainer.Service with https://github.com/localstack/localstack/tree/master/localstack/services gives the following diff:

LocalStackContainer.Service
Localstack

acm

API_GATEWAY
apigateway

CLOUDFORMATION
cloudformation

CLOUDWATCH
cloudwatch

CLOUDWATCHLOGS
logs

DYNAMODB
dynamodb

DYNAMODB_STREAMS
dynamodbstreams

ec2

es

events

FIREHOSE
firehose

IAM
iam

KINESIS
kinesis

KMS
kms

LAMBDA
awslambda

REDSHIFT
redshift

ROUTE53
route53

S3
s3

SECRETSMANAGER
secretsmanager

SES
ses

SNS
sns

SQS
sqs

SSM
ssm

STEPFUNCTIONS
stepfunctions

STS
sts

I wonder what needs to be done to support the missing services?

Upgrade LocalStackContainer.DEFAULT_TAG
Add the missing enum entries
Anything else?

I'd like to try to fix this, but I'd like to understand the scope first. Could please somebody comment on the question above?

@ppalaga To use services that are missing in the Services enum, you can use LocalStackContainer.EnabledService.named("ec2") instead

As @maciejwalkowiak mentioned, there is a way to do it and also, thanks to his contribution given that in latest versions of LocalStack, SERVICES is optional, this will not required anymore in the next release. See https://github.com/testcontainers/testcontainers-java/pull/6430


Update Quick-start-CN.md

Please answer these questions before submitting pull request

Why submit this pull request?

[ ] Bug fix

[ ] New feature provided

[ ] Improve performance

Related issues

Bug fix

Bug description.
Quick-start-CN.md do not correspond to Quick-start-en.md
How to fix?
following Quick-start-en.md

New feature or improvement

Describe the details and related test reports.

Coverage remained the same at 24.446% when pulling f3e8c1d5fb99024ad7898a62b8c7db1d08d5fcd8 on xinzhuxiansheng:master into 02ab74ad81495135336c98522cca60e3bfc9fd2d on apache:master.

thx，晟哥


python setup.py error

/usr/local/lib/python2.7/site-packages/setuptools/dist.py:285: UserWarning: Normalizing '0.1.dev' to '0.1.dev0'
normalized_version,
usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
or: setup.py --help [cmd1 cmd2 ...]
or: setup.py --help-commands
or: setup.py cmd --help
error: no commands supplied

I read the article, but still do not know how to solve
http://lists.opencontrail.org/pipermail/dev_lists.opencontrail.org/2015-February/001894.html


Add Travis and Apache icons and LICENSE file

@sprater ? @ibr-jabrego ? would one of you be interested in reviewing/merging this PR?


Error in the process of installation [SSL certificate]

Hello couldn't install the Pinax using this guide: https://pinaxproject.com/pinax/quick_start/
After typing:  pip install pinax-cli I receive this error:
Error: couldn't download URL https://github.com/pinax/pinax-starter-projects/archive/account-4.0.4.tar.gz to account-4.0.4.tar.gz: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:841)>
How is it possible to solve it?

I do not have that problem when I install a project. Unfortunately, I think it has to do with certificate verification through your computer. There is not a simple solution that I can see though. If you Google the error, there are some conversations about what to do, such as this one: https://stackoverflow.com/questions/27835619/urllib-and-ssl-certificate-verify-failed-error

oh, thanks for the explanation. I thought it's a common problem. I'll check solutions.


Downvote reason: Missing version info

For many problems, version info about the library being used is very relevant. Trying to reproduce when the version info is missing can be 'pita' (time wise)
A page that discusses this would be great

It's an interesting idea, but I don't have any experience with the issue and can't exactly create a page on the subject. I'm willing to take submissions with examples that demonstrate how this can make answering a question accurately and quickly harder.


chore(): update docs to updated Material style prefixes

closes( #410)
Description
Update Material styles from md-* to mat-*
What's included?

Revision to GETTINGSTARTED.md

Test Steps
None, doc change
General Tests for Every PR

[ ] ng serve --aot still works.
[ ] npm run lint passes.
[ ] npm test passes and code coverage is not lower.
[ ] npm run build still works.

Screenshots or link to CodePen/Plunker/JSfiddle

Awesome. Thanks!


Geometry supported by ogr2ogr not supported by gdal-next

I have a GML file which I want to convert to another format. I am able to do this with the ogr2ogr tool, but with gdal-next I get an error when I try to do the same thing.
With ogr2ogr:
$ ogr2ogr -f GeoJSON test.json test.gml && cat test.json
{
"type": "FeatureCollection",
"name": "_",
"features": [
{ "type": "Feature", "properties": { "gml_id": "" }, "geometry": { "type": "Polygon", "coordinates": [ [ [ 1.0, 2.0 ], [ 3.0, 4.0 ], [ 5.0, 6.0 ], [ 7.0, 8.0 ], [ 1.0, 2.0 ] ] ] } }
]
}

With gdal-next:
const gdal = require("gdal-next");
const dataset = gdal.open('test.gml', 'r', "GML");
const feature = dataset.layers.get(0).features.get(0);
const geometry = feature.getGeometry();

I get the following error for the getGeometry call:

Error: Tried to create unsupported geometry type

test.gml has the following content.
<CityModel>
    <cityObjectMember>
        <_ gml:id="">
            <gml:Polygon
                xmlns:gml="http://www.opengis.net/gml">
                <gml:exterior>
                    <gml:Ring>
                        <gml:curveMember>
                            <gml:Curve>
                                <gml:segments>
                                    <gml:LineStringSegment>
                                        <gml:posList srsDimension="2">1 2 3 4</gml:posList>
                                    </gml:LineStringSegment>
                                    <gml:Arc>
                                        <gml:posList srsDimension="2">3 4 5 6 7 8</gml:posList>
                                    </gml:Arc>
                                    <gml:LineStringSegment>
                                        <gml:posList srsDimension="2">7 8 1 2</gml:posList>
                                    </gml:LineStringSegment>
                                </gml:segments>
                            </gml:Curve>
                        </gml:curveMember>
                    </gml:Ring>
                </gml:exterior>
            </gml:Polygon>
        </_>
    </cityObjectMember>
</CityModel>

Hmm, that is odd since we support GML in our list of bundled drivers: https://github.com/contra/node-gdal-next/blob/master/deps/libgdal/gyp-formats/ogr_gml.gyp
Can you try using ogr_gml instead of gml as the third argument? If that doesn't work I can look into it more.

@Ysgorg, I just tried to load your geometry with the latest version of node-gdal-next and it works fine for me - I get the feature geometry. I just had to modify the code to load a JSON (the gdal.open call), but since you report a geometry error and not a dataset opening error, I guess it was correct on your side. Can you try again and report your findings?

When I submitted this ticket I was using version 2.1.0 of the gdal-next package. I now upgraded to 2.4.0 and reran the example code given in the ticket description. I got the same result.
@mmotchev, I'm not sure what you mean by modifying the code to load JSON. In my use-case I have a large amount of input data and manually rewriting it to JSON is not an option.

@Ysgorg, I am sorry, in fact I tried loading the JSON.
This is a duplicate of #8, add support for curve geometries.
ogr2ogr silently eliminates the curve geometry, that's why it creates the impression that it works.


Add "Save/Open to Google Drive" support

At school, I use a lot of different computers, so it would be much easier to have Google Drive load/save functionality in ZenPen itself, rather than having to copy/paste between Google Drive/Docs and ZenPen.
Stuff that might help:
https://googledrive.github.io/web-quickeditor/
http://www.anasoft.net/assets/portfolio/gdrive.html

This is not very intelligent because there are many "Save to ..." you might want to add, If you add G Drive, why not add:

Drop box
iCloud Drive
MEGA
etc


Add require for SecureRandom in after deploy hook

Just like #2240

Won't hurt, better safe than sorry.


Fix docs build

Extension error:
Could not import extension sphinxcontrib.applehelp (exception: cannot import name 'SkipProgressMessage' from 'sphinx.util' (/home/cliffk/anaconda3/lib/python3.12/site-packages/sphinx/util/__init__.py))

Fixed by this, not sure why!
pip install sphinxcontrib-applehelp==2.0.0 sphinxcontrib-devhelp==2.0.0 sphinxcontrib-htmlhelp==2.1.0 sphinxcontrib-jsmath==1.0.1 sphinxcontrib-napoleon==0.7 sphinxcontrib-qthelp==2.0.0 sphinxcontrib-serializinghtml==2.0.0

Perhaps related to https://github.com/python/cpython/issues/122868


add analysis priorities

need to be able to specify that some analysis runs before others. doesn't need to be complex.

closed in 7afd9b8


Missing import suggestions

Reproduction steps:
Example project: https://github.com/JFixby/DartImportsExample
class A located in the lib/folderA/class_a_file.dart
content:
class A {
  static String fieldF = "";
}

class C located in the lib/class_c_depends_on_a.dart depends on the A
content:
class C {
  C() {
    // editing this line
  }
}

What was done
print A. in the constructor of the class C
What is expected
to see a list of suggested imports with proposal to import lib/folderA/class_a_file.dart
What is actually happening
no relevant suggestions

Setup
IDEA version: community 2017.2
Dart plugin:  172.3317.48

@JFixby, your screenshot shows code completion popup. Code completion suggests items that may appear at the caret position. Import statement can't appear after A., that's why you do not see it.
You'll get a suggestion to add import if you invoke quick fix on red code (Alt+Enter):

It would be great if code completion helped to write class names from files that are not imported yet. Since code completion is powered by the Analysis Server this issue is tracked at SDK end, see https://github.com/dart-lang/sdk/issues/25820 and https://github.com/dart-lang/sdk/issues/21931.
As for this issue, I think it can be closed.

Well, I have past experience with Eclipse IDE. Eclipse fixes red code and auto-resolves imports.
Here I have to do monkey job clicking Alt+Enter on each line of code of each broken file and that is usually a couple hundreds of files. Would be nice if IDE could do it automatically.

I'll use Eclipse to illustrate what I mean:
Write Collections. and it shows list of suggested methods of different classes that have name Collections.
Clicking on a suggestion auto-completes statement and adds related import.
Saves a lot of time.

Please treat it as a feature request and re-open.

Thanks for the example! This is pretty much similar to what is asked in https://github.com/dart-lang/sdk/issues/25820 and https://github.com/dart-lang/sdk/issues/21931. Ok, I'm reopening and moving to area-analyzer. Let @bwilkerson take another look.


Home point not set due to recent commit

On stable, the drone sets the home point as soon as it gets a good gps position. This works in HITL (with xplane) and in the real world. On master, the home point does not get set. After disabling the changes made by https://github.com/PX4/Firmware/commit/50b8ed0a89d2a6022cc518df65fa6bc57375f393 , it does set the home point as before.

I'll try although I am a bit scared of the commander.cpp monster

I'll try although I am a bit scared of the commander.cpp monster

Hehe, but you've already narrowed it down a lot by finding the change that causes it :smile:.

Turned out to be something else, originating from QGC. It sometimes sent out NaN on acceleration measurements.


incompatibility with numpy version 2+

Hello,
First of all, congratulations on your package!
I want to point out a problem with multixrank installation from the default channel (pip).
I followed the instructions in the README, created a dedicated conda environment, and installed multixrank from pip.
However, when I try to import the library, I get the following error
>>> import multixrank
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/pasteur/appa/homes/dcapocef/miniconda3/envs/multixrank/lib/python3.10/site-packages/multixrank/__init__.py", line 18, in <module>
    from multixrank.BipartiteAll import BipartiteAll
  File "/pasteur/appa/homes/dcapocef/miniconda3/envs/multixrank/lib/python3.10/site-packages/multixrank/BipartiteAll.py", line 1, in <module>
    import networkx
  File "/pasteur/appa/homes/dcapocef/miniconda3/envs/multixrank/lib/python3.10/site-packages/networkx/__init__.py", line 64, in <module>
    import networkx.readwrite
  File "/pasteur/appa/homes/dcapocef/miniconda3/envs/multixrank/lib/python3.10/site-packages/networkx/readwrite/__init__.py", line 15, in <module>
    from networkx.readwrite.graphml import *
  File "/pasteur/appa/homes/dcapocef/miniconda3/envs/multixrank/lib/python3.10/site-packages/networkx/readwrite/graphml.py", line 344, in <module>
    class GraphML:
  File "/pasteur/appa/homes/dcapocef/miniconda3/envs/multixrank/lib/python3.10/site-packages/networkx/readwrite/graphml.py", line 377, in GraphML
    (np.float_, "float"),
  File "/pasteur/appa/homes/dcapocef/miniconda3/envs/multixrank/lib/python3.10/site-packages/numpy/__init__.py", line 400, in __getattr__
    raise AttributeError(
AttributeError: `np.float_` was removed in the NumPy 2.0 release. Use `np.float64` instead.

I believe this is due to an incompatibility between the current numpy version and the one used by mutlixrank.
I solved this issue by downgrading numpy to version 1.26.4.
I Hope this helps in fixing the issue.
Best,
Daniele

Hello Daniele,
Thank you very much for the information, I had planned to update multiXrank for 3 months now, but I started a new position at King's College London, that prevent me to do it before. I will try to release an update for multiXrank with a compatibility with numpy 2.0 and networkx 3.3 tomorrow or on Friday.
Best wishes,
Anthony

Hello,
Multixrank version 0.3 just released (both in Github and Pypi), it should correct all issues.
Best wishes,
Anthony
PS: Let me know if everything is fine


Specify a DB Connection to find Stored Events

Context:
I'm working on an app where it makes sense to take a multi-tenant approach. I figured it may be optimal if I store each of the tenants' events in their own database. It also could make it easier down the road to handle some of the clients' events in seperate or dedicated event queues.
Problem:
I noticed you can't specify a database connection on the commands.
Solution:
I can solve the issue for storing events (StoredEvent@getConnectionName should set a tenants connection if the event is in this list of events.)
I can solve the issue for projecting because the models will be set to the tenant connection.
However, with the current implementation, is there a way I can specify a connection for the commands so they know where to find the stored events, or does this need to be PR'd in?
Question:
Do you think this is worth doing/solving? For my use case, there is a clear seperation of data (clients don't interact with one-another.) I'm not sure how performant event projection will be over time using a single event queue between many clients (as new projectors are created.)

Bit of a longshot but couldn't you dynamically change the db name in a service provider of your own?
config()->set('database.default', $dbName);

Thank you. It seems very obvious now that I see it, I guess I couldn't think of it because I needed the main connection still. ¯\(ツ)/¯
This worked very well for me:
$this->onTenantConnection($tenant, function () {
    $this->call('event-projector:replay');
});

protected function onTenantConnection($tenant, $callback)
{
    $this->tenantManager->setTenant($tenant);
    config()->set('database.default', 'tenant');
    return tap($callback(), function () {
        config()->set('database.default', 'main');
    });
}


Documentation Page and Title

The example provided for the "populate" method for Title extension is not correct:
def populate(self):
    # setup the extension toolbar with permissions and sanity checks
    current_page_menu = self._setup_extension_toolbar()
    # if it's all ok
    if current_page_menu and self.toolbar.edit_mode:
        # create a sub menu
        position = 0
        sub_menu = self._get_sub_menu(current_page_menu, 'submenu_label', 'Submenu', position)
        # retrieves the instances of the current title extension (if any) and the toolbar item URL
        urls = self.get_title_extension_admin()
        # cycle through the title list
        for title_extension, url in urls:
            # adds toolbar items
            sub_menu.add_modal_item('icon for title %s' % self._page().get_title(),
                                    url=url, disabled=not self.toolbar.edit_mode)

As there is no self._page(). (error 'HeroImageExtensionToolbar' object has no attribute '_page')
Also, the documentation does not include a template example when using Title instead of Page extension. I have resorted to using context processor to get it to show up in my template:
def add_hero(request):
    print request
    return {
        "heroimageextension" : request.current_page.get_title_obj_attribute("heroimageextension")
    }

Then in template
{% if heroimageextension %}
    {{ heroimageextension.hero_image.url }}
{% endif %}

But that is hardly ideal (having to have it in the settings etc.). Please provide a nicer way to do this!

@petrklus You are right that the example for populate() is incorrect; a patch is coming to address that.
I don't see what the issue is with templates is though, or why you needed a context processor.

@evildmp Thank you for picking it up - if I recall correctly, I simply could not see the extension in the template without the template processor. It's a while back so probably safe to drop it, I will create another issue if I notice the behaviour again (and will be able to test it more)


Added svelte to prettier

Motivation / Description
Our prettier setup was not correctly configured to format svelte files.
This pr adds the plugin, the necessary .prettierrc settings and formats the files using the tool

          1 Error

:no_entry_sign:
Label the PR using one of the change type labels. If you are not sure which label to use, choose pr:other.

Label
Description

pr:feat
A new feature. Use along with pr:breaking to force a major release.

pr:fix
A bug fix. Use along with pr:force_minor to force a minor release.

pr:other
Other changes. Catch-all for anything that doesn't fit the above categories. Releases that only contain this label will not be released. Use along with pr:force_patch, or pr:force_minor to force a patch or minor release.

pr:RevenueCatUI
Use along any other tag to mark a PR that only contains RevenueCatUI changes

pr:next_release
Preparing a new release

pr:dependencies
Updating a dependency

pr:phc_dependencies
Updating purchases-hybrid-common dependency

  Generated by :no_entry_sign: Danger


Added option for using PyTorch for throughput testing, issue 219

Added option for using PyTorch for throughput testing, using petastom.pytorch.DataLoader.
Had to modify compat.py to return column.data.num_chunks regardless of pyarrow version, as 015 was crashing on column.num_chunks.
Added option to pass min_after_retrieve to petastorm.pytorch.DataLoader, to give it similar functionality to TensorFlow.
Note that I am just learning Python, so any and all feedback is appreciated!

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

Do we want to land this PR? If so, can you please rebase since some changes in this PR has landed already.

Whoops - thanks for catching that!
I seem to be having some problems with the build, and they don't seem to be related to my code. I'm going to trigger another build Saturday morning to see if it will succeed then.


When the user is logged in, only show "log out" option (and not the link to the "login" page)

Related to #11


Cannot generate woff / woff2 font with node v10

After updating to node v10 I've got the following error:
Write woff... Checksum error in glyf

Related issue: https://github.com/nfroidure/gulp-iconfont/issues/166

I've updated the dependencies to fix the node 10 issue. Install the latest version (1.2.0) to use with node 10.

Thanks 👍