Weapon Qualities are being concatonated rather than added

Hi guys, if you go drag-and-drop Accurate (or other ranked qualities) onto a weapon multiple times (say, a grenade), instead of going 1+1=2, it goes 1+1=11, which is (probably) just a string concatenation ("1"+"1" = "11"). Note that, when using the item, it is using that information, so a Accurate 3 ("111") grenade rolls One hundred and Eleven boost dice(!)

Dats some accuracy right there. 😂

Putting this here as a reminder to get a pull request for this. The foundItem.data.rank is not an integer. Probably a better fix to change it when it first gets made.
item-sheet-ffg.js:910
foundItem.data.rank = parseInt(itemObject.data.rank, 10) + parseInt(foundItem.data.rank);

Testing this with V10.290, SWFFG 1.80-alpha5 on a fresh world and newly created items and it does not have this issue.

I also can't reproduce it on a migrated world - guess it got fixed at some point


fix(games.json): fix epic namespaces as epic string

Closes #565

I also updated my script to handle such cases better, available at:
https://github.com/felipecrs/AreWeAntiCheatYet/blob/master/tools/fill_epic_namespaces.sh


Is it a way to draw bold text with custom font?

I would use similar method
graphics.draw(Text("Test", at: Vec2(x: 0, y: 15)))

but I would set font and size also.

Font size can already be set using e.g.
graphics.draw(Text("Test", fontSize: 14, at: Vec2(x: 0, y: 15)))

Bold text isn't currently supported, but if Cairo supports this it shouldn't be too hard to add (adding a fontWeight property to Text, then updating the implementation of draw(_ text: Text) in CairoGraphics). Feel free to open a PR!


Added services to Google Travel Time, added update_interval to configuration

Description:
Google Travel time allows you to use optimistic, best_guess, or pessimistic for travel time estimation. You can also set avoid options like Highways or Tolls. Since the API has a limit of the number of free calls, it doesn't make sense for a user to have to setup 3 travel time sensors if they want to toggle between time estimates for a given mode of travel.
I added two services to update the travel model or avoid option with optional entity_id if you want to update a specific sensor, or if you want to update all of them.
Also, with an API limit, it doesn't make sense to have a fixed 5-minute interval for a sensor. If you have a lot, you may want a slower update rate, if you have a few you may want a faster update rate. I added an update_interval parameter to the configuration portion of the sensor.
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#4590
Example entry for configuration.yaml (if applicable):
- platform: google_travel_time
  name: Travel Time
  api_key: !secret gcm_key
  origin: work
  destination: home
  update_interval: 120

Checklist:

[x] The code change is tested and works locally.

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

This PR seems to have gone stale. Closing it. You can reopen it when you're ready to finish it.


[test] adding craype tag for check that depends on Cray PE

Will fix UES-572

@jgphpc Can you answer my  questions here? This is PR is ready  to  go otherwise.

@jenkins-cscs retry none


[cinder-csi-plugin] fsck error, mount error

/kind bug

The binaries affected:

[ ] openstack-cloud-controller-manager
[x] cinder-csi-plugin
[ ] k8s-keystone-auth
[ ] client-keystone-auth
[ ] octavia-ingress-controller
[ ] manila-csi-plugin
[ ] manila-provisioner
[ ] magnum-auto-healer
[ ] barbican-kms-plugin

What happened:
tying to create pvc
cinder-csi-plugin logs
I0129 13:12:58.786740       1 mount_linux.go:287] `fsck` error fsck from util-linux 2.29.2
fsck.ext2: Bad magic number in super-block while trying to open /dev/vdc
/dev/vdc:
The superblock could not be read or does not describe a valid ext2/ext3/ext4
filesystem.  If the device is valid and it really contains an ext2/ext3/ext4
filesystem (and not swap or ufs or something else), then the superblock
is corrupt, and you might try running e2fsck with an alternate superblock:
    e2fsck -b 8193 <device>
 or
    e2fsck -b 32768 <device>

E0129 13:12:58.803489       1 mount_linux.go:140] Mount failed: exit status 32
Mounting command: mount
Mounting arguments: -t ext4 -o defaults /dev/disk/by-id/virtio-5b77f374-4351-4b8f-b /var/lib/kubelet/plugins/kubernetes.io/csi/pv/pvc-efbb1829-06bc-43e5-adbc-c15ffbab10e3/globalmount
Output: mount: wrong fs type, bad option, bad superblock on /dev/vdc,
       missing codepage or helper program, or other error

       In some cases useful info is found in syslog - try
       dmesg | tail or so.

I0129 13:12:58.809304       1 mount_linux.go:322] Disk "/dev/disk/by-id/virtio-5b77f374-4351-4b8f-b" appears to be unformatted, attempting to format as type: "ext4" with options: [-F -m0 /dev/disk/by-id/virtio-5b77f374-4351-4b8f-b]
I0129 13:13:01.388338       1 mount_linux.go:326] Disk successfully formatted (mkfs): ext4 - /dev/disk/by-id/virtio-5b77f374-4351-4b8f-b /var/lib/kubelet/plugins/kubernetes.io/csi/pv/pvc-efbb1829-06bc-43e5-adbc-c15ffbab10e3/globalmount

pvc seems like works good
[core@ate-tc3w4dzt6znh-master-0 ~]$ kubectl get pvc
NAME                   STATUS   VOLUME                                     CAPACITY   ACCESS MODES   STORAGECLASS   AGE
csi-pvc-cinderplugin   Bound    pvc-efbb1829-06bc-43e5-adbc-c15ffbab10e3   50Gi       RWO            standard-csi   142m

used nginx.yaml to test pvc , mounted to /var/lib/www/test
root@nginx:/var/lib/www/test# ls
ls: cannot open directory '.': Permission denied

What you expected to happen:
How to reproduce it:
Anything else we need to know?:
Environment:

openstack-cloud-controller-manager version: v1.17.0
OpenStack version: train
Kubernetes v1.17.1
Fedora CoreOS 31
Cinder CSI Version: latest

@yzhelezko The fsck thing is not error actually, it's k8s is trying to check for issues on disk but in cinder-csi case before file system formatting. You can see from the log that the formatting and mounting are both successful in the end.


sort by a multi-line 'select as' query

Allow it to grab the last 'as' in a multi-line select query (within a select query). Also does one line.
Would solve something complicated like this:
->select('table.begin_at', DB::raw('(SELECT table2.id FROM table as t ...) as file_id'))

We'll be on L4 for a while due to a large project, hope you don't mind the requests.

@lisa-fehr, I don't mind these pull requests and I appreciate the help. Anyways, can you please make the script case insensitive? Maybe something like below...
$matches = explode(' as ', Str::lower($str));

Good idea. Added Str::lower($str)

Released on v3.6.12. Thanks!


Make peer dependency on font-awesome less optimistic

Currently dependency is too optimistic:
https://github.com/gowravshekar/font-awesome-webpack/blob/master/package.json#L32
please fix the version. In fact, I'll make a PR soon-ish doing just that :)

Bump. I think this is causing errors for me with extras and spinning files no longer there.

Bumped the font-awesome required version to 4.3.0.
If Font-Awesome alters the less file namings agin, use "Custom Configuration" - font-awesome.config.js in future to prevent issues like this.


Attach hooks to all methods?

Is this a package I can use to check that the user is authenticated before running any method? And throwing an error otherwise?

Yes, you could do that.

There's no function to actually add hooks on a global basis, every method has to be specified. What I'm getting at is something like Underscore's pick or omit, but for the hooks. Would you guys accept a PR?

Sure send us a PR man!

Would be useful for this:
https://github.com/meteor/meteor/issues/4652


Prevent stale reference to model's config_.Name()

This can be non-deterministically reproduced by loading and unloading
a model with dynamic sequence batching enabled, and running
tritonserver binary with --log-verbose=1.
The destructor of DynamicBatchScheduler, when --log-verbose=1, prints
the following:
  LOG_VERBOSE(1) << "Stopping dynamic-batcher thread for " << model_->Name()
                 << "...";

That is actually printed by the BatcherThread() function; however,
~DynamicBatchScheduler waits to return until it has run.
Unfortunately, model_->Name() is implemented as:
const std::string& Name() const { return config_.name(); }

config_ has already been destroyed by the time that the scheduler's
destructor is run, so the reference to the name is stale. This stale
reference sometimes causes a segfault (or sometimes not, depending
upon how long it takes to reach a null byte in the invalid memory)
when unloading a model that looks like this:
I0427 20:19:40.309255 106 model_repository_manager.cc:700] AsyncLoad()
'vista_conformer-ctc-decoder-cpu-streaming'
I0427 20:19:40.309388 106 model_repository_manager.cc:939]
TriggerNextAction() 'vista_conformer-ctc-decoder-cpu-streaming'
version 1: 1
I0427 20:19:40.309409 106 model_repository_manager.cc:975] Load()
'vista_conformer-ctc-decoder-cpu-streaming' version 1
I0427 20:19:40.550437 106 ctc-decoder-library.cc:22]
TRITONBACKEND_ModelFinalize: delete model state
I0427 20:19:40.550490 106 triton_backend_manager.cc:101] unloading
backend 'riva_asr_decoder'
I0427 20:19:40.594447 106 sequence_batch_scheduler.cc:866] Stopping
sequence-batch reaper thread...
[Thread 0x7fbb0bfff000 (LWP 128) exited]
--Type <RET> for more, q to quit, c to continue without paging--

Thread 7 "tritonserver" received signal SIGSEGV, Segmentation fault.
[Switching to Thread 0x7fbb20c15000 (LWP 127)]
0x00007fbbffedc959 in ?? () from /usr/lib/x86_64-linux-gnu/libc.so.6
(cuda-gdb) bt
 #0  0x00007fbbffedc959 in ?? () from
 #/usr/lib/x86_64-linux-gnu/libc.so.6
 #1  0x00007fbc001edb28 in std::basic_streambuf<char,
 #std::char_traits<char> >::xsputn(char const*, long) () from
 #/usr/lib/x86_64-linux-gnu/libstdc++.so.6
 #2  0x00007fbc001df824 in std::basic_ostream<char,
 #std::char_traits<char> >& std::__ostream_insert<char,
 #std::char_traits<char> >(std::basic_ostream<char,
 #std::char_traits<char> >&, char const*, long) ()
   from /usr/lib/x86_64-linux-gnu/libstdc++.so.6
   #3  0x00007fbc00722f4c in
   nvidia::inferenceserver::DynamicBatchScheduler::BatcherThread(int) ()
   from /opt/tritonserver/lib/libtritonserver.so
   #4  0x00007fbc00182de4 in ?? () from
   /usr/lib/x86_64-linux-gnu/libstdc++.so.6
   #5  0x00007fbc00600609 in start_thread ()
      from /usr/lib/x86_64-linux-gnu/libpthread.so.0
      #6  0x00007fbbffe70293 in clone () from /usr/lib/x86_64-linux-gnu/libc.so.6

The workaround is to swap the order of scheduler_ and config_ in
src/core/model.h, since destructors of member variables run bottom-up.
TritonModel::SetConfiguredScheduler()
depends upon config_ having already been set, but this is the case
already because the constructor for TritonModel has already run by the
time SetConfiguredScheduler() is run.

@tanmayv25 Now that #135 is fixed can we close this PR?


