prisma:engine spans is missing on NestJS + Apollo

I am currently implementing PrismaInstrumentation with NestJS+Apollo(GraphQL).
I want to retrieve spans about prisma, but prisma:engine is missing(retrieve only prisma:client).
Could anyone tell me how to retrieve prisma:engine?
tracer.ts
import { SemanticResourceAttributes } from '@opentelemetry/semantic-conventions';
import { registerInstrumentations } from '@opentelemetry/instrumentation';
import {
  SimpleSpanProcessor,
} from '@opentelemetry/sdk-trace-base';
import { NodeTracerProvider } from '@opentelemetry/sdk-trace-node';
import { OTLPTraceExporter } from '@opentelemetry/exporter-trace-otlp-http';
import { PrismaInstrumentation } from '@prisma/instrumentation';
import { FastifyInstrumentation } from '@opentelemetry/instrumentation-fastify';
import { HttpInstrumentation } from '@opentelemetry/instrumentation-http';
import { GraphQLInstrumentation } from '@opentelemetry/instrumentation-graphql';
import { Resource } from '@opentelemetry/resources';

(() => {
  // Configure the trace provider
  const provider = new NodeTracerProvider({
    resource: new Resource({
      [SemanticResourceAttributes.SERVICE_NAME]: 'allabout-aggregation-api',
    }),
  });
  provider.register();
  provider.addSpanProcessor(new SimpleSpanProcessor(new OTLPTraceExporter()));

  // Register your auto-instrumentors
  registerInstrumentations({
    tracerProvider: provider,
    instrumentations: [
      new HttpInstrumentation(),
      new FastifyInstrumentation(),
      new GraphQLInstrumentation(),
      new PrismaInstrumentation(),
    ],
  });
})();

main.ts
import './tracer';
import { NestFactory } from '@nestjs/core';
import {
  FastifyAdapter,
  NestFastifyApplication,
} from '@nestjs/platform-fastify';
import { AppModule } from './app.module';

async function bootstrap() {
  const app = await NestFactory.create<NestFastifyApplication>(
    AppModule,
    new FastifyAdapter(),
    {
      logger: ['error', 'warn'],
    },
  );
}
bootstrap();

The following image represents the content displayed in Jaeger. While prisma:client is visible, prisma:engine is not being displayed.

Prisma Version
4.8.0

Could you please provide more information about what version of Prisma are you using, and whether you are using Data Proxy or the local query engine? Could you please also post contents of the generator block from your schema file? Thank you!

Thank you for your you reply.
I use Prisma: 4.8.0, and local query engine. I have defined the generator block as follows.
generator client {
  provider        = "prisma-client-js"
  // 2022年2月現在preview機能だが、findUniqueをdeletedAtで絞るため仕方なく使っている
  previewFeatures = ["extendedWhereUnique", "tracing"]
}

Can you try upgrading to a more recent version to see if that fixes the missing spans?

Same behavior but only on OSX m1

Is that also true for you @ishii1648 or are you on another OS and platform?

I'm also seeing only prisma:client:* traces but nothing for prisma:engine:*. I'm testing with prisma v5.0.0 and Datadog using a setup like this: https://github.com/DataDog/dd-trace-js/issues/1244#issuecomment-1648606069.

I have also been having this issue. Turns out that defining multiple new PrismaClient instances messes things up a bit. My gut feel is that this error can be related. Tracked here: https://github.com/prisma/prisma/issues/20779

Given https://github.com/prisma/prisma/issues/20779, I assumed that maybe hot reloading in NestJS triggers this issue, but I still couldn't reproduce this after a few tries:

Compared to the original report, my reproduction didn't include Apollo in the stack, so maybe that's relevant.
@ishii1648 could you please share a reproduction we could run?

@ishii1648 @larskarbo @morlz could you please help us with a reproduction?

I am also facing this issue with NestJS (without Apollo) (details here).
Since the issue seems related to #20779, I tried the workaround mentioned there: adding engineType = "binary" in my generator, and that fixed the issue.
Of note: I implemented the access to the PrismaClient as recommended by NestJS, using a service:
import { Injectable, OnModuleInit } from '@nestjs/common';
import { PrismaClient } from '@prisma/client';

@Injectable()
export class PrismaService extends PrismaClient implements OnModuleInit {
  async onModuleInit() {
    await this.$connect();
  }
}

And then I use this service in various other services/modules.
I traced that class and found that onModuleInit() is called multiple times when the NestJS app starts, so I guess that for each module or service where I use PrismaService, a new PrismaClient is created, which makes this issue a duplicate of #20779.
I'll try to see with NestJS if that (onModuleInit() being called multiple times) is expected. There might be a bug there too.

As we've tried to reach @ishii1648 twice without success, we're now closing this issue.
For future readers, we recommend trying out this answer.
Please leave a comment below with a minimal reproduction repository if you see this issue, and we'll reopen it. Thanks!


Update style.css

because of overflow : hidden; the text was not visible which was downwards.

@Scoopy0510 Can you share a screenshot of which bug you have fixed?

@Scoopy0510 Can you share a screenshot of which bug you have fixed?
Yeah, In Attendance javascript code, the overflow property was marked as hidden that's why the text which was at the bottom was not visible. I am attaching that one. In the image below ,it is seen that the text below that is not visible as it is hidden.

But the live website doesn't have any such bug 🤔

 **But the live website doesn't have any such bu

 But the live website doesn't have any such bug 🤔

Is that so. Sorry.
I am new to Open source idk I had found this error in that code only.I am attaching it here what I had saw.

@Scoopy0510 Don't be sorry, everyone here is to learn
I think there is no mistake in the code if it was there then it would have been reflected on the website.
Look after some other issues or try to find bugs in other projects
Good Luck 👍🏻

@Scoopy0510 Don't be sorry, everyone here is to learn I think there is no mistake in the code if it was there then it would have been reflected on the website. Look after some other issues or try to find bugs in other projects Good Luck 👍🏻

That's good to hear this from you, Thank you.


Support extra ruby path

LGTM!!!

thanx!

shipped
https://forge.puppetlabs.com/lamanotrama/unicorn_sysvinit

:shipit: :v:


Adding my Repo

Hey @waleedrehman31, the added repository doesn't comply with the criteria mentioned here for a new project. Therefore, closing this PR for now. Thanks for contributing anyways! 🙌


Guidance on how to modify the solution to allow certain URLs through the XSS filter etc

Hi!
We've been following the WAFv2 implementation guide and have managed to get the solution up and running for our app. Overall most things seem to be working great, however we have a few requests that are now returning 403s due to false positives (e.g. XSS, image upload, etc).
Is there any guidance available on how we should go about modifying the solution to accommodate the false positives (without resorting to the UI)?
Seems like it would be a pretty common scenario. Maybe we could modify the solution to include a simple 'URL allow list' config value (or something)...?
Thanks!

I agree.
In general, we need to enrich the Web ACL with more custom rules.
In my case, I need to allow certain hosts to perform a lot of requests or to send requests with a sql-injection-like queries, without blocking them.

Thanks for the suggestions. Custom rules are not in the scope of this solution at this time, but we have added your requests to our backlog and will evaluate them for future releases. For now, you can use AWS WAF console to create custom rules or feel free to modify the source code (webacl and main templates) to add custom rules that fit your use case.

@aijunpeng Thanks for the response. While I'm definitely interested in this as an official feature, what I was actually asking for is some guidance in how the solution can/should be modified to suit (what seems like) a pretty common requirement...

You can try updating the statement for XssRule in aws-waf-security-automations-webacl.template source code, or updating the XssRule via WAF console -> Rule JSON editor.

close the ticket due to user inactivity


How to run flywayMigrate after the ./gradlew run?

Running uaa with postgres doesn't execute the alter db scripts. I guess this can be achieved with the gradle flyway definition.
Can you please add a way to achieve this into the docs?

Take a look at the .travis.yml script to see how to apply the PostgreSQL profile. if a migrate didn't happen, those tests wouldn't be passing

https://github.com/cloudfoundry/uaa/commit/308bd975c7f9398a8b8f523b4395b22963ad5600

I am closing this issue. After reviewing all the details, the system will always run a flyway migrate as it starts up. The documentation of how to apply the profile is available, it is not immediately clear.
We will add command line options to set the profile, and to enable/disable the flyway clean script during Tomcat startup and update the documentation as such

Thanks for this. Just a little concerned that the flyway migrate script only runs when you run a test. Please correct me if I'm wrong.
However, is there a flag to add to gradlew run in order to execute flyway migrate as well?

@charyorde When you do
./gradlew run

It will launch a Tomcat container, and deploy the UAA war file. This war file reads the DB configuration from the uaa.yml, and will connect to what ever DB you have specified in the spring_profiles setting.
The war file (the UAA application) always runs the migrate script. However, if you haven't specified the MySQL profile, the UAA simply starts an in memory database and runs the migration scripts on the in memory DB.
In the following commit we are also providing you with the ability to configure the spring profiles for gradlew run on the command line, as such
./gradlew -Dspring.profiles.active=default,mysql run

The migrations scripts always run. They are part of the application lifecycle. However, you must configure which DB they run on.

Here we show how 'flyway.migrate' always runs when the app is started up and loads the Spring XML files
https://github.com/cloudfoundry/uaa/blob/master/common/src/main/resources/spring/data-source.xml#L35

@fhanik Thanks for the detailed explanation.
However, here's my manual test that caused the illusion on gradlew run:
bash-3.2$ cat ~/javaprojects/uaa/common/build/resources/main/org/cloudfoundry/identity/uaa/db/postgresql/V1_6_0__ExtendAuthzApprovalUsername.sql
ALTER TABLE authz_approvals ALTER COLUMN username TYPE VARCHAR(255);

I was expecting to see username column on authz_approvals table but I was only able to find it in authz_approvals_old. Don't know where authz_approvals_old comes from though ;)
uaa=# \d authz_approvals;
                                Table "public.authz_approvals"
     Column     |            Type             |                   Modifiers
----------------+-----------------------------+------------------------------------------------
 user_id        | character varying(36)       | not null
 client_id      | character varying(36)       | not null
 scope          | character varying(255)      | not null
 expiresat      | timestamp without time zone | not null default now()
 status         | character varying(50)       | not null default 'APPROVED'::character varying
 lastmodifiedat | timestamp without time zone | not null default now()
Indexes:
    "new_authz_approvals_pkey" PRIMARY KEY, btree (user_id, client_id, scope)

uaa=#  \d authz_approvals_old;
                              Table "public.authz_approvals_old"
     Column     |            Type             |                   Modifiers
----------------+-----------------------------+------------------------------------------------
 username       | character varying(255)      | not null
 clientid       | character varying(36)       | not null
 scope          | character varying(255)      | not null
 expiresat      | timestamp without time zone | not null default now()
 status         | character varying(50)       | not null default 'APPROVED'::character varying
 lastmodifiedat | timestamp without time zone | not null default now()
Indexes:
    "authz_approvals_pkey" PRIMARY KEY, btree (username, clientid, scope)

