S3 CRT Client

Reverts slackhq/kaldb#609.
⚠ Depends on NO_PROXY support before this can be merged - https://github.com/awslabs/aws-c-http/issues/413.
Today there exists no way to initialize the S3 CRT client from Java using no proxy, if the HTTP_PROXY/HTTPS_PROXY environment values are set. These are directly read from native C code when an empty proxy config is detected.
Attempting to initialize the proxy with null values does not bypass this behavior, nor does attempting to initialize the proxy with an empty string (throws errors in native code). Attempting to set the http.proxy/https.proxy Java values to empty also does not stop reading the ENV values, as this is also detected as an empty proxy config. The only way to avoid this behavior is to completely unset the proxy values from the system environment values, as mentioned in the issue above.
Initial benchmarks with this code (even running through the proxy) showed download speeds took less than 50% of the original time. As soon as some form of no_proxy support is added, or there exists a way to force disable reading of the env variables this should be revisited.

We have a workaround in place for the time being, using a squid sidecar and then providing the env variables aws.s3.crt.proxy.scheme / aws.s3.crt.proxy.host / aws.s3.crt.proxy.port. This seems to be working as expected, and we can revert this portion once the fix is rolled into the upstream.


Set LAPACK path

Trying to avoid linking problems with OSX Accelerate framework.
Suggested by @isuruf in #3

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.


[#18] - defer removal of flash message

[x] Prevent flash message removal on mouse over per https://github.com/poteto/ember-cli-flash/issues/18
[x] Resume exit timers with remaining time per https://github.com/poteto/ember-cli-flash/pull/28#issuecomment-81991231
[x] Tests

[x] Add unit test for deferring removal
[x] Add integration test for mouseover events

@poteto Do you think this in a good place to merge? I'd love to close out that issue.

LGTM


[wip]: adding transfer fee upgrade integration test

Description
Adding the scaffolding here in draft. Please do not review yet.
closes: #3993

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[ ] Provide a commit message to be used for the changelog entry in the PR description for review.
[ ] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

closing this for now, I'll ressurect the branch when the time comes


iOS - Expense Request - App crashes when the expense is paid by the admin in OD

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v9.0.79-0
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Yes, reproducible on both
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5381717&group_by=cases:section_id&group_order=asc&group_id=296765
Issue reported by: Applause Internal Team
Device used: iPhone 13
App Component: Money Requests
Action Performed:
Preconditions:
Must be executed in a Collect workspace. We need:
Admin account
Employee account
The OD steps can be executed in a web browser, no need to execute them in the main testing device.
Employee - NewDot

Navigate to the workspace chat and Submit expense
Navigate to the report conversation
Send a message in the report conversation

Admin - OldDot

Log in as the admin in OldDot on another device
Navigate to the report from the Collect workspace. Should be in processing status
Reimburse the report (use the pay elsewhere option)

Employee - NewDot

Navigate to the paid report

Expected Result:
The app does not crashes, the employee can see the system message that the admin paid the report elsewhere
Actual Result:
App crashes when the expense is paid by the admin in OD
Workaround:
Unknown
Platforms:

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[x] iOS: Standalone
[x] iOS: HybridApp
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/746b978c-af30-4104-9d12-fec8a7c6e4d8
Bug6702756_1735268832248!My_iPhone_I_Console.txt
View all open jobs on GitHub

Working part time through the holidays and prioritizing customer issues. Will look at this ASAP


switchoff function of ws2812b just switches off the first n lights

I have an LED stripe with 240 leds which I trz to controll with this class. But using the switchoff function just switches off the first few lights.

@thigg still need help? or can be closed?

Closed due to age. Please open a new one, if problems still exists.
Thank you


What's your favorite ice cream flavor :ice_cream: :ice_cream: ?

@acabunoc

Thanks for making a good first bug! I'm wondering, how will this contribute to your project?

Hello @ankuhl  , I am Vasant Joseph. I am a class 12 student. I have skills in designing(Photoshop, premiere pro), presentation, and anything which is given to me, which I can do. I am sincere to my work. If there is anything which I can do to help, please inform me. This is my first time at Mozilla global sprint.
I have experience by participating in a variety of hackathons.
I am interested to help.

Hello @DigitalGrassroots  , I am Vasant Joseph. I am a class 12 student. I have skills in designing(Photoshop, premiere pro), presentation, and anything which is given to me, which I can do. I am sincere to my work. If there is anything which I can do to help, please inform me. This is my first time at Mozilla global sprint.
I have experience by participating in a variety of hackathons.
I am interested to help.

i had done a research on internet governence in India, UK, And USA. If you want I can share the research.

Hello @VasantJ3 ! Welcome to your first Global Mozilla Sprint and we are excited you would like to contribute to @DigitalGrassroots as it is our first time too. I m a team lead and here to help you.
We could really use your design skills for a cover page and some illustrations as we are working on an outcome report from our first Ambassadors program. Please shoot us an email at<EMAIL_ADDRESS>for further guidelines.
Welcome to Mozilla Sprint!


Class based view and base.html

@aasis21
I'm replacing the functional views with class based views as it would be better for post and get methods separately.
Also. We need a base.html . Its not a good habit to repeat code.

We are not repeating code,index.html is being extended in all template.

That is the same thing bro. base.html is conventionally extended . Not index.html

@lordaniket06
Fine man , change it..


Build cleanup/modernization

Use Directory.* props, remove explicit imports and other redundancies from project files.

Add Directory.Build.rsp so that msbuild behaves as dotnet build with parallelism and minimal verbosity on by default.

Simplify build props as much as possible while moving things around. For example, remove unnecessary conditions.

Use official artifacts layout feature to move all artifacts to bld. I am still getting used to some of the choices it makes here myself, but I think we should give it an honest try before abandoning it because it makes life much better to not have to battle msbuild ourselves every time a new build artifact ends up outside the bld\ folder.

Put TargetFramework(s) directly in csproj files. Everything in VS breaks without TargetFramework and that's really annoying when working on the build and there's something off. The DRY value here is too low to be worth it, IMHO.

Enable nullable and implicit usings throughout

Actually reference Nerbank.GitVersion and Microsoft.SourceLink.GitHub. There was a PackageReference Update that wasn't updating anything.

Enable .NET Analyzers

Enable latest analysis level since we're clean on .NET 9 SDK analysis in this repo.

Update .NET Framework min version from 4.6.2 to 4.7.2. There are various things fixed in tooling by this and I seriously doubt that any partner is actually running on anything lower in 2024. 4.7.2 was released more than six years ago and every Windows release since has that or higher in-box.

Update .gitignore to latest dotnet new .gitignore template + custom exclusion for Directory.Build.rsp

PR validation improvements:

Use release config.
Split restore into its own step. Helps when tracking build issues over time to see which step is actually failing easily, and which step is slowing down builds.
Don't build again for test step.

Hold off on merging, I'll fix the nits.


DeadObjectElimination: delete dead arrays for which the "destroyArray" builtin is inlined.

This is needed for non-OSSA SIL: in case we see a destroyArray builtin, we can safely remove the otherwise dead array.
This optimization kicks in later in the pipeline, after array semantics are already inlined (for early SIL, DeadObjectElimination can remove arrays based on semantics).
rdar://73569282
https://bugs.swift.org/browse/SR-14100

@swift-ci test

@swift-ci benchmark

Performance: -O

Regression
OLD
NEW
DELTA
RATIO

FlattenListLoop
1653
2514
+52.1%
0.66x (?)

FlattenListFlatMap
4821
6695
+38.9%
0.72x (?)

NSStringConversion.MutableCopy.Rebridge
810
954
+17.8%
0.85x (?)

ObjectiveCBridgeFromNSSetAnyObjectToStringForced
88000
95500
+8.5%
0.92x (?)

Code size: -O
Performance: -Osize

Improvement
OLD
NEW
DELTA
RATIO

FlattenListFlatMap
6666
6097
-8.5%
1.09x (?)

Code size: -Osize
Performance: -Onone
Code size: -swiftlibs

How to read the data
The tables contain differences in performance which are larger than 8% and
differences in code size which are larger than 1%.
If you see any unexpected regressions, you should consider fixing the
regressions before you merge the PR.
Noise: Sometimes the performance results (not code size!) contain false
alarms. Unexpected regressions which are marked with '(?)' are probably noise.
If you see regressions which you cannot explain you can try to run the
benchmarks again. If regressions still show up, please consult with the
performance team (@eeckstein).

Hardware Overview
  Model Name: Mac Pro
  Model Identifier: MacPro6,1
  Processor Name: 12-Core Intel Xeon E5
  Processor Speed: 2.7 GHz
  Number of Processors: 1
  Total Number of Cores: 12
  L2 Cache (per Core): 256 KB
  L3 Cache: 30 MB
  Memory: 64 GB

@swift-ci smoke test and merge

Hi,
I was looking at why the Android CI has been failing since https://ci-external.swift.org/job/oss-swift-RA-linux-ubuntu-16.04-android/9117/ and it seems to be because of the changes introduced in the test in this commit. No need to fix it or anything, I am just trying to understand the changes, and how it might be have affected only Android.
This is the failure.
******************** TEST 'Swift(android-armv7) :: SILOptimizer/dead_array_elim.swift' FAILED ********************
Script:
--
: 'RUN: at line 1';   /home/ubuntu/jenkins/workspace/oss-swift-RA-linux-ubuntu-16.04-android/buildbot_linux/swift-linux-x86_64/bin/swift-frontend -target armv7-unknown-linux-android -I /home/ubuntu/android-ndk-r19c/sysroot/usr/include -I /home/ubuntu/android-ndk-r19c/sysroot/usr/include/arm-linux-androideabi -L /home/ubuntu/android-ndk-r19c/sources/cxx-stl/llvm-libc++/libs/armeabi-v7a -L /home/ubuntu/android-ndk-r19c/toolchains/arm-linux-androideabi-4.9/prebuilt/linux-x86_64/lib/gcc/arm-linux-androideabi/4.9.x -L /home/ubuntu/android-ndk-r19c/toolchains/arm-linux-androideabi-4.9/prebuilt/linux-x86_64/arm-linux-androideabi/lib  -module-cache-path /home/ubuntu/jenkins/workspace/oss-swift-RA-linux-ubuntu-16.04-android/buildbot_linux/swift-linux-x86_64/swift-test-results/armv7-unknown-linux-androideabi/clang-module-cache -swift-version 4  -ignore-module-source-info -typo-correction-limit 10  -O -emit-sil -primary-file /home/ubuntu/jenkins/workspace/oss-swift-RA-linux-ubuntu-16.04-android/swift/test/SILOptimizer/dead_array_elim.swift | /usr/bin/python3.5 /home/ubuntu/jenkins/workspace/oss-swift-RA-linux-ubuntu-16.04-android/swift/utils/PathSanitizingFileCheck --sanitize BUILD_DIR=/home/ubuntu/jenkins/workspace/oss-swift-RA-linux-ubuntu-16.04-android/buildbot_linux/swift-linux-x86_64 --sanitize SOURCE_DIR=/home/ubuntu/jenkins/workspace/oss-swift-RA-linux-ubuntu-16.04-android/swift --use-filecheck /home/ubuntu/jenkins/workspace/oss-swift-RA-linux-ubuntu-16.04-android/buildbot_linux/llvm-linux-x86_64/bin/FileCheck  /home/ubuntu/jenkins/workspace/oss-swift-RA-linux-ubuntu-16.04-android/swift/test/SILOptimizer/dead_array_elim.swift
--
Exit Code: 1

Command Output (stderr):
--
/home/ubuntu/jenkins/workspace/oss-swift-RA-linux-ubuntu-16.04-android/swift/test/SILOptimizer/dead_array_elim.swift:58:16: error: CHECK-NEXT: is not on the line after the previous match
// CHECK-NEXT: integer_literal $Builtin.Int{{[0-9]+}}, 21
               ^
<stdin>:99:8: note: 'next' match was here
 %18 = integer_literal $Builtin.Int32, 21 // user: %19
       ^
<stdin>:82:55: note: previous match ended here
 debug_value %0 : $String, let, name "stringParameter", argno 1 // id: %1
                                                      ^
<stdin>:83:1: note: non-matching line after previous match is here
 %2 = integer_literal $Builtin.Int32,<PHONE_NUMBER> // user: %3
^

Input file: <stdin>
Check file: /home/ubuntu/jenkins/workspace/oss-swift-RA-linux-ubuntu-16.04-android/swift/test/SILOptimizer/dead_array_elim.swift

-dump-input=help explains the following input dump.

Input was:
<<<<<<
         .
         .
         .
        94:  %13 = struct $String (%12 : $_StringGuts) // users: %16, %15
        95:  retain_value %0 : $String // id: %14
        96:  release_value %13 : $String // id: %15
        97:  release_value %13 : $String // id: %16
        98:  release_value %0 : $String // id: %17
        99:  %18 = integer_literal $Builtin.Int32, 21 // user: %19
next:58            !~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~              error: match on wrong line
       100:  %19 = struct $Int (%18 : $Builtin.Int32) // user: %20
       101:  return %19 : $Int // id: %20
       102: } // end sil function '$s15dead_array_elim31testDeadArrayAfterOptimizationsySiSSF'
       103:
       104: // static Array._adoptStorage(_:count:)
         .
         .
         .
