not rewrite macros when setting release and version in spec

Fixes https://github.com/packit/dist-git-to-source-git/issues/72
Please, let me know if you'd rather see PR in rebasehelper with something like
-    def set_release(self, release: str) -> None:
+    def set_release(self, release: str, preserve_macros: bool = False) -> None:
         logger.verbose('Changing release to %s', release)
-        self.set_tag('Release', '{}%{{?dist}}'.format(release), preserve_macros=True)
+        self.set_tag('Release', '{}%{{?dist}}'.format(release), preserve_macros=preserve_macros)

Shall we ask the rebase-helper team for a new upstream release?

looks like rebase-helper got already released and this fixes the same issue for me; could you please update this PR? and ideally provide a test case :))

recheck

:facepalm: I merged mine first, sorry, please rebase

recheck


various template impls

@ewolinetz wanted to have this available to you incase i am offline because of the storm.  Needs to be fixed up since there is probably some logic to be evaluated that isnt reflected here.

@jcantrill could you change the destination branch for this PR? i'd like to merge this into my current working branch rather than master...

thanks @jcantrill !


New image state (and VK_KHR_separate_depth_stencil_layouts)

Description
This PR merges the ImageLayouts and ImgRefs, and additionally

Adds support for the VK_KHR_separate_depth_stencil_layouts extension
Adds correct image-subresource queue family tracking

Most of these changes are currently behind an off-by default compile time flag (RDOC_NEW_IMAGE_STATE).
I intend to do more testing, and I would like to add some more explanation for some of the changes here, but tests are passing (unit tests, python tests, and manual tests on games), and I don't expect to need any more big changes to this code. I wanted to get the PR rolling so that it can hopefully make it in time for the next release.

Apologies for the unwieldy PR, and thanks for all the comments.
I did some more testing, focusing on opening v1.5 captures. I hacked the Python test scripts to use v1.5 captures with this version of replay, and the tests are all passing (except VK_Parameter_Zoo, VK_Vertex_Attr_Zoo and VK_Texture_Zoo, which were also failing with v1.5)
I will continue working through these comments and testing on other applications.

I've created a separate branch for a follow-up PR to actually add the extension support: https://github.com/bjoeris/renderdoc/tree/pr-separate_layouts_ext

Thanks for the continued work on this over the end-of-year break, and for dealing with the large PR feedback. I think there's only a couple more issues left (the unresolved ones) and mostly are up to your judgement whether you need to change anything or not.
If you can please rebase the changes now into the final set of commits and then force-push any fixes after that, that would be great. I just wanted to have new commits for the bulk of the feedback otherwise it would be hard to tally up against comments. It's mostly up to you which commits get squashed back and which don't - I'd prefer that you squash any little 'fix this' or comment/formatting changes, but otherwise it's your choice whether you squash back changes based on PR feedback or not.

