Only build the Boost libraries we do actually use

This should reduce rare build time dramatically.
Thanks to the amazing work of @bincrafters we can do this now much more easier than we our current Conan boost package.
I can do once again the CI/build part, but I have actually no idea what Boost libs we do require nor how to easily find that out.
@aschaeffer @a-teammate Could somebody of you please provide a list with the Boost libs we are currently using?

I think the list is in cmake/require_thirdparty_dependencies.cmake

Thanks!

find_package(Boost REQUIRED
COMPONENTS
thread
random
filesystem
regex
program_options
system
)

I'm sorry to say but the Bincrafters packages are not currently compatible with the find facilities of CMake.  They can only be brought in by using the conanbuildinfo.cmake and linked to with ${CONAN_LIBS}.  This is one of a few unfortunate challenges which exist in the space between CMake's design of find_package and systems like Conan which dynamically generate CMake files.

@solvingj We are currently using @lasote's boost package. What are you doing differently that CMake find_package doesn't work?

Ah okay good to know! so we need to reference the above list of components individually in our dependencies.py and afterwards import them in cmake as we do it with separate conan libs.
Could work :)
Good that our list is so short currently :)

Would it somehow be possible to create an empty super package with some custom findBoost methode, where every single Boost Library package is registering itself there?
-- conan-boost - "empty target", searches for existing Boost.xyz targets, exports custom findBoost.cmake methode
-> Install conan-boost-thread
-- conan-boost - finds Boost.Thread, CMake find methode can find Boost.Thread through findBoost
---- conan-boost-thread

The problem is that conan install and cmake generation are operating at different stages.
You first do conan install (with all boost libs) and afterwards cmake.. (otherwise nothing can be found since nothing was installed)
Which would bring us back to the beginning.
A solution would be to couple cmaking and conan installing less loosely. i.e. by letting cmake execute conan install each time another boost lib got requested. I just doubt that's worth the effort.

Yes, but can't CMake just look for other CMake targets at runtime?

Yes, it's perfectly workable.  The unfortunate thing I was pointing out is that we can't satisfy the existing findBoost() stuff that's out there, so our packages don't just "drop in" to that.  You have to change your CMake configs to use our packages, and we're a bit disappointed about that.

Actually, we just had a promising suggestion come in here:
https://github.com/bincrafters/community/issues/26
If you have time, please take a look and potentially give it a try. I'll also have some of our team try it out as well, but your feedback would be invaluable.

