Fix to maintain correctness when out-of-order ZK updates are received

It is possible that CuratorInventoryManager receives ZK path events which are duplicate and out-of-order. At broker, this corrupts the information about historical server to served segment mapping(when "batch" segment announcer is used) and leads to following errors when queries are sent...
"No partition chunk found for [SegmentDescriptor{interval=2015-01-27T00:00:00.000Z/2015-01-28T00:00:00.000Z, version='2015-01-28T10:37:48.739Z', partitionNumber=43}]! Looks like segments were dropped while queries were still in queue"
Our quick workaround was to keep restarting broker and coordinator periodically to refresh/correct the cache time to time.
This patch should fix the issue (as long as events are not missed and ADD event comes before REMOVE)
We thought of 2 options

Keep a path to version cache inside CuratorInventoryManager (where cache keys are invalidated after configured amount of time, say 60 minutes) and ignore the event if higher version of same path event has been seen already.This would put more memory pressure on the broker and coordinator.
When an event arrives, don't use the cached data from the event but get latest data from ZK directly. Then, even if events come out of order, data used is always latest. This option is easier to implement, does not put additional memory requirement and We chose to use it to solve the problem.

Some evidence from the logs regarding out-of-order and duplicate events...
2015-02-16 11:18:32,412 INFO [ServerInventoryView-0] io.druid.curator.inventory.CuratorInventoryManager - CHILD_UPDATED[2015-02-11T22:01:43.441Z3] with version[3]
2015-02-16 11:19:04,773 INFO [ServerInventoryView-0] io.druid.curator.inventory.CuratorInventoryManager - CHILD_UPDATED[2015-02-11T22:01:43.441Z3] with version[5]
2015-02-16 11:20:09,039 INFO [ServerInventoryView-0] io.druid.curator.inventory.CuratorInventoryManager - CHILD_UPDATED[2015-02-11T22:01:43.441Z3] with version[4]
2015-02-16 11:20:41,420 INFO [ServerInventoryView-0] io.druid.curator.inventory.CuratorInventoryManager - CHILD_UPDATED[2015-02-11T22:01:43.441Z3] with version[5]
2015-02-16 11:20:41,437 INFO [ServerInventoryView-0] io.druid.curator.inventory.CuratorInventoryManager - CHILD_UPDATED[2015-02-11T22:01:43.441Z3] with version[6]
2015-02-16 11:21:13,737 INFO [ServerInventoryView-0] io.druid.curator.inventory.CuratorInventoryManager - CHILD_UPDATED[2015-02-11T22:01:43.441Z3] with version[6]
2015-02-16 11:22:51,089 INFO [ServerInventoryView-0] io.druid.curator.inventory.CuratorInventoryManager - CHILD_UPDATED[2015-02-11T22:01:43.441Z3] with version[7]
2015-02-16 11:23:23,400 INFO [ServerInventoryView-0] io.druid.curator.inventory.CuratorInventoryManager - CHILD_UPDATED[2015-02-11T22:01:43.441Z3] with version[8]
2015-02-16 11:27:08,895 INFO [ServerInventoryView-0] io.druid.curator.inventory.CuratorInventoryManager - CHILD_UPDATED[2015-02-11T22:01:43.441Z3] with version[7]
2015-02-16 11:27:41,591 INFO [ServerInventoryView-0] io.druid.curator.inventory.CuratorInventoryManager - CHILD_UPDATED[2015-02-11T22:01:43.441Z3] with version[9]
2015-02-16 11:27:41,705 INFO [ServerInventoryView-0] io.druid.curator.inventory.CuratorInventoryManager - CHILD_UPDATED[2015-02-11T22:01:43.441Z3] with version[8]
2015-02-16 11:32:32,751 INFO [ServerInventoryView-0] io.druid.curator.inventory.CuratorInventoryManager - CHILD_UPDATED[2015-02-11T22:01:43.441Z3] with version[10]

I'm curious if this is at all related to https://github.com/druid-io/druid/pull/1109

@drcrallen looks like #1109 is solving a different problem.

@himanshug are you saying that curator is not properly updating its cache?

@himanshug can you share which version of ZooKeeper you are using? ZooKeeper should have certain guarantees with respect to ordering, but there also have been some fixes related to message ordering in recent versions.

@xvrl it has been observed on ZK 3.4.5 . Also, I'm not entirely sure if this is a zookeeper issue alone, curator might cause this in corner cases when connection etc goes down and it retries stuff.
so, it is a good idea to not rely on cached data but get latest from zk when event is observed.

@himanshug Thanks for this fix. This particular situation has been a white whale for me. It is extremely difficult to us to reproduce in production but it has occurred. The logs around curator inventory manager were specifically added to catch this problem.

@fjy I think the logs certainly helped as it is so difficult to reproduce and understand otherwise. thanks.

@xvrl @himanshug I think we should file a bug with either Curator or Zookeeper to let them know we have this particular problem.

@himanshug Can we add a comment explaining that this is just a workaround?
Also, it would be great to address the following to make sure we track down the root cause:

Check if this is still an issue with ZooKeeper 3.4.6, there seem to be bugfixes related to child updates in 3.4.6, such as this one
File a corresponding issue with Curator, since PathChildrenCache should not update its cache if it sees an older version.

This seems like a Curator bug, or possibly a ZK bug expressing itself in Curator (probably not ZOOKEEPER-1667 though, since that sounds like it affects watcher kind but not order).
I think we should try to fix Curator rather than working around the problem in Druid. Curator already keeps a path-to-version cache so it should be possible to do @himanshug's first suggestion there (refuse to apply out of order updates) without increasing memory use.

@xvrl I've updated the pull request description.
It is really difficult to test and reproduce it on 3.4.6 though.
also, created an issue with curator https://issues.apache.org/jira/browse/CURATOR-191

@gianm I agree that it should be fixed in curator/zookeeper but it may be OK to have this patch in druid
(1) for safeguarding ourselves
(2) to have some fix while it is being corrected in curator.

@himanshug Are you saying you have observed the problem on 3.4.6 as well, or are you saying that it is hard to test and reproduce on 3.4.5 and therefore you have not tried upgrading yet?

@xvrl we have tried and seen only on 3.4.5 . None of our systems are running 3.4.6 yet. we can't upgrade zookeeper on our druid cluster (due to various other reasons and my team doesn't own the druid cluster in question).
and, it is very difficult to reproduce on a test setup with zk-3.4.6 .

Hey guys, in an attempt to move forward, I propose we merge this PR as it solves a fairly critical problem and open a new issue to investigate the root cause in Curator/ZK. I agree the fix should probably be there, but it does take time to interact with their communities to get the problem resolved. How does that sound to everyone?

According to this thread this may also be an issue with 3.4.6, so I am on board with working around this. I also agree with @gianm that addressing it in curator would be cleaner and may also avoid the added network calls to get the node data every time.
As long as we:

actively seeking a fix / workaround at a lower level, and
add a comment in the code to indicate this is a  temporary workaround

then I am on board with merging this as a temporary fix.
@himanshug would you mind adding a comment and leading the effort to try to submit a patch to curator to address the problem as you described in the first solution you had in mind?

https://github.com/apache/curator/pull/68

@xvrl got pulled into some stuff today and cudn't reply msgs but @cheddar is way faster it seems :)

+1 on @xvrl's comment. I think we can merge this if we comment the code that we should remove the workaround once we update to a curator version with the fix and file a new issue to do the update.

@xvrl @fjy added the comment with curator bug and info that its a temporary workaround.


ModuleNotFoundError: No module named 'dateutil' in v1.0.1

After upgrading to version 1.0.1 from 0.0.56, my existing scripts utilizing the blackduck module break with the following exception:
Traceback (most recent call last):
  File "myScript.py", line 6, in <module>
    from blackduck.HubRestApi import HubInstance
  File "/opt/python-3.9.2/lib/python3.9/site-packages/blackduck/__init__.py", line 2, in <module>
    from .HubRestApi import HubInstance
  File "/opt/python-3.9.2/lib/python3.9/site-packages/blackduck/HubRestApi.py", line 55, in <module>
    from .Utils import object_id
  File "/opt/python-3.9.2/lib/python3.9/site-packages/blackduck/Utils.py", line 9, in <module>
    import dateutil.parser
ModuleNotFoundError: No module named 'dateutil'

Looking through the code, I do see python-dateutils defined in the requirements.txt file, but it doesn't seem like the full set of required packages are defined in the setup.py file, so pip doesn't know to install it.
The obvious workaround is for me to pip install python-dateutils manually when I need to use the blackduck module, but ideally running pip install blackduck would automatically install everything it needs.

Fixed setup.py @gsnyder2007 or @skiyooka - could one of you update pypi?

PyPI is updated with 1.0.2 and I tested pip install blackduck with just pip and setuptools installed and it pulls everything in correctly. Thank you @trv-tsingkof for the report and @AR-Calder for the fix.


inline css is not included after update to 1.0.0-alpha2

If i disable javascript in dev tools, to take a look at the html structure i can see our css is gone.
Before we did the update it was included like this.

This results in a flash when we load the page :(
The css  looks ok after hydration. I have looked at https://github.com/nuxt/nuxt.js/issues/754
Nothing is mentioned about css changes. Can someone help?
Thanks
Arun

I have the same problem. Please look at this.

Hi, have you tried with the 1.0.0-alpha4 ?

It works fine in alpha 4. Thanks!! :)


Problem with RemoteStartTransaction use case (triggering Authorize)

Hello.
I'm currently facing a problem on the RemoteStartTransaction use case, where this method sent by the CSMS triggers another method (AuthorizeRequest) sent by the Charging Station.

CSMS sends a RemoteStartTransaction request to Charging Station and awaits for its response;
Charging Station, when it receives the request, instead of responding to it, sends an Authorize request  to CSMS, and awaits for its response.
CSMS receives that request, generates the response and sends the response - I'm sure of this (used Wireshark to check if the response is in fact sent).
PROBLEM: Charging Station doesn't receive the response, and blocks, because it keeps awaiting for it.