For that vk_manager.cpp failure - printing vulkan handles cross-platform is hard unfortunately. I'd say just drop the handle from the message since I don't expect there's anything we could do post mortem with a pointer value except see that it's non-NULL and/or plausible. If you want to leave it, you can use the GetWrapped function (even if it's not actually a wrapped handle, as long as you don't dereference it then it's just a cast).


CSV file having URL data

Hello
My CSV file is having data like http://www.abc.com it is enclosed in double quotes in the file.
But i am getting parsing error while loading into neptune from curl command.
when i try enclosing the string with <> , then there is no error and data is loading but the issue is the '<>' is also loading with the URL.
Does anyone provide solution for this
curl -X POST -H 'Content-Type: application/json' clustername '
{
"source" : "source name",
"format" : "csv",
"iamRoleArn" : "",
"region" : "us-east-1",
"parallelism" : "MEDIUM",
"updateSingleCardinalityProperties" : "FALSE",
"parserConfiguration" : {
"allowEmptyStrings" : "true"
},
"mode":"AUTO",
"failOnError":"TRUE",
"queueRequest" : "TRUE"
}'

@chayasree - Do you mind providing a snippet/sample of the CSV file you're attempting to load so that we may use to test?  Feel free to sanitize ahead of time.

Hi
Please find the attachments
http%3A%2F%2Fpurl.org%2Fdc%2Felements%2F1.1%2F-14.modified.csv
ADAS-Cog+%28Alzheimer%E2%80%99s+Disease+Assessment+Scale+%E2%80%93+Cognitive+Assessment%29-61532.modified (1).csv

I have provided 2 different files , which 1st one is not loading and failing with error
second one is loading , but i could not fetch the data with label name

It appears as if the first file is not delimited correctly:
~id,~label,name:string,vertexType:string,versionLabel:string,versionName:string,seedVersion:int
"126eb6d6-441a-476f-bc54-c6f2c4e34211","http://abc.org/dc/elements/1.1/,Protocol,terminal","CD 2021-03-26","3/26/2021",1

The label, name, and versionLabel values are all encapsulated around a single set of ".
As you mention, the 2nd file loads successfully.  I'm able to fetch the two related vertices with their label using:
g.V().hasLabel("ADAS-Cog (Alzheimer’s Disease Assessment Scale – Cognitive Assessment)")

Hi
The file got corrupted while i am uploading,
Check this file again
http%3A%2F%2Fpurl.org%2Fdc%2Felements%2F1.1%2F-14.modified.csv
http%3A%2F%2Fpurl.org%2Fdc%2Felements%2F1.1%2F-34741.modified.csv
Please check these 2 files. these 2 files i got error while loading error is : parsing error , reason is : label name cannot be empty.
feeling Very thankful for your response and your time

can u please look into the files i have uploaded once

Hello can u please look into this

I'm not seeing the same issue.  Both of those files load correctly:
{
  "status": "200 OK",
  "payload": {
    "feedCount": [
      {
        "LOAD_COMPLETED": 2
      }
    ],
    "overallStatus": {
      "fullUri": "s3://<bucket_path_redacted/",
      "runNumber": 1,
      "retryNumber": 0,
      "status": "LOAD_COMPLETED",
      "totalTimeSpent": 3,
      "startTime":<PHONE_NUMBER>,
      "totalRecords": 18,
      "totalDuplicates": 0,
      "parsingErrors": 0,
      "datatypeMismatchErrors": 0,
      "insertErrors": 0
    }
  }
}

g.V().hasLabel('http://abc.org/dc/elements/1.1/').valueMap(true)

[{'vertexType': ['terminal'],
  'versionLabel': ['CD 2021-03-26'],
  <T.label: 4>: 'http://abc.org/dc/elements/1.1/',
  <T.id: 1>: '126eb6d6-441a-476f-bc54-c6f2c4e34211',
  'name': ['Protocol'],
  'versionName': ['3/26/2021'],
  'seedVersion': [1]}]

g.V().hasLabel('http://purl.org/dc/elements/1.1').valueMap(true)

[{'vertexType': ['terminology'],
  'versionLabel': ['CT 2021-03-26'],
  <T.label: 4>: 'http://purl.org/dc/elements/1.1',
  <T.id: 1>: '9ddd715b-67c6-4073-bf1a-eb1544aaf0ac',
  'name': ['CDASH'],
  'versionName': ['2021-03-26'],
  'seedVersion': [1]},
 {'vertexType': ['terminology'],
  'versionLabel': ['CT 2021-03-26'],
  <T.label: 4>: 'http://purl.org/dc/elements/1.1',
  <T.id: 1>: '961c2cd6-5bb1-4648-9833-6fac1e8d0ba0',
  'name': ['SDTM'],
  'versionName': ['2021-03-26'],
  'seedVersion': [1]}]

I notice in your load command that you have allowEmptyStrings set to True.  If you use that and you have a row that does not have a label (just ""), then that would throw the parser error that you're seeing above.  As empty strings are not allowed label values.  If you left allowEmptyStrings set to False and supplied no value for a label, Neptune uses a default label name of "vertex" in its place.  You may want to check your load files for any rows that have an empty string ("") in the label field.
Since this is not related to any of the utilities in this repo, I'm going to close this issue.  If you continue to have problems with bulk loading, please make a post to https://repost.aws or open an AWS Support Case and we can continue to work on this issue there.


[Test][Text] Reduce runtime of text tutorial

Description of changes:
Reduce the runtime for the text notebooks.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-805-2 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-805/2/index.html

Job PR-805-8 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-805/8/index.html

Would it be do-able to speed up the text CI build by another 2x so it is within the 10-15 min range?

It might be difficult to accelerate it further. Is it possible for us to merge this in first?

Job PR-805-9 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-805/9/index.html

@Innixma I'll merge this in.

@sxjscience If it is difficult to speed up further it isn't a big deal, having CI at ~45 minutes is already a significant improvement. Keeping it at ~20-25 minutes is also more realistic to when we expand the tutorials for the other tasks in preparation for v0.1 release as they will get longer too.


Can't login using all OAuth services

(At least I tested google and github)
I started remark42 with docker, using my nginx as a proxy. API requests and web version are working.
But when after authorization in google or github the redirection to my site takes place (to __my_site__/auth/github/callback?code=***&state=***), I wait 30 seconds, after which the text {"error": "exchange failed"} is displayed in the browser.
Please tell me where I made the mistake.

docker logs after click "login with github"
remark42  | 2019/10/28 03:30:53.942 [DEBUG] {provider/oauth2.go:80 provider.Oauth2Handler.LoginHandler} login with github
remark42  | 2019/10/28 03:30:53.946 [DEBUG] {provider/oauth2.go:115 provider.Oauth2Handler.LoginHandler} login url https://github.com/login/oauth/authorize?client_id=69c967ac2eaecefce58c&redirect_uri=https%3A%2F%2Fremark42.modos189.ru%2Fauth%2Fgithub%2Fcallback&response_type=code&state=00000000000000000000000000005b0dd106f34d, claims={"aud":"store-iitc-modos189-ru","exp":1572253253,"jti":"0000000000000000000000000000bf42acb09278","nbf":1572251393,"handshake":{"state":"00000000000000000000000000005b0dd106f34d","from":"https://remark42.modos189.ru/web/iframe.html?selfClose"}}
remark42  | 2019/10/28 03:30:53.948 [INFO]  {logger/logger.go:120 logger.(*Middleware).Handler.func1.1} GET - /auth/github/login?from=https://remark42.modos189.ru/web/iframe.html?selfClose&site=store-iitc-modos189-ru - dde7ba6b30f5 - 302 (247) - 4.785571ms
remark42  | 2019/10/28 03:30:54.571 [DEBUG] {provider/oauth2.go:140 provider.Oauth2Handler.AuthHandler} token with state<PHONE_NUMBER>0000000000000000005b0dd106f34d

docker logs after 30 second timeout
remark42  | 2019/10/28 03:31:54.573 [INFO]  {rest/httperrors.go:17 rest.SendErrorJSON} exchange failed - Post https://github.com/login/oauth/access_token: dial tcp <IP_ADDRESS>:443: i/o timeout - 500 - <IP_ADDRESS> - /auth/github/callback?code=00000000000000421b52&state=00000000000000000000000000005b0dd106f34d [caused by auth/provider/oauth2.go:143 provider.Oauth2Handler.AuthHandler]
remark42  | 2019/10/28 03:31:54.573 [INFO]  {logger/logger.go:120 logger.(*Middleware).Handler.func1.1} GET - /auth/github/callback?code=00000000000000421b52&state=00000000000000000000000000005b0dd106f34d - dde7ba6b30f5 - 500 (28) - 1m0.003340052s
remark42  | 2019/10/28 03:31:54.573 [WARN]  {lgr/adaptor.go:16 lgr.(*Writer).Write} http: superfluous response.WriteHeader call from github.com/go-chi/chi/middleware.Timeout.func1.1.1 (timeout.go:40)

docker-compose.yml
version: '2'

services:
    remark:
        build: .
        image: umputun/remark42:latest
        container_name: "remark42"
        hostname: "remark42.modos189.ru"
        restart: always

        logging:
          driver: json-file
          options:
              max-size: "10m"
              max-file: "5"

        ports:
            - "8042:8080"

        environment:
            - REMARK_URL=https://remark42.modos189.ru
            - SITE=store-iitc-modos189-ru
            - SECRET=***
            - STORE_BOLT_PATH=/srv/var/db
            - BACKUP_PATH=/srv/var/backup
            - DEBUG=true
            - AUTH_GOOGLE_CID=***
            - AUTH_GOOGLE_CSEC=***
            - AUTH_GITHUB_CID=***
            - AUTH_GITHUB_CSEC=***
        volumes:
            - ./var:/srv/var

nginx config
server {
    listen      <IP_ADDRESS>:443 ssl http2;
    server_name remark42.modos189.ru ;
    ssl         on;
    ssl_certificate      __path__;
    ssl_certificate_key  __path__;
    ssl_dhparam          __path__;

    location / {
         proxy_redirect          off;
         proxy_set_header        X-Real-IP $remote_addr;
         proxy_set_header        X-Forwarded-For $proxy_add_x_forwarded_for;
         proxy_set_header        Host $http_host;
         proxy_pass              http://<IP_ADDRESS>:8042/;
    }
}

curl -V https://www.google.com from docker image
sudo docker run -it --entrypoint /bin/sh umputun/remark42:latest

/srv # curl -V https://www.google.com

curl 7.64.0 (x86_64-alpine-linux-musl) libcurl/7.64.0 OpenSSL/1.1.1a zlib/1.2.11 libssh2/1.8.0 nghttp2/1.35.1
Release-Date: 2019-02-06
Protocols: dict file ftp ftps gopher http https imap imaps pop3 pop3s rtsp scp sftp smb smbs smtp smtps telnet tftp
Features: AsynchDNS IPv6 Largefile NTLM NTLM_WB SSL libz TLS-SRP HTTP2 UnixSockets HTTPS-proxy

iptables -L -t nat
Chain PREROUTING (policy ACCEPT)
target     prot opt source               destination
DOCKER     all  --  anywhere             anywhere             ADDRTYPE match dst-type LOCAL

Chain INPUT (policy ACCEPT)
target     prot opt source               destination

Chain POSTROUTING (policy ACCEPT)
target     prot opt source               destination
MASQUERADE  all  --  <IP_ADDRESS>/16        anywhere
MASQUERADE  all  --  <IP_ADDRESS>/16        anywhere
MASQUERADE  all  --  <IP_ADDRESS>/16        anywhere
MASQUERADE  all  --  <IP_ADDRESS>/16        anywhere
MASQUERADE  udp  --  <IP_ADDRESS>           <IP_ADDRESS>           udp dpt:openvpn
MASQUERADE  tcp  --  <IP_ADDRESS>           <IP_ADDRESS>           tcp dpt:http-alt

Chain OUTPUT (policy ACCEPT)
target     prot opt source               destination
DOCKER     all  --  anywhere            !<IP_ADDRESS>/8          ADDRTYPE match dst-type LOCAL

Chain DOCKER (2 references)
target     prot opt source               destination
RETURN     all  --  anywhere             anywhere
RETURN     all  --  anywhere             anywhere
DNAT       udp  --  anywhere             anywhere             udp dpt:openvpn to:<IP_ADDRESS>:1194
DNAT       tcp  --  anywhere             anywhere             tcp dpt:8042 to:<IP_ADDRESS>:8080
# Warning: iptables-legacy tables present, use iptables-legacy to see them

to me, it looks like your container can't reach the internet. Can you try curl -v https://www.google.com or curl -v https://github.com (note - -v should be lower case)

With curl, the container has Internet access.
But it turned out that there was some problem because of the server upgrade that was solved by the reboot.
So the problem is solved. Thanks.


Shall we trust content length for RequestBodyLimitAction?

Currently we record the body per chunks as it is coming and when reach the limit we decide to trigger the RequestBodyLimitAction. As buffering is expensive, in specific cases like Envoy it is worth to investigate if we should trust content-length header for rushing this decision even before the buffering.

Sending a content length but not sending a payload of the same size is invalid http. So I think it's fine to read it for eager limit checking given even if it wasn't accurate the request would be rejected.

we should trust content-length header for rushing this decision even before the buffering
I see value in doing this if the LimitAction is deny, but it requires an extra API method that accepts the content length, checks the LimitAction, and directly rises an interruption if it is above the limit, doesn't it?


Installation failed of C API on Ubuntu

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:/
TensorFlow installed from (source or binary): Tensorflow C API
TensorFlow version: Latest
Python version:
Installed using virtualenv? pip? conda?:No
Bazel version (if compiling from source):/
GCC/Compiler version (if compiling from source): Update
CUDA/cuDNN version: NaN
GPU model and memory: NaN

*THE GUIDE IS NOT COMPLETE / THAT IS NOT WORKING
TF_VERSION ISN'T RECOGNIZE AS A FUNCTION
LOVE

I think it might have been resolved. I am closing the issue. Please open new ticket if you see similar issue again. Thanks!


Update TCF Experience with Legal Basis Info

Is your feature request related to a specific problem?
Continuing to expand TCF experience
Describe the solution you'd like
Update TCF Experience with legal basis information to facilitate a FE filter between consent and legitimate interest
A description of what you want to happen.
Describe alternatives you've considered, if any
A description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Eveb though according to GVL, a Purpose always have to Consent basis, it seems like it is possible that this Purpose is on a privacy declaration that doesn't have a Consent basis, especially if that system is not in the GVL.
So in practice, it sounds like for the purposes/special purposes tab, I need to calculate a list of possible legal bases across all systems, and then for the vendor tab, for each purpose/special purpose, the list of legal bases just on that system.
In line with how we've been doing things, I should rely on some other mechanism to enforce that the appropriate data is on the systems. I look at the system directly instead of the GVL.

Show just non-gvl system purposes under both filters while we figure out what to do with non-gvl systems?


Implementing ng2-translate module

I'm submitting a ...
[ ] bug report
[ ] feature request
[X] question about the decisions made in the repository

In fact I have no idea whether this is a bug or not but I will try to explain it so you can tell me if I am doing something wrong.
I am trying to implement the ng2-translate module. I could do it in pages.module but I am unable to do it in the nga.module.
I created a custom module called Pipes that exports the TranslateModule and when I implement it in pages.module by referencing to pipes.module and adding it to the imports array, everything seems to be ok.
Then I tried to extend it to nga.module in the same way, importing the PipesModule from source file and adding it to import section but despite the fact that I have no error, the translations of every single page are not loaded.
The following is the code I am doing, can you help me?
pipes.module.ts:

import { NgModule } from '@angular/core';
import { BrowserModule } from '@angular/platform-browser';
import { FormsModule, provideForms } from '@angular/forms';

import { TranslateModule, TranslateStaticLoader, TranslateLoader } from 'ng2-translate/ng2-translate';
import { Http, HttpModule } from '@angular/http';

@NgModule({
    declarations: [
    ],
    imports: [
        TranslateModule.forRoot({
          provide: TranslateLoader,
          useFactory: (http: Http) => new TranslateStaticLoader(http, 'assets/langs', '.json'),
          deps: [Http]
        })
    ],
    exports: [
        TranslateModule
    ]
})

export class PipesModule { }

pages.module.ts

import { NgModule }      from '@angular/core';
import { CommonModule }  from '@angular/common';

import { routing }       from './pages.routing';
import { NgaModule } from '../theme/nga.module';
import { PipesModule } from '../pipes.module';

import { Pages } from './pages.component';

@NgModule({
	imports: [
	  	CommonModule,
	  	NgaModule,
	  	routing,
	  	PipesModule
	],
	declarations: [Pages]
})

export class PagesModule { }

nga.module.ts

import { NgModule }      from '@angular/core';
import { CommonModule }  from '@angular/common';
import { ReactiveFormsModule, FormsModule } from '@angular/forms';
import { RouterModule } from '@angular/router';

import { PipesModule } from '../pipes.module';

import {
  BaThemeConfig
} from './theme.config';

import {
  BaThemeConfigProvider
} from './theme.configProvider';

import {
  BaAmChart,
  BaBackTop,
  BaCard,
  BaChartistChart,
  BaCheckbox,
  BaContentTop,
  BaFullCalendar,
  BaMenu,
  BaMsgCenter,
  BaMultiCheckbox,
  BaPageTop,
  BaPictureUploader,
  BaSidebar,
  BaInternationalization,
  BaStripedTable,
  BaPreview,
  BaDialog
} from './components';

import {
  BaScrollPosition,
  BaSlimScroll,
  BaThemeRun
} from './directives';

import {
  BaAppPicturePipe,
  BaKameleonPicturePipe,
  BaProfilePicturePipe
} from './pipes';

import {
  BaImageLoaderService,
  BaThemePreloader,
  BaThemeSpinner
} from './services';

import {
  EmailValidator,
  EqualPasswordsValidator
} from './validators';

const NGA_COMPONENTS = [
  BaAmChart,
  BaBackTop,
  BaCard,
  BaChartistChart,
  BaCheckbox,
  BaContentTop,
  BaFullCalendar,
  BaMenu,
  BaMsgCenter,
  BaMultiCheckbox,
  BaPageTop,
  BaPictureUploader,
  BaSidebar,
  BaInternationalization,
  BaStripedTable,
  BaPreview,
  BaDialog
];

const NGA_DIRECTIVES = [
  BaScrollPosition,
  BaSlimScroll,
  BaThemeRun
];

const NGA_PIPES = [
  BaAppPicturePipe,
  BaKameleonPicturePipe,
  BaProfilePicturePipe
];

const NGA_SERVICES = [
  BaImageLoaderService,
  BaThemePreloader,
  BaThemeSpinner
];

const NGA_VALIDATORS = [
  EmailValidator,
  EqualPasswordsValidator
];

@NgModule({
  declarations: [
    ...NGA_PIPES,
    ...NGA_DIRECTIVES,
    ...NGA_COMPONENTS
  ],
  imports: [
    CommonModule,
    RouterModule,
    FormsModule,
    ReactiveFormsModule,
    PipesModule
  ],
  providers: [
    BaThemeConfigProvider,
    BaThemeConfig,
    ...NGA_VALIDATORS,
    ...NGA_SERVICES
  ],
  exports: [
    ...NGA_PIPES,
    ...NGA_DIRECTIVES,
    ...NGA_COMPONENTS
  ]
})
export class NgaModule {
}

+1

I got it working doing the next steps:

First I created a core module which is a shared module and I want it visible app wide (like your pipes module is meant to be), it contains not only translate pipes but other stuff too. Like this (notice specially the exports section):

...
import { TranslateModule } from 'ng2-translate/ng2-translate';
...
@NgModule({
  imports: [
      CommonModule,
      TranslateModule.forRoot()
  ],
  **exports**: [
      TranslateModule,
      ...
  ]
})
export class CoreModule {
    constructor (@Optional() @SkipSelf() parentModule: CoreModule) {
        if (parentModule) {
            throw new Error('CoreModule is already loaded. Import it in the AppModule only');
        }
    }
}

I imported that core module into the app module like this:

...
import { CoreModule } from './common/core.module';
...
@NgModule({
  imports: [ // import Angular's modules
    BrowserModule,
    HttpModule,
    RouterModule,
    FormsModule,
    ReactiveFormsModule,
    appRouting,
    HomeModule,
    PagesModule,
    CoreModule
]
...

Now the translate module is app wide available, however, every time you need to access to the translate service you still have to import it in each component / service where you use it (you also could bootstrap it, but I wouldn't recommend it, you don't always need TranslateService).

If I am not missing anything, that's about it.
That should work and I believe having a common shared module where the translate module is included makes sense since you will need it app wide.
Hopefully this helps.

marcos, thank you very much for the response!
I could not make it work using the "| translate" pipe in the template.
1- I created the pipes.module as in my first post (adding the constructor you added).
2- I imported it into app.module. (I can access it in the .component but not in the .html file)
The error is that the 'translate' pipe is not recognized: "Template parse errors: The pipe 'translate' could not be found".
Regards!

Hi, @nicokiva
The most important part is the "exports" section. You have to export the module that holds the translation logic so that when you import that in the app module it will be application wide available.
Honestly the constructor is not a big deal, you can skip making it the way I did if you want, those lines are only to throw a more friendly error when the core module is included twice. It is a good thing to do to help yourself, but it doesn't change the behaviour in a normal application in everything is correctly configured.
So, if you are getting that error is because your component can't see the pipe and that means your translate module is not actually available in that component's context.
Can you write your app structure and highlight the component that is not able to see the translate module?
Also, can you write at least the related parts of app.module and others modules involved in your issue so we can maybe see something you may be missing?
Thanks

Marcos! yes, of course.
src/app/app.Module
import { NgModule, ApplicationRef, ReflectiveInjector } from '@angular/core';
import { BrowserModule } from '@angular/platform-browser';
import { FormsModule, ReactiveFormsModule } from '@angular/forms';
import { HttpModule } from '@angular/http';
import { RouterModule } from '@angular/router';
import { removeNgStyles, createNewHosts } from '@angularclass/hmr';

/*
 * Platform and Environment providers/directives/pipes
 */
import { ENV_PROVIDERS } from './environment';
import { routing } from './app.routing';

// App is our top level component
import { App } from './app.component';
import { AppState } from './app.service';
import { GlobalState } from './global.state';
import { NgaModule } from './theme/nga.module';
import { PagesModule } from './pages/pages.module';
import { PipesModule } from './pipes.module';

import { Configuration } from './app.configuration';

import { IService } from './services/interfaces/account.service';

import { AccountService } from './services/account.service';
import { AccountServiceMock } from './services/mocks/account.service';

import { ProfileService } from './services/profile.service';
import { ProfileServiceMock } from './services/mocks/profile.service';

// Application wide providers
const APP_PROVIDERS = [
  AppState,
  GlobalState
];

/*
  If mockServices then load mock services (has sense only for those that calls backend)
  else load original services.
*/
let SERVICES = [];
var injectorOptions = [];
if (Configuration.mockServices) {
  SERVICES.push({ provide: AccountService, useClass: AccountServiceMock });
  SERVICES.push({ provide: ProfileService, useClass: ProfileServiceMock });
} else {
  SERVICES.push({ provide: AccountService, useClass: AccountService });
  SERVICES.push({ provide: ProfileService, useClass: ProfileService });
}

/**
 * `AppModule` is the main entry point into Angular2's bootstraping process
 */
@NgModule({
  bootstrap: [App],
  declarations: [
    App
  ],
  imports: [ // import Angular's modules
    BrowserModule,
    HttpModule,
    RouterModule,
    FormsModule,
    ReactiveFormsModule,
    NgaModule,
    PagesModule,
    routing,
    PipesModule
  ],
  providers: [ // expose our Services and Providers into Angular's dependency injection
    ENV_PROVIDERS,
    APP_PROVIDERS,
    SERVICES
  ]
})

export class AppModule {

  constructor(public appRef: ApplicationRef, public appState: AppState) { }

  hmrOnInit(store) {
    if (!store || !store.state) return;
    console.log('HMR store', store);
    this.appState._state = store.state;
    this.appRef.tick();
    delete store.state;
  }

  hmrOnDestroy(store) {
    const cmpLocation = this.appRef.components.map(cmp => cmp.location.nativeElement);
    // recreate elements
    const state = this.appState._state;
    store.state = state;
    store.disposeOldHosts = createNewHosts(cmpLocation);
    // remove styles
    removeNgStyles();
  }

  hmrAfterDestroy(store) {
    // display new elements
    store.disposeOldHosts();
    delete store.disposeOldHosts;
  }
}

src/app/pipes.module

import { NgModule, Optional, SkipSelf } from '@angular/core';
import { BrowserModule } from '@angular/platform-browser';
import { FormsModule, provideForms } from '@angular/forms';

import { TranslateModule, TranslateStaticLoader, TranslateLoader } from 'ng2-translate/ng2-translate';
import { Http, HttpModule } from '@angular/http';

@NgModule({
    declarations: [
    ],
    imports: [
        TranslateModule.forRoot({
          provide: TranslateLoader,
          useFactory: (http: Http) => new TranslateStaticLoader(http, 'assets/langs', '.json'),
          deps: [Http]
        })
    ],
    exports: [
        TranslateModule
    ]
})

export class PipesModule {

  constructor (
    @Optional() @SkipSelf() parentModule: PipesModule
  ) {

    if (parentModule) {
        throw new Error('CoreModule is already loaded. Import it in the AppModule only');
    }

  }

}

src/app/theme/nga.module

import { NgModule }      from '@angular/core';
import { CommonModule }  from '@angular/common';
import { ReactiveFormsModule, FormsModule } from '@angular/forms';
import { RouterModule } from '@angular/router';

import {
  BaThemeConfig
} from './theme.config';

import {
  BaThemeConfigProvider
} from './theme.configProvider';

import {
  BaAmChart,
  BaBackTop,
  BaCard,
  BaChartistChart,
  BaCheckbox,
  BaContentTop,
  BaFullCalendar,
  BaMenu,
  BaMsgCenter,
  BaMultiCheckbox,
  BaPageTop,
  BaPictureUploader,
  BaSidebar,
  BaInternationalization,
  BaStripedTable,
  BaPreview,
  BaDialog
} from './components';

import {
  BaScrollPosition,
  BaSlimScroll,
  BaThemeRun
} from './directives';

import {
  BaAppPicturePipe,
  BaKameleonPicturePipe,
  BaProfilePicturePipe
} from './pipes';

import {
  BaImageLoaderService,
  BaThemePreloader,
  BaThemeSpinner,
  BaSimpleSpinner,
  BaHttpWrapperService
} from './services';

import {
  EmailValidator,
  EqualPasswordsValidator
} from './validators';

const NGA_COMPONENTS = [
  BaAmChart,
  BaBackTop,
  BaCard,
  BaChartistChart,
  BaCheckbox,
  BaContentTop,
  BaFullCalendar,
  BaMenu,
  BaMsgCenter,
  BaMultiCheckbox,
  BaPageTop,
  BaPictureUploader,
  BaSidebar,
  BaInternationalization,
  BaStripedTable,
  BaPreview,
  BaDialog
];

const NGA_DIRECTIVES = [
  BaScrollPosition,
  BaSlimScroll,
  BaThemeRun
];

const NGA_PIPES = [
  BaAppPicturePipe,
  BaKameleonPicturePipe,
  BaProfilePicturePipe
];

const NGA_SERVICES = [
  BaImageLoaderService,
  BaThemePreloader,
  BaThemeSpinner,
  BaSimpleSpinner,
  BaHttpWrapperService
];

const NGA_VALIDATORS = [
  EmailValidator,
  EqualPasswordsValidator
];

@NgModule({
  declarations: [
    ...NGA_PIPES,
    ...NGA_DIRECTIVES,
    ...NGA_COMPONENTS
  ],
  imports: [
    CommonModule,
    RouterModule,
    FormsModule,
    ReactiveFormsModule
  ],
  providers: [
    BaThemeConfigProvider,
    BaThemeConfig,
    ...NGA_VALIDATORS,
    ...NGA_SERVICES
  ],
  exports: [
    ...NGA_PIPES,
    ...NGA_DIRECTIVES,
    ...NGA_COMPONENTS
  ]
})
export class NgaModule {
}

Having this, every component into src/app/pages can see the translation pipe but no component in src/app/theme/components can see it (at least I tried with baMessageCenter and baBackTop). Better said the .component files are injected with the translate module but the template does not recognize the translate pipe.

Hi @nicokiva
Well, I believe you're missing importing "pipesModule" in the nga.module imports section.
I don't see anything else. When you have multiple modules, you have to include that "shared module" (pipesModule) in each module you make use of it.
I know that sounds like if you import a module inside app.module it should be inherited down the stream to other children, but Angular2 doesn't work that way in order to isolate as much as possible the modules in stand-alone pieces and avoid circular references or very-hard-to-follow ones... I had myself quite a hard time getting my head around that, but it makes sense after all.
Hopefully this helps.

Yeah! it worked. I love you!

I added the import to nga.module and it worked!
I thought that as it was app wide after including it in the app.module, it was going to work but after importing it in the nga.modules, it worked!

Well... I do not know what happened but there is another issue I have now.
If I import PipesModule in app.module to make it app wide and then I import it in pages.module, everything is OK. After that if I import it in the nga.module, every translation from the page is lost (it is shown the keys I use to get them)...

@nicokiva
Import TranslateService in your app.module and inject it on constructor to set de default language, doing so the application will load with the choose language, like that:
constructor(public appRef: ApplicationRef, public appState: AppState, translate: TranslateService) {
  translate.setDefaultLang('en');
 }


RFC 9134 Version Support

We recently ran into a case where we two venders with correct implementation of different versions of RFC-9134 (https://datatracker.ietf.org/doc/html/draft-ietf-payload-rtp-jpegxs-18) did not opererate together.  Since the JPEG-XS specification for packetization is evolving does it make some sense to put into the receiver caps and parameters the specific range or specific values for RFC 9134 version supported by a receiver?
The controller could then "reason" about what common format spans the target receivers and adjust to send that version of packetization with the SDP file containing the specific version of JPEG-XS packetization employed.  This would give us maximum flexibility for supporting a wide range of hardware.
Thoughts?

Tbh, I would rather focus on the already published RFC and assume here that implementations of outdated drafts will be fixed to conform to the final spec.

Is there such a thing though as a final spec for these new media types? Looking over NDI I see there are currently 12 versions over the last 6 years with major and minor versions (https://en.wikipedia.org/wiki/Network_Device_Interface).    I suspect we will see improvements in each of these new media types that can introduce confusion for operators trying to connect incompatible equipment that say "JPEG-XS" capable. Having the vender's that produce JPEG-XS equipment with NMOS capabilities put explicitly in their receiver capabilities the versions supported could be useful.

Can’t speak for NDI, but once an RFC is published it is done.  Errata can
be filed (9134 has an errata already) but no real changes.
Yes some people built to early drafts, and they need to update their
products - we also run into this case in the field and it’s a problem.
On Tue, Feb 1, 2022 at 4:48 AM rbgodwin-nt @.***> wrote:

Is there such a thing though as a final spec for these new media types?
Looking over NDI I see there are currently 12 versions over the last 6
years with major and minor versions (
https://en.wikipedia.org/wiki/Network_Device_Interface). I suspect we
will see improvements in each of these new media types that can introduce
confusion for operators trying to connect incompatible equipment that say
"JPEG-XS" capable. Having the vender's that produce JPEG-XS equipment with
NMOS capabilities put explicitly in their receiver capabilities the
versions supported could be useful.
—
Reply to this email directly, view it on GitHub
https://github.com/AMWA-TV/bcp-006-01/issues/9#issuecomment-1026653430,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACPBR3F3HXNYHXF5EFUB7NDUY6T7VANCNFSM5NFQ6L2A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Creating an RFC takes many steps, and always starts with drafts. These drafts are versioned, but are in itself never approved RFC's. Vendors often start implementations, based on a draft RFC, but need to update their implementations once the final RFC is published.
What is more, once an RFC is published, it can and will never be changed again (not even to fix a typo). So, in my opinion, there is no use in adding version support for RFC 9134. The RFC itself is the version. If updates to the specification are needed, the new version will also be a different RFC.

Closed per discussion in activity group, 18 May 2022


Need to explicitly specify rules for the users table

Describe the bug
I can easily create a user keeping the config as shown in https://www.spaceuptech.com/docs/user-management/overview
However, to perform any read or write operation, we need to explicitly specify the rules.
Also, users is a very common name. The programmer may already have such a table.
Lets rename the table to something uncommon, like SpaceCloud_Users_Table.
So that there is no conflict in the table names
Expected behaviour
Either include in the docs that you have to explicitly specify, or give permissions internally itself.
Actual behaviour
Explicit Rules are currently required.
Steps to reproduce
Call the Profile or Profiles routes.
Your environment

Space Cloud version: v0.9.0
OS: -
The client api version (if applicable): -
Browser (if applicable): -

If this bug restricts your use of space-cloud, give it a thumbs up reaction, so that we can determine which bugs need to be fixed immediately.
👍

@AliabbasMerchant we should specify in the docs that rules are required to read the users collection. User management isn't going to be a production ready module. It's just for easy prototyping

@AliabbasMerchant if I have addressed your concern then let's close this issue


Can't set categories or tags as in the example

I've been playing with the demo code and while it's creating a post with no problem, it's not setting the category or tags. Here's what I've got to create a post:
wp=Client('http://500kelvin.doesntexist.com/xmlrpc.php','ben','cactus42')
post=WordPressPost()
post.title = 'Come As You Are'
post.content = "Blah blah blah"
post.terms_names = {'category':'music','post_tag':'testing testing'}
post.post_status = 'publish'
theNewPostID = wp.call(NewPost(post))
I do have a Music category defined, and I've tried using both 'Music' and 'music' in the call, but posts always come in uncategorized. What am I missing?
Thanks

From the docs: Note that terms_names is a dictionary with taxonomy names as keys and list of strings as values.


Providing an absolute path to a DAT file causes the XML parsing to fail

Placing the DAT file in the same directory and using only the filename with extension works fine. Current suspicion is that quoted paths are the issue but have not confirmed yet.

Finally figured this one out, and the issue extended to the Move Library functions as well. Basically, this would occur when providing quotes with the filepath, as is often necessary (but not in the case of this script) when providing file paths that have spaces or other special characters in the string.
This issue was resolved by stripping away any surrounding quotes from paths input by the end user prior to using the string as a path.


pyroxy

I am using python 3.9.1
it used to work when i added my own proxies
now i am getting error
pyroxy eror

Info. What exact error?

raceback (most recent call last):
File "/home/whitmorekyle84/MHDDoS/start.py", line 19, in
from PyRoxy import Proxy, Tools as ProxyTools, ProxyUtiles, ProxyType, ProxyChecker
ImportError: cannot import name 'Proxy' from 'PyRoxy' (unknown location)

I don't normally get this error
when i put my own proxies i get it

There might be issues with your proxies, empty spaces, IPv6, username and passwords, idk what PyRoxy supports so just try to let it get its own proxies, they work fine as long is as it even uses them
Otherwise full remove PyRoxy and reinstall requirements

/files/proxies/your_proxy.txt
is this way right

Yes, you can just check where it stores it by letting MHDDoS get proxies for you once

File "start.py", line 28, in
from PyRoxy import Proxy, ProxyChecker, ProxyType, ProxyUtiles
ModuleNotFoundError: No module named 'PyRoxy'
i'm too met this problem

Can u help me plz?


SOLR-16810: Under certain situations Solr produces managed schema XML with duplicate fields

https://issues.apache.org/jira/browse/SOLR-16810

Description
While persisting the ManagedIndexSchema as XML, non-printable characters in field names get escaped as #nn;, where nn is the decimal representation of the non-printable character. For example, if the field name has the byte 0x14, it gets escaped as #20;. This in indistinguishable from the literal #20; in the field name. If we have two fields - one with the non-printable character and the other with the literal string, two fields get generated with the same name. Loading the resulting XML, naturally, causes an exception. To fix this, any occurrence of literal # in the field name should be escaped, with say ##.
A second problem is that while escaping happens when generating XML, the corresponding unescaping does not happen on loading it.
Solution
Both the suggested fixes are here. There are tests to expose the bugs and the fixes that pass the tests.
Tests
Added two sets of tests - one for IndexSchema.java and the other for XML.java
Checklist
Please review the following and check all that apply:

[x] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[x] I have created a Jira issue and added the issue ID to my pull request title.
[x] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[x] I have developed this patch against the main branch.
[x] I have run ./gradlew check.
[x] I have added tests for my changes.
[ ] I have added documentation for the Reference Guide

Thank you StaleBot.....   I just checked the JIRA and I was last to chime in, so I'll take this and try and get it over the finish line.


Support 3.25.2 sqlite

3.25.2 was released 2018-09-25 and has a few new features that seem pretty useful, specifically support for window functions and new features to alter table such as rename column.

already supported :)
https://github.com/mattn/go-sqlite3/blob/c7c4067b79cc51e6dfdcef5c702e74b1e0fa7c75/sqlite3-binding.h#L127

Ah nice. I was only looking at the releases. I see a bunch of work on a 2.0 release. Are their plans for a 1.10 release?

tagged v1.10.0.


Fixed duplicate inputs

Grab and interact scripts now use InputActionAsset of the PlayerInput component to register per-player input instead of global input
Interact script has been refactored into Interactor and Interactable. this makes it follow the actor - object behaviour simular to the grab & grabable interaction

To ensure this hotfix truly works it should be tested on multiple machines since unity sometimes displayed undefined behaviour


ci: add python 3.10 to workflows

With python 3.10 set to release in a few months, it would be a good idea to ensure code works on it while developing it.
If it breaks at any point, that's a good sign of a regression.

Decided internally to wait until 3.10 is stable and our dependencies all officially support 3.10.


🎆 Update image conversion transform to handle EPS images

Currently this only converts EPS -> PNG. Other conversion requires inkscape which has a CLI bug loading EPS files: https://gitlab.com/inkscape/inkscape/-/issues/3524
There is a bunch of commented code we can revive it that issue gets resolved. We can also just delete the commented code and not worry about until EPS -> PNG becomes a problem...
I also tested some other PDF/SVG conversions and they seem to work as expected.

🚀


Doesn't work?

I'm not able to get this plugin working.. I've done:
-Install package in my PCL
-Install package in my .Droid project
-Added the receiver to my manifest at Properties/AndroidManifest.xml
-Copied your demo code (excluding the cancel)
-Run/debug
But no notification is shown. No error is shown or logged. No crash.
Am I missing something or doing something wrong?
(PS: cannot try other platforms because I'm using Xamarin Studio on Windows..)

@PieterAelse
I fixed some issues. You can download sample project to see how to use it:
https://github.com/B1naryStudio/Xamarin.LocalNotifications/tree/master/Samples/LocalNotifications.Samples.Droid


Changed eval to exec

Changing eval to exec resolves conflicts with some commands that include colons and ampersands.

Anything I can do to help with this @bandesz ?

@nwalke sorry, I wasn’t watching my own repo and I only saw the issue and the PR now. I’ll look into this asap.


When I use your pre-trained model 550000.pt, the following error occurs

When I execute the command : python generate.py --size 256 --ckpt ./checkpoint/550000.pt
Traceback (most recent call last):
File "generate.py", line 76, in
g_ema.load_state_dict(checkpoint["g_ema"])
File "/home/sd02/anaconda3/envs/pytorch_py36/lib/python3.6/site-packages/torch/nn/modules/module.py", line 1045, in load_state_dict
self.class.name, "\n\t".join(error_msgs)))
RuntimeError: Error(s) in loading state_dict for Generator:
Missing key(s) in state_dict: "noises.noise_0", "noises.noise_1", "noises.noise_2", "noises.noise_3", "noises.noise_4", "noises.noise_5", "noises.noise_6", "noises.noise_7", "noises.noise_8", "noises.noise_9", "noises.noise_10", "noises.noise_11", "noises.noise_12".

Please replace g_ema.load_state_dict(checkpoint["g_ema"]) with g_ema.load_state_dict(checkpoint["g_ema"], strict=False)

Please replace g_ema.load_state_dict(checkpoint["g_ema"]) with g_ema.load_state_dict(checkpoint["g_ema"], strict=False)


Implement GetKey trait for Xpriv and Vec<Xpriv> with necessary …

Improvements

Implemented the GetKey trait for Xpriv, allowing key retrieval based on key requests and secure key derivation.
Modified the get_key function to support both specific fingerprint-based and general derivation path lookups, ensuring compatibility with both master and child keys.
Added support for the Vec<Xpriv> collection, enabling efficient key retrieval from a set of Xpriv instances.
Improved error handling and fixed issues with trait bounds and collection handling.
Fixed incomplete PartialEq and Eq implementations for Xpriv to ensure proper comparison and trait compliance.
Added constructor example for better usage of the Xpriv struct and trait functionality.

Implementarea metodei get_key pentru HashSet<Xpriv> caută primul PrivateKey valid din colecție folosind get_key pe fiecare xpriv, returnând None dacă nu găsește niciunul valid.


Remote Follow

Is there any remote follow button? I search the domain of the blog that is using this plugin, but I can not find it.

That would be cool, too, also to have the endpoint be able to reply directly to people who engage with them, that way it will be consistent across both the fediverse and the site they are coming from.


Use OpenAI ChatGPT Plugin via Python API fails with maximum number of tokens error

System Info
0.0.314
Who can help?
@hwchase17 @agola11 @eyurtsev
Information

[ ] The official example notebooks/scripts
[X] My own modified scripts

Related Components

[X] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[X] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
from langchain.chat_models import ChatOpenAI
from langchain.agents import load_tools, initialize_agent
from langchain.agents import AgentType
from langchain.tools import AIPluginTool

tool = AIPluginTool.from_plugin_url("https://scholar-ai.net/.well-known/ai-plugin.json")

llm = ChatOpenAI(temperature=0, streaming=True, model_name="gpt-3.5-turbo-16k-0613", max_tokens=16000)
tools = [tool]

agent_chain = initialize_agent(
    tools, llm, agent=AgentType.OPENAI_FUNCTIONS, verbose=True
)

agent_chain.run("What are the antiviral effects of Sillymarin?")

And I get
InvalidRequestError: This model's maximum context length is 16385 tokens. However, you requested 17952 tokens (1855 in the messages, 97 in the functions, and 16000 in the completion). Please reduce the length of the messages, functions, or completion.

Expected behavior
The idea is that should provide sensible information based on the Plugin Scholar AI.
It would be terrific, if you could help us here. 🙂

@hwchase17 @agola11 @eyurtsev The output of the generated response was not particularly helpful, because even, if I set max_tokens really low like this
tool = AIPluginTool.from_plugin_url("https://scholar-ai.net/.well-known/ai-plugin.json")

llm = ChatOpenAI(temperature=0, streaming=True, model_name="gpt-3.5-turbo-16k-0613", max_tokens=100)
tools = [tool]

agent_chain = initialize_agent(
    tools, llm, agent=AgentType.OPENAI_FUNCTIONS, verbose=True
)

agent_chain.run("What are the antiviral effects of Sillymarin?")

I get InvalidRequestError: This model's maximum context length is 16385 tokens. However, your messages resulted in 16552 tokens (16455 in the messages, 97 in the functions). Please reduce the length of the messages or functions.


Added content for passwordless authentication

Learn Build status updates of commit 6383fd7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoftgraph/docs-conceptual/authentication-commands.md
:white_check_mark:Succeeded
View (graph-powershell-1.0 || graph-powershell-beta)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 1084f6c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoftgraph/docs-conceptual/authentication-commands.md
:white_check_mark:Succeeded
View (graph-powershell-1.0 || graph-powershell-beta)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@f4rty - Requesting for your review on this change. Thanks.

May be remove the first (FIDO) from 'Fast ID Online (FIDO) v2.0 (FIDO2)'. Feels like a pleonasm.
Also, I think we need to emphasise the great usability of Passwordless, too, replacing this 'Passwords are a primary attack vector.'

Learn Build status updates of commit d894117:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoftgraph/docs-conceptual/authentication-commands.md
:white_check_mark:Succeeded
View (graph-powershell-1.0 || graph-powershell-beta)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@timayabi2020, please merge this.

@CelesteDG , this is ready for merging. Thanks.


The database does not appear in the sqlLab menu

Bug description
Hello everyone. Sorry, my English is not good.
I have more than 200 database connections (SQL Server). Connection established successfully, no problems. Only when I look in SqlLab at the database selection, the database only displays 100 databases. Meanwhile, when I search for my other databases, it doesn't appear. Is it really the limit of 100? If I don't want a limit, where should I change the code? Thank you.
How to reproduce the bug

Screenshots/recordings
No response
Superset version
3.0.3
Python version
3.9
Node version
16
Browser
Chrome
Additional context
No response
Checklist

[ ] I have searched Superset docs and Slack and didn't find a solution to my problem.
[ ] I have searched the GitHub issue tracker and didn't find a similar bug report.
[ ] I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.

I had a similar issue with sqlite connection.
Establishing a connection, I am assuming wont 100% guarantee that its actually linked to your DB. Check you connection path or even try sqlalcheny connection to define you absolute DB path.


Prepend psiimport path instead of overwriting $PYTHONPATH

Description
When psi4 binary is found but the Python package is not we should not overwrite the $PYTHONPATH before importing psi4 as that may cause other required packages to be not found anymore. Hence prepend it.
See https://github.com/MolSSI/QCEngine/issues/292#issuecomment-1337320606
Changelog description
Fix IndexError in PSI4 Harness when psi4 Python package is not found but the binary is.

Thanks, I think I agree. In answer to your

So why not simply import psi4 and directly access psi4.executable?

it's because which_import finds the location w/o actually importing. so the pythonpath adjustment is to get enough dependencies locatable so that import psi4 can work and psi4.executable can be accessed.

it's because which_import finds the location w/o actually importing. so the pythonpath adjustment is to get enough dependencies locatable so that import psi4 can work and psi4.executable can be accessed.

And that's what I don't understand: You find the location w/o importing to then adjust the PYTHONPATH with the found location to import it in a subprocess. But the location can only be found if PYTHONPATH already contains it so the import would work. So why changing it? Or did I miss how which_import works?

it's because which_import finds the location w/o actually importing. so the pythonpath adjustment is to get enough dependencies locatable so that import psi4 can work and psi4.executable can be accessed.

And that's what I don't understand: You find the location w/o importing to then adjust the PYTHONPATH with the found location to import it in a subprocess. But the location can only be found if PYTHONPATH already contains it so the import would work. So why changing it? Or did I miss how which_import works?

The overall aim for this def found() is to make sure a paired bin/psi4 and psi4/__init__ are found and ready to load. There's some configuring that the program knows above, so it's best to use psi4 --module and psi4.executable. https://github.com/psi4/psi4/blob/master/psi4/init.py#L38-L42 The overall pattern for the QCEngine harnesses is that found() looks for the installation (but doesn't run or import) and version() does a simple run. https://github.com/MolSSI/QCEngine/blob/master/qcengine/programs/nwchem/runner.py#L73-L87
You're correct that which_import is finding the signature of the psi4 package but not actually trying to import it (https://github.com/MolSSI/QCElemental/blob/next/qcelemental/util/importing.py#L36). So when that returns True, it may still be that not all dependencies required for psi4 are importable. My first thought (and how I answered above) was that the env modification was to add a likely dependency location to sys.path so that import psi4 worked and psi4.executable could be accessed. But I entirely agree with your analysis that that is redundant. My second thought was that the PYTHONPATH truncation for env was to restrict paths so that, say a psi4.py couldn't interfere with the targeted psi4 installation. But I'm not thinking of what scenario would hit that. I've removed the env complication in #391 and will try that locally for a few days.
Would that satisfy your intent the same as this PR? Hopefully I'll have a bright idea (or hit a bug) in the meantime about what I was doing.

Would that satisfy your intent the same as this PR? Hopefully I'll have a bright idea (or hit a bug) in the meantime about what I was doing.

But why the subprocess? What about the more common:
try:
  import psi4
except ImportError:
  return None
return psi4.executable[:-5]

If you still want the subprocess please include at least error handling, see the linked comment:

So I'd strongly suggest to add any kind of error checking first before accessing exc["stdout"]

I.e. at least check the exit code and possibly that exc["stdout"] is not empty which both suggests that the import failed.

@Flamefire, in case you're still monitoring this issue, I think I've got it fixed at #418 . If you see any deficiencies, please let me know.

@Flamefire, in case you're still monitoring this issue, I think I've got it fixed at #418 . If you see any deficiencies, please let me know.

Looks good I think. If you want to be safe I'd also check the exit code as suggested in my prior comment, i.e. exc["proc"].wait(timeout=30) == 0

Looks good I think. If you want to be safe I'd also check the exit code as suggested in my prior comment, i.e. exc["proc"].wait(timeout=30) == 0

Thanks, done, and I caught another bug along the way.


Tolerations taints code cleanup

Removing duplicated code. Broken down the changes into smaller steps for easier review.
@damemi @seanmalloy PTAL

/lgtm
/assign @aveshagarwal @ravisantoshgudimetla
@ingvagabund looks like Travis CI failed to create the kind cluster. Just need to re-run the Travis CI job, but I don't have access to trigger the job again.
https://travis-ci.org/github/kubernetes-sigs/descheduler/jobs/677873044

/close
/reopen

/open

Looks like it would be easier to just push --force :-|
@ravisantoshgudimetla @aveshagarwal do you have permissions to reopen the issue?

/reopen

/open

/reopen

@aveshagarwal can you try /reopen one more time?

/approve


Invalid cursor position saved

I don't know if there are any alternative to the current behavior, but the "save cursor position" feature is strange when you use suplemon as your git editor for instance:
Everytime you type git commit -a, suplemon is invoked so that you can type your comment. But git actually opens a file COMMIT_EDITMSG. If you type, say, a 4 lines commit, then on your next commit the cursor will be down to be 4th line because this is still the "same" file COMMIT_EDITMSG. The same strange behavior happens for git rebase and such obviously.
One possible behavior could be to restore cursor position only when the file content is the same as previously (checksum or equivalent)? I'm not sure it make senses to restore cursor position when file has been modified by any other software - what do you think?

Good point! I've noticed this a few times but haven't been too bothered with it. However your use case is such that this feature might be rather annoying. I think the best approach is to save a hash/checksum of the file too, and only restore the cursors if the hash is the same as before. I'll fix this for the next release.

This is now fixed in the dev branch. Please test it if you have time.

Cool, thanks, I'll give it a try today.

So far so good. Did not see any issue yet.

As of today, I only saw one strange behavior once but not sure it was related to this bug. Apart of that it works great :).


Add  function

What is the problem / feature ?
env_int is a logical addition to this toolset.
How did it get fixed / implemented ?

Added an env_int function.
Made some internal adjustments to how things were pulled from the environment so that handling of default values could be delegated to each individual env_* function.

How can someone test / see it ?
Look at the README.
Here is a cute animal picture for your troubles...

One small question, otherwise :+1:.


Removing any reference of Conference ID reset from the page

Removing any reference of Conference ID reset from the page since it's no longer supported.

@Mshaikh-msft Please implement the suggested changes. Thanks

@Mshaikh-msft Please implement the suggested changes. Thanks

Done

@get-itips @tseward

@tiburd Please merge this PR. Thanks


smarter js detector logic to trace tree of scripts

necessary for the more complex script setups. complement to disambiguation + [dev] block persistence issue netlify/netlify-dev-plugin#131

Closing due to https://github.com/netlify/cli/issues/1495


Release 1.3.1

Codecov Report

Merging #44 (ac8b233) into master (c92f7df) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #44   +/-   ##
=======================================
  Coverage   85.11%   85.11%
=======================================
  Files          16       16
  Lines         900      900
=======================================
  Hits          766      766
  Misses        134      134

Flag
Coverage Δ

unittests
85.11% <100.00%> (ø)

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

Impacted Files
Coverage Δ

cloudscale_cli/version.py
100.00% <100.00%> (ø)

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 c92f7df...ac8b233. Read the comment docs.


Avoid stomping on an existing FileProvider

https://github.com/aspnet/Identity/blob/dev/src/UI/IdentityDefaultUIConfigureOptions.cs#L62

You fixed this already right?


It keeps pinning all ChatGPT tabs

Describe the bug
I keep various ChatGPT tabs for multiple convos I'm having and use their titles to know which tab is of which convo.
ChatGPTBox keeps pinning these tabs when I reload them, which hides their titles and I'm unable to find them. When I unpin a tab it comes out of its place and I have to move it back every time.
To Reproduce
Steps to reproduce the behavior:

Open multiple convos on normal/non-pinned tabs.
Reload one of these tabs.
The reloaded tab is automatically pinned.
Unpin it.
Reload it again.
It's automatically pinned again.

Expected behavior
I don't know if there are use cases where automatically pinning is useful, but for my use case which relies on tabs title it's annoying.
It seems the automatic pinning was implemented to solve some issue that happens when no ChatGPT tab is opened. But automatically pinning a tab isn't the proper solution. Just warn that at least 1 tab must remain open and let user choose how and where this tab will be.
Please complete the following information):

OS: Win10
Browser: Firefox 112
Extension Version: 2.3.0

You said makes a lot of sense, I will change the behaviour in the next version so that the tab won't be pinned by default, but instead provide a button for the user to decide whether to pin it


rename transformation commands to keep them consistent.

I should have made this a PR - but wasn't thinking, so I can revert if needed.
The latest 0.2.0 release renames some of the transformation functions, to keep more in line with our tokens2counts and tokens2pos commands.
Specifically:

lowercase has become tokens2lower
uppercase has become tokens2upper
stem has become tokens2stem
topbigrams has become tokens2topbigrams

This keeps them all contained together in the --help documentation, and makes it more clear what content they are expecting. The tradeoff is they are slightly more verbose.
Thoughts @iros / @arnicas ?
changed in 2685e9425e092ccb7fb39e428d0304249b33e144

Sorry +1


Fetch Reddit link title

it's a problem with Reddit, the title is always "Reddit - Dive into anything" with auto fetch.
is there a way to fix it? thank you!

not really a solution, but the plugin works fine on old.reddit.com

Me too. It was working all right until like a week ago :(


[Bug]: [Fixed Layout] Text widget in Auto height doesn't hug contents until its prompted

https://user-images.githubusercontent.com/104895602/229086967-c1491717-36e7-497a-ad98-4889f55809f1.mp4

Happens with all text boxes, PFB for stat box

FYI: this is happening with all widgets that has auto height property


Keep assert and with keywords as is

Describe the bug
V8 will remove support for the assert keywords, so Node.js and Chrome will follow suit.

Hey, quick update on this. Node.js is planning to remove support for assert in v22 (which will be released in April) and Chrome in v126 (which will be released in May).

https://github.com/denoland/deno/issues/17944#issuecomment-2027005952
I'm trying to implement this in Deno as well (https://github.com/denoland/deno/pull/23541), but it doesn't work because swc commonizes the assert and with keywords. Please add a new option to pass those keywords as is to V8.
Input code
import foo from "./foo.json" with { type: "json" };
import bar from "./bar.json" assert { type: "json" };
console.log(foo);
console.log(bar);

Config
{
  "jsc": {
    "parser": {
      "syntax": "typescript",
      "tsx": false
    },
    "target": "es2022",
    "loose": false,
    "minify": {
      "compress": false,
      "mangle": false
    },
    "experimental": {
      "keepImportAttributes": true
    }
  },
  "module": {
    "type": "es6"
  },
  "minify": false,
  "isModule": true
}

Playground link (or link to the minimal reproduction)
https://play.swc.rs/?version=1.5.0&code=H4sIAAAAAAAAA8vMLcgvKlFIy89XSCvKz1VQ0tMHsvWyivPzlBTKM0syFKoVSioLUq0UlCBitdZcmRA9SYlFcD1ANlRPYnFxKlASU1dyfl5xfk6qXk5%2BugbQCk1UEaABmtYAe1o9F40AAAA%3D&config=H4sIAAAAAAAAA21QOw6DMAzdOQXy3KFi6NCtY4ceIqUGpU1IZBsJhLg7CSSUSt3i9%2FNzpqIs4c01XMspPMPgFTHSPgeEx07UEBCQ0SPXpL3AKbPCkWqUYVyheWNAFLUo0YVcnasqOcA4x5gdCbO608143Fk76wmZf4VRqrrW4N%2BNOHgkbTG0NcesD6K%2FW%2B9IbiKkn71gzBXqk79IGWDdq1%2FD01%2FEe7cLLvAV5bJ7MdD8yM41dV4AwZk7Q1cBAAA%3D
SWC Info output
No response
Expected behavior
import foo from "./foo.json" with {
    type: "json"
};
import bar from "./bar.json" assert {
    type: "json"
};
console.log(foo);
console.log(bar);

Actual behavior
import foo from "./foo.json" with {
    type: "json"
};
import bar from "./bar.json" with {
    type: "json"
};
console.log(foo);
console.log(bar);

Version
1.5.0
Additional context
related options:
https://github.com/swc-project/swc/blob/c76cd9eccf080da67d1c8d75063f8286dfc4acc1/packages/types/index.ts#L607-L616C9

Closing as not planned. This require modification of AST, but it does not worth it.

@kdy1 Then, could you add importAssertions: boolean to TsParserConfig to disable the assert keywords in TypeScript?

I'll accept a PR, but it should named differently so the default value can be false


Add CIAT subjects to item view sidebar

We had meant to have all center-specific subjects show on the left sidebar if the current item had them. ILRI subjects (and others) show up for sure, but CIAT (and others) is missing.

ILRI item: https://cgspace.cgiar.org/handle/10568/70108
CIAT item: https://cgspace.cgiar.org/handle/10568/70052

The list is configured somewhere in the CGIAR base theme (0_CGIAR), so we just need to double check that all centers that have specific subject metadata are present there so children themes can inherit the logic.

I just had a look and these are in item-view-DIM-helper.xsl in the 0_CGIAR base theme. The following subjects are missing: bioversity, ciat, cta, humidtropics, iita.

An example item from CCAFS that includes both CCAFS and Bioversity subjects. First from CGSpace (without patch), only the CCAFS subjects are visible:

After adding the correct templates to the XMLUI theme both subjects are visible:

I fixed a corner case with it failing to print a space, but now I notice that sometimes there is a semicolon at the end (even in the above screenshot), but it's like this on production already and I never noticed. I will merge this now and just look at that as a separate issue later.


Update KKBOX to 5.2.80

Thank you for the contribution. It was merged directly as 60245402bd4d929167cd3a796b5a7522fca119ee to keep commit history cleaner. Your contribution is still credited to you.


Add routing handler

This would be very useful after allowing data to cascade.
It could also be implemented as a plugin.

Can you explain what you mean by a routing handler?

By routing handler I meant a feature that provides routing, so that we can define our app's routes paths and the components that should be loaded for every route.

This sounds like something you can solve through conditionals in the --body. With #26, this should become easier to use.

On 21 Jun 2019, at 11:55, Okba<EMAIL_ADDRESS>wrote:
By routing handler I meant a feature that provides routing, so that we can define our app's routes paths and the components that should be loaded for every route.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


ci(pytest): cleanup tox.ini and pytest.ini

Previously the test root directory was configured both in the tox.ini and the
pytest.ini.
This patch moves the configuration into the pytest.ini and fixes some
whitespaces too.
Signed-off-by: Florian Scherf<EMAIL_ADDRESS>

Codecov Report

Merging #209 (d6f8a93) into master (4ed11fa) will decrease coverage by 0.17%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #209      +/-   ##
==========================================
- Coverage   61.71%   61.53%   -0.18%
==========================================
  Files          67       67
  Lines        4764     4763       -1
  Branches      921      784     -137
==========================================
- Hits         2940     2931       -9
- Misses       1570     1583      +13
+ Partials      254      249       -5

Impacted Files
Coverage Δ

lona/html/abstract_node.py
88.57% <0.00%> (-2.00%)
:arrow_down:

lona/server.py
68.02% <0.00%> (-0.95%)
:arrow_down:

lona/view.py
54.28% <0.00%> (-0.58%)
:arrow_down:

lona/html/widget_data.py
32.38% <0.00%> (-0.48%)
:arrow_down:

lona/view_runtime.py
68.27% <0.00%> (-0.29%)
:arrow_down:

lona/html/node.py
59.37% <0.00%> (ø)

lona/html/widget.py
57.40% <0.00%> (ø)

lona/html/node_list.py
57.14% <0.00%> (ø)

lona/shell/commands/lona_server_state.py
0.00% <0.00%> (ø)

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 4ed11fa...d6f8a93. Read the comment docs.

@maratori: Thanks for reviewing :)


Not getting locations in Samsung devices

Describe the bug
This plugin works well  in android and ios but only the issue found in samsung devices. it doesnt give any data in samsung devices.
To Reproduce
Install the plugin and open in samsung note 8 with android  9.
it gives the stationary location some times but not any other location.
Expected behavior
Should work in all the devices
Screenshots
Desktop (please complete the following information):
Smartphone (please complete the following information):
Samsung note 8 with android  9
Additional context
Given the battery optimization and still it doesnt work

Please share the configuration and plugin logs. Otherwise I'll be forced to close this issue.

@HarelM
// locationProvider: BackgroundGeolocation.DISTANCE_FILTER_PROVIDER,
locationProvider: BackgroundGeolocation.ACTIVITY_PROVIDER,
// locationProvider: BackgroundGeolocation.RAW_PROVIDER,
desiredAccuracy: BackgroundGeolocation.HIGH_ACCURACY,
debug: true,
stationaryRadius: 1,
distanceFilter: 2,
interval: 10,
fastestInterval: 10,
activitiesInterval: 30,
startForeground: true,
notificationIconLarge: "screen",
notificationIconSmall: "screen",
this is my configuration and i tried all the providers, and when i use activity provider and debug true it shows a popup detected Still activity but nothing else

How can debug plugin logs ?

Raw provider is your best bet to see any activity.
Getting the plugin logs is done by calling getLogs (or something similar, not in front of a PC right now).
I would advise to open a GPS app to see that the GPS on the device actually sends location events...

when i check status  it says that   services enabled false in samsung devices.other devices working fine. i will try to get logs right now. i tried all providers

when i check the logcat in android studio i found this @HarelM
2022-07-09 12:44:58.890 671-671/? E/audit: type=1400 audit(1657350898.883:397990): avc:  denied  { write } for  pid=3743 comm="atrace" name="enable" dev="tracefs" ino=7036 scontext=u:r:shell:s0 tcontext=u:object_r:debugfs_tracing_debug:s0 tclass=file permissive=0 SEPF_SM-A536E_12_0001 audit_filtered
2022-07-09 12:44:58.898 3737-3737/? E/GRPC: server_chttp2.cc:40] {"created":"@1657350898.898531869","description":"No address added out of total 1 resolved","file":"external/grpc_repo/src/core/ext/transport/chttp2/server/chttp2_server.cc","file_line":396,"referenced_errors":[{"created":"@1657350898.898519869","description":"Unable to configure socket","fd":3,"file":"external/grpc_repo/src/core/lib/iomgr/tcp_server_utils_posix_common.cc","file_line":217,"referenced_errors":[{"created":"@1657350898.898479677","description":"Address already in use","errno":98,"file":"external/grpc_repo/src/core/lib/iomgr/tcp_server_utils_posix_common.cc","file_line":190,"os_error":"Address already in use","syscall":"bind"}]}]}
2022-07-09 12:44:58.899 3737-3737/? E/StudioTransport: Server failed to start. A port number wasn't bound.

this error getting with the time interval i set in the config. Anyone having a samsung device with same issue?? found in galaxy series too

@ivinantony Have you managed to fix the issue somehow? I also have Samsung (S21) and if I use the ACTIVITY_PROVIDER no events are fired. For the same configuration it is enough to switch to RAW_PROVIDER and events are properly fired. All of them.

I also not found any solution foe samsung devices. Which provider works for you??

No clue, I use raw provider and haven't heard any complains.
The following is my configuration:
https://github.com/IsraelHikingMap/Site/blob/main/IsraelHiking.Web/src/application/services/geo-location.service.ts

RAW_PROVIDER is not battery usage sensitive. @ivinantony do you maybe tried the DISTANCE_FILTER_PROVIDER?

Currently i am also using rawprovider and one user told its not working for him in samsung s20 ultra and android 12. Let me compare my configuration with @HarelM


ISPN-9818 upgrade tomcat version

https://issues.jboss.org/browse/ISPN-9818

@jsenko I upgraded this component that seems used in the integration tests of spring boot mainly. Is it a problem for datagrid?

@wburns changes reported


Addition of the HDMI OUT toggle within the integration

My Yamaha reciever (RX-V773) has the ability within the Yamaha AVR app to toggle the various HDMI outputs. It features two outputs which are both mirrored and attached to Zone 1 within the receiver. This gives me the ability to sometimes use my TV and sometimes use my projector depending on the situation.
I was wondering if it might be possible to add the ability to toggle these HDMI outputs within HA as can be done within the Yamaha AVR app. I've attached a screenshot of what the setting looks like in the app.
Im happy to help test and pull logs from the device if necessary as I understand not all receivers may have this function.

You are in luck :)
I recently started on that by a request in https://github.com/mvdwetering/yamaha_ynca/discussions/119
I was close to finished, but then there was an issue with the HA 2023.6.0 release which required some immeditate attention due to the connection issue. I will pick this up again soon.
(that is assuming your receiver uses the same commands as the TSR-700, but that is very likely it does)