>>>>>>

--

********************

From my understanding of the test (and the JIRA ticket), it seems that the final value 21 is indeed created, but there's seems to be something in between constructing the return value and returning it, and the debug_value line. Those lines are retain_value/release_value, which seems like something is being constructed and destroyed (maybe what these changes tried to remove).
This is Android 32 bits (64 bits works fine), and if I understand correctly Darwin doesn't have a lot of 32 bits platforms lying around anymore, but https://github.com/apple/swift/pull/35737/files#diff-c34c7dab63c3329cf8671662271a8d64c1ee918eff414b91196fe5df89883265R259 looks like something that might not be true for a 32 bit platform.
Any insight will be appreciated. If nobody have any idea, I can simply disable the test with an XFAIL.

@drodriguez Thanks for letting me know!
I fixed it in https://github.com/apple/swift/pull/35776


Suggestion: limit output to N countries

I'm mostly interested in looking at the progression of the disease in the current top 10 countries. Would it be possible to specify a cutoff number, where only the top N results are shown?
For example: corona --sort cases -n 10

I welcome your contributions on that.

Done.

Implemented. Update the CLI: npm i -g corona-cli


add new font-face mixin with less formats required

basic support should be available with .woff and .otf formats, .eot is an optional extra for IE8 and below, and .woff2 can be added optionally for modern browsers
this also means we won't need to @extend fonts anymore and overwriting them should therefore be much easier
@incuna/frontend please review (I've tested it locally in a project and it seems to work fine)

@grahamgilchrist more examples added, is the css output ok?

Nice docs :)

@hippogriffic  merge conflicts.

master merged

looks good !

Did we check the CSS output when we have the eot option on?

I checked it in chrome, but chrome doesn't recognise .eot so it doesn't even try to find it... will have to check in browser stack when we use it in a project

yeah, I just mean look in the generated CSS to see if it outputs two src lines or not with that option on. Might break IE8 if it does.


chore(whale): add build support

What this PR does / why we need it:
This enables whale to be built successfully
A few changes has to be made to make this work as expected:

Changed all /src imports to /dist imports.

The reason for this is because /src does not exist after running lerna build, and since the imports themselves drilled into non existent /src import errors occurred after build process completed.
Now builds run fine with /dist, but since /dist does not exist during testing we have to perform another work around by pathing any packages with /dist import to /src. Hence the added bespoke /dist targets in tsconfig and jest.config

Changed src/main to src/index

Since docker builds targets the package build src and not a specific file index has to be used for automatic resolution.
See Dockerfile

Which issue(s) does this PR fixes?:

Partially solves

https://github.com/DeFiCh/jellyfish/issues/977

Additional comments?:
Follow up for PR:

https://github.com/DeFiCh/jellyfish/pull/1272

The custom pathing for whale /dist targets is not ideal but until there is better support for this sort of behavior inherent in jest and typescript this is the only way I see this working for now. Another potential work around would be to refactor how imports are used to remove the /dist requirement, such as:
import { address } from '@defichain/whale-api-client'

// access namespace functionality from exported (* as address)
// address.AddressToken
// address.AddressHistory

or change our packages to export without casting
export * from './api/Address'
// instead of
export * as address from './api/Address'

I think until we have a defined way we want to approach this, the following work around should suffice to get the beginning of our whale build steps in jellyfish operational.

Biggest concern is the mapper stuff which we can refactor after unblocking the build & release process and migrating everyone from whale into jellyfish/apps/whale.

Yes, to recap some conversation I am having with @eli-lim we are thinking:

Merge this PR with mapped workarounds to get builds successfully working.
Create release workflow PR to make whale part of jellyfish releases.
Officially move all development efforts from whale to jellyfish/whale.
Start further refactoring of jellyfish whale, restructuring and fixing imports (and in turn remove mapping workarounds)

This should get us fully migrated.


[QUERY] Can I define a custom query that returns a single or Optional entity object?

Query/Question
I want to define a custom query using @Query annotation that I know will return either one document or none. I tried to define the Repository interface like below.
  @Query(       "SELECT * FROM c WHERE c.partitionKey = @partitionKey AND c.someField = @someField")   Optional<SomeEntityClass> find(       @Param("partitionKey") String partitionKey,       @Param("someField") String someField);
But I got the following exception when I run a test.
Caused by: java.lang.ClassCastException: class java.util.ArrayList cannot be cast to class xxx.SomeEntityClass (java.util.ArrayList is in module java.base of loader 'bootstrap'; xxx.SomeEntityClass is in unnamed module of loader 'app')
I also tried change the method to return SomeEntityClass instead of Optional, it gives the same error. If I change the return type to List, then it works fine.
I suspected the current azure spring data cosmos version only support to return a List instead of a single object or Optional. Is it the case?
Why is this not a Bug or a feature Request?
Not sure if this is a bug or just a missing feature or something I did is wrong.
Setup (please complete the following information if applicable):

OS: Windows 10
IDE: Eclipse
Library/Libraries: com.azure.spring:azure-spring-boot-starter-cosmos:3.6.0

At this link in the "Native SQL Select @Query with Index Parameters" section you can see a SELECT *... that returns an individual BOOK object. So I am assuming if multiple results are found this query is just returning the first record.
@kushagraThapar I will look into this unless you think this is bad practice for our API.

