[BUG] Unable to use date_histogram as a transform group

Describe the bug
When trying to create a transform job on the "Sample web logs" index to aggregate on 5min buckets, I get the following response, even if the documentation states that date_histogram is available and has a "field" field:
{
  "error" : {
    "root_cause" : [
      {
        "type" : "illegal_argument_exception",
        "reason" : "Invalid field [field] found in date histogram"
      }
    ],
    "type" : "illegal_argument_exception",
    "reason" : "Invalid field [field] found in date histogram"
  },
  "status" : 400
}

To Reproduce
Steps to reproduce the behavior (on Opensearch-Dashboards):

Generate the "Sample web logs" though the "Add sample data" menu
Create the following transform job (I'm using "Dev Tools" > "Console"):

PUT _plugins/_transform/agregate_level_1
{
  "transform": {
    "enabled": true,
    "continuous": false,
    "schedule": {
      "interval": {
        "period": 1,
        "unit": "Minutes",
        "start_time":<PHONE_NUMBER>
      }
    },
    "description": "Agregate data on 5min buckets",
    "source_index": "opensearch_dashboards_sample_data_logs",
    "target_index": "transform_level_1",
    "data_selection_query": {
      "match_all": {}
    },
    "page_size": 1,
    "groups": [
      {
        "date_histogram": {
          "field": "@timestamp",
          "interval": "5m"
        }
      }
    ],
    "aggregations": {
      "data_transfer": {
        "sum": {
          "field": "bytes"
        }
      }
    }
  }
}

You get the error as reply:

{
  "error" : {
    "root_cause" : [
      {
        "type" : "illegal_argument_exception",
        "reason" : "Invalid field [field] found in date histogram"
      }
    ],
    "type" : "illegal_argument_exception",
    "reason" : "Invalid field [field] found in date histogram"
  },
  "status" : 400
}

Note: the result is the same with "field": "timestamp" instead of "field": "@timestamp" (both objects exist and have the same type/value)
Expected behavior
The transform job is created and produce a new index with buckets of 5min
Plugins
opensearch-cluster-master-0 opensearch-alerting                  <IP_ADDRESS>
opensearch-cluster-master-0 opensearch-anomaly-detection         <IP_ADDRESS>
opensearch-cluster-master-0 opensearch-asynchronous-search       <IP_ADDRESS>
opensearch-cluster-master-0 opensearch-cross-cluster-replication <IP_ADDRESS>
opensearch-cluster-master-0 opensearch-index-management          <IP_ADDRESS>
opensearch-cluster-master-0 opensearch-job-scheduler             <IP_ADDRESS>
opensearch-cluster-master-0 opensearch-knn                       <IP_ADDRESS>
opensearch-cluster-master-0 opensearch-ml                        <IP_ADDRESS>
opensearch-cluster-master-0 opensearch-observability             <IP_ADDRESS>
opensearch-cluster-master-0 opensearch-performance-analyzer      <IP_ADDRESS>
opensearch-cluster-master-0 opensearch-reports-scheduler         <IP_ADDRESS>
opensearch-cluster-master-0 opensearch-security                  <IP_ADDRESS>
opensearch-cluster-master-0 opensearch-sql                       <IP_ADDRESS>

Screenshots
NC
Host/Environment:

Kubernetes (MicroK8s):

Client Version: v1.26.5
Kustomize Version: v4.5.7
Server Version: v1.26.5

Helm repo:

opensearch              https://opensearch-project.github.io/helm-charts

Helm dependencies:

dependencies:
  - name: opensearch
    version: "1.20.2"
    repository: '@opensearch'
  - name: opensearch-dashboards
    version: "1.13.1"
    repository: '@opensearch'

Additional context
In the end I would even like to aggregate on more fields to have the following groups if it's allowed:
    "groups": [
      {
        "date_histogram": {
          "field": "@timestamp",
          "interval": "5m"
        }
      },
      {
        "terms": {
          "source_field": "url.keyword",
          "target_field": "url"
        }
      },
      {
        "terms": {
          "source_field": "clientip",
          "target_field": "ip"
        }
      }
    ],

I can't upgrade to a much greater version than 1.3.10 for now, I'm bound to it because of my service provider... :disappointed:

Hello, I also reproduced this behavior with the following chart versions (+ adding the corresponding appVersion for each):

opensearch

appVersion: 2.8.0
version: 2.13.2
appVersion: 2.4.1
version: 2.9.1
appVersion: 1.3.10
version: "1.20.2"

opensearch-dashboards

appVersion: 2.8.0
version: 2.11.1
appVersion: 2.4.1
version: 2.7.0
appVersion: 1.3.10
version: "1.13.1"

Hello, here are some new elements...
I checked also through the transform job creation webUI (Opensearch Dashboards > Index Management > Transform Jobs) for all the above versions and here are my results:

1.3.10 => broken webUI at step 2 (Define transform), only a blank page is shown on Firefox, unable to test
2.4.1 => only grouping option is by terms, even on @timestamp field and only aggregation is by count
2.8.0 => same as 2.4.1

I'm really surprised this feature is not supported, as it is supposed to be available according to the documentation of all those versions:

1.3.x => https://opensearch.org/docs/1.3/im-plugin/index-transforms/transforms-apis/#request-body-fields
2.4.x => https://opensearch.org/docs/2.4/im-plugin/index-transforms/transforms-apis/#request-body-fields
2.8.x => https://opensearch.org/docs/2.8/im-plugin/index-transforms/transforms-apis/#request-body-fields

All are stating the same:

Option
Data Type
Description
Required

groups
Array
Specifies the grouping(s) to use in the transform job. Supported groups are terms, histogram, and date_histogram. For more information, see Bucket Aggregations.
Yes if not using aggregations.

Hello, it seems the issue is more the way parameters are documented and/or errors reported, as when removing all parameters a more useful error is shown:
PUT _plugins/_transform/agregate_level_1
{
  "transform": {
    "enabled": true,
    "continuous": false,
    "schedule": {
      "interval": {
        "period": 1,
        "unit": "Minutes",
        "start_time":<PHONE_NUMBER>
      }
    },
    "description": "Agregate data on 5min buckets",
    "source_index": "opensearch_dashboards_sample_data_logs",
    "target_index": "transform_level_1",
    "data_selection_query": {
      "match_all": {}
    },
    "page_size": 1,
    "groups": [
      {
        "date_histogram": {}
      }
    ],
    "aggregations": {
      "data_transfer": {
        "sum": {
          "field": "bytes"
        }
      }
    }
  }
}

{
  "error" : {
    "root_cause" : [
      {
        "type" : "illegal_argument_exception",
        "reason" : "Source field must not be null"
      }
    ],
    "type" : "illegal_argument_exception",
    "reason" : "Source field must not be null"
  },
  "status" : 400
}

It seems there was an attempt to document this, but being at the same level as groups and aggregations this line never comes (at least for me) as an expected replacement of the field parameter of terms, histogram or date_histogram:

Option
Data Type
Description
Required

source_field
String
The field(s) to transform.
Yes

Maybe it should be more useful and obvious if described as an extra line in groups description...
The target_field and its behavior (if not specified, it equals source_field from what I was able to test) should also be documented at the same place...

Hello, any news about the date_histogram not allowing @timestamp as source field ? as this remains a bug related to date_histogram while used in transform jobs to me...
The documentation issue was a first wall I faced to try using it, but the key issue here is date_histogram not being able to use a date field as source... Kind of sad according to the name :sweat_smile:

This is the supported format for date_histogram in transform job:
        "date_histogram": {           "source_field": "timestamp",           "calendar_interval": "minute"         }

Hello @kvitali,
Thanks for pointing the good format, though I won't be able to confirm it works, as we had to move to Elasticsearch since then...
I guess this should be either:

fixed in the Transform API to match the format of the Search API (much cleaner, but would be a breaking change)
or documented in https://opensearch.org/docs/latest/aggregations/bucket/date-histogram/ that field becomes source_field when used within the Tranform API


Update create-trigger-transact-sql.md

There are 2 functions:
@@ROW_COUNT or ROWCOUNT_BIG()

@baleng : Thanks for your contribution! The author, @CarlRabeler, has been notified to review your proposed change.


undefined local variable or method `current_user' for #RailsAdmin::MainController:0x00007ff680a0e8e8 Did you mean? _current_user

This is ralls_admin.rb
RailsAdmin.config do |config|
config.actions do
dashboard
index
new
export
bulk_delete
show
edit
delete
show_in_app
end
end

https://github.com/sferik/rails_admin/issues/3228#issuecomment-894754042


Refactor Awesome-books to use classes

create DOM element class and constructor
create a Local storage constructor
create an initial function to get the data and create the elements on loading

Add the delete method to the LocalStorage class as requested
Make the error message in red color


deleteUser returns a response with FormatException

I made an edge function that is calling supabase.auth.admin.deletUser(....). It is deleting the user as expected but it returns just
"User deleted: {
  "user": {}
}"

which is then leading to an format exception.
This is my called edge function i call within flutter.
serve(async (req: Request) => {
  try {
    // Create a Supabase client with the Auth context of the logged in user.
    const supabaseClient = createClient(
      // Supabase API URL - env var exported by default.
      Deno.env.get('SUPABASE_URL') ?? '',
      // Supabase API ANON KEY - env var exported by default.
      Deno.env.get('SUPABASE_ANON_KEY') ?? '',
      // Create client with Auth context of the user that called the function.
      // This way your row-level-security (RLS) policies are applied.
      { global: { headers: { Authorization: req.headers.get('Authorization')! } } }
    );
    // Now we can get the session or user object
    const {
      data: { user },
    } = await supabaseClient.auth.getUser();

    const supabaseAdmin = createClient(
      Deno.env.get('SUPABASE_URL') ?? '',
      Deno.env.get('SUPABASE_SERVICE_ROLE_KEY') ?? ''
    );
    const { data: deletion_data, error: deletion_error } = await supabaseAdmin.auth.admin.deleteUser(user.id, true);
    if (deletion_error) throw deletion_error;

    return new Response("User deleted: " + JSON.stringify(deletion_data, null, 2), {
            headers: { "Content-Type": "application/json" },
            status: 200,
          });
  } catch (error) {
    return new Response(JSON.stringify({ error: error.message }), {
      headers: { 'Content-Type': 'application/json' },
      status: 400,
    });
  }
})

Is it wrong to catch the result from deletUser? If so, how do I recognize then if it worked?
flutter: ----------------FIREBASE CRASHLYTICS----------------
flutter: FormatException: Unexpected character (at character 1)
User deleted: "fd4f1aab-76af-41b3-95e8-22048f81defa"
^
flutter:
#0      _ChunkedJsonParser.fail (dart:convert-patch/convert_patch.dart:1383:5)
#1      _ChunkedJsonParser.parseNumber (dart:convert-patch/convert_patch.dart:1250:9)
#2      _ChunkedJsonParser.parse (dart:convert-patch/convert_patch.dart:915:22)
#3      _parseJson (dart:convert-patch/convert_patch.dart:35:10)
#4      JsonDecoder.convert (dart:convert/json.dart:610:36)
#5      JsonCodec.decode (dart:convert/json.dart:216:41)
#6      jsonDecode (dart:convert/json.dart:155:10)
#7      _compute
_isolates_io.dart:109
<asynchronous suspension>
flutter: ----------------------------------------------------

The actual exception seems then to be thrown because _event.next has a bad state for whatever reason. There is basically no information that points me to the reason.

Your result from the edge function is not json. You have to wrap it with {}

Thank you that was the cause


Update Cache in-memory for RP

Can we get a sample for caching which targets the Razor Pages/Page Models workflow which is now recommended? I can't find one anywhere...

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

ID: 13f5f8f9-18dd-9c12-9ed1-fcc6b44ba7fd
Version Independent ID: 640b4f36-5886-7b69-1c95-cfc8574a02fd
Content: Cache in-memory in ASP.NET Core
Content Source: aspnetcore/performance/caching/memory.md
Service: asp.net-core
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

It's pretty much the same thing, but rather than a controler you have a page model. I've scheduled this for update.

Thanks!
Is it possible to do on Partial Views?
e.g. I have a sidebar which displays some menus, and shows status of each menu item (# items, etc.). The Partial View receives a Model, but not a page model.
If I add a cache taking constructor to the model, will it work?

fixed in #7792


Why is coveralls not a dev dependency?

Apologies if this has been discussed prior, I was unableto find an issue regarding it.
Is there a reason that the coveralls module is not a dev dependency? I understand that @isaacs likes and uses it, but I am unclear why it is required for the usage of tap, coverage generation is covered by nyc, correct?
questions and comments on this account are not related to my work and are personal only

Tap includes a coveralls reporter built in.

I don't actually use coveralls, myself.  We've been planning on removing the coveralls reporter at some point, since it seems to not be very well maintained, brings in a bunch of deprecated modules, etc.  But it is properly a "dependency" (as in, "used for some actual real features") and not a "devDependency" (as in, "used in the development of tap itself".)  I guess we could move it to an optional peerdep in the next major?

+1 on removing/making optional for all the reasons you mentioned. Looks like coveralls is maintaining a GitHub action now in favor of their travis stuff.


Remove usage of deprecated logger.warn function

logger.warn() is deprecated, suggest logger.warning() in README instead.

Coverage remained the same at 92.593% when pulling 267600f526f8726762c33287348455dd7c9a533e on johnthagen:patch-1 into baacedb3f7b0b8c857e0d9f04a9eee6abba9122f on xolox:master.

Thanks for the suggestion! I've merged #34, #37 and #38 and published the changes to PyPI as v7.2.


FINERACT-1410 :Allow Payment types to map to all types of Ledgers

Description
Describe the changes made and why they were made.
Ignore if these details are present on the associated Apache Fineract JIRA ticket.
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[x] Write the commit message as per https://github.com/apache/fineract/#pull-requests

[x] Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.

[x] Create/update unit or integration tests for verifying the changes made.

[x] Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.

[x] Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/api-docs/apiLive.htm with details of any API changes

[x] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the developer mailing list for guidance, if required.)

FYI our guidelines for code reviews are at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide.

@ankita10r @avikganguly01 in the testings I see these messages:
org.apache.fineract.integrationtests.SkipRepaymentOnMonthFirstTest
Test testSkippingRepaymentOnFirstDayOfMonth() FAILED (3.8s)
org.opentest4j.AssertionFailedError: expected: <30> but was: <32>
at app//org.apache.fineract.integrationtests.SkipRepaymentOnMonthFirstTest.tearDown(SkipRepaymentOnMonthFirstTest.java:71)
Test checkRepaymentSkipOnFirstDayOfMonth() SKIPPED
FAILURE: Executed 86 tests in 2m 16s (1 failed, 2 skipped)
Are they expected?

Dear Victor,
Most tests do not pass (at least for me) I think i have seen more than 10
tests fail, so Ignore them most of the time. before you send a PR below are
the steps you must do
STEPS TAKEN
1a- Add your Changes
git add .
1b- Apply Spotless
./gradlew spotlessApply
2- Add your Changes
git add .
3- To Build War / Jar for deployment (any one below)
./gradlew clean bootWar  -- boots and builds war file
./gradlew clean war      -- builds war file
./gradlew bootJar        -- builds jar file
Then you commit and push PR
When these 3 sections pass on your PC then your PR should be OK
My Kind Regards
Francis Guchie Kirago
Skype: francisguchie
*Telegram: *232 79 19 44 07
*Whatsapp: *232 79 19 44 07
LINKEDIN: https://www.linkedin.com/in/francis-guchie-kirago-a4379617/
twitter: @FrancisGuchie
On Tue, Oct 19, 2021 at 7:49 PM Víctor Romero @.***>
wrote:

@ankita10r https://github.com/ankita10r @avikganguly01
https://github.com/avikganguly01 in the testings I see these messages:
org.apache.fineract.integrationtests.SkipRepaymentOnMonthFirstTest
Test testSkippingRepaymentOnFirstDayOfMonth() FAILED (3.8s)
org.opentest4j.AssertionFailedError: expected: <30> but was: <32>
at
app//org.apache.fineract.integrationtests.SkipRepaymentOnMonthFirstTest.tearDown(SkipRepaymentOnMonthFirstTest.java:71)
Test checkRepaymentSkipOnFirstDayOfMonth() SKIPPED
FAILURE: Executed 86 tests in 2m 16s (1 failed, 2 skipped)
Are they expected?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/fineract/pull/1894#issuecomment-947053724, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFNCABTVWJ7KN4EA477EXE3UHXDWJANCNFSM5GG2FFOQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@avikganguly01 @francisguchie @ankita10r @IOhacker
This has now broken the build, and all the other builds are failing.
We should never merge failing builds please.
I will try to revert this change - please can you fix the failing tests before merging again.

And yes, if you look at the code and the failing test, it's obvious that this PR broke the test.
But the point is: it should be the responsibility of the person submitting the PR to ensure the build passes before it is submitted, not for us to prove that the failure is due to their PR.

Hi @avikganguly01 (cc @awasum @fynmanoj)
Unfortunately I disagree with your description of the current state of the Fineract branch. Looking at the pull request history, I've merged around 350 PRs over the two years or so. For each of these, I've ensured all the tests pass before merging. So, in my view, this is not an unrealistic expectation. In fact, this is the only way we can make sure things improve rather than degrade.
I agree there are still some flaky tests. But we've made a huge amount of progress on this over the last year: I've fixed at least 30 such tests myself. There are still a couple of known issues with:
a) performance of Travis (sometimes the build takes too long due to load on the Travis infra, and some tests time out)
b) month end (some tests fail on last / first day of months, due to the fact that different months have different number of days and this confuses the checking routines for interest calculation)
I agree we should fix these. But these handful of issues should not be an excuse for allowing code to be merged that breaks additional tests, without even investigating why the tests are failing. That will take us even further away from our goal of having reliable tests.


VS Code Figma toolkit improvements

Here are some of the improvements and updates that will be added to the VS Code Figma toolkit:

[x] Update the quick pick component to include multi-select scenarios
[x] Add left + right placement of panels
[x] Update the Settings UI template to include the new hover state for the light theme
[x] Fix a few bugs in the Settings UI template
[x] Fix button hover interaction

Moved this out but maybe it's actually done?


Unable to add bot in supergroup

I'm unable to add the bot in super groups.
I get the following error.
Method: channels.inviteToChannel
Result: {"_":"rpc_error","error_code":400,"error_message":"CHAT_ADMIN_REQUIRED"}
Stack: Error
at u (https://web.telegram.org/js/app.js:18:3697)
at https://web.telegram.org/js/app.js:18:4996
at l (https://web.telegram.org/js/app.js:8:11738)
at https://web.telegram.org/js/app.js:8:11910
at m.$eval (https://web.telegram.org/js/app.js:8:19316)
at m.$digest (https://web.telegram.org/js/app.js:8:17747)
at https://web.telegram.org/js/app.js:8:19403
at a (https://web.telegram.org/js/app.js:7:2845)
at https://web.telegram.org/js/app.js:7:4299
at n (https://web.telegram.org/js/app.js:2:17073)

What does this have to do with Telegram Bot in general, or telepot in particular?


I don't know why but even though I have filled up the pull_request_template.md file, it is not showing on the preview section here but shows in the "Files Changed" section. Kindly waive this anomaly (maybe) if possible and go to the "File Changed" section for the edited pull_request_template.md file.

Thank you very much for your suggestions! I am working on it.


[WIP] Move Build watching to FastlaneCore

WIP
remove code duplication of build processing watching. as suggested by @KrauseFx
currently testing it.

This looks super promising, excited about all the duplicate code to be removed 👍
Let me know when this is ready for the first review

ok looking good so far - ready for review.
going to close this PR - and submit the splitted ones.


Applied changes after upstream changes

I redid the previous changes of this fork's divergence on top of the new changes in the upstream repo. Changes in upstream made so we didn't have to change as much this time (now BOM stripping and sourcemaps are optional). The browserifiability may have been broken, how can I test this? (I don't know how this module is being used in the browser).

The browserifiability may have been broken, how can I test this? (I don't know how this module is being used in the browser).

Inside the one and only, node-ipfs-api :D
https://www.npmjs.com/package/vinyl-fs-browser this is @jbenet's fork that is used by node-ipfs-api
https://github.com/ipfs/node-ipfs-api/blob/master/package.json#L11-L13

OK, the node-ipfs-api tests pass with this, and a simple manual test with browserify seems to work (although we should set up the tests to run in the browser).

@jbenet Could you please give me publish access for this repo on npm? (Or just publish the latest master).

@mappum does it work when you drag + drop files to the menubar of electron-app?

(or probably the drag+drop webui)

@mappum you can publish access, but pls pls confirm things are not broken. people depend on these modules now. ideally make ipfs-api run browser tests?

Confirmed working on latest webui.

Thanks!
—
Sent from Mailbox
On Thu, Oct 1, 2015 at 11:50 PM, ᴍᴀᴛᴛ ʙᴇʟʟ<EMAIL_ADDRESS>wrote:

Confirmed working on latest webui.
Reply to this email directly or view it on GitHub:
https://github.com/jbenet/vinyl-fs/pull/2#issuecomment-144910786


Layout not fitting screen when orientation changed.

If I change the orientation in Iphone, the swiper won't view items as stretched. However when I reload as orientation changed, it fits the horizontal screen. How can I fix this?

First Open

Orientation Changed

Reload after orientation changed

Rotate again to horizontal screen

If you are not clear what I am telling here you might want to hightlight images by dragging them.


3.0.x docs build

What does this PR (Pull Request) do?
Fixes docs build in docker by upgrading python build and splitting up yum install from command that uses what was just installed (python3).  This avoids a spurious race condition in building docs in docker.

[x] This PR fixes #3466 for the 3.0.x branch.

Which Traffic Control components are affected by this PR?

Documentation

What is the best way to verify this PR?
./pkg -v docs

Test is documentation builds.  No CHANGELOG.md necessary.  No documentation change needed.
The following criteria are ALL met by this PR

[x] This PR includes tests OR I have explained why tests are unnecessary
[x] This PR includes documentation OR I have explained why documentation is unnecessary
[x] This PR includes an update to CHANGELOG.md OR such an update is not necessary
[x] This PR includes any and all required license headers
[x] This PR ensures that database migration sequence is correct OR this PR does not include a database migration
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3479/
Test PASSed.


[glm] build failure - SHA512 hash mismatch

Operating system
Windows 11 10.0.22621.3155 x64 Enterprise
Compiler
Visual Studio Community 2022 Version: [17.9.2]  VS Tools Version: [14.39.33519]
Microsoft (R) C/C++ Optimizing Compiler Version [19.39.33521] for x64
Vcpkg Information
Vcpkg Version [2024-02-27-29a99e84b4d7bf30e43466b1572ea6b392dcf4e3]
Vcpkg Scripts Version [2024-02-27-3dd44b931481d7a8e9ba412621fa810232b66289]
Steps to reproduce the behavior
`.\vcpkg.exe install glm`

Failure logs
Installing 39/72<EMAIL_ADDRESS>Building<EMAIL_ADDRESS>-- Note: glm only supports static library linkage. Building static library.
-- Downloading https://github.com/g-truc/glm/archive/1.0.1.tar.gz -> g-truc-glm-1.0.1.tar.gz...
[DEBUG] To include the environment variables in debug output, pass --debug-env
[DEBUG] Disabling metrics because vcpkg.disable-metrics exists
[DEBUG] Trying to load bundleconfig from D:\vcpkg\vcpkg-bundle.json
[DEBUG] Failed to open: D:\vcpkg\vcpkg-bundle.json
[DEBUG] Bundle config: readonly=false, usegitregistry=false, embeddedsha=nullopt, deployment=Git, vsversion=nullopt
[DEBUG] Feature flag 'binarycaching' unset
[DEBUG] Feature flag 'compilertracking' unset
[DEBUG] Feature flag 'registries' unset
[DEBUG] Feature flag 'versions' unset
[DEBUG] Feature flag 'dependencygraph' unset
Downloading https://github.com/g-truc/glm/archive/1.0.1.tar.gz
[DEBUG] Trying to hash D:\vcpkg\downloads\g-truc-glm-1.0.1.tar.gz.6100.part
[DEBUG] D:\vcpkg\downloads\g-truc-glm-1.0.1.tar.gz.6100.part has hash c6c6fa1ea7a7e97820e36ee042a78be248ae828c99c1b1111080d9bf334a5160c9993a70312351c92a867cd49907c95f9f357c8dfe2bc29946da6e83e27ba20c
error: Failed to download from mirror set
error: File does not have the expected hash:
url: https://github.com/g-truc/glm/archive/1.0.1.tar.gz
File: D:\vcpkg\downloads\g-truc-glm-1.0.1.tar.gz.6100.part
Expected hash: 0296a9bac8a114f50aa719b402d66524a4028aeec72649da1964faa80116f3f1ae5fa13d741dbaf9f4bcdb0722c0e44a4e85e8fc16dda641f4ba03f4a2c755d3
Actual hash: c6c6fa1ea7a7e97820e36ee042a78be248ae828c99c1b1111080d9bf334a5160c9993a70312351c92a867cd49907c95f9f357c8dfe2bc29946da6e83e27ba20c
[DEBUG] D:\vcpkg-tool\src\vcpkg\base\downloads.cpp(1030):
[DEBUG] Time in subprocesses: 0us
[DEBUG] Time in parsing JSON: 10us
[DEBUG] Time in JSON reader: 0us
[DEBUG] Time in filesystem: 76us
[DEBUG] Time in loading ports: 0us
[DEBUG] Exiting after 1.1 s (1118400us)
CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:32 (message):
  Failed to download file with error: 1
  If you are using a proxy, please check your proxy setting. Possible causes are:

  1. You are actually using an HTTP proxy, but setting HTTPS_PROXY variable
     to `https://address:port`. This is not correct, because `https://` prefix
     claims the proxy is an HTTPS proxy, while your proxy (v2ray, shadowsocksr
     , etc..) is an HTTP proxy. Try setting `http://address:port` to both
     HTTP_PROXY and HTTPS_PROXY instead.

  2. If you are using Windows, vcpkg will automatically use your Windows IE Proxy Settings
     set by your proxy software. See https://github.com/microsoft/vcpkg-tool/pull/77
     The value set by your proxy might be wrong, or have same `https://` prefix issue.

  3. Your proxy's remote server is out of service.

  If you've tried directly download the link, and believe this is not a temporary
  download server failure, please submit an issue at https://github.com/Microsoft/vcpkg/issues
  to report this upstream download server failure.

Call Stack (most recent call first):
scripts/cmake/vcpkg_download_distfile.cmake:270 (z_vcpkg_download_distfile_show_proxy_and_fail)
scripts/cmake/vcpkg_from_github.cmake:106 (vcpkg_download_distfile)
ports/glm/portfile.cmake:3 (vcpkg_from_github)
scripts/ports.cmake:170 (include)
error: building glm:x64-windows failed with: BUILD_FAILED
Elapsed time to handle glm:x64-windows: 1.2 s
Additional context
I downloaded the file manually and checked SHA512 with hashtools.
The SHA512 from the downloaded file is c6c6fa1ea7a7e97820e36ee042a78be248ae828c99c1b1111080d9bf334a5160c9993a70312351c92a867cd49907c95f9f357c8dfe2bc29946da6e83e27ba20c which matches the one reported by vcpkg as being incorrect, does not match the SHA512 from the portfile.cmake of 0296a9bac8a114f50aa719b402d66524a4028aeec72649da1964faa80116f3f1ae5fa13d741dbaf9f4bcdb0722c0e44a4e85e8fc16dda641f4ba03f4a2c755d3

I just encountered the same issue a couple of minutes ago! Good to know there is a fix for it!


New package: UserBenchmark.UserBenchmark version <IP_ADDRESS>

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Installation failed with exit code -1978335226 2024-02-09 18:34:15.880 [CLI ] ShellExecute installer failed: 1

(Automated response - build 720.)


[shelter-ui] Button has low contrast in dark theme

I found that Button has low contrast if user use dark theme.
Example with catppuccin-mocha. Both client are in dark mode.
Shelter (in Dorion client)

Compared to Vencord (in official Discord client). This is the same as Discord's default button style

I expect Button style should be similar to Discord's default button style in dark theme. But correct me if this is intended.

Thank you for your reply. You are correct that it seems it's due to catppuccin theme isn't using --button-filled-brand-text variable as button text color. However, it seems shelter-ui is also not using --button-filled-brand-text as button text color. Instead shelter-ui button use --interactive-active.
https://github.com/uwu/shelter/blob/061ab9bba8c9a7540a45efcc400e64bbf6f891ba/packages/shelter-ui/src/button.tsx#L28-L30

And it seems --interactive-active is also used in other component such as text in selected friend in friend list. This is how it looks if I set --interactive-active to #000000.

May I know which one is the correct variable to use for button text color? Is it --button-filled-brand-text or --interactive-active?

I do believe shelter UI is wrong about which variable to use here, it should be button-filled-xxxx-text, and I do plan to change this, but this theme would still sadly be broken

but this theme would still sadly be broken

It's okay no worries I could help to discuss this in catppuccin repo. Nice to hear that it's acknowledged at least. Maybe we could reopen the ticket to track the fix for shelter-ui? 👀

that is actually a sensible idea yeah sure

Speaking of themes, is there any reference theme implementation that we could use as base/reference for making themes? So far I found BetterDiscord variables but let me know if there's better/more preferable reference.

I'm not aware of any better resource, or any resources at all really for theming other than just seeing what others do and mimicking that.
quite honestly my method back when I did theming was just to find the thing I wanted to change and see what variable it used, then see if overriding that had the desired effect without breaking everything else.


brightness: failed to get brightness of display 0x2b107b14 (error -536870201)

brightness: failed to get brightness of display 0x2b107b14 (error -536870201)

Please try the current master and see if it works. If it doesn't, what display/OS/Mac are you using?

Closing as I haven't heard back. Please feel free to comment if this is still a problem.

Same issue for me

@MattAlp Can you be more specific — are you using the current version of brightness built from Git? If so, what display and what macOS version are you using? Does adjusting brightness work from the keyboard/Displays System Preferences? (If not, then brightness isn't going to help you — see the README).

Sure;

I have obtained brightness from Brew at the time of posting this issue. I am using my Mac display and external display monitor connected over USB-C + an HDMI adapter, on a 2016 MBP running 10.15.7 Catalina.
Adjusting brightness works fine through conventional flows.
Interestingly, the brightness does change when running from the terminal, as expected, but then returns to the previous level after a short delay (this is unrelated to notifications or DnD settings).

OK, I think that's the problem - the Homebrew build is not updated
with the latest changes. I don't know how to fix that - perhaps you
can try reporting it to Homebrew?
On Mon, Mar 22, 2021 at 11:55:02AM -0700, Matt wrote:

Sure;

I have obtained brightness from Brew at the time of posting this issue. I am using my Mac display and an external display monitor connected over USB-C + an HDMI adapter, on a 2016 MBP running 10.15.7 Catalina.
Adjusting brightness works fine through conventional flows.
Interestingly, the brightness does change when running from the terminal, as expected, but then returns to the previous level after a short delay (this is unrelated to notifications or DnD settings).

--
--Nicholas


Broadcasting bug

https://github.com/jonniedie/ComponentArrays.jl/blob/85f99a369b2cf7204fabe30ed5a458af18d5b349/src/broadcasting.jl#L74
The line above creates problems in the situation the comment above it claims that it improves. If f changes the type of the numerical values, they will not fit in new_data.

Thanks! That was a good catch.
Closed by 5c4242e.


Organize Full Technology Stack

Base repository is forked from: https://github.com/get-convex/turbo-expo-nextjs-clerk-convex-monorepo/
Web UI is written in typescript, and is React native.
Backend database is powered by Convex and allows for real-time and transactional behavior (important at scale)
LLM and Embedding components are provider-agnostic, but most development is done using TogetherAI for compute
For MVP, Vectara simplifies document ingestion/chunking/retrieval with RAG-as-a-service and a compatible SDK.
Moving forward, Convex Auth can replace Clerk for authentication!

Orchestration Concerns:

There have been some great improvements in recent LLMs, and function-calling is now baked natively into models like the Llama-3.1 family. For instance: https://docs.together.ai/docs/llama-3-function-calling is a great resource by TogetherAI for handling such a test using a specific subset of models. This, however, is not very flexible to LLM or provider! I should not invest too many hours in non-robust components, but for MVP/proof-of-concepts, they work.
All steps should conform to LangGraph framework in any given workflow. This means approaching each task with state organized and the solution space well defined. This will be more important as RAG gets more complicated, like adding filter kwargs to a search in a vector space.

The flow will look a little something like this:

User visits textbook-ai website, signs in, and is directed to app page
User uploads a PDF (Like Intro to Physics) into memory
A TextbookSchema is generated and assured to be valid (Uses LLM component)
Chunk the textbook with appropriate metadata (page number, other relevant fields) - Note that a function getMetadataByPage would still need information that should be collected @ step 3. Something like mapping the page ranges to the chapter/sections (Should be in table of contents)
Add these chunks and their metadata to a Convex table
Create embeddings for each chunk

At this point, indexing is done!

After indexing a textbook, here's an abstracted version of what flows should look like:
INPUT: The active settings, the current state, and the user input (button press or string input)
OUTPUT: GOAL (Retrieved content, Content + Text, or just Text (covers fail case)
The flow:

Collect inputs
Identify GOAL, or fail (if input can't be resolved to a valid goal)
Begin appropriate LangGraph workflow!
Stream steps to completion or fail (Ensures deterministic failure behavior with handled side effects)
Serve GOAL (e.g. Excerpts of the textbook content, with the appropriate pages/sections provided, + add a summary cuz why not, AI is gimmicky but it can be helpful)

The reason to make this a touch more complicated with LangGraph (at least for now) is because of the side effects when handling eval functions or other substeps that may fail, but can be soundly reasoned through.

I still agree with most of the design patterns here! Here's a lil AI generated strategy based on these notes:
Here's a comprehensive product plan organized from your technical notes:

# TextbookAI - Product Development Plan

## Product Overview
TextbookAI is an intelligent learning platform that transforms standard textbooks into interactive, searchable content with AI-powered features. The platform allows users to upload educational materials and interact with them in novel ways through advanced AI capabilities.

## MVP Feature Set

### Core Functionalities
1. User Authentication
   - Secure sign-in system
   - Personal user dashboard

2. Document Processing
   - PDF upload capability
   - Automatic textbook structure analysis
   - Smart content indexing

3. Interactive Features
   - Intelligent content search
   - Context-aware responses
   - Real-time content navigation

## Technical Architecture (MVP)

### Frontend
- React Native (TypeScript)
- Responsive web interface
- Real-time updates support

### Backend
- Convex database for real-time data handling
- TogetherAI for AI compute
- Vectara for document processing (RAG-as-a-service)
- Clerk for authentication (Phase 1)

## User Flow

1. **Onboarding**
   - User visits website
   - Creates account/signs in
   - Arrives at dashboard

2. **Content Upload**
   - User uploads PDF textbook
   - System processes document
   - Automatic content structuring

3. **Interaction**
   - Search within textbook
   - Receive AI-enhanced responses
   - Access relevant excerpts with page references

## Development Phases

### Phase 1 (MVP)
- Basic authentication
- PDF upload and processing
- Simple search and retrieval
- Basic AI interactions

### Phase 2
- Enhanced AI capabilities
- Advanced search features
- Improved content structuring
- Migration to Convex Auth

### Phase 3
- Additional document format support
- Advanced analytics
- Collaborative features
- Custom AI model optimization

## Technical Considerations

### Scalability
- Real-time database capabilities
- Transactional behavior support
- Efficient content processing

### Security
- Secure authentication
- Data encryption
- Privacy compliance

### AI Integration
- Provider-agnostic architecture
- Flexible LLM integration
- Structured workflow management

## Success Metrics
1. User adoption rate
2. Document processing speed
3. Search accuracy
4. User engagement time
5. System reliability

## Timeline
- MVP Development: 2-3 months
- Testing & Refinement: 1 month
- Initial Launch: End of Q1

## Resource Requirements
1. Development Team
2. Cloud Infrastructure
3. AI Computing Resources
4. Testing Environment

This plan provides a structured approach to building TextbookAI, focusing on delivering core functionality in the MVP while maintaining flexibility for future enhancements. The architecture is designed to be scalable and maintainable, with clear progression paths for feature development.

The timeline/resource requirements will certainly be affected by scale, but for now, we're not far from the complete vision, even abstracted past textbooks :)


Update type hints for session

Updating type hints to what's currently reflected on php.net as per https://youtrack.jetbrains.com/issue/WI-60588

Wow... Inspection found lots of issues... Or is this because it's using the older version of the code, that's being modified?

Thank you for taking your time & submitting the proposal. I've commented on some of the found inspection reports: we would appreciate it if you could address the issues I've highlighted & submit new changes

Now only annotator fails, but, I do not think I can do anything about it, since this what we have in PHP itself.

Thanks!
Unfortunately the changes didn't pass PhpStorm Stubs Tests. Please allow me to explain why and what can be done here.

First 16 failed tests failed on checking the declared stubs type VS their actual type run by the runtime. Example: Method SessionHandler::open has invalid return type. With proposed changes it returns bool type while in fact it doesn't return anything: https://3v4l.org/v3ZBA. You can also see this hinted in docs on php.net: Note this value is returned internally to PHP for processing..
So, to address this problem we have to remove return type hints from SessionHandler, SessionHandlerInterface & SessionUpdateTimestampHandlerInterface methods

Failures 17-48 are caused by Method xxx with @since 'x.x'  has parameter 'x' with typehint 'x' but typehints available only since php 7. Let's inspect SessionHandler::destroy for example: it has @since 5.4 tag meaning that you theoretically can run the code that calls this method using PHP5.4. With public function destroy(string $id) signature this would cause PHP fatal error because typehints available only since php 7.
I understand that php.net documentation & sources at php-src are all indicating a string typehint for this method, bit with language level < 7 that's not possible. Calling this method in 5.4 is possible: https://3v4l.org/SLJi6.
That means that all method typehint changes in SessionHandler and SessionHandlerInterface should be reverted.


Add two device images

Add Heltec Wireless Paper V1.0 and Lilygo T3S3 device images.

Thanks for contributing! 🥳


Marketing homepage redesign

Redesign of the Epic Stack Marketing homepage, which corresponds to the page we'll build in one of the segments of the "Unleash The Designer In You" Epic Web workshop.
https://github.com/epicweb-dev/epic-stack/assets/485747/2e4ca359-8ec3-45be-ae62-e5e9df401289
Animations have been revisited and implemented with CSS only — no more GSAP or Framer Motion 👍
The design supports dark mode.

See how you feel about this new commit.
The keyframes are defined as a Tailwind preset inside the _marketing route folder (I was not able to export it directly from the _marketing+/index.tsx route).
Since there is already a logos folder with loads of SVG in there, I assumed that was a safe place to put something that might be deleted by the user.
If deleted, the Tailwind config (which imports that preset) will flag that the preset doesn't exist and should be removed.

I'm not sure about the error you had, but I think you're someone was good 👍

Just occurred to me to look at the dark mode version and I think we could improve the logos a fair bit for that. Maybe just invert?

Whoops yes! I did something wrong — this is how it was supposed to look. I put the grayscale on the parent and it grayscaled the tiles as well:
https://github.com/epicweb-dev/epic-stack/assets/485747/63264cdb-426a-4c96-8958-a255d4693823
The idea of grayscale is the page is slightly less colorful, but the logos still show their colors on hover 👍
But I am not convinced it works without changing the logo colors — so maybe I'll just go with lighter tiles on dark.
Will open a new PR 👍


feat : add support to send sns topic name

what
It is not possible to assign a custom name to the SNS.
why
When creating an SNS, giving a descriptive name to the task it is intended for is important to understand the purpose at a glance.
Important
To merge this feature, it is necessary to first review and merge this, and change source of module sns_topic
Support SNS name on module sns

/terratest


Issue 232/link data to city

#232

@BrunoSalerno I'm not sure what I should translate Data to for Spanish/French so let me know what I need to update it to


Use smaller base image for website-builder

@uidoyen

PR https://github.com/debezium/docker-images/pull/200 merged.

PR https://github.com/debezium/docker-images/pull/200 merged.


How can the port on which the application runs can be changed?

Please put back default express/node.js port.
If it will be required to change the port on which the application runs how this can be done?

You can change it in the config/environment/production.js file here: https://github.com/distributev/the-app/blob/master/server/config/environment/production.js#L45

I see it can be changed through command line when app is started.
Can you show me an example with the command to start the app with a different port?


Add tests for the files

So for now we would be testing only the nighthawk api right @kushthedude

Yes, a simple tdd should to the trick for api.
On Fri, 6 Nov, 2020, 12:25 Sladyn<EMAIL_ADDRESS>wrote:

So for now we would be testing only the nighthawk api right @kushthedude
https://github.com/kushthedude
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/layer5io/nighthawk-go/issues/11#issuecomment-722911996,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKQMTLVPPPTUV7QBHH2G653SOOMW3ANCNFSM4QJSSFHQ
.


CRD docs: polish epic

The initial work to get generated docs up and running was done: https://github.com/stackabletech/documentation/issues/328
Now we need to fill the docs with content.
And then we need to update the CRD docs for all operators, similar to the druid PR:
### Products
- [ ] https://github.com/stackabletech/airflow-operator/pull/354/
- [ ] https://github.com/stackabletech/druid-operator/pull/494
- [ ] https://github.com/stackabletech/hbase-operator/pull/425
- [ ] https://github.com/stackabletech/hdfs-operator/pull/433
- [ ] https://github.com/stackabletech/hive-operator/pull/394
- [ ] https://github.com/stackabletech/kafka-operator/pull/645
- [ ] https://github.com/stackabletech/nifi-operator/pull/537
- [ ] https://github.com/stackabletech/spark-k8s-operator/pull/319
- [ ] https://github.com/stackabletech/superset-operator/pull/431
- [ ] https://github.com/stackabletech/trino-operator/pull/510
- [ ] https://github.com/stackabletech/zookeeper-operator/pull/749
- [ ] Commons
- [ ] Listener
- [ ] OPA
- [ ] Secret
- [ ] EDC
- [ ] Hello World

There is a style guide here: https://docs.stackable.tech/home/nightly/contributor/style-guide#_crd_documentation
Also, all the CRD docs should be linked under "Reference" in our regular docs.
### Products
- [ ] Airflow
- [ ] Druid
- [ ] HBase
- [ ] Hadoop HDFS
- [ ] Hive
- [ ] Kafka
- [ ] NiFi
- [ ] Spark
- [ ] Superset
- [ ] Trino
- [ ] ZooKeeper
- [ ] Commons
- [ ] Listener
- [ ] OPA
- [ ] Secret
- [ ] EDC
- [ ] Hello World

also this: https://github.com/stackabletech/crddocs/issues/7

stackablectl now uses -c kind instead of -k to create a kind cluster inline, and the cluster is a single node one. The getting started guides still refer to -k in some places.


[node, client] Consolidate linked transfer into HTLC

All of the linked hash apps can be consolidated into the HTLC app. All of the other attributes can simply be hashed into the preImage. This will reduce surface area.

Closing this because we found that it's not actually possible


Ctrl-b/f dont seem to do anything in terms of scrolling documentation

Make sure you have done the following

[X] I have updated to the latest version of blink.cmp
[X] I have read the README

Bug Description
Pressing  or  doesn't seem to scroll documentation at all
https://github.com/user-attachments/assets/6801495f-5cf5-4607-986b-d336563d4fd4
Relevant configuration
require("blink.cmp").setup({
    keymap = "default",
    -- experimental auto-brackets support
    accept = { auto_brackets = { enabled = true } },
    -- experimental signature help support
    trigger = { signature_help = { enabled = true } },

    windows = {
      documentation = {
        auto_show = true,
      },
    },
  })

neovim version
NVIM v0.10.2
blink.cmp version: branch, tag, or commit
v0.5.0

It doesn't scroll because the entire documentation is already being shown in your examples


JSON Marshal 库

windows10  go 1.18.6, occured UnsupporetedCPU ,maybe it‘s too old to run Sonic  in Cloudwego/Hertz

请先看看文档：https://www.cloudwego.io/zh/docs/hertz/reference/json/


Fix exception when updating order line

After the Model Extending change, we have missing that the order line observer trigger an exception with the morph name.
class_implements(): Class product_variant does not exist and could not be loaded

Thanks @lguichard although this is a duplicate of #1964


NumberPrompt takes only int and float. By design?

I tried using long, double, and decimal for the generic type in NumberPrompt. The code compiled and ran just fine, but no numbers of any sort were recognized for any of those three types, only when using int and float. I am writing up NumberPrompt in the wiki as accepting int and float, under the assumption that it's not supposed to deal with other numeric types.

I should have mentioned I tried uint as well, with the same result.

Looks to me like we should just cover all the numbers here.

added long, double, decimal with unit tests


style: Fix overflow of text.

This PR fixes the overflow of long names in tooltips and empty feed title.
Fixes: #27282

Screenshots and screen captures:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Thanks for your work here, @Vector73.

For the tooltip, the 280px max-width probably deserves a design discussion on CZO. We'd want to get consensus on what's a likely good upper value.
For the word-wrap portion, I'm not sure that that's the way to go. This too probably deserves a design discussion, but we should also probably figure out the cases under which names are presented on a single line with an ellipsis (...) and when they are allowed to wrap.

What do we do now for the DMs screen? A before/after screenshot comparison would be a good next step here.

I have updated the screenshots in PR description.

Thanks, that's super helpful!
Is 3 lines the max that a name might go to in the tooltip given our name length limits? If so, this seems like a fine solution to me.

The DMs screen also looks basically fine to me in the full screenshot, and is a clear improvement over the current state.

@karlstolley What are your thoughts? Since this is an edge case, I don't think we necessarily need a design discussion to get to a perfect solution, as long as it looks reasonable to us.

Sure. I've raised my concerns, but it is an edge case and I'm fine with merging this.

Yeah, this is progress, might as well merge and we can refine later if it feels warranted.


smartbond UART power management

This adds power management functionality to SmartBond(tm) serial driver.
With this PR:

UART allows platform to go to sleep handling PM_DEVICE_ACTION_RESUME/PM_DEVICE_ACTION_SUSPEND messages.
Platform can be woken up on UART RX activity
Platform can be keep awaken when DTR line is asserted (optional feature of selected GPIO)
DT now has definition of UART2 and UART3

This change requires #61857 and #61844 and #61941
Signed-off-by: Jerzy Kasenberg<EMAIL_ADDRESS>

@andrzej-kaczmarek @dcpleung please take a look

Just reminder that this requires #61857

Still waiting for #61857 to be processed

Still waiting for #61857 to be accepted

needs rebase

rebased to fix conflicts and @ydamigos comment addressed


Implement standard API endpoint: GetStateValidatorBalances

PR Description
Implement standard API endpoint: GetStateValidatorBalances. Builds on #2985
Documentation

[x] I thought about documentation and added the documentation label to this PR if updates are required.

Closing this in favor of the super-set PR: https://github.com/PegaSysEng/teku/pull/3003


SVG rendering?

I'm developing a game using SVG ( http://play.esviji.com/ ) and I feel a physics engine could help me on some topics.
All physics engines I've seen use only Canvas rendering, none of them uses SVG.
Is there a way Matter could provide both?

Definitely, d3 is not the option if you want interactive physics in your app. DON'T USE FOR IT. It's a wonderful library but not for that.


VRMC_materials_mtoon-1.0

https://github.com/vrm-c/vrm-specification/tree/master/specification/VRMC_materials_mtoon-1.0

I did see https://github.com/tbillington/bevy_toon_shader, but now sure how different it is from mtoon.
Might be best to just create a separate bevy_mtoon plugin for this.

https://github.com/unavi-xyz/bevy_shader_mtoon


doc: Getting started refers to 2.x-SNAPSHOT as latest

ref. https://www.dropwizard.io/en/latest/getting-started.html#tutorial
I assume at one point this was being automatically updated? But doesn't answer how it got stuck at a snapshot release?
edit: I see the rst uses a release variable. Maybe an ordering issue with cutting the 2.x branch and overriding the docs?

Similar for Other Versions page

@OneCricketeer Thanks for reporting this! ❤
The problem was indeed that latest was still pointing to the release/2.1.x branch as its source. I've changed it to release/4.0.x and rebuilt that documentation version.

I still feel the tutorial should use the latest release? Not refer to a snapshot?

I still feel the tutorial should use the latest release? Not refer to a snapshot?

I agree, this has been a tradeoff we chose to reduce the manual effort for maintaining this particular part of the documentation.
These code snippets are real, runnable projects, for example https://github.com/dropwizard/dropwizard/blob/release/4.0.x/docs/source/examples/getting-started/pom.xml, and as such they show the latest state of the release/4.0.x branch.
This being said, the stable documentation is based on the latest Git tag and there it's showing the latest release version:
https://www.dropwizard.io/en/stable/getting-started.html#tutorial
I'll try making stable the default target instead of latest on ReadTheDocs.


Add Support for Kibana to not have a public IP address but still be deployed

+1 I don't see any reason why having Kibana automatically has a public IP. It should be a preference with the default true so it can be disabled.


Implement --no-install and --redirects for "paket update".

This pull request adds the switch --no-install to paket update, and handles paket update --redirects also for the "single package update" case.
Additionally, the newly introduced InstallerOptions and SmartInstallOptions have been refactored, such that they match their usage better.

LGTM. Thanks


fix: Force patch release 'Return from function on reject' (#255)

This PR adds an empty commit to force a patch release with changes in #255. Related issue - #264.

That didn't work or why did you close it?

I have no idea why this got closed. Must have pressed the close by mistake.

Probably because of no changes... GitHub trying to be smart ¯_(ツ)_/¯

I'll commit directly to master.

See https://github.com/okonet/lint-staged/pull/267


relative links does not use current file's directory

What version of starlight are you using?
0.9.1
What version of astro are you using?
3.0.6
What package manager are you using?
pnpm
What operating system are you using?
Linux
What browser are you using?
Firefox
Describe the Bug
given following structure:
.
└── src
    └── content
        └── docs
            └── guides
                ├── a.md
                └── b.md

https://github.com/withastro/starlight/assets/54838975/ddd6dfd4-1c6f-40c1-98b1-baff0f224ed9
[b](b)

[./b](./b)

both links are resolved to /guides/a/b, whereas expected path is /guide/b. is this intended behavior?
Other cases
https://github.com/withastro/starlight/assets/54838975/11129d82-684d-4c5f-95b9-2fb59e0505ee
vscode resolves relative markdown links using current file's directory.
docusaurus resolves relative markdown links using current file's directory.
html resolves relative links using current file's directory.
Link to Minimal Reproducible Example
https://stackblitz.com/edit/github-asbwyq-xpyz1g?file=src%2Fcontent%2Fdocs%2Fguides%2Fa.md
Participation

[ ] I am willing to submit a pull request for this issue.

After some investigation I've found out that astro/starlight didn't modify markdown links. It's just that links like /foo/bar is actually /foo/bar/index.html, therefore links like ./baz resolved to /foo/bar/baz. I wrote a simple remark plugin that resolves the link:
import { isAbsolute, join } from "node:path"

import type { Root } from "mdast"
import { visit } from "unist-util-visit"

// starts with ./ or ../
const relativePath = new RegExp(`^\\.\\.?/`)

export const fixRelativeLinks = () => (tree: Root) => {
  visit(tree, "link", (node) => {
    if (isAbsolute(node.url)) {
      return
    }
    if (!relativePath.test(node.url)) {
      return
    }
    // const prev = node.url
    node.url = join("..", node.url)
    // console.log(`${prev} -> ${node.url}`)
  })
}

Thanks for the issue. I think this is expected behaviour for now although depending on your usage it may not always be what you want. I wonder if you’d be interested in wrapping up your remark plugin into a published package? If so, we could add it to the plugins section of our showcase page for others who want the same behaviour.

Closing for now as something that can be fixed with a user plugin. Would still love to see this published or a blog post explaining how to set this up if someone is interested.


fix problem with encoding

fix problem with encoding

I can't merge this pull request because it would break things for people using the pagecache that are using a charset other than utf-8.
You can, however, get that header to be output by default by changing 2 settings in php.ini:
http://php.net/manual/en/ini.core.php#ini.default-mimetype
http://php.net/manual/en/ini.core.php#ini.default-charset

in opencart there is not any encodings other than uft-8

There are for other themes.
Here's an issue someone opened, as a concrete example: https://github.com/budgetneon/pagecache/issues/10
That user needed windows-1254.   Hardcoding UTF-8 will create issues for existing, documented users of the pagecache.
And, setting the correct default charset for php is something a site owner should do anyway.


More examples

Hi,
I've been following your work for about a month and I faved all your posts on /r/unixporn.
Your bars are absolutely wonderful.
But all of all the screenshots you've posted on reddit, non of them have the actual lemonbuddy config file.
Is it possible to add them to the repo? Or do a lemonbuddy.github.io website, and post them as a showcase?
Thanks in advance!

Try checkout his dotfiles repo. As he warned, the config might be too new or too old. So, be careful when put it into your config file.

Oooh gawd, I missed that repo!
I was always in the .config/lemonbuddy folder. Thanks a lot.


Fixes: getQuery function and fetch functions

fetch function to directly give entry/asset/ct object instead of enclosing in another object.
getQuery function fixed to return query object.

Created the PR mistakenly to master branch causing the checks to not run. Closing this and raising another PR.


Aggregate state not being saved after executing an async method

This is likely not an issue with the Eventuous framework, but I can't figure out why an aggregate's state doesn't get saved after executing an async method on the aggregate. I made sure that async/await is used where needed.
Here is a link to a sample project (it has only the bare minimum to reproduce this issue):
https://github.com/claudiuchis/Eventuous.Sample
There are 2 events:

WidgetCreated - created by calling the post endpoint on the WidgetController
WidgetReacted - created as a reaction to the WidgetCreated event after calling an "external" service.

The 2nd event (WidgetReacted) is not appended (e.g. saved) to the stream, so it looks like the reaction doesn't get a chance to complete, and thus the event doesn't get saved. No clue why, as the service does await on the command issued by the reactor.
Any help is much appreciated.

A real world scenario is sending an email after a user signs up.

As I thought, there is no issue with the framework. The issue was with my code, I was using the non-async version of the OnExisting when adding the async handler. I switched to OnExistingAsync and the problem went away.

Yeah, there are some docs lacking in that regard. I plan to have a better API coverage in the docs when the new version gets out.


Break long invocation chains if they have more than 4 (3?) invocations in them.

If we're breaking, we should do before the first invocation so the indentation is more consistent:
e.g.
Enumerable
    .Range(...)
    .Where(...);

Agreed, it uses this logic currently. <= 4 keeps on the same line, otherwise you get some weird spacing with the one version
        one.Range______________________________________________()
            .Where__________________________________________________();

        this.Range______________________________________________()
            .Where__________________________________________________();

        Enumerable
            .Range______________________________________________()
            .Where__________________________________________________();

Currently it doesn't break a chains until it exceeds the max line length.


Updated chrome - now ui.New fails to start

Chrome version: Version 110.0.5481.177 (Official Build) (64-bit)
These are the errors from ui.New()
2023/03/08 13:25:26 websocket.Dial ws://<IP_ADDRESS>:45031/devtools/browser/396a03af-3f1b-400b-8294-9c0f2f25cd29: bad status
exit status 1

Running chrome on its own - like lorca calls it generates:
[0308/095229.172548:WARNING:bluez_dbus_manager.cc(247)] Floss manager not present, cannot set Floss enable/disable.
[0308/095229.190683:WARNING:sandbox_linux.cc(393)] InitializeSandbox() called with multiple threads in process gpu-process.

From another project - it looks like adding this argument resolves the issue:
--remote-allow-origins=\*

Also seems related to - https://github.com/zserge/lorca/issues/183

But doing it like https://github.com/zserge/lorca/issues/183 only wakes up Chrome, not the GUI as it originally did

https://github.com/zserge/lorca/pull/185 should fix this. Hopefully it is accepted upstream.


外部链接失效

问题描述
外部链接无法直接打开，已确定是docsify的问题 (docsifyjs/docsify#1467)，等待修复。

已使用plugin暂时修复

hi,How did you do it? At present, I am also troubled by this problem

hi,How did you do it? At present, I am also troubled by this problem

By adding a plugin.
Im lazy to type or copy it. You can view the source code of https://slimefun-wiki.guizhanss.cn/


Undefined method `warning' for Logger:0x0000561690c6b180 Did you mean warn?

ONC recently removed a test from Tab 6 of the g.10 test suite. Following this change, the input json file we had historically used contained inputs that Inferno no longer recognized. When trying to run the tests, our POST to https://inferno.healthit.gov/suites/api/test_runs was met with a 500 and the following response: {"errors":"undefined method 'warning' for #\u003cLogger:0x0000561690c6b180\u003e\nDid you mean?  warn"}.
I believe this is coming from this line of code.
We updated our json to remove the no longer used inputs and are now seeing tests pass normally so this is non-blocking/non-urgent on our end, just wanted to report this little bug since I imagine others may run into it following the recent change as well.

Thanks, we'll take care of that. Itshould be warn rather than warning.

We also need to document when inputs change in the release notes... you shouldn't have to discover that through warnings in the log.  FYI @Jack-Fraser16.  Thanks @kayadanae!


Spm support

NOTE: Currently requires Xcode 12beta4 do build and test because of the binary OpenSSL SPM dependency!  Also, the SPM module configuration may make this version incompatible with previous versions because of import changes!
Reimplements the ECC functions that were previously OpenSSL based using libtomcrypt and libtommath from https://github.com/libtom.  All functions that were previously provided have been implemented:

Public key recovery from private secp256k1 and secp256r1 keys.
Public key recovery from signature and message hash for secp256k1 and secp256r1 keys.
Recid matching for keys from signature and message hash.
Signing with secp256k1 keys.

Unused Cocoapods artifacts have been removed.  The podspec file has been updated but not tested yet and is likely not to work.  Enabling Cocopods will be undertaken in follow up PR's.
The default functions are currently still pointing to OpenSSL.  The libtom implementations have been created along side them temporarily for review and side by side testing.  OpenSSL will be removed before the final PR is removed.  Once OpenSSL is removed, the swift-tools dependency will be dropped down to 5.2 to hopefully utilize Xcode 11.

This repository and work have been folded into eosio-swift.


Refactor todos

[x] Readd embeds
[x] Switch to env vars
[x] Rewrite current config system
[x] Readd total visits support
[x] Add multi-device support
[ ] Maybe re-do website frontend?
[x] Change successful beat behavior to how it was before (returning unix timestamp)
[ ] Allow multiple tokens
[x] Total uptime on stats page

I feel this can be closed for now


Tk button style messes up Matplotlib toolbar

When using Matplotlib with the tk backend, the default styles for the tk widgets mess up the toolbar:

It can be avoided without too much trouble by subclassing matplotlib.backends.backend_tkagg.NavigationToolbar2Tk and editing its _Button method, but it's still something to be aware of in case there's a better solution for it.
(Also, this wouldn't need to be an issue if Matplotlib caught up with the times and just used ttk, and used the png images for the tkinter toolbar which have transparent backgrounds...)

@daniilS , I'm actually working on cleaning up the tkinter widgets now. Would you mind posting some sample code that I can use to test while I'm making those changes?

Sure, here's a minimal example that creates an empty plot embedded in a tkinter window (it looks like it doesn't happen when letting Matplotlib create its own window: https://gist.github.com/daniilS/bd8a70b17b67ebc716345bc5c922131c
The culprit itself is here: https://github.com/matplotlib/matplotlib/blob/master/lib/matplotlib/backends/_backend_tk.py#L561
Maybe this is really a Matplotlib issue, but I doubt this would be particularly high priority for them. I'm happy to provide an example for the documentation on how to subclass NavigationToolbar2Tk to avoid this, if you reckon that's enough of a solution.

yeah, I don't think there's an easy ttkbootstrap way to make this look better because of how the buttons are configured inside matplotlib. You can use unicode characters to get a pretty nice looking substitute that would work with ANY theme, and without having to use any images.  I'm sure this will render differently depending on the font set, etc... and there are other character options available to represent the same idea.

from tkinter import ttk
from ttkbootstrap import Style

chars = {
    'home': '\U0001F3E0',
    'left': '\U0001F878',
    'right': '\U0001F87a',
    'magnify': '\U0001f50d',
    'slider': '\U0001f39a',
    'move': '\u2931',
    'disk': '\U0001f5aa'
}

style = Style('yeti')
root = style.master
style.configure('char.primary.TButton', font='Helvetica 20')
frame = ttk.Frame(root, padding=20)
frame.pack(fill='both')

ttk.Button(frame, text=chars['home'], style='char.primary.TButton').pack(side='left', padx=1)
ttk.Button(frame, text=chars['left'], style='char.primary.TButton').pack(side='left', padx=1)
ttk.Button(frame, text=chars['right'], style='char.primary.TButton').pack(side='left', padx=1)
ttk.Separator(frame, orient='vertical').pack(side='left', fill='y', padx=5)
ttk.Button(frame, text=chars['move'], style='char.primary.TButton').pack(side='left', padx=1)
ttk.Button(frame, text=chars['magnify'], style='char.primary.TButton').pack(side='left', padx=1)
ttk.Button(frame, text=chars['slider'], style='char.primary.TButton').pack(side='left', padx=1)
ttk.Separator(frame, orient='vertical').pack(side='left', fill='y', padx=5)
ttk.Button(frame, text=chars['disk'], style='char.primary.TButton').pack(side='left', padx=1)

root.mainloop()

So, the tk widgets aren't really the focus on this package, but I think I was able to get them to look ok with the primary colors for whatever theme is chosen. The other colors are available via the colors property for manual overrides.

I have no idea how to remove the relief from the spinbox. None of the option database settings I tried worked. Also, I was not able to add any styles to the Scrollbar. However, since these are two widgets that are available in the ttk widget set, I don't really see this as an issue.

Looks like matplotlib did recently switch to using the png images in tkinter 8.6, so that solves the transparency issue. Annoyingly there's no good way to style the tk.checkbutton when it's using the indicatoron=False mode, which explains why the pan and zoom buttons don't get styled. I'll create a subclass of NavigationToolbar2Tk for my code which uses ttk widgets, and am happy to share it if you want to include it in the documentation in case anyone else runs into the same issue.

Yeah for sure. If you send me a link to your code I can consider putting it into the gallery. I'm sure it would be helpful for others who use matplotlib and want make make it looks like it was designed in the last 10 years. ;-)

Sure, I'll upload it and link you the project repository once it's reached a somewhat presentable state (what example could be more exciting than a data analysis tool!). It's a shame that tkinter has such a bad rep because people always associate it with the default tk look :/

@israel-dryer I finally got around to writing a proper patch that makes the matplotlib toolbar go from looking like this:

to this:

when embedding a plot in a tkinter window with ttkbootstrap. The file for the patch is here, and all you need to do is call its applyPatch method in a project at any point before displaying a matplotlib graph.
It might be worth adding a file like this to to the widgets module and including a note in the documentation in case anyone else ever runs into the same problem. The patch is only necessary for embedded plots as ttkbootstrap doesn't affect matplotlib's own popups, but it also has the somewhat nice side effect of causing the popups to use the native ttk theme instead of the old tk look.


fix data race of Consumer.closeHandler

Hi, I met a data race when I use consumer.NotifyClose(), this pull request tries to fix it.
WARNING: DATA RACE
Read at 0x00c0006ae160 by goroutine 1489:
  github.com/rabbitmq/rabbitmq-stream-go-client/pkg/stream.(*Coordinator).RemoveConsumerById()
      /home/gpadmin/go/pkg/mod/github.com/rabbitmq/rabbitmq-stream-go-client@v1.1.0/pkg/stream/coordinator.go:85 +0x15a
  github.com/rabbitmq/rabbitmq-stream-go-client/pkg/stream.(*Consumer).Close()
      /home/gpadmin/go/pkg/mod/github.com/rabbitmq/rabbitmq-stream-go-client@v1.1.0/pkg/stream/consumer.go:218 +0x65a

Previous write at 0x00c0006ae160 by goroutine 1463:
  github.com/rabbitmq/rabbitmq-stream-go-client/pkg/stream.(*Consumer).NotifyClose()
      /home/gpadmin/go/pkg/mod/github.com/rabbitmq/rabbitmq-stream-go-client@v1.1.0/pkg/stream/consumer.go:95 +0xf6

Thank you, @kensou97 . Can you please rebase the PR?

Thank you, @kensou97 . Can you please rebase the PR?

done

FYI:

https://github.com/rabbitmq/rabbitmq-stream-go-client/releases/tag/v1.1.2

We are aware of some data race problems in this client. We are working on a new client version (v2) where the code is easier and more flexible. We don't have an ETA but aim to release something after the summer.


StyleContainer for dynamic React Native styles

Environment: React Native
Dynamic styles are right now resolved directly from the style prop.
This causes several misadvantages

React Native throws warnings everytime a "wrong" value is applied (e.g. stateful values)
Every property gets iterated on every render
Stateful selectors as introduced with beta5, cannot be combined

Proposal
There's nothing like a unique className which we can use to reference the styles. We may generate those and applied during the initial render. This would be the best solutions as far as I know, because you would not need to clone the element if only pure styles are passed in.
e.g.
const styles = StyleSheet.create({
   box: {
      color: 'red',
      fontSize: 12,
      backgroundColor: props => bg.color,
      'fat=true': {
         fontWeight: 'bold'
      }
   }
})

would return
{
   box: {
      color: 'red',
      fontSize: 12,
      _dynamic: 'd0'
   }
}

while the StyleContainer now has a reference for d0 which returns:
{
   backgroundColor: props => bg.color,
   'fat=true': {
      fontWeight: 'bold'
   }
}

Basic testing on native worked out quite well. Black-box tests will do the rest.


Feature: Computed fields

add ability to add virtual fields to Models which are not stored in database.
// ent/schema/user.go

type User struct {
	ent.Schema
}

// Fields of the User.
func (User) Fields() []ent.Field {
	return []ent.Field{
		field.String("first_name").,
		field.String("last_name"),
		field.String("full_name").Computed()
	}
}

// ent/user_resolvers.go

func UserFullNameResolver(u User) string {
  return u.firstName + " " + u.lastName
}

Hey @sijad,
There's an option to do it today using templates, so I'm not so convinced it should be on the schema definition. See: https://entgo.io/docs/schema-fields/#additional-struct-fields

yes, I'm not sure why I didn't find it sooner...
thanks!


Ability to change Timidity/Fluidsynth polyphony channel count in the config file

I couldn't find any setting like that, while chorus/reverb is fairly freely customizable in fluidsynth case.

Regrettably timidity++ seem not to have this option, but polyphony is a hardcoded number.
I'll have a deeper look when I have time, but a priori i can't escape having to patch it.


Deprovision of APB fails with multiple attempted APB launches, secret mount failure

Bug: Deprovision of APB fails with multiple APB launch, Secret Mount Failure

Possibly related to issue https://github.com/openshift/ansible-service-broker/issues/341
What happened:

Reset catasb cluster

Provision APB into "myproject" (logs)

Click "Delete" button on provisioned service once

Two deprovision APBs attempt to start simultaneously

Both deprovision APBs fail to start due to volume mount failure (logs)

ASB logs show failure to delete service instance (logs)

ASB logs excerpt:

[2017-08-07T20:42:33.917Z] [ERROR] ERROR - failed to delete service instance - client.Error{Code:100, Message:"Key not found", Cause:"/service_instance/e7df3940-29dd-42d9-a9e4-e95f19e04042", Index:0x14}
--
  | [2017-08-07T20:42:33.917Z] [ERROR] broker::Deprovision error occurred.
  | [2017-08-07T20:42:33.917Z] [ERROR] 100: Key not found (/service_instance/e7df3940-29dd-42d9-a9e4-e95f19e04042) [20]

What you expected to happen:
Deprovision APB pod runs as expected.

Would this be service-catalog bug?

@rthallisey I am going to try and replicate. I think that this might be fixed but want to test.
If our APB is failing to start because of a secret mount not available, then that I think would be our problem and not service catalog.


Finish setup of example project

Finish setup of example project

Done -- documentation is also ready. See repository.


Add GUI Testing

Ripme version: ¯\(ツ)/¯
Java version:  ¯\(ツ)/¯
Operating system:   ¯\(ツ)/¯
Exact URL you were trying to rip when the problem occurred:  ¯\(ツ)/¯
Please include any additional information about how to reproduce the problem: ¯\(ツ)/¯

Seeing as Translations are being added, it would be a good idea to add in some GUI unit testing... sometime.
AssertJ seems like a good library to use.

While we're at it we should add some unit tests for the translations (Making sure the locale the user specifies in the config files is used, making sure all strings are translated ect)

How about adding new ComboBox to choose language that user uses?

@zinuzian

How about adding new ComboBox to choose language that user uses?

It's done in PR #625 and should be in ripme 1.7.49


Suggestion for replacing "kosher salt" closes #7

My local shop doesn't sell kosher salt. Here I propose how that can be replaced.

Using regular salt instead is a good idea. Reopening the PR and accepting it.


Unable to change PullUpController's height dynamically

I have a UILabel with numberOfLines set to 0. Its height constraint is >=30. I want to change pullUpController's height depends on its subview's height. I don't have any method to call after the layout is updated.

Hello there,
did you try using the method updatePreferredFrameIfNeeded(animated: Bool)?
Note that you should update pullUpControllerPreferredSize (or pullUpControllerPreferredLandscapeFrame) before call that method.

Yes. I have this code in ChildVC:PullUpController
override var pullUpControllerPreferredSize: CGSize {
    if #available(iOS 11.0, *) {
        return CGSize(width: UIScreen.main.bounds.width, height: contentView.bounds.height + self.view.safeAreaInsets.bottom)
    } else {
        return CGSize(width: UIScreen.main.bounds.width, height: contentView.bounds.height + self.bottomLayoutGuide.length)
    }
}
override func viewDidLayoutSubviews() {
    updatePreferredFrameIfNeeded(animated: true)
}

But ChildVC's y posistion is maxY in parent view controller.
Parent view controller's frame -> (0,0,414,736)
Child view controller's frame -> (0,736,414,0)
Child view controller's subview frame -> (0,-268,414,268)//268 is contentView.bounds.height

Hi @MarioIannotta
Is there a way to do it?
I am trying to do something similar where I want to expand the pullUpControllerPreferredSize (height) manually but it doesn't work.

public override var pullUpControllerPreferredSize: CGSize {
return CGSize(width: UIScreen.main.bounds.width, height: maxHeight)
}

Then  I am changing the maxHeight value and calling:

viewController.updatePreferredFrameIfNeeded(animated: true)

The viewController doesn't update its height. Is that possible to do?


fix: removed @semantic-release/git from deps

:tada: This PR is included in version 2.0.16 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Create "How to get started contributing article"

Per discussion at contributor WG meeting.
Article would be targeted at potential new contributors, and oriented around "how do I find something to contribute to".    Based partly on Destiny's talk and Callum's article on KBE.

cc @Cali0707 since I can't assign him.

Commenting so I can help here

@jberkus @Deafveloper how should we start working here? Does starting with a google doc make sense? Something else?

Thanks for reaching out! I’m not entirely sure what the next step is, so I’ll wait for guidance on how we should proceed. Should I tag @jberkus here, or would it be better to DM for clarification?


error: "uninitialized /etc detected" when compiling. causes stages to be skipped

This issue was originally opened in a different repo that uses osbuild:
Link to original issue in Sample images.
@pingou commented on that issue:

This is an error message produced by osbuild, I've asked the osbuild folks and the feedback was:
this error message is misleading, but 1) should fix that and 2) maybe investigate the underlying issue, but should be harmless. ticket
would be great

Would you mind opening a ticket at https://github.com/osbuild/osbuild for them to track this work?

below, copasta from original issue:
Description
getting error when building image:

/usr/lib/tmpfiles.d/journal-nocow.conf:26: Failed to resolve specifier: uninitialized /etc detected, skipping
All rules containing unresolvable specifiers will be skipped.

as described in error message, it doesn't stop the build, just skips that stage.
getting same error in various building types: different extensions (tar/img/qcow2), different setups(vm/container/sudo permissions).
Fix ?
maybe "/etc" initialization should be done earlier in the build flow?
notice that after the build is done, able to boot image/container, and to access the "/etc" folder, so seem it is initialized later in the build flow.
to reproduce
run any build that has stages such as: "org.osbuild.kernel-cmdline", examples and full logs below:
make cs9-qemu-minimal-regular.x86_64.tar

minimal-regular.x86_64.tar log)
make cs9-qemu-minimal-regular.aarch64.qcow2

minimal-regular.aarch64.qcow2 log

I have never investigated this (beyond making sure it is indeed harmless), and I wonder if it could be tmp file snippets without corresponding sysuser snippets (or whatever they are called). I.e., tmp files might rely on packages configuring users, but in our case that's not happened, so if systems was responsible for configuring the users it would all be fine. A bit of a guess, but if that's the case, it would be great if someone were to file bugs in the relevant packages.

@dvdhrm - can you have a look at this example:
org.osbuild.first-boot :
a file is created to track if there is a need to run first-boot commands:
ConditionPathExists=/etc/osbuild-first-boot

so, if this always fail:
open(f"{tree}/etc/osbuild-first-boot", 'a').close()

(as ,  "uninitialized /etc..." in this stage),
meaning there is no option to custom executing commands on first boot.
this was also seen in logs (and the actual image), that the first-boot commands i added weren't executed,
hence this stage, eventually, will always be skipped...

Thanks for the patience!

@dvdhrm - can you have a look at this example: org.osbuild.first-boot : a file is created to track if there is a need to run first-boot commands:
ConditionPathExists=/etc/osbuild-first-boot

so, if this always fail:
open(f"{tree}/etc/osbuild-first-boot", 'a').close()

(as , "uninitialized /etc..." in this stage), meaning there is no option to custom executing commands on first boot.

This stage does not print uninitialized /etc, nor is it related to this warning. Can you elaborate why you think it is? What am I missing?

this was also seen in logs (and the actual image), that the first-boot commands i added weren't executed, hence this stage, eventually, will always be skipped...

What logs? The first-boot command will only be executed when you boot the image. Osbuild will never boot images, nor execute any first-boot units.
Lets try to clear some things up first:

hence this stage, eventually, will always be skipped: I don't think the stage is skipped, but rather the first-boot unit you created, right? The stage is executed as part of osbuild, and there is no way to skip stages. However, the stage might be malfunctioning. I think this is what you are referring to?
The warning you mentioned is printed when systemd-tmpfiles is invoked. This can happen both when we invoke it manually, and when post-install scripts invoke it (when RPMs install tmpfiles, they trigger a re-run of tmpfiles). Hence, the warning can occur multiple times in an osbuild run.
The warning is triggered when a tmpfiles-specifier requires /etc. The only specifier in current systemd that requires this is the machine-id (I might be missing some, but I checked the code I found nothing else). This means, if a tmpfile configuration uses %m in their configuration, systemd-tmpfiles needs to read the machine-id from /etc to know what path to create/modify/update. This obviously cannot happen if there is no machine-id in /etc. Hence, systemd-tmpfiles skips such configurations if /etc is empty.

Note that we really want to avoid initializing /etc in osbuild, since we would be required to generate a machine-id, which then would be identical on each deployment of the image. This would prevent us from creating re-usable, or sharable images.
I tried tracing all the occurrences of this warning, and they all seem harmless. Let me know if I missed something! However, it is quite unlikely that RPMs use the machine-id in their tmpfile-paths, with few exceptions for historical reasons. If you are sure you ran into this issue, I would gladly hear the use-case and we can try figuring out how to solve this!
Having traced this all, I am quite certain the warning is not related to the issue you describe. However, I am having a hard time trying to understand what your exact issue is. Can you elaborate on what you are doing, what you expect happening, and what you are seeing going wrong?

@dvdhrm, thnx for explaining the meaning of the warning: "uninitialized /etc...", and why it's not related to the "first-commands" issue.
could there be some documentation/changing warning message to show this could be normal behavior for this flow?
maybe changing this message could save some confusion for others in the future?
we've notice that "first-command" is using systemd for execution, which explains why "first-commands" didn't work when building a tar image, and running as a docker container that doesn't have it's own systemd to run these commands on first boot.
in conclusion, closing this issue as this isn't a bug.


helppppp!

my problem is when i want use the powershell to send farsi phrase to piper, the powershell does not support the persian unicode and i want to know that you proceed this project, how you overcome this because i cannot find any solution i the internet.

@Sarvinkhan ,
The issue you're facing is related to how PowerShell displays and handles text encodings. Here's how to overcome it and run the command with your Persian phrase:

Setting Default Encoding (PowerShell v6+):

If you're using PowerShell Core v6 or later, it defaults to UTF-8 encoding. You can verify this by running:
[System.Text.Encoding]::Default
If it's not UTF-8, add the following line to your PowerShell profile ($PROFILE) to set it permanently:
$PSDefaultParameterValues['*:Encoding'] = 'utf8'

Explicit Encoding for the Phrase (All Versions):

Even if PowerShell is set to UTF-8, the way you entered the phrase might not be interpreted correctly.
Use the [Text.Encoding]:: methods to explicitly encode the Persian text as UTF-8 before piping it to the command:
$phrase = "سلام و درود بر شما"
$encodedPhrase = [Text.Encoding]::UTF8.GetBytes($phrase) | ConvertTo-Hex

echo ($encodedPhrase -join "") | .piper.exe --model .modelsfa_IR-amir-medium.onnx --output_dir .outputs

This encodes the phrase as UTF-8, converts the bytes to a hexadecimal string for clarity, and then pipes it to the echo command.
3. Font Selection:
Ensure your PowerShell console uses a font that supports displaying Persian characters. Common options include Segoe UI or Lucida Sans Unicode. You can change the font in the console properties.

Verifying Output:

After running the command, check the output directory (.outputs) to see if the processed file contains the Persian characters correctly.

@Sarvinkhan , The issue you're facing is related to how PowerShell displays and handles text encodings. Here's how to overcome it and run the command with your Persian phrase:

Setting Default Encoding (PowerShell v6+):

If you're using PowerShell Core v6 or later, it defaults to UTF-8 encoding. You can verify this by running:
[System.Text.Encoding]::Default
If it's not UTF-8, add the following line to your PowerShell profile ($PROFILE) to set it permanently:
$PSDefaultParameterValues['*:Encoding'] = 'utf8'

Explicit Encoding for the Phrase (All Versions):

Even if PowerShell is set to UTF-8, the way you entered the phrase might not be interpreted correctly. Use the [Text.Encoding]:: methods to explicitly encode the Persian text as UTF-8 before piping it to the command:
$phrase = "سلام و درود بر شما"
$bytes = [Text.Encoding]::UTF8.GetBytes($phrase)
$encodedPhrase = [System.BitConverter]::ToString($bytes, [String]::Empty)

echo ($encodedPhrase -join "") | .piper.exe --model .modelsfa_IR-amir-medium.onnx --output_dir .outputs

This encodes the phrase as UTF-8, converts the bytes to a hexadecimal string for clarity, and then pipes it to the echo command. 3. Font Selection:
Ensure your PowerShell console uses a font that supports displaying Persian characters. Common options include Segoe UI or Lucida Sans Unicode. You can change the font in the console properties.

Verifying Output:

After running the command, check the output directory (.outputs) to see if the processed file contains the Persian characters correctly.

if your discord channel is still available, eould please give me an invite link?


Is junction tree necessary?

I think that molecular graph can be directly generated from latent vector of graph as junction tree is made from latent vector of junction tree. Have you ever tried it and it is worse than JTVAE?

Hi,
Yes. I mentioned this possibility in my paper -- generating a graph node by node from its latent representation. In fact, this paper implemented this (https://arxiv.org/abs/1803.03324), though their model does not have a encoder (so pretty much like a language model). Their performed experiments on ZINC dataset, and showed show that 89.2% molecules are valid when sampled, while we get 100%. Unfortunately, they didn't report results on Bayesian optimization.
In molecule domain, previous work already tried to generate SMILES strings of molecules. If you look at how SMILES strings encode molecules, you will realize that generating SMILES is almost the same as generating a graph node by node.


Usage with vanilla puppeteer?

Can the this repository be used as Puppeteer plugins, similar to how we use puppeteer-extra-plugin-stealth?
// puppeteer-extra is a drop-in replacement for puppeteer,
// it augments the installed puppeteer with plugin functionality
const puppeteer = require('puppeteer-extra')

// add stealth plugin and use defaults (all evasion techniques)
const StealthPlugin = require('puppeteer-extra-plugin-stealth')
puppeteer.use(StealthPlugin())

It would be good if we could use unblocked with puppeteer's API

I never responded to this. Unfortunately, it would be possible to do, but would require a lot of work. This entire project works around puppeteer's limitations in scraping, which includes going to the protocol level of the platform. To really get there, we'd have to re-implement all of puppeteers apis on top of the current stack. That's not a goal of the project at the moment, so I'm going to mark this as won't fix. If you are wanting to take this on, I think the approach would be to create it with plugins and identify areas that need to be added to plugins to make it possible.


Error when running example

When running the example I get the following error:
Error: Compilation failed
Compiling ...-- NAMING ERROR --------------------------------------------------- src/Main.elm
I cannot find a Firestore.Sub.ChangeType type:
492| handleChange : Model -> Firestore.Sub.ChangeType -> Firestore.Document.Document -> Cmd Msg
^^^^^^^^^^^^^^^^^^^^^^^^
Could you help me to get this running?
Thank you.
S.

I updated the example to the latest API. Sorry about the delay :\
One will still have to plug their own firebase project config in to get things working.


edit must be readwrite

[Enter feedback here]
spotted errors -

Old
New

ChannelSettings.Edit.Group
ChannelSettings.ReadWrite.Group

TeamsTab.Edit.Group
TeamsTab.ReadWrite.Group

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

ID: 11c68c28-b6d6-ef7e-f8f9-ebebe487378b
Version Independent ID: c5985867-3795-4280-ef0d-1d66ee30da6e
Content: Resource-specific consent in Teams - Teams
Content Source: msteams-platform/graph-api/rsc/resource-specific-consent.md
Product: msteams
GitHub Login: @laujan
Microsoft Alias: lajanuar

This is already fixed.


Route53 issues

Versions
Please specify real version numbers or git SHAs, not just "Latest" since that changes fairly regularly.
| Sarama | Kafka | Go |
|v1.27.1|aiven 2.7|1.14|
|  |  |  |
Configuration
What configuration values are you using for Sarama and Kafka?
tlsConfig := tls.Config{}
caCertPool := x509.NewCertPool()
	caCertPool.AppendCertsFromPEM(caCertUSW2)
	caCertPool.AppendCertsFromPEM(caCertUSE1)
	tlsConfig.RootCAs = caCertPool
config := sarama.NewConfig()
	config.Producer.Return.Successes = true
	config.Net.TLS.Enable = true
	config.Net.TLS.Config = tlsConfig
	config.Version = sarama.V0_10_2_0
	config.Net.SASL.Handshake = true
	config.Net.SASL.Enable = true
	config.Net.SASL.User = "XXXXXX"
	config.Net.SASL.Password = "XXXXXX"

Logs
When filing an issue please provide logs from Sarama and Kafka if at all
possible. You can set sarama.Logger to a log.Logger to capture Sarama debug
output.

I wasn't aware of this logger and will include the terminal output.

logs: CLICK ME

NewSyncProducer  kafka: client has run out of available brokers to talk to (Is your cluster reachable?)

Problem Description
We are trying to utilize route53 with aiven kafka as the solution.  We have worked with their experts who are not sure why the overall solution won't work.  My code is fine for directly hitting the service.  There are two issues with using a route 53 endpoint as the broker URL.
One is that if you utilize t509 certificates then hostname validation will cause a failure.  The workaround is to set something like
tlsConfig.ServerName = "dyanmciaivenurl"

The second issue we noticed is that providing this route53 URL to the SyncProducer results in a Kafka not leader for partition error.
Could we get some help on these two issues?
To reproduce:
Setup a free account with aiven.
Setup route53 for aiven dynamic url.
Write a producer with your route53 url.
Attempt to produce messages.

@jackingle you'd need to include the full logger output for us to diagnose further, and you haven't described your Route53 configuration, but I imagine that the issue you are seeing is that the initial connection to the hostnames you provide to Sarama are simply used to "bootstrap" connectivity to the cluster, and then the client will rely on the hostnames returned in the "advertised listeners" configuration on the server side for determining how to connect to each broker in the cluster. Unless you've a) got a 1:1 mapping of brokers to route53 hostnames and b) overridden the advertised listeners with the route53 hostnames, then you will be unlikely to be able to connect successfully

Apologies for the miss. For Route53 we're strictly routing direct
route53uri.io > aivendyanmicurl.com
One important piece to note if you haven't gotten the chance to check is that with Aiven Kafka you do not have access to brokers.  You provide an Aiven URL in the place you would provide a broker.  As such our 1:1 mapping is route53 hostname to aiven hostname.

@dnwe -
The use case @jackingle is referring to is a small extension to Sarama go library to disable hostname validation. Kafka SASL client library allows for this by setting property "ssl.endpoint.identification.algorithm" to "" essentially asking to disable hostname validation. Why would one need to disable hostname validation? It is for cases where you would like a CNAME mapping like "kafka.mydomain" (in for e.g. route53) to point to the actual Kafka cluster with "actual.kafka.cloud.com". Is this making sense?

@jackingle - the following will do what you are looking for. I've verified it. No changes in Sarama go is required.
https://github.com/digitalbitbox/bitbox-wallet-app/blob/b04bd07852d5b37939da75b3555b5a1e34a976ee/backend/coins/btc/electrum/electrum.go#L76-L111