That's awesome. Let me know if you'd like me to complete any testing on my unit once you've completed the feature and I'll keep and eye out on the upcoming release notes. Cheers!

I just release 7.4.0 which added support for HDMIOUT toggles on TSR-700, I expect that your receiver uses the same commands and should therefore also work.

That's awesome. Thank you very much. I'm still on HA 2<IP_ADDRESS> so once my provider TrueCharts pushes the update to 2023.6.0 I'll be able to update the integration and test it. I'll let you know if there are any issues. Thanks for your work!

I just updated and that seems to have worked a treat. Thank you very much for your hard work!

Good to hear it works.
I will be closing the issue as it has been implemented


[bug] Update error distribution while annotating

If possible we should update this in real time:

Resolved in #1103


Introduce special syntaxes to modify existing array fields

This is just an idea to solve #22.
This PR introduces two special syntaxes to construct new array:

${field_name} + ["to be added 1", "to be added 2"]
${field_name} - ["to be removed 1", "to be removed 2"]

For more details, please see descriptions added to the README.
I think this will help me to define a post process filter to modify existing array fields like https://github.com/clear-code/sharetary/blob/master/sample/fluent-plugin-github-activities.conf#L343
On the other hand, this PR possibly breaks backward compatibility. If there is any existing configuration to output string like above syntaxes, it will construct unexpected results...