@trande4884 returning just the first object is not intuitive, also, based on the query execution, the returned list can have any ordering, which will cause the query to return different results every time the query is executed in case it has more than one return values.
@zhoujia1974  - I don't think it is possible to return any other type than list for any queries for cosmos db. For optional return types, we can figure it out based on the return type, however, for the above reason it won't be ideal to do so.

Closing this issue because of inactivity. @zhoujia1974 please feel free to open it if your problem still persists.


feat: support for basic satisfy logic

Problem: we want to be able to submit a jobspec and do a satisfy request against the graph.
Solution: this WIP adds the parsing of a jobspec (from simple parameters into the Jobspec, which is a library proper, and in the future we can have converters from other kinds of job files or even Kubernetes abstractions). The Jobspec goes from the calling client directly to the graph database, which is a GRPC endpoint, and that logic is exposed from the graph database interface. Since this is an in-memory database provided directly by rainbow, this means it is technically still hitting the rainbow server, but in the future this can be a separate graph database GRPC endpoint. Note that auth will need to be worked out for when the graph is separate - likely when a cluster registers it will have permission/auth as part of the schema. As is implemented here, we then hit the actual in memory graph, and run a Satisfy -> DFSForMatch function, which right now is just doing a top level check for resources of each known cluster against the Jobspec. I am doing this because I do not yet know what the DFS should look like (I need to look at Fluxion code) but the response returns a simple list of clusters that can satisfy the resources from a global (totals) sense. We also need to think about what information about the match (beyond the cluster name) should go back to the calling client to be sent to rainbow. Note that the client is handling these two requests, and the user is just making one request with their job.
In the picture below, rainbow (the server with the graph database paired) is running on the left, and the client on the right. You can see (on the right) that I register and submit two jobs. The first is within the resources of the cluster I registered (and accepted) and the second is not (no matches).

Small update - more detailed output for checking the attributes, and I added a recursive function call to recurse into the jobspec (which was not properly happening before). Go doesn't have the ability to do "while (for) there are objects in this array" because the length of the array is pre-determined. So this recursive approach is probably best.

This is good. Merging to work on subsystem registration.


Flyway upgrade from 9.16.0 to 10.10.0 fails flyway migration

Which version and edition of Flyway are you using?
I am using Flyway 10.10.0
If this is not the latest version, can you reproduce the issue with the latest one as well? (Many bugs are fixed in newer releases and upgrading will often resolve the issue)

Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
sbt 1.7.1
"org.flywaydb" % "flyway-core" % 10.10.0
java version: 11
Which database are you using? (Type & version)
PostgreSQL 42.7.1
Which operating system are you using?
Linux
What did you do? (Please include the content causing the issue, any relevant configuration settings, the SQL statement(s) that failed (if any), and the command you ran)
I'm trying to upgrade the Flyway version from 9.16.0 to 10.10.0.
After updating the version I get the following compiler error:
java.lang.UnsupportedClassVersionError: org/flywaydb/core/Flyway has been compiled by a more recent version of the Java Runtime (class file version 61.0), this version of the Java Runtime only recognizes class file versions up to 55.0

Note: same code with flyway 9.16.0 working fine
flyway code in scala:
Fway
  .configure()
  .dataSource(config.url, config.username, config.password)
  .locations(config.location)
  .schemas(config.schema)
  .baselineOnMigrate(true)
  .table("flyway_schema_history")
  .load()

What did you expect to see?
I expected the migration to go smoothly
What did you see instead?

Hi @Bekturdi thank you for your issue. This is because Flyway is compiled for Java 17 since version 10.0.0, please see the release notes for the Java Compatibility. Please see the API documentation page for the minimum version of Java that is now supported by Flyway.

@Barry-RG Thanks for the quick reply!


AppendNet does not exist in Kicad 8

Hi @gauravmm,
we discovered another small bug: in KiCad 8, this AppendNet is no longer available.
You can see it here:
Kicad 7 API
Kicad 8 API
When commenting out this line, it still works though. Maybe it is not needed anymore?
Best
Hannes

will be fixed by #7 (sorry for the noise, hadn't read the comments over there)

No problem, thanks for the report!


hotfix processes: update the contributing guide

When twgit hotfix start is used, twgit fire a new branch, based on the last commit of stable branch. Creating, for instance, a branch named  hotfix/0.2.3
Then, when twgit hotfix finish is used, twgit merge the hotfix branch into stable, tag it, etc.
Problem is: When multiple hotfix PR are created in parallel, the async party begins, and a hotfix 0.2.5 could be mergeable before 0.2.3.
So, with hm-admin here is the hotfix process:
twgit feature start fix-blablabla
gaa && gc...
g push myfork
...
review
merge
...
twgit hotfix start
twgit feature merge-into-hotfix fix-blablabla
twgit hotfix finish

ps ironic note: I appreciate github warning me with this message while I’m filling this issue:

Please review the guidelines for contributing to this repository.

So now, it’s not just async party time, it’s circular reference party time too! 🎉 🎈 🎂

Twgit should stay an intern process. We can accept external contribution which not using it so IMO, we should put that in the contributing file

I think what you want to say is: “we should not put that in the contributing file”.
But ok, I agree, let’s close this issue ;)


Fix to throw errors properly

Thank you for maintaining, I love this library :)
I fixed a promise error handling, could you review it?
If a promise causes an error, the error is not thrown. Then it causes a timeout error.
const assert = require('assert');
const parallel = require('mocha.parallel');

parallel('test', () => {

  it('should throw an AssertionError', () => {

    return Promise.resolve()
      .then(() => assert(false));
  });
});

// v0.15.1
  1) test should throw an AssertionError:
     Error: Timeout of 2000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.

// ↓
  1) test should throw an AssertionError:

      AssertionError: false == true
      + expected - actual

      -false
      +true

Good catch! Added a failing test in https://github.com/danielstjules/mocha.parallel/commit/908b29e751e9ed776a5668a5d55594310718ab85 that's fixed by your PR. Thanks! :)

Just published 0.15.2 !

Thanks!


php8.2 based on @jbboehr's branch

Stack created with [Sapling]

-> #108

php8.2 based on @jbboehr's branch

Python and PHP both broke separately, so merging both even though neither are passing CI


Works with ST3

Seems to be working with version 3 as well :) Maybe Readme should be updated?

Good point, thanks 😄. I've updated the readme now to mention the versions supported.


Testing out chef-provisioning 1.2 and knife-windows 1.0 RC

\cc @chef/client-core

This is the only way forward

yep

Bring 'em all together!


yoman and vs errors during installation (spa services ver 0.7.2)

To repeat

Open VS 2017 RC , create test solution (creating class lib project)
Open console, go to the solution folder create folder  for test JavaScriptServices folder e.g. "TestJavaScriptServices " go to it.
Run yo aspnetcore-spa choose "react + redux" (as it is described in http://blog.stevensanderson.com/2016/05/02/angular2-react-knockout-apps-on-aspnet-core/)

Expected result:
yoman works without errors
Observerd result
yoman error:

Other info:
npn -v
3.10.10
dotnet --version
1.0.0-preview4-004233

My C:\Program Files\dotnet\sdk\1.0.0-preview4-004233 doesn't contain subfolder "Microsoft\VisualStudio"

Two possible reasons for this:

You haven't got .NET Core 1.1.0 SDK installed. This is required, even if you also have VS2017RC installed.
Or, you're using an old version of generator-aspnetcore-spa. Old versions don't produce global.json files, but those files are needed to specify which SDK version to use.

So please check you have installed .NET Core 1.1.0 SDK (not only older or newer versions). Also please make sure you're on the latest generator-aspnetcore-spa.
If that doesn't fix it, please reopen with details. Thanks!

SDK installation helps.
But it sill work as not described in documentation http://blog.stevensanderson.com/2016/05/02/angular2-react-knockout-apps-on-aspnet-core/

it seems you do not need md/cd some-empty-directory since now yoman script manages folders in iteractive dialog.
after dotnet run I see those errors

Hi. It sounds like your goal is to add a SPA project into an existing VS2017RC solution. If so, here are the steps:

Install the latest Yeoman and generator (npm install -g yo generator-aspnetcore-spa)
In VS2017RC, create a new solution

File -> New -> Project
Expand Templates -> Other Project Types -> Visual Studio Solutions
Choose Blank Solution
Pick a name (e.g., MyBlankSolution), click OK

In a command prompt, change directory to where the solution is (e.g., cd c:\Users\<youruser>\Documents\Visual Studio 2017\Projects\MyBlankSolution)
Run mkdir MyReduxProject, then cd MyReduxProject, then yo aspnetcore-spa
Choose React with Redux, then choose .csproj, then accept the name MyReduxProject, and wait for it to restore dependencies and build the client-side files
Back in Visual Studio, right-click on your solution name in Solution Explorer and choose Add -> Existing Project

Browse to the directory containing MyReduxProject.csproj (e.g., C:\Users\<youruser>\Documents\Visual Studio 2017\Projects\MyBlankSolution\MyReduxProject)
Select MyReduxProject.csproj and click OK

Check you can build it by pressing Ctrl+Shift+B. There will be no build errors.
Run it by pressing Ctrl+F5. It will appear in a browser.

I'm not sure how what you've done is different from this. If you are following some different procedure that you believe is correct but still doesn't work, can you please post exact repro steps, starting from creating an entirely new solution in a blank directory, with the same level of detail as in the instructions above? Thanks!

I have did the same, only instead of Blank solution I have created new "Core project" as it was recommended after SDK installation for VS 2017 (I guess for initialization).
Going to try it with blank solution.

I can compile it, but can't start (Ctrl+F5 doesn't work):