Thanks all the same.

Look through the other migration scripts. You will see each change as it is performed. Alternatively, you can enable debug output. and see each SQL command being executed.
So it seems all your migrations are run. Do a 'select * from schema_version' to see if your migrations have been applied or not. It will list all versions there. Migrations are not run everytime, only when they are needed as determined by the content in the schema_version table
If all seems ok, I will ask you to close this issue.


feat: Allow All Users to Perform LDAP Search Lookups in their org

fixes https://github.com/casdoor/casdoor/issues/3042

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


UNable to debug remotely

Repro steps:

Action: appService.StartRemoteDebug
Error type: Error
Error Message: Unable to establish connection to application: WebApp is stopped, try sending a request to start it up
Version: 0.13.0
OS: win32
VS Code Version: 1.33.1

Call Stack
TunnelProxy.<anonymous> (dist/extension.bundle.js:333:934898)
Generator.throw (<anonymous>)
a (dist/extension.bundle.js:333:931125)

Hi @himsoftware,
Sorry that you're having this issue.  Has this problem consistently reproed?  If it is due the Web App being stopped, could you try restarting it, waiting a minute, and then try remote debugging?


Projects do not load

currently, the website cannot recognize projects.

currently, the website cannot recognize projects.

Strange. I'll look into it soon, if you want. (can't access a desktop, currently signed in from a school laptop via chrome://chrome-signin )

Also, Th

currently, the website cannot recognize projects.

Strange. I'll look into it soon, if you want. (can't access a desktop, currently signed in from a school laptop via chrome://chrome-signin )

Don't know if this will help, but I've noticed the website now loads just a black screen...
I'm assuming this error from the console could help on pinpointing what the major problem is.
TypeError: Cannot read properties of undefined (reading 'toLowerCase')
at App.js:32:12
at Array.filter ()
at i.value (App.js:30:8)
at $o (react-dom.production.min.js:187:188)
at Qo (react-dom.production.min.js:186:173)
at $i (react-dom.production.min.js:269:427)
at Pu (react-dom.production.min.js:250:347)
at xu (react-dom.production.min.js:250:278)
at Au (react-dom.production.min.js:250:138)
at vu (react-dom.production.min.js:243:163)

It appears I may have messed something up… It has problems with search sorting.

https://scratchbackup.github.io/projects.json is an array of project IDs. Let's say we are looking at ID 1. It's just a string, and string.title will be undefined. So you will need to either fetch from the Scratch API or change the structure of your array to contain the title as well.


Switch to fullscreen

I'm quite certain it's in front of me, but I wasn't able to work it out: how can I switch to fullscreen mode?
We are talking about a native playground here.

Assuming you mean exclusive fullscreen mode, this isn't directly supported at the moment. It appears bgfx doesn't support this either.

well, apart from it's too bad, is there a way to have a borderless fullscreen window?

Sure. If you don't need exclusive fullscreen, you can create the window for the targeted platform to be windowless. That isn't technically part of the Babylon Native library and more on the app/UI side.

Is there a documentation somewhere about this?
Maybe I shall continue reading bgfx docs?

Is there a documentation somewhere about this?

What platform are you targeting?

Maybe I shall continue reading bgfx docs?

Only if you are planning to contribute a fix to Babylon Native or bgfx.

Well, back then I thought there will be a generic "in-babylon-native" call to switch to fullscreen. So I thought I'll cover all the supported platforms.
arm64 is the most important one currently. I'm trying to build stuff that runs on an Nvidia Jetson Nano.

one more, I know it's not the right place, but since we are in a conversation:
void Graphics::Impl::DisableRendering()
{

   assert(m_renderThreadAffinity.check());

If I leave this line in the source in Core/Graphics/Source/Graphics.cpp then the app crashes on each exit.

a generic "in-babylon-native" call to switch to fullscreen

That's why I assumed you mean exclusive fullscreen and even then it only makes sense for desktop. A generic fullscreen feature probably doesn't make much sense for Babylon Native.

arm64 is the most important one currently. I'm trying to build stuff that runs on an Nvidia Jetson Nano running Ubuntu.

Linux is what I was looking for :-) The CPU architecture doesn't really matter. As for the specifics, I don't know how to do it in Linux, but a search for "x11 make window fullscreen" seems to have some relevant results.

Core/Graphics/Source/Graphics.cpp then the app crashes on each exit.

Yeah, we know about this one. It's an assert though, not a crash. We will fix it soon. It's rather annoying. ;-)

Many thanks. The picture is clear now.
Many thanks for this awesome product!


In post.html_content, are special chars escaped properly?

^^

I don't know, the documentation of the used API (ScratchDB) doesn't provide information about that. Maybe try it out yourself


Allow setting ForcedTags using the acls.hujson config file

This pull request introduces a declarative way to set ForcedTags on nodes by recursively expanding aliases. These declarative forced tags are considered separately from the ones stored in the database.
Continuation of #1490
{
    "hosts": {
        "a": "fd7a:115c:a1e0::1",
        "b": "fd7a:115c:a1e0::2"
    },
    "forcedTags": {
        "tag:some-tag": [
            "a",
            "tag:some-other-tag"
        ],
        "tag:some-other-tag": [
            "b"
        ]
    }
}

[x] read the CONTRIBUTING guidelines
[x] raised a GitHub issue or discussed it on the projects chat beforehand
[x] added unit tests
[ ] added integration tests
[ ] updated documentation if needed
[ ] updated CHANGELOG.md

Dear @networkException ,
Can you please explain a little bit more about this approach?

What is it good for?
How can it be used better than the "ones stored in the database" ?
Are these extra tags show/used in some of the UI ?

Maybe reviewers would understand it better too, why did you commit this.
(Personally I'm not.)

I do not think we want to support more things in the policy file. It is currently has a lot of moving parts, and it does not work as intended. Adding more will make it harder for us to get it into shape. If there is a good case for this, it can be revisited in the future.


Added Prathima Kadari as Mentor

I'm looking forward to mentor newbies to open source and get them used to github.

@prathimacode-hub hey, if you have any projects please send the pr or dm me i will add it.

Sure, I have many open source projects. Can I add them here directly or through a PR. @iamdevvalecha

@prathimacode-hub hey, if you have any projects please send the pr or dm me i will add it.

I had added my projects through PR. Kindly check on to them. @iamdevvalecha


mjollnir block has no gravity

Version: MythicBotany-1.16.4-1.3.15
Example: Be flying with mjollnir when absorption runs out.
Outcome: Mjollnir block is placed in the air and stays there.
Expected Outcome: Mjollnir block is placed in the air and falls like an anvil.

This is intended


Update query for Supplemental table in marker paper

Query is at https://github.com/x-atlas-consortia/hra-pop/blob/main/queries/hra/ct-per-organ-per-tool.rq
Expected number: 58.

I see. Can we amend the query to only use the finest levels.

@bherr2
Can you please re-run the query?

With @bherr2
58 is the number of unique CTs Azimuth can identify in kidney across all levels. Filtering by lowest level is hard because different tools label annotation levels differently.


something is corrupting weights

Today generations started returning black squares for opendalle sdxl. Yesterday those generations were working fine.
After digging into it I found there are some tensors in the unet that are now set to nan.  The hash value of the weights has changed. Weird.

Has not re-occurred. Will re-open if I get more information.


Optimisation question

I was looking the tick system when something is questioning me with this system.
Why not replace this code:
static uint32_t sys_time = 0;
static volatile uint8_t tick_irq_flag;
LV_ATTRIBUTE_TICK_INC void lv_tick_inc(uint32_t tick_period)
{
    tick_irq_flag = 0;
    sys_time += tick_period;
}
uint32_t lv_tick_get(void)
{
    uint32_t result;
    do {
        tick_irq_flag = 1;
        result = sys_time;
    } while(!tick_irq_flag);     /*'lv_tick_inc()' clears this flag which can be in an interrupt. Continue until make a non interrupted cycle */

    return result;
}

by this code:
static volatile uint32_t sys_time = 0;
LV_ATTRIBUTE_TICK_INC void lv_tick_inc(uint32_t tick_period)
{
    sys_time += tick_period;
}
uint32_t lv_tick_get(void)
{
    return sys_time;
}

The initial code is faster i guess ?  sys_time usage will not be optimized by the compiler.
why use do { ... } while loop,  it prevents what? if an interrupt happen, the sys_time is just updated ?
i am curious.

On a 32 bit processor sys_time += tick_period would be an atomic operation, but on an 8 or 16 bit processor it would not be. tick_irq_flag and do{ ... } while loop ensure that the sys_time is unfettered by a tick interrupt when it is sampled.

Hi,
@DeonMarais64 I just started to write the same answer :)

thank for your answer! To complement:

Why not put volatile on sys_time since this variable is in an interruption? the fact that it is to modify only in this interuption? I do not have to use the volatile keyword on sys_time any more?

Put volatile on sys_time or not much affects the performance (speed of treatment)?

If the new code is faster, can we integrate it in lvgl with an option (preprocessor) that will be selected according to the architecture (8-16-32-64- ..)?
E.g: sys_tick can be used in 64 bits on 64 bits architecture. No need to control whether it overflow either.
We can use a new section in lv_conf to gather all the small optimization options. E.g:

/*=================
   Performance. setting
 *=================*/
#define LV_ATTRIBUTE_TICK_INC                 /* Define a custom attribute to `lv_tick_inc` function */
#define LV_ATTRIBUTE_TASK_HANDLER             /* Define a custom attribute to `lv_task_handler` function */
#define LV_ARCHITECTURE  32
typedef int32_t  lv_pixel_t                               /*   X / Y coordinate  size  */
....

volatile is for the optimization. The compiler might optimize out a variable if it's not changed in the code. And if the variable is changed in the interrupt the compiler won't see the change.
As the loop needs to run twice very very rarely your suggestion can have only a very little effect in performance but would add a new configuration option which can be a root of issues if set wrong.
But I maybe I don't see the full picture. In your case, this update would have a bigger effect?

Hi,
There is often a tradoff between absolute optimisation and simplicity. IMO they way it is is a good compromise.
In this instance using the volatile keyword for sys_time would do no harm and is unlikely to offer much ITO opsimisation, it would only realy be necesarry if sys_time were a global variable. But that would mean whomever is referencing it would need to provide an atomic modify guard similar to what is already been done for each reference instance in cases where processor word size does not match sys_time's type.

About bit field started at the end of issue #298, the current writing is this:
typedef enum
{
    LV_SB_MODE_OFF 	= 0x0,   	/*Never show scrollbars*/
    LV_SB_MODE_ON  	= 0x1,   	/*Always show scrollbars*/
    LV_SB_MODE_DRAG = 0x2,   	/*Show scrollbars when page is being dragged*/
    LV_SB_MODE_AUTO = 0x3,   	/*Show scrollbars when the scrollable container is large enough to be scrolled*/
	LV_SB_MODE_HIDE = 0x4,	 	/*Hide the scroll bar temporally*/
	LV_SB_MODE_UNHIDE = 0x5,	/*Unhide the previously hidden scrollbar. Recover it's type too*/
} lv_sb_mode_t;