Can you do this with enable_ruby true without applying this patch?

Introducing this kinds of syntax makes things be complex like

why don't you support * operator, or other operators for the array
don't you support number addition?

I instead already supported enable_ruby option. I am not willing to merge this.

I couldn't find out how to do this without patch, because there seems to be no way to refer all (unknown number of) members of existing array field and expand them as is. Instead I only can refer just specific member with its known index.
Anyway, I agree that such a special syntax makes thing complex.

Writing conf like:
  <source>
    type dummy
    tag dummy
    dummy {"message":"foo","time":"1432732710","members":["Alice"]}
  </source>
  <match dummy>
    type record_reformer
    renew_time_key time
    tag reformed.${tag}
    enable_ruby true
    <record>
      members ${members + ["foo"]}
    </record>
  </match>
  <match reformed.**>
    type stdout
  </match>

outputs like
2015-05-27 22:18:30 +0900 reformed.dummy: {"message":"foo","time":"1432732710","members":"[\"Alice\", \"foo\"]"}

So, the problem is that record-reformer treats (almost) everything as strings.
Supporting conversion of such an array form with fluent-plugin-typecast would be sweet.

Thank you for the quick response and explanation of the alternative way based on the typecast.
I close this PR because I have no idea to solve future issues you'll be worried.