Try restarting Visual Studio. I once had it doing that, and restarting it made the problem go away. Seems like it was a bug in VS2017 RC.

Restart helps. Thank you.

When I move moved folder "MyReduxProject" (with all content) to subfolder: e.g. webapps\MyReduxProject application stop worked. Message box which points to error in tmp folder:

Please share more information about project internals! Somewhere is hidden dependencies on the folders strucure that is not declared in configuration file!

The configuration data for the page is invalid error is a general ASP.NET configuration/permissions issue - it's not specific to anything to do with these project templates.
To resolve it, I'd suggest a web search for that error message and looking for what you might not have configured correctly.

@SteveSandersonMS , please just do not think that I want to load my work on you, just give me a chance to explain:
I think every user of spaservices sees on that that this is not just "for learning purposes", not only expect that it is MS recommended way to setup SPA environment, but also expect to use it as backbone for his own projects. That means:

it is possible switch core to framework
it is possible  to move project a) to other folder b) to other solution

Yes, to support this in documentation it is an additional work for you. You can refuse this but please  just inform on first page that it was never tested by you.
When now you suggest just "web search" not MSDN page this means that even you do not know where it is described (some pieces somewhere in ".NET blog"). Because of permanent revolution in asp core tooling there is no docuemntation about this.
The most actual topic about switching core services is github issue https://github.com/dotnet/cli/issues/4996
Note: they suppose to drop
And there are no topics about moving projects folders from one folder to another (except we now that this was not working in the VS 2015 and as I see doesn't work in VS 2017 RC).
There are no other person in the world who could now better how to manage asp.core projects, because your spaservices are most complex presetup environments at a moment.

I was only saying that I don't know what the issue is because the error you've posted is a general IIS/ASP.NET one and isn't specific to anything in these templates. The 500.19 error is widely discussed online so if you search for that error message you will find plenty of advice on diagnosing and fixing it.

Thank you for your patience.
I have tried to create gif-video with moving to subfolder, and then rebuild/run, and on fifth or sixth attempt of recording moving to subfolder succeed.
So it somehow works. I can launch and see app in browser.
But together I see  an error in VS Error List:  MSB4131, The "FilesWritten" parameter is not supported by the "GenerateRuntimeConfigurationFiles" task. Verify the parameter exists on the task, and it is a gettable public instance property.

This will never finish...


Asecam Camera (659N7)

Hello,
Is the firmware compatible with ASECAM cameras? The current version of the camera is V5.00.R02.000659N7.10010.140d00..ONVIF 16.12

Do you have more information about the camera like the model number or pictures of the components inside? That will give us a better idea if it could be supported.


[JS] fix: update setup for echo/search/oauth bot/ME/adaptive card

Linked issues
closes: #980 #981 #982
Details
Provide a list of your changes here. If you are fixing a bug, please provide steps to reproduce the bug.
Change details

rename  MicrosoftAppId, MicrosoftAppPassword
update README
remove npm install from teamsapp.local.yml

Attestation Checklist

[x] My code follows the style guidelines of this project

I have checked for/fixed spelling, linting, and other errors

I have commented my code for clarity

I have made corresponding changes to the documentation (we use TypeDoc to document our code)

My changes generate no new warnings

I have added tests that validates my changes, and provides sufficient test coverage. I have tested with:

Local testing
E2E testing in Teams

New and existing unit tests pass locally with my changes

Additional information

Feel free to add other relevant information below

Thanks for these updates! Could you also add a table of contents for echoBot and searchCommand, under the sample description / before 'Showcase' (similiar to AI-ME Samples)

Thanks, I missed that. Added TOC for these two samples.

@kuojianlu Please also update the sample.env files with the comment!
https://github.com/microsoft/teams-ai/blob/main/js/samples/05.chatModeration/sample.env#L1

Discussed with @kuojianlu -- these don't need sample env file.


[corechecks] Add Helm cluster check

What does this PR do?
This PR adds a new cluster check that collects information about the Helm releases deployed in the cluster.
For each Helm release deployed, the check reports the helm.release gauge set to 1 with the following tags:

helm_release
helm_chart_name
helm_namespace
helm_revision
helm_status
helm_chart_version
helm_app_version

Additional Notes

Not in this PR, but may be implemented in the future: add events to the check (helm release deployed, deleted, etc.). Also, add extra tags to the metrics from Pods managed by Helm.
Helm supports multiple storage backends. The default in v3 is Kubernetes secrets. That's what this check supports. In the future we might add support for other backends.
This PR needs changes in our Helm chart and the Operator. The reason is that this check needs permissions to read Kubernetes secrets. At the moment, in order to use this check, the user needs to set those permissions manually.
The check is not enabled by default.
I needed the changes in https://github.com/DataDog/datadog-agent/pull/10900 to make this check work in the cluster check runners. Not sure if this is something specific to my environment.

Describe how to test/QA your changes

Deploy the agent in Kubernetes. The check is not enabled by default, if you're using the Helm chart you can use this:

clusterAgent:
  enabled: true
  confd:
    helm.yaml: |-
      instances:

If you're testing this before we adapt our Helm chart and Operator as mentioned above, you'll need to add permissions to read secrets manually in the cluster-agent and the check runners cluster roles.
Deploy some Helm chart in your cluster. The datadog one also counts if that's what you used.
For each Helm release deployed in your cluster, check that the metric helm.release is reported and that it's tagged with all the tags listed above.

The check should run properly on the DCA and the cluster check runners (this can be controlled with the cluster_check option (set to true to run in check runners). When configured to run on the DCA, try 2 replicas and make sure that the check only runs in the leader.
Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] The appropriate team/.. label has been applied, if known.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the config template has been updated.

I think copyright dates are handled some other way?  At any rate, those aren't related to this PR.  Let's fix those in another PR, if necessary.

@clamoriniere Ready for review


[core] Running a local Lilypad network on windows had some hiccups

chown commands had to be sudod - figure out a way for hardhat files to be owned by the correct user
chain path had to be set outside of current folder or docker build commands would fail if chain is running because data folder is locked and there is a COPY . .  statement in the docker files (easy, just copy the right files)

https://github.com/Lilypad-Tech/lilypad/pull/208

https://github.com/Lilypad-Tech/lilypad/issues/106

checkout the branch: arsen/fix-windows-docker-permission
chain-clean
chain
chain-boot
if that works, we can approve this


Add go.work and go.work.sum to global allowlist

Description:
Explain the purpose of the PR.
 Using workspaces leads to the generation of go.work and go.work.sum files, which behave similarly to go.mod and go.sum files. Since the other Go-specific files are in the global allowlist, I think it makes sense to also add  these files.
Checklist:

[x] Does your PR pass tests?
[ ] Have you written new tests for your changes?
[x] Have you lint your code locally prior to submission?

love it. Thanks @AislingHPE


Attribution for podcast directories and webapps - audio query string proposal

This isn't, per se, anything for an RSS namespace: but I'd be keen to at least float this idea and see what happens. If there's a better place for this - I feel there must be - then I'd be really grateful for a pointer. It is, however, something that needs agreement and standardising somewhere; and has a particular benefit for The Podcast Index.
The problem
If I listen to a podcast on Podfriend on my Windows laptop, then this is the useragent that will present itself to my podcast host:
Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/86.0.4240.111 Safari/537.36
Here's the useragent from a listen to a podcast within a Google Search result on that same laptop:
Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/86.0.4240.111 Safari/537.36
...here I am listening to a podcast using Podchaser's inbuilt player:
Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/86.0.4240.111 Safari/537.36
...here I am listening to a podcast on a clever embedded Twitter card:
Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/86.0.4240.111 Safari/537.36
...and here is a podcast on the Apple Podcasts website:
Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/86.0.4240.111 Safari/537.36
The observant among you will have spotted that these are all exactly the same. A podcast host is unable to differentiate between any of these plays, and a podcast host is left without any idea where their marketing has worked and which podcast service is driving the traffic to them.
The result? Podtrac reports that the second-most popular podcast player is... Google Chrome. Which it is. But it isn't.
Another problem
If you hit 'play' in Pocket Casts on an iPhone but you haven't downloaded the audio first, the app will try to 'progressively download' the audio - or 'stream' it in commonly-used language. It does this using the AppleCoreMusic library, and to a podcast host, it looks like this:
AppleCoreMedia/<IP_ADDRESS>A393 (iPhone; U; CPU OS 14_0_1 like Mac OS X; en_us)
Hit 'play' in Apple Podcasts on an iPhone without downloading the file first? It looks like
AppleCoreMedia/<IP_ADDRESS>A393 (iPhone; U; CPU OS 14_0_1 like Mac OS X; en_us)
Hit 'play' in Airr, a new iPhone podcast app? It looks like
AppleCoreMedia/<IP_ADDRESS>A393 (iPhone; U; CPU OS 14_0_1 like Mac OS X; en_us)
And hit 'play' in Castro, and it looks like
AppleCoreMedia/<IP_ADDRESS>A393 (iPhone; U; CPU OS 14_0_1 like Mac OS X; en_us)
That's because this is the useragent of the library, and it can't be changed. Similar things happen on the Apple Watch.
The result? Podcast hosts either attribute all of this to Apple Podcasts (because it says the word 'Apple' in it), or they don't report it at all, or they break it out to 'AppleCoreMedia' which doesn't help anyone.
Solutions
One solution is to use the referer HTTP header. But that doesn't always help with web-based plays (referers are often set to be blank for privacy; or report an email client or a social media service); and it doesn't help at all with the AppleCoreMedia problem.
Another solution is to give every podcast directory a different RSS feed which is bespoke to that platform. Some podcast hosts do this: but it isn't great to have lots of URLs floating around which have the same content with a different address (and is against best practice). It's also hard if you supply https://example.com/rss/for-apple into Apple Podcasts, and then Overcast goes and grabs that for its own use.
A better solution is to use the RSS useragents repo - to spot the RSS calls being made, and then to add a query string to the audio. So, if your RSS feed is called by a useragent you recognise like Overcast/1.0, you can then set the audio to be https://example.com/audio/mygreatpodcast.mp3?parsedby=overcast and you can then be reasonably sure that any use of this audio is from Overcast - even in the web player it produces.
Here is an example of the different useragents being used on a day's worth of data. The RSS useragent approach works really well.
BUT Podcast Index throws a spanner in the works here. The Podcast Index RSS useragent starts Podcastindex.org/ but that then means that every piece of audio that uses Podcast Index's API is tagged with parsedby=podcastindex. If twenty apps use Podcast Index's APIs, then those apps are invisible to podcasters, and we don't know which is performing best.
Proposal
Podfriend is a web app. It's unable to modify the browser useragent, or to add any bespoke headers.
If it wishes to signal to a podcaster that "this play is coming from Podfriend", it could send a unique indication in its request for the audio.
My proposal is that this specific query string name is uniformly called _from, and the value should be an unambiguous name for the service or app - normally a domain or pseudo-domain (which we may, optionally, wish to list in the OPAWG rss useragents list. The use of the underscore in _from is unlikely to clash with anything.
So, as an example:
https://example.com/audio/mygreatpodcast.mp3?_from=podfriend.com
Where a querystring is already there, the query string should be appended with a &, like:
https://example.com/audio/mygreatpodcast.mp3?version=45&_from=podfriend.com
Adding this audio query string in a uniform fashion would enable any host or analytics service to spot this query string for analytics; and help developers get the credit they deserve. It would work whether the podcast is downloaded or streamed, and be unlikely to be stripped or removed. (Only iVoox, in my tests, seems to remove all query strings from audio URLs).
Most importantly, it would be the end of "the second most popular podcast platform is Google Chrome" - which is nonsense.
Thoughts?

I support "_from=" 100%. It's a simple and elegant solution that's easy to implement on both sides.

I like the idea of tackling problems like this, but I'm not sure where they are best addressed. This is definitely outside the scope of what Podcastindex is currently undertaking.

@tomrossi7 I entirely agree, it's outside scope (though affects Podcast Index more so than others).
Any clue as to where might I put this? If only I knew someone who wrote a daily newsletter or something.

@jamescridland I do think it could be something handled by OPAWG. At least right now that's my best bet.
Then we need to make sure to link a lot to the slack from podcastindex.social to get more eyes on it?

@jamescridland I posted about this on mastodon with an idea that we could maybe help with.

I vote for ?_from= 👍

But what will happens if the audio file URL already has a ?_from= in the RSS feed ?
(Castopod already adds it based on opawg/podcast-rss-useragents)

I think I would personally go for something like ?_appName=, which I feel is less ambigious, if I encounter "?_from" I'm thinking it could be all sorts of things, but "appName" I fully understand instantly. But if everyone thinks ?_from is better, then I am still perfectly fine with that.
Also, the reason why I don't think "ua" is all that good is that we do not want to recreate the UserAgent string - because that one will be sent with the request anyway, with platform information etc.. We simply want to pass on a bit of extra information.

I named it ?s= (s as in service) but ?_from= or ?_appName=, any name will work for me, as long as it doesn't take too long to pick one. 😉

@benjaminbellamy Good question. I think the point of trying to standardise this is to highlight that apps and players can overwrite any existing _from value.
For Podfriend, they will get an RSS feed from me with ?_from=podcastindex --because the feed has been parsed by Podcast Index. I would like to see _from=podfriend though, since Podfriend is handling the final play.
One thought is that we could ask for these to be added - _from=podfriend,podcastindex but I'm not sure how useful that is in reality.
Given the removal of slugs elsewhere in Podcast Index, we should be using domains or pseudo domains here, I suppose.

(This will be in Podnews on Monday. It would be good to drive a bit of traffic here, and let's see what the reaction is).

Good call @jamescridland .

I've moved this to https://github.com/opawg/podcast-rss-useragents/issues/12
Please come and continue the discussion over there! :)


Skipping USER when using a tool

Is your feature request related to a problem? Please describe.
When I use tools, the sequence of messages goes:

USER asking a question
LLM responding with a TOOL call
AGENT responds with a TOOL result
USER is prompted for a response

Describe the solution you'd like
I think the right (at least for my use) sequence should be:

USER asking a question
LLM responding with a TOOL call
AGENT responds with a TOOL result
LLM responds to the original question using results from the tool

So USER should not be added to the list of responders in Task.step() if pending message is a response from the AGENT.
Describe alternatives you've considered
I know I can skip human prompt. Is there another way? Maybe there is a way to tag a tool such that it adds "LLM" as a recipient?
Additional context

Maybe there is a way to tag a tool such that it adds "LLM" as a recipient?

Indeed, there's a SEND_TO routing you can add as prefix to the string returned by the tool handler method, see example here:
https://github.com/langroid/langroid/blob/main/examples/docqa/extract-then-chat.py#L76
We haven't gotten around to documenting this yet.

Incidentally, we don't want to enforce that after agent_response() the result should always be handled by llm_response() -- one reason is that the result of a tool could be a huge amount of text (e.g. think of a sql query) and we want to allow user to intervene (when interactive=True) to either allow the LLM to respond, or insert their own message

@pchalasani thanks for the response! Will give it a go.
I'm new to the field, maybe we are thinking about different use cases. I am interested in RAG, where I don't want user to see/act upon the response from a tool/function. As far as user is concerned, tool is just a part of the agent not a separate entity. I want LLM to transform the response and tailor it to the original user prompt. Huge amount of text, in this case, should be handled by the tool/function itself. For example, limit the result of the sql query to a manageable size.
Don't get me wrong, I think SEND_TO is a perfect solution. Maybe worth wrapping it (optionally) so it is added automatically to the tool response.

Huge amount of text, in this case, should be handled by the tool/function itself.

I only listed on possible reason (i.e. result size) why we'd want the user to have a chance to "intervene" before an LLM sees a tool result. As you said this scenario can be handled by putting the burden on the tool handler method to cut the result size (which still may not be perfect, since the handler method would potentially need to do some parsing etc).
Another reason for user intervention could be that the tool result may contain sensitive info, and the user gets a chance to check and override.
So we made a design choice here: instead of creating a special rule ("send all tool results directly to LLM") and have possible undesirable scenarios where results shouldn't be seen by the LLM, we rather put the onus on the developer who is writing the tool handler, to add a SEND_TO to explicitly send it to LLM if they are confident that is fine to do, in their specific scenario.
Another way is to have two separate agents. Assistant directly interfaces with the user, takes their high-level questions, and generates questions to RagAgent (which would have interactive=False).  See an example of this in examples/docqa/doc-chat-2.py and other similar variants.

Maybe worth wrapping it (optionally) so it is added automatically to the tool response.

This could be one way, e.g. via a config flag. We're thinking about this or other good ways to let the developer control message flow in various states.

Forgot to mention one other thing. When you set up a task with interactative=True, then it will wait for user input after each valid response from a non-human entity (i.e. LLM or Agent tool handler). One big reason we do it this way is to avoid infinite loops between LLM and AGENT responses -- e.g.

LLM generates a tool,
tool handler returns result or err
LLM generates tool again
tool handler returns result or err
...

If you really want a scenario where it takes your first question, and then subsequently becomes non-interactive, then you can either use SEND_TO or a multi-agent set up as mentioned above.
Another possibility is you could write your own loop:
task = lr.Task(..., interactive=False)
while True:
    query = ... prompt user for their question...
    result = task.run(query)

Hi @pchalasani, when I set interactive=False, after tool calling, the agent return same message continuelly. Any suggestion for this? By the way, I'm using gpt-3.5, is it related?
>>> Starting Agent Search (1) gpt-3.5-turbo-1106
{
    "request": "search",
    "content": "langchain"
}
Stats: N_MSG=2, TOKENS: in=232, out=17, max=1024, ctx=16385, COST: now=$0.0003, cumul=$0.0003,
tot=$0.0003 (gpt-3.5-turbo-1106)
Agent:
Langchain is a blockchain platform that aims to provide a decentralized infrastructure for language-related service providers, such as translators, interpreters, and language tutors. It uses blockchain technology to facilitate secure and transparent transactions within the language industry.
Stats: N_MSG=4, TOKENS: in=254, out=45, max=1024, ctx=16385, COST: now=$0.0003, cumul=$0.0006,
tot=$0.0006 (gpt-3.5-turbo-1106)
Langchain is a blockchain platform designed to support language-related services such as translation, interpretation, and language tutoring. It aims to provide a decentralized infrastructure for language professionals and clients, using blockchain technology to facilitate secure and transparent transactions.
Stats: N_MSG=6, TOKENS: in=304, out=45, max=1024, ctx=16385, COST: now=$0.0004, cumul=$0.0010,
tot=$0.0010 (gpt-3.5-turbo-1106)
Langchain is a blockchain platform that focuses on providing decentralized solutions for language-related services, such as translation, interpretation, and language tutoring. It aims to create a secure and transparent environment for language professionals and clients to interact and transact.
Stats: N_MSG=8, TOKENS: in=354, out=47, max=1024, ctx=16385, COST: now=$0.0004, cumul=$0.0015,
tot=$0.0015 (gpt-3.5-turbo-1106)
Langchain is a blockchain platform that is designed to support language-related services such as translation, interpretation, and language tutoring. It aims to provide a decentralized infrastructure for language professionals and clients, using blockchain technology to facilitate secure and transparent transactions.
Stats: N_MSG=10, TOKENS: in=406, out=47, max=1024, ctx=16385, COST: now=$0.0005, cumul=$0.0020,
tot=$0.0020 (gpt-3.5-turbo-1106)
Langchain is a blockchain platform that focuses on providing decentralized solutions for language-related services, such as translation, interpretation, and language tutoring. It aims to create a secure and transparent environment for language professionals and clients to interact and transact.
Stats: N_MSG=12, TOKENS: in=458, out=47, max=1024, ctx=16385, COST: now=$0.0006, cumul=$0.0025,
tot=$0.0025 (gpt-3.5-turbo-1106)

@frankyuan This is expected. You have to have some way to signal that the task is done, to exit the loop. That could either be by

explicit signals from the agent entities, e.g. prefixing msg with DONE -- there are numerous examples using DONE (search for "DONE" under examples/), or
via task constructor arguments like done_if_response, done_if_no_response (see e.g. examples/docqa/chat_multi_extract.py), see release notes

It's not clear what you are trying to accomplish in the above example, which is probably a dummy example just for illustration.

@frankyuan This is expected. You have to have some way to signal that the task is done, to exit the loop. That could either be by

explicit signals from the agent entities, e.g. prefixing msg with DONE -- there are numerous examples using DONE (search for "DONE" under examples/), or
via task constructor arguments like done_if_response, done_if_no_response (see e.g. examples/docqa/chat_multi_extract.py), see release notes

It's not clear what you are trying to accomplish in the above example, which is probably a dummy example just for illustration.

Thanks @pchalasani, I will try your suggestion.


Switch runelite panel sides from right to left

I play with Runelite on my left screen, and the panel falls closest to the middle, it would be nice to switch the panel to the left side of the client so I can easier see the game
Describe the solution you'd like
Have the panel side toggle able
Describe alternatives you've considered
I don't see any clear alternatives besides moving the game to my right monitor which I don't want to do
Additional context
Add any other context or screenshots about the feature request here.

Duplicate of #1641


Add create class code action

Description:
$Title
Describe your problem(s)
For now we suggest Create record 'Person' for the following scenario,
public function main() {
    Person person = new Person();
}

But this is an object creation and Person is a class not a record.  This behaviour due to the diagnostic unknown type 'Person'(BCE2069). ( and the diagnostic is same at Person and Person())

Quick fix we now provide,

Describe your solution(s)
We can improve this to give Create class 'Person' code action.

We are closing this issue due to inactivity. If you need further assistance or have additional input, feel free to reopen it. Thank you for helping us maintain a relevant and focused issue list!


rename amp_app, update libamp and haas100 uart driver

Is your feature request related to a problem? Please describe.
amp_demo is not a right name easy to misunderstanding. uart recv failed
Describe the solution you'd like
rename amp_demo to amp_app , update uart.c use ringbuf to recv
Additional context

 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.xinweng.ljh seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


"CDVImgDownloader.h" not found for ios

Hi, when trying to use this plugin on for ios, I got the error "CDVImgDownloader.h" not found. Please advise, thanks.

Thanks, @Z-Hau for raising the issue. Please try to uninstall and install again. Meanwhile, I will try to check what went wrong. At present, I don't have Mac to test, if anyone can debug and fix it I'd really appreciate it.

Hey @Z-Hau, fixed the issue. Now please install the latest release i.e v1.0,8. "npm i cordova-plugin-downloadimage-to-gallery"
Hit star on GitHub and let me know if you find any issue.
Thank you @Z-Hau

@Z-Hau Alright, will have a look, thanks !

@saisreeharsha Unfortunately, more errors.

This is the latest version

Sorry for that, I am on it. Will update u asap!


Add RequestError to know if error was due to request failing

Adds RequestError to Error in the case the request fails

Fixes #31

Thank you for the PR, can you please add some accompanying tests.

Done, also added a test for RealClock so it now has 100% test coverage.
$ go test --cover
OK: 12 passed
PASS
coverage: 100.0% of statements

Thank you


FPS-105: Fix broken links (and other minor fixes)

Fixed broken links. I opted for absolute paths rather than relative paths, because links built with relative paths occasionally break when swapping between languages (not sure exactly what triggered this, but it's almost certainly a bug in the JS somewhere)
Updated links to the zendesk site to point internally instead
Added an ES translation for 'Overview of Kiva's API' (translation from Mateo)
Adding firebase config (seems reasonable/good to have this in the repo; we can later automate firebase deployments)
Deleted redundant /en/ i18n directory: the default locale is English, so the default /docs path shows up as English in the dropdown (the code was copy/pasted and I spent a few minutes trying to figure out why my changes weren't getting picked up)

Please test!
(Do we do a test deployment for these PRs?) I used the firebase-cli to deploy locally.
From /fps-sdk/portal, run yarn clear && yarn build, and then run the firebase emulator as outlined in the linked page.

we cannot delete the en files. in order for translations to work within the docusaur toolset, the en files must exist even though our default pages are english.

@matt-raffel-kiva have you tested this? I tested with the firebase emulator it and it works. I'll be pretty disappointed if docusaurus requires us to copy/paste this directory.

yes I have tested this...hence why I ask the files be added back

This PR was closed accidentally. I was trying to delete a comment.


Support for SSH git access with passphrase protected key

Is there any configuration option to provide the passphrase for the SSH key that accesses an SSH based git endpoint? The only way i could get it to work with<EMAIL_ADDRESS>was to strip the passphrase from the key. This should be added as it appears possible w/ jgit via a CredentialsProvider
http://stackoverflow.com/questions/12502452/how-do-you-set-the-configuration-for-jschconfigsessionfactory-for-jgit-so-that-p

It would be a really nicer feature for git backend support.


SDK-916 : Error logs when certificate submission fails

Jira ticket

A bit off-topic: Please check your git config, somehow your name is set to = :wink:


Possible to change VM IP?

Is it possible to change the VM IP from what it currently defaults to for me <IP_ADDRESS> to something else (for my case, I want <IP_ADDRESS>)? Any instructions would be welcome.

OS X assigns the IP dynamically, it would take some research to figure out how to change the subnet as well as assign a static IP. I'll put it on my list of things to figure out.

i wasn't able to find a reliable way of doing this, unfortunately


Merge bytecodealliance/wasmtime with typed continuations

This patch merges the changes to bytecodealliance/wasmtime with the
typed continuation stuff. This patch also targets our
wasm-tools#typed-continuations branch.
Currently, there are a bunch of failures regarding function references, which I have not looked deeper into (I assume some of them are to do with unimplemented features). List of failures:

wast::Cranelift::spec::function_references::br_on_non_null
wast::Cranelift::spec::function_references::br_on_non_null_pooling
wast::Cranelift::spec::function_references::br_table
wast::Cranelift::spec::function_references::br_table_pooling
wast::Cranelift::spec::function_references::call_ref
wast::Cranelift::spec::function_references::call_ref_pooling
wast::Cranelift::spec::function_references::func_bind
wast::Cranelift::spec::function_references::func_bind_pooling
wast::Cranelift::spec::function_references::let_pooling
wast::Cranelift::spec::function_references::linking
wast::Cranelift::spec::function_references::linking_pooling
wast::Cranelift::spec::function_references::r#let
wast::Cranelift::spec::function_references::return_call_ref
wast::Cranelift::spec::function_references::return_call_ref_pooling
wast::Cranelift::spec::function_references::select
wast::Cranelift::spec::function_references::select_pooling
wast::Cranelift::spec::function_references::table
wast::Cranelift::spec::function_references::table_pooling
wast::Cranelift::spec::function_references::type_equivalence
wast::Cranelift::spec::function_references::type_equivalence_pooling

Yay!! This is awesome!    Tests aren't quite right (call_ref should certainly work), but I think it's just a new message mismatch thing with an easy fix.    Let's merge.     I have to run to class, but I'll work more on this tonight (when you likely won't be around).   So, please let me know at the end of your day what I can work off of and should/shouldn't change.   I may be able to start looking at some of the new instructions.     I can try to apply this patch on the new branch this evening, or feel free to try it out yourself.
diff --git a/crates/wast/src/wast.rs b/crates/wast/src/wast.rs
index d9f885acf..b1a44ced8 100644
--- a/crates/wast/src/wast.rs
+++ b/crates/wast/src/wast.rs
@@ -328,6 +328,8 @@ impl<T> WastContext<T> {
             // specifies which element is uninitialized, but our traps don't
             // shepherd that information out.
             || (expected.contains("uninitialized element 2") && actual.contains("uninitialized element"))
+            || (expected.contains("null function") && actual.contains("uninitialized element"))
         {
             return Ok(());
         }

One thing that's on my mind as tests fail, remember that cargo locks the git dependency that was on #typed-continuations when you first built the new Cargo.toml, so you may need to update those dependencies (only, using -p) if you see errors related to anything we recently (or in the future) merged


Add Jenkinsfile for bundle workflow

Description
Add a Jenkinsfile describing a pipeline job that builds and assembles a new bundle nightly and publishes all artifacts to S3.
This job depends on the following plugins:
aws-steps
 Pipeline-utility-steps for parsing YAML.
jdk 14 and maven should be available as tools with these labels:
 jdk "JDK14"
maven "maven-3.8.2"

Python3, pip3, and pipenv are also required on the agents to run the build scripts.
Issues Resolved
closes #212
Check List

[ x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Is there an issue for ARM that will make sure we update this?

There is not -  I will make one.
I am thinking of updating this file to in parallel build for arm as a separate step.  This is so that we can link the arm & x64 builds to the same execution and ensure they are triggered at the same time. So we'd have something like this:
/builds///.  Thoughts?

Should we be building --snapshot by default?


Does not handle gaps in tool numbers

If you have a 3 tools with numbers 0, 1, 3 (eg. relating to physical position on printer) then the code thinks there are
4 tools installed.

Works now!


Hide volumetric fog, SSIL and SDFGI on Mobile/Compatibility renderers

These features are only supported in the Clustered Forward renderer.
See https://github.com/godotengine/godot/issues/55880.

I've never liked us doing this in Godot 3 and I'm not sure if we should be doing so in Godot 4.
In Godot 3 I kind of got it with how different the GLES2 and GLES3 renderers became which made it unrealistic to create a game that targeted both renders.
But with Godot 4 this is far more likely where someone creates a game that will use the desktop renderer when deployed to high end computers and the mobile renderer when deployed on lower end or mobile devices.
You don't want to get into a situation where settings are lost just because the developer switches to the mobile renderer to test their game.

@BastiaanOlij I suppose we could use the read-only property hint which was added in 4.0. This way, settings are still shown but you can't toggle them.

@Calinou maybe, my point merely is that someone might want to support both and removing the settings or even making them read only, just makes that harder to do.
That said, I guess such a developer will need to build something on top of this anyway because you would likely want to have the settings that are shared between the two renderers be different. Looking for instance at the limitations our R10G10B10A2 buffers on the mobile renderer give will likely mean that you'll want to tweak settings so they look good when the mobile renderer is used.
It almost begs the question if the two renderers need their own set of settings, now there is a hornets nest I'd rather not touch :)

Closing per the above comments.


Some pages are crashing on the live site

Not sure if you are aware of this.
https://hotwire.io/documentation/turbo/handbook/02_drive

Yeah, sadly the state of git submodules on Heroku is not great. It's currently failing because the automatic deployment via GitHub doesn't initialize git submodles.
I'm not sure what the best way would be to incorporate the upstream documentation without have to actually copy it into the repo.
I guess the other option is that we fetch the documentation at runtime via HTTP requests and cache it in the app.

How about fetching them in the build step? Would that be an alternative? It would definitely be more performant.

Right, that's a good idea. We'd just need to make sure that you can also run and see the content locally while developing, so it should probably be something like a rake-task that spits out the generated files.

Yes. a Script that you could run from time to time that will output the files in the directory, and you manually commit to the repo.
Then they will be available as regular HTML files


Add breaking-change triage in release notes

With this config, PRs labelled "breaking-change" will automatically be categorized as such in release notes

Codecov Report

Merging #284 (3b18200) into main (fb310b7) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #284   +/-   ##
=======================================
  Coverage   66.89%   66.89%
=======================================
  Files          73       73
  Lines        4157     4157
=======================================
  Hits         2781     2781
  Misses       1197     1197
  Partials      179      179

Flag
Coverage Δ

unittests
66.89% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@jotak 👍


[Question] - Different Signatures for same input

Hi,
I'm trying to port my python code over to golang and trying to replicate ECDSA P192 signing logic from python to golang(wrapper over openssl).
ECDSA:
//S.CURVE is P192 and Hash function is sha256
fe_ecdsa.sign(message, s._key, curve=s.CURVE, hashfunc=s.HASH

The output signature is always the same when I sign using this library. When I use openssl, the signature varies for the same input(same keys)
I understand that this may not be a bug with this library. I'm just wondering if there is anything that this library does to not get randomized signatures.
Thanks!

Hey there, it could well be that openssl is using a randomly generated nonce, which would cause non-deterministic signatures for a static message and keypair. This library uses RFC6979 to generate deterministic signatures by using a deterministic nonce derived from the input message.

Thanks for the explanation! I will close the issue


New Website Logo: Stateless

Please fill the following out:

Name: Stateless
Homepage url: https://www.bestateless.com/
Brand Guidelines/Licensing: We have complete ownership of our logo
Logo:

Duplicate of #14424


sorting virtual fields on UTable

Description
currently columns has sortable?: boolean but only really works when the column key is present in the row.
I believe it should become sortable?: boolean|((row:any) => any) and use this function to determine a and b for sort.
basically going down to replacing these get() calls with the given sortable function.
    const rows = computed(() => {
      if (!sort.value?.column || props.sortMode === 'manual') {
        return props.rows
      }

      const { column, direction } = sort.value

      return props.rows.slice().sort((a, b) => {
        const aValue = get(a, column)
                    // ^^^
        const bValue = get(b, column)
                    // ^^^

        const sort = columns.value.find((col) => col.key === column)?.sort ?? defaultSort

        return sort(aValue, bValue, direction)
      })
    })

unfortunately, I'm still too new and burned out to figure out the correct implementation.
is this change something that would be accepted? any suggestion on how to proceed?
Additional context
this functionality is needed in order to sort columns which key isn't a field in the row but assembled via <template #foo-data /> from the full row instead.
but It's also relevant to those columns where the field exists but isn't used as-is. the second can reduce the need for custom sort implementations as defaultSort would be able to sort correctly what the proposed sortable function returns.
it could also make sense to complement this with an optional transformation function per column to compose the value used by the table when the data slot isn't provided and for sorting simply providing sortable: true and using the value function.

I wonder if we could take advantage of useSorted for this.

I would be happy to implement the basics of this if someone can tell me what would be the recommended approach and knowing the maintainers would actually consider the PR.

Revisiting this I've noticed that if we do add a sort based on a virtual field, and since sortable: true if the user decide to sort based on another field, how would they be able to sort back to that virtual field?
Isn't it more convenient to add an actual field by transforming the data after the fetch?

Revisiting this I've noticed that if we do add a sort based on a virtual field, and since sortable: true if the user decide to sort based on another field, how would they be able to sort back to that virtual field?

the sort object carries column key and direction, by virtual field a column with key but with undefined row[key].

Isn't it more convenient to add an actual field by transforming the data after the fetch?

very good idea indeed. would that be done with something line const { data, pending, refresh, error } = useFetch().then((ctx) => transform(ctx))?

very good idea indeed. would that be done with something line const { data, pending, refresh, error } = useFetch().then((ctx) => transform(ctx))?

Even more simpler! Both useFetch and useAsyncData provide a transform option that lets you transform the data after it has been fetched.
In the following example (although not perfect) should give you the idea, combining the city and street so that the sort function will sort them combined:
<script setup lang="ts">
const columns = [{
  key: 'id',
  label: 'ID'
}, {
  key: 'name',
  label: 'Name',
  sortable: true
}, {
  key: 'address',
  label: 'Address',
  sortable: true
}]

const { data: people } = await useFetch('/users', {
  baseURL: 'https://jsonplaceholder.typicode.com',
  transform: (users) => {
    return users.map(user => ({
      id: user.id,
      name: user.name,
      address: `${user.address.city} - ${user.address.street}`
    }))
  }
})
</script>

<template>
  <UTable :columns="columns" :rows="people" />
</template>

brilliant! thank you so much @Sandros94 . this closes it I think.


Image-to-Image Translation using ReStyle

Hey!
First of all, great work!
I just wanted to ask whether there is any documentation for trying the ReStyle Encoder for Image-to-Image Translation?
I am working on generating real images from sketches in a non-facial domain and have already tried the vanilla psp image-to-image translation pipeline.
I saw your comment here saying that the ReStyle Encoder is better suited for non-facial domain and thus wanted to try it.
I have already tried setting the source and the target in the data config as the folder to sketch and real images respectively. It generates images as attached below. Shouldn't the input here be a sketch image?

Thanks in advance!

Sorry for the confusion.
So this is how my config looks like -

The source, as it can be seen, are the sketch images and the target are the real images.
The data is paired by using the same filename for the sketch and the real image.
Using this config, I am seeing the result attached in the previous message above.
Is that correct for the task of Image-to-image translation for generating realistic images from sketches?

Ahh got it.
Yeah it would make more sense if the source image is picked from the correct domain if we are working across multiple domains.
Let me put this up for training and see.
Thanks a lot for your help! :)

A few questions related to all this.
What is the significance of the transforms?  Why are they as they are in the repo?
And does this mean the model is trained on a 256x256 image? Won't that affect things if the model is meant for 1024?


Mapping IllegalArgumentException to SettingsException extend OpenSearchException in AbstractScopedSettings class

Description
Mapping IllegalArgumentException to SettingsException extend OpenSearchException in AbstractScopedSettings class so that the helpful error messages can be reported.
Issues Resolved
resolve #4745
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #4792 (1d3b642) into main (657d559) will decrease coverage by 0.09%.
The diff coverage is 55.55%.

@@             Coverage Diff              @@
##               main    #4792      +/-   ##
============================================
- Coverage     71.00%   70.90%   -0.10%
+ Complexity    58161    58095      -66
============================================
  Files          4708     4708
  Lines        277522   277522
  Branches      40181    40181
============================================
- Hits         197055   196785     -270
- Misses        64326    64558     +232
- Partials      16141    16179      +38

Impacted Files
Coverage Δ

.../opensearch/common/settings/SettingsException.java
66.66% <0.00%> (-8.34%)
:arrow_down:

...search/common/settings/AbstractScopedSettings.java
89.71% <58.82%> (+0.23%)
:arrow_up:

...adonly/AddIndexBlockClusterStateUpdateRequest.java
0.00% <0.00%> (-75.00%)
:arrow_down:

...readonly/TransportVerifyShardIndexBlockAction.java
9.75% <0.00%> (-58.54%)
:arrow_down:

...n/admin/indices/readonly/AddIndexBlockRequest.java
17.85% <0.00%> (-53.58%)
:arrow_down:

...indices/readonly/TransportAddIndexBlockAction.java
20.68% <0.00%> (-41.38%)
:arrow_down:

.../admin/cluster/reroute/ClusterRerouteResponse.java
60.00% <0.00%> (-40.00%)
:arrow_down:

...h/action/ingest/SimulateDocumentVerboseResult.java
60.71% <0.00%> (-39.29%)
:arrow_down:

...luster/routing/allocation/RoutingExplanations.java
62.06% <0.00%> (-37.94%)
:arrow_down:

...pensearch/action/ingest/DeletePipelineRequest.java
31.25% <0.00%> (-37.50%)
:arrow_down:

... and 496 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Yep. Agreed. I am wondering if we don't backport this PR to 2.x and wait for 3.0 release?

It is still a potentially impactful change for developers and users even if it is "allowed" in a major version, so if we can do this in a backward compatible way that is preferable.
I think we discussed this before, but I'm coming back around to the idea of changing this bit of code:
https://github.com/opensearch-project/OpenSearch/blob/f5ea1a4d8067896080f61ab5f88cdcd307ee3c84/server/src/main/java/org/opensearch/ExceptionsHelper.java#L106-L107
to be:
} else if (t instanceof IllegalArgumentException) {
    return "Invalid argument" + "[" + t.getMessage() + "]";

The question is whether we'd be leaking information that is meant to be suppressed when detailed error reporting is turned off. @reta do you have any thoughts? It seems pretty innocuous to me, and InvalidArgumentExceptions are often constructed with helpful user-facing messages (as is the case with the settings).

@xuezhou25 coming from: https://github.com/opensearch-project/OpenSearch/issues/5222
Looks like there are genuine failures from this PR.
Are you still working on this? If so, probably lets chat.

In build 9282:
The failed test is introduced in commit https://github.com/opensearch-project/OpenSearch/commit/da94dbb94e16a9766049006e666b02dd89b5bf7e on 12/22/2022
REPRODUCE WITH: ./gradlew ':server:test' --tests "org.opensearch.extensions.ExtensionsManagerTests.testHandleAddSettingsUpdateConsumerRequest" -Dtests.seed=EC17246CC904E1E0 -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=ar-BH -Dtests.timezone=America/Inuvik -Druntime.java=19

org.opensearch.extensions.ExtensionsManagerTests > testHandleAddSettingsUpdateConsumerRequest FAILED
    SettingsException[Setting is not registered for key [falseSetting]]
        at __randomizedtesting.SeedInfo.seed([EC17246CC904E1E0:4A8CBBFBDAADD80B]:0)
        at app//org.opensearch.common.settings.AbstractScopedSettings.addSettingsUpdateConsumer(AbstractScopedSettings.java:249)
        at app//org.opensearch.common.settings.AbstractScopedSettings.addSettingsUpdateConsumer(AbstractScopedSettings.java:476)
        at app//org.opensearch.extensions.AddSettingsUpdateConsumerRequestHandler.handleAddSettingsUpdateConsumerRequest(AddSettingsUpdateConsumerRequestHandler.java:73)
        at app//org.opensearch.extensions.ExtensionsManagerTests.testHandleAddSettingsUpdateConsumerRequest(ExtensionsManagerTests.java:738)

In build 9355, the same failure as above https://github.com/opensearch-project/OpenSearch/pull/4792#issuecomment-1379324786

@xuezhou25 Can you please rebase your branch with the latest commits?


during install harbor, there's many error log

cannot set status to error: cannot set conditions to error: cannot prepare owner status: cannot update status: Operation cannot be fulfilled on harbors.goharbor.io "sample": the object has been modified; please apply your changes to the latest version and try again: cannot prepare owner status: cannot update status: Operation cannot be fulfilled on harbors.goharbor.io "sample": the object has been modified; please apply your changes to the latest version and try again

This is because the object has been outdated when doing status updates. No high influence so far, we can do some improvements later when we get a stable version.

Close as it is not valid anymore.


isort inconsistencies for the same version but different Python versions

I've run into a bit of a puzzle and am not entirely sure what's going on. It seems extremely similar to #379, but reversed this time. I'm currently using isort version 4.3.4, and I've managed to reproduce this failure case only when downgrading to Python 3.7.1 from Python 3.7.4 locally.
This error doesn't make a whole lot of sense as it only occurs under Python 3.7.1. If this condition is not met, I can't reproduce this. I understand this probably a pretty low priority bug because it is only reproducible for me under such specific circumstances, but do you have any idea what might be going on? Interestingly, the formatter yapf had a very similar problem when jumping between these versions and I'm not sure if both of these projects use a feature that has changed between those two versions that you're aware of?

Just in case this is useful, I had a similar problem in which I was getting a different result in the CI environment and in tox. In the end, what was different was that in the CI environment the package was installed in development mode and in tox it wasn't.
In this case, the fix was either telling isort that my package was a known first party import (setup.cfg):
[isort]
known_first_party = <my_package>

or telling tox to install the package in development mode (tox.ini):
[testenv]
usedevelop = true

Thanks for the response. I'll see if any of the available options help here.


ITI-65: Need guidance for List.status when grouping with XDS/XDR/XCDR

Section Number <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>
Issue There are two issues with Folder List.status when grouping MHD with XDS/XDR/XCDR: 1. The mappings to XDS in section <IP_ADDRESS> and <IP_ADDRESS> do not map the value "current" to "Approved" 2. XDS Submission Sets and Folders are not allowed to be deprecated. What should happen if an ITI-65 request includes a different status?
Proposed Change 1. Add mappings to <IP_ADDRESS> and <IP_ADDRESS>. 2. Add guidance to reject any List.status other than "current".
Priority: Medium

@jlamy I am not understanding a solution.

The code mapping is found in Table 2:<IP_ADDRESS>.2.1-1. -- see http://build.fhir.org/ig/IHE/ITI.MHD/branches/master/ITI-66.html
MHD is not the right place for rules around "not allowing deprecation".

so I am not understanding any change to make.


Services: missing example of the error case in hello_services

We're not sure how to handle an error when writing a service. We didn't see an example of error cae in https://github.com/flutter/flutter/blob/master/examples/hello_services/android/app/src/main/java/com/example/flutter/ExampleActivity.java
We'd love to know:

how to signal an error back up to Dart from the Java and ObjC code

fyi @faisalabid

@szakarias @mravn-google don't we have this now?

In https://github.com/flutter/flutter/tree/master/examples/platform_services, I mean

It's part of platform_services now, yes.


Logs not Generating

Dear Kozmer,
Thanks for this POC. I am able to follow the do the POC.
I wanted to forward the logs from the host to some SIEM and create detection rules.
Please let me know which files contains the logs generated.

You could probably get a shell in the Docker container and check for logs, this is something I have not done so I wouldn't be much help here.. Sorry!


http_in plugin respond with 200 OK to GET /

Is your feature request related to a problem? Please describe.
I am trying to run Fluentd by deploying a docker container to an Azure App Service. App Service is designed for running HTTP APIs and websites and expects an application to respond with HTTP 200 to GET / before it will route any requests to it.
Describe the solution you'd like
I would like to have http_in respond with 200 OK to a GET / request.
Describe alternatives you've considered
Running the http_in plugin behind nginx within the same container andn having nginx handle GET /. This isn't preferred for a few reasons though.

Additional context
N/A

Is this reasonable? I am happy to work on it & open a PR.

I would like to have http_in respond with 200 OK to a GET / request.

@JoshKeegan Should be fixed by #3373.

I would like to have http_in respond with 200 OK to a GET / request.

@JoshKeegan Should be fixed by #3373.

Thankyou for the quick response & PR!