quickcheck-based testing

Hello :)
I'm excited to see automerge coming to the Rust ecosystem!
One of the most effective time-saving engineering techniques I used while writing the first version of https://github.com/rust-crdt/rust-crdt (and most of my projects since then) was to use quickcheck to generate randomized operations, then permute that sequence of operations, then assert that all permutations of applications converged to an identical final state after all operations were applied. You can also clone them to assert idempotency. This is orders of magnitude more effective than most example/table-driven testing. Personally, I hate writing tests, but I take a huge amount of satisfaction when the machine spits back a failing example that illuminates a bias I encoded in the system.
The really nice thing is that quickcheck will then shrink the failing set of operations down to the minimum set that still causes the bug to jump out, effectively telling a story for you to more easily debug what happened compared to staring at a chain of 1000 random operations and not knowing what could have gone wrong. It basically is a way to write like 50 lines of test code that then generates millions of interesting tests that keep mining for bugs in your implementation, avoiding the "pesticide paradox" where your code just becomes immune to your static tests.
I initially just cloned this repo to implement something similar, but I realized that some of the functionality is a bit scattered across various sub-crates, which adds friction to testing.
I see that Backend is doing a bit of the merging work, scattering the logic of the CRDT across the protocol and backend subcrates. The reason why this adds friction is because it is quite convenient to conditionally derive the Arbitrary trait which allows random instances of a type to be synthesized during testing, but if you want to do this for multiple crates in a way that is only done at testing-time, you have to start adding compile-time features (the #[cfg(test]) attribute is not transitive to dependencies during testing).
So, there are 2 options to still accomplish this, if you are interested:

introduce a testing conditional compilation feature that then triggers the Arbitrary trait to be derived for various types
merge backend and protocol, or at least push the least upper bound function for the automerge CRDT into protocol

would you be interested in either of these, to facilitate making automerge-rs far more reliable through the use of generative testing?

oh yeah, this also unlocks using hit-tracing fuzzers to find even more bugs in combination with quickcheck https://github.com/rust-fuzz/arbitrary

Hey! Great to have you here, I've been following rust-crdt for a while, I really like what you've done there.
Vis a vis model based tests. I've had a crack at building them for automerge before, in the javascript implementation here. I ran into two problems:

I found it very difficult to generate a useful Arbitrary instance for automerge operations directly. The problem is that an operation is not valid in it's own right, it depends on previous operations. For example, an operation setting the key of an object to some value needs to know the operation ID of the operation which created the parent object. I ended up instead working in a language of operations on paths into the state of the CRDT. This worked but it was quite awkward
Even once I had gotten the above test working, I couldn't get it to explore any interesting parts of the state space. Partly this was to do with performance being a problem, which meant that I couldn't generate a lot of examples, but it's also that the interesting parts of the algorithm are quite unlikely to occur in entirely random mutations. I ended up in situations where my unit tests were surfacing bugs (usually to do with many concurrent list insertions) which I couldn't get the model based test to reproduce.

Now, I am in general very enthusiastic about model based testing and it seems a shame to me that a project with such an obvious property to test as a CRDT doesn't have a model based test so I would be very interested in any experience you have with model based testing that you can bring to bear. On that basis, let's explore what changes we might make to make model based testing possible.
To your point about logic being scattered across several crates. I assume that you're referring to the backend/frontend split? Could you explain to me how that is a problem for property based tests? The main property I had been thinking of testing was something along the lines of "if two instances of the CRDT have applied the same set of operations, then they have the same state". The backend contains the entire state of the CRDT so it should be possible to apply operations to two different instances of the backend in arbitrary orders and then assert that they have the same state right?
There are additional properties that we would want to test to do with the way that the frontend and backend communicate, for example "the backend always emits patches to the frontend in the order that change requests were received from the frontend".
I guess to my mind the biggest problem I have with model based testing here is that I can't figure out how to generate an interesting, arbitrary set of operations. But yeah, would welcome your thoughts on all of the above.


style: style adjusting of gap

PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.


rest api

For bug reporting only! If you're posting a feature request or discussion, please ignore.
Expected Behavior
When following the documented method for querying customer information in a shop, I am expecting to get a single result back.
$req = $shop->api()->rest('GET','/admin/api/customers/search.json', ['query' => 'email:test@email.com'])['body'];
Current Behavior
When running the above query, it is as if the query is not actually run and instead all results are returned. I have confirmed a single customer exists with that email. Yet it returns 50 results not just the single customer.
Failure Information
N/A
Steps to Reproduce
Run the above query using an email you know exists

Well, after firmly planting my face into the keyboard, repeatedly, I determined the issue. An undocumented thingy....
In the Shopify dev docs and here in the wiki, it only assumes a customer search needs one query parameter. Well, thats false it appears. You must include 'limit' of 1.... I am working to confirm this but so far thats what it seems.

Glad you found the solution @unplugged216 - I'm going to close this ticket now.

hey @unplugged216 how you resolved this? as per your conversation i added a limit but still getting all records

Hey there!
Here is an example, demonstrating what did not work and what worked for me.
What did not work
$shop->api()->rest('GET','/admin/api/2022-10/products.json', [ 'product_type' => 'Demo Type'])['body'];
This seemed to fix my issue
$shop->api()->rest('GET','/admin/api/2022-10/products.json', ['limit' => '250', 'product_type' => 'Demo Type'])['body'];
Please not I am also using the latest API. I opted in this call to set the API version manually instead of in ENV as I have noticed the ENV vars are not getting loaded consistently by the package.

Glad you found the solution @unplugged216 - I'm going to close this ticket now.

@Kyon147 ,
I would recommend not closing this as my solution may only work for me. This is a documentation issue in this package. The package provides a wiki demonstration that does not work.... Closing an issue without resolve is unsettling for the project I feel.

Hi @unplugged216
Happy to reopen, that's my misunderstanding reading it as a API issue rather than a package one. I tried a direct api call, and don't see any issues - so there could be a bug in the shopify-basic-api package but would need some more investigation.


Several compiler flags like --iree-llvmcpu-loop-unrolling have no effect

Thanks @banach-space for reporting that in #16092.
The problem is here:
https://github.com/openxla/iree/blob/6ab1ed8e94d1e4a2ba07abbcab4f80302631ab8b/compiler/src/iree/compiler/Tools/iree_compile_lib.cc#L183-L187
The ireeCompilerGlobalInitialize(); at line 183 is where we register target backends, which is where we read the flag. But at that point, it still just returns the default flag value, because we haven't yet initialized command-line flags, as that is done below by ireeCompilerSetupGlobalCL() at line 185.
We can't simply reorder these calls, because ireeCompilerSetupGlobalCL dereferences the globalInit pointer, which is still NULL before ireeCompilerGlobalInitialize().
Call stack to where we just get the default value of the flag in LLVMCPU initialization, notice ireeCompilerGlobalInitialize(); at iree_compile_lib.cc:183 in the stack:
llvmLoopUnrolling=0

Breakpoint 1, mlir::iree_compiler::IREE::HAL::LLVMTargetOptions::getFromFlags ()
    at /home/benoit/iree/compiler/src/iree/compiler/Dialect/HAL/Target/LLVMCPU/LLVMTargetOptions.cpp:450
450       target.pipelineTuningOptions.SLPVectorization = llvmSLPVectorization;
(gdb) bt
#0  mlir::iree_compiler::IREE::HAL::LLVMTargetOptions::getFromFlags ()
    at /home/benoit/iree/compiler/src/iree/compiler/Dialect/HAL/Target/LLVMCPU/LLVMTargetOptions.cpp:450
#1  0x00007ffff2a79f00 in mlir::iree_compiler::IREE::HAL::registerLLVMCPUTargetBackends(std::function<mlir::iree_compiler::IREE::HAL::LLVMTargetOptions ()>)
    (queryOptions=...) at /home/benoit/iree/compiler/src/iree/compiler/Dialect/HAL/Target/LLVMCPU/LLVMCPUTarget.cpp:911
#2  0x00007ffff177431e in mlir::iree_compiler::registerHALTargetBackends()::$_0::operator()() const (this=<optimized out>)
    at /home/benoit/iree/compiler/src/iree/compiler/Tools/init_targets.cc:28
#3  0x00007ffff17742d6 in mlir::iree_compiler::registerHALTargetBackends () at /home/benoit/iree/compiler/src/iree/compiler/Tools/init_targets.cc:25
#4  0x00007ffff15751ee in mlir::iree_compiler::embed::(anonymous namespace)::GlobalInit::GlobalInit (this=<optimized out>)
    at /home/benoit/iree/compiler/src/iree/compiler/API/Internal/CompilerDriver.cpp:251
#5  ireeCompilerGlobalInitialize () at /home/benoit/iree/compiler/src/iree/compiler/API/Internal/CompilerDriver.cpp:1248
#6  0x00007ffff1776f26 in mlir::iree_compiler::runIreecMain (argc=<optimized out>, argv=<optimized out>)
    at /home/benoit/iree/compiler/src/iree/compiler/Tools/iree_compile_lib.cc:183
#7  0x00007fffec4280d0 in __libc_start_call_main (main=main@entry=0x555555555780 <main(int, char**)>, argc=argc@entry=9, argv=argv@entry=0x7fffffffdb18)
    at ../sysdeps/nptl/libc_start_call_main.h:58
#8  0x00007fffec428189 in __libc_start_main_impl (main=0x555555555780 <main(int, char**)>, argc=9, argv=0x7fffffffdb18, init=<optimized out>,
    fini=<optimized out>, rtld_fini=<optimized out>, stack_end=0x7fffffffdb08) at ../csu/libc-start.c:360
#9  0x00005555555556b5 in _start ()

@stellaraccident , I think the problem is in embedding_api.h. By exposing separate API entry points for ireeCompilerGlobalInitialize and ireeCompilerSetupGlobalCL, it forfeits control over the ordering of CL initialization vs ireeCompilerGlobalInitialize, and together with the fact that ireeCompilerGlobalInitialize does care about CL option values, the problem seems intractable with the current API.
I think we need to add another API entry point combining both, making it an internal detail in which order CL vs other things are initialized. And then, users of the API that want to enable the CL should use that. The existing ireeCompilerGlobalInitialize entry point can stay for users who don't want the CL at all. ireeCompilerSetupGlobalCL should probably go away, since it can't be used correctly. WDYT ?