Openai Assistant async _await_for_run method is not really async

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
The function _await_for_run inside openai_assistant/base.py has a sleep invocation, in the file only a sleep function is imported that is the time.sleep implementation which is blocking.
awaiting asyncio.sleep instead would be the correct solution to avoid blocking invocations in an async function.
In particular this code:
    async def _await_for_run(self, run_id: str, thread_id: str) -> Any:
        in_progress = True
        while in_progress:
            run = await self.async_client.beta.threads.runs.retrieve(
                run_id, thread_id=thread_id
            )
            in_progress = run.status in ("in_progress", "queued")
            if in_progress:
                sleep(self.check_every_ms / 1000)
        return run

should become:
    async def _await_for_run(self, run_id: str, thread_id: str) -> Any:
        in_progress = True
        while in_progress:
            run = await self.async_client.beta.threads.runs.retrieve(
                run_id, thread_id=thread_id
            )
            in_progress = run.status in ("in_progress", "queued")
            if in_progress:
---------------await asyncio.sleep(self.check_every_ms / 1000)
        return run

in addition to this asyncio should be imported somewhere in that file.
I may open a pull request to fix this but I would be able to do so in the beginning of the next week.
Error Message and Stack Trace (if applicable)
No response
Description
I'm trying to create a fastAPI endpoint to serve langchain completions and I noticed that increasing check_every_ms would block completely my endpoint for the specified ms instead of asyncrhonously awaiting the specified time.
Considering the high response time of some openai models it is not an unlickely situation increasing that number to avoid useless excess traffic every second.
I include system info below but this issue is present also in the current langchain repo.
System Info
System Information

OS:  Linux
OS Version:  #41~22.04.2-Ubuntu SMP PREEMPT_DYNAMIC Mon Jun  3 11:32:55 UTC 2
Python Version:  3.11.6 (main, Jul  5 2024, 16:48:21) [GCC 11.4.0]

Package Information

langchain_core: 0.2.9
langchain: 0.2.1
langchain_community: 0.2.1
langsmith: 0.1.81
langchain_cli: 0.0.25
langchain_text_splitters: 0.2.1
langserve: 0.2.2

Packages not installed (Not Necessarily a Problem)
The following packages were not found:

langgraph

Yeah sounds like a change we'd want to make -- will you open a PR?

I can open it, but I have to wait until Monday.
Is that ok?

Ofc will be great! Thank you!

Opened PR.


Fix Process Botlist is in Helpers

open helpers.watchlist to import process_botlist.
with this change it will not start watchlist.py correctly

The fix for watchlist seems logical. But why did you disable the lunarcrush code in the marketcollector?

Oh sorry. I don't want to send this in my pull request too.
Lunar doesn't work for now. API v3 should be offline and v4 is paid

Better include a lunarcrush_enabled param instead of completely disable it by code.
Funny enough I started with all of these script because of Lunarcrush, first scripts where called altrank.py and galaxyscore.py back then.


number of docs in ES is less than number of docs in Mongo after sync

I have a mongo cluster with 3 sharded nodes and 5 collections and want to connect it to ElasticSearch 2.x.
After sync everything is fine for 4 collections but the number of docs in ES is less than Mongo for the biggest collection(around 80M docs).
Also I tried to run the connector several times. After sync the number of docs in ES is around 60M but in Mongo there are 80M records.
MongoDB 3.2.9
Python 2.7.6
Ubuntu 14.04.4 LTS
ES 2.x

Updates:
Logs are here. The approximated number of docs for mongo nodes are correct.
2017-01-20 14:27:35,862 [INFO] mongo_connector.oplog_manager:120 - OplogThread: Initializing oplog thread
2017-01-20 14:27:35,884 [INFO] mongo_connector.connector:405 - MongoConnector: Starting connection thread MongoClient(host=[u'X'], document_class=dict, tz_aware=False, connect=True, replicaset=u'db')
2017-01-20 14:27:35,885 [INFO] mongo_connector.oplog_manager:120 - OplogThread: Initializing oplog thread
2017-01-20 14:27:35,889 [INFO] mongo_connector.connector:405 - MongoConnector: Starting connection thread MongoClient(host=[u'X'], document_class=dict, tz_aware=False, connect=True, replicaset=u'db2')
2017-01-20 14:27:35,891 [INFO] mongo_connector.oplog_manager:120 - OplogThread: Initializing oplog thread
2017-01-20 14:27:35,895 [INFO] mongo_connector.oplog_manager:605 - Bulk upserting approximately 27964103 docs from collection 'X'
2017-01-20 14:27:35,896 [INFO] mongo_connector.connector:405 - MongoConnector: Starting connection thread MongoClient(host=[u'X'], document_class=dict, tz_aware=False, connect=True, replicaset=u'db3')
2017-01-20 14:27:35,896 [INFO] mongo_connector.oplog_manager:605 - Bulk upserting approximately 28639433 docs from collection 'X'
2017-01-20 14:27:35,902 [INFO] mongo_connector.oplog_manager:605 - Bulk upserting approximately 22505012 docs from collection 'X'

I fixed the problem.
Its because db.collection.count() in sharded clusters is not accurate in mongo. To get the actual number of docs need to run this:
db.collection.aggregate(
...    [
...       { $group: { _id: null, count: { $sum: 1 } } }
...    ]
... )

So mongo-connector is actually replicated all the documents?

Yes. mongo-connector is working fine.
On Sat, Jan 21, 2017 at 7:11 AM, Shane Harvey<EMAIL_ADDRESS>wrote:

So mongo-connector is actually replicated all the documents?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mongodb-labs/mongo-connector/issues/630#issuecomment-274231416,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABDS9SQJXAeF9_Ij6YKbr3lblgLBZdFHks5rUX5MgaJpZM4LpZrO
.

--
Reza Bakhshandeh

Thanks!


Template per Response Code

Hey,
would it be possible to define a template per response code? This would make it easier, to render a different template, e.g. in case of validation errors of API input or other situations (e.g. item not found ...).
I understand that it would likely also be possible to have that logic within the template (if the response object is made available), but it might be need lees boilerplate code to be able to see that directly in the Python code.
Henning

Hi,
I'd recommend implementing an exception handler middleware that converts exceptions to HTML responses. The middleware has access to the request, it can check whether it was an hx-request or not if it matters for your use-case.