/*Data of page*/
typedef struct
{
    lv_cont_ext_t bg; /*Ext. of ancestor*/
    /*New data for this type */
    lv_obj_t * scrl;            /*The scrollable object on the background*/
    lv_action_t rel_action;     /*Function to call when the page is released*/
    lv_action_t pr_action;      /*Function to call when the page is pressed*/
    struct {
        lv_style_t *style;          /*Style of scrollbars*/
        lv_area_t hor_area;            /*Horizontal scrollbar area relative to the page. (Handled by the library) */
        lv_area_t ver_area;            /*Vertical scrollbar area relative to the page (Handled by the library)*/
        uint8_t hor_draw :1;        /*1: horizontal scrollbar is visible now (Handled by the library)*/
        uint8_t ver_draw :1;        /*1: vertical scrollbar is visible now (Handled by the library)*/
        uint8_t mode     :3;        /*Scrollbar visibility from 'lv_page_sb_mode_t'*/
    } sb;
    uint8_t arrow_scroll :1;		/*1: Enable scrolling with LV_GROUP_KEY_LEFT/RIGHT/UP/DOWN*/
} lv_page_ext_t;

the next writing seems more readable/ usefull for you ?  (It is minor)
enum
{
    LV_SB_MODE_OFF 	= 0x0,   	/*Never show scrollbars*/
    LV_SB_MODE_ON  	= 0x1,   	/*Always show scrollbars*/
    LV_SB_MODE_DRAG = 0x2,   	/*Show scrollbars when page is being dragged*/
    LV_SB_MODE_AUTO = 0x3,   	/*Show scrollbars when the scrollable container is large enough to be scrolled*/
	LV_SB_MODE_HIDE = 0x4,	 	/*Hide the scroll bar temporally*/
	LV_SB_MODE_UNHIDE = 0x5,	/*Unhide the previously hidden scrollbar. Recover it's type too*/
};
typedef uint8_t lv_sb_mode_t;

/*Data of page*/
typedef struct
{
    lv_cont_ext_t bg; /*Ext. of ancestor*/
    /*New data for this type */
    lv_obj_t * scrl;            /*The scrollable object on the background*/
    lv_action_t rel_action;     /*Function to call when the page is released*/
    lv_action_t pr_action;      /*Function to call when the page is pressed*/
    struct {
        lv_style_t *style;          /*Style of scrollbars*/
        lv_area_t hor_area;            /*Horizontal scrollbar area relative to the page. (Handled by the library) */
        lv_area_t ver_area;            /*Vertical scrollbar area relative to the page (Handled by the library)*/
        uint8_t hor_draw :1;        /*1: horizontal scrollbar is visible now (Handled by the library)*/
        uint8_t ver_draw :1;        /*1: vertical scrollbar is visible now (Handled by the library)*/
        lv_sb_mode_t mode     :3;        /*Scrollbar visibility from 'lv_page_sb_mode_t'*/
    } sb;
    uint8_t arrow_scroll :1;		/*1: Enable scrolling with LV_GROUP_KEY_LEFT/RIGHT/UP/DOWN*/
} lv_page_ext_t;

The difference is in the new way to declare the lv_sb_mode_t enum to be the same size as uint8_t?
Also if i recall correctly bitfields should be at least 2bits wide, with the most significant bit being the sign bit, but don't quote me on that, i'm just thinking loud.

@Zaltora
It's a great mixture of the two way of thinking! Congrats! :)
I was thinking about what happens if this way you place two bitfield "below" each other. I thought maybe they will be interpreted as sperate bytes but I was wrong. It works well even if with a mixed typedef combo like this:
˙``c
typedef uint8_t enum_8_typedef;
typedef uint16_t enum_16_typedef;
typedef struct {
uint8_t         field_A :1;
uint8_t         field_B :2;
enum_8_typedef  field_C :3;
enum_16_typedef field_D :3;
enum_8_typedef  field_E :3;
enum_16_typedef field_F :3;
}bit_field_struct_t;

It's still 2 bytes (`sizeof(bit_field_struct_t) is `2``). So I think we can adopt this declaration format because it improves the readability while keeps memory usage low!

@C47D
I often use 1 bit wide bitfields and they work for me as simple 0/1 "numbers". I never heard about the sign bit in bit fields. Can show a description of it?

@C47D
Thank you!
It says: "a 1-bit signed field on a two's complement machine can only take the values 0 or -1"
I didn't know it but it's quite interesting. However, it's about: signed field. So I think until we use unsigned data it"s fine to use 1-bit bitfield too. Do you agree?

It is fine to use 1-bit bitfield, I use this way too. However i will take a look about portability. Using bitfield with register seem not good anymore for me.
I never test to use following uint8_t and uint16_t. It's not dangerous to do that ? In your case, it is feel to work and use only 2 bytes but i am not sure of the behavior for all compiler.

Using bitfield with register seem not good anymore for me.

Why?

I never test to use following uint8_t and uint16_t. It's not dangerous to do that ? In your case, it is feel to work and use only 2 bytes but i am not sure of the behavior for all compiler.

I also think its dangerous :) Using only uin8_t types looks much safer. I just would like to know an extreme mixing of types still works at least in GCC.

I didn't know it but it's quite interesting. However, it's about: signed field. So I think until we use unsigned data it"s fine to use 1-bit bitfield too. Do you agree?

I do agree. Anyway, if it worked so far (1bit bitfields) you shouldn't change it.

@kisvegabor  the order is not guarantate. e.g with a 16 bit register for a driver:
typedef union
{
    struct {
        uint16_t cvrf : 1;   // LSB or MSB ?? Compiler dependancy  (Need be LSB to work)
        uint16_t tcf  : 1;
        uint16_t pvf  : 1;
        uint16_t wf   : 3;
        uint16_t sf   : 1;
        uint16_t cf   : 3;
        uint16_t cen  : 1;
        uint16_t wen  : 1;
        uint16_t scc3 : 1;
        uint16_t scc2 : 1;
        uint16_t scc1 : 1;
        uint16_t      : 1;     //MSB  or LSB ?? compiler dependancy (Need be MSB to work)
    };
    uint16_t mask_register;
} ina3221_mask_t;

If you describe a register like this, it will be compiler dependant. In one case, the register description is false.

In general, it's not a problem if the bitfield declaration and its usage remain in the same code because the compiler will reference the fields as it declared them.
However, if you use it in a message then it's possible that the other system will interpret it differently. And in case of LittlevGL the colors might be problematic because the images are generated externally with a hypothetical bit order.
Anyway, I'd like to keep the current way until somebody reports an issue about it. It seems usually the same order is used in the compilers.

Thank you for the ideas guys!
I think the final conclusion is the
 lv_sb_mode_t mode     :3;

bitfield would be better than the current approach. So if you have time and intention, a Pull Request would be welcome. :)


unable to load model when using ollama(local) provider

What happened?
i can not load model,but the address has exist ollama model,my friend can load model with same address

the version of idea is

Relevant log output or stack trace
No response
Steps to reproduce
No response
CodeGPT version
2.7.1-233
Operating System
Windows

I think this is probably due to the trailing "/" in your Base host. Try to remove it and test it again.
I will create a PR to mitigate this annoying problem

not useful by remove trailing slashes


chore(4.4): release 1.7.8

:robot: I have created a release beep boop
1.7.8 (2022-11-14)
Bug Fixes

add detailed oxShibboleth apploggers https://github.com/GluuFederation/docker-oxshibboleth/issues/48 (0733a3b)
keep oxsshipoleth disabled by default (7632a89)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/GluuFederation/cloud-native-edition/releases/tag/v1.7.8 :sunflower:


Code workflow

This is the rest merge from work-flow branch to master branch

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

good to go.


404 at /machine/drivers/

I found a broken link : https://docs.docker.com/machine/drivers/
I arrived on this page through: https://www.google.com/

Thanks for reporting; this content is deprecated, but looks like we missed a redirect for that specific URL; I opened https://github.com/docker/docker.github.io/pull/13836 to add the missing redirect 👍


chore(changeset): baseBranch should the branch that PRs are based on / created against

This also fixes an issue where users who just cloned the repo, and thus don't have the main branch locally, are unable to create a changeset.

To be sure I'm not totally in the wrong here, and that we don't break changeset again -- I'm requesting a review from @ernestoresende who set up changeset.

Upon further evaluation, this PR has one additional implication, as I believe it will change the GitHub Action for change-sets on the main branch to instead create a PR against beta/develop.
If we are okay with this workflow, I believe it will work like this after merging this PR:

Like before, merge to main when you want to do a release.
It will create a PR against beta/develop.
A maintainer merges said PR into beta/develop.
The maintainer merges/pushes (locally with the CLI, or however) back into main to get it caught back up (so the changesets and version bump are present in the main branch).

We could try to minimize manual intervention or change this around, but I think a stable, working procedure may be more important for now.
@danstepanov if you approve, we can merge this (so the likelihood of someone having an issue with bunx changeset is as small as possible) and I'll follow up with the name switch to develop.

Delineated some more thoughts on a new RFC:

https://github.com/danstepanov/create-expo-stack/discussions/86

But as it regards to this PR, the GitHub Action is hardcoded to run on a specific branch mentioned by name. The changes in this PR should not affect how the GitHub Actions run.

But as it regards to this PR, the GitHub Action is hardcoded to run on a specific branch mentioned by name. The changes in this PR should not affect how the GitHub Actions run.

Not quite. You're right that it's hardcoded to run against the main branch -- but the changeset GH action determines what branch to open it's auto-generated pull request  against depending on the baseBranch.

Delineated some more thoughts on a new RFC

Checking this out now! 🙌

@sammoore what's the story on this PR?


Screen

Hi! Thanks for your job, your plugin is the unique one that works with Air Purifier 3C
I’d like to disable the “Screen” brightness from HomeKit, my interest is just to have the Air Purifier without screen control
Do you know how can I remove from HomeKit?
Second question,  any changes to let appear the temperature and humidity sensors?
This is my configuration:
"accessories": [],
"platforms": [
    {
        "platform": "miot",
        "devices": [
            {
                "name": "Purificatore",
                "ip": "192.168.178.xxx",
                "token": "adxxxxxxxxxxb",
                "deviceId": "3xxxxxxx",
                "model": "zhimi.airpurifier.mb4",
                "pollingInterval": 5,
                "buzzerControl": false,
                "childLockControl": false,
                "ledControl": false,
                "swingControl": false,
                "moveControl": false,
                "fanLevelControl": false,
                "ioniserControl": false,
                "dryControl": false,
                "modeControl": false,
                "lightModeControl": false,
                "motorControl": false,
                "pm25Breakpoints": [
                    5,
                    12,
                    35,
                    55
                ]
            }
        ]
    }
]

}