Global flag ordering and the race to the top of main is going to be the death of me.
Yes, we can change that in any way we want: that part of the API is just for tightly coupled tools to work right.
Even if it has issues, we may need to keep the current cl init function for a bit. Various things in Python land use it as an escape hatch to set flag values in arbitrary ways (it's already unsafe and private but useful in a pinch).

I think I was wrong in my above assessment.  The call to LLVMTargetOptions::getFromFlags() in the call stack in the issue description above, which was getting default values because it was called before CL initialization, doesn't matter. Its return value is not even used, it's called just for static registration purposes, AFAIU.
Instead, the problem was what https://github.com/openxla/iree/pull/16153 fixes.


use oc instead of kubectl for OpenShift artefacts

fixes #341
If OpenShift controller like DeploymentConfig is used in Kedge file,
than oc binary will be used instead of kubectl for commands like
kedge apply, kedge create, kedge delete

Don't understand why is Coveralls still complaining.
I've even added more tests than there was before.

@kadel for now this is good, but then this will be using the oc binary only in case of DeploymentConfig, for everything else it'll be using kubectl - which will work fine - but then the end user needs to have both the binaries to deploy to OpenShift.
I think this is another place where we could use information from the --distribution flag.
This problem will still be there once we move to kubernetes/client-go and openshift/client-go.
kubernetes/client-go should still work with OpenShift, but it's just the inconsistency this brings to our operations since we have no information about the distribution this is being run on.

@kadel for now this is good, but then this will be using the oc binary only in case of DeploymentConfig, for everything else it'll be using kubectl - which will work fine - but then the end user needs to have both the binaries to deploy to OpenShift.

Once we have Routes in the logic should change a bit. oc should be used if at least one OpenShift specific resource is used.
Once we stop using binaries, and use openshift/client-go this problem will go away.
It will be possible to use openshift library for everything, even for creating Kubernetes artifacts on Kubernetes

I think this is another place where we could use information from the --distribution flag.

If stick with using oc and kubectl binaries, that it might make sense to add some flag like --use-oc to force calling oc for everything. I think that this might be better than --distribution flag.

Lets keep this open, until we merge Routes, than we can figure out how we will do this

I've updated this.
Now it is not based on just a controller.
oc is used for all artifacts if there is at least one generated OpenShift  specific object (DeploymentConfig, Route)
So if k8s Deployment is used as controller object, and there is a Route, than oc will be used.

Rest LGTM!

@kadel this should say oc error or something else, right now it says kubectl error.

replaced with "failed to execute command: ....."

@kadel  Haha, I hope I convince you for --distribution one day :laughing:
LGTM!


Question about Versions 0.2.x | 0.4.2 | 1.0

Hi,
currently I am using 0.2.8 and I am not using any npm or gulp only NuGet.
But I am using TypeScript with typings from
https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/angular-ui-router
that works fine so far.
Now I would like to Upgrade and the only compatible Version I've found without npm was 0.4.2 where I cann use the typings from DefinitelyTyped is this version getting bugfixes if there are some bugs or is this also a "dead" version like 0.2.8.
And when should I use 1.0 instead of 0.4.2? Will there be a NuGet compatible version for 1.0 in the future with "plain" typings?

0.4.2 is still the latest "stable" release and is not even 2 weeks old.  0.4.2 is the latest in the legacy branch.  We jumped from 0.2.x to 0.3.x and 0.4.x because there were some potential breaking changes, as documented in the release notes.  If you want to stay on the legacy branch, I recommend moving to 0.4.2.  0.4.2 will continue to get any critical bug fixes (such as the recent change in angular 1.6.x promise handling).  Other than those critical bug fixes, it will not be developed any further.
On the master branch, 1.0.0-rc.1 is the latest release.  rc.1 looks good, so I don't think we'll need an rc.2.  The next release will likely be 1.0 final, so now would be a good time to think about the update.
We don't have any control over NuGet published version.  Some third party published it there.  We only publish to npm (and for legacy reasons, we still publish the ng1 releases to bower).
We also aren't involved in the DefinitelyTyped typings.  That was also published by a third party (which is totally awesome).  DefinitelyTyped "plain typings" are sort of a band-aid or stop-gap solution for non-typescript projects.  Now that ui-router is written in Typescript, there's not much of a reason to publish separate typings to DefinitelyTyped.  The official typings are distributed and updated in each released version.
Hope these answers are helpful


Add unwrap_or_return_uninit!()

This macro can be used to conveniently bubble up errors from
init_from_closure() in a ? operator like fashion.
I found this macro convenient in my code, I thought I'd try upstreaming it.

Bump @nbdd0121


WIP fix utility group transformation

I am using the regex of vite plugin, thanks to @antfu (i stole it) :octocat:
It needs some more refactoring, but should be finished soon
Closes #18

i might refactor this sometime, but for now it looks the best way with only one line


read_excel(sheet_name=None) not working  (#512)

What do these changes do?
Fix read_excel when sheet_name=None, return corrected type (OrderedDict)
#512

[X] passes flake8 modin
[X] passes black --check modin
[X] tests added and passing

Thanks @ipacheco-uy! I went ahead and merged with master to fix the travis compatibility issue.


JS tasks block python tasks

When running the following code in Jupyter lab:
import ipyjs
w = ipyjs.ExampleWidget()
code = '''
function wait(seconds) {
  var start = new Date();
  console.log(start)
  while((new Date() - start) / 1000 < seconds);
}

var begin = performance.now();

console.log("start");

wait(1);
wait(1);
wait(1);
wait(1);
wait(1);
wait(1);
wait(1);
wait(1);
wait(1);
wait(1);
'''
w.eval(code)

I would like to see, from the python side, some loading dots, as can be seen in the code.
However, what occurs is that the python output stops while the custom js code is being evaluated.
This is probably expected behavior from jupyter_ui_poll, but for the purposes of this test library, it would be nice to be able to see loading dots in order to be able to tell when the js custom code has finished evaluating.
Is there way to do this? Which widget events should I reply? Does kernel.do_one_iteration() suffice for this?

@Kirill888 Hello, I'm not sure if you will see this, but I wanted to ask if you had any ideas on this issue. I saw the work you've done with jupyter-ui-poll and was wondering if you had any thoughts on this.

@AnthonyAndroulakis I'm no expert on JS side of things, but I think your problem is that your widget is blocking execution of any other JS code, so no messages get sent to the python side since JS is single threaded and your code seems to be busy-polling. I suggest you try with JS code that uses timer event callbacks instead of a while loop.

I see, thank you for response and your suggestion. I'll try that out.


Galran/new geometry module

Introducing the new Geometry module, which currently contains types (Vec3, Vec4 and Transform) and some utility functions.
The Transform type is a replacement for the RigidBodyTransform that was part of the CSG part of the package. The new Transform is used to position surfaces inside a CSG tree, but also emitters in 3D and everywhere else a 3D transform is needed.
The changes are all over the package, including the documentation, so please take the time and go through them.

Codecov Report

Merging #70 (48f1aaf) into main (a118833) will decrease coverage by 0.68%.
The diff coverage is 37.74%.

@@            Coverage Diff             @@
##             main      #70      +/-   ##
==========================================
- Coverage   52.41%   51.73%   -0.69%
==========================================
  Files          54       54
  Lines        5992     6089      +97
==========================================
+ Hits         3141     3150       +9
- Misses       2851     2939      +88

Impacted Files
Coverage Δ

src/Examples/Examples.jl
1.14% <0.00%> (ø)

src/Geometry/Geometry.jl
0.00% <ø> (ø)

src/Geometry/Primitives/Curves/Bezier.jl
56.70% <ø> (ø)

src/Optical/Eye.jl
0.00% <0.00%> (ø)

src/Optical/Lenses.jl
37.16% <0.00%> (ø)

src/Optical/OpticalRay.jl
60.71% <ø> (ø)

src/Visualization/Visualization.jl
6.06% <0.00%> (ø)

src/Geometry/Primitives/NonCSG/Triangle.jl
74.28% <16.66%> (-5.41%)
:arrow_down:

src/Geometry/Transform.jl
31.33% <31.33%> (ø)

src/Geometry/CSG/CSG.jl
84.94% <50.00%> (+0.05%)
:arrow_up:

... and 6 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 a118833...48f1aaf. Read the comment docs.

Do we think this merits a bump to v0.4.0 (breaking change)?

I fixed everything except the 'I' - i don't understand exactly if this is a "laze" definition or how to use it efficiently, so i left the identitytransform function for now (i removed the identityT function).

@galran did you push the latest stuff? I looked at the Transform.jl file and it seems the same as before.

Pushing it right now. Will probably finish in 3 min.
ran
From: BrianGun @.>
Sent: Friday, April 2, 2021 1:09 PM
To: microsoft/OpticSim.jl @.>
Cc: Ran Gal @.>; Mention @.>
Subject: Re: [microsoft/OpticSim.jl] Galran/new geometry module (#70)
@galranhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fgalran&data=04|01|rgal%40microsoft.com|4c108e914ea44ed119ab08d8f6133086|72f988bf86f141af91ab2d7cd011db47|1|0|637529909576969943|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=ZNGfCy3Xhg%2F25vV2FfVm%2BnXvTGR%2BsZrfAK31Vzfp7lw%3D&reserved=0 did you push the latest stuff? I looked at the Transform.jl file and it seems the same as before.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FOpticSim.jl%2Fpull%2F70%23issuecomment-812691167&data=04|01|rgal%40microsoft.com|4c108e914ea44ed119ab08d8f6133086|72f988bf86f141af91ab2d7cd011db47|1|0|637529909576969943|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=AOKKQYGJjYuSfBKqIyPrlb5ucfu4dEmnvV1EexL2z80%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FATBK6XSMIN23AFAULFW4OITTGYP6VANCNFSM42FULF2A&data=04|01|rgal%40microsoft.com|4c108e914ea44ed119ab08d8f6133086|72f988bf86f141af91ab2d7cd011db47|1|0|637529909576979941|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=%2Bh%2FivMcKlZpPUjdpKH%2FLqSv%2FIPA6dHXI91LYEjNQ6%2F8%3D&reserved=0.


Migstorage secret discovery refactor

Improved resiliency.
Driven by ref name.
Closes #362

We chatted this one over offline and feel that to implement the wider feature here is out of scope for this PR. Submitting a PR that correctly dereferences the secret ref will fix the immediate linked issue (https://github.com/fusor/mig-ui/pull/363/files#diff-d63dd651509698d746484d8ad4394199R203), and we're going to close this one.


Heroku problem with redis conection

Hi everyone. I recently upgraded the redis plan (free to Premium 0) of my staging app on heroku.
with this configuration:
celery==3.1.18
django==2.1.15
djangorestframework==3.9.0
gunicorn==19.8.1
kombu==3.0.37
redis==2.10.3
I am getting this error on sentry:
ConnectionError: null
File "django/core/handlers/exception.py", line 34, in inner
response = get_response(request)
File "django/core/handlers/base.py", line 126, in _get_response
response = self.process_exception_by_middleware(e, request)
File "django/core/handlers/base.py", line 124, in _get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)
File "django/views/decorators/csrf.py", line 54, in wrapped_view
return view_func(*args, **kwargs)
File "django/views/generic/base.py", line 68, in view
return self.dispatch(request, *args, **kwargs)
File "rest_framework/views.py", line 495, in dispatch
response = self.handle_exception(exc)
File "rest_framework/views.py", line 455, in handle_exception
self.raise_uncaught_exception(exc)
File "rest_framework/views.py", line 492, in dispatch
response = handler(request, *args, **kwargs)
File "rest_framework/generics.py", line 192, in post
return self.create(request, *args, **kwargs)
File "saf/apps/contratos/views.py", line 34, in create
self.perform_create(serializer)
File "rest_framework/mixins.py", line 26, in perform_create
serializer.save()
File "rest_framework/serializers.py", line 214, in save
self.instance = self.create(validated_data)
File "saf/apps/contratos/serializers.py", line 350, in create
club_lpf_destino=validated_data.get('club_lpf_destino'))
File "contextlib.py", line 52, in inner
return func(*args, **kwds)
File "saf/apps/contratos/service.py", line 81, in create
author=user, club=user.is_saf(), saf=user.is_club()
File "saf/apps/common/service.py", line 25, in _send_notifications
users=users_to_notify, title=title
File "saf/apps/notifications/service.py", line 34, in create
self.mobile_notificator.create(notification, mobile_users)
File "saf/apps/notifications/service.py", line 54, in create
self.send_notification.delay(payload)
File "celery/app/task.py", line 453, in delay
return self.apply_async(args, kwargs)
File "celery/app/task.py", line 559, in apply_async
**dict(self._get_exec_options(), **options)
File "celery/app/base.py", line 353, in send_task
reply_to=reply_to or self.oid, **options
File "celery/app/amqp.py", line 305, in publish_task
**kwargs
File "kombu/messaging.py", line 172, in publish
routing_key, mandatory, immediate, exchange, declare)
File "kombu/connection.py", line 470, in _ensured
interval_max)
File "kombu/connection.py", line 382, in ensure_connection
interval_start, interval_step, interval_max, callback)
File "kombu/utils/init.py", line 246, in retry_over_time
return fun(*args, **kwargs)
File "kombu/connection.py", line 250, in connect
return self.connection
File "kombu/connection.py", line 756, in connection
self._connection = self._establish_connection()
File "kombu/connection.py", line 711, in _establish_connection
conn = self.transport.establish_connection()
File "kombu/transport/virtual/init.py", line 809, in establish_connection
self._avail_channels.append(self.create_channel(self))
File "kombu/transport/virtual/init.py", line 791, in create_channel
channel = self.Channel(connection)
File "kombu/transport/redis.py", line 466, in init
self._disconnect_pools()
File "kombu/transport/redis.py", line 484, in _disconnect_pools
self._async_pool.disconnect()
File "redis/connection.py", line 898, in disconnect
connection.disconnect()
File "kombu/transport/redis.py", line 866, in disconnect
channel._on_connection_disconnect(self)
File "kombu/transport/redis.py", line 498, in _on_connection_disconnect
raise get_redis_ConnectionError()

