Update tests for B extension

Description

Add B extension to ISA strings
B == Zba + Zbb + Zbs

Related Issues

N/A

Ratified/Unratified Extensions

[ ] Ratified
[X] Unratified

List Extensions

https://github.com/riscv/riscv-b

Reference Model Used

[X] SAIL
[X] Spike
[ ] Other - < SPECIFY HERE >

Mandatory Checklist:

[X] All tests are compliant with the test-format spec present in this repo ?
[X] Ran the new tests on RISCOF with SAIL/Spike as reference model successfully ?
[] Ran the new tests on RISCOF in coverage mode
[ ] Link to Google-Drive folder containing the new coverage reports (See this for more info): < SPECIFY HERE >
[ ] Link to PR in RISCV-ISAC from which the reports were generated : < SPECIFY HERE >
[X] Changelog entry created with a minor patch

Optional Checklist:

[X] RISCV-V CTG PR link if tests were generated using it : https://github.com/riscv-software-src/riscv-ctg/pull/100
[ ] Were the tests hand-written/modified ?
[ ] Have you run these on any hard DUT model ? Please specify name and provide link if possible in the description
[ ] If you have modified arch_test.h Please provide a detailed description of the changes in the Description section above.

For reasons Allen and I don't entirely understand, some riscof installations require XLEN in the RVTEST_CASE macro to select the correct set of tests to build.  Some tests such as the I suite already have this, but I've been having to add the XLEN check to many of the newer test suites.  Would it be possible to modify your script to generate tests that include the XLEN check?
For example, in rv32i_m/B/src/andn-01.S
please change
RVTEST_CASE(0,"//check ISA:=regex(.*I.B.);def TEST_CASE_1=True;",andn)
to
RVTEST_CASE(0,"//**check ISA:=regex(.32.);**check ISA:=regex(.*I.B.);def TEST_CASE_1=True;",andn)

The RVTEST_CASE macro is part of the template that is generated by the CTG
(compatibility Test Generator) program
(when tests aren't hand generated, as many priv and non-ISA oriented tests
seem to be moving to)
So, this change sounds like it could be part of either RISCOF, but more
likely part of CTG, not part of arch-test;
I would file this issue in that repo.
I'm having the template part of the CTG code updated to be much much
simpler (and moving the complexity to existing arch-test macros) and all
the tests re-generated, and this could be part of it.
On Mon, Jan 29, 2024 at 5:05 AM David Harris @.***>
wrote:

For reasons Allen and I don't entirely understand, some riscof
installations require XLEN in the RVTEST_CASE macro to select the correct
set of tests to build. Some tests such as the I suite already have this,
but I've been having to add the XLEN check to many of the newer test
suites. Would it be possible to modify your script to generate tests that
include the XLEN check?
For example, in rv32i_m/B/src/andn-01.S
please change
RVTEST_CASE(0,"//check ISA:=regex(.*I.B.);def TEST_CASE_1=True;",andn)
to
RVTEST_CASE(0,"//**check ISA:=regex(.32.);**check ISA:=regex(.*I.B.);def
TEST_CASE_1=True;",andn)
—
Reply to this email directly, view it on GitHub
https://github.com/riscv-non-isa/riscv-arch-test/pull/427#issuecomment-1914657675,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHPXVJRCBOL3M7TWK5KBKZTYQ6NBHAVCNFSM6AAAAABB2KKHFCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJUGY2TONRXGU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

This gets generated from the string in the cgf -       "check ISA:=regex(.*I.B.)".  Fixing this will need the cgf file to be updated to  add the xlen filter and then regenerate the files.  My PR to add B just got merged  into dev branch of CTG so  I cannot update that PR unfortunately. A  new PR would be  needed.

require XLEN in the RVTEST_CASE macro to select the correct set of tests to build

I have noticed it will  generate a  warning of by selecting more tests into the test data base but nothing bad besides that. I am not sure why we cannot just fix riscof  to look at both the test ISA and the ISA string for doing the test selection.
But seems like Allen has a  better fix in mind.

Thank you.  Let's defer to Allen's recommendation.

I guess I don't understand why earlier tests were generated "correctly," and these new ones weren't.
That's the first step - I don't want this to happen again.

David: is this actually an issue? The report says he tested it with Spike vs. Sail, and it passed.
IS this necessary (in this case) or just nice to have?
Presumably the tests won't be run if the RVTEST_ISA string doesn't match

As far as I can tell, the changes to this test were only in those boilerplate macros; the tests themselves haven't changed.
It's really hard to review this when all the diffs have a "Large diffs are not rendered by default" message.
The shouldn't be for a change like this.
The right way to do this (IMHO, of course) is to use sed to just replace/add the change comments and RVTEST_ISA/RVTEST_CASE  macros.

I think we need a "best practices" document...
If this is actually a functional problem, then let's fix it by doing what I suggest above, either by taking the current tests in the repo and running sed of them, replacing this PR, or by a new commit using sed on this PR.


Mongo DB Upgrade to 5.0 having compatibitlity issue with Apache Beam included mongo-java-driver.

I am from the Jaguar Landrover (JLR UK) team and we are using Apach beam Dataflow Pipeline to ingest JLR vehicle data into Mongo DB from more than a Year. On production we have upgraded our Mongo DB from 4.4 to 5.0. And We have Apach Beam Pipeline running with version 2.34 on GCP.  And after upgrading we are experinceing connection issue.
And as per Mongo DB Documentation says this Apache Beam dependecy mongo-java-driver is not compatible with Mongo DB 5.0.
https://www.mongodb.com/docs/drivers/java/sync/current/compatibility/#compatibility
!image-2022-05-05-17-16-23-236.png!
Atelast we should have Mongo Java Driver 4.4 or higher with Mongo DB 5.0
And inside the Apach Beam version i can see Mongo Java Driver version is till 3.12.x so Please provide your valudable help in this issue. Our Application is on JLR Production and it's causing critical issues.

Please have a look at this on urgent basis as it's blocking on production.

Imported from Jira BEAM-14420. Original Jira may contain additional context.
Reported by: dneema.

Beam 2.38 has bumped driver version to 3.12.10 which should support MongoDB 5.0 server: https://github.com/mongodb/mongo-java-driver/blob/d5e75f4a72976c70a2371cad82c5769abbe6e124/driver-core/src/main/com/mongodb/connection/ServerDescription.java#L64


Create c-doubly_linked_list.c

Hey! I have added a code for doubly linked list in C language.
Thanks for the chance.

same implementation already exists in the repo .
Thanks for contributing.


Memory pool

Memory pool implementation. Use case is to implement a logger that utilizes a message pool. Source pulled from old logger branch and updated here in this PR: https://github.com/awslabs/aws-c-common/tree/logger
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Where you able to maintain API compatibility with the pool in c-io? That would save some time.

we should pull the memory pool out of aws-c-io, to use this.


Create bbzsogr.txt

Looks like this should be http://bbzsogr.so.ch/ ?

No, www.bbzsogr.ch is redirected to bbzsogr.so.ch, but the official domain is called bbzsogr.ch !!

No, www.bbzsogr.chhttp://www.bbzsogr.ch is redirected to bbzsogr.so.ch, but the official domain is called bbzsogr.ch !!
The email adresses look like<EMAIL_ADDRESS>Von: Tom MacWright<EMAIL_ADDRESS>Gesendet: Freitag, 20. März 2015 15:19
An: leereilly/swot
Cc: Mosimann Daniel
Betreff: Re: [swot] Create bbzsogr.txt (#638)
Looks like this should be http://bbzsogr.so.ch/ ?
—
Reply to this email directly or view it on GitHubhttps://github.com/leereilly/swot/pull/638#issuecomment-84029515.

@tmcw It appears @dmosimann is right. bbzsogr.so.ch doesn't have an MX record :/


Add a User-Agent header to the cloudflare pack to make it work again

Cloudflare
Status

bugfix
complete

Description
Add a User-Agent header to over come changes to the Cloudflare API.
Checklist (tick everything that applies)

[X] Metadata: pack.yaml, icon, structure (required for new packs)
[X] Version bump (required for changed packs)
[X] Code linting (required, can be done after the PR checks)
[X] Tests (not required but really recommended)

LGTM, thanks!

Thanks @Kami, all tests passed :-D


handle CheckMetadataHash extension

related: https://github.com/polkadot-fellows/runtimes/pull/337
will be affected on Jul 1st
The following will be added to  SignerPayloadJSON:
/**  The metadataHash for the CheckMetadataHash SignedExtension */
metadataHash?: HexString
/** The mode for the CheckMetadataHash SignedExtension, i*/
mode?: number
and assetId

needs "@polkadot/api" update

https://hackmd.io/Z0t8kSkxR1W1oqtgvDLVXQ


[Tooltip]: ChangeDetection is triggered to often

What is the expected behavior?
Change detection runs once when component is shown or become hidden
What is the current behavior?
When move mouse over button ChangeDetection is run a lot of times in short period of time
What are the steps to reproduce?
Providing a StackBlitz reproduction is the best way to share your issue.
StackBlitz starter: https://stackblitz.com/edit/angular-2g1mvi
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
"@angular/material": "7.3.7",

@dmkorol can you provide more information? How many times are you seeing change detection run? On which application were you measuring? How were you measuring?

can you provide more information?
How many times are you seeing change detection run?
There is online reproduction(link above): https://stackblitz.com/edit/angular-2g1mvi
Please open the console to see results during moving on the button(it reproduce on all browsers)
For example, got 19 execution when I moved the cursor on the button

How were you measuring?
I add a function on the view {{val()}} that write to console each time when it is run (when change detection is triggered on the view)

On which application were you measuring?
In the real application, there is a lot of places where the tooltips were put, most of the component is a dummy and don't use change any custom change detection strategies. In some cases, there are a lot of information and tables.
Desired behavior that tooltip should run change detection only once onShow or onHide.

can you provide more information?
How many times are you seeing change detection run?

There is online reproduction: https://stackblitz.com/edit/angular-2g1mvi
Please open the console to see results during moving on the button(it reproduce on all browsers)
For example, I got 19 executions of val(), when I moved the cursor on the button.

How were you measuring?
On which application were you measuring?

I add a function on the view {{val()}} that write to console each time when it is run (when change detection is triggered on the view)
Desired behavior the tooltip should run change detection only once onShow or onHide.

This is raping my app !
look

I found out if remove the second param from ComponentPortal https://github.com/angular/components/blob/5284a57b2f24319359d0528522feeca9c1393751/src/material/tooltip/tooltip.ts#L336
a problem will disappear.
do we really need to set it?

A lot of this comes from the fact that we have Angular animation events for when an animation starts and when it finishes. There's an in-progress PR (#19432) that decouples the tooltip from the Angular's change detection cycle and should resolve this issue.


Allow cleaner implementation api of sandbox mode

Thank you for merging in my previous request.
I would like to please request one last change which will allow a user to be able to pass through a value to the function to be able to toggle the sandbox with one variable change (eg: the env value).
This change will allow the following:
public function toSendGrid($notifiable): SendGridMessage
  {
      return (new SendGridMessage(config('services.sendgrid.templates.default')))
          ->enableSandboxMode(config('services.sendgrid.sandbox_enabled'))
          ->payload([
              ...
          ]);
  }

Without the change this PR introduces, the user is forced to make use of a temporary variable as indicated below
public function toSendGrid($notifiable): SendGridMessage
    {
        $sendGridMessage = (new SendGridMessage(config('services.sendgrid.templates.default')))
            ->payload([
                    ...
                ]);

        if (config('services.sendgrid.sandbox_enabled')) {
            return $sendGridMessage
                ->enableSandboxMode();
        }

        return $sendGridMessage;
    }

This change will not affect the current behaviour with the variable defaulting to true but gives the developer a slightly cleaner implementation for updating the value to enable or disable sandbox mode.

Hi @zbrody, thank you for the PR. I agree with your main idea. I also think setSandboxMode is a more suitable for that usecase.
This way, it will be closer to how Sendgrid handled this in their SDK:
https://github.com/sendgrid/sendgrid-php/blob/285381257100b73aa50d8d70f0bcfb1f48b63747/lib/mail/Mail.php#L1748-L1769

This is now addressed in our latest release. Thanks @zbrody for the contributions.


Whenever I Run spotdl on OSX Mojave it returns an AttributeError no matter what.

What is the purpose of your issue?

[x] Bug
[ ] Feature Request
[ ] Question
[ ] Other

Description

I am running spotdl on a 2017 iMac with python 3.8 installed. I have updated and even reinstalled spotdl a few times. Whenever I run the spotdl command it doesn't work and I always get the following error (see below). I have had no issues when I installed it and ran it on a 2013 macbook air. The macbook is not mine and so I would like to get spotdl to run on my iMac. Any help is greatly appreciated.
Log

Traceback (most recent call last):
File "./spotdl", line 5, in
from spotdl.command_line.main import main
File "/Users/me/Library/Python/3.8/lib/python/site-packages/spotdl/init.py", line 2, in
from spotdl.command_line.core import Spotdl
File "/Users/me/Library/Python/3.8/lib/python/site-packages/spotdl/command_line/core.py", line 26, in
from spotdl.command_line.arguments import ArgumentHandler
File "/Users/me/Library/Python/3.8/lib/python/site-packages/spotdl/command_line/arguments.py", line 31, in
_CONFIG_BASE = spotdl.util.merge_copy(
File "/Users/me/Library/Python/3.8/lib/python/site-packages/spotdl/util.py", line 64, in merge_copy
return merge(base.copy(), overrider)
File "/Users/me/Library/Python/3.8/lib/python/site-packages/spotdl/util.py", line 69, in merge
for key, value in overrider.items():
AttributeError: 'NoneType' object has no attribute 'items'

Try running v3. If the issue still exists, we'll look into it.
@thewitchkingleader , reopen this issue if the problem persists with v3


.m4s files

@hongzimao @ravinet @vibhaa
Hello everyone can anyone help me out how .m4s files are getting called. i.e based on what condition. Please guide me to the code snippet

@hongzimao @ravinet Please help me out

We did not modify the exact logic to retrieve the .m4s files, which is controlled by DASH (https://github.com/hongzimao/pensieve/tree/master/dash.js). We only altered the bitrate adaptation (ABR) logic.
Not sure if this relates to what you were asking for but we inject the ABR algorithm here: https://github.com/hongzimao/pensieve/blob/master/video_server/myindex_RL.html#L34. At video streaming time, we interact with the video client here: https://github.com/hongzimao/pensieve/blob/1120bb173958dc9bc9f2ebff1a8fe688b6f4e93c/dash.js/src/streaming/controllers/AbrController.js#L503-L602 (I honestly forgot how we did it, and here's how I searched for this code: first I noticed we set ABR algorithm here, then I code searched setAbrAlgorithm and traced here, hope this BTS process helps you). After this ABR logic, the downstream process transitions to getPlaybackQuality and later should instruct DASH to retrieve the corresponding chunk.


[ask] "inspect":     {     } property result find at waterline

hi, i have problem.  i have model but my model property field not have inspect. but in find result that property has there.. why its  happen, my old model its not have it before..
thanks
ps) sorry for my broken english..

Hi @wandyirawan, I'm not sure I understand the problem... can you perhaps provide code snippets and expected results?


Add support for using parameter values in creating directories

Let's say I have the following directory structure in a template directory -
template
 |
 + - {{dir_name}}-00
   |
   + - example.txt

and my .scaffold.toml file looks like following
name = "example"
# Omitted for brevity ...

[parameters]
  [parameters.dir_name]
  type = "string"
  message = "Enter the value for directory name."
  required = true
  default = "default-directory"

When I run cargo scaffold templates/ -n example, this should generate following directory structure -
example
  |
  + - default-directory-00
      |
      + - example.txt

This is supported by ffizer and cargo generate as well. May be it's already supported but then not documented properly. I quickly tried this with a simple template, but it generates the directory with {{dir_name}}-00 directory name and not the rendered name,

In fact render_path already exists. Needs a really small fix to make this available during creating a directory. Sending a PR for that.

Fixed by #45


Add L.last

Codecov Report

Merging #64 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #64   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines         438    439    +1
=====================================
+ Hits          438    439    +1

Impacted Files
Coverage Δ

src/partial.lenses.js
100% <100%> (ø)
:white_check_mark:

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 2b210e1...92cb08f. Read the comment docs.

Looks good. Thanks!


[QUESTION] Disable automatic snapshot after disk extend

What's the task? Please describe
I want to disable automatic snapshot after disk extend.
I use below configs but doesnt works. Whenever ı extend my disk, Longhorn get automatic snapshot and I can not remove it.
Describe the sub-tasks
Additional context
defaultSettings:
  defaultDataPath: /data/longhorn/
  defaultReplicaCount: 1
  autoCleanupSystemGeneratedSnapshot: true
  autoCleanupRecurringJobBackupSnapshot: true
  storageReservedPercentageForDefaultDisk: 3
  storageMinimalAvailablePercentage: 3

persistence:
  defaultClassReplicaCount: 1

@c3y1huang Yes, I mean volume expansion. I got my answer.
Thank you.


Test - Maven - TearDown Script fails to teardown EMR Serverless Application

Description
Previously there was a bug with the teardown script where it would fail if an EMR Serverless Application was not in the correct state. This is still the case if scripts/test/maven/tearDown.sh is used.
Steps to reproduce

Run the quick system tests "scripts/test/maven/buildDeployTest.sh" "$SHORT_ID" "$VPC" "$SUBNETS"
After the tests have finished run scripts/test/maven/tearDown.sh "$SHORT_ID"-main
Teardown fails
Check the Cloud Formation logs for BulkImportEMRServerlessNestedStack
See error

Expected behaviour
Any EMR Serverless Applications deployed are stopped and destroyed as part of the teardown script.  It should act the same as the other teardown scripts.
Screenshots/Logs
Resource handler returned message: "Invalid request provided: Application 987 must be in of the following statuses [CREATED, STOPPED]. Current status: STARTED (Service: EmrServerless, Status Code: 400, Request ID: 123)" (RequestToken: 456, HandlerErrorCode: InvalidRequest)
Background
The class TerminateEMRServerlessApplications already handles stoping an EMR Serverless application and is already used via TearDownInstance. The offending class is TearDownMavenSystemTest I think it might have something to do with the clients that are used.

This is only an issue if the script it called with just a SHORT_ID


Problem with boolean and decimal values

I'm seeing this result. bool and bool_t should be true, bool_f should be false, and dec should be 1.23
{
  "id": "gwashington",
  "data": {
    "fam1": {
      "bin": [
        {
          "value": "abc",
          "labels": [],
          "timestamp": "1523953458013000"
        }
      ],
      "bool": [
        {
          "value": "",
          "labels": [],
          "timestamp": "1523953458013000"
        }
      ],
      "bool_f": [
        {
          "value": "",
          "labels": [],
          "timestamp": "1523953458013000"
        }
      ],
      "bool_t": [
        {
          "value": "",
          "labels": [],
          "timestamp": "1523953458013000"
        }
      ],
      "dec": [
        {
          "value": 1,
          "labels": [],
          "timestamp": "1523953458013000"
        }
      ],
      "jadams": [
        {
          "value": 1,
          "labels": [],
          "timestamp": "1523953458013000"
        }
      ],
      "str": [
        {
          "value": "hello",
          "labels": [],
          "timestamp": "1523953458013000"
        }
      ]
    }
  }
}

Environment details

OS: macos 10.13.3
Node.js version: 8.10.0
npm version: 5.6.0
yarn version: 1.3.2
google-cloud-node version: master branch (0.13)

Steps to reproduce
    const entries = [
        {
          method: 'insert',
          key: 'gwashington',
          data: {
            fam1: {
                jadams: 1,
                bool: true,
                bool_t: true,
                bool_f: false,
                dec: 1.23,
                str: "hello",
                bin: Buffer.from('abc'),
            }
          }
        }
      ];

    await table.mutate(entries);

I was able to reproduce.
cc @kolodny

Bigtable stores all it's fields in binary with no encoding information. The typical way to put data inside bigtable or extract it out is by manually doing the encoding/decoding. The library currently helps out by converting numbers to Buffers otherwise it just passes whatever the value is to Buffer.from and hopes for the best. Ideally every field should be encoded before being passed to table.mutate.
In the future we may consider allowing passing some Mutation like API to simplify the encoding/decoding actions to and from the table

One thing to note based on the above, any data previously entered in that fashion should be considered bad and should be deleted
I'll close this issue for now, feel free to reopen if you have any followup questions or issues

Thanks. I would consider throwing on everything except string buffer and null/undefined instead of potentially storing bad values. After testing to see that string/int works you may move on thinking that also bool and decimal works (like I almost did).
An option that enables the lib to use JSON.parse() and JSON.stringify() on non-buffer values could be handy.
JSON.parse(JSON.stringify(null)) // returns null
JSON.parse(JSON.stringify(1.23)) // returns 1.23
JSON.parse(JSON.stringify(new Date())) // returns string 2018-04-22T05:04:10.213Z
JSON.parse(JSON.stringify(true)) // returns true
JSON.parse(JSON.stringify('')) // returns ''
JSON.parse(JSON.stringify(undefined)) // fails

Thanks @moander, I opened #102 to get a discussion about that point started.


Timing out

Describe the bug
Message did not complete it's response is popping up and I'm having to continually resubmit (burning tokens?)
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-cn8pgb
Steps to reproduce
Just run the prompt I put in
Expected behavior
Should complete building the code!
Screen Recording / Screenshot

Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 129
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 72941454,
  "usedJSHeapSize": 69686498,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = astrontelstar
Chat ID = 2cb753887026

Additional context
No response

Hi! Thanks for reporting this issue!
Is this consistent? Can you try again and see if it is still happening? Sometimes messages can get lost for a variety of networking reasons.
If this is a repeated issue and you still can't continue, we can add this to out internal bug tracker.

@astrontelstar resolved in #1073


Format modified configuration files always

Description of PR

What component(s) does this PR affect?

[X] Back-End (Database, Microservices, Containers, etc)
[ ] Front-End (Flask, API, etc)
[ ] Docs
[ ] Build System

Does the PR require changes on other components? If yes, please mark the components:

[ ] Back-End (Database, Microservices, Containers, etc)
[ ] Front-End (Flask, API, etc)
[ ] Docs
[ ] Build System

Related Issue

Resolves #258
Solution

Type

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Docs update
[ ] None of the above

Checklist:

[X] I have read the contributing guide and my code conforms to the guidelines.
[ ] This change requires a change in the documentation.
[ ] I have updated the documentation accordingly.

:running: Benchmark Results :running:

BGP Updates (updates/sec): 1124
Hijacks Detected (hijacks/sec): 100

I will need to test how this looks like for the user, and whether it creates any unnecessary changes. Will re-ping after testing. @pgigis please also take a look at this (and if possible, test the dev container version checking config comparison; I will do the same before proceeding with PR integration).

Update: seems to work fine. I will update the CHANGELOG and merge by Wednesday. No more large diffs while learning! :)


Air Conditioners Showing as nodeProfile

I'm testing out echonetlite2mqtt to use in my home. I have a number of different echonetlite devices, including several air conditioners.
It seems like my air conditioners aren't being discovered as air conditioners, but instead as nodeProfile objects, and they never seem to update after being discovered. If I restart echonetlite2mqtt, some get discovered, but not all.
I have tried some other echonetlite software, and these air conditioners do get discovered as air conditioners. Is there perhaps a race somewhere?
What logging and information can I provide you?

This is the list of properties for one of the air conditioners. Notice the manufacturer(MC) code of 8, which is Daikin:
{
"operatingStatus": "true",
"version": "010c0100",
"id": "fe0000089c50d148d05800000000000000",
"faultDescription": 0,
"manufacturer(MC)": "000008",
"uid": "342d",
"selfNodeInstances": "000001",
"selfNodeClasses": 2,
"selfNodeInstanceListS": {
"numberOfInstances": 1,
"instanceList": [
"013001"
]
},
"selfNodeClassListS": {
"numberOfClasses": 1,
"classList": [
"0130"
]
}
}

@aftertommy
Thank you for your report.
I reproduced it with my Daikin air conditioner.
But 3 months ago it was fine.
Maybe, a firmware update for the air conditioner may have changed its behavior.
In my environment, the cause was that the ID of the nodeProfile and the ID of the air conditioner were the same.
I will fix this issue in a few days.
Also, detailed logs are not output. I plan to add functionality.

Thank you! Please let me know if I can help with gathering more info.

I’m curious, because I’m still learning how this protocol works — are you saying that Daikin is reusing the nodeProfile SEOJ  value but sending  property codes for the air conditioner class? I’m trying to understand how sometimes these get identified as normal air conditioners (and I’m trying to make sure I understand how echonet lite works).

@aftertommy
I fixed this issue.
If you are using docker, use latest or tag:v2.0.1 or newer.
If you're running on node.js, clone the latest code.
In v2.0.1, I changed the nodeProfile Id if the nodeProfile and the device Id are the same.
This will show both the device and the nodeProfile.
If your problem was not due to duplicate Ids, please download the logs at the bottom of the "Logs" page and send them to me.

it looks to be working. thank you so much! oh man im so glad you have no idea.


Add Crushing Recipes for Allthemodium Alloy Dusts

Describe the solution you'd like
I suggest adding crushing recipes for the 3 Allthemodium Alloy Dusts from Allthemodium Alloy Ingots from Allthemodium.

I can add them as default recipes, but wouldn't ATM add the recipes if they desired that?
Or asking the other way around: Are there other ways to crush ingots into dust that are roughly on the same gameplay tier as occultism crusher?
Then I can add them without accidentally unbalancing their pack :)

This is ATM dev Toblerone0508, these dusts are currently unobtainable and don't have uses, I'm planning on adding uses to them in the future and recipes to get them but I thought I might as well suggest that these recipes are added to Occultism itself.

Excellent, then I'll add them :)


create taskWatchdogExample.c file

This taskWatchdog timer library is much easier to understand the basics of Watchdogs.
#include "freertos/FreeRTOS.h"
#include "esp_wifi.h"
#include "esp_system.h"
#include "esp_event.h"
#include "esp_event_loop.h"
#include "nvs_flash.h"
#include "driver/gpio.h"
#include <esp_task_wdt.h>
#include <stdio.h>
#include <stdint.h>
#define WATCHDOG_TIMEOUT_S 5
void myTask(void *myData)
{
while (1)
{
printf("Registering myTask with the Task Watchdog timer.\n");
esp_task_wdt_add(NULL);

	for (int i = 0; i < 5; i++)
	{
		vTaskDelay(1000 / portTICK_PERIOD_MS);
		printf("Tick :%d\n", i);
	}

printf("Reset and Delete watchdog registration of this task.\n");
esp_task_wdt_reset();
	esp_task_wdt_delete(NULL);
}
vTaskDelete(NULL);

}
void app_main(void)
{
printf("App starting\n");

printf("Initializing the task watchdog.\n");
esp_task_wdt_init(WATCHDOG_TIMEOUT_S, true);

printf("Creatign a new task.\n");
xTaskCreate(myTask, "My Task", 2048, NULL, 1, NULL );

}

 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.

@manish5897 There is already example at https://github.com/espressif/esp-idf/tree/master/examples/system/task_watchdog, is there anything that this PR adds?

Closing due to lack of response (also the facts that this PR won't even compile). Please feel free to raise new PR if there is any feature addition to existing example.


enable cross-tab parameter delivery

This patch enables send image to other tab when the image browser is opened on a separate tab.
Known caveats: When there are multiple Gradio tab opened, clicking on send button may cause the parameters to be sent to all of them.

Thank you for the great contribution! I really appreciate the feature you added.
However, there is a small issue with the index.js file. This file is generated by index.tpl.js when running 'yarn deliver-dist', so it is best not to manually modify it. I will add a separate reminder for this.
Regarding the issue of potentially triggering multiple pages simultaneously, It might be possible to generate a random key in advance to determine the correspondence between them. If the received data does not contain the correct key, we can simply ignore that event. However, this approach may result in the keys not matching after a page refresh.


Why queue two frame at most instread of read the latest frame?

https://github.com/alvr-org/alvr-visionos/blob/6005e8c20c8d82c39f05463c8a5b6f9abc1ac93d/ALVRClient/Renderer.swift#L598
should it read the latest one frame in the queue?
two frame cache would cause 22 ms around latency when the render rate is 90Hz.

stability reasons mostly, the buffer allows small circumstantial frame drops to be less noticable and large frame drops to be used as an opportunity to fill the buffer


No releases

https://github.com/copy/v86/releases

They are located under tags => https://github.com/copy/v86/tags.

There should be a release unless an error happened with the workflow or if @copy deleted it.
But in the meantime, anyone with a GitHub account can download a zip containing the latest build pretty easily by going to the check summary:

There should be a release unless an error happened with the workflow or if @copy deleted it.

I didn't delete it, but it only showed up as a draft. I've published it now, let's hope it stays published.

Seems to be working now.


Changed link to https in mkdocs.yml template

I'm not changing these in license headers because there is another PR to update these headers (see: #44)

Thanks, @webimpress.


NoClassDefFoundError ApnsPushNotification Class

Hi, I have this problem:
Exception in thread "ActiveMQ Session Task-1" java.lang.NoClassDefFoundError: com/relayrides/pushy/apns/ApnsPushNotification
Caused by: java.lang.ClassNotFoundException: com.relayrides.pushy.apns.ApnsPushNotification
at org.apache.catalina.loader.WebappClassLoaderBase.loadClass(WebappClassLoaderBase.java:1295)
at org.apache.catalina.loader.WebappClassLoaderBase.loadClass(WebappClassLoaderBase.java:1147)
... 10 more

I have added this one like a maven dependency on my project.
Thanks.

I see that you've moved this question to the mailing list; this appears to be more of a general support question than a bug report, so I'll close this issue and do my best to help you on the mailing list.

Thanks, I created a new question on the maining list y think
https://groups.google.com/forum/#!topic/pushy-apns/CWvgNPp9yqk
I hope you can help me and thank you so much really :)
2017-01-26 17:21 GMT-04:00 Jon Chambers<EMAIL_ADDRESS>
Closed #418 https://github.com/relayrides/pushy/issues/418.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/relayrides/pushy/issues/418#event-938464616, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADnNjMFF-7z3vXYkHVVL5uT7pof-LNjcks5rWQ5DgaJpZM4LtrMa
.


Max fraction infectivity filter and barcode fraction consistency filter should be moved from plate QC to curve fitting QC

One of the most common types of errors that is observed experimentally with this method is a single well in which a single barcode is detected at higher than expected levels. I suspect that this is a result of variation in vRNA production by infected cells. We are able to mitigate the impact of this issue by using multiple barcodes per strain and by making sure that we have adequate coverage of each barcoded strain so that we are averaging over a large number of infected cells - this is why there is an upper limit of barcoded strains that I suggest to be used in this method. With this study, we used 113 barcodes, and and infected 50,000 cells with a TCID50 of 25,000. This should be approximately 150-220 infectious virions for each barcode in each well. To maintain a minimum of 100 virions per barcode, our libraries should contain approximately 50-80 strains, if each is represented by 3 unique barcodes. It may be may possible to increase this number by increasing the cell number in each well and/or decreasing the number of barcodes associated with each variant to 2.
When high vRNA corresponding to a barcode is detected, this typically occurs within a single well and for that well, a small number of barcodes (typically 1, sometimes 1-3) are detected at a higher than expected fraction infectivity. When this issue occurs in a selection condition, it triggers a max fraction infectivity error, and when this occurs in a no-serum well, it triggers a barcode fraction consistency error. I suggest that this type of error is handled by removing specific barcodes from specific wells. However, currently as these flags occur in the initial plate processing QC, the only mechanisms by which these errors can be removed is either by removing the entire well from analysis or the barcode from analysis for that entire plate.
In terms of barcode_frac_consistency, as we use a large number of no-serum controls and calculate a ratio of barcode to spike-in for each of these wells, then use the median value for that as our 100% fraction infectivity, I do not think we need to worry about removal of barcodes that trigger this error, as our calculations should be robust to this error so long as it is not occurring across many wells for the same barcode. I propose that this error should only be triggered in the event that the same barcode fails in multiple wells.
For max_fraction_infectivity, again as this is typically occurring in a single well for each barcode where this is observed, I think prior to fitting curves with the fraction infectivity data, we can remove fraction infectivity measurements that are higher than expected (i.e. >5 or >6).

In the new version 2.0.0, there is per-barcode per-well filtering that does this.

In the new version 2.0.0, there is per-barcode per-well filtering that does this.


feat(api): Add CORS support

Add first-class support for CORS to falcon. Includes a CORS object
for specifying CORS configuration, CORS middleware for applying a
CORS object, and a kwarg to the API class that will take a CORS
object and configure the middleware for you. Allows over-riding
the CORS configuration on a per-resource basis.
Resolves issue #303

I closed and re-opened to add a few small tweaks


Rejected client side version number

Added Flux Networks 1.16.5-<IP_ADDRESS> to the latest release of the Enigmatica 6 modpack (minecraft version 1.16.5), and now am receiving mismatched mod list from multiple users. Triple checked the version number on both ends, even redownloaded the mod, still same behavior.

[Server] [User Authenticator #9/INFO] [minecraft/ServerLoginNetHandler]: UUID of player [user] is [uuid]
[Server] [Netty Epoll Server IO #44/ERROR] [ne.mi.fm.ne.NetworkRegistry/NETREGISTRY]: Channels [fluxnetworks:main_network] rejected their client side version number
[Server] [Netty Epoll Server IO #44/ERROR] [ne.mi.fm.ne.FMLHandshakeHandler/FMLHANDSHAKE]: Terminating connection with client, mismatched mod list

Server:

Client:

so what the network version showed in the log? by finding Version test of

Closed, it should not happen in 6.2.0 (will release later).


Field: containers does not exist during manifest generation

What happened?
I'm using localstack to simulate a AWS bucket in minikube. For this I need to inject the AWS key and secret environment variables to the containers. According to the docs the "prometheusSpec" should have a containers field, which allows overriding containers. (Through it's stated in the docs that it's advanced and might break any time...
When I run the build I get "RUNTIME ERROR: Field does not exist: containers".
local k = import 'ksonnet/ksonnet.beta.4/k.libsonnet';
local t = import 'kube-thanos/thanos.libsonnet';
local pvc = k.core.v1.persistentVolumeClaim;
local storageClass = k.storage.v1.storageClass;
local podDisruptionBudget = k.policy.v1beta1.podDisruptionBudget;

local prometheusDeploymentName = "k8s";
local monitoringNamespace = "monitoring";

#######################
# Proemtheus Operator
#######################

local kp =
  (import 'kube-prometheus/kube-prometheus.libsonnet') +
  // Adds Thanos Sidecar to the promi instance.
  (import 'kube-prometheus/kube-prometheus-thanos-sidecar.libsonnet') +
  (import 'kube-prometheus/kube-prometheus-all-namespaces.libsonnet') +
  // This removes KubeControllerManagerDown and KubeSchedulerDown alerts & rules. These are not required as these run in GKE's managed (and inaccessable) plane.
  (import 'kube-prometheus/kube-prometheus-managed-cluster.libsonnet') +
  // This seems to fix a current generation bug at: https://github.com/prometheus-operator/kube-prometheus/issues/824
  (import 'github.com/thanos-io/thanos/mixin/config.libsonnet')
  {
    _config+:: {
      namespace: "monitoring",
    },

    prometheus+:: {
      prometheus+: {
        spec+: {
          version: 'v2.19.2',
          thanos: {
            baseImage: 'quay.io/thanos/thanos',
            objectStorageConfig: {
              key: 'thanos.yaml',
              name: 'thanos-objectstorage'
            },
            version: 'v0.16.0'
          },

          containers: [
            if c.name == 'thanos-sidecar' then c {
              env+: [{name: "AWS_ACCESS_KEY_ID", value: "test"},{name: "AWS_SECRET_ACCESS_KEY", value: "test"},],
            } else c
            for c in super.containers
          ],

        },
      },
    },
  };

#####################
# Manifest Generation
#####################

# Prometheus operator
{ ['setup/0namespace-' + name]: kp.kubePrometheus[name] for name in std.objectFields(kp.kubePrometheus) } +
{
  ['setup/prometheus-operator-' + name]: kp.prometheusOperator[name]
  for name in std.filter((function(name) name != 'serviceMonitor'), std.objectFields(kp.prometheusOperator))
} +
{ ['prometheus-' + name]: kp.prometheus[name] for name in std.objectFields(kp.prometheus) }

Did you expect to see something different?

I expected it so succeed or to be removed from the documentation. I dig down in the vendor folder and saw that the mixin file for the Prometheus CRD contained the containers field correctly... So I'm a bit clueless.

How to reproduce it (as minimally and precisely as possible):

I use the latest version - see jsonnet file:

{
  "version": 1,
  "dependencies": [
    {
      "source": {
        "git": {
          "remote": "https://github.com/prometheus-operator/kube-prometheus.git",
          "subdir": "jsonnet/kube-prometheus"
        }
      },
      "version": "v0.7.0"
    },
    {
      "source": {
        "git": {
          "remote": "https://github.com/thanos-io/kube-thanos.git",
          "subdir": "jsonnet/kube-thanos"
        }
      },
      "version": "v0.17.0"
    }
  ],
  "legacyImports": true
}

IIUC your use case, you should set the containers field. The Prometheus operator will do a strategic merge patch between your input and the generated statefulset definition.
    prometheus+:: {
      prometheus+: {
        spec+: {
          version: 'v2.19.2',
          thanos: {
            baseImage: 'quay.io/thanos/thanos',
            objectStorageConfig: {
              key: 'thanos.yaml',
              name: 'thanos-objectstorage'
            },
            version: 'v0.16.0'
          },

          containers: [
            {
              name: 'thanos-sidecar',
              env: ...,
              ...,
            }
          ],

          ...

IIUC your use case, you should set the containers field. The Prometheus operator will do a strategic merge patch between your input and the generated statefulset definition.
    prometheus+:: {
      prometheus+: {
        spec+: {
          version: 'v2.19.2',
          thanos: {
            baseImage: 'quay.io/thanos/thanos',
            objectStorageConfig: {
              key: 'thanos.yaml',
              name: 'thanos-objectstorage'
            },
            version: 'v0.16.0'
          },

          containers: [
            {
              name: 'thanos-sidecar',
              env: ...,
              ...,
            }
          ],

          ...

 Thanks @simonpasquier !

Works. Thanks @simonpasquier !


Align swift sdk configuration subcommand with proposal text

Per the proposal text:

swift sdk configure <identifier> <target-triple>, which allows users to provide additional search paths and toolsets to be used subsequently when building with a given Swift SDK. Specifically, multiple --swift-resources-path, --include-search-path, --library-search-path, and --toolset options with corresponding paths can be provided, which then will be stored as configuration for this Swift SDK. swift sdk configure <identifier> --show-configuration will print currently set paths, while swift sdk configure <identifier> --reset will reset all of those at once.

@swift-ci test

@swift-ci test linux

@swift-ci test windows

@swift-ci test

@swift-ci test windows

@swift-ci test

@swift-ci test

@swift-ci test


NTR: Digestive System Surgery

NTR: digestive system surgery
Parent: MAXO:0000004 surgical procedure
Def: Surgery performed on any part of the digestive system.
NCIT:C15330
xref: NCIT:C15330
cc @lopierra
needed for INCLUDE

Any updates on adding these terms?

added new surgery by site pattern. Updated or added several new terms. This term will be available after next release.


Linear tasks not fusing in dask.dataframe

Some tasks within dask.dataframe don't seem to fuse well, even though I've turned on fusion and have a linear chain of tasks.  I've worked things down to the following example:
import pandas as pd
import dask.dataframe as dd
import dask

df = pd.DataFrame({'x': []})
ddf = dd.from_pandas(df, npartitions=1)

ddf = ddf.map_partitions(lambda x: x)
ddf['x'] = ddf['x'].astype('int8')
ddf = ddf.map_partitions(lambda x: x)
ddf['x'] = ddf['x'].astype('int8')

dask.config.set(fuse_ave_width=5)
ddf = dask.optimize(ddf)[0]
ddf.visualize('dask.png')

Many operations will fuse, but something about these kinds of tasks don't let them.  I'm finding it difficult to find out why.  cc @eriknw in case he has a moment to look this over.

Interesting.  I'll take a look this evening.

Thanks Erik!
On Mon, Jul 23, 2018 at 1:07 PM, Erik Welch<EMAIL_ADDRESS>wrote:

Interesting. I'll take a look this evening.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/3807#issuecomment-407131034, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASszONJ6CWi8u9TT06p_eFB-CUk6dTqks5uJgLJgaJpZM4VbV3y
.

Aha.  We've both seen this before.  You showed it here: https://github.com/dask/dask/pull/1979#issuecomment-280968528
In my immediate reply, I said:

The optimized graphs may look linear and fusible, but they're not fused because e.g. ('wrapped-#1', 0) is used twice by ('add-exp-mul-sub-#0', 0)

I experimented with a "fancier fuse" a few comments later: https://github.com/dask/dask/pull/1979#issuecomment-281230317
Is it time to revisit "fancier fuse" that creates a task that is like a self-contained dask graph?

Ah, right, so we would end up computing the the intermediate result twice.

Is it time to revisit "fancier fuse" that creates a task that is like a self-contained dask graph?

Possibly, yes.  Do you have an estimate on the performance impact and the human time it would take to develop.  I suspect that you would end up being the person to do this.  Is this something that you have time and interest in doing?

Good questions.  For performance reasons, we would ideally want to include this optimization in a single call to fuse.  My previous experiment used two passes.
I am interested in this and am inclined to work on it as I am able.  I am unavailable to work on in this week.
If my memory serves, I think @jcrist also experimented a while ago with creating objects that encapsulated and computed compound tasks that include multiple occurrences of a single dependency (like what we see in this issue).  This may be a better starting point than my previous experiment.

Just checking in here.  @eriknw you mentioned that you wouldn't have time last week.  Do you anticipate having time to work on this in the near-to-moderate future?

Yeah, I plan to dive in this weekend.


Closes #1

Clarified validations.py
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

Huh


[Documentation]: Links is not working in multiple docs page

Documentation Issue Description
There are several links that are not working on the documentation pages:
Agent page see here:

Learn more
Our Putting It All Together section has more on agents

Chatbots page see here:

Building a chatbot tutorial

Q&A see here:

Searching Pandas tables

Text to SQL

For further examples of Q&A use cases, see our Q&A section in Putting it All Together.

Documentation Link
https://docs.llamaindex.ai/en/stable/use_cases/agents.html
https://docs.llamaindex.ai/en/stable/use_cases/chatbots.html
https://docs.llamaindex.ai/en/stable/use_cases/q_and_a.html

Hi, I'd like to work on this issue. However, I could see that @0xDigest referenced this issue.


Implement Download Email Submission Form

Context
The type of Implementation: New feature (non-breaking change which adds functionality)
When users click the "Download" button on the download page, our application displays the "Download Start Processing" view includes a form that requests the following information from users:

an input field to specify a preferred email address to receive download notifications
a checkbox to accept the terms of use (required)
a checkbox to subscribe to the refine.bio team update (optional)
a button to start the download processing

Problem or idea
Mockups: Download Start Processing View's Email Form
Based on the above, we'll implement the download email submission view.
Solution or next step

Implement the download email form component
Create any sub-components or helpers as necessary

This view was included in 44 - Implement My Dataset Download Status Views, thus closing this ticket, thank you!


Update setup.py

Support for the python 3.10 in order to run on Databricks runtime 13.3.
Describe the big picture - why are we doing this?
Any additional details to clarify code in the PR?
How is this tested?
Pull Request Checklist

[ ] Pull request includes a description of why we are doing this
[ ] CHANGELOG has been updated
[ ] Version in _version.py has been updated
[ ] All tests in the tests folder pass with a local build
[ ] README has been updated (if applicable)
[ ] Documentation in docs has been updated (if applicable)
[ ] requirements.txt and requirements-dev.txt have been recompiled (if applicable)
[ ] Docker image can be built using docker build -t collie .

Screenshots or GIFs?

Need to update to python 3.10 for Databricks runtime 13.3

@mkmrabby gotcha! Relatedly, I have a couple Collie changes I would love to be able to make - would I be able to tag you / someone else on the team for a review on those PRs?


Solve union issue with io-ts

PASS  src/tests/account_coverage.spec.ts
PASS  src/tests/tryout.spec.ts
PASS  src/tests/account_guarantor.spec.ts (17.176s)
FAIL  src/tests/account.spec.ts
● Test suite failed to run
TypeError: Cannot read property 'name' of undefined

  146 | import * as t from 'io-ts';
  147 |
> 148 | export const RTTI_ResourceList = t.union([
      |                                    ^
  149 |     RTTI_Account,
  150 |     RTTI_ActivityDefinition,
  151 |     RTTI_AdverseEvent,

  at node_modules/io-ts/lib/index.js:764:59
      at Array.map (<anonymous>)
  at getUnionName (node_modules/io-ts/lib/index.js:764:25)
  at Object.<anonymous>.exports.union (node_modules/io-ts/lib/index.js:770:35)
  at Object.<anonymous> (src/Union/RTTI_ResourceList.ts:148:36)
  at Object.<anonymous> (src/Resource/RTTI_Account.ts:6:1)
  at Object.<anonymous> (src/__tests__/account.spec.ts:1:1)

Test Suites: 1 failed, 3 passed, 4 total
Tests:       5 passed, 5 total
Snapshots:   0 total
Time:        22.055s

When solved, I may make resources 'exact'

Solved,
hahaha, I didn't make a recursion of union as all types do !

Fixed in 4.0.4


Iot/added deprecation errors

create_thing and update_thing now raise exceptions if thing type is deprecated.
@bblommers - Who is responsible of adding the X to the implementation details? also the release notes - who keeps track of which new methods are implemented? (as i've seen 2.2.4 is not shown yet there)

Codecov Report

Merging #4192 (bbb117d) into master (27c9b31) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #4192   +/-   ##
=======================================
  Coverage   95.07%   95.08%
=======================================
  Files         480      480
  Lines       53031    53035    +4
=======================================
+ Hits        50419    50427    +8
+ Misses       2612     2608    -4

Flag
Coverage Δ

servertests
55.25% <0.00%> (-0.01%)
:arrow_down:

unittests
95.00% <100.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

moto/iot/models.py
91.08% <100.00%> (+0.60%)
:arrow_up:

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 27c9b31...bbb117d. Read the comment docs.

Hi @arielb135, thanks for the PR!

@bblommers - Who is responsible of adding the X to the implementation details? also the release notes - who keeps track of which new methods are implemented? (as i've seen 2.2.4 is not shown yet there)

The maintainers will update both before a release, so don't worry about that 👍


compound type (Union) gets preferentially parsed as string

Checks

[x] I added a descriptive title to this issue
[x] I have searched (google, github) for similar issues and couldn't find anything
[x] I have read and followed the docs and still think this is a bug

Bug
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
             pydantic version: 1.7.3
            pydantic compiled: True
                 install path: /usr/local/lib/python3.7/site-packages/pydantic
               python version: 3.7.11 (default, Jul  6 2021, 12:42:50)  [Clang 12.0.0 (clang-1<IP_ADDRESS>)]
                     platform: Darwin-19.6.0-x86_64-i386-64bit
     optional deps. installed: ['typing-extensions', 'email-validator']

from pydantic.dataclasses import dataclass
from typing import Dict, Union

@dataclass
class MyDataclass:
	c: Dict[str, Union[str, bool, int]]

print(MyDataclass(c={"a": 1, "b": True, "c": "d"}))

output: MyDataclass(c={'a': '1', 'b': 'True', 'c': 'd'})
expected output: MyDataclass(c={'a': 1, 'b': True, 'c': 'd'})
As you can see from the repro above, it seems that the str component of the Union. If I change the order to [bool, int, str], the output becomes: MyDataclass(c={'a': True, 'b': True, 'c': 'd'}). I'm not familiar at all with how the underlying code works, but my guess is that this is giving preference to types given their order in the Union. I find this behavior quite odd, especially considering that the objects in the input already conform to the types specified in the dataclass. Is this intended behavior?

I believe this is intended. From the Unions section of the docs, emphasis mine:

The ordering of unions is supposed to be ignored:
https://github.com/python/cpython/blob/main/Lib/typing.py#L564

I believe this is intended. From the Unions section of the docs, emphasis mine:

The ordering of unions is supposed to be ignored:
https://github.com/python/cpython/blob/main/Lib/typing.py#L564

The ordering is ignored when a type checker is reading an annotation to determine what type a variable is. If we have x: str | int, then yeah, that's equivalent to x: int | str.
But that's not what pydantic is doing. Pydantic is a parsing library, so when it sees that a value is supposed to be parsed into a Union, it has to decide which member to use. It can't parse something into a str and int simultaneously. It uses the order to make that decision in a predictable way.

Regardless, though, looks like #2902 has landed!

But that's not what pydantic is doing.

Yes, I eventually worked that out after hours of pulling my hair out. Unfortunately for someone who's very familiar with how mathematical unions are supposed to work, this behaviour was extremely surprising.


[branch-2.1] Picks "[Fix](partial update) Fix DORIS_SEQUENCE_COL is not set for newly inserted rows in partial update #40272"

picks https://github.com/apache/doris/pull/40272

run buildall

run buildall


Controller out-of-memory kill when cluster has too many secrets

/kind bug
What steps did you take and what happened:
Given a cluster with a high count of Secrets (in the order of about 8k secrets), when and ISVC is reconciled the KServe controller tries to load into memory all Secrets in the cluster. Because the high count of Secrets, the KServe controller will reach its memory limit and the cluster will kill the pod. The cluster may re-create the pod but, given unchanged conditions, the pod will be killed continuously.
What did you expect to happen:
KServe controller should work fine even if the cluster has a high count of Secrets.
What's the InferenceService yaml:
Not relevant. Any ISVC will lead to the crash.
Anything else you would like to add:
On a quick code review, looks like KServe catches the secrets when using an informer/watch. However, KServe may not need to watch for changes in Secrets and, apparently, Secrets are always queried by name. If this is true, it may be better to query directly the API Server rather than via a cache of the Secrets, preventing the need of an Informer/watch.
Environment:

KServe Version: 0.11.2 - should also happen in  master

/assign


Fix member session return type

Just looks like sdk-codegen wasn't pulled to grab https://github.com/stytchauth/sdk-codegen/pull/182 :)

Woops yup, need to regenerate.


[FEATURE] Investigate using Hardtanh for training models

Using Hardtanh as the WaveNet activation function seems to work well, and has the potential to be faster than the "fast" tanh. It also has the benefit of being supported directly by pytorch.

I've been investigating this some more, and I haven't yet seen any downside to training ESR using Hardtanh. Some more varied testing would be good, though.
Playback performance is definitely improved, although by how much seems to be compiler-specific. I tested with VS and clang, and saw about a 7% speed increase on the former, and 4% on the latter.
Clang performance was also significantly better than VS - so we may want to be aware of that in general.
I got closer to a 15% increase in speed with gcc on Raspberry Pi - so a lot depends on compiler and platform.

Do you notice any difference in the spectrum? Eg aliasing behavior?

Do you notice any difference in the spectrum? Eg aliasing behavior?

I haven't done any spectral analysis - that isn't something I really have much experience with.

Checking back through issues, it looks like this is supported by both this repo as well as the plugin, so I believe that resolves this issue.
If we wanted to e.g. define an architecture that used it (e.g. as part of nano), we could look at that separately.


How can i set the plus button?

like the Roles in the main page About

@CrazyDavis did you know how to make it ?

You can make it by

setting yOffSet property in the RAMAnimatedTabBarItem instance

You can make it by

setting the yOffSet in the RAMAnimatedTabBarItem
use bigger image icon than the other tabs


[tests] Add support for running EndToEnd tests on helix with the aspire workload

Problem:

We want to run some EndToEnd tests on CI, which can dotnet run an aspire project,
and allow individual tests to interact with the services.
This requires:

Ability to build, and run an aspire project - IOW, a sdk with the aspire workload installed.
docker

Also, allow using TestProject.* in tests/testproject, in two modes:

in-tree test run which directly reference aspire projects, and repo targets
out-of-tree test runs which uses a SDK+workload based on local build output

Solution:
SDK+workload

SDK with version from global.json is installed into artifacts/bin/dotnet-{none,latest}

The aspire workload is installed into the sdk using the nuget packages in artifacts/packages with a version like 8.0.0-{dev,ci}

this also allows running tests that do dotnet new aspire-starter; dotnet run same as a user
this utilizes an internal nuget from dotnet/runtime

TestProject

This can switch between the two test run modes using the msbuild property $(TestsRunningOutOfTree)

when running in-tree the test project directly references hosting targets, and aspire projects via ProjectReference
when running out-of-tree the ProjectReferences and imports are replaced with PackageReferences to the Aspire nugets

Default is to run in-tree

Helix

The tests are built on the build machine
The testproject, and the sdk+workload is sent to helix

where the tests run using dotnet from the sdk+workload path

Since docker is needed to helix, this is enabled only for Linux in this PR. Blocked on https://github.com/dotnet/dnceng/issues/2067 for windows support.

TL;dr or How do I use this?
Steps to prepare for out-of-tree runs

.\build.cmd -pack - to build all the nugets
dotnet build tests/Aspire.EndToEnd.Tests/Aspire.EndToEnd.csproj /t:InstallWorkloadUsingArtifacts /p:Configuration=<config>

this will install the sdk, and the aspire workload using the nugets from artifacts/packages

at this point the sdk is usable outside the repo too. Note: you need to add artifacts/packages/$(Configuration)/Shipping as a nuget source so the locally built packages can be picked up.

Using it from VS

No difference when running the EndToEnd tests in in-tree mode
For switching to out-of-tree add <TestsRunningOutOfTree>true</TestsRunningOutOfTree> to tests/Aspire.EndToEnd.Tests/Directory.props before any imports.

tests cannot be run at this point as they will fail complaining about artifacts/bin/dotnet-latest being missing
Install the sdk+workload following the steps above
Run/debug the tests normally now, and they will be using the sdk
Also note that in this case the testproject is run from the bindir for Aspire.EndToEnd.Tests, so a path like artifacts/bin/Aspire.EndToEnd.Tests/Debug/net8.0/testassets/testproject/

Using it from command line

No difference when running the EndToEnd tests in in-tree mode
Install the sdk+workload following the steps above
When running the tests you can either:

set <TestsRunningOutOfTree>true</TestsRunningOutOfTree> to tests/Aspire.EndToEnd.Tests/Directory.props before any imports
or set the environment variable TestsRunningOutOfTree=true

Other details

tests/Shared/WorkloadTesting - has sources essentially from dotnet/runtime's Wasm.Build.Tests
TestProject supports skipping individual resources like --skip-resources oracledatabase,cosmos
.runsettings with 15min timeout
xunit.runner.json - to get diagnostic messages
The core infrastructure is not specific for Aspire.EndToEnd.Tests, and will get used by other test projects in the future

Microsoft Reviewers: Open in CodeFlow

[ ] Questions:

[ ] should a forced container cleanup be done at the end?
[ ] nuget-cache - Is it OK to expect the dev to manually delete aspire packages from the cache?

PR uses a new nuget-cache path for every run, which is slow on slow connections, but good on fast ones

[ ] Microsoft.NET.Sdk.Aspire: set PackageType='' to the manifest nuget is notdotnetplatform, and can be installed directly by the workload testing tasks. Check that nothing else is affected, and this is an acceptable change.

TODO:
- [ ] nuget.helix.config - Use the root nuget.config and insert the built-nugets source

Talked offline with @radical, marking as ready for review as this is ready for feedback.

@eerhardt I tried to add oracledb back, but it failed because the test timeout was 5minutes. I think we continue with that in a follow up PR, so reverting that change.

Follow-up work is being tracked in https://github.com/dotnet/aspire/issues/2687 .


When tables don't have an update data binding, make the values readonly

Some tables are only insert and delete capable, so the cells shouldn't be editable on existing rows.

Changed so that the enabled property for TableRows, when set to false will only allow insert changes.


Bump internal version files to 1.1.0

Fix #1381
These internal version files are used to:

output service versions as part of the startup message
tag staging Docker images sent to Nexus from master nightly build

Version must be incremented so as to not mingle Fuji work with prior
Edinburgh work in Nexus staging.
Signed-off-by: Trevor Conn<EMAIL_ADDRESS>

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@8bfb242). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1386   +/-   ##
=========================================
  Coverage          ?   17.02%
=========================================
  Files             ?       78
  Lines             ?     8789
  Branches          ?        0
=========================================
  Hits              ?     1496
  Misses            ?     7142
  Partials          ?      151

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 8bfb242...643e928. Read the comment docs.


Inconsistent file sizes in binary .scn during import

I have found a strange issue when importing scene from .dae file using "Instanced Sub-Scenes" option. This option creates a separate scene (binary .scn) for each object in the imported .dae file. After the initial import, I have a .scn file with size, for example, 3411 bytes. Now, if I open this scene and save it again (using Save Scene As and .scn format) it has only 1437 bytes. Is it intended behavior? Below you can find the bigger version of the file, right after the import from .dae - try to save it manually.
Cube.zip

Another tip - if you convert it to a text format (.tscn), both versions, the smaller and the bigger, have identical size and content.

@shinyclaw Can you still reproduce this in Godot 3.2.3 rc4 or at least provide the original model? It's impossible to test importer with the imported model.

No answer, so closing. Please comment if you can still reproduce it, and make sure that an up-to-date reproduction project is included.


Rename tests method to more explicit names

A good convention is something like :

methodName_should_do_something
methodName_when_condition_should_do_something
methodName_when_condition_should_throw_exception

Similar request: Refactor tests structure in compas-sct


Support Narrow Types in BitCast Folding Rule

This change adds support for narrow types in the BitCastScalarOrVector folding rule. According to Section 2.2.1 of the SPIR-V spec, types that are narrower than 32 bits are automatically either sign extended, or zero extended depending on the type. With that guaranteed, we should be able to use the first 32-bit word of any narrow type for the folding logic without performing any special conversions.
In order to reduce code duplication, this change moves the GetU32BitValue and GetU64BitValue functions from IntConstant to ScalarConstant. Without this move, we would have needed an identical version of GetU32BitValue on FloatConstant.

 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.

Just pushed up some new test cases in the IntegerInstructionFoldingTest suite that cover 16-bit type casting. I handled the 16-bit floats in there too because I didn't see a clean way to integrate them into the FloatInstructionFoldingTest since it uses 32-bit C++ floats as the comparison value. I would have had to do some ugly casting to make them work there, so I figured it made more sense to treat them as integers. (Plus it also allows the 32-bit sign/zero extension logic to be tested)

Just added two more test cases to help get some coverage for 8-bit constants.
Please let me know if anything else is missing! 🙂

Thanks for the test cases and for your patience in waiting for the review.

No problem! Thank you for the review! 🙂


 It will now always copy index.html to 404.html without any option to adjust. If there is no index.html, than a message is printed.

Please test<EMAIL_ADDRESS>


Shift + direction key is unrecognized

When typing shift with any direction key, that key combination is unrecognized, and shows an icon that has a question mark inside a square:

I think this and #138 are duplicates of #106.
@akitchen any progress on this? 🙂  Being able to display Shift + direction key strokes would be lovely. I would love to help, unfortunately I won't be able to make a pull request as I don't know C! 😬  But maybe we can help with thoughts if you need any? (see https://github.com/keycastr/keycastr/issues/140#issuecomment-560005953)
Thank you for this app!

Yes, you're right about the duplicates @ashugeo . This is fixed on the master branch and will be in the next release v0.9.9


--replicas flag has been deprecated

Commands in chapter 8 rely on the depreciated replicas. This is several places throughout the book.
It was removed in 1.18.
https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG/CHANGELOG-1.18.md#deprecation-4
kubectl run be-default \

--image=gcr.io/kuar-demo/kuard-amd64:blue
--replicas=3
--port=8080
Flag --replicas has been deprecated, has no effect and will be removed in the future.
pod/be-default created

The closest alternative I could come up with for a command such as
kubectl run alpaca-prod --image=gcr.io/kuar-demo/kuard-amd64:blue --replicas=3 --port=8080 --labels="ver=1,app=alpaca,env=prod"

would be the three commands
kubectl create deployment alpaca-prod --image=gcr.io/kuar-demo/kuard-amd64:blue --replicas=3 --port=8080

kubectl label --overwrite deployments ver=1 app=alpaca env=prod --selector="app=alpaca-prod"

kubectl label --overwrite pods ver=1 app=alpaca env=prod --selector="app=alpaca-prod"

So, basically, the labels for the resources (the deployment and pods) have to be specified in separate commands.

Thank you for raising this issue. We've updated this in the 3rd edition to something similar to the following
# First, create the deployment
$ kubectl create deployment alpaca-prod --image=[gcr.io/kuar-demo/kuard-amd64:blue](http://gcr.io/kuar-demo/kuard-amd64:blue) --replicas=2
# After the deployment is created, you can add some labels
$ kubectl label deployments alpaca-prod --overwrite ver=1 app=alpaca env=prod


Solve problem with {} tags in identifier

Description
In the indicator class, we use the identifier attribute as the unique identifier of a given class. Since this was also used as the output variable name for the created netCDF, we introduced formatting tags in the identifier attribute. The problem is that the same identifier is also used in other contacts (finch) to identify uniquely the process, but before the class is called, we can't format the identifier string.
We should have two distinct attributes: the unique class identifier, and the output variable name. The identifier should not contain {}, but the output variable name could.
More generally, we should clarify which attributes need to be clearly defined without arguments tags, and which can include tags.

I agree 100%.. should be a relatively quick fix

Also noticed that with the changes in https://github.com/Ouranosinc/xclim/pull/173  {thresh} in id (or proposed  variname) will now contain the string as thesholds : e.g. '30 degC' instead of the previous '30'.  Thus an index of 'txgt_30' becomes 'txgt_30 degC' or similar ...


Mark groovy and scala plugins as supported with the configuration cache

In the user manual and mention this in the release notes

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.

Thanks, Paul!


global ngModelOptions ignored when ngModelOptions provided in field definition.

So I have a need to add the tab index a fied resides on to the ngModelOptions so when its time to validate, I can ignore specific validation errors by parsing the errors array, which contains error codes, which in turn contains the array of ngModelController objects specific to that error.  I can grab the fields tab from the ngModelController.$options to check the tab it's part of and know if I should ignore it or not.
Problem is I noticed when I add the ngModelOptions property to the field's form definition, it ends up not applying the global formDefaults' ngModelOptions property on it.
I believe the offending spot is under the sfPathProvider, inside the $get method's postProcessFn method.
It parses through obj.keys, and when obj[attr] === undefined, it applies the schemaDefaults (lookup's saved stdForm object) attribute to it.  Seems like it would be good to set ngModelOptions property as a merge between the stdForm settings to capture the global form settings, and also pass in the field's for definition specific options if they exist.
Which leads to the inquiry, besides updating the schema forms file to merge the ngModelOptions, is there another way to lookup the form definition settings when all I have is the ngModelController of that field?  Also have a need to grab the path somehow.

Hi @kyse are you referring to within a directive of your own? Without knowing that I would be thinking about importing the sfField scope for the info.
Thanks for the info on the merge script issue, that does seem like something worth updating, the script is now in json-schema-form-core in the merge.js file and the options attribute passed into the merge function is whatever is added to the sf-options attribute, so it would be relatively easy to change.
I will make this issue an enhancement and take a look at updating it tonight :)

Yes inside a directive, once I broadcast for the form to validate, if there are errors, I'm checking the formController's $errors array to grab all the ngModelControllers.  I think I may go about this differently, and just store the form definition for that field in a factory so I can easily look it up.  Since I'm using custom GetterSetter's to apply the model value so I can make use of customized array targeting, the built in lookup store you guys use gets overridden since my keys can be Entity.Person[].FirstName.

There is an issue kicking about regarding showing form status on a tab for the validity of the form within its tab pane, the solution offered in that may be of interest to you as well perhaps.


Is it possible to take a Screenshot that contains layers (UIKit elements) and the OpenGL elements?

Hi,
First of all, thanks for your sample. It´s great.
I want to add a UIButton to take a screenshot after my image is recognized and my scene appears. I could achieve to take a screenshot where the scene (opengl) appears but the uilabel (uikit) doesn´t.
Is it possible to do this?
Thanks very much for your support.
Regards, Juan

Hi, @jarrillaga
Maybe, You need to be combining each snapshots (OpenGL and UIKit).
as below:
http://stackoverflow.com/questions/11299682/programmatically-take-a-screenshot-combining-opengl-and-uikit-elements


Added support for Special opcodes when displaying opcode names.

Now special opcodes won't all just display "Special" on debug printout. They'll say "Special: <special_op>".
This applies the patch to the base code, and is independent of patch #44. That patch has a separate fix that's coming, as soon as I figure out how to update a pull request.

To update a pull request, simply push to the branch that the pull request references.
I'm a bit confused how these two PR's fit together though, can you clarify that a bit?

This is the updated pull request; I successfully pushed the update and squashed.
They're related because this pull request prints instructions, and the bug relates to printing instructions. So I applied the fix to the place where this pull request prints an instruction, and the other fix is for where your code prints an instruction.

Hey, would it be possible for you to merge this and #44 into one PR, and add a specialized print function to reduce the duplication between them? I'd also prefer if you squashed them into one commit.

Sure thing.


docs(share_plus): #1299 document XFile.fromData

Description
Document the use of temporary directories and why developers should care about it.
Related Issues

closes #1299

Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I titled the PR using Conventional Commits.
[x] I did not modify the CHANGELOG.md nor the pubspec.yaml files.
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze) does not report any problems on my PR.

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

[ ] Yes, this is a breaking change (please indicate that with a ! in the title as explained in Conventional Commits).
[x] No, this is not a breaking change.

@Coronon fyi

Thanks, please do not merge yet: Your wrote 9getTemporaryDirectory

Fabulous 🎉

Thank you @nohli !


Handle universal module format a little better

The previous version errors out on the client side if another library provides a bad implementation of a "require" function.  This pattern is more resilient both in Node and the client browser.

Thanks. The changes seem sensible. However, the code was completely reformatted (two times it seems) which will make very hard to backtrace any bugs. (you can take a look at our code style here)
Can you remake the PR, touching only the lines you changed?

Yep, you got it.


Boolean Inputs Not Recognized

When selecting Boolean true/false as the type for Up/Down values respectively, an error message is generated during execution:
Input property msg.payload should contain the specified low or high value.
This was tested using a Trigger:State node as the input, set to output Boolean values. But the same behaviour can be reproduced using simple Inject nodes as well.
I can work around this by setting Up/Down to type Number and then using Switch and Change nodes to remap the true/false input to 1/0 instead. A bit of a hack.
Are Booleans actually supported for Up/Down values?

Hi @Boh1,
Thanks for reporting this issue!
I have added a fix for this on Github.  Note that this fixed version is not available on NPM yet, because @tsknightstorm and I have been working recently on a major release of this node, but were waiting for feedback from the community.  Will add a reminder on Discourse, because would like some folks to test the new calibration feature before we release this on NPM.
If you want to try meanwhile whether this fix works correctly (and also didn't mess up other value types), you can install it directly from this Github repository.  To do that you need to install git and then execute (from within your .node-red folder):
npm install bartbutenaers/node-red-contrib-button-events

Test feedback is appreciated!
Bart


serializer: update tests for date intervals

changes the UI serializer test for date interval to apply the unicode space char change as in Babel. See https://github.com/python-babel/babel/commit/d76db13414a6104019edc0dbafa82410e8d08e63

Babel 2.12 upgrades CLDR to v42, which introduces this change to the unicode space char.

How would this affect the final UI? I'm wondering if we should decode it to utf-8 here https://github.com/inveniosoftware/marshmallow-utils/blob/master/marshmallow_utils/fields/babel.py#L75
That would be the lowest module where the value is produced.

there will be no noticeable impact to the UI, except date ranges not jumping to the next line inside the text - which is even better. u2009 is a "thin space", better visually.


clang-tidy auto fixes for performance, coreguidlines and modernize checks

These are the fixes clang-tidy was able to do automatically by running a subset of checks it supports. For this invocation i filtered it to do all modernize, performance and core guidelines checks.
The full list of checks is available here
http://clang.llvm.org/extra/clang-tidy/checks/list.html

    ) PURE;

Leave this too -- here the SAL is meaningful.

Refers to: dev/Effects/microsoft.ui.composition.effects_impl.h:79 in f1b8511. [](commit_id = f1b851109061a41c1290a681e16de80f3aa2e673, deletion_comment = True)

#include "Microsoft.UI.Composition.Effects_impl.h"

revert

Refers to: dev/Materials/Reveal/RevealBrush.cpp:10 in f1b8511. [](commit_id = f1b851109061a41c1290a681e16de80f3aa2e673, deletion_comment = True)

/azp run


Output buffer problem on website Created event

When I create a new tenant and the WebsiteEvent event is emitted, the script outputs a string of 8 characters. After I return the created website in the json response, the json payload is not parsed because of this string.
This happens in the Hyn\Tenancy\Repositories@create method on the line where the Created event is emitted:
    $this->emitEvent(         new Events\Created($website)     );
If I clean the output buffer after the event, everything works as it should
 ob_start();

 $this->emitEvent(
    new Events\Created($website)
);

$output = ob_get_contents();

ob_end_clean();

var_dump($output); # Here I get this output: string(8) "        "

Information

hyn/multi-tenant version: 5.4.*
laravel version: 5.8.*
database driver and version: mariadb-10.4
webserver software and version: nginx-stable
php version: 7.4.2

Sorry guys... My mistake. One migration file had spaces before the <?PHP tag.
This one was hard to find. Spend a few hours to find out.

Glad you figured it out 😊


v1.11 backports 2023-02-18

[ ] #23763 -- envoy: Avoid empty typeURL for all resources (@sayboras)

Due to breaking change of NetworkPolicy (i.e. name attribute is removed), API update requires backport one more commit.

Once this PR is merged, you can update the PR labels via:
$ for pr in 23763; do contrib/backporting/set-labels.py $pr done 1.11; done

/test-backport-1.11

All required jobs are succesfully ran, aditional review is also in. Marking this ready to merge so that tophat can take a look as well.


chore: add 14.18.1 base image

Hello @flotwig what else is needed to get this PR merged ?

Not yet?


Update to 2.0.0.

Checklist

[x ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy
[x] Ensured the license file is being packaged.

Restricts r-base to <3.5 since the necessary dependencies are not available for R 3.5 yet.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Great! Thanks a lot :-)


Upgrade pyarrow dependency to 14.0.1

See:

CVE-2023-47248
pyarrow-hotfix

My project depends on delta-lake-reader, and I cannot upgrade pyarrow to a version which mitigates the above vulnerability:
ERROR: Cannot install delta-lake-reader==0.2.15 and pyarrow==14.0.1 because these package versions have conflicting dependencies.

The conflict is caused by:
    The user requested pyarrow==14.0.1
    delta-lake-reader 0.2.15 depends on pyarrow<10.0.0 and >=9.0.0

@Goldensunboy Published a version with updated pyarrow version. Hope it solved your issue


README.md: logo is broken

 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.

Thanks @joedf, I had actually fixed it on the other side of it back then, but really appreciate the contribution! I think your change is great as more resilient than an external domain, however as you did not sign our CLA I'm not able to merge this.

Thanks @joedf, I had actually fixed it on the other side of it back then, but really appreciate the contribution! I think your change is great as more resilient than an external domain, however as you did not sign our CLA I'm not able to merge this.

It's signed now.

It's signed now.

/rebase

/rebase


AUI 2043 added test

i left a few comments on one of the commits so you can take a look at those. other things i noticed was that your first two commits have the same commit message.
it looks like the second commit added a test, but also did a bunch of general source formatting? you should probably separate those things into two different commits.
then the last commit says "added test", but youre actually just making a small change/source formatting the test. so the commit messages are a little confusing.
basically you should have 3 commits like this...
AUI-2043 when "required" is included in a set of multiple validators, make it the default validator
AUI-2043 added test
AUI-2043 general source formatting
the last thing is to make sure to not commit the project files.
ill go ahead and close this pull request so that you can make those changes and send a new one. if you need help with git to reconstruct the commits etc let me know... and i think steven should also be able to help if you want to ask him


Can't build with Mirror Network Profiler imported

Describe the bug
The title says it, if you have Mirror Profiler imported in the project, you can't make a build which is annoying because you have to remove and import it.
Expected behavior
Get the build :)

Also getting this, i have to remove and then import again after.
@RamiroOliva  Could you post the red console log here?  If i remember correctly, its something about UI.

See if this fixes the issue...

See if this fixes the issue...

This worked.
Should we export with the new hierarchy and pin the adjusted version in Discord/Sponsor section?

Not sure we can right now.  The above is in pinned discord msg. Closing this.


Service protection API limits link broken

The link to the article "Service protection API limits" is broken.
The link used in the article is this:
https://docs.microsoft.com/en-us/powerapps/developer/data-platform/api-limits.md
The correct link I think is this:
https://docs.microsoft.com/en-us/powerapps/developer/data-platform/api-limits

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

ID: 51381035-2f12-61f5-0f18-3c8b7c36dd55
Version Independent ID: 1981a463-4ef8-0111-88e4-8ee912436000
Content: Priority-based throttling - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/data-entities/priority-based-throttling.md
Service: dynamics-365-cross-app
Product: ****
Technology: ****
GitHub Login: @hasaid
Microsoft Alias: sunilg

@elcanelo, Thanks for providing this feedback. I'm not seeing the broken link in the Priority based throttling topic. Can you point out where you found the broken link?

The broken link is with in the Dual-write limits for live synchronization topic.
Specifically under the From Finance and Operations apps to Dataverse table. There is a link titled Service protection API limits which if clicked shows a 404 error.

@elcanelo, thank you for the information. The link should be fixed now.


Fixed build warnings

Dummy test wasn't actually running because it wasn't marked as a [Fact]. Once I added that, it failed because the shader was not found. I am pretty sure that the one I chose is the correct shader.

Single - This should be used instead of Equal to ensure a collection only has one element. Warning raised by xunit.

Internal to Public changes - First, it's an easy fix for the warning that a field will never get set. Second, both of these are part of the unit tests, so their visibility doesn't much matter.

All UnitTests pass

DummyTest is actually not a real test -- it's just something I use to run one-off tests on Mac or Linux, because there's no proper test explorer there. I'd be fine with just deleting the test actually.
The other stuff looks good to me.

Dummy test removed!

@CloneDeath Looks good now. Thanks for the fixes!


feat: add cli structure and run command

Description

References

Review Checklist

[ ] I have added documentation for new/changed functionality in this PR or in openfga.dev
[ ] The correct base branch is being used, if not main

Codecov Report

Merging #42 (2b864f2) into main (cda3998) will decrease coverage by 0.29%.
The diff coverage is 14.60%.

@@            Coverage Diff             @@
##             main      #42      +/-   ##
==========================================
- Coverage   76.66%   76.37%   -0.30%
==========================================
  Files          69       72       +3
  Lines        8022     8045      +23
==========================================
- Hits         6150     6144       -6
- Misses       1452     1479      +27
- Partials      420      422       +2

Impacted Files
Coverage Δ

cmd/openfga/main.go
0.00% <0.00%> (ø)

pkg/cmd/root.go
0.00% <0.00%> (ø)

pkg/cmd/run.go
0.00% <0.00%> (ø)

pkg/cmd/service/service.go
82.30% <86.66%> (ø)

server/commands/check.go
84.08% <0.00%> (-1.51%)
:arrow_down:

storage/postgres/utils.go
83.43% <0.00%> (-0.64%)
:arrow_down:

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 cda3998...2b864f2. Read the comment docs.


airline colorschema disappears after jedi finishing codes indication, how to fix it?

environment

vim: 7.4.52
vim-airline: v 0.8
OS: ubuntu 14.04 amd64
terminal: konsole

actual behavior
coding in Python, and the plugin jedi which is shipped by vim-airline can automatically show some coding indications
expected behavior
want the colorschema does not disappear after finishing auto-filling.

I found that the airline would come back if I jump between different windows in Vim using Ctrlwn   start a new window, jump out and jump back...
I think this is because the jedi does not release the window focus or vim-airline does not capture the latest window.

here is two conditions that vim-airline the toolbar would come back

jumping between different windows in Vim

re-sizing the terminal window since the vim-powerline would automatically recognize the re-sizing motion

after jedi giving grammar guides:

after re-sizing my terminal or jumping windows between main window or NERDTree.

how to fix this?

this problem was solved.
here's the reference.
https://github.com/vim-airline/vim-airline/issues/78#issuecomment-21161085
and here's my vim-airline settings in .vimrc
" airline settings
let g:airline#extensions#tabline#enabled = 1
let g:airline#extensions#tabline#left_sep = ' '
let g:airline#extensions#tabline#left_alt_sep = '|'
let g:airline_powerline_fonts = 1
let g:airline_exclude_preview=1

this line let g:airline_exclude_preview=1    fix this issue.
Hope this could be helpful for others who have the same issue. :smiley:


The "Unifying Return Type" example is not clear

What is the type of issue?
Documentation is confusing
What is the issue?
In the following sentence, the code example below does not properly convey what "struggles to unify the return type" means

Unifying Return Type. In situations where TypeScript struggles to unify the returned effect type, Effect.suspend can be employed to resolve this issue. For example:

import { Effect } from "effect"

const ugly = (a: number, b: number) =>
  b === 0
    ? Effect.fail(new Error("Cannot divide by zero"))
    : Effect.succeed(a / b)

const nice = (a: number, b: number) =>
  Effect.suspend(() =>
    b === 0
      ? Effect.fail(new Error("Cannot divide by zero"))
      : Effect.succeed(a / b)
  )

Where did you find it?
https://effect.website/docs/guides/essentials/creating-effects

🤦♂I have somehow read this far into the documentation without hovering the mouse over any code block, so I didn't realize you could just hover over the symbols to see which type they have!

The difference is the return type (visible on hover over ugly and nice).
Ugly:
Effect.Effect<never, Error, never> | Effect.Effect<number, never, never>

Nice:
Effect.Effect<number, Error, never>

Unifying, here, means that the two diverging / union return types are merged into one.


Feature/update earthkit plots

This PR:

revamped the polytope examples
published the polytope examples
updated the version of earthkit-data and earthkit-plots


Weights for m & l are for resolution 224

I get this error when I try to put image with resolution 384 into m or l models :
Input 0 is incompatible with layer model_8: expected shape=(None, 224, 224, 3), found shape=(1, 384, 384, 3)

If you load model by:
model = tf.keras.models.load_model('../models/efficientnetv2-s-21k.h5')

The input shape is fixed at 224, 224, 3. If you need a new resolution, can set input shape as dynamic:
# Define model and load weight
import efficientnet_v2
model = efficientnet_v2.EfficientNetV2(model_type="b0", input_shape=(None, None, 3), survivals=None, dropout=0.2, classes=1000, classifier_activation=None)
model.load_weights('../models/efficientnetv2-b0-imagenet.h5')

model(np.ones([1, 224, 224, 3])).shape
# TensorShape([1, 1000])
model(np.ones([1, 384, 384, 3])).shape
# TensorShape([1, 1000])

It's working thank you very much! Just a little thing: when loading the weights in this configuration, tf expects the model to have around 21000 classes, but I've found a workaround: define the model with around 21000 classes, load the weights, then put a new head onto it with the right number of classes.

That can be done easier, by using by_name=True, skip_mismatch=True, and it will report those layers mismatched:
import efficientnet_v2
model = efficientnet_v2.EfficientNetV2L(input_shape=(None, None, 3), survivals=None, dropout=1e-6, classes=1000)
model.load_weights('../models/efficientnetv2-l-21k.h5', by_name=True, skip_mismatch=True)
# WARNING:tensorflow:Skipping loading of weights for layer predictions due to mismatch in shape ((1280, 1000) vs (1280, 21843)).
# WARNING:tensorflow:Skipping loading of weights for layer predictions due to mismatch in shape ((1000,) vs (21843,)).

model(np.ones([1, 772, 772, 3])).shape
# TensorShape([1, 1000])


[pubsub/jetstream] Add support for concurrencyMode

Description
This adds support for the concurrencyMode metadata field which will send spawn a new goroutine per message like the RabbitMQ implementation.
Issue reference
n/a
This was requested by a user of NATS/JetStream to support this parallel mode. /cc @Moscagus
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

Thanks for the quick review. Addressed the comments and rebased the branch.

@bruth thanks for the PR.

@bruth for reasons of high availability and scalability, I use at least 2 pods per application. Because of this I must configure the "DeliverGroup" ("queueGroupName") property in "push" mode.
Looking at the "PR" changes I see that "concurrencyMode" is not applied when the "queueGroupName" is set.
Therefore, at the scalability level, in my use case I remain the same as before the change. That is, if I want to parallelize the app by 100, I must upload 100 pods.
Is it possible to make the same change to ?:
subscription, err = js.jsc.QueueSubscribe(req.Topic, queue, natsHandler, nats.Bind(streamName, consumerInfo.Name))
Thanks

Bah, that was an oversight.. sorry about that, will open another PR in a minute.

Here is the new PR: https://github.com/dapr/components-contrib/pull/3232

@bruth thanks for the new PR.

@Moscagus new PR just got merged, FYI.

@bruth thanks


go-metro integration

Hi!
Documentation states that "The check is currently not available with Datadog Agent v6.".
I really missed this integration since I migrated my project to using datadog-agent v6.

Where is go-metro?


Demo predictions and Node JS App predictions are not the same result

nsfwjs.com (load model: inceptionv3: ['/model/', { size: 299 }]) image result:
Identified as Porn
Porn - 48.19%
Neutral - 40.21%
Sexy - 10.87%
Hentai - 0.61%
Drawing - 0.12%

NodeJS app (load model: inceptionv3: ['/model/', { size: 299 }]) same image result:
[
    {
        "className": "Neutral",
        "probability": 0.6607071757316589
    },
    {
        "className": "Sexy",
        "probability": 0.18770809471607208
    },
    {
        "className": "Porn",
        "probability": 0.14371605217456818
    },
    {
        "className": "Hentai",
        "probability": 0.004498853348195553
    },
    {
        "className": "Drawing",
        "probability": 0.0033697152975946665
    }
]

My Code:
const express = require('express')
const multer = require('multer')
const jpeg = require('jpeg-js')

const tf = require('@tensorflow/tfjs-node')
const nsfw = require('nsfwjs')

const app = express()
const upload = multer()

let _model

const convert = async (img) => {
  // Decoded image in UInt8 Byte array
  const image = await jpeg.decode(img, true)

  const numChannels = 3
  const numPixels = image.width * image.height
  const values = new Int32Array(numPixels * numChannels)

  for (let i = 0; i < numPixels; i++)
    for (let c = 0; c < numChannels; ++c)
      values[i * numChannels + c] = image.data[i * 4 + c]

  return tf.tensor3d(values, [image.height, image.width, numChannels], 'int32')
}

app.post('/nsfw', upload.single("image"), async (req, res) => {
  if (!req.file)
    res.status(400).send("Missing image multipart/form-data")
  else {
    const image = await convert(req.file.buffer)
    const predictions = await _model.classify(image)
    image.dispose()
    res.json(predictions)
  }
})

const load_model = async () => {
  _model = await nsfw.load('https://nsfwjs.com/model/', {size: 299})
}

// Keep the model in memory, make sure it's loaded only once
load_model().then(() => app.listen(8080))

I tried the model files locally, same result.

Have you tried tfjs-node to decode the images?
Just for info.  Which one of the two was correct?  I assume the website was correct, and the node was wrong?

https://github.com/infinitered/nsfwjs/discussions/540


Recursive functions in inner scopes are not possible

Function initialization have to be able to be recursive outside the outer scope.

Gonna guess this is what you mean:
start :: fn do
    a :: fn do
        a'
    end
end


Add a Codacy badge to README.md

Coverage remained the same at 56.72% when pulling 045c52b959de6a89b7b87a681ac54e0c1f485b54 on codacy-badger:codacy-badge into 7c460d61467faf64ec761417f84bba8487dbd21b on onnovalkering:master.