As for screen, try to set the screenControl property to true, that should enable screen controls in HomeKit.
As for temperature and humidity, it looks like the air purifier does not support that so it will not show up. At least it seems not to be reported by purifier over the specs.

As for screen, try to set the screenControl property to true, that should enable screen controls in HomeKit.
As for temperature and humidity, it looks like the air purifier does not support that so it will not show up. At least it seems not to be reported by purifier over the specs.

"accessories": [],
"platforms": [
    {
        "platform": "miot",
        "devices": [
            {
                "name": "Purificatore",
                "ip": "<IP_ADDRESS>",
                "token": "adfb4337ce7861580eaa8573847f77fb",
                "deviceId": "359581494",
                "model": "zhimi.airpurifier.mb4",
                "pollingInterval": 5,
                "buzzerControl": false,
                "childLockControl": false,
                "ledControl": false,
                "swingControl":  #false,'
                "ScreenControl": true,
                "moveControl": false,
                "fanLevelControl": false,
                "ioniserControl": false,
                "dryControl": false,
                "modeControl": false,
                "lightModeControl": false,
                "motorControl": false,
                "pm25Breakpoints": [
                    5,
                    12,
                    35,
                    55
                ]
            }
        ]
    }
]

}
Done. But the tile persist in HomeKit (my goal is to remove it)

If you want to remove it then you have to set screenControl to false, please note that it needs to be written with a small s at the beginning.

If I can butt in, I had some such issues while developing a homebridge plugin of my own; when an accessory is cached, it retains services it registered between restarts of the server, even if the service itself is removed. The plugin should remove these services when they are not needed anymore. Likewise, when I started using this plugin, I initially used the screen switches but they were cluttering HomeKit and so I disabled the switch in config.json, and the switches persisted (while doing nothing, ofc).
As a stopgap solution, you can remove the entire accessory from homebridge cache (menu->homebridge settings->remove a single accessory from cache and remove just your device). The plugin will rediscover it from scratch and then the extra switches will be gone. Only caveat is the device will be set up as new in HomeKit and you'll have to set your automations again.


git+ssh support

Just moving an internal project to duniverse and needed these changes for support. This supports adding a opam repo to the duniverse switch and then pulling git+ssh packages from it when it's on a private git server.

thanks!


Add Milestone and improve more

Motivation and Context
We sometimes need to validate pr whether milestone is set, so I've prepared milestone model.
Description
Other changes as below

[x] add milestone model
[x] change iso8601 date string type to Date type using .iso8601 date decoding strategy (all date string get from GitHub API are formatted by iso8601 )
[x] resolved DecodingError of fixtures/eidolon_609.json.
[x] change iso6801 on comments, which means rubbers and plastic houses to iso8601.

And I've added milestone structure on fixtures/eidolon_609.json to check decoding json correctly.

@SD10 Thank you for quick review! I fixed all😄

lol 🦆

Ah, yeah, this is something I hadn't really thought of - in DangerJS we only provide interfaces for the objects, but it's fine to access keys in the DSL without specifying them into the DSL. Looks like there'll be more easy low-hanging fruit like this 👍


Feature Request: Yeelight Meteorite

Hi,
just because you wrote FeatureRequest welcome. I would love to be able to use my Yeelight Meteor Lights. Is there any hope for me?

Update:  this basically works with my Yeelight Meteorite - just downloaded today's version.  I'm able to control the downlight (temp, brightness, on/off) and separately the background light (temp, colour, brightness, on/off).  The only issue I've found is that it doesn't remember the background light colour/temp settings after editing it.  Almost perfect!

@lyance @cellcortex I finaly setupped my whole homebridge again and now i actually got it to work too. Thank you for giving me hope ;) Main lights react snappy but i cant turn it off via klicking on the icon but instead i have to klick on it and pull the brightness all the way up to 100% or 0%, is there anything i can configure so that i can just click on it?
The Backround lights show up in homebridge but not yet in my home app, also i can turn a light on or off once and then they dont react for about 30 sec anymore. Also no way to switch colors.


无法获取到“系统信息”

hello,
我遇到了无法获取系统信息的情况，如下图

日志部分如下：

><EMAIL_ADDRESS>dev /home/milo/all-in-one
> cross-env EZM_ENV=local nodemon index.js

[nodemon] 2.0.20
[nodemon] to restart at any time, enter `rs`
[nodemon] watching path(s): *.*
[nodemon] watching extensions: js,mjs,json
[nodemon] starting `node index.js`
Debugger listening on ws://<IP_ADDRESS>:53055/948db6af-273c-4967-8bed-2d3e0fd3c109
For help, see: https://nodejs.org/en/docs/inspector
Debugger attached.
2023-07-29 10:16:22,301 INFO 1082516 [master] node version v18.12.1
2023-07-29 10:16:22,302 INFO 1082516 [master] @xprofiler/console version 1.0.5
2023-07-29 10:16:22,305 INFO 1082516 [master] node version v18.12.1
2023-07-29 10:16:22,305 INFO 1082516 [master] @xprofiler/manager version 1.0.1
2023-07-29 10:16:22,307 INFO 1082516 [master] node version v18.12.1
2023-07-29 10:16:22,307 INFO 1082516 [master] @xprofiler/wsserver version 1.0.3
2023-07-29 10:16:22,886 INFO 1082516 [master] agent_worker#1:1082586 started (568ms)
2023-07-29 10:16:22,898 INFO 1082516 [master] agent_worker#1:1082579 started (588ms)
2023-07-29 10:16:22,904 INFO 1082580 [RemoteConfig] loading remote config and save to /home/milo/all-in-one/run/remote_config.json
2023-07-29 10:16:22,930 INFO 1082516 [master] agent_worker#1:1082580 started (617ms)
2023-07-29 10:16:23,481 INFO 1082640 [RemoteConfig] loading remote config from /home/milo/all-in-one/run/remote_config.json
2023-07-29 10:16:23,499 INFO 1082653 [RemoteConfig] loading remote config from /home/milo/all-in-one/run/remote_config.json
2023-07-29 10:16:23,624 INFO 1082516 [master] @xprofiler/wsserver started on http://<IP_ADDRESS>:9190 (1317ms)
2023-07-29 10:16:23,633 INFO 1082586 [xtransit] xtransit start.
2023-07-29 10:16:23,634 INFO 1082586 [xtransit] websocket client connecting to ws://<IP_ADDRESS>:9190...
2023-07-29 10:16:23,655 INFO 1082586 [xtransit] websocket client has been connected.
2023-07-29 10:16:23,669 ERROR 1082639 [-/<IP_ADDRESS>/-/17ms GET /] getAppSecret http failed: RequestError: connect ECONNREFUSED <IP_ADDRESS>:8543, POST http://<IP_ADDRESS>:8543/xtransit/app_secret -1 (connected: false, keepalive socket: false, agent status: {"createSocketCount":1,"createSocketErrorCount":0,"closeSocketCount":0,"errorSocketCount":1,"timeoutSocketCount":0,"requestCount":0,"freeSockets":{},"sockets":{"<IP_ADDRESS>:8543:":1},"requests":{}}, socketHandledRequests: 1, socketHandledResponses: 0)
headers: {}, raw data: [object Object]
2023-07-29 10:16:23,699 ERROR 1082639 [-/<IP_ADDRESS>/-/47ms GET /] getAppSecret http failed: RequestError: connect ECONNREFUSED <IP_ADDRESS>:8543, POST http://<IP_ADDRESS>:8543/xtransit/app_secret -1 (connected: false, keepalive socket: false, agent status: {"createSocketCount":2,"createSocketErrorCount":0,"closeSocketCount":1,"errorSocketCount":2,"timeoutSocketCount":0,"requestCount":0,"freeSockets":{},"sockets":{"<IP_ADDRESS>:8543:":1},"requests":{}}, socketHandledRequests: 1, socketHandledResponses: 0)
headers: {}, raw data: [object Object]
2023-07-29 10:16:23,715 INFO 1082516 [master] @xprofiler/console started on http://<IP_ADDRESS>:8443 (1413ms)
2023-07-29 10:16:23,724 INFO 1082579 [xtransit] xtransit start.
2023-07-29 10:16:23,727 INFO 1082579 [xtransit] websocket client connecting to ws://<IP_ADDRESS>:9190...
2023-07-29 10:16:23,744 INFO 1082579 [xtransit] websocket client has been connected.
2023-07-29 10:16:23,747 ERROR 1082639 [-/<IP_ADDRESS>/-/10ms GET /] getAppSecret http failed: RequestError: connect ECONNREFUSED <IP_ADDRESS>:8543, POST http://<IP_ADDRESS>:8543/xtransit/app_secret -1 (connected: false, keepalive socket: false, agent status: {"createSocketCount":3,"createSocketErrorCount":0,"closeSocketCount":2,"errorSocketCount":3,"timeoutSocketCount":0,"requestCount":0,"freeSockets":{},"sockets":{"<IP_ADDRESS>:8543:":1},"requests":{}}, socketHandledRequests: 1, socketHandledResponses: 0)
headers: {}, raw data: [object Object]
2023-07-29 10:16:23,751 ERROR 1082586 [xtransit] Error: Command failed: df -P
df: /run/user/1000/doc: Operation not permitted

2023-07-29 10:16:23,802 ERROR 1082639 [-/<IP_ADDRESS>/-/151ms GET /] getAppSecret http failed: RequestError: connect ECONNREFUSED <IP_ADDRESS>:8543, POST http://<IP_ADDRESS>:8543/xtransit/app_secret -1 (connected: false, keepalive socket: false, agent status: {"createSocketCount":5,"createSocketErrorCount":0,"closeSocketCount":3,"errorSocketCount":4,"timeoutSocketCount":0,"requestCount":0,"freeSockets":{},"sockets":{"<IP_ADDRESS>:8543:":2},"requests":{}}, socketHandledRequests: 1, socketHandledResponses: 0)
headers: {}, raw data: [object Object]
2023-07-29 10:16:23,803 ERROR 1082639 [-/<IP_ADDRESS>/-/152ms GET /] getAppSecret http failed: RequestError: connect ECONNREFUSED <IP_ADDRESS>:8543, POST http://<IP_ADDRESS>:8543/xtransit/app_secret -1 (connected: false, keepalive socket: false, agent status: {"createSocketCount":5,"createSocketErrorCount":0,"closeSocketCount":3,"errorSocketCount":5,"timeoutSocketCount":0,"requestCount":0,"freeSockets":{},"sockets":{"<IP_ADDRESS>:8543:":2},"requests":{}}, socketHandledRequests: 1, socketHandledResponses: 0)
headers: {}, raw data: [object Object]
2023-07-29 10:16:23,840 ERROR 1082639 [-/<IP_ADDRESS>/-/103ms GET /] getAppSecret http failed: RequestError: connect ECONNREFUSED <IP_ADDRESS>:8543, POST http://<IP_ADDRESS>:8543/xtransit/app_secret -1 (connected: false, keepalive socket: false, agent status: {"createSocketCount":6,"createSocketErrorCount":0,"closeSocketCount":5,"errorSocketCount":6,"timeoutSocketCount":0,"requestCount":0,"freeSockets":{},"sockets":{"<IP_ADDRESS>:8543:":1},"requests":{}}, socketHandledRequests: 1, socketHandledResponses: 0)
headers: {}, raw data: [object Object]
2023-07-29 10:16:23,861 INFO 1082516 [master] @xprofiler/manager started on http://<IP_ADDRESS>:8543 (1556ms)
2023-07-29 10:16:23,869 INFO 1082580 [xtransit] xtransit start.
2023-07-29 10:16:23,870 INFO 1082580 [xtransit] websocket client connecting to ws://<IP_ADDRESS>:9190...
2023-07-29 10:16:23,891 ERROR 1082579 [xtransit] Error: Command failed: df -P
df: /run/user/1000/doc: Operation not permitted