Hi! GitHub issues are for reporting bugs and features with the repository in question. The above issue is a support request, rather than a bug or feature, and is also not about the Python buildpack - so it doesn't belong here.
For support requests, please open a support ticket:
https://help.heroku.com


Duplicate agents in properties table

Hello,
I am not sure of whether this is a bug or a feature:
https://paperswithcode.com/sota/visual-question-answering-on-gqa-test2019

Is it ok that agent is repeated? I was assuming that I would only have 1 agent if all other fields (paperUrl, date, source, etc) are the same.
Thank you

redundant_tuplas_by_metric_pwc.xlsx
I don't know if this will be of any use, but I identified the tuplas of experiments that contain more than one value when grouping by task, dataset, agent, paperDate, paperUrl and metric.

@dhamaris Thanks for opening this issue. Looking at the original source of where the result came, it appears that the submissions are unique in the sense that they come from two different authors.

I am not sure how to fix what you are trying to achieve. Not sure about it, but I am inclined to think that the results correspond two different experiments by two different users. Maybe they use similar model, backbone, experimental setup, or even same code base, as it is common in public community leaderboards such as Kaggle. If the results are obtained from external leaderboards like in this case, it is possible we will see this type of results. If the results are added on our website directly, it is less common as results are added directly through papers, helping to preserve one model to one result relationship.


Fix OpenShift resource URI caching - clear cache on service stop

Summary
Prevents the cached value from distorting status checks on stopped resources.
This fixes infinispan-client failures observed in e.g. https://github.com/quarkus-qe/quarkus-test-suite/pull/975.
This is a follow-up to
https://github.com/quarkus-qe/quarkus-test-framework/pull/625, which introduced the caching in order to fix issues in INTEROP environment. See e.g.
https://mpts-prod.psi.redhat.com/dashboard/run/?id=8896.
Please check the relevant options

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Dependency update
[ ] Refactoring
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] This change requires a documentation update
[x] This change requires execution against OCP (use run tests phrase in comment)

Checklist:

[ ] Example scenarios has been updated / added
[ ] Methods and classes used in PR scenarios are meaningful
[x] Commits are well encapsulated and follow the best practices

run tests


No Grassium-Z in the Item choices

I see the other Z-move items for the calcs except at first glance, no Grassium-Z choice.

Fixed https://github.com/Zarel/honko-damagecalc/commit/d6ec797ac33a756e1c16ac8275a4027de7554434


Don't run in CI

When the progress bar runs in my CI (jenkins) it produces an insane amount of output. Maybe you could use the package 'isCi' to disable the progress bar in the CI environment

Yeah. It's already shipped in 1.3.0 ;)
https://github.com/nuxt/webpackbar/blob/master/CHANGELOG.md


Feature Request: Allow lookup from path to look at two parts of the path

As the code is right now, lookupFromPathIndex can only be one number.
If I have a URL that breaks the locale up into two parts such as /en/us/restOfPath, then I can only grab the language portion of the path. I want to be able to construct the full locale en-US from the path and not just the 'en' portion.

hm...guess that is out of the things we could support - we support locale in one position - no support for building locale out of multiple path segments -> that is to custom.
But you can easily add your own detection logic: https://github.com/i18next/i18next-browser-languageDetector#adding-own-detection-functionality


Verified only user search

Solves: https://github.com/PEC-CSS/acm-website-backend/issues/108

files changed from multiple PRs, request a review after

merging the first PR
merging main into this branch


Add link to releases on every repository page

At the moment you can click to go to the releases page only when you're on the Code tab in a repository.
As I use the releases page quite often, I'd suggest to place a link to releases on every tab of the repo. Perhaps as another tab?
I hope others would find this useful as well.

:+1: Great idea! I use the releases page a lot too. Another tab sounds good.
Interested in doing a pull request?

Sure, I can suggest a PR later today or tomorrow though. That's why I wanted to make a suggestion and see if it's worth it.

Cool. We could use this icon https://octicons.github.com/icon/tag/ (already part of the font icon on the site), unless you have a better suggestion.


Client side cloud error logging.

We can setup error logging in each of our client web apps. Can track what bugs users are getting in JS and WASM. Get some metrics, detect things we may not know. Also don't have to rely on user stories to fix bugs.
Bunch of tools for this, can even upload source maps with it and track line numbers and full stack traces.

I'll agree with this, we should slap in one of the usual services.


Add Find by ios class chain

Change list
This is to add capability to use class chain selectors for Finding elements.
Types of changes

[ ] Bugfix (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)

Details
            AppiumWebElement lastSettingsCellItem = MobileAppMgr.CurrentAppDriver.FindElement(MobileBy.IosClassChain("**/XCUIElementTypeCell[-1]")); string labelAttribute = lastSettingsCellItem.GetAttribute("label"); lastSettingsCellItem.Tap(1, 250);
Appium Logs for above code are available here

@TikhomirovSergey Made requested updates.

@mykola-mokhnach Made the fixes.


Reinstall mypy

I accidentally removed mypy when I installed ruff, but ruff is not a type checker.
I will reinstall mypy.
https://docs.astral.sh/ruff/faq/#how-does-ruff-compare-to-mypy-or-pyright-or-pyre

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (0d60974) to head (1555f9f).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #130   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines           83        83
=========================================
  Hits            83        83

Flag
Coverage Δ

unittests
100.00% <ø> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Keyboard service doesn't commit the last edit when it's dismissed

We were getting the proper values, but we thought we were still composing, so
the text was still underlined.
Fixes #115

@jason-simmons @Hixie


How can I make evilginx capture session multiple times

Currently evilginx take one session per victim, but I want to capture everytime even if the victim is same