The main motivation to switch to a modularized Boost package was the build time. In the meantime we had several huge changes to our build chain, including that we are now linking in the package directly into our own Conan repository (#599).
While I'm still happy that there is a modularized Boost package I don't think any longer that Inexor would benefit in a meaningful way from it (because of all the changes).
It would be nice to get this PR merged (cc @lasote) https://github.com/conan-community/conan-boost/pull/124 but beyond that it seems quite fine as it is right now.
Thanks for all the input and work! 😄


Pdf

Please add pdf support sir🥹

coming soon...😊

it will be possible to read books with images etc right? like how normal pdf viewer works or it will only display text?
any update regarding this?

well this is what i am talking about if it shows like this then awesome. if only text then issues


[ADD] DCOS-2301 pip version check to install script

@ssk2 Please review

@ytalashkocv - how did you test this? I can't figure out how to install an old version of pip.

@ytalashkocv a couple of comments - this breaks the install script on Linux. Not sure how to test it either.

@ssk2 For now version checks without bc. Please review.

Thank you @ytalashkocv!


Make Jenkins stash names unique

Change Jenkins stash names to be unique
fixes #1524

bors r+

bors r+


Polyfill issue with vueJs3 and Webpack5

Hi everyone,
I am facing an issue with a fresh install of VueJs 3 with TypeScript. It seems Webpack 5 does not allow auto-polyfilling anymore
and a lot of your dependencies cannot be resolved.

I cannot apply the suggested workaround in webpack.config.js as this file does not exist in my project.
Installing dependencies in my project and defining missing paths in tsconfig.json does not seem to work either.

Is there any workaround for my case or is it just impossible for now to use with VueJs3?
Thanks in advance
Environment:

Node.js Version:  16
Redis Server Version: 6.2.6
Node Redis Version: 4.0.4
Platform: Bullseye

ERROR in<EMAIL_ADDRESS>8:17-34

Module not found: Error: Can't resolve 'crypto' in '/app/node_modules/@node-redis/client/dist/lib'

BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
	- add a fallback 'resolve.fallback: { "crypto": require.resolve("crypto-browserify") }'
	- install 'crypto-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
	resolve.fallback: { "crypto": false }

node-redis is not intended to run in the browser, what are you trying to do?

node-redis is not intended to run in the browser, what are you trying to do?

Ok that's what I feared , I've read that a couple of times since. I just want to store the result of an API call, so I'm creating a redis client into my Vue component to get the cached data if exists or call my endpoint with axios. It seems I'm misusing something.


IngressRoute should provide additional printed columns

Kuberentes v1.11.0 introduced a new feature for CRDs called server server side printing.  Basically this allows you to define additional columns in your CRD that will be output when kubectl get is run.
As an example:
apiVersion: apiextensions.k8s.io/v1beta1
  kind: CustomResourceDefinition
  metadata:
    name: crontabs.stable.example.com
  spec:
    group: stable.example.com
    version: v1
    scope: Namespaced
    names:
      plural: crontabs
      singular: crontab
      kind: CronTab
      shortNames:
      - ct
    additionalPrinterColumns:
    - name: Spec
      type: string
      description: The cron spec defining the interval a CronJob is run
      JSONPath: .spec.cronSpec
    - name: Replicas
      type: integer
      description: The number of jobs launched by the CronJob
      JSONPath: .spec.replicas
    - name: Age
      type: date
      JSONPath: .metadata.creationTimestamp

I think it would be a nice feature to add column outputs for IngressRoute CRDs.

As an example, this is what output might look like:
$ k get ingressroute
NAME      FQDN              ALIASES            TLS SECRET   FIRST ROUTE   STATUS    STATUS DESCRIPTION
apps      apps.heptio.com                                   /             valid     valid IngressRoute
blog                                                        /blog         valid     valid IngressRoute
heptio    heptio.com        [www.heptio.com]   heptio-tls   /             valid     valid IngressRoute

Relevant upstream issues to improve the First Route column to output multiple paths (and improve array output like Aliases): https://github.com/kubernetes/kubernetes/issues/67268 and https://github.com/kubernetes/kubernetes/pull/67079


startgnome not working on pixelbook for kali-rolling

name: kali-rolling
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/kali-rolling...
crouton: version 1-20180127033952~master:de1361af
release: kali-rolling
architecture: amd64
xmethod: xorg
targets: gnome-desktop
host: version 10032.86.0 (Official Build) stable-channel eve
kernel: Linux localhost 4.4.86-11979-g09b49365a3c4 #1 SMP PREEMPT Mon Jan 8 23:19:25 PST 2018 x86_64 GNU/Linux
freon: yes
Unmounting /mnt/stateful_partition/crouton/chroots/kali-rolling...

chronos@localhost ~/Downloads $ sudo startgnome
Entering /mnt/stateful_partition/crouton/chroots/kali-rolling...
dbus[11327]: Unknown username "geoclue" in message bus configuration file

_XSERVTransmkdir: Owner of /tmp/.X11-unix should be set to root

X.Org X Server 1.19.6
Release Date: 2017-12-20
X Protocol Version 11, Revision 0
Build Operating System: Linux 4.9.0-5-amd64 x86_64 Debian
Current Operating System: Linux localhost 4.4.86-11979-g09b49365a3c4 #1 SMP PREEMPT Mon Jan 8 23:19:25 PST 2018 x86_64
Kernel command line: cros_secure console= loglevel=7 init=/sbin/init cros_secure oops=panic panic=-1 root=/dev/dm-0 rootwait ro dm_verity.error_behavior=3 dm_verity.max_bios=-1 dm_verity.dev_wait=1 dm="1 vroot none ro 1,0 4710400 verity payload=PARTUUID=3f2571c0-62b0-d841-90eb-59db0e1c63cc/PARTNROFF=1 hashtree=PARTUUID=3f2571c0-62b0-d841-90eb-59db0e1c63cc/PARTNROFF=1 hashstart=4710400 alg=sha1 root_hexdigest=aedbb505e6b4fbab947c7d148c244a7728e62645 salt=24ba52177cab9d5967052e6d94dd31474990b5017bff3a9d70695e100d4802c1" noinitrd vt.global_cursor_default=0 kern_guid=3f2571c0-62b0-d841-90eb-59db0e1c63cc add_efi_memmap boot=local noresume noswap i915.modeset=1 tpm_tis.force=1 tpm_tis.interrupts=0 nmi_watchdog=panic,lapic
Build Date: 26 January 2018  04:30:21PM
xorg-server 2:1.19.6-1 (https://www.debian.org/support)
Current version of pixman: 0.34.0
        Before reporting problems, check http://wiki.x.org
        to make sure that you have the latest version.
Markers: (--) probed, (**) from config file, (==) default setting,
        (++) from command line, (!!) notice, (II) informational,
        (WW) warning, (EE) error, (NI) not implemented, (??) unknown.
(++) Log file: "/tmp/Xorg.crouton.1.log", Time: Fri Feb  2 19:28:29 2018
(==) Using system config directory "/usr/share/X11/xorg.conf.d"
Error org.freedesktop.DBus.Error.UnknownMethod: Method "ReleaseDisplayOwnership" with signature "" on interface "org.chromium.LibCrosServiceInterface" doesn't exist

method return time=1517617709.844501 sender=:1.12 -> destination=:1.105 serial=5135 reply_serial=2
   boolean true
xf86EnableIOPorts: failed to set IOPL for I/O (Operation not permitted)
crouton: version 1-20180127033952~master:de1361af
release: kali-rolling
architecture: amd64
xmethod: xorg
targets: gnome-desktop
host: version 10032.86.0 (Official Build) stable-channel eve
kernel: Linux localhost 4.4.86-11979-g09b49365a3c4 #1 SMP PREEMPT Mon Jan 8 23:19:25 PST 2018 x86_64 GNU/Linux
freon: yes
Terminated
Running exit commands...
/usr/bin/xinit: connection to X server lost
Hangup

waiting for X server to shut down Error org.freedesktop.DBus.Error.UnknownMethod: Method "TakeDisplayOwnership" with signature "" on interface "org.chromium.LibCrosServiceInterface" doesn't exist

method return time=1517617712.939902 sender=:1.12 -> destination=:1.107 serial=5137 reply_serial=2
   boolean true
(II) Server terminated successfully (0). Closing log file.

Unmounting /mnt/stateful_partition/crouton/chroots/kali-rolling...
Sending SIGTERM to processes under /mnt/stateful_partition/crouton/chroots/kali-rolling...

If known, describe the steps to reproduce the issue:
Typing startgnome causes the screen to go black, a blaack mouse appears in the centre of the screen, and then it crashes.
Note that kde and xfce works fine, I prefer gnome though.

@ShadowRi5ing,
Not sure about kali but gnome works on Ubuntu trusty & xenial for me.
Maybe try adding the 'xiwi' xmethod and see how it works for you.
Hope this helps,
-DennisLfromGA

Just tried xenial and gnome works fine. Xiwi also doesn't work with gnome on kali. I think it may be a bug with the crouton installation of kali. I've tried uninstalling and installing about 4 times.


Internalised profile declarations to be inferred based on type/platform

Fixes #67

Coverage increased (+0.1%) to 88.385% when pulling a1c33d8510cb0d4b177c5134c35f325ac1bc6f48 on task-internalise-profiles into 1ef31d602454d4f7451cbf68a3c43951c208cf6e on develop.


feat: Added Vanuatu Vatu to currency list

Description

Added missing currency for Vanuatu

Type of change

New feature (non-breaking change which adds functionality)
Addition to list of constants

/ok-to-test sha=b48a35b

@mohanarpit can you help merge this PR

@nzidol Thank you so much for your contribution towards making Appsmith a better product. 🎉


[BP-1.13][FLINK-23947] Improve logging of granting/revoking leadership in DefaultDispatcherRunner

Backport of #17033 to release-1.13.

Rebased the branch to fix the NOTICE issue in flink-dist.

Could you rebase the PR again? 😢

Thanks for the review @zentol. Sure, I'll rebase this PR. Merging this PR then.


add PATCH verb to apiserver

This add's a PATCH verb to all resources who's RESTStorage implements RESTGetter and RESTUpdater. Patch implements the merge patch defined in rfc7386. I'm opening this to discuss what how the pkg/client.Client Patch api should look, or if it should be included in the Client api. The difficulty is that when a patch is converted into an api type, information is lost to zero values of fields, thus a patch must be applied as a json string or a map[string]interface{} in order to maintain all information. I will be refactoring kubectl update --patch as well.
 curl localhost:8080/api/v1beta1/pods/web \
    --request PATCH  \
    -H "Content-Type:application/merge-patch+json" \
    -d '{"labels":{"new":"label"}}'

Fixes #4578

Assigning to @bgrant0607, feel free to rebalance. Maybe @smarterclayton is also interested.

I'd like to sketch out how patch fits in with the other changes going on in update (ie whether we should reorganize update so we can avoid double reads) and how we distinguish intent on PATCH to allow status.

We can hold this pending more discussion on the use case / api of PATCH.

@ghodss and @nikhiljindal may also be interested.

Work on status shouldn't hold up patch, IMO. I'll add some thoughts about status to #2726.

What's the rush to get patch in?  Agree it's nice, but I'd put the priority on getting our existing verbs working before we add new ones.

On Feb 23, 2015, at 4:36 PM, Brian Grant<EMAIL_ADDRESS>wrote:
Work on status shouldn't hold up patch, IMO. I'll add some thoughts about status to #2726.
—
Reply to this email directly or view it on GitHub.

No huge rush, but configuration (reconcile) will depend heavily on patch, and I see it as a separate issue from status.
The refactoring you are doing, which admittedly is partly in support of status, is a bigger reason to go slowly with this.

I should have a prelim status pull up tomorrow - we can circle the wagons on what we want to do with update, then if necessary we can do the simple patch now and implement the "correct" patch. I want to close on finalizers and acl so we know whether we need to implement per property policy rules in our engine, with that we probably have the last of the near term update reqts closed (except splitting into two objects which belongs below the storage interface anyway).

On Feb 23, 2015, at 5:53 PM, Brian Grant<EMAIL_ADDRESS>wrote:
No huge rush, but configuration (reconcile) will depend heavily on patch, and I see it as a separate issue from status.
The refactoring you are doing, which admittedly is partly in support of status, is a bigger reason to go slowly with this.
—
Reply to this email directly or view it on GitHub.

One more question. Is the "patch body contains a resourceVersion" condition something we want to validate on input? If there is no resourceVersion should be just update against the current resource?

I'm uncomfortable encouraging clients to not patch with resource version.  However, arbitrary patch isn't unreasonable to support.  I would not do anything specific around resource version right now, and we can add it in later.

On Mar 3, 2015, at 2:22 PM, Mike Danese<EMAIL_ADDRESS>wrote:
One more question. Is the "patch body contains a resourceVersion" condition something we want to validate on input? If there is no resourceVersion should be just update against the current resource?
—
Reply to this email directly or view it on GitHub.

Current iteration will honor resource version when it is passed in the patch body, otherwise it will use the resourceVersion returned in the first Get so the get/patch/write cycle will be controlled by resourceVersion as well.

LGTM, waiting for rerun of Travis, sorry for the long delay.

@ghodss How would this PR need to change in order to support custom list merging?

I'm going to merge this as-is (Shippable passed, Travis is backed up), but we've decided to change merge semantics: #4889. I'm counting on lack of documentation and kubectl integration to prevent rapid proliferation of usage. :-)

That's helpful actually - I will open a new PR that enhances this with list merging.


Check pgsql's query privilege

Fixes #3849.
Changes proposed in this pull request:

check pgsql's query privilege

Codecov Report

Merging #5099 into master will decrease coverage by 0.01%.
The diff coverage is 0.00%.

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

Pull Request Test Coverage Report for Build 10877

0 of 11 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 57.286%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-scaling/sharding-scaling-postgresql/src/main/java/org/apache/shardingsphere/shardingscaling/postgresql/PostgreSQLDataSourceChecker.java
0
11
0.0%

Files with Coverage Reduction
New Missed Lines
%

sharding-scaling/sharding-scaling-postgresql/src/main/java/org/apache/shardingsphere/shardingscaling/postgresql/PostgreSQLDataSourceChecker.java
1
0%

Totals

Change from base Build 10876:
-0.03%

Covered Lines:
11668

Relevant Lines:
20368

💛  - Coveralls

Any unit test for PostgreSQLDataSourceChecker.java?

A new pr will add the unit test of PostgreSQLDataSourceChecker.java and MySQLDataSourceChecker


DEM for Italy

Hi,
here's a DEM for Italy
https://tinitaly.pi.ingv.it/

Hi @sabas, thanks for the suggestion, added in https://github.com/DahnJ/Awesome-DEM/commit/06b1c8f175e053937449b465da756acaca1fc9ed


draugr minions with "needle" melee weapon do not attack

Describe the bug
Draugr minions with the "needle" weapon do not attack.  They will aggro on nearby creatures, close in to melee range, and just stand there.    Skeleton minions do not have this issue.  Confirmed in a fresh world with no other mods.
To Reproduce (if applicable)
Spawn any draugr minion with the requisite needles in your inventory and approach an enemy creature.
Expected behavior (if applicable)
The draugr minions should attack when in melee range.
Screenshots (if applicable)
N/A
Desktop

OS:  Windows 10
Mod version:  2.4.4
Game version:  0.214.2

Thank you. This is because the weapon uses an animation that skeletons have, but draugr do not have. I will need to create a different version for them. It will be fixed in the next version


With preferRest: true option set, I am unable to connect to auth emulator

Describe your environment

Operating System version: Arch Linux
Firebase SDK version: 11.4.0
Firebase Product:  auth, firestore, storage
Node.js version: 18.x
NPM version: 8.19.1

Additional related dependencies
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
Describe the problem
With preferRest: true option set, I am unable to connect to auth emulator
Steps to reproduce:
Here is my firebase initialization code
import { initializeApp, applicationDefault } from 'firebase-admin/app';
import { getAuth } from 'firebase-admin/auth';
import { initializeFirestore } from 'firebase-admin/firestore';

export const app = initializeApp({
  credential: applicationDefault(),
  projectId: process.env.GCLOUD_PROJECT,
});

export const auth = getAuth(app);
---export const firestore = initializeFirestore(app);
+++export const firestore = initializeFirestore(app, { preferRest: true });

When I run my application with preferRest: true, I get this error. It seems like emulators are not detected with this option.
Error: Could not load the default credentials. Browse to https://cloud.google.com/docs/authentication/getting-started for more information.
  at GoogleAuth.getApplicationDefaultAsync (/home/node/app/node_modules/google-auth-library/build/src/auth/googleauth.js:209:19)
  at GoogleAuth.getClient (/home/node/app/node_modules/google-auth-library/build/src/auth/googleauth.js:635:17)
  at GrpcClient.createStub (/home/node/app/node_modules/google-gax/src/fallback.ts:280:27)
Caused by: Error:
  at L0.getAll (/home/node/app/lib/server.js:113:509361)
  at ta.get (/home/node/app/lib/server.js:113:32416)
  at Function.findByIdentifier (/home/node/app/src/database/user.ts:115:44)\n

Application works correctly without preferRest option.

I am also unable to reproduce this.

I am experiencing the same.
From what I can tell, the function refreshTokenNoCache is using the constant below:
 const url = OAuth2Client.GOOGLE_OAUTH2_TOKEN_URL_

did you set the environment variables for Firestore and Auth emulator?

I can confirm those environment variables are set.
Applications works correctly without preferRest: true, I am also facing this exact issue in another project.

I have this issue too. When you enable preferRest, it seems like the auth emulator tries to setup credentials for the google-gax fallback?
It has the correct address set in the env passed into createStub and customServicePath is true, but this.auth.getClient() errors out.
https://github.com/googleapis/gax-nodejs/blob/main/src/fallback.ts#L280

Hey everyone, thank you for your patience on this. We have identified the root cause and the team is currently working on a fix.

@lahirumaramba Can you suggest some workaround while https://github.com/googleapis/gax-nodejs/issues/1409 is open? Perhaps we can mock/stub some methods so it works for tests?

Also running into this.

We have some momentum on a fix, but in the meantime there are two possible workarounds.
First, you could disable preferRest when using the emulator.
    const db = initializeFirestore(app, {
        preferRest: !process.env.FIRESTORE_EMULATOR_HOST
    });

Second, you can create this auth object for use with the emulator.
import { initializeApp, applicationDefault } from 'firebase-admin/app';
import { initializeFirestore } from 'firebase-admin/firestore';

export const app = initializeApp({
    credential: applicationDefault(),
    projectId: process.env.GCLOUD_PROJECT,
});

// Workaround pt 1 - add this createEmulatorAuth function
function createEmulatorAuth() {
    return {
        async getClient() {
            return {
                async getRequestHeaders() {
                    return {Authorization: 'Bearer owner'};
                }
            };
        },
    };
}

async function main() {
    const db = initializeFirestore(app, {
        preferRest: true
    });

    // Workaround pt 2 - Pass emulator auth here
    if (process.env.FIRESTORE_EMULATOR_HOST) {
        db.settings({
            auth: createEmulatorAuth()
        });
    }

    await db.collection("foo").add({time: Date.now()})
    const result = await db.collection("foo").get();

    console.log(result.size);
}
main();

I don't seem to have any trouble connecting to auth emulator with preferRest.  My config:
config/firebase.ts
import { initializeApp } from 'firebase-admin/app';
import { getAuth } from 'firebase-admin/auth';
import { getFirestore } from 'firebase-admin/firestore';
import { getStorage } from 'firebase-admin/storage';

const app = initializeApp();

const auth = getAuth(app);

const db = getFirestore(app);

db.settings({
  ignoreUndefinedProperties: true,
  preferRest: true,
});

const storage = getStorage(app);

export { auth, db, storage };

I stand to be corrected. Although my app runs fine with auth emulator when preferRest is true, unit tests now time out and fail. This fixed it for me:
db.settings({
  ignoreUndefinedProperties: true,
  preferRest: !process.env.JEST_WORKER_ID,
});


[Dotnet] How to use Kiota MultiPart with IFormFileCollection or IFormFile

I am using Kiota to generate an HTTP client within a shared project among multiple solutions. Everything is functioning as expected, except for one specific use case related to a POST resource. The resource and command are defined as follows:
    [HttpPost]
    public async Task<ActionResult> Post([FromForm] CreateUserCommand command)
        => Ok(await _mediator.Send(command));

The CreateUserCommand includes an IFormFileCollection for attachments:
    public class CreateUserCommand : IRequest<bool>
    {

        public CreateUserCommand ()
        {
        }

        public string Email { get; set; }

        public string FirstName { get; set; }

        public string LastName { get; set; }

        public string Address { get; set; }

        public string Phone { get; set; }

        public IFormFileCollection Attachements { get; set; } = new FormFileCollection();
      }

Kiota generates the following method signature:
    public async Task<Int32BooleanResultResponse?> PostAsync(MultipartBody body, Action<RequestBuilderPostRequestConfiguration>? requestConfiguration = default, CancellationToken cancellationToken = default)

Here is an example of how I attempted to construct the MultipartBody:
so in my request i need to send the user informations with attachements file, does any one knows how to do it, i did not find anything in the documentation
           var multipartBody = new MultipartBody();

            multipartBody.AddOrReplacePart("email", "application/json", applyCommand.Email);
            multipartBody.AddOrReplacePart("address", "application/json", applyCommand.Address);
            multipartBody.AddOrReplacePart("firstName", "application/json", applyCommand.FirstName);
            multipartBody.AddOrReplacePart("lastName", "application/json", applyCommand.LastName);
            multipartBody.AddOrReplacePart("phone", "application/json", applyCommand.Phone);

Can anyone provide guidance on how to properly structure the request to send user information along with attachment files using Kiota? Any insights or examples would be greatly appreciated.

Hi @linkingas,
Thanks for using kiota and for reaching out.
Can you provide the OpenAPI description for that endpoint please?

Hello, Thank you baywet,
'/v{version}/User':
post:
tags:
- User
summary: Create User
description: Create User
parameters:
- name: version
in: path
required: true
schema:
type: string
requestBody:
content:
multipart/form-data:
schema:
type: object
properties:
Email:
type: string
FirstName:
type: string
LastName:
type: string
Address:
type: string
Phone:
type: string
            Attachements:
              type: array
              items:
                type: string
                format: binary
        encoding:
          Email:
            style: form
          FirstName:
            style: form
          LastName:
            style: form
          Address:
            style: form
          Phone:
            style: form
          Attachements:
            style: form

Thanks for the additional information.
Can you try to update your code to:
var multipartBody = new MultipartBody();

multipartBody.AddOrReplacePart("Email", "application/x-www-form-urlencoded", applyCommand.Email);
multipartBody.AddOrReplacePart("Address", "application/x-www-form-urlencoded", applyCommand.Address);
multipartBody.AddOrReplacePart("FirstName", "application/x-www-form-urlencoded", applyCommand.FirstName);
multipartBody.AddOrReplacePart("LastName", "application/x-www-form-urlencoded", applyCommand.LastName);
multipartBody.AddOrReplacePart("Phone", "application/x-www-form-urlencoded", applyCommand.Phone);

(notice both the case change for the part name and the format change)

I'm not sure that part of the description is actually correct as I'm not sure an array of array of bytes makes sense at all.
Simply try this for each of the of the files you want to upload
multipartBody.AddOrReplacePart("filename", "application/octet-stream", fileStream); // filename must be unique

@baywet thank you for your response, as my files are dynamic i can not use a stream (will not have the metadata informations) and unique part (see the screenshot )

you should have access to the stream from the blob once the file has been selected by the user (I'm not sure what kind of app you're building here, but assuming front end through blazor).
The blob should also contain the file name.

The frontend is built using angular but i don't have any issue in this side, the frontend is sending a Blob to a BFF , the BFF is calling the api to create a user with some files, the files can be the cv , cover letter ...
now if the bff  send a stream as mentioned in this code
multipartBody.AddOrReplacePart("filename", "application/octet-stream", fileStream); // filename must be unique
i will not have the metadata  of the file

The filename/partname will be in the Content-Disposition part header that Angular sends to you BFF (assuming you're using multipart there too, forms would work as well). Parsing that incoming request header should give you the information you need to forward things to the backend API.


Trouble with gitlab container

I thought I had this figured out but, no, it doesn't seem to like me. Here's my docker-compose.yml for a gitlab instance. I don't want gitlab to take over the ports for ssh/https, so I've pointed them to 23/444 respectively. When I look at the nginx conf I can't immediately see anything wrong. But I'm getting a 502 bad gateway when I navigate to http://g.1beb.ca
gitlab:
  image: 'gitlab/gitlab-ce:latest'
  restart: always
  environment:
   - VIRTUAL_HOST=g.1beb.ca
   - VIRTUAL_PORT=8080
  ports:
   - 444:443
   - 23:22
   - 8080:8080
  volumes:
   - '/home/admin/gitlab/data:/etc/gitlab'
   - '/home/admin/gitlab/logs:/var/log/gitlab'
   - '/home/admin/gitlab/config:/var/opt/gitlab'

When I run ...
$docker exec nauseous_lichterman cat /etc/nginx/conf.d/default.conf

This is the resulting conf file:
# If we receive X-Forwarded-Proto, pass it through; otherwise, pass along the
# scheme used to connect to this server
map $http_x_forwarded_proto $proxy_x_forwarded_proto {
  default $http_x_forwarded_proto;
  ''      $scheme;
}
# If we receive Upgrade, set Connection to "upgrade"; otherwise, delete any
# Connection header that may have been passed to this server
map $http_upgrade $proxy_connection {
  default upgrade;
  '' close;
}
gzip_types text/plain text/css application/javascript application/json application/x-javascript text/xml application/xml application/xml+rss text/javascript;
log_format vhost '$host $remote_addr - $remote_user [$time_local] '
                 '"$request" $status $body_bytes_sent '
                 '"$http_referer" "$http_user_agent"';
access_log off;
# HTTP 1.1 support
proxy_http_version 1.1;
proxy_buffering off;
proxy_set_header Host $http_host;
proxy_set_header Upgrade $http_upgrade;
proxy_set_header Connection $proxy_connection;
proxy_set_header X-Real-IP $remote_addr;
proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
proxy_set_header X-Forwarded-Proto $proxy_x_forwarded_proto;
server {
	server_name _; # This is just an invalid value which will never trigger on a real hostname.
	listen 80;
	access_log /var/log/nginx/access.log vhost;
	return 503;
}
upstream g.1beb.ca {
			# gitlab_gitlab_1
			server <IP_ADDRESS>:8080;
}
server {
	server_name g.1beb.ca;
	listen 80 ;
	access_log /var/log/nginx/access.log vhost;
	location / {
		proxy_pass http://g.1beb.ca;
	}
}
upstream r.1beb.ca {
			# rstudio_rstudio_1
			server <IP_ADDRESS>:8787;
}
server {
	server_name r.1beb.ca;
	listen 80 ;
	access_log /var/log/nginx/access.log vhost;
	location / {
		proxy_pass http://r.1beb.ca;
	}
}

To update, I looked at the nginx logs and I see:
nginx.1    | 2016/03/23 01:54:34 [error] 166#166: *320 connect() failed (111: Connection refused) while connecting to upstream, client: <IP_ADDRESS>, server: g.1beb.ca, request: "GET / HTTP/1.1", upstream: "http://<IP_ADDRESS>:8080/", host: "g.1beb.ca"
nginx.1    | 2016/03/23 01:54:34 [error] 166#166: *320 connect() failed (111: Connection refused) while connecting to upstream, client: <IP_ADDRESS>, server: g.1beb.ca, request: "GET /favicon.ico HTTP/1.1", upstream: "http://<IP_ADDRESS>:8080/favicon.ico", host: "g.1beb.ca", referrer: "http://g.1beb.ca/"

Interesting, I wasn't able to get it working with docker-compose but a direct command worked like a charm:
sudo docker run -i  \
    --hostname localhost  \
    --publish 444:443 \
    --publish 8080:8080 \
    --publish 23:22 \
    --name gitlab \
    --restart always \
    --volume /home/admin/gitlab/config:/etc/gitlab \
    --volume /home/admin/gitlab/logs:/var/log/gitlab \
    --volume /home/admin/gitlab/data:/var/opt/gitlab \
    --volume /home/admin/gitlab/logs/reconfigure:/var/log/gitlab/reconfigure \
    -e VIRTUAL_HOST=g.1beb.ca gitlab/gitlab-ce

Can you try your compose config without the "ports" sections and "VIRTUAL_PORT=8080". nginx-proxy does not need the proxied container to publish its port to the host, it connects to it over the internal docker network (using the port EXPOSE:ed in the Dockerfile).

That worked. Thanks!

Ok good. But i don't understand why it worked when using docker directly, which port did you test? 444 maps a exposed port at least hmm

It only seemed to be problematic when I set port 8080 or something of that nature.


Add guidance for region deployment

Hi team, it could be helpful to add a couple of regional options for this solution.
gpt-4o-realtime-preview and the semantic ranker are not available in all regions. what region would be preferred for users to deploy the resources?

This repository doesn't contain infra as code, so I wonder what you are specifically looking for. Is this a request to have better documentation in this repository?
To answer your question;

GPT-4o-realtime-preview is available for global deployments in East US 2 and Sweden Central regions, more information can be found here.

You can leverage the Azure AI Search regions list to understand if your region of choice supports semantic ranker. For Europe this would be North Europe or France Central for example. Please note that semantic ranker is not required for this accelerator, but might provide more relevant search results.

Hi mick, yes that answers my question.
The README mentions prerequisites for service deployment. But it was not clear from the README in what regions those services need to be deployed to support the application.


Resultspage: the link leading to the external profile does not work

the link redirects to /index

remove last slash
URL has to look like  href="#/app/profile/{{item.profileID}}/{{item.huggID}}"

remove last slash
URL has to look like href="#/app/profile/{{item.profileID}}/{{item.huggID}}"


Need to copy Contents/images folders for files that are processed

Our parser has escaped the limited subset I intended us to work on.
But, that's not a problem.  It's working quite well, with only 16 warnings.
But, there are lots of occasions when images are missing. This seems to be in files that aren't in a Britain_xx folder.  It appears that file parsing that starts from a category file (Britain_legacy/Britain_legacy.html) always copies over the content/images content.
But, file parsing that was generated from shopping list does not copy the content over.
Hmm, we may end up with more intelligent processing here (such as when we process the standard countries), but for now it would be convenient if the Content/Images folders were duplicated each time we process a file.  Aaah, but there are lots and lots of them, so let's check if it's necessary first (Contents/Images doesn't exist for that category.

Fixed in #421.


fix: swarm api

Making the extension compatible with the latest manifest v3 Swarm extension.

I've added tests for dappUrlToId function. Please take a look if examples are correct.


Uses a resizable panel for Topics/Queue

The Topic/Queues left panel is occasionally too narrow to display the whole names and the scrolling is cumbersome:

This fix enables a splitter between the Topics/Queues panels and the Messages panels.

@armannaj  -- could I bother you with a review on this PR, please?


Early Digi app exports/imports without author name

What is the issue
The app will be updated with vascular plants taxon spine to includes author names. However, the already imported specimen records did not have author associated with the taxon name.
We have to retroactively add the author name to records where that name is available in the specimen image.
There are ~ 15.000 vascular plant records as of 19-12-2023 in Specify. The period is May 2023 to December 19. 2023. These records will eventually all have an image associated.
Description
The imported specimen records ought to have the author name along with the taxon name if available. The author is often written after the taxon name on the label and might thus be retrieved from the image associated with the specimen.
Unless we have a pipeline that can read the author name, this will require a lot of manual work. The images can be identified by the associated barcode, yet that has to be read via a pipeline.
When that pipeline goes operational we can go from the specimen record straight to the image and look for the author name so that can be added. Waiting for the pipeline to be ready is the only feasible way forward.
Why is it needed/relevant ?
There is the issue of homonyms and disambiguation which author names can help solve.
Estimate level of effort required.
Hard
How to approach it?
It would be preferable for the pipeline to identify the barcodes for the images and then rename those images with the barcode. This enables us to go specimen by specimen.
If a pipeline can read the authors - or at least some of the authors - that would be tremendously helpful. Reading handwritten author names and adding them to the taxonomic name is yet another issue (separate though).

New Solution:

Export a file from Specify with [full taxon name], [Storage], [Author], [Notes/remarks]

Who has the ball on this issue?

There is a utility ready now. It is a Excel sheet with names , storage location and dropdowns for each name. I am submitting this to Bhupjit and Pip for review.

Workflow on producing the spreadsheet:

I discussed with @jlegind that we'd also need to include taxon table primary key, i.e. taxon ID, so the results can be easily used to update the taxon records with the assigned author name. In doing so, we may need to be wary of any cases where the same taxon record is assigned different author names depending on the specimen in question.

Taxon identifiers are added to the spreadsheet utility. @PipBrewer @bhsi-snm
authorDropdown.xlsx
Feel free to test the spreadsheet.

How about we find time to present this to the digitizers?

The digitizers have begun filling in this sheet. The corresponding GitHub ticket for the Herbarium Sheet Digitization project is https://github.com/NHMDenmark/Herbarium-Sheets-workstation/issues/74 :)

The author name sheet has been filled and https://github.com/NHMDenmark/Herbarium-Sheets-workstation/issues/74 has been closed. There are, however, some remaining issues that need to be fixed before this can be imported into Specify again.

All remaining work is incorporated into ticket https://github.com/NHMDenmark/Herbarium-Sheets-workstation/issues/117


furnace: 0.6pre9 -> 0.6pre16

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for furnace is: Multi-system chiptune tracker compatible with DefleMask modules
meta.homepage for furnace is: https://github.com/tildearrow/furnace
meta.changelog for furnace is: https://github.com/tildearrow/furnace/releases/tag/v0.6pre16
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.6pre16 with grep in /nix/store/svkis9fs7mv5sxlk0jpz4d4anm1ji4k7-furnace-0.6pre16
found 0.6pre16 in filename of file in /nix/store/svkis9fs7mv5sxlk0jpz4d4anm1ji4k7-furnace-0.6pre16

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
furnace

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/svkis9fs7mv5sxlk0jpz4d4anm1ji4k7-furnace-0.6pre16 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A furnace https://github.com/r-ryantm/nixpkgs/archive/b73bf4bd04ac532dcc3abbc9309a33472a0a4108.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/b73bf4bd04ac532dcc3abbc9309a33472a0a4108#furnace

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/svkis9fs7mv5sxlk0jpz4d4anm1ji4k7-furnace-0.6pre16
ls -la /nix/store/svkis9fs7mv5sxlk0jpz4d4anm1ji4k7-furnace-0.6pre16/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

furnace

Maintainer pings
cc @OPNA2608 for testing.

https://github.com/NixOS/nixpkgs/pull/254979


Following update in mockups

No status icons for 0 values in migration plans
Migration table headers adjustments
Removed PV icon from plan details
Same style icons for plan status
Updated migrations icon
Updated migrations counter to be a link
Added formated time under patternfly standards

Closes #589
Migrations icon

Stage finished - state display

Final migration finished - state display

Time display changes

@Danil-Grigorev Is this waiting on me to do something? Best I can tell from looking at the code it looks OK, but I can't really tell without seeing the changes in the UI.

No status icons for 0 values in migration plans

Is this in a mockup somewhere? @Danil-Grigorev


GKE v2 UI, cannot Edit a registred cluster

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

Register a GKE cluster with node pools
Go to edit cluster
Upgrade k8s verion
Click Save

Result:
The Edit will gets stuck in Saving state
Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): v2.5-6f71d4c7346c63a60c671d59185e1595fe826ac7-head
Installation option (single install/HA):

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):
Machine type (cloud/VM/metal) and specifications (CPU/memory):
Kubernetes version (use kubectl version):

(paste the output here)

Docker version (use docker version):

(paste the output here)

This is a UI issue, it's not handling the possibility that most attributes of a registered cluster will be null:
Uncaught TypeError: Cannot convert undefined or null to object
    at n.willSave (component.js:600)
    at n [as willSave] (index.js:443)
    at n.driverSave (cluster-driver.js:53)
    at e.n._join (backburner.js:1079)
    at e.n.join (backburner.js:790)
    at d (index.js:168)
    at index.js:3698
    at o (index.js:146)
    at index.js:3697
    at n.save (index.js:3659)

component.js:
599      if (!Ember.get(config, 'masterAuthorizedNetworks.enabled')) {
600        delete config.masterAuthorizedNetworks.cidrBlocks;
601      }

passed on v2.5-4e70ed29ccc5de7c61516ba0367c7c4904989b56-head


Changes v2

Why?
Improvement related to v2. This PR changes kanji.sh from a single-page collection of PDFs to a well-designed web app.
What Changed?
There are lots of things that changed with this PR. Please see the list.

[x] Logo change
[x] Adds the waiting list for the read feature.
[x] Adds pagination for the write feature.
[x] Adds privacy policy & terms of use. Since I'm using analytics, it is good to let others know what data is being collected.

Code Quality is an issue to be solved in another PR.


Thrown errors not catchable

On Mac OS 10_11 (El Capitan), safari 9 HRIR does not lead properly, but gives this error:
[Error] [Omnitone] HOARenderer: HRIR loading/decoding failed. (@658.19ms)
This is for both HOA and FOA, this in itself might not be an issue for this project, but I am unable to catch the error in any way.
The error is thrown during the initialize() method, since that is a promise, the .then(success, error) or .catch(error) should work. It does not.
When adding a .then(function() {console.log('success'); }, function() {console.log('error'); }); to the initialize call, it prints nothing and the exception is thrown anyway.
When trying to provoke this behavior on another machine I called the renderer with hrirPathList that would fail, this resulted in another exception being called. The same behavior in not being able to catch the exception persists though.
Recreate this issue this way:
var foaRenderer = Omnitone.createFOARenderer(this.audioCtx, {hrirPathList: ['hello', 'path']}); foaRenderer.initialize().then(function() {     console.log('Successful'); }, function(error) {     console.log('error', error); });
Expected result:
[error] [Omnitone] BufferList: XHR error while loading "hello(Not Found) [log] error false
Actual result:
[error] [Omnitone] BufferList: XHR error while loading "hello(Not Found)

Is this Safari specific?
Thanks for reporting and I'll investigate soon.

Hrir loading/decoding is. But as I tried to specify, the issue of not receiving the exception is the bigger problem. And that can be provoked by providing faulty hrirPaths in the config.
Sorry if this was unclear.

Hrir loading/decoding is

If the default HRIR is failing on Safari, I think that's a bigger issue for Omnitone. I assume you found this bug while trying custom HRIR files.
From my experience, Safari's dev tool is not good at notifying users when a promise fails. With that said, I agree that we have a bug in the exception handling.

Yes, but notice the version of Mac OS and safari this was tested on, it works fine on newer versions. No this was not found when testing custom hrir files, but when testing browser support.

Thanks! Would love to know about the result of the browser support testing.

The browser testing is not as extensive as it perhaps should be. That older safari version is the only one so far.


Translation in angular

Im using emoji-picker-element in my angular app, I'm using angular 18.0.1 and I'm trying to translate the label like search placeholder etc etc and I have no idea how to do that I tried to set the locale to fr but I guess that's not the correct way to to do it
This is my component typescript code
@Component({
    selector: 'post-comments',
    standalone: true,
    imports: [
        AsyncPipe,
        MatIconModule,
        MatButtonModule,
        TranslocoModule,
        MatProgressSpinnerModule,
        NgOptimizedImage,
        MatTooltipModule,
        MatRippleModule,
        RouterModule,
        MatDividerModule,
        DateToNowPipe,
        MatFormFieldModule,
        MatMenuModule,
        MatInputModule,
        ReactiveFormsModule,
    ],
    templateUrl: './post-comments.component.html',
    changeDetection: ChangeDetectionStrategy.OnPush,
    schemas: [CUSTOM_ELEMENTS_SCHEMA],
})

CUSTOM_ELEMENTS_SCHEMA is required for the emoj
My component HTML code
 <emoji-picker class="light" locale="fr" (emoji-click)="addEmojiForEdit($event)" />

I tried to follow the docs and create a new picker object using the constructor but how I'm gonna link that new created object with the emoji-picker component ?

I'm not familiar with Angular, but please take a look at the Internationalization section in the README. In short you can call picker.i18n = ... with the i18n object. In your HTML above try setting an i18n attribute, or else use some equivalent of refs to grab a handle to the element and set it manually.

Thank you i fixed by importing my fr i18n object in my ts file
import fr from 'emoji-picker-element/i18n/fr';
fr = fr;

in my HTML file
<emoji-picker class="light" [i18n]="fr" (emoji-click)="addEmoji($event)" />


Timeouts on WKWebView image snapshot

Hi, one more issue from me 😇 It looks like WKWebView is never performing image snapshot when there is no content loaded. I have the case where one of my views contains WKWebView that have no content initially. If I've debugged that correctly image snapshot on web view waits until the page finishes loading. But when there is no page loading it never occurs resulting in a timeout.
I am wondering if making it possible to take a snapshot when there is no content in web view is a good solution here.
Workaround for this is to load any file from disk into, even empty one.

Ah, good point, embedded web views shouldn’t time out if possible. Will look into this soon! Thanks for so thoroughly testing things 🙌

Ok, I will try to reproduce it outside or find if there is some other case here.

This makes timeout:
func testWebView() {
        class VC: UIViewController {
            override func loadView() {
                let browser: WKWebView = .init()
                view = browser
                browser.load(URLRequest(url: URL(string: "url")!))
            }
        }
        let vc = VC()
        vc.view.frame = .init(x: 0, y: 0, width: 200, height: 200)
        assertSnapshot(matching: vc, as: .image)
    }

It turns out that it was loading url, but it was kind of invalid.

I am wondering if loading other valid URL like base64 data which is accepted by browsers will behave similarly.

Ah. I think it’s kind of correct to fail in this case. Any way to modify your code to avoid the loading for the test? Or to load a valid HTML string instead?

(Up for other suggestions, too! We just want to wait for loading web pages to load.)

Going to close this issue and track the improvement here #136.


Resource requests for multiple jobs limited by first one submitted

Steps to reproduce the problem
Run two serverless jobs concurrently, the first one using 1 worker and the second one using 3.
Using the basic getting started running_program.ipynb notebook, make the following updates:
To ensure jobs run concurrently, add a pause in source_files/pattern.py around L19:
import time
time.sleep(120)

Then, update the running the pattern section of the notebook to launch jobs with different resource configurations:
from quantum_serverless import Configuration
job = serverless.run("my-first-pattern")
job2 = serverless.run("my-first-pattern", config=Configuration(workers=3))

(note: I've also tried this when setting auto-scaling to true on both jobs with no change in behavior)
What is the current behavior?
A Ray cluster with two pods (one head, one worker) are launched and both workloads are run on that cluster.
$ k get po
NAME                                 READY   STATUS    RESTARTS   AGE
c-mockuser-a1a35d28-head-4pf5d       2/2     Running   0          95s
c-mockuser-a1a35d28-worker-g-m7qgm   1/1     Running   0          95s
gateway-796cfb4d5b-cbslp             1/1     Running   0          6m50s
kuberay-operator-654bf75dcb-4tvcw    1/1     Running   0          6m50s
postgresql-0                         1/1     Running   0          6m50s
prepuller-5v2xg                      1/1     Running   0          6m50s
scheduler-fbb99cb54-mlxrt            1/1     Running   0          6m50s

What is the expected behavior?
At a minimum, I would expect the cluster to be resized to add the additional requested workers.
Not sure if the best behavior would be to start a new Ray cluster for the additional job, given the differing resource requests. I could see arguments in favor of both approaches...
How to Fix
When a job is submitted, we check to see if there's an existing compute resource:
https://github.com/Qiskit-Extensions/quantum-serverless/blob/7dfe5bbe644b15cca75691639fb898ce0f2da53e/gateway/api/schedule.py#L43-L45
and if so, we reuse it:
https://github.com/Qiskit-Extensions/quantum-serverless/blob/7dfe5bbe644b15cca75691639fb898ce0f2da53e/gateway/api/schedule.py#L49-L53
In some use cases, this may not be the ideal behavior, so we may want to revisit this decision...

Probably @IceKhan13 can give us a better insight but if I remember correctly something that it surprised me in the past is that Ray tries to allocate the workload where it can. So if you have two jobs that will consume 4 cpus and you have a cluster with 4 cpus available it will try to run those two jobs in that cluster instead of create two clusters or more workers. So maybe it's happening something similar in this case.
Said that, I never tried to run any workload with that configuration. I think it's a good idea to start testing different configurations for the workloads and monitor the behavior.

This will be fixed by #1337


Full Thumbnail Image Not Visible in API Listing and Overview Pages in Publisher and DevPortal

Description:
When importing APIs to the API Manager, it was observed that the thumbnail image of some APIs were not fully visible in the API Listing and Overview Pages in the Publisher and DevPortal.
API Listing Page

API Overview Page

Steps to reproduce:

Import [1].
Go to the API Listing page and observe the thumbnail image. It is not fully visible.
Click the API and go to the Overview page and observe the image at the top of the Overview page. It is not fully visible.

Affected Product Version:

APIM Latest

Environment details (with versions):

OS: Ubuntu 20.04.1 LTS (64-bit)

Suggested Labels:

Import-third-party-APIs

[1] https://raw.githubusercontent.com/APIs-guru/openapi-directory/master/APIs/googleapis.com/abusiveexperiencereport/v1/openapi.yaml

This issue is addressed through this one with a different approach https://github.com/wso2/product-apim/issues/10755


Updated to look for components in the western-up-react node module

I think workspaces are being used initially here, so I'm gonna merge


Activate/deactivate sing-in button.

The sign-in button is now activated only if both input fields are non
empty.

LGTM. Thanks!


fix dependency of include files under learn/building_programs mini-book

I had to do this fix to the dependencies generator, to be able to build a project in the case when the makefile is not in the same directory as the source files. I hope this makes sense!

(I just noticed that I accidentally did not fork the project... I hope this is fine, else I can close and re-submit a proper PR)

I'm surprised this is actually used, I wrote this guide to show how involved the makefile setup can become and motivate people to use fpm or other high-level build systems with proper Fortran support ;).

I'm surprised this is actually used, I wrote this guide to show how involved the makefile setup can become and motivate people to use fpm or other high-level build systems with proper Fortran support ;).

fpm will come first for me! But I also want to update my naive build workflow using make with this nice and robust one.


make-sqlite reports table is locked

When run TestCreateFile or TestCreateFileOnProtectedBranch, it reports,
2017/05/05 10:07:21 [...ea/models/webhook.go:641 DeliverHooks()] [E] UpdateWebhook: database table is locked
2017/05/05 10:07:21 [.../runtime/asm_amd64.s:2197 goexit()] [E] UpdateHookTask [2]: database table is locked

This will not stop the tests but it needs to be fixed.

continue via #2040


Not compatible with Node.js 14

node_modules/uWebSockets.js/uws.js:31
                throw new Error('This version of µWS is not compatible with your Node.js build:\n\n' + e.toString());
                ^

Error: This version of µWS is not compatible with your Node.js build:

Error: Cannot find module './uws_linux_x64_83.node'
Require stack:
node_modules/uWebSockets.js/uws.js

Works fine with Node.js 13

Yes you see Node.js 14 was released yesterday and I am working on it.

It runs now, but as per usual we have significant drops in performance with this new version:
Node.js 12.15.0:
Running 10s test @ http://localhost:9001
2 threads and 10 connections
Thread Stats   Avg      Stdev     Max   +/- Stdev
Latency    96.15us   33.36us   1.42ms   84.71%
Req/Sec    51.66k     7.07k   69.29k    66.83%
1038175 requests in 10.10s, 70.30MB read
Requests/sec: 102790.74
Transfer/sec:      6.96MB
Node.js 13.1.0:
Running 10s test @ http://localhost:9001
2 threads and 10 connections
Thread Stats   Avg      Stdev     Max   +/- Stdev
Latency   269.75us    2.61ms  59.44ms   99.44%
Req/Sec    50.95k     6.15k   63.07k    67.82%
1023792 requests in 10.10s, 69.32MB read
Requests/sec: 101369.67
Transfer/sec:      6.86MB
Node.js 14.0.0:
Running 10s test @ http://localhost:9001
2 threads and 10 connections
Thread Stats   Avg      Stdev     Max   +/- Stdev
Latency   110.34us   35.47us 436.00us   85.36%
Req/Sec    45.04k     5.12k   55.53k    71.29%
904947 requests in 10.10s, 61.27MB read
Requests/sec:  89599.46
Transfer/sec:      6.07MB
Remains to be seen if those drops are persistent or will be fixed in following patch releases. It was probably since v8 I saw this large of a drop in performance with one single Node.js version though, so it frightens me what they are doing to the project.

hopefully they used an unsigned int in their performance throttle - it will eventually drop so low it might wrap around to ludicrous speed again ( Node.js 37.0.0? )

The sad thing is that Node.js 0.8.0 from the initial months was actually quite rapid for being a JS runtime. But since that day numbers have fallen steadily as more and more clowns poke around in the code. Ryan's initial design was almost decent.

https://github.com/nodejs/node/issues/32226 kanske

Hard to tell, I have no ArrayBuffers in use for this test only String. Other tests that include ArrayBuffer show big losses, but not really anything more than the loss with String, so I don't think this is exactly the same. What is clear though, is that performance has been properly messed up with this release.
It could be ArrayBuffer related, I mean the only way forward with a JIT is to move around the cost and shove it under the mat. You can't really improve performance anything more than what we already did in the 80s - it's all just a matter of moving things around and marketing it differently.
Look at V8's Ignition marketing - it's an interpreter because they came to the conclusion JITs are slow and take up a lot of CPU time to JIT, which overall slow things down because most code is not a single for loop of 5 billion iterations doing the exact same thing over and over, so they removed it and went back to interpreter as it has no JIT:ing cost.
It's just hilarious to follow - nothing has changed since the 1980s just different brands on the same old algorithms.


[SECURITY VULNERABILITY] -

Summary
Short summary of the problem. Make the impact and severity as clear as possible. For example: An unsafe deserialization vulnerability allows any unauthenticated user to execute arbitrary code on the server.
Product
[product]

Details
Give all details on the vulnerability. Pointing to the incriminated source code is very helpful for the maintainer.
PoC
Complete instructions, including specific configuration details, to reproduce the vulnerability
Impact
[impact]
Remediation
Propose a remediation suggestion if you have one. (This is just a suggestion, as the maintainer might have a better idea to fix the issue.)
Credit
List all researchers who contributed to this disclosure.
If you found the vulnerability with a specific tool, you can also credit this tool.
Contact
[contact]

There is no information attached to this issue. It is just the default filler information.


Editor tab shadow is rendered on top of contrast border

"workbench.editor.tabSizing": "shrink",
"workbench.colorCustomizations": {
	"contrastActiveBorder": "#0011ff",
}

/verified


(Finished) pull-request for GUNNARP (40x40 panel) support

Hi
I bought a Gunnarp panel today.
I will be submitting a pull-request for the GUNNARP 40x40 panel some time soon; expect somewhere begin next week as I am tight on time this weekend.
This message is just to let people know that it's coming, and if they have some issues, they could add it to this ticket.
Note I do not have access to the round version of the lamp; it'd be good to see if there's someone who has it.
I will also endevour a pull-request for the zigbee2mqtt.io documentation soon after, supplying docs for how to get this lamp working.
Cheers.

Created pull-request.

Merged, thanks!


[MacOS][Apple M2 Pro]There's sound but black screen

Tried release mode and debug mode, It's the same whether
  media_kit: ^1.1.11
  media_kit_video: ^1.2.5 # For video rendering.
  media_kit_libs_video: ^1.0.5 # Native video dependencies.

or
  media_kit:
    git:
      url: https://github.com/media-kit/media-kit.git
      ref: main
      path: media_kit
  media_kit_video:
    git:
      url: https://github.com/media-kit/media-kit
      path: media_kit_video
  media_kit_libs_video:
    git:
      url: https://github.com/media-kit/media-kit
      path: libs/universal/media_kit_libs_video

It also appears in android's debug mode, but release works fine

Clone the repository, run media_kit_test. Does it work well?

git clone ...
cd media_kit_test
flutter run

Clone the repository, run media_kit_test. Does it work well?

Yes, it is

Then it's a bug in your code.

if there is bug in my code. I think android's release mode shouldn't work well
print log
flutter: VideoOutput.Resize
flutter: {rect: {top: 0.0, left: 0.0, height: 0.0, width: 0.0}, id:<PHONE_NUMBER>, handle:<PHONE_NUMBER>}
flutter: NativeVideoController: Texture ID:<PHONE_NUMBER>
2024-11-30 23:20:43.769 yoyo[26787:84174920] TextureGL: resize: 1920.0x1080.0
flutter: VideoOutput.Resize
flutter: {id:<PHONE_NUMBER>, handle:<PHONE_NUMBER>, rect: {left: 0.0, width: 1920.0, top: 0.0, height: 1080.0}}
[ERROR:flutter/shell/platform/embedder/embedder_external_texture_metal.mm(100)] Could not create external texture:<PHONE_NUMBER>
[ERROR:flutter/shell/platform/embedder/embedder_external_texture_metal.mm(100)] Could not create external texture:<PHONE_NUMBER>
[ERROR:flutter/shell/platform/embedder/embedder_external_texture_metal.mm(100)] Could not create external texture:<PHONE_NUMBER>
[ERROR:flutter/shell/platform/embedder/embedder_external_texture_metal.mm(100)] Could not create external texture:<PHONE_NUMBER>
[ERROR:flutter/shell/platform/embedder/embedder_external_texture_metal.mm(100)] Could not create external texture:<PHONE_NUMBER>
[ERROR:flutter/shell/platform/embedder/embedder_external_texture_metal.mm(100)] Could not create external texture:<PHONE_NUMBER>

If there is bug in my code. I think android's release mode shouldn't work well.

We have written implementation (separately) for each platform by hand... there can be differences as to how things work under the hood. There can be some race condition etc. Just use the recommended way to initialize Player & VideoController as mentioned in our README or described in the media_kit_test app.

Also... do leave the solution if you the issue for future visitors.

Also... do leave the solution if you the issue for future visitors.

I fixed it with delete the macos directory and recreate it, Maybe there was a problem with the previous macos project Settings


About Soundhax

My 3DS is in 9.0.0-16 K, and I tried Soundhax, and I failed in download otherapp "File not found"
So I ask him I can't download, and he said

..What should I do?

You need to ask for help on Nintendo Honebrew: https://discord.gg/C29hYvh

Thanks!


Update Readme and Contribute to Oct07-2020 release instructions

fixes: https://github.com/Azure/azure-sdk-for-cpp/issues/693

/check-enforcer override

3.13?

yes, first wanted to merge the other version PR to then update this one :D

Update the link to vcpkg commit version before merging.


SQMAGOPC-423: Change OrderInterface::loadByIncrementId calls to order factory implementation

Could you make these changes from master branch, with a pull request to back to master?
We want to deliver these fixes to Paytrail before recurring payment feature is complete.

Changes done on branch made from master and merged into master


Support specifying user networks when creating pods

This is my attempt at implementing the wiring to support #4718 as-is.
This --network argument is slightly different than the --net/--network argument accepted when creating containers. I'm sure there's also other inconsistencies and such. If this general direction is welcomed then I'll revise as necessary; however I haven't managed to run the tests yet to know how well my test works. If waiting for a proper refactor as discussed in #4718 is preferred then ignore this PR i guess.

/ok-to-test

Thanks @danopia.  First thing you need to do is sign your PR.
git commit -a --amend -s
git push --force
Should fix this, and then the tests should run.
/approve
Need @mheon and @haircommander to review, I know they had some thoughts on redoing the podman pod create stuff.

thanks @danopia the code LGTM
more philosophically, it feels a bit wasteful to me to add patches to a code base that is destined to fundamentally change. However, I can't commit any time to making that change, and I appreciate the time you've put in here, so this feeling is non-blocking

Changes LGTM


I ran sudo dlite install and now my mac is hosed

Something was changed with /var/empty/.../Application support
Now chrome, box and probably more apps are completely dead. Can't start.
I don't dare to reboot until I know what it did.

I have yosemite 10.10.5, it's an imac with hybrid drive.

whoa, that's pretty severe. how did you run it exactly? just sudo dlite install? from what shell? the installer doesn't delete anything, and altering the files it does shouldn't have any impact like you're seeing. can you give me any more information at all?

Mail and all the other programs are dead too. It's like it wiped out application support.
I can see the files are there (except /var/empty is indeed empty, what is that supposed to do?).
I don't appear to have any accidentally root owned directories in my home.
It did this from zsh.
It started as soon as "Building disk image" appeared.

When I launch a new terminal all my zsh settings are gone.

I get a plain bash.

What exactly is it doing in that startup?

It has moved my home direcotry to /var/empty!!!

Where is that set?

Not the contents of home, but the variable setting that denotes the location of my home. In case that was unclear.

the only thing i can think of is that something in your shell corrupted the environment variables that sudo sets (i.e. $SUDO_USER, $SUDO_UID, $SUDO_GID) but even then it would only cause the installer to create a file in the wrong directory. nowhere in the code do i change any environment variables, so that shouldn't be possible.
can you run the Console app? if so can you try to poke around in there and see if there are any errors related to dlite?

Super quiet in the logs actually. This is all I have around that time
1/26/16 9:32:16.000 AM kernel[0]: VNOP_IOCTL(_DKIOCCSMAP) returned an unexpected error code=30
1/26/16 9:32:16.000 AM kernel[0]: VNOP_IOCTL(_DKIOCCSMAP) returned an unexpected error code=30
1/26/16 9:32:16.000 AM kernel[0]: VNOP_IOCTL(_DKIOCCSMAP) returned an unexpected error code=30

Just to clarify, I ran it just like that. Downloaded, chmod a+x, ran sudo ~/Downloads/dlite install
MD5 (/Users/pbergen/Downloads/dlite) = 8b84bcf7ee7b57c733dc61f618305c53

I have no SUDO_* env vars at all.

the SUDO_ environment variables are only set when running a command through sudo, if you run sudo env you should see them.
doing some searching through OS X error codes, it sounds like error code 30 signals that a disk volume was taken offline and the OS wants it to be reinserted. you might try running sudo dmesg and reading through output, but i suspect that your physical disk may be going bad and was taken offline as a result of that. can you run disk utility? if so, do all of your volumes show as currently online and mounted?

So the problem is with Open Directory. It somehow killed off my network account. I'm working on figuring out the repair for it.
You might want to check what could possibly have caused that in your installer.

i don't do anything with the user's account.. i create a few files in their home directory and set the ownership of those to the user, but that's it. i have no idea what on earth could've caused your account to be removed. is your home directory on some sort of shared volume? maybe creating a 30GiB file over the network to a shared volume caused the issue

i'm going to close this issue, but please do let me know if you find out anything more about why this may have happened.

Whenever I launched dlite install it would simply hang the whole system. No disk access was possible anymore, and any process that tried to would become a zombie immediately.
I noticed that I also have a hybrid drive. Keeping console.app open during the install showed me that core storage was failing repeatedly. Of course all those log messages have not been flushed to disk, along with a few modifications I did just before launching, so I cannot post them here at the moment.
Interestingly enough, when trying to build a 5gb image nothing of the sort happened. Building a 10gb image causes the issue immediately. I'll try to make a video of that happening as soon as I can.

Correction, it does happen even when creating a 5gb image with -d 5, consistently. Every time I have to force shutdown the mac and reboot. I guess I had a single lucky run earlier, but I cannot see what the difference could be.
Anyway, here is the a screenshot of console.app. I put the marker just before starting sudo dlite install. As soon as the first message appears, the mac is unusable for anything that requires disk. After a while even console.app hangs. The only solution is to force reboot via long press of the power button.

@zenoamaro any updates? my iMac keeps crashing too :(

@rjayako no updates, I checked again a few weeks ago but it would still hose ~90% of the time.

I'm not a dlite user but I've had my system (also iMac / Yosemite / Fusion Drive) hosed by this same issue several times last year after creating large files (downloads in Transmission in my case). Tried a full reformat and reinstall twice and watched it happen again each time with the same error messages in the logs, which I've been Googling every once in a while ever since.
I had the exact same symptoms and log messages as reported by @philipbergen and @zenoamaro, but I'd also get a flood of "Keychain Not Found" error dialogs. After that my login.keychain and seemingly every other file that was in use at the time was truncated to 0 bytes.
Neither Apple Hardware Test or the Genius Bar could find any issues so I eventually tried splitting the Fusion Drive before reinstalling and I haven't had any issues since. I also upgraded to El Capitan shortly after this, but I don't think that had anything to do with it. As far as I can tell, that thread and this one are the only search results for these error messages.

@zenoamaro just curious to know if that's the case what your walk around is?


Share LCMS transform across threads

cmsFLAGS_NOCACHE disables the 1-pixel cache and makes calling cmsDoTransform() thread-safe. So instead of creating an LCMS transform for each thread, one can pass this flag and share a single transform across all threads.
I made an attempt to benchmark this, but the benefits (in terms of performance) are minimal, and I'm not sure whether it can be properly measured. Therefore, this should be considered as a general improvement.

Benchmark results
Benchmarked with a photo of Wat Arun (JPEG, 7360x4912, 29MB) containing the common sRGB IEC61966-2.1 profile.
Run with:
$ SKIP_TEST=1 ./ci.sh release -DJPEGXL_ENABLE_SKCMS=OFF
$ build/tools/benchmark_xl --input "~/wat-arun.jpg" --codec jxl:fast:d1

Before:
Compr               Input    Compr            Compr       Compr  Decomp  Butteraugli
Method             Pixels     Size              BPP   #    MP/s    MP/s     Distance    Error p norm           BPP*pnorm   Errors
---------------------------------------------------------------------------------------------------------------------------------
jxl:fast:d1      36152320  6101574    1.35019251877   1  45.831  36.947   1.68089497   0.58562449946  0.7907058179756201        0
Aggregate:       36152320  6101574    1.35019251877 ---  45.831  36.947   1.68089497   0.58562449946  0.7907058179756201        0

After:
Compr               Input    Compr            Compr       Compr  Decomp  Butteraugli
Method             Pixels     Size              BPP   #    MP/s    MP/s     Distance    Error p norm           BPP*pnorm   Errors
---------------------------------------------------------------------------------------------------------------------------------
jxl:fast:d1      36152320  6101574    1.35019251877   1  45.905  37.082   1.68089497   0.58562449946  0.7907058179756201        0
Aggregate:       36152320  6101574    1.35019251877 ---  45.905  37.082   1.68089497   0.58562449946  0.7907058179756201        0

I'm not entirely sure that benchmark_xl even measures CMS speed, there might be other ways to check that - but I also think that this is a Good Thing (tm) independently of the potential speed benefits :)


Problems when comparing PNG to JPG

import lpips
import sys
loss_fn = lpips.LPIPS(net='alex') # vgg o squeeze
img0 = lpips.im2tensor(lpips.load_image(sys.argv[1]))
img1 = lpips.im2tensor(lpips.load_image(sys.argv[2]))
print ('Distances: %.3f'%(loss_fn.forward(img0,img1)))

RuntimeError: The size of tensor a (127) must match the size of tensor b (255) at non-singleton dimension

I can successfully compare a jpg against a png. However, I get this error when comparing 2 images that have different resolutions. I can solve this problem by a combination of resizing the image and the canvas to make them match as optimally as possible.

Yes, the images need to be of the same size


Clone / Copy sets fromServer flag to 'true'

When cloning or copying a restangular model the fromServer attribute is always set to 'true'.
Because of this when you create a new model on the client, and call the save() method it does a PUT as opposed to a POST.
The fromServer attribute should reflect the original entity.

I think this is related to this issue, there's something strange going in .clone():
http://jsfiddle.net/vladimir_ze/docz3fuz/
Some properties being reset to default values after using .clone().

Sometimes it's set to undefined. To be refactored + unit tests should be done.


爱企查日常任务 CX12011: "点赞观点" 出错

日志如下
开始任务：点赞观点
(node:20009) UnhandledPromiseRejectionWarning: ReferenceError: nid is not defined
at dotask (/ql/scripts/raw_Scripts_ql_aiqicha.js:188:9)
at async aqc (/ql/scripts/raw_Scripts_ql_aiqicha.js:223:9)
(Use node --trace-warnings ... to show where the warning was created)
(node:20009) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag --unhandled-rejections=strict (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:20009) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

更新没 按理说不会啊

更新了

@lxr760 你进去脚本里看

我是放在青龙里跑的


kittycad say command like cowsay but its an ascii kittycad

https://github.com/uvipen/ASCII-generator

@gserena01 - adding you to this for impl for this quarter (heads up in tmorrow's 1:1 we're going to be moving around a few of your 'prioritized tasks' for this quarter)


[DCOS-41824] Enforce data service version under test

This PR sets the installed version of a package during testing to stub-universe. It ensures that we always test the newly built version and not by accident the universe version.

@elezar I have integrated an enum with two different options: STUB_UNIVERSE, LATEST_UNIVERSE to make the version more explicit. I didn't completely understand the idea behind ANY_VERSION since the version is always translated to a string.
One thing we could consider would be to change the API in a follow-up because most of the versions we want to install are either the latest from universe or stub-universe. Unfortunately, allowing only those two might also break other things I am currently not aware of.


Enable SSL Certificate Verification

A simple change to enable passing of verify down to requests so that there is the option of doing SSL Certificate Verification. Currently the options for this are quite limited due to the cert by default being self signed and having a CN of SolarWinds-Orion.
Also bumped the packaging version too...

Thanks for pushing to pypi too.


Creating NavBar

updated css file and header js file to create header.
Created footer.js for the footer details and updated it.

added navbar and footer


node module problem

there is some error appear while run this project. Please help me if any one solve this issue
I already install webpach dev, webpack-cli, node-sass and pace also but I con not solve this problem

Thank you for creating this issue. However, issues need to follow one of our templates so that we can clearly understand your particular circumstances.
Please help us help you by recreating the issue using one of our templates.
Please open an issue in ng repo. You have invalid configuration, please read the error message.

You have the enforce option for module.rules[1][4], but it is not allowed


Warnings about private keywords appear when used as setups or teardowns even though the keywords are in the same file

When setting a private keyword that is located in the suite file as suite/test setup or teardown, warnings about each keyword being private appear.
RF version: 6.0.2
Python: 3.10
OS: Windows 10 Pro 21H2 build 19044.2486
You can reproduce the problem with a simple suite like this:
*** Settings ***
Suite Setup    PrivateSetup
Suite Teardown    PrivateTeardown

*** Test Cases ***
Calling Private Keywords From Setup/Teardown In The Same File As The Test Case
    [Setup]    PrivateSetup
    log    Do something!
    [Teardown]    PrivateTeardown

*** Keywords ***
PrivateSetup
    [Tags]    robot:private
    Log    This will raise warning!

PrivateTeardown
    [Tags]    robot:private
    Log    This will raise warning!

Output:
[ WARN ] Keyword 'PrivateSetup' is private and should only be called by keywords in the same file.
==============================================================================
Poista Dataa
==============================================================================
Calling Private Keywords From Setup/Teardown In The Same File As T... | PASS |
------------------------------------------------------------------------------
Poista Dataa                                                          | PASS |
1 test, 1 passed, 0 failed
==============================================================================
Output:  C:\robotti\lokit\output.xml
Log:     C:\robotti\lokit\log.html
Report:  C:\robotti\lokit\report.html
[ WARN ] Keyword 'PrivateSetup' is private and should only be called by keywords in the same file.
[ WARN ] Keyword 'PrivateTeardown' is private and should only be called by keywords in the same file.
[ WARN ] Keyword 'PrivateTeardown' is private and should only be called by keywords in the same file.

Process finished with exit code 0

This is actually by design. Private keywords can be used without a warning only by keywords in the same file. Using a private keyword directly in a test causes the same warning as you get when using them as test/suite setups/teardowns. Using private keywords as keyword teardown is fine.
I don't see big problems making it possible to use private keywords defined in a test case file as suite/test setup/teardown without a warning, though, and the change wouldn't be backwards incompatible either. It can be considered if you have a good use case for it and especially if you would be interested to provide a PR.

Its actually really common for me to have suite specific keywords such as setup/teardown in the suite file itself. It isolates test-specific logic from commonly used keywords. Those keywords usually just call public keywords elsewhere, but with varying sequences based on the test at hand.
Of course I could always just move those keywords to another resource file, but then it would seem like that the resource file would be ok to use with other suites, when in fact the keywords were meant for a specific suite.
I can try to find time this month and familiarize myself with the codebase for a PR.

Having setup/teardown keywords in the suite file is fine and often a very good idea. What use case you have for making them private?

Having setup/teardown keywords in the suite file is fine and often a very good idea. What use case you have for making them private?

Hmm... yeah I guess this is kinda niche 😅
My first thought was that since the keywords were are supposed to be used by the suite-file only, I should private them. But I don't import keywords from other suite-files anyway, so I guess there is no real use case.
The reason I made this issue was because robotframework-lsp kept suggesting keywords from .robot files, but I realize now that privating them doesn't help either. I'm closing this issue, since it's more of an issue with robotframework-lsp or it's configuration.


:white_check_mark: Test: Verify all test apis

Description
Concise description of proposed changes
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1141  👈
#1140
main

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

Join @roninjin10 and the rest of your teammates on  Graphite


Curious as to why keyspace in Cassandra sets replication factor of 1?

We have been having stability challenges with our cassandra cluster... which we are working on.  But we are noticing that if we lose one node in our cluster, we lose data.  When I describe the cadence keyspace it says the replication factor is 1.  What's the rationale there?  We are new to cassandra administration, but it would seem you would want a replication factor of at least 2 if you don't want to lose data when a node goes down.
cqlsh> describe keyspace cadence

CREATE KEYSPACE cadence WITH replication = {'class': 'SimpleStrategy', 'replication_factor': '1'}  AND durable_writes = true;

@ryanwalls replication factor of 1 is used for developer setup for running Cadence server on laptop.  For production cluster we recommend to use replication factor of 3.  Please use 'cadence-cassandra-tool' to create both the cadence and cadence_visibility keyspaces.  It already has an option to configure the replication factor.
cadence-cassandra-tool --ep $CASSANDRA_SEEDS create -k $KEYSPACE --rf $RF

Going to leave this open until the README for the cadence-cassandra-tool is updated with this info.  https://github.com/uber/cadence/blob/master/tools/cassandra/README.md


chore: upgrade go dependencies

from: https://github.com/jenkins-x/jx3-versions

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign jstrachan
You can assign the PR to them by writing /assign @jstrachan in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pr
be6b610e3165fc44257164bfc790f83b81016bc9
link
/test

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.


Intial Port to Swift 3.0

Current Tests all run without error

If you use Cocoapods you can try it out right now using:
pod 'FutureKit', :git => 'https://github.com/FutureKit/FutureKit.git', :branch => 'v3'
Or Carthage
github "FutureKit/FutureKit" "v3"

Can we merge this to master please?


Negetive Values in Y Axis

I am seeing -ve values in Y-Axis, although I do not have any -ve values in my data set.
I have a row data and I am trying to draw a timeseries graph.
[['2016-6-1',1330,20],
['2016-6-2',1222,21],
['2016-6-3',1111,33]]
I have the following definition for y axis:
y: {
min:0,
label:"Count"
}
In the Y-Axis I see   values like -100 as the first value.
Now, this happens for only a few datasets and not all. But this is very much reproducible.
I can provide the exact dataset if required. Thanks.

Could you provide a sample of the dataset? And I don't know what "-ve" is. Could you provide the a sample of what you're generating?

We get a lot of reports and the more info you give us, the more likely we'll be able to address it in a timely fashion.
Please read the contributing guidelines, then add the following information to your issue:

Operating system and web browser (with version)
What version of the library
If you were doing anything unusual, like using C3 in conjunction with a particular library or framework
Ideally, a JSFiddle example depicting the specific problem really helps us out.

Use this JSFiddle template

Thank you for your cooperation — we rely on users like you to help make C3 the best charting library it can be!

I'm experiencing this. Here's the code:
var myChart = c3.generate({
      bindto: '.js-languages',
      axis: {
        y: {
          min: 0,
          max: 100
        }
      },
      data: {
        columns: [
          ['JavaScript', 30, 80, 100, 60, 50, 75],
          ['Ruby', 70, 20, 0, 40, 50, 25]
        ]
      }
    });

The 3rd value in the "Ruby" set is 0 and I'm even setting the y min to be 0, but it's still showing -10 on the y-axis. See screenshot:

I was able to solve visually for this problem somewhat by setting smaller padding values and hiding the outer tick:
c3.generate({
  bindto: '.js-languages',
  axis: {
    y: {
      min: 0,
      max: 100,
      padding: {
        top: 4,
        bottom: 4
      },
      tick: {
        outer: false
      }
    }
  },
  data: {
    columns: [
      ['JavaScript', 30, 80, 100, 60, 50, 75],
      ['Ruby', 70, 20, 0, 40, 50, 25]
    ]
  }
});

Here is my example:
var x = ["TRANS_DATE", "2016-10-08", "2016-10-09", "2016-10-10",
"2016-10-11", "2016-10-12", "2016-10-13", "2016-10-14"];
var y = ["INT_COUNT", 932, 1825, 59, 28, 4, 1315447, 21214553];
console.log(x);
console.log(y);
var _chart = c3.generate({
bindto:"#_chart",
size:{height:"180"},
data:{
x:'TRANS_DATE',
columns:[x,y]
},
axis:{
x: {type: 'timeseries', tick: {format: '%m-%d'} }
}
});
And here is how the chart looks like:
[image: Inline image 1]
On Mon, Sep 5, 2016 at 12:59 PM, Matt Pardee<EMAIL_ADDRESS>wrote:

I was able to solve visually for this problem somewhat by setting smaller
padding values and hiding the outer tick:
c3.generate({
bindto: '.js-languages',
axis: {
y: {
min: 0,
max: 100,
padding: {
top: 4,
bottom: 4
},
tick: {
outer: false
}
}
},
data: {
columns: [
['JavaScript', 30, 80, 100, 60, 50, 75],
['Ruby', 70, 20, 0, 40, 50, 25]
]
}
});
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/c3js/c3/issues/1722#issuecomment-244786459, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADy6nDnxQGqpmWvu_N_5fJnny3fIJk-bks5qnEqJgaJpZM4IzK8z
.

Here's a screenshot of this problem demonstrated on the Axis Range example on the c3 website.

and here's the hack to fix (h/t @mattpardee):

Though I found padding: { bottom: 0 } to be preferable to 4.


Browserify, gulp build

A builded version has nothing to do here.
Maybe on prepublish. And gulp seems overkill.
I think I will handle that using npm scripts only.

I just wanted to be able to consume it via the browser and not have to set
up commonJS. It's a fantastic library. Great job.
On Mon, May 11, 2015 at 11:41 PM Maxime Thirouin<EMAIL_ADDRESS>wrote:

A builded version has nothing to do here.
Maybe on prepublish. And gulp seems overkill.
I think I will handle that using npm scripts only.
—
Reply to this email directly or view it on GitHub
https://github.com/harthur/color/pull/58#issuecomment-101139660.


Fix Bug in greetings.yml file

Task
Improve greetings.yml file with proper syntax
Procedure
Comment below if interested in this issue
Once assigned start working
Write proper commit message in PR

Hi Can I take this up please?

Yes @nish-d  go for it 👍

@Swarnimashukla Can you please review my PR. Happy to accomodate any more suggestions/ changes.


Sync eng/common directory with azure-sdk-tools for PR 999

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/999 See eng/common workflow

@tjprescott
This is one of the sync PR for azure-sdk-tools PR https://github.com/Azure/azure-sdk-tools/pull/999
We have a guidance on how to check in these PRs. https://github.com/Azure/azure-sdk-tools/blob/master/eng/common/README.md#workflow
Engineer team will take care of these PRs, so you can simply ignore it next time you have seen these ones. :)

@sima-zhu got it. I just saw that our repo is accumulating a number of these and figured to get them merged.


'sudo lxd init' should enable zfs compression by default

We've just completed some zfs testing, and have determined that, generally, enabling zfs compression reduces disk usage by 20-25%, at a cost of 2-3% CPU overhead, on most systems.  Please enable zfs compression, when the user sets up zfs storage with 'lxd init'.

It's been enabled for a while: f75a9136daac62aabfd209f3a6121ef13e23a8d5 and should be enabled in lxd's >= 2.3

Yep, that's all been done already a couple weeks ago :)

;-)  Cool.
:-Dustin
On Wed, Oct 5, 2016 at 10:38 AM, Stéphane Graber<EMAIL_ADDRESS>wrote:

Closed #2461 https://github.com/lxc/lxd/issues/2461.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lxc/lxd/issues/2461#event-813460394, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAzMhruVIzNyE1Asg11p_35gXc62Gps2ks5qw8RpgaJpZM4KO8M4
.


Create page layout design.

It's always useful to add a Pull Request description, that tells what exact changes are being made.
For example it could looks like this:
This commit changes:

Feature number 1
Feature number 2

Also, you don't need a dot (.) after a Pull Request name.

Also, adding a photo could be a separate commit, since it's a non-intersecting change.
But you don't need to change it now.


Add technical documentation for districts

What does this PR do?
Adds a one-page summary of what's needed for a new district to set up Student Insights from a technical point of view, requested by @snoopyuri so that he can use it in initial meetings with potential new districts.

@kevinrobinson I'm going to 🚢 this one as a selfie, tho feel free to update/elaborate on it! @snoopyuri reviewed a draft earlier this afternoon. 👍


[Form Provider Refactor] NewContactMethodPage

Coming from https://github.com/Expensify/App/issues/25397#issuecomment-1743272245, we created a new Form provider that works with functional components and we need to refactor all usages of the old Form component.
This issue is specific to refactoring src/pages/settings/Profile/Contacts/NewContactMethodPage.js
cc @kowczarz
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01fd2034b42e8f51ab
Upwork Job ID:<PHONE_NUMBER>472392704
Last Price Increase: 2023-10-04

Hello! I'm Kamil Owczarz from Software Mansion, the expert agency and I would like to work on this task.

Thanks @kowczarz do you have ideas for a proposal yet?

@laurenreidexpensify that's a followup for this issue https://github.com/Expensify/App/issues/25397#issuecomment-1743272245. I think it would be reasonable to skip proposal similarly as it was done in function components migration and TypeScript migration.

Yep! No proposal needed here. Assigning @kowczarz to the job! Thanks for working on it.

Is there a draft PR up for this one yet?

Hey, not yet, there are 10 similar issues and I will work on them this week. I will ping you as soon as it will be ready.

waiting on PR

Do we have a PR up yet?

Not yet, we started with the NewDatePicker, DateOfBirthPage, EditRequestCreatedPage forms first. Should get to this one soon!

Hello, I'm Tymoteusz Kałuzieński from Software Mansion, the expert agency, and I would like to work on this task. Kamil reassigned it to me.

@cdOut can you please follow the instructions in this comment?

Hi @luacmartins, @mountiny has already added @cdOut to the upwork whitelist (details in slack dm), but it takes some time to be deployed.

Cool, in that case I'll assign them to the issue!

Payment Summary -

External issue reporter - N/A
Contributor that fixed the issue - @cdOut payment N/A
Contributor+ that helped on the issue and/or PR - @abdulrahuman5196 $500 payment issued in Upwork


Add UV_CONSTRAINT environment variable to provide value for --constraint

Summary
This PR is adding UV_CONSTRAINT environment variable as analogous to PIP_CONSTRAINT to allow providing constraint file via environment variable. Implementing this will simplify adoption of uv in testing procedure in projects that I'm involved (testing using tox).
This was my motivation for opening #1841 that is closed in favor of #1789 which was closed without implementing this feature.
In this implementation, I have used space as a separator as analogous to pip. This introduces an obvious problem if the path contains space. Another option could be to use standard separator (: - UNIX like, ; - Windows). Which one did you prefer?
Test Plan
It is my first contribution and first rust coding experience. It will be nice if one could point how I should implement testing this.

Test failure looks like network error

I think space-delimiting is ok, since it's at least compatible with pip and the other arguments we added.


Fix: build image error

What type of PR is this?
/kind cleanup
/kind bug

Build docker image error for path not exists, I think this can be avoided by adding e2e test, which will have to build a image. Caused by https://github.com/kubernetes-sigs/lws/pull/43

When developing rollout, I found some codes located at wrong place, e.g.

makeCondition and setCondition is defined for leaderworkset api, but located at pod_controller
constructWorkerStatefulSetApplyConfiguration is used for pod_controller, but its tests located at lws_controller_test.go

Try to make the rollout PR small enough for easier reviewing.

What this PR does / why we need it
Which issue(s) this PR fixes

Fixes #
Special notes for your reviewer
No logic changed.
Does this PR introduce a user-facing change?

None

/lgtm
Good catch


added turkish lira symbol

Adding missing turkish lira symbol
Refrences:
Turkish lira wiki
Central Bank of Turkey
xe.com refrence

@sksethi25 thanks for spotting this!


Add an arbitrary interface to the router and tcpserver

I'm not entirely convinced by this, as it will only let you set a single interface, when you start the server/router. But, I can't think of anything else, so...
deferred to @CaptainHayashi

Seems better than what we have already, and that interface could always have a map/struct keyed on requests.
Make sure the tests pass, though!  Also maybe add some new ones to make sure the interface{} is passed in properly?

not really sure what such a test would look like...

@LordAro meh, don't worry about it, as long as the existing ones pass.  I was thinking some sort of function that gets passed in a string, checks to make sure that it's 1) there, 2) a string and 3) the correct string, and then sends back a yes/error response that's checked by the test runner.  Seems overcomplicated though.


Draft: Add a capability version check to the Session

This commit adds a check to the session to ensure the capability of the server is compatibility the current client. This ensures all functions and features are equally supported and defined both in the server and the client.

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

Comparison is base (edab765) 100.00% compared to head (9c52e31) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #49   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           32        32
  Lines         1872      1889   +17
  Branches       298       300    +2
=========================================
+ Hits          1872      1889   +17

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