Do you have any idea why this might happen? The callback of the @on(Action.RemoteStartTransaction) is awaiting for the finalization of the Authorize.
If you know how to fix the problem, please inform me.
Best regards
Code Samples:
OCPP Client:
Key notes - globals.control_variables.remote_start_transaction_event is a asyncio.Event and it is set when the OCPP client receives the response of the RemoteStartTransaction (executed by other entity).
async def request_authorize(self, kwargs):
        request = call.AuthorizePayload(
            id_tag=kwargs['id_tag']
        )

        response = await self.call(request)

        return response

@on('RemoteStartTransaction')
async def on_remote_start_transaction(self, **kwargs):
        p = BrokerPayload(
            intent=constants.REMOTE_START_TRANSACTION,
            content=kwargs,
            type=constants.REQUEST
        ).to_dict()

        # send payload to broker, broker will forward it to destination
        BrokerUtils.send(p, log_path)

        # waits for response
        await globals.control_variables.remote_start_transaction_event.wait()

        result = call_result.RemoteStartTransactionPayload(

     status=globals.control_variables.remote_start_transaction_payload['status']
        )

        # reset control variables
        globals.control_variables.remote_start_transaction_event.clear()
        globals.control_variables.remote_start_transaction_payload = None

        # returns respective payload
        return result

CSMS
async def request_remote_start_transaction(self, kwargs):
        request = call.RemoteStartTransactionPayload(
            id_tag=kwargs['id_tag']
        )

        if 'connector_id' in kwargs.keys():
            request.connector_id = kwargs['connector_id']

        if 'charging_profile' in kwargs.keys():
            request.charging_profile = kwargs['charging_profile']

        response = await self.call(request)
        return response

@on('Authorize')
async def on_authorize(self, **kwargs):
        Utils.log_file(file_path=log_path, message="AuthorizeConf sent")
        return call_result.AuthorizePayload(
            id_tag_info={
                "status": "Accepted"
            }
        )

It's solved, I'd to override the ChargePoint.start method, since it makes the reading of websocket messages dependent of the message handler execution. This means that the charge point will only read the websocket messages once the handling of the request is finalized.
charge_point.py
async def start(self):
    while True:
        message = await self._connection.recv()
        LOGGER.info('%s: receive message %s', self.id, message)
        await self.route_message(message)

The solution is quite simple. In the custom charge point, just override the function 'start' to look like this:
async def start(self):
    while True:
        message = await self._connection.recv()
        LOGGER.info('%s: receive message %s', self.id, message)
        asyncio.create_task(self.route_message(message))


models.os.getMaxSatisfyingVersion cannot return a dev image

This is due to semver.maxSatisfying(['2.0.8+rev1.prod','2.0.8+rev1.dev'], '2.0.8+rev1.dev') returning '2.0.8+rev1.prod', so even with maximum specificity you still get given the prod image (presumably because p > d and build metadata is ignored for comparsions)

Fixed and released in 7.0.1


Kernel debugger presence leakage through KUSER_SHARED_DATA

The following will detect a kernel debugger on any system running Windows 2000 or later:
#include <stdio.h>
#include <conio.h>
int main(int argc, char* argv[])
{
    unsigned char b = *(unsigned char*)0x7ffe02d4;
    if ((b & 0x01) || (b & 0x02))
        printf("Kernel debugger detected!\n");
    else
        printf("No kernel debugger detected\n");
    _getch();
    return 0;
}