**Hi 👽 ! If you are interested in ⚙Advanced Phishing⚙ and the ✨Evilginx3✨ tool then I offer you Google Gmail and Custom Phishlets with a ✅Free Tutorial✅ which arrives if the 📈Telegram reaches 50 Subscribers📈
⏳ GOOGLE GMAIL PHISHLET 2024 WITH PROOF ⏳
->
⛔https://www.youtube.com/watch?v=nA47UknIE6A⛔
🔓 Ask your questions and a free tutorial from 50 subs on telegram !!!
->
📩https://t.me/Noremorse1011📩**


build.sh -> Latex Error: File "beramono.sty" not found

By installing texlive-fonts-extra as mention in here


Initialized Resources #70

Codecov Report
All modified lines are covered by tests :white_check_mark:

Comparison is base (f2825b4) 97.09% compared to head (5d831d5) 96.96%.
 Consider uploading reports for the commit dd9c59d to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##               main     #117      +/-   ##
============================================
- Coverage     97.09%   96.96%   -0.13%
- Complexity      643      645       +2
============================================
  Files           172      174       +2
  Lines          1409     1418       +9
  Branches         47       47
============================================
+ Hits           1368     1375       +7
- Misses           35       37       +2
  Partials          6        6

Files
Coverage Δ

...confidence/junit5/engine/quality/ResourceThat.java
100.00% <100.00%> (ø)

...confidence/junit5/engine/resource/Initialized.java
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


coverity 1021972: Dereference after null check

it looks good. thanks.

[approve ci]

clang-analyzer build successful! https://ci.trafficserver.apache.org/job/clang-analyzer-github/751/

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/523/

clang format failed! https://ci.trafficserver.apache.org/job/clang-format-github/514/

AU check successful! https://ci.trafficserver.apache.org/job/autest-github/510/

Intel CC build successful! https://ci.trafficserver.apache.org/job/icc-github/639/

FreeBSD11 build successful! https://ci.trafficserver.apache.org/job/freebsd-github/2206/

clang-analyzer build successful! https://ci.trafficserver.apache.org/job/clang-analyzer-github/767/

Linux build successful! https://ci.trafficserver.apache.org/job/linux-github/2102/


Update dependabot/fetch-metadata action metadata field

Why:
Closes #12138
What's being changed:
Change metadata field according to the docs
Check off the following:

[x] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@iamandrewluca
Thanks so much for opening a PR! I'll get this triaged for review :zap:

👋🏻 - I'll take a look at this but will first check with the relevant team as my knowledge in this area is very limited. Thanks for your patience 🙂

Thanks for the technical review @brrygrdn ✨
@iamandrewluca 👋🏻 - can you add @brrygrdn's suggestions to your PR, so I can give it a quick check and merge it? Thank you!

@mchammer01 @brrygrdn updated the PR.
It seems that I configured my action wrong then
https://github.com/iamandrewluca/holiday-time/blob/e35e02a93cbef5e5dcad8efbd7965d33d203ad0a/.github/workflows/dependabot.yml#L19
It sems to work because of !=


Add ability to remove permissions when disabling user

Closes #957
Tests to come
Review Checklist

Tests were added to cover all code changes
Documentation was added / updated
Code and tests follow standards in CONTRIBUTING.md

Codecov Report

Merging #958 into master will decrease coverage by 0.01%.
The diff coverage is 75%.

@@            Coverage Diff             @@
##           master     #958      +/-   ##
==========================================
- Coverage   90.04%   90.02%   -0.02%
==========================================
  Files          48       48
  Lines        6519     6519
==========================================
- Hits         5870     5869       -1
- Misses        649      650       +1


tiffs with npages not dividing by pageCount will give a warning instead an error

Alternative implementation to #279
Thinking of adding a flag to fromtif() that would default to the current behavior of rising an error.
Also I am sending the warning using the warning module, I saw another way using the spark logger:
log4jLogger = sc._jvm.org.apache.log4j
LOGGER = log4jLogger.LogManager.getLogger(__name__)
LOGGER.warning("number of planes...")

@sofroniewn @freeman-lab What do you guys think?

Why don't we forget the warning stuff, because doing warnings on the workers well is hard, and instead add a keyword argument like discard_extra that will discard extra planes instead of giving an error (as we do now). Ok if you want at the same time to use our own warnings to log the file name.

Warnings should be done like this:
https://github.com/thunder-project/thunder/blob/0f2e29fbe106a7b50dffeb6f056aba4c107b82a7/thunder/utils.py#L4

@freeman-lab Thanks for the comments!
I think it is ready to merge.

Excellent, looks good to me @boazmohar , merging in!


Yeelight Lightstrip Plus

Device model
Yeelight Lightstrip Plus
Device name
yeelink.light.strip2
Does the device require a MiCloud connection?
Yes
Anything special about the device?
"Poll failed 4 times in a row! Stopping polling and trying to reconnect! Reason: Error: Call to device timed out"

If you say that this device requires a MiCloud connection then you need to specify your MiCloud credential sin the config.json and set the forceMiCloud property to true. That should work for your device.

I tried to force MiCloud but it's still not working. I'm not sure if it require MiCloud, but I guess so. Some other variant of Lightstrip Plus (yeelink.light.strip2) as I saw required that.

What error do you get when you force it to use MiCloud?


Identifier empty after creating an amendment.

Steps to reproduce:

Create a motion, it gets the identifier 1
Set the amendment prefix to ÄA-
create an amendment, its identifier is empty.
-> It should be Äa- 1. This causes issue #3478

Had the workflow set to complex. This forbids to set an identifier.


@conda-forge-admin,please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/tsfresh-feedstock#17.


fix: when the jobs parameter is an empty array, block unnecessary runCodeGenerationJobs

What kind of change does this PR introduce?
fixed sometimes log will be NaN%
Did you add tests for your changes?
no
Does this PR introduce a breaking change?
no
What needs to be documented once your changes are merged?
no

this should be fixed in DefaultStatsPrinterPlugin

this should be fixed in DefaultStatsPrinterPlugin

Sorry, my pr description is wrong. I have corrected my pr title description.

Thanks


XmlRecordReader's escape-function is missing escaping for gt and lt

I saw that you fixed the issue with the reader passing unescaped content to the mapper in 5.1.0 - however, it seems that the readers escape function only escapes 3 entities, missing gt and lt (<>).
This causes issues when element's contain these character, as they become unencoded by the reader and then interpreted by the mapper.
For example:
<ValueTemplate><derive expr="{ becomes
<ValueTemplate><derive expr="{
Now the mapper throws an exception because the attribute "expr" of supposed element (which it is not supposed to be) "derive" does not have an opening quotation mark.

Thank you for pointing out this issue!

I saw that you fixed the issue with the reader passing unescaped content to the mapper in 5.1.0

I guess you are talking about this: https://github.com/j-easy/easy-batch/blob/master/easybatch-xml/src/main/java/org/easybatch/xml/XmlRecordReader.java#L166
right?

however, it seems that the readers escape function only escapes 3 entities, missing gt and lt (<>).

There should be a good reason why < and > were not escaped, but to be honest, I don't remember it.

This causes issues when element's contain these character

Is this allowed in xml? According to your example, the XmlRecordReader takes this:
<ValueTemplate><derive expr="{
and puts this in the payload of the xml record:
<ValueTemplate><derive expr="{
is this the case?