2023-07-29 10:16:23,901 INFO 1082580 [xtransit] websocket client has been connected.
2023-07-29 10:16:24,062 ERROR 1082580 [xtransit] Error: Command failed: df -P
df: /run/user/1000/doc: Operation not permitted

之后就一直是这条消息不断滚动了:
2023-07-29 10:17:23,751 ERROR 1082586 [xtransit] Error: Command failed: df -P
df: /run/user/1000/doc: Operation not permitted

我尝试搜索这个问题，但是又看不懂相关内容。。。。

就是采集 system 信息的时候报错了，所以没上报。
df -P 自动获取 /run/user/1000/doc 目录因为没权限导致的失败，你可以手动配置 xtransit 的上报目录：

disks 数组里配置你有权限的目录即可，比如 process.cwd()

感谢，一切正常了  💕


Update vminsights-performance.md

Removed references to health feature.
Put note section for the 'Top N Charts' page to more prominently display that these charts will only show the top 5 resources.

@guywi-ms

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


dnm: add one more test to fetch all cdc logs

What problem does this PR solve?
test only, dnm
What is changed and how it works?
test only
Check List
Tests

Unit test
Integration test

Release note
None

/hold

/run-all-tests

/run-all-tests

/run-kafka-integration-test

/run-kafka-integration-test

/run-integration-tests

/run-all-tests


Python3?

Just started using your code.  I also just started using python 3.5.   Do you have any plans for 3.5 support?  If not I'll give it a shot.

I wasn't really planning doing that so if you could help with it, that would be awesome :)

Cool, I've forked your code.  I'm testing Py3 now.   I've also added the skeleton for setup tools so it can be installed as a package.   Have you ever considered registering this on pypy?
Also, seems a bit brittle now.  Know what's going on here? https://github.com/mbernico/boruta_py/blob/master/boruta/examples/iris.ipynb

ValueError                                Traceback (most recent call last)
 in ()
----> 1 feat_selector.fit(X,y)
/home/mike/dev/boruta_py/boruta/boruta_py2.py in fit(self, X, y)
169         """
170
--> 171         return self._fit(X, y)
172
173     def transform(self, X, weak=False):
/home/mike/dev/boruta_py/boruta/boruta_py2.py in fit(self, X, y)
311         else:
312             # and 2 otherwise
--> 313             ranks = ranks - np.min(ranks) + 2
314         self.ranking[not_selected] = ranks
315
/home/mike/anaconda/envs/Python2.7/lib/python2.7/site-packages/numpy/core/fromnumeric.pyc in amin(a, axis, out, keepdims)
2357     else:
2358         return _methods._amin(a, axis=axis,
-> 2359                               out=out, keepdims=keepdims)
2360
2361
/home/mike/anaconda/envs/Python2.7/lib/python2.7/site-packages/numpy/core/_methods.pyc in _amin(a, axis, out, keepdims)
27
28 def _amin(a, axis=None, out=None, keepdims=False):
---> 29     return umr_minimum(a, axis, None, out, keepdims)
30
31 def _sum(a, axis=None, dtype=None, out=None, keepdims=False):
ValueError: zero-size array to reduction operation minimum which has no identity

I'll hope to get it incorporated into scikit learn actually, but distributing it via pypy would be a nice planB..
Thanks for adding setup.py and fixing the bug when all features are selected.


 LODs fix? fxaa fix? which ones are already included in lazy_pack?)
if i want cool stability, cool graphics, etc.
(i7 11700, rtx 3080)
im asking bcs im confused and its not clear which mods already come with the lazy pack and which dont.....

this information is available in both the release notes, and the description of the github page
https://github.com/HolographicWings/TOTK-Mods-collection
as always with lazy packs, you shouldn't mix them with other graphics mods - the lazy pack produces the optimal combination.
all of your suggestions are either redundant, or degrade the visual quality as they weren't meant to be used in conjunction with chuck's more recent iterations of DynamicFPS.
tldr; don't mix anything. its got you covered.
feel free to add whatever cheats and 'real' mods like, i dont know, bettersages?
if you use the lazy pack, you shouldnt use anything else from this repo.
except aspect ratios i guess. keep in mind that they share files with controller mods, so if you want to use both, you will have to merge the files in question using the developers own tool:
https://github.com/Fruithapje21/TOTK-Ultrawide

tysm, i little confused bcs i see "lod improvement 1.12", and think its not on 1.20, it hasnt been updated yet, i dont know))
love you <3


[v4-beta.15] New ion-tabs doesn't work with router-outlet

Bug Report
Ionic Info
ionic info
✔ Gathering environment info - done!

Ionic:

   ionic (Ionic CLI)             : 4.3.0 (/usr/local/lib/node_modules/ionic)
   Ionic Framework               : @ionic/angular 4.0.0-beta.15-0
   @angular-devkit/build-angular : 0.9.0-beta.3
   @angular-devkit/schematics    : 0.9.0-beta.3
   @angular/cli                  : 6.2.6
   @ionic/angular-toolkit        : 1.1.0

Cordova:

   cordova (Cordova CLI) : 8.1.2<EMAIL_ADDRESS>   Cordova Platforms     : android 7.1.1, browser 5.0.4, ios 4.5.5
   Cordova Plugins       : cordova-plugin-ionic-keyboard 2.1.2, cordova-plugin-ionic-webview 2.0.3, (and 18 other plugins)

System:

   Android SDK Tools : 26.1.1 (/Users/heb/Library/Android/sdk)
   ios-deploy        : 1.9.4
   NodeJS            : v11.0.0 (/usr/local/Cellar/node/11.0.0/bin/node)
   npm               : 6.4.1
   OS                : macOS
   Xcode             : Xcode 10.1 Build version 10B61

Describe the Bug
New ion tabs/tabbar doesn't trigger ion-router-outlet when tab is clicked.
Steps to Reproduce
Steps to reproduce the behavior:

Create a new Ionic tab demo via ionic start myApp tabs --type=angular
change into myApp and update the used @ionic/angular version via npm i<EMAIL_ADDRESS>afterwards start the app via ionic serve, after a few seconds the homepage will open in your browser with no tabbar visible. This is due to the fact that the generated demo is not updated to the latest changes yet.
Next, open the file tabs.page.html, and replace the content with

<ion-tabs>
  <ion-tab-bar color="dark">
    <ion-tab-button tab=”home”>
      <ion-icon name="home"></ion-icon>
      <ion-label>Home</ion-label>
    </ion-tab-button>
    <ion-tab-button tab=”about”>
      <ion-icon name="information-circle"></ion-icon>
      <ion-label>About</ion-label>
    </ion-tab-button>
    <ion-tab-button tab=”contact”>
      <ion-icon name="contact"></ion-icon>
      <ion-label>Contact</ion-label>
    </ion-tab-button>
  </ion-tab-bar>

  <ion-tab tab="home">
    <ion-router-outlet name="home"></ion-router-outlet>
  </ion-tab>
  <ion-tab tab="about">
    <ion-router-outlet name="about"></ion-router-outlet>
  </ion-tab>
  <ion-tab tab="contact">
    <ion-router-outlet name="contact"></ion-router-outlet>
  </ion-tab>
</ion-tabs>

Reload the app in the browser
The tabbar is now visible and Home is selected, though clicking on About or Contact doesn't have any effect
In my main app (where I found this issue) the tab is triggered but still router-outlet is not loaded. But If I replace the ion-router-outlet with e.g. ion-content that content is proper loaded and displayed when the tab is clicked.

Expected Behavior
It looks like the migration steps mentioned in the beta.15 release notes are incomplete or not accurate enough. E.g. the old code has a href element for each tab (in addition to the router-outlet) but the release note don't really explain what happened to that href as it is gone in the new approach.

Wondering about this as well. Having persistent nav stacks when switching between tabs is one of the most important features of Ionic IMO. Understandably v4 is still in beta, but aside from this tab refactor, I haven't seen much mention of that missing functionality. At least with the tab refactor it fixes whatever bug was causing the app to lag when switching back to a tab that had a stack, but it still requires an href on the button, which pops back to the root page.

same issue here.
First tab is the only one that works.

https://github.com/ionic-team/starters/pull/526/files
working using that example.

Thats the right way to implement tab bar in new beta 15:
<ion-tabs>

  <ion-tab tab="home">
    <ion-router-outlet name="home"></ion-router-outlet>
  </ion-tab>
  <ion-tab tab="about">
    <ion-router-outlet name="about"></ion-router-outlet>
  </ion-tab>
  <ion-tab tab="contact">
    <ion-router-outlet name="contact"></ion-router-outlet>
  </ion-tab>

  <ion-tab-bar slot="bottom">

    <ion-tab-button tab="home" href="/tabs/(home:home)">
      <ion-icon name="home"></ion-icon>
      <ion-label>Home</ion-label>
    </ion-tab-button>

    <ion-tab-button tab="about" href="/tabs/(about:about)">
      <ion-icon name="information-circle"></ion-icon>
      <ion-label>About</ion-label>
    </ion-tab-button>

    <ion-tab-button tab="contact" href="/tabs/(contact:contact)">
      <ion-icon name="contacts"></ion-icon>
      <ion-label>Contact</ion-label>
    </ion-tab-button>

  </ion-tab-bar>

</ion-tabs>

@MrSparklle solution is indeed working. The missing part was the href on the button, which isn't mentioned in the migration release notes.
This issue can be closed.

Shouldn't be closed...
My tabs component (using login page first then -> goto tabs page) works in beta 13. Then i update it today and it doesn't work.
Then i follow the solution above (using the href) and the content of each page still missing. So, only tabs with no content.

@hebra just you that open the issue can close it.


dma fix and dma_async example

I was trying to put together an 'async' DMA example that basically continues looping without waiting for DMAs to finish but I ran into some issues.
As according to the comments for wait:
    /// [wait] will busily wait until the transfer is finished. If you
    /// don't want this, please call this function only once you know that the
    /// transfer has finished.

I assumed the only other Transfer function, is_active, was the intended way of checking this (@hannobraun). Unfortunately, it modifies the registers when returning false, meaning a non-blocking call to wait after checking active would get stuck in wait. That is to say, the following fails:
if !transfer.is_active() {
   // gets stuck because internal calls to `is_active` always returns true
   transfer.wait();
}

I've modified is_active to have an flag for clearing the registers, making usage within wait clear flags, but calls to Transfer::is_active will not clear flags.

Yes, is_active was intended to do exactly that. And it did at one point, but I must have broken it with later changes.
I haven't reviewed the example, but the changes to the DMA API look good to me.

CI appears to be failing for the L0x1, can we resolve this before merging?


Need option to auto size the width of Columns in the RESULTS of an mssql query

Type: Feature Request
Something akin to "resultsGrid.autoSizeColumns": true is needed.
If it already exists, make sure it is documented and findable via Help or the Command Pallette?
VS Code version: Code 1.77.1 (Universal) (b7886d7461186a5eac768481578c1d7ca80e2d21, 2023-04-04T23:23:33.779Z)
OS version: Darwin arm64 21.6.0
Modes:
Sandboxed: Yes

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.77.3. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

more Settings options to control the RESULTS output in general would be nice, e.g.

settings that could relocate the results window to the bottom output panel
autosize to the width of columns
set the width of columns for various data types
etc...

I'm pretty sure mssql querying is provided by an extension rather than being a core part of VS Code.

apologies - feel free to close this ticket if results display and associated settings would be controlled by the extension author.


[new]: json_query(json_string, query)

Check the idea has not already been suggested

[X] I could not find my idea in existing issues

Edit the title above with self-explanatory function name and argument names

[X] The function name and the argument names I entered in the title above seems self explanatory to me.

BigFunction Description as it would appear in the documentation
wrap https://github.com/jmespath/jmespath.js
Examples of (arguments, expected output) as they would appear in the documentation

json_query('{"foo": [{"first": "a", "last": "b"}, {"first": "c", "last": "d"}]}',  "foo[*].first")
--> [ 'a', 'c' ]

json_query(
    '{"foo": [{"age": 20}, {"age": 25}, {"age": 30}, {"age": 35}, {"age": 40}]}',
    "foo[?age > `30`]"
)

--> [ { age: 35 }, { age: 40 } ]

Congrats @sidaliSadi !


Attach external address to force closures

Background
Love using external wallet for my node; all my opens are from an external address, and all incoming and outgoing channel opens have a custom close address attached to them, thanks to bos intercepting even incoming channels.
What bothers me is the large elephant in the room: why no force closures tho?
Your environment

version of lnd 14.2
which operating system (uname -a on *Nix) latest raspios 64bit
version of btcd, bitcoind, or other backend 22
any other relevant environment details //

Steps to reproduce
Have a node. Try setting an external address to send funds to on force closure. Notice it is not possible.
Expected behaviour
Being able to have force closures send funds (or alternatively, swipe) to a specific external address; where the channel is already existing, give the user the ability to set the same coop close address, also for the force closure event.
Actual behaviour
Force closures send funds to hot wallet. Hot wallets are bad and should be avoided (or, as per my postgres sync/streaming replication guide PR discussion), NEVER use hot wallets! They can lead to funds loss!

This would be an extremely valuable improvement, external closing addresses for force closure is just as if not more important of a function as cooperative closure external address support.

Suggested User Story:
A less-trusted node operator managing funds for another wallet, could be the same entity but different software. One "cold storage" wallet that has on-chain funds, for example a Ledger. One "hot wallet" running LND, with the xpub from the ledger. All cooperative and force close on-chain operations close to an address generated by this external wallet xpub, ensuring no off-chain fees get stored on the node itself. Channel opens are Multi-sig, the node creates the transaction, and it requires the cold storage wallet to sign the transaction, to actually fund the channel open, again, using an address from the xpub on the node for the withdrawal address in all scenarios.
This can allow users who have a cold storage source of funds to operate a Lightning Node without necessarily trusting the software running it as much as is currently required, it also reduces the attack surface area to loss of funds due to a compromised node and malicious force closure and subsequent sweeping of the node's wallet, or extraction of the node's private key and draining of on-chain funds as channels close.
The Node software cannot liquidate the node into an untrusted flow, lowering the barrier to entry for adoption of Lightning by more conservative potential noderunners, or by reducing the anxiety of existing noderunners and allowing them to move more of their cold storage onto the Lightning Network with reduced potential counterparty risk.
I believe this scenario would greatly improve adoption of Lightning in general, and LND specifically as the go-to solution for building tools around generating passive income from user cold storage sources, and while still more #Reckless than a fixed cold storage, it represents far less risk than the current solution of keeping a live decrypted hot wallet on nodes that are typically not heavily secured.

What bothers me is the large elephant in the room: why no force closures tho?

This is possible, but if you want to leave it the output in the chain (non-breach case), the wallet then needs to understand how to do that special spend (witness script reveal, etc). With 0.15, the key/script import in the wallet is more advanced now, so it's more feasible.
In the protocol both sides specify two keys: one for HTLCs and one for the settled balance. The HTLCs you need to be able to timeout or settle them, but the settled balance can be "colder". So we just need to make that hook available, similar to how ppl can specify the upfront shutdown address. From there we'd update the wallet to understand that new output type so it can generate a valid witness.

The above flew way, way above my head, but what I understoof from this is that the feature can indeed be added. This is so cool

but what I understoof from this is that the feature can indeed be added.

Indeed...we have the technology!


setAnchorOffset is not there

There is no way to set the anchor point anymore?

I just realized also, I might have killed the method when porting it into a library. I'll fix it. Thanks

Thanks 🙌

I want num of pixels instead of float threshold for offset argument :)

I just port them back from previous commits, sorry for the delay


How does one set this up for use in an IDE such as Eclipse?

This is a newbie question, I know, but as far as setting this up so that it can be used and compiled into a .jar file, I'm completely lost. Would like to learn how to do this so I can actually start work on using this.

Watch his video about making a better client thatn yay in an hour. Might help you!

This is a newbie question, I know, but as far as setting this up so that it can be used and compiled into a .jar file, I'm completely lost. Would like to learn how to do this so I can actually start work on using this.

still need help? JulianCode69#1337

@JulianCode1 xD


The response.text quick accessor only works when the response contains a valid Part, but none was returned. Check the candidate.safety_ratings to see if the response was blocked.

Description of the bug:
The response.text quick accessor only works when the response contains a valid Part, but none was returned. Check the candidate.safety_ratings to see if the response was blocked.
Actual vs expected behavior:
Traceback (most recent call last):
File "D:\Study\codes\Gemini_score.py", line 64, in
strr = str(id) + ":" + response.text
File "D:\Study\codes\venv\lib\site-packages\google\generativeai\types\generation_types.py", line 401, in text
raise ValueError(
ValueError: The response.text quick accessor only works when the response contains a valid Part, but none was returned. Check the candidate.safety_ratings to see if the response was blocked.
Any other information you'd like to share?
THIS IS MY CODES:
import pandas as pd
import pdb
import google.generativeai as genai
import os
genai.configure(api_key="xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx")
model = genai.GenerativeModel('gemini-1.5-pro')
response = model.generate_content(system_prompt)
strr = str(id) + ":" + response.text
system_prompt is a string

@141forever, Similar feature request #282 is already filled. Requesting you to please follow and +1 the similar issue for updates and close this thread. Thank you!

@singhniraj08 I'm facing similar as the OP, since the finish_reason is always OTHER and the issue you linked is talking about safety I'm not sure it is related problem.
My prompt is as simple as translating a JSON object with the content "Hi Peter, how's it going?" along with related metadata like random bson ObjectId. Using the same prompt, I can easily get the result from ChatGPT, but Gemini (both Pro and Flash) just responds with finish_reason=OTHER so I don't even know what the problem is with the prompt.

Additional information:
With the same prompt I always get the expected result via the public gemini website https://gemini.google.com
With https://aistudio.google.com/app/prompts/new_chat , it is failed every time.
I can share the prompt via email if you guys are interested

@trunglebka, #282 is for implementation of more helpful error messages when the response is blocked by Gemini because of safety or other reasons. The SDK doesn't control the service's responses. If you feel the response is blocked which shouldn't be the normal case, we would suggest you to use "Send Feedback" option in Gemini docs. Ref: Screenshot below. You can also post this issue on Discourse forum.

@singhniraj08 Thanks for pointing that out, I wasn't paying close attention

Im having the same issue, Im doing some rewriting text about healthcare USML dataset. And in a specific question I got this error.

@baptvit @141forever
set the safety_settings and generate the content.
https://ai.google.dev/gemini-api/docs/safety-settings?hl=en

@architectyou yeah, Ive tried to set the safety_settings
response = GEMINI_MODEL.generate_content(prompt, safety_settings=[
    {"category": "HARM_CATEGORY_DANGEROUS_CONTENT", "threshold": "BLOCK_NONE"},
    {"category": "HARM_CATEGORY_SEXUALLY_EXPLICIT", "threshold": "BLOCK_NONE"},
    {"category": "HARM_CATEGORY_HATE_SPEECH", "threshold": "BLOCK_NONE"},
    {"category": "HARM_CATEGORY_HARASSMENT", "threshold": "BLOCK_NONE"},
    {"category": "OTHER", "threshold": "BLOCK_NONE"}
])
response.text

But even though I set up, I had to catch the exception:
def rewrite_sentence(sentence_original, model=GEMINI_MODEL, template_prompt=TEMPLATE_PROMPT):
    try:
        prompt= template_prompt.replace("<original_sentence>", sentence_original)
        response = model.generate_content(prompt, safety_settings=[
    {"category": "HARM_CATEGORY_DANGEROUS_CONTENT", "threshold": "BLOCK_NONE"},
    {"category": "HARM_CATEGORY_SEXUALLY_EXPLICIT", "threshold": "BLOCK_NONE"},
    {"category": "HARM_CATEGORY_HATE_SPEECH", "threshold": "BLOCK_NONE"},
    {"category": "HARM_CATEGORY_HARASSMENT", "threshold": "BLOCK_NONE"}
])
        return response.text
    except Exception as e:
        return "Error text"

Same error here, the safety settings have no effect on this particular issue

Facing same issue for reviewing code (which is supposed to be written badly): The response.text quick accessor only works when the response contains a valid Part, but none was returned. Check the candidate.safety_ratings to see if the response was blocked.

Facing same issue for reviewing code (which is supposed to be written badly): The response.text quick accessor only works when the response contains a valid Part, but none was returned. Check the candidate.safety_ratings to see if the response was blocked.

Facing same issue as well

Facing the same issue here

I'm still seeing the issue. Does google even care?

You need to pass Safety Settings. Here is how I did in one of my open source repo: https://github.com/ajitesh123/auto-review-ai/blob/main/backend/llm.py#L177-L198
(Please star my repo if this solves for you :))

You need to pass Safety Settings. Here is how I did in one of my open source repo: https://github.com/ajitesh123/auto-review-ai/blob/main/backend/llm.py#L177-L198
(Please star my repo if this solves for you :))

Sadly changing the  safety sessions has no effect on this issue

Same error here too

@phil-scholarcy @OGsiji I can look into this issue for you. Can you please post a minimal working code sample here that caused this error for you? I tried OP's original code sample and I am unable to hit that error.


Update to pysam version 0.8.4

Our version of pysam has been pinned at 0.8.3 for a while now, and using 0.8.4 causes problems. We should update to 0.8.4.

Fixed in #809.


Add python 3.12 to starters builds

Motivation and Context

Noticed that 3.12 builds were missing!
How has this been tested?

Checklist

[ ] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Assigned myself to the PR
[ ] Added tests to cover my changes

distutils removed in Python 3.12 🤔

distutils removed in Python 3.12 🤔

Yeah, https://docs.python.org/3.12/whatsnew/3.12.html#distutils

They've fixed it on pyspark but it'll only be out in 4.0.0 https://issues.apache.org/jira/browse/SPARK-45390
A solution I saw was installing setuptools 🤔

Adding setuptools to the requirements.txt for spaceflights-pandas-viz for python 3.12 does solve the issue but the tools.feature test will fail still because it pulls the last released version of starters.
Here's what we can do -

Separate PR for adding setuptools to the requirements.txt will only run tests till 3.11 so can be merged
Delete and re-tag 0.19.5 on starters
Then add "3.12" to the builds

What do you think @ElenaKhaustova @astrojuanlu @noklam?
Or we could wait till pyspark 4.0.0 comes out.
We have already added support for python 3.12 in the last release on Kedro, this should technically have been a part of 0.19.4

Is this a build or project requirements? Usually setuptools is a build requirements and we tried very hard in the past to remove it from kedro library depedencies (it's big and slow, used for micropkg)

@noklam just adding it to the requirements.txt for the spaceflights-pyspark-viz starter (only if python_version>3.11)
https://stackoverflow.com/questions/77233855/why-did-i-got-an-error-modulenotfounderror-no-module-named-distutils
Wouldn't need it after pyspark releases 4.0.0

After reading the JIRA issue, I think it's okay to included it in requirements.txt conditionally for >=3.12. It's a shame PySpark didn't support this. We should leave a comment in the requirements too.

All done! ✅


How to display countries in GoAccess?

Currently I am able to view info about the country at "VISITOR HOSTNAMES AND IPS" table only.
Is there a configuration already to add a new table i.e "COUNTRIES" like in "GEO LOCATION" table but with countries?
Using:

Vagrant machine
./configure  --enable-geoip=mmdb
with GeoLite2-Country.mmdb
exporting to html.

BTW great tool!

Currently the Geolocation panel should list countries that visited your server. The demo displays this panel at the bottom of the page. Is that what you are looking for?

I guess you are referring to "GEO LOCATION" table? If so, then no that's not what I want. Almost. I want the same but country specific. I am looking for something that will list the countries instead of continents. For example (in a poor copy paste format from html):
1	705,129 (58.18%)	101,762 (61.69%)	9.26 GiB (68.79%)	415.51 ms	3.39 d	1.23 h	UK
2	410,542 (33.87%)	33,606 (20.37%)	3.22 GiB (23.90%)	557.34 ms	2.65 d	5.05 m	Germany
3	51,287 (4.23%)	16,161 (9.80%)	361.81 MiB (2.63%)	507.70 ms	7.23 h	5.00 m	Greece
4	21,476 (1.77%)	8,792 (5.33%)	463.83 MiB (3.37%)	139.37 ms	49.89 m	1.65 m	Japan
Is there a configuration to do that? Or am I missing something already implemented?

Never mind. Found it. The continent is clickable.. Nice!

That's right. Pretty much any panel with the little caret ▶ on the first column is expandable.

BTW I am using Vimium. This should generate a button on any link or clickable item. That's why I failed to find the carets at first.

Got me confused as well. So let me add, though in nCurses there's no "click", same applies here – let's say it's "enter-able" :wink:


Behave like relaton-iso for undated citations

Right now a query for JIS B 0001 is returning the entry for JIS B 0001:2019. relaton-jis needs to behave like relaton-iso: if the citation is undated, it needs to return the citation without the year, and with the year-specific record added on as an instance.

Is blocking https://github.com/metanorma/metanorma-jis/issues/23

Implemented in v 1.14.2
https://github.com/relaton/relaton-jis/blob/main/spec/fixtures/jis_x_0208.xml


Option to disable vibration

Please give an option to disable the vibration. It is a battery killer and after some games really annoying :)
Thanks for the game!
Best,
Til

I couldn't +1 this enough. Every time I hit a tile my phone vibrates and it feels like I'm being punished for hitting the tiles. I can understand some people may like it, but for me, it's just annoying (also the reason I don't play the Flappy Droid Android 5 easter egg, which does the same) :(

I considered that at some point, but when I found out that disabling the vibration at my phone's level did the same, I lost interest in making this option available... Since that's a cheap workaround and the option is quite simple to add, I'll add it soon ;)

Done !
New release here : https://github.com/316k/android-tap-the-black-tiles/releases/tag/v1.1
It should also be updated on F-Droid eventually.


gotop: 3.5.2 -> 3.5.3

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/xxxserxxx/gotop/releases.
meta.description for gotop is: "A terminal based graphical activity monitor inspired by gtop and vtop"
meta.homepage for gotop is: "https://github.com/xxxserxxx/gotop"
meta.changelog for gotop is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 3.5.3 with grep in /nix/store/1wrg69lgasb5m61507p696q78bl4f5q4-gotop-3.5.3

directory tree listing: https://gist.github.com/b44a60a65f37d51c34858fb91125f94d

du listing: https://gist.github.com/4c54e5f7ba7aea89a42b577533b265bb

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

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
gotop

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/1wrg69lgasb5m61507p696q78bl4f5q4-gotop-3.5.3 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's 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 gotop https://github.com/r-ryantm/nixpkgs/archive/347320954a7e7c508b86d9f2e355919eaa52f9c9.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/1wrg69lgasb5m61507p696q78bl4f5q4-gotop-3.5.3
ls -la /nix/store/1wrg69lgasb5m61507p696q78bl4f5q4-gotop-3.5.3/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 1

1 package built:
- gotop

Maintainer pings
cc @magnetophon @kalbasit for testing.

Result of nixpkgs-review pr 89834 1

1 package built:
- gotop


✨ When file is opened or focused, jump to it in project navigator

Is your feature request related to a problem? Please describe.
When a tab is clicked, or a file is opened via the quick open overlay, jump to that file in the project navigator, even if it is layers deep in a nested file structure.

related #123

Partially fixed in #123.
If the folder that the file the user has opened is in a directory that is collapsed in the project navigator, we need to recursively open directories until we can scroll to it in one motion (not opening one folder after the other).

Has this been @lukepistrol ?

What is the status of this?

[ ] ###

https://github.com/CodeEditApp/CodeEdit/issues/184#issuecomment-1079380600


Set WORDS_BIGENDIAN if needed

The irtkCifstream file contains the following code (may also other I/O classes):
#ifdef WORDS_BIGENDIAN
  _Swapped = false;
#endif

I an guessing via CMake's TEST_BIG_ENDIAN


3.8.0 - error on ?? (nullish coalescing operator)

🐛 Bug Report
Version 3.8.0 just got released, including a nullish coalescing operator (??). This breaks in certain environment, like Jest.
  ● Test suite failed to run

    Jest encountered an unexpected token
    This usually means that you are trying to import a file which Jest cannot parse, e.g. it's not plain JavaScript.

    [...]

    Details:

    /github/home/Library/Caches/Bit/capsules/44d80e1c05a5191ec871aec76c25c66afd3b0734/node_modules/.pnpm/registry.npmjs.org+@react-aria+interactions@3.8.0_react@17.0.2/node_modules/@react-aria/interactions/dist/main.js:558
        ref = ref ?? newRef;
                   ^

    SyntaxError: Unexpected token '?'

      1 | import React, { ReactNode, useRef } from 'react';
    > 2 | import { useButton } from '@react-aria/button';
        | ^
      3 | import { Link } from<EMAIL_ADDRESS>

🤔 Expected Behavior
code should have been compiled down to:
ref = (_ref = ref) !== null && _ref !== void 0 ? _ref : newRef;

Usually, typescript handles that.
😯 Current Behavior
Code remains with the original ?? operator:
const $103a536ab6395e4d$export$27c701ed9e449e99 = /*#__PURE__*/ ($parcel$interopDefault($hZjuk$react)).forwardRef(({ children: children , ...props }, ref)=>{
    let newRef = $hZjuk$react.useRef();
    ref = ref ?? newRef; // <---------------------
    let { pressProps: pressProps  } = $213abf3f34aca140$export$45712eceda6fad21({
        ...props,
        ref: ref
    });
    let child = ($parcel$interopDefault($hZjuk$react)).Children.only(children);
    return(/*#__PURE__*/ ($parcel$interopDefault($hZjuk$react)).cloneElement(child, // @ts-ignore
    {
        ref: ref,
        ...$hZjuk$reactariautils.mergeProps(child.props, pressProps)
    }));
});

💁 Possible Solution

I solved it with a temporary solution using the transformIgnorePatterns option in Jest.
https://jestjs.io/docs/configuration#transformignorepatterns-arraystring
transformIgnorePatterns: ["node_modules/(?!(@react-aria|.pnpm/.*[+/]@react-aria.*|.pnpm/@react-aria.*)/)"]

I also had problems with nullish coalescence and optional chaining in storybook.

Same problem here with @internationalized/number version 3.0.4 and Webpack (which I can't update atm). Version 3.0.3 worked fine; version 3.0.4 has optional chaining in dist/module.js. I worked around that by pinning the version for now.

Same problem here with @internationalized/number version 3.0.4 and Webpack (which I can't update atm). Version 3.0.3 worked fine; version 3.0.4 has optional chaining in dist/module.js. I worked around that by pinning the version for now.

when any solution - please share
have exactly the same(

Nullish coalescing is supported natively in all browsers we support, representing over 92% of users globally. If you are using an old version of a build tool without support, I would recommend upgrading that. https://caniuse.com/mdn-javascript_operators_nullish_coalescing

Having the same problem with multiple files, using Webpack 4.
On what I understand by reading on optional-chaining with Webpack, we need to upgrade to Webpack 5 to support new versions of react-aria. https://stackoverflow.com/questions/59972341/how-to-make-webpack-accept-optional-chaining-without-babel
Solution : Webpack 5
Webpack rely on Acorn as JavaScript parser. Acorn is supporting optional chaining from version 7.1.0 (2020-06-11) https://github.com/acornjs/acorn/blob/master/acorn-loose/CHANGELOG.md#710-2020-06-11
Webpack
Accoring to https://webpack.js.org/blog/2020-10-10-webpack-5-release/ Webpack updated Acorn from 5 to 8 on Webpack 5.0.0
Solution : Babel - @babel/plugin-proposal-optional-chaining
I didn't played a lot with Babel parsing node_modules, but maybe there is a way to load react-aria with babel and @babel/plugin-proposal-optional-chaining.
ERROR in<EMAIL_ADDRESS>95:24
Module parse failed: Unexpected token (95:24)
File was processed with these loaders:
 * ./node_modules/babel-loader/lib/index.js
You may need an additional loader to handle the result of these loaders.
|   let scopeRef = $bmn4K$useRef([]);
|   let ctx = $bmn4K$useContext($0238e4b796d1715f$var$FocusContext);
>   let parentScope = ctx?.scopeRef;
|   $bmn4K$useLayoutEffect(() => {
|     // Find all rendered nodes between the sentinels and add them to the scope.
<EMAIL_ADDRESS>2:0-129 45:6-25 130:17-46
 @ ./node_modules/react-aria/dist/module.js

ERROR in<EMAIL_ADDRESS>90:37
Module parse failed: Unexpected token (90:37)
File was processed with these loaders:
 * ./node_modules/babel-loader/lib/index.js
You may need an additional loader to handle the result of these loaders.
|   } = props;
|   let data = $d786742a15de2e64$export$3585ede4d035bf14.get(state);
>   let isDisabled = props.isDisabled ?? state.disabledKeys.has(key);
|   let isSelected = props.isSelected ?? state.selectionManager.isSelected(key);
|   let isFocused = state.selectionManager.focusedKey === key;
 @ ./node_modules/react-aria/dist/module.js 22:0-79 22:0-79 22:0-79 22:0-79

ERROR in<EMAIL_ADDRESS>847:98
Module parse failed: Unexpected token (847:98)
File was processed with these loaders:
 * ./node_modules/babel-loader/lib/index.js
You may need an additional loader to handle the result of these loaders.
|         let eraIndex = eras.indexOf(value.era);
|         if (eraIndex < 0) throw new Error('Invalid era: ' + value.era);
>         eraIndex = $6384333fb108a063$var$cycleValue(eraIndex, amount, 0, eras.length - 1, options?.round);
|         mutable.era = eras[eraIndex]; // Constrain the year and other fields within the era, so the era doesn't change when we balance below.
|
<EMAIL_ADDRESS>4:0-80 146:34-54
 @ ./node_modules/react-aria/dist/module.js

Ah, looks like the latest version of storybook is still on webpack 4. Unbelievable!
Not sure about jest. The latest version worked for me.

Apparently webpack 5 is opt in with storybook. Has anyone tried this? https://storybook.js.org/blog/storybook-for-webpack-5/

we have the same issue

We faced the issue starting from last night on CI without any code change.
Currently trying to update to webpack5 in our storybook based component library.

If you're using yarn, you may also reduce the number of things you need to update using resolutions.
https://github.com/adobe/react-spectrum/issues/2855#issuecomment-1040946387
"resolutions": {
    "acorn": "8.0.1"
},

It's not just about nullish coalescing or optional chaining, but private class methods as well.
ERROR in<EMAIL_ADDRESS>1120:4
Module parse failed: Unexpected character '#' (1120:4)
You may need an appropriate loader to handle this file type, currently no loaders are configured to proc
s this file. See https://webpack.js.org/concepts#loaders
|     // i.e. a ZonedDateTime should not be be passable to a parameter that expects CalendarDate.
|     // If that behavior is desired, use the AnyCalendarDate interface instead.
>     #type;
|     constructor(...args){
|         let [calendar, era, year, month, day] = $de531a00a7b9094c$var$shiftArgs(args);

Browser support is documented here: https://react-spectrum.adobe.com/react-aria/interactions.html#supported-browsers-and-platforms
Private class properties are also pretty widely supported, but it looks like they should be compiled (due to safari). Looking into it. https://caniuse.com/mdn-javascript_classes_private_class_fields
However, webpack being a lower support target than browsers is a new one we hadn't considered. I would definitely recommend upgrading if you can, but we are discussing what to do about it now and will have an update soon. Sorry for the inconvenience and thanks for your patience.

Can confirm switching to webpack5 builder in Storybook fixed the issue.
Latest migration docs for Storybook + Webpack5

We have temporarily reverted our browserslist config to a lower browser target such that optional chaining and nullish coalescing are now transpiled in our published packages. That should keep things working for the time being.
However, we would like to update this again in the future so that we can avoid transpiling syntax that is supported by our browser targets in order to reduce bundle sizes and improve runtime performance. Please upgrade to webpack 5 (or parcel? 😉😉) and/or the latest version of jest as soon as you can. We plan to re-introduce this change in July 2022. Given that webpack 5 has already been available for over a year (released in October 2020), we hope this gives everyone ample time to upgrade their dev dependencies.
Sorry again for the unexpected inconvenience.

I recommend you keep an older browser support for longer. The ?? operator is available for only about half a year, I usually keep browser support for at least 3 if not 5 years. I can't really tell users to update their browsers, and sometimes even upgrading Webpack is not possible.
As an accessibility tool, I'd expect it not to leave out 8% of the world population, more than the population of screen reader users 😉

According to caniuse, most browsers released nullish coalescing support around Feb-March 2020. That's just about 2 full years ago, if you can believe it!
Most browsers are evergreen these days, so if you specifically choose not to update, you're making some kind of statement. I'm not sure waiting much longer would make a difference to that number. At some point, things are gonna start breaking for them.

Most browsers are evergreen these days, so if you specifically choose not to update, you're making some kind of statement.

Yes and no 😛  Some people are just locked on the browser version, because of their OS/hardware: https://blog.jim-nielsen.com/2022/a-web-for-all/.
On a side note: to be honest I'm not really sure if this should be a library author's concern 🤔

Yeah exactly. If you choose you can compile your node_modules according to your browser targets. This was more of an unfortunate case of build tools themselves being outdated more than browsers.


Field display name

We should have the ability to configure the validator so that it can read field names off of tags such as "json" and use that when displaying the error.
Suggestion
defaultValidator := validator.Default()

defaultValidator.DisplayNameTag = "json"

type Foo struct {
    FirstName string `validator:"min(1),max(20)" json:"first_name"`
}

if errs := validator.Validate(Foo{}); errs.Any() {
    errs.PrintAll() // first_name cannot be shorter than 1 characters.
}

Feature implemented. Closing issue.


Adding local server recipe

Fix #18
@darkwing, this is mainly based on API analytics demo. Do you want to take a look at this implementation?

I'm going to merge to have this available for the talk tomorrow.


False positive for “RET504: Unnecessary variable assignment before return statement”

Code similar to the following will return a false positive for RET504, as the variable assignment here is in fact not unnecessary or optional:
def foobar() -> str:
    result = get_foobar_from_db()
    print(f"Got {len(result)} number of results from db.")
    return result

Good catch.


Add link to options step in wizards

Added link for image types to options step in both create and update wizard. Ticket #694

Codecov Report

Merging #152 (2c84637) into master (5baf8c7) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master     #152   +/-   ##
=======================================
  Coverage   25.49%   25.49%
=======================================
  Files          78       78
  Lines        1369     1369
  Branches      353      353
=======================================
  Hits          349      349
  Misses        942      942
  Partials       78       78

Impacted Files
Coverage Δ

src/components/form/ImageOutputCheckbox.js
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


be able to select the precise hours to take the medications instead of day periods.

That's in my todos but I'd like to keep the first iteration (https://github.com/waseefakhtar/dose-android/milestone/1) simple for now w.r.t UX and development.


Using npmregistries for private packages, is failing to load package. Because of CORS error.

Bug report
Packages affected

[ ] sandpack-client
[Y] sandpack-react

Using npmregistries for private registry is failing to load package, because of CORS error.
Used npmregistry for loading packages from custom registry
npmRegistries: [
{
proxyEnabled: true,
enabledScopes: ['@scope'],
limitToScopes: true,
registryUrl: "custom registry url",
registryAuthToken:"token"
},
],
Error info: 'https://1-4-0-sandpack.codesandbox.io' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: The value of the 'Access-Control-Allow-Origin' header in the response must not be the wildcard '*' when the request's credentials mode is 'include'.
Link to sandbox: link (optional)
Your Environment

Software
Name/Version

Sandpack-client version

Sandpack-react version
1.5

Browser
chrome

Operating System
windows10

Hey @appanasantosh, thanks for reporting this issue. However, given the amount of details in this description, I can't really debug and understand what's happening. Please, instead of mentioning a piece of documentation, give a real snippet of code (hide sensitive information) and add more information, such as: a network screenshot, sandpack error, or anything else that make it easier to understand the problem.
Feel free to reach out to me on Discord, and then we can make a session to debug it.

Hi @danilowoz here is the screenshot of the error when trying to import the private registry package.

In the code I have mentioned the details about the registry in prop
npmRegistries: [
{
proxyEnabled: true,
enabledScopes: ['https://github.com/scope'],
limitToScopes: true,
registryUrl: "custom registry url",
registryAuthToken:"token"
},
]

Most registries don't add the cors headers, so unless you have a proxy like verdaccio in place or have added cors headers to your registry in some other way, it will return this error.
Don't think there's a lot we can do here unless you share more details about the registry itself as the issue is with the registry not sandpack.

Actually the registries are from private azure artifacts space.

@appanasantosh, the configuration should look something like this:
npmRegistries: {
  enabledScopes: ["@codesandbox"],
  registryUrl: "//my-registry.domain.com",
  limitToScopes: true,
  registryAuthToken: "SECRET" // optional value, if public
}

proxyEnabled must be false and the enableScopes must contain the package scope instead of an URL.

I have used the same configuration before, but the same exception was thrown as mentioned in above comments.

Hey @appanasantosh, indeed, there was a CORS issue on the latest bundler version. It has been fixed on the latest Sandpack version and https://github.com/codesandbox/sandpack/pull/554

@danilowoz I have installed latest version(1.5.2) and tested it. Still I am having the same issue.

Can you try again with 1.5.3? It should work now

Hey, it should be fixed on v1.5.4, thanks again for reporting it 👍

Thanks @danilowoz. CORS issue is fixed, it is working fine now.


Improve multi-tier block management

There are 2 main changes in this PR:

Eliminate cascading eviction by allowing writes to go anywhere in the worker when there is no space on target.
Alleviate performance problems of the first, by employing an async task for moving blocks around based on access pattern.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8451/
Test FAILed.

Jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8452/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8541/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8562/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8566/
Test FAILed.

Jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8632/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8672/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8680/
Test FAILed.

jenkins, test this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8681/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8685/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8686/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8687/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8688/
Test PASSed.

@calvinjia @bf8086
I stopped growing this PR. PTAL.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8705/
Test FAILed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8717/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8742/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8743/
Test PASSed.


document contribution to window topic

Hi:
In the paper, it says: "3) A ranking of every MEPs contributions relative to all window and dynamic topics in the corpus"
Is there a function that out put this result? Or we need to calculate it by self.

This refers to MEP (author)-document counts for each window and dynamic topic. Since the MEP metadata was very specific to our application and database, the function is not included here.
Regards,
Derek.