So, the problem is that record-reformer treats (almost) everything as strings.

Another idea, how about it?: output the value of a field in the type same to the source field's one, if both source and destination fields' name are also same. For example, if the source is {"array":["Alice"]} and the rule is array ${array + ["Bob"]} then the output will be {"array":["Alice","Bob"]}.
This behavior sounds natural as a filter. The restriction "the out_record_reformer outputs any reformed field's value as a string, and the value always have to be re-converted into expected type by another plugin filter/out_typecast" looks good for complete backward compatibility, but looks confusable for beginners including me.

For example, if the source is {"array":["Alice"]} and the rule is array ${array + ["Bob"]} then the output will be {"array":["Alice","Bob"]}.

Currently, record-reformer supports a config like
array ${array + ["Bob"]}foo

so this idea breaks lower version compatibility.
What I am thinking is to provide type specification, but it is not done yet because I could not come up with good syntax to provide it before.
It would become just like typecast plugin, so I am sending a PR for fluentd itself to make typecast plugin be a bundled plugin https://github.com/fluent/fluentd/pull/589 (this is especially for the already bundled record_transformer).
Introducing types option for record-reformer and record-transformer would be another way, the implementation is very easy because Fluentd api is available > https://github.com/sonots/fluent-plugin-filter_typecast/blob/master/lib/fluent/plugin/filter_typecast.rb


Fix typo in simple_tensorflow_serving

Fix missed  f in self.app = Flask("simple_tensorflow_serving", template_folder='templates')

Nice catch and thanks for your contribution :)


Provide Compliance Data in JSON

Benefit: Users can download each of the following groupings for analysis in JSON format
Allowance-Based Compliance (For each trading program)
NOx Averaging Compliance (ARP)
*need to determine if these groupings are appropriate

Duplicate


beautify the list for form-select components

Why not compose the dropdown component :

<select> is the native HTML tag that handles picking values itself. dropdown is a visual menu without functionality. You can put links in it.
But if you want dropdown to work as a functional <select> element, you should write custom JS to handle the functionality (or use libraries like Headless UI - [example])
I wish there was a way to style the elements of native <select> element using CSS but it's not possible.


Add the SpiHandle:Config struct to SSD130x4WireTransport:Config

This is a fix for issue #506. It seems a number of people have been running into this, and it was an oversight on my part, so I fixed it.
@stephenhensley - I added constructors to the Config structs that invoke Defaults() as you mention in the issue description.  This should be fully backward compatible, but I don't have time to test against an SPI transport OLED.  Perhaps you could quickly verify this against a Daisy Patch?  Otherwise it's little more than a little code shuffling, and should not introduce any breaking changes.
Cheers,
Sam