The headers and CRT functions are only there to make the output readable; if you want you can turn this into a one liner that doesn't use any syscall or even a single DLL import!
So how does it work? Well, 0x7ffe02d4 is actually 0x7ffe0000 + 0x2d4. 0x7ffe0000 is the fixed user mode address of the KUSER_SHARED_DATA structure that contains data that is shared between user mode and the kernel (though user mode doesn't have write access to it). I stumbled upon this struct in the Windows Research Kernel sources, but it's actually part of the DDK and even mostly-commented (some fields better than others). The struct has some interesting properties: (a) its address is fixed and has been in all Windows versions since it was introduced. (b) its user mode address is the same in 32 bit and 64 bit mode. (c) all offsets and sizes are strictly fixed, and new fields are only ever appended or added in place of unused padding space. Hence why this program will work in 32 bit Windows 2000 and 64 bit Windows 10 without recompiling.
I found some much more detailed documentation of the struct at http://www.geoffchappell.com/studies/windows/km/ntoskrnl/structs/kuser_shared_data.htm. Other than that the struct seems to be relatively unknown (at least for anti-debug purposes), I didn't find many references to it other than people trying to use the time fields as jump pads for malware... The 'b' in the program at +0x2d4 is the 'BOOL KdDebuggerEnabled' field. When I found the struct I tested it with
BOOL KdDebuggerEnabled = *(BOOL*)SharedUserData->KdDebuggerEnabled; // exact same code as above

Since I am not a filthy Javascript programmer, I do all my boolean checks with strict type equality to avoid issues with MS's ever-changing giant forest of typedefs that change your program depending on which headers you include. I found that if (KdDebuggerEnabled == TRUE) evaluated to false on both my own machine and a debugged VM, so at first I thought it was just an outdated field kept for compatibility reasons. But closer inspection revealed that the value is actually a bitfield and not a boolean, of which the first two bits are set to 1 if a debugger is attached! So if you do if (KdDebuggerEnabled) like a pleb, the check will work! Microsoft rewarding laziness and bad form again.
After I found out about this (and then afterwards of course found the above link, where the bitfield factoid was already documented...) I tried to nuke the field at both its KM and UM address from a kernel module - only to find that it was immediately overwritten with 0x3 again. Then I remembered I had a kernel debugger and set a HWBP:

0: kd> ba w 1 0xFFFFF780000002D4
0: kd> g
Breakpoint 0 hit
kdbazis!KdReceivePacket+0x75f:
fffff80000ba2bff 8b45e7          mov     eax,dword ptr [rbp-19h]
1: kd> kb
RetAddr           : Args to Child                                                           : Call Site
00 fffff80000ba24bc :<PHONE_NUMBER>000008<PHONE_NUMBER>000001<PHONE_NUMBER>000000<PHONE_NUMBER>000000 : kdbazis!KdReceivePacket+0x75f
01 fffff80002885e1d : fffffa800410c280 fffff8a0003cf890<PHONE_NUMBER>000000 fffffa8004284c30 : kdbazis!KdReceivePacket+0x1c
02 fffff8000288873f :<PHONE_NUMBER>000001 fffff880009f1180<PHONE_NUMBER>000002<PHONE_NUMBER>02625a : nt!KdPollBreakIn+0xec`

(kdbazis.dll is VirtualKD's replacement for kdcom.dll, I recommend it if you want to do kernel debugging in a VM without wanting to kill yourself)
Trying this with kdcom.dll won't work, it seems to go into an infinite loop where it keeps sending itself messages that the breakpoint has been hit, which cause it to write to the breakpoint address, which triggers the breakpoint... etc. But the basic idea is the same regardless of which DLL you use: the debugger interface writes to the address on every KdReceivePacket, and kdcom.dll additionally writes to it in KdSendPacket as well.
The good news is that both DLLs can be patched not to do this with no negative consequences. Since VirtualKD is open source, patching it is left as an exercise for the reader... Patching kdcom.dll is more of a challenge, since it is one of the first modules to be loaded at boot and seems to have stricter code signing requirements than other drivers (test signing is not enough). So I found runtime patching to work best. This is possible from TitanHide and I've written a 'PatchKdCom()' to do this, but no PR yet as the code is quite messy and could use a review at least. Or maybe it's not even interesting enough to warrant adding; I've never seen anyone use KUSER_SHARED_DATA for anti-debug (as far as I'm aware anyway, but I'm definitely going to be setting more HWBPs in the future ;)).
Gist here: https://gist.github.com/anonymous/b5024c25634fc36e699cd9d041224531
Some issues:

Because there are so many almost-but-not-quite-the-same patches, the function is huge for what it really does
All of the repeated memcmp's are basically an unrolled loop that shouldn't have been unrolled. Is there a clean way to do this with the C-style (no STL) C++ you have to use in drivers? The problem is that the patches are of variable size, so you can't make a 2D array with constant dimensions
The patch is one way - if you want the rat behaviour back, you need to reboot :)

Definitely very interesting! I pretty sure this isn't used because most people don't use a kernel debugger to do debug protections anyway but perhaps its an interesting trick to use...

That's true. I figured this might be of interest since TitanHide users are probably more likely to have a kernel debugger attached than most. It is possible to debug user mode applications using a kernel debugger, it's just very cumbersome and rarely gives any advantage over regular ring 3 debugging. But if I am making even minor changes to TitanHide for example, I will always have a debugger attached to the VM, if only to catch the BSODs and to have a usable log window.
Btw, I found that someone else had previously also found out about the magic 0x7ffe02d4, and found that it is actually used in ntdll!RtlIsAnyDebuggerPresent (which I had also never heard of...)
RtlIsAnyDebuggerPresent checks PEB->IsBeingDebugged for the user mode process, and KUSER_SHARED_DATA->KdDebuggerEnabled for kernel mode, and returns true if either one is set.

As for the patching, probably some nifty macros could make the code easier to maintain or maybe even std::array? I think it would be a good addition but at the moment I don't really feel like updating TitanHide (mostly because I stopped using it myself 😄)


Dead Link in README.md

Hi, there is a dead link in README.md for Differential Equations in Data Science "Python Tutorial".
It takes the user to an online jupyter notebook link with a 404 Error.

Should we remove it?

Should be fixed with Pull Request #167

Thanks for the catch everyone!


[CI][Java] "AMD64 manylinux2014 Java JNI" is failed with nonexistent include path

Describe the bug, including details regarding any error messages, version, and platform.
https://github.com/apache/arrow/actions/runs/9087726441/job/24975987957?pr=41658#step:7:1378
FAILED: src/gandiva/precompiled/print.bc /build/cpp/src/gandiva/precompiled/print.bc
cd /build/cpp/src/gandiva/precompiled && /opt/vcpkg/installed/amd64-linux-static-release/tools/llvm/clang-17 -std=c++17 -DGANDIVA_IR -DNDEBUG -DARROW_STATIC -DGANDIVA_STATIC -fno-use-cxa-atexit -emit-llvm -O3 -c /arrow/cpp/src/gandiva/precompiled/print.cc -o /build/cpp/src/gandiva/precompiled/print.bc -isystem -isystem /x86_64-redhat-linux -isystem -lpthread -I/build/cpp/src -I/arrow/cpp/src
clang-17: error: no such file or directory: '/x86_64-redhat-linux'

This is related to #41451.
Component(s)
Continuous Integration, Java

Issue resolved by pull request 41661
https://github.com/apache/arrow/pull/41661


Update to keycloak 19

Release notes: https://www.keycloak.org/2022/07/keycloak-1900-released

the version could be a variable too.

the version could be a variable too.

the version could be a variable too.

Done!


Cant get extraTags working in log4j.xml

Hi there,
I am trying to use the extraTags in log4j.xml but it is not taking affect. Is there an issue with my config or is there a possible bug?
The MDC data is appearing in sentry but under additional data but not on the right as tags
<appender name="sentry" class="net.kencochrane.raven.log4j.SentryAppender">             <param name="dsn" value="https://excluded/" />            <param name="extraTags" value="User,Company,Email,ID" />             <filter class="org.apache.log4j.varia.LevelRangeFilter">                 <param name="levelMin" value="WARN"/>             </filter> </appender>

I don't see anything wrong with the configuration and I was able to replicate the intended behavior using a similar configuration and an adaptation of our log4j example project that included two MDC values.
<?xml version="1.0" encoding="UTF-8" ?>
<!DOCTYPE log4j:configuration SYSTEM "log4j.dtd">
<log4j:configuration xmlns:log4j="http://jakarta.apache.org/log4j/">
    <appender name="sentry" class="net.kencochrane.raven.log4j.SentryAppender">
        <param name="extraTags" value="foo,bar" />
    </appender>
    <root>
        <priority value="debug" />
        <appender-ref ref="sentry" />
    </root>
</log4j:configuration>


Homepage

Get the homepage implemented.

First draft of the home page structure is up.

Looking good.
Will get back to you on banner photo.
The featured trail map should be one in Tahoe or Yose. Could probably use the Bayview Trail Loop is a good one to start with.
http://www.outerspatial.com/trips/1250/edit


Picture display problem

May I ask why this happens to me?

Open up a new issue with a different image if you want this to be reviewed. That content is not appropriate for GitHub.


"heavier" item art

It's the opposite of a lighter. Instead of adding fire to things, it removes fire from things. It has to look pretty heavy. There is no reference art for this made-up tool, but, I think it should look like it functions in an analog way, like a lighter would. That is, it's nothing high-tech. Very much open to interpretation.

you know it kinda reminds me of a fire extinguisher

maybe, but its function is a lot cleaner than a fire extinguisher. it doesn't 'put out' the fire, it removes the fire.

still, a fire extinguisher might be a good starting point for inspiration.

yes, considering fire extinguishers are known for being heavy

Asset: https://drive.google.com/file/d/16fXVPRtgVonW-xMWhidnxbGKivouCv8s/view?usp=sharing
Credit: XKZ


SSL/TLS does not seem to be supported

SSL/TLS query parameters don't seem to be supported. Something like mysql_async::Pool::new("mysql://myuser:password@<IP_ADDRESS>/db?sslMode=VERIFY_CA") would fail because it wouldn't recognize the ssl-mode query parameter. The error looks like:
thread 'tokio-runtime-worker' panicked at 'called `Result::unwrap()` on an `Err` value: UnknownParameter { param: "ssl-mode" }', /Users/caleb/.cargo/registry/src/github.com-1ecc6299db9ec823/mysql_async-0.30.0/src/conn/pool/mod.rs:124:41

It seems like the parsing is just not wired up properly because there is a struct opts.ssl_opts.

Hi.
Regarding SSL query params - not sure about the reason, but support is not implemented, sorry. You'll have to use OptsBuilder for now.
Regarding sslMode - it seem to be a libmysqlclient feature, but mysql_async is a pure-rust driver that doesn't depend on it:
If opts.ssl_opts is given then you should assume REQUIRED + VERIFY_IDENTITY + VERIFY_CA.

with_danger_skip_domain_validation turns off VERIFY_IDENTITY
with_danger_accept_invalid_certs  turns off VERIFY_CA
Otherwise - DISABLED

Thanks for the quick response @blackbeam. Totally fine if this isn't a high priority.

You'll have to use OptsBuilder for now.

This is the fix we ended up using downstream. We manually parse out an SSL query parameters and then manually add the ssl_opts with the OptsBuilder. We then pull the SSL query parameters out so that it doesn't blow up when trying to parse the rest of the address.
fn build_opts(address: &str) -> Result<Opts, mysql_async::Error> {
    let url = Url::parse(address)?;

    let use_ssl = url
        .query_pairs()
        .any(|(k, v)| is_ssl_param(&k) && v.to_lowercase() != "disabled");

    let query_without_ssl: Vec<(_, _)> = url
        .query_pairs()
        .filter(|(k, _v)| !is_ssl_param(k))
        .collect();
    let mut cleaned_url = url.clone();
    cleaned_url.set_query(None);
    cleaned_url
        .query_pairs_mut()
        .extend_pairs(query_without_ssl);

    Ok(OptsBuilder::from_opts(cleaned_url.as_str())
        .ssl_opts(if use_ssl {
            Some(SslOpts::default())
        } else {
            None
        })
        .into())
}

// Using it
let opts = build_opts(address)?;
let connection_pool = mysql_async::Pool::new(opts);

CC @dicej


Remove unused rijndael encryption

Removing unused rijndael encryption, since the cryptography package is already imported and favored over it.
Fixes #464
Checklist

[ ] I have added tests that prove my fix is effective or that my feature
works
[ ] I have added docstrings to newly created methods and classes
[ ] I have optimized the code at least one time after creating the initial
version
[x] I have updated the README.md or I am verified that this
is not necessary
[x] I have updated the readthedocs documentation or I
verified that this is not necessary
[x] I have added a consice human-readable description of the change to
CHANGELOG.md

@igormp thanks!


Upgrade Designer to Kamelets 4.7.0

Describe the solution you'd like
Upgrade Designer to Kamelets 4.7.0
Variant
Web Application
Container Management (if applicable)
None

Hello @mgubaidullin. Can this be assigned to me? I want to get familiar with how karavan works internally and working on this will help me understand it better.

Sorry @Anik-Bardhan but I need to unassign
Usually Kamelet upgrade is quite simple procedure, but this time Kamelet catalog comes with fasterxml.jackson version that is not compatible with the latest Vertx we use in generator.


Update INSTALL.md

add option to reload the shell without exiting.

Thanks for this, but I don't actually recommend sourcing ~/.zshrc multiple times in the same session. exec zsh seems to be a better option, but neither of them are as fool-proof as starting a new session from scratch, so I think that is what I would like to continue suggesting.


"ReferenceError: self is not defined" error occurring in unstable branch

I created a minimal project that just performs the related commonjs requires (https://github.com/metincansiper/cose-bilkent-test) to reproduce this error . In this line (https://github.com/metincansiper/cose-bilkent-test/blob/main/package.json#L13) I am switching the cose-bilkent branch that is used (after refreshing the node_modules). When I am using the unstable branch of cose-bilkent I am seeing the following error:
/Users/siperm/Documents/Workspace/cose-bilkent-test/node_modules/cytoscape-cose-bilkent/cytoscape-cose-bilkent.js:10
})(self, function(__WEBPACK_EXTERNAL_MODULE__281__) {
   ^

ReferenceError: self is not defined
    at Object.<anonymous> (/Users/siperm/Documents/Workspace/cose-bilkent-test/node_modules/cytoscape-cose-bilkent/cytoscape-cose-bilkent.js:10:4)
    at Module._compile (internal/modules/cjs/loader.js:1144:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1164:10)
    at Module.load (internal/modules/cjs/loader.js:993:32)
    at Function.Module._load (internal/modules/cjs/loader.js:892:14)
    at Module.require (internal/modules/cjs/loader.js:1033:19)
    at require (internal/modules/cjs/helpers.js:72:18)
    at Object.<anonymous> (/Users/siperm/Documents/Workspace/cose-bilkent-test/index.js:1:21)
    at Module._compile (internal/modules/cjs/loader.js:1144:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1164:10)

It works fine with the master branch of cose-bilkent.

@metincansiper Can you please verify that this problem is fixed now?

@hasanbalci than you! It is working.


BTable selected-items issues

Describe the bug
BTable and v-model:selected-items has  some issues

when table items changes (for example using a provider property that paginates items)
when selected-items have a pre-selection

for example using  provider that paginates, switching a page will loose the selected color on table.
The primaryKey property solves the table selection issue BUT the select-mode 'single' doesn't work in any case
At the end items inside the BTable are not the same even if they are the same object...
Reproduction
https://stackblitz.com/edit/github-xfblxu?file=src%2Fcomponents%2FComp.vue,src%2FApp.vue
Used Package Manager
pnpm
Validations

[X] Have tested with the latest version. This is still alpha version and sometime things change rapidly.
[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

I'd need a reproduction for both, using a provider and using this pre-selection. I'm confused as to what you're trying to do and what is expected

The given reproduction works after #1877 . The other issues mentioned do not have a reproduction. Closing


[BEAM-2885] Add a Local FS implementation of the Artifact Staging API

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

R: @lukecwik

R: @kennknowles

Coverage decreased (-0.05%) to 69.5% when pulling c31b70a66b30bcce12cbbd352e40ede3a784f052 on tgroh:local_fs_artifact_staging_service into 5dab9e109cffefb69ac6be22b2d902d5579797e1 on apache:master.

Coverage decreased (-0.05%) to 69.499% when pulling 7423b43aeba2fd4072a794122085a7dfb1ac4df4 on tgroh:local_fs_artifact_staging_service into 5dab9e109cffefb69ac6be22b2d902d5579797e1 on apache:master.


Force TLS1.2 connections.  Only accept and make TLS1.2 connections. DO not accept downgrades to any other version/protocol.

This is required in server and client code

[ ] PoC code @cconstab
[x] Root server
[x] Secondary server
[ ] Client  code
[ ] Other places ?

Moved out of backlog whilst we wait for https://github.com/dart-lang/sdk/issues/37173 to be resolved (it looks like an outcome is near).

Now we just need to wait for dart-lang/sdk 8062c4c to find its way into a release. As it's a breaking change I think this might not turn up until 2.15

2.15 is here needs to be tested next!

2.15 is already in production, as it's been carried along by FROM dart in our Dockerfiles
Some quick tests:
If no TLS version is specified then default behaviour is 1.3:
$ openssl s_client -connect 8fe461b7-9d18-5ccf-9e3b-9ffe4cd09571.hornet.atsign.zone:1912 -ign_eof -brief
CONNECTION ESTABLISHED
Protocol version: TLSv1.3
Ciphersuite: TLS_AES_256_GCM_SHA384
Requested Signature Algorithms: ECDSA+SHA256:RSA-PSS+SHA256:RSA+SHA256:ECDSA+SHA384:RSA-PSS+SHA384:RSA+SHA384:RSA-PSS+SHA512:RSA+SHA512:RSA+SHA1
Peer certificate: CN = 8fe461b7-9d18-5ccf-9e3b-9ffe4cd09571.hornet.atsign.zone
Hash used: SHA256
Signature type: RSA-PSS
Verification: OK
Server Temp Key: X25519, 253 bits
@

If we force TLS 1.2 that works fine:
$ openssl s_client -connect 8fe461b7-9d18-5ccf-9e3b-9ffe4cd09571.hornet.atsign.zone:1912 -tls1_2 -ign_eof -brief
CONNECTION ESTABLISHED
Protocol version: TLSv1.2
Ciphersuite: ECDHE-RSA-AES256-GCM-SHA384
Requested Signature Algorithms: ECDSA+SHA256:RSA-PSS+SHA256:RSA+SHA256:ECDSA+SHA384:RSA-PSS+SHA384:RSA+SHA384:RSA-PSS+SHA512:RSA+SHA512:RSA+SHA1
Peer certificate: CN = 8fe461b7-9d18-5ccf-9e3b-9ffe4cd09571.hornet.atsign.zone
Hash used: SHA256
Signature type: RSA-PSS
Verification: OK
Supported Elliptic Curve Point Formats: uncompressed
Server Temp Key: X25519, 253 bits
@

But if we force TLS 1.1:
$ openssl s_client -connect 8fe461b7-9d18-5ccf-9e3b-9ffe4cd09571.hornet.atsign.zone:1912 -tls1_1
CONNECTED(00000003)
140164297622848:error:141E70BF:SSL routines:tls_construct_client_hello:no protocols available:../ssl/statem/statem_clnt.c:1112:
---
no peer certificate available
---
No client certificate CA names sent
---
SSL handshake has read 0 bytes and written 7 bytes
Verification: OK
---
New, (NONE), Cipher is (NONE)
Secure Renegotiation IS NOT supported
Compression: NONE
Expansion: NONE
No ALPN negotiated
Early data was not sent
Verify return code: 0 (ok)
---

or TLS 1.0:
openssl s_client -connect 8fe461b7-9d18-5ccf-9e3b-9ffe4cd09571.hornet.atsign.zone:1912 -tls1
CONNECTED(00000003)
140024612635968:error:141E70BF:SSL routines:tls_construct_client_hello:no protocols available:../ssl/statem/statem_clnt.c:1112:
---
no peer certificate available
---
No client certificate CA names sent
---
SSL handshake has read 0 bytes and written 7 bytes
Verification: OK
---
New, (NONE), Cipher is (NONE)
Secure Renegotiation IS NOT supported
Compression: NONE
Expansion: NONE
No ALPN negotiated
Early data was not sent
Verify return code: 0 (ok)
---

It's a similar picture with the root servers.
@cconstab you had additional items on the original checklist, but I'm not sure how applicable they are, so I think this can be closed.

@cpswan thanks for the testing!
Everything covered as this is now part of the base Dart Language..

testing bot connection


WB making Lookup active giving an error

After creating a lookup and when trying to activate it, WE was giving the following error:
Database error occurred retrieving Views dependent on the Lookup Path.

This should have been fixed by #54. Please update your repo and retest.

Venkat, can you confirm issue has been sorted and close the issue?

Hi Ian. I did fetch upstream to my branch to update the forked repo with the genevaers/wb main repo. Then i did mvn install to build the project. However, it is giving me a build failure saying CopyBookImporter errors


Add a couple of helper methods to DeclBaseName

This is in preparation for a bunch of upcoming PRs to migrate various places over from Identifier to DeclBaseName

@swift-ci Please smoke test

@swift-ci Please smoke test


Memory optimization when using aws s3

The changes here make it so that there are no longer references being
held to the s3 objects allocated during XMLToStruct in the s3 libraries.
In my testing, when doing a listObjectsV2 against ~383,000 objects, this
change saves about 1/2 of the memory uses. (1gb of use without these
changes and ~500mb with them)

Thanks @ebressler . Could you explain why this matters? Esp since because for the most part we don't keep the resulting BlobItemOutput around? Is it because of https://github.com/kahing/goofys/blob/master/internal/dir.go#L514 ?
The reason I am asking is it's likely that other backend would have the same problem too, and it'd be nice to fix it once instead of each backend_* fixing it each time

Hey @kahing ,thanks for the question. I believe the problem is that since memory pointers were used to the original strings, it kept a reference all the way back to the S3 object. Since the inodes get put into the cache and never get purged from the cache unless they are accessed, they reference count never goes to 0. I am sure that the same problem would be there for other backends, but I don't have a way to check that right now easily.  I am pretty sure it is not just that basename slice you point to. However, it could be that plus inside of SetFromBlobItem (https://github.com/kahing/goofys/blob/master/internal/dir.go#L532) that this is happening. I will try making the change there instead and see if that solves it because I didn't like that I had to sprinkle 3 different ways of doing the same thing across the file because the data filling BlobItemOuput was gathered slightly differently in each case.
Anyway, for more detail before you had a chain that looked something like:
inode->BlobItemOuput->ListObjectV2Output->s3.ListObjectsOutput where there was always at least 1 pointer reference to a part of a string in a struct going all the way back.
This keeps the s3ListObjectsOutput around. Showed up very clearly in a pprof.


Help debugging an issue

I know, this is only for issues in regards to the Serenity platform however I have been trying to figure something out for the last 4 hours and can't figure it out. I had no one else to ask, so I'm asking you guys.
I have my site on a test server connected to a test database. I then copy the entire contents of that folder to my production server. I verify that it's still working on the production server with the test server connection. I go to Administration/Role and attempt to edit permissions and I get the following error. I do not get this on the test server.
Does anyone have any idea at all what it could be?

EDIT: I can edit roles when I select a user. However, if I go to edit permissions for that user or role, I get the above error message.
EDIT 2: I have completely removed the Northwind database from everything and it works fine on the test server. It's just the move over to production on that production box where it finds the issue.

I have a feeling that it's something on the server side but just have no idea where it could be.

https://github.com/volkanceylan/Serenity/issues/1599

@volkanceylan ... my friend. Thank you. Just..thank you.


Center horizontally?

I like this implementation you went with here! Listening to cursor events instead of remapping individual bindings is a cool idea
Could add zs and zH to center horizontally in addition to vertically (code). When in 'centering mode' I prefer also have the cursor horizontally centered as well

Glad you like it.
That's an interesting idea. I have text wrap enabled, so horizontal centering wasn't really a forethought for me, but I can look at adding it. I started playing around with it and right now I can have it center horizontally if you move toward the end of the line, moving the text left. Is that what you're thinking?

Yes, I believe so. I changed the normal! zz to normal! zzzszH so when the cursor tries to move to the right hand side of the window, the viewport also moves right (which is the same as the text moving left)

Cool, I'll work on something this weekend. I think I'll make it configurable as well, just so that way people can turn it on and off if they want it or not

Feature added. Update the plugin and enable the feature in the plugin config. Thanks again for the idea!


Make easy_install command less strict (fixes #1405)

Summary of changes
This PR implements the changes proposed in issue #1405:

The --user option is always available. A warning is issued if the user site directory is not available.
The error shown when the install directory is not in PYTHONPATH has been turned into a warning.

Closes #1405
Pull Request Checklist

[ ] Changes have tests
[x] News fragment added in changelog.d. See documentation for details

Discussion
We should probably adapt the message when the install directory is not on PYTHONPATH since it is no longer a hard error. Any opinions?
Also, this is my first PR for setuptools, so I am interested in feedback as well as guidance on how/where to implement the tests. Would TestEasyInstallTest from setuptools/tests/test_easy_install.py be appropriate?

Is there any interest in this PR? According to #1405, there seems to be a consensus about the approach taken here. Since this issue is preventing me from adopting PEP 517/518 projects more widely, I would appreciate some feedback.

I'm interested. Been watching this PR since it was submitted.
I don't have much to add other than the current status is quite a hurdle on our project. The tradeoffs for the workarounds are not great.
For what is worth PR lgtm.

Thank you very much!


feat: Add verify proof calls to private kernels

Ok, it's failing exactly the same as master, this PR is good to merge


Plugin update issue

Hi,
I have a problem with update my plugin. I try to add new file with plugin but I get window with information:
*Gateway time-out Error code 504,
*plugins.qgis.org Host Error,
*What happened?  The web server reported a gateway time-out error,
*What can I do? Please try again in a few minutes.
I get information about errors in Console (Chrome DevTools):
*POST https://plugins.qgis.org/plugins/wody_polskie_wms/version/add/ 504 (Gateway Time-out)
I've been trying to update the plugin for a few days.
Can you help me?
Łukasz Ścisłowski

Hi Łukasz,
I can at THIS moment (when logged in) just upload a plug to
https://plugins.qgis.org/plugins/add/
can you try that too?
You are using a script? Maybe there is something with the api endpoint (@timlinux ?)

Thank you for your very quick reply.
I also tried to update via https://plugins.qgis.org/plugins/add/ but the error is the same. I didn't use a script to update.
@.***
From: Richard Duivenvoorde @.>
Sent: Thursday, April 13, 2023 1:43 PM
To: qgis/QGIS-Website @.>
Cc: Łukasz Ścisłowski (KZGW) @.>; Author @.>
Subject: Re: [qgis/QGIS-Website] Plugin update issue (Issue #1148)
Hi Łukasz,
I can at THIS moment (when logged in) just upload a plug to
https://plugins.qgis.org/plugins/add/
can you try that too?
You are using a script? Maybe there is something with the api endpoint @.***https://github.com/timlinux ?)
—
Reply to this email directly, view it on GitHubhttps://github.com/qgis/QGIS-Website/issues/1148#issuecomment-1506821649, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AWQUJLB7I5SGBAMUTG2NFLTXA7RDLANCNFSM6AAAAAAW46O7GM.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

I'll sent an email to the dev list...

I just uploaded a plugin without issue. Could you retry @LScislowski  ?

update error.pdf

I've also tried on a different browser (Chrome, Edge) and different network infrastructure. Maybe it's a problem with my file? But my plugin file works when I install it directly in the program.

Could you share it here ?

wody_polskie_wms_1_0_2.zip

@dimasciput could you take a look please?

Hi @LScislowski,
It's because the homepage URL in the metadata cannot be reached. Can you please update the link first?

I changed my homepage URL and it works. I'm waiting for approval now.
Thank you very much for your help.


[Add]

Details
Add candy color button which animates on hover effect.
Screenshots

Type of Contribution

[ ] Update to an existing Animated Button
[X] Adding a new Button
[ ] Resolving a bug
[ ] Proposal to the Repository
[ ] Changes related to documentation or README.md
[ ] Other Changes

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue
[X] I am a GSSoC'23 contributor

Its a level-2 task , so please add level-2 tag. Thank You!

Sorry for the inconvinience, but we are closing all the issues! Please open a new issue if you want to contribute anything. Also now you can create a new button from the website itself! You will get points for those too! I will add the instructions on how to create a new button soon and create an empty pr for the same to get points!


No access to command/search history

In normal vim you can press / or : to start a search or a command, and then you can press the up arrow to access previous commands.
This does not work with this extension, it just shows you an empty textbox. This is quite frustrating when your regex contains a mistake and you need to undo and apply it again. Unless you have the foresight to copy it before executing, you'll have to type it again.

readme

It only works once though? Like if I press ctrl+p it gives me the last command but if I press it again, nothing happens.

Okay, it works only if you delete content here

In the command_line.ts:
The "Up" and "Down" finds a command in history that can be prefixed by the current command, this also includes itself.
private onHistoryUp = async (): Promise<void> => {
    // recall older command-line from history, whose beginning
 // matches the current command-line
    await this.neovimClient.input("<Up>");
    ...
};

private onHistoryDown = async (): Promise<void> => {
    // recall more recent command-line from history, whose beginning
 // matches the current command-line
    await this.neovimClient.input("<Down>");
    ...
}

To get the normal history commands without this prefix check, the API calls S-Up and S-Down can be used.
private onHistoryUp = async (): Promise<void> => {
    await this.neovimClient.input("<S-Up>");
    ...
};

private onHistoryDown = async (): Promise<void> => {
    await this.neovimClient.input("<S-Down>");
    ...
}

Neovim Command Line History

My workaround for this issue is:

C-u C-p to (clear content then) go to the previous item
C-u C-n to (clear content then ) go to the next item

I also combined the two commands into one shortcut using the 'multi-command' extension.
{
"key": "ctrl+k",
"command": "extension.multiCommand.execute",
"args": {
"sequence": [
"vscode-neovim.delete-all-cmdline",
"vscode-neovim.history-up-cmdline"
]
},
"when": "neovim.mode == 'cmdline_normal'"
},
{
"key": "ctrl+j",
"command": "extension.multiCommand.execute",
"args": {
"sequence": [
"vscode-neovim.delete-all-cmdline",
"vscode-neovim.history-down-cmdline"
]
},
"when": "neovim.mode == 'cmdline_normal'"
},

I think the issue is not solved, I am not understanding the meaning of "up and down arrow key being prefixed"
It is mapped to another function by neovim/ vscode?
How to work around?
Btw instead of fixing the up and down arrow key.
How about adding command history on top of the list of auto complete suggestions?
Which file I could check to archive this feature?

Very annoying to click C-u C-p instead of up arrow.
Not to mention that C-u scrolls away from current match, then C-p scrolls again.
Trying both vscode-neovim and vscodevim, vscodevim does not have this issue at all.

This is because vscodevim does not support suggestions and has the commandline in the status bar instead of the quickpick window, and vscode has broken the ability to bind arrow keys in the quickpick window. https://github.com/microsoft/vscode/issues/98479

and has the commandline in the status bar instead of the quickpick window

Perhaps it's a better design desicios, that gives the extension full control over the command line.
Another very annoying issue is that C-r doesn't work on the command line. If I want to paste some register as part of the command (very useful for :s for example)

and has the commandline in the status bar instead of the quickpick window

Perhaps it's a better design desicios, that gives the extension full control over the command line. Another very annoying issue is that C-r doesn't work on the command line. If I want to paste some register as part of the command (very useful for :s for example)

+1

I have large monitor and commandline in the statusbar doesn't work for me. But if anyone wants to contribute by introducing a commandline presentation option - welcome! :)

and has the commandline in the status bar instead of the quickpick window

Perhaps it's a better design desicios, that gives the extension full control over the command line. Another very annoying issue is that C-r doesn't work on the command line. If I want to paste some register as part of the command (very useful for :s for example)

Is there a feature request for this? I have been trying to figure out if this is possible for 2 hours now and figure out an ok workaround.
For example, sometimes I want to yank a word yaW and search for it /<C-r>0.
Instead, you yank it to the clipboard registers, i.e. "*yaW or maybe "+yaW and then search for it with /<C-v> . Only issue is that the clipboard registers seem a bit slow in VS Code in Windows 10 (in my machine, at least).

Is there a feature request for this? I have been trying to figure out if this is possible for 2 hours now and figure out an ok workaround.

I started a workaround in https://github.com/vscode-neovim/vscode-neovim/pull/657 but it has a fundamental issue (vscode does not tell the plugin where the cursor is, so it can only append to the end of the box), and a fixable bug (some of the characters are like CR are interpreted directly).

When I press
:<up> I don't get the last command.  Is this a different issue?

I think using the statusbar like vscodevim and nvim itself (bottom of the screen) is way better... There are many issues with this current setup

@darianmorat the tradeoff is no completion menu


feat(metrics): Add source labels to WASM metrics

Description:
This change adds a simple Lua HTTP filter to add the necessary source metadata
for SMI metrics to each request's headers which are read in the WASM
extension and used to tag the generated metrics.

This PR is part of several which together extend Envoy to generate custom
metrics needed to implement the SMI metrics spec (#984).
Here is a map of the PRs as they're currently planned:

(#2479) ref(injector): pass pod object to getEnvoySidecarContainerSpec
(#2488) feat(metrics): add WASM metrics source
(#2503) docs(metrics): Add docs for WASM stats
(#2517) feat(metrics): Add WASM feature flag
(#2546) feat(metrics): Add stats.wasm to proxy config
(#2554) feat(injector): Add name, workload name, workload kind to PodMetadata
(YOU ARE HERE) feat(metrics): Add source labels to WASM metrics
feat(metrics): Add dest labels to WASM metrics
feat(metrics): Add "unknown" for dest labels on local replies
feat(metrics): clean up WASM metrics in Prometheus config
feat(metrics): add flag to enable WASM metrics
tests(e2e): add WASM metrics test

The remaining changes can be found here: https://github.com/openservicemesh/osm/compare/main...nojnhuh:wasm-metrics

Affected area:

New Functionality      [ ]
Documentation          [ ]
Install                [ ]
Control Plane          [X]
CLI Tool               [ ]
Certificate Management [ ]
Networking             [ ]
Metrics                [X]
SMI Policy             [ ]
Security               [ ]
Tests                  [ ]
CI System              [ ]
Performance            [ ]
Other                  [ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?
No

Codecov Report

Merging #2571 (0b2f2f5) into main (3696a57) will decrease coverage by 0.42%.
The diff coverage is 91.22%.

@@            Coverage Diff             @@
##             main    #2571      +/-   ##
==========================================
- Coverage   63.19%   62.77%   -0.43%
==========================================
  Files         152      154       +2
  Lines        7027     7127     +100
==========================================
+ Hits         4441     4474      +33
- Misses       2569     2636      +67
  Partials       17       17

Flag
Coverage Δ

unittests
62.77% <91.22%> (-0.43%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/envoy/lds/connection_manager.go
91.17% <66.66%> (-2.38%)
:arrow_down:

pkg/envoy/lds/response.go
61.01% <87.50%> (+1.01%)
:arrow_up:

pkg/envoy/lds/wasm.go
90.19% <88.23%> (-0.99%)
:arrow_down:

pkg/envoy/lds/ingress.go
83.01% <100.00%> (ø)

pkg/envoy/lds/inmesh.go
67.27% <100.00%> (ø)

pkg/envoy/proxy.go
68.51% <100.00%> (+23.35%)
:arrow_up:

pkg/configurator/validating_webhook.go
70.34% <0.00%> (-9.66%)
:arrow_down:

pkg/endpoint/mock_provider.go
0.00% <0.00%> (ø)

pkg/endpoint/types.go
100.00% <0.00%> (ø)

pkg/certificate/rotor/rotor.go
88.00% <0.00%> (+4.00%)
:arrow_up:

Continue to review full report at Codecov.

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


Fix glyph index for bitmap fonts and glyph positions in the TextEdit.

Fix glyph index for bitmap fonts (#56348).
Fix TextEdit glyph position rounding (follow up #55905, makes TextEdit to use float glyph positions).
To keep glyphs sharp, draw_glyph do rounding in some cases (we probably should use FreeType sub-pixel positioning, since rounding can cause uneven spacing with some fonts/scales). But in general glyph positions can be non-integer because of hinting and oversampling, also there's no need to round positions for MSDF fonts, and it should be taken ito account when calculating next glyph position.

Fixes #56607, fixes #55996, fixes #56348

Thanks!


The build speed

When I change the Kotlin native code , it will cost a lot of time to finish the build task (Xcode)
task named :shared:linkDebugFrameworkIos  will cost about two minutes
any advice to improve the build speed ?

the host is MacBook Pro (15-inch, 2019)

This isn't really in scope of this plugin. Linking speed is about the kotlin compiler itself. It's not perfect right now, but JetBrains is working on it!


how to restrict download permissions  of files to users  in cvat

My actions before raising this issue

[ ] Read/searched (https://github.com/opencv/cvat/issues/4441)
[ ] Searched past issues

I want to restrict the users to download the  jobs from CVAT , and give access to only selected users.
change the export behaviour permissions.
Expected Behaviour
To restrict download to users and only authorised and selected users can download the job/job files
Current Behaviour
all users can download the jobs from dataset

all users can download the jobs from dataset

That is not exactly true. Only staff assigneed to a job (or its task, or its project) can export the job as a dataset.
Generally Open Policy Agent rego files are responsible for permissons (in case with jobs: cvat/apps/iam/rules/jobs.rego), consider changing them according your needs and rebuilding CVAT with -f docker-compose.dev.yml and ```--build`` arguments.


csvParse, autoType, readFileSync, return string keys

when I wrote the code in node as follows, It will return string keys.
my local  csv file :
country,population
China,1439323.774
India,1380004.385
United States of America,331002.647
Indonesia,273523.621
Pakistan,220892.331
Brazil,212559.409
Nigeria,206139.587
Bangladesh,164689.383
Russian Federation,145934.46
Mexico,128932.753
Japan,126476.458
Ethiopia,114963.583
Philippines,109581.085
Egypt,102334.403

run the code to get data from local csv file:
  const data = csvParse(
    readFileSync(join("data", "population.csv"), "utf8"),
    autoType
  );

and the result of data is (return string keys of country):

the correct key name should be country ,  not 'country'
I try logging data[0].country, data[0]["'country'"] , all the result is undefined, I don't know why...
but the result of data[0].population  is good
when I only use csvParse, the problem still happen
  const data = csvParse(readFileSync(join("data", "population.csv"), "utf8"));
  console.log(data);

The confusion probably comes from the way you're printing the array? If the key is indeed 'country', you should see this:

[
  { "'country'": 'China', population:<PHONE_NUMBER> },
  columns: [ 'country', 'population' ]
]

can you try this?
   echo 'console.log({country: "China", population: 1.439})' | node

The confusion probably comes from the way you're printing the array? If the key is indeed 'country', you should see this:
[
  { "'country'": 'China', population:<PHONE_NUMBER> },
  columns: [ 'country', 'population' ]
]

can you try this?
   echo 'console.log({country: "China", population: 1.439})' | node

The confusion probably comes from the way you're printing the array? If the key is indeed 'country', you should see this:
[
  { "'country'": 'China', population:<PHONE_NUMBER> },
  columns: [ 'country', 'population' ]
]

can you try this?
   echo 'console.log({country: "China", population: 1.439})' | node

I try logging data.country, data["'country'"] ,  then result is undefined

@Fil  it works , is I do this

Ah, maybe the initial problem is because your file starts with a BOM (https://en.wikipedia.org/wiki/Byte_order_mark)? This could explain what you see.

Ah, maybe the initial problem is because your file starts with a BOM (https://en.wikipedia.org/wiki/Byte_order_mark)? This could explain what you see.

yes , you are right , thanks ! saved my life!

No, it's something that should be handled by the ingestion process (node's fs, in this case).

No, it's something that should be handled by the ingestion process (node's fs, in this case).

maybe, but if d3.csvParse can make it done would be perfect! I finally found this, also a csv Parser. https://csv.js.org/parse/options/bom/


UAT | Registration | Only two id options

Describe the bug
Apart from Aadhar, the only option for id card is PAN. In addition, Aadhar is available as an option both for OTP verification as well as manual id upload.
To Reproduce
Steps to reproduce the behavior:

Go to 'citizen login page'
Click on 'Register'
Scroll down to 'upload document' -> manual id
See error

Expected behavior
User should get other options apart from PAN for manual verification. Aadhar should not show in manual verification.
Screenshots

Desktop (please complete the following information):

OS: Windows [e.g. iOS]
Browser: Edge [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

master data in solution

@shambhavigoel1002  can you please add the Master data

Completed can be moved for qa testing @Ramu-kandimalla

Hi @Ramu-kandimalla Please move this too

Hi @rajeshcherukumalli  Please verify and confirm in UAT environment

Regarding_Type of ID

Env.UAT

url: https://dristi-kerala-uat.pucar.org/digit-ui/citizen/dristi/home/registration/upload-id

reference link:
-                 Litigant_ https://jam.dev/c/2174f88e-63b8-456a-96ef-d97b9b640ce2
-                  Advocate_https://jam.dev/c/1b92d33f-e090-4e74-88e2-029adcfe30d8
-                  Advocate Clerk_https://jam.dev/c/3cc8b3af-88f6-443b-a2a6-21c8f48c2e3e

Test cases:
1.verify that in 'Type of ID' dropdown, for user all the option are enable.
2.verify that user able to select 'Passport' type of id.
3.verify that user able to select 'Pan Card' type of id.
4.verify that user able to select 'Driving License' type of id.
5.verify that user able to select 'Voter ID' type of id.
6.verify that user able to select 'Aadhar' type of id.
7.verify that selected option by user,visible in 'Type of ID' dropdown.

Hi All This issue is fixed and deployed in UAT


How to understand the shape of "nbr_fea_idx" in the code

I couldn't understand "the indices of M neighbors of each atom"
Could you give me a simple example about the matrix of "nbr_fea_idx".
Thank you!

Thanks for your question. nbr_fea_idx is the indices of the M neighbors for each node. For instance, you have two nodes 0 and 1. Each node is connected to the other and has one neighbor. In this case, M = 1 and nbr_fea_idx = [[1], [0]].

Thanks for your answer.
if node 0 is connected to a new node 2, but node 1 is not. In this case, what M and nbr_fea_idx look like?


Reject a Compose file with unexpected fields

As Compose specification evolves and introduce new fields/features, a Compose implementation MAY just reject a compose file using a higher level version than the one it has been built on.
On the other hand, even if a compose file uses a newer version it doesn't mean it also use recent fields, and implementation would not be able to use it.
Last but not least, as we made version: "3" and alias for "latest" there's no way to guarantee a Compose file do not target a more recent specification than the one being implemented.
So, a Compose implementation need to be able to reject a compose file which uses fields introduced by a newer version of the specification. Schema validation can be used here, but we also can make the code stricter regarding this scenario: mapstructure used internally to map yaml sections into go structs can track unmapped attributes.  We can rely on this to offer informative diagnostic to the end-user.

I'd like to work on this issue as well :)

closing as obsolete, as we removed the version requirement


Pieces Copy button bug

When I click the copy button multiple times, it creates something like this:

Ideally, however many times I copy, it shouldn't keep on wrapping up the code in the markdown code block iteratively.
Maybe fix this by assigning to a new variable instead of this.codeblock? https://github.com/pieces-app/obsidian-pieces/blob/6efbdd9448e1f4df056021f3d44b2ec41f7ddb8a/main.js#L26917

Hey @ajkdrag, sorry for the late reply again!
Is this still still occurring for you on the latest version of the plugin?

Hello @ajkdrag , thank you for bringing this to our attention! I've just pushed a build that should resolve this issue, please let me know if you have any more problems!

Thanks for the fix. It's all good.


Alow multiple projects to deploy

Allow multiple projects to deploy

I've updated the log data bucket policy to include the staging and prod roles now that I've run them on those accounts.


Height-For-Width layouts and widgets

Rudolf Cardinal appears to have a real solution for the Label layout problem.
https://stackoverflow.com/questions/14238138/heightforwidth-label

Rewrite the layouts.
The best approach I've found is to have layouts calculate their children's geometry assuming some sort of standard rectangle (Qt itself often starts with a 640x480 one when drafting a layout); offering height information based on this draft; then, when setGeometry() is called, if the resulting height (based on a potentially new width) doesn't match what the layout previously advertised, re-constraining by calling parent->setFixedHeight().
This allows you to use arbitrary widgets, and HFW widgets only need to support hasHeightForWidth() and heightForWidth() to have the new layouts (and their parent widgets, and any ancestor layouts using this mechanism) adjust their height.
It can lead to some redrawing, but often not too much, as it happens on a per-layout not per-widget basis.

I've put C++ code at http://egret.psychol.cam.ac.uk/code/2017_01_16_qt_height_for_width/ for the following layouts:

BoxLayoutHfw, VBoxLayoutHfw, HBoxLayoutHfw -- replacements for QBoxLayout etc.
GridLayoutHfw -- replacements for QGridLayout.
FlowLayoutHfw -- replacement for Qt's FlowLayout (http://doc.qt.io/qt-5/qtwidgets-layouts-flowlayout-example.html).

and the following widgets:

AspectRatioPixmapLabel -- image maintaining its aspect ratio;
LabelWordWrapWide -- word-wrapping label that tries to use as much horizontal space before it word-wraps.
VerticalScrollArea -- as its name suggests, a vertical scroll area, but one that supports height-for-width cleanly.

Those layout and widgets are at these paths

https://github.com/ucam-department-of-psychiatry/camcops/tree/master/tablet_qt/widgets
https://github.com/ucam-department-of-psychiatry/camcops/tree/master/tablet_qt/layouts

The CamCOPS authors have relicensed those layouts and widgets as LGPL https://github.com/ucam-department-of-psychiatry/camcops/pull/340
So this would be a complete solution to Qt layout problem, but it would require a few steps.

Create a new repository cardinal-qtwidgets for Python bindings to all of those layouts and widgets. LGPL license. Depends on Qt.
Create a new repository pyedifice-cardinal-elements for Edifice Base Elements for all of those layouts and widgets. Depends on Qt and cardinal-widgets and pyedifice. LGPL license.

Would resolve #118


Allow label to be hidden on all fields

Allow label to be hidden on all fields. This is an option in the Visual Options section when editing the config of any field.

thank you @holmesadam, I added the support for hidden labels to all Bolt Pro fields too.


raw_css not working

On panel 0.5.1, this seems to no longer be working:
pn.extension(raw_css="""
.scrolling {
  overflow: scroll;
}
""")
pn.pane.Str('lfdblafdlsa', width=40, height=20, css_classes=['scrolling'])

The scrolling class gets populated on the pane, but the css doesn't seem to be available.

raw_css should be a list, should definitely warn or error though


Autocomplete incorrectly shows in non-templates

CloudFormation format detection doesn't seem to work for Autocomplete suggestions. They show in all YAML files, not just CloudFormation YAML templates. This seems separate from the linter, which correctly doesn't run on non-templates.
Maybe there's a configuration option I'm missing?
Example from an OpenAPI YAML file:

It seems this issue is related to a different extension. Duh, and sorry!

It seems this issue is related to a different extension. Duh, and sorry!


Update PS Script to have consistent use of AZ module

Hi
The following document has a PS script to create keyvault for data export service.  It is a bit inconsistent in its use of AzureRM and AZ modules, can it be updated to AZ only please?
https://github.com/MicrosoftDocs/power-platform/blob/master/power-platform/admin/replicate-data-microsoft-azure-sql-database.md
e.g. change New/Get-AzureRmResourceGroup with New/Get-AZResourceGroup
New/Get-AzureRmKeyVault to New/Get-AZRMKeyVault
Thanks
Sam

@shammyowens
Thanks for the feedback. We've updated the content.


fix(product-actions): use correct uniq function to ensure unique sfa attributes

Summary
Fix a problem for product update actions where duplicate SFA attributes were generated.
Description
If multiple variants have the same SFA attribute, and this somehow changed (added, removed, changed), multiple equal actions were generated.
The problem was that the wrong lodash function was used, which allows to filter out from an array non-unique items.
TODO

Tests

[x] Unit

@PhilippSpo here we go 🙈😅

Coverage remained the same at 95.729% when pulling e9f98a3d008dee949503d95f5f3e7fe0e4475e9a on fix-product-actions-sfa-duplicate-attributes into d04c3790f956c81d389bb57b702cf5dc2209f498 on rewrite-2.0.

Coverage remained the same at 95.729% when pulling e9f98a3d008dee949503d95f5f3e7fe0e4475e9a on fix-product-actions-sfa-duplicate-attributes into d04c3790f956c81d389bb57b702cf5dc2209f498 on rewrite-2.0.

Released in 2.0.0-alpha.44.
/cc @PhilippSpo


No alive nodes found in your cluster

This error maybe in your host bro.. If your using laradock. you should change your elasticquent.config to elasticsearch:9200 or localhost:9200 maybe in these two.

@edrianjansolatorio The same error i am facing on aws ec2
Elasticsearch\Common\Exceptions\NoNodesAvailableException: No alive nodes found
in your cluster in
/var/www/html/vendor/elasticsearch/elasticsearch/src/Elasticsearch/ConnectionPool/StaticNoPingConnectionPool.php:53


Witget issue

iPhone 6s; iOS 12.1.1; JB b41
at witget center all stock witgets (shortcuts, battery, calendar etc.) display message "can not be loaded".

iPhone 6s; iOS 12.1.1; JB b41
at witget center all stock witgets (shortcuts, battery, calendar etc.) display message "can not be loaded".

Please use the issue template and close this ticket, it is useless and won't help any of the devs fixing anything.
Secondly, your problems are  by 99% caused by incompatible tweaks you irresponsibly installed without thinking.
Thirdly, restore RootFS and wait for a full release, public betas  aren't meant for new and inexperienced users.
With regards
Un4givenPG.

Template not used, also didn't follow guidelines.


[lighthouse] add docs for a11y audits

@robdodson here's the rest of the a11y docs... plz review

@kaycebasques you're crushing these!

@ebidel 💪

These are likely false positives on test failures, will investigate today.

@kaycebasques - please add {# wf_md_include #} to the top of query.md. That will prevent many of the tests from failing (#4076).

Thanks for review, @robdodson. Plz see my replies to your comments, I have a couple questions.
I'll wait on @alice's response too, as that may have a biggish impact on the docs

@kaycebasques - whenever this is ready, remove the DO NOT MERGE tag, then merge and let me know it's merged so I can push it live.


Adding the parsing logic to parse the auth config from feature_store.yaml file

Adding the parsing logic to parse the auth config from feature_store.yaml file. Adding basic models.

I am closing this PR and created a new PR due to the issues with the rebase. Please follow the PR further updates and code.
https://github.com/RHEcosystemAppEng/feast/pull/36


compose: Remove draft older than 30 days.

This removes drafts older than 30 days as they become irrelevant
for the user. Those drafts are removed before populating them.
It's difficult to test manually drafts older than 30 days, so I have only one chance to test drafts older than 30 days and it worked. This is the screenshot of object from dev tools (and  GIF  of that draft from my another PR) of the drafts that date backs to 12 June 2017 which get removed after implementation.
Here's after is how it looks after getting removed older.png
And this is how it looks when we don't have any draft. NoDraft.png
Fixes: #7602.
Also, it might worth taking look into #7903, my another PR, in which I've also added timestamps and sorted them according to their timestamp.

@shubhamdhama can you post a screenshot?
@sampritipanda can you give this a careful review?

This function should have some sort of unit tests. A particular issue I'd want the tests to ensure we don't have is cases where the subtraction is in some unit other than days (e.g. seconds!) since that's a classic bug for a feature like this.

@timabbott  Do you mean the case of draft_lifetime to be in seconds? But the logic is only for the cases when draft lifetime is in days(I should add a comment that draft_lifetime is in days), so do I need to give the flexibility to change the logic for seconds/minute case also (which I think is very rare to possible).
Also, drafts.js have a very low node-unit test coverage, Is there any reason why we haven't them, if not, I'll add some tests.

Added delete as a hotkey to delete a draft.

Also, drafts.js have a very low node-unit test coverage, Is there any reason why we haven't them, if not, I'll add some tests.

@shubhamdhama A lot of the testing is done using casper tests in frontend_tests/casper_tests/14-drafts.js and the node tests only test the localStorage handler. Casper tests don't count towards coverage. The casper tests are missing some of the newer features like the timestamp, the pointer and backspace to delete as well. Adding more thorough node tests would probably be a good idea.

Thanks, @sampritipanda, and yes, I'm writing some node-tests for the format_drafts to see the sorting of drafts and remove_old_drafts. And for casper_tests it is difficult to figure out how to test the changes for time stamps but I'll try to create Casper tests for the pointer, backspace changs.

I merged the "delete" hotkey change.  The other one I guess is waiting on tests?

(I would probably not bother with casper tests for this; they have a maintenance cost that is probably too much for this feature).

@timabbott Can you please this https://github.com/zulip/zulip/pull/8398#issuecomment-371139940 doubt.

@timabbott I've added some test for remove_old_drafts. Can you please clear this https://github.com/zulip/zulip/pull/8398#issuecomment-371139940.

@timabbott Do you mean the case of draft_lifetime to be in seconds? But the logic is only for the cases when draft lifetime is in days(I should add a comment that draft_lifetime is in days), so do I need to give the flexibility to change the logic for seconds/minute case also (which I think is very rare to possible).

No, I just want to be sure the code works.

Okay, so I've already added tests for them.
On Mar 12, 2018 09:57, "Tim Abbott"<EMAIL_ADDRESS>wrote:

@timabbott https://github.com/timabbott Do you mean the case of
draft_lifetime to be in seconds? But the logic is only for the cases when
draft lifetime is in days(I should add a comment that draft_lifetime is in
days), so do I need to give the flexibility to change the logic for
seconds/minute case also (which I think is very rare to possible).
No, I just want to be sure the code works.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip/pull/8398#issuecomment-372190808, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AVNVCDgDXjDQWtByGQ4opNPHSq9YEkVpks5tdfkugaJpZM4SHNvA
.

This looks great, merged, thanks @shubhamdhama!  We could probably do better with extending the tests to cover the draft deletion feature (i.e. have one that's 35 days old and check it gets removed).

Should this notice be sticky at the top of the modal? Otherwise it feels like unexpected data loss.

I think it's OK where it is -- you scroll to the top to see what the oldest drafts are, and you learn there.  Making it sticky I think draws too much attention to it at times you don't need it.
(I also think 30 days is sufficiently generous that almost no chat messages still have value after that)

Yep, it is!


support parsing ISO 19115* family metadata into triples

Closing as this is done in https://github.com/DataONEorg/slinky/tree/feature_update_graph_pattern.


Documentation note

Am I terribly confused? when defining "Transfer function", documentation say:
"where n is generally greater than or equal to m (for a proper transfer function)"
Isn't it the other way around?
http://python-control.readthedocs.io/en/latest/conventions.html
Thanks ahead.

Yup, it's backwards.  I think it will make the most sense to use n for the denominator (since n is the usual variable for the number of states).  Marking this as a bug to fix.

This is fixed in the current master (checked on readthedocs).


object mode crash with list comprehension and any()

Reporting a bug

[X ] I am using the latest released version of Numba (most recent is visible in
the change log (https://github.com/numba/numba/blob/master/CHANGE_LOG).
[X ] I have included below a minimal working reproducer (if you are unsure how
to write one see http://matthewrocklin.com/blog/work/2018/02/28/minimal-bug-reports).

weird interaction with max of list comprehension and any of boolean array, either of them works fine alone but combined together will crash numba:
$ cat bug.py

import numpy as np
from numba import jit

@jit
def foo(data):
    # commenting out this line prevents the crash:
    max_len = max([len(m) for m in data['y']])

    mask = data['x'] == 0
    if any(mask):
        print('ahaha')

data = {'x': np.arange(5), 'y': [[1], [2, 3]]}
foo(data)

$ python bug.py
Traceback (most recent call last):
  File "bug.py", line 17, in <module>
    foo(data)
  File "/home/jarno/miniconda3/envs/cropbot/lib/python3.6/site-packages/numba/dispatcher.py", line 367, in _compile_for_args
    raise e
  File "/home/jarno/miniconda3/envs/cropbot/lib/python3.6/site-packages/numba/dispatcher.py", line 324, in _compile_for_args
    return self.compile(tuple(argtypes))
  File "/home/jarno/miniconda3/envs/cropbot/lib/python3.6/site-packages/numba/compiler_lock.py", line 32, in _acquire_compile_lock
    return func(*args, **kwargs)
  File "/home/jarno/miniconda3/envs/cropbot/lib/python3.6/site-packages/numba/dispatcher.py", line 655, in compile
    cres = self._compiler.compile(args, return_type)
  File "/home/jarno/miniconda3/envs/cropbot/lib/python3.6/site-packages/numba/dispatcher.py", line 82, in compile
    pipeline_class=self.pipeline_class)
  File "/home/jarno/miniconda3/envs/cropbot/lib/python3.6/site-packages/numba/compiler.py", line 904, in compile_extra
    return pipeline.compile_extra(func)
  File "/home/jarno/miniconda3/envs/cropbot/lib/python3.6/site-packages/numba/compiler.py", line 367, in compile_extra
    return self._compile_bytecode()
  File "/home/jarno/miniconda3/envs/cropbot/lib/python3.6/site-packages/numba/compiler.py", line 835, in _compile_bytecode
    return self._compile_core()
  File "/home/jarno/miniconda3/envs/cropbot/lib/python3.6/site-packages/numba/compiler.py", line 822, in _compile_core
    res = pm.run(self.status)
  File "/home/jarno/miniconda3/envs/cropbot/lib/python3.6/site-packages/numba/compiler_lock.py", line 32, in _acquire_compile_lock
    return func(*args, **kwargs)
  File "/home/jarno/miniconda3/envs/cropbot/lib/python3.6/site-packages/numba/compiler.py", line 253, in run
    raise patched_exception
  File "/home/jarno/miniconda3/envs/cropbot/lib/python3.6/site-packages/numba/compiler.py", line 244, in run
    stage()
  File "/home/jarno/miniconda3/envs/cropbot/lib/python3.6/site-packages/numba/compiler.py", line 456, in stage_objectmode_frontend
    cres = self.frontend_looplift()
  File "/home/jarno/miniconda3/envs/cropbot/lib/python3.6/site-packages/numba/compiler.py", line 418, in frontend_looplift
    flags=loop_flags)
  File "/home/jarno/miniconda3/envs/cropbot/lib/python3.6/site-packages/numba/transforms.py", line 191, in loop_lifting
    cfg = compute_cfg_from_blocks(blocks)
  File "/home/jarno/miniconda3/envs/cropbot/lib/python3.6/site-packages/numba/analysis.py", line 241, in compute_cfg_from_blocks
    cfg.add_edge(k, target)
  File "/home/jarno/miniconda3/envs/cropbot/lib/python3.6/site-packages/numba/controlflow.py", line 83, in add_edge
    assert dest in self._nodes
AssertionError: Failed in object mode pipeline (step: object mode frontend)

I'm running numba 0.40.0 and 0.25.0 on python 3.6.6

workaround is to put the max of list comprehension into a separate function, this doesn't crash:

import numpy as np
from numba import jit

@jit
def foo(data):
    ml = max_len(data['y'])
    mask = data['x'] == 0
    if any(mask):
        print('ahaha')

@jit
def max_len(xs):
    return max([len(m) for m in xs])

data = {'x': np.arange(5), 'y': [[1], [2, 3]]}
foo(data)

Thanks for the report. I can reproduce. I think the root cause is the same as: https://github.com/numba/numba/issues/3289. Adding forceobj=True as a kwarg in the @jit decorator also "fixes" it.


select an account error

Verify latest release

[X] I verified that the issue exists in the latest chatgpt release

Verify webapp is working

[X] I verify that the ChatGPT webapp is working properly for this account.

Environment details
Node.js v19.3.0
OS : Windows
Browser version : 109.0.5414.119
Describe the Bug
Recently, when i use API with google auth, i get this after adding Email and password :

the api is not working properly even with the latest version, I only get responses for a few requests, after 4 or 5 requests it crashes.
what is the main reason for this, should i change the authentication method ?

Try upgrading to the latest version; we're now using the official API, which no longer uses puppeteer so problems like this should go away completely: https://github.com/transitive-bullshit/chatgpt-api/releases/tag/v4.0.0


DocketReport ERROR_STRINGS "This case was administratively closed" too aggressive.

I think the ERROR_STRING for 'This case was administratively closed' may be causing some docket htmls to be incorrectly flagged as invalid.
Are there any other heuristics that might be used to identify whatever error page this is being used to detect?
Here's are some cases that include the phrase on the docket:
lamd 3:17-cv-01352
nysd 1:16-cv-01697
nynd 9:17-cv-01287
nynd 9:16-cv-00251
nynd 9:17-cv-01331
flmd 8:17-cv-02878
pawd 2:17-cv-01035
pawd 2:17-cv-00020
nynd 9:17-cv-00627
cod 1:17-cv-02042
nynd 9:16-cv-01369
nynd 9:17-cv-00938
ctd 3:17-cv-00518
nynd 9:17-cv-00861
flmd 8:16-cv-00088
ctd 3:16-cv-00890
nynd 9:16-cv-00757
flsd 1:17-cv-21992
nynd 9:17-cv-01051
cacd 2:16-cv-05381
nynd 9:17-cv-00661
cod 1:16-cv-02648
nynd 9:16-cv-00101
nynd 9:17-cv-00710
nynd 9:17-cv-00939
gand 1:18-cv-03187
nysd 1:17-cv-08171
pawd 2:16-cv-01470
nynd 9:17-cv-01216
txnd 4:18-cv-00245
txwd 5:20-cv-01380
txnd 3:16-cv-00643
ilcd 4:16-cv-04127
gand 1:17-cv-00361
pawd 1:16-cv-00050
nynd 9:17-cv-01295
txed 4:20-cv-00168
nynd 9:16-cv-00087
cod 1:17-cv-02174
pawd 2:17-cv-00614
nynd 9:17-cv-00058
flmd 6:16-cv-00486
It looks like this phrase occasionally appears in the body of the entry descriptions:

Interesting, @nadahlberg, thanks for reporting this. Let me see if I can figure out who added it and why.

Looks like the answer is...me! I added it back in 2017 in 8de4670aab9242b2d2c2b514e7c17f65f75ee990.
I think we could make this more precise by tweaking the ERROR_STRING to have asterisks or perhaps to remove the error string altogether.
The string in the test case is:
*** This case was administratively closed.***

Want to try that Nathan? If not, I can put it on a backlog to tweak this.

Thanks @mlissner! I didn't get any hits for this, so I'm comfortable using it for my purposes. Testing on ~40M entries for context.
Is a one-line PR helpful or did you want to diy / do more due diligence?

Yeah, a one liner should do it! Maybe worth adding a test that should be parsed properly so we don't regress?


Replace all E40k difference notes with footnotes.

I experimented with a few approaches, and managed to find something that seems to work across various situations and within the constraints of Asiciidoc and Antora.
I think I could go ahead and publish this now — but it wouldn't hurt for you to cast your eyes over it, if you find the time. If you can't get to it anytime soon I'll probably go ahead, but let me know if you'd like me to hold off.
Once we've published this, I'll pull the tweaks into ongoing PR #36 so that's up to date and easier to merge to the master branch later.

I think these changes work well enough, so I'll go ahead and merge into master rather than have this hanging around and getting out of sync.


Work shifts index lens

lens should use same period as other work lenses
Originally created by Tom Smyth at 2018-01-17 15:44, migrated from redmine-#7841

Tags changed from work to work, work mvp

By Tom Smyth at 2018-01-17 20:52

Tags set to work

By Tom Smyth at 2018-01-17 20:45

Subject changed from Work assignments index lens to Work shifts index lens

By Tom Smyth at 2018-03-08 02:48

Priority changed from ? to 2

By Tom Smyth at 2018-01-17 20:55

Status changed from New to Closed

By Tom Smyth at 2018-03-08 13:53

Version set to 6.0

By Tom Smyth at 2018-04-29 11:24