Yes, that is the case, exactly as you have described.
The issue with the reader is that the underlying reader it uses to retrieve payloads already does un-escaping of XML-entities.
So before passing it on the mapper, you pretty much have to re-escape the whole thing.
"<derive expr="{" is valid content for an XML, since it's properly escaped.

Hi @Mercious
I've updated the XmlRecordReader to escape < and > in addition to other special characters.
Now, even if the input is escaped, the reader will keep it intact.
The fix is in v5.2.0-SNAPSHOT. It would be great if you give it a try and tell me if it works fine for you now. Thank you in advance!
Kr
Mahmoud

@Mercious Have you got a chance to test the fix?

@benas Yes, works fine now - thanks for the fix!

Great! thanks for testing 👍


added 3 more stylers and tests

only 2 of those items can be tested.   So I tossed them in kitchen sink.

:+1:


Pass minify parameter so minimization will happen in build.

I see in #44 the background on why we are not pushing the minified JS in our documentation at the moment. With that said, there is a checkout.min.js being built but it is not minified as the parameter to enable the minification is not passed. This PR enables it so the minified version exists correctly even if not (yet) noted in documentation.
I fully understand this may be something we do not want to do because of the discussion in #44, but I think having it there for merchants who want to use it but not having the documentation point to it can be helpful.
For reference, enabling minification shrinks the file by about 50%. It is still large, but this is more manageable.
$ la dist/checkout.4.0.47*js
-rw-r--r--  1 ehummel  110057925   463K Mar  6 13:34 dist/checkout.4.0.47.min.js
-rw-r--r--  1 ehummel  110057925   849K Mar  6 13:34 dist/checkout.4.0.47.js

@ErikMHummel Can I just download the dist/checkout.4.0.176.js file on master branch, and include it in my script tag, and be assured that this is the smallest possible version?


Feature/rental forecasting

Rental Forcasting Folder
Location

data/model/rent
add new rental folder with larger dataset

Details

create forecast model for studio - 3 bedrooms

Committed Features

users can now see a 10 year projection for studio -3 bedrooms in desired city.

 DS will create endpoint that shows rental predictions.


[Event Request] Add Event OnBeforeCheckWizardAction Table 5064 Interaction Template

Hello. Please add an Event OnBeforeCheckWizardAction in Table 5064 Interaction Template
field(18; "Wizard Action"; Enum "Interaction Template Wizard Action")
        {
            Caption = 'Wizard Action';

            trigger OnValidate()
            var
                InteractionTmplLanguage: Record "Interaction Tmpl. Language";
                IsHandled: Boolean;

            begin

                OnBeforeCheckWizardAction(Rec, IsHandled);  <--- NEW EVENT
                if IsHandled then
                    exit;

                if InteractionTmplLanguage.Get(Code, "Language Code (Default)") then
                    if ((InteractionTmplLanguage."Custom Layout Code" <> '') and ("Wizard Action" <> "Wizard Action"::Merge) and ("Word Template Code" = '')) or
                       ((InteractionTmplLanguage."Custom Layout Code" = '') and ("Wizard Action" = "Wizard Action"::Merge) and ("Word Template Code" = '')) or
                       (("Word Template Code" <> '') and ("Wizard Action" = "Wizard Action"::Import))
                    then
                        Error(Text003, FieldCaption("Wizard Action"), "Wizard Action", TableCaption, Code);
            end;
        }

    [IntegrationEvent(false, false)]
    local procedure OnBeforeCheckWizardAction(var InteractionTempl: Record "Interaction Template"; var IsHandled: Boolean)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Error: You are accessing the Scene#data object which is no longer used.

the Scene DataModel changed, and the module needs to access the _id value directly:
Error: You are accessing the Scene#data object which is no longer used. Since V10 the Document class and its contained DataModel are merged into a combined data structure. You should now reference keys which were previously contained within the data object directly.
    at logCompatibilityWarning (commons.js:1692:19)
    at Scene._logV10CompatibilityWarning (commons.js:6617:14)
    at get data [as data] (commons.js:6517:24)
    at GamingTablePlayer.pullFocus (gaming-table-player.js:107:47)
    at keyDown (gaming-table-player.js:122:21)
logCompatibilityWarning @ commons.js:1703
_logV10CompatibilityWarning @ commons.js:6617
get data @ commons.js:6517
pullFocus @ gaming-table-player.js:107
keyDown @ gaming-table-player.js:122
foundry.js:676

Basically "game.scenes.viewed.data._id" needs to be changed to "game.scenes.viewed._id"

Bug should be resolved by v1.0.1


Request: Interleave merge

Hi @lelegard ,
At time I'm very satisfied with the interleave parameter of the FILE input. With this approach is quite easy to mux different SPTS. Using the tsstuff tool it's possible to convert any SPTS to a fixed CBR and then mux any number of thse SPTS with the same bitrate using the interleave approach. Finally using the reduce plugin it's possible to remove all NULL stuff and target a final CBR for the MPTS. To complete the processing at the end it's mandatory to use the pcradjust plugin to smooth the PCRs in the MPTS.
So, now I want to try to reproduce this in real-time. But the merge plugin lacks the interleave functionality. The idea is very simple: read 1 packet from the PIPE and 1 packet from the CHAIN. And if one of the queues is empty then inject a NULL packet.
What you think?

It does not work like that.
The file plugin is an input plugin. It is the source of the chain. It has complete freedom on which packet to inject and when. Everything is possible (in theory).
The merge plugin is a packet processor plugin. It never inserts new packets. It can only transform (or delete) existing packets in the chain. As any other packet processor plugin which needs to insert something, it can only replace existing null packets. In other words, if you use the merge plugin on a stream where there is no null packet, nothing will be merged. The merge plugin has no freedom to decide when to insert a packet from the merge stream. It is dependent on the content of the main stream from the chain.

Hi @lelegard ,
Yes, you're right!
What then of a pipe input with interleave functionality? The idea will be a new input module that creates N child process and reads from all of them. Could that work?

What then of a pipe input with interleave functionality?

I would prefer an --interleave option to tsswitch. This would be more generic. When I have time...

What then of a pipe input with interleave functionality?

I would prefer an --interleave option to tsswitch. This would be more generic. When I have time...

Good idea! In this case (when you have time, no problem), the tsswitch will read from all inputs at time (or from a subset of them)?

tsswitch will read from all inputs at time (or from a subset of them)?

All of them, this is the purpose of --interleave.


Removed non-beta textures, fixed cacti, flowers, rails, bed, nether stuff, redstone.

Glowstone
Beds
Pumpkins
Redstone
Csctus
Netherrack
Soulsand
Extra: Removed items not in version.

There are merge conflicts that need to be resolved before I can accept this pull request.

I'm not quite sure I understand... Do I need to recopy the folder from your repo, and apply the new image over it?

Link

That doesn't really help... I just cloned my branch of the repo locally, pushed it there via git, and then submitted a pull request via the web portal.  Any ideas?

I guess the easiest way would be to make a fresh fork and just toss your current texture into it.


png files optimization(loss less)

Used zopfli with no options to reduce image size.
Use WinMerge to verify no loss of png files.
[Before]
png files 1,501,003 bytes
[After]
png files 1,168,250 bytes
List which issues are fixed by this PR. You must list at least one issue.

reducing the resources used by users

If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@christopherfujino
Thanks for your review.
That was certainly true.
I have changed the PR objective to reducing the resources used by users.


Change Request: support CSS-in-JS

Environment
Node version: v22.11.0
npm version: v10.9.0
Local ESLint version: v9.16.0 (Currently used)
Global ESLint version: Not found
Operating System: darwin 24.1.0
What problem do you want to solve?
I'd like the plugin to work for CSS-in-JS solutions like styled-components, emotion and linaria.
What do you think is the correct solution?
In stylelint, you can use PostCSS parser for that purpose. Here is an example: https://github.com/hudochenkov/postcss-styled-syntax
So I guess https://github.com/eslint/css/issues/29 is related
Participation

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

Additional comments
No response

@rakleed please explain what you mean by "support CSS-in-JS". Some examples would be helpful. (I'm not sure what exactly I'm supposed to be looking at in those links. Because I don't use CSS-in-JS, I'll need you to provide as much context as possible.)

This means that you can write CSS directly in JS files. See examples:

https://github.com/styled-components/styled-components#example
https://emotion.sh/docs/introduction#framework-agnostic
https://github.com/callstack/linaria#syntax

So you need to check the JS files, find such CSS code in template literals, and lint it.

In stylelint, you can use PostCSS parser for that purpose. Here is an example: https://github.com/hudochenkov/postcss-styled-syntax
So I guess #29 is related

Ah gotcha. This would actually be a multi-step process to get this working for ESLint.
First, we'd need to finish up work on this RFC:
https://github.com/eslint/rfcs/pull/105
That would give ESLint the ability to pick out specific parts of a JavaScript file and mark it as CSS. ESLint would then be able to present all of the CSS and JS problems in one file.
The problem of interpolated values is one I'm not sure how to address.
I'll leave this issue open to track this request. Just be aware it's a non-trivial amount of work so likely won't be completed anytime soon.


Parsing into a concrete syntax tree.

Hello Lukas,
I was wondering what your opinion is on adding support for parsing xml documents into a concrete syntax tree i.e. an AST that contains location information such as the source location of equals signs in attributes and other syntactical elements.
Is this perhaps on your TODO list? if not, would you ever accept any PRs for that, even if it made use-cases that don't need a CST slower, or should a to-CST-parser exist independently of the to-AST-parser to not make the to-AST-parser slower?

The event based parser (which is now used for all parsing operations) has an option to keep track of source locations of individual events. While this does not allow to keep track of exact positions of each token, it gives enough information for things like error reporting, printing the location of elements, or even some basic syntax highlighting.
Speed and memory consumption are a general concern. There are users that want to be able to process GBs of XML data on mobile devices. So yeah, I am very careful of adding features that are not generally useful but that come at a high cost for everybody.
I played around propagating the location information from events to the DOM nodes, which is relatively easy to do in the current setup. However, I didn't pursue the idea further due to the lack of a strong user-case and the question what would happen with the location data if the DOM was mutated?

Thank you for the response. Given the requirement that GBs of data need to be parsed, forcing all users to parse into a CST sounds like it would definitely introduce an unacceptable amount of overhead.


website has a black background at start

The background color of the app is currently default, which leads to a really awkward white screen while UI loads over it. This PR makes it black, which vibes nicer with the coming app loading screen etc.

Wasn't able to reproduce on my chrome. I think this is probably a fine edge case, it's a small aesthetic detail and one line of css. The other options around  this seem to be painful.


[improve](stream load) set NumberLoadedRows to zero when stream load transaction failed (#41946)

pick (#41946)
Set NumberLoadedRows to zero when stream load failed.
before:
stream load result: {
    "TxnId": 8589,
    "Label": "c8e7c4fe-56b2-4e3b-b4cc-4f2a94cdd003",
    "Comment": "",
    "TwoPhaseCommit": "false",
    "Status": "Fail",
    "Message": "[DATA_QUALITY_ERROR]too many filtered rows",
    "NumberTotalRows": 3,
    "NumberLoadedRows": 1,
    "NumberFilteredRows": 2,
    "NumberUnselectedRows": 0,
    "LoadBytes": 77,
    "LoadTimeMs": 78,
    "BeginTxnTimeMs": 0,
    "StreamLoadPutTimeMs": 4,
    "ReadDataTimeMs": 0,
    "WriteDataTimeMs": 72,
    "ReceiveDataTimeMs": 7,
    "CommitAndPublishTimeMs": 0,
    "ErrorURL": "XXX"
}

now:
stream load result: {
    "TxnId": 8589,
    "Label": "c8e7c4fe-56b2-4e3b-b4cc-4f2a94cdd003",
    "Comment": "",
    "TwoPhaseCommit": "false",
    "Status": "Fail",
    "Message": "[DATA_QUALITY_ERROR]too many filtered rows",
    "NumberTotalRows": 3,
    "NumberLoadedRows": 0,
    "NumberFilteredRows": 2,
    "NumberUnselectedRows": 0,
    "LoadBytes": 77,
    "LoadTimeMs": 78,
    "BeginTxnTimeMs": 0,
    "StreamLoadPutTimeMs": 4,
    "ReadDataTimeMs": 0,
    "WriteDataTimeMs": 72,
    "ReceiveDataTimeMs": 7,
    "CommitAndPublishTimeMs": 0,
    "ErrorURL": "XXX"
}

run buildall

run buildall

run buildall


fix: add more exports

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


ShouldBeEncrypted job contract not working as expected

Laravel Version: 8.23.1
PHP Version: 7.4.1
Database Driver & Version: MariaDB 10.5.8

Description:

PR #35527 introduced Job Encryption, and its code was tidied up later in f80f647852106942e4a0ef3e9963f8f7a99122cf.
I tried adding ShouldBeEncrypted to two separate database job classes — an event listener and a Notification — both of which implement ShouldQueue.
The jobs are added to the database, but the command property is not encrypted.

Steps To Reproduce:

Open src/Illuminate/Queue/Queue.php in your IDE
Set a breakpoint at line 154
You will see that $job instanceof ShouldBeEncrypted evaluates to false, and the job is not encrypted.

The $job property appears to be an instance of Illuminate\Queue\CallQueuedHandler@call when entered in to the database. I'm not sure why this would be, as the queue handler internals are a little out of my depth... any clues would be appreciated. Thanks in advance.

This currently works only with jobs dispatched from the bus Job::dispatch() or dispatch(new Job). It doesn't work with events or notifications because I thought these records won't hold any sensitive data.

This currently works only with jobs dispatched from the bus Job::dispatch() or dispatch(new Job). It doesn't work with events or notifications because I thought these records won't hold any sensitive data.

@JackWH we'll make this work with other queueables soon.

@JackWH we'll make this work with other queueables soon.

issue was resolved here, https://github.com/laravel/framework/pull/36036

issue was resolved here, https://github.com/laravel/framework/pull/36036


Enable full survey access

@ajstewart this is a big one, but it needed to be because of all the changes in the full survey data specs.
Specifically,

We no longer use "epochs". The terminology is retained in all VAST software to save the hassle of changing everything, but they're more like groups than epochs. Specifically, each epoch no longer necessarily contains all fields, and each field can be observed multiple times in one epoch. The reason for this is that the pilot observations were scheduled such that VAST monopolised the telescope for as long as required to observe all fields, which is extremely inefficient and therefore not feasible for the full surveys. Instead, we're planning on redefining epochs to be all observations on a given day (or maybe week).
The field names no longer have an "A" on the end, which was a leftover from when observations were potentially going to be interleaved. Allowing access to data that uses both naming schemes is not trivial, but we can now handle it.


Fix identity panel for project_admin

We are enabling a custom keystone policy file for horizon because
the horizon keystone policy and keystone policy need to be slightly
different when supporting the project admin. The horizon keystone policy
file requires project admin to be part of the admin required rule.
The project_id was also being used in the policy where project scope
did not matter. This has been corrected.
This fixes a bug where horizon fails to load parts of horizon
if the user has the project_admin role.

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

ok to test

identity:validate_token and identity:validate_token_head were also previously unscoped and may need to be changed as well

retest

retest

Also this should be independent of the policy_for_horizon boolean. We will need the keystone_policy.json to exist regardless of customized horizon in order for the Identity panel to work.

@amwong are you stating that this PR needs more work?

It looks like zfeldstein protected the policy_for_horizon variable here in this commit:
https://github.com/blueboxgroup/ursula/commit/9d12eee58b62e353a9345b57602032596b3a9957
It looks like he fixed the horizon panel for cloud_admin and put a feature flag for horizon customization. I think project_admin would fall under this. IMO if the customization feature flag needs to be removed, it should be in a different commit and we should ask zfeldstein if we should remove the horizon customization feature flag altogether.

retest

retest

retest

I'm going to merge, we can adjust after.


Connection with administrator user impossible

Entered to /scp/login.php use the docker-compose values:
ADMIN_USER=deleted
ADMIN_PASS=deleted
What is elsewhere? Or other data?
The user has 5 characters, I also tried the ADMIN_EMAIL

Change the ADMIN_USER for another one with more characters since it only had 5 and it started to work.


Android Icon doesn't update

I change my icon.png... When I run whith android CLI I keep old icon... On ios I have the new one.
I tried to delete platforms folders and re install but I keep the old bad icon...
I forget something?

Yes, correct, This is the issue I'm facing as well. @AlexDisler can you take a look at this issue?

I'm also experiencing the issue in iOS - I ran the script once, edited icon.png and splash.png but neither updated.

Me too. Update the master icon.png, rerun and the output looks OK but nothing is updated.
Even purging all the files first (find www/platforms/ -type f -name '*png' -exec rm {} \;) does not work. Something must be checking for the folders and skipping the work if they are there ?

if you're using a cordova version lower than 6.0.0 try:
npm i -g<EMAIL_ADDRESS>

I'm having a similar problem since updating to the new version.
I could run before with the single icon.png and things worked fine for IOS/Android.
Now when running I see the first 'Checking Project & Icon' heading and all is ok. Then all I see is 'Generating Icons for ios' and the process exits.
I have imagemagick installed and have tried using both the seperate icon-android/ios.png files and single icon.png.
Any help would be appreciated!

Seems like my error is related to a problem trying to create the folders to export icons to.
iOS output is targeting /Images.xcassets/AppIcon.appiconset/
And the line https://github.com/AlexDisler/cordova-icon/blob/master/index.js#L171
is failing trying to do recursively create the the nested folder structure..
if i get time i'll have to do a pull request :)
you could use https://www.npmjs.com/package/wrench or something...

Should be resolved by https://github.com/AlexDisler/cordova-icon/pull/69


Davidson algorithm should allow sparse matrix input

Currently it only works for LinearOperators, but it should also work for sparse matrices, right?

Absolutely, mostly within 20 iterations for dimension at the order of 10^6 or even higher.
Actually it works for not only sparse matrices, but also for more general matrices when they're diagonal dominant, so that the approximation (H - \lambda I)^{-1} \approx (D_H - \lambda I)^{-1} is valid.
-- Seems no easy way to insert equations in discussion :(

Davidson's key inputs are:

linear operator (scipy.sparse.linalg.LinearOperator);
linear operator's exact diagonal elements (without calculating the whole matrix of course);

With a sparse matrix as the starting point, I'd suggest converting it to a LinearOperator as well, just as the QubitOperator case, and then they can share the same underlying Davidson Algo implementation, which had better be decoupled for clarity.


Provide a way to indicate the CommandContext has been set.

TIL that the Context only gets set after module constructors have run. This means that some of the code I had from the beginning, like this excerpt from MpGameModuleBase, never worked.
protected MpGameModuleBase(TService gameService)
{
    gameService.OpenToJoin.TryGetValue(Context.Channel.Id, out _open); //BANG
}

I already have an idea on a way that would encapsulate this relatively cleanly, PR shortly.

Fixed by #502


Zend\Http\Headers::addHeaderLine() doesn't check MultipleHeaderInterface

[X] I was not able to find an open or closed issue matching what I'm seeing.
[X] This is not a question. (Questions should be asked on chat (Signup here) or our forums.)

Adding a Header not implementing MultipleHeaderInterface via addHeaderLine, will add the Header multiple times. I'd expect it to be/stay unique.
Background: I'm trying to overwrite an already (multiple times) set ContentType Header. Because the Header is set twice via addHeaderLine(), it is present multiple times. But via Headers->get('Content-Type') i only get the first instead of all Content-Type headers. Of cause: because it's not implementing MultipleHeaderInterface. :/
In general i'd expect addHeader() and addHeaderLine() to behave the same way. Or am I wrong? ... Of cause Instanciating the Header for the multiple-check would destrory the "lazy-loading" argument in the docs. :/
If i'm wrong, all ZF-internal Header modifications should use addHeader() instead of addHeaderLine() to prevent this behaviour. In this particular Case Zend\View\Strategy\JsonStrategy adds a Content-Type Header via addHeaderLine() and therefore doesn't re-use/overwrite an already set ContentType Header. May be, this is a bug in zend-view, too.
I'd prefer to implement the "multiple-check" into addHeaderLine().
I'd like to contribute some code but i'm nut sure, what the best/right solution is. ... any thoughts/suggestion?
Code to reproduce the issue
$headers = new Headers();
$headers->addHeader(new ContentType(null, 'text/plain')); // i.e. in Controller
$headers->addHeaderLine('Content-Type', 'text/javascript'); // i.e. in JsonStrategy:134
echo $headers->count().PHP_EOL;
echo $headers->get('Content-Type)->getMediaType();

Expected results
1
text/javascript

Actual results
2
text/plain

This repository has been closed and moved to laminas/laminas-http; a new issue has been opened at https://github.com/laminas/laminas-http/issues/2.


Add back support for Langchain Evalchains

Describe the Feature
with V0.1, support for langchain EvalChain is broken, add that back in

Any ETA? Or some architecture/code scaffolds so at least I can help accelerate this?

hey @leehanchung , you could invoke (or ainvoke()`) it in a loop and create the dataset to evaluate.
refer this notebook.
This might take some time for me to fix -worst case 2 weeks. I apologize for the hard time 🙁


Fixes and Node monitoring

Minor fixes (Documentation, debug line, non-updating variable in Grafana)

New Python server which allows remote monitoring on a distant Prometheus instance. Needs three env variables :
INFRABOX_PORT : on which port open the http server
DISTANT_PROM_ADDR : the http address (and port) of the distant server to scrape from
INSTANCE_LABEL_PREFIX : the prefix to add to the instance prometheus label of scraped metrics

Updated the python server making prometheus metrics from the PostGreSQL DB to add node metrics

Added a Node Inspector Dashboard

I forgot to update old dashboard with the new prometheus metric name


Fails to attach storage with error message 'could not find host by using initiators'

I trying to configure OCP 4.9.11 running on vSphere <IP_ADDRESS>00 utilising NPIV for my compute nodes against a FlashSystem 5030 array using the v1.8.0 CSI driver.  I can successfully create a PVC that provisions a PV and I can see that appear in the storage console.
When I try to attach the PVC to a pod (in this case configuring the openshift-image-registry component) it fails to complete the action .  The volumeAttachment record reports
Status.Attach Error.Message:  rpc error: code = NotFound desc = Host for node: nvme_nqn: , fc_wwns : [''], iscsi_iqn : iqn.1994-05.com.redhat:ca8c232fba  was not found, ensure all host ports are configured on storage
Using 'oc logs -f pod/ibm-block-csi-controller-0 ibm-block-csi-controller -n ibm-block-csi-driver' in my namespace I can see the following errors:
2022-01-20 14:28:40,777 DEBUG   [140349823231744] [SVC:153;60050763808120A550000000000000B5] (array_mediator_svc.py:get_host_by_host_identifiers:705) - could not find host by using initiators: nvme_nqn: , fc_wwns : [''], iscsi_iqn : iqn.1994-05.com.redhat:ca8c232fba
2022-01-20 14:28:40,778 ERROR   [140349823231744] [SVC:153;60050763808120A550000000000000B5] (exception_handler.py:handle_exception:36) - Host for node: nvme_nqn: , fc_wwns : [''], iscsi_iqn : iqn.1994-05.com.redhat:ca8c232fba  was not found, ensure all host ports are configured on storage
Traceback (most recent call last):
 File "/driver/controller/controller_server/exception_handler.py", line 44, in handle_common_exceptions_with_response
   return controller_method(servicer, request, context)
 File "/driver/controller/controller_server/csi_controller_server.py", line 249, in ControllerPublishVolume
   lun, connectivity_type, array_initiators = array_mediator.map_volume_by_initiators(volume_id,
 File "/opt/app-root/lib64/python3.8/site-packages/decorator.py", line 232, in fun
   return caller(func, *(extras + args), **kw)
 File "/opt/app-root/lib64/python3.8/site-packages/retry/api.py", line 73, in retry_decorator
   return __retry_internal(partial(f, *args, **kwargs), exceptions, tries, delay, max_delay, backoff, jitter,
 File "/opt/app-root/lib64/python3.8/site-packages/retry/api.py", line 33, in __retry_internal
   return f()
 File "/driver/controller/array_action/array_mediator_abstract.py", line 19, in map_volume_by_initiators
   host_name, connectivity_types = self.get_host_by_host_identifiers(initiators)
 File "/driver/controller/array_action/array_mediator_svc.py", line 706, in get_host_by_host_identifiers
   raise array_errors.HostNotFoundError(initiators)
controller.array_action.errors.HostNotFoundError: Host for node: nvme_nqn: , fc_wwns : [''], iscsi_iqn : iqn.1994-05.com.redhat:ca8c232fba  was not found, ensure all host ports are configured on storage```

Digging through your code I can see that I'm dying on line 741 of 'ibm-block-csi-driver/controller/array_action/array_mediator_svc.py '.  If I understand the basics of the code correctly, your issuing a 'lshost' on the storage array and then a 'lshost <id>'. and then searching through the result looking for nqn, WWPN or iscsi_name values.  If you find one, you set a connectivity type.  In my case, the code fails to find these keys.

If I run a 'lshost <num>' command I get the following output:
```id 22
name <ocpwork101_fqdn>
port_count 4
type generic
mask<PHONE_NUMBER>111111111111111111111111111111111111111111111111111111
iogrp_count 2
status online
site_id
site_name
host_cluster_id
host_cluster_name
protocol scsi
status_policy redundant
status_site all
WWPN 281C000C29000047
node_logged_in_count 2
state active
WWPN 281C000C29000046
node_logged_in_count 2
state active
WWPN 281C000C29000045
node_logged_in_count 2
state active
WWPN 281C000C29000044
node_logged_in_count 2
state active
owner_id 0
owner_name Openshift

Questions

Why is this failing? I don't understand why it can't see the WWPN values clearly showing.
What does CSI driver do when the VMs are running on vSphere?  Is it going to attach the LUN to the VM using a vSphere RDM disk or something else?

Thanks for any help you can offer in resolving this issue.
SteveP

After digging about how NPIV on VMWare works AND talking to some experts in this space...
VMWare requires all LUN's to be presented using RDM disks.  From within the VM itself, it's not aware that it has fibre connection to the storage array and therefore dynamically created LUNs using the CSI driver can't be mapped into the VM when required.
Closing defect.

Closing defect.


ImportError: cannot import name 'simps' from 'scipy.integrate'

Before posting a bug report

[X] I have searched exisisting GitHub issues to make sure the issue does not already exist.

Expected behavior
import pacakage successfully
Actual behavior
After installation, when doing
from strawberryfields.apps import points, plot,
it reports the import error.
I've checked scipy.integrate, and the current version has simpson instead of simps method
Reproduces how often
100%
System information
File /[python enviroment]/strawberryfields/apps/__init__.py:36
     [34](https://file+.vscode-resource.vscode-cdn.net/opt/miniconda3/envs/py312/lib/python3.12/site-packages/strawberryfields/apps/__init__.py:34) import strawberryfields.apps.plot
     [35](https://file+.vscode-resource.vscode-cdn.net/opt/miniconda3/envs/py312/lib/python3.12/site-packages/strawberryfields/apps/__init__.py:35) import strawberryfields.apps.points
...
---> [27](https://file+.vscode-resource.vscode-cdn.net/opt/miniconda3/envs/py312/lib/python3.12/site-packages/strawberryfields/backends/states.py:27) from scipy.integrate import simps
     [29](https://file+.vscode-resource.vscode-cdn.net/opt/miniconda3/envs/py312/lib/python3.12/site-packages/strawberryfields/backends/states.py:29) from thewalrus.symplectic import rotation as _R
     [30](https://file+.vscode-resource.vscode-cdn.net/opt/miniconda3/envs/py312/lib/python3.12/site-packages/strawberryfields/backends/states.py:30) from thewalrus.symplectic import xpxp_to_xxpp

ImportError: cannot import name 'simps' from 'scipy.integrate'

Source code
No response
Tracebacks
No response
Additional information
No response

Hi @wangqiuoe ,
Thank you for opening this issue. I'm not being able to replicate your problem. It looks to me like an installation issue.
I recommended that you create a new virtual environment to avoid any installation issues. You can create a virtual environment with Conda and install Strawberry Fields as follows:

Install Miniconda following the instructions here.
Open your terminal (mac) or command line (Windows).
Create a new Conda environment with: conda create --name <name_of_your_environment> python=3.10
Activate the environment with: conda activate <name_of_your_environment>
Install Strawberry Fields with: python -m pip install strawberryfields
Install other useful packages with: python -m pip install jupyter matplotlib

Note that you will be installing 3 packages here: Strawberry Fields, Jupyter, and Matplotlib. Also, note that where it says <name_of_your_environment> you can choose any name that you want.
If you're still having trouble in your newly created environment please post here the output of sf.about()
import strawberryfields as sf
sf.about()

Please let us know if this resolves your issue!

Hi @CatalinaAlbornoz
I have followed your method and it still ends at the same point as before
`Traceback (most recent call last):
File "/home/qcl/Downloads/strawberryfields-master/examples/gate_teleportation.py", line 2, in
import strawberryfields as sf
File "/home/qcl/anaconda3/envs/sf/lib/python3.10/site-packages/strawberryfields/init.py", line 24, in
from . import apps
File "/home/qcl/anaconda3/envs/sf/lib/python3.10/site-packages/strawberryfields/apps/init.py", line 36, in
import strawberryfields.apps.qchem
File/home/1/anaconda3/envs/sf/lib/python3.10/site-packages/strawberryfields/apps/qchem/init.py", line 34, in
import strawberryfields.apps.qchem.dynamics
File/home/qcl/anaconda3/envs/sf/lib/python3.10/site-packages/strawberryfields/apps/qchem/dynamics.py", line 73, in
import strawberryfields.utills import operation
File "/home/qcl/anaconda3/envs/sf/lib/python3.10/site-packages/strawberryfields/utils/init.py", line 21, in
from .program_funstions import *
from import File/hone/qcl/anaconda3/envs/sf/lib/python3.18/site-packages/strawberryfields/utils/program_functions.py", line 28, in
fron strawberryfields.engine import LocalEngine
File "/home/qcl/anaconda3/envs/sf/lib/python3.18/site-packages/strawberryfields/engine.py", line 29, in
from strawberryfields.device import Device
File "/home/qcl/anaconda3/envs/sf/lib/python3.10/site-packages/strawberryfields/device.py", line 23, in
from strawberryfields.io import to_program
File "/home/qcl/anaconda3/envs/sf/lib/python3.10/site-packages/strawberryfields/io/init.py", line 26, in
from strawberryfields.program import Program
File " /home/qcl/anaconda3/envs/sf/lib/python3.18/site-packages/strawberryfields/program.py", line 62, in
from strawberryfields.compilers import Compiler, compiler_db
File "/hone/ /qcl/ /anaconda3/envs/sf/lib/python3.10/site-packages/strawberryfields/compilers/init.py", line 40, in
from .xcov import Xcov
File "/home/qcl/anaconda3/envs/sf/lib/python3.10/site-packages/strawberryftelds/comptlers/xcov.py", line 25, in
import strawberryfields.ops as ops
File "/home/qcl/anaconda3/envs/sf/lib/python3.10/site-packages/strawberryfields/ops.py", line 33, in
from .backends.states import BaseFockState, BaseGaussianState, BaseBosonicState
File "/home/qcl/anaconda3/envs/sf/lib/python3.10/site-packages/strawberryfields/backends/init.py", line 72, in
from .gaussianbackend import GaussianBackend
File "/home/qcl/anaconda3/envs/sf/11b/python3.10/site-packages/strawberryflelds/backends/gausstanbackend/init.py", line 16, in
from .backend import GaussianBackend
"/home/qcl/anaconda3/envs/sf/lib/python3.10/site-packages/strawberryflelds/backends/gausstanbackend/backend.py", line 23, in
from strawberryfields.backends.states import BaseGaussianState
File "/home/qcl/anaconda3/envs/sf/lib/python3.10/site-packages/strawberryfields/backends/states.py", line 27, in
from scipy.integrate import simps
ImportError: cannot import name 'simps' from 'scipy.integrate' (/home/qcl/.local/lib/python3.10/site-packages/scipy/integrate/init_.py)`
I have python==3.10

Hi @umar1kfarooq-king  ,
You can try using Google Colab. This would be the recommended option.
If you need to install locally:
If the local Conda instructions don't work you can try creating the virtual environment with venv.
The document attached shows the instructions on how to create the virtual environment and install PennyLane. In this case instead of installing PennyLane you can install Strawberry Fields.
PennyLane installation instructions 2024.pdf
I hope this helps.

@CatalinaAlbornoz
I just installed strawberryfields and facing same issue with import.
I'm using Python 3.10

I checked scipy, they don't have a function scipy.integrate.simps (at least not anymore). The function called scipy.integrate.simpson:
https://docs.scipy.org/doc/scipy/reference/generated/scipy.integrate.simpson.html

OK. scipy deprecated simps
https://github.com/scipy/scipy/issues/21070

Hi @ZiadHatab,
Thank you for confirming the deprecation of simps. I'll see what we can do on our end. In the meantime please install a version of SciPy older than 1.14


Plot Negative Stages

Currently, we cannot plot negative stages for the frequency function. However, places like New Orleans are plenty negative, so this is a reasonable feature.
       A related issue is that we need to be able to plot negative stAGES

Originally posted by @rnugent3 in https://github.com/HydrologicEngineeringCenter/HEC-FDA/issues/1025#issuecomment-1850718077

e.g.

also

The Y Axis is log, which doesn't like negatives. Poor design on my part makes this not super easy to swap out for a linear on the fly when we swap to stages. This can be fixed. It'll just take some rethinking.


Cannot be generated automatically

I use Express.
My router is written down
router
  .put('/xxx', () => {})
  .post('/xxx', () => {})
  .post('/xxx/:id', () => {})

I found that writing like this is invalid.
If you change it like this, it will work
router.put('/xxx', () => {})
router.post('/xxx', () => {})
router.post('/xxx/:id', () => {})

so...
Can't use the first way of writing?

Hi, I'm not a maintainer for this code base, but I faced a similar issue before.
There is a closed issue for this one and the latest version should resolve it. I suggest you to install the latest version.

Hi hsiangfeng!
As pratikIT95 said, this has been fixed in the latest version. Update with:
npm install --save<EMAIL_ADDRESS>

@pratikIT95 & @davibaltar
Thanks for answering. : )
But.
I am using the latest version as invalid > swagger-autogen": "^2.1.0",
Here is my example
example
can be seen
router/index.js
and
router/user.js
is different writing methods.
But it can only output router/user.js
Is my setting incorrect?

Hi hsiangfeng. I identified the problem. It occurs for two factors.

As you are working with routes, you need to pass the 'root' of your route as your endpoint file, which in your case is the app.js file. Then your swagger.js will look like this:

const swaggerAutogen = require('swagger-autogen')();
const outputFile = './swagger_output.json';
const endpointsFiles = ['./app.js'];
swaggerAutogen(outputFile, endpointsFiles)

Through your example it was possible to identify a bug in lib .. :( to work around the problem for now, in your routes/index.js file, do not break the line of the first method after the "router", that is, instead of doing:

router
.get (...
.post (...
.put (...
do it:
router.get (...
.post (...
.put (...
Here it worked! This "breaking the line" bug after the "route" I will resolve here as soon as possible. Try it there and see if it works.
Thank you for reporting this issue!

@davibaltar oh! is working!
Thanks a lot too you library, I really like it. : )
I look forward to its correction.

Thanks hsiangfeng! If you have more problems, report it to us.
I published now the version 2.1.1 to fix this bug.
I will keep this issue open for a while if someone has the same problem and has an old version.

Thanks @davibaltar! new version is working!
If it is changed to this, it will be invalid
router
  .use(middleware)
  .put('/xxx', () => {})
  .post('/xxx', () => {})
  .post('/xxx/:id', () => {})

But it works if it is like this
router.use(middleware)
  .put('/xxx', () => {})
  .post('/xxx', () => {})
  .post('/xxx/:id', () => {})

Hi @hsiangfeng. Thanks for letting me know. Really I forgot to verify the "use" case. In the next version I will publish with this correction. Any other problem, let us know. Thanks!

Hi @hsiangfeng. The version 2.2.1 fixed this. If the problem persists in your project, please open a new issue with more details, please.

Hi @hsiangfeng. The version 2.2.1 fixed this. If the problem persists in your project, please open a new issue with more details, please.


Vitals app

Created vitals app
Vital service is present in registration and clinical app

Coverage decreased (-0.08%) to 13.408% when pulling ddca23384aef87fb70ef5646b14e49aecb6dff1f on vitals-app into edf5423ef0012cdd498a1ad7ead6b891ec5853e0 on master.


Rake chewy:sync should work for complex "outdated_sync_field" values

Expected behavior
If an outdated_sync_field do not have a simple value, it should still be possible to use chewy:sync on that index.
Actual behavior
If one defines an index with an updated_at similar to this one:
    field :updated_at, type: :date, value: ->(obj) do
      [obj.updated_at, obj.obj2.updated_at].compact.max
    end

Then chewy:sync will fail because inside supports_outdated_sync?:
        def supports_outdated_sync?
          updated_at_field = root.child_hash[outdated_sync_field] if outdated_sync_field
          !!updated_at_field && updated_at_field.value.nil?
        end

The condition updated_at_field.value.nil? will always be false since a proc is being used.
Steps to reproduce the problem
Run rake chewy:sync on a index that defines the value of updated_at as a proc.
Version Information
Share here essential version information such as:

Chewy version 7.3.4
Elasticsearch version
Ruby version 3.3
Rails version 7.0

I would gladly jump on this if a maintainer could give me a few pointers to start with.

@sasha370 @AlexDavidoiu


[ux] infer chat template for local models

https://www.reddit.com/r/LocalLLaMA/comments/1fkogmk/comment/lo0qrzq/?utm_source=share&utm_medium=web3x&utm_name=web3xcss&utm_term=1&utm_content=share_button

it is fixed in #88


Elixir ExUnit: Not a Test File

Sorry in advance, I know you receive these questions all of the time, but I simply cannot figure out how to get vim-test to run my test suite when I run the command in a buffer that is not a test file. I am using Elixir and ExUnit. All my tests are in the following format (is this the correct regex???):
 let test#elixir#exunit#file_pattern = '*_test\.exs'
I am using vim-rooterhttps://github.com/airblade/vim-rooter so that I can make fzf search in my project root (and not just the pwd). I thought it might be interfering with vim-test's need to know the pwd, but I removed it and still had the issue.
Thanks in advance - vim-test really is fantastic!

What is the name of the file you are trying to run tests for?

That would help, wouldn’t it?
physics_test.exs

Roger

On Nov 12, 2017, 9:36 AM -0500, Mathew Attlee<EMAIL_ADDRESS>wrote:
What is the name of the file you are trying to run tests for?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/janko-m/vim-test/issues/246#issuecomment-343741500, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAl76W8x0c3Tcpeh3CZKk3UrvjVW1yFGks5s1wJUgaJpZM4Qa5uR.

That should work. Are you running TestFile when that has focus on the buffer, or are you trying to run it from physics.exs. If the latter you need to set up projectionist alternate mappings, I've added this to the documentation #247

Note that vim-test uses regexes, not globs, so '*_test\.exs' won't work because that's not a valid regex, you would need to use '.*_test\.exs'. Also, note that by default vim-test uses its own regex mechanism which is similar but in many cases different than the POSIX standard, the POSIX standard can be turned on via \v. Note that the pattern also doesn't have to match the entire file, so you can remove the .*, and you should also anchor the end. With all this in mind, the following should work:
let test#elixir#exunit#file_pattern = '\v_test\.exs$'

@janko-m should we update the default pattern? I assumed it was fine because it passes the specs

@codeinabox Yes, I think we should add \v to all patterns, because I cannot think of any reason we would want the regexes to behave in the way we're all used to 😃

@codeinabox Node that in Vim regexes, characters . and $ still behave the same as in POSIX regexes, so that's why it works. But for example ( characters are taken literally in Vim regexes, while \( would be the special character which creates a regex group; in POSIX standard it's the opposite.

This should now be addressed in master with #248


Rasa interactive duplicates some slot events

Rasa version: 2.0.0
Rasa SDK version (if used & relevant):
Rasa X version (if used & relevant):
Python version: 3.7.0
Operating system (windows, osx, ...): osx
Issue:
When using rasa interactive cli to generate training stories, some slot set events are duplicated in the generated stories like this:
- story: interactive_story_2
  steps:
  - intent: find_restaurant
  - action: restaurant_form
  - active_loop: restaurant_form
  - slot_was_set:
    - requested_slot: cuisine
  - slot_was_set:
    - cuisine: french
  - slot_was_set:
    - cuisine: french
  - slot_was_set:
    - requested_slot: num_people
  - intent: inform
    entities:
    - num_people: '2'
  - slot_was_set:
    - num_people: '2'
  - action: action_deactivate_loop
  - active_loop: null
  - slot_was_set:
    - requested_slot: null
  - action: utter_french_indian_other

Error (including full traceback):
No error, just duplicated slot events in the generated training stories.
Command or request that led to error:
rasa interactive

Content of configuration file (config.yml) (if relevant):

Content of domain file (domain.yml) (if relevant):

Thanks for the issue, @amn41 will get back to you about it soon!
You may find help in the docs and the forum, too 🤗

Have the same issue #7476

https://rasa.com/docs/rasa/playground The example on the playground page have the same issue.

https://rasa.com/docs/rasa/playground The example on the playground page have the same issue.

when rinning interactive mode, this happens:

The form asks for the email also after it is given, but automatically overrides the extracted entity with the complete user input


SSLlabs is hanging when "Certificate name mismatch" OR no SSL available

The SSLlabs scanner hangs when the result is "Certificate name mismatch " or when "SSL is not available".

I think this was fixed when we moved to the API


Pr/make web worker friendly

With this change fetchEventSource can be used from within a web worker

Hello, is there any blockers to merge this PR?

I'd also like this PR to be merged, this would be a huge help, thanks!

@vishwam hey just bumping this again, if you have a chance to look at it, thanks!

I'm also very interested in seeing this PR getting merged. In regards to the other open PRs, is this project still in use at microsoft/ Azure? Is it still maintained?

FYI adding
Object.defineProperty(global, 'self', {
    writable: true,
    enumerable: false,
    configurable: true,
    value: global,
});

in index.ts will also make this node-compatible!

Do you have any plans to support the running of service workers?

it seems this project is abandoned, so I have forked it to make it compatible with Cloudflare Workers here and published our forked version on npm at @magiccircle/fetch-event-source-cfworker.
I do not intend to maintain this fork, but dropping a link here in case it's helpful to others!