@recursinging Thanks, Sam!
This looks great, I saw some of the forum activity you're talking about, and was hoping we'd be able to carve out some time this month to get this in there.
Thanks for making it happen!
Looks good to me. We'll do a quick check that everything still works with the existing Daisy Patch code, and get it merged.


metadata validation at dataset level not feature level

the properties mentioned in inspire profile validation at https://github.com/soilwise-he/SoilWise-documentation/blob/main/tech/docs/technical_components/metadata_validation.md are mentioned in the SO spec for metadata at feature level, these properties are well embedded in the INSPIRE data model and are outside the scope of dataset level metadata, my suggestion is to remove them here, instead we could describe the elements which are required by inspire at dataset level, which are not in the EUSO profile. But I doubt if this is usefull.

Thank you for the suggestion. I agree that the current implementation of INSPIRE validation is not particularly useful for the SWR metadata. However, since it is already integrated within Hale and, to my knowledge, was one of the initial requirements, it can still serve as an example for the first iteration, demonstrating that "we tried it, but it didn't work as expected."
Regarding the INSPIRE dataset-level metadata, it could potentially be added in the next iteration. However, I am uncertain if it makes sense to maintain both INSPIRE and EUSO profiles. At this stage, it appears that only the EUSO profile will be retained in the subsequent iterations.


Uncaught ReferenceError: Buffer is not defined

When executing wrangler publish I'm getting a Buffer is undefined.

same issue here. @elsell how did solve this?
I did both npm install -g<EMAIL_ADDRESS>and npm install<EMAIL_ADDRESS>It still gives the same error
cli output 👇 (click to expand)
$ wrangler publish
 Running npm install && npm run build

up to date, audited 611 packages in 3s

52 packages are looking for funding
  run `npm fund` for details

3 moderate severity vulnerabilities

To address all issues, run:
  npm audit fix

Run `npm audit` for details.

><EMAIL_ADDRESS>build
> webpack

asset worker.js 90.9 KiB [compared for emit] (name: main) 1 related asset
modules by path ./node_modules/@glenstack/ 21.7 KiB
  modules by path<EMAIL_ADDRESS>6.52 KiB 6 modules
  modules by path<EMAIL_ADDRESS>11.8 KiB
<EMAIL_ADDRESS>1.28 KiB [built] [code generated]
<EMAIL_ADDRESS>1.26 KiB [built] [code generated]
<EMAIL_ADDRESS>2.78 KiB [built] [code generated]
<EMAIL_ADDRESS>2.64 KiB [built] [code generated]
<EMAIL_ADDRESS>3.87 KiB [built] [code generated]
<EMAIL_ADDRESS>3.33 KiB [built] [code generated]
./src/index.ts 1.02 KiB [built] [code generated]
./node_modules/tweetnacl/nacl-fast.js 60.5 KiB [built] [code generated]
crypto (ignored) 15 bytes [built] [code generated]
webpack 5.40.0 compiled successfully in 1299 ms
 Build completed successfully!