FastAPI must resolve dependencies (and validate the route's data inputs) before calling the route, so if the API's input is incorrect, FastAPI will raise an HTTPException with 422 status code before calling the route (and its fasthx wrapper). As the fasthx decorator is not executed, it can not solve such an issue.
Regarding the response code: if your route returns a Response object, FastHX decorators will simply return it without doing anything. If your route doesn't return a Response object and you define status codes in the FastAPI path operation decorator, that's out of of reach for FastHX, so it can't do anything.

I assume you only want to differentiate between a successful response and error cases. If the returned error HTML is fairly generic (part of its content could come from HTTPException or a custom subclass with extra properties), then a middleware would be a good solution for you. This lib could implement a generic, ready to use middleware, but I'd need more user feedback (and some consensus) on how it should work. This could also be an APIRouter-specific thing, then again, a specialized APIRouter subclass could be added to the lib, but without input from several users on how it should work, I'm hesitant to add such a feature. And finally, as outlined above, a route-specific decorator couldn't handle input errors, because the request doesn't reach the decorator, so it can not be a complete solution on its own.
Peter

Closing for lack of response.


Prevent publication completion if Future completed exceptionally

Closes #395

Superseded by the solution discussed for #395. Thanks nonetheless!


Consider including implicitly

No sense in making Stylus users write @import 'lost'.

This makes install directions different for both. Will leave out for now.


[FEATURE] check if our implementation of sign does this

A way to trivially implement sign
https://twitter.com/pshufb/status/1575766441500213255?t=Emg-8jBEPjct0rTO8XtSDA&s=19

This sign function only return +1/-1 so that is our signnz function which currently do that:
           if constexpr( unsigned_value<T> ) { return one(eve::as(a)); }
    else if constexpr( floating_value<T> ) { return bit_or(one(eve::as(a)), bitofsign(a)); }
    else if constexpr( signed_value<T> )
    {
      using elt_t = element_type_t<T>;
      return (a >> (sizeof(elt_t) * 8 - 2)) | T(1);
    }

the unsigned case is obvious, the other are doing the masking thing also specified in the tweet answers.
There is also a pedantc version that do that + nan handling.
We can probably test the max(min) variant in place of the non unsigned one and check code gen

On x86, for integers, we don't have all min/max till SSE4 at which point we have _mm_sign_epi which is obv. better.
On x86 for reals:
signnz(eve::sse_abi_v0::wide<float, eve::fixed<4l> >):
        endbr64
        minps   xmm0, XMMWORD PTR .LC0[rip]
        maxps   xmm0, XMMWORD PTR .LC1[rip]
        ret
signnz2(eve::sse_abi_v0::wide<float, eve::fixed<4l> >):
        endbr64
        andps   xmm0, XMMWORD PTR .LC2[rip]
        orps    xmm0, XMMWORD PTR .LC0[rip]
        ret

Sounds about the same to me.

On other architecture, we got min/max earlier, e.g AARCH64:
signnz(eve::arm_abi_v0::wide<float, eve::fixed<4l> >):
        fmov    v2.4s, 1.0e+0
        fmov    v1.4s, -1.0e+0
        fmin    v0.4s, v0.4s, v2.4s
        fmax    v0.4s, v0.4s, v1.4s
        ret
signnz2(eve::arm_abi_v0::wide<float, eve::fixed<4l> >):
        movi    v1.4s, 0x80, lsl 24
        adrp    x0, .LC0
        ldr     q2, [x0, #:lo12:.LC0]
        and     v0.16b, v1.16b, v0.16b
        orr     v0.16b, v0.16b, v2.16b
        ret

It looks better indeed.


Users SNBAT see deprecation warnings when using CLI

Expected Behavior
Suppressed output
Current Behavior

No longer getting this behavior so I am closing.


triton.txt

I'm a student of Triton International College, Kathmandu, Napal

@HemYavav First stage of verification passed, official domain is triton.edu.np as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@HemYavav Thank you for your patience while waiting for a review and processing your request. Pull request merged.


bug: Collection-repeat  - Maximum call stack size exceeded error

Type: bug
Platform: android 4.4 browser
Hi,
Am using collection-repeat to iterate around 1800 records and am getting the maximum call stack reached error. Am pasting the HTML code here & attaching the screenshot below.

NOTE: I tried to iterate only through collection-repeat by converting entire dataset into an array and, search filter was not working properly.

Greetings @OSSDeveloper!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

Hi,
Problem solved. I converted entire dataset into array and applied custom filter. Its working fine now. Thanks.

Greetings @OSSDeveloper!
I've closed this issue because my sensors indicated it was old and inactive, and may have already been fixed in recent versions of Ionic. However, if you are still experiencing this issue, please feel free to reopen this issue by creating a new one, and include any examples and other necessary information, so that we can look into it further.
Thank you for allowing me to assist you.


2 - Use IDs instead of label values for category mapping?

For now mapping is done like this:
"Theme": {
                "Flore": "Faune et flore",
                "Faune": "Faune et flore",
                "Géologie": "Eau et géologie",
                "Point de vue": "Forêt",
                "Eau": "Eau et géologie",
                "Histoire et patrimoine": "Histoire et culture",
                "Savoir-faire": "Histoire et culture",
                "Agropastoralisme": "Agriculture et élevage",
            },

As foreign keys in the API are 90% of the time only displayed as IDs in the main objects route (Trek, POI...), this functionment needs additional API requests in order to retrieve the value corresponding to those IDs.
Pros:

no additional API requests;
simplify mapping functions;
robust to category label changes ("MTB" to "Mountain Bike" for instance), thus reduces maintenance load.

Cons:

risk of false category data without error raised (if categories IDs change, or if the whole meaning of a category changes, e.g. "MTB" to "Canoe") whereas label value mapping would raise an error to any change.

Selected solution: IDs are used in the GAG app mapping, and categories' labels from last aggregation are displayed in the mapping web app for human control.


OSD 7598 - boilerplate update + transitioning to crd v1

Need to hold these PRs pending boilerplate updates to handle CRD v1 in the appsre pipeline script for bundle generation.

/hold

bp commit level is good (includes crd v1 + fix)
New CRD installs cleanly, and oc explain works.
/lgtm

/approve


nixos/bluetooth: add input and network service configs

Description of changes
Fixes #204418
CC: @jbit for testing this out.

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

The code looks good. @jbit can you help with testing?

Yup! I was in the middle of rebuilding my system with this change when I received  your message :smile:
The change works great on my setup, thanks @ymatsiuk for taking a look at my issue report so quickly!
I'm not sure if there are automated tests you want me to run, but I tried the manual way as detailed below.
Testing:
I used a local nixpkgs checkout with 1d61d168a6fe30eb42c02fff8a097a5c224c7ec9 cherry picked on top.
After rebuilding, I ended up with a bluez package with the expected new symlinks:
# ls -l /nix/store/gk5nwc6ilvgl5da96xkfgffc460y1x6g-bluez-5.64/etc/bluetooth/
total 0
lrwxrwxrwx 1 root root 25 Jan  1  1970 input.conf -> /etc/bluetooth/input.conf
lrwxrwxrwx 1 root root 24 Jan  1  1970 main.conf -> /etc/bluetooth/main.conf
lrwxrwxrwx 1 root root 27 Jan  1  1970 network.conf -> /etc/bluetooth/network.conf

I manually created a /etc/bluetooth/input.conf file filled with only failure, ran systemctl restart bluetooth, and looked at journalctl -u bluetooth, which contained an error proving the bluetoothd is actually processing the input.conf file.
Dec 07 21:37:05 catbag bluetoothd[569459]: profiles/input/manager.c:load_config_file() Parsing /nix/store/gk5nwc6ilvgl5da96xkfgffc460y1x6g-bluez-5.64/etc/bluetooth/input.conf failed:
Key file contains line “failure” which is not a key-value pair, group, or comment

I then added the following to my configuration.nix file:
  hardware.bluetooth = {
    enable = true;
    powerOnBoot = true;
    input = { General = { IdleTimeout = 1; }; };
  };

Which resulted a /etc/bluetooth/input.conf file with the contents:
[General]
IdleTimeout=1

After restarting the bluetooth daemon again, my input devices now go to sleep after one minute of inactivity.
Result
Everything works as expected. :+1: from me

it's about time i merge this :)


[evergreen-travis] Update .travis.yml node versions

This is an automated PR, by the evergreen-travis project.
It helps you to keep the versions of node in your travis CI configuration up to date with new releases.

If you do not want to receive these PRs then remove your repository from our repository list
If there is a problem with the generated travis.yml, or something isn't right with this PR then please raise an issue

Coverage remained the same at 100.0% when pulling 98e50b7193f62958365b0714c9508bb888f54570 on evergreen-travis:evergreen-travis-1479402670680 into 607d56189f4ffd441221ace6fd0d714447528c3b on Kikobeats:master.


How to rewrite URL using Request Transformer plugin?

How do I rewrite URL using Request Transformer plugin?
In the docs, there is no example URL rewrite/replacement (https://docs.konghq.com/hub/kong-inc/request-transformer/#enabling-the-plugin-on-a-route).
For example, I want to rewrite /foo using route, is this right?
apiVersion: networking/v1beta1
kind: Ingress
metadata:
  name: <route>
  annotations:
    kubernetes.io/ingress.class: kong
    plugins.konghq.com: <request-transformer-example>
spec:
  rules:
  - host: rewrite.bar.com
    http:
      paths:
      - path: /foo(/|$)(.*)
        backend:
          serviceName: echo
          servicePort: 80

apiVersion: configuration.konghq.com/v1
kind: KongPlugin
metadata:
  name: <request-transformer-example>
config:
  replace:
    uri:
    - /foo(/|$)(.*):/(/|$)(.*)
plugin: request-transformer

For example, above will result:
rewrite.bar.com/foo rewrites to rewrite.bar.com/
rewrite.bar.com/foo/ rewrites to rewrite.bar.com/
rewrite.bar.com/foo/bar rewrites to rewrite.bar.com/bar
What I want to achieve is similar to what in NGINX controller docs (https://kubernetes.github.io/ingress-nginx/examples/rewrite/#rewrite-target).

I just went through something similar (assuming that I'm understanding what you're asking correctly). I configured my ingress like this:

---
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  name: kong-httpbin
  annotations:
    kubernetes.io/ingress.class: kong
    konghq.com/plugins: stripper
spec:
  rules:
  - host: host.example.com
    http:
      paths:
      - path: /foo/(.*)
        backend:
          serviceName: httpbin
          servicePort: 8000

---
apiVersion: configuration.konghq.com/v1
kind: KongPlugin
metadata:
  name: stripper
config:
  replace:
    uri: '/$(uri_captures[1])'
plugin: request-transformer

Closing as resolved.

Closing as resolved.

This is not working for me, I tried the above still the request to the backed is the same, it does not strip foo out

Hi. any updates ?
i have similar issue


Huge memory consumption

The application loads a 6Mb Gif and the memory consumption increases from 100Mb to 1.3Gb.

Hi @gundone,
This is a duplicate of many other issues. It's a known problem that I'm not going to fix, because it's inherent to the way this library works. I made another library, XamlAnimatedGif, whose goal was specifically to reduce memory usage (at the cost of higher CPU usage).
That being said, if you have a 6MB GIF, you should probably consider converting it to a video...


Clean up code for in-person workshop delivery:

Move VPC to single NAT gateway for reduced cost, faster spinup time.
Use an explicit provider for the Pinecone Index to remove the need for the API key to be stored in an env var as well as Pulumi config.
Fix Markdown linting issues.
Move frontend containers to private subnet for improved security posture.
Only allow egress from LB to VPC CIDR, only on the port used by the frontend service.

Problem
Describe the purpose of this change. What problem is being solved and why?
Solution
Describe the approach you took. Link to any relevant bugs, issues, docs, or other resources.
Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[x] Infrastructure change (CI configs, etc)
[x] Non-code change (docs, etc)
[ ] None of the above: (explain here)

Test Plan
Describe specific steps for validating this change:
Spin up the refarch, verify the frontend service can still be hit.

Verified this today with a fresh deployment and tear down. Thanks for the contributions!


A restore method on the model

Restores a deleted instance of a Deletable model.
(Need to decide if it restores just the instance or all cascading instances. )

https://github.com/upgrad/django-deletes/pull/6 solves it trivially without tracking changesets.


Using a Delayed Future in Widget test stops the test from completing

@felangel thanks for this fantastic package, you've saved mocking for us in the NS age!
I've have run into an issue though and I'm not sure if this is a case of my not doing something correctly but I've found that trying to use Future.delayed() in a mocked method in a widgetTest causes the test to hang and never complete.
My code is:

testWidgets('home page shows progress indicator when loading',
      (tester) async {
    final api = MockApiService();
    final lvm = LoginViewModel(api);

    when(api).calls(#requestLoginCode).thenAnswer((_) async {
      await Future<void>.delayed(Duration(seconds: 1));
      return APIResult.Error;
    });

    final w = ProviderScope(
      overrides: [loginProvider.overrideWithProvider(Provider((ref) => lvm))],
      child: HomePage(),
    );

    await tester.pumpWidget(wrapForTest(w));

    await lvm.requestCode('12345');

    await tester.pump();

    await tester.pump(Duration(seconds: 2));

    final hasSpinner = tester.any(find.byType(CircularProgressIndicator));

    expect(hasSpinner, true);
  });

where requestCode() calls the ApiService objects requestLoginCode() method which returns a Future that completes once its network API call (in the real, nnon mocked ApiService class) completes.
If I remove the Future.delayed() everything works as expected, BUT I want the delay in there because I'm trying to test that a CircularProgressIndicator is displayed during the period before the Future returned by requestLoginCode completes.
I've included the Riverpod setup code, but I don't think its relevant here.
My suspicion is this is maybe due to widgetTests running inside a FakeAsync zone though according to this SO question this approach should work (or at least used to work with mockito).

Hi @maks 👋
Thanks for opening an issue!
Are you still able to reproduce this in the latest release (v0.0.2-dev.2)? Thanks! 🙏

@maks any updates or can this be closed?

@felangel sorry for the slow reply, its been a long weekend holiday here.
I've upgraded to 0.0.2-dev.5 and changed my test to now use new type-safe syntax:
    when(() => authRepo.requestOtp('12345')).thenAnswer((_) async {
      await Future<void>.delayed(Duration(seconds: 1));
      return AuthResult.error(400, 'test error');
    });

But unfortunately still seeing the test hang.

@maks no worries thanks for getting back to me!
Just to clarify, you said this was not a problem using mockito?
I think this is expected because in a widget test you will need to use tester.runAsync(() {...}) in this situation.

@felangel actually I realised when I looked again at my test that I was making it more complicated than it needed to be as I could just override my providers state and not even need to worry about the async behaviour.
I still think there maybe an issue with async handling as I would have expected that FakeAsync to work with the Future I was declaring inside the FaskAsync Zone, but I'll close this for now until I get a chance to see if I can write a simple standalone test to reproduce the issue. Thanks again for looking at it!


Do not show invalid route in elevate

This was reported a couple weeks ago via our bug bounty (and again over
this week end): the redirectTo parameter on the elevate route (which is
used to allow reloading this page) can be used to spoof the error
message that's shown.
While this is arguably reasonably unlikely to convincingly trick
someone, it's still worth fixing, which this patch does.
--
Fixes: https://trello.com/c/hNdb8op1/29-prevent-social-engineering-via-impersonate-s-redirectto
cc @fancyremarker @chasballew

Nice, thanks for this


Encoding Error: 'ascii' codec can't encode character '\xe0' in position 41: ordinal not in range(128)

Description
When attempting to use the llm command-line interface (CLI) on Windows, I encounter an encoding error related to the ASCII codec. Despite setting environment variables to enforce UTF-8 encoding, the issue persists. Additionally, configuring models results in errors indicating that certain models are unknown or not recognized.

Steps to Reproduce

Set Up the Environment:

Operating System: Windows 10
PowerShell Version:
Name             : ConsoleHost
Version          : 5.1.22621.4111
InstanceId       : e0d61ad7-8a35-4d6f-9199-74f95ce5a7e9
UI               : System.Management.Automation.Internal.Host.InternalHostUserInterface
CurrentCulture   : en-US
CurrentUICulture : en-GB
PrivateData      : Microsoft.PowerShell.ConsoleHost+ConsoleColorProxy
DebuggerEnabled  : True
IsRunspacePushed : False
Runspace         : System.Management.Automation.Runspaces.LocalRunspace
Python Version: 3.11
Poetry Version: Poetry (version 1.8.2)
llm Version: 0.16

Configure Environment Variables:
$Env:PYTHONIOENCODING = "utf-8"
$Env:PYTHONUTF8 = "1"

Activate Poetry Virtual Environment:

Navigate to a project directory (ensure it's not within site-packages):
mkdir C:\Users\samue\llm-project
cd C:\Users\samue\llm-project
poetry init --no-interaction
poetry add llm
poetry shell

Run the llm Command with ASCII Input:
llm 'hi'

Expected Behavior: The command should execute without encoding errors and return a response from the language model.
Actual Behavior:
Error: 'ascii' codec can't encode character '\xe0' in position 41: ordinal not in range(128)

Run the llm Command with Non-ASCII Input:
llm 'Café'

Actual Behavior:
Error: 'ascii' codec can't encode character '\xe0' in position 41: ordinal not in range(128)

Attempt to Add and Set a Model:
llm models add groq-llama3.1-70b --path "C:\Path\To\groq-llama3.1-70b"
llm models set default groq-llama3.1-70b

Actual Behavior:
Error: Unknown model: groq-llama3.1-70b

List Available Models:
llm models list

Actual Output:
Available models:
- groq-llama3.1-70b

Attempt to Use the Default Model:
llm 'hi'

Actual Behavior:
Error: 'groq-llama3.1-70b' is not a known model

Expected Behavior

The llm command should handle both ASCII and non-ASCII inputs without encountering encoding errors.
Adding and setting models should recognize and configure valid models without errors.

Actual Behavior

Encountering an encoding error when using non-ASCII characters, specifically:
Error: 'ascii' codec can't encode character '\xe0' in position 41: ordinal not in range(128)

Errors related to unrecognized models when attempting to add or set a default model:
Error: 'groq-llama3.1-70b' is not a known model

Environment

Operating System: Windows 10
PowerShell Version: [
Name             : ConsoleHost
Version          : 5.1.22621.4111
InstanceId       : e0d61ad7-8a35-4d6f-9199-74f95ce5a7e9
UI               : System.Management.Automation.Internal.Host.InternalHostUserInterface
CurrentCulture   : en-US
CurrentUICulture : en-GB
PrivateData      : Microsoft.PowerShell.ConsoleHost+ConsoleColorProxy
DebuggerEnabled  : True
IsRunspacePushed : False
Runspace         : System.Management.Automation.Runspaces.LocalRunspace]
Python Version: 3.11
Poetry Version: [Please provide the output of poetry --version]
llm Version: 0.16
Installation Path: C:\Users\samue\AppData\Roaming\pypoetry\venv\Lib\site-packages\llm\
Virtual Environment Path: C:\Users\samue\AppData\Roaming\pypoetry\venv\Lib\site-packages\llm\llm-test\.venv

Environment Variables
$Env:PYTHONIOENCODING = "utf-8"
$Env:PYTHONUTF8 = "1"

Additional Context

Attempted Solutions:

Set PowerShell's output encoding to UTF-8.
Configured Python environment variables (PYTHONIOENCODING and PYTHONUTF8) to enforce UTF-8 encoding.
Activated the Poetry virtual environment within a separate project directory.
Tried adding and setting different models, including a local model (groq-llama3.1-70b), which resulted in errors.

Error Messages:
Error: 'ascii' codec can't encode character '\xe0' in position 41: ordinal not in range(128)

Error: 'groq-llama3.1-70b' is not a known model

Commands Run:
llm 'hi'
llm 'Café'
llm models add groq-llama3.1-70b --path "C:\Path\To\groq-llama3.1-70b"
llm models set default groq-llama3.1-70b
llm models list
llm 'hi'

Attempts to Use llm-cmd:
llm-cmd hi

Result:
llm-cmd : The term 'llm-cmd' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that
the path is correct and try again.

Notes:

Attempting to run poetry shell within the site-packages\llm directory results in an error due to the absence of a pyproject.toml file.
The issue persists even after setting environment variables within the Poetry shell.

llm -m claude-3-5-sonnet-latest --key llm-claude-3 'Hello'

Error: 'ascii' codec can't encode character '\u221a' in position 0: ordinal not in range(128)

facing the same issue, running aarch macOS, iterm2, zsh


Update Hardhat

@sebastianst I totally forgot that, but if you mean the update to hardhat-toolbox (and i guess you do) we already did that in #28.

I think we meant that, thanks!


[terraform] Simple full node network on one validator

This adds a network of fullnodes attached to the first validator, with
1-10 fullnodes configurable.
Test Plan: Deployed to mgorven workspace. fullnode startup currently
broken due to 4c209a863, but worked previously.

Please don't land, requires coordination in libra-ops.

Working now, upstream_peers config was missing.

@bors-libra: delegate+

:v: @mgorven can now approve this pull request

@bors-libra r=phoenix-antigravity

:pushpin: Commit ecac135 has been approved by phoenix-antigravity

:hourglass: Testing commit ecac1352060252b860edb43afc892295a45eed59 with merge 5ba3162a21fab7471c11ff8c098853b489e33ddd...

:broken_heart: Test failed - checks-circle_commit_workflow

@bors-libra retry

:hourglass: Testing commit ecac1352060252b860edb43afc892295a45eed59 with merge 91ef1654b5519e41fdfbcded7c402c6dfc711675...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: phoenix-antigravity
Pushing 91ef1654b5519e41fdfbcded7c402c6dfc711675 to master...


float32

Hello, thank you for implementing this great application. However, for your train_denoiser.ipynb, I ran it on my Google Colab and it gave me a mismatched type of float32. If I change all float32 types to float64, it fixes the error. This can be something to keep in mind in the next update of your code.

Sure. Will check and get back.

This is likely because of the TensorFlow version mismatch. I used TensorFlow 2.4.1. Besides if you change the dtype of the noise to float32 it should not happen.
noise = tf.experimental.numpy.random.randn(batch_size, 32, 32, 3) * self.sigma
noise = tf.cast(noise, tf.float32)


Why all compartment ids are not consistent?

Hi
I wonder why "c_i" is the only compartment of the model that is not a single letter?
i exists in BiGG (http://bigg.ucsd.edu/compartments/i) and could be used like this instead of c_i
Also in BiGG the extracellular compartment is e, not s that is eyespot.
Would you mind to change them?

Thanks for getting in touch @smoretti . Could I ask where did you encounter c_i? I'm only asking because I couldn't find it in the yml file. Here is the compartments section:
- compartments: !!omap
    - s: "Extracellular"
    - p: "Peroxisome"
    - m: "Mitochondria"
    - c: "Cytosol"
    - l: "Lysosome"
    - r: "Endoplasmic reticulum"
    - g: "Golgi apparatus"
    - n: "Nucleus"
    - i: "Inner mitochondria"
    ```

I found it in the SBML file

Also in BiGG the extracellular compartment is e, not s that is eyespot.
Would you mind to change them?

this might be a good idea

I wonder why "c_i" is the only compartment of the model that is not a single letter?

This probably was due to an issue of the SBML exporting function, but should be fixed in the next release.

The i to c_i conversion is reported as a bug in RAVEN.

the c_i issue has been fixed in the latest release v1.10

Also in BiGG the extracellular compartment is e, not s that is eyespot. Would you mind to change them?

In addition to the BIGG models, yeastGEM also use e as abbreviation of extracellular compartment. Therefore it makes sense to apply it to Human-GEM.
The modification to model file would be simple and straightforward. But I'm afraid this would affect some functions in HumanGEM and RAVEN. @JonathanRob what do you?

I'm also open to this suggested change in abbreviation. But yes, this should be done carefully because it will require additional changes and will break backwards compatibility in some cases.
Since we're suggesting changes to the extracellular compartment abbreviation, we should also consider changing other compartment(s). For example, in Human-GEM, peroxisome is abbreviated as p, whereas I believe COBRA models typically use x to abbreviate the peroxisome. So it would make sense to me to change all of the abbreviations so that they are consistent with whichever scheme is used most often (i.e., in COBRA-like models).

I wonder if there is any existing standards defined for cellular compartment abbreviations somewhere, like SBML or COBRA?

I wonder if there is any existing standards defined for cellular compartment abbreviations somewhere, like SBML or COBRA?

@edkerk you seem to be more familiar with the standards - do you know of any compartment abbreviations?

To my knowledge there is no universal standard, although the closest to this would be the compartment abbreviation convention that is used in BiGG models:

BiGG ID
Name

c
cytosol

e
extracellular space

p
periplasm

m
mitochondria

x
peroxisome/glyoxysome

r
endoplasmic reticulum

v
vacuole

n
nucleus

g
golgi apparatus

u
thylakoid

l
lysosome

h
chloroplast

f
flagellum

s
eyespot

im
intermembrane space of mitochondria

cx
carboxyzome

um
thylakoid membrane

cm
cytosolic membrane

i
inner mitochondrial compartment

mm
mitochondrial intermembrane

w
wildtype staph aureus

y
cytochrome complex

KBase/SEED had a compartment list very close to this one in the past. It does not look to be available online any more.
Some pieces of the former list are there: https://github.com/ModelSEED/ModelSEEDDatabase/blob/master/Templates/*/Compartments.tsv in the different Templates sub-directories.
In MetaNetX/MNXref we built a "SEED" compartment list over the years.
Despite of the differences between the two lists used by BiGG and SEED, both abbreviation changes for "Extracellular", from s -> e (proposed by @smoretti), and "Perxisome", from p - > x (proposed by @JonathanRob), make sense and would promote the convenience and compatibility of Human-GEM.
I would suggest to take steps in this implementation, maybe begin with the modification to Perxisome compartment abbreviation.

The abbreviation x is already assigned to the boundary compartment in Human-GEM. The change from p to x may affect some existing implementations (e.g. addBoundaryMets).

How about using b for the boundary compartment? Theoretically it would clash with the abbreviation list in MetaNetX shown above by @smoretti, but only if the model would be imported into MetaNetX after running addBoundaryMets. So, to my understanding, this would be a problem in practice.


SSR fails with NODE_ENV=production

I am doing SSR with<EMAIL_ADDRESS>and node v6.5.0 and.

When NODE_ENV=development server side works correctly.
When NODE_ENV=production, SSR fails with the following error:

Invariant Violation: Minified React error #23; visit http://facebook.github.io/react/docs/error-decoder.html?invariant=23 for the full message or use the non-minified dev environment for full errors and additional helpful warnings.
    at reactProdInvariant (myapp/node_modules/react/lib/reactProdInvariant.js:32:15)
    at Object.onlyChild [as only] (myapp/node_modules/react/lib/onlyChild.js:34:166)
    at ApolloProvider.render (myapp/node_modules/react-apollo/ApolloProvider.js:33:31)
    at getChildFromComponent (myapp/node_modules/react-apollo/server.js:16:26)
    at getQueriesFromTree (myapp/node_modules/react-apollo/server.js:54:24)
    at getQueriesFromTree (myapp/node_modules/react-apollo/server.js:53:9)
    at getDataFromTree (myapp/node_modules/react-apollo/server.js:72:14)

The error can be decoded at http://facebook.github.io/react/docs/error-decoder.html?invariant=23, and is referred to the fact that the prop children of ApolloProvider is undefined.
How to reproduce
This is my code, a bit simplified:
Root.js
const Root = ({
    history: _history,
    routes,
    routerState,
    client,
    store
}) => {
    let component;
    if (__SERVER__)
        component = <RouterContext history={_history} {...routerState} />;
    else
        component = <Router history={_history} routes={routes} render={scrollHandlingMiddleware} />;
    return (
        <ApolloProvider client={client} store={store}>
                {component}
        </ApolloProvider>
    );
};

server.js (snippet)
const Root = require("../client/containers/Root").default;
const client = createClient();
const store = createStore({
    client
});
const routes = getRoutes();
const history = createMemoryHistory(req.path);

match({ history, routes, location: req.originalUrl }, (err, redirectLocation, routerState) => {
    if (err) return next(err);
    else if (redirectLocation)
        return res.redirect(302, redirectLocation.pathname + redirectLocation.search);
    else if (!routerState)
        return res.status(404).send("Not Found");
    const rootContainer = React.createElement(Root, {
        history,
        routerState,
        routes,
        client,
        store
    });

    return getDataFromTree(rootContainer)
    .then(context => {
        hydrateOnClient({
            state: context.store.getState(),
            markup: renderToString(rootContainer)
        });
    });
});

Just run the server with NODE_ENV=production node server.js
Debug
In my scenario only in production ApolloProvider is being rendered with the property given to the Root component, which makes the application crash.
I reduced the problem to this piece of code.
I added console.log(e) in the empty catch block, and I noticed to following:

in production the try block doesn't throw.
in development the try block throws with the error caught error TypeError: Can't add property props, object is not extensible

If I remove the block my application works again.
So apparently in production Component.props is set to the property of the parent component. This is not correct, because Component is given to the next iteration, but it has the wrong props and context.
So I wonder, why the try catch blog to overwrite Component if it is frozen?
The bug is introduced in version 0.5.5 by https://github.com/apollostack/react-apollo/pull/205.
Workaround
Use<EMAIL_ADDRESS>instead
Version

<EMAIL_ADDRESS><EMAIL_ADDRESS>

I'm also seeing something similar, my code is similar to above but without the router match wrapper.
Non-minified but in production error is reported at https://github.com/apollostack/react-apollo/blob/master/src/server.ts#L59 as:
Invariant Violation: <Subscriber channel="location"> must be rendered in the context of a <Broadcast channel="location">

I have created a PR to show how the problem would be fixed in my case. I would like feedback on this one from @jbaxleyiii as he worked on #205

@rewop, @tmeasday is working on this over the next couple of days!

I assume this is fixed in 0.6.0


Added tests for BaseViewModel class

Closes #39

          1 Warning

:warning:
Branch name should have release/, hotfix/, fix/, housekeep/ or feature/ prefix.

  Generated by :no_entry_sign: Danger


refactor serializable

close #26

Codecov Report

Merging #27 into preview3-parity will increase coverage by 0.02%.
The diff coverage is 96.64%.

@@                 Coverage Diff                 @@
##           preview3-parity      #27      +/-   ##
===================================================
+ Coverage            96.01%   96.04%   +0.02%
===================================================
  Files                   42       42
  Lines                 3592     3637      +45
===================================================
+ Hits                  3449     3493      +44
- Misses                 143      144       +1

Impacted Files
Coverage Δ

neo3/network/protocol.py
64.13% <40.00%> (-1.14%)
:arrow_down:

neo3/network/payloads/consensus.py
89.88% <81.81%> (-0.82%)
:arrow_down:

neo3/contracts/manifest.py
100.00% <100.00%> (ø)

neo3/core/cryptography/bloomfilter.py
100.00% <100.00%> (ø)

neo3/core/serialization.py
100.00% <100.00%> (ø)

neo3/core/types/uint.py
100.00% <100.00%> (ø)

neo3/network/capabilities.py
100.00% <100.00%> (ø)

neo3/network/message.py
100.00% <100.00%> (ø)

neo3/network/node.py
76.72% <100.00%> (ø)

neo3/network/payloads/address.py
100.00% <100.00%> (ø)

... and 10 more

Continue to review full report at Codecov.

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


try upgrading ubuntu

replace ubuntu 20 with ubuntu 24

nope, there isn't an ubuntu 24 runner type provided by github.

nope, there isn't an ubuntu 24 runner type provided by github.

https://github.com/actions/runner-images

Yes, a beta only mid-May https://github.com/actions/runner-images/issues/9691 :-(
You could use a docker image, but I am unsure it is worth the effort for 2 weeks.


Nicer clock in tortoise beacon

Motivation
Inefficient use of the clock in tortoise beacon code.
Changes

Use layer timings to wait for pre-calculated tortoise beacon window start time.
Check for clock timeouts.
Avoid running multiple instances of the beacon code (was a bug).
Use actual waitAfterEpochStart (was a bug).

Test Plan
Existing unit and system tests.
DevOps Notes

[x] This PR does not require configuration changes (e.g., environment variables, GitHub secrets, VM resources)
[x] This PR does not affect public APIs
[x] This PR does not rely on a new version of external services (PoET, elasticsearch, etc.)
[x] This PR does not make changes to log messages (which monitoring infrastructure may rely on)

@nkryuchkov Good comments, will fix.
Also, this currently breaks the multi node sim. I think I know how to fix it, will let you know when to re-review.


mark some slow tests

Description
Some testcase speedups I had sitting on a branch

Processing this pull request

Merged into branches master in fa7244c99d2d0a5ea11fb3a63579125844a69e8a


Big Image

Everytime a user enters Foundry for the first time, the first token he saw a image hover, the image opens on it full size. The image became normal when it hover on another token.

I've been having the same issue. Tried the module by itself and couldn't replicate the problem, so it must be a conflict with something else. Are you using TheRipper's Tidbits module by any chance? I've found that this bug happens to me when I'm hovering over a token before the loading screen appears. In the console, the rendering of the ImageHover is mentioned right before a warning from that module.

I've looked a bit more into this and what I've found is that when the image appears big, it is always followed by the second message in the screenshot.
Foundry VTT | Retrieved and compiled template modules/image-hover/templates/image-hover-template.html
Maybe the bug happens because that first image hover is loaded before the template is compiled, becoming so huge? I don't know a thing about how these things work, just giving some ideas.


hotsos/core: add name aliasing support

at the moment, scenarios are using the long import paths in order to reference to a Python property. this feature allows assigning an alias to a Python class or variable in order to make using plugin property interfaces easier.
Added aliases to the plugins.
Updated existing scenarios to use aliases.
Fixes #912

The aliasing naming seems a bit odd to me. For example, juju.base, kernel.calltrace, 'ceph.daemon.all-osds` and so on give the impression it's a package/class hierarchy when they are just a given name.

Given we could mix with the original way of writing/accessing the properties, the use of dots seems a little misleading.

The dot separation represents a hierarchy, where the first part is the plugin name, then the optional subsections, and finally the property name. I made it this way so if we write a YAML schema(*another important QOL improvement that will make it easier to write scenarios) for the scenarios for IDE/editor assistance in the future, it would be convenient for tab completion.
As for the possible confusion, we can easily distinguish an alias from the full path as the package names always have to start with hotsos. (*unless we start supporting external plugins), so if it starts with "hotsos", it's a full property path, otherwise, it's an alias.

The dot separation represents a hierarchy, where the first part is the plugin name, then the optional subsections, and finally the property name.

Now you say that, it makes sense. But it's not obvious otherwise. Also nothing prevents anyone from giving arbitrary names. So might be useful to have a section ("Alias naming") in documentation on the naming convention.

@dosaboy if it makes sense to land this one just let me know and I'll rebase it


Nested merge vars

Is there a way to send a complex object through merge vars? Mandrill supports any json object to be sent through merge vars. It would be veeeeeeery nice to be able to send something like a list that has many dicts which also have lists, etc.

Djrill uses Python's standard json package to encode the Mandrill API calls, so dict and list merge vars should get converted to json in the usual ways.
What happens when you try it?

The problem was that one of my nested objects wasn't a standard python dict. It was a different kind of dict, so I had to convert the object to json first and then re parse the info so that everything was json serializable. It works. Thanks

By the way, I had to use a custom encoder for json transformation.


if two nodes behind nat without public ip, can they make mesh vpn?

if two nodes behind nat without public ip, can they make mesh vpn?

Currently no.
Technically yes. (with hole-punching algorithm)

for eample, node a and b behind nat, if I have a third node c with public ip and port, and can I use this for a relay,just  transfer for some nat port information ,that is to say under the c's help a b make a connection hole? is it feasible?

If you have the node C, this server can either help punch a hole between A and B. If the hole is established, A and B can communicate directly without further help from C.
However, whether hole punching can succeed is determined by multiple factors (e.g. the configuration of the NAT gateway). If hole punch fails, node C can relay data between A and B.
Currently VxWireguard-Generator only supports relaying. The other mode is possible but I do not have time to implement it.

Random onlooker here.
You’d have to use a STUN server and neither nodes can have a symmetric NAT. At least that is my amateur understanding.
Cool project.

FYI, if you want a full-mesh VPN, you might want to try ZeroTier or Tinc.
Also, I2P is a good choice (better than Tor) if you want an anonymous global-wide mesh VPN.

@innerop
VpnCloud works really well, is fast, supports TUN or TAP and its easy to configure...
https://vpncloud.ddswd.de/features/

Depending on what the OP is trying to accomplish: Althea (routing firmware and mesh privacy architecture) uses wireguard to secure the client to the exit node and a second WG tunnel to secure each hop between routers in a Babel Mesh to create a crypto incentiveised commmunity p2p global scale network.  Its interesting!

@innerop
VpnCloud works really well, is fast, supports TUN or TAP and its easy to configure...
https://vpncloud.ddswd.de/features/

thanks for the reference! nice project!


Permissions Issues Since Updating to Latest

Since updating to latest version, seeing these errors in the actual-http-api logs:
PostError: PostError: unauthorized
at throwIfNot200 (/usr/src/app/node_modules/@actual-app/api/dist/app/bundle.api.js:39687:31)
at postBinary (/usr/src/app/node_modules/@actual-app/api/dist/app/bundle.api.js:39753:17)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async _fullSync (/usr/src/app/node_modules/@actual-app/api/dist/app/bundle.api.js:42602:35)
at async /usr/src/app/node_modules/@actual-app/api/dist/app/bundle.api.js:42503:32 {
type: 'PostError',
reason: 'unauthorized',
meta: undefined
}
PostError: PostError: unauthorized
at throwIfNot200 (/usr/src/app/node_modules/@actual-app/api/dist/app/bundle.api.js:39687:31)
at postBinary (/usr/src/app/node_modules/@actual-app/api/dist/app/bundle.api.js:39753:17)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async _fullSync (/usr/src/app/node_modules/@actual-app/api/dist/app/bundle.api.js:42602:35)
at async /usr/src/app/node_modules/@actual-app/api/dist/app/bundle.api.js:42503:32 {
type: 'PostError',
reason: 'unauthorized',
meta: undefined
}
unhandledRejection Error: We had an unknown problem opening “”.
at handlers.api/sync (/usr/src/app/node_modules/@actual-app/api/dist/app/bundle.api.js:29500:27)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
Actual is on latest as well.

Hello @culpeppers , these errors are expected to happen from time to time (I am not really sure why, I am guessing some file or cache gets stuck or something). If you are running actual-http-api using docker, using the flag --restart=on-failure should take care of restarting the container whenever these failures happen.
Are you using docker?, if so, does this failure keep happening even after the container restarts?
I upgraded to latest (both containers) day before yesterday and it has been working for me. Initially it gave me some errors, I think it's because actual and actual-http-api needs to be upgraded at the same time, but after 1 min and 1 restart it started working again.

I'm using it on Unraid under Docker. Would I add that as an env variable, or as something else?
Thanks!

Reading few posts looks like there are extra parameters (or advanced options), where you can add the --restart=on-failure flag. Let me know if it works.

https://www.reddit.com/r/unRAID/comments/10wvn9b/way_to_automatically_restart_a_stopped_container/
https://forums.unraid.net/topic/55582-run-containers-with-restart-unless-stopped/

Sorry for the delay. Thank you for looking into that for me. Adding that did the trick, thank you!!


Internal air blocks visualization

When viewing the interior of a ship, through the model viewer, the void spaces are not obvious. It would help to highlight them or color these spaces so that it is clear what the "interior" of the ship is.

Implemented in ab368d468255e6cec2c5a251a4b6a4db4552720a


feat: support external ca certificate import when creating a cluster

Signed-off-by: zhuzhenfan<EMAIL_ADDRESS>
What type of PR is this?
/kind feature

What this PR does / why we need it:
support external ca certificate import when creating a cluster
Which issue(s) this PR fixes:

Fixes #377
Special notes for reviewers:
kcctl create cluster --ca-cert ca.crt --ca-key ca.key

Does this PR introduced a user-facing change?

None

Additional documentation, usage docs, etc.:

@x893675 @lixd

/cc @lixd @qinyer

/approve

/cherrypick master


Add reject flow to collab close offer

We shouldn't automatically accept any collab close offer, equally I don't think it reasonable to automatically force close those dlc channels either.
I propose we implement a flow to reject a collab close offer.

Proposal: implement a simple if check which does not automatically accept a collab close offer. In a later phase we will then implement the reject flow.

Proposal: implement a simple if check which does not automatically accept a collab close offer. In a later phase we will then implement the reject flow.

see https://github.com/get10101/10101/issues/2156


Add pagination when listing immutableTagRules

immutableTagRule ressource didn't handle pagination. This commit fixes it.

hi @gaglimax
thank you !


Solid harm prefact

The inner product obtaining our expansion coefficients erroneously contained the solid harmonic prefactor. I added code to divide through by the prefactor. This causes the expansion coefficients to actually converge to the underlying atomic gaussian, which is necessary.
I currently cleaning up an example notebook that shows that the expansion of an isotropic gaussian does indeed converge.
However, an anisotropic gaussian does not yet converge... I have some suspicions.

hey, i had a rather quick look, but was the purpose here to modify the density expansion coefficients? I am afraid that adding the additional 1/sqrt{2l+1} affects how higher order features are constructed down the line

hey, i had a rather quick look, but was the purpose here to modify the density expansion coefficients? I am afraid that adding the additional 1/sqrt{2l+1} affects how higher order features are constructed down the line could you perhaps give me an overview of how the density expansion is now normalized?

Hi Jigyasa,
I made these changes under the assumption that a linear combination of expansion coefficients and basis functions should recreate the underlying gaussian. I identified that this 1/sqrt(2l+1) factor was erroneously included and needed to be divided out in order to recreate the underlying gaussian.

To reiterate one more time, I'm trying to unify equations 15 and 16 within the AniSOAP paper, but right below equation 16, we make an incorrect statement saying that

with $R_l^m(r) = r^l * Y_l^m(\hat{r})$

This is incorrect, because actually $R_l^m(r) = \sqrt{\frac{4\pi}{2l+1}}r^l * Y_l^m(\hat{r})$. Hence I need to divide out this factor, because in the code, we are calculating $R_l^m$, but we actually want $r^l * Y_l^m(\hat{r})$ (eqn 15 in the paper)

okay, thank you! i was a little unsure of the prefactor being there in the definition of real-valued spherical harmonics but all good if this makes the anisoap and isotropic gaussian expansion consistent. (incidentally, this made me realize there's a typo in Eq 17, l' must be equal to l)

That’s fine, only a suggestion

On Apr 30, 2024, at 6:10 PM, arthur-lin1027 @.***> wrote:
@arthur-lin1027 commented on this pull request.
In anisoap/representations/ellipsoidal_density_projection.py https://github.com/cersonsky-lab/AniSOAP/pull/38#discussion_r1585643426:

solid_harm_prefact = np.sqrt((4 * np.pi) / (np.arange(lmax + 1) * 2 + 1))

scaled_sph_to_cart = []
for l in range(lmax + 1):

   scaled_sph_to_cart.append(sph_to_cart[l] / solid_harm_prefact[l])

cannot actually make this change, because sph_to_cart is a list of 5-dimensional arrays that change depending on l. np.divide cannot understand how to unify the dimensions so I just use a for loop, plus it's more explicit here.
—
Reply to this email directly, view it on GitHub https://github.com/cersonsky-lab/AniSOAP/pull/38#discussion_r1585643426, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALKVP3WED2WUGY6YX6IK7LLZAAQGBAVCNFSM6AAAAABGOL3ILSVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAMZSG43DSNJRHE.
You are receiving this because your review was requested.


Gravity skewed?

Recently unable to place anything in the right direction.
Even adjusting things with any form of screwdriver is not solving it.
No idea what caused it, just incredibly annoying.

Please include specific steps to reproduce, such as:

`/tp xxx yyy zzz'
Attempt to place a item_string at x, y, z.
Attempt to use screw driver to rotate it such that param2 = n.

Document what happened, vs what you expected to happen.

We met in-game, and player Staff11 gave a demo.  Any item that they place seems to start w/ param2 = 18, so its on its side.  Chests, armor stands, etc...
Player logged out, then back in as a brand new character, and problem did not exhibit.

There is nothing weird in they raw player data:
~$ ./bin/sqlite3 -line ./worlds/world/players.sqlite "select * from player where name in ('Staff11', '2hit');"
             name = 2hit
            pitch = 25.7999992370605
              yaw =<PHONE_NUMBER>41406
             posX =<PHONE_NUMBER>5
             posY = 45
             posZ =<PHONE_NUMBER>75
               hp = 20
           breath = 10
    creation_date = 2022-05-23 20:56:53
modification_date = 2022-05-23 21:02:30

             name = Staff11
            pitch = 13.0600004196167
              yaw =<PHONE_NUMBER>82813
             posX =<PHONE_NUMBER>25
             posY = 15
             posZ =<PHONE_NUMBER>75
               hp = 12
           breath = 10
    creation_date = 2022-02-05 15:59:52
modification_date = 2022-05-23 20:55:47

Further experiments suggest that it is related to the "Rhotator" mod.
https://github.com/entuland/rhotator

Staff11 dropped their entire inventory, and relogged.
Problem still present.
We restarted the server (/shutdown).
Problem appears to be gone (nodes place normally now).
Staff11 re-equiped a rhotator, used it, then relogged.
Problem re-appeared.
Staff11 again emptied their inventory.
Server was restarted yet again.
Problem persists (nodes are placed w/ weird param2 values).

So the repro is inconsistent.

Hulpful forum post?
https://forum.minetest.net/viewtopic.php?t=23234

Hypothesis: The rhotator is storing per-player rhotator configs in mod-storage.  The setting is applied when the player places at item, even if the rhotator is no longer in their inventory.

The rhotator tool was designed this way.  WAI.


[W5.11][T09-A2]Goh Zu Wei

Added a NameSortCommand and a EmailSortCommand that sorts persons' name and email in an alphabetical order respectively.
Added the list functionality to AddCommand and DeleteCommand.

good job!


The pluign falsely detects invalid links (the links work)

Describe the bug
For my docs page, the link validation fails, even though all links are functional.
I am using the following base URL /obsidian-meta-bind-plugin-docs.
Links look like this [link](/obsidian-meta-bind-plugin-docs/guides/...)
Those are the error I am getting:

To Reproduce
I am not quite sure why this happens, but I believe it has to do with the base URL.
Expected behavior
The link checker does not detect any broken links.
How often does this bug happen?
Every time
System Info
"@astrojs/starlight": "^0.15.0",
"astro": "^4.0.3",
"starlight-links-validator": "^0.5.0"

Additional Context
No response

Thanks for your report :raised_hands:
This should now be fixed in v0.5.1.
Before publishing the new version, I tested locally on your documentation website, and indeed, you were right, the links were valid as expected:

I think this still happens for static files. For example, the file public/document.pdf with a base: '/obsidian-meta-bind-plugin-docs/' would get the link /obsidian-meta-bind-plugin-docs/document.pdf.

I think this still happens for static files. For example, the file public/document.pdf with a base: '/obsidian-meta-bind-plugin-docs/' would get the link /obsidian-meta-bind-plugin-docs/document.pdf.

Thanks for letting me know @JuanM04. This should now be fixed in v0.5.2.


Object map of {type} vs Array of {type}

It is quite easy to document Array properties for which all items are of the same type
ex:
{
    /**
     * @property {Movie[]} movies
     */
    movies: [grease, grease2, dirtyDancing]
}

But what if I need to document a property which is instead a map with identical value type
Could we use such notation?
{
    /**
     * @property {Keyword{}} keywords
     */
    keywords: {
        "module": {hits:42, lastHit:<PHONE_NUMBER>888, rating: 4.5},
        "plugin": {hits:51, lastHit:<PHONE_NUMBER>888, rating: 3.8},
        "extension": {hits:51, lastHit:<PHONE_NUMBER>888, rating: 3.8}
    }
}

I found this notation
/**
 * @type Object<number,string>
 */
var obj = {};
obj[1] = 'bar';

https://google.github.io/styleguide/javascriptguide.xml#JsTypes
Is it supported ? (or planned to be?)

For your first example, you can document the type as Object<number, Keyword>. For the second example, yes, Object<number, string> is correct.


[ci] Specify permissions for tvm bot

This adjusts some error reporting for tvm-bot and manually specifies the permissions it should run with to hopefully alleviate the 403 errors when merging PRs
cc @Mousius @areusch

will let you merge


Selecting semi-hard examples, possible bug?

Hello,
In the tripletselect layer, we aim to choose semi-hard negative examples. meaning, negative examples that are within a margin alpha comparing to the positive example.
In your custom layer, you sort both a_p and a_n examples so that the highest distance is first.
Don't we want the highest a_p distance and the smallest a_n distance so they might violate the semi-hard definition?
The following code with my suggested fix:
`        archor_feature = bottom[0].data[0]
    for i in range(self.triplet):

        positive_feature = bottom[0].data[i+self.triplet]
        a_p = archor_feature - positive_feature
        ap = np.dot(a_p,a_p)
        aps[i+self.triplet] = ap
        aps = sorted(aps.items(), key = lambda d: d[1], reverse = True)

    for i in range(self.triplet):

        negative_feature = bottom[0].data[i+self.triplet*2]
        a_n = archor_feature - negative_feature
        an = np.dot(a_n,a_n)
        ans[i+self.triplet*2] = an
    #ans = sorted(ans.items(), key = lambda d: d[1], reverse = True)  # guyn - seems to me like a bug, we want the lowest distance to be first
    ans = sorted(ans.items(), key = lambda d: d[1], reverse = False)`

What do you think?
Guy

I think so. Your mentioned strategy is chosen by the CVPR15 FaceNet paper. According to that paper, we should find the 'hard-positive' (the largest-distance positive) and 'hard-negative' (the smallest-distance negative). But I think maybe the author choose another strategy?


GEOMESA-795 Write attribute histogram results to accumulo "_stats" table

Attribute histogram job results can now be written back to accumulo in a {feature-name}_stats table by setting the "geomesa.hist.write.to.accumulo" to true.

I think this can be merged, maybe that one thing I noted above should be addressed first however.


[servicebus] Best way to determine if topic/subscription exists

This is a request to have a documented way to determine whether a topic/subscription exists or not.
I've been trying to work out the best way to determine if a service bus topic/subscription exists or not. I might be missing something, but I was hoping to be able to call GetTopic/GetSubscription on the ServiceBus admin client and infer the topic/subscription existence from error returned. i.e.
package main

import (
	"context"

	"github.com/Azure/azure-sdk-for-go/sdk/azidentity"
	"github.com/Azure/azure-sdk-for-go/sdk/messaging/azservicebus/admin"
)

func main() {
        credential, err := azidentity.NewDefaultAzureCredential(nil)
	if err != nil {
		panic(err)
	}

	adminClient, err = admin.NewClient("<ex: myservicebus.servicebus.windows.net>", credential, nil)
	if err != nil {
		panic(err)
	}

	resp, err := adminClient.GetSubscription(context.Background(), "topicName", "subName", &admin.GetSubscriptionOptions{})
	if err != nil {
		panic(err)
	}
	_ = resp
}

At the moment, GetTopic/GetSubscription returns a 200 http response error so it is not possible to tell whether the topic/subscription exists or not.  In github.com/Azure/azure-service-bus-go it was possible to use IsErrNotFound to determine a topic/subscriptions existence from topic/subscription manager. I.e.
package main

import (
	"context"

	servicebus "github.com/Azure/azure-service-bus-go"
)

func main() {
	namespaceName := "mynamespace"
	namespace, err := servicebus.NewNamespace(servicebus.NamespaceWithEnvironmentBinding(namespaceName))
	if err != nil {
		panic(err)
	}

	tm := namespace.NewTopicManager()

	_, err = tm.Get(context.Background(), "topicName")
	if err != nil {
		if servicebus.IsErrNotFound(err) {
			// topic exists!
		}
		// something went wrong
	}
}

The only way I have currently found to do this is to list the subscriptions and match based on the subscription name. Would be nice to get some clarity on this.

I'll take a look. The normal expectation is that the service would return a 404 if the asked-for-entity does not exist. I'll look at what's going on there.

Thanks, I traced one of the requests through for GetSubscription and I can see that a 200 response is returned from the http request.  It suprised me that a 404 was not being returned.

Hi @bvwells, just wanted you to know I haven't ghosted you on this, we're still trying to understand what the right way is to expose this in the SDK as it's a bit of an unusual API. Will update this issue when I've got a conclusion/patch.

Thanks for the update @richardpark-msft. There is no rush with this one as I have a workaround for the issue by listing topics and subscriptions to see whether they exist or not.

Thanks for your patience on that - this should be a bit more efficient.


in building JMeter source  meet  EXCEPTION_STACK_OVERFLOW (Execution failed for task ':src:dist:runGui'.）)

Steps to reproduce the problem
the jmeter source get by git，the jmeter version is lastest.
when execute  ./gradlew.bat runGui  meet EXCEPTION_STACK_OVERFLOW .
how to fix the problem? i modify DEFAULT_JVM_OPTS in gradlew.bat ,but it not work. what the jvm_args should set to?
Getting Build Failed with exception while building using ./gradlew.bat runGui：
PS D:\xxx\jmeter> ./gradlew.bat runGui
Type-safe project accessors is an incubating feature.

Configure project :
Building JMeter 6.0.0-SNAPSHOT

Configure project :src:components
RenderInBrowser is excluded from compilation. If you want to compile it, add -PenableJavaFx

Configure project :src:dist-check
Certain tests will be skipped as they depend on external services and fail too often. Please add -PenableFlaky to enable them: [batchHttp4ImplDigestAuth, batchHttp4ImplPreemptiveBasicAuthJava, batchHttp4ImplPreemptiveBasicAuth, batc
hSlowCharsFeatureHttpClient4, batchSlowCharsFeatureJava, batchTCP_TESTS, batchTestKeepAlive, batchTestRedirectionPolicies, batchJDBC_TESTS, batchResponseDecompression]

Task :src:dist:runGui
WARN StatusConsoleListener The use of package scanning to locate plugins is deprecated and will be removed in a future release
WARN StatusConsoleListener The use of package scanning to locate plugins is deprecated and will be removed in a future release
WARN StatusConsoleListener The use of package scanning to locate plugins is deprecated and will be removed in a future release
WARN StatusConsoleListener The use of package scanning to locate plugins is deprecated and will be removed in a future release
================================================================================
Don't use GUI mode for load testing !, only for Test creation and Test debugging.
For load testing, use CLI Mode (was NON GUI):
jmeter -n -t [jmx file] -l [results file] -e -o [Path to web report folder]
& increase Java Heap to meet your test requirements:
Modify current env variable HEAP="-Xms1g -Xmx1g -XX:MaxMetaspaceSize=256m" in the jmeter batch file
Check : https://jmeter.apache.org/usermanual/best-practices.html
================================================================================
An unrecoverable stack overflow has occurred.

A fatal error has been detected by the Java Runtime Environment:
EXCEPTION_STACK_OVERFLOW (0xc00000fd) at pc=0x00007ff89b91c5b7, pid=17424, tid=15452
JRE version: Java(TM) SE Runtime Environment (17.0.10+11) (build 17.0.10+11-LTS-240)
Java VM: Java HotSpot(TM) 64-Bit Server VM (17.0.10+11-LTS-240, mixed mode, sharing, tiered, compressed oops, compressed class ptrs, g1 gc, windows-amd64)
Problematic frame:
C  [awt.dll+0xec5b7]
No core dump will be written. Minidumps are not enabled by default on client versions of Windows
An error report file with more information is saved as:
D:\2024\source\jmeter\bin\hs_err_pid17424.log
If you would like to submit a bug report, please visit:
https://bugreport.java.com/bugreport/crash.jsp
The crash happened outside the Java Virtual Machine in native code.
See problematic frame for where to report the bug.

Task :src:dist:runGui FAILED
checksum-dependency elapsed time: 2340ms, configurations processed: 69 (add -PchecksumTimingsPrint to print detailed timings)
checksum-dependency elapsed time: 2623ms, configurations processed: 219 (add -PchecksumTimingsPrint to print detailed timings)

FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':src:dist:runGui'.

Process 'command 'D:\ly\sw\jdk-17.0.10\bin\java.exe'' finished with non-zero exit value 1

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Get more help at https://help.gradle.org.

Deprecated Gradle features were used in this build, making it incompatible with Gradle 9.0.
You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins.
For more on this, please refer to https://docs.gradle.org/8.5/userguide/command_line_interface.html#sec:command_line_warnings in the Gradle documentation.
BUILD FAILED in 13s
161 actionable tasks: 18 executed, 143 up-to-date
JMeter Version
JMeter 6.0.0-SNAPSHOT
Java Version
17
kotlin Version
1.9.20
gradle Version
8.5
OS Version
windows

hs_err_pid17424.log

Have you tried increasing -Xss here: https://github.com/apache/jmeter/blob/09c3f810dda7efaf1510ac062454d978d90a0741/src/dist/build.gradle.kts#L643 ?
It says there was "free space=243k" in stack, however, it is hard to tell what is wrong here.
Have you tried Java 21 or a Java from a different vendor?

i change the jdk to Java 21(21.0.2), i meet the problem as below, It seems that I can only use java 17 ? :
10:17:08: Executing 'runGui'...
Starting Gradle Daemon...
Gradle Daemon started in 2 s 495 ms
Type-safe project accessors is an incubating feature.
FAILURE: Build completed with 2 failures.
1: Task failed with an exception.

What went wrong:
A problem occurred configuring project ':build-logic-commons:gradle-plugin'.

Failed to calculate the value of task ':build-logic-commons:gradle-plugin:compileJava' property 'javaCompiler'.
No matching toolchains found for requested specification: {languageVersion=17, vendor=any, implementation=vendor-specific} for WINDOWS on x86_64.
No locally installed toolchains match and toolchain download repositories have not been configured.

Try:

Learn more about toolchain auto-detection at https://docs.gradle.org/8.5/userguide/toolchains.html#sec:auto_detection.
Learn more about toolchain repositories at https://docs.gradle.org/8.5/userguide/toolchains.html#sub:download_repositories.
Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Get more help at https://help.gradle.org.
==============================================================================

2: Task failed with an exception.

What went wrong:
Failed to query the value of property 'buildFlowServiceProperty'.

Could not isolate value org.jetbrains.kotlin.gradle.plugin.statistics.BuildFlowService$Parameters_Decorated@65b3a32e of type BuildFlowService.Parameters
A problem occurred configuring project ':build-logic-commons:gradle-plugin'.
Failed to calculate the value of task ':build-logic-commons:gradle-plugin:compileJava' property 'javaCompiler'.
No matching toolchains found for requested specification: {languageVersion=17, vendor=any, implementation=vendor-specific} for WINDOWS on x86_64.
No locally installed toolchains match and toolchain download repositories have not been configured.

Try:

Learn more about toolchain auto-detection at https://docs.gradle.org/8.5/userguide/toolchains.html#sec:auto_detection.
Learn more about toolchain repositories at https://docs.gradle.org/8.5/userguide/toolchains.html#sub:download_repositories.
Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Get more help at https://help.gradle.org.
==============================================================================

BUILD FAILED in 28s
10:17:40: Execution finished 'runGui'.
the java version as below:
C:\Users\Administrator>java -version
java version "21.0.2" 2024-01-16 LTS
Java(TM) SE Runtime Environment (build 21.0.2+13-LTS-58)
Java HotSpot(TM) 64-Bit Server VM (build 21.0.2+13-LTS-58, mixed mode, sharing)

hs_err_pid20116.log

I forgot that Java version should be configured with parameters (see ./gradlew parameters)
For instance:
./gradlew runGui -PjdkTestVersion=21

I forgot that Java version should be configured with parameters (see ./gradlew parameters)
For instance:
./gradlew runGui -PjdkTestVersion=21

Thank you for answering my question.

in java 17, i set the stack size from 256k to 1m is ok,  and after execute ./gradlew.bat runGui  , the jmeter can open.

in java 21 , execute ./gradlew.bat runGui -PjdkTestVersion=21, hava error  as below:
D:\xxx\jmeter> ./gradlew.bat runGui -PjdkTestVersion=21
Type-safe project accessors is an incubating feature.
FAILURE: Build completed with 2 failures.
1: Task failed with an exception.

What went wrong:
A problem occurred configuring project ':build-logic-commons:gradle-plugin'.

Failed to calculate the value of task ':build-logic-commons:gradle-plugin:compileJava' property 'javaCompiler'.
No matching toolchains found for requested specification: {languageVersion=17, vendor=any, implementation=vendor-specific} for WINDOWS on x86_64.
No locally installed toolchains match and toolchain download repositories have not been configured.

Try:

Learn more about toolchain auto-detection at https://docs.gradle.org/8.5/userguide/toolchains.html#sec:auto_detection.
Learn more about toolchain repositories at https://docs.gradle.org/8.5/userguide/toolchains.html#sub:download_repositories.
Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Get more help at https://help.gradle.org.
==============================================================================

2: Task failed with an exception.

What went wrong:
Failed to query the value of property 'buildFlowServiceProperty'.

Could not isolate value org.jetbrains.kotlin.gradle.plugin.statistics.BuildFlowService$Parameters_Decorated@2c02cbc8 of type BuildFlowService.Parameters
A problem occurred configuring project ':build-logic-commons:gradle-plugin'.
Failed to calculate the value of task ':build-logic-commons:gradle-plugin:compileJava' property 'javaCompiler'.
No matching toolchains found for requested specification: {languageVersion=17, vendor=any, implementation=vendor-specific} for WINDOWS on x86_64.
No locally installed toolchains match and toolchain download repositories have not been configured.

Try:

Learn more about toolchain auto-detection at https://docs.gradle.org/8.5/userguide/toolchains.html#sec:auto_detection.
Learn more about toolchain repositories at https://docs.gradle.org/8.5/userguide/toolchains.html#sub:download_repositories.
Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Get more help at https://help.gradle.org.
==============================================================================

BUILD FAILED in 2s

i hava another question :  jmeter freeze when open a jmx with a Big request param.
in the jmx, i have a httpsample , and it's  request param is a big json ,about 600k, when load the file ,the jmeter hangs up.
how to solve the problem.
I don't want to use CSV to deliver the request param.
jmx_slow.zip

{languageVersion=17, vendor=any, implementation=vendor-specific} for WINDOWS on x86_64.
No locally installed toolchains match and toolchain download repositories have not been configured.

That means Gradle can't detect Java 17 on your machine.
By default JMeter uses Java 17 for the build, and you could use ./gradlew -PjdkBuildVersion=21 -PjdkTestVersion=21 runGui to use 21 for both build and test.

in jdk 21, i try ./gradlew.bat -PjdkBuildVersion=21 -PjdkTestVersion=21 runGui , but some error happen as front:

when use ./gradlew.bat runGui , how to pass a jmx file path?

"jmeter freeze" is caused by https://github.com/bobbylight/RSyntaxTextArea/issues/41:
A workaround is to disable line wrapping (see jsyntaxtextarea.linewrap in jmeter.properties)
"AWT-EventQueue-0" #36 [67591] prio=6 os_prio=31 cpu=155639,07ms elapsed=209,99s tid=0x000000013be47c00 nid=67591 runnable  [0x000000016ecdd000]
   java.lang.Thread.State: RUNNABLE
	at org.fife.ui.rsyntaxtextarea.TokenImpl.getWidthUpTo(TokenImpl.java:601)
	at org.fife.ui.rsyntaxtextarea.TokenUtils.getSubTokenList(TokenUtils.java:132)
	at org.fife.ui.rsyntaxtextarea.WrappedSyntaxView$WrappedLine.calculateLineCount(WrappedSyntaxView.java:1188)
	at org.fife.ui.rsyntaxtextarea.WrappedSyntaxView$WrappedLine.handleDocumentEvent(WrappedSyntaxView.java:1439)
	at org.fife.ui.rsyntaxtextarea.WrappedSyntaxView$WrappedLine.insertUpdate(WrappedSyntaxView.java:1463)
	at org.fife.ui.rsyntaxtextarea.WrappedSyntaxView.insertUpdate(WrappedSyntaxView.java:706)
	at javax.swing.plaf.basic.BasicTextUI$RootView.insertUpdate(java.desktop@21.0.2/BasicTextUI.java:1711)
	at javax.swing.plaf.basic.BasicTextUI$UpdateHandler.insertUpdate(java.desktop@21.0.2/BasicTextUI.java:1978)
	at javax.swing.text.AbstractDocument.fireInsertUpdate(java.desktop@21.0.2/AbstractDocument.java:227)
	at org.fife.ui.rsyntaxtextarea.RSyntaxDocument.fireInsertUpdate(RSyntaxDocument.java:187)
	at javax.swing.text.AbstractDocument.handleInsertString(java.desktop@21.0.2/AbstractDocument.java:781)
	at javax.swing.text.AbstractDocument.insertString(java.desktop@21.0.2/AbstractDocument.java:740)
	at javax.swing.text.PlainDocument.insertString(java.desktop@21.0.2/PlainDocument.java:131)
	at javax.swing.text.AbstractDocument.replace(java.desktop@21.0.2/AbstractDocument.java:699)
	at javax.swing.text.JTextComponent.setText(java.desktop@21.0.2/JTextComponent.java:1725)
	at org.apache.jmeter.gui.util.JSyntaxTextArea.setInitialText(JSyntaxTextArea.java:305)
	at org.apache.jmeter.protocol.http.config.gui.UrlConfigGui.configure(UrlConfigGui.java:291)
	at org.apache.jmeter.protocol.http.control.gui.HttpTestSampleGui.configure(HttpTestSampleGui.java:134)
	at org.apache.jmeter.gui.tree.JMeterTreeModel.addComponent(JMeterTreeModel.java:162)
	at org.apache.jmeter.gui.tree.JMeterTreeModel.addSubTree(JMeterTreeModel.java:131)
	at org.apache.jmeter.gui.tree.JMeterTreeModel.addSubTree(JMeterTreeModel.java:131)
	at org.apache.jmeter.gui.tree.JMeterTreeModel.addSubTree(JMeterTreeModel.java:123)
	at org.apache.jmeter.gui.GuiPackage.addSubTree(GuiPackage.java:555)
	at org.apache.jmeter.gui.action.Load.insertLoadedTree(Load.java:201)
	at org.apache.jmeter.gui.action.Load.loadProjectFile(Load.java:132)
	at org.apache.jmeter.gui.action.Load.loadProjectFile(Load.java:103)
	at org.apache.jmeter.gui.action.Load.doActionAfterCheck(Load.java:90)
	at org.apache.jmeter.gui.action.AbstractActionWithNoRunningTest.doAction(AbstractActionWithNoRunningTest.java:44)
	at org.apache.jmeter.gui.action.ActionRouter.performAction(ActionRouter.java:83)
	at org.apache.jmeter.gui.action.ActionRouter.lambda$actionPerformed$0(ActionRouter.java:65)
	at org.apache.jmeter.gui.action.ActionRouter$$Lambda/0x000000e0015201f8.run(Unknown Source)
	at java.awt.event.InvocationEvent.dispatch(java.desktop@21.0.2/InvocationEvent.java:318)

"jmeter freeze" is caused by https://github.com/bobbylight/RSyntaxTextArea/issues/41:
A workaround is to disable line wrapping (see jsyntaxtextarea.linewrap in jmeter.properties)
"AWT-EventQueue-0" #36 [67591] prio=6 os_prio=31 cpu=155639,07ms elapsed=209,99s tid=0x000000013be47c00 nid=67591 runnable  [0x000000016ecdd000]
   java.lang.Thread.State: RUNNABLE
	at org.fife.ui.rsyntaxtextarea.TokenImpl.getWidthUpTo(TokenImpl.java:601)
	at org.fife.ui.rsyntaxtextarea.TokenUtils.getSubTokenList(TokenUtils.java:132)
	at org.fife.ui.rsyntaxtextarea.WrappedSyntaxView$WrappedLine.calculateLineCount(WrappedSyntaxView.java:1188)
	at org.fife.ui.rsyntaxtextarea.WrappedSyntaxView$WrappedLine.handleDocumentEvent(WrappedSyntaxView.java:1439)
	at org.fife.ui.rsyntaxtextarea.WrappedSyntaxView$WrappedLine.insertUpdate(WrappedSyntaxView.java:1463)
	at org.fife.ui.rsyntaxtextarea.WrappedSyntaxView.insertUpdate(WrappedSyntaxView.java:706)
	at javax.swing.plaf.basic.BasicTextUI$RootView.insertUpdate(java.desktop@21.0.2/BasicTextUI.java:1711)
	at javax.swing.plaf.basic.BasicTextUI$UpdateHandler.insertUpdate(java.desktop@21.0.2/BasicTextUI.java:1978)
	at javax.swing.text.AbstractDocument.fireInsertUpdate(java.desktop@21.0.2/AbstractDocument.java:227)
	at org.fife.ui.rsyntaxtextarea.RSyntaxDocument.fireInsertUpdate(RSyntaxDocument.java:187)
	at javax.swing.text.AbstractDocument.handleInsertString(java.desktop@21.0.2/AbstractDocument.java:781)
	at javax.swing.text.AbstractDocument.insertString(java.desktop@21.0.2/AbstractDocument.java:740)
	at javax.swing.text.PlainDocument.insertString(java.desktop@21.0.2/PlainDocument.java:131)
	at javax.swing.text.AbstractDocument.replace(java.desktop@21.0.2/AbstractDocument.java:699)
	at javax.swing.text.JTextComponent.setText(java.desktop@21.0.2/JTextComponent.java:1725)
	at org.apache.jmeter.gui.util.JSyntaxTextArea.setInitialText(JSyntaxTextArea.java:305)
	at org.apache.jmeter.protocol.http.config.gui.UrlConfigGui.configure(UrlConfigGui.java:291)
	at org.apache.jmeter.protocol.http.control.gui.HttpTestSampleGui.configure(HttpTestSampleGui.java:134)
	at org.apache.jmeter.gui.tree.JMeterTreeModel.addComponent(JMeterTreeModel.java:162)
	at org.apache.jmeter.gui.tree.JMeterTreeModel.addSubTree(JMeterTreeModel.java:131)
	at org.apache.jmeter.gui.tree.JMeterTreeModel.addSubTree(JMeterTreeModel.java:131)
	at org.apache.jmeter.gui.tree.JMeterTreeModel.addSubTree(JMeterTreeModel.java:123)
	at org.apache.jmeter.gui.GuiPackage.addSubTree(GuiPackage.java:555)
	at org.apache.jmeter.gui.action.Load.insertLoadedTree(Load.java:201)
	at org.apache.jmeter.gui.action.Load.loadProjectFile(Load.java:132)
	at org.apache.jmeter.gui.action.Load.loadProjectFile(Load.java:103)
	at org.apache.jmeter.gui.action.Load.doActionAfterCheck(Load.java:90)
	at org.apache.jmeter.gui.action.AbstractActionWithNoRunningTest.doAction(AbstractActionWithNoRunningTest.java:44)
	at org.apache.jmeter.gui.action.ActionRouter.performAction(ActionRouter.java:83)
	at org.apache.jmeter.gui.action.ActionRouter.lambda$actionPerformed$0(ActionRouter.java:65)
	at org.apache.jmeter.gui.action.ActionRouter$$Lambda/0x000000e0015201f8.run(Unknown Source)
	at java.awt.event.InvocationEvent.dispatch(java.desktop@21.0.2/InvocationEvent.java:318)

in jdk 21, i try ./gradlew.bat -PjdkBuildVersion=21 -PjdkTestVersion=21 runGui , but some error happen as front:
https://github.com/apache/jmeter/issues/6262#issuecomment-2029438219

It looks like the build requires Java 17 anyway, so you need to have both Java 17 and 21.
See https://docs.gradle.org/current/userguide/toolchains.html regarding the automatic toolchain search and download.

https://github.com/apache/jmeter/issues/6262#issuecomment-2029446617
i primary want to debug Jmeter source with a jmx file, how to dubug in GUI mode with a given jmx file?
in org.apache.jmeter.JMeterTest, i see many runNonGui test case , is there any runGui test case ?

how to dubug in GUI mode with a given jmx file?

Launch GUI in debug mode, in GUI click "file -> open" and so on.

thank you very much , i think There are probably no more questions about this issue