Error: Something went wrong with the request to Cloudflare...
Uncaught ReferenceError: Buffer is not defined
  at line 357 in btoa
  at line 361 in exports.authorization
  at line 222 in<EMAIL_ADDRESS>  at line 68 in<EMAIL_ADDRESS>  at line 3073
  at line 3083
  at line 3085
 [API code: 10021]

Hi @chocnut maybe it will help you
if (typeof Buffer === 'undefined' && typeof global !== 'undefined') {
    global.Buffer = {
      // @ts-ignore
      isBuffer: s => typeof s === 'string',
      // @ts-ignore
      from: s => ({ toString: (m) => m === 'base64' ? encode(s).toString() : (m || '') }),
      // @ts-ignore
      byteLength: s => s.length,
    };
  }

I forked this project and applied the fix you suggested. I had to fix a reliance on Buffers in one more place, but now it works!
https://github.com/Code6226/glenstack/tree/master/packages/cf-workers-discord-bot


Running the worker on a remote system.

Hi
This is not really a bug, but rather a question about the possibility of running the worker on different system
than the actual wrapper library and communicate over TCP/IP instead of sockets, which would allow any architecture
and OS to use this code while running the worker on a "remote" linux vm or whatever.
Thanks

That'd probably induce much more overhead than current implementation that is based on shared memory.

The other missing piece is some kind of daemon sitting in a VM and spawning those worker processes (there is one worker per CDM instance) and that is, honestly, out of scope.


card styled

Please change the folder structure to get accepted . Happy coding :wink:
Cards -> cards-1 -> "files"

keep contributing 👍


Next Greater Element III

Enter your question -
Given a positive integer n, find the smallest integer which has exactly the same digits existing in the integer n and is greater in value than n. If no such positive integer exists, return -1.
Note that the returned integer should fit in 32-bit integer, if there is a valid answer but it does not fit in 32-bit integer, return -1
Enter link to the question(if question belongs to any online platform) -
https://leetcode.com/problems/next-greater-element-iii/
Tags for the question(eg - Array, Basic, Stack, etc.) -
Math ,two pointer, string

for hacktober fest can you please assign me this question of leetcode. I will Try to solve this problem

@nik132-eng,
Kindly add your solution to "leetcode" folder.
Deadline - 03/10/2021


[Settings] ComboBox fix on FZ page

Summary of the Pull Request

Removed the TextBlock that was acting as a Header and used the Header property of the ComboBox to fix inconsistent spacing.
Set the IsEnabled property to resolve: #10089

Quality Checklist

[X] Linked issue: #10089
[ ] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[ ] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

@niels9001
thanks for the fix!


error: conflicting declaration 'typedef char int8_t'

(Note that I'm a bit noobish to c++, especially when it comes to using other people's libraries.)
Using example windows code (with the included cpp), the program broke on run time.
Here's the full error:
||=== Build: Debug in ClientTest (compiler: GNU GCC Compiler) ===|
easywsclient.cpp|12|warning: ignoring #pragma comment  [-Wunknown-pragmas]|
main.cpp|4|warning: ignoring #pragma comment  [-Wunknown-pragmas]|
easywsclient.cpp|33|error: conflicting declaration 'typedef char int8_t'|
stdint.h|27|error: 'int8_t' has a previous declaration as 'typedef signed char int8_t'|
easywsclient.cpp|35|error: conflicting declaration 'typedef long int int32_t'|
stdint.h|31|error: 'int32_t' has a previous declaration as 'typedef int int32_t'|
easywsclient.cpp|36|error: conflicting declaration 'typedef long unsigned int uint32_t'|
stdint.h|32|error: 'uint32_t' has a previous declaration as 'typedef unsigned int uint32_t'|
easywsclient.cpp||In function 'socket_t {anonymous}::hostname_connect(const string&, int)':|
easywsclient.cpp|92|error: '_snprintf_s' was not declared in this scope|
easywsclient.cpp|93|error: 'getaddrinfo' was not declared in this scope|
easywsclient.cpp|108|error: 'freeaddrinfo' was not declared in this scope|
easywsclient.cpp||In function 'easywsclient::WebSocket* {anonymous}::from_url(const string&, bool, const string&)':|
easywsclient.cpp|468|error: '_snprintf_s' was not declared in this scope|
||=== Build failed: 10 error(s), 2 warning(s) (0 minute(s), 1 second(s)) ===|
It's probably me doing something stupid, but I was just wondering what went wrong here. (Again, using the example code and including the cpp file.)
Windows 10, using Code Blocks.

Hi @Kertopher, this code hasn't been yet written to work with the Code::Blocks / MinGW compiler; But let me know what steps you do to get it working. And adding these compiler flags might help you get a head start. Let me know:
-Dsnprintf=snprintf
-D_WIN32_WINNT=0x0501
-D_MSC_VER=1600

@dhbaird Has this been optimized for windows yet?


Add game replay saving, add checkpoint callback to save some replays along with the models

By default saves 5 replay matches (itself against itself) every 100K steps during training along side the model. Saved to .\models\.
Adds:

Game() ability to write replay outputs.
SaveReplayAndModelCallback callback that can be used in training to save both model checkpoints along-side replays. Useful for examining the behavior that your model is learning.
Change example training script default param n_steps to be the same as batch size.
Action sequences now takes additional argument pointing to current game for better programmatic actions. I think this feature is unused, but will cause a failure if anyone else is using actions sequences unfortunately.

Note, I've done more testing and this isn't ready to merge yet. It seems to have some bugs now that I tested more replays, working on preparing a fix now.

Great idea with replays. Can be nice to have seed in the replay filename. Will it be possible to reproduce the replay with a manual run? Similarly to how it is reproduced in tests.


Add Solution for LeetCode Problem 1315

Add Solution for LeetCode Problem 1315. Sum of Nodes with Even-Valued Grandparent
Given the root of a binary tree, return the sum of values of nodes with an even-valued grandparent. If there are no nodes with an even-valued grandparent, return 0.
A grandparent of a node is the parent of its parent if it exists.

Example 1:

Input: root = [6,7,8,2,7,1,3,9,null,1,4,null,null,null,5]
Output: 18
Explanation: The red nodes are the nodes with even-value grandparent while the blue nodes are the even-value grandparents.

Example 2:

Input: root = [1]
Output: 0

Constraints:

The number of nodes in the tree is in the range [1, 104].
1 <= Node.val <= 100

Great implementation


Which features should we pass to ARIMA?

Currently, if the time series problem is multivariate, ARIMA will receive an exact copy of the input data to to fit on.
Reproducer,
from evalml.automl import AutoMLSearch
from unittest.mock import patch
from evalml.demos import load_weather
import pandas as pd

ARIMA_PATH = "sktime.forecasting.arima.AutoARIMA"

SIZE_OF_FIRST_SPLIT = 914

X, y = load_weather()
X.ww["other_feature"] = y - 1

automl = AutoMLSearch(X, y, "time series regression", verbose=True,
                      problem_configuration={"date_index": "Date",
                                             "forecast_horizon": 1,
                                             "gap": 0,
                                             "max_delay": 10})

with patch(f"{ARIMA_PATH}.fit") as mock_fit, \
    patch(f"{ARIMA_PATH}.predict", side_effect=lambda fh, X: pd.Series(range(X.shape[0]))):
    automl.search()

expected_answer = X.ww.copy()
expected_answer = X.ww.iloc[:SIZE_OF_FIRST_SPLIT, :]
expected_answer.index = pd.DatetimeIndex(expected_answer['Date'], "D")
expected_answer = expected_answer.ww.drop(["Date"])

pd.testing.assert_frame_equal(mock_fit.call_args_list[0].kwargs['X'], expected_answer)

I think this is problematic because the other_feature in this case may not be known in advanced and this could be a case of data leakage - training a model on data that would not be available at predict time.
The other pipelines receive the input features after feature-engineering, namely lagging and soon-to-be rolling windows, so the other_feature would not be given as-is to the pipeline. For that reason, the other pipelines do not suffer from the same problem as ARIMA.
We need to decide:

Should we even pass features to ARIMA? Why not just pass in the target values
If we want to pass in some features, we need to make sure there is no leakage.

Step 1 of this issue is reaching consensus on the above decision, then we can decide how to best implement that.

This was fixed in #3313


Setup existing DW tests to run in CFO

Not needed for 1.35 (2.4).
this will be done in the future

OpenShift CI adjusted in https://github.com/openshift/release/pull/45214
Migrated tests are executed for every PR in ODH/CFO.


Use original src and width for srcset if sizes are not provided

fixes https://github.com/frontity/saturn-theme/issues/137

:tada: This PR is included in version 0.9.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


ci: add cruft autoupdate GitHub Action

Summary
Add a GitHub Action so downstream projects can autoupdate via cruft with this template. Note that this is currently only possible with public HTTPS links when initially running cruft create or cruft link.
I explored with SSH links, but it seems I'd need to add deploy keys to get this working, or add a secondary actions/checkout step in the Workflow that accepts a GitHub PAT token. This might be prohibitive for some users, especially those with downstream private repositories. For these users, the current manual syncing process remains.
Checklists
I/we (contributor) confirm that the code, and analyses in this Pull Request (developments) meets the following requirements:

[x] code runs
[x] developments are ethical, and secure
[x] contributor has made proportionate checks that the developments are correct
[x] minimum usable documentation is written in plain English in the docs folder
[x] all pre-commit hooks pass
[x] test suite passes
[x] code coverage is maintained, or increased

Additional comments
N/A

I've also removed caching when making the Poetry installation, as this seems to regularly break.


“Implement a chat message list” refers to “ChatScreenChat” instead of “ChatScreenState”

Step 8 under “Implement a chat message list” refers to “ChatScreenChat” instead of “ChatScreenState”

Fixed. Marking as closed.


The proxy shouldn't ask the destination service to resolve localhost addresses

This is an extension of #357. The following logic from https://tools.ietf.org/html/rfc6761#section-6.3 should be implemented in the proxy:

The domain "localhost." and any names falling within ".localhost."
are special in the following ways: [...]

Name resolution APIs and libraries SHOULD recognize localhost
names as special and SHOULD always return the IP loopback address
for address queries [...] Name resolution APIs SHOULD NOT send queries for
localhost names to their configured caching DNS server(s).

In particular, we shouldn't send queries for localhost names to the Destinations service.

This is a little more complicated than #357 since we have to decide what to do about IPv6. One solution is assume localhost addresses resolve to <IP_ADDRESS> and then fix this when we implement proper IPv6 support (#91).

This is blocked on #155


Update New-HTMLText.ps1

Removed ")" exclusion from $FindMe - any string in [text](url) format should now match, regardless of number of brackets in url.
No longer require lazy matching for $LinkURL - it should always end in ")" which can then be excluded.
Should fix #200

Great